IJulia in local environment, dead kernel

Current Outcome:
Recently, I freshly installed Julia 1.6.2 in my computer. I made a dedicated environment for a project with Pkg.generate("envName") along with Python venv. After activating python venv and installing jupyter with pip, I open Julia and set environment variable as below:

ENV["PYTHON"] = Sys.which("python")
ENV["JUPYTER"] = Sys.which("jupyter")

Those 2 commands set the environment variable to the binaries in the venv folder. After activating environments with Pkg.activate("envName"), I proceed with installing IJulia with Pkg.add("IJulia") within generated TOML file. The installation went normally without any error, and when I run IJulia.notebook(dir="path/to/project"), jupyter notebook opened just fine. However, when I open a notebook with IJulia kernel, it tries to start the kernel for a while and then throws "Dead Kernel" prompts to me.
Expected Outcome :
The kernel started successfully and I can use Julia in my notebook.
Action taken :
I tried updating IJulia and build it again [Fail], I tried installkernel("Julia", "--project="path/to/project.toml") [Fail]. I also tried to open jupyter notebook directly from cmd (not Julia REPL), and it throws me this message
ERROR: LoadError: ArgumentError: Package IJulia not found in current path:
- Run `import Pkg; Pkg.add("IJulia")` to install the IJulia package.

Stacktrace:
 [1] require(into::Module, mod::Symbol)
   @ Base .\loading.jl:893
in expression starting at C:\Users\arify\.julia\packages\IJulia\e8kqU\src\kernel.jl:1
[I 11:32:40.564 NotebookApp] KernelRestarter: restarting kernel (1/5), new random ports
ERROR: LoadError: ArgumentError: Package IJulia not found in current path:
- Run `import Pkg; Pkg.add("IJulia")` to install the IJulia package.

Stacktrace:
 [1] require(into::Module, mod::Symbol)
   @ Base .\loading.jl:893
in expression starting at C:\Users\arify\.julia\packages\IJulia\e8kqU\src\kernel.jl:1
[I 11:32:43.580 NotebookApp] KernelRestarter: restarting kernel (2/5), new random ports
ERROR: LoadError: ArgumentError: Package IJulia not found in current path:
- Run `import Pkg; Pkg.add("IJulia")` to install the IJulia package.

Stacktrace:
 [1] require(into::Module, mod::Symbol)
   @ Base .\loading.jl:893
in expression starting at C:\Users\arify\.julia\packages\IJulia\e8kqU\src\kernel.jl:1
[I 11:32:46.605 NotebookApp] KernelRestarter: restarting kernel (3/5), new random ports
ERROR: LoadError: ArgumentError: Package IJulia not found in current path:
- Run `import Pkg; Pkg.add("IJulia")` to install the IJulia package.

Stacktrace:
 [1] require(into::Module, mod::Symbol)
   @ Base .\loading.jl:893
in expression starting at C:\Users\arify\.julia\packages\IJulia\e8kqU\src\kernel.jl:1
[I 11:32:49.613 NotebookApp] KernelRestarter: restarting kernel (4/5), new random ports
ERROR: LoadError: ArgumentError: Package IJulia not found in current path:
- Run `import Pkg; Pkg.add("IJulia")` to install the IJulia package.

Possible cause:
jupyter notebook doesnt find IJulia in the place project.toml directed, or maybe jupyter notebook doesnt find the project.toml at all

I made a dedicated environment for a project

IJulia must be installed in the global/default Julia environment, unless you use the IJulia.installkernel function to install a kernel that launches Julia in your environment.


Introduce context on view component

Is your feature request related to a problem? Please describe.
Tooltips generated from a glossary should show up only on the first occurrence on a page.
Describe the solution you'd like
Volto provides a context of the View component for the case that blocks need to know about each other.
Wrap the RenderedView component with a context provider.
Describe alternatives you've considered
Hints where to hook in the collection of blocks without a change in packages/volto/src/components/theme/View/View.jsx are welcome. As an alternative to the wrapper with context provider, I see only the shadowing of the View component, which is obviously no option in an add-on.
Additional context
https://github.com/rohberg/volto-slate-glossary/pull/8
Per default, glossary terms are searched per slate leaf. Which leads to a tooltip on every match.

For the record: I want this. :-)

Closing with a solution without the need of a change request in Volto. Details in the discussion of the PR #6395.


Add Lua version

Lua is a versatile language. It's also very fast for a dynamic language. It's a simple program and can be improved, because right now I'm coping the table to another and allocate memory as well 2 times.

Language
Simple
Optimized
Notes

grep
0.04
0.04
grep baseline; optimized sets LC_ALL=C

wc -w
0.25
0.25
wc baseline; optimized sets LC_ALL=C

C
0.75
0.16

Rust A
1.08
0.27
by Andrew Gallant

Lua
1.21

Rust B
1.23
0.21
also by Andrew: bonus and custom hash

C++
1.44
0.79
"optimized" isn't really optimized

Perl
1.55

by Charles Randall

Python
1.65
0.98

Go
1.76
0.31

AWK
2.00

optimized uses mawk

Shell
2.16
1.70
optimized does LC_ALL=C sort -S 2G

These are the results from my benchmark.

I like this a lot, thanks! However, this looks more like the optimized version, as it reads in large chunks and then shuffles down the "rest". Could you make a simple.lua that's as simple as possible, and then move this version to optimized.lua?
Also, you probably want to lowercase the whole buffer, rather than word at a time.
Could you also please update test.sh/benchmark.py, and add credits to the README (you may need to merge the latest master).

Here is a simple Lua version with basically the same performance as Python:
local counts = {}
local function insert(s) counts[s] = (counts[s] or 0) + 1 end
string.gsub(io.read("a"):lower(), "([^%s]+)", insert)

local t = {}
for k in pairs(counts) do table.insert(t, k) end
table.sort(t, function(a, b) return counts[a] > counts[b] end)

for _, v in ipairs(t) do print(v .. ' ' .. counts[v]) end

I added simple and changed the optimized version. Thanks to catwell for the simple version. Added credits as well.

I like that simple version, @catwell -- that looks great. However, there's something wrong with this PR, as it has 84 commits attached and a whole bunch of other stuff. Can you please rebase off the latest master and squash into a single commit?


Message stopped strangely (by length?), but the button is still the red stop button

Describe the bug
问题描述
Some long responses seem to stop before it should, and the button is still a red stop button, even if it's stopped.
To Reproduce
如何复现
Don't figure out a 100% method to reproduce it, but it looks like there might be some logical bugs can be catched.
Expected behavior
期望行为
The message continues only when it should, and when it's finished, the button become a green send button
Screenshots
截图说明
From debug window:

Please complete the following information):
请补全以下内容

OS: Ubuntu 22.04
Browser: Firefox v115.0
Extension Version: v2.4.0
Model: GPT-4-32k on Azure OpenAI
Configs:

Max token: 15000
Temperature: 0.2

fixed in https://github.com/josStorer/chatGPTBox/commit/3bef0c0c98f639f3506f2602d3de64351ffbe8f3

https://github.com/josStorer/chatGPTBox/releases/tag/v2.4.1

Awesome, thanks!


Always report skipped feature hits for the original layer

In the process of getting to the bottom of #4822, it occurred to me why so many iterations of patching the select interaction fixed one issue, only to create another: hit detection of features should return the same features and layers, no matter be features selected or not.
I think this fix makes the affected code easier to read and understand, and turns the select interaction into one that could almost be loved.
Fixes #4822.

Nice branch name 😉

Nice branch name 😉

Agreed.
Other things I like:

Net 7 lines removed!
And that is with something like 15 test lines added!

Things that make me scared:

This code always breaks.
This changes a test in a way that makes it seem like applications would break, but I'm so scared of the select interaction that I don't know if this is expected behavior or not.

In summary:

Andreas is awesome for his perseverance with the select interaction.
I hope that someone else who uses the select interaction can review this.
It would be really nice to have a clear understanding of what was broken before and what is fixed now.

@tschaub I wrote a blog post about the whole story: http://ahocevar.net/openlayers/2015/02/17/trilogy-of-error.html. I hope it explains the whole evolution of these problems.

@tschaub:

This changes a test in a way that makes it seem like applications would break, but I'm so scared of the select interaction that I don't know if this is expected behavior or not.

There is no real need to make this change, so I changed it back to the way it was.
The key change of this pull request is that skipped features are now hit-detected the same way as if they had never been chandged on their original layer. This means that the also won't be hit-detected again when duplicated on an unmanaged layer. Which is what the Select and Modify interactions do.

It would be really nice to have a clear understanding of what was broken before and what is fixed now.

When a feature is selected, the Select interaction calls ol.Map#skipFeature() with the selected feature, and copies the feature to its internal unmanaged layer. Before this change, an already selected feature was reported by ol.Map#forEachFeatureAtPixel to come from a null layer. After this change, it is still reported to come from the layer that it was selected from.
For user created unmanaged layers, nothing changes.

I wrote a blog post about the whole story: http://ahocevar.net/openlayers/2015/02/17/trilogy-of-error.html.
@ahocevar So if you knew that all one year before (see publishing date), why didn't you fix it earlier? ;-)

Thanks @ischas - I shouldn't be writing blog posts at 2am.

@ahocevar 404

@ischas Took a few seconds to update on the server. Now it's live.

Andreas is awesome for his perseverance with the select interaction.

No doubt. All of us are benefited at his cost.

Nice work @ahocevar, did some testing with an application of mine that makes good use of the Select interaction here and there and it works as expected.
This also makes #3453 easier to fix so I've rebased that on this PR.

Anyone brave enough to review this? @fredj or @tsauerwein maybe?

LGTM, thanks Andreas


Test summary start time in locale causes issues with junit reporter

Summary
The syncTests.ts and asyncTests.ts build test summary JSON with start times in local locale. Later, when using these summaries to generate a junit report, the Date constructor fails to construct a valid date, causing a failure at;
RangeError: Invalid time value
    at Date.toISOString (<anonymous>)
    at JUnitReporter.format (/Users/jpower/Code/sfpowerscripts/packages/core/node_modules/@salesforce/apex-node/src/reporters/junitReporter.ts:32:61)

Should the test summary just have an ISO date string for start time instead, given intention to programmatically parse again later in junit reporter?
Steps To Reproduce:

Setup machine to have a locale that will produce a date time string like: Sat Jul 22 2023 9:54:03 a.m. (like en-CA)
Run apex tests via apex-node
Use junit reporter to save results

Expected result

Expect correctly parsed date

Actual result

Error, invalid time value

Additional information
Apex library Version:  "@salesforce/apex-node": "1.1.0"
OS and version: Darwin TRAC-C02F314EML7L 22.5.0 Darwin Kernel Version 22.5.0: Thu Jun  8 22:22:22 PDT 2023; root:xnu-8796.121.3~7/RELEASE_X86_64 x86_64

This should be fixed with https://github.com/forcedotcom/salesforcedx-apex/pull/334. Please reopen if the problem still occurs.


MacOS Enso needs top foreground to run a project

Enso doesn't continue opening project flow if I switch with cmd+tab to some other program (e.x. terminal or browser). Not sure for how long it works that way.
Steps to reproduce:

Open Enso
Click local project to run
Immediately hit cmd + tab to some other window (browser or terminal)
Proces pauses at

[info] [2023-07-05T13:01:57.598Z] [org.enso.projectmanager.infrastructure.languageserver.LanguageServerBootLoader] Language server booted [LanguageServerConnectionInfo(<IP_ADDRESS>,54546,62744)].

Switch back after e.x 1 minute to Enso
Process resumes initialisation

https://github.com/enso-org/enso/assets/2855109/6983b901-c50a-4618-9812-ec0c28603e75

@xvcgreg tested on windows, seems fine
Uploading<PHONE_NUMBER>_shareX.mp4…

@somebody1234 it looks like project manager related, but can you check whether maybe we pause communication from react with PM? I am also having an impression that when you do that with cloud project it won't switch to workspace when running in background

@PabloBuchu I believe it is the browser throttling the JS when it is not in the view.
https://github.com/electron/electron/issues/9567 might be relevant.

@mwu-tow that did the trick
in app/lib/client/src/config.ts:155 we have this option backgroundThrottling it set by default to true meaning the Enso works like above. Shouldn't we set it by false or at least for nightly builds? @wdanilo @sylwiabr

@PabloBuchu we should set it to "false" by default in all builds IMO.


date range placeholder

in search facet instructions for user

ask tech services archivists if just by year is enough

Added a slider input control for to and from year values

Added a slider for from and to date inputs, await user testing

Using 1800 as default from date
Future enhancement will use earliest date in search result set


Implement to_hdf5 and from_hdf5 methods on base class

This PR implements to_hdf5 and from_hdf5 which are responsible for managing the context of hdf5 files passed to the archive methods. The to_hdf5 method adds metadata including package version, serializer version, and object so that the hdf5 entirely describes the object.
In addition, the start of a testing suite has been included with an associated action for running the tests.
The requirements file still referenced pmd-beamphysics, so I changed the reference to openpmd-beamphysics for compat with conda-forge installations.

Thanks @jacquelinegarrahan !


Crash on x86_64

I just compiled tracy and ran any test. but it crashes at the beginning.
Kernel: Linux m1cha-NB 3.13.0-35-generic #62-Ubuntu SMP Fri Aug 15 01:58:42 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
OS: Ubuntu 14.04.1 LTS
GCC: Ubuntu 4.8.2-19ubuntu1
crash:
--------------------------------------------------------------------------------
Whoops: No such process
Function: tracy_wait_event, File: tracy-event.c, Line: 501
Arguments: PTRACE_GETSIGINFO, pid (0), ((void *)0), &siginfo
--------------------------------------------------------------------------------
tests/fork(tracy_backtrace+0x1f)[0x40334d]
tests/fork(tracy_wait_event+0x91f)[0x40708a]
tests/fork(tracy_main+0x40)[0x40352b]
tests/fork(main+0x76)[0x401e03]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf5)[0x7f11f31e2ec5]
tests/fork[0x401cc9]
[0x0]
[0x0]
tests/fork(_start+0x0)[0x401ca0]
[0x7fffdb42ccf0]
tracy_main: tracy_wait_Event returned NULL

base, bash, inject-simple, fuzz-write tests where working on my machine:
[mariotti@localhost src]$ uname -a
Linux localhost.localdomain 3.6.11-4.fc16.x86_64 #1 SMP Tue Jan 8 20:57:42 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux
But I removed the -mx32 switch before compiling. btw, why is it there? It looks like a strange switch.
Tests are kinda working, but a run on a different exe gave me this:
ESC[1;33m24992 System call: write (4) Pre: 1ESC[0m
ESC[1;33m24992 System call: write (4) Pre: 0ESC[0m
and the "write" is not catch by tracy.... running a simple base test.
Why that?

base, bash, inject-simple, fuzz-write tests where working on my machine:
[mariotti@localhost src]$ uname -a
Linux localhost.localdomain 3.6.11-4.fc16.x86_64 #1 SMP Tue Jan 8 20:57:42 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux
But I removed the -mx32 switch before compiling. btw, why is it there? It looks like a strange switch.
Tests are kinda working, but a run on a different exe gave me this:
ESC[1;33m24992 System call: write (4) Pre: 1ESC[0m
ESC[1;33m24992 System call: write (4) Pre: 0ESC[0m
and the "write" is not catch by tracy.... running a simple base test.
Why that?

Hi,
The -mx32 (x32 mode) switch is used only to generate the file arch/amd64/syscall_x32.h ; as far as I know/recall. It's added by somewhat recent GCC's, maybe yours is too old?
Also - I think the base test should catch the write? What exe did it not work on?
On this topic: I do have an x86_64 machine again, so I could test this later -- although I don't recall getting problems last time I tried it.

Hi,
The -mx32 (x32 mode) switch is used only to generate the file arch/amd64/syscall_x32.h ; as far as I know/recall. It's added by somewhat recent GCC's, maybe yours is too old?
Also - I think the base test should catch the write? What exe did it not work on?
On this topic: I do have an x86_64 machine again, so I could test this later -- although I don't recall getting problems last time I tried it.

okey ;) my machine has an old OS:
[mariotti@localhost Linux x86+64bit]$ cat /etc/redhat-release
Fedora release 16 (Verne)
and I guess and old gcc:
[mariotti@localhost Linux x86+64bit]$ gcc --version
gcc (GCC) 4.6.3 20120306 (Red Hat 4.6.3-2)
But I think tracy related things are not that fast growing as the web ;)
I would expect basic OS hooks. It compiles and runs, at least few tests.
Indeed, you write too, the uname dependency, should be made a bit better. ;)
I think the main problem is x86_64 and not i386 ... as I compile as an amd64.
The exe I was testing is from: http://www.cgm-online.com/rc-heli-simulator/_download.html
Do not panic, I am not a cracker ;)
But I would not mind to prove a different licence protection.
i.e. technically "tracy" gives the possibility to a "low" technically skilled person to crack or better
hijack some licence protections.
At least for what is running in the user space.
Might this be indeed the problem? I was running in the user space....

the -mx32 is given as switch to gcc... i.e. does not generate any file. in principle..
but it is assumed for an x86_64 architecture which given with the amd64 uname ..
a simple google gave me this:
http://stackoverflow.com/questions/10860367/is-the-mx32-gcc-flag-implemented-correctly
http://stackoverflow.com/questions/24041752/compile-c-program-for-x86-with-gcc-doesnt-use-avx-instruction-set
I am not that expert, but if there is a dependency it should be on the running OS and eventually on
the guest application.

The -mx32 flag on GCC tells GCC to work in 'x32' mode, x32 mode is not amd64 (x86_64) and also not i386/i686 (x86); it's a separate thing that was added but it used by practically no one. Our support for it is also untested. That being said, it should not interfere with your usage of tracy, mostly. Unless your compiler does not have the flag yet -- then I suggest removing the x32 support altogether (I can help with this). We're definitely also supporting older Linux systems.
Perhaps you can run make clean, and then make, and show me the output? (Without modifications to the file)
So what you noticed is that certain system calls are not logged by Tracy. More specifically, using 'base' on the binary you linked. It's possible that their binary is i686 instead of amd64 bit. Tracy should be able to detect this and provide you with the proper 32 bit system call, when you use the right functions. (tracy_set_hook with the proper abi argument)
tracy_set_hook(t, "write", TRACY_ABI_X86, )
Let me know. :)

Alternatively, you can compile tracy in 32 bit mode, if you're only going to work on 32 bit binaries. It may be easier in certain cases.

Okey, for the moment removing the -mx32 switch seems to work. Random hand made test runs seems to compile and run on simple commands. Like ./fuzz-write ls
Please note that everything goes under amd64 even if this is an intel, i.e. the x86_64 flag is used.
which I think it is okey somehow at present as it should not differ too much...if not at all.
Switching to TRACY_ABI_X86 gets traced the calls indeed. But of course the results are a bit unexpected as, if I get it well, we are on a 64 bit machine using a 32 bit calls. I am sorry I got till now
little time so I cannot give you more specific details. Simply random files are created on tracing write ;)
I will try to reproduce it and come back....

btw.. all in the user space.. kind of comfortable indeed ;)

amd64 is the generic name for x86_64. They are the same. So intel 64 bit is "amd64". On a 64 bit machine you can execute 32 bit system calls, it's just a bit more difficult to get it right if your tracer is 64 bit.
More details would be appreciated.

I think this issue is fixed now. see: https://github.com/MerlijnWajer/tracy/commit/f83ed148f3b20fcddee56dd6ef64720790ac73ea

Yes, it is likely the same issue as the one @v1kko and I ran into today. I'll create a CREDITS file and credit you as well, if you want, @mariotti / @fabmariotti :-)

I'm tired -- I should have mentioned @M1cha . :-)


Bug GioWriterImpl Ports

Nested ports are not correctly writtein to graphml

[ ] Create graphml test file to have nested ports
[ ] Use generic AbstractReaderTest to verify

See spec http://graphml.graphdrawing.org/xmlns/1.1/graphml-structure.xsd
<xs:sequence>
<xs:element ref="desc" minOccurs="0"/>
<xs:choice minOccurs="0" maxOccurs="unbounded">
<xs:element ref="data"/>
<xs:element ref="port"/>
</xs:choice>
</xs:sequence>

This test is still required, the impl should work already


Implementing GEP 724: Refresh Route-Gateway Binding

What type of PR is this?
/kind feature
What this PR does / why we need it:
Implements GEP #724
Which issue(s) this PR fixes:
Fixes #724
Does this PR introduce a user-facing change?:
- Routes attach to Gateways with direct object references
- Gateway Route selector has been removed
- Gateway Route group/kind restriction has changed into an optional list

Note: This PR does not include any docs yet. I will follow up with that either in this PR or a follow up.

Thanks for the great feedback on this @stevesloka @hbagdi and @howardjohn! I think I've responded to everything now, PTAL. I'm planning on covering docs and additional examples in a follow up PR so this one doesn't become too massive.

Looks great, just a few nits to tidy up :+1:

Thanks for the great feedback on this PR! I think I've addressed everything now, PTAL.

Only a few small nits from me, LGTM otherwise.

Thanks for the feedback @youngnick! PTAL.

/lgtm


tests: net: dhcpv4_server: Add test suite for DHCPv4 server library

Add test suite covering DHCPv4 server library functionality. The existing DHCPv4 client tests were moved to a subdirectory.

Thanks @pdgendt, I've addressed your comments.


getSignedAndUploadToS3 is not catching exceptions occurred in getSignedURL

hi. we've noticed that if our endpoint that makes presigned url returns error (500 or 502), vue2Dropzone plugin crashes with exception
vue2Dropzone.js?2af3:185 Uncaught TypeError: Cannot convert undefined or null to object
at Function.keys ()
at Dropzone.eval (vue2Dropzone.js?2af3:185)
steps to reproduce:

raise an error in "signingURL" endpoint
try to upload a file to AWS S3

actual behavior: unhandled JS exception
expected behavior: event vdropzone-error OR vdropzone-s3-upload-error  emitted
thanks

Thanks for your feedback. We will look into it.


Add v3 examples

Related to #641

/progress 10
Started to work on it

/progress 90
Fixing some examples

/progress 100
Done


uses (JS) const for stack and slab declarations

While this is rather a minor thing, while reading through the guide I noticed that for JS stack and slab the let keyword is used, where in fact it should be a const.

Thanks!


ValueError: attempt to get argmin of an empty sequence in Diarization!

@kadirnar
I have received the following error when running the diarization pipeline:
ValueError: attempt to get argmin of an empty sequence

I ran the diarization example exactly. The error seems related to the code below in the diarization pipeline.
 # align the diarizer timestamps and the ASR timestamps
for segment in new_segments:
    # get the diarizer end timestamp
    end_time = segment["segment"]["end"]
    # find the ASR end timestamp that is closest to the diarizer's end timestamp and cut the transcript to here
    upto_idx = np.argmin(np.abs(end_timestamps - end_time))

The pipeline is also too slow, taking 10 minutes for a 5-minute audio file. Can you provide a detailed example of setting hqq for diarization?

What code are you using? Which GPU?

What code are you using? Which GPU?

Hi,
from whisperplus.pipelines.whisper_diarize import ASRDiarizationPipeline
from whisperplus import download_youtube_to_mp3, format_speech_to_dialogue

audio_path = download_youtube_to_mp3("https://www.youtube.com/watch?v=mRB14sFHw2E")

device = "cuda"  # cpu or mps
pipeline = ASRDiarizationPipeline.from_pretrained(
    asr_model="openai/whisper-large-v3",
    diarizer_model="pyannote/speaker-diarization-3.1",
    use_auth_token=False,
    chunk_length_s=30,
    device=device,
)

output_text = pipeline(audio_path, num_speakers=2, min_speaker=1, max_speaker=2)
dialogue = format_speech_to_dialogue(output_text)
print(dialogue)

RTX 3090; however, it is not related to the GPU. It seems that the returned result is empty in some places.

@kadirnar, any update on this?

I will test it today.

@kadirnar any news? got the same problem on rtx4090 in runpod

@kadirnar, Could you please kindly update us regarding this issue as it makes it impossible to use this great package?

@KolosDan @behroozazarkhalili

Have you done all the installations correctly? Have you used hf_token?

@kadirnar generally long inputs and long silence parts trigger this error, found example to trigger this error - https://youtu.be/SbZ5ONmdwgM?si=0QcdTMcamqyngqzU
For context: we download the input (an example video URL with -f 140), convert it to MP3 using ffmpeg, and then send it to the worker. Then it does processing for 20 min and fails.

Error content
{"error_type": "<class 'ValueError'>", "error_message": "attempt to get argmin of an empty sequence", "error_traceback": "Traceback (most recent call last):\
  File \"/usr/local/lib/python3.10/dist-packages/runpod/serverless/modules/rp_job.py\", line 134, in run_job\
    handler_return = handler(job)\
  File \"/handler.py\", line 22, in handler\
    diarization = pipeline(audio_path)\
  File \"/usr/local/lib/python3.10/dist-packages/whisperplus/pipelines/whisper_diarize.py\", line 171, in __call__\
    upto_idx = np.argmin(np.abs(end_timestamps - end_time))\
  File \"/usr/local/lib/python3.10/dist-packages/numpy/core/fromnumeric.py\", line 1325, in argmin\
    return _wrapfunc(a, 'argmin', axis=axis, out=out, **kwds)\
  File \"/usr/local/lib/python3.10/dist-packages/numpy/core/fromnumeric.py\", line 59, in _wrapfunc\
    return bound(*args, **kwds)\
ValueError: attempt to get argmin of an empty sequence\
", "hostname": "xxx", "worker_id": "xxx", "runpod_version": "1.6.2"}

Source code of runpod worker
import runpod
from runpod.serverless.utils import rp_download
from whisperplus.pipelines.whisper_diarize import ASRDiarizationPipeline
from whisperplus import download_youtube_to_mp3, format_speech_to_dialogue
import os

def handler(job):
    job_input = job["input"]
    file = rp_download.file(job_input["url"])

    audio_path = file['file_path']

    pipeline = ASRDiarizationPipeline.from_pretrained(
        asr_model="openai/whisper-large-v3",
        diarizer_model="pyannote/speaker-diarization-3.1",
        use_auth_token=os.getenv('HF_TOKEN'),
        chunk_length_s=30,
        device="cuda",
    )

    diarization = pipeline(audio_path)
    return diarization

if __name__ == "__main__":
    runpod.serverless.start({"handler": handler})

@kadirnar
Yes, It is not related to the installation, though. I highlighted the root cause of the error in the code I already submitted.

Thank you for the detailed explanation. I'll try again this evening. I install Runpod gpu for each trial. So it's a little late. Today I will solve this problem. Thank you for your interest.

Hi.
I just wanted to add that I also ended in the same issue.
In my case I have tested with both :
device = "cpu"
and
device = "mps"
The code I used is 100% the one of the example in the repo for Speaker Diarization.
All the installation, requirements, Models, HF token and  pyannote permissions are fine.
I cloned the repo just minutes ago.

What code are you using? Which GPU?

Hi,
from whisperplus.pipelines.whisper_diarize import ASRDiarizationPipeline
from whisperplus import download_youtube_to_mp3, format_speech_to_dialogue

audio_path = download_youtube_to_mp3("https://www.youtube.com/watch?v=6sUwRiIncKU")

device = "cuda"  # cpu or mps
pipeline = ASRDiarizationPipeline.from_pretrained(
    asr_model="openai/whisper-large-v3",
    diarizer_model="pyannote/speaker-diarization-3.1",
    use_auth_token=False,
    chunk_length_s=30,
    device=device,
)

output_text = pipeline(audio_path, num_speakers=2, min_speaker=1, max_speaker=2)
dialogue = format_speech_to_dialogue(output_text)
print(dialogue)

RTX 3090; however, it is not related to the GPU. It seems that the returned result is empty in some places.

I tested it with A40 on the Runpod platform and it works.

@kadirnar generally long inputs and long silence parts trigger this error, found example to trigger this error - https://youtu.be/SbZ5ONmdwgM?si=0QcdTMcamqyngqzU

could you test it with longer example?

For you to replicate
Device I tried cuda, cpu, mps, all ends with the same error..
from whisperplus.pipelines.whisper_diarize import ASRDiarizationPipeline
from whisperplus import download_youtube_to_mp3, format_speech_to_dialogue

audio_path = download_youtube_to_mp3("https://www.youtube.com/watch?v=2P8J9OWK3TI")

device = "cpu"  # cuda / cpu / mps
pipeline = ASRDiarizationPipeline.from_pretrained(
    asr_model="openai/whisper-large-v3",
    diarizer_model="pyannote/speaker-diarization-3.1",
    #use_auth_token=False,
    use_auth_token="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",
    chunk_length_s=30,
    device=device,
)
output_text = pipeline(audio_path, num_speakers=3, min_speaker=1, max_speaker=3)
dialogue = format_speech_to_dialogue(output_text)
print(dialogue)

How did you do the installation? Can you write them all? @IzzyHibbert

How did you do the installation? Can you write them all? @IzzyHibbert

Pls forget it. Was an issue with flash attention not properly installed. Thanks

@kadirnar
FROM runpod/pytorch:2.2.0-py3.10-cuda12.1.1-devel-ubuntu22.04

RUN apt-get update -y && apt-get install -y ffmpeg

COPY requirements.txt requirements.txt
RUN pip install -r requirements.txt
RUN pip install whisperplus git+https://github.com/huggingface/transformers
RUN pip3 install torch torchvision torchaudio

COPY handler.py handler.py
COPY start.sh start.sh
RUN chmod +x start.sh

CMD /start.sh

@Terisback ,
I don't know the runpod code. Can you just run the sample code?

@Terisback @behroozazarkhalili
Can you write your installation steps?

What code are you using? Which GPU?

Hi,
from whisperplus.pipelines.whisper_diarize import ASRDiarizationPipeline
from whisperplus import download_youtube_to_mp3, format_speech_to_dialogue

audio_path = download_youtube_to_mp3("https://www.youtube.com/watch?v=6sUwRiIncKU")

device = "cuda"  # cpu or mps
pipeline = ASRDiarizationPipeline.from_pretrained(
    asr_model="openai/whisper-large-v3",
    diarizer_model="pyannote/speaker-diarization-3.1",
    use_auth_token=False,
    chunk_length_s=30,
    device=device,
)

output_text = pipeline(audio_path, num_speakers=2, min_speaker=1, max_speaker=2)
dialogue = format_speech_to_dialogue(output_text)
print(dialogue)

RTX 3090; however, it is not related to the GPU. It seems that the returned result is empty in some places.

I tested it with A40 on the Runpod platform and it works.

I also followed the installation steps exactly as mentioned in the repository's GitHub. I copied and pasted the sample code exactly as mentioned above, and tried to run it on RTX 3090, but I got the same error. I think the issue is related to the part of the code I quoted above. For now, I'm going to use another package since this one isn't working well.


[build] use Central Feed Services as package repositories

Description
Our pipeline failed due to not following CFS (Central Feed Service) compliance: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2895367&view=logs&j=f9b44729-c575-586c-d77f-59f393b22beb&t=0bdfb5d1-f75b-5aa3-bfc9-71c755c7e013&l=16
More specifically:

Multiple feeds declared.
CFS0001: missing sibling .npmrc file
CFS0013: package source has value that is not an Azure Artifacts feed.

From the following guides, it looks like we should remove all default repositories (e.g. npmjs.org and nuget.org), and adopt Azure Artifact feeds and set default repositories as the upstream:

Mutliple feeds declared: https://eng.ms/docs/cloud-ai-platform/devdiv/one-engineering-system-1es/1es-docs/secure-supply-chain/project-artemis/secure-supply-chain-task/nuget-multifeed
CFS0001: https://eng.ms/docs/cloud-ai-platform/devdiv/one-engineering-system-1es/1es-docs/secure-supply-chain/project-artemis/central-feed-services-cfs#1-nodejs-projects-are-missing-feed-configuration-addnpmconfig-fixnpmrc-issue-nodefault-cfs0001-cfs0002
CFS0013: https://eng.ms/docs/cloud-ai-platform/devdiv/one-engineering-system-1es/1es-docs/secure-supply-chain/project-artemis/central-feed-services-cfs#5-nugetconfig-files-have-feeds-that-are-not-azure-artifacts-fixnugetconfig-issue-hasnonaa-cfs0013

Another solution is to make error suppression work again. For some reason, it doesn't work.


Move batch to pbft event manager

Description
This changeset removes the batch plugin thread, and instead has it rely on the event manager thread introduced in #1586.
Motivation and Context
This is the second in a series of changesets (see PR #1586) designed to eliminate the plugin threads from PBFT.  As this is in a series, it does not explicitly fix any particular issue, but it is work towards eliminating the race conditions (by eliminating go routines).
As with #1586, this changeset contains some hacky things and TODOs, pieces marked for eventual removal, etc.  This is intentional.
If @corecode could please review.
How Has This Been Tested?
This has passed the pbft unit tests locally, and the CI should confirm the changeset doesn't break anything.  Because there is no new function, just re-routing of execution flow, the existing tests should be adequate.
Checklist:

[X] I have added a Signed-off-by
[X] Either no new documentation is required by this change, OR I added new documentation
[X] Either no new tests are required by this change, OR I added new tests
[X] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Jason Yellick<EMAIL_ADDRESS>

Rebased, because of conflict.

LGTM

@jyellick This one has a merge conflict

@tuand27613 @corecode thanks for the review. LGTM


Apply coupon compatibly v3

Hi @icoderman,
first of all, thanks for your job. Really appreciate it!
As from the title i'm trying to apply coupon in an order but seems compatible only with version V3.
In fact if I apply coupon, the order will filled with coupon information but the price still remains the same.
I've read about work-around, trying to change the total price of the order and pushing it, but also this is unreachable for me.
Any suggestion?
Emiliano

Hi @Pelellone thank you! I would be happy to approve your PR which implements V3 support and coupons :)

Hi @icoderman!
I tried to add V3 compatibility for enabling coupons in order.
All works adding just V3("v3") in ApiVersionType.
I've tested order and coupons of course.
Hope that help you!


Add hyperlink support to external links column

This is needed for the initial step for SAMS integration (see #1390).
The URL pattern should come to SHEETS_EXTERNAL_LINK_LABELS, which needs to be extended for this. Will add a separate task for that if not yet there.

Done.


GstGUIImager: get / set properties not thread safe

get_imager_ts() is used in planner and scripting. While get_image() is thread safe, these functions are not thread safe

done


migration plan for IBM Cloud modules to terraform-ibm-modules

Modules Testing files refer these variables - https://github.com/cloud-native-toolkit/action-module-verify/blob/main/env/ibm_ocp48/terraform.tfvars

Re-grouped the services by category:
Account

[X] https://github.com/cloud-native-toolkit/terraform-ibm-resource-group
[X] https://github.com/cloud-native-toolkit/terraform-ibm-access-group
[X] https://github.com/cloud-native-toolkit/terraform-ibm-account-users
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-add-access-group-users
[X] https://github.com/cloud-native-toolkit/terraform-ibm-account-access-group
[X] https://github.com/cloud-native-toolkit/terraform-ibm-iam-service-authorization
[X] https://github.com/cloud-native-toolkit/terraform-ibm-onboard-fs-account
[X] ~https://github.com/cloud-native-toolkit/terraform-ibm-fs-access-groups~

VPC / VSI

[ ] ~https://github.com/cloud-native-toolkit/terraform-ibm-vpc-vpn~
[X] https://github.com/cloud-native-toolkit/terraform-ibm-ocp-vpc
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpc-ssh
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpc
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-vpe-gateway
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpc-subnets
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpc-gateways
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-vpn-server
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-iks-vpc [ Need Cluster ]
[ ] ~https://github.com/cloud-native-toolkit/terraform-ibm-vsi-openpages~
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vsi-proxy
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpc-load-balancer
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpc-vsi
[X] https://github.com/cloud-native-toolkit/terraform-ibm-flow-log
[X] https://github.com/cloud-native-toolkit/terraform-ibm-vpn-gateway
[ ] https://github.com/cloud-native-toolkit/terraform-vsi-bastion

Gateway

[ ] https://github.com/cloud-native-toolkit/terraform-ibm-transit-gateway

Object Storage

[X] https://github.com/cloud-native-toolkit/terraform-ibm-object-storage
[X] https://github.com/cloud-native-toolkit/terraform-ibm-object-storage-bucket

Event Stream

[X] https://github.com/cloud-native-toolkit/terraform-ibm-event-streams
[X] https://github.com/cloud-native-toolkit/terraform-ibm-event-streams-topic

KMS

[ ] https://github.com/cloud-native-toolkit/terraform-ibm-container-registry
[X] https://github.com/cloud-native-toolkit/terraform-ibm-kms
[X] https://github.com/cloud-native-toolkit/terraform-ibm-kms-key

Deprecated Services from Sep 30th

[X] https://github.com/cloud-native-toolkit/terraform-ibm-cert-manager
[X] https://github.com/cloud-native-toolkit/terraform-ibm-cert-manager-cert

Storage

[  ]~https://github.com/cloud-native-toolkit/terraform-ibm-portworx~ (Moved back and Archived )
[X] https://github.com/cloud-native-toolkit/terraform-ibm-db2-warehouse
[X] https://github.com/cloud-native-toolkit/terraform-ibm-etcd
[X] https://github.com/cloud-native-toolkit/terraform-ibm-mongodb

HPCS

[ ] https://github.com/cloud-native-toolkit/terraform-ibm-hpcs
[X] https://github.com/cloud-native-toolkit/terraform-ibm-hpdbaas-mongodb
[X] https://github.com/cloud-native-toolkit/terraform-ibm-hpcs-initialization
[X] https://github.com/cloud-native-toolkit/terraform-ibm-hpdbaas-postgresql

Cluster config/software

[ ] https://github.com/cloud-native-toolkit/terraform-ibm-log-analysis-bind
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-cloud-monitoring-bind
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-cloud-operator

Cloud Services

[X] https://github.com/cloud-native-toolkit/terraform-ibm-watson-studio
[X] https://github.com/cloud-native-toolkit/terraform-ibm-log-analysis
[X] https://github.com/cloud-native-toolkit/terraform-ibm-cloud-monitoring
[X] https://github.com/cloud-native-toolkit/terraform-ibm-key-protect
[X] https://github.com/cloud-native-toolkit/terraform-ibm-appid
[X] https://github.com/cloud-native-toolkit/terraform-ibm-activity-tracker
[X] https://github.com/cloud-native-toolkit/terraform-ibm-redis
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-image-registry

More Work TBD

[ ] https://github.com/cloud-native-toolkit/terraform-ibm-watson-assistant [ Module is just single main.tf]
[ ] https://github.com/cloud-native-toolkit/terraform-ibm-vpc-hpvsi [ Compile Error in initial version ]

As per discussion,

Sean will do first module and create docs for process.  I am going to follow and repeat/rinse for above list.
Create the TASK if the module takes more than hour(s)

Snippet of process

Move the module from our catalog to IBM target repo via Github Settings
Update the catalog.yaml ID of the module to include the target repo
Make sure the module name have prefix "Toolkit" or "TZ" as well as it does not conflict with target repo since module names already exist
Validate the Github action workflow testing is completed after move to target repo.

Box note from Sean to support this migration

Validated as well as expanded additional information during my 1st journey.
https://ibm.ent.box.com/notes/1001181926527?s=5tmk9s3houumkvl1xyz8bvp765adpxm3

2 modules are migrated as well as validated from BOM

Next Step:

Proposed maintenance window 12-4 pm CST to do this migration activity.
Migration module is On Hold till we get consensus across from consumers to perform this migration activity.

@balasgit can we please progress this work and not limited to 4hr use the whole 8hr CST window but over communicate what is moving and when, the quicker we get this done, the easier it is for us to move onto the next task

Absolutely @mjperrins we can always try to improve to get it done !! will do best :-)

Migration error for IBM cloud monitoring module which requires code fix..

Mongo DB expects the resource group to be present in the Target Cloud Account which i dont have access

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-mongodb/runs/8042605124?check_suite_focus=true

Updates

Re-grouped the services by category which will be easier to come up with strategy such as resources required on the Target Cloud Account which will expedite our migration approach since majority of time spent on.
Our module README needs to be updated with the resources required on target cloud account as Pre-req which is challenging to do reverse engineering and identify.

Released the Mongodb module
Working on the weird issue Cluster not found..

Verified the VPC module build test and here are tickets which am planned to work today. I have just captured issues but didn't had a time to analyze yet.
VPC Build Test
[ ] https://github.com/cloud-native-toolkit/software-everywhere/issues/475
[X] https://github.com/cloud-native-toolkit/software-everywhere/issues/474
[ ] https://github.com/cloud-native-toolkit/software-everywhere/issues/473
[ ] https://github.com/cloud-native-toolkit/software-everywhere/issues/472
[X] https://github.com/cloud-native-toolkit/software-everywhere/issues/471
[X] https://github.com/cloud-native-toolkit/software-everywhere/issues/470
[ ] https://github.com/cloud-native-toolkit/software-everywhere/issues/469
[X] https://github.com/cloud-native-toolkit/software-everywhere/issues/465
[ ] https://github.com/cloud-native-toolkit/software-everywhere/issues/464

End-2-End Testing 4 migrated modules (https://ibm-cloudplatform.slack.com/archives/C039DRPL77U/p1662479225843029)

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-transit-gateway/issues/9
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-container-registry/issues/12
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-object-storage-bucket/issues/34

40 modules are migrated
Migrated Repo
https://github.com/orgs/terraform-ibm-modules/repositories?q=terraform-ibm-toolkit
Catalog Build
https://github.com/cloud-native-toolkit/software-everywhere/blob/main/catalog.yaml
Addressed couple of issues but some pending issues requires more debugging which I will do tomorrow
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-portworx/issues/20
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-kms-key/issues/20
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-cert-manager/issues/9
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-db2-warehouse/issues/5 [ error happens during destroy after merging with main. Test re-running now as I share update in slack )
Ask:
Pls open issue at the module if you face errors as I will check first thing tomorrow morning.

Build Errors

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-hpcs/issues/24
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-hpcs-initialization/issues/9
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-hpdbaas-mongodb/issues/11
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-hpcs-initialization/issues/10
https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-image-registry/issues/36

Remaining Modules

Build Error
-[ ]  https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-iks-vpc/issues/6

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-add-access-group-users/issues/6

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-vpe-gateway/issues/25

https://github.com/cloud-native-toolkit/software-everywhere/issues/472

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-container-registry/issues/12

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-hpcs/issues/24

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-image-registry/issues/36

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-iks-vpc/issues/6

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-log-analysis-bind/issues/40

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-cloud-monitoring-bind/issues/25

https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-cloud-operator/issues/5

@seansund
We need to have cluster for these modules test @ Golden Eye.
Test Cases used are "OCP4_latest" , "OCP_VPC" , ocp44_vpc in the Workflow file as result it returns Cluster not found for these modules.
In Cluster installation
-[ ] https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-log-analysis-bind/issues/40
-[ ] https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-cloud-monitoring-bind/issues/25
-[ ] https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-cloud-operator/issues/5
-[ ] https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-image-registry/issues/37
-[ ] https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-iks-vpc/issues/6

Will discuss cluster pre-req on Thrusday office hours since Tues office hours didn't started.

For the in-cluster installation modules, we need to update the tests to provision a new cluster. I've already made the changes for the log-analysis-bind and cloud-monitoring-bind modules. You can use this commit as a template for the other ones - https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-cloud-monitoring-bind/pull/24/commits/465f6b0e155e738d70740c9177ad8fc4d1889985


poc error

poc
To test this go through the Registration workflow and look at the browser console after submitting page 1 Operation Information and the final Registration Submission pages. We are throwing an error on the submission page in the API that can be removed if you want to test successful submission.
Again, I might be missing something here so happy to jump on a call and discuss or else hear any other ideas/criticisms of this! Thanks.

Heya, thanks for this! I was trying to avoid any error handling anywhere except MultiStepBase, but what you've got here is just a couple lines and I think that works better than useEffect. I'll pull some of this code over into the PR later today


added basic functionality of AuthenticationService

Codecov Report

:exclamation: No coverage uploaded for pull request base (hba@2f905b7). Click here to learn what that means.
The diff coverage is 0%.

@@          Coverage Diff           @@
##             hba    #5452   +/-   ##
======================================
  Coverage       ?   82.17%
  Complexity     ?    11228
======================================
  Files          ?     1357
  Lines          ?    44289
  Branches       ?     5574
======================================
  Hits           ?    36394
  Misses         ?     5278
  Partials       ?     2617

Impacted Files
Coverage Δ
Complexity Δ

...io/crate/operation/auth/AuthenticationService.java
0% <0%> (ø)
0 <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 2f905b7...a467cb6. Read the comment docs.


Add the link of supposed example in topology-spread-constraints.md

Ref: #39218

Hi @kbhawkey @natalisucks
Could you please have a look

@sftim I have updated the PR, please take a look now

@sftim I have updated the PR, please take a look now

Actually, I'm happy for any contributor to review this PR.

Thanks @divya-mohan0209


Warn if rancher compose has been upgraded but not rancher server

There have been issues where people have upgraded rancher compose, but haven't upgraded rancher server.
Since rancher-server and rancher-compose are pinned to each other in terms of compatibility, we should warn users if their version of rancher-compose doesn't work with their older rancher-server version.
Note: If you upgrade rancher-server, but forget to upgrade rancher-compose, we already have a warning that a newer version of rancher-compose is available.

Isn't this covered by #1725 ?

In this situation we can probably error out.  For example, with Docker if I use a 1.9 client against a 1.8 server it just displays an error.

@kaos The note that I covered is what was fixed for #1725.


Why are there two assets at the same time? Why are the ~assets and ~/assets

Why are there two assets at the same time?
Why are the ~assets and ~/assets

@pi0 @alexchopin

Hi @yuchonghua,
~assets is a Vue.js (vue-router) alias and can be used inside <template> and <style> tags.
~/assets is a Nuxt.js alias and can be used inside <script> tag.

~assets and ~/assets can be unified

If not uniform, feel the new version of nuxt use is very strange, even the name of each place is not the same
@alexchopin

@alexchopin
Yes ah, I have encountered this problem, why engage in two standards?
Is not it awkward to develop?

@alexchopin Would you please fix this situation?

@Atinux Do not you think it's not a troublesome thing? Even with a path to use a different logo to visit? The

Now vue file introduction style or problems
\ping @alexchopin @pi0 @Atinux

\ping @alexchopin @pi0 @Atinux

Just as a test, does change it to @/assets/web/... fixes problem?

Use ~assets/web/images/... can be normal

I am also experiencing this problem

I am also experiencing this problem

Now vue-loader loading style inside the css have this problem, what is the reason for this problem?

git clone https://github.com/ddvjs/nuxt-assets-bug.git
cd nuxt-assets-bug
npm i
npm run dev

I also encountered this problem. But have not found a solution

I have the same problem，Can some one tell me how to do this?

After some investigations, it seems this problem is not causing by vue-loader/transofrm-require that was fixed for templates usage. Currently before finding fix, as @yuchonghua suggested the only workaround is using ~assets/web/images/...

2ff8b5a should finally allow ~/assets and ~/static usage everywhere. Keeping this thread open until next release if anyone has better workaround :)

thank @pi0 very much

Will be available in rc5.

Very much hope that rc5 can be released faster.
/ping @pi0

Very much hope that rc5 can be released faster.
/ping @Atinux @pi0


Upgrade Existing AWS 3.6 cluster to 3.7

Is is possible and if so are there any docs available to upgrade an existing 3.6 install to 3.7?
If this is possible does it also create the auto scaling groups that are now in openshift-ansible?
Thanks

Is is possible and if so are there any docs available to upgrade an existing 3.6 install to 3.7?

https://docs.openshift.com/container-platform/3.6/install_config/upgrading/index.html

If this is possible does it also create the auto scaling groups that are now in openshift-ansible?

Not sure about that, maybe @sdodson or @kwoodson would know?

Yes, you can follow the docs in https://github.com/openshift/openshift-ansible/tree/master/playbooks/aws which describe how to deploy a scale group. We're working on automated workflows to support upgrades now.
This is not yet supported for OCP customers though.

I have provisioned via the reference architecture scripts, will the upgrade mentioned above work with the reference architecture scripts to upgrade or should we be using openshift-ansible directly?

@daveconde so the instances themselves will not be re-provisioned with ASGs. The deployment of the aws code from this repo creates a static inventory which you can then use https://docs.openshift.com/container-platform/3.7/upgrading/automated_upgrades.html#preparing-for-an-automated-upgrade and the manual upgrade process

Will it create a ASG for new nodes to be provisioned in to as part of the upgrade?

@daveconde,
The upgrade playbook for 3.9 will create scale groups based upon an AMI that you would be required to build.

Thanks


Add run command to python-sdk so users do not need SQL CLI on their a…

…iflow instances to run DAGs
Description
What is the current behavior?
Currently run_dag only exists in the SQL CLI library. This means that users do not have the sql-cli installed can not take advantage of the run_dag function.

What is the new behavior?
This function makes the run_dag function a part of the astro library so users can directly call it via the astro SDK in their DAGs. It also means that the astro CLI can use the run_dag function without installing the SQL CLI.
Does this introduce a breaking change?
Checklist

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

Yes, I agree with @tatiana.
It would be so much nicer if we could have that in core airflow or as separate python package. Then we would not have to maintain two implementations - run_dag is not specific to astro-sdk-python DAGs nor astro-sql-cli workflows. It works for all kind of DAGs, right? If we put it into the sdk, I am afraid, we are being slowed down by astro-sdk-python releases. If we have it separately, we can more quickly iterate and release changes.

We decided to not go ahead with this proposal, and we moved the feature to a separate repo:
https://github.com/astronomer/astro-run-dag
The discussion and reasoning can be found at:
https://docs.google.com/document/d/19rSL2nQljdg0SI5NREKpVls-uSXxYVFk4Nq_6VLQO8A/edit#


chore(datasets): Remove deprecation warning from Table Dataset

Description
Remove deprecation warning as suggested here -  Need to remove code that's marked as "to be removed": ibis dataset: https://github.com/kedro-org/kedro-plugins/blob/main/kedro-datasets/kedro_datasets/ibis/table_dataset.py#L123-L128
Development notes

Removed the deprecation warning from Table Dataset
Updated release note

Checklist

[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[x] Added a description of this change in the relevant RELEASE.md file
[ ] Added tests to cover my changes
[ ] Received approvals from at least half of the TSC (required for adding a new, non-experimental dataset)

@deepyaman should filepath and file_format also be removed as arguments? I see in the history that you did remove them from the docstrings already.

Yes, please!

Yes, please!

Hi @deepyaman , I may not be fully aware of the context but what happens to the load function if we remove the parameters filepath and file_format ?  I see a docstring mentioning this
So to summarize, this will affect 2 functions load and _describe. For _describe I can remove the keys from the returned dict. I am not clear on load
Thank you

Yes, please!

Hi @deepyaman , I may not be fully aware of the context but what happens to the load function if we remove the parameters filepath and file_format ? I see a docstring mentioning this
So to summarize, this will affect 2 functions load and _describe. For _describe I can remove the keys from the returned dict. I am not clear on load
Thank you

All this logic can be removed; ibis.FileDataset should fully cover these functionalities.
        if self._filepath is not None:
            if self._file_format is None:
                raise NotImplementedError
            reader = getattr(self.connection, f"read_{self._file_format}")
            return reader(self._filepath, self._table_name, **self._load_args)

        else:


zipfile.BadZipfile: File is not a zip file

Hi everyone,
We periodically see the following error in an environment in which pex files are periodically (somewhat frequently) rotated on disk as the version increases and we do a new release. My suspicion is that we see this error when we rotate the pex file while it is executing or unpacking.
Have others seen this and give me a tip on i) root cause and ii) how to safely prevent?
Thanks! :+1:
Traceback (most recent call last):
  File "/usr/lib64/python2.6/runpy.py", line 122, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib64/python2.6/runpy.py", line 34, in _run_code
    exec code in run_globals
  File "__main__.py", line 25, in <module>
  File ".bootstrap/_pex/pex_bootstrapper.py", line 99, in bootstrap_pex
  File ".bootstrap/_pex/pex.py", line 55, in __init__
  File ".bootstrap/_pex/pex_info.py", line 76, in from_pex
  File "/usr/lib64/python2.6/contextlib.py", line 16, in __enter__
    return self.gen.next()
  File ".bootstrap/_pex/common.py", line 76, in open_zip
  File "/usr/lib64/python2.6/zipfile.py", line 696, in __init__
    self._GetContents()
  File "/usr/lib64/python2.6/zipfile.py", line 716, in _GetContents
    self._RealGetContents()
  File "/usr/lib64/python2.6/zipfile.py", line 728, in _RealGetContents
    raise BadZipfile, "File is not a zip file"
zipfile.BadZipfile: File is not a zip file

@sholsapp this would make sense if the pex file is being truncated while being executed (e.g. a slow network copy process writes the first 1/5th of the file at the moment that pex is trying to unzip it).
how are you "rotating" the pex file exactly (i.e. what command/setup)?
have you considered the ol' create-then-atomically-rename ("echo blah > $FILE.tmp && mv $FILE.tmp $FILE" trick?

@sholsapp, Kris asked some good questions that were never answered. These would need answers to provide further help. I'm going to close this as an answered question, but please re-open if you are still struggling with this and have more details you can provide.


Allow querying of individual devfile JSON.

Which area this feature is related to?

/area api
Which functionality do you think we should add?
Allow querying of individual devfile JSON.
Why is this needed? Is your feature request related to a problem?
This will allow a more efficient build process of the registry viewer. In addition, if we support /index/all we should support individual querying.
Detailed description:
I want to be able to access the same information from https://registry.devfile.io/index/all but for individual devfiles.
Describe the solution you'd like
If I want to access the java-maven JSON data I want to access it at /index/java-maven.
Describe alternatives you've considered
N/A
Additional context
N/A

closing as rotten (created > 2 years). Feel free to re-open if it's not a requirement.


[iOS] NSManagedObjectContext save observer

Context observer for automatic message creation for entities saved with an NSManagedObjectContext. Also prevents the MSCoreDataStore from trying to re-insert/update/delete the entity and just creates the server message as discussed in issue #568

Hi @damienpontifex, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, AZPRBOT;

I have some general comments, but I think the core is good.  Since this is a prototype branch, I'm happy taking this as is.  Let me know if you want to address anything right now.

Made the small changes for the boolean check and some comments above on the error handling. Happy to merge and continue to test and work on otherwise

OK, I am going to merge this in, as I would like to try out the changes/etc and I know others have expressed interest in this topic.

I have added some unit tests for this class. Should I just create another pull request to include them?


There seems to be a duplicity with eigen code

https://github.com/anboto/Anboto/tree/master/plugin/eigen
There is a subfolder src, then subfolder Eigen with another src again and seemingly same files. Maybe some of them needs to be deleted?

Thank you Mirek, you are right. Hopefully removed.

As it has been a long time since this problem has been solved and there are no further comments, this issue will be closed.


CSV Output mode

the ability to generate a CSV report would be immensely useful to compare findings across multiple runs and multiple profiles.

That's a great enhancement, reviewing your PR right now. Thanks!

Enhancement added in PR #53
Thanks @MrSecure


Proposal to manage Grafana chart as dependency

As of now, Grafana is integrated as a deployment in the DevLake helm chart. I'd like to propose removing it and leveraging the official chart as a dependency. This would help keep up to date with Grafana's improvements and corrections, and it would standardise its usage.

Look good to me, @matrixji @IronCore864 what do you think?

Good idea. Does Grafana's official chart support provisioning dashboards?

Good idea. Does Grafana's official chart support provisioning dashboards?

Yes it does.

I will do some investigation on this one. Please assign this issue to me.

I did some research, and I want to support this proposal. Below are my detailed proposal; requesting for comments:

1 Background
The current situation:

DevLake supports both Docker compose, and Helm install
self-created Grafana Docker image with customized dashboard/datasource definition
Grafana Docker image releases together with DevLake, the Grafana version isn't the latest and can't be specified
DevLake Helm chart defines a customized Grafana Deployment/Service/Ingress/ConfigMap

2 Proposal
Disconnect DevLake with Grafana. By "disconnect", I mean:

remove the grafana folder from apache/incubator-devlake helm chart
move the customized dashboard/datasource definition to the apache/incubator-devlake-helm-chart repo
remove Grafana related K8s manifests from apache/incubator-devlake-helm-chart
use the official Grafana helm chart as a dependency for the DevLake helm chart

3 Pros and Cons

reduce DevLake Helm chart complexity
no reinventing wheels
customize Grafana version

Sounds good to me overall.
However, we should also address the following problems:

The dashboards are highly dependent on the Domain Layer definition from the devlake repo, How do we keep track of them? e.g. dashboards developed for v0.18 might not work on v0.17. Keep in mind we have to make old dashboards should be upgraded or downgraded automatically while users upgrade/downgrad the helm chart. @IronCore864
Currently, we have some dashboards specifically designed for data verification (for both Testing and Debugging), should we deploy the helm in order to do so? if so, we should set up an automated CD system for testing. @IronCore864  @abeizn
How do we and users develop new dashboards or upgrade existing ones, we need documents to guide users on how to do so and submit their contribution. @IronCore864  @Startrekzky

How do we release


Automatic documentation building and publishing

Build documentation after successful testing by travis-ci and publish it automatically to [github-pages] (pygeo.github.io/pycmbs).

documentation now pushed to ReadTheDocs


bugfix: Match ports in docker-compose.yml with README.md, add docker …

…instructions

Thanks for contributing! This PR is several different changes (changing ports, changing README organization, changing Cypress versions) mixed together. Could you submit PRs that are more narrowly scoped?


Make it works with Kotlin

support +1

+1

I love Kotlin
But I think we can use extension function.
Is it need?

i think it is need
postfix is awesome

+1

there are tons of kotlin extensions and you can add your own. You don't need a plugin for that.


Anyone can help?

These nodes do not use UNETLoader, this is probably a mixup and you're trying to use:
https://github.com/huchenlei/ComfyUI-IC-Light-Native with wrong models.


bootique-cayenne-test presence breaks BQModuleProviderChecker.testMetadata

An initial proposal for the issue bootique/bootique-cayenne#38.

It seems correct, but still doesn't feel quite right. I need to think about the problem a bit more.

Closing. We are working on an alt solution


Update preview image in roasalin demo

Before submitting
Please complete the following checklist when submitting a PR:

[ ] Ensure that your tutorial executes correctly, and conforms to the
guidelines specified in the README.

[ ] Remember to do a grammar check of the content you include.

[ ] All tutorials conform to
PEP8 standards.
To auto format files, simply pip install black, and then
run black -l 100 path/to/file.py.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Title:
Summary:
Relevant references:
Possible Drawbacks:
Related GitHub Issues:

Hey @ottoak, I am fixing all the validation related errors I am hitting as part of #887 . We should not need this PR for separate fix

oops, thanks for the FYI @rashidnhm - I'll close this!


Refactor/storybook use infinite scroll

Closes #2506
Description
Replaced size with fontSize in Storybook use-infinite-scroll hook and updated the docs
Is this a breaking change (Yes/No):
No

Hi @ChinoUkaegbu , thanks for your PR!!
I'm sorry but you don't have to change size prop used in the  Heading component.
The size prop used in the Icon component is deprecated.
https://github.com/yamada-ui/yamada-ui/blob/main/packages/components/icon/src/icon.tsx#L21

I see. So the Heading 'size' should remain the same but the Loading 'size' should be changed to 'fontSize'?

I've made the changes from the review!

Thanks!


Use card mode for cncf-landscape shortcode

The repeated icons are causing confusing.
closes: #25074

Really pleased how easy the shortcode makes this fix!
/lgtm

Sure. Thanks @tengqm .
/approve


Cloud-init.md documentation  does not reflect alle the current changes

hi together,
I am quite new to terraform proxmox driver. I could not  get it to run properly by the documentation and all the cloud-init issues around 3.0.1rc1 or lasrt 2.9 confused me.
could anybody with knowledge please spend 5 minutes and update the
https://github.com/Telmate/terraform-provider-proxmox/blob/master/docs/guides/cloud_init.md
Thank you so much !
Cheers,
Timo

First of all, I appreciate the work you've done maintaining this tf provider.
I'm also new to proxmox and I had to follow  a couple of outdated guides and I must admit that figuring out how the new blocks work is a really frustrating experience.
I'd really appreciate if someone could offer some guidance and point me in the right direction.
My main struggle was working with the new disks and network blocks.
If someone could update the example posted in this repository, I'm sure it will inevitably draw more people and get them up to speed in a matter of minutes.
Thanks again!

Here's an example I've used for disks:
https://github.com/Telmate/terraform-provider-proxmox/blob/master/examples/pxe_example.tf#L52

Yes, that's a great starting point. I've also managed to import the state of previously created resources.
So given the pxe example and my own TF state I was able to recreate my cluster with Terraform.
Perhaps I can contribute and create a PR with another more beginner-friendly example once I clean it my code and add more comments.
Here's the TF command I used to export my TF state into an already existing module where <vm_name> is the name of the actual TF resource defined in that module.
terraform import 'proxmox_vm_qemu.<vm_name>' <host_name>/proxmox_vm_qemu/<vm_id>


VS 2017

Any chance we'll have a version for VS 2017?

I dont't know. Honestly, the only thing i've recently did is to move it to GitHub. SCSF depends on GAX/GAT, which became OpenGAX later. And there's currently no release for 2017. https://github.com/MicrosoftArchive/OpenGAX/issues/3

Hi, does something changed about GAX for VS2017 since Sep 30, I'm trying to find but without success


Malformed flow version directory name

I'm trying to install run flow-typed install but I'm getting the following error:
UNCAUGHT ERROR: Error: npm/sinon_v2.x.x/flow_v0.28.x: Malformed flow-version directory name! Expected the name to be formatted as 'flow_all' or 'flow_-?v' or 'flow_v-?' or 'flow_v-v'(for a range).

Nevermind. My flow-typed was outdated. I was in version 2.0.0. After updating to 2.1.5 it worked perfectly.


Remove m11.sesinnovation.com from cp-collected-iocs.intel

This entry is malformed and should be removed until the correct entry is
added.

This addresses issue: https://github.com/CriticalPathSecurity/Zeek-Intelligence-Feeds/issues/14

The offending line has been removed. The updated feeds are published.

Thanks @Patrick-Kelley !


running "cargo fmt" on clean master creates modifications

I ran cargo fmt (following CONTRIBUTING.md) and I get the following diff:
$ git diff
diff --git a/src/cargo/core/resolver/context.rs b/src/cargo/core/resolver/context.rs
index 2c2281d6..c423cf3e 100644
--- a/src/cargo/core/resolver/context.rs
+++ b/src/cargo/core/resolver/context.rs
@@ -244,9 +244,11 @@ impl Context {
         if !reqs.used.is_empty() {
             let pkgid = s.package_id();

-            let set = Rc::make_mut(self.resolve_features
-                .entry(pkgid.clone())
-                .or_insert_with(|| Rc::new(HashSet::new())));
+            let set = Rc::make_mut(
+                self.resolve_features
+                    .entry(pkgid.clone())
+                    .or_insert_with(|| Rc::new(HashSet::new())),
+            );

             for feature in reqs.used {
                 set.insert(InternedString::new(feature));
diff --git a/src/cargo/ops/resolve.rs b/src/cargo/ops/resolve.rs
index 68965388..6d4e6d2d 100644
--- a/src/cargo/ops/resolve.rs
+++ b/src/cargo/ops/resolve.rs
@@ -479,11 +479,16 @@ fn register_previous_locks<'a>(

             // Ok if nothing matches, then we poison the source of this
             // dependencies and the previous lock file.
-            debug!("poisoning {} because {} looks like it changed {}",
-                   dep.source_id(),
-                   member.package_id(),
-                   dep.name());
-            for id in resolve.iter().filter(|id| id.source_id() == dep.source_id()) {
+            debug!(
+                "poisoning {} because {} looks like it changed {}",
+                dep.source_id(),
+                member.package_id(),
+                dep.name()
+            );
+            for id in resolve
+                .iter()
+                .filter(|id| id.source_id() == dep.source_id())
+            {
                 add_deps(resolve, id, &mut avoid_locking);
             }
         }

is this surprising?
for reference the HEAD of master I'm using the current one: d8b20310217e5e2bc515c13111ab9a7789709e42.

@dwijnand the contributing md is wrong, unfortunately. We no longer run rustfmt on CI, because formatting changes with different major versions of Rust, and it causes problems around new release, and for backports as well. So instead we'll run rustfmt on the codebase from time to time.
In the meantime, the general approach is to run current stable rustfmt only on the changed files.


CITZ ADM Procument Options Decision Note

Describe the issue
Develop a note for the ADMs around the procurement options for the MCS pilot.
Which Sprint Priority is this issue related to?
2
Additional context
Add any other context, attachments or screenshots
Definition of done

Completed decision note reviewed by Gen and Paul

Latest draft of note has been sent to procurement for review.


Move corePKCS11 to latest tag

Move corePKCS11 to latest tag
Description
Move corePKCS11 to latest tag
Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aggarw13 thanks for catching the manifest update. Updating History.txt is out of scope in this PR IMO.


[Frontend] Add TRITON_PRINT_AUTOTUNING_ALL flag

When the TRITON_PRINT_AUTOTUNING_ALL envvar is set to 1, we print all of the autotuning configurations for a given kernel in a sorted order based on the kernel timings including the timing data.
For example:
Printing ALL Multiple Triton autotuning Configs with timings in sorted order for kernel JITFunction(_ragged_hstu_attn_fwd):
Triton autotune config: [BLOCK_M: 64, BLOCK_N: 64, num_warps: 4, num_stages: 2]; Triton autotune timing: [1.4173120260238647, 1.4117120504379272, 1.4213504791259766]
Triton autotune config: [BLOCK_M: 64, BLOCK_N: 32, num_warps: 4, num_stages: 4]; Triton autotune timing: [1.4553600549697876, 1.4507968425750732, 1.4593983888626099]

The purpose of this change is to bring more clarity to how different configs stack up; it is especially useful as we are lately beginning to autotune on on operations (ie tma and others) that may not bring benefit to all configurations equally.

[x] I am not making a trivial change, such as fixing a typo in a comment.

[x] I have written a PR description following these
rules.

[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[ ] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[ ] This PR does not need a test because FILL THIS IN.

Still deciding how to test this one, it is a print debugging feature.

Select one of the following.

[x] I have not added any lit tests.
[ ] The lit tests I have added follow these [best practices]

Actually it could be obtained through the profiler. Let me know if it's sufficient or you still need a new feature

Actually it could be obtained through the profiler. Let me know if it's sufficient or you still need a new feature

That sounds really useful, and I do think profiling data would be great to go with this (especially if it shed light on TMA or caching behavior)
But I do think having this sort of print out as a quick debugging feature has been helpful as a matter of convenience.

This sounds useful to me as it's pretty straightforward to use.
The profiler will give more comprehensive metrics for sure and it can be a good alternative too.


Logos from simpleicons.org having space in its name not loading

Are you experiencing an issue with...

[X] shields.io
[ ] My own instance
[ ] gh-badges NPM package

:beetle: Description
Logos from simpleicons.org not rendering when the logo has space in it's name. For example : "Code Climate"

:link: Link to the badge
https://img.shields.io/codeclimate/maintainability/einsteinpy/einsteinpy.svg?logo=Code Climate&style=flat-square

Just leaving a note in case anyone comes across this issue.
The Code Climate logo was added to simple-icons on v1.9.22 and our server is currently running v1.9.21,
The icon should load correctly once we update simple-icons.

Also, keep in mind that the canonical / recommended way to refer to simple-icons logos is now with a hyphen separator, e.g. ?logo=visual-studio-code.

https://img.shields.io/badge/foo-bar-blue.svg?logo=visual-studio-code


Refactor configuration management

Currently, the configuration is passed in via a big yaml file that gets parsed into a dictionary. This is a bit inconvienient as it neither documents what configuration values are available and it pollutes the main function. Maybe it would be nice to refactor it out into a separate class?

Currently working on this and making progress.
TODO

[X] Write a config parser as seperate class
[ ] Change main to use the new parser
[ ] Adapt methods to the new config class where needed


Some unicode characters aren't included in all fonts

Example : accented letters in M+ fonts

Package version 3.0.3 under Linux, if it's relevant

Sure, not all fonts cover all of unicode. Or rather, no font has full coverage. Nothin much I can do about that, that's why there is a "secondary font" setting that'll provide the glyphs that you're missing.

Sure, but doesn't M+ (for example) cover all of it ? I don't seem to have any problem with the same font in urxvt :

I'm using their woff (webfont) release, which limits Latin coverage to Basic Latin. All the glyphs (Kanji etc) make M+ too massive to include completely. Some other fonts are also huge (i.e. tens of megabytes) and not included with their complete set. If you like using M+ I recommend just installing it on your machine.


Add tin.py output to MultiQC report

When adding the rseqc/tin the TIN files are being found by MultiQC (v1.11) but I can't seem to be able to get rid of the markdup.sorted.bam extension as shown in the image below:

This doesn't work either:
https://github.com/nf-core/rnaseq/pull/745/commits/169d5470b7de4da2757471dccd773a9f9dc98465
So for now I am using the wrong search path intentionally as added in https://github.com/nf-core/rnaseq/pull/745/commits/dabdfee63ed5dceb638c96180b96b3f37c59e7d6. This means the files will all be staged in the MultiQC work directory but they won't be rendered in the report.
This behaviour needs to be tested with the dev version of MultiQC and added back in properly in the next release of the pipeline.
For now, users should be able to use their own --multiqc_config to add the TIN metrics if they so wish.

This will be fixed in MultiQC v1.12 https://github.com/ewels/MultiQC/issues/1484

MultiQC v1.12 was released recently but I found an issue with the rendering of the custom content of dupRadar that needs to be addressed before we bump versions and fix this issue too https://github.com/ewels/MultiQC/issues/1629

Fixed in https://github.com/nf-core/rnaseq/pull/874/commits/77d49ddeea097599601fb5253aface07d63839f7


Personal webparts: Token renewal operation failed due to timeout

Category

[ ] Question
[x] Bug
[ ] Enhancement

Expected or Desired Behavior
When manually adding the sharepoint-starter-kit.sppkg, all webparts, including personal webparts, should work.
Observed Behavior
Personal webparts, e.g. e-mail, give "Token renewal operation failed due to timeout"
Steps to Reproduce
Tenant at Targeted Release for Everyone
Manually upload sharepoint-starter-kit.sppkg to appcatalog
Create a teamsite or communication site, add a personal webpart
Personal webpart should give Token renewal operation failed due to timeout error
Btw, using Apply-PnPTenantTemplate to provision starterkit.pnp gives me "Another object with the same value for property identifierUris already exists" (see issue #278). So that's why I want to do it manually..

Managed to deploy the starterkit via provisioning and with working API permissions. Was related to this problem: #278


OpenALTO Architecture

References:

https://people.csail.mit.edu/gsmaragd/publications/CoNEXT2019/CoNEXT2019.pdf

Google Orion reference: https://www.usenix.org/system/files/nsdi21-ferguson.pdf

The architecture diagram


White navbar on ios and empty page on android when using the backbutton

Ionic version:  (check one with "x")
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Current behavior:
From root: I pushed a page, then another. From here, I used the back button in the nav bar. On android, the whole page is white/empty. On iOs, the navbar is white. However, everything is still here, just hidden (clicking/tapping where buttons should be still work).
I don't think I used tricky css or html but I'm pushing pages through a event. (Root page subscribe to an event and child emit the event. Then root page push the correct data). The navbar back button isn't caught.
Root (List Page) is not lazy loaded ? (not module) but every child pushed is.
Looking with chrome inspector, I noticed this: (modal-infographie is NOT a modal and is the page I am supposed to see) (this css does not come from my code)

element.style {
  z-index: 99;
  opacity: 0;
  transform: translateY(40px);
}
.ion-page.show-page {
  opacity: 1;
}

Putting !important to opacity: 1 or removing opacity: 0 showed the page again (removing transform put the navbar at the correct place).
If I use navctrl.pop() instead of the back button, it work correctly.
It looks like this issue but my ionic is up to date and the project was created a few months ago (using aws-starter)
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.1.0
    Cordova Platforms  : android 6.3.0 ios 4.5.4
    Ionic Framework    : ionic-angular 3.9.2

System:

    Node : v8.9.1
    npm  : 5.5.1
    OS   : Windows 7

Environment Variables:

    ANDROID_HOME : not set

Misc:

    backend : pro

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having?
Said repo does not have to be your actual project but just a minimal app that demonstrates the issue you are seeing. I have several sample / test applications that do similar navigation and do not see this issue in any of them, so I will need to see what your code is doing.
Thanks for using Ionic!

Wish I could but I'm running tight on schedule on this current project.
I worked around this problem by doing it an other way but I still have the code somewhere. I was a pretty strange way to navigate between page though.

OK, that's cool. If you are able to provide one in the future I can re-open the issue at that point and have a look. In the meantime, I will close this.
We are currently working full-speed-ahead on v4, and once that is out this may or may not still be an issue. If this is still an issue then, certainly let us know.


Could not connect to host

when i try any function i got Could not connect to host
here is my code for example
public function validate_address(){
    $address=[

        'line1'=>'line1',// recommended no required,

        'country_code'=>'AE',

        'city'=>'Dubai'

    ];

    $data = Aramex::validateAddress($address);

    dd($data);

}

NOTE1 : when i try to upload on server it is working fine but in my local device it give me Could not connect to host
NOTE2 : i'm using test env

are you fixed?


jql 6.0.3 test failure

👋 trying to upgrade jql to the latest release, 6.0.3, but failed with the following failure
jql --raw-string --inline  '"cats".[2:1].[0]."third"' /private/tmp/jql-test-20230422-25743-2ghswh/example.json
Unable to parse input .[2:1].[0]."third" after Key Selector "cats"

While just "cats" as QUERY string, it works out fine
jql --raw-string --inline  '"cats"' /private/tmp/jql-test-20230422-25743-2ghswh/example.json
[{"first":"Pixie"},{"second":"Kitkat"},{"third":"Misty"}]

relates to https://github.com/Homebrew/homebrew-core/pull/129018

Hi @chenrui333 ,
That's because I've released a major version with a lot of breaking changes which are documented in the readme.
This is how you should do the query now: '"cats"[2:1][0]"third"'. Basically the tokens have been simplified in the grammar, the dots are not necessary.
Please let me know if that doesn't work as expected.

By the way, I've just realized that you are a Homebrew maintainer 😄 ! Thanks for your work!
Second thing, the release is incorrect. That's a minor thing but it should be jql-v6.0.3-aarch64-apple-darwin.zip not jql-jql-v6.0.3-aarch64-apple-darwin.zip.

Actually found a bug, will fix :).

@chenrui333 a new patch version has been released with the fix and the correct name :). Please let me know if you have any issue.
N.B.: I will update the readme with some more complete / cookbook examples since the queries are different compared to v5.

Worked for us, thanks @yamafaktory!


feat(controller): Remove the excessive logging of node data

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[x] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I've signed the CLA.
[x] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[ ] My builds are green. Try syncing with master if they are not.
[x] My organization is added to USERS.md.

@jessesuen @sarabala1979 could you think of any reason these log statements log the whole node? Did it used to have String(). None of these statements are structured or really greppable, so I don't think changing them will be a problem.


Updated convert_idrsa_pub function

@daedalus

Since p and q are None in pub_key, priv_key = generate_keys_from_p_q_e_n(None, None, e, n) it will generate pub_key with an object and privkey = None, and since you are checking for the condition if pub_key and priv_key: it will always fail...

Since both p and q = None in pub_key, priv_key = generate_keys_from_p_q_e_n(None, None, e, n) it will generate pub_key !=None and privkey = None, and since you are checking for the condition if pub_key and priv_key: it will always fail...

OK.
Thank you i got it.
Thanks :)

Since both p and q = None in pub_key, priv_key = generate_keys_from_p_q_e_n(None, None, e, n) it will generate pub_key !=None and privkey = None, and since you are checking for the condition if pub_key and priv_key: it will always fail...

So, Should i update the code by removing private_key or close the PR.

Please update to simething like this:
pub_key, _ = generate_keys_from_p_q_e_n(None, None, e, n)
if pub_key:
   ...

Please update to something like this:
pub_key, _ = generate_keys_from_p_q_e_n(None, None, e, n)
if pub_key:
   ...

OK. :)

@daedalus review.

Please replace the variable priv_key  with: _

Please replace the variable priv_key with: _

DONE.

also: if n != None and e != None: can be changed toif n and e:

also: if n != None and e != None: can be changed toif n and e:

apologies, I m currently new to python

Merged, thanks!!!


Document which encryption libraries are used for HTTPS calls, and whether and when we use the OS

If this library is used to make HTTPS calls, what is an answer to the question "Does your app implement any standard encryption algorithms instead of, or in addition to, using or accessing the encryption in Apple’s OS" on providing Export Compliance Information in Apple's AppStore?

Is there any update here?


Add docsy docs template for site

Need to add site to linter blacklist here: https://github.com/GoogleContainerTools/kpt-functions-sdk/blob/master/.github/workflows/ci.yaml#L23

@frankfarzan to use this template you will need to build the docs html from the hugo site with make build.  This will place the html in the docs folder.
You should move the current docs content into the site folder first.
To view the docs locally run make server


cross-publish for 2.13

This does the simplest possible thing for cross-publishing for 2.13. There's more work to be done, like moving common code back into the scala dir and setting up tests for 2.13 code, but I'd like to merge this, to test that Travis builds the appropriate snapshots as expected.

This now runs tests for both 2.12 and 2.13 against our Docker services. 2.12-specific test dependencies have been removed.


Track detailed event for Google Analytics

https://developers.google.com/analytics/devguides/collection/gtagjs/events

https://betterprogramming.pub/how-to-conditionally-change-index-html-in-react-de090b51fed3


Fix Edit links if version is referenced by annotated tag

A better version of #3238 that fixes #1637, #1820 etc. thanks to responses from git ML.
<EMAIL_ADDRESS>Annotated tags in Git are special. GitHub is unable to find commit corresponding to tag hash, because annotated tags are pointing to tag meta data and need to be dereferenced first.

@ericholscher this is another PR that I think it's ready for merge and it's not too big. Can you take a look at this one?

I'd like to see a test that shows what this is fixing? I don't understand it otherwise really.

Specifically, a test in our code, so that we know it doesn't break in the future :)

I guess we don't have much for tests on the VCS code, so will go ahead and merge this.

I think we introduce a bug here after merging the PR:
[21/Dec/2017 14:21:39] readthedocs.vcs_support.utils:99[15182]: INFO Lock (poliastro): Releasing
[21/Dec/2017 14:21:39] readthedocs.doc_builder.environments:344[15182]: ERROR (Build) [poliastro:latest] need more than 1 value to unpack
Traceback (most recent call last):
  File "/home/humitos/rtfd/code/readthedocs.org/readthedocs/projects/tasks.py", line 209, in run_setup
    self.setup_vcs()
  File "/home/humitos/rtfd/code/readthedocs.org/readthedocs/projects/tasks.py", line 339, in setup_vcs
    update_imported_docs(self.version.pk)
  File "/home/humitos/.pyenv/versions/2.7.14/envs/rtfd/lib/python2.7/site-packages/celery/local.py", line 191, in __call__
    return self._get_current_object()(*a, **kw)
  File "/home/humitos/.pyenv/versions/2.7.14/envs/rtfd/lib/python2.7/site-packages/celery/app/task.py", line 380, in __call__
    return self.run(*args, **kwargs)
  File "/home/humitos/rtfd/code/readthedocs.org/readthedocs/projects/tasks.py", line 619, in update_imported_docs
    } for v in version_repo.tags
  File "/home/humitos/rtfd/code/readthedocs.org/readthedocs/vcs_support/backends/git.py", line 95, in tags
    return self.parse_tags(stdout)
  File "/home/humitos/rtfd/code/readthedocs.org/readthedocs/vcs_support/backends/git.py", line 121, in parse_tags
    commit_hash, name = row
ValueError: need more than 1 value to unpack

This is what I get from inside the RTD code:
(Pdb++) self.run('git', 'for-each-ref', '--format="%(if)%(*objectname)%(then)%(*objectname)%(else)%(objectname)%(end) %(refname)"', 'refs/tags')
[21/Dec/2017 14:31:26] readthedocs.vcs_support.base:62[15749]: INFO VCS[poliastro:latest]: git for-each-ref --format="%(if)%(*objectname)%(then)%(*objectname)%(else)%(objectname)%(end) %(refname)" refs/tags
[21/Dec/2017 14:31:26] readthedocs.vcs_support.base:70[15749]: INFO VCS[poliastro:latest]: "cf0274439e04abc164254ed92d2bf9cc17c53dce refs/tags/v0.1.0"
"53111940f2c0dc44c729724c8a5cb19815a75448 refs/tags/v0.1.1"
"f1ac1904cb48074d2dd503399001d7dc9438472b refs/tags/v0.1.2"
"91f76e251226ac8855801a59026421b72605ddce refs/tags/v0.2.0"
"39668fae67cdd40f63513b7b5d161ee18e90064b refs/tags/v0.2.1"
"0b9044e3f75ebe048f271f83e8054c2cc880607b refs/tags/v0.3.0"
"4c2b72283ba9674c9d223207ef9d5dbf6a7778ef refs/tags/v0.3.0rc1"
"16b4b6d496fdddfea816fab6cc0ed8062180d189 refs/tags/v0.3.1"
"d888e8045011403dc30c59c1d5f376198f7bf257 refs/tags/v0.3.2"
"72f3b62bab1432948b94d3fbfb38132d7d1065a0 refs/tags/v0.3.3"
"c7b914953fe58a37f2b52274eca8210fe6eec225 refs/tags/v0.4.0"
"7f9b8208e64d6604638eedb25e27345a77e65032 refs/tags/v0.4.1"
"d08c9baf2d8d12577075d64d172f8f4103591d4c refs/tags/v0.4.2"
"564b1d19a9a2fe0f7306d833638bea839d05fbdd refs/tags/v0.4.3"
"d56e4718dd31aa651e6ed036b1607abe16aa94f0 refs/tags/v0.5.0"
"3d05513e6a78987fb10e9b9d2dd29933cb5c3939 refs/tags/v0.6.0"
"93d229833dc1b9838ab167e791d655da103a0c9f refs/tags/v0.7.0"
"72c4cf841c77bb74d1dead34a28415e42610d51c refs/tags/v0.7.0b1"
"44dbc6e26f2d0a6b395a910c39c9e8131c8946b2 refs/tags/v0.8.0"

(0, u'"cf0274439e04abc164254ed92d2bf9cc17c53dce refs/tags/v0.1.0"\n"53111940f2c0dc44c729724c8a5cb19815a75448 refs/tags/v0.1.1"\n"f1ac1904cb48074d2dd503399001d7dc9438472b refs/tags/v0.1.2"\n"91f76e251226ac8855801a59026421b72605ddce refs/tags/v0.2.0"\n"39668fae67cdd40f63513b7b5d161ee18e90064b refs/tags/v0.2.1"\n"0b9044e3f75ebe048f271f83e8054c2cc880607b refs/tags/v0.3.0"\n"4c2b72283ba9674c9d223207ef9d5dbf6a7778ef refs/tags/v0.3.0rc1"\n"16b4b6d496fdddfea816fab6cc0ed8062180d189 refs/tags/v0.3.1"\n"d888e8045011403dc30c59c1d5f376198f7bf257 refs/tags/v0.3.2"\n"72f3b62bab1432948b94d3fbfb38132d7d1065a0 refs/tags/v0.3.3"\n"c7b914953fe58a37f2b52274eca8210fe6eec225 refs/tags/v0.4.0"\n"7f9b8208e64d6604638eedb25e27345a77e65032 refs/tags/v0.4.1"\n"d08c9baf2d8d12577075d64d172f8f4103591d4c refs/tags/v0.4.2"\n"564b1d19a9a2fe0f7306d833638bea839d05fbdd refs/tags/v0.4.3"\n"d56e4718dd31aa651e6ed036b1607abe16aa94f0 refs/tags/v0.5.0"\n"3d05513e6a78987fb10e9b9d2dd29933cb5c3939 refs/tags/v0.6.0"\n"93d229833dc1b9838ab167e791d655da103a0c9f refs/tags/v0.7.0"\n"72c4cf841c77bb74d1dead34a28415e42610d51c refs/tags/v0.7.0b1"\n"44dbc6e26f2d0a6b395a910c39c9e8131c8946b2 refs/tags/v0.8.0"\n', u'')
(Pdb++)

and this is from my own computer:
[humitos@julia:latest|(8685e25...)]$ git for-each-ref --format="%(if)%(*objectname)%(then)%(*objectname)%(else)%(objectname)%(end) %(refname)" refs/tags
cf0274439e04abc164254ed92d2bf9cc17c53dce refs/tags/v0.1.0
53111940f2c0dc44c729724c8a5cb19815a75448 refs/tags/v0.1.1
f1ac1904cb48074d2dd503399001d7dc9438472b refs/tags/v0.1.2
91f76e251226ac8855801a59026421b72605ddce refs/tags/v0.2.0
39668fae67cdd40f63513b7b5d161ee18e90064b refs/tags/v0.2.1
0b9044e3f75ebe048f271f83e8054c2cc880607b refs/tags/v0.3.0
4c2b72283ba9674c9d223207ef9d5dbf6a7778ef refs/tags/v0.3.0rc1
16b4b6d496fdddfea816fab6cc0ed8062180d189 refs/tags/v0.3.1
d888e8045011403dc30c59c1d5f376198f7bf257 refs/tags/v0.3.2
72f3b62bab1432948b94d3fbfb38132d7d1065a0 refs/tags/v0.3.3
c7b914953fe58a37f2b52274eca8210fe6eec225 refs/tags/v0.4.0
7f9b8208e64d6604638eedb25e27345a77e65032 refs/tags/v0.4.1
d08c9baf2d8d12577075d64d172f8f4103591d4c refs/tags/v0.4.2
564b1d19a9a2fe0f7306d833638bea839d05fbdd refs/tags/v0.4.3
d56e4718dd31aa651e6ed036b1607abe16aa94f0 refs/tags/v0.5.0
3d05513e6a78987fb10e9b9d2dd29933cb5c3939 refs/tags/v0.6.0
93d229833dc1b9838ab167e791d655da103a0c9f refs/tags/v0.7.0
72c4cf841c77bb74d1dead34a28415e42610d51c refs/tags/v0.7.0b1
44dbc6e26f2d0a6b395a910c39c9e8131c8946b2 refs/tags/v0.8.0
[humitos@julia:latest|(8685e25...)]$

the difference is that from inside RTD the output comes with " at the begining and the end of the line, so it's not well split by the csv reader.

Just propose a hotfix at #3437.

Just for people coming here and thinking this PR fixed the issue: it was reverted, and is currently not fixed yet (see https://github.com/rtfd/readthedocs.org/pull/3441)


Document base image process

Easier to review at: https://github.com/lcarva/atomic-reactor/blob/base-image-doc/docs/base_images.md

Coverage remained the same at 87.656% when pulling bce1382a5d434022f2dc2304daf24e8841bbea4b on lcarva:base-image-doc into 9e58e600860284fb48d4d65fd4b181c11385266d on projectatomic:master.

Coverage remained the same at 87.656% when pulling 4140c7ff51210c88cfdc01cab46cc255e5e7b756 on lcarva:base-image-doc into 9e58e600860284fb48d4d65fd4b181c11385266d on projectatomic:master.

Coverage remained the same at 87.656% when pulling 347992821a017a56cd5d954c6a65cec1bc76a553 on lcarva:base-image-doc into 9e58e600860284fb48d4d65fd4b181c11385266d on projectatomic:master.

LGTM


テキスト入力欄の右クリックメニュー

説明 / Description
#700
チェックリスト / Checklist

MUST

[ ] npm test passed
[ ] npm run lint was applied without warnings
[ ] changes of /docs/webapp not included (except release branch)
[ ] console.log not included (except script file)

RECOMMENDED (it depends on what you change)

[ ] unit test added/updated
[ ] i18n

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (de9fcb6) 35.92% compared to head (6d6d14c) 35.92%.

Files
Patch %
Lines

src/background/index.ts
0.00%
13 Missing :warning:

src/common/i18n/text_template.ts
0.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #706   +/-   ##
=======================================
  Coverage   35.92%   35.92%
=======================================
  Files         164      164
  Lines       25826    25851   +25
  Branches     1625     1625
=======================================
+ Hits         9277     9286    +9
- Misses      14937    14953   +16
  Partials     1612     1612

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