Youtube-dl in Ubuntu 14.04 return redirect link, not original video link

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.01.18. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.01.18

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[Ubuntu 14.04]
$ youtube-dl -g https://www.youtube.com/watch?v=-Sl8zi1ISdw -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-g', u'https://www.youtube.com/watch?v=-Sl8zi1ISdw', u'-v']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.01.18
[debug] Python version 2.7.6 - Linux-3.13.0-74-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: ffmpeg 3.2.2, ffprobe 3.2.2
[debug] Proxy map: {}
https://r5---sn-ogueln7k.googlevideo.com/videoplayback?id=f9297cce2d4849dc&itag=248&source=youtube&requiressl=yes&mv=u&mn=sn-ogueln7k&mm=31&pl=16&ms=au&nh=IgpwcjAzLm5ydDE5KgkxMjcuMC4wLjE&ratebypass=yes&mime=video/webm&gir=yes&clen=69865286&lmt=1476762069197443&dur=241.583&signature=46753D9BFEF1BD8B0233E30452A7EB352437DE06.0EBB7BF9F6643246B3783E175CC823069DD387D1&mt=1484812962&key=dg_yt0&upn=zWYblsaiNak&ip=<IP_ADDRESS>&ipbits=0&expire=1484834626&sparams=ip,ipbits,expire,id,itag,source,requiressl,mv,mn,mm,pl,ms,nh,ratebypass,mime,gir,clen,lmt,dur
https://r5---sn-ogueln7k.googlevideo.com/videoplayback?id=f9297cce2d4849dc&itag=251&source=youtube&requiressl=yes&mv=u&mn=sn-ogueln7k&mm=31&pl=16&ms=au&nh=IgpwcjAzLm5ydDE5KgkxMjcuMC4wLjE&ratebypass=yes&mime=audio/webm&gir=yes&clen=3904190&lmt=1476761860096521&dur=241.641&signature=0D14BD2832CC9F43E9DCBD7C47677B1AB59B089E.6BA858B819E15C3BC42B82CC63BC0949DE08FF31&mt=1484812962&key=dg_yt0&upn=zWYblsaiNak&ip=<IP_ADDRESS>&ipbits=0&expire=1484834626&sparams=ip,ipbits,expire,id,itag,source,requiressl,mv,mn,mm,pl,ms,nh,ratebypass,mime,gir,clen,lmt,dur

[Mac OS]

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
I'm using youtube-dl with -g option to play original youtube video in my website.
I made simple server that return original video url.
It's all okay when I test in my local Mac.
However, when I deploy my youtube-dl server to Ubuntu 14.04, It's little bit strange.
I'm using same version of youtube-dl(2017.01.18) and same version of python(2.7.12) in both Mac and Ubuntu, but they give me different url.
The main problem is the url from Ubuntu is not real original video url. It redirects to another video url.(In above case,  https://r2---sn-3u-bh2r.googlevideo.com/videoplayback?pl=16&ei=NXSAWLe3KIqJ4gK3l6SIBw&upn=Wgx7FL9FhmU&ipbits=0&requiressl=yes&itag=22&ratebypass=yes&dur=241.696&lmt=1476952144664504&source=youtube&sparams=dur,ei,expire,id,ip,ipbits,ipbypass,itag,lmt,mime,mm,mn,ms,mv,nh,pcm2cms,pl,ratebypass,requiressl,source,upn&key=cms1&mime=video%2Fmp4&expire=1484834965&id=o-AEo6sQA7hwMJULAegASwEhfi2ZdV2gpJELqOXnDuNIzS&signature=5EC2ACF647F1BCA174DD3A9B470B81C91349E363.29E250BFE6C3A7B09E7ACAFBFC4E1C4509D6529F&ip=<IP_ADDRESS>&redirect_counter=1&req_id=9deccb23abc7a3ee&cms_redirect=yes&ipbypass=yes&mm=31&mn=sn-3u-bh2r&ms=au&mt=1484813143&mv=m&pcm2cms=yes)
I need real video link because i have to load same video a lot of time. When i load the videos with wrong link, It's became really slow...(I think it's because of browser cache or something..)
Why is it different and can you fix it?
Thanks.

URLs starting with https://r2---sn-3u-bh2r.googlevideo.com are original links.

When i load the videos with wrong link, It's became really slow...

Most likely that's due to incorrect CDNs. https://github.com/rg3/youtube-dl/issues/9001#issuecomment-201991818 may help.

@yan12125 Thanks for your comment.
However, your suggestion doesn't work in my case.
There is no problem when i use one video tag.
But, with many video tag with same src, it has a problem.
I found that the problem is chrome doesn't cache redirected video, so when I add new video dynamically,  chrome unnecessarily requests the same video every time.
It makes my website extremely slow.
In Firefox and Safari, everything is fine.
Also, I found that chrome cache all videos well with direct video link.
I can't get redirect location because of CORS and w3c specification in browser.
So, is there any way to get direct video link for every user in any location from youtube-dl server?
I think it's impossible due to security reasons, but if you have any suggestion please help me.

I don't quite understand your problems, but I guess that's a Chrome problem rather than YouTube or youtube-dl. In my experience Firefox usually follows w3c standards while Chrome often invent their own standards.


Doesn't work cross projects.

To my understanding it is possible to link multiple projects as workspaces in the same sanity studio.
I configured mine and installed this plugin and datasets from other projects are selectable which is exactly what I want, but two issues came up:
Let's suppose that I have two projects (A & B),  both with two datasets (A & B)
I cannot select to copy documents from Project-A/Dataset-A to Project-B/Dataset-A. It only works when selecting a dataset with a different name in the destination project.
It is also not possible to duplicate data to another project at all because the same token is used to first read from source project and then write to the target project.
Wouldn't it be possible to fetch the secret in the destination project and use it's api key to allow cross-project duplications.

Yea, this still doesn't work. I have two different projects both with 'production' dataset and it does not work. The destination entry is not alterable (but shows both configured projects with their project IDs). I installed on an older version of sanity studio and also a clean install of the latest sanity studio (3.5.1 as of this writing). Still doesn't work. My whole reason for wanting this was to copy only some data between projects intended for different websites while keeping their own projects to themselves.

any update on this? projects showing but greyed out.

After some small fix, it seems working.
I created a PR for this fix : https://github.com/sanity-io/cross-dataset-duplicator/pull/55


Link to cli binaries from UI

Is your feature request related to a problem? Please describe.
If a developer is just coming on board, it would be very helpful if an admin could give access to the UI then have a link there to pull the latest cli tooling. While it's definitely not hard to find with google, it's just one less potential source of frustration for a new user.
Describe the solution you'd like
On the UI, we should include a link somewhere to the right release for the targeted cluster

@scures can you pls confirm if this is the intended link?
If, so the link seems to work, but the cli downloads fail for me. Also the "See all packages". Is perhaps this intended to be fixed on another ticket?

We should have added additional information to the issue. The URL is as Sorin said, made up using the version of epinio. When running locally this is set as an env var when running the backend (see https://github.com/epinio/ui/blob/dev/docs/developer/install-latest-standalone-ui.md#to-run-the-latest-version-of-the-ui ./jetstream command. To test this you could set that to v1.1.0 which should produce a valid link

Let's make sure the linked version matches the epinio-server version currently deployed and not always the latest epinio binary version.

@jimmykarily It's the version we show bottom left of the UI, normally supplied via chart value, so should already align correctly on epinio release. On a slightly related note, I think we have a separate issue to update where that version comes from (api /info request rather than helm chart --> ui backend env var)

We should have added additional information to the issue. The URL is as Sorin said, made up using the version of epinio. When running locally this is set as an env var when running the backend (see https://github.com/epinio/ui/blob/dev/docs/developer/install-latest-standalone-ui.md#to-run-the-latest-version-of-the-ui ./jetstream command. To test this you could set that to v1.1.0 which should produce a valid link

@richard-cox  I tried to check this and I am not sure it works as expected.
I followed your advice and set environment within ./jetstream to 1.1.0 instead that to dev:

However I still get broken links.
When I look at the error, it seems not only a version issue but another part of the URL. For the Apple binary for instance, it points to this URL that does not work: https://github.com/epinio/epinio/releases/download/1.1.0/epinio-darwin-arm64 instead of  https://github.com/epinio/epinio/releases/tag/v1.1.0/epinio-darwin-arm64. So it seems the /download/ part should be replaced by /tag/ and the version should be preceeded by a v. Is this something to be done here or will be automatically set once into Epinio?

Just realized of an error on the last comment.
It was suggested  by @richard-cox to create the environment with v.1.1.0 and I just added 1.1.0. ok, I will replace that, but I understand still the switch download for tag still applies. Checking it again.

Actually it works correctly when the environment is set right. @richard-cox all good here, pls disregard the previous comments.
Pasting screenshot of this:

Ok, closing issue. Checked in latest epinio dev and working. Added to our test plan.


Don't assume a *NIX system

In your code, you're silently assume a *NIX system (macOS or GNU/Linux).
For example https://github.com/pitchmuc/adobe_analytics_api_2.0/blob/04e29001025a6440e972efe6b1b9f292aac956a8/adobe_analytics_2/aanalytics2.py#L42 could break on a windows machine.
Join paths using os.path.join instead.

Thanks again.
I have seen that it causes some issue, especially for the Certificate retrieval.
When I try to read the path that users are providing for their certificate, it breaks on windows.
 line77  with open(_pathToKey, 'r') as f:
I prefer to use pathlib than os.path but I need to find a way to decrypt the path before fetching it

Could you provide an example of a path before and after decryption?
Perhaps I can provide some tips.

I think I found a way to deal with opening the file on different OS.
The problem is that, I ask for the path to their file, what I did before was trying to open the file directly and people may enter NIX path for window system. So it created problems.
Now I pass it to the Path class from the pathlib library.
It seems to automatically detect the path and find.
I tried with a NIX path on my window machine and it was able to detect the file.
I use it on 2 locations :
importConfigFile function
#file is the string argument of the function that can be a path or the name of the file.
if file.startswith('/'):
        file = "."+file
    with open(Path(file), 'r') as file:

retrieveToken function
# _pathToKey is the string that contains the path.
if _pathToKey.startswith('/'):
        _pathToKey = "."+_pathToKey
    with open(Path(_pathToKey), 'r') as f:

What I do, I see if the string start with a slash and assume that if this is case, the user wanted to use a relative path.
Wondering if I should encapsulate that into a try except function now.

I didn't know about this os.sep, I will try to use it in the print output so it match user expectation !
Thanks

I have updated the code now to reflect the user machine :
This
print(' file created at this location : ' + _cwd + '/config_admin.json')

becomes
print(f" file created at this location : {os.getcwd()}{os.sep}config_admin.json")


Reserving 'yigit' as username

@yigitkonur, your PR cannot be merged because the CI checks fail. It says that you haven't set up your my-links repository for adding your links.


segment_key_values doesn't create explicit segments when passed as a parameter into why.log

Description
when calling whylogs like this:
why.log(df, segment_key_values={segment_key: segment_value})

whylogs should return a SegmentedResultSet with an explicit segment defined by the segment_key_values

[x] I have reviewed the Guidelines for Contributing and the Code of Conduct.

fixed


Adding an IndexedDB Storage

This pull request introduces an IndexedDB storage implementation for Axios caching. IndexedDB provides a client-side storage solution that can be used to store and retrieve data for efficient caching of Axios requests.
Key Changes:
Added a new module for IndexedDB-based storage.

Created functions for finding, setting, and removing data from IndexedDB storage.
Initialized an IndexedDB database and object store for Axios caching.
Integrated the IndexedDB storage into the Axios caching system.

Code Highlights:

getIndexedDbStorage(db: IDBPDatabase, storeName: string): BuildStorage: This function creates an IndexedDB storage object for Axios caching. It includes methods for finding, setting, and removing data based on the provided IndexedDB database and store name.
buildIndexedDbStorage(): Promise: This function initializes the IndexedDB storage by creating the necessary object store and returns an AxiosStorage object, which can be used for caching Axios requests.

Note:

Since the indexed is only available in browsers, no tests can be designed for this storage without mocking.
Due to the nature of the IndexedDB interface, a web API was not possible because of asynchrony.

Due to the nature of the IndexedDB interface, a web API was not possible because of asynchrony.

You can use new Promise((resolve, reject) => ...) to convert callbacks to promises.

Since the indexed is only available in browsers, no tests can be designed for this storage without mocking.

I cannot merge this without tests, we are already mocking a DOM object to be used, I think it can be used here too.
https://github.com/arthurfiorette/axios-cache-interceptor/blob/main/test/dom.ts

Actually, some of my comments above were misleading... I look into idb and raw indexedDb usage and we should stick with idb. It seems waaay more easy to handle and integrate with. Just change it to a peerDependency. I'm not sure if our build step would include idb source into the bundle or just a import to it. A closer look to it should be done after the initial implementation.

I'll have a deeper look into testing idb in the next days/ weeks. Maybe it is possible with the fake-indexeddb package.

Maybe it is possible with the fake-indexeddb package.

Just use this property within the tests.
// test/dom.ts
dom.window.indexedDB

Yes that is good.
I just had the idea if it wouldn't be better to combine the IndexedDB storage with the buildWebStorage interface.
Extending the WebStorage API to a "BrowserStorage Interface". IndexedDB is just like localstorage and sessionstorage by default only available in browsers and not in Node.js.

I just had the idea if it wouldn't be better to combine the IndexedDB storage with the buildWebStorage interface.

I don't think so, they share different APIs, local/session storage is sync and idb is async. The more split storage methods are the better it will be, as tree shaking can remove more code from being sent to the browser.

@arthurfiorette The jsdom indexedDB Property seems to be undefined. Can you take a look at it?

Just use this property within the tests.
// test/dom.ts
dom.window.indexedDB

@lucajung maybe something in their docs would help?
https://github.com/jsdom/jsdom

@lucajung is this ready for review?

Currently, I'll just mention https://github.com/arthurfiorette/axios-cache-interceptor/issues/669#issuecomment-1820679994 at documentation when searching for IndexedDb support.


Bump attr and relax its version constraints

attr uses calver, so specifying "^19" clearly does not
work as intended
Fix #123
Although, to be honest I'm considering dropping the dependency on attr and use dataclasses from the stdlib instead, but that would mean bumping the minimum required version of Python to 3.7

haven't used calver but poetry does the same thing with ^19, meaning 19.something but not 20.something - you need >=19.0.0 too
anyway thanks for the speedy response :)

Sorry I'm confused. The patch i mad replaced "^19" with ">=20". Are you OK with that or do you want ">= 19" ?

Sorry I'm confused. The patch i made replaced "^19" with ">=20". Are you OK with that or do you want ">= 19" ?

no worries. i'm happy either way. i'm on v20 anyway. was just chit-chatting :)

OK, then


changes for aca py 0.7.4

So many files.  Is this generated code?  I see in the changes some are copyright dates and version updates -- but the page takes a long time to load :-)
I raise this because I'm wondering about the ACA-Py changes that drive this. Is the number of files changed reasonable, or did something happen between 0.7.3 and 0.7.4 that we should know about?
Thanks!

So many files. Is this generated code? I see in the changes some are copyright dates and version updates -- but the page takes a long time to load :-)

Looks like most of the changes are due to a line with a space in it (in a comment) is replaced with a line with no space :-D
I'm guessing a change to a template that is used to generate code.

Sorry for that. Yes, these are generated classes. I do not know where the space in the header is coming from, but from what is currently in the main branch there aren't any breaking changes in the api. The only api change so far is an additional "auto_verify" flag in the presentation exchanges. But as not everything is merged yet that might change.


Pooling: passing "sequence of n (low, high) integer pairs" resulting in TypeError

Trying to pass a tuple or list of tuples to a pool operation's padding parameter gives out the following errors:
TypeError: Unknown padding type: (1, 1).
TypeError : unhashable type: 'list'
Sample code for reproducing the bug:
from flax import nn
from jax import random

class FlaxModel(nn.Module):
    def apply(self, x):
        x = nn.max_pool(x, (3, 3), strides=(2, 2), padding=[(1, 1), (1, 1)])
        return x

rng = random.PRNGKey(0)
model, _ = FlaxModel.init_by_shape(rng, [(1, 100, 100, 1)])

Indeed looks like our code doesn't support padding that's a sequence of pairs. @hawkinsp has said that the version of JAX on HEAD added support for this, we should add a test and plumb it through correctly. (Or in the meanwhile if that's impossible, support this by manually padding before calling into lax.reduce_window)

Yes, JAX at head supports a sequence of (low, high) padding pairs.
Flax probably still needs to do some work to add batch and feature dimensions to what the user provides. JAX and XLA don't have opinions about which dimensions are batch and which are feature, but Flax is documented to only accept padding for the spatial dimensions.


Laravel / Vue 3 Install Issues

The installation guide for adding Inertia with Vue 3 to Laravel is out dated or inaccurate. I suggest going through the installation pages on a boilerplate Laravel app to witness.
For those who also spent hours following the install guide I'd suggest taking the install guide as far as you can, then visit the Laravel.com pages below to piece together the lapse in documentation.
https://laravel.com/docs/10.x/vite#vue
https://laravel.com/docs/10.x/vite#inertia

We'd really appreciate PR's for this, thanks.


Add ability to Undock from Explorer

The Changes, Branches and Tag windows in Team Explorer all have the option to "Undock from Team Explorer" allowing the developer to have the Stash window and the team explorer visible at the same time.
Could you add this option to the Stash window?

Yes, great suggestion! It required relatively easy code changes, so I've just added this function.
I'll release it soon.

yea, I just lost a little work, because I didn't know that stash doesn't automatically include untracked changes... so I'll be pinning this window to always be visible now...

I've just released a new version. Could you please get it (Git stash extension) and test new function.

I ran into an issue with this undocking feature. After undocking, if I view a stash info, it opens it in the team explorer window but after that point, the undocked window does not respond to any clicks even after clicking on back in team explorer.

Hmm. Interesting.  I just tried to undock and didn't get this issue.  One thing @voshchinskiyvitya , the stash info window opens in Team Explorer once the Stashes window is undocked.

@nvmkpk, it's very strange behavior. I've used standard Visual Studio Team Explorer extensibility. What about other undocked windows? Do they work?

@WDrake98, good suggestion, I'll try to add undocking functionality for "Stash info" page too.

I've just released a new version (Git stash extension). Now Stash Info page also can be undocked.


uncaught exception

My logs show several occurrences of the following error:
node-app[XXX]: error: UNCAUGHT EXCEPTION TypeError: Cannot read property 'name' of undefined#012    at RiakNode._connectionClosed (/usr/share/XXX/Staging-d-O79Y6AIIG/node_modules/basho-riak-client/lib/core/riaknode.js:565:27)#012    at emitTwo (events.js:106:13)#012    at RiakConnection.emit (events.js:191:7)#012    at RiakConnection._emitAndClose (/usr/share/XXX/Staging-d-O79Y6AIIG/node_modules/basho-riak-client/lib/core/riakconnection.js:137:18)#012    at RiakConnection._connHandleEnd (/usr/share/XXX/Staging-d-O79Y6AIIG/node_modules/basho-riak-client/lib/core/riakconnection.js:144:14)#012    at emitNone (events.js:91:20)#012    at Socket.emit (events.js:185:7)#012    at endReadableNT (_stream_readable.js:934:12)#012    at _combinedTickCallback (internal/process/next_tick.js:74:11)#012    at process._tickDomainCallback (internal/process/next_tick.js:122:9)

I assume, from the comment a couple lines above, that command / command.name may be undefined, which may cause these uncaught exceptions.
Let me know if you need an issue or anything.

Thanks for the PR. Just out of curiosity, what are you doing when this happens? As you can see here, all commands are assigned a name during construction:
https://github.com/basho/riak-nodejs-client/blob/master/lib/commands/commandbase.js#L53-L55

Thank you, I have added your commit to PR #164


Add Explore All Courses on Landing Screen

Description
LEARNER-8182
This PR adds the support of the Explore All Courses button on the landing screen.
Testing

[ ] On clicking the Explore all courses button opens course discovery.

On iPhone 5 the button is overlapping in landscape orientation

@salman2013 I've fixed the UI issue on iPhone 5s.


Remove scripts that are no longer used

These used to be used more when Probot was an app instead of a framework. I don't use any of these any more, and I'm guessing nobody else does either.

I do wish we had a better way of testing Probot as we build it (rather than requiring from another app's dir), but that's out of the scope of this PR so godspeed ⚡

I sometimes run ./bin/probot.js run ./test/fixtures/example.js (just need a .env and .pem in the local directory).  Otherwise I use npm link to try it out against an existing app locally.


Question: Is there a way to add channels without having to compile the whole app?

I tried to edit channelInfoList.json without any effect.

Zapp fetches updated channel infos from https://api.zapp.mediathekview.de/v1/channelInfoList, which override the local channels. You can let this url (strings/config.xml) poitn to another url and serve the channels you like - but I doubt this is possible without recompiling.
Compiling should not be too difficult however.

Ok thanks


Keep marker shown until touching another point of the screen

Hi!
Is it possible to show the marker when the column is touch and instead of hiding it when the user stop pressing the column, continue to show it until the user presses it again or presses on another column?
From what I see, that is not possible out of the box (please let me know if I am wrong).
If it is not supported, is it something you would plan to add?
Thanks :)

Hello @ampeixoto, unfortunately, this is not supported. However, it is a feature that we’d like to implement soon.

@ampeixoto I'm doing this for linechart, might work for column too:
var persistentMarkers by remember { mutableStateOf(emptyMap<Float, Marker>()) }
var persistentMarkers by remember { mutableStateOf(emptyMap<Float, Marker>()) }

markerVisibilityChangeListener = object : MarkerVisibilityChangeListener {
                override fun onMarkerMoved(
                    marker: Marker,
                    markerEntryModels: List<Marker.EntryModel>
                ) {
                    persistentMarkers = mapOf(markerEntryModels.first().entry.x to marker)
                }

                override fun onMarkerShown(
                    marker: Marker,
                    markerEntryModels: List<Marker.EntryModel>
                ) {
                    val markerXValue = markerEntryModels.first().entry.x

                    persistentMarkers =
                        if (persistentMarkers.containsKey(markerXValue)) emptyMap() else
                            mapOf(markerXValue to marker)
                }
            }

lineChart(
...
persistentMarkers = persistentMarkers,
...


Delete the local copy of eigen that is shipped with tensorflow

TensorFlow now pulls the eigen code directly from the eigen upstream repository. There is no need to keep a local copy anymore.

Can one of the admins verify this patch?


moved fedora to the top

issue reported can be reproduced with fedora 22

This looks good to me.
@bcoe do you want to sign off? (this moves RHEL further down the list, which effects your PR #6)

make it so :+1:

Just published 2.5.0 to npm


Unexpected panic when using callback

Hi!
I was trying out the new syscall/js/callback package, and I'm wondering if I'm using it correctly. The entire source for my test can be found on my github.
I'm getting a panic when passing a syscall/js/callback.Callback to syscall/js.Value.Call. My example looks like this (abbreviated):
promise := js.Global.Call("fetch", "http://google.com")
promise.Call("then", callback.New(func(args []js.Value) {
    fmt.Println(args)
}))

This results in the following panic:
panic: invalid value wasm_exec.js:59:6
wasm_exec.js:59:6
goroutine 1 [running]: wasm_exec.js:59:6
syscall/js.ValueOf(0x3dc60, 0xc01e2d0, 0x1) wasm_exec.js:59:6
    $GOPATH/src/github.com/neelance/go/src/syscall/js/js.go:95 +0x90 wasm_exec.js:59:6
syscall/js.makeArgs(0xc077d68, 0x1, 0x1, 0x691a0, 0xc01e2b8, 0x691a0) wasm_exec.js:59:6
    $GOPATH/src/github.com/neelance/go/src/syscall/js/js.go:130 +0x7 wasm_exec.js:59:6
syscall/js.Value.Call(0x1a, 0x74e9f, 0x4, 0xc077d68, 0x1, 0x1, 0x1a) wasm_exec.js:59:6
    $GOPATH/src/github.com/neelance/go/src/syscall/js/js.go:141 +0x2 wasm_exec.js:59:6
github.com/johanbrandhorst/wasm-experiments/fetch/fetch.Fetch(0x1, 0xc0d2000, 0xc028b10, 0xc01e260, 0x10, 0x10, 0x0, 0x0, 0x0, 0x0, ...) wasm_exec.js:59:6
    $GOPATH/src/github.com/johanbrandhorst/wasm-experiments/fetch/fetch/fetch.go:73 +0x1f wasm_exec.js:59:6
main.main() wasm_exec.js:59:6
    $GOPATH/src/github.com/johanbrandhorst/wasm-experiments/fetch/main.go:25 +0xc

@johanbrandhorst This may or may not be related, bu the http://google.com URL doesn't permit CORS, so the code you tried to execute is expected to fail in browser:

What happens if you try it on another URL, one that permits CORS? For example, https://api.github.com:

(Or did you try running this via Node.js? If so, CORS is probably not applicable.)

@shurcooL Cool, I will switch out the URL, but the error occurs before the request is even made as far as I can tell.
I ran the example in Firefox. In my testing I wasn't able to use node without installing third party Fetch API polyfills.

Also, have you tried this?
 promise.Call("then", callback.New(func(args []js.Value) {
 	fmt.Println(args)
-}))
+}).Value)

Since callback package isn't documented yet, it's not clear to me how it's meant to be used.

I couldn't see any way to interact with the Callback type other than using Dispose. Changing to use .Value removes the panic! Thanks for the tip!

It might be worthwhile to keep this open until @neelance gets a chance to see this.
I suspect there are API change opportunities to consider here, ones that could make the user experience better.
For example, perhaps js.Call should recognize callback.Callback type and report a better error. Or maybe it should extract its Value automatically (and then Value potentially be made unexported).
Given this is still a WIP, these things may already be on @neelance's mind and in future plans. :)

Good points all, I'll reopen it and allow @neelance to choose what to do :).

Good feedback! Now that I merged the callback package into syscall/js I was able to allow Callback to be passed to Value.Call directly and make Value unexported.


Changes made to the Order section to be merged with the upgraded site

order.css had some few styling different my code and main branch code, one example being flex-direction set to 'column' in main and 'row' in my code, due to which the page was  askew. Fixed it.

Hey @Shybymh  please create the  PR to UI-Improvement branch not the main branch . Also resolve the conflict and test out the working of code before pushing it !! last time your code broke the main repo , which i just fixed !! make sure you always pull the repo before you create the PR


Adding private network to VMs

Hi,
I managed to get your plugin running. Great work!
My xenserver has an additional private network named "External". My VMs need access to this network using DHCP to access the internet. How can I add this network to my VMs automatically? I tried config.vm.network "private_network", type: "dhcp"
but this does not add anything.
I use Host Internal Management Network (himn) for accessing the VMs and rsync for provisioning (eth0). So I need to add an additional network interface (eth1).
Thanks for your support!
Markus

Hi,
thanks, this worked for me!
I wish you a great new year.
Best wishes
Markus


nps confirming w/ a prompt

I through this on stack overflow last summer, and it remains unanswered.
https://stackoverflow.com/questions/50770212/adding-a-inquirer-confirmation-prompt-to-nps-package-scripting
essentially, how do I have nps call a prompt that calls a nps script? I get the error about scripts resolving to strings.
Thanks!
Somewhat relates to https://github.com/kentcdodds/nps/issues/123

Hey Falieson, thanks for the issue.
Would it be possible for you to make a wrapper script with inquirer.js and trigger release on callback? Related: this https://github.com/SBoudrias/Inquirer.js/issues/684
At least for nps, this sounds like a feature request to support an async function as a script. It's something that we've discussed in the past and have decided not to implement, to avoid overcomplicating nps, see #184. Sorry about that.


SPM seems able to grab source from the target repo. Maybe we can rewrite the debug path to that?

Debug info points at absolute paths. That's why building from source is generally better for visibility. However, SPM grabs the repo source and adds it to Xcode. You can browse, but not debug. If we can override the paths, we might be able to have the equivalent of "source jars".

We've figured this out. Updates soon...


Copy disk labels to snapshot labels

Added option to copy labels from a disk onto the snapshot labels

Hi @timwsuqld ,
Thanks for the PR - can you give one-liner on what you've added?
Also, would be great if you could update the README with your new option.
Cheers,
Jack

@jacksegal  I've pushed another commit. Basically, we just list all the labels of the source disk, and apply those to the snapshot. Useful for filtering snapshots based on the original disk labels. (We actually backup our snapshots with another tool, so having the labels help us find the snapshots we actually want backed up)

Thanks @timwsuqld - looks good to me!


Add message_helpers.py

Fixes #49

Question: should we optionally run the validator at the end of the function?

Question: should we optionally run the validator at the end of the function?

I don't think we need to since we have these independent unit tests


Running Node.js Bot Builder Framework with Luis.ai behind a corporate firewall.

I'm trying to build a chatbot POC for my team, but I keep running into an authorization issue with Luis.ai when I try to test locally. I am assuming it stems from a proxy issue, but on my end I've set my npm config to those dictated by my company. The issue still pertains so I'm assuming the proxy settings are being ignored causing the traffic to be blocked.
I've tried what this user did, but to no avail.
Is there a standard fix for this issue?

Hi @phillykid
It's hard to say without knowing a bit more. Are you able to debug your bot to find out where exactly the exception is happening?
Would you be able to test with a simple node application (not bot) that makes a call to LUIS, to see if that is correctly utilizing the proxy setting?
Basic, simple query such as:
https://<region>.api.cognitive.microsoft.com/luis/v2.0/apps/<LuisApplicationID>?verbose=true&timezoneOffset=0&subscription-key=<subscriptionkey>&q=hello
Also, as you mentioned authorization, please make sure you have the correct region/subscriptionkey/applicationid, and that it is running against the correct resource.

I tested with postman and the call comes back with json, but I tested on a node application(non-bot) and I get the error ENOTFOUND.

Hmm, interesting. So that seems to me that the node applications are not properly using the proxy server set in the node configuration. Do other requests (not to luis) from node application (non-bot) use the proxy settings and work successfully?

Well it seems that all non .corporate.com traffic is getting axed when running from node. I added my proxy settings directly into the API calls using the option in the requests library and it solves the issue for getting responses back from outside the network. How would I go about adding the proxy settings for any Luis.ai request being made?

So, I found this:
https://stackoverflow.com/questions/18586902/node-js-global-proxy-setting/27825729#27825729
You could look to see if you could get that tunneling to work.
The issue with running in the bot is that currently, the framework ignores proxy. This is due to a dependancy on msrest.
The only other thing would be to try make the request outside of the SDK apis. i.e. request directly to LUIS endpoint, without using SDK mechanisms (but still from within the bot). Using something like axios or fetch.

I looked at the tunneling tool, but it was not working. I was able to find the more updated version called global-agent and that allowed me to make direct API calls to Luis without having to specify the proxy in the request itself. The Luis SDK is still being blocked so will have to look into making the request outside of it.

Yeah, unfortunately, that's your option for the moment. Keep an eye on that bug and perhaps this one as well.


homebrew update to 2024.7.16 broke yt-dlp

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Updated yt-dlp on my apple silicon mac via homebrew which broke the whole yt-dlp cli leading to the same  python traceback error I posted below. So not even the debug info work at all. Also tried to update to master via yt-dlp command. But I get the same error.
I tried to delete and install it but no luck.
I tried to reinstall yt-dlp with --head so the whole thing would be installed from source but still, no luck same error.
Until the update to 2024.7.16 it worked without a problem now it does not work at all (and switching back to older verisons in homebrew is a bit of a pain):/
(dont wonder about the folder of the homebrew setup , homebrew is not installed in its default "/opt" dir but tried it on another mac with the default homebrew setup with the same outcome below...
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU
Traceback (most recent call last):
  File ".homebrew/bin/yt-dlp", line 5, in <module>
    from yt_dlp import main
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/yt_dlp/__init__.py", line 18, in <module>
    from .cookies import SUPPORTED_BROWSERS, SUPPORTED_KEYRINGS
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/yt_dlp/cookies.py", line 23, in <module>
    from .aes import (
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/yt_dlp/aes.py", line 5, in <module>
    from .dependencies import Cryptodome
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/yt_dlp/dependencies/__init__.py", line 65, in <module>
    import requests
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/requests/__init__.py", line 164, in <module>
    from .api import delete, get, head, options, patch, post, put, request
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/requests/api.py", line 11, in <module>
    from . import sessions
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/requests/sessions.py", line 15, in <module>
    from .adapters import HTTPAdapter
  File ".homebrew/Cellar/yt-dlp/2024.7.16/libexec/lib/python3.12/site-packages/requests/adapters.py", line 81, in <module>
    _preloaded_ssl_context.load_verify_locations(
FileNotFoundError: [Errno 2] No such file or directory

It looks like a problem with the certifi package. Have you tried running brew doctor?

`brew doctor
Your system is ready to brew.`

additionally tried to reinstall it, no change

FYI: the yt-dlp homebrew package is not maintained by us. If there is indeed a problem with the yt-dlp brew package, we aren't the ones who can fix it. You would want to report it to homebrew-core.

homebrew is not installed in its default "/opt" dir but tried it on another mac with the default homebrew setup with the same outcome below...

Per the homebrew docs, installing outside of the default prefixes it not supported.
The only thing I can tell you at this point is that the requests library is not able to find the CA certificate bundle provided by certifi, probably because it's looking in the wrong path(s) (possibly because of the non-standard brew installation).

Ah okay, then I will try my luck there, regading the default prefix, like I said, tried it on another mac with the default homebrew setup with the same outcome 🤷.
Thanks


Crystal Reports Parameters

Parsing of SQL used for an enterprise reporting team contains Crystal Reports Parameter Syntax. Added Token to stop parser from failing and will work on implementing the class instead of just skipping.
TOKEN: /* Crystal Report Tokens */

{
  	/* Parameter example {?MyParameter}*/
	<CRYSTAL_PARAMETER : "{?"(~[])*"}">
	/*Time Stamp Example {ts '2016-07-31 00:00:00'} -- */
|   <CRYSTAL_TM_STAMP : "{ts"<WHITESPACE>(["0" - "9"]){4}"-"(["0" - "9"]){2}"-"(["0" - "9"]){2}<WHITESPACE >(["0" - "9"]){2}":"(["0" - "9"]){2}":"(["0" - "9"]){2}"}">

}

Changes Unknown when pulling 56497af552c53a4fbd6c451f86d8a076a776f6a6 on master into ** on jjtree-start**.

Changes Unknown when pulling 56497af552c53a4fbd6c451f86d8a076a776f6a6 on master into ** on jjtree-start**.

Changes Unknown when pulling 56497af552c53a4fbd6c451f86d8a076a776f6a6 on master into ** on jjtree-start**.

Could it be that this pull request is corrupted?  So many commits, no diff possible...

Changes Unknown when pulling afe1011bdd646968adc73aa6830b3f3ef217ff07 on master into ** on jjtree-start**.

Changes Unknown when pulling 1dabcbc56f9b1589256a5d9f35c2904c2bbf70c8 on master into ** on jjtree-start**.

Changes Unknown when pulling 3e740100498aa22be0b334c4f39fe5fc7cbc7b76 on master into ** on jjtree-start**.

Changes Unknown when pulling 22c8b48fde56659c5609cb51c0b7ab71c1e67da8 on master into ** on jjtree-start**.

Changes Unknown when pulling 275fbbe87fa0adab9181a1fd80ba313c88f32f3c on master into ** on jjtree-start**.

Changes Unknown when pulling 9d680a6ec4b9f078f8b3da1d1bd7d9dff10086c1 on master into ** on jjtree-start**.

Changes Unknown when pulling 81de7a60dd1a9f36ee072b3f6a76aa375f93965f on master into ** on jjtree-start**.

Is this active anymore as a PR. I close it, due to messages I receive for every new commit.


KAFKA-16921: Migrate test of connect module to Junit5 (Runtime subpackage)

Migrate test of connect module to Junit5
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hi @chia7712 , @m1a2st
I have do some changes, PTAL 😄

Hi @chia7712, I have make some changes based on your example to keep StandaloneHerderTest has minimum changes during migration, PTAL 😸

Hi @chia7712, I have make some changes based on feedback, PTAL 🔹


feat: update DAB transactions

Signed-off-by: Tim Schmidt<EMAIL_ADDRESS>Description:
update DAB transactions
Related issue(s):
Fixes #1268
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Could you please provide files, which I can use for testing?


AIX 7.2 nodes unable to run extended.functional tests

I've disabled both nodes from running test jobs:
https://ci.adoptopenjdk.net/label/ci.role.testDISABLE/
See:
https://ci.adoptopenjdk.net/view/Test_functional/job/Test_openjdk11_j9_extended.functional_ppc64_aix/121/consoleFull
https://ci.adoptopenjdk.net/view/Test_functional/job/Test_openjdk16_j9_extended.functional_ppc64_aix/52/console

I suspect this is a dup of https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/2059 (although perhaps the title needs to be more generic!) ... I'm closing this - if you disagree, re-open


FR: option to escape hashtag in article

In Obsidian, inline hashtag is considered as valid tag. Thus, if an article I clipped contains hashtag, it would pollute my own tags space.
Also related to #159 .

Support this change also. Finding I have an insane number of tags due to hashtags in articles that are synced.

I ran into the same problem. It would be great if hash symbols, emojis, parentheses, and other clutter could be filter from the title while importing. It might be a too big ask, but having a regex replace for the title would be awesome.


Tracking links break pdf generation with docsifytopdf

Describe the bug
I get the following error when running npm run convert.
Alternatively, please generate a pdf in the repo. Thanks.
INFO:
Build with settings:
{
"pathToStatic": "static",
"mainMdFilename": "main.md",
"removeTemp": true,
"contents": "docs/_sidebar.md",
"pathToPublic": "./pdf/readme.pdf",
"pdfOptions": {
"format": "A4"
},
"emulateMedia": "print",
"pathToDocsifyEntryPoint": "."
}
ERROR:
run error
Error: ENOENT: no such file or directory, lstat '/home/me/generative-ai-for-beginners/01-introduction-to-genai/README.md?WT.mc_id=academic-105485-koreyst'
To Reproduce
Steps to reproduce the behavior:

Run 'npm run convert'
See error

Expected behavior
Generate pdf

.


User provided certs pushed from control. vars reorg

This PR

reorgs defaults/vars into private/public concept per comments from @sdodson
allows users to provide external facing certs for hawkular without needing to push them to the target

cc @mwringe @bbguimaraes

@bbguimaraes updated...

@mwringe: should we also support overriding heapster.cert, heapster.key,
heapster_client_ca.cert and heapster_allowed_users as the deployer
currently does?  I think you said there's not much sense in supporting some of
these, but I can't remember specifically which.

@bbguimaraes I think the discussion with @mwringe was there was minimal reason to allow users to provide heapster keys and certs because they are internal components where we can generate them.

@bbguimaraes updated.  removed the dependency on the named certificates since I didnt feel like it was required now (ever?).  Regardless of where the control machine is, the expectation will be ansible has access to the user provided certs and keys which seems reasonable; there should be no need to push them anywhere.  Please review and comment if I have resolved your issues.


BGP peering statement in this page is not entirely true.

Hi, I am from virtual wan support team. The information provided in this article about the BGP peeing between an NVA in a spoke VNET and secure hub is not entirely true.
"BGP peering between an NVA in a spoke VNet and a secured virtual hub (hub with an integrated security solution) is supported if Routing Intent is configured on the hub. BGP peering feature is not supported for secured virtual hubs where routing intent is not configured."
From our verification :

BGP peering between an NVA in a spoke VNet and a secured virtual hub (hub with an integrated security solution) is NOT supported if Routing Intent is configured on the hub. The BGP Neighbour will be up but routes exchange won't happen as the routing intent preventing vnet propagation routes into default routable ###

BGP peering feature is supported for secured virtual hubs where routing intent is not configured.
Both BGP session and route exchange work normally.

Please help review and update it internally.
Thanks.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9205b7a3-51f0-70fb-5125-97fba938f421
Version Independent ID: e9d061eb-38a3-808f-a11c-4246289a64c3
Content: About BGP peering with a virtual hub - Azure Virtual WAN
Content Source: articles/virtual-wan/scenario-bgp-peering-hub.md
Service: virtual-wan
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@robrong Thanks for your feedback. I have asisgned this thread to the author for further review.

#assign:cfields475

#assign:wtnlee

Data path does not work as expected. Closing issue.

#close

#please-close


Contact. Auth.

Fix bugs in auth
Make contact reading permission
Put contacts (who is sign up) in database firebase
Init Contact fragment

Some improvements
Fix bugs in authentication
Make contact reading permission
Put contacts (who is sign up) in cloud database
Init Contact fragment


rspamd: 1.9.2 -> 1.9.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/rspamd/versions.
meta.description for rspamd is: '"Advanced spam filtering system"'.
meta.homepage for rspamd is: '"https://rspamd.com"
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.9.4 with grep in /nix/store/nx9bhlq1x39f9xcx37zxr1h6byd4bd12-rspamd-1.9.4
directory tree listing: https://gist.github.com/67ba0a66636839dd821e067a53a0ee81
du listing: https://gist.github.com/9657b53ce57d7ba3f4ae460e867b6eb5

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
rspamd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nx9bhlq1x39f9xcx37zxr1h6byd4bd12-rspamd-1.9.4 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A rspamd https://github.com/r-ryantm/nixpkgs/archive/af6001a0898cf55a120d2619cb61fdf4f19aa686.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nx9bhlq1x39f9xcx37zxr1h6byd4bd12-rspamd-1.9.4
ls -la /nix/store/nx9bhlq1x39f9xcx37zxr1h6byd4bd12-rspamd-1.9.4/bin

cc @avnik @fpletz for testing.

Built and tested locally :+1:


Fix point cloud crash in IE

This fixes a crash in IE where the tile_translucentCommand uniform is discovered by gl.getActiveUniform but is not included in the uniform map. The fix is to to not declare tile_translucentCommand if handleTranslucent is false, which is the case for point clouds.
Chrome and Firefox must be smarter about realizing the uniform was not actually being used, so this bug did not surface on those browsers.

@lilleyse, thanks for the pull request! Maintainers, we have a signed CLA from @lilleyse, so you can review this at any time.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

Looks great, thanks @lilleyse!


Bridge.Testing.dll not found, can't build projects

Hi @Meai1!
The Bridge/Tests project(s) should be unloaded for now. Those projects require a private Bridge.Testing package. That package will be available very soon. For now you'll need to exclude the projects from the Solution.

Please use Bridge.NoTest.sln to load the Solution without the Testing projects.
See #1718.


ReportActionItemSingle cannot read property replace of undefined

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Request money from a new user
Notice the app crashes

Expected Result:
App doesn't crash
Actual Result:
App crashes
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub

We can close this issue since we reverted the offending PR - https://github.com/Expensify/App/pull/22303


Clean up Python 2.x compatibility code

six package, other compatibility packages like backports and mock, __future__ imports, and so on.
Related: Issue #186 (using Python 3.x-only features)

Completed by PR #383.


Error from watchers not captured

Version
2.6.10
Reproduction link
https://jsfiddle.net/shenron/kbuymoz0/
Steps to reproduce
Create a watcher, and reject a promise inside.
What is expected?
Vue should capture the reject with errorCaptured
What is actually happening?
The reject has not been captured

Duplicate of https://github.com/vuejs/vue/issues/10009


Rename occurences of Kubefed to KubeFed

Per the Wednesday May 15 WG meeting consensus.
cc @abrennan89

@pmorie Is this to be changed in docs only or in the source code too like function names?

@Jaanki I have an open issue for this in docs; https://github.com/kubernetes-sigs/kubefed/issues/891

@abrennan89 I was thinking to take this up. I see #891 is not assigned to anyone. If you are not planning to work on it, I would be happy too :)

@Jaanki This is already being worked on as part of #912

The PR author is not in the ks org so assignment was not possible.

Superceded by #936


Update tutorials for Nov 2021 package update

The tutorial notebooks need to be updated to reflect the refactor and illustrate the use of new functionality.

The quickstart tutorial has been updated. I may not have time to get to the other tutorials soon, but the main functionality is demonstrated there.

Tutorial 1 (covering basis frequencies, non-negativity constraint, and initialization) has been updated.

Tutorials 2 and 3, covering extrapolation and outlier detection, have been updated.

All tutorials updated


Render .txt files inline

Let's show the content of text files in the page!

I completed this feature here https://github.com/feross/instant.io/pull/44

Merged - thanks!


Global sync fix

fixes #13. The Master shares its delta to all slaves

Seems fine to me, merging.


fix #471 again-Service check TTL

Though #471 had been closed, I think that the fix was not enough.
The real reason is, when the service was registered, the TTL was not set.
So we need to set TTL when register the service while RegisterHealthCheck() is false and heartbeatProperties.isEnabled() is true.
The settings are:
spring.cloud.consul.discovery.register-health-check=false
spring.cloud.consul.discovery.heartbeat.enabled=true
I had fixed it with my SpringCloud Finchley, and copied the changes to the master branch.
And I find that function createCheck() had been fixed with moving HealthCheckCriticalTimeout to the top of that function.
But there are still two places needed to be fixed!

Why do you keep closing and opening new PRs?

There are ci errors so I had to resubmit it. I'm sorry for it.

No you don't. Pushing a new change should trigger CI and we can to. Resubmitting is worse.


add basic support for an intuitive JSON based route format

I mentioned the possibility of adding JSON support to CelesteMagicTimer in a comment on a previous PR. This adds proof of concept support to celeste_timer.py by accepting a JSON route in the new format and automatically generating a route that works with the existing code.
I'm curious if you have any thoughts about the JSON format I came up with. It seems extremely intuitive to me:

Splits are listed in the file in the order that they actually appear in the livesplitter.

The "level" concept is a built-in natural result of using JSON, instead of using a list with a "level" value. Splits can be nested arbitrarily deep.

All the splits are given in the JSON itself, rather than (awkwardly, IMO) combining the last subsplit of each split with the split itself. No more split.names = ["City", "Chasm"] kind of stuff.

Because it's just text, it's a much easier format for most users to edit to meet their own needs. It would also be easier for someone to write a route editor designed around the format even if they weren't familiar with the internals of this program.

Again, though, this is just a proof of concept. I think if it seems nice to you and works well it might be worthwhile to write a version of celeste_timer.py that makes it the default, and adds support for text based PB files and so on. I think the natural hierarchical syntax might allow for a cleaner timer implementation.
Might also be worth coding something that can avoid using eval for common triggers (chapter endings, room entrance, etc etc).

I added a timer built specifically to use the JSON route format. It seems pretty solid although I'm not happy with every aspect of it. I think it might be reasonable to merge something like this, so let me know what you think if you have a look at it. I tried to comment the timer very extensively to make it easy to maintain for someone who's not me (and therefore not familiar with the code).
It does everything I need, but I've left a couple of FIXMEs in for possible improvements.
Let me know if you like this and I can make any final changes or squash the commits together if you want.

I'm super unsure how to feel about this one. I think the idea is good, but I feel real uncomfortable inheriting all this code which makes very different design decisions than the ones I made. I see you've still got it marked as a draft, so I'll let you keep working on it while I work on making the pickle-based routes more intuitive.


Temporary UoM Vocabulary for UAT

A hastily assembled skos Units of measure vocabulary
This is so UAT can continue and select values to input results.
This file should be overwritten or removed once we have the correct vocabulary but this is essential to move forward with UAT and system development.

No Longer Applicable


Test "Blocking additions test should be able to load a lot of redundant triples into limited memory" sometimes fails

Very rarely, unclear if related to fastutil map change.

If it does not return assumed fixed by https://github.com/uzh/triplerush/commit/798a37766dd0e634fddf08f3c83492621d887c55

Assuming fixed by https://github.com/uzh/triplerush/commit/798a37766dd0e634fddf08f3c83492621d887c55


unix: reset signal disposition before execve()

Signal dispositions are inherited by child processes.  Libuv itself
does not touch them (if you don't use uv_signal_start(), that is)
but the embedder might and probably does in the case of SIGPIPE.
Reset the disposition for signals 1-31 to their defaults right before
execve'ing into the new process.
But caveat emptor:

This does open a race window where blocked signals can get delivered
in the interval between the pthread_sigmask() call and the execve() call
(and may end up terminating the process)

Fixes: nodejs/node#13662
CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/294/

Would it make sense adding an option so the user can decide what behavior it prefers (just in case there's people relying in the current behavior)?

Yes.  I was going to propose that if people weren't on board with this.

+1. We've been carrying a similar patch in Julia for years.

@libuv/collaborators Can I get a review?  Your opinion on whether or not to add a flag is welcome too.
(I'm currently leaning towards not adding a flag but happy to do it if everyone feels it's a good idea.)

Rebase + new CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/308/ - all green and landed in d0a27ba...11563e1.  Just to be on the safe side I did a local integration test with node.js and all was green there too.


Fix Rails default log level

This changed in https://github.com/rails/rails/commit/229fd2a02fc694b4b7756445b6647777aa94e25d - the advice about changing your log level still stands though.

Thanks!

Thanks!


Uncaught Error: Call to private Dotenv\Repository\Adapter\EnvConstAdapter::__construct() from context 'Illuminate\Support\Env'

This is the full error I'm receiving
 Uncaught Error: Call to private Dotenv\Repository\Adapter\EnvConstAdapter::__construct() from context 'Illuminate\Support\Env' in /var/www/vendor/illuminate/support/Env.php:58
I have a project using slim (it's not a laravel project), where I have a public/index.php file that has the following piece of code
require_once __DIR__ . '/../vendor/autoload.php';

$dotenv = Dotenv\Dotenv::create(__DIR__ . '/..');
$dotenv->load();

So I go up one folder since my index.php file is in the public folder.
Am I missing something?
Thank you

This can only happen if you have an incompatible version of illuminate support and dotenv.

If you have Laravel 8 components, you need ^5.2 of this package, and if you have Laravel 7 components, you need ^4.1.

Or Laravel 6 LTS needs ^3.3.

@GrahamCampbell  I'm not using Laravel at all, does this package work with just vanilla php? (or slim framework, the one I'm using)

I'm not using Laravel at all

But it seems you, or something, is at least a Laravel component which is incompatible (i.e. outdated).

Right, Good point let me see who included Illuminate, and as @GrahamCampbell  said I'll match the correct version of this package. Thanks!


Fixes #141 - Garbage collector to delete performance data older than 14days

Tested the changes locally and removed leftover. Merging it


Entity type "Dataset" should probably be "Assay"

Here are the entitytypes in the ES I have access to:
    - key: "Sample"
      doc_count: 2462
    - key: "Dataset"
      doc_count: 173
    - key: "Donor"
      doc_count: 76

curl -XGET "http://localhost:9200/entities/_search?format=yaml" -H 'Content-Type: application/json' -d'
{
 "size":"0",
 "aggs" : {
  "my-aggregation" : {
   "terms" : { "field" : "entitytype.keyword" }
   }
 }
}'

I suspect Dataset is what we at Harvard are calling Assay.
@ngehlenborg : I think the concept of Dataset matters to you, and this isn't what you have in mind? Is it important to change this, or could it be left as is?

@cborromeo @shirey We need to talk about this either make changes in the neo4j or the script.

@mccalluc Dataset is a more generic term.  A dataset may not always point to data generated from a single assay- for example a dataset that is derived from other, multiple datasets.  It also allows for multiple assays to be part of one dataset (not that we particularly want to do that, but we can if we would ever need to)
@etlds FYI

That works for me; If we do add 'Assay' to ES, give me a heads up? Thanks.


How to control the SVG path dimensions?

I understand this is a generic SVG question, though I am assuming a few people will struggle with the same query when using qr-image.
The SVG output produces a path and size, e.g.
{ size: 29,
  path: 'M0 0h7v7h-7zM9 0h5v1h1v-1h1v1h2v-1h3v2h-1v-1h-1v1h1v1h-1v1h1v1h-3v3h-1v1h-1v-1h-1v-2h1v1h1v-1h-1v-1h1v-1h1v-1h1v-1h-1v1h-1v1h-2v-1h1v-1h-1v1h-1v-1h-1v-1h-1v1h1v2h-1v-1h-2v2h1v1h-1v2h-1v-6h1zM22 0h7v7h-7zM1 1v5h5v-5zM23 1v5h5v-5zM2 2h3v3h-3zM24 2h3v3h-3zM20 3h1v1h-1zM11 5h1v1h-1zM10 6h1v1h1v-1h1v3h2v1h1v1h-2v1h-1v-2h-1v-2h-1v2h-3v-1h1v-1h1zM18 6h1v1h-1zM20 6h1v2h-2v-1h1zM0 8h1v1h-1zM2 8h5v1h-2v1h-1v-1h-1v1h-2v-1h1zM17 8h2v1h-2zM22 8h5v1h-1v1h-1v1h1v1h-2v1h-1v-1h-1v1h1v1h1v-1h2v2h-1v1h-1v1h-1v-2h-1v-1h-1v2h-1v-2h-1v-1h2v-2h3v-1h-5v-1h3zM16 9h1v1h-1zM27 9h2v1h-2zM0 10h1v1h-1zM6 10h1v1h-1zM11 10h1v2h-1zM17 10h1v1h-1zM26 10h1v1h-1zM1 11h2v1h-1v1h-1zM4 11h1v1h2v-1h2v1h1v1h2v-1h1v2h-1v2h-1v-2h-1v1h-1v1h-1v-1h-2v-1h3v-1h-3v1h-1v1h1v1h-1v4h-1v1h-2v-1h1v-1h1v-1h-1v1h-1v-1h-1v3h-1v-4h3v-1h1v-1h-2v-2h1v1h1zM16 11h1v1h-1zM19 11h1v1h-1zM14 12h2v1h-2zM17 12h2v1h-2zM26 12h2v1h-2zM16 13h1v1h-1zM28 13h1v1h-1zM15 14h1v1h1v1h3v2h-1v-1h-4v-1h-1v-1h1zM18 14h1v1h-1zM27 14h1v1h1v1h-1v1h1v1h-2v-1h-2v-1h2zM1 15h1v1h-1zM6 16h1v1h-1zM12 16h1v1h-1zM21 16h1v1h-1zM7 17h1v1h-1zM9 17h2v1h-1v1h-1zM13 17h2v1h1v1h-2v1h-1v1h-1v-2h1zM22 17h1v1h-1zM24 17h1v1h-1zM6 18h1v1h-1zM11 18h1v1h-1zM18 18h1v1h-1zM25 18h2v1h-2zM7 19h1v1h2v1h-1v1h-1v-1h-3v-1h2zM19 19h3v1h1v-1h2v1h2v-1h1v2h-1v1h-1v-1h-1v3h2v1h-7v-1h-1v-2h1v-2h-1zM14 20h2v1h-2zM17 20h2v1h-2zM10 21h2v1h-1v2h-1v1h-1v2h-1v-4h1v-1h1zM16 21h1v1h1v1h-3v-1h1zM21 21v3h3v-3zM28 21h1v1h-1zM0 22h7v7h-7zM22 22h1v1h-1zM27 22h1v2h-1zM1 23v5h5v-5zM14 23h1v1h-1zM2 24h3v3h-3zM11 24h2v1h-1v1h1v-1h2v1h-1v1h-1v2h-1v-2h-1v1h-2v-1h1v-2h1zM15 24h3v1h-3zM18 25h1v1h1v1h-2zM27 25h2v1h-1v2h-1zM15 26h1v2h4v1h-5v-1h-1v-1h1zM21 26h5v1h-1v1h-1v1h-2v-1h1v-1h-1v1h-2v-1h1zM8 28h1v1h-1zM26 28h1v1h-1z' }

The size is equal to the size of one module.
But what if you want to display QR code of a particular size, e.g. 200x200px?

I might be wrong here, but the approach I've taken is to use the module size to define a viewBox of an SVG path. Then you are basically left to size svg element in whatever way you want.
I have created a gist https://gist.github.com/anonymous/68bb278cb4945ba645fdb61105201e83.

Created this issue for others record (and just in case my approach is not optimal; @alexeyten please correct me in that case).

29 is the size on entire picture in modules. Since SVG is scalable there is no pixels until you want to rasterize it.
Here is SVG image with exactly 200px width and 200px height.
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 29 29" width="200" height="200">
    <path d="M0 0....z"/>
</svg>


Styles: Icons

[ ] Make icon library searchable.
[ ] Demonstrate color and size changes.
[ ] Provide zip of all icons for designers
[ ] Remove link from icon name in Icon Library like:
zoom-out.svg
Download

Un-assigning myself so my Issue list is current. Feel free to assign to me when/if it leaves the backlog.

Closing, added details and consolidated with https://github.com/cagov/design-system/issues/779


llm-vm not running, though pip install method

When I install llm-vm though pip:
pip install llm-vm

And I try to run something like this:
from llm_vm.client import Client
model_name = "bloom"
client = Client(big_model=str(model_name))
response = client.complete(prompt=prompt, temperature=temp)
print(response)

I get the following error:
  File "/home/mehmet/SandBox/_git/internet-hosted/github/ANARCHY/benchllm/env/lib/python3.10/site-packages/llm_vm/completion/data_synthesis.py", line 57, in DataSynthesis
    @backoff.on_exception(backoff.expo, openai.error.RateLimitError)
AttributeError: module 'openai' has no attribute 'error'

:point_up: I believe this error was fixed with this PR https://github.com/anarchy-ai/LLM-VM/pull/392, so I think the issue here is maybe the PyPi package was not updated.
But, after manually fixing this error I ran into this error:
ModuleNotFoundError: No module named 'hnswlib'
/home/mehmet/.pyenv/versions/3.10.4/lib/python3.10/tempfile.py:837: ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/tmp/tmpjo_8rwvu'>
  _warnings.warn(warn_message, ResourceWarning)

I fixed this issue by installing the following package:
pip install hnswlib

We need to fix this issue. I think the root cause is because the PyPi package has not been updated.

Hey I thought I had fixed this but I guess my branch wasn't up to date. Just pushed a new build with the changes. Kindly confirm if it works for you

Just tested it, it works now. Thanks for the fix Vik


Warning: This synthetic event is reused for performance reasons.

Receiving
Warning: This synthetic event is reused for performance reasons. If you're seeing this, you're accessing the property `clientX` on a released/nullified synthetic event. This is set to null. If you must keep the original synthetic event around, use event.persist(). See https://fb.me/react-event-pooling for more information.

When using onDrag in drag from outside example

Still an issue


[TextField] When changing font size, the text is not aligned.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Similiar Issues:
https://github.com/mui-org/material-ui/issues/8017
Current Behavior 😯
When overriding font size, the text does not align correctly.
Expected Behavior 🤔
Text should be in the center, there should be a prop accepting font size. Currently, it's using transformation to move the label upwards on click. The coordinates do not account for different font size.
Steps to Reproduce 🕹
Here's my following code:
const theme = createMuiTheme({
  typography: {
    fontFamily: 'Averta',
    fontWeightRegular: 500,
  },
 overrides: {
    MuiTextField: {
      root: {
        fontSize: '20px'
      },
    },
    MuiInputLabel: {
      shrink: {
        fontSize: '20px',
      },
      root: {
        fontSize: '20px',
      },
}
});

an example of such text field would look like:

As you can see, the text isn't aligned, it's meant to be aligned at 12px. To exaggerate the problem, look at the following:

It's even more obvious with larger fonts.
Context 🔦
I'm trying to change the height of the textfield component so that it matches our figma height and it's font-size (56px in height) and 20px font size. I assume it's also a problem to align with just using inputbase and outlined input.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.5

React
16.13.0

Browser
chrome-latest

TypeScript
3.8.3

@OriginLive Did you try to customize the transition?

@oliviertassinari could you give an example of how to override a label transition and how to center it correctly? Here's an example in codesandbox:
https://codesandbox.io/s/busy-pare-oj517

@OriginLive Definitely! Here we go:
const theme = createMuiTheme({
  typography: {
    fontFamily: "Averta",
    fontWeightRegular: 500
  },
  overrides: {
    MuiOutlinedInput: {
      root: {
        fontSize: "20px"
      }
    },
    MuiInputLabel: {
      root: {
        fontSize: "20px"
      }
    }
  }
});

https://codesandbox.io/s/lingering-haze-hnop1

@oliviertassinari
thanks, however that example also changes the height of the field. The idea here is to have a 20px font in 56px height.

@OriginLive Ok, updated

@oliviertassinari can you explain how you got the 19px in "translate(14px, 19px) scale(1)" for future reference please.

19px is just because it feels right, in #8017, I think that we come to the conclusion that we can't easily write CSS that display great value for any potential configurations.

I assume padding is the same? Damn, it would be nice to have that calculated somewhere.

The padding vaue is less important, I couldn't get a "round" line-height value based on the font and font-size so I went with something that would be just right below 56px, and fixed it with an explicit height CSS property.

@oliviertassinari just to let you know, there's also a way to use:
    MuiInputBase: {
      root: {
        fontSize: '20px',
        lineHeight: '23px',
      },
      input: {
        height: '0.95em',
      }
    },

instead of
MuiOutlinedInput: {
      input: {
        padding: "16px 14px",
        height: 56,
        boxSizing: "border-box"
      }
    },

part.
The em part scales better i believe. Not quite sure how it affects everything else, but i feel like border-box with height seems a bit hackish.


[Docs] Document how to setup AEM 6.3 indexes to be case-insensitive

In AEM 6.3, indexes can be configured to be case-insensitive. We should document how this can be done for the damAssetLucene index so restrictions can be case-insensitive.
See https://issues.apache.org/jira/browse/OAK-3574 for details

Let Oak docs take care of this


How to set key color of a *ebiten.Image?

How to set a key color for a *ebiten.Image? A key color normally used to set image background transparent. Thanks.

Hi,
Ebiten treats only 32bit-colored images, Ebiten doesn't have such notion.
https://golang.org/pkg/image/#NewPaletted A paletted image can have a transparent key color, and Ebiten's NewImageFromImage can accept this image.

Thanks for your reply, this is what i did currently. But can I do it with ColorM or Blend?

Yes. ColorM can be applied to any Ebiten images at DrawImage.

Can you please provide some example. I'm trying to write tool work with game archives from old game, and it uses png image without transparency. I would like to be able to render UI for that game based on ui configs.

Get an image image.Paletted
Replace an index in color.Palette with a transparent color
Pass the image to ebiten.NewImageFromImage(https://pkg.go.dev/github.com/hajimehoshi/ebiten/v2#NewImageFromImage)


Reduce the latency of casting Array to ndarray

resolve #63
Use NTuple instead of ShapeType(Union of Tuple), this reduces most of latency:
julia> x = rand(2,2);

julia> @time py_cast(Py, x)
  0.272697 seconds (626.84 k allocations: 34.489 MiB, 4.73% gc time, 61.58% compilation time)
...

Codecov Report

Merging #64 (3549d52) into main (87a15e9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   60.76%   60.76%
=======================================
  Files          11       11
  Lines         915      915
=======================================
  Hits          556      556
  Misses        359      359

Impacted Files
Coverage Δ

TyPython/src/CPython.Julia.jl
95.45% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


🐛 Bug Report: executeShellCommand does not write to stream correctly.

📜 Description
During the research I performed to create this ticket, it's unclear whether this is a backstage issue, or a winston logger issue.
When I use the executeShellCommand in my custom action, and I pass the logger as the writeable stream, I do not get anything on the writeable interface.
👍 Expected behavior
It should write to the stream that is passed in so that I can see the stdout and stderr of the command.
👎 Actual Behavior with Screenshots
It actually does not write to the stream. Instead, I get a console log:
[winston] Unknown logger level: undefined
Which led me to look into how the stream is written, adjusting the code in the plugin to use .info, instead of .write logs the stream via an info log, but I don't think that's accurate.
👟 Reproduction steps
Implement the executeShellCommand in a custom module. Ensure you pass the context's logger to the child process in the options parameter. Here is how I have mine configured:
        const result = await executeShellCommand({
          command,
          args,
          options: {
            // shell: true,
            cwd: ctx.workspacePath,
          },
          logStream: ctx.logger,
        });

Use this custom module in a template and use a simple command like this:
    - id: run-echo
      name: echo
      action: custom:execShellCommand
      input:
        command: echo
        args:  [
          "Hello World!"
        ]

See that the output of that command is not written to the logger as expected.
📃 Provide the context for the Bug.
No relevant context.
🖥 Your Environment
Backstage version "1.27.0"
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Are you getting any TypeScript error on this line? logStream: ctx.logger, those should not be compatible. You can use ctx.logStream, although that's deprecated so better do what the deprecation message suggests instead.

Hi @Rugvip ,
I'm also facing the same issue...ctx.logStream works fine but is deprecated. The deprecated message says  @deprecated — - use ctx.logger instead but when I use ctx.logger winston says the loglevel is missing [1] [winston] Unknown logger level: undefined... I'm a little lost here but will probably keep on using the logstream for now...

@elaine-mattos hey :wave: sorry I think we should have been a little more helpful here with the deprecation message.
ctx.logStream is indeed deprecated, and the alternative is to wrap up the ctx.logger in a stream yourself similar to this:
const logStream = new Writable({
  defaultEncoding: 'utf8',
  write(chunk, _encoding, next) {
    ctx.logger.verbose(chunk.toString().trim());
    next();
  },
});

Does this help? Where are is logStream being used? Maybe you can migrate to just using logger instead?

Ah darn, I think we should have deprecated that too when we deprecated the logStream so that you pass in logger instance instead and create the log stream in the executeShellCommand.
Do you want to raise a PR for this @elaine-mattos? :pray:

Sure!

@elaine-mattos hey 👋 sorry I think we should have been a little more helpful here with the deprecation message.
ctx.logStream is indeed deprecated, and the alternative is to wrap up the ctx.logger in a stream yourself similar to this:
const logStream = new Writable({
  defaultEncoding: 'utf8',
  write(chunk, _encoding, next) {
    ctx.logger.verbose(chunk.toString().trim());
    next();
  },
});

Does this help? Where are is logStream being used? Maybe you can migrate to just using logger instead?

Where is the class Writeable? I can't seem to find it..

@billyatroadie import { Writable } from 'stream' I think should do it


example: null not showing null

RapiDoc version 9.0.0, using example: null in API definition does not render null in the example section. For example:
  id_reference:
    type: integer
    description: ID reference
    nullable: true
    example: null

Shows 0 instead of null. I think it is a regression introduced in commit https://github.com/mrin9/RapiDoc/commit/86e9727a44e86cf7e6524a07d03876c46d6e7825. This line returns false if schemaObj.example is null, so example goes undefined and testing it for null does not work as intended.

thanks for reporting, will get to it soon


Bump Go version to 1.17

I would like to request to bump Go version to 1.17 in Knative project (probably after 0.26 release).
/kind feature
There is no rush but Go 1.17 needs //go:build instead of/in addition to // +build
so a lot of code need to be changed.
[1] https://golang.org/doc/go1.17

FYI @knative/technical-oversight-committee

cc @knative/productivity-wg-leads

Do we want to bump just the versions we build with, while keeping go.mod versions at 1.16 or do we want to make use of any new functionality of 1.17? If only the former, that's okay. Updating go.mod though might be a tad early at least for our (Red Hat's) productization pipeline. Dunno if others have similar constraints.

We'd just update the toolchain with go1.17 - each repo can mark their min go version the the go.mod themselves.


(tiller): Use grpc-go 1.3.0

https://github.com/grpc/grpc-go/releases/tag/v1.3.0 released with "separate incoming and outgoing metadata in context" (https://github.com/grpc/grpc-go/pull/1157)
IMO, grpc-go should be updated to 1.3.0 in which ever release Rudder changes are merged to ensure only necessary metadata is forwarded from Tiller backend to Rudder grpc service via context.
cc: @technosophos @nebril

https://github.com/grpc/grpc-go/issues/1178 explains the api changes in grpc-go package.

This has been bumped several times, so I am bumping to Upcoming - Minor. @adamreese would like your opinion on if this is a breaking change or something we can pull into a minor version. I think this should be just fine to pull in, but wanted to be sure


Delegate method "selectedDate(_ date: Date)" not called, if a day already selected.

I can't get a date, if a day already selected.
I think, nothing will be break if the method
calendarDelegate?.selectedDate?(day.date)
will call before the line
guard day.state == .available else { return }
original code:
` extension VACalendarView: VAMonthViewDelegate {
func dayStateChanged(_ day: VADay, in month: VAMonth) {
    switch selectionStyle {
    case .single:
        guard day.state == .available else { return }

        calendar.deselectAll()
        calendar.setDaySelectionState(day, state: .selected)
        calendarDelegate?.selectedDate?(day.date)

    case .multi:
        calendar.setDaySelectionState(day, state: day.reverseSelectionState)
    }
}

}`

STUCKED ON SAME.

hey I wanted to navigate my viewcontroller on selected date & I got sucked . but after did some check & edit .. finally I came to know....
when you have made a selection,
to multiple ,
in demo
//    calendarView.selectionStyle = .multi
& then,
on, VACalendarViewDelegate
you use ,
delegate methods ,
below,
// use this method for single selection style
@objc optional func selectedDate(_ date: Date)
// use this method for multi selection style
@objc optional func selectedDates(_ dates: [Date])
you will get action on both unselected & selected.
This is temporary & worked for me ,
hope it'll be useful for you also,
check my code.
&
MAIN THIS ONE IS IN LATEST DEMO I'VE DOWNLOADED ON "21ST AUG 2019"
check my code below,
extension scheduleViewController: VACalendarViewDelegate {
func selectedDate(_ date: Date) {
//print("selected Date-->",date)
let VC = self.storyboard?.instantiateViewController(withIdentifier: Constants.Class.ClientSchedualDialyVC) as!
ClientSchedualDialyVC
self.navigationController?.pushViewController(VC,animated:true)
}

func selectedDates(_ dates: [Date]) {
    calendarView.startDate = dates.last ?? Date()
    let VC = self.storyboard?.instantiateViewController(withIdentifier: Constants.Class.ClientSchedualDialyVC) as!
    ClientSchedualDialyVC
    self.navigationController?.pushViewController(VC,animated:true)    }

}


Clearer message for "Communities"

Suggestions from website review on 2021-08-21:

Options for how to improve "communities" on the menu:

[ ] Keep the intermediate page (https://marmaladeai.gatsbyjs.io/communities), but spread the three out horizontally and have an associated image and/or icon with each, so you really know you are going into your world.
[ ] Use dropdown/hamburger menu but expand "communities" to "freelance communities" (and associate an icon with each community).

Individual community pages:

[x] Drop the roadmap entirely. Competitors will look for that.
[x] Move the graphics down into the page, so you see the text about the solution above the fold.
[x] Move the "Behind the Scenes" section to a specific page on this, e.g. a “How does it work?” page.
[x] Digital marketing niche page: "How can Marmalade AI help me succeed in digital marketing?"

"How does it work?" page:

[ ] Take content from individual community pages.

-- 2021-9-26 (Craig: seems like we've done this by using the "keep the communities page" approach, and the addition of just a brief text at the bottom about expecting more after Early Access, seems to wrap it up?)

Fixed by #160


Request Self Assessment Link

Users should be able to generate a self assessment link. This will allow the beneficiary to complete it separately from the charity user. This is complicated because beneficiarys don't have app accounts.
Is blocked by #26.

add new route which takes the same arguments as the new meeting route
create a new meeting in the database (same routine as the new meeting route)
create JWT (using the method implemented in #25)
create link (maybe /meeting/remote/{jwt})
return link

 Self assessment (via link)


GitHub sources are checked out with git://

Hi there!
I saw this since the warning was introduced when using the git protocol to fetch dependencies with bundler. It's probably good to show this error message but I'm a fan of the github:  option when using gem.
Example:
gem 'something', github: 'dude/something'

This will write something like this into the lock file:
GIT
  remote: git://github.com/dude/something.git
  revision: e9a5a089355067232d61bb11475a5f4da13854d3
  specs:
    something (0.0.1)
      ...
      ...

So I think we should change the github option to use either https or even ssh to clone the repo. Because when I use the GitHub flag, I don't want to see the error message that I checked out the repo with the git protocol.
Cheers, keep up the good work :+1:

We have done this for 2.0, but it's a backwards incompatible change to make for 1.x. You can redefine the github git shortcut, or use the full git option in the meantime.

Wow, nice reaction time! :smile:
Oh I see. Thank you!

@segiddins What's the plan for the 2.0 release? Next year or even later?

As soon as we can get it finished, see https://github.com/bundler/bundler/issues/4856 for progress

Okay, thanks!
If I can help with anything, message me.


Fix preflight checks

Description
This PR fixes the HasLicense check failure in the preflight scans for csm-authorization-sidecar image in RedHat portal. This is necessary for RedHat to publish authorization sidecar image in its catalog
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Backward compatibility is not broken

How Has This Been Tested?
Preflight checks passes:

e2e pass

e2e pass


Undocumented caching-like behaviour

I'm seeing some strange caching-like behaviour in some of my tests. Can't seem to find it documented anywhere, and it seems to me like it's unintentional
The app that is being tested is a thin wrapper around a third-party user management software. It can only respond to PATCH requests, and setting a field in the body to null should delete that field from the user in the third-party system
The issue arises when I run these two tests right after one another:
###
# @title add subscription level
  // Randomly generated each key-value pair, so that we know the entries are from _this_ run
  const crypto = require("crypto")
  const value = `httpyac-${crypto.randomBytes(20).toString('hex')}`;

  exports.value = value
}}

PATCH {{$host}}/metadata/v1/
...defaultHeaders

{
  "userId": "<SOME_ID>",
  "access": {
    "subscription": "{{value}}"
  }
}

?? status == 200

{{
  const { equal } = require("assert");

  const user = await users.get({id:"<SOME_ID>"})
  test("got right values back: all fields updated", () => {
    equal(user.subscription, value)
  })
}}

###
# @title delete subscription field from user
PATCH {{$host}}/metadata/v1/
...defaultHeaders

{
  "userId": "<SOME_ID>",
  "access": {
    "subscription": null
  }
}

?? status == 200

{{
  const { equal } = require("assert");

  let user = await users.get({id:"<SOME_ID>"})
  user = await users.get({id:"<SOME_ID>"})

  test("subscription field deleted", () => {
    equal(user.subscription, undefined)
  })
}}

Doing this, user.subscription still retains the value as it was computed by the previous test, resulting in the following error:
subscription field deleted (AssertionError [ERR_ASSERTION]: 'httpyac-f750382b199e8a48093bf67cd0225fb35803e5d3' == undefined - /Users/audunmo/work/cloud-account-metadata/test/httpyac/delete.http:79:5

However, checking application logs, outgoing http responses, and the user management system, I can see that the deletion actually went through. This clued me in to that there may be some caching going on here.
Interestingly, performing any arbitrary javascript before the request causes the caching to be avoided, and the second test to pass as expected.
# @title delete subscription field from user
{{
  const x = 2
}}
PATCH {{$host}}/metadata/v1/
...defaultHeaders

{
  "userId": "<SOME_ID>",
  "access": {
    "subscription": null
  }
}

?? status == 200

{{
  const { equal } = require("assert");

  let user = await users.get({id:"<SOME_ID>"})
  user = await users.get({id:"<SOME_ID>"})

  test("subscription field deleted", () => {
    equal(user.subscription, undefined)
  })
}}

So for me, this seems like a bug. I'm assuming that what's happening here is that there's a proxy intended to cache responses from the app that's under testing, and that it's set as a global proxy. This intercepts all requests, and the check that I have outgoing to the third-party service is being "caught in the cross-fire" so to speak.
I don't love my workaround. Is there any other approach to doing this? @no-proxy on the test has no impact. Setting Cache-Control: no-store on the either the tests or the third-party client also does nothing

@audunmo httpyac already caches a few values between requests. This involves the variables that can be reused in the next request. but I think the cause is the logic behind users.get. How is this created?
The best would be a minimal example with e.g. postman-echo.com to reproduce the error. In the current example there are too many unknowns and noise:

not defined defaultHeaders and host
missing script start in line 2
is @title needed to produce bug? Is ?? status == 200 needed to produce bug?
logic of await users.get is unknown and why is it called two times in a row in second request?


Enabling optional Timestamp fields tests

This is done to test against PR#46 --- Ensure that Timestamp doesn't fail on null on optional properties #46 ---

testPortCallServiceTypeCodeField

testModeOfTransportField

testVesselPositionField

testEventLocationField

testEventLocationField

testFacilitySMDGCodeField

fix added to schema to allow nulling of vesselCallSignNumber.

Note: This not expected to pass the test on OVS master

Can we split this into two - the part that can be merged now and the parts pending a fix in OVS.

Can we split this into two - the part that can be merged now and the parts pending a fix in OVS.

Will do, waiting on a response.

This is deprecated as all tests on JITV1/OVS are enabled


Use Host IP address if PrimaryIP is empty

Description
Description
When host network is selected on rancher container, traefik returns as primary IP address "". This results in proxying to http://:PORT. With this PR if the IP address is empty, the returned IP address is host's address where the container is running.
Follow up description: #1498
This is a quick-fix and should be made differently in future. Probably it should start by detecting what network was chosen on a container and based on that return the proper IP address.

Thanks for your interest in Traefik ! 👍
You cannot manually edit files in the vendor.

Rule #1 of open-source: no is temporary, yes is forever.

— Solomon Hykes https://twitter.com/solomonstre/status/715277134978113536
We don't want quick-fix and in respect of this rule I must close your PR.
We will be very happy if you open a "non quick-fix" PR on master.


feature: enriched output with both speech and braille

In https://github.com/zorkow/speech-rule-engine/issues/438#issuecomment-727638329 I had asked how to "merge" speech and braille output.
As per discussion with @zorkow, while this approach should work for now there might be problems in the long run, e.g., when heuristics are configured differently for each format.
The long-term solution would be for SRE to provide an additional option to create enriched output containing multiple formats.

As per F2F, there shouldn't really be a problem as long as speech and braille are generated with the same heuristic setup (which is currently not directly confgurable anyway).
I would still be nice to have a convenient function as part of SRE to get both with a single API call.

As per F2F, there shouldn't really be a problem as long as speech and braille are generated with the same heuristic setup (which is currently not directly confgurable anyway).
I would still be nice to have a convenient function as part of SRE to get both with a single API call.

Just to record this here, too. I've found a simple example where this is no longer the case. Something like '(1\;2) will lead to incompatible tree structures since the braille heuristic for cycles kicks in and ignores the space whereas the speech heuristics makes the space a times; in particular the space gets a semantic-id in speech but not braille, messing up mapping the two trees afterwards.

I don't know if this should be filed as a separate issu but using my hacky way of combining trees I found an example where I have no braille but a speech label, e.g., \prod _{j\geq 1}(1-x^j)^{-1}, on the leaf of \prod.


PointCloud2 output in RViz is frozen and does not update

Camera Model: R200
Firmware Version: <IP_ADDRESS>
Operating System & Version: Ubuntu 16.04
Kernel Version (Linux Only): <IP_ADDRESS>-upboard
I am able to generate a depth and color image output, but the pointcloud2 output does not update in real time after less than a minute. It updates for a bit, but then the screen freezes and only the depth and color images continue to work. Does anyone know how to resolve this issue? Thanks.

[Realsense Customer Engineering Team Comment]
Hi @Clare-Meng
would like to know if you still need help with this issue?
thanks.

@RealSense-Customer-Engineering I have this exact same issue with the exact hardware/versions.
Camera Model: R200
Firmware Version: <IP_ADDRESS>
Operating System & Version: Ubuntu 16.04
Kernel Version (Linux Only): <IP_ADDRESS>-upboard
The pointcloud2 in rviz stops working after ~1min of running. It comes back if I restart the r200 nodelet, only to freeze again.
When reading the rostopic echo /camera/depth_registered/points, the data continues to output after the pointcloud2 freezes, but it eventually seems to get stuck (is there a delay in the data output?)
rostopic echo /camera/depth/points never stops.
Thanks, Daniel

Kindly let me know the resolution. I face the same. Similar configurations.

Precisely the same issue is occurring for me.  The color and depth image windows update, but the pointcloud2 window stops updating after about 30 seconds.  When I close rviz and reopen it, the pointcloud2 windows does not display anything.  I have ctrl-C the window where I launched roslaunch realsense_camera r200_nodelet_rgbd.launch and relaunch it to get another 30 seconds of point cloud viewing.
rostopic echo /camera/depth/points and rostopic echo /camera/depth_registered/points both put out tons of information to the screen.


e2e test for elasticsearch token propagation

Signed-off-by: Ruben Vargas<EMAIL_ADDRESS>
Which problem is this PR solving?

e2e tests for token propagation to ES

Short description of the changes

This is an integration test that make sure that the token reached ES storage.

To keep the number of jobs low, could we add this test to ES_INTEGRATION_TEST test. There are already other tests related to ES.

Ok, I addressed all comments, @yurishkuro thanks for the review!


TSpanView.java uses or overrides a deprecated API

Deprecated API used
> Task :react-native-svg:compileDebugJavaWithJavac
Note: ***/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/TSpanView.java uses or overrides a deprecated API.

Environment info
React Native Environment Info:
    System:
      OS: macOS 10.15
      CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
      Memory: 93.41 MB / 16.00 GB
      Shell: 5.7.1 - /bin/zsh
    Binaries:
      Node: 12.12.0 - /usr/local/bin/node
      npm: 6.11.3 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 13.1, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
      Android SDK:
        API Levels: 28, 29
        Build Tools: 28.0.3, 29.0.0, 29.0.0, 29.0.0, 29.0.2
    IDEs:
      Android Studio: 3.5 AI-191.8<IP_ADDRESS>77832
      Xcode: 11.1/11A1027 - /usr/bin/xcodebuild
    npmPackages:
      react: ^16.8.3 => 16.11.0
      react-native: 0.59.9 => 0.59.9
    npmGlobalPackages:
      react-native-ble-plx: 1.0.3
      react-native-cli: 2.0.1
      react-native-create-library: 3.1.2
      react-native-git-upgrade: 0.2.7
      react-native-permissions: 1.1.1

Library version: 9.13.3
Steps To Reproduce
Install app with react-native run-android
Describe what you expected to happen:
Library with no use of deprecated code

Hmm, I used to get that as well, but am currently unable to replicate it, here's my info:
react-native info
info Fetching system and libraries information...
System:
    OS: macOS 10.15.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 216.95 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 12.12.0 - /usr/local/bin/node
    Yarn: 1.19.1 - /usr/local/bin/yarn
    npm: 6.13.0 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
    Android SDK:
      API Levels: 27, 28, 29
      Build Tools: 28.0.3, 29.0.2
      System Images: android-28 | Intel x86 Atom_64, android-28 | Google Play Intel x86 Atom, android-29 | Google APIs Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>00203
    Xcode: 11.2/11B52 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.3 => 0.61.3
  npmGlobalPackages:
    react-native-cli: 2.0.1

node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/TSpanView.java: uses or overrides a deprecated API. Recompile with -Xlint:deprecation for details.
Receiving the same issue. New react-native project with the package installed.  Perhaps it has something to do with trying to minify with R8?