Support Authorization Code Grant

Thanks for the great work.  I have a Feature Request - I don't think it would take too much to support Authorization Code Grant with a Public Client (no client secret).  This seems to be recommended more these days over implicit for Single Page Apps.  Also, it would give the opportunity to implement a refresh token.  Thanks

Here are some supporting docs -

<EMAIL_ADDRESS>https://developer.okta.com/blog/2019/08/22/okta-authjs-pkce

I'd be interested in submitting a PR, but would like some guidance and where to the HTTP calls would live, what HTTP library to use, etc.


TypeError: unsupported operand type(s) for +: 'long' and 'NoneType'

Hello,
I am using Ghidra 9.2.3 and I got an issue when testing a binary with your script :

Thanks in advance,

Hi, Can you share the binary?

@meowmeowxw I just sent you it on your protonmail mail ;)

Received, ghidra2dwarf only works on ELF binaries not PE


Carousel buttons: tap targets are not sized appropriately

I guess we should put them a bit further from each other
https://pagespeed.web.dev/analysis/https-main--ororafresh--hlxsites-hlx-live/18wrg5sd25?form_factor=mobile

Fixed as per the Pull Requests - https://github.com/hlxsites/ororafresh/pull/95


Better CurrencyPair

Current CurrencyPair api can cause problems with really big or small factors. For example 1 Indonesian rupiah is worth 7.5×10-5 U.S. dollars (according to google). Should I really provide such factors? IMHO there should be api that takes two Money instances and take advantage for better calculation precision. This approach you can see on this site http://www.nbp.pl/kursy/xml/LastA.xml where 10000 Indonesian rupiah is worth 2.9904 Polish zloty (at this moment). You can think about it like there is TWO factors.

@mdrost Can you explain with causes problems?

Since the description of this issue is too limited, I am closing it. Because I can somewhat understand what you mean, I listed a possible solution to the list of ideas for the next major version.


run locally

I want to use this example locally without docker https://fastapi-keycloak.code-specialist.com/full_example/ but this give me this error message

client_secret = client > client_id > credential > Secret
admin_client_secret = admin_cli > credential > Secret

and please specify that fastapi_keycloak will work for new keycloak version 17
The error message is
(venv) (base) sana@sana:/office/My Projects/keycloak_implementation$ uvicorn main:app --reload --port 8001
INFO:     Will watch for changes in these directories: ['/office/My Projects/keycloak_implementation']
INFO:     Uvicorn running on http://<IP_ADDRESS>:8001 (Press CTRL+C to quit)
INFO:     Started reloader process [18160] using watchgod
Process SpawnProcess-1:
Traceback (most recent call last):
File "/usr/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/uvicorn/subprocess.py", line 76, in subprocess_started
target(sockets=sockets)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/uvicorn/server.py", line 60, in run
return asyncio.run(self.serve(sockets=sockets))
File "/usr/lib/python3.8/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "uvloop/loop.pyx", line 1501, in uvloop.loop.Loop.run_until_complete
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/uvicorn/server.py", line 67, in serve
config.load()
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/uvicorn/config.py", line 458, in load
self.loaded_app = import_from_string(self.app)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/uvicorn/importer.py", line 21, in import_from_string
module = importlib.import_module(module_str)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/office/My Projects/keycloak_implementation/./main.py", line 7, in
idp = FastAPIKeycloak(
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/fastapi_keycloak/api.py", line 129, in init
self._get_admin_token()  # Requests an admin access token on startup
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/fastapi_keycloak/api.py", line 288, in _get_admin_token
response = requests.post(url=self.token_uri, headers=headers, data=data)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/requests/api.py", line 117, in post
return request('post', url, data=data, json=json, **kwargs)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/requests/sessions.py", line 515, in request
prep = self.prepare_request(req)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/requests/sessions.py", line 443, in prepare_request
p.prepare(
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/requests/models.py", line 318, in prepare
self.prepare_url(url, params)
File "/office/My Projects/keycloak_implementation/venv/lib/python3.8/site-packages/requests/models.py", line 392, in prepare_url
raise MissingSchema(error)
requests.exceptions.MissingSchema: Invalid URL 'None': No scheme supplied. Perhaps you meant http://None?

Hi @bwithai,
It seems like the token_endpoint isn't a valid URL. Could you please paste the content of your Keycloak's OIDC config? (/auth/realms/master/.well-known/openid-configuration) for confirmation?
As for your question regarding Keycloak v17, there are no known issues with v17 yet. We didn't fully test it yet though.

Okay, so for now I don't think we can support Keycloak v17 with little effort without losing the backward compatibility. I have to suggest using Keycloak v16 for now

It's v16 but still token_endpoint give this error

please give me some clue if you know how to deal with it

I can't figure out your problem with this information. But I would assume you're having a Keycloak, not an issue with our package, we can't support you with Keycloak.


[stable/grafana] Fix TLS definition in Grafana ingress

Signed-off-by: Nicolas Hug<EMAIL_ADDRESS>
What this PR does / why we need it: fix TLS definition in Grafana ingress (https wasn't working)
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #6213
Special notes for your reviewer:

/assign @sameersbn


Building, Installing, Running... Getting past imagemagick's missing references

Hi I am attempting to build this on Ubuntu 20.04, and this happens:
root@Baileys:/home/paul/goswi/goswi-master# go build -tags no_pkgconfig gopkg.in/gographics/imagick.v3/imagick
gopkg.in/gographics/imagick.v3/imagick
/root/go/pkg/mod/gopkg.in/gographics/imagick.v3@v3.5.0/imagick/affine_matrix.go:9:10: fatal error: MagickWand/MagickWand.h:
No such file or directory
9 | #include <MagickWand/MagickWand.h>

  |          ^~~~~~~~~~~~~~~~~~~~~~~~~

compilation terminated.
I can do some env variables to suppress the error(s) in build, but install goes back to the same error.  And run doesn't, at all.
I guess it begs the question do you really truly need imagemagick just to convert profile pics? I read all your musings on the reasons for using it, but imagemagick is a disaster of a project to try to build from. Just my opinion though.
I did actually modify the #include to wand/magick_wand.h which is a valid include, but then the build dies on 3 type cast errors.
Have you got this project to build, and how? Is there anything I could help with? I don't know Go at all but I'm pretty good with other things and Go doesn't seem very scary as a language.
Regards

I got this to work on a fresh install of ubuntu 22 doing the following:
apt-get install golang-go

apt-get install libjpeg-dev libpng-dev libopenjp2-7-dev gcc g++ libltdl-dev

git clone https://github.com/ImageMagick/ImageMagick.git ImageMagick-7.1.1

cd ImageMagick-7.1.1

./configure --with-modules --enable-delegate-build --with-quantum-depth=16 --with-jp2 --with-openjp2=yes --with-jpeg=yes --with-png --with-png=yes

make

make install

apt-get install libmagickwand-dev

cd ..

export CGO_CFLAGS_ALLOW='-Xpreprocessor'

git clone https://github.com/GwynethLlewelyn/goswi.git

cd goswi

cp config.sample.ini config.ini

nano config.ini

nano main.go (Need to fix the ROBUSTserver variable to ROBUSTServer)

go run .

It would be helpful to extend the readme on actually how to start this with go run . as I suspect some might go for just starting main.go and that fails.
Not sure if that imagemagick install that way is even necessary, but I know that's how jp2 support is accomplished with it. Assuming the magickwand hands off to the system install of imagemagick it should work to show the profile images. Though really what you want for this is a separate service that just converts and caches the images somewhere so you can directly serve the png or jpeg files. On the fly conversion is not great as that's really heavy, you want to do that in a controlled way to not overload the system.

The warnings during startup about things not to be used for production or blindly trusting everything is probably not good for long term or production use as the warnings suggest. That's something worth looking into as security, as lackluster as it is with OpenSim, we don't want to make worse.
Seems a lot of effort for just a web panel, not just for setting up imagemagick, but the code in general. I gravitate to php for these things for the ease of use as it doesn't require generating a webserver from scratch. It's, if I see that correctly, a single application that runs as service, so how would it scale? Think rewriting some grid services or an asset server in golang might be a good option, not so much a web panel. Just a personal opinion.

My apologies for only replying today — I have way too many overlapping things to do, and I confess that this was not very high in my priorities.

Aye, 'some assembly required'. Or, rather, 'some Go knowledge required'. In other words: no, I'm not going to teach you how to get a working Go environment to get gOSWI compiled. It's not too difficult, though:

You can use any package manager, such as apt on Debian/Ubuntu Linux, or homebrew on macOS. Almost all will have a package called go or  golang.
If you're on Ubuntu Linux, I recommend using @longsleep's PPA: https://ppa.launchpadcontent.net/longsleep/golang-backports/ — Simon keeps a much more recent version of Go up to date (from the Debian equivalent) than the 'official' Ubuntu repositories
If all else fails: Go is surprisingly easy to get and install — have a look at their official installation page

The config.ini file configuration instructions are in the supplied config.sample.ini; they should be obvious to anyone who is used to configure OpenSimulator.
The connection to the database may not be obvious. It uses a Data Source Name (DSN) which was inspired by what PHP's PEAR::DB uses — more precisely, the 'advanced' format mentioned on the PEAR manual — just dropping the prefix, i.e.: [user[:password]@][net[(addr)]]/dbname[?param1=value1&paramN=valueN] (brackets indicate optional parameters; the parentheses are required to connect to a remote host and/or a local socket which is not the default one)
I've added that comment to the  config.sample.ini file as well.
Finally, the source code is provided 'as-is', which essentially means that there isn't any free support. Caveat utilitor.

Now, I'm aware that the major source of trouble is to get a working, functional copy of ImageMagick 7 on Debian/Ubuntu. Even though ImageMagick 7 has been released nine years ago (!), it still remains unsupported on those distros. However, there is a pre-compiled package for an upcoming Debian release (still 'under testing') which does work under Ubuntu as well: it has 'only' twenty (!!) dependencies that must be additionally downloaded, but then Ubuntu's apt will pretty much figure out what it additionally needs and pull those from its usual package repos.
In order to show how this works in practice, look at the Git Actions for this repository, which will build, from scratch, a working Go + ImageMagick 7 in order to compile gOSWI. Please note that, every time a Git Actions workflow is run, it starts with an empty, pristine Ubuntu installation and begins to install everything else. This means that within a certain timeframe (since everything will be continuously upgraded!), you can rest assured that it is possible to have everything properly configured to get it working.
The magic happens here — where you can see which packages have been added — and of course, by clicking on the Action itself, you can check what has actually happened and replicate it on your own system. Take into account that this will only work on Ubuntu 24.04 or an equivalent recent Debian/Ubuntu derivative distro.
(Note: macOS users have it much easier, since, at the time of writing, Homebrew supports ImageMagick 7.1.1-41 directly).
That said...
Indeed, this is a 'personal project' which was created with two main goals: allowing my grid users to have a quick way to change their passwords — because they keep forgetting them — and providing a 'splash screen' with a list of regions to directly teleport to, as well as a working map. Very secondarily, it also provides some statistics (mostly for statistic aggregators) and little more than that.
Since I don't have a huge grid to work with, a more distributed solution was not really required. Go apps are pretty good at launching gazillions of lightweight threads in parallel to deal with whatever is thrown at them; the question is mostly if you wish to deploy a distributed system to ease the load and to make the single point of access a bit more reliable. In this scenario, and assuming a transparent backend with its own built-in redundancy checking (namely, the MySQL database), there is nothing strictly forbidding to deploy several instances of gOSWI running simultaneously — assuming that the front-end load balancing system is able to deal with cookies (i.e., guaranteeing that the same HTTP stream is kept alive for the same user, until they log out or there is a network disconnect). A minimalistic approach, using DNS to provide round-robin distribution of load, and possibly having nginx acting as a distributed-load front-end server, as described by documents such as these or these  — I would personally recommend one of those hash-based solutions, but, again, YMMV.
I agree that 'this is just a webpanel'. It was never meant to be anything else! I'm also fond of PHP, of course, and I can agree that for 'just a webpanel', doing everything in PHP would have been easier. Or — perhaps not. I would very likely need to use one of those massive frameworks, such as Laravel or Symphony — both of which are challenging in their own way. Go has its own 'framework' built into its standard libraries; and I use Gin Gonic on top of that to make everything even easier to work with. In other words — there is not much work to do except grab some HTML/CSS and paste it on a handful of templates, and voilà, you have a full-blown web application in an instant, without really needing to 'learn' the complexities of one of the 'popular' PHP frameworks.
Regarding JPEG2000 and the usage of ImageMagick 7 for converting between formats... with that, I cannot disagree with your proposals, you're quite right on everything you mentioned: it's like killing a fly with an atomic bomb, since ImageMagick 7 is immensely complex and there is no port to Go, just a layer on top of the C/C++ libraries (and even that is a small nightmare to manage), which needs a 'special' feature of Go (known as Cgo) that allows Go to make calls to a C/C++ library 'as if' it were written in Go natively. The problem is that this bit of code needs to be compiled with a C/C++ compiler, and that is an additional layer of complexity of its own. Generally speaking, Go will have no problem in launching whatever C/C++ compiler is available and do a reasonably good job at making the integration run smoothly. But that, in turn, requires struggling with specific dependencies just to get it to compile — and my 'quick & dirty' approach is shown on the go.yml workflow.
That said, I guess that, for future versions, I might simply resort to forking a magick process instead — it should also work that way, at the cost of a slight performance hit (spawning/forking whole processes has way more overhead than the built-in, lightweight 'goroutines'). Then again, unlike PHP, this is all compiled to native machine code, and should certainly have as much performance as you can expect from it.

You are amazing! Thank you for you!


Making it compatible with Tensorflow v2

Hi @BeastBreath , thanks so much for making this upgrade! Currently I dont have any time to review the changes. Is it good to put in your separate repo? And I will put a link to that.


feat(hlgroups): define nvim-navic highlight groups

Closes #62 by defining nvim-navic highlight groups.
Thank you for this relaxing theme.

@rebelot ping?


Add Neo4j event publication repository

First version of Neo4j support for Spring Modulith.
Example adapted from the other modules.
Uses Spring Data Neo4j's Neo4jClient to have a low-level abstract while still participating in the ongoing Spring transaction.
For query creation, the Neo4j-CyperDSL got pulled in as a dependency.
I am not sure if the Neo4jEventPublication is even needed. Maybe it should directly implement the TargetEventPublication without the "adapter" implementation.

That's polished and merged, thanks!


Allow SQLite to be defined as URL / with paramters

Use case:
I want to be able to pass PRAGMA parameters to sqlite3 so I can enable Journal WAL and busy timeout:
file:mydbfile.db?&_journal_mode=WAL&_busy_timeout=5000
I want to be able to do this so I can back up my database with litestream.

Chiming in here, I have set up litestream and did an initial replica of our original headscale DB. The headscale DB restored correctly with our kubernetes init container, and reading the docs it looks like litestream sets the mode to WAL anyway on startup?

Further to this,I added a new user, restarted the container and it re-pulled in the latest changes. So not sure if we need to be setting those options inside headscale?

If I'm understanding correctly #1348 is towards this end by introducing a DBUrl config param which is passed directly into the driver rather then generiting from DBHost, DBUser, etc


Crash on Starting Widget-marker

Did call run.bash and got the following error message.
>> bin\dart bin\widget-maker.snapshot
Failed to init NativeSymbolResolver (SymInitialize 87)
flutter: Observatory listening on http://<IP_ADDRESS>:49683/

===== CRASH =====
version=2.1.0 (Tue Nov 13 18:22:02 2018 +0100) on "windows_x64"
ExceptionCode=-1073741819, ExceptionFlags=0, ExceptionAddress=00007FFA952CF4F6
Dumping native stack trace for thread 1ff0
Stack dump aborted because InitialRegisterCheck failed.

I'm on Windows 10 x64 bit with the C++ redist installed.

I'm sorry but I'm not sure what the problem might be. I hope future updates will fix this!

I also encountered the same problem. Here's the error log

Failed to init NativeSymbolResolver (SymInitialize 87) [ERROR:e:\b\build\slave\windows_engine\build\src\flutter\shell\common\engine.cc(112)] Engine run configuration was invalid. [ERROR:e:\b\build\slave\windows_engine\build\src\flutter\shell\platform\embedder\embedder_engine.cc(61)] Could not launch the engine with configuration. flutter: Observatory listening on http://<IP_ADDRESS>:63015/


Create a way for the Support team to manually delete an org (and thereby delete the associated users and Okta artifacts)

Once we have a path to delete organizations and their associated artifacts, Support needs to be able to delete those organizations before the automatic 45-day deletion job runs.
Unlike the scheduled job, we'll need to give support the ability to delete id-verified organizations. However, we'll still need guardrails in place to ensure that they can't delete organizations that have submitted any test results.

I think this can be change to a one-time job to delete all unused organizations (i.e. orgs that aren't reporting tests.) However, it's worth a discussion to determine if this is still valuable - what's the harm in keeping these organizations around?


fix(venv): make relocatable activation scripts support ksh

It transpires that detecting the directory a script was sourced from is non-trivial across bash, ksh and zsh.
The previous version was a one-liner and supported bash and zsh but not ksh.
It is possible to keep the one-liner and add ksh support, but that is mutually-exclusive with zsh.
Therefore, the only way to square this circle is to add an if block. A silver lining here is that although longer, the script is probably easier to follow as there is less code-golfing going on.

© 2024 Morgan Stanley.
THIS SOFTWARE IS CONTRIBUTED SUBJECT TO THE TERMS OF THE APACHE LICENSE V.2.0 AND THE MIT LICENSE.
  THIS SOFTWARE MAY BE REDISTRIBUTED TO OTHERS ONLY BY EFFECTIVELY USING THIS OR ANOTHER EQUIVALENT DISCLAIMER IN ADDITION TO ANY OTHER REQUIRED LICENSE TERMS.

Can you note why ksh is important to support in particular? There are a lot of shells out there. Are we going to support all of them?

Can you note why ksh is important to support in particular? There are a lot of shells out there. Are we going to support all of them?

I agree that trying to support all of them is futile.
The Korn shell in particular is very much entrenched in a lot of enterprise environments at least.
Not sure about zsh -- I recall ohmyzsh being trendy some years ago... never used it much myself. I personally do not object to excluding it, but others might?

I think Zsh is very important, I believe it's the default on macOS.

I'm fine with this as it's only used for --relocatable, right?

I'm fine with this as it's only used for --relocatable, right?

Yes. I will actually go back and drop the sourced'ness checks for all except bash (which is what currently exists). I am testing them right now and are really brittle stuff -- would not want them to impact real users.

Phew, that was not fun. Tested (by hand) across bash, ksh and zsh. So far so good. zsh in particular is where the > /dev/null trick comes from -- TIL it has a 'loud' cd command.

I guess my qualm with ksh (and curiosity as to why it is important) is that it's not included in our existing uv-shell::Shell variants. I'd like to have a solid list of shells we support, rather than support for some here and others there.
I presume it's present in enterprise because of red hat?

I guess my qualm with ksh (and curiosity as to why it is important) is that it's not included in our existing uv-shell::Shell variants. I'd like to have a solid list of shells we support, rather than support for some here and others there.

That is a fair concern. I had a brief look at the module and it doesn't look terribly complicated (save for configuration_files() which requires a bit of research because I haven't got a clue as to how it's used). How would you like me/us to approach this -- same PR or track separately?

I presume it's present in enterprise because of red hat?

The correlation is rather uncanny indeed. Although I would not be surprised if that the affinity for ksh predates widespread RHEL (or even Linux!) adoption.

Ok @zanieb it turns out it was a fairly trivial addition considering there's already a pattern of supporting multiple POSIX shells with largely the same code.

Thank you. This seems reasonable to me, especially now that we added Ksh to our shell enum.

Late to this, but, FWIW, ksh is the default shell on AIX which I think a fair number of IBM customers still use. So, not silicon-valley, but certainly banks and the like.


Forgotten variable in CSS

Expected behavior
Don't have undefined CSS variables.
Actual behavior
There are undefined CSS Variables
Steps to reproduce

https://raw.githubusercontent.com/sampotts/plyr/master/dist/plyr.css

If you look there, you'll see this declaration:
::-webkit-slider-runnable-track{
  background:0 0;
  border:0;
  border-radius:3px;
  height:6px;
  transition:all .3s ease;
  -webkit-user-select:none;
  user-select:none;
  background-image:linear-gradient(to right,currentColor var(--value),transparent var(--value))
}

Which has the --value variable there. Comes from here https://github.com/sampotts/plyr/blob/600f0eb8a3cf49f5268e5c15487797d496bc19b6/src/sass/components/sliders.scss#L22
It does seem the only element having a var notation. Keen to submit a PR but unsure what the intended value was.

It’s required to set the lower fill on the range input that WebKit lacks. It’s not forgotten 👍
Sam

On 20 Mar 2018, at 8:15 pm, Antonio Laguna<EMAIL_ADDRESS>wrote:
Expected behavior
Don't have undefined CSS variables.
Actual behavior
There are undefined CSS Variables
Steps to reproduce
https://raw.githubusercontent.com/sampotts/plyr/master/dist/plyr.css
If you look there, you'll see this declaration:
::-webkit-slider-runnable-track{
background:0 0;
border:0;
border-radius:3px;
height:6px;
transition:all .3s ease;
-webkit-user-select:none;
user-select:none;
background-image:linear-gradient(to right,currentColor var(--value),transparent var(--value))
}
Which has the --value variable there. Comes from here https://github.com/sampotts/plyr/blob/600f0eb8a3cf49f5268e5c15487797d496bc19b6/src/sass/components/sliders.scss#L22
It does seem the only element having a var notation. Keen to submit a PR but unsure what the intended value was.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

And what's that value?
variable '--value' is undefined and used without a fallback [postcss-custom-properties]

Actually, nevermind found out!

Sorry for the abrupt answer. Basically all browsers have capacity to allow you to style the "lower fill" of an <input type="range"> slider. Edge even allows you to style both sides! WebKit however is missing this, for whatever reason. In v2 I dealt with this by displaying a <progress> behind the slider but always felt it was a hack and then discovered it was possible to use JS + CSS variables (since WebKit supported them ages ago) to set the values of a background gradient instead. This is the JavaScript part of it:
https://github.com/sampotts/plyr/blob/master/src/js/controls.js#L15

Thanks a lot for the inside. When I inspected the shadow dom and saw the value progressing then it struck me. Really clever!


Add basic node setup and static configuration file

Basic node setup includes:

Bootstrapping with static config file for single node setup (transport and basic parameters)
Implementation of Gossipsub
Saving peer store to disk

This will enable starting up a single node that reads the config file. From this, we'll be able to move onto connecting to other nodes that can specify configurations for connecting to the bootstrap node.

"Implementation of Gossipsub" will not be included. Other basic nodes setup include:

configs to setup a peer, providing options
Checks to perform,
Disk read/write
Custom operations to perform at startup
etc.


Create appveyor.yml

will need to be enabled

Coverage decreased (-1.8%) to 26.857% when pulling ee82346fa104446c95d48dfd2b94848d51a9ed59 on tk/appveyor into 10c84a114393f6d863ff3bbdabdd96c0b411ad59 on master.

ping @amitmurthy could you turn this on?

Bump!

I guess it should be in Settings -> Services. Couldn't find Appveyor there. What is it called?

No, you enable it through a webhook. Sign up on appveyor then enable the repo through their projects page.

Done.


Make headlamp-plugin faster

headlamp-plugin checks whether there are updates to it every time we run it. While this is nice and useful in a normal use, if we run the command repeatedly, this feature does impact dev experience by taking a long time when we may just be running a simple command, as every check takes usually 1.5-2.0 sec .
So I suggest we only check for updates a limited number of times per day. Maybe once a day is even enough.
If the update is to be skipped, we could print a message telling the user how to force check the updates.
cc/ @illume

Are you having this issue in just the npm start command?
That should usually be only run once, and then the command just sits there watching for changes - with no update check done. I understand it would be more of a problem when the npm start was buggy and it had to be run on each change.

During the "watch" bug it was certainly annoying. But also in live demos it usually just seems like it takes too long.
I think it's worth making faster or only running sporadically, but certainly not urgent.


KFM with Blocked File Types

When/how long does this take to apply? i am testing this with Known Folder Move and after adding ISO to the block list i can still add new iso to my desktop which syncs up to OneDrive

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

ID: a3a0cebd-7e82-6631-bdee-031b3b84f74d
Version Independent ID: ee04db29-a6db-2874-2573-d80f0b415308
Content: Block syncing of specific file types
Content Source: OneDrive/block-file-types.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@Naz9mm Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @Naz9mm , I tried in my lab and took +- 1 hour to apply.
I did not recognize a need to edit the article.
Hope this information helps you.

@Naz9mm Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Support database migrations

Support migrations for the sqlite and mysql database schemas.

question: is there an opportunity for Backend-agnostic migrations, supporting a kind of an ORM migration approach?
E.g., today there is a diag.Backend interface for inspecting data from supporting backend.Backends for diagnostic purposes.  Perhaps a similar interface could support Backend-agnostic migrations.


Feature/add configuration

Configuration Objects for Cloudinary Base.
Configurations were separated based on concerns and likelihood of change. (Cloud Configuration are global per website, while URL might not be)

@asisayag2  this was before the demo and updates.
Please see this PR with all the new updates, I'm closing this PR for now.
https://github.com/cloudinary/cloudinary-js-base/pull/20


Adding more context to "failed to collect operation" errors

Hey there 👋
This is related to / a follow-up https://github.com/kamilkisiela/graphql-hive/issues/1220
Perhaps I'm missing something but is there any way to enrich these error log messages with more context?  I know that the logger can be supplied at startup but I'm looking to add some dynamic/runtime context related to the currently executing request, maybe similar to how envelop offers a useErrorHandler configuration
In our personal circumstance we use an apollo koa server, and we keep a KoaContext with a requestId on it that gets generated by the client - this lets us track requests across our ecosystem of microservices.  Ideally, we would love to enrich the error thrown here with our context.  This way, any error can be easily associated with the particular HTTP request logs that triggered the error
At the very least, it would be great if the error call included the query document that it was trying to parse.

Pardon the slight detraction but how do you configure the logger on startup?


enable support of non uniform workgroup size by default

Support of non uniform workgroup size also allow to split a NDRange in
several smaller regions.
We need that feature available by default to run the CTS on small
embedded GPUs even in CL1.2.
Allow to disable to feature with '-uniform-group-size'.
This is related to the work in kpet/clvk#367

I meant we should enable it by default in clvk :). I'm not sure we want it enabled by default in clspv.


AppVeyor notification

Hi,
Some time ago I was notified by email that https://ci.appveyor.com/project/conda-forge/thermotools-feedstock/builds/34834887 failed and I really wonder why I was notified on that. Do you have any explanation? Thanks.

Sorry, I do not have an explanation for that.


Anonymous links not handled correctly

It appears that the following happens when anonymous references are used.
http://logs.openstack.org/13/161113/6/check/gate-openstack-manuals-tox-checkniceness/e2e5ba6/console.html
The rst that caused this @ https://review.openstack.org/#/c/161113/6/doc/hot-guide/source/existing_templates.rst
It appears links/references that are anonymous don't have line numbers so can't be accessed in this manner (and therefore blow up the linting error handler).

I will take a look by the end of this weekend. Thanks!

To add a little bit more info, in this case I had used double trailing underscores yet still tried to treat it as a named link. Example:
The OpenStack TripleO Heat Templates repository__ contains a variety of heat templates that are included in the tripleo-heat-templates codebase.
.. _OpenStack TripleO Heat Templates: http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/
I'm not sure if the check would have been confused if I had used a properly formatted anonymous link, but at least the reported problem is reproducible with the above.

Yea, I saw that. In any case though, the linter should either be reporting or ignoring it, not throwing errors.

Alright, took a look. It was an easy fix: the error had no line associated with it since it was more about the file. To fix it, we moved from ['line'] to .get('line').
The error itself should read something like "Anonymous hyperlink mismatch: 1 references but 0 targets."
This has been patched and released in 0.11.0.


Anaconda Inc

Anaconda Inc is under travel restrictions policy. WFH only required for all staff.

can you share a source for this?

Please see attached email from CEO to all employees
On Wed, Mar 11, 2020 at 5:45 PM Nathan Manousos<EMAIL_ADDRESS>wrote:

can you share a source for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phildini/stayinghomeclub/issues/288#issuecomment-597917150,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJLCXXIS44YNUAV7QFYBJILRHAIBZANCNFSM4LF552WA
.


Poor performance for trace calls

Describe the feature
Im using debug_traceCall through IPC very extensively (executing a lot of simulations in parallel with arbitrary bytecode). I think it's around 100-1000 sims per block and up to 20-60 simultaneous requests. Previously I was running this same code on geth IPC and was getting decent times - like 200-400ms for 80 requests (it's 4 times * 20 simultaneous requests).
With reth my average time for these same requests are 400-800ms. Sometimes it becomes 2s+, while in geth I was getting everything under 1 second. I'm using --rpc-max-tracing-requests 1000.
Is there anything I can tweak?
Additional context
No response

what kind of tracing requests?


Type definition does not contain enumeration for NavigationFailureType

Version
3.4.2
Reproduction link
https://jsfiddle.net/dmop5w9b/
Steps to reproduce
I want to check for navigation-errors of a certain type. So there is the new VueRouter.isNavigationFailure function which takes a type as second parameter. But because of the missing type definition for this type, I can not call this function without typescript-errors.
if (VueRouter.isNavigationFailure(err, VueRouter.NavigationFailureType.duplicated)) {
 ...
}

Typescript error is: Property 'duplicated' does not exist on type 'NavigationFailureTypeE'.ts(2339)
I provided a fiddle with the code, but did not get the errors there... Those errors just appear in VSCode.
What is expected?
No typescript errors
What is actually happening?
Typescript errors

Thanks!

Still not possible to use NavigationFailureType or isNavigationFailure
https://router.vuejs.org/guide/advanced/navigation-failures.html#detecting-navigation-failures is not working

import VueRouter from 'vue-router';

if (VueRouter.isNavigationFailure(err)) {
          switch (err.type) {
            case VueRouter.NavigationFailureType.duplicated: {
....


Regenerates jsctl cli docs

The result of make docs-gen with the expanded config path referring to my home directory replaced with ~
Signed-off-by: irbekrm<EMAIL_ADDRESS>

We should probably have a check for this that it's clean. Thanks for adding it.


parentScroll

Please update documentation for  [parentScroll],
was [parentScroll]="scrollingBlock.nativeElement" becoming [parentScroll]="scrollingBlock"

Good catch. I'll do this.
However, the documentation isn't completely wrong.
These are both valid:
<random-angular-component #element1>
    <virtual-scroll [parentScroll]="element1.nativeElement">
    </virtual-scroll>
</random-angular-component>

<div #element2>
    <virtual-scroll [parentScroll]="element2">
    </virtual-scroll>
</div>

The issue is that parentScroll must be assigned a native javascript DOM element reference. the angular #elementRef syntax will give you a different javascript variable type based on the type of element. If it's a native HTML element such as a DIV, it'll give you a native javascript DOM element reference. If it's a custom angular component, the DOM element will be wrapped in an Angular ElementRef https://angular.io/api/core/ElementRef in which case you'll need to call .nativeElement to get to the javascript DOM element reference.

I've updated the documentation based on your request. Please re-open if this is still incorrect.


As a developer, I would like to consume messages from Kafka topic and store it in the CouchDB

Write a sink connector to move data from Kafka Broker to CouchDB.

This is done!


Fix --foo=./file.ext parsing

Fixes #10.

Please consider releasing a new version that includes this (or any other) fix. Thanks!

Damn, you're fast. Thanks for the pull request. Releasing new version in a min.

you're fast

Right back at you. Thanks for releasing v0.1.4 so quickly!


Peer Connection issue

It is connecting any device to any other device randomly.

#30 related?
If you have additional question, please open again:)


[Bug] New feature from 1.5.7 allowing Cache Control headers to be set on Edge is not honoured by client

The browser client request does not appear to be honouring the Cache-Control header coming from the server which is a new feature in 1..5.7.

This is an effect of not setting the Expires header in Edge and not a fault of this SDK.


Ideas/Questions about future of this project

Hello, this is a cool project and I'm evaluating to use it for more. I have a few questions:
TF Module
Wouldn't it be nice to have this as a terraform module and expose every important value as a variable?
The target right now is playground/testing, but I think this is also a good base for more serious things.
License
Currently this code has no License, is this on purpose?

Hi @toabi, that's a good idea. Would you like to create a PR?

Okay cool, yes I will work on this now.
How do you feel about renaming of files/resources/… - I think some things could get better names.

Well, all that is done :)


feat(zio): support Native for Scala 2.12 and 2.13

In my previous PR I missed 2.12 and 2.13. They should work without any code changes.

When this PR is merged and released I will continue extend the Tapir zio Native support.


[PYTHON] add RUFF to the installTool list

this speaks for itself.

Thanks for the request. @bamurtaugh has shared this issue in the community channel to gather feedback and interest.
Adding this to the backlog!


long topics get truncated badly in room info with no way to scroll

It's possible to scroll topic now


Add individual

Added BaseCaseMixin
Fixed so adapter.individual() works with all adapters

Looking good :baby_bottle:


[bump] build-tools: 0.13.0 => 0.14.0 (minor)

Bumped build-tools from 0.13.0 to 0.14.0 in preparation to release 0.13.0.
Command used:
flub release -g build-tools

Thanks for the review, @dannimad!


Initialize cdk-tree component datasource in the constructor

Due to an error Cannot read properties of undefined (reading 'getTreeData'), had to move datasource initialization to the constructor

I managed to analyze the cause of the problem. The root cause of the issue is the tsconfig target ES2022. As it turns out targets ES2021 and ES2022 behave quite differently in some aspects. The following class:
class Data {
    getTest() {
        return 'test';
    }
}

class Test {
    test1 = this.data.getTest();
    test2 = (() => { console.log("data", this.data)});

    constructor(private data: Data) {
        console.log("getData", this.data.getTest())
    }
}

const test = new Test(new Data())
test.test2();

will be compiled to the form (ES2021):
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
class Data {
    getTest() {
        return 'test';
    }
}
class Test {
    constructor(data) {
        this.data = data;
        this.test1 = this.data.getTest();
        this.test2 = (() => { console.log("data", this.data); });
        console.log("getData", this.data.getTest());
    }
}
const test = new Test(new Data());
test.test2();

On the other hand, setting the target to ES2022 we get:
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
class Data {
    getTest() {
        return 'test';
    }
}
class Test {
    data;
    test1 = this.data.getTest();
    test2 = (() => { console.log("data", this.data); });
    constructor(data) {
        this.data = data;
        console.log("getData", this.data.getTest());
    }
}
const test = new Test(new Data());
test.test2();

As we can see, after compiling, we refer to the class dependency at a time when we do not yet have a class reference created.
Hence, in target ES2022 we get the error
[ERR]: Cannot read properties of undefined (reading 'getTest')

You can find similar differences in the static class fields. The entire thread, along with solutions, can be seen here.
So the solution to our problem is to either initialize the fields of the class in the constructor or restore in tsconfig target to 2021

One more thing that needs addressing, though, is that we didn't get a test failure for this.

@gkalpak we have this usecase covered


Fix path in mysql_data_persistence test

What this PR does / why we need it:
Fix mysql_data_persistence chaos (e2e) test with correct path references
Which issue this PR fixes : fixes #261

Thank you @Roshanjossey ! Hope this is the beginning of many more PRs !


Add setting to enable/disable tooltip hover during Screencast Mode.

This request is related to #15667.

I'm loving the Screencast Mode, and I use it often during presentations. The tooltip when hovering over code usually gets in the way though. I know there is a editor.hover.enabled setting to toggle the tooltip but during regular coding work I like it to be enabled. Only during screencasts I'd like it to be disabled. Could there be a setting as part of the Screencast Mode to enable/disable the hover?

A configurable ScreenCast mode would benefit a lot of streamers: https://twitter.com/mattstratton/status/1456595913716158465?t=1Er9tGIs0DKQKrW_mU-DTg


Eslint Airbnb config is not compatible with Vuex.

no-param-reassign rule makes the Airbnb Eslint config incompatible with Vuex. In Vuex parameters mutation is required - this is how Vuex works.
So In order to fix this issue, the following override is needed:
"no-param-reassign": [
  "error",
  {
    "props": true,
    "ignorePropertyModificationsFor": [
      "state",
      "acc",
      "e",
      "ctx",
      "req",
      "request",
      "res",
      "response",
      "$scope"
    ]
  }
],

PS
All properties except state are in the ignorePropertyModificationsFor array by default.

I used to disable this rule by adding eslint-disable for each vuex file, and your config seems better. I think we could use eslint conf's overrides field to make a finer restriction.

This worked for me, after restarting dev environment.
I created a new .eslintrc.js file in my store folder and added them there
// airbnb eslint is incompatible with vuex, so we add:
"no-shadow": ["error", { "allow": ["state"] }],
"no-param-reassign": [
  "error",
  {
    "props": true,
    "ignorePropertyModificationsFor": [ // All properties except state are in the ignorePropertyModificationsFor array by default.
      "state",
      "acc",
      "e",
      "ctx",
      "req",
      "request",
      "res",
      "response",
      "$scope"
    ]
  }
],

This worked for me, after restarting dev environment.

Thanks for this comment. Errors went away in my IDE but project still wasn't working and I couldn't see why until I saw this.


[BUG] Rollup Query fails in pytest with "year 0 is out of range"

Describe the bug
The following test fails in python when using spark 3.1.0
@pytest.mark.parametrize('data_gen', all_gen, ids=idfn)
@ignore_order
def test_cache_expand_exec(data_gen):
    def op_df(spark, length=2048, seed=0):
        cached = gen_df(spark, StructGen([
            ('a', data_gen),
            ('b', IntegerGen())], nullable=False), length=length, seed=seed).cache()
        cached.count() # populate the cache
        return cached.rollup(f.col("a"), f.col("b")).agg(f.count(f.col("b")))
    assert_gpu_and_cpu_are_equal_collect(op_df)

Steps/Code to reproduce bug
write the above test in python or pyspark shell.
Expected behavior
The test should pass
Additional context
Investigate why the above test is failing. The following error is reported
return datetime.datetime.fromtimestamp(ts // 1000000).replace(microsecond=ts % 1000000)
ValueError: year 0 is out of range

This makes it sound like the timestamp is going out of range which we have been circumventing by limiting the dates used for generating a timestamp in python tests

@sameerz should we mark this as p1 as per @revans2 suggestion?

Until we know why it is failing and the extend of the possible data corruption we need to treat this as a P1

Is this still a P1 for the 0.3 release if it's specific to Spark 3.1.0?

This is no longer reproducible. We should keep this around as a lower priority until Spark 3.1 is released

Unable to reproduce this error while running with the latest Spark-3.1.0-SNAPSHOT. We should de-prioritize this until the Spark-3.1.0 release

Closing as this isn't reproducible


pocket-updater-utility: 2.31.0 -> 2.34.0

Description of changes
https://github.com/mattpannella/pocket-updater-utility/releases/tag/2.32.2
https://github.com/mattpannella/pocket-updater-utility/releases/tag/2.33.1
https://github.com/mattpannella/pocket-updater-utility/releases/tag/2.34.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2660

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1129

@ofborg eval


guide: explain about encrypted buckets in external data mgmt

See this ticket: https://github.com/iterative/dvc/issues/2701
May be it worth creating a summary in that ticket and link to it from the docs.

Closing as external outputs were removed.


influxctl 2.1.0

Adds features and functionality introduced in influxctl 2.1.0.
@powersj, are the influxctl cluster commands meant only for internal use? If so, I'll add a note to them. They're included in the CLI help output, so I still think they should be included in the public documentation. But I can make their purpose clear.

[x] Rebased/mergeable

Thanks @powersj. I marked influxctl cluster create|update as draft, so the markdown files are still there, but they won't be publicly available. If these will never be available to users, I can go ahead and remove them completely.


Update Code Editor

Resolves:   #363
AutosizeCodeEditor is used from the library volkovlabs/components;
Added syntax support  for code editor: TYPESCRIPT, SQL, PGSQL, MySQL, PYTHON, JAVA, PHP, GO, C++, C, C#, RUBY

@asimonok Ready for Review


合并最新代码

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

合并最新代码


added videohub integration

Coverage decreased (-3.88%) to 84.49% when pulling 02c6fd16a527286c884c68c8d852ded1e54f308e on vh-integration into 0340c462d4f256f8aa5f5d7cd11242054e6e810f on master.

don't need this any more


Simulator Unresponsive after Interactive Phase

Hello,
So I am working through the simulator, once i click the "Start Interactive Phase" and it opens a new window for the Connector URL i selected, the "Get Data" button is then not responsive, any suggestions or ideas. I have tried in both Chrome and Firefox. Thanks!

As soon as you open the index.html page, could you open the inspect option in chrome and paste the console logs that appear once you click "Get data" option ? This will help debug the issue.

@jolllof, as @ani0989 suggested, grabbing any errors you can find in the logs will help diagnose the issue.
You can right click anywhere in the popup window of the interactive phase, click 'Inspect' then navigate to the 'Console' tab in order to find the logs.

Hi - I am having this same issue. Here's my error message:
Warning: Failed Context Types: Calling PropTypes validators directly is not supported by the prop-types package. Use PropTypes.checkPropTypes() to call them. Read more at http://fb.me/use-check-prop-types
Thanks!

Please help reopen if there is still an issue. Thanks for contributing to webdataconnector!


Raw AD record display missing some bytes?

This shows #9 Name having 26 characters when shown "As String", which is correct.
But the display "As Array" only shows the last 24 characters.

That'st interesting.
I'll have a look!


Update black

Minor housekeeping stuff: the new major version of black has been out for a while. Let us not get stuck with the old one.
Changes:

Updates dependencies and CI checks to use black 24.*.
Re-formats code with the new black.
Adds a tox environment so that we can run tox -e format to run black and isort.

This became a little messy. Closing.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 21.995%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fabrictestbed_extensions/fablib/slice.py
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1160

Relevant Lines:
4699

💛  - Coveralls


Parse "not determined" Clouds conditions

Summary

Parsing METARs w\ not determined cloud feature results in a corrupted parsing.
Expected Behavior
Cloud features such as:

BKN013///
//////TCU

Should:

not break parsing;
be reflected in a model somehow.

Current Behavior

All METAR features aft go to the RMK property.
Steps to Reproduce

Parse:

METAR EGLL 042350Z AUTO 35010KT 320V020 5000 -RADZ BKN008/// OVC015/// //////CB 06/04 Q0993 TEMPO 4000 RADZ
EGSS 042320Z AUTO 36008KT 4100 -RADZ OVC003/// //////TCU 07/06 Q0992

(these are real METAR issues by the authorities)

Resolved with #8


The polygon is rendered inverted

When I add a polygon, the entire area of the map is filled, and the polygon is "cut" out of it. Why is this happening? I expect a simple polygon to be rendered.
Here is a live example https://codesandbox.io/s/lsz8qh
This is my topojson
{   "type": "Topology",   "objects": {     "collection": {       "type": "GeometryCollection",       "geometries": [{ "type": "Polygon", "arcs": [[0]] }]     }   },   "arcs": [     [       [-12.260535998018383, 62.14792267500428],       [-13.974493697178218, 35.17713854785707],       [10.820761017340033, 35.82827731956981],       [48.070775012424235, 43.55191645077076],       [43.9572765344397, 70.27381439382373],       [17.105272580928016, 71.36229082737557],       [-12.260535998018383, 62.14792267500428]     ]   ] }
rendering now

what polygon i expect to see

Because the coordinates in the array are counterclockwise
if positioned clockwise, the polygon will be drawn correctly
"arcs": [
  [
    [-12.260535998018383, 62.14792267500428],
    [17.105272580928016, 71.36229082737557],
    [43.9572765344397, 70.27381439382373],
    [48.070775012424235, 43.55191645077076],
    [10.820761017340033, 35.82827731956981],
    [-13.974493697178218, 35.17713854785707],
    [-12.260535998018383, 62.14792267500428]
  ]
]


Add webpack support

Set up webpack-style src/dist folders
Set up package.json
Set up webpack build optimization (minimizing)

Meant to target my fork, sorry!

No worries at all, glad to see that you enjoy it.
I'd suggest vite or esbuild though, if you really want a bundler ;)

Oh thank you, I didn't know about those 💡


How to remove the tooltip on markers when onclick is fired?

I have created a few markers on the map and when we hover over them the label appears. When we click on them, I am loading a different component but the label stays on the screen and does not go away. I tried calling the onMarkerOut of the Component but no luck. Could you please point me in the right direction?

@iamanoopmk I know this is an old issue but in case anybody else run into this issue, this is the workaround I found: pass a ref prop to your VectorMap component and in your click handler get the map object and hide the tip:
Your VectorMap component looks like this:
<VectorMap
map={...}
ref={"map"}
onRegionClick={this.handleRegionClick}
/>
And the handler looks like this:
handleRegionClick = (event,data) => {
this.refs.map.$mapObject.tip.hide();
}
There's probably a cleaner way but this workaround worked for me.

I'm having the same problem except that onClick I'm calling a different route. I tried the workaround you propose but it return me an error. Is there another way to work around this problem?
Thanks in advance :)

It cant get uglier than this:
 setTimeout(()=> {       Array.from(document.getElementsByClassName("jvectormap-tip")).forEach((el) => {         el.style.display = 'none'       });     },100);

I am having the same issue when I call useState from the parent component on onRegionClick. @maxencelefort's solution didnt help. @KingAMS's solution works, but yes it would be nice to fix this.

It cant get uglier than this:
 setTimeout(()=> { Array.from(document.getElementsByClassName("jvectormap-tip")).forEach((el) => { el.style.display = 'none' }); },100);

thank you kind sir

Is there a (less hacky) fix for this yet?

I found a solution how to remove tooltip from a DOM, when destroy component
const Map = ({ country, height, values, isZoomButton }) => {
  const refMap = useRef<any>(null);
  const handkeHover = useCallback((e, el, code) => {
    const value = country[code];
    if (!value) {
      el.html(el.html())
      return;
    };
    el.html(`${el.html()}: ${value.toLocaleString("en")}`);
  }, [country])
  useEffect(() => {
    return () => {
      if(refMap.current) {
        refMap.current.tip.remove();
      }
    }
  }, [refMap])
  return <div>
    <VectorMap
      map={"world_mill"}
      zoomOnScroll={true}
      zoomButtons={isZoomButton}
      containerStyle={{
        width: "100%",
        height
      }}
      ref={(map) => {
        if(refMap.current) {
          refMap.current.tip.remove();
        }
        if(map) {
          refMap.current = map.$mapObject
        }
      }}
      containerClassName="map"
      onRegionTipShow={handkeHover}
    />
  </div>
}

Clean up  DOM elements when component is no longer in use
    useEffect(() => {     return () => {       document.querySelectorAll('.jvectormap-tip').forEach((element) => {         element.remove();       });     };   });


Torch version in requirements

Hi. It's not a bug, just a little thing.
In requirements.txt you mentioned torch>=1.7.1 but 1.7.1 don't have "rounding_mode" param in "div" function (you use it in decoders/seq2seq.py). It was added in 1.8.0.  There was a separate function "torch.floor_divide" In 1.7.1.

I encountered the same error in seq2seq.py and spent some time figuring out that it actually requires at least torch 1.8.0.
I do consider it as a bug.

Fixed in the development branch, thank you!  Now the requirement is:
torch>=1.8.0,<=1.8.1
Note that pytorch 1.9 works as well. The only annoying thing is that there are some warnings raised internally by pytorch that they will be fixed in the next version (I already raised an issue in the pytorch repo for the warnings).


[now dev] Use empty string for missing matches

Rather than undefined. This matches the behavior in production.
Fixes #2375.

Codecov Report

Merging #2376 into canary will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           canary   #2376   +/-   ##
======================================
  Coverage    10.9%   10.9%
======================================
  Files         250     250
  Lines        9031    9031
  Branches      990     991    +1
======================================
  Hits          985     985
  Misses       7945    7945
  Partials      101     101

Impacted Files
Coverage Δ

src/commands/dev/lib/dev-router.ts
75.47% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16b99a6...9fa3ac2. Read the comment docs.


Setup drawer right side does not work

Hi. Thank you again for the awesome drawer.
I have a question. I want to setup drawer on right side. I am using your code example but it does not work ( when I clicked on Open button, No drawer displayed from right side)
drawerController.drawerDirection = .Right
drawerController.drawerWidth     = 200

Do I have do anything in order to make Right-Drawer work?
Thanks!

@haducloc  can you tell us bit more , are you setting using the code or with storyboard?

I am not using Storyboard. What I did setup as below:
func application(application: UIApplication, didFinishLaunchingWithOptions launchOptions: [NSObject: AnyObject]?) -> Bool {
// Override point for customization after application launch.
    let mainViewController   = MainViewController()
    let drawerViewController = DrawerViewController()
    let drawerController     = KYDrawerController()
    drawerController.mainViewController = UINavigationController(
        rootViewController: mainViewController
    )
    drawerController.drawerViewController = drawerViewController

    // To make Right Drawer
    drawerController.drawerDirection = .Right
    drawerController.drawerWidth     = 200

    window = UIWindow(frame: UIScreen.mainScreen().bounds)
    window?.rootViewController = drawerController
    window?.makeKeyAndVisible()

    return true
}

@haducloc Hi!
As shown below , please try to change the order in which to set the drawerDirection.
let drawerController     = KYDrawerController()

// To make Right Drawer
drawerController.drawerDirection = .Right
drawerController.drawerWidth     = 200

drawerController.mainViewController = UINavigationController(
    rootViewController: mainViewController
)
drawerController.drawerViewController = drawerViewController

I'm sorry, but for this bug will at a later date will correspond
@hardikdevios Thanks! :+1:

Thank you ykyouhei. I will test it

how can i set it to the right side with Storyboard? @haducloc


Allow overriding parameters of an overridden method

Suggestion

🔍 Search Terms

Override, parameters, variance, variant, contravariant, covariant, cast, casting
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

Allow overriding parameters of an overridden method

#54427

Looplicate of #54427, see comment

Looplicate of #54427, see comment

Disagreed.
Although both of these issues are referring to an update on the current implementation of method overriding, they are not referring to the same part of it.

In what way is this currently not allowed?

No

An actual example would be nice.

Absolutely.
I updated https://github.com/microsoft/TypeScript/issues/54426#issue-1729064272 to feature one.
Thanks

In what way is this currently not allowed?

No

That’s not a yes/no question…

That’s not a yes/no question…
It reminds me of that “Q: how many X do you want? A: yes” meme. 😄

Lol yes sorry, I read it "in that way", my bad.
Check the example tab in https://github.com/microsoft/TypeScript/issues/54426#issue-1729064272

My “looplicate” comment was meant to be humorous (hint: that’s not a real word) but I see the joke failed. Oh well, I’ll never get a job as a comedian.
But still: was it really necessary to file two issues in immediate succession about two parts of the same thing? Now we get to have two nearly identical conversations.
To wit: in both of these you’re either asking for something already supported or something unsafe. In both cases the current behavior is stable and unlikely to change.  In both cases it’s not clear what the motivation is: why do you want to be able to make subclass methods completely unassignable to superclass methods? Breaking subtyping intentionally would need a fairly convincing argument to be considered… but I don’t see one here.  It’s like saying cars shouldn’t warn when the driver isn’t wearing their seat belt with the motivating example “I want to drive without wearing a seat belt”.  The answer’s going to be “no, sorry; consider wearing the seat belt or using a //@car-ignore-error directive no lawsuits kthxbai”

My “looplicate” comment was meant to be humorous (hint: that’s not a real word) but I see the joke failed.

Don't worry, I got the joke at least. (though admittedly it took me a good minute 😅)

My “looplicate” comment was meant to be humorous (hint: that’s not a real word) but I see the joke failed. Oh well, I’ll never get a job as a comedian.

Whoops, my bad, I thought you thought the two issues were in a kind of "recursive duplicate" state

was it really necessary to file two issues in immediate succession about two parts of the same thing?

Maybe not, indeed.
I thought that there is a different debate for each, may be wrong, my bad if it turns out being true.

something already supported

Ah? I don't find any way to do it.
Also, the "Allow overriding parameters of an overridden method" concept is an existing thing in some OOP languages such as Java.

I don't find any way to do it.

Safely overriding methods with contravariant parameters is already allowed:
class Child1 extends Parent {
  override method(x: string | number) { } // no error
}

class Child2 extends Parent {
  override method(x: string, y?: string) { } // no error
}

The thing you're asking for is demonstrably unsafe:
class Child1 extends Parent {
  override method(x: number) { console.log(x.toFixed());  }
}

let parent: Parent;
parent = new Child1(); // should be safe but isn't
parent.method("abc"); // RUNTIME ERROR, x.toFixed is not a function

class Child2 extends Parent {
  override method(x: string, y: string) { console.log(y.toUpperCase()) }
}
parent = new Child2(); // should be safe but isn't
parent.method("abc"); // RUNTIME ERROR, y is undefined

Playground link
TS does allow some unsafe things, indeed it allows covariant parameters in method overrides, which are also unsafe. But you're asking to add new unsafe things and haven't presented anything resembling a compelling reason to do so.

the "Allow overriding parameters of an overridden method" concept is an existing thing in some OOP languages such as Java.

No, Java does not allow that.  It allows overloading methods, which is different.  The method in the parent class would still be callable on an instance of the subclass, if you pass in the right arguments.  But even if you were right about this, it wouldn't be sufficient to justify adding the feature to TypeScript.  For that, we'd still need to see some compelling TypeScript-specific use cases for it.
What are these use cases?

TS cannot support multiple overload implementations, so that will not happen.  But there's no need to do that; if you actually want overloading, you can do it with a single implementation that checks its inputs.  And even that isn't needed, you can just use unions in the input parameters.  TS already supports overriding with wider parameters, as I mentioned earlier. Again, for your example it looks like
class Child1 extends Parent {
  override method(x: string | number) { } // no error
}

class Child2 extends Parent {
  override method(x: string, y?: string) { } // no error
}

Doesn't that meet your needs?  You check inside the method what x is, or if y is defined, and then move on from there.  Without needing the TS language to be changed.
I don't think I'll engage further here if you don't mind.  You should probably close this feature request, or you can leave it open and it will most likely be declined.  Either way I think I've said everything I can think of multiple times now, so I'm just adding noise that isn't helping anyone.  Good luck!

I guess I will find my way anyway, thanks for your time!


README: Add fail-fast option in all matrix examples

Should there be an explanation why fail-fast is required or what it does?

I don't think that fail-fast should be used in the readme. That's specific about how GitHub Actions handles matrices; unrelated to MSYS2 itself. Many users might want the default fail-fast behaviour indeed.
However, I agree it would be interesting to add a comment with a reference to the corresponding page in GitHub's docs, and a brief explanation of why using it might be desirable.

Thanks for the review. I have added a comment with a reference to the corresponding page in GitHub's docs.

Thanks!


Block access to the same addresses as original Via

In Via we use Squid to block outbound traffic to dangerous locations, for example to end-points which might divulge AWS secrets exposed within the container for configuration:

https://github.com/hypothesis/via/blob/master/conf/squid.conf

We need a similar system to block access in Via 3's HTML component.
It seems like a bit of overkill to have Squid there just for that. I think an iptables config would do the job, but I don't really know how to do that, and it's probably a bad thing to guess at.
Testing notes

You can go to Via 3 in QA and enter any URL for test at: https://qa-via3.hypothes.is/

Some good URLs

https://qa-via3.hypothes.is/route?url=http%3A%2F%2Fexample.com
https://qa-via3.hypothes.is/route?url=https%3A%2F%2Fexample.com
https://qa-via3.hypothes.is/route?url=https%3A%2F%2Fh.readthedocs.io%2F_%2Fdownloads%2Fen%2Flatest%2Fpdf%2F

Non standard ports

https://qa-via3.hypothes.is/route?url=http%3A%2F%2Fwww.asnt.org%3A8080%2FTest.html

A bad config URL

The following should be blocked: https://qa-via3.hypothes.is/route?url=+http%3A%2F%2F<IP_ADDRESS>%2Flatest%2Fmeta-data%2F
However it's already caught by Cloudflare, so we can't know

A branch has been pushed for approval: https://github.com/hypothesis/deployment/pull/105

I hate to be the bearer of bad news, but the requested functionality is proving very difficult if not impossible to guarantee automation in all conditions with our Elastic Beanstalk Docker implementation and iptables. The fundamental problem relates to the internal usage of IP ranges that we are looking to block the application running inside Docker from accessing. I have been able to manually configure a solution that appears to work on VIA3 QA. However, I suspect there may be unexpected problems we are yet to realise.
The manual solution blocks the unwanted IP and Port access. However, when it comes to providing an automated means to implement this over and over again the solution falls down. This problem specifically relates to the docker0 network deployed by elastic beanstalk which we have little influence over, and the manner in which IP addresses are allocated within the <IP_ADDRESS>/12 subnet we are looking to block. During my manual configuration I can check the IP that has been allocated. However, through the automation of an ebextension I do not have access to this information. I can make a good guess, but it will not be a reliable implementation.
I think implementing at the network layer is a no go, and we are going to have to provide a solution within the application layer using some kind of HTTP proxy.
I do have a couple of other ideas if we want to continue researching a solution outside of the application, but I think we should discuss these with an understanding of how complex implementing proxy would be.
I hope this all makes sense. Let me know if you would like any further detail.
For now, I will proceed with the closure of this issue.

To add to the above, my understanding from a Slack discussion is that the problem with the iptables configuration is that when a process in the container makes a DNS or HTTP request (and perhaps other things), the response packet gets blocked before it reaches the container.

Excuse the dumb question, but if we can list the things we want to block with Squid, can't we use a similar static list for IP tables within the container?

Quick update - I am making progress here and believe a workable solution is in the making. I have created an internal issue to track my work: https://github.com/hypothesis/playbook/issues/342

A real quick recap of our discussions about blocking ports:

The main thing this is here to solve is accessing bad hosts (like localhost), not any particular port
We don't think the inclusion of port blocking was intentional in Squid, it's probably just copy paste from an example config
Blocking wide port ranges to match the Squid config has a performance cost in iptables
There's no attack you can do to us in port X you can't do on port 80, or 443

The summary is we don't think there's any advantage to blocking outbound ports which we make HTTP requests on. It also runs the risk of breaking sites. Therefore the port blocking part has been removed completely.

A solution has been deployed. Closing this ticket. Please let me know if you have any further questions.


UI FIX: Pokemon image is too small.

The pokemon image that is displayed in the card is too small as shown below:

Propose fix:
Note: If you can, you should use the existing CSS class provided and modify the said class to solve this problem.

The pokemon image should have the same height and width as the card (Parent); ideally should look like the image shown below:

Uploading the solution in a bit

Closing since #52 was merged.


SCC connector not updating new findings

Forseti version - 2.18.0
Issue :
Test bucket violation created (read permissions to everyone).
After run the email attached CSV report do recognize and report it as violation.
In SCC it's shown as new asset (aka seen and refreshed inventory) but the violation (findings) are not updated (showing last status) no matter what is chosen, nor where is clicked for refresh.
Where can we debug this issue and how to decide if an issue or mis-communication  / mis-configuration?

Hi @ttodo-softserve, can you please clarify if you have completed the integration between Forseti and CSCC?  Did you follow the steps here in this guide?
https://forsetisecurity.org/docs/latest/configure/notifier/index.html#cloud-scc-notification
For example, does anything else show up in CSCC for you, or is it only this test bucket that did not show up?
If you have setup CSCC already, you can set the forseti server into debug mode.  And then check if the following debug level shows up.
On the server vm, run this command:
forseti server log_level set debug

See the stackdriver log of the server vm like this:

Look for these debug logs:
https://github.com/forseti-security/forseti-security/blob/dev/google/cloud/forseti/notifier/notifiers/cscc_notifier.py#L246
https://github.com/forseti-security/forseti-security/blob/dev/google/cloud/forseti/notifier/notifiers/cscc_notifier.py#L257
Please let us know.

@blueandgold - thanks for clarifying! Appreciated.
Opening the debug I found the error - configuration.
Sorry for bothering, at least I've learned how to debug it on-the-fly.
Findings are updated after the forseti run is finished and results are updated to SCC.
Last question - when does the inventory update in SCC?

Upon each forseti run finish
or it's needed to do manually Assets - > RE-SCAN in SCC
or after an hour they will be reflected

Cheers and thanks again!

@ttodo-softserve Inventory in SCC is created and updated by SCC itself, which I believe happens periodically every few hours (you don't have to manually trigger a re-scan).  Forseti only updates the findings.
I will go ahead and close this issue.  If you have any other questions, please feel free to re-open this one, or a new issue.  Thanks.

Perfect! Thank you!


Unpacking azure-cli (2.0.47-1~xenial) over (2.0.45-1~xenial) freezes on microsoft/vsts-agent:ubuntu-16.04-standard

I am hosting my own Azure DevOps agents as Docker containers on AKS starting from the image:

FROM microsoft/vsts-agent:ubuntu-16.04-standard

Trying to install the latest version of az cli on the agents as per this article.
Always on the unpacking step, the console just freezes with no sign of downloading packages or error whatsoever:

Unpacking azure-cli (2.0.47-1~xenial) over (2.0.45-1~xenial) ...

Is there a way to see progress on this? The Unpacking just seems frozen and the exec fails at some point.
When I try to kubectl exec -it myvstsagent /bin/bash I get the following error:
kubectl exec -it vstsagent-jb2-5dd6598f66-wr4pp /bin/bash -n vstsagents
error: unable to upgrade connection: pod does not exist
However, I can see the pod is running:
kubectl get po -n vstsagents
NAME                                           READY     STATUS    RESTARTS   AGE
vstsagent-jb2-5dd6598f66-wr4pp   1/1       Running        0          24m

To see the progress of an apt operation, pipe to debconf-apt-progress. See this StackOverflow question.


v1.0 refers to an old release

We recently got a Dependabot PR (https://github.com/ministryofjustice/hmpps-probation-integration-services/pull/3619) attempting to upgrade our action version from 0.2.6 to v1.0, however it appears v1.0 refers to an older commit from 26/08/2020.
Should this tag be removed or renamed?

Same is happening for all GitHub Dependabot users. Please remove or rename the initial release from 2020, which is labeled v1.0.

this has been changed now


I want to add some processing to offline players in realtime multiplayer

I want to add some processing to offline players in realtime multiplayer.
what should I do?

Resolved over Gitter, not an issue.


Small tweaks

Creating a Requirements file, Gitignore, set API keys to environment variables

I have a branch with a few tweaks, just assign the above issue to me and I will make a pull request.

@david-marko


Router.back with force:true

When you use the back of the router, even though it brings you to the right url, in the background, it's processing another page that it should not.
I have an event of pageInit which logs the page name that has been initialized.
Then if I navigate from page1 -> page2 -> page3 -> page4 to go back to page2, the page2 will be shown properly but there will be a page1 init in the background.
I've tried to back in both Javascript and HTML:
HTML
Back

Javascript
mainView.router.back({'url': 'page2.html', 'force': true});

Log
Init :: page 2
Init :: page 3
Init :: page 4
Init :: page 2
Init :: page 1

Note
This only happens when I use the Router.back, everything works fine with Router.load. However, I want to use the back function for the reverse animation.

This is expected behaviour

I thought that when you use the Router.back function with the url parameter, it's supposed to go back to the specified url. So why is it normal behaviour that it's going to the provided url, and doing a back in background?

yes because for swipe back to work the previous page must be in DOM.
F7 only keep 3 pages in DOM so Page 1 left the DOM when Page 4 was loaded.
page:init wasn't run for Page 2 when you went back to it because it was still in DOM from last time
However it was run for Page 1 because it was reentering the DOM and needs to be ready for swipeback to work

use $router. reloadPage(url)


Cant take any game status from listener.

I try use example project. An result 0. Zero callback in console. Can some one help me?

Did you copy the cfg file to enable GSI?
Sent from my iPhone

On Jun 5, 2016, at 14:11, Alexander<EMAIL_ADDRESS>wrote:
I try use example project. An result 0. Zero callback in console. Can some one help me?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

solved problem. Wrong path. One more question. Can listener return game UI state? Like - callback when game ready. From ingame console:

ChangeGameUIState: DOTA_GAME_UI_STATE_LOADING_SCREEN -> DOTA_GAME_UI_DOTA_INGAME

I don't think GSI itself provides such information. But if someone wants to fork the repo and add support for reading Dota's log/console files. Feel free to do so.

Hi. I'm having trouble to receiving game statuses. Maybe i doing something wrong. I'll be glad if you guide me. So i created cfg file with such uri -> "http://localhost:4000/ ".
I launch game in play with bots mode, and nothing happened.Perhaps I have misunderstood something?

Omg, i put cfg in wrong path. Now it's okay.

Can someone reconfirm the full path for the config file. I'm pretty sure the path listed in the readme file is wrong as that isn't a valid path for Reborn. (Am I wrong?)

@sheldonj The path is correct. Official logitech integration uses that path, as well as PlaysTV.

@sheldonj Double check your directory name for typo, cause i'm stuck on same (i think same)problem.

Thanks guys, at work right now but will check.

I definetly have the config in the correct place, I can test my endpoint for receiving posts no problem and thats work.  Just not seeing any posts from Dota2, nor do I see any gamestate configs loaded in the console. I wonder what else it could be.

Run your program as Admin, it might not be able to create network listener.

Im not having any issues receiving network requests. I can send requests to the endpoint with Postman and it works just fine. it just seems that Dota2 client isn't actually sending the requests. When you launch the game do you see any thing in the Dota2 console about your gamestate_integration loading?

@sheldonj it little bit weird. But i still think that you something missed in config. If you don't see any requests from D2 it could be problem in config file, directory placement/namin or mby wrong Uri for listening game states. Also check Uri in your program and cfg file.

Make sure your .cfg file complies with the requirements listed for CSGO.
https://developer.valvesoftware.com/wiki/Counter-Strike:_Global_Offensive_Game_State_Integration

I ended up figuring it out, totally rookie mistake. I had just reinstalled a new computer (and I hadn't turned show file extensions on. So I was creating the config file with a .cfg.txt file extension by mistake, sorry for wasting everyone's time.

Hi there. Can someone confirm correct work with latest patch?

I played the game today with my keyboard lighting utility. It was operating just fine.

@antonpup  can you pls check *.cfg file path

The .cfg path did not change.


Is this project / repository maintained anymore?

Serious question....there are currently 42 open pull requests ranging back to 2015! Can we rely on any updates  / issues being processed?
kind regards
Martin

This repository is now collaboratively maintained by MrRio/James Hall and yWorks. A new release is definitively planned (probably mid/end Q2). Currently, we are quite occupied with other things, so the response time is quite high and pull requests are not merged immediately, but this situation should improve in the nearer future.

thank you very much for this answer and keep up the good work!


Intermittent authentication issues

I'm seeing issues with librespot intermittently failing to authenticate with Spotify. About half the time it will auth fine, the other half, it'll panic with "Authentication failed".
> RUST_BACKTRACE=full ./target/debug/librespot --username <redacted_correct_username> --password <redacted_correct_password>  --backend pipe --name Scrobbler
INFO:librespot: librespot aa86ebf (2017-07-09). Built on 2017-07-16.
WARN:mdns: Failed to register IPv6 receiver: Error { repr: Os { code: 49, message: "Can\'t assign requested address" } }
INFO:librespot::session: Connecting to AP "lon6-accesspoint-a21.ap.spotify.com:4070"
thread 'main' panicked at 'Authentication failed', /Users/davidfutcher/code/rust/librespot/target/debug/build/librespot-3c13ee3527cd9ab5/out/lib.rs:365
stack backtrace:
   0:        0x102f2ea63 - std::sys::imp::backtrace::tracing::imp::unwind_backtrace::hcc709dd93d82a051
   1:        0x102f3286d - std::panicking::default_hook::{{closure}}::he392668e67778d9e
   2:        0x102f3247b - std::panicking::default_hook::h2ade4aa81d4f8204
   3:        0x102f34fe7 - std::panicking::rust_panic_with_hook::h4573918fc9f80a34
   4:        0x1024b5de3 - std::panicking::begin_panic::h0b1eb0ebf12edac1
   5:        0x10269d160 - librespot::connection::authenticate::{{closure}}::h491faa13958321b4
   6:        0x10245425b - <futures::future::and_then::AndThen<A, B, F> as futures::future::Future>::poll::{{closure}}::{{closure}}::hd96b1d1983e98ece
   7:        0x10250009d - <core::result::Result<T, E>>::map::h94574c79f2fc8796
   8:        0x102452bb5 - <futures::future::and_then::AndThen<A, B, F> as futures::future::Future>::poll::{{closure}}::ha736e5897b455cb3
   9:        0x1025c7a8a - <futures::future::chain::Chain<A, B, C>>::poll::hd8581d0da50a829e
  10:        0x102451f23 - <futures::future::and_then::AndThen<A, B, F> as futures::future::Future>::poll::hcb4726b2369fd3be
  11:        0x1025e78d6 - <alloc::boxed::Box<F> as futures::future::Future>::poll::hb66495c588857fb5
  12:        0x1025c131b - <futures::future::chain::Chain<A, B, C>>::poll::h4c6b3c2e0fa8b777
  13:        0x102451e93 - <futures::future::and_then::AndThen<A, B, F> as futures::future::Future>::poll::hb039b3204627240c
  14:        0x10264664e - <futures::future::map::Map<A, F> as futures::future::Future>::poll::h1b2bbd294a6a0e33
  15:        0x102413dc6 - <alloc::boxed::Box<F> as futures::future::Future>::poll::h3db0fa643d2683cb
  16:        0x1024237a2 - <librespot::Main as futures::future::Future>::poll::h184fc0c4b39a1ebf
  17:        0x1023e036c - <futures::task_impl::Spawn<F>>::poll_future::{{closure}}::hc40a91b557b5eade
  18:        0x1023e0527 - <futures::task_impl::Spawn<T>>::enter::{{closure}}::hb1da2be55168403e
  19:        0x1024180d3 - futures::task_impl::set::{{closure}}::h196e435af1bd9dbf
  20:        0x1023e63f1 - <std::thread::local::LocalKey<T>>::with::ha138070957478d2a
  21:        0x10241800f - futures::task_impl::set::he094d1281dd89e6b
  22:        0x1023e0486 - <futures::task_impl::Spawn<T>>::enter::h88e4fea210ad3907
  23:        0x1023e02ef - <futures::task_impl::Spawn<F>>::poll_future::h49a47c2cd84ac369
  24:        0x1023d06ee - tokio_core::reactor::Core::run::{{closure}}::h1a510271d9c3a902
  25:        0x1023da6c4 - <scoped_tls::ScopedKey<T>>::set::h25adf020d1dcbc80
  26:        0x1023d0549 - tokio_core::reactor::Core::run::h5f9256539d4bee9a
  27:        0x10242473b - librespot::main::h98d5680561ad6527
  28:        0x102f3611a - __rust_maybe_catch_panic
  29:        0x102f35400 - std::rt::lang_start::ha1f4f04346e51ef5
  30:        0x102424909 - main

This is the same problem as issue #220, and has been fixed in PR #218. Just waiting on a merge from @plietar...

Yep confirm it's fixed by PR #218, cheers @sashahilton00.

Glad to hear it. Please close the issue :)


[INTERNAL] Bump terser from 5.29.1 to 5.29.2

Cherry-pick of https://github.com/SAP/ui5-builder/pull/1000

coverage: 94.608%. remained the same
when pulling 4d979f041ecc1f19d306646441ea4db56f8b7412 on update-terser
into 4661a164132c5edcc801111d69bf68a2f312bee0 on v3.


Game crashes using Lost template

What you were trying to do
What actually happened
How to reproduce

Step 1
Step 2
Step 3

Log details and game version
Computer details

Just started the game, looked around at settings, then came back, started a single player game selected the "Lost" template, and the game crashed.
https://pastebin.com/E3NrgHbi

So, looking through the log, the line java.lang.OutOfMemoryError: Java heap space indicates that this crash was caused by running out of memory.
If you try reducing settings like view distance as well as launching with more memory available it should fix your problem.
If you still get this crash even with this then please re-open the issue as there is a possibility there could be a memory leak of some sort.


Upgrade Opera.app to v28.0.1750.48

Coverage remained the same at 83.75% when pulling 5a4fd4cf054e83b95e40490c2b1ea2df56bffed4 on Ngrd:update-opera into fc0647bec996877ef41984511d012d2da72046e6 on caskroom:master.


Mock App function circuits in Goblin recursion tests

The real kernel circuit will perform two recursive verifications (eventually, folding verifications). Currently we don't have a mocked app circuit at all in the goblin recursion tests so we simply construct two recursive verifications of the previous kernel proof as a stand in approximation. We need to actually construct an app proof since that will be a significant computation. For our purposes the app proof can simply be a circuit of given size comprised entirely of arithmetic gates.

Closed by https://github.com/AztecProtocol/aztec-packages/pull/4242


LargeantAnalysis module added

Modified version of previous AnalysisExample module

Old AnalysisExample module is located at https://github.com/DUNE/lbnecode, which had not been updated for 9 years. The old AnalysisExample is based on previous LArG4. While this new LargeantAnalysis module is based on the LArG4 Main.

If the purpose is to provide an example, then there is a maintained version in larexamples
https://github.com/LArSoft/larexamples/tree/develop/larexamples/AnalysisExample
I think that's where the one in lbnecode was copied from.
What is added/modified here to make it useful for duneopdet, specifically?


Rework entry page based on requirements from last meeting

DESC

update base design of entry page according to new requirements
https://www.figma.com/file/xmDdlfAuqdL5gFd1TF1Z6N/reinheritHOME?node-id=0%3A1
figma based
repeat ReInHerit colors on different locations on page

TODOs

[x] implement
[x] include image attributions
[x] include images to static folder
[x] repair base layout on rest of the subpages

implemented


Spring Data CosmosDB won't work If azure.cosmosdb.database is not set

CosmosDB Starter version: 2.0.7
Spring Boot version: 2.0.6

Description
When azure.cosmosdb.database is not set in application.properties or informed somehow, the DocumentDbAutoConfig fails with the following message:
An exception occurred while running. null: InvocationTargetException: Error creating bean with name 'todoListController': Unsatisfied dependency expressed through field 'todoItemRepository'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'todoItemRepositoryCosmosDB': Unsatisfied dependency expressed through method 'setDocumentDbOperations' parameter 0; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'com.microsoft.azure.spring.data.cosmosdb.core.DocumentDbOperations' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}

Detailed stack trace attached: stacktrace.log
Expectation
Looking at the source code, it is indicated that azure.cosmosdb.database is optional. Therefore, it should create a database either with a default name, or with the same name as the initial collection (defined with @Document in a model class).

Hi @brunoborges, thanks for reporting issue, have opened a PR to fix the issue to allow printing accurate error message, now azure.cosmosdb.database is required.


I've noticed that there's like 6 missing enchantment books from the legacy console creative menu.

https://github.com/Wilyicaro/Legacy-Minecraft/assets/39068731/4f51f51c-f9ce-4242-a392-52d662bde24b

Can you list them for me? I don't remember, as this organization wasn't made by me

Can you list them for me? I don't remember, as this organization wasn't made by me
Quick shot III
Piercing IV
Multishot
Sweeping edge III
Swift sneak III
Soul speed III


Problems using Mailer from /sender address

Laravel Version: 5.4
PHP Version: 7.0
Database Driver & Version: MySQL 5.7

Description:
I have Mailer functionality built into my business app. I'm using smtp to send out client emails. The emails should be send from the client email address but instead are sent from the system. In other words the sender appears as the system email no matter what I try. I do not have a default from address in my mail config file nor in my .env file.  I'm sure the email is sent to the Mailer since these are the errors I get:
If I use $sender as a dependency as shown in the code, it sends but from the system email. When I send an email it sends from<EMAIL_ADDRESS>not the user email
If I use $from as a dependency to inject to the email I get an even worse error.

Below is another error I get when I tried to debug the Mailer class in the framework. In the image the sender email is correctly passed to the Mailer send method.

According to the documentation this is a lot easier than what I'm experiencing.
Steps To Reproduce:
Here is my Mailable class
class UserEmail extends Mailable
{
use Queueable, SerializesModels;
public $sender,$name, $subject, $content, $attach;

/**
 * Create a new message instance.
 *
 * @return void
 */
public function __construct($sender, $name, $subject, $content, $attach)
{
    //
    $this->sender = $sender;
    $this->name = $name;
    $this->subject = $subject;
    $this->content = $content;
    $this->attach = $attach;
}

/**
 * Build the message.
 *
 * @return $this
 */
public function build()
{

                return  $this->from($this->sender, $this->name)
                          ->view('mailbox.sendmail')
                          ->subject($this->subject)
                          ->with(['t_body' => $this->content, 'title' => $this->subject])
                          ->withSwiftMessage(function ($message) {
                              $message->getHeaders();
                          });

                      for ($i = 0; $i < count($this->attach); $i++) {
                          $this->attach($this->attach[$i]);
                      }
      }

}

And in my Emailcontroller I'm sending the variables
  $sender = (isset($configuration['serveruser']) && $configuration['serveruser'] != '') ?
  $configuration['serveruser'] : Auth::user()->email ;
                $name = Auth::user()->name;
                $subject = $request->subject;
                $content = $request->input( 'content' );
                $attach = $files;

   Mail::to($request->to)->send((new UserEmail($sender, $name,$subject, $content, $attach))->onQueue('emails'));

mail.php
  //    'from' => [
 //        'address' => '',
//        'name' => '',
//    ],

Our system is live and I can't fix this and users will not want to send emails. Is it a bug? according to the documentation, if I set $this->from(address, name) , it sends from that email address.

Can you try setting the config directly instead of an array through the ternary?

Please ask on the forums, this repo is for bug reporting only. You can use https://laracasts.com/discuss or https://laravel.io/forum which are forums with a very large community of developers helping each other.

I just noticed I am also having this issue after updating to Laravel 5.4 which from what I understand uses SwiftMailer but it is completely ignoring the From field and instead of sending emails directly from the app email.

From field on 5.5 seems to be working fine for me.

@themsaid you said that with laravel 5.3 and I updated to 5.4. Notifications delay isn't working either and it again said to work in the documentation. what kind of bugs or problems do you actually solve here? My app is live and this is a real issue and you have closed it twice without helping .. the last time you said 5.3 was not supported and I guess this time 5.4 isn't @taylorotwell please help with these 2 issues if possible.

@nivanmorgan thing is I really can't replicate any of these issues on 5.4 or 5.5
Delay is working fine for me as well as the sender field in Mailers, I pointed you towards the forums because it's a more active QA community where you can share your issue and people will try to help, including myself.
We only keep bug reports here so we can work on it and fix, but if we can't replicate the issue then it's probably a mis-configuration from your side, so the forums is a better place for this.
I hope you get me point.

@themsaid you were right. I upgraded to 5.5 and it still didn't send from the sender  address so I changed to mailgun and it worked  fine. notify I'm not sure of though. I had use jobs to delay it. Thx

i'm facing same issue with laravel 5.7.. can some help me why from field in not working and sending default email instead of client email


Kubeconfig issues on crc vm

This is the last issue I’ve got with the setup atm. Whenever I try to use the OC command to do anything cluster related, is days the kubeconfig file isn’t defined(not exact words) and I should point the KUBECONFIG path env variable to the config file. I’ve tried doing that with the /etc/kubernetes/kubeconfig file and also copying the installation config file to the vm and using that but the first one just says permission denied and the second one says openshift-machine-config-operator doesn’t have access to any namespaces.
How can I get around that?

After a successful snc.sh run, the kubeconfig file is in crc-tmp-install-data/auth/kubeconfig

Agni thought it only needed to be copied, sorry. How would I run it?

Also I'm just using whatever snc happened to install.

export KUBECONFIG=crc-tmp-install-data/auth/kubeconfig; ./oc get nodes

Closing the issue as I don't think there's a bug here.

Error from server (Forbidden): nodes is forbidden: User "system:serviceaccount:openshift-machine-config-operator:node-bootstrapper" cannot list resource "nodes" in API group "" at the cluster scope
Found a bug for ya😅 that happens whenever I try to do anything oc related


storage: Clarified Append API comment.

👋🏽 I found the newly changed Appender interface commentary very vague, so attempted to fix it. Can you help me verify if I understood it right?
cc @tomwilkie @roidelapluie @codesome @csmarchbanks  🤗
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

@bboreham would mind taking look as well on the comment clarification? What would be the most clear? (tiny change)

Awesome, adjusted the interface commentary. Could you double check @tomwilkie @roidelapluie ?

Hello @bwplotka, we have looked at it during the bug scrub , the interface has changed with the addition of examplars and native histogram. We are closing this issue, feel free to rebase and adapt the PR to the new interfaces if you still find it appropriate.


Added note regarding minDate and maxDate

The minDate and maxDate props require that the date format be the same as what is specified in the format prop.
Related to issue #89.

Coverage remained the same at 97.368% when pulling ccc58f9dfa0c7981435a59b5b8ea9dcacb4e21ca on robertcoopercode:patch-1 into 2a209bd4f11a84ff4543fbde930fed60dcbc0f68 on xgfe:master.

RIP Pull Request


(1.14) [FLINK-24065][connector] Upgrade the state of TwoPhaseCommitSink to support empty transaction after finished

This PR picks https://github.com/apache/flink/pull/17120 onto release-1.14

@flinkbot run azure


Rename rosflight package to rosflight_io

With the current naming scheme, there has consistently been confusion as to what actually is "rosflight" and what the documentation on the website is referring to. To make things more clear, we are going to call "rosflight" the entire flight controller stack, excluding rosplane and roscopter. This includes the firmware, the rosflight_io node, the simulator, and all other supporting packages directly related to rosflight.
The rosflight firmware will be called rosflight_firmware instead of just firmware, and the package that contains the rosflight_io node will be renamed to be rosflight_io.

I also moved the udp board implementation directly into the firmware repository since the udp board doesn't require ROS, and moving it simplified and clarified the rosflight_firmware package found in this directory.

After some discussion on pull request #408 on the firmware repo, we decided moving udp_board to the firmware wasn't the best place for it. Putting it next to the SIL in the rosflight_sim package seems like a better idea.