showWorkflows Buttons / text aren't aligned

Summary
What happened/what you expected to happen?
https://workflows.apps.argoproj.io/event-flow/argo?showWorkflows=true&selectedNode=argo/Sensor/log&tab=events
https://user-images.githubusercontent.com/2119212/106316570-f8662100-623a-11eb-8efa-faf81e15b462.mov
Diagnostics
What Kubernetes provider are you using?
What version of Argo Workflows are you running?
Paste a workflow that reproduces the bug, including status:
kubectl get wf -o yaml ${workflow}

Paste the logs from the workflow controller:
kubectl logs -n argo $(kubectl get pods -l app=workflow-controller -n argo -o name) | grep ${workflow}

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

thank you - @jsoref - would you like to submit a PR to help fix some of these issues?

I think for the time being I'm going to skip the UI items I've filed recently. I'm going to work on some of the CI items. If no one picks them up, I might circle back in the summer.

Not a fan of the bot.

Help wanted

I don't like 🤖

Do you have an issue with the buttons not having padding between them as well? Or is that the norm for this application?

I'd expect padding between the buttons, yes.


Study obstacles towards making mlkem-native C90 compliant

Acceptance criteria:

[ ] A list of issues/idioms that need to be addressed to make mlkem-native C90 compliant.

If it turns out to be simple, just open a PR and fix it.

I gave this a try. I worked on a PoC for x86_64 here - this does build with -std=c90 without warnings, but not everything is implemented the way we would want it yet.
Here are the obstacles I found:

C++ style comments are not allowed in ISO C90: We would have to replace all // comments with /* */ comments. Pain: medium.  (Easy to do in general, but a bit off work if we want it clean. To test, I removed all // comments in 0c1f4450dcc90c120a3c0522a6295df6f30272f7)
‘for’ loop initial declarations are only allowed in C99 or C11 mode: We have to pull out all variable declarations out of the loop. Pain: low  (implemented by be247bd4255c287d4eb1befe334e3528e0020513)
 use of C99 long long integer constant (e.g., 0x0000000000000001ULL). OpenSSL works around this by setting -Wno-long-long (done in b4bf0872e13a3e6d5d8a54817295f11732cb427a)
 anonymous variadic macros were introduced in C99 - the macros in debug.h would have to defined differently when compiling without debug flags. Pain: low
(Changed in cabcd7957ad6c970e3a92297fdcc28383c9e84b7)
ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]. We'll have to move all declaration to the top. Pain: low.
(This sounds painful, but requires surprisingly few changed - see 7a881c3915e6e5d69e9ae3a84f67ad5da9c163be)
No restrict - we rarely use this anyway, but maybe we want to in the future. We can define it separately, so that modern compiler can still apply the optimizations. Pain: low (Done in 631a4024ec8017855abd501b3fcf791a7efad96b)
No inline - this is a bigger painpoint because if we do remove the inline on C90 builds, the compiler complains about a lot of unused functions in headers (as not all CUs are using them). We will have to silence those using __attribute__((unused)) which is what OpenSSL is doing too. Pain: low (66a998a106b0b1e47e2262c9ab90a92fcc7976ba)
Minor change for initializing pointer arrays done in 1783b7d60fc1102445987abaf20458b4428c8a61.

Overall: I don't want to switch to C90 for our own builds, but achieving C90 compliance requires moderate changes of which I find the comments the most annoying, but I can live with that.
What do you think @hanno-becker @rod-chapman? I can open a PR that changes everything nicely over the weekend.

What is the requirement to do this? Are there potential customers/users who demand C90 compliance?


error corrected in create_lmdb_dataset.py

I have updated
datalist[i].strip('\n').split('\t')
with
datalist[i].strip('\n').split(' ')
to make it correct with the repo readme for create_lmdb_dataset.py.

It may be better to fix readme file instead of fixing create_lmdb_dataset.py?

I think the representation given in the readme is quite understandable changing made it hard to understand.

Hi @2113vm,
I think there is no mistake in the readme.

Splitting by "\t" is better than splitting by " " because your labels can contain " ". And you can't split line correctly. Although you can write something like that
img_p, *labels = line.split(' '),
labels = ' '.join(labels)

but it only solves the problem that you did.

Hi @2113vm
Got your point hence closing this.😊


Dashboard-notifications

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@maziigwe please fix your conflict


ONVIF not working anymore?

Hi,
upgrading from 0.1.3 to 0.1.5 made ONVIF stop working on may 2 cameras (y21ga)...
My Home Assistant cannot find the cameras anymore using the ONVIF integration, and also an Android app called Onvifer I downloaded for testing, cannot find any cameras.
So I made a reset, reconfigured from scratch, but I still have that problem.
Can you help me understanding what's wrong?
My configuration is as follows.
Thanks!

Did you select the correct network interface?

Same problem, I upgraded from 0.1.3 to 0.1.5 and onvif stopped working.
Changed to previous version 0.1.4 and it works now.

What's your model?

Did you select the correct network interface?

My cameras have wifi only, so I presume yes.

Check the command line of the onvif command.
Parameter --ifs

What's your model?

model: y21ga

I have the same problem as partner Ermanno Baschiera, with previous version it works fine for me,
thanks for your job.

What's exactly the problem?
Automatic discovery or onvif?
If you run Onvif Device Manager, does the cam appear automatically in the list?
If not, if you enter the cam manually with the ip address, does it works?

With my y21ga, onvif works properly.

I have problems too recording video clip from the ffmpeg stream.
My cameras are simply configured as:
`camera:

platform: ffmpeg
input: rtsp://<IP_ADDRESS>/ch0_1.h264
name: living_room_camera
#extra_arguments: -rtsp_transport tcp -analyzeduration 15
platform: ffmpeg
input: rtsp://<IP_ADDRESS>/ch0_1.h264
name: outdoor_camera
#extra_arguments: -rtsp_transport tcp -analyzeduration 15
In the log file I can see a lot of these errors:2021-03-28 20:58:20 INFO (MainThread) [homeassistant.setup] Setup of domain recorder took 0.2 seconds
2021-03-28 21:00:04 DEBUG (SyncWorker_2) [homeassistant.components.stream.recorder] Starting recorder worker thread
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8754).
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8461).
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8788).
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 9050).
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8908).
2021-03-28 21:00:04 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8760).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8997).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8934).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8988).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 9087).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 9071).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8946).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8809).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8824).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8786).
2021-03-28 21:00:05 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
2021-03-28 21:00:06 ERROR (recorder_save_worker) [libav.h264] Invalid NAL unit size (0 > 8810).
2021-03-28 21:00:06 ERROR (recorder_save_worker) [libav.h264] Error splitting the input into NAL units.
`
The strange thing is that if I play with mplayer or vlc the video is perfect so probably is something related to the code implemented in Home Assistant.

I'm testing it with vlc and ffmpeg and I can't see any error.
Tomorrow, I will try to re-install from scratch.


[TensorIR][M2a] Storage Align

This PR is part of the TensorIR upstreaming effort (#7527), which adds the one schedule primitive storage_align.
Co-authored-by: Siyuan Feng<EMAIL_ADDRESS>Co-authored-by: Ruihang Lai<EMAIL_ADDRESS>

Thanks @MasterJH5574 @vinx13! Yes, we should add the primitives accordingly in TracedSchedule

Will do the review today!

Thanks @jinhongyii @junrushao1994! I'm gonna to merge it and send it to mainline.

@vinx13 Hey I am curious on the semantics of StorageAlign: why it works on an output buffer, instead of the buffer allocation side? I imagine that it could simplify the logic a bit (don't need to go up straight to find the alloc side)

@vinx13 Hey I am curious on the semantics of StorageAlign: why it works on an output buffer, instead of the buffer allocation side? I imagine that it could simplify the logic a bit (don't need to go up straight to find the alloc side)

The semantic is to annotate producer block (multiple producers are allowed, the annotations will be unified during lowering). IMO the easiest way to specify the buffer in the API is block + write_buffer_index as write_buffer_index is more obvious than alloc_buffer_index, what do you think?

I dont have strong opinion though, just quick thoughts when reading the error checking code path. @tqchen What do you think?

I wrote the origin code that "go up straight to find the allocation site", which is first used in primitive SetScope. SetScope also specify the target buffer by one of its producer block and a write_buffer_index.
From my POV, the interface was designed like this mainly because CacheRead and CacheWrite specify the target buffer by a block and a index. Therefore I inherited the interface in CacheRead and CacheWrite.
At that time I thought that "specifying a buffer directly on the allocation side is impossible", but after a second thought today... Of course we can do that.
So ATM I think specifying the target buffer from the allocation side by alloc_buffer_index is much more convenient. Would like to hear your opinions :-)

Using alloc_buffer_index is possible. My only concern is that the index is not obvious to users when calling schedule primitives without converting the module to tir script and looking at it.

Yes, this is also my concern. But on the other hand, it's also not straightforward for users to know the write_buffer_index of a buffer in a block. So the two ways are similar from this angle.
To clarify, most of the time a block only writes one buffer, and most of the time a block allocate zero or one buffer... Hard to choose...

Okay, seems that we reach a conclusion to use write_buffer_index.


[Icon Request] Support for semantic-release configuration files

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: filename
Icon Name: semantic-release
Sample original Icon:

Filenames: .releaserc, .releaserc.yaml, .releaserc.yml, .releaserc.json, .releaserc.js, release.config.js

More info: https://semantic-release.gitbook.io/semantic-release/v/next/usage/configuration#configuration-file

Is there a light icon for this?

oh I don't know, but in my browser (with force-dark-mode), it looks like this:

please add an icon...


[Bug]:  --disable-enr-auto-update flag for LighthouseBeaconService should not have value

Environment
production
Bug Description
Based on documentation the --disable-enr-auto-update is a flag not a map.

  --disable-enr-auto-update
      Discovery automatically updates the nodes local ENR with an external
      IP address and port as seen by other peers on the network. This
      disables this feature, fixing the ENR's IP/PORT to those specified on
      boot.

ℹ controls/roles/update-changes/molecule/215/prepare.yml should be updated.
💁  Currently LighthouseBeaconService service is erroring and not able to start

Launcher Version
2.1.5
Operating System
Mac
Affected Service (if applicable)
No response
Logs (if applicable)
Usage: lighthouse beacon_node --debug-level <LEVEL> --network <network> --execution-endpoint <EXECUTION-ENDPOINT> --execution-jwt <EXECUTION-JWT> --eth1-blocks-per-log-query <BLOCKS> --datadir <DIR> --http-address <ADDRESS> --metrics --metrics-address <ADDRESS> --disable-upnp --validator-monitor-auto --slasher --slasher-dir <PATH> --port <PORT> --enr-tcp-port <PORT> --enr-udp-port <PORT> --disable-enr-auto-update <--http|--gui|--staking>

error: unexpected value 'false' for '--disable-enr-auto-update' found; no more were expected

Acknowledgment

[X] I have checked the existing bugs and confirmed this issue is not already reported.

Thank you for the report. The bug is already fixed by @NeoPlays with #1912
You can just update your node to get the Lighthouse hotfix:


add new docs contributor videos

This is a Feature Request

What would you like to be added
A learning path for new docs contributors. Topics include: getting started with git, how the k8s community uses github, signing the CNCF agreement.
The learning path would include existing external content where appropriate (github training), but further includes custom videos for project specific knowledge.
The learning path may include a list of links, a youtube playlist, etc.
Why is this needed
New contributors often take a bit to adjust to how the k8s docs community works. This learning path would help ease that transition.
Comments

This was discussed at the quarterly planning meeting in October, and the group decided it was worth actioning.

related to
adding git tutorial content to k8s website https://github.com/kubernetes/website/issues/20485
concept of a skill tree https://github.com/kubernetes/website/issues/18887

/assign

/sig contributor-experience

@geoffcline I think this is a duplicate of https://github.com/kubernetes/community/issues/4953
Is this OK to close?

yep! I didn't think to search there.

/close


⚠ Bofrost.at has degraded performance

In a7300d9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1420 ms

Resolved: Bofrost.at performance has improved in 6f88dd4 after 6 minutes.


Update create_cloned_field to use a global cache

Closes https://github.com/tiangolo/fastapi/issues/4644 by instantiating a global dictionary to store cloned fields. This prevents repeated copying of models which can have significant performance affects. Please see the issue for more details.
This uses a WeakKeyDictionary so we avoid holding a reference to the model. I doubt this will have much practical affect since response model types are often globally defined, but it seems like good practice. This should not increase memory overhead since all the models generated by create_cloned_field will be in scope for the scope of the FastAPI application anyway.
It assigns the dictionary as a default value instead of moving the parameter to an explicitly global value. This allows a new cache to be passed if desired. However, mutable default values can be confusing and it would be fair to move it out of the function signature.
#4644 includes profiling indicating the performance increases here.
As mentioned in the issue, https://github.com/tiangolo/fastapi/issues/894#issuecomment-576484427 refers to an alternative approach that would require work in Pydantic that would make this function irrelevant. That seems better in the long, but this is an easier patch for now.

Does not close #4644 - see comment in the linked issue.

Isn't this a similar solution as what has been proposed here? #4105

@orfisko Yeah it is! I missed that implementation since it wasn't linked to an issue. The choices here are a little different:

The cloned_types parameter is retained. This prevents a breaking change to the function signature and allows a caller to opt-out of the cache or provide a different one.
A WeakKeyDictionary is used. This is mostly a matter of best-practice preventing a cache from creating strong references to types. This could prevent increased memory usage if types are defined dynamically. I doubt it'll have much practical effect though.

Does not close #4644 - see comment in the linked issue.

For others who read this, this comment refers specifically to an issue with using dynamically generated models with ForwardRef fields with this patch. Even then, you can work around the problem by updating forward refs. If someone provides an MRE, I can look into resolving the issue. This does fix the memory consumption and initialization times, see the update from the same user in https://github.com/tiangolo/fastapi/issues/4644#issuecomment-1091825267.

when is this going to merge and release?

Note that coverage is failing because this utility is not explicitly tested and consequently there's not coverage for actually using a custom cloned_types value.
@tiangolo The CI failures at https://github.com/tiangolo/fastapi/actions/runs/3535147825/jobs/5932836062 appear to be related to an change in GitHub actions that breaks Python when using actions/cache@v3. We encountered this at Prefect as well, resetting the cache resolved the issue.

Is this ready to ship? Anything I can do to help push this across the finish line?

This is ready to ship although it does not currently meet coverage requirements for the project. The maintainer has marked this as an issue to investigate when they have time.

@tiangolo any chance we could get this merged, if it's ready to go? this is a huge win, making both app startup and test startup significantly faster.

Indeed @tiangolo. This would be a nice win all around :)

@tiangolo What's required to get this merged?

Thanks for keeping this up to date @madkinsz. Personally I think this is an improvement over the proposal in #4105 and would be my preference. @tiangolo, sorry to bug but can you tell shed light on what's holding this up?

Hey team, looking forward for this merge :)

Awesome, thanks a lot @madkinsz! 🙇
Thanks everyone for the comments, sorry for the long delay. 😅 🙈
I've been working on the migration to Pydantic v2, which will make this clone_field logic unnecessary altogether, as everything will be done on the Pydantic-core (Rust) side. Nevertheless, I'll first make a release with this improvement.
This will be available in FastAPI 0.96.0, released in the next hours. 🚀

@huonw I'm making you an honorary co-author of this PR for your work on https://github.com/tiangolo/fastapi/pull/4105 that was before this. I'm taking this one just for the simplification in the changes/diff, the weakref, and keeping the signature. 🤓


Bump Ruby versions

Just updated Ruby versions for TravisCI. 🙏
https://www.ruby-lang.org/en/news/2019/10/02/ruby-2-4-9-released/
https://www.ruby-lang.org/en/news/2019/10/01/ruby-2-5-7-released/
https://www.ruby-lang.org/en/news/2019/10/01/ruby-2-6-5-released/

Thank you!


needed css for valueless attributify classes in safelist not generated

UnoCSS version
"0.49.4"
Describe the bug
Hi, i am new to this wonderful repo and i was trying out unocss for my very first time.
i was trying to do something like this:
<p :bg-blue="someCondition">Some paragraph!</p>
i know for runtime cases like this i should put my class in safelist:
safelist: 'm-auto bg-blue'.split(' '),
so i expect to not worry about using 'bg-blue'  anyway. however the above scenario will not work unless i use 'bg-blue' in a static valueless attributify situation somewhere else so the corresponding css can generate at build time. eg:
<div bg-blue> ... </div>
also this is my unocss.config.ts for attributify preset:
presetAttributify({       trueToNonValued: true     }),
Reproduction
https://github.com/alisdg/unocss-test
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[ ] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[x] The provided reproduction is a minimal reproducible example of the bug.

app.vue 14 line hidden bg-red is match regex

app.vue 14 line hidden bg-red is match regex

hi @macheteHot . I don't really get your point. I know the red paragraph is working fine.
let me simplify the issue, i think [bg-blue=""], [bg-blue=true] styles should also be generated alongside .bg-blue for bg-blue class if it is in the safelist

i don't know if this is the right thing to do, but this can be solved by writing a variant for all my classes in safelist and adding [bg-blue=""], [bg-blue=true] selectors manually.

Since safelist's tokens are just added directly to the list of tokens to be processed without modification, you may want to do additional processing to make attributify tokens. Something like
safelist: 'm-auto bg-blue'.split(' ').flatMap(x => [x, `[${x}=""]`, `[${x}=true]`]),


Change the _idx_to_token list to a dictionary

PR types
Function optimization

PR changes
Change the _idx_to_token list to a dictionary

Description
Optimize the problem of issue https://github.com/PaddlePaddle/PaddleNLP/issues/599

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

Resolve #599


single-thread API: add usrsctp_get_timeout()

PR #339 makes usrsctp work in single thread environments by forcing the application call usrsctp_handle_timers() periodically. While it's good enough (it does work), the optimized way of doing it would be by having an API like DTLSv1_get_timeout() and DTLSv1_handle_timeout in OpenSSL and BoringSSL.
usrsctp already has usrsctp_handle_timers() so the only missing API would be usrsctp_get_timeout().
This is, the application provides an input to the lib (via usrsctp_socket(), usrsctp_close(), usrsctp_connect(), usrsctp_conninput(), etc) and retrieves from the lib (via usrsctp_get_timeout()) the time that must elapse before the lib (usrsctp) would need to do something (like sending a retransmission, a chunk or whatever). And then the app would set its own timer with that expiration value and call usrsctp_handle_timers() when it expires. This would be much more efficient that the current approach in which the app must call usrsctp_handle_timers() every N ms just in case the lib has something to do at that time.
As @tuexen said:

usrsctp_get_timeout() should give the app the number of milliseconds until the next timer runs off. It should walk the (unordered) list of timers to compute that value. It would even be more efficient if the timer list would be ordered, but then starting a timer would be more complicated.

IMPORTANT: It should be very clear and well documented when the app should call usrsctp_get_timeout(), this is: the app should call usrsctp_get_timeout() and prepare a new timer after calling which usrsctp functions and after which usrsctp events.

why was this closed?
fixed by https://github.com/sctplab/usrsctp/pull/591

What do you mean? This issue was closed because it was done here: https://github.com/sctplab/usrsctp/issues/349

#339 adds
static unsigned get_tick_count(void)
{
#ifdef _WIN32
	return GetTickCount();
#else
	struct timeval te;
	gettimeofday(&te, NULL); // get current time
	unsigned milliseconds = te.tv_sec*1000LL + te.tv_usec/1000; // calculate milliseconds
	return milliseconds;
#endif
}

but that looks very different from #591
int
usrsctp_get_timeout(void)
{
	return sctp_get_next_tick();
}

int
sctp_get_next_tick(void)
{
	int ret;
	sctp_os_timer_t *c;

	SCTP_TIMERQ_LOCK();
	c = TAILQ_FIRST(&SCTP_BASE_INFO(callqueue));
	if (c) {
		uint32_t min_delta = UINT32_MAX;
		while (c) {
			uint32_t delta = c->c_time - ticks;
			min_delta = (delta < min_delta) ? delta : min_delta;
			c = TAILQ_NEXT(c, tqe);
		}
		ret = min_delta;
	} else {
		ret = -1;
	}
	SCTP_TIMERQ_UNLOCK();

	return ret;
}

usrsctp_get_timeout is used in frida-core

@milahu do you mean that PR #591 (not merged and unfortunately obviously ignored) is not a replacement but an addition and improvement to make single thread behavior way better than before?


@now/bash won't work

@now/bash won't work
now version: 15.0.5
build: @now/bash
$ git clone https://github.com/zeit/now-examples
$ cd bash
$ now

then,
$ curl https://bash.atarumix.now.sh
An error occurred with your deployment

NO_STATUS_CODE_FROM_LAMBDA

Perhaps this problem is from version 15.0.2 or 15.0.3.

Hello!
We are working on a fix as we speak. For now, use the following now.json:
{
  "version": 2,
  "public": true,
  "builds": [
    { "src": "index.sh", "use": "@now/bash" , "config": { "useBuildUtils": "@now/build-utils@canary" } } }
  ]
}

Hi @atarumix
Just following up here. Are you still using @now/bash? I would like to invite you to our slack workspace to get some feedback.


Frequency resolution of f_hat

Hi,
I have a question regarding the frequency resolution for the vector of Fourier coefficients.
No matter what I do, whether I use equal M and N, or one being a multiple of the other, it doe's not change the frequency resolution of the coefficients given in f_hat. For equidistant FFT, there is is a relationship for df, which is the reciprocal of M * dt. This obviously does not apply here, since df stays always at df = 1 Hz.
Is there a chance to change this behavior, or am I doing something wrong in principle?
Here is my little test to check the right frequencies and amplitudes. I can see that the 200 Hz, for example, is always at index 200.
import numpy as np
import nfft as nf

n = 2**13
dt = 1.0e-4

x = np.linspace(0.0, n*dt, n)
y = 2. * np.sin(2. * np.pi * 2.5 * x) + 0.5 * np.sin(2. * np.pi * 50. * x) + 1.25 * np.sin(2. * np.pi * 200. * x)

f_hat = nf.nfft_adjoint(x, y, n)

import matplotlib.pyplot as plt
plt.plot(2.0/n * np.abs(f_hat[n//2:]))

plt.grid()
plt.xlim(195, 205)

plt.savefig('test.png')

THX!
Patrick

I'll now answer my own question in case someone had the same issue.
The thing is, that df = 1/(Mdt) holds. One simply has to scale the total sampling time T = Mdt to unity and apply this to the sampling location vector "x". The frequency is then index * df – with df = 1/T.

I want to increase the frequency resolution from 1Hz to 0.5Hz. I divide all sampled time sequences with 2. It seems to work fine.
Thus, it seems to divide all sampled time sequences with a factor K(K>1) to increase the frequency resolution from 1Hz to (1/K)Hz. Do you mean that?
plt.figure()
plt.subplot(3,1,1)
plt.title('nfft')
plt.plot(nfft_adjoint(np.arange(0,win*1e-3,1e-3), np.hanning(win) * csi_12[:,sc_index], 1000),color='red')
plt.subplot(3,1,2)
plt.title('nfft')
plt.plot(nfft_adjoint(np.arange(0,win*1e-3,1e-3), np.hanning(win) * csi_12[:,sc_index], 1000)[250:-250],color='red')
plt.subplot(3,1,3)
plt.title('nufft')
plt.plot(nfft_adjoint(np.arange(0,win*0.5e-3,0.5e-3), np.hanning(win) * csi_12[:,sc_index], 1000),color='red')
plt.show()


Fix image bugs

Coverage remained the same at 60.413% when pulling 0ac6fed6a7e156d178da130ab7575a6988d58611 on JustinDrake:patch-14 into 33daeb30b8261b53c89dd8189ac1b9fd2de01add on OpenBazaar:master.


Metrics view: Add "The last day" to time range selector

Would be nice to have an option for "The last day" to look at memory and cpu metrics over time. It's a big jump from "The last hour" to "The last week" and setting a range under "Custom time window" is several clicks.

Removed from milestone, code added downstream with issue there too.


docs: Update lzv2 README

Added details on package structure and new diagram and added current pkg version to kpt commands

@davelanglois-ssc added a disclaimer on the package versions. I think keeping the current versions in the commands keeps things easier for now until we can add in a command to grab the correct versions for automation.


embedders, vector store, ds should be easier to find

local.New is where the constructor for a local embedder is, it doesnt make sense


Moving translation.properties from Microsoft/java-to-csharp-api-generator repo

translation.properties has been deleted from the generator repo by this PR https://github.com/Microsoft/java-to-csharp-api-generator/pull/30


File path completion is not triggered

On a javascript file(haven't tried it on any other type of file) filepath completion does not trigger.
I have nvim-cm-tern, Languageclient-neovim installed and all the python modules mentioned in the README.

Just pushed an update, try the newest version.

Tried the new update and it works like a charm! Thanks!


UnitFlagManager: Null-Exception

 Issue by JHCD
Monday Apr 10, 2017 at 17:03 GMT
Originally opened as https://github.com/CQUI-Org/cqui/issues/490

With the newest CQUI master (2017-04-10) I get the following Exception every turn-end:
Runtime Error: C:\Users\User\Documents\My Games\Sid Meier's Civilization VI\Mods\cqui\Assets\UI\UnitFlagManager.lua:1504: attempt to index a nil value
stack traceback:
	C:\Users\User\Documents\My Games\Sid Meier's Civilization VI\Mods\cqui\Assets\UI\UnitFlagManager.lua:1504: in function 'OnPlayerTurnActivated'
	[C]: in function 'func'
	[C]: in function '(anonymous)'

 Comment by chaorace
Tuesday Apr 11, 2017 at 01:39 GMT

I was able to replicate this issue

 Comment by SpaceOgre
Saturday May 06, 2017 at 07:52 GMT

I'm not able to replicate this. Are there certain steps I need to take?

 Comment by JHCD
Monday May 08, 2017 at 16:59 GMT

They are only sporadic... one day yes, the other not...


Clarification on Bot WebApp communication to the Azure Bot Service

Can there please be clarity on the communicaiton flow from the AppService hosting the bot to the Azure Bot Connector service?
My understanding is that the Bot would still need to communicate outbound from the AppService to the Bot Connector service.
Can you please confirm if Teams would also need to communicate to the Bot Connector service?
So that process would be direct to the Bot Connector, via public IP/internet.
Lastly, Private Link does not outline the Azure Bot Service/Connector as an available service that PL supports- see https://docs.microsoft.com/en-us/azure/private-link/availability.
So does this solution align to that?

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

ID: 62849df7-a49c-f17e-6876-c3da5954789a
Version Independent ID: a412df8a-f876-d501-6aec-5bc1f010deaa
Content: Secure your Microsoft Teams channel bot and web app behind a firewall - Azure Example Scenarios
Content Source: docs/example-scenario/teams/securing-bot-teams-channel.yml
Service: architecture-center
Sub-service: example-scenario
GitHub Login: @ajafry
Microsoft Alias: aljafry

@LucianFrango Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@MayankBargali-MSFT @LucianFrango Hi,
do you know if there is any update on the case? I am interested and I don't know how to do it, I have the same problem that @LucianFrango

@LucianFrango,
Sorry for the delay.
The bot application logic is hosted in Azure App Service. The App service is moved to private mode:

The incoming message is managed by App Service private endpoint. It is not possible to be accessed by the internet.

The outgoing message is controlled by App Service Integration. All the traffic is sent to a private Vnet, and then controlled by Vnet rules.

The Vnet has an Azure Firewall that exposes a single public IP address that clients can use to communicate with the underlying bot services. The route table, associated with it, instructs Azure to route traffic that isn't within the address prefix of any other route to the subnet where the Azure Firewall instance is deployed. Then the outbound App service traffic arrives at the Azure Firewall which manages the internet connectivity, and in that way the message arrives to Teams.
The incoming traffic from Teams arrives at the Azure Firewall, and the app domain name is translated to the private link IP inside the VNET, finally the traffic arrives at the App Service.
I will close the ticket.
If you would like to help add more details to the article, you might consider submitting an update as a contribution, see: http://aka.ms/aac/contribute for details.
#please-close


Use :focus-visible instead of :focus across the app.

discussion: https://chat.zulip.org/#narrow/stream/6-frontend/topic/select.20boxes

[ ] Using :focus-visible instead of :focus across the app if we want to modify appearance of focus outline across the app. This will remove any outline on interactive elements on click, but show outline while navigating via keyboard.

[ ] To use browser default outline color by removing all overrides.

We will need to remove boostrap overrides.
Override all :focus outlines with:
:focus:not(:focus-visible) {
   outline: none;
}

There are a lot places where we override :focus outline. So, they need to be audited manually.

@zuliport claim

@zulipbot  claim

claim

@zulipbot claim

@zulipbot claim

This issue is not for beginners, figuring out how to do this in a way that doesn't break anything is the main challenge.

@amanagr , I've been doing some research lately on how to tackle this issue. Would send a PR as soon as its ready

@zulipbot claim

@zulipbot help wanted


Custom routes

There needs to be somewhere to register routes that are not inferred like PageLogic and ApiLogic is.
Example: I want to be able to serve custom images at URIs such as example.com/pics/my-cat. It would be good if a custom view-logic-path could be registered somewhere to allow this type of request to be handled anywhere I like, such as<EMAIL_ADDRESS>or even simply<EMAIL_ADDRESS>(where /pics is replacing the default /page).
There is currently no way of doing this in WebEngine, because a browser requests an image with an Accept header that WebEngine doesn't have a route for, so a RoutingException is thrown.

It should also be possible to custom route individual URIs or URI patterns. This requirement has just come up in a project and can't be avoided.
Page at /statement lists out all statement files that the user has uploaded to their account. /statement/original-filename.csv should download the original CSV, but currently this would have to be hacked as a dynamic page logic that outputs the file contents.

This would close #278 too.

I've just had an idea. I think it's a good idea.
Instead of adding a new place to define routes, outside of the current routing system of the page/api directory, provide a CusomRouteLogic extension to the Logic base class.
If a developer just needs simple routing, WebEngine already provides that by putting the route files into the correct places.
However, if a developer needs fully customised routing, they can do so by providing the routing rules within the CustomRouteLogic. For example, in an online motorbike shop, there may be many path derivatives, like /shop/bikes/triumph/daytona-900, /shop/parts/triumph/handlebars, /shop/clothing/helmets/supersafe9000. This is certainly possible to achieve by using the @ rules of file routing, but can become cumbersome if there are many changing rules, or even rules that are dynamic depending on certain data sources.
In this case, a new extension of Logic class can provide an abstract function: route(RequestInterface $request):Route.
This can be placed in a _route.php file within any directory of the /page directory, inlcuding the /page directory itself if fully custom routes are required across the whole application.

Moving to Router repo.

Closed by Router tests.


ContentControl

How can I insert ContentControl or input text into an existing contentcontrol?
Thanks a lot!

At the moment we don't support ContentControl (I even didn't knew it existed).
Some docs fo future implementation:

https://learn.microsoft.com/en-us/office/dev/add-ins/word/create-better-add-ins-for-word-with-office-open-xml#work-with-content-controls
https://stackoverflow.com/questions/63170082/insert-a-content-control-with-ooxml

If someone/you want to tackle this it would be great :-) If not, probably if I will be bored enough I'll take a look what Content Control is and will decide.
Quick look for a custom form based on:

https://support.microsoft.com/en-us/office/create-forms-that-users-complete-or-print-in-word-040c5cc1-e309-445b-94ac-542f732c8c8b

Added a single element, and it looks like this

using DocumentFormat.OpenXml.Wordprocessing;
using DocumentFormat.OpenXml;

namespace GeneratedCode
{
    public class GeneratedClass
    {
        // Creates an SdtBlock instance and adds its children.
        public SdtBlock GenerateSdtBlock()
        {
            SdtBlock sdtBlock1 = new SdtBlock();

            SdtProperties sdtProperties1 = new SdtProperties();
            SdtId sdtId1 = new SdtId(){ Val = 734359063 };

            SdtPlaceholder sdtPlaceholder1 = new SdtPlaceholder();
            DocPartReference docPartReference1 = new DocPartReference(){ Val = "DefaultPlaceholder_-1854013440" };

            sdtPlaceholder1.Append(docPartReference1);
            SdtContentText sdtContentText1 = new SdtContentText();

            sdtProperties1.Append(sdtId1);
            sdtProperties1.Append(sdtPlaceholder1);
            sdtProperties1.Append(sdtContentText1);

            SdtContentBlock sdtContentBlock1 = new SdtContentBlock();

            Paragraph paragraph1 = new Paragraph(){ RsidParagraphAddition = "00A7755C", RsidRunAdditionDefault = "00A7755C", ParagraphId = "21E9D7DC", TextId = "7A60725A" };

            Run run1 = new Run();
            Text text1 = new Text();
            text1.Text = "Test";

            run1.Append(text1);

            paragraph1.Append(run1);

            sdtContentBlock1.Append(paragraph1);

            sdtBlock1.Append(sdtProperties1);
            sdtBlock1.Append(sdtContentBlock1);
            return sdtBlock1;
        }

    }
}

It's similar to Watermark and a few other elements we already kind of support.
I assume this is Content Control you mean right?

Test Controls Diff.docx

Here is the way i figure out to get all contentcontrol (richtextbox) in a document and find one whose tag is what I am looking for:
            /// <summary>
	/// 查找并返回指定Word文档里的所有内容控件集合
	/// </summary>
	/// <param name="document">指定文档</param>
	/// <returns></returns>
	public List<SdtElement> ContentControls(WordprocessingDocument document)
	{
		List<SdtElement> contentControls = new List<SdtElement>();
		var allElements = document.MainDocumentPart.Document.Body.ChildElements;
		foreach (var element in allElements)
		{
			if (element is SdtElement)
			{
				contentControls.Add((SdtElement)element);
			}
			else
			{
				getChildren(element, contentControls);
			}
		}
		return contentControls;
	}

	/// <summary>
	/// 查找指定元素的所有层级的子元素
	/// </summary>
	/// <param name="element">指定的元素</param>
	/// <param name="contentControls">符合要求的元素的集合</param>
	private void getChildren(OpenXmlElement element, List<SdtElement> contentControls)
	{
		if (element.HasChildren == true)
		{
			foreach (OpenXmlElement child in element.ChildElements)
			{
				if (child is SdtElement)
				{
					contentControls.Add((SdtElement)child);
				}
				else
				{
					getChildren(child, contentControls);
				}
			}
		}
	}

	/// <summary>
	/// 从内容控件集合里找到指定Tag的内容控件
	/// </summary>
	/// <param name="contentControls">内容控件集合</param>
	/// <param name="tagName">指定的Tag</param>
	/// <returns></returns>
	public SdtElement ContentControl(List<SdtElement> contentControls, string tagName)
	{
		foreach (SdtElement item in contentControls)
		{
			var tag = item.SdtProperties?.GetFirstChild<Tag>();
			if (tag != null && tag.Val.HasValue)
			{
				if (tag.Val == tagName)
				{
					return item;
				}
			}
		}
		return null;
	}

It's a bug I think:

https://github.com/EvotecIT/OfficeIMO/issues/47
https://github.com/EvotecIT/OfficeIMO/issues/49

It seems to be a bug in the Open-XML-SDK. I guess workaround would be to copy document before opening it up, and use Load that way. It's a different issue, so lets no hijack this thread here.


⚠ DesiTorrents has degraded performance

In 76ebf1b, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1775 ms

Resolved: DesiTorrents performance has improved in 83bbea1.


type declarations

This PR will add typescript support to the lib.
#67

:tada: This PR is included in version 1.8.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SPARK-32393][SQL] Fix postgres bpchar array support

What changes were proposed in this pull request?
This PR fixes the support for char(n)[], character(n)[] data types. Prior to this change, a user would get Unsupported type ARRAY exception when attempting to interact with the table with such types.
The description is a bit more detailed in the JIRA itself, but the crux of the issue is that postgres driver names char and character types as bpchar. The relevant driver code can be found here. char is very likely to be still needed, as it seems that pg makes a distinction between char(1) and char(n > 1) as per this code.
Why are the changes needed?
For completeness of the pg dialect support.
Does this PR introduce any user-facing change?
Yes, successful reads of tables with bpchar array instead of errors after this fix.
How was this patch tested?
Unit tests

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Could you add tests in PostgresIntegrationSuite then check if the test can pass on your local env? (Note: our testing framework, Jenkins, does not run it).

bpchar is internally used in postgresql (https://www.postgresql.org/docs/current/typeconv-query.html), but users can use it in DDL;
postgres=# create table t (v bpchar);
CREATE TABLE

So, I think its okay to add it for better interoperability.

btw, thanks for your contribution, @kujon !

ok to test

Test build #126374 has started for PR 29192 at commit 1000377.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30990/
Test PASSed.

Could you add tests in PostgresIntegrationSuite then check if the test can pass on your local env? (Note: our testing framework, Jenkins, does not run it).

will do!

Test build #126374 has finished for PR 29192 at commit 1000377.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126374/
Test FAILed.

@kujon Any update? If you get stuck, please let me know.

Hi @maropu, I just need a few more days. I'm the meantime, I struggled finding said file, do you want me to add a fresh integration test?

This one: https://github.com/apache/spark/blob/master/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/PostgresIntegrationSuite.scala
You can add test cases there, then run the tests via ./build/mvn -Pdocker-integration-tests -Dtest=none -DwildcardSuites=org.apache.spark.sql.jdbc.PostgresIntegrationSuite

kindly ping.

It seems that it's too hard requirement for a new contributor. How do you want to proceed this, @maropu ?

Can one of the admins verify this patch?

I reopened this again, @kujon .
Hi, @maropu . It seems that this PR aims to support CREATE TABLE t (a char(64)[]); instead CREATE TABLE t (a bpchar).

Can one of the admins verify this patch?

ok to test

Test build #127252 has started for PR 29192 at commit 1000377.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31872/
Test PASSed.

Thank you for your first contribution, @kujon .
We added you to the Apache Spark contributor group and assign SPARK-32576 to you.
Welcome!

Test build #127252 has finished for PR 29192 at commit 1000377.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127252/
Test FAILed.

@dongjoon-hyun apologies, I am quite short on time recently. Thanks for accepting the PR. I'll add the tests when I find some spare time.

Ahh, I see it's being tackled in https://github.com/apache/spark/pull/29397 already. Thank you for that!

Never mind, @kujon! Thanks for your contribution!

Yep. Thank you, @kujon and @maropu . We are collaborating! :)


Does Confluent Kafka dotnet support KSQL or Stream Processing the way Java supports ?

Description
Does Confluent Kafka dotnet support KSQL or Stream Processing the way Java supports ?
How to reproduce
Checklist
Please provide the following information:

[ ] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[ ] Confluent.Kafka nuget version.
[ ] Apache Kafka version.
[ ] Client configuration.
[ ] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[ ] Critical issue.

This isn't supported yet by the .NET client.


Develop

Codecov Report

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

@@          Coverage Diff          @@
##           master    #47   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines         544    546    +2
  Branches       19     19
=====================================
+ Hits          544    546    +2

Impacted Files
Coverage Δ

src/puml-writer.ts
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 8d7de01...bc0975b. Read the comment docs.


Stuck on executing tests of apk with entrypoint test_bundle.dart

Steps to reproduce

Do the setup mentioned in the docs for Android
Run the command : patrol test --flavor dev -t integration_test/app_test.dart

Actual results

Shows : Completed building apk with entrypoint test_bundle.dart
But stuck on : Executing tests of apk with entrypoint test_bundle.dart

Logs

Logs
In next major release, patrolTest method will be intended for UI tests.
If you want to use Patrol in your widget tests, use patrol_finders package.

For more information, see https://patrol.leancode.co/patrol-finders-release
Disable this warning by setting the PATROL_FINDERS environment variable.

No device specified, using the first one (<IP_ADDRESS>:5555)
• Building apk with entrypoint test_bundle.dart...
✓ Completed building apk with entrypoint test_bundle.dart (14.5s)
• Executing tests of apk with entrypoint test_bundle.dart on device <IP_ADDRESS>:5555...

logs_with_verbose.txt

Patrol version
patro: ^2.3.0
Patrol Doctor output

Patrol Doctor output
In next major release, patrolTest method will be intended for UI tests.
If you want to use Patrol in your widget tests, use patrol_finders package.

For more information, see https://patrol.leancode.co/patrol-finders-release
Disable this warning by setting the PATROL_FINDERS environment variable.

Patrol CLI version: 2.2.0
Program adb found in /Users/shrijanregmi/Library/Android/sdk/platform-tools//adb
Env var $ANDROID_HOME set to /Users/shrijanregmi/Library/Android/sdk/
Program xcodebuild found in /usr/bin/xcodebuild
Program ideviceinstaller found in /opt/homebrew/bin/ideviceinstaller

Flutter Doctor output

Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[!] Flutter (Channel stable, 3.13.0, on macOS 14.0 23A344 darwin-arm64, locale en-NP)
    ! Warning: `flutter` on your path resolves to /Users/shrijanregmi/Projects/Flutter/sdk/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/shrijanregmi/fvm/versions/3.13.0. Consider adding /Users/shrijanregmi/fvm/versions/3.13.0/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/2.19.6/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/shrijanregmi/fvm/versions/3.13.0. Consider adding /Users/shrijanregmi/fvm/versions/3.13.0/bin to the front of your path.
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.82.3)
[✓] Connected device (3 available)
[✓] Network resources

Same issue for me too.
This is my issue.
• Building apk with entrypoint test_bundle.dart...
✓ Completed building apk with entrypoint test_bundle.dart (3.9s)
• Executing tests of apk with entrypoint test_bundle.dart on emulator-5554...
See the report at: file:///Users/balajiks/Projects/Flutter/project_name/build/app/reports/androidTests/connected/flavors/elements/index.html
✗ Failed to execute tests of apk with entrypoint test_bundle.dart on emulator-5554 (Gradle test execution failed with code 1) (8.7s)
Exception: Gradle test execution failed with code 1
See the logs above to learn what happened. Also consider running with --verbose. If the logs still aren't useful, then it's a bug - please report it.

It happened to me also, the exact same thing. Running --verbose option it seems that it does not get any tests on the way:
: > Task :app:connectedDebugAndroidTest
: Starting 0 tests on Android(AVD) - 13
:
Instrumentation did not complete:
Instrumentation code: 0
Instrumentation result bundle:
    FAILURE: Build failed with an exception.

    * What went wrong:
    Execution failed for task ':app:connectedDebugAndroidTest'.

✗ Failed to execute tests of apk with entrypoint test_bundle.dart on emulator-5554 (Gradle test execution failed with code 1) (47.7s)

In my case, I am new on patrol. I wanted to migrate some tests, but unable to do so. Followed the 'Getting started' in the docs. I changed the name test to 'test1', still having the same issue. Trying to debug it I found this error (executing it with debug with Intellij Idea, maybe it is not the way to do so).
ClientException with SocketException: Connection refused (OS Error: Connection refused, errno = 111), address = localhost, port = 39400, uri=http://localhost:8081/initialize
This exception is raised when the NativeAutomator is calling initialize().
@bartekpacia can you give me some steps to debug properly? Maybe I can pull more usable info about this 😄

Hey guys,
My issue is different than yours. I don't get any error message - nor the execution fails. It's just stuck on > Task :app:connectedDevDebugAndroidTest and does not proceed to anything.

executing it with debug with Intellij Idea, maybe it is not the way to do so).

You must use patrol_cli to run Patrol integration tests (the ones that have nativeAutomation: true set in patrolTest(). Unfortunately running from IDE directly, with debugging attached, is not supported right now. See #1777.

@shrijanRegmi Please create a new issue.

@shrijanRegmi Please create a new issue.

Why should I create a new issue? I have created the issue with the same exact problem that I am having, if you check the title, description, logs. Everyone else is talking about their own error in the replies.

Why should I create a new issue?

Ah sorry, I didn't notice you're the OP.
Anyway, all of you should attach output of adb logcat to your comments/issues, otherwise it's just guesswork.

Please try using latest patrol v2.3.1.

Not working for me, even with the new patrol_cli v2.2.1 .

After test with the lastest version of patrol and patrol_cli
10-13 16:33:37.842 12268 12268 E AndroidRuntime: java.lang.RuntimeException: Unable to instantiate instrumentation ComponentInfo{com.example.app.test/pl.leancode.patrol.PatrolJUnitRunner}: java.lang.ClassNotFoundException: Didn't find class "pl.leancode.patrol.PatrolJUnitRunner" on path: DexPathList
[[zip file "/system/framework/android.test.runner.jar", zip file "/system/framework/android.test.mock.jar", zip file "/system/framework/android.test.base.jar", zip file "/data/app/~~0Yz2ga10iem80x3Y0KHqyg==/com.example.app.test-dP2SCiL-0sJs9esoT6Bsdw==/base.apk", zip file "/data/app/~~uWZcvtvqhdKkXBdGQn2onQ==/com.example.app-gjJPfFXTmY-pXTZ_-AMFTQ==/base.apk"],nativeLibraryDirectories=[/system/lib64, /system_ext/lib64]]
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.app.ActivityThread.initInstrumentation(ActivityThread.java:6939)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.app.ActivityThread.handleBindApplication(ActivityThread.java:6723)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.app.ActivityThread.-$$Nest$mhandleBindApplication(Unknown Source:0)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2134)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.os.Handler.dispatchMessage(Handler.java:106)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.os.Looper.loopOnce(Looper.java:201)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.os.Looper.loop(Looper.java:288)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.app.ActivityThread.main(ActivityThread.java:7898)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at java.lang.reflect.Method.invoke(Native Method)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:936)
10-13 16:33:37.842 12268 12268 E AndroidRuntime: Caused by: java.lang.ClassNotFoundException: Didn't find class "pl.leancode.patrol.PatrolJUnitRunner" on path: DexPathList[[zip file "/system/framework/android.test.runner.jar", zip file "/system/framework/android.test.mock.jar", zip file "/system/framework/android.test.base.jar", zip file "/data/app/~~0Yz2ga10iem80x3Y0KHqyg==/com.example.app.test-dP2SCiL-0sJs9esoT6Bsdw==/base.apk", zip file "/data/app/~~uWZcvtvqhdKkXBdGQn2onQ==/com.example.app-gjJPfFXTmY-pXTZ_-AMFTQ==/base.apk"],nativeLibraryDirectories=[/system/lib64, /system_ext/lib64]]
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:259)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at java.lang.ClassLoader.loadClass(ClassLoader.java:379)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at java.lang.ClassLoader.loadClass(ClassLoader.java:312)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        at android.app.ActivityThread.initInstrumentation(ActivityThread.java:6935)
10-13 16:33:37.842 12268 12268 E AndroidRuntime:        ... 10 more
10-13 16:33:37.856 12268 12268 I Process : Sending signal. PID: 12268 SIG: 9

That's weird. Do you have ProGuard enabled?

Yes, when I get home I will try to keep your files for testing

patrol: ^2.4.0-dev.3 solved my issue and I was able to run everything normally on an emulator. However, it still does not work on my physical Android device.

@shrijanRegmi Did 2.3.1 also work for you?

In my case I have to set minify=false and only working with patrol: ^2.3.1.

Disabling ProGuard is not always an option, especially for release builds.
We should advise users to exclude Patrol's classes in their proguard-rules.pro.

Sure, I have only advised it for debugging purposes (buildType = debug) . I couldn't make it work trying to keep the classes in proguard file.

If you think your issue is ProGuard-related, see #1804.
If ProGuard is disabled in your build and it still fails to execute, then it's most likely an error in your setup. Make sure to thoroughly go over Getting Started again.
I'm closing this since there's no minimal, reproducible sample was provided by the OP.
Feel free to open a new issue with all the required information provided.


Update AWS SDK for JavaScript to v3

Within the Airnode logs after updating Node.js from 14 to 18 in #1647.
2023-02-13 23:49:28 (node:15) NOTE: The AWS SDK for JavaScript (v2) will be put into maintenance mode in 2023.
2023-02-13 23:49:28
2023-02-13 23:49:28 Please migrate your code to use AWS SDK for JavaScript (v3).
2023-02-13 23:49:28 For more information, check the migration guide at https://a.co/7PzMCcy

tested in #1635 via HTTP gateways and signed gateways using master hash e1746a1d123809fb09edd9e24e307c910731f7b6


Manager, Reporter and Gesture tests

Codecov Report
Patch coverage is 94.71% of modified lines.

Files Changed
Coverage

Sources/Trap/Config.swift
ø

Sources/Trap/Datasource/Datasource.swift
ø

Sources/Trap/Datasource/Gesture/Pinch.swift
0.00%

Sources/Trap/Manager/Reporter.swift
ø

Tests/TrapTests/Mocks/CBCentralManager.swift
ø

Tests/TrapTests/Mocks/UIEvent.swift
ø

Tests/TrapTests/Mocks/UITouch.swift
ø

Tests/TrapTests/Manager/StorageTests.swift
83.33%

Tests/TrapTests/Mocks/MockCollector.swift
83.33%

...sts/TrapTests/Mocks/UISwipeGestureRecognizer.swift
84.37%

... and 33 more

:loudspeaker: Thoughts on this report? Let us know!.


Mouse Utilities not working properly when using Windows Ink

Microsoft PowerToys version
0.65.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Mouse Utilities
Steps to reproduce
Use a Windows Ink compatible device (a graphical tablet in my case) and try using Mouse Utilities
✔ Expected Behavior
To show and follow the pointer while using the pen
To show clicking circles while using the pen
To follow the pointer with crosshairs while using the pen
❌ Actual Behavior
Not showing or following the pointer (find my mouse)
Not showing any yellow or blue circle when clicking (mouse highlighter)
Not following the pointer while using Ink device pen (mouse pointer crosshairs)
Other Software
Software: Wacom Center
Graphical Tablet: One by Wacom Medium (CTL-672)
Tablet Driver Version: 6.4.0-8

I had same issues while using Microsoft Edge. It is possible that Windows 10 misidentified some software as a game.  For microsoft owns products，this should not happen.

I encounter this same issue all the time and it's endlessly frustrating. As much as I avoid Windows Ink in general, some applications only work with a pentablet by having it enabled. Thankfully the configuration software my tablet comes with lets you set Ink enabled/disabled per application, removing the pain that is having to toggle it on for that one app that doesn't just use Wintab like a sensible program and back off again, but with the ones that do this issue can't be avoided. This might not be fixable purely with changes to PowerToys though—I have a separate mouse crosshair utility which I downloaded years ago and isn't quite as good as the one in PowerToys, and vexingly it works in some programs with Windows Ink enabled, but not all or likely most. (To be explicit, the PowerToys crosshair works with none.)
To elaborate on the actual behavior, when the tablet is used, the crosshair stays put at the last position the cursor was when moved with the mouse.

/dup https://github.com/microsoft/PowerToys/issues/20504


refactor: switch Dec to cosmossdk.io/math

Progress towards: https://github.com/osmosis-labs/osmosis/issues/6099
What is the purpose of the change
This PR is similar to: https://github.com/osmosis-labs/cosmos-sdk/pull/481
Now, it focuses on creating aliases and switching the in-repo Dec to cosmossdk.io/math LegacyDec
Brief Changelog

type alias Dec to cosmossdk.io/math `LegacyDec
Delete in-repo implementations

Testing and Verifying
This change is already covered by existing tests
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no)
How is the feature or change documented? (not applicable

Codecov Report

Merging #482 (75958de) into roman/sdkmath-int-uint (8053a2d) will decrease coverage by 0.11%.
The diff coverage is 0.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                     @@
##           roman/sdkmath-int-uint     #482      +/-   ##
==========================================================
- Coverage                   61.53%   61.43%   -0.11%
==========================================================
  Files                         596      595       -1
  Lines                       55580    55064     -516
==========================================================
- Hits                        34203    33826     -377
+ Misses                      19308    19198     -110
+ Partials                     2069     2040      -29

Files Changed
Coverage Δ

types/math.go
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes


High CPU consumption

Environment
Kubernetes cluster set up via kops
Setup explained in the post (as of 2017-12-15), which created a pod (copy-paste from running configuration):
{
  "kind": "Pod",
  "apiVersion": "v1",
  "metadata": {
    "name": "warp-controller-3266622471-0wff5",
    "generateName": "warp-controller-3266622471-",
    "namespace": "default",
    "selfLink": "/api/v1/namespaces/default/pods/warp-controller-3266622471-0wff5",
    "uid": "cf97545d-e1a9-11e7-805b-0a424d12d852",
    "resourceVersion": "5580173",
    "creationTimestamp": "2017-12-15T15:08:30Z",
    "labels": {
      "pod-template-hash": "3266622471",
      "run": "warp-controller"
    },
    "annotations": {
      "kubernetes.io/created-by": "{\"kind\":\"SerializedReference\",\"apiVersion\":\"v1\",\"reference\":{\"kind\":\"ReplicaSet\",\"namespace\":\"default\",\"name\":\"warp-controller-3266622471\",\"uid\":\"cf8b3a31-e1a9-11e7-805b-0a424d12d852\",\"apiVersion\":\"extensions\",\"resourceVersion\":\"5580142\"}}\n",
      "kubernetes.io/limit-ranger": "LimitRanger plugin set: cpu request for container warp-controller"
    },
    "ownerReferences": [
      {
        "apiVersion": "extensions/v1beta1",
        "kind": "ReplicaSet",
        "name": "warp-controller-3266622471",
        "uid": "cf8b3a31-e1a9-11e7-805b-0a424d12d852",
        "controller": true,
        "blockOwnerDeletion": true
      }
    ]
  },
  "spec": {
    "volumes": [
      {
        "name": "cloudflare-warp-cert",
        "secret": {
          "secretName": "cloudflare-warp-cert",
          "defaultMode": 420
        }
      },
      {
        "name": "default-token-bbdv5",
        "secret": {
          "secretName": "default-token-bbdv5",
          "defaultMode": 420
        }
      }
    ],
    "containers": [
      {
        "name": "warp-controller",
        "image": "quay.io/stackpoint/warp-controller:beta",
        "command": [
          "/warp-controller",
          "-v=6"
        ],
        "resources": {
          "requests": {
            "cpu": "100m"
          }
        },
        "volumeMounts": [
          {
            "name": "cloudflare-warp-cert",
            "readOnly": true,
            "mountPath": "/etc/cloudflare-warp"
          },
          {
            "name": "default-token-bbdv5",
            "readOnly": true,
            "mountPath": "/var/run/secrets/kubernetes.io/serviceaccount"
          }
        ],
        "terminationMessagePath": "/dev/termination-log",
        "terminationMessagePolicy": "File",
        "imagePullPolicy": "Always"
      }
    ],
    "restartPolicy": "Always",
    "terminationGracePeriodSeconds": 30,
    "dnsPolicy": "ClusterFirst",
    "serviceAccountName": "default",
    "serviceAccount": "default",
    "nodeName": "ip-172-20-44-106.ec2.internal",
    "securityContext": {},
    "schedulerName": "default-scheduler",
    "tolerations": [
      {
        "key": "node.alpha.kubernetes.io/notReady",
        "operator": "Exists",
        "effect": "NoExecute",
        "tolerationSeconds": 300
      },
      {
        "key": "node.alpha.kubernetes.io/unreachable",
        "operator": "Exists",
        "effect": "NoExecute",
        "tolerationSeconds": 300
      }
    ]
  },
  "status": {
    "phase": "Running",
    "conditions": [
      {
        "type": "Initialized",
        "status": "True",
        "lastProbeTime": null,
        "lastTransitionTime": "2017-12-15T15:08:30Z"
      },
      {
        "type": "Ready",
        "status": "True",
        "lastProbeTime": null,
        "lastTransitionTime": "2017-12-15T15:08:34Z"
      },
      {
        "type": "PodScheduled",
        "status": "True",
        "lastProbeTime": null,
        "lastTransitionTime": "2017-12-15T15:08:30Z"
      }
    ],
    "hostIP": "<IP_ADDRESS>",
    "podIP": "<IP_ADDRESS>",
    "startTime": "2017-12-15T15:08:30Z",
    "containerStatuses": [
      {
        "name": "warp-controller",
        "state": {
          "running": {
            "startedAt": "2017-12-15T15:08:33Z"
          }
        },
        "lastState": {},
        "ready": true,
        "restartCount": 0,
        "image": "quay.io/stackpoint/warp-controller:beta",
        "imageID": "docker-pullable://quay.io/stackpoint/warp-controller@sha256:4fe8ec7d8847f959438be57198b7da69c77f5ecfc264a16aea62ff2e98ab413a",
        "containerID": "docker://f1f3b4af3a81063e811c5c8e8eebb419400d43fa2bb65e612e92a6f492be576d"
      }
    ],
    "qosClass": "Burstable"
  }
}

and an ingress definition:
kind: Deployment
apiVersion: apps/v1beta1
metadata:
  name: httpbin
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: warp-service-app
    spec:
      containers:
      - name: httpbin
        image: kennethreitz/httpbin:latest
        ports:
        - containerPort: 8080
---
kind: Service
apiVersion: v1
metadata:
  name: warp-service
spec:
  selector:
    app: warp-service-app
  ports:
  - protocol: TCP
    port: 80
    targetPort: 8080
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: warp-service
  annotations:
    ingress.kubernetes.io/rewrite-target: /
    kubernetes.io/ingress.class: cloudflare-warp
spec:
  rules:
  - host: httpbin.svstaging.net
    http:
      paths:
      - path: /
        backend:
          serviceName: warp-service
          servicePort: 8080

Expected
Low CPU consumption
Actual
Ingress works as expected, but the controller consumes 100% CPU consumption over 12+ hours period, then 80% (probably b/o other pods activation)

While nothing suspicious in the logs:
I1217 12:01:43.398224       1 controller.go:570] Validation ok for starting warp-service/1
I1217 12:01:43.838907       1 reflector.go:276] github.com/cloudflare/cloudflare-warp-ingress/pkg/controller/controller.go:281: forcing resync
I1217 12:01:43.839346       1 controller.go:130] No annotation found for kubernetes.io/ingress.class
I1217 12:01:43.839491       1 controller.go:133] Annotation kubernetes.io/ingress.class=cloudflare-warp
I1217 12:01:48.850426       1 reflector.go:276] github.com/cloudflare/cloudflare-warp-ingress/pkg/controller/controller.go:280: forcing resync
I1217 12:01:48.852048       1 controller.go:266] Watching endpoint default/warp-service
I1217 12:01:48.852282       1 controller.go:545] Start or Stop warp-service
I1217 12:01:48.852426       1 controller.go:570] Validation ok for starting warp-service/1
I1217 12:02:41.835812       1 reflector.go:405] github.com/cloudflare/cloudflare-warp-ingress/pkg/controller/controller.go:280: Watch close - *v1.Endpoints total 0 items received
I1217 12:02:41.838125       1 round_trippers.go:405] GET https://<IP_ADDRESS>:443/api/v1/namespaces/default/endpoints?resourceVersion=5809647&timeoutSeconds=453&watch=true 200 OK in 2 milliseconds
I1217 12:02:43.403461       1 reflector.go:276] github.com/cloudflare/cloudflare-warp-ingress/pkg/controller/controller.go:279: forcing resync
I1217 12:02:43.403537       1 controller.go:205] Watching service default/warp-service
I1217 12:02:43.403557       1 controller.go:545] Start or Stop warp-service
I1217 12:02:43.403564       1 controller.go:570] Validation ok for starting warp-service/1
I1217 12:02:43.846662       1 reflector.go:276] github.com/cloudflare/cloudflare-warp-ingress/pkg/controller/controller.go:281: forcing resync
I1217 12:02:43.846704       1 controller.go:130] No annotation found for kubernetes.io/ingress.class
I1217 12:02:43.846711       1 controller.go:133] Annotation kubernetes.io/ingress.class=cloudflare-warp
I1217 12:02:48.859712       1 reflector.go:276] github.com/cloudflare/cloudflare-warp-ingress/pkg/controller/controller.go:280: forcing resync
I1217 12:02:48.859777       1 controller.go:266] Watching endpoint default/warp-service
I1217 12:02:48.859805       1 controller.go:545] Start or Stop warp-service

PS: it could be related to my prior experiments with multi-rule ingress configuration (as discussed here ). While I reverted the ingress configuration to the correct one, the controller process remained the same and could have been affected.

Unfortunately, I'm also experiencing this jump in CPU usage after a long period of normal usage.

The ingress controller is deployed in a testing environment with very low traffic. I can't (currently) find any team member action that might explain triggering the jump in CPU usage. My logs and config look to match those posted by @vovayartsev. I'm happy to post anything additional if it helps.

Updating the cloudflare-warp dependencies to the latest https://github.com/cloudflare/cloudflare-warp may resolve this problem.

commit https://github.com/cloudflare/cloudflare-warp-ingress/commit/0501efe0be569cfe625954753447bbd3870f3d7b updates dependencies, built into docker image quay.io/stackpoint/warp-controller:v0.3.0
( quay.io/stackpoint/warp-controller@sha256:b4a8598498709513f5b1f228a759419a808b8b30fda289a6b8b15b61d4cf6229 )

Did the cpu drop?

Sorry for not circling back to this thread... I had left my follow-up feedback on the Cloudflare community forums.
Yes, the CPU usage dropped back to "normal" after updating to the latest version of Warp (at the time).
I can't comment on recent performance, though. We removed Warp and the ingress controller from our system about a month ago due to other issues.


minor(blocksid/seagate): Add BlockSID and Seagate ports feature

Fixes https://github.com/bluecmd/go-tcg-storage/issues/26

:tada: This PR is included in version 0.7.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot select any accounts. I already allowed it in the privacy settings

Thanks for opening this issue.
It seems that when notes are stored in IMAP/email accounts, they are identified as IMAPNote(s) instead of ICNote(s), and the same goes for the folders, accounts, and attachments. I made the logic that identifies the type of item more tolerant in the linked commit.
I was able to reproduce your exact issue, where it gets stuck querying an account (whose name is just the x-coredata uri), and my linked commit fixed the issue -- to be released in v0.4 build 3 shortly.


Gc dlauthenticator

This PR contains the changes to the dlauthenticator class structure that's needed to accommodate the jupyterhub cloud/cluster authenticator needs.
More specifically the pre_spawn_start method.

@iglesu I know this was merged but I just realized we should probably give the readme an update at some point to include basic info about the different authenticator classes and what configuration is needed and also perhaps a snippet on how to easily run the test suite. I can create an issue for this so we can circle back around :muscle:

@iglesu I know this was merged but I just realized we should probably give the readme an update at some point to include basic info about the different authenticator classes and what configuration is needed and also perhaps a snippet on how to easily run the test suite. I can create an issue for this so we can circle back around 💪

yes absolutely! I'll add that, thank you!


replace fig with simpler original

the original figure (from https://about.gitlab.com/2017/08/25/gitlab-and-reproducibility/) is simpler and less cluttered. I don't think we need logos for tools in the figure

Is the old file to be kept?

no indeed not, sharp eyes as usual! the old file is now deleted

Thanks for the change!


TST Try pipenv for CI

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@93d8ff2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   74.62%
=========================================
  Files             ?       30
  Lines             ?     2428
  Branches          ?        0
=========================================
  Hits              ?     1812
  Misses            ?      616
  Partials          ?        0

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 93d8ff2...797b1c3. Read the comment docs.


[Bug Report] AttributeError: 'VideoRecorder' object has no attribute 'path'

Describe the bug
Using gymnasium wrapper RecordVideo causes AttributeError on calling env.step()
I am using it inside a SyncVectorEnv after a RecordEpisodeStatistics wrapper but the same error is reproduced with single usage of the wrapper.
Code example
import gymnasium as gym

def make_env(gym_id):
    def thunk():
        env = gym.make(gym_id)
        env = gym.wrappers.RecordEpisodeStatistics(env)
        env = gym.wrappers.RecordVideo(env, "videos", episode_trigger=lambda t: t % 100 == 0)
        return env
    return thunk

envs = gym.vector.SyncVectorEnv([make_env(args.gym_id)])
observation = envs.reset()
for _ in range(200):
    action = envs.action_space.sample()
    observation, reward, terminated, truncated, info = envs.step(action)

--------------------------
Stack Trace:
    observation, reward, terminated, truncated, info = envs.step(action)
  File "/Users/stella/RL/lib/python3.9/site-packages/gymnasium/vector/vector_env.py", line 198, in step
    return self.step_wait()
  File "/Users/stella/RL/lib/python3.9/site-packages/gymnasium/vector/sync_vector_env.py", line 151, in step_wait
    ) = env.step(action)
  File "/Users/stella/RL/lib/python3.9/site-packages/gymnasium/wrappers/record_video.py", line 172, in step
    self.close_video_recorder()
  File "/Users/stella/RL/lib/python3.9/site-packages/gymnasium/wrappers/record_video.py", line 185, in close_video_recorder
    self.video_recorder.close()
  File "/Users/stella/RL/lib/python3.9/site-packages/wandb/integration/gym/__init__.py", line 66, in close
    m = re.match(r".+(video\.\d+).+", getattr(self, path))
AttributeError: 'VideoRecorder' object has no attribute 'path'

System info
Python 3.9.13
Gymnasium 0.27.1
Installed via pip 23.0.1
MacOS Ventura 13.2.1
Additional context

This was tested using environments CartPole-v1 and LunarLander-v2. Both produce the same error.

The code example was followed by a tutorial that was using the attribute record_video_trigger on RecordVideo that I assume was an older version of gym, so could this be a backward compatibility problem?
Tutorial: https://www.youtube.com/watch?v=MEt6rrxH8W4&t=522s&ab_channel=Weights%26Biases

Thank you in advance!
Checklist

[X] I have checked that there is no similar issue in the repo

This is a problem with WandB integration; which version of WandB do you have? I am not able to reproduce the error with the latest 0.14.0

I am also using version 0.14.0 installed with pip

Okay, I was importing wandb before gymnasium.
The error arises because you don't specify the render_mode; change your code in this way:
- env = gym.make(gym_id)
+ env = gym.make(gym_id, render_mode="rgb_array")

There is a warning saying this, however, no error should be raised; I will propose a fix to WandB


Bluetooth: host: Remove bt_l2cap_chan_send_cb

EATT is the only user of bt_l2cap_chan_send_cb, and not necessary. We
can maintain the same functionality without it.
Instead of passing and storing the callback into l2cap, we can store it
in a callback queue in the ATT bearer struct.
This will allow us to remove that internal API later, in order to
streamline the l2cap API.

The test in this PR will not pass until https://github.com/zephyrproject-rtos/zephyr/pull/65678 is merged


chore: fmt sdk

#234
#251  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Add zh-CN machine translations.

The goal of this PR is to add bilingual Chinese subtitles based on the accurate English subtitles (#384) with machine translation.
Why use bilingual subtitles for collaboration?
Since our localized version subtitles are done by machine translation, we may need to reference to the original (English) ones, and the bilingual subtitles might could help, since in this type of subtitle, we are putting the localized words on top and English on the bottom, so it will be easier for reviewers to review. And when we publish the localized ones to YouTube, we could add a new function to generate_subtitles.py to remove the English lines.

If we are OK with the bilingual subtitles plan, I could use the tools to generate other languages, too.
These zh-CN translations were also:

Replaced 您 to 你
Added space between English and Chinese
Corrected the translation of "Hugging Face"
Corrected part of Transformers translations

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi Lewis, thanks for your great support!
For adding the new function to the script, considering this is not kind of urgent work, I'd like to introduce Steven @tyisme614 to give it a try, buuut... if we're not finished this after all the zh-CN subtitles are reviewed, I'll have to ask your help to do it, WDYT? 🥺

My vote goes to merge this PR now, since @xianbao and me are planning to organize the other Chinese language volunteers for the further collaboration, directly open PR to the main repo will be easier for us.
Thanks again for the great help, Lewis and Tiezhen!

Done and thank you for working on making the course accessible to Chinese speakers!


Gem will work on 4.2.1

Tested on Rails-4.2.1 version in my current project and everything is working fine.

Thanks for the PR, @kuldeepaggarwal! Would you try removing the activesupport dependency to see if that clears up the security warning.
Since activesupport is a dependency of activerecord, I don't think there is an issue with removing it.

Sure!!

@anarchocurious Updated the PR

Can you please release a new version so that I can refer to my project's Gemfile?

I just pushed it out as v0.1.2.


Paths complete with backslash on Windows 10

I'm using PyCharm 2019.2 on Windows 10. Seems like relative paths work fine with forward slash (e.g. starting with "./" or "../" plus Ctrl + Space), but if I enter 'C:/' no suggestions come up. If I use backslashes, e.g. 'C:\' this will work for autocompletion, but the resulting string is inappropriate for python (\ is an escape character). The desired behavior would be to have C:/ produce the appropriate suggestions, and new directories would complete with an additional /
PyCharm 2019.2 (Professional Edition)
Build #PY-192.5728.105, built on July 23, 2019
Runtime version: 11.0.3+12-b304.10 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
GC: ParNew, ConcurrentMarkSweep
Memory: 1949M
Cores: 8
Registry:
Non-Bundled Plugins: CMD Support, com.pinkfloyded.idea-file-path-autocomplete.idea-file-path-autocomplete

Hi Patrick, thanks for the report. This issue is fixed in 0.4.1. Please try it out.


az synapse workspace create should provide the option to specify the Managed Resource Group name.

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
az synapse workspace create
Describe the bug
Deploying an Azure Synapse workspace requires a separate managed resource group. This resource group should be able to be created when running the az synapse workspace create CLI command. Currently this is not possible. This is clearly an omission and should be rectified.
To Reproduce
az synapse workspace create
Expected behavior
Should give the user the option to create the managed resource group. It doesn't
Environment summary
Azure CLI 2.39.0 on Windows 10
Additional context
None

route to CXP team

Adding Service team to look into this ask.
@wonner @zesluo Could you please look into this and provide an update ?

Hi @plynch66, we will add a parameter "--managed-resource-group-name" to support managed resource group feature. Wish to deploy on the next release. Thank you.


English locale displayed twice in Web UI

Since we bumped to Flask 3.0.x and flask-babel 4.0.x the English locale is shown twice in the language selection dropdown

Fix in https://github.com/PiSCSI/piscsi/pull/1291
Before, BABEL.list_translations() didn't list "en" in the tuple of supported locales, but they do now. Removing a workaround fixes the problem.


tf.data.Dataset.padded_batch() should support padding to nearest N bytes

System information
TF 1.5
Python 3.6.4 (Anaconda)
Problem description
If using tf.data.Dataset.padded_batch() on input with variable size, it looks like there is significant overhead first time a batch with examples of size X is used on GPU. If batch with examples of size X is used second time, the execution is approx. 4x faster. (I suppose it is due to a way tensorflow handles data.)
In case the input size varies a lot, this slows the computation enormously. It could be solved by padding the batch to nearest N bytes so that there is only limited number of sizes pushed onto GPU and overhead thus becomes negligible.
Solution
It would be great if padded_batch supported argument pad_to_nearest_bytes or/and allowed_batch_sizes for enumeration of possible sizes.
Proof:
TIME    BATCH_SIZE (2nd dimension = the data size)
0.15205 20480
0.06180 20480
0.80608 147456
0.24141 147456
0.74360 135168
0.21659 135168
0.58724 98304
0.16206 98304
0.05387 20480
0.05694 20480
0.53993 90112
0.15452 90112
0.23547 147456
0.23576 147456

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

This can already be done with more flexibility using Dataset.map(lambda x: tf.pad(...)), so I don't think there's any need at the present time to extend Dataset.padded_batch() to support it.


As shown in the remarks, according to the documentation, write HelloServer, but this configuration file is not found. Is the configuration file for this service generated manually or automatically generated by a tool?

E：Automatically generated, use the ps and grep instructions to find and display this process path. There is a conf directory under the corresponding directory, which contains this configuration file.
C：自动生成，使用ps和grep指令查找显示这个进程路径。对应目录下有个conf目录，里面含有这种配置文件。


Integrate aws-ebs-csi-driver plugin into the deployment (/terraform-aws-eks)

Request related to a new offering from AWS: https://docs.aws.amazon.com/eks/latest/userguide/managing-ebs-csi.html. CSI add-on is now available to public over EKS managed add-ons.
Till some time ago 'ebs-csi' functionality available as 'unmanaged plugin' in other terraform modules, AWS now offers ebs-csi driver as EKS managed add-on, the unmanaged deployment conflicts with RBAC if in later step you want to manage it as 'EKS managed add-on'

EXAMPLE of unmnaged:
source = "DrFaust92/ebs-csi-driver/kubernetes"
version = "2.13.0"
once aws-ebs-csi-driver is installed as unmanaged and you try to transfer it to managed you get a conflict error, the only way is to 'OVERWRITE'.
error: Conflict with "HashiCorp" using rbac.authorization.k8s.io/v1: .rules

cluster_addons = {
coredns = {
resolve_conflicts = "OVERWRITE"
}
kube-proxy = {}
vpc-cni = {
resolve_conflicts = "OVERWRITE"
}
}
Please add new cluster_addons

this is an upstream issue on the EKS service side - there isn't any change that we can make within the module to fix this

I just wanted to check whether this was available now?
My cluster addons looks like this
 cluster_addons = {
    coredns = {
      resolve_conflicts = "OVERWRITE"
    }
    kube-proxy = {}
    vpc-cni = {
      resolve_conflicts        = "OVERWRITE"
      service_account_role_arn = module.vpc_cni_irsa.iam_role_arn
    }
    aws-ebs-csi-driver = {
      resolve_conflicts = "OVERWRITE"
    }
  }

And i can see in the aws dashboard that the aws-ebs-csi-driver is active.
However im having some issues when attempting to create a pvc from a storage class

yes the managed addon is available by EKS now - https://docs.aws.amazon.com/eks/latest/userguide/managing-ebs-csi.html

okay, thanks for clarifying!

@bryantbiggs Follow up question when integrating aws-ebs-csi-driver using this module. I seem to have a permissions issue when trying to setup a pvc. It's constantly in a state of Pending and suggests there isent the correct permissions

Warning  ProvisioningFailed    115s  ebs.csi.aws.com_ebs-csi-controller-bb878b5db-tgw4t_39cd0e10-7a75-46d0-9821-6766be0b0fee  failed to provision volume with StorageClass "elasticsearch": rpc error: code = Internal desc = Could not create volume "pvc-afc5056b-abd3-4fce-bed6-b5b309943ff6": could not create volume in EC2: UnauthorizedOperation: You are not authorized to perform this operation. Encoded authorization failure message: Q8Sb1SzHYNEOEh1k_k-awG7IRswCqE5XdGgqgxUa0draxmo6pFtu8zq8C8VXIjxhJfKjzLY6Aqc_jqB4TVm-K3TsmKZocpnCx8H39XvnkJ_MK3hM_ATkAxvN0Lxkhf9WdvLWIC4KEeKaeQ_rSl5IlMgh7hPH_wB4RxisIlWd7K1DlMscEMGUDg1ut0XoUG_JC66Sb_6zyDNKBL6O1rkfEmGBvkMq-g7UcQZ0_KyfPOSjklDruHBEQVIZ15pXo5O9qy4kSteRHrB6XzBRRJEigQhRzfScyXSuhMJ9XP8OTNDeBzRvKAY2bow-_IEQZb0oGwH1YVEsNAF3n7ZeGlE3qGcPRzHy6KIor57Hj9qRGymYV8H3rFqOrzv6uQnHRA2q47Fp0vwTRuPdhqZVU-5fP2UjgD_aRE7de0I1K5D6ZtopOqk8ptKqTP507cAPhxRhnUmWjl9Ep4sToOkiniZz00aNkp69hmhmMxv17w1VUFHaD-jV3GjvvAfQ2_nv5JnYKggQ-BV-m70wPhMrqzyBEDZs8fNkGp4NgsvEi3eIKwQqxDzkDbkL997RMgE39corjNegG9Ks2IVFQlknE_EJtGb1CR9gvUS84C9mJvvBS-WBoV6UKtkj5rcv-imIm5CAW0zWZ89JuZtt4sgzxQ

Can you see what i might have missed?
  cluster_addons = {
    ...
    aws-ebs-csi-driver = {
      resolve_conflicts        = "OVERWRITE"
      service_account_role_arn = module.ebs_csi_irsa.iam_role_arn
    }

  }

  module "ebs_csi_irsa" {
  source  = "terraform-aws-modules/iam/aws//modules/iam-role-for-service-accounts-eks"
  version = "~> 4.12"

  role_name             = "${local.cluster_name}-ebs-csi"
  attach_ebs_csi_policy = true

  oidc_providers = {
    ex = {
      provider_arn               = module.eks.oidc_provider_arn
      namespace_service_accounts = ["kube-system:aws-ebs-csi-driver"]
    }
  }

  tags = local.tags
}

My best guess is that the above module is not enough, I cannot see the serviceaccount that corresponds to the role aws-ebs-csi-driver in my k8s cluster.
{
   "Version": "2012-10-17",
   "Statement": [
       {
           "Sid": "",
           "Effect": "Allow",
           "Principal": {
               "Federated": "arn:aws:iam::700849607999:oidc-provider/oidc.eks.eu-west-2.amazonaws.com/id/B02660CC7E55090032325E1CD715816B"
           },
           "Action": "sts:AssumeRoleWithWebIdentity",
           "Condition": {
               "StringEquals": {
                   "oidc.eks.eu-west-2.amazonaws.com/id/B02660CC7E55090032325E1CD715816B:aud": "sts.amazonaws.com",
                   "oidc.eks.eu-west-2.amazonaws.com/id/B02660CC7E55090032325E1CD715816B:sub": "system:serviceaccount:kube-system:aws-ebs-csi-driver"
               }
           }
       }
   ]
}

kubectl get serviceaccounts -n kube-system | grep "ebs"*
ebs-csi-controller-sa                       1         3d22h
ebs-csi-node-sa

I just want to confirm that am i right in thinking i still need to  create this serviceaccount. aws-ebs-csi-driver and attach the role to it? Or should that have been taken care of already by the setup of the cluster?
resource "kubernetes_service_account" "ebs_csi_driver" {
  metadata {
    name      = local.service_account_name
    namespace = local.service_account_namespace
    annotations = {
      "eks.amazonaws.com/role-arn" = module.ebs_csi_irsa.iam_role_arn
    }
  }
}

Oh i just spotted what might be issue, https://docs.aws.amazon.com/eks/latest/userguide/csi-iam-role.html
The service account should be named  ebs-csi-controller-sa
https://github.com/terraform-aws-modules/terraform-aws-iam/blob/master/examples/iam-role-for-service-accounts-eks/main.tf#L84-L98

can you post here the complete working code please


Bump version to 0.2

And bump compat bounds for LibCImGui.jl so it can be used with the 1.89.* releases.

Codecov Report
Patch and project coverage have no change.

Comparison is base (33e106d) 6.97% compared to head (f7a5037) 6.97%.

: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.
Additional details and impacted files
@@          Coverage Diff           @@
##           master      #6   +/-   ##
======================================
  Coverage    6.97%   6.97%
======================================
  Files           9       9
  Lines         301     301
======================================
  Hits           21      21
  Misses        280     280

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Scatter shows error when connecting

Fixed: Scatter removed in https://github.com/dfuse-io/ice/commit/1d81394d16765f8c12ed2a50debe31cef63fbc69