AIA PSF modelling in Python - [merged]

In GitLab by @nshirman on Sep 3, 2019, 09:41
Merges master -> master
Converted IDL file for modeling PSFs to Python.

In GitLab by @wtbarnes on Sep 3, 2019, 10:22
Closes #21

In GitLab by @wtbarnes on Sep 3, 2019, 10:25
I'd like to see a few tests before this gets merged though I'm not necessarily opposed to a separate PR (as long as it is in quick succession!).
I'm happy to help with these. Is there a notebook/script somewhere that shows how to use this to deconvolve an image?

In GitLab by @nshirman on Sep 3, 2019, 10:45
Mark made a notebook to compare the IDL and Python PSFs (after I made them into fits images), and I added in my code there and used Mark's comparison code to create a residual for each passband.
compare_psfs.ipynb
compare_psfs.html
The fits files are in the 'shirman' folder in sanhome, if you wanted to use those.

In GitLab by @wtbarnes on Sep 3, 2019, 15:11
Commented on aiapy/psf/aia_psf.py line 237
Call this width_x,width_y? In general, descriptive variable names are better.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 3
Remove any editor specific comments. This first docstring should be just a short very short description of what is in this file. For example, see top of aiapy/response/channel.py

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 7
I don't think each function name needs to start with "aia_*" since this is part of the larger aiapy package

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 1
Remove this line

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 6
I think this can be removed as it does not appear to be used anywhere.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 20
Remove this parameter altogether.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 22
Remove. Just use use_preflightcore

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 8
Thanks for adding docstrings to every function! Could you format these to follow NumPy docstring format? We want to be consistent with how sunpy and astropy do it. I've found this guide from the astropy docs to be very helpful when writing/formatting docstrings.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 97
You can remove this and just make the default value for False

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 112
Thinking about it a bit more, I don't think this function needs to exist. All of this can just go in a dictionary in this same file. Rather than calling a function, just use the dictionary. Unless there's some reason it needs to go in its own function?

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 249
Remove this. Just use xtomult

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 213
Remove this altogether. We don't need to wrap numpy functions.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 221
Can you describe in this docstring the mathematical operation that you're actually doing here? You can write LaTeX in restructured text that will get rendered nicely in Read the Docs. As an example of how to do this, see the docstring for the effective_area method in aiapy/response/channel.py

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 115
What are these filenames? Is the user expected to have these on their local machine? Can they be downloaded from somewhere?

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 284
Remove this. In general, we should avoid merging code with print statements in it (though often useful for debugging!)

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 258
This could be done in a loop and you could put dx and dy all in a single array, e.g.
d_xy = np.zeros(2,4)
for i in range(4):
    d_xy[0,i] = d * np.cos(meshinfo[f'angle{i+1}'] / 180. * np.pi)
    d_xy[1,i] = d * np.sin(meshinfo[f'angle{i+1}'] / 180. * np.pi)

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 254
I'm likely just ignorant, but what exactly is being iterated over here? Could this be vectorized?

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 245
What is k?

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 279
What is 0.18?

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 367
See comment in first function about this same parameter

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 307
What is 0.82?

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 374
Remove this parameter altogether.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 308
Better to keep naming conventions consistent. In Python, variables tend to be all-lowercase. See PEP-8/naming-conventions for additional info.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 370
wavelength should require units on it. See Channel.__init__ in aiapy/response/channel.py as an example.

In GitLab by @nshirman on Sep 3, 2019, 15:36
A lot of the things mentioned above, such as variables/variable names, numbers, print statements, the filenames in the first function, function names, and the way the functions are structured, is because I was translating from the original IDL file, and that's how everything was named and structured there. I thought it'd be easier to compare the IDL and Python version if they were structured similarly. If that's not important, since we won't be using the IDL version, I can go through and make those changes.
aia_psf.pro

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 381
Can just return this line directly

In GitLab by @wtbarnes on Sep 3, 2019, 15:24
The name of this file can just be psf.py. See my comment on function names above.

In GitLab by @wtbarnes on Sep 3, 2019, 15:22
Commented on aiapy/psf/aia_psf.py line 10
Add an __all__ = ['aia_calc_psf'] here as this is the primary user-facing function that we want out of this file. Then, in aiapy/psf/__init__.py we can add a from .psf import * to just get this function.

In GitLab by @nshirman on Sep 3, 2019, 17:10
Commented on aiapy/psf/aia_psf.py line 1
changed this line in version 2 of the diff

In GitLab by @nshirman on Sep 3, 2019, 17:10
Commented on aiapy/psf/aia_psf.py line 3
changed this line in version 2 of the diff

In GitLab by @wtbarnes on Sep 3, 2019, 17:03
I just merged #20 which added the CI config. Before merging this, let's rebase this on top of the current master branch so that the testing pipeline runs on this branch first.
Ignore this we can merge this and worry about the CI stuff later

In GitLab by @wtbarnes on Sep 3, 2019, 15:36
One other note: it looks like you're writing this in Python 2.7? aiapy is compatible with >=3.6 so I would recommend updating!

In GitLab by @nshirman on Sep 3, 2019, 17:11
Commented on aiapy/psf/aia_psf.py line 6
changed this line in version 3 of the diff

In GitLab by @nshirman on Sep 3, 2019, 17:10
added 1 commit
44f39ae5 - Update aia_psf.py
Compare with previous version

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 237
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 3, 2019, 17:11
added 1 commit
8ab8f619 - Update aia_psf.py
Compare with previous version

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 213
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 7
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 20
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 22
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 8
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 284
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 221
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 258
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 249
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 370
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 307
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 308
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 374
changed this line in version 4 of the diff

In GitLab by @nshirman on Sep 11, 2019, 10:45
Commented on aiapy/psf/aia_psf.py line 381
changed this line in version 4 of the diff

In GitLab by @wtbarnes on Sep 12, 2019, 11:16
Commented on aiapy/psf/aia_psf.py line 370
The input should be an astropy quantity with units. If it needs to be cast as a string later, then that is easy enough to do.

In GitLab by @nshirman on Sep 11, 2019, 10:46
Commented on aiapy/psf/aia_psf.py line 370
wavelength is a string here, so would it still need units? Or how would we require them in that case?

In GitLab by @markcheung on Sep 12, 2019, 11:50
I had a notebook in the AIA Notebooks repo (under AIA_HUB) which did this. But it needs to be updated to use PSFs from Nina's routine.
I need some education about how to help someones code. If everything needs to be resolved before a merge request is approved, do I need to make a clone of their fork first?

In GitLab by @nshirman on Sep 11, 2019, 10:45
added 1 commit
4264119a - Update aia_psf.py
Compare with previous version

In GitLab by @nshirman on Sep 13, 2019, 12:30
Commented on aiapy/psf/aia_psf.py line 112
changed this line in version 6 of the diff

In GitLab by @nshirman on Sep 13, 2019, 12:30
Commented on aiapy/psf/aia_psf.py line 115
changed this line in version 6 of the diff

In GitLab by @nshirman on Sep 12, 2019, 13:38
added 1 commit
c13f8ff9 - Update aia_psf.py
Compare with previous version

In GitLab by @nshirman on Sep 13, 2019, 12:30
added 1 commit
bff62358 - Shorten getmeshinfo function for filling in meshinfo dictionary
Compare with previous version

In GitLab by @nshirman on Sep 26, 2019, 15:15
Commented on aiapy/psf/aia_psf.py line 245
changed this line in version 8 of the diff

In GitLab by @nshirman on Sep 17, 2019, 09:12
Should be able to now!

In GitLab by @nshirman on Sep 26, 2019, 15:15
Commented on aiapy/psf/aia_psf.py line 97
changed this line in version 8 of the diff

In GitLab by @nshirman on Sep 13, 2019, 12:43
added 1 commit
f88fe1e1 - Add link to documentation pdf
Compare with previous version

In GitLab by @wtbarnes on Sep 17, 2019, 02:21
@nshirman If you check the box to allow commits from upstream members, @markcheung should be able to push commits to your branch. See instructions here

In GitLab by @nshirman on Sep 26, 2019, 15:15
Commented on aiapy/psf/aia_psf.py line 254
changed this line in version 8 of the diff

In GitLab by @nshirman on Sep 26, 2019, 15:15
Commented on aiapy/psf/aia_psf.py line 279
changed this line in version 8 of the diff

In GitLab by @nshirman on Sep 26, 2019, 15:15
Commented on aiapy/psf/aia_psf.py line 10
changed this line in version 8 of the diff

In GitLab by @nshirman on Sep 26, 2019, 15:15
Commented on aiapy/psf/aia_psf.py line 367
changed this line in version 8 of the diff

In GitLab by @nshirman on Sep 26, 2019, 15:15
added 1 commit
34ee7931 - Update aia_psf.py
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 15:21
added 17 commits
34ee7931...16273f61 - 16 commits from branch LMSAL_HUB/aia_hub:master45809277 - Merge branch 'master' of https://gitlab.com/LMSAL_HUB/aia_hub/aiapy
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:00
added 1 commit
868e69c5 - new commit to psf folder 2
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 15:52
added 1 commit
64545697 - new commit to psf folder
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 15:48
added 18 commits
3517680c - Update README.rst3fcd17de - Upload New File1054758d - Update aia_psf.pyb366a605 - Update aia_psf.py0cddb3f5 - Update aia_psf.py0bd470c1 - Update aia_psf.py037b8801 - Update aia_psf.py775e88e8 - Update aia_psf.pyed04811f - Update aia_psf.py44f39ae5 - Update aia_psf.py8ab8f619 - Update aia_psf.py4264119a - Update aia_psf.pyc13f8ff9 - Update aia_psf.pybff62358 - Shorten getmeshinfo function for filling in meshinfo dictionaryf88fe1e1 - Add link to documentation pdf34ee7931 - Update aia_psf.py45809277 - Merge branch 'master' of https://gitlab.com/LMSAL_HUB/aia_hub/aiapy312c957f - commiting new
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:17
added 1 commit
1ceebaab - Delete toolkit.py
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:17
Commented on aiapy/psf/toolkit.py line 1
changed this line in version 13 of the diff

In GitLab by @wtbarnes on Sep 30, 2019, 16:16
Commented on aiapy/psf/toolkit.py line 1
Let's remove this file and let @markcheung make a separate merge request with his deconvolution code.

In GitLab by @wtbarnes on Sep 30, 2019, 16:01
It'd be great if we could also add some tests here. In a previous comment, you linked to a notebook where you show a comparison with a PSF from a FITS file. We could track these as test data and then just do a diff between the two to make sure the difference stays within the acceptable error.
We can also call the IDL code from Python though this is considerably slower and cannot be run without an IDL install.
This could potentially be deferred to another PR as well

In GitLab by @nshirman on Sep 30, 2019, 16:20
Commented on aiapy/psf/init.py_keep line 1
changed this line in version 16 of the diff

In GitLab by @wtbarnes on Sep 30, 2019, 16:17
Commented on aiapy/psf/init.py_keep line 1
Can you remove the _keep in both this filename and the psf.py file?

In GitLab by @nshirman on Sep 30, 2019, 16:19
added 1 commit
4c31c4a8 - Delete psf.py
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:19
added 1 commit
382be094 - Update __init__.py
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:20
added 1 commit
4f1ae09f - Delete __init__.py_keep
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:30
added 2 commits
a6a6d754 - update file namesb0700365 - test
Compare with previous version

In GitLab by @nshirman on Sep 30, 2019, 16:23
Commented on aiapy/psf/init.py_keep line 1
Will do, for the psf.py file it says someone edited the file at the same time as me so I can't change the name at the moment.

In GitLab by @wtbarnes on Sep 30, 2019, 16:24
Commented on aiapy/psf/init.py_keep line 1
Hmm that's strange. This is in the GitLab GUI? If that doesn't work, I'd try the "add-commit-push" route from the command line.

In GitLab by @nshirman on Sep 30, 2019, 16:34
added 1 commit
84888b88 - Delete psf.py_keep
Compare with previous version

In GitLab by @markcheung on Sep 30, 2019, 18:24
approved this merge request

In GitLab by @nshirman on Sep 30, 2019, 16:34
Commented on aiapy/psf/init.py_keep line 1
Yes that worked, thanks!

In GitLab by @wtbarnes on Oct 1, 2019, 18:04
resolved all threads

In GitLab by @wtbarnes on Oct 1, 2019, 17:59
unmarked as a Work In Progress

In GitLab by @wtbarnes on Oct 1, 2019, 17:59
approved this merge request

In GitLab by @wtbarnes on Oct 1, 2019, 18:07
merged

In GitLab by @wtbarnes on Mar 23, 2020, 13:30
mentioned in commit dfe298e8bed8f3c4edf99a04b1327d06ad90c8f9

In GitLab by @wtbarnes on Oct 1, 2019, 18:07
mentioned in commit 627100810c56a0f444a42942143933c8419eac91


Video streaming audio not playing correctly

Hello, I'm using this same code in my system to implement the video call, but every time I start the call, my audio doesn't play on the other end of the line, and for this to work correctly I need to refresh the page where the call is being played so that my audio can be transmitted normally, could you please help me? Thanks in advance!

We are also experiencing the same and we have to refresh everytime is this has been @PedroHenrike20  can you please share the solution here it would be helpful for all of us

Hello, the solution to my problem was to update the component in my project, because I had made a bifork of it to make a change to the code and I wasn't getting the updates, so I extracted this component from my fork and updated it and then it came back to work normally, try to update and check again, because I didn't do anything else, although I published this issue and so far I haven't received any solution other than this modification I made. Any news keep me informed, please


bugged jwt audience validation library

What happened?
currently used library version (github.com/go-jose/go-jose/v3 v3.0.0) does not support multiple audience in expected jwt as there is a bug in the code
if len(e.Audience) != 0 {
	for _, v := range e.Audience {
		if !c.Audience.Contains(v) {
			return ErrInvalidAudience
		}
	}
}

in above code, it requires claims' audience are included in every expected audience
What did you expect to happen?
in latest commit (github.com/go-jose/go-jose/v3 v3.0.1-0.20230907165853-260aa2669b7e) the code is fixed where it claims' audience only need to appear once in expected audience
if len(e.Audience) != 0 {
	flag := false
	for _, v := range e.Audience {
		if c.Audience.Contains(v) {
			flag = true
			break
		}
	}
	if !flag {
		return ErrInvalidAudience
	}
}

How'd it happen?

verify a jwt token with expected audience containing multiple string

Hi @peterphoenix, thanks for reporting this issue. Could you help me understand what you're trying to do? Are you exposing the same service under multiple Pomerium routes, such that the service does not know which specific audience value to expect?

We are using auth0 with multiple audiences that has /userinfo in aud also
https://auth0.com/docs/secure/tokens/access-tokens/get-access-tokens#multiple-audiences

I think there might be some confusion between the IdP access tokens (such as from Auth0) and the JWTs that Pomerium itself issues (when setting the pass_identity_headers route option).
My understanding is that this SDK is intended only for use with the JWTs issued by Pomerium. (See https://www.pomerium.com/docs/capabilities/getting-users-identity for more information about this.)
If you'd like to work directly with IdP access tokens, you might be better off using a general-purpose JWT library directly, whether that is the go-jose library or another of the many options listed at https://jwt.io/libraries.

Hi @kenjenkins thanks for the clarification! Yeah, I was a bit confused with IdP access tokens and Pomerium's JWT. At first, I thought the audience could be different with the issuer, depending on the upstream. But it seems like for Pomerium's JWT case the issuer and audience will always be the same. If so, then there shouldn't be a problem with the current library.


New script for making SBML files that doesn't use KBaseFBAModeling an…

…d corrects bug in SBML download

Is this code included in develop?  If so we should close without merging.

I think this can be closed now, since this made it into develop and was merged to staging from there.

Just confirmed that this is already in staging, so I'm going to close it.


Sound wave won't play if startTime is different than 0.

After the sound wave is imported and setted on the audio component, if I execute a Play node with StartTime different from 0. , it will play no sound.

Are you using the latest version of the plugin? The plugin used to have this problem, but it has already been fixed.
If not, are you using the same node or which node are you using? I've tested in different ways, both on Windows and Android and everything works as it should

I'm using the plugin for ue4.25, Downloaded from release page on 13/01/2021 (previously I got from MarketPlace and couldn't get it to work at all).
I'm using an audio component, in which I set sound with the imported .wav and play with these nodes:

Just to be sure, I'll try to download again the zip for 4.25

I downloaded the RuntimeAudioImporter4250.zip from release and it worked!

It worked for normal sound, but for Ambisonics (4 channels) sound It doesn't .
I uploaded 2 ambisonics sounds in here if you wan't to test: https://we.tl/t-Mh5SMYfv2A
I also added 1 line in RuntimeAudioImporterLibrary.cpp
USoundWave* ReadyUSoundWave = GetSoundWaveObject(this, (const uint8*)WaveData, WaveDataSize, Status); // A ready USoundWave static object
					ReadyUSoundWave->bIsAmbisonics = 1; // added this line to make sure all imported audios are flagged as ambisonics

with this setup, the problem is only with the start time, if different from 0. it won't play.

Thanks for the message, I'll take a look and deal with this shortly.

I have an assumption that I missed something: some platforms do not use WAV, but OGG Vorbis format, which is why decompression does not take place in full.  I will fix this as soon as I can.

Thank you very much!

I will update the plugin soon, more information can be found in this chat - https://t.me/unrealblog

Fixed.

Hi there,
I still have this problem. Playing with start time of 0 works like a charm.
I am using UE 4.27 on windows.
When I installed the plugin from the marketplace it crashed when using a starttime other than 0.
I then found this thread and downloaded the latest 4.27 zip.
Now it doesn't crash anymore, but it doesn't play anything either.
It doesn't matter if I try to import an mp3 or a wav file.
Btw, altough I downloaded 4.27 zip it says it was built with 4.26


Creation of new Fluent API through DirectPrinter class

Fixes #82
@lukevp, I finally got back to this.
The end result is exactly what you suggested in #82
var DirectPrinter = new DirectPrinter(emitter, printer);
DirectPrinter.CenterAlign()
             .Print("Centered Text")
             .LeftAlign()
             .Print("Left Aligned Text")
             .Send();

There's an internal buffer that holds the byte arrays being added. Once the Send method is called, the buffer is sent to printer.Write and the buffer is cleared out. This means that the code below will send "ABC" to the printer and then send only "DEF" to the printer shortly after
var DirectPrinter = new DirectPrinter(emitter, printer);
DirectPrinter.Print("ABC").Send()
             .Print("DEF").Send();

Sorry I got lazy on the unit tests, but following the same principle we should have at least 1 test per command, to guarantee that calling a command through the emitter produces the exact same result as calling it through the DirectPrinter class. Right now that's easily seen, but in a future with multiple emitters and refactoring, unit tests would raise the red flag if something not expected happened.
Talking about raising flags, have you considered an auto pipeline? So you can have a build and tests check every merge/commit the repo gets on the master branch? You usually put a badge with the build status on the README.md
For example this one that I'm using on my repo. It uses Azure DevOps Pipelines:

looks good to me!  I think I'd prefer to use GitHub Actions since it's simpler and more tied in with GitHub.   Are you interested in developing a workflow.yml file?  If not I could give it a try.

Site thing. I've never done so with GitHub Actions, but I'll take a look how It works

I think we should also update the README.md when you release the new version, adding usage examples for the fluent API


[Bug] imgsrv.roll20.net proxy delivering content insecurely

Roll20 uses TLS by default, yay! But the content proxy server (imgsrv.roll20.net) isn't set up to deliver content over TLS, so the browsers are complaining.
Firefox is the most strict in this regard. For passive content (e.g., images), this isn't a big deal as Firefox allows the content. But for "active" content (e.g., fonts) the content is blocked completely.
This makes it impossible to use webfonts in our character sheets on Firefox.

Yeah, this issue is really causing me pain and it seems reasonably easy to fix.

I'm in agreement.  I'd like to see this addressed as well.  Has anyone put a suggestion on the suggestion forum?  I'd sacrifice one of my votes to add it to this if others go along as well.  One vote isn't likely to get it noticed….

Place a link here and I'll vote for it.

Okay, I spent a bit of time trying to look at this again (to familiarize myself with the issues), but I have much to learn in this area.  If I figure it out I'll make the suggestion and let you know.  Otherwise, somebody who's already got a handle on it can do that and I'll commit a vote (after I figure out which one I'm sacrificing :-( )

I don't get why this isn't resolved yet.  All it requires is setting up a TLS cert on the proxy server.  They could even buy and use a wildcard cert on all of their services.

@IanDominey Tell me about it...

2017 and this still isn't fixed :(

Reminder to remediate .

Why is it closed?
The bug is present by this day.


JSONata for Python seems broken

Hi,
I am trying to use JSONata for python by using the python wrapper available here: https://pypi.org/project/pyjsonata/ I can't seem to install this, pip install command fails with:
 pip install pyjsonata
Defaulting to user installation because normal site-packages is not writeable
ERROR: Could not find a version that satisfies the requirement pyjsonata (from versions: none)
ERROR: No matching distribution found for pyjsonata

Is there a plan to fix the PyPI build? I see that the make file needs to be written in python as mentioned here:
Source packages are currently broken until I can be bothered to rewrite jsonata-c’s Makefile in Python, as required by setuptools.

I am planning to take a dependency on JSONata in python and java and was hoping to understand if this is available for customer use. Any help here would be greatly appreciated.
Thanks

@anshikg you're probably getting that error because there isn't a pre-compiled (wheel) version that matches your architecture (see e.g. https://pypi.org/project/pyjsonata/0.0.1a9/#files for what's available). For me at least, installation via pip does work in principle, as long as there's a published version that matches the machine it's being installed on.
Until the root cause (which you correctly point to) is fixed, you can still build your own wheel manually by following the instructions in the README, and then install that. It worked reasonably well on a Mac.

I think I have an idea on how to fix this in pyjsonata. Btw I'm moving the issue to that repo since it's really a pyjsonata build problem.
Terribly sorry for taking a year to respond, I've had github email notifications turned off as they're so noisy, but it's made me miss quite a lot.


fix: Postgre\Connection::reconnect() TypeError in pg_ping()

Description
Fix #9043
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

Hi @ping-yee ,  one last rebase please and we can merge this. Thank you!

Thank you, @ping-yee . I rebased your branch.


Add support for Puppet Plans

Use the tasks mode of the Puppet parser to syntax check plans in the plans/ directory of a module. Fixes #95.

Looks like the related issue was resolved.


[Tour of Beam] [Frontend] Minor improvements

Minor UI improvements.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @alexeyinkin

R: @damccorm

Actually, looks like https://github.com/apache/beam/actions/runs/5001733504/jobs/8965458156?pr=26735 is failing and this failure seems new/not occurring on master (https://github.com/apache/beam/actions/workflows/playground_frontend_test.yml?query=branch%3Amaster).
@nausharipov could you please take a look? Would be great if we could just generally get green on our CI failures

@damccorm as Alexey mentioned, Playground frontend tests are being fixed. Fixing Tour of Beam integration test is on Dmitry.
This PR doesn't contain important and urgent changes. I'll add some more improvements and send it for review after the test fixes are merged.


[Concurrency] Warn about deadlock potential in withTaskCancellationHandler docs.

withTaskCancellationHandler gets called with the task status record lock held, so care needs to be taken if the cancellation handler acquires any locks of its own. Update the docs comment to describe this.
rdar://111686260

@swift-ci please smoke test


Wrong detection under yarn on windows

Test suite:

package.json

{
	"name": "test",
	"version": "1.0.0",
	"main": "index.js",
	"license": "MIT",
	"scripts": {
		"test": "node test.js"
	},
	"dependencies": {
		"cross-spawn": "^6.0.5"
	}
}

test.js

var crossSpawn = require('cross-spawn');
var cmd = crossSpawn.sync('yarn', ['--version'], {
	stdio: 'inherit',
});
console.log('yarn --version return:', cmd.status);
console.log('path of child process:', cmd.options.envPairs.filter(e => /^path=/i.test(e)));
process.exit(cmd.status);

command to run
yarn install
yarn node test.js
yarn run test

my output
PS D:\Projects\test> yarn node test.js
yarn node v1.12.1
1.12.1
yarn --version return: 0
path of child process: [ 'Path=xxxxxxxxxxxxxxx' ]
Done in 0.83s.

PS D:\Projects\test> yarn run test
yarn run v1.12.1
$ node test.js
The filename, directory name, or volume label syntax is incorrect.
yarn --version return: 1
path of child process: [ 'Path=A:\\USR-TMP\\yarn--1541088945270-0.3043972176672871;xxxxxxxxxxxxxxxxxxxx' ]
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

generated yarn and node in my $TEMP folder, contains bash script.
problem
this is also a yarn issue.
But yarn not want to fool this module, so the detect logic must have a bug. ( Or maybe there is a bug in nodejs core 🤣 )

Ok, finally, I found this is a yarn bug. yarnpkg/yarn#6631


Weird behavior of tf.math.reduce_max

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: None
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.12.0-0-ga6d8ffae09 1.12.0
Python version: 3.6.6
Bazel version (if compiling from source): None
GCC/Compiler version (if compiling from source): None
CUDA/cuDNN version: 9.0/7
GPU model and memory: GeForce GTX 1080 Ti, 11G

Describe the current behavior
import numpy as np
import tensorflow as tf

# tf.enable_eager_execution()

def test_bug():
    input = tf.convert_to_tensor(np.ones((1, 1, 1, 1), dtype=np.float32))
    x = tf.layers.conv2d(input, 1, (1, 1), activation="tanh",
                         kernel_initializer=tf.initializers.constant(2),
                         use_bias=False)
    max_out = tf.math.reduce_max(x, [1, 2])

    with tf.Session() as sess:
        sess.run(tf.global_variables_initializer())
        m = sess.run(max_out)
        m1, _ = sess.run([max_out, x])
        print('m =', m)
        print('m1 =', m1)
        assert np.allclose(m, m1)

test_bug()

I'd suppose that the assertion should pass given above code. However, tf.math.reduce_max looks very strange:
m = [[2.]]
m1 = [[0.9640276]]

Describe the expected behavior
m and m1 should have the same value and the assertion should pass.
m = [[0.9640276]]
m1 = [[0.9640276]]

Code to reproduce the issue
Listed above.
Other info / logs
If eager execution is enabled, then the value of x and max_out is the same.

@rmlarsen this looks like another bug like the earlier one we fixed where grappler is rewriting a node which is being fetched. Can you take a look?

I think this bug has been fixed in nightly. Please reopen / comment if I'm wrong.


Fix cookie refresh and authenticate caching

Fixes https://github.com/aspnet/Security/issues/309
Also addresses Authenticate being called repeatedly which was a regression (ticket should be generated once and cached)
Added a ShouldRenew for the CookieValidateContext which explicitly tells the cookie to renew the cookie.  Identity used to call SignInDirectly but that would trigger infinite validation loops.  Not sure this is the right fix but it appears to work.  Thoughts? @Tratcher

Updated splitting notification refresh from sliding refresh as discussed

:shipit:

b9f152ebb1c37845a89889ab7ae80a1395a40455


how can i fixed this issue(win7, npm 5.3.0, vc++ 2013/2015)

npm ERR! code ELIFECYCLE
npm ERR! errno<PHONE_NUMBER>
npm ERR<EMAIL_ADDRESS>watch: webpack-dev-server --hot --inline --progress --c olors --port 5000
npm ERR! Exit status<PHONE_NUMBER>
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>watch script.
npm ERR! This is probably not a problem with npm. There is likely additional log
ging output above.
npm ERR!      C:\Program Files\nodejs\node_cache_logs\2017-08-30T13_49_47_656Z-d
ebug.log

Sorry, you didn't follow the issue template, so we can't help you.


[FR]: Haystack integration

Proposal summary
Haystack is a popular library for working with LLMs, especially when working on RAG pipelines.
Would be great to have an integration so that tracks and chains are logged to the Opik platform.
Motivation
No response

Haystack integration is now released: https://www.comet.com/docs/opik/tracing/integrations/haystack


http payload hang on exit

To prove that we understood these to issues correctly, we ran a couple of tests.  The first was to change g_fn.write to a function which uses a syscall(SYS_write) to make sure we're not self-interposing ourselves.  With this build, curl never hung.
As a second test, (without the g_fn.write change we talked about as the first test), I commented out our close, fclose, and fcloseall calls from src/wrap.c.  Instead, I used funchook to patch __close() and __close_nocancel().
As a note, we got this idea for the second fix by running curl under gdb.  When we tried to set a breakpoint on __close_nocancel_nostatus(), it resulted in something like 60 or 80 breakpoints...  Which is consistent with this function being implemented as an inline function.
Checking the glibc source code we see this implementation of __close_nocancel_nostatus():
static inline void
__close_nocancel_nostatus (int fd)
{
__close_nocancel (fd);
}

So when we're interposing close(), we'd expect to miss these 60 or 80 spots that call __close_nocancel_nostatus().  None of these will ever call the close() that we provide.

When I added code to address both issues, I found that the go integration tests intermittently failed.  I think the go failures were always in statically linked go applications.  These tests were failing because the go applications were crashing with segfaults.
I didn't have time to completely understand why, but it appears to be timing related, and these crashes only occurred when the changes to patch __close and __close_nocancel were in place.
Since we're trying to get 0.7.2 out the door, I'm choosing to back out these changes to __close and __close_nocancel.  The change to use scope_write for g_fn.write is sufficient to avoid this hang on process exit, and there is no loss of stability according to our integration tests.


Create a job with sources

Hi!
it is possible to create a job with attached sources instead of specifying a folder location for them?

What do you mean by attached sources? Can you give an example for this?

I mean, job is created empty by default. It is possible to specify directory only for project sources. Maybe, it is possible to create a job together with project sources via api call? Attach a ZIP with sources or smth else?

You can created a job using a config.xml ...having a source directory or a zip file of sources is at the moment not possible but it might be a good improvement...What about offering a patch/pull request for this enhancement ?

Thanks!


add some persistence benchmarks to test various systems

Secretly, I wrote these so I could see what the results were on appveyor. Since I had to write them anyway, now we have an easy thing for users to run on various filesystems and setups that will give us information about how the building blocks of the contractmanager perform.
And these are building blocks we will of course be reusing as we update the persist stuff in the other packages.

This PR has been a challenge. But I think I got it now.

This is ready for merge


check_arabic_high_hamza always SKIPs

I may be misunderstanding, but it looks to me as if this code:
https://github.com/fonttools/fontbakery/blob/033a59ef84f267a4f13230258fb2c522dd3d037c/Lib/fontbakery/checks/universal/__init__.py#L913-L920
always produces a SKIP since, according to fontTools documentation, ttFont.getGlyphSet() returns

a dict-like object mapping glyph names to glyph objects

— it is not keyed by USV.
While on the subject, it may be unusual for a font to have ARABIC_LETTER_HIGH_HAMZA but not have ARABIC_LETTER_HAMZA, but in the case some such font comes along it seems this check should still test that ARABIC_LETTER_HIGH_HAMZA isn't a mark even if it can't test the approximate glyph size. Therefore I suggest the above SKIP happen only if ARABIC_LETTER_HIGH_HAMZA is absent. The presence/absence of ARABIC_LETTER_HIGH_HAMZA should then enable/disable the 2nd part of the check.

ARABIC_LETTER_HAMZA not in glyph_set or ARABIC_LETTER_HAMZA not in glyph_set
is clearly wrong anyway.


Place .PHONY in front of the commands

I was lazy and used the Update branch button in the Github UI 😄

Closing as the Makefile can definitely be removed. I'm running sanity checks on the mongo-orchestration folder (as drivers-evergreen-tools can use files from the project directory) before creating a PR to drop this as part of PHPLIB-1058.


Problem with exponential notation

When I put:
1 * 10 ^ 50
The result is: -5,376,172,055,173,529,600
When it really should be: 1e + 50
Or even if I put:
2000000000000000000*5
the result is: -8.446.744.073.709.551.616
When it should be: <IP_ADDRESS>.000.000.000
I know that for most people it is not common to do an operation of such a large number, but it gives the feeling that when large operations are made it gives errors, it would be good if you fix it please, since I am making a calculator and I do not want him to give me errors of this type.
Also I was testing the math_expresions package and it does it well but I prefer this one as it has more options like sinh, etc ...
If you could update that would be awesome.

Thanks for raising that! I think you're right – it's better to avoid working with the integer type in most use cases for this library.
In any case, this should work the way you want it to as of version 0.8.12.

Hello @ram6ler.
Now it works perfect thank you very much for such a quick response, that helps me a lot and it will surely help more people who use this very good package. Thanks again :)


New  trigger

Triggers if run on an Desktop preview.
Now you can do:
<Panel ux:Name="panel" Background="#F00" >
    <Desktop>
        <Change panel.Background="#0F0" />
    </Desktop>
</Panel>

Use case: I want some elements to be visible in iOS while others in Android, since I am using both <Android /> and <iOS/> the Desktop preview becomes empty.
Now I can using <Desktop /> to show whatever I need.

On the surface this looks good to me as a fix, but I think a better long term solution would be a more general mechanism that allows mixing of these flags, say a <Platform Is="iOS, Desktop"/> or similar (names are hard).

I don't like this. It makes it look like desktop is a supported export-target, which it isn't really.
I also think the semantics will be technically wrong; if running in Android emulation on ChromeOS, we'd be incorrectly reporting as non-desktop.
In general, I expect the lines between mobile and desktop to blur more in the future, making distinctions like these hard to support. We should instead strive to make tools that makes it easy to detect the platform-features that matter, and respond reasonably to that.

I agree. It might confuse some users. How about changing the name? Maybe, <Fallback />?  Or I can try @Sunjammer suggestion <Platform Is=“iOS, Desktop”

I think ux:Condition is really what you want here. Sadly, that's a long-standing, unimplemented planned feature.
But, we need to get out of the "exposing discrete platforms"-mindset, and focus on high-level, semantic tools. I'd prefer if we didn't cement the "desktop" vs "mobile" idea any more...

I think we have do occassionally have need of this trigger. Full cross-platform support will always be lagging slightly behind and such a trigger could be a decent stop-gap measure.
That said, I agree with @kusma that there is a naming problem. "Desktop" isn't right here, especially since Preview is meant to mimic Mobile. But I don't know what a good name for this is -- I'm okay with something long/ugly since it should rarely be used.

How about <NotMobile />? I don't know, like @Sunjammer said, names are hard. I personally find many uses for this implementation, however it's your call guys.


10x React Performance Blog Post

Looks like this failed due to "access rights". I figure I don't have to worry about this right?

Looks good, couple minor spelling issues I can fix myself tomorrow if you don't. I'll merge after that's fixed. Don't worry about the test. Thanks for the post!

Sure sounds good! Thanks!

I took a second look and saw some glaring mistakes. I fixed a few of them right now.

First sentence: 'from 3 seconds to less that 200 milliseconds' can you proof read it once more? Sorry.

ok. i think this is a little cleaner

Cleaned it up a tiny bit and posted, thanks!


Scrub secrets

This PR provides a way to get the 0.2 feature that would scrub secrets from logs in case if it leaks.
For now, I only handle env var, and the scrub writer is heavily mocked. Also, the files need to be moved around, there will be a refactor later in the PR advancement.
I'll update this description along the implementation.

I don't really know how to get access to the secrets values from the shim.
Since the shim is quite simple and only deals with files and exec, it doesn't have access to the buildkit secretStore.
By creating and using the _DAGGER_SCRUB_SECRET, I have access to the secrets' names, but to get their values, I need to actually connect to buildkit session and get access to the secret store. I don't know if it's what we want, and I don't know how to do it from the shim without creating a Frankenstein.
Can you enlighten me @aluzzardi, @sipsma ?

Just read the secrets like a user would: from ENV or from the File
The proposal I wrote mentioned passing env names and file paths, not secret IDs

Closes #4413 #3173
Once the PR is ready to review, we should double check that it solves the two issues above before merging.

@samalba : Are you sure about #3173?
@aluzzardi @sipsma @vito : I was hoping to set the _DAGGER_SCRUB_SECRETS env var in core/secret.go, but somehow, from the shim, it seems it's not set. Am I missing some layer where I need to pass an env var along?
My attempt at setting it is in the last WIP commit . If you can enlighten me, as the reading + scrubbing env var seems functional on the shim side.

@dolanor

I was hoping to set the _DAGGER_SCRUB_SECRETS env var in core/secret.go, but somehow, from the shim, it seems it's not set. Am I missing some layer where I need to pass an env var along?

Yep you guessed it. To pass an env var to the shim you should set it via LLB in (core.Container).Exec, just like this:
https://github.com/dagger/dagger/blob/b9518e4cd458057a51656055c3fee4770de12449/core/container.go#L906-L908
You can use the Secrets field to know what secret var names + paths to set:
https://github.com/dagger/dagger/blob/b9518e4cd458057a51656055c3fee4770de12449/core/container.go#L86-L99

leaving https://github.com/dagger/dagger/issues/1892 for future reference.

Testing the correct integration with the shim etc, I found 2 bugs. With a big help from @grouville .
https://github.com/dagger/dagger/issues/4607
https://github.com/dagger/dagger/issues/4608
The current PR won't fix them. I propose we review this PR as is as it is at feature parity with the 0.2 implementation.
I'll do follow up PR to fix those different issues.
PS: after thinking a little bit, I guess @vito 's approach in concourse might work (but I fear it might not work for other chars than \n)

The commits are always squashed now, we can't merge without squashing in the GitHub interface.

This is now waiting on you @vito @aluzzardi

I fixed everything that was found. Good for another pass
cc @vito @aluzzardi @sipsma

Just re-based. It was quite atrocious because my first commits were in conflict in the ./cmd/shim/main.go before I had a dedicated file for it.
So after a few unsuccessful rebases, I decided to squash my first commits until I split the logic in another file.
Then a simple rebase on upstream/main worked automatically.
The service code was the tricky one to merge automatically. But still, thanks @vito for it, I want to play with it :).


add cumulative metrics granularity info

this pr adds suuport for cumulative metrics granularity options info per notion spec
[ ] Outstanding questions need resolving
[ ] Needs review by PM @JStein77
[ ] Needs docs review (optional)
docs project

Depends on PR https://github.com/dbt-labs/dbt-core/pull/10350


FormData should not be passed to $http config as data

Method append undefined on the FormData instance passed to the formDataAppender function.
Steps to reproduce:
1 - Install angular-http-case-converter
2 - Install angular-upload-file
3 - Configure an input file as the rails example (with the formDataAppender function)
4 - Run the application and try to upload something
angular-upload-file version: 2.1.1
angular: 1.3.3
angular-http-case-converter: 0.1.1
Long explanation:
I'm using angular-http-case-converter to normalize formats between JavaScript (camelCase) and Ruby (snake-case), so during the transformation process it creates a new object transfering the properties of the original object to the new one while converting their format.
The problem is that the object being passed when I use angular-file-upload is FormData and this process replaces the original object making an error occur when the request transformer calls the append method that doesn't exist on the new object created by the interceptor that converted the format.

There is a difference between the data being used in $http and $upload. For http the data is being sent as one json object so processing could happen on the whole object but for $upload the properties specified in the data: {p1: 1, p2: 2} are treated as form data fields. That's like when you are uploading using regular html form and you have some fields that are being send along with the file as part of multipart/form-data request. This way on the server side you can access those fields by their name from the request body. So for $upload transformRequest functions will be called on the value of each property not the data object itself.
If you want to modify the property names at the root level of data then you can either do it before sending it to $upload or since version 2.2.0 you can specify option sendDataAsJson:true which will convert the whole data as one object to json and send it with the form data name data.
This way the transformRequest will be called on the whole data object and you can modify it as you want.
The only difference is that on the server side the form data name is 'data' which contains the whole data object and individual properties will not be added to multipart form data.

Hi Danial, congratulation for the great job you've been doing with angular-file-upload!
Maybe I've failed to explain myself, but looking to your answer I can comment that I understand it mainly because I've read the code. Passing it as an object or as field names is indifferent to who is passing the object, but for the chain of interceptors it is way different, and both ways are subject to the encoding phase that will get the data passed and serialize it to be transmitted and then used by the server-side.
The thing is that angular-file-upload internally passes a FormData instance to $http and it becomes impossible to modify or even read its property before the request is made except for the transformRequest function that executes right before the request is made. The suggestion I've made/implemented/pull-requested is that internally we pass the original data object to $http and just when the `transformRequest

I see what you mean now. Yea I think your approach is cleaner.
I will consider it for the next release.

Version 3.0.0 will not touch the data option and will send it as form data field with the name data to the server by default. If you set sendObjectsAsJsonBlob it will send it as json blob otherwise json strings.
There is also an option 'fields' which is similar to form data field and each key in that object will be sent individually as one form data field.

:+1: congratulations and thanks for fixing it so quickly ;)


Issues when trying to build application

root@debian:~/ellipto-iptv# yarn build
yarn run v1.22.19
$ react-scripts build
/bin/sh: 1: react-scripts: not found
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
root@debian:~/ellipto-iptv# yarn start
yarn run v1.22.19
$ react-scripts start
/bin/sh: 1: react-scripts: not found
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
root@debian:~/ellipto-iptv#

root@debian:~/ellipto-iptv# yarn build yarn run v1.22.19 $ react-scripts build /bin/sh: 1: react-scripts: not found error Command failed with exit code 127. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. root@debian:~/ellipto-iptv# yarn start yarn run v1.22.19 $ react-scripts start /bin/sh: 1: react-scripts: not found error Command failed with exit code 127. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. root@debian:~/ellipto-iptv#

Have you run yarn install? By doing so, it will install react-scripts.

So to confirm, clone repository, cd to ellipto-iptv, then run Yarn install?Sent from my iPhoneOn 18 Oct 2023, at 09:26, Md Mobinur Rahaman @.***> wrote:
@.:/ellipto-iptv# yarn build yarn run v1.22.19 $ react-scripts build /bin/sh: 1: react-scripts: not found error Command failed with exit code 127. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. @.:/ellipto-iptv# yarn start yarn run v1.22.19 $ react-scripts start /bin/sh: 1: react-scripts: not found error Command failed with exit code 127. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. @.***:~/ellipto-iptv#
Have you run yarn install? By doing so, it will install react-scripts.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

So to confirm, clone repository, cd to ellipto-iptv, then run Yarn install?Sent from my iPhoneOn 18 Oct 2023, at 09:26, Md Mobinur Rahaman @.> wrote: @.:/ellipto-iptv# yarn build yarn run v1.22.19 $ react-scripts build /bin/sh: 1: react-scripts: not found error Command failed with exit code 127. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. @.:/ellipto-iptv# yarn start yarn run v1.22.19 $ react-scripts start /bin/sh: 1: react-scripts: not found error Command failed with exit code 127. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. @.:~/ellipto-iptv# Have you run yarn install? By doing so, it will install react-scripts. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Yes. Are you able to build it?

root@debian:~# cd ellipto-iptv
root@debian:~/ellipto-iptv# yarn install
00h00m00s 0/0: : ERROR: [Errno 2] No such file or directory: 'install'
root@debian:~/ellipto-iptv#
Do I need to install react first?

root@debian:~# cd ellipto-iptv root@debian:~/ellipto-iptv# yarn install 00h00m00s 0/0: : ERROR: [Errno 2] No such file or directory: 'install' root@debian:~/ellipto-iptv#
Do I need to install react first?

You don't need to install React separately. The dependencies of this web app, including React and all other packages specified in the package.json file, will be automatically installed when you execute either the npm install or yarn install command.
It seems that Yarn is not currently installed on your system. To resolve this, you can globally install Yarn by using the command npm install --global yarn. After installing Yarn, you can proceed by running yarn install and then execute yarn build to successfully build this project.

So followed your instructions and I get the following:
root@debian:~# npm install --global yarn
added 1 package in 1s
root@debian:~# git clone https://github.com/MobinurRahaman/ellipto-iptv.git
Cloning into 'ellipto-iptv'...
remote: Enumerating objects: 419, done.
remote: Counting objects: 100% (63/63), done.
remote: Compressing objects: 100% (37/37), done.
remote: Total 419 (delta 41), reused 37 (delta 26), pack-reused 356
Receiving objects: 100% (419/419), 305.59 KiB | 2.50 MiB/s, done.
Resolving deltas: 100% (256/256), done.
root@debian:~# cd ellipto-iptv
root@debian:~/ellipto-iptv# yarn install
yarn install v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning "@emotion/react ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@emotion/react > @emotion/babel-plugin ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "@types/react@>=16".
warning "react-scripts > eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "react-scripts > eslint-config-react-app ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "react-scripts > react-dev-utils ><EMAIL_ADDRESS>has unmet peer dependency "typescript@>= 2.7".
warning "react-scripts > eslint-config-react-app > @typescript-eslint/eslint-plugin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".
[4/4] Building fresh packages...
Done in 37.04s.
root@debian:~/ellipto-iptv# yarn build
yarn run v1.22.19
$ react-scripts build
Creating an optimized production build...
Browserslist: caniuse-lite is outdated. Please run:
npx update-browserslist-db@latest
Why you should do it regularly: https://github.com/browserslist/update-db#readme
<--- Last few GCs --->
[401:0x55f89f2a9580]    22365 ms: Scavenge (reduce) 249.5 (258.0) -> 249.3 (258.8) MB, 1.0 / 0.0 ms  (average mu = 0.637, current mu = 0.463) allocation failure;
[401:0x55f89f2a9580]    22580 ms: Mark-sweep (reduce) 250.5 (259.0) -> 249.6 (259.5) MB, 212.8 / 0.0 ms  (average mu = 0.414, current mu = 0.052) allocation failure; scavenge might not succeed
<--- JS stacktrace --->
FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
1: 0x7f1e1d3f4a98 node::Abort() [/lib/x86_64-linux-gnu/libnode.so.108]
2: 0x7f1e1d2f40ab  [/lib/x86_64-linux-gnu/libnode.so.108]
3: 0x7f1e1d768e60 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/lib/x86_64-linux-gnu/libnode.so.108]
4: 0x7f1e1d76921b v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/lib/x86_64-linux-gnu/libnode.so.108]
5: 0x7f1e1d93a2f5  [/lib/x86_64-linux-gnu/libnode.so.108]
6: 0x7f1e1d94f673 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/lib/x86_64-linux-gnu/libnode.so.108]
7: 0x7f1e1d92bcde v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/lib/x86_64-linux-gnu/libnode.so.108]
8: 0x7f1e1d92d074 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/lib/x86_64-linux-gnu/libnode.so.108]
9: 0x7f1e1d90dd7d v8::internal::Factory::NewFillerObject(int, v8::internal::AllocationAlignment, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/lib/x86_64-linux-gnu/libnode.so.108]
10: 0x7f1e1dce656a v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/lib/x86_64-linux-gnu/libnode.so.108]
11: 0x7f1e1d679b79  [/lib/x86_64-linux-gnu/libnode.so.108]
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Thank you for your assistance with this.

Cancel that It was my fault I hadn't allocated enough ram to the Container. Thank you so much for your assistance.


EC2 describe_instance_types

Confirm by changing [ ] to [x] below:

[x] I've searched for previous similar issues or discussions and didn't find any solution

Platform/OS/Device
OSX
Describe the question
session = boto3.Session(region_name="us-east-2")
    ec2_client = session.client('ec2')

    result = ec2_client.describe_instance_types(
        Filters=[
            {
                'Name': 'network-info.efa-supported',
                'Values': [
                    'true'
                ]
            },
        ],
    )
    for instance in result['InstanceTypes']:
        print(instance['InstanceType'])

let ec2_conf = aws_sdk_ec2::config::Config::builder()
            .credentials_provider(aws_auth_providers::default_provider())
            .region(Some(Region::new("us-east-2")))
            .build();

        let ec2_client = aws_sdk_ec2::Client::from_conf(ec2_conf);

        let filters = vec![Filter::builder()
            .set_name(Some("network-info.efa-supported".to_string()))
            .set_values(Some(vec!["true".to_string()]))
            .build()];

        let result = ec2_client
            .describe_instance_types()
            .set_filters(Some(filters))
            .send()
            .await?;

        Ok(result
            .instance_types
            .unwrap_or_default()
            .into_iter()
            .filter_map(|instance| instance.instance_type)
            .collect::<Vec<_>>())

In both cases I get: [I3en24xlarge, C6gn16xlarge, G4dnMetal]
For
 aws ec2 describe-instance-types \
--region us-east-2 \
--filters Name=network-info.efa-supported,Values=true \
--query "InstanceTypes[*].[InstanceType]" \
--output text

I get 20 instances?!? What am I doing wrong?

interesting! Can you use --debug with the CLI and RUST_LOG='smithy_http=trace' (you'll need add tracing_subscriber::fmt::init()) on the SDK to compare what request is going across the wire in both cases? That should hopefully narrow it down.

cli.txt
rust.txt
Thank you for the timely reply!!!

Note that neither MaxResults nor set_max_results gave any changes. I didn't bother with pagination.

looks the CLI is implicitly paginating for you behind the scenes. Pagination is on our roadmap (#47) but isn't implemented yet. Until then, you'll need to manually construct a new request from the pagination token of the previous request.


Can‘t be deployed  to cloudflare？

I deploye it to cloudflare，through it display sucss ,  I can’t open the site. I tried 3 times  and don't know why...
Success: Finished cloning repository files
No build command specified. Skipping build step.
Note: No functions dir at /functions found. Skipping.
Validating asset output directory
Deploying your site to Cloudflare's global network...
 Uploading... (99/99)

✨ Success! Uploaded 0 files (99 already uploaded) (0.42 sec)

✨ Upload complete!
Success: Assets published!
Success: Your site was deployed!
Deploying your site to Cloudflare's global network...
 Uploading... (99/99)

✨ Success! Uploaded 0 files (99 already uploaded) (0.42 sec)

✨ Upload complete!

I was able to deploy to Cloudflare Pages, just choose AstroJS in the framework preset and add the build command: npm run build. I'm using Bun, so that's why I have bun run build  in the framework preset.
Also make sure to update AstroJS to the latest version.


[Feature Request]:  Support adding previously generated embedding to index

Feature Description
I can generate embeddings for my nodes and then add the nodes to the VectorStoreIndex.
They are added to the vector store
There are no embeddings gnerated for them, because they already exist
Reason
https://github.com/run-llama/llama_index/blob/main/llama-index-core/llama_index/core/indices/vector_store/base.py
Here in multiple places embeddings already set in nodes are ignored and new embeddings are generated
Value of Feature
Much easier and consistent way to deal with self generated embeddings
Avoidance of possible extra compuation

@suung this is already supported
https://github.com/run-llama/llama_index/blob/769e76a28aa497ec9885975a09eb8eec24713597/llama-index-core/llama_index/core/indices/utils.py#L132
If node.embedding is not none, the embedding will not be computed

The vector index calls the above function here
https://github.com/run-llama/llama_index/blob/769e76a28aa497ec9885975a09eb8eec24713597/llama-index-core/llama_index/core/indices/vector_store/base.py#L145

Hey @logan-markewich
thanks for getting back so real quick :)
Looks right and I might have missed it.
I was coming across this, because I was adding my own embeddings, adding the documents to the store and then by chance realised, that the open ai embeddings were called by default still.
If I come across it again, I will take a better look.


Improve the description of config password.policy.minimum.digits

Description
Some operators interpret "minimum digits" as "minimum number of characters" and confuse the setting password.policy.minimum.digits with password.policy.minimum.length. This PR changes the description of password.policy.minimum.digits and two other related settings to make the distinction clearer.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[X] Minor

Screenshots (if appropriate):
How Has This Been Tested?
I applied the changes in a local lab and verified the descriptions were updated through the UI.

@blueorangutan package

@rohityadavcloud a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7108

code lgtm
@winterhazel
could you like to improve description of special characters as well ?

@winterhazel could you like to improve description of special characters as well ?

@winterhazel , will you honour this request? (I don't think it is a blocking request for change but a good suggestion to complete the improvement) We can always do it later, if we remember ;)

code lgtm
@winterhazel could you like to improve description of special characters as well ?

@weizhouapache sure, I have changed the description of password.policy.minimum.special.characters to indicate which characters are considered special.

thanks @winterhazel
code lgtm

textual changes only => merging


[Plugin] Life-cycle hooks

Implements life-cycle hook plugins.
This shows only before-install-all hook, but other hooks can be easily added in the similar fashion.

This needs to go behind the feature flag

Since the hooks will be called from many places, I added the feature flag in the Plugin#hook method. This will make removing the flag easy.

@segiddins r?

@segiddins is this ready now?

@homu r+

:pushpin: Commit c22094a has been approved by segiddins

:lock: Merge conflict

:umbrella: The latest upstream changes (presumably #4788) made this pull request unmergeable. Please resolve the merge conflicts.

@asutoshpalai can you rebase this so we can merge it? Thanks!

@homu r+

:pushpin: Commit de65807 has been approved by segiddins

:zap: Test exempted - status


Unable to install Kubelogin in windows10

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks install-cli
Errors:
The command failed with an unexpected error. Here is the traceback:
<urlopen error [WinError 10054] An existing connection was forcibly closed by the remote host>
Traceback (most recent call last):
  File "urllib\request.py", line 1348, in do_open
  File "http\client.py", line 1282, in request
  File "http\client.py", line 1328, in _send_request
  File "http\client.py", line 1277, in endheaders
  File "http\client.py", line 1037, in _send_output
  File "http\client.py", line 975, in send
  File "http\client.py", line 1454, in connect
  File "ssl.py", line 512, in wrap_socket
  File "ssl.py", line 1070, in _create
  File "ssl.py", line 1341, in do_handshake
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 658, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 692, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acs/custom.py", line 368, in k8s_install_cli
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acs/custom.py", line 386, in k8s_install_kubectl
  File "urllib\request.py", line 216, in urlopen
  File "urllib\request.py", line 519, in open
  File "urllib\request.py", line 536, in _open
  File "urllib\request.py", line 496, in _call_chain
  File "urllib\request.py", line 1391, in https_open
  File "urllib\request.py", line 1351, in do_open
urllib.error.URLError: <urlopen error [WinError 10054] An existing connection was forcibly closed by the remote host>

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks install-cli

Expected Behavior
Environment Summary
Windows-10-10.0.19044-SP0
Python 3.10.4
Installer: MSI

azure-cli 2.36.0 *

Additional Context

route to CXP team

@hemasundarjambula Did you try upgrading your Azure CLI version as I am not able to reproduce the problem.

@hemasundarjambula Please let us know if you are still having issues.

I had upgraded to 2.46 .0 version but unable to install the Kube login in Windows10 PC.
@.***
Thanks &Regards
Hemasundar jambula | JHITOps | Micro Services
US Division Shared Application Services
Office: F3 building  | Manyata Tech Park | Bangalore, India | M. +91 9886-523604
<EMAIL_ADDRESS>| johnhancock.comhttp://www.johnhancock.com/
Please dial Prime line support number for priority issues :<PHONE_NUMBER>
Please consider the environment before printing this email.
OOO Alert : Dec 19,20,21,22,23
INTERNAL
From: SaurabhSharma-MSFT @.>
Sent: Tuesday, April 4, 2023 9:57 AM
To: Azure/azure-cli @.>
Cc: Hemasundar Jambula @.>; Mention @.>
Subject: [EXTERNAL] Re: [Azure/azure-cli] Unable to install Kubelogin in windows10 (Issue #25945)
CAUTION This email is from an external sender, be cautious with links and attachments.
@hemasundarjambulahttps://github.com/hemasundarjambula Please let us know if you are still having issues.

Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-cli/issues/25945#issuecomment-1496021130, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQRCOYS7WZKHPYFSVPMTARTW7QSBZANCNFSM6AAAAAAWJBMP3Y.
You are receiving this because you were mentioned.Message ID: @.***>
STATEMENT OF CONFIDENTIALITY The information contained in this email message and any attachments may be confidential and legally privileged and is intended for the use of the addressee(s) only. If you are not an intended recipient, please: (1) notify me immediately by replying to this message; (2) do not use, disseminate, distribute or reproduce any part of the message or any attachment; and (3) destroy all copies of this message and any attachments.

@hemasundarjambula It looks like you are having issues with your network or connections and thus getting this error.  Can you please try to install CLI on a different machine or in a different network.
Also, refer to other methods of installing kubelogin as listed here if that works for you.


popcornjs.org demo video broken

Issue reported: https://github.com/mozilla/popcorn-js/issues/455
Reported by: @minshallj
After the commit that removed twitter was merged (#332), the popcornjs.org demo video has been broken. At 29 seconds (Will Ferrell dancing) all content stops appearing next to the video. Google developer tools report this error:
Uncaught TypeError: Popcorn(...).listen(...).footnote(...).footnote(...).image(...).googlemap(...).googlemap(...).flickr(...).twitter is not a function
I don't really know anything about popcornjs, but wherever the twitter stuff went should be included along with http://popcornjs.org/code/dist/popcorn-complete.js on line 158 in the source for popcornjs.org

Last link seems broken as many examples


Revert "[text_input] prepare for custom text input sources"

Reverts flutter/flutter#72803
The PR seems to cause some internal tests to fail. A repro of the failure can be found in https://github.com/flutter/flutter/issues/74347.
Temporarily reverting the PR.

/cc @jpnurmi

/cc @jpnurmi


Getting segmentation faults when submitting empty BQMs to FixVariablesComposite

Might be coming from dimod - not sure.

I believe https://github.com/dwavesystems/dimod/pull/917 fixes this.
Checked with:
sampler = FixVariablesComposite(DWaveSampler(), algorithm='roof_duality')
sampler.sample(dimod.BQM('SPIN'))

Great! Closed by https://github.com/dwavesystems/dimod/pull/917


Split daemon and processSignatures

Would be nice (specially for testing) to be able to run kms ad-hoc and not as a daemon.
Just moving the code within the setInterval out of there and make it reachable by another command line option.
Cheers

@Hathoriel Would be good to have this divided. So we can also use this as an npm module...


Node.js  Loaders Team Meeting 2023-06-20

Time
UTC Tue 20-Jun-2023 18:00 (06:00 PM):

Timezone
Date/Time

US / Pacific
Tue 20-Jun-2023 11:00 (11:00 AM)

US / Mountain
Tue 20-Jun-2023 12:00 (12:00 PM)

US / Central
Tue 20-Jun-2023 13:00 (01:00 PM)

US / Eastern
Tue 20-Jun-2023 14:00 (02:00 PM)

EU / Western
Tue 20-Jun-2023 19:00 (07:00 PM)

EU / Central
Tue 20-Jun-2023 20:00 (08:00 PM)

EU / Eastern
Tue 20-Jun-2023 21:00 (09:00 PM)

Moscow
Tue 20-Jun-2023 21:00 (09:00 PM)

Chennai
Tue 20-Jun-2023 23:30 (11:30 PM)

Hangzhou
Wed 21-Jun-2023 02:00 (02:00 AM)

Tokyo
Wed 21-Jun-2023 03:00 (03:00 AM)

Sydney
Wed 21-Jun-2023 04:00 (04:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Loaders Team+Meeting+2023-06-20&iso=20230620T1800
or https://www.wolframalpha.com/input/?i=06PM+UTC%2C+Jun+20%2C+2023+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1SHBPJVM5-rjI3L85q4OE5ZO6yoCgjFBjaRDfxuFyNME/edit

Agenda
Extracted from loaders-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
Invited

Loaders team: @nodejs/loaders

Observers/Guests
Notes
The agenda comes from issues labelled with loaders-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/97506450082

For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

Possibly message port via register()

Would love to be part of the "message port in register()" discussion, but unfortunately I am not available at the time of the meeting.

Let's postpone that discussion til the following meeting then?

Would love to be part of the “message port in register()” discussion, but unfortunately I am not available at the time of the meeting.

This can also happen async. I don’t have any particular ideas for how to implement this, so if you have a design in mind please feel free to propose it in an issue in this repo. I think what we were generally aiming for is:

Provide a way to access the communications port between the application and loaders threads; on the application thread side it could perhaps be returned by register, and on the loaders side? import { messagePort } from 'node:module'? Or just do the API on both sides? Or one of the “hooks” could be the handler for the port, like we have export async function resolve we could also have export async function onMessage or whatever?
Once that lands, follow up with a PR to get rid of the globalPreload hook. Update the docs to recommend everyone use --import with register, and whatever would’ve been in a globalPreload sloppy script can now be in whatever code is run via --import (i.e. everything else that’s in the file that calls register). In this PR or as a follow-up, deprecate the --experimental-loader flag.

On a separate track, I’m eager to see https://github.com/nodejs/node/pull/47999 completed, as that finishes one of the three remaining to-do items in our “get to stable” milestone. @aduh95 where did you leave off with that, and is there anything in particular that you’d welcome help with on that one?
Closing because it seems like we’re not meeting this week.


Interaction.PlotActions improvement

A fix for #3304
This fix stores the plot actions assigned to the Interaction in seperate field during disabled periods (e.g, when dragging axis lines).
It restores the actions on Enable
/// <summary>
/// Disable all mouse interactivity
/// </summary>
public void Disable()
{
    //back up the enabled plot actions
    EnabledPlotActions = Actions;
    Actions = PlotActions.NonInteractive();
}

/// <summary>
/// Enable mouse interactivity using the default mouse actions
/// </summary>
public void Enable()
{
    Actions = EnabledPlotActions;
}

Elegant solution! Thanks for this PR @Nils-Berghs, I'll merge it now and it will be in the next release 🚀


Is it possible to specify an output directory?

Apologies if this is a really silly question, but I would like to be able to specify the output directory for particular conversions (perhaps in the YAML) so that if I write an article in Obsidian the resulting copy is stored in same folder as under the system set up with a full-blown word processor. Could this be possible? I had a quick, ignorant, scan of the code and couldn't see anywhere that the output directory was specified.

@seatrout apologies if I misread your requirements, first time responder…
Have you seen the Export Folder option in the settings under Obsidian Settings → PLUGIN OPTIONS → Pandoc Plugin → scroll down a bit to see the "Export folder": this is not specific for a particular conversion but at least allows you to set a common export folder for all export files.

Perhaps this could be extended to allow diff export paths for specific export formats in future…
Hope this helps.

Thanks! I had missed that. But it makes life much easier


Fix Flaky Test in MetadataManagerSuite

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #3331 (cd3c10c) into master (f88c4cb) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #3331      +/-   ##
============================================
- Coverage     51.58%   51.52%   -0.06%
  Complexity       13       13
============================================
  Files           475      475
  Lines         26475    26475
  Branches       3694     3694
============================================
- Hits          13656    13641      -15
- Misses        11496    11505       +9
- Partials       1323     1329       +6

Impacted Files
Coverage Δ

...pache/kyuubi/server/metadata/MetadataManager.scala
83.97% <100.00%> (ø)

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...uubi/engine/spark/events/SparkOperationEvent.scala
88.88% <0.00%> (-5.56%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
76.13% <0.00%> (-3.41%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
78.00% <0.00%> (-2.00%)
:arrow_down:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
74.07% <0.00%> (-0.93%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
82.60% <0.00%> (-0.63%)
:arrow_down:

...rc/main/scala/org/apache/spark/ui/EnginePage.scala
78.93% <0.00%> (-0.30%)
:arrow_down:

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
96.89% <0.00%> (-0.09%)
:arrow_down:

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

cc @pan3793

cc @pan3793

thanks, merged to master and branch-1.6


Update functions/Get-DbaCustomError.ps1

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[X] Documentation
[ ] Build system

Purpose

Update example to reflect current parameter name. The -SqlInstance is NOT an array

🎉 🎈 🎉 🎈 🎉 🎈


Disable automatic updates for Flatcar in the E2E tests

Flatcar has automatic updates that are triggered 10 minutes after the instance comes up and then each hour after that. If there are updates to be applied, Flatcar will apply those updates and then reboot the instance 5 minutes after applying updates.
This can cause stability issues and make tests fail. See #2318 and #2319 for example.
We should disable automatic updates in the E2E tests to reduce the flakiness rate of Flatcar tests. That can be done via Ignition as described here: https://flatcar-linux.org/docs/latest/setup/releases/update-strategies/

This has been implemented by #3391, #3393, and #3398.
/close


searchResponse.total() is different from searchResponse.businesses().size() ?

If the number of business to return is limited (e.g. params.put("limit", "3");), then searchResponse.businesses().size() will be 3 but searchResponse.total() seems to return the # of businesses that match the criteria without the limit. Is this expected ?

Hello @Nevvea7,
Yes, it's an expected behavior. "Total" is the number of businesses that matches the searching criteria instead of the number of business results in the response.


WIP: add track id to async api calls

API calls should be logging a more appropriate tracking id. It already exists but seems not be used consistently enough.

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✖debian. JID-1147

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1215

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-1218

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests


F-Droid can't build

For https://github.com/deckerst/aves/releases/tag/v1.9.1
...
Downloading android-x64-release/linux-x64 tools...                 848ms

Running Gradle task 'assembleLibreRelease'...
Checking the license for package Android SDK Platform 34 in /opt/android-sdk/licenses
License for package Android SDK Platform 34 accepted.
Preparing "Install Android SDK Platform 34 (revision: 2)".
"Install Android SDK Platform 34 (revision: 2)" ready.
Installing Android SDK Platform 34 in /opt/android-sdk/platforms/android-34
"Install Android SDK Platform 34 (revision: 2)" complete.
"Install Android SDK Platform 34 (revision: 2)" finished.

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':app:compileLibreReleaseKotlin'.
> No matching toolchains found for requested specification: {languageVersion=8, vendor=any, implementation=vendor-specific}.
   > No locally installed toolchains match (see https://docs.gradle.org/8.0/userguide/toolchains.html#sec:auto_detection) and toolchain download repositories have not been configured (see https://docs.gradle.org/8.0/userguide/toolchains.html#sub:download_repositories).

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 7m 45s
Running Gradle task 'assembleLibreRelease'...                     466.7s
Gradle task assembleLibreRelease failed with exit code 1

Not sure I understand what is missing, it has Java17, it wants 8?
fyi https://gitlab.com/fdroid/fdroiddata/-/commit/e3c4fb8fb5f244423ff2c29e9e8b6a22321eea08

I think it complains that gradle v8 toolchain is missing (not java v8).
On Thu, 24 Aug 2023, 16:22 Licaon_Kter, @.***> wrote:

For https://github.com/deckerst/aves/releases/tag/v1.9.1
...
Downloading android-x64-release/linux-x64 tools...                 848ms
Running Gradle task 'assembleLibreRelease'...
Checking the license for package Android SDK Platform 34 in /opt/android-sdk/licenses
License for package Android SDK Platform 34 accepted.
Preparing "Install Android SDK Platform 34 (revision: 2)".
"Install Android SDK Platform 34 (revision: 2)" ready.
Installing Android SDK Platform 34 in /opt/android-sdk/platforms/android-34
"Install Android SDK Platform 34 (revision: 2)" complete.
"Install Android SDK Platform 34 (revision: 2)" finished.
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':app:compileLibreReleaseKotlin'.

No matching toolchains found for requested specification: {languageVersion=8, vendor=any, implementation=vendor-specific}.
No locally installed toolchains match (see https://docs.gradle.org/8.0/userguide/toolchains.html#sec:auto_detection) and toolchain download repositories have not been configured (see https://docs.gradle.org/8.0/userguide/toolchains.html#sub:download_repositories).

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 7m 45s
Running Gradle task 'assembleLibreRelease'...                     466.7s
Gradle task assembleLibreRelease failed with exit code 1
Not sure I understand what is missing, it has Java17, it wants 8?
fyi
https://gitlab.com/fdroid/fdroiddata/-/commit/e3c4fb8fb5f244423ff2c29e9e8b6a22321eea08
—
Reply to this email directly, view it on GitHub
https://github.com/deckerst/aves/issues/725, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKBEXKNVRX7LVTANBIUGRDXW5PRPANCNFSM6AAAAAA35DREOU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

What is a gradle 8 toochain ?

@licaon-kter I've tried something, it may help! Releasing v1.9.2 now. Please let me know whether the build works for you now.

Started a local build. brb

Does it still fail?


Feature Request: Name Printer

Simple one, probably would go with efforts to have config files. Being able to set a printer name would be quite helpful to us running multiple printers. (I've currently got 4 printers all running Mariner for instance)

Seconded!

The raspberry pi zero w uses mdns, for example I changed my host name for my printer to sonicmini4k1, that way I navigate to http://sonicminic4k1:5000 to see the interface. If people got in the habit of changing their host name, it’d be relatively easy to then put the host name in the title of Mariner

The raspberry pi zero w uses mdns, for example I changed my host name for my printer to sonicmini4k1, that way I navigate to http://sonicminic4k1:5000 to see the interface. If people got in the habit of changing their host name, it’d be relatively easy to then put the host name in the title of Mariner

Interesting, I've never dug into it, but the host name stuff has never worked for me on windows with the raspberry pi zero w units over wifi, only when setup and connected via USB. (Not sure if this is a windows issue, and I need to install bonjour/etc, or if the multi-cast udp just isn't being forwarded across the wifi extender that the units are connected too.)
I actually just modified my system hosts file the other day to work around this issue on a single system.

One can just change the title in index.html as well. I'm doing both hostname+title but I could probably pass the hostname as a variable for the title somehow.

I just implemented this. If you run mariner from the master branch (I haven't released a new version yet with this feature), you can configure the display name for the printer from /etc/mariner/config.toml.

How can I run master ?

You can download from the artifacts generated by continuous integration. For example, for the latest commit on master as of today: https://github.com/luizribeiro/mariner/actions/runs/845877861


Database box setPassword.sh script problem on Windows (Git EOL)

On Windows hosts, the setPassword.sh scripts for the Oracle Database boxes fail. This seems to be the same Git for Windows end-of-line conversion issue that was reported recently for the Kubernetes box (#68).
Environment: Windows 10 Professional x64 Version 1803, VirtualBox 5.2.18, Vagrant 2.1.5, Git for Windows 2.19.0.
The following from the Oracle Database <IP_ADDRESS> box shows the issue. The 11g XE and 18c setPassword.sh scripts generate the same error.
[vagrant@oracle-12201-vagrant ~]$ sudo su - oracle
Last login: Sun Sep 16 16:54:02 -06 2018
[oracle@oracle-12201-vagrant ~]$ ./setPassword.sh TopSecret
-bash: ./setPassword.sh: /bin/bash^M: bad interpreter: No such file or directory

To confirm that end-of-line conversion is the problem, I made a copy of the setPassword.sh script with the CR characters removed. The copy worked correctly.
[oracle@oracle-12201-vagrant ~]$ tr -d '\r' < setPassword.sh > setPassword2.sh
[oracle@oracle-12201-vagrant ~]$ chmod a+rx setPassword2.sh
[oracle@oracle-12201-vagrant ~]$ ./setPassword2.sh TopSecret
The Oracle base remains unchanged with value /opt/oracle
...
<script runs normally>
...
SQL> Disconnected from Oracle Database 12c Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production

I've verified that the same .gitattributes file that @AmedeeBulle created to fix this issue for the ContainerRegistry and Kubernetes boxes also fixes it for the database boxes.
If you agree that this is a legitimate issue, and that adding .gitattributes files is the right solution, I'd be happy to submit a PR.

My bad -- I though I was the only one running shell scripts from the guests, so I didn't looked at other boxes.
PR is welcome, thanks for spotting this.

Thank you, sir.  PR submitted.

Merged! Thanks for the contribution, @PaulNeumann


Get service version out of retryRpc loop

getServiceVersion() is a simple unauthenticated code, that is called after a successful connection.
Calling it with retryRpc utility opens a way for some allowed exceptions to fuel an infinite loop.
Fixes #11228

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8947/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8949/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8973/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.2 please


Package engine "Node >= 10.0.0" causing installation problem in Node 15.4.0

Describe the bug
Can not install with node 15.4.0 using yarn.
To Reproduce
Get node 15.4.0, install yarn 1.22.5, then
$ yarn add nes.css

Expected behavior
NES.css installs correctly
Screenshots

Environment:

OS: Windows
Browser: N/A (not a concern)
Build environment (i.e. NodeJS): N/A (not building)

Suggestion(s) for fixing this issue
Remove the engine field from package.json, or update it.
Additional context
IDK, thanks.

This problem has already been fixed by #418 but has yet to be published. It would be nice if a new version could be released on NPM.

How can yarn add using the develop branch?

Sorry for the late answer. I'm using yarn install --ignore-engines to install the dependencies for my project. To add the package you can edit the package.json file by hand.

#469 depicts the same issue. If it's fixed upstream, what does it take to get it into a release? This is a pretty breaking bug after all


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1938 ms

Resolved: Bofrost.be performance has improved in 3651b31.


how to add component in the overlay area

how to make component with zIndex in the transparent area

Hi!
I'm closing this issue because it's a question and not a bug report. Please don't take it personally, I'm just trying to make it easier to maintain this project by separating questions from bug reports 🙏.
If you're still looking for feedback/answers, could you please ask your question in a new discussion?
Thank you!


Fixed pdfjs cmaps path resolution

I use pdf-visual-diff in my repo and got the following error.
node ➜ /workspaces/pdf-test (master) $ yarn ts-node index.ts
yarn run v1.22.19
$ /workspaces/pdf-test/node_modules/.bin/ts-node index.ts
Warning: loadFont - translateFont failed: "UnknownErrorException: Unable to load binary CMap at: /workspaces/pdf-test/node_modules/pdf-visual-diff/node_modules/pdfjs-dist/cmaps/Adobe-Japan1-UCS2.bcmap".
Warning: Error during font loading: Unable to load binary CMap at: /workspaces/pdf-test/node_modules/pdf-visual-diff/node_modules/pdfjs-dist/cmaps/Adobe-Japan1-UCS2.bcmap
Warning: loadFont - translateFont failed: "UnknownErrorException: Unable to load binary CMap at: /workspaces/pdf-test/node_modules/pdf-visual-diff/node_modules/pdfjs-dist/cmaps/Adobe-Japan1-UCS2.bcmap".
Warning: Error during font loading: Unable to load binary CMap at: /workspaces/pdf-test/node_modules/pdf-visual-diff/node_modules/pdfjs-dist/cmaps/Adobe-Japan1-UCS2.bcmap

The correct cmaps path  is
/workspaces/pdf-test/node_modules/pdfjs-dist/cmaps/Adobe-Japan1-UCS2.bcmap
but
/workspaces/pdf-test/node_modules/pdf-visual-diff/node_modules/pdfjs-dist/cmaps/Adobe-Japan1-UCS2.bcmap
(/workspaces/pdf-test is base dir of my repo.)
This PR fixes pdfjs-dist path resolution.

@moshensky
Hi. Please review this PR when you have time.

@krml4913 I took the liberty to merge your PR. I have released 0.7.1 with your fix. Thank you :)


Add fr-config-delete package

fr-config-delete package to clean out a tenant - i.e. remove
Static config

Journeys
Scripts
Custom endpoints
Connectors
Mappings
RCS instances

Dynamic config

Managed objects

Agents

Sync state

etc

This feature would be very helpful in our workflow so that we don't have to keep track of deleted items. Has there been any progress on this?


Improve the DFD printing

Dear OWASP,
I am using the latest version 1.3.1 on Mac. If the DFD is too big, the diagram is not correctly printed. The scroll bars appear but the diagram is not complete. Check the screenshot

Thanks

Hello @lukfe85 - this is a very good point. It is something that we hope to change for version 2.0 later this year. The new area for Threat Dragon is in the OWASP github organisation - do you want to raise the issue there?
https://github.com/OWASP/threat-dragon-core/issues
Cheers, Jon

Thanks @jgadsden I did it.


chore: export cmder.SetFirstKeyPos to support build module commands

fix https://github.com/go-redis/redis/issues/1990

Thanks. This is an internal API so we can change it at any point.


Updated DeveloperGuide

Coverage remained the same at 65.474% when pulling 79afab3a2fc0d7819e3b91f4c52d44a0bbdbb46d on jomcruz93:master into<PHONE_NUMBER>4e90a82f610c59c548b0e96727af2c on CS2113-AY1819S1-T16-1:master.


Replace s3 acl creation with bucket policy for flink on eks

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

What is the outcome that you are trying to reach?
Currently, the flink on eks module is still using s3 bucket ACL. It is not recommended to use s3 acl by AWS, and if the bucket blocks all the public access, then the deployment of flink on eks failed. Here are the logs:
╷
│ Error: error creating S3 bucket ACL for flink-operator-doeks-flink-logs-20230510141128566600000005: AccessControlListNotSupported: The bucket does not allow ACLs
│ 	status code: 400, request id: 9X4ZR5H18VX1ZQ83, host id: dwfHXqf2F1UjYcDnoUPlo9pSloivD9HSixRgn1V6IlwuopthKSglnK7J6CyCiTD9hSSC5FXYjQo=
│
│   with module.s3_bucket.aws_s3_bucket_acl.this[0],
│   on .terraform/modules/s3_bucket/main.tf line 41, in resource "aws_s3_bucket_acl" "this":
│   41: resource "aws_s3_bucket_acl" "this" {

Describe the solution you would like
Using Bucket policy instead of s3 acl.

Describe alternatives you have considered
Nope.

Additional context

@hitsub2 Thanks for reporting the issue. This is using a s3 upstream module and I hope this doesn't require any change to the upstream module.
Would you be able to raise a PR if this is a simple change of removing the line https://github.com/awslabs/data-on-eks/blob/b058d5ee731754d2327ef63a3e8fcd9b51cc4c13/streaming/flink/addons.tf#L258?


ZEPPELIN-3345. Don't load interpreter setting when it exist interpreter.json but no interpreter-setting.json found

What is this PR for?
It usually happens in dev enviroment. e.g. I have livy interpreter in interpreter.json due to history testing but no livy folder under {ZEPPELIN_HOME}/interpreter after I rebuilt the project without building livy module. This PR would just remove that invalid interpreter setting when loading from interpreter.json
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3345

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

hey @zjffdu thanks for making all these improvements, could we make sure there is at least one review before merging?
@Leemoonsoo