skip function

https://github.com/JuliaEarth/Variography.jl/blob/b1e401eb8514fad469d9514693412caf6f9cda9c/src/empirical.jl#L53
Continuing the discussion from Zulip:
Here I would prefer defining
skip(::Type{VariogramAccumAlgo}, i::Int, j::Int) = ...
skip(::Type{OtherAlgo}, i::Int, j::Int) = ...

outside the function, to avoid the closure, and then just call skip(VariogramAccumAlgo, i, j) && continue inside the loop.
(I'm not sure if the custom definition of iterators apply easily, as I don't know what neighbors return)

@lmiq the current approach is not capturing variables from scope, right? Just returning a lambda.

Also, can we continue the discussion in #geostats.jl in Zulip? It is easier and more dynamic. These GitHub issues we use for actual issues, and centralize all of them in the GeoStats.jl repo instead of submodules.


Cannot create new user account

I just downloaded Bash on Windows, and nearing completion it asked me to create a user account.  I forgot what happened, but I either canceled the prompt or it crashed.  Now when I open BashOnWindows it is always root.
I can't find any documentation on how to re-create a new user account (I know it doesn't matter that much since "root" is actually running not as Admin anyway -- but still).
Any help will be much appreciated.

open cmd.exe , then type:
lxrun /setdefaultuser
For more information visit: https://aka.ms/wslusers  :facepunch:

@ddfznt Thanks for the help!
@fpmchu Let me know if you continue to have issues after running lxrun /setdefaultuser

I also tried it and it worked fine for me. Thanks!

Awesome thanks!


Stats functions rewrite

Switches to much shorter SQL implementations
Fixes a bug with the wrong standard deviation being used
Decreases tolerance for error on the tests
Adds versions for all number types, not just numeric.

There is a minor (~10%) performance gain from the SQL version, but on a table with reals the new versions are over 4x faster.

Stale


Suggestion: Add jsonb support

Thanks for this great crate!
Currently this generates:
#[derive(Queryable, Debug, Identifiable)]
#[primary_key(key)]
pub struct Setting {
    pub key: String,
    pub value: Jsonb,
}

for:
table! {
    settings (key) {
        key -> Varchar,
        value -> Jsonb,
    }
}

which doesn't compile because Jsonb isn't imported.
It would be great if it instead generated the correct serde type
#[derive(Queryable, Debug, Identifiable)]
#[primary_key(key)]
pub struct Setting {
    pub key: String,
    pub value: serde_json::Value,
}

added. will be released in the next version. (>= 0.3.8)

@jacob-pro 0.3.9 released . it should contains the above feat.

thank you :)


Add user not found and password mismatch errors on /login route

Enhance the /login route by adding error handling for cases where the user is not found or the password does not match. Use proper error messages and status codes.
Reference:

JWT Authentication in Express

@dead8309 can you assign me


Navbar wobbles vertically when scrolling in Firefox 61

When scrolling vertically in Firefox 61 the navbar wobbles vertically. This does not happen with Chromium 67

Visit: http://myyasuda.github.io/sphinxbootstrap4theme/examples.html
Click "Set up" in the navbar
Scrol vertically

I released modified version (0.5.2) of this issue.

Thanks a lot for another very quick fix!


feat: fix up local wallet nullchain

Description

Fixing up function calls so that null chain works with a full local wallet once more
Testing

Tested with running full local wallet
Breaking Changes

Should be none
Closes

@cdummett Yes agreed, can just duplicate one or some of the existing tests with a full wallet and check they pass too


Top scores should be also per mode, rather than just per beatmap

Just an AND gamemode = ? in the query for saveScore (UPDATE)

WHO CARES ABOUT SCOREBOARD BUGS

WE HAVE JUST FIXED #ILBUGDELLENNEPPI!


Update SafeEventEmitterProvider type

This package now uses the SafeEventEmitterProvider type from the eth-json-rpc-provider package, rather than the equivalent provider time previously exported from this package. The old provider type has been removed.
The withBlockTracker test helper required more substantial updates to remain compatible with this type. The old FakeProvider class was replaced with a real but empty provider that has its sendAsync method stubbed out to behave identically to the sendAsync method on FakeProvider. It should be equivalent.

Oh good catch! I think I've managed to remove them all in https://github.com/MetaMask/eth-block-tracker/pull/130/commits/79f0eb070b38152ad0caee58197e7c018ab217df


PlayerEx: Director's Cut Incompatibility

Mod Name
PlayerEx: Director's Cut
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/playerex-directors-cut
Minecraft version
1.20.1
Describe the bug
Crashes when pressing singleplayer
Steps to reproduce

Download the mod and its dependencies
Launch the game
Click "singleplayer"
Crashes
...

Logs
https://pastebin.com/WDqX86de
Additional context
No other mods were installed
### Tasks

Duplicate of #699


You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
**Disclaimer: This message and any attachments are intended for the use of
the addresses(s) only and may be confidential and/or legally privileged. If
the reader is not the intended recipient, DO NOT READ, notify sender and
delete this message. In addition, be aware that any disclosure, copying,
distribution or use of the contents of this message is strictly prohibited.
The contents of this message, while possibly falling under the exceptions
of the Inspection of Public Records ACT [NMSA Chapter 14, Article2] may be
subject to inspection by the public.

sex

This has been already implemented, tho the point of updating the flixel side codes to the newer one is a good catch, you should keep this pr, but only update it for to do update the flixel codes

No longer needed


No way to set/remove default translation languages

I have not been able to find any good, documented way to set the languageService languages when wanting the options to be English and Russian. Ideally there would be a way to set the languages, but alternatively a remove method would be good. The solution I came up with (which seems hacky to me) is:
//Remove french, add Russian
config.directory.languageService.languages.pop();
var langDef = {
code: 'ru',
name: 'русский язык'
};
config.directory.languageService.addLanguage(langDef);

Duplicate of #812


Cv 59

Implemented Secrets in  main function(main.py) of WZDx code in GCP.

In main.py the from gcloud import storage doesn't seem to be needed. Can we remove?


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 2054 ms

Resolved: Bofrost.be performance has improved in b2fb19d after 9 minutes.


js-api.md documents non-existing methods

In dat/docs/js-api.md, several methods are documented which don't actually exist.

cat
dump
config

I'm not sure how much of an issue that is, but thought I'd mention it ;)

@chuesler thanks for the heads up, sorry for the late reply, your issue got a bit burried. check out the beta branch right now for current development.

hey @chuesler we just merged the dat beta. Check out the new JS api at http://github.com/maxogden/dat-core


Is NextResultSet implemented for driver.Rows in go-ora?

I am using go_ora.Connection.QueryContext(...) that returns driver.Rows.
I want to use NextResultSet call like in database/sql:Rows
go-ora version: 2.8.19
What I am trying to do:
...
conn, err := go_ora.NewConnection(connectionString, nil)
if err != nil {
	return err
}
...
err = conn.Open()
if err != nil {
	return err
}
...
driverRows, err := conn.QueryContext(ctx, query, args...)
if err != nil {
	return err
}

...
// driverRows.NextResultSet()?
for driverRows.NextResultSet() {
	for driverRows.Next(values) == nil {
		...
	}
}

Maybe I should use as in Golang standard library?
nextResultSet, ok := rs.rowsi.(driver.RowsNextResultSet)
if !ok {
	doClose = true
	return false
}

// Lock the driver connection before calling the driver interface
// rowsi to prevent a Tx from rolling back the connection at the same time.
rs.dc.Lock()
defer rs.dc.Unlock()

rs.lasterr = nextResultSet.NextResultSet()
if rs.lasterr != nil {
	doClose = true
	return false
}

I tried to find implementation of NextResultSet in go-ora source code same as in godror:
https://github.com/godror/godror/blob/010a6de91d2687e91c1563126d3b76578c9771b5/rows.go#L776-L814
But only found this commented line:
https://github.com/sijms/go-ora/blob/78d53fdf18c31d74e7fc9e0ebe49ee1c6af0abda/v2/data_set.go#L24

As I see DataSet does not implement NextResultSet as DataSet can not be converted to driver.RowsNextResultSet


"make sure y'all add a little more space bw different entries"

additionally, fix the crowding of metadata


[stable/cockroachdb] Complete refactor and capabilities extension

This is a hard backwards-incompatible rework of the stable/cockroachdb chart.
Fixes #18399
Possibly fixes #18402 (at least in all tested cases by me it has passed)
Motivation
The current 2.x.x version of stable/cockroachdb is hard to reuse in a variety of situations.
Specifically, it bounds you to:

use only PersistentVolumes for storing data
use only public-accessible Docker images
doesn't allow to tweak affinity rules and tolerations

All this made it hard for us to reuse Helm chart both for production/staging/local-dev environments and even numerous E2E tests.
BC Breaks
The version of the chart with this PR is bumped up to 3.0.0, reflecting the following backwards-compatibility breaks:

Values of the chart are fully renamed to follow camelCase style (instead of UpperCase) widely used in Helm chart's ecosystem (fixes #18399).

Additionally, values passed as CLI args to cockroach follow the kebab-case style to represent a direct mapping from values to these CLI args and omit possible misunderstandings.

Labels chart, heritage, release and component are replaced with more idiomatic helm.sh/chart, app.kubernetes.io/name, app.kubernetes.io/instance, app.kubernetes.io/managed-by and app.kubernetes.io/component.
Explicitly differentiate the listening ports in Pods (conf.port/conf.http-port) and in Services (service.ports).
Remove the Name value. Now, the name is idiomatically consists of release name + chart name and can explicitly overwritten with nameOverride and fullnameOverride values.
Remove the Component value. Now the components of the chart may be differentiated with specifying custom labels for them, if required.

Added

Support nameOverride and fullnameOverride values.
Ability to specify credentials for pulling used Docker images from private registries.
Ability to label and annotate almost any chart's resource either dependently and independently (see values.yml for details).
Ability to configure affinity rules, tolerations taints and nodeSelector both for StatefulSet and initialization Job.
Ability to persist data in:

hostPath for those who doesn't want to poke (or is not able to) with PersistentVolumes;
emptyDir which is convenient for temporary snapshot-based deployments (when the container already holds its data and persistence is unnecessary);
persistentVolume which is still the default one and now supports additional labels and annotations.

Ability to specify custom ingress rules for NetworkPolicy in addition to existing ones.
Ability to reuse entrypoint of the used Docker image (previously it was hardly overwritten), which adds the ability to run custom logic before the cockroach starts (in our case, for example, this is the data restoration from a snapshot, widely used for numerous E2E tests).
Ability to deploy in standalone mode (cockroach start-single-node) without clustering.
Ability to configure logtostderr verbosity.
Ability to configure cluster-name and disable-cluster-name-verification.
Ability to configure more CLI args as confchart values. While still allowing specify custom CLI extra args (via statefulset.args).

Additionally
From the chart description I've noticed that there are some chart instructions outside, specifically on CockroachDB's site:

https://www.cockroachlabs.com/docs/v19.2/orchestrate-cockroachdb-with-kubernetes-insecure.html

I'll create PR and update those once (and if) this PR is merged.
Checklist

[x] DCO signed
[x] Chart Version bumped (to 3.0.0)
[x] All the variables are documented in the README.md
[x] README.md info is updated and upgrading from previous versions is described.
[x] Title of the PR starts with chart name (e.g. [stable/cockroachdb])

ack @a-robinson @DuskEagle @joshimhoff @keith-mcclellan

/ok-to-test

Update:

Remove vim comment from templates/_helpers.tpl.
Run init Job when conf.join values is set.
Fix mounting certs in init Job when tls.enabled: no.
Make restartPolicy of init Job be always OnFailure (not just on tls.enabled: yes), so omit warnings like "incorrect value 'Always' for 'restartPolicy'".
Merge with latest master and stable/cockroachdb chart updates.

ack @DuskEagle

Hi @tyranron, thanks for taking the time for this huge refactor. Overall, I think this is a big improvement, and I'm excited to see it get in.
I've been doing some testing locally and creating CockroachDB clusters has worked exactly as expected. I did however run into a couple of issues when changing the number of replicas from 1 to 3 on an existing deployment:

Because the cockroachdb-init job did not run when there was only one replica, when we change the number of replicas, the job gets created and attempts to initialize an already-initalized cluster. This results in the job running forever, re-attempting to initialize the cluster every five seconds.
When changing the number of replicas, we change the --join flag in the StatefulSet spec. This causes the existing pod to restart once the two new pods are ready. This will actually happen whenever you scale up the number of replicas, which is unfortunate. But it's especially bad when moving from 1 to 3 replicas, because the existing ranges have not been replicated over to the new pods yet. This causes the new pods to crash until the cockroachdb-0 pod has restarted.

For the first issue, it might be simplest just to always pass a --join flag to cockroach start, even for a one-node cluster. Alternatively, we could fix the cockroachdb-init job to exit successfully if it is attempting to init an already-initialized cluster. Both approaches seem good to me.
The second issue is a bit trickier. I think this problem is actually one of the reasons why we didn't change the --join flag before. In theory, we could set the .spec.updateStrategy.type to OnDelete to ensure that existing pods aren't restarted. But we really only want that update strategy when changing the number of replicas, not when changing other parts of the statefulset spec (such as the CockroachDB version).
As long as the join flag includes just one running cockroachdb pod, then new pods will be able to join the cluster and gossip with all existing pods. So the join flag only has to contain enough pods so that you can still scale even if some pods are down for whatever reason (such as an AZ outage).
So, the simplest approach might be to continue to just hard-code the join flag to three pods (even for a one-pod cluster, this is harmless). Alternatively, we could include instructions to always set the updateStrategy to OnDelete when changing the number of replicas and to set the updateStrategy to RollingUpdate when performing any other kind of update, but that might be harder for users of the chart.

@DuskEagle thank you for so detailed feedback and giving info tips about how CockroachDB clustering works.

Alternatively, we could fix the cockroachdb-init job to exit successfully if it is attempting to init an already-initialized cluster. Both approaches seem good to me.

Actually, the idempotence of init Job is quite a good thing despite the issues above. I believe this should be first-class supported in /cockroach init command, but at the moment I've solved with some hairy bash magic.

So, the simplest approach might be to continue to just hard-code the join flag to three pods (even for a one-pod cluster, this is harmless).

I didn't know it's harmless. So now, after you've described the bigger picture I see that 3 hardcoded Pods is the best solution in this situation without introducing Kubernetes Operators in the game.
I've documented reasoning about hardcoded Pods for future contributors.

Update:

Hardcode --join Pods always to 3 (if the flag is not specified explicitly).
Make init Pod idempotent (succeeds if luster has been initialized already).

Set terminationGracePeriodSeconds: 0 for init Job as it doesn't handle OS signals anyway.

Rebased onto latest master.

@DuskEagle after investigating the cockroach start-single-node command all the described above makes even more sense.
So, with latest update I've introduced conf.single-node value (defaults to no), which gives the ability to run CockroachDB in a standalone mode without requiring initialization (so no init Job). This would be quite useful for some temporary non-production deployments (like E2E tests, etc). I've documented the value and placed the explicit warning on it about possible consequences.

Update:

Provide conf.single-node value which deploys Pods in standalone mode without initialization Pod.

[x] value documented in values.yaml;
[x] README updated.

Support conf.cluster-name and conf.disable-cluster-name-verification values.

[x] value documented in values.yaml;
[x] README updated.

Return printing of executed command in initialization Pod.
Remove --port flag usage in initialization Pod and add port to --host flag directly.
Rebase onto latest master.

Thanks for the update! I've tested scaling clusters and running the init job against an already-initialized cluster, and both now work exactly as expected. I also tested starting a cluster using the single-node helm value and that worked as well.

So now, after you've described the bigger picture I see that 3 hardcoded Pods is the best solution in this situation without introducing Kubernetes Operators in the game.

Introducing a public operator is something we've been talking about internally. We think it could make several common operations simpler, such as creating or scaling secure clusters, or upgrading between major versions of CockroachDB. If you have any feedback you'd want to give on this please feel free to bring it up on our Github, Forums, or Slack.
Thanks again for all these improvements to this Helm chart!
/lgtm /approve

/lgtm


failed and error and im lost

Status: Downloaded newer image for smy20011/dreambooth:latest
docker: Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error running hook #0: error running hook: exit status 1, stdout: , stderr: Auto-detected mode as 'legacy'
nvidia-container-cli: initialization error: WSL environment detected but no adapters were found: unknown.

I think you need to open docker program you installed and make sure docker is running.

i think its running but its new to me and confusing a bit

https://forums.docker.com/t/docker-run-container-failing-with-gpus-all-nvidia-container-cli-initialization-error-wsl-environment-detected-but-no-adapters-were-found-unknown/130452
It seems like a driver problem. Make sure you have latest driver installed.

i have latest drivers both GPU and CUDA


Sort keys before deleting.

Fixes a potential panic in the multidelete code.
@Workiva/dtbackend

Raven
Number of Findings: 0

+1


Combine Again

@helloreuben we don't support Django 1.11, as well as Python 3.6 yet.

@webyneter I knew that, the PR was a mistake - I was playing around.. Sorry to inconvenience. Would you by any chance know of a timeline to support Django 1.11 as well as Python 3.6

@helloreuben no problem. I can assure you that when all the dependencies listed in the ticket are upgraded to support Django 1.11, and ones we carry out overall testing a 1.11 release will be shipped.


[QUESTION]

How can I make use of starlette background feature inside FastApi?

afaik you have to explicitly use a JSONResponse, but @tiangolo might have a way better view on this.
I was unable to find a way to pass the background kwargs to a FastAPI Response without it.
here's a working example shamelessly adapted from the starlette test cases
def test_multiple_tasks2():

    TASK_COUNTER = 0
    def increment(amount):
        nonlocal TASK_COUNTER
        TASK_COUNTER += amount
    app = FastAPI()

    @app.get("/")
    async def root():
        tasks = BackgroundTasks()
        tasks.add_task(increment, amount=1)
        tasks.add_task(increment, amount=2)
        tasks.add_task(increment, amount=3)
        message = {'status': 'successful'}
        return JSONResponse(message, background=tasks)

    client = TestClient(app)
    response = client.get("/")
    assert response.json() == {'status': 'successful'}
    assert TASK_COUNTER == 1 + 2 + 3

Yes - this works, and to reinforce the warning above - if you don't return JSONResponse with the background=tasks parameter - it won't initiate the background tasks.

Thanks for your help here guys @euri10  and @wshayes !
It is now integrated into FastAPI in a more simple way, similarly to when using the Request object directly, just declare a function parameter of type BackgroundTasks and add tasks to it directly. The rest is handled by FastAPI. And now you don't have to return a JSONResponse directly.
Here are the new docs: https://fastapi.tiangolo.com/tutorial/background-tasks/
In short:
from fastapi import BackgroundTasks, FastAPI

app = FastAPI()

def write_notification(email: str, message=""):
    with open("log.txt", mode="w") as email_file:
        content = f"notification for {email}: {message}"
        email_file.write(content)

@app.post("/send-notification/{email}")
async def send_notification(email: str, background_tasks: BackgroundTasks):
    background_tasks.add_task(write_notification, email, message="some notification")
    return {"message": "Notification sent in the background"}

As this is now solved, I'll close this issue. But feel free to add more comments or create new issues.


Wrong stack trace in errors shown in combination of for..of loops and JsSetException

When throwing an error using JsSetException inside a for..of loop error.stack does point to the last line of the for loop instead of the line which has caused the real exception.
for..in however does work properly
throwing inside JavaScript works properly even inside for..of loop
try..catch and rethrow works properly
Example "for..of":
1 function crash() {
2 	Sys.CauseException();
3 }
4
5 for (let x of [1,2,3]) {
6 	crash();
7 }

error.stack:
Error: Error invoking CLR method CauseException()
   at Global code (test.js:6:2)

Example "for..in":
1 function crash() {
2 	Sys.CauseException();
3 }
4
5 for (let x in [1,2,3]) {
6 	crash();
7 }

error.stack:
Error: Error invoking CLR method CauseException()
   at crash (test.js:2:2)
   at Global code (test.js:6:2)

Example "rethrow":
1 function crash() {
2 	try {
3 		Sys.CauseException();
4 	} catch(error) } { throw error; }
5 }
6
7 for (let x in [1,2,3]) {
8 	crash();
9 }

error.stack:
Error: Error invoking CLR method CauseException()
   at crash (test.js:4:19)
   at Global code (test.js:8:2)`

throwing the error in C#:
var error = JavaScriptValue.CreateError(JavaScriptValue.FromString("Error invoking CLR method CauseException()"));
JavaScriptContext.SetException(error);
return JavaScriptValue.Invalid;

See https://github.com/microsoft/ChakraCore/issues/3685.

Yeah the problem is the internal workings of for...of which also cause #3685 - though the fix may not be the same.
In the bytecode emitter for of loops are re-written as follows:
Input JS:
for (let x of [1,2,3]) {
  crash();
}

Re-written version (roughly - may have missed a condition somewhere):
const iterator = [1,2,3][Symbol.iterator]();
const next = iterator.next;
let shouldCallReturn = false;
try {
  while(true)
  {
    shouldCallReturn = false;
    const result = next.call(iterator);
    if (! (result instanceof Object))
      throw new TypeError("Iterator should return an object");
    if (result.done)
      break;
    let x = result.value;
    shouldCallReturn = true;
    crash();
  }
} catch (e) {
  if (shouldCallReturn) {
    try { iterator.return(); shouldCallReturn = false; } catch (e) { try {iterator.close()} catch (e) {}}
    throw e;
  }
} finally {
  if (shouldCallReturn) {
    if (typeof iterator.return === "function") {
      const result = iterator.return();
      if (! (result instanceof Object))
        throw new TypeError("Iterator should return an object");
  }
}

Yes that is what the spec wants for a for...of implemented using simpler operations - chakracore does it this way as it's easier for the JIT to optimise than using a more complex operation. The problem is that the exception handling gets a little confused - as it thinks there's a try/catch/finally block when the original code didn't contain it.

@rhuanjl
I am curious why running a for..in loop does produce the correct stack trace.
Does the byte-code of for..in not have any internal try/catch?

@goozie81 That's right. The bytecode for for-in looks something like this:
  Line   1: for (var i in obj);
  Col    1: ^
    0009   ProfiledLdRootFld    R2 = root.obj #0 <0>
    000f   ProfiledInitForInEnumerator R2  uint:0  <0>
    0014   ProfiledLoopStart    uint:0
    0016   ProfiledLoopBodyStart uint:0

  Line   1: var i in obj);
  Col    6: ^
    0018   BrOnEmpty            x:0026 (   9)  R2  uint:0
    001d   ProfiledStRootFld    root.i = R2 #1 <1>
    0023   Br                   x:0016 ( -16)
    0026   ProfiledLoopEnd      uint:0

Thinking about this - the only issue here is that the stack trace is wrong - so just need some way to capture the stack trace from the initial throw rather than re-generating it after calling .return I figure that that should not be too hard.

Note however that will fix this issue but won’t fix #3645


Question on API id

If a developer wishes to develop more than one client that uses the Telegram API/TDLib, is that possible?
I know that my.telegram.org does not seem to expose the possibility of requesting more than one API ids.
Note: just a question out of curiosity. I do not need another api id at this point.

It is not possible to create multiple api_id for the same phone number.

Got it, thanks!


bugfix(Logger): move aut token to the beginning of the log message

Each log message contains the authentication token, and logentries
allows the token to be everywhere in the message sent.
Previously the token was at the end of the message.
In case the message contained another UUID, logentries used it as
an authentication token and skipped the real one that was at the end of
the message.
For this reason, all the logs with a UUID inside where not saved in
logentries.

There were the following issues with your Pull Request

Commit: 1994ea9545275d7be87c0ac59ccb499066c7e874
Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Guidelines are available at https://github.com/sysdia-solutions/logger_logentries_backend

This message was auto-generated by https://gitcop.com


Server error raised when an institute is created first time without p…

…opulating institutes

The error occurred as the disk space ran out. df commend showed 100%, The
issue solved after deleting some files.
On Tue, 5 Jan 2021 at 22:58, Dinidu Hewage<EMAIL_ADDRESS>wrote:

…opulating institutes
You can view, comment on, or merge this pull request online at:
https://github.com/lk-gov-health-hiu/pssp/pull/204
Commit Summary

Server error raised when an institute is created first time without
populating institutes

File Changes

M src/main/java/lk/gov/health/phsp/bean/InstitutionController.java
https://github.com/lk-gov-health-hiu/pssp/pull/204/files#diff-44e9bcd3243819309acda2a449c9eb144d261aa28ca54dc47954666f0054b40e
(4)

Patch Links:

https://github.com/lk-gov-health-hiu/pssp/pull/204.patch
https://github.com/lk-gov-health-hiu/pssp/pull/204.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lk-gov-health-hiu/pssp/pull/204, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJLRQR6CNYF3VMZO2NKXD3SYND4LANCNFSM4VVV2NWA
.

--
Dr M. H. B. Ariyaratne
MBBS(Ruhuna), MSc(Biomedical Informatics), MD(Health Informatics)
SLMC Reg No. 19157
Senior Registrar in Health Informatics
Dr M. H. B. Ariyaratne
MBBS(Ruhuna), MSc(Biomedical Informatics), MD(Health Informatics),
Visiting Fellow - University of Southampton
Clinical Informatics Research Unit,
Care of the National Blood & Transplant Service,
MP 852, Coxford Road,
Southampton, SO16 5AF, UK
Official Addresses
Senior Registrar in Health Information,
Management Development & Planning Unit,
Ministry of Health and Indigenous Medicine,
385, Rev. Baddegama Wimalawansa Thero Mawatha,
Colombo 10, Sri Lanka.
Phone    : + 94 11 2675449 - Ext 274
Residence
36/A, Thalgahahena Lane,
Galle, Sri Lanka.
Mobile    : + 94 71 5812399
Residence : + 94 91 2241603
Email<EMAIL_ADDRESS>

The error occurred as the disk space ran out. df commend showed 100%, The
issue solved after deleting some files.
On Tue, 5 Jan 2021 at 22:58, Dinidu Hewage<EMAIL_ADDRESS>wrote:

…opulating institutes
You can view, comment on, or merge this pull request online at:
https://github.com/lk-gov-health-hiu/pssp/pull/204
Commit Summary

Server error raised when an institute is created first time without
populating institutes

File Changes

M src/main/java/lk/gov/health/phsp/bean/InstitutionController.java
https://github.com/lk-gov-health-hiu/pssp/pull/204/files#diff-44e9bcd3243819309acda2a449c9eb144d261aa28ca54dc47954666f0054b40e
(4)

Patch Links:

https://github.com/lk-gov-health-hiu/pssp/pull/204.patch
https://github.com/lk-gov-health-hiu/pssp/pull/204.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lk-gov-health-hiu/pssp/pull/204, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJLRQR6CNYF3VMZO2NKXD3SYND4LANCNFSM4VVV2NWA
.

--
Dr M. H. B. Ariyaratne
MBBS(Ruhuna), MSc(Biomedical Informatics), MD(Health Informatics)
SLMC Reg No. 19157
Senior Registrar in Health Informatics
Dr M. H. B. Ariyaratne
MBBS(Ruhuna), MSc(Biomedical Informatics), MD(Health Informatics),
Visiting Fellow - University of Southampton
Clinical Informatics Research Unit,
Care of the National Blood & Transplant Service,
MP 852, Coxford Road,
Southampton, SO16 5AF, UK
Official Addresses
Senior Registrar in Health Information,
Management Development & Planning Unit,
Ministry of Health and Indigenous Medicine,
385, Rev. Baddegama Wimalawansa Thero Mawatha,
Colombo 10, Sri Lanka.
Phone    : + 94 11 2675449 - Ext 274
Residence
36/A, Thalgahahena Lane,
Galle, Sri Lanka.
Mobile    : + 94 71 5812399
Residence : + 94 91 2241603
Email<EMAIL_ADDRESS>

I think the change is not necessary. The error occurred as there was no disk space left in the HDD. can we discard these changes? Thanks

I think the change is not necessary. The error occurred as there was no disk space left in the HDD. can we discard these changes? Thanks

What I see is that we still can see this in our local environment which are not having any disc space issue.
What I saw was that we are going to execute the applicationController.getInstitutions().add(selected) method when the institute variable in the applicatoinController is null. It's something similar to the following.
MyTest obj = null; obj.toString();
So the exception fired.
This happens only if we try to insert institutes right after rebuilding and deploying the application. We need to go to insert institutions without going to list institution page.
One other thing I saw with the code without the fix. Although we came to the insert institutes page after visiting the list institution page. the first institution we add gets duplicated in the list institution page. that's because we call the fillItems();  method after inserting the institution and we add that same institution again to the applicationController institution list with the method applicationController.getInstitutions().add(selected) .
This correction was done to overcome the above issues.

What I see is that we still can see this in our local environment which are not having any disc space issue.
What I saw was that we are going to execute the applicationController.getInstitutions().add(selected) method when the institute variable in the applicatoinController is null. It's something similar to the following.
MyTest obj = null; obj.toString();
So the exception fired.
This happens only if we try to insert institutes right after rebuilding and deploying the application. We need to go to insert institutions without going to list institution page.
One other thing I saw with the code without the fix. Although we came to the insert institutes page after visiting the list institution page. the first institution we add gets duplicated in the list institution page. that's because we call the fillItems();  method after inserting the institution and we add that same institution again to the applicationController institution list with the method applicationController.getInstitutions().add(selected) .
This correction was done to overcome the above issues.


Upgrades

General upgrades. Increased some specification on dev dependencies. Fixed some warnings in kafka events.

Just a note: the redis_events/pyproject.toml file didn't get the same love with the upgrades!
The poetry.lock was updated but not the toml
Mentioning it because we are recently making use of the redis_events plugin. And I'm curious if some of the upgrades break anything in the integration, or if it was missed.

Just a note: the redis_events/pyproject.toml file didn't get the same love with the upgrades! The poetry.lock was updated but not the toml
Mentioning it because we are recently making use of the redis_events plugin. And I'm curious if some of the upgrades break anything in the integration, or if it was missed.

I did have an issue with the redis_events plugin in particular. So I installed from the devcontainer which didn't get the global updates. The script was trying to upgrade aries-cloudagent to 0.12.0-rc1which I didn't want. I think it might be a caching issue or something because all the other plugins were correctly on 0.11.0.
I'll try and figure out why that was happening but there wasn't any problems with those upgrades that I know of.


Add telemetry tracer/metric enable flag and integ test

Description
Add static settings for Tracer and Metrics features in the Telemetry so that users should be able to enable either one or both. Also add the integration test cases for metric framework.
Related Issues
Resolves #10359

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] GitHub issue/PR created in OpenSearch documentation repo for the required public documentation changes (https://github.com/opensearch-project/documentation-website/issues/3951)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta, As discussed over PR #10241 creating the PR to introduce the separate static flags for Tracing and Metrics.

@reta, Please take a look as per your convenience, it would be great if we can merge this in 2.11 release. This will be helpful for users to experiment with tracing and metrics. Thanks!

@reta, your thoughts on the approach here please?

@reta, one thing now irrespective of which telemetry is enabled (tracing or metrics), we are bringing up the entire OpenTelemetry instance which contains both Tracing and Metrics. Now plugin should be able to close the entire telemetry.
For example let's say we have enabled just MetricsFeature from core. Otel plugin will create the OpenTelemtry instance with both metrics and tracing sdk. Core will assign the tracer to a NoopTracer.INSTANCE and MetricsRegistry the actual instance. Now when there is a Node::close call, core will shut down the  NoopTracer and the actual MetricsRegistry instances which intern will close their corresponding telemetries, since tracer will be a NoopTracer it won't close the TracingTelemetry. Now, since Otel has by default initialised TracingTelemetry also, so it will remain open. We have couple of ways to address this.

Otel since initialising both Tracing and Metrics telemery and core is not aware of this. So Otel should be able to close both on any of the telemetry:close call.
We can make the Telemetry closable and OtelTelemetry should be able override the close behaviour.
Core should add a explicit check and if any of the telemetry exists irrespective of the feature flag then close it from the corresponding factories(TracerFactory and MetricsRegistryFactory).

To me option 2 looks more organic approach as it defines the responsibilities clearly. What do you suggest?

@reta, one thing now irrespective of which telemetry is enabled (tracing or metrics), we are bringing up the entire OpenTelemetry instance which contains both Tracing and Metrics. Now plugin should be able to close the entire telemetry.

Correct @Gaganjuneja , I think we could wrap OpenTelemetry into RefCountedReleasable: the tracer and metrics would increment the reference if needed, the last one will close the instance down.

LGTM, thanks @Gaganjuneja , could you please resolve the conflicts?


XlaRuntimeError when jitting certain reductions of large (intermediary) array

Description
Error for amin, amax, and logsumexp:
XlaRuntimeError: INTERNAL: Failed to execute XLA Runtime executable: run time error: custom call 'xla.gpu.func.launch' failed: Failed to launch CUDA kernel: fusion_4 with block dimensions: 5120x1x1 and grid dimensions: 30733x1x1: CUDA_ERROR_INVALID_VALUE: invalid argument.

No errors for sum, mean, or prod.
Minimal reproducing example:
https://colab.research.google.com/drive/1hXANR-lRbHV73SfD5J7FqAwTqLEqsT4P?usp=sharing
The original algorithm uses sum/mean and runs fine for this size of an array. However, I wanted to see if log-space calculations would result in a lower relative error.
What jax/jaxlib version are you using?
jax v0.4.1 jaxlib v0.4.1
Which accelerator(s) are you using?
GPU
Additional system info
Google colab
NVIDIA GPU info
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 460.32.03    Driver Version: 460.32.03    CUDA Version: 11.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  Tesla T4            Off  | 00000000:00:04.0 Off |                    0 |
| N/A   61C    P0    27W /  70W |  13758MiB / 15109MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
+-----------------------------------------------------------------------------+

@cheshire Can you find someone to look at this XLA issue?

I don't have access to the colab, and is probably not the best way to share the bug.
Is it possible to file an XLA bug and ideally attach input HLO?

@deasmhumhna can you open access to the colab, at least?

@deasmhumhna can you open access to the colab, at least?

It's open now.

I can repro on V100 with the nightly JAX.
A lightly smaller repro:
import jax
import jax.numpy as jnp

from functools import partial

@jax.jit
def test(x, reduction=None):
    # jnp.amax, jax.nn.logsumexp also fail
    return jnp.amin(jnp.square(x[:, None, :] - x).sum(axis=-1))

The issue is that we try to launch a kernel with 5120 threads per block, but this is limited to 1024:
fusion with block dimensions: 5120x1x1 Full error:
jaxlib.xla_extension.XlaRuntimeError: INTERNAL: Failed to execute XLA Runtime executable: run time error: custom call 'xla.gpu.func.launch' failed: Failed to launch CUDA kernel: fusion with block dimensions: 5120x1x1 and grid dimensions: 30733x1x1: CUDA_ERROR_INVALID_VALUE: invalid argument.

Here is an HLO that reproduce the error:
HloModule jit_test

region_0.14 {
  Arg_0.15 = f32[] parameter(0)
  Arg_1.16 = f32[] parameter(1)
  ROOT add.17 = f32[] add(Arg_0.15, Arg_1.16)
}

region_1.19 {
  Arg_0.20 = f32[] parameter(0)
  Arg_1.21 = f32[] parameter(1)
  ROOT minimum.22 = f32[] minimum(Arg_0.20, Arg_1.21)
}

ENTRY main.24 {
  Arg_0.1 = f32[50176,3]{1,0} parameter(0), sharding={replicated}
  reshape.4 = f32[50176,1,3]{2,1,0} reshape(Arg_0.1)
  broadcast.6 = f32[50176,1,3]{2,1,0} broadcast(reshape.4), dimensions={0,1,2}
  reshape.7 = f32[50176,3]{1,0} reshape(broadcast.6)
  broadcast.8 = f32[50176,50176,3]{2,1,0} broadcast(reshape.7), dimensions={0,2}
  reshape.5 = f32[1,50176,3]{2,1,0} reshape(Arg_0.1)
  broadcast.9 = f32[1,50176,3]{2,1,0} broadcast(reshape.5), dimensions={0,1,2}
  reshape.10 = f32[50176,3]{1,0} reshape(broadcast.9)
  broadcast.11 = f32[50176,50176,3]{2,1,0} broadcast(reshape.10), dimensions={1,2}
  subtract.12 = f32[50176,50176,3]{2,1,0} subtract(broadcast.8, broadcast.11)
  multiply.13 = f32[50176,50176,3]{2,1,0} multiply(subtract.12, subtract.12)
  constant.3 = f32[] constant(0)
  reduce.18 = f32[50176,50176]{1,0} reduce(multiply.13, constant.3), dimensions={2}, to_apply=region_0.14
  constant.2 = f32[] constant(inf)
  ROOT reduce.23 = f32[] reduce(reduce.18, constant.2), dimensions={0,1}, to_apply=region_1.19
}

Tracked internally as P1 in b/268425676

Thanks for the convenient reproducer, Frédéric.
I found the culprit and have a fix in-flight.

If I use upstream TF and upstream JAX, I have this output:
<function amin at 0x7f5efb0b4d30> None
<function amax at 0x7f5efb0b4940> None
<function logsumexp at 0x7f5efac68430> None
<function sum at 0x7f5efb0b4160> None
<function mean at 0x7f5efb0ba670> None
<function prod at 0x7f5efb0b4550> None

So it is fixed upstream.
Thanks for the report and the nice notebook.


Running PHP Stan having conflict with Laravel Breeze's VerifyEmailController file

Laravel Version: 9.48.0
Laravel Breeze: 1.18.0
PHP Version: 8.0.2
Database Driver & Version:

Description:
When running PHPStan to check bugs, it is returning error Parameter #1 $user of class Illuminate\Auth\Events\Verified constructor expects  Illuminate\Contracts\Auth\MustVerifyEmail, App\Models\User|null given.
Checking on the Illuminate\Auth\Events\Verified class, method __construct expects a @param \Illuminate\Contracts\Auth\MustVerifyEmail  $user but in VerifyEmailController sends $request->user()
Steps To Reproduce:
install phpstan and run ./vendor/bin/phpstan analyse --memory-limit=1G; composer dump-autoload
I know this is a very minor bug.

I am still new to breeze so I dont know how this will affect the overall code.

Heya, a PHPStan issue is not a bug so please feel free to send in a PR if you like.


Task - Type material - depository

More information on depository is needed on search (museum, collection, country, city, etc.)

We added what we can for now.  This will depend on switching to GBIFs repositories.


Replace reflect.SliceHeader with unsafe.Slice

Fixes part of #20
This PR fixes the go vet warning caused by the use of reflect.SliceHeader() by replacing it with unsafe.Slice().

I'm not 100% this actually works as desired, as the unsafe and reflect libraries both feel like voodoo to me. There don't seem to be tests in this area, but maybe I missed them?

This requires a version bump for Go:
./main.go:246:14: unsafe.Slice requires go1.17 or later (-lang was set to go1.15; check go.mod)

This is probably reasonable, since Go supports two major releases at a time:
https://go.dev/doc/devel/release#policy
And we're on Go 1.21, meaning that anything older than Go 1.19 is unsupported.
OTOH, I can see how dropping support for language release from less than 3 years ago could seem a bit aggressive. ¯\_(ツ)_/¯

I agree that the bump is reasonable. We loosely follow the Go support policy, as following those minor releases is a good idea and generally safe. We currently only test in CI with 1.19+. Older versions of Go may or may not work.


iOS implementation of ConsumePurchaseAsync just returns null despite being async

Bug Information
The iOS implementation is just an empty expression-bodied method that returns null, despite the method being async. Using await on this method causes a null reference exception.
Version Number of Plugin: 2.0.0
Device Tested On: iPhone 8+
Simulator Tested On: [none]
Version of VS: 15.9.7
Version of Xamarin: <IP_ADDRESS>
Versions of other things you are using:
Steps to reproduce the Behavior
Just use CrossInAppBilling.Current.ConsumePurchaseAsync(...) in iOS;
Expected Behavior
the method should not return null, because it is async
Actual Behavior
the method returns null
Code snippet
var task = CrossInAppBilling.Current.ConsumePurchaseAsync("", "");
if (task == null)
{
// try breakpoint here
]
Screenshotst

I agree if a bug it should be fixed. FYI I thought we didn't actually need to use this method on iOS per the latest web page documentation.
https://jamesmontemagno.github.io/InAppBillingPlugin/PurchaseConsumable.html
Consumables are unique and work a bit different on each platform and the ConsumePurchaseAsync may need to be called after making the purchase:

Apple: As soon as you purchase they are consumed

I used it cross-platform, and for me it defeats the purpose of being cross-platform if I need to determine the OS

On iOS they are consumed immediately with null.... so if on iOS then don't try to consume it. Check with Xamarin.Essentials

Then the OS should be determined even in the cross-platform parts of the codes!?

You can determine the OS in the shared code. Unfortunately all platforms aren't the same and you need to have some cases to handle this as this is how I designed the API. You are free to fork it and use the code directly.

Would it be better to throw an exception in this case? Documentation is great, but a well worded exception would be more so. There seem to be a lot of edge cases to trip over...


Proposal: remove amount of supported environment variables

Puppeteer supports environment variables to allow people to configure things
like:

where to download the browser from
which product to download/run (Chrome/FF)
which Chromium revision to download
which browser to run (either via providing a product, or a binary executable)
whether to skip the download of Puppeteer

These are all reasonable things that should be configurable, but I'd like to
propose that we slim down the number of ways we let people configure these.
For example, the downloadBrowser function in install.ts checks three
different places for environment variables:
const product =
  process.env.PUPPETEER_PRODUCT ||
  process.env.npm_config_puppeteer_product ||
  process.env.npm_package_config_puppeteer_product ||
  'chrome';

I didn't even know until reading this code that npm_config_* variables
existed, and similarly for npm_package_config. I would wager that most people
are using environment variables if they need to configure Puppeteer's behaviour.
I would like to suggest we move to only supporting environment variables, so
that the code above would look like so:
const product = process.env.PUPPETEER_PRODUCT || 'chrome';

We can do this incrementally to help users who do rely on the old settings:

Ship a release that logs a warning if we detect use of npm_config_* or
npm_package_*.
In a future major version (giving people reasonable time to deal with the
warnings) remove support for those environment variables.

Any thoughts welcome :) If you rely on npm_config_* or npm_package_*
variables please do let me know; I'd like to understand why those options are
preferable over environment variables.

@mathiasbynens @OrKoN @hanselfmu please LMK WYT!

The advantage of using npm_package_* variables is that you can keep all your configuration in package.json instead of providing the variables in the script commands which might be messy. Do you know when and why it was added in the first place?

That seems like a reasonable benefit; but it's not something I've seen anyone do across the community/ecosystem, so I'm wondering if anyone does take advantage of that. Even within devtools we use environment variables.
I'm not sure on the original context - maybe @mathiasbynens knows more?

I'm not sure on the original context - maybe @mathiasbynens knows more?

This was introduced to address #648. (Environment variables were only introduced later, in #3190.) It seems this is a common/expected mechanism to support proxy configurations through package.json.
A more conservative version of this change would be to get rid of the process.env.npm_ usage only for non-proxy settings, but I could see how that might be confusing as well.
I'm not sure what to think about this change, to be honest.
cc @mjzffr since this impacts the way product is determined -- would y'all be okay with this change?

My main thought and motivation for suggesting this was that I was unaware of the npm_* environment variables and suspect few people rely on them or use them. If that's the case, it's a small bit of maintenance burden we can remove. If we're not sure, let's leave it be for sure. I may then invest some time writing some helpers to make it easier to support this set of config for each given environment variable that we care about.

I don't have a good sense of how these are used in the wild, but I've been surprised a couple of times by what some Puppeteer-based frameworks and test harnesses rely on. Maybe reach out to a few?
Another idea: what about refactoring the env checks into a general utility method to reduce the maintenance burden? So you could call e.g. product = getEnv('puppeteer_product') || 'chrome'

Perhaps it would make sense to deprecate the usage of these env variables for a few releases before removing? E.g.,
function deprecate(value) {
  console.log('Deprecated...');
  return value;
}

const product =
  process.env.PUPPETEER_PRODUCT ||
  deprecate(process.env.npm_config_puppeteer_product) ||
  deprecate(process.env.npm_package_config_puppeteer_product) ||
  'chrome';


Add gem test

#5

Following this, I added CI for Gem.
Currently, the test fails because of the following issue.

#6

I will rebase this. Then All tests will succeed.

Now all tests have succeeded.
Only the new gem test uses the latest Fluentd, and other tests use v1.12 for now.
When the following fix of Fluentd is merged, other tests also start to use the latest Fluentd.

https://github.com/fluent/fluentd/pull/4061

Or perhaps we can adjust gemfile or gemspec of this repository.

When the following fix of Fluentd is merged, other tests also start to use the latest Fluentd.

It is enough to just wait PR merged, IMHO.
But it is good practice to test with latest version, so how about using latest Fluentd in gem job?
I mean inserting gem install fluentd to gem job.

Thanks for your review!


Create quine program in python

Self Printing Program in Python
This is to address issue #13
@thevirtualbuddy 😃

Great work @AmirNaghibi

PR to #28


Need help reward video does not work

This is my code based on sample. I want to test using admob test id.
//Prepare your AdMob Ad Unit Id for your banner and interstitial
    var admobid = {};
    admobid = { // this is sample test id
        banner: 'ca-app-pub-6869992474017983/9375997553',
        interstitial: 'ca-app-pub-6869992474017983/1657046752'
    };

// prepare reward video
    function admobRewardVideoAd(){
        if(AdMob) AdMob.prepareRewardVideoAd({
            adId: admobid.reward,
            isTesting: true,
            autoShow: true
        });
    }

//1. try call after device ready
document.addEventListener("deviceready", admobRewardVideoAd, false);

//2. try call the reward video
AdMob.showRewardVideoAd();

//3. try call the reward video
admobRewardVideoAd();

I try 3 ways, all of them still not show the reward video. Any hints?
Thank you

AdMob has no much video sources, you need add some mediation to some other Ad networks, like vungle, adcolony, etc.
I am working on a new plugin cordova-plugin-vungle, and will add a mediation adapter cordova-plugin-admob-vungle but not finished yet.
You can try downloading the mediation adapters from admob website then add it manually, or wait my vungle plugin done.

See #511, same issue, let's follow up in that tracker.


[Java] Format pom.xml files using Apache Maven pom.xml 2008 convention

Describe the enhancement requested
pom.xml files used by the project seems to follow direct conventions regarding indentation or section ordering which makes harder to push changes to those files.
One of the most used convention for pom.xml files is the one used by Apache Maven itself.
There are a couple of plugins which do support this convention but spotless seems to be one of the most common options, especially if the project decides to also use for Java code formatting.
Component(s)
Java

Issue resolved by pull request 41174
https://github.com/apache/arrow/pull/41174


optimize InstancesByPort: not calling getPodByIP twice

Please provide a description for what this PR is for.
This is a minor optimization
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[x] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
c2326f185d019984b181adb4012312502967a417
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/cherrypick release-1.3

@rshriram: new pull request created: #16872

In response to this:

/cherrypick release-1.3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Fix bad text in toasts; give RawLabel and Label better names

See our capitalization policy to explain some of the proposed text changes here.
When a toast has two full ideas, I've punctuated them as sentences, like

Download failed. Please try again from your browser.

Otherwise, I've left out any punctuation, to leave the message quick and not too prose-like. I'm not sure if that's exactly the choice we want.
Greg, I'd be happy for you to make any improvements/fixes to the user-facing text before merge without another round of review. I mainly just wanted to clean up the more glaring errors in these.

Thanks for the review! Revision pushed.

Thanks! Looks good; merging.
I'll also do a quick tools/tx-sync and we'll see what that Transifex UI looks like.


Scrolling and editable

If you have a grid with editable, and are in the process of editing a field then you scroll the grid, the scrolling is somewhat flicker-y and funny.  Refer this plunker: http://plnkr.co/edit/WoKn84HKBlPvOOJiNaF6?p=preview
Select a field to enter edit mode, then scroll the grid.

This looks to now be just another instance of the scrollTo and setFocus issue that we have.  Closing as a duplicate.


Improve login/registration copy

Detailed Description
The error messages around login and registration could use copywriting. Additionally, we should use sentence case on all of the error messages.

@jfrankl Do you have suggestions for what these should be, or any examples to crib from?


add a way to get a list with every movie and every tv serie

this is very useful to create apps like popcorn time, and could be implemented to PT when plugin system were ready ;)

Solved with tvShows() method


A mistral long context - MegaBeam-Mistral-512K

Hi, thanks for the project! could you please evaluate https://huggingface.co/aws-prototyping/MegaBeam-Mistral-7B-512k on the latest RULER benchmark. Thanks!

Sure! I put the results on the leaderboard (under our evaluation) although I saw you have tested on your own. This is a pretty good long-context model. It would be great if we can have numbers to show its short context performance (MMLU, MTBench, or something on open llm leaderboard).

Thank you for testing it! Yes, I just wanted to confirm if our eval is consistent with yours (which appears to be the case). I will take a look at the short context benchmark although we have focused solely on the "long" context when training this model.


Delegate call issue & SKPhotoBrowserOptions bug

Hello! I have to start by saying that this is an incredible project!
Now, in the last versions, it seems that you've removed the delegate call inside 'open func determineAndClose()' at SKPHotoBrowser.swift @ line 238. It should be:
open func determineAndClose() {
delegate?.willDismissAtPageIndex?(currentPageIndex)
animator.willDismiss(self)
}
Apparently you've also changed some of your if parameters to not look at SKPhotoBrowserOptions at all (example: display delete button) and correct me if I'm wrong, but there are no proper setters for the SKPhotoBrowser boolean.

Thanks your feedback(sorry for late reply), it maybe fixed at latest ver. let me know if you have any issues there.
https://github.com/suzuki-0000/SKPhotoBrowser/blob/e8af321d593b76eb27cdd52526e482678af0fd20/SKPhotoBrowser/SKPhotoBrowser.swift#L217


Add note to typescript section about typescript version compatibility

Problem Description:
The Typescript documentation does not currently call out that the glamorous types rely on 2.4+ language features.
While they are still usable pre 2.4, the experience is not as good.
The following bug report, even if unrelated to the version, has screenshots of issues that will occur with typescript versions <2.4.
https://github.com/paypal/glamorous/issues/278
Suggested Solution:
Add a note to the typescript section calling out the compatible typescript version.

https://github.com/kentcdodds/glamorous-website/pull/276


The y=mx+b sliders are oriented backwards

The sliders look like this, but drag up and down:

Discovered in https://github.com/phetsims/bamboo/issues/49

I switched to use HSlider and will open a separate issue about this newly discovered VSlider problem.  It looks like this now:

@pixelzoom is this what was intended? Or did you want vertical sliders?

phetsims/sun#733 is fixed, so I reverted to VSlider for DemoLinearEquationPlot.
Back to @samreid for review.

Looks great, thanks, closing.


Adds Vault Verb

Fixes #1883

Once conflict's done LGTM if you want to leave the duplicate code for now.


Add .editorconfig file

To start with just .php files. According to the Developer Style Guide:

Use tabs for indentation (not spaces or a mixture of both)
Use a tab stop of 8, rather than 4
Ensure there is NO trailing whitespace at the end of a line, for example spaces or tabs when there is no more text afterward
Ensure there is NO whitespace on empty lines. For example, a line must not contain only spaces or only tabs

Additionally, I've added:

UTF-8 character encoding
LF line endings
Final newline

Proposed .editorconfig file (.php files for now):
; This file is for unifying the coding style for different editors and IDEs.
; More information at http://editorconfig.org

root = true

[*.php]
indent_style = tab
tab_width = 8
end_of_line = lf
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

GitHub will also read and apply this style to the repo.
See #1664.

Can you please add *.inc too?

:+1:


Soft restart causes segfault on game

Using linux build of test project, commenting out setting static fields seem to prevent this but otherwise it will segfault after about a minute

Might have fixed, test running the game for a minute after soft restart

Looks like its fixed, closing


Grpc "stream removed" in Azure

What version of gRPC and what language are you using?
1.19.0
What operating system (Linux, Windows,...) and version?
Linux & Windows
What runtime / compiler are you using (e.g. python version or version of gcc)
dotnet core
What did you do?
We put our grpc server on Azure and then we try to call these grpc servces locally. Everything works fine at first, but it can't be called normally after waiting for five minutes.
There is sample code.
GrpcConsoleApp.zip
What did you expect to see?
How to solve my problem
Anything else we should know about your project / environment?
We tried AWS and Aliyun, no similar problems. 😂😂😂😂

Just like this:

Are you using an Azure LB? Perhaps the TCP idle connection is killing you? To test this you can bump that up from the default (I think it is 4 or 5 minutes?) up to 30 and see if the app stops working after 30 minutes instead.
You may need to turn on keep-alive otherwise. Check out keepalive.md Other people can speak to whether or not "Stream removed" is the appropriate error you should be receiving.

@Wisgary
We add the following code on server side, but the problem still exists.
var options = new List<ChannelOption>()
            {
                new ChannelOption("GRPC_ARG_KEEPALIVE_TIME_MS", 1000 * 60 * 1),
                new ChannelOption("GRPC_ARG_KEEPALIVE_TIMEOUT_MS", 1000 * 10),
                new ChannelOption("GRPC_ARG_HTTP2_MIN_SENT_PING_INTERVAL_WITHOUT_DATA_MS", 1000 * 10),
                new ChannelOption("GRPC_ARG_HTTP2_MAX_PINGS_WITHOUT_DATA", 0),
                new ChannelOption("GRPC_ARG_KEEPALIVE_PERMIT_WITHOUT_CALLS", 1)
            };

We can't receive any packets about HTTP2 pings, 😭😭😭😭😭😭😭

@Wisgary
When the client's sleep time is greater than the Azure TCP idle timeout time, the grpc call will be failed with "Stream Removed"

Hi yunfan - if you look at grpc_types.h
#define GRPC_ARG_KEEPALIVE_TIME_MS "grpc.keepalive_time_ms"
You want the value rather than the symbol for your channel options - so "grpc.keepalive_time_ms".

@Wisgary
(⊙o⊙)？😂😂😂😂，It seems that I made a mistake, thx.


ImagePanelComponent not drawing image centered

Previously (right) it was drawing the image centered, currently (left) it does not.

Fixed in #643


[stable/postgresql] init-scripts configmap is skipped with non-null .Values.initdbScripts

Is this a request for help?: No

Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT

Version of Helm and Kubernetes:
N/A
Which chart:
stable/postgresql
What happened:
.Values.initdbScripts is not honored
What you expected to happen:
.Values.initdbScripts should be honored
How to reproduce it (as minimally and precisely as possible):

Edit values.yaml to include:
initdbScripts:
  test.sh: |
    #!/bin/sh

Deploy stable/postgresql chart

Run kubectl get configmaps -l release=<release>

Observe no <release>-init-scripts configmap created

Anything else we need to know:
Missing condition at https://github.com/helm/charts/blob/master/stable/postgresql/templates/initialization-configmap.yaml#L1

Thanks for submitting the issue and the PR!

I miss this issue and the PR and sent this other PR that is already merged.  Please check my comment in your PR.


Improve server list screen

Server list window display all available LMP servers.
Right now that screen is very basic so the code must be improved.
The related code is contained in:
https://github.com/DaggerES/LunaMultiPlayer/tree/master/Client/Windows/ServerList
The idea would be to display the server name, description, players and as much data as possible in a "grid" style.

Finished


feat(enter): make enter work inside form

What: Allow enter key to submit form, if downshift is inside a form tag.

Why: So form can be submitted with enter key if focus is on downshift input.

How: Moved event.preventDefault() so it only executes if menu is open

Checklist:

[ ] Documentation - N/A
[x] Tests (kind of, added a story book... may need unit test? Wasn't sure)
[ ] Ready to be merged
[x] Added myself to contributors table

Codecov Report

Merging #161 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #161   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines         267    267
  Branches       64     64
=====================================
  Hits          267    267

Impacted Files
Coverage Δ

src/downshift.js
100% <100%> (ø)
:arrow_up:

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 3bafbcd...ae542b0. Read the comment docs.


Use jolicode/castor=^0.17.0

[ ] Fix Build macOS pipeline

@jacquesbh I have an issue with static PHP if you know why https://github.com/monsieurbiz/SyliusPluginMaker/actions/runs/9527809570/job/26264831724?pr=5

@maximehuran This is the same issue as the one in castor's pipeline.


Error when I run "pip install -e ."

Collecting scipy>=1.6.0 (from pyntcloud->multi-part-assembly==0.2.0)
Using cached scipy-1.10.1-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (58 kB)
ERROR: Exception:
Traceback (most recent call last):
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/pkg_resources/init.py", line 3367, in _dep_map
return self.__dep_map
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/pkg_resources/init.py", line 3147, in getattr
raise AttributeError(attr)
AttributeError: _DistInfoDistribution__dep_map
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/requirements.py", line 36, in init
parsed = _parse_requirement(requirement_string)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/_parser.py", line 62, in parse_requirement
return _parse_requirement(Tokenizer(source, rules=DEFAULT_RULES))
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/_parser.py", line 80, in _parse_requirement
url, specifier, marker = _parse_requirement_details(tokenizer)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/_parser.py", line 118, in _parse_requirement_details
specifier = _parse_specifier(tokenizer)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/_parser.py", line 214, in _parse_specifier
parsed_specifiers = _parse_version_many(tokenizer)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/_parser.py", line 229, in _parse_version_many
tokenizer.raise_syntax_error(
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/_tokenizer.py", line 167, in raise_syntax_error
raise ParserSyntaxError(
pip._vendor.packaging._tokenizer.ParserSyntaxError: .* suffix can only be used with == or != operators
torch (>=1.8.*)
~~~~~~^
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 105, in _run_wrapper
status = _inner_run()
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 96, in _inner_run
return self.run(options, args)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/cli/req_command.py", line 67, in wrapper
return func(self, options, args)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/commands/install.py", line 379, in run
requirement_set = resolver.resolve(
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/resolver.py", line 95, in resolve
result = self._result = resolver.resolve(
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 546, in resolve
state = resolution.resolve(requirements, max_rounds=max_rounds)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 427, in resolve
failure_causes = self._attempt_to_pin_criterion(name)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 239, in _attempt_to_pin_criterion
criteria = self._get_updated_criteria(candidate)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/resolvelib/resolvers.py", line 229, in _get_updated_criteria
for requirement in self._p.get_dependencies(candidate=candidate):
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/provider.py", line 247, in get_dependencies
return [r for r in candidate.iter_dependencies(with_requires) if r is not None]
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/provider.py", line 247, in
return [r for r in candidate.iter_dependencies(with_requires) if r is not None]
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/resolution/resolvelib/candidates.py", line 401, in iter_dependencies
for r in self.dist.iter_dependencies():
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_internal/metadata/pkg_resources.py", line 247, in iter_dependencies
return self._dist.requires(extras)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/pkg_resources/init.py", line 3070, in requires
dm = self._dep_map
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/pkg_resources/init.py", line 3369, in _dep_map
self.__dep_map = self._compute_dependencies()
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/pkg_resources/init.py", line 3379, in _compute_dependencies
reqs.extend(parse_requirements(req))
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/pkg_resources/init.py", line 3434, in init
super().init(requirement_string)
File "/root/anaconda3/envs/assembly0/lib/python3.8/site-packages/pip/_vendor/packaging/requirements.py", line 38, in init
raise InvalidRequirement(str(e)) from e
pip._vendor.packaging.requirements.InvalidRequirement: .* suffix can only be used with == or != operators
torch (>=1.8.*)

Hummm, I never saw this error before. Searched a bit and this seems to be the closest one. Maybe you want to downgrade pip and retry?
Also this error seems to be triggered by a package that requires torch. Maybe you can try manually install all packages here? I don't really remember the exact version of these packages, but I think pytorch around 1.10 should work

After I  I updated the pip version
pip install pip==24
the issues fixed.
However, when I continue to train:  python scripts/train.py --cfg_file global-32x1-cosine_200e-everyday.py
Traceback (most recent call last):
File "scripts/train.py", line 137, in
cfg = importlib.import_module(os.path.basename(args.cfg_file)[:-3])
File "/root/anaconda3/envs/assembly0/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'global-32x1-cosine_200e-everyday'

Glad that it works. So for training, as stated here, you need to use the full path of the config file, e.g. configs/global/global-32x1-cosine_200e-everyday.py


Unidecode1.2.0

Type: Incorrect
Summary:
Unidecode1.2.0
Details:
Should be LGPL-2.0-or-later found in Files (https://clearlydefined.io/file/8177f97513213526df2cf6184d8ff986c675afb514d4e68a404010521b880643) and License file in Download Files at Unidecode-1.2.0-py2.py3-none-any.whl
Resolution:
Update to LGPL-2.0-or later
Affected definitions:

Unidecode 1.2.0

Correcting to GPL


combine observables from different scopes

Hello
I am using a library which can be initiated and uses a callback to to inform the application that a message is received. How can I instantiate an observable in the callback function and use it in the main loop?
i.e.:
void callback(){
push an event to an observer in the main loop
}
int main(){
  initLibrary(callback);

  // Observe here the observable in the callback function

}

here is an example of how to tie libuv fs callbacks to rxcpp.
// wrap up uv filesystem requests as observables
template<typename F>
auto rx_uv_fs_request(F&& f) {
    return create<uv_fs_t*>([f](auto out) {

        // observables are lazy, this is a COLD observable
        // each call to subscribe will open the file.
        // build the per-subscribe state
        struct uv_request_state {
            explicit uv_request_state(subscriber<uv_fs_t*> o) : out(move(o)) {}
            ~uv_request_state() {
                uv_fs_req_cleanup(&request);
            }
            uv_fs_t request;
            subscriber<uv_fs_t*> out;
        };
        // not ideal, would like to remove these allocations
        auto state = make_unique<uv_request_state>(out.as_dynamic());
        state->request.data = state.get();

        auto req = const_cast<uv_fs_t*>(&state->request);

        // call the function with a callback and the request dedicated to this call
        int result = f(req, [](uv_fs_t* req){
            // take over the lifetime of the state
            unique_ptr<uv_request_state> state(static_cast<uv_request_state*>(req->data));
            req->data = nullptr;

            if (report_fail(req->result, state->out)) {
                return;
            }

            state->out.on_next(req);
            state->out.on_completed();
            return;
        });
        if (report_fail(result, state->out)) {
            // the callback will not be called, cleanup the state
            return;
        }
        // the callback will be called, defer lifetime to the callback
        state.release();
    });
}

auto rx_uv_fs_close(uv_loop_t* loop,
                          uv_file file){
    return rx_uv_fs_request([loop, file](uv_fs_t* req, uv_fs_cb cb){
        return uv_fs_close(loop, req, file, cb);
    });
}

Hello
Thanks for the example. I however notice that you use a lambda without variable capture. I would like to replace:
int result = f(req, [](uv_fs_t* req){ ...
by
int result = f(req, [&](uv_fs_t* req){ ...
However I get a number of type errors because it's not possible to cast the lambda type to the function type.
Best regards
Pieter

Yes, the callback must allow a pointer to be stored in the request so that some state can be allocated and stored in the request and used on the callback so that no captures are required in the lambda.
Or worse, there must be some shared global that is used to pass the state to the callback.


Manual Control Feature

Hello, I was looking to mess around with the newly added "Manual Control" feature, but noticed that a PyPI release has not been made with this feature included.
Are there any plans for a new PyPI release soon?
EDIT: I have gone ahead and installed the new changes via the command 'pip install --upgrade git+git://github.com/mavlink/MAVSDK-Python.' However, I would still like to know if a release is planned soon. Thanks!

Good point, I'll make a release later this week :+1:

@JonasVautherin Thanks! Is there any way an example for manual control could be thrown together before then?

MAVSDK-Python 0.12.0 is out: https://github.com/mavlink/MAVSDK-Python/releases/tag/0.12.0. Let us know if it works with manual control!

BTW, there is a C++ manual control example here. Would be amazing if you could contribute a PR with an example in python for MAVSDK-Python :blush:

Awesome! I'll get it downloaded and look into contributing an example if I figure it out. Thanks a lot


Lua port

I ported the basic example from the README into Lua using LÖVE
https://github.com/ravener/love-life

Great Job and very nice capture of the fly :)

I have added this link to the "Other Ports" section in readme so that people with Lua background benefit from your implementation.
Best

I ported the basic example from the README into Lua using LÖVE
https://github.com/ravener/love-life

By the way, I just had a look at your GitHub profile and read that you are from Afghanistan and you are programming on an Android phone because you don't have a Computer!
You are truly amazing and inspirational.
Keep up the good work

@hunar4321 Thanks! Will try my best.


AllowAnonymous does not work

I created a simple page in my BlazorApp (InteractiveAuto)  and set it to be public. Here is the simple code:
`@page "/test"
@attribute [AllowAnonymous]

Test

@code {

}

The page loads briefly with the word "Test" on the page but then redirected to the login page. I couldn't figure out why. Can you provide some guidance as how I can create public pages that do not require authentication/authorization?
By the way, I created these pages in both the server app and the client app, same.
Thanks!

Hello @franky1017
For guidance on Blazor authentication capabilities, refer to Blazor documentation on the topic.
Normally in a Blazor application pages are available to unauthenticated users by default, unless you mark the page with an [Authorize] attribute.

I have specified [AllowAnonymous] on the page. The page briefly shows the
content and within a second, it redirects to login.
I am not doing anything on the page, just static content. Anything you can
think of that will redirect the page to login after a brief second? I've
never had this behavior with other Blazor app before and this is the first
time I am using your component. Any advice is appreciated.
Frank
On Fri, Jun 21, 2024 at 3:00 AM Yuriy Durov @.***>
wrote:

Hello @franky1017 https://github.com/franky1017
For guidance on Blazor authentication capabilities, refer to Blazor
documentation
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/?view=aspnetcore-8.0
on the topic.
Normally in a Blazor application pages are available to unauthenticated
users by default, unless you mark the page with an [Authorize] attribute.
—
Reply to this email directly, view it on GitHub
https://github.com/BitzArt/Blazor.Auth/issues/16#issuecomment-2182130078,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJETLLPPIF3SDLDRRPOEAFTZIPFSJAVCNFSM6AAAAABJUERXOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSGEZTAMBXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@franky1017
It seems like something may be interfering with your Blazor request pipeline. Try creating a new Blazor project and then following the steps from Blazor.Auth documentation and see if it works for you.
Also, I can take a look at your solution if you attach it to this issue.

Never mind, it's my bad. I had a redirect in the MainLayout. Even though
the page uses a public layout, the main one is referenced in Routes. That's
why it's executed every time.
Frank
On Fri, Jun 21, 2024 at 1:25 PM Yuriy Durov @.***>
wrote:

@franky1017 https://github.com/franky1017
It seems like something may be interfering with your Blazor request
pipeline. Try creating a new Blazor project and then following the steps
https://bitzart.github.io/Blazor.Auth/02.getting-started.html from
Blazor.Auth documentation and see if it works for you.
Also, I can take a look at your solution if you attach it to this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/BitzArt/Blazor.Auth/issues/16#issuecomment-2183152511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJETLLNBD74ZKGMTKRFIES3ZIRO2BAVCNFSM6AAAAABJUERXOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBTGE2TENJRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix problem with hosts in the ingress.yaml file

According to the spec:
https://kubernetes.io/docs/concepts/services-networking/ingress/
http should be on the same level as host.

Fixed and updated the resource version!
Thanks for reporting it.


Error in first sample code: Help needed.

Hello,
I can not resolve this issue. Can someone help?
File "/media/M2Disk/Projects/CompVis/DepthEstimation/ZDepth/zoeenv/lib/python3.11/site-packages/torch/nn/functional.py", line 3924, in interpolate
raise TypeError(
TypeError: expected size to be one of int or Tuple[int] or Tuple[int, int] or Tuple[int, int, int], but got size with types [<class 'numpy.int64'>, <class 'numpy.int64'>]

def call(self, x):
width, height = self.get_size(*x.shape[-2:][::-1])
return nn.functional.interpolate(x, (int(height), int(width)), mode='bilinear', align_corners=True)

Hey @shersoni610 I had this exact same error.  I fixed it by setting up the virtual environment like they describe in the readme.

Clone the repo and cd ZoeDepth
Create the virtual environment using conda:

conda env create -n zoe --file environment.yml
conda activate zoe

After that everything from the sample code worked fine.

Hello Hari,
I got the following error:
File "/Users/.cache/torch/hub/intel-isl_MiDaS_master/midas/backbones/beit.py", line 47, in _get_rel_pos_bias
new_sub_table = F.interpolate(old_sub_table, size=(int(new_height, new_width)), mode="bilinear")
^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: int() base must be >= 2 and <= 36, or 0

Hello Hari,
Sorry, my mistake. Your solution worked.

@jagennath-hari hanks for help. An extra i changed in sanity.py at 37 as:
torch.hub.help("intel-isl/MiDaS", "DPT_BEiT_L_384", force_reload=False)
Otherwise, it downloads again and overwrites the changes I made in the beit.py code.


[feat] GitHub Repo Label Automation

MagicUI GitHub Actions Automation
Description:
This workflow triggers when an issue is opened or reopened. It defines four jobs: “error,” “help,” “question,” “feature” and "component." Each job corresponds to specific conditions based on the issue title. For example, the error job runs if the issue title indicates a bug, while the help job activates for titles related to fixing, assistance, or typos. Each job identifies the issue using environment variables and adds an appropriate label (such as bug, help wanted, question, or enhancement) using the gh issue edit command.
Workflow Name and Trigger:
The workflow is named “Search & Update Issue Label by Unnamed.”
It triggers when an issue is opened or reopened.
Jobs:

The workflow defines four jobs: error, help, question, and feature.
Each job corresponds to a specific condition based on the issue title.

Job Conditions:

The error job runs if the issue title starts with “bug,” “bugs,” “[bug],” or contains the word “bug.”
The help job runs if the issue title starts with “fix,” “[fix],” contains “help,” “help needed,” “HELP,” “HELP NEEDED,” “assist,” or “typo.”
The question job runs if the issue title starts with “question,” “[question],” or contains the word “question.”
The feature job runs if the issue title starts with “feature,” “[feature],” “enhancement,” or “[enhancement],” or contains “enhancement” or “feature.”
The component job runs if the issue title starts with “component,” “[component],” or contains the word “component.”

Job Steps:

Each job performs the following steps:
Identifies the issue using environment variables (GH_TOKEN, GH_REPO, NUMBER).
Adds a specific label (bug, help wanted, question, or enhancement) to the issue using the gh issue edit command.

https://github.com/magicuidesign/magicui/pull/171/commits/4562cbf069b01061a97abe3c034256f7a53b5859
This is a GitHub Actions workflow file that automates a welcoming message to new contributors who open an issue or pull request in a repository.
The workflow, named "Greet New Contributor", is triggered when a new issue or pull request is opened. The message is customized based on whether it's an issue or pull request, and includes emojis to make it friendly and welcoming.
The workflow runs on an Ubuntu environment and uses the GITHUB_TOKEN secret to authenticate the interaction. The message is posted as a comment on the issue or pull request, making the contributor feel welcome and acknowledged by the community.
This automation aims to create a positive experience for new contributors, encouraging them to engage more with the project and fostering a sense of community.


Is there a way to authenticate against the plugin?

Basic authentication doesn't work.
https://user:password@kibana:5601

I need this for my Kibana plugin which goes to a dashboard, does a screenshot of it and send the image by email.

Duplicate of https://github.com/floragunncom/search-guard-kibana-plugin/issues/37


Service integration policies cause race-condition with Step Function creation/modification

Description
When creating a Step Function using the service integration policies provided by the module, there's a race condition which causes the Step Function creation/update to be triggered before policies are successfully created, causing an error:
│ Error: updating Step Functions State Machine (arn:aws:states:eu-central-1:obfuscated-account-id:stateMachine:obfuscated-step-function-name): AccessDeniedException: 'arn:aws:iam::obfuscated-account-id:role/obsfucated-role' is not authorized to create managed-rule.
│       status code: 400, request id: df276253-c571-45cf-8865-04b99a7ff0f5
│
│   with module.step_function.aws_sfn_state_machine.this[0],
│   on .terraform/modules/step_function/main.tf line 15, in resource "aws_sfn_state_machine" "this":
│   15: resource "aws_sfn_state_machine" "this" {
│

If your request is for a new feature, please use the Feature request template.

[X] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: 3.1.0
Terraform version: 1.5.5
Provider version(s):

+ provider registry.terraform.io/hashicorp/aws v5.5.0

Reproduction Code [Required]
data "aws_caller_identity" "current" {}

data "aws_region" "current" {}

locals {
  codebuild_events_arn = "arn:aws:events:${data.aws_region.current.name}:${data.aws_caller_identity.current.account_id}:rule/StepFunctionsGetEventForCodeBuildStartBuildRule"
}

module "step_function" {
  source  = "terraform-aws-modules/step-functions/aws"
  version = "3.1.0"

  name = var.step_function_name

  type = "standard"

  definition = templatefile("${path.module}/template.asl.json", {
    [...]
  })

  logging_configuration = {
    include_execution_data = true
    level                  = "ALL"
  }

  service_integrations = {
    xray = {
      xray = var.enable_xray_integration
    }

    codebuild_StartBuild_Sync = {
      codebuild = [
        var.codebuild_arn
      ]
      events = [local.codebuild_events_arn]
    }

    stepfunction_Sync = {
      stepfunction = [var.foo_step_function_arn, var.bar_step_function_arn]
      events       = true
    }

    lambda = {
      lambda = [
        var.lambda_foo_function_arn,
        var.lambda_bar_function_arn
      ]
    }
  }
}

Steps to reproduce the behavior:
Are you using workspaces? No.
Have you cleared the local cache (see Notice section above)?: Yes
List steps in order that led up to the issue you encountered:

simply run terraform plan and apply

Expected behavior
The policies should be created before the step function is created.
Actual behavior
The step function creation fails because the policy was not created successfully before the step function.
Terminal Output Screenshot(s)

Additional context
As you can see, the step function starts the modification BEFORE the policy has been created. By running it a second time, the apply works without issues.

Please try upgrading Terraform AWS provider to be newer than 5.11.0 (you have 5.5.0). Also, the code in the module is written in the way that it handles dependencies between resources correctly (StepFunction machine, IAM role, IAM policy, IAM policy attachment), so there is nothing we should be doing in the module but rather in provider.
Please open issue in the provider repository.


feat: add unstable_mapSliceZone()

Types of changes

[ ] Chore (a non-breaking change which is related to package maintenance)
[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This PR adds an experimental unstable_mapSliceZone() helper. The helper maps a Slice Zone's Slices using a set of mapping functions, one for each type of a Slice Zone's Slice.
Developers often need to transform or augment Slices when building relatively complex websites. We currently do not offer an official solution for this need, thus developers come up with their own non-standard solutions.
This function is primarily recommended for use on the server where Slices can be augmented, reshaped, or trimmed-down.
The solution provided in this PR is general, flexible, and tested.
Example
The following example reshapes Code Block Slices (id: code_block) by replacing it with an object containing a single codeHTML property. The property contains HTML produced from a highlight() function, which could be a function that calls a syntax highlighting library, like Shiki.
async function highlight(code: string, language: string): Promise<string> {
  // Process the code and return HTML.
}

const mappedSliceZone = await unstable_mapSliceZone(page.data.slices, {
  code_block: async ({ slice }) => ({
    codeHTML: await highlight(slice.primary.code, slice.primary.language),
  }),
});

Any occurance of a Code Block Slice will be replaced with the mapped version. Note how the function provided to the code_block property receives the slice object. Other data is included, such as slices, and index.
A third parameter (context) can be provided to unstable_mapSliceZone(), whose contents will be passed to the function in the context property. The context object is most helpful when mapping functions are defined outside the unstable_mapSliceZone() calling scope, such as in a different module.
const mappedSliceZone = await unstable_mapSliceZone(
  page.data.slices,
  {
    code_block: async ({ slice, context }) => ({
      codeHTML: await highlight(slice.primary.code, slice.primary.language),
    }),
  },
  context
);

Unstable
This helper is marked as unstable_ in its function name and @experimental in its TSDocs. It is being added as a way for developers to test it in production and provide feedback before being marked as stable.
Checklist:

[x] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

🦦

Codecov Report

Merging #302 (180a5a8) into master (7cce22a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##           master     #302    +/-   ##
========================================
  Coverage   99.94%   99.95%
========================================
  Files          49       50     +1
  Lines        5764     6031   +267
  Branches      308      319    +11
========================================
+ Hits         5761     6028   +267
  Misses          3        3

Impacted Files
Coverage Δ

src/helpers/unstable_mapSliceZone.ts
100.00% <100.00%> (ø)

src/index.ts
100.00% <100.00%> (ø)

Made some suggestions, but actually, are you also using this PR to export and reuse looser types for <SliceZone /> component implementations? 🤔

Thanks for catching the clearly incorrect TSDocs! Can you tell I copied it over from the initial @prismicio/react implementation? 😅
The types should not have been exported. I also changed the loose SliceLike type to Slice, but kept SliceGraphQLLike since this function should support the GraphQL API where we don't always know the shape of the object.

I'm not sure about the _unstable prefix, is it that unstable, as in, is there still chances for us to get rid of it? If yes, then sure, let's keep it, otherwise I'd consider dropping it.

unstable_ is a React convention to mark a function as volatile. Its implementation or APIs may change and should be considered unfinished. Once we determine that this function works well and is useful, we can remove the unstable_ prefix and deem it stable.
This function designed for a common, but advanced, use case, so I'm not 100% sure it should be treated the same as our other helpers, like asDate() and asLink(). For now, we can safely treat it as an experiment with no downsides.
What do you think though? Should we just commit to adding it to @prismicio/client as a stable helper?

The types are failing due to the changes in #304. Once we publish #304, which I'll include as part of publishing this PR, the type errors should go away.


Upload to google drive.

Is there a way to upload files after box? If not natively, could you guys recommend a script, please?
Thanks!

I think rclone would be the best way to proceed here.
You can set that up to sync a folder on raspberry to google drive folder.
You could either just sync the media folder itself or if you want to be more selective then use the box script to copy the captured files you want to the local rclone folder
https://rclone.org/

Is there a way to make it (rclone) run after box? As far as I understood, I should make a shell script file called "end_box.sh" and give it permissions. Is it right?

Yes endbox.sh can be used. You need to make sure it is owned by user www-data (chown) and given execution permissions.
If you only wanted to sync at intervals you could use the management_command facility in scheduler. Say set to run every 30 minutes.

Thanks!


Remove observers when screen is not visible anymore

Hi guys,
How am I able to remove the observers I am listening for in my ViewController when I navigate to an other screen?
Looking forward for the solution!
Cheers,
Kevin.

Hi @kevinvugts,
Check out ReactiveKit docs on Disposing and see if that helps.


Implement GitHub actions code testing for Python

Task to do
Implement GitHub Actions to test python server code
Reason
Perform basic quality checks on code
Result
GitHub action(s) test python code

Closing as duplicate from #531


Update example.html docs with HTTP server section and info about pushState support

I've noticed it's pretty common for tutorials and docs everywhere to explain HTML5 History and then say "Oh yeah btw pushState won't work unless you redirect everything to index.html lol" - and then leave it at that. I did some searching and realised nobody's actually explained how to do it in Apache, which is what a huge number of people use..
So I published a quick guide about setting up .htaccess on Apache for pushState - I think it's worthwhile to include this info in the docs, along with other common server configs.
Personally I was searching for this info for ages so I hope it can help out a few other people...
In this pull request I've added a section to the example.html docs for HTTP Server Setup, which would contain all the info people need to get their apps running, locally and in production - this would cover supporting pushState URLs, as well as all the other bits and bobs needed for various servers.
It's cool with me if you want to change the text, restructure it, remove the link to the article or merge it into another section - seems to fit well, but I didn't have a style to base it on since all the other sections are placeholders :o)
I've also added a note and link in the readme with a bit about this, where it was already being mentioned (again, feel free to change.)
Ok! that's all for now.

:+1: to making this more obvious.


How to invoke the bot when ShowCard action button is clicked?

On an adaptive card sent in Teams Conversation, I want to invoke a bot when ShowCard action button is clicked. The bot receives the information like message Id, user Id, etc. I will invoke my service to perform validations and return the data applicable for the user. I will wrap this data in an Adaptive card and respond to the bot. The Teams client will show the returned adaptive card.
Is this feasible? Can you please share document links or code pointers?

URL: https://adaptivecards.io/explorer/Action.ShowCard.html

This is not currently on our roadmap. Take a look at the 1.4 Refresh/Action.Exectue feature to see if they help address your scenario.