aioesphomeapi-2.6.1 removal routine broken

First, Happy new Year 2020!
And thanks a lot for your fabulous work in the last years.

Home Assistant release with the issue:

0.103.5
Last working Home Assistant release (if known):
sorry, don't know
Operating environment (Hass.io/Docker/Windows/etc.):

bare metal install on x86_64 Gentoo Linux, no docker, no virtualenv
Integration:

ESPHome aka aioesphomeapi-2.6.1
Description of problem:
While removing an ESPHome device from integrations list, about 1500 lines of errors are written to server.log. The device disappears (in UI), but if re-added, all it's entities show up with "_2" added to their entity_id. A manual clean up in .storage/core.entity_registry is neccesary to remove it entirely.
.storage/esphome.{key} is not deleted.
In the following example, esphome.c5d7a702e6174a0aa680a6a654ffcc82 is the key of the device to be removed,
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
panel_iframe:
  esphome:
    title: ESPHome
    icon: mdi:chip
    require_admin: true
    url: https://{hostname}:6053/#

Everything else is added via Configuration -> Integrations -> ESPHome
Traceback (if applicable):
attached as file, simply is too big. starts with:
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/homeassistant/core.py", line 673, in _async_remove_listener
    self._listeners[event_type].remove(listener)
  File "/usr/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 550, in __eq__
    if self.unique_id is None or other.unique_id is None:
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/__init__.py", line 575, in unique_id
    if not self._static_info.unique_id:
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/sensor.py", line 87, in _static_info
    return super()._static_info
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/__init__.py", line 538, in _static_info
    info = self._entry_data.info[self._component_key].get(self._key)
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/__init__.py", line 533, in _entry_data
    return self.hass.data[DATA_KEY][self._entry_id]
KeyError: 'c5d7a702e6174a0aa680a6a654ffcc82'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.7/logging/__init__.py", line 1025, in emit
    msg = self.format(record)
  File "/usr/lib/python3.7/logging/__init__.py", line 869, in format
    return fmt.format(record)
  File "/usr/lib/python3.7/site-packages/colorlog/colorlog.py", line 119, in format
    message = super(ColoredFormatter, self).format(record)
  File "/usr/lib/python3.7/logging/__init__.py", line 608, in format
    record.message = record.getMessage()
  File "/usr/lib/python3.7/logging/__init__.py", line 369, in getMessage
    msg = msg % self.args
  File "/usr/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 565, in __repr__
    return "<Entity {}: {}>".format(self.name, self.state)
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/__init__.py", line 589, in name
    return self._static_info.name
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/sensor.py", line 87, in _static_info
    return super()._static_info
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/__init__.py", line 538, in _static_info
    info = self._entry_data.info[self._component_key].get(self._key)
  File "/usr/lib/python3.7/site-packages/homeassistant/components/esphome/__init__.py", line 533, in _entry_data
    return self.hass.data[DATA_KEY][self._entry_id]
KeyError: 'c5d7a702e6174a0aa680a6a654ffcc82'
Call stack:
  File "/usr/lib/python-exec/python3.7/hass", line 11, in <module>
    load_entry_point('homeassistant==0.103.5', 'console_scripts', 'hass')()
  File "/usr/lib/python3.7/site-packages/homeassistant/__main__.py", line 388, in main
    exit_code = asyncio_run(setup_and_run_hass(config_dir, args))
  File "/usr/lib/python3.7/asyncio/runners.py", line 43, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.7/asyncio/base_events.py", line 566, in run_until_complete
    self.run_forever()
  File "/usr/lib/python3.7/asyncio/base_events.py", line 534, in run_forever
    self._run_once()
  File "/usr/lib/python3.7/asyncio/base_events.py", line 1771, in _run_once
    handle._run()
  File "/usr/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/lib/python3.7/site-packages/homeassistant/helpers/entity_platform.py", line 432, in async_remove_entity
    await self.entities[entity_id].async_remove()
  File "/usr/lib/python3.7/site-packages/homeassistant/helpers/entity.py", line 483, in async_remove
    self._on_remove.pop()()
  File "/usr/lib/python3.7/site-packages/homeassistant/core.py", line 616, in remove_listener
    self._async_remove_listener(event_type, listener)
  File "/usr/lib/python3.7/site-packages/homeassistant/core.py", line 681, in _async_remove_listener
    _LOGGER.warning("Unable to remove unknown listener %s", listener)
Unable to print the message and arguments - possible formatting error.
Use the traceback above to help find the error.

Additional information:

And here's the complete log:
ESPHome-Removal-Error.log

Ping :-)

The device disappears (in UI), but if re-added, all it's entities show up with "_2" added to their entity_id

That is expected (and not related to the other errors you described). When removing a device from the config entries, its entity registry items aren't removed. That is because of the HA core, not the ESPHome integration (basically, removing+re-adding an integration should retain the data in entity registry; but if the unique IDs change by for example changing the ESP, well _2 will be added).

.storage/esphome.{key} is not deleted.

IMO, not really an issue (file is small, and the key is random so when you add a new config entry it will not conflict).

While removing an ESPHome device from integrations list, about 1500 lines of errors are written

That is more of an issue. I looked at it a while back, and the issue is that the core somehow accesses some Entity attributes after the Entity has already been removed (so potentially also more of a core issue).

OK, does not look good, but can live with that. Think I'll set up another test machine for the lab/experimental stuff, so the real registry won't be poisoned too much with test entities. From time to time the registry has to be cleaned anyway. Thanks a lot, and also for ESPHome in general, like it very much.


Uncaught Error: Class 'OdooClient\Client' not found

Im Testing using Odoo 10 CE on internal network
I got Fatal error: Uncaught Error: Class 'OdooClient\Client' not found in C:\xampp\htdocs\odoo-client-master\index.php:10 Stack trace: #0 {main} thrown in C:\xampp\htdocs\odoo-client-master\index.php on line 10
how can i fix this ?
and here my codes
Thanks before

Don't know if this would be accepted here, but i found a composer library built upon robroypt/odoo-client. Basically what it does his help bundle and autoload the libraries into your PHP project. Check it out here:
[https://github.com/tbondois/odoo-ripcord]( tbondois /
odoo-ripcord )
all you need is a simple composer require tbondois/odoo-ripcord and viola this whole library is pacaged for you


move checking ptr tracking on item pop into cold helper function

Before:
Benchmark 1: cargo miri run --manifest-path bench-cargo-miri/serde1/Cargo.toml
  Time (mean ± σ):      6.729 s ±  0.050 s    [User: 6.608 s, System: 0.124 s]
  Range (min … max):    6.665 s …  6.799 s    5 runs

Benchmark 2: cargo miri run --manifest-path bench-cargo-miri/unicode/Cargo.toml
  Time (mean ± σ):     20.923 s ±  0.271 s    [User: 20.386 s, System: 0.537 s]
  Range (min … max):   20.580 s … 21.165 s    5 runs

After:
Benchmark 1: cargo miri run --manifest-path bench-cargo-miri/serde1/Cargo.toml
  Time (mean ± σ):      6.562 s ±  0.023 s    [User: 6.430 s, System: 0.135 s]
  Range (min … max):    6.544 s …  6.594 s    5 runs

Benchmark 2: cargo miri run --manifest-path bench-cargo-miri/unicode/Cargo.toml
  Time (mean ± σ):     20.375 s ±  0.228 s    [User: 19.964 s, System: 0.413 s]
  Range (min … max):   20.201 s … 20.736 s    5 runs

Nothing major, but we'll take it I guess. :shrug:
Fixes https://github.com/rust-lang/miri/issues/2132

@saethlin do you think this makes sense, or should we close the issue without further action?

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

Rebased after @saethlin 's PR landed, and found another cold path in the same item_popped function. It now contains barely any code itself.  Strangely it seems like a 2σ regression on unicode.
So overall my inclination is to land this? I can't see how it could hurt, anyway...

Hm, give me a bit to twiddle with this...

Oh yeah this is perfect, item_popped inlines now and the hot path through it doesn't call any functions, which is the transformation I was hoping for :+1:
I'm not surprised this is ~perf neutral on most functions and shows the most effect on slice-get-unchecked, because that benchmark stresses this code path more than anything else. A 2σ regression could easily be noise due to the scheduler or some minor perf effect we don't have control over.

item_popped inlines now

Should we add #[inline] or even inline(always) to to ensure this remains the case, or is that a bad idea?

I think that #[inline(always)] is a bad idea, and #[inline] doesn't apply because we aren't trying to enable cross-crate inlining. In my experience applying it when a function isn't being inlined almost always results in slower code, and the way to get the inlining you want is to extract cold paths into #[inline(never)] functions as you've done here.
Ideally LLVM could extract cold paths into functions via PGO but I've never seen that happen. And unlike profile data we know what paths are guaranteed to be cold, as opposed to where we just don't have good profiles yet.

Okay, fair. I thought #[inline] would also shift some inlining weights, not just help for cross-crate things, but I don't really know.
Thanks for the review!
@bors r+

:pushpin: Commit 3bd0e8a2cac5b6141261b11c0d8b12e94ca908d3 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 3bd0e8a2cac5b6141261b11c0d8b12e94ca908d3 with merge af2c50fb89840852c5181c6e8aeb7be94d85f8df...

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

It does add an inlining hint, but the hint is small, and some people including me and some people in LLVM keep wanting to get rid of it, and the fact that both rustc's implementation of #[inline] and C/C++ inline do two things is kind of a problem because you can't get the ODR behavior or the cross-crate inlining without the inline hint as well.
And the fact that it adds a hint I think means that it would be worse to add in a situation like this. What I would really like is a #[rustc::warn_if_not_inlined] or something like that.

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing af2c50fb89840852c5181c6e8aeb7be94d85f8df to master...


Git Synchronize changes doesn work

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.62.2
OS Version: Ubuntu 20.04

Steps to Reproduce:

I did changes in repository files
I click on synchronize files to pull and push commits
This is the error output

> git pull --tags origin dev
remote: HTTP Basic: Access denied
fatal: Authentication failed for 'https://gitlab.com/juantorbay/agripac-acua.git/'

This is the other error:

> git pull --tags origin dev
remote: HTTP Basic: Access denied
fatal: Authentication failed for 'https://gitlab.com/juantorbay/agripac-acua.git/'

I get the same error when I try to clone a repository.

VS Code Version: 1.62.2
OS Version: Pop!_OS 21.04

Steps to Reproduce:

Click Clone Repository
Select a private Repository and allow access in the browser pop-up
I get this error

> git clone https://github.com/pintariching/DelavnicaNET.git /home/tilen/Documents/DelavnicaNET --progress
remote: Repository not found.
fatal: Authentication failed for 'https://github.com/pintariching/DelavnicaNET.git/'

I fixed the problem temporarily by following the tutorial on this website: https://florianwinkelbauer.com/posts/2021-05-27-git-gnome-keyring/.

I had same thing pop up on Friday 12th November 2021 except it was with Github and I fixed it by getting a token issued by github and adding it to git credentials on my local machine

Hi all, I have the same problem, I did try the method mentioned by @pintariching but it did not work. @dannyb2018, could you please describe the steps you took that solved the problem? Thanks.
FYI: it worked fine for me two days ago but I encounter the error yesterday. here is the error I see from my computer (Ubuntu Linux 21.04, VS code 1.62.2):
git push
remote: Repository not found.
fatal: Authentication failed for ...

Hi,
I had the same issue with vscode when I was trying to clone a private repository to my local machine.
Ubuntu: 18.04 LTS
VSCode version: 1.62.2
The error also occurs for other git workflows like pull,push as well.
The error for the git workflows are as follows:
git clone
remote: Repository not found.
fatal: Authentication failed for:

git pull , git push
remote: Repository not found.
fatal: Authentication failed for

Remove the credentials for git that you currently have

Add libsecret:
sudo apt-get install libsecret-1-0 libsecret-1-dev
cd /usr/share/doc/git/contrib/credential/libsecret
sudo make

make libsecret the credential helper:
git config --global credential.helper /usr/share/doc/git/contrib/credential/libsecret/git-credential-libsecret

Get a token for logging into Github:
https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token

Do a git push and it will ask for user name - use your normal one - and when it asks for the password give the token

So far so good for me.  No idea why it stopped working first

@dannyb2018: Thanks a lot for your prompt response!
a follow-up question, could you please elaborate on the 1st step a little more?
How did you remove the credential? is there an instruction to do it on VS code or a linux command for it? additional info would be appreciated. I think I perform steps 2-5 fine.
Thanks!

I think that he means that by removing credentials, you can use either a GUI tool like seahorse or a CLI tool like secret-tool (you can get them with sudo apt-get install seahorse or sudo apt-get install secret-tool)
Some distributions already have these installed and you can find them if you search for something like keyring.

Thanks, @dannyb2018 and @pintariching for your response. I did try your suggestions and a few other random things from stackoverflow.com that I cannot recall and reproduce what I did including restarting my computer, uninstallation, and reinstallation of GitHub extensions. looks like git push works fine for me now.
Hope that someone could reproduce/confirm the fix and share it here so that others could be benefited from it.

I'm facing the same issue, starting from today after I've updated to latest VScode version. I'm not using gnome-keyring or seahorse to store git credentials.

VS Code Version: 1.62.2
OS Version: Ubuntu 20.04

Pulling from private gitlab server using gnome-terminal concludes successfully. Pulling from VSCode terminal results in error:
remote: HTTP Basic: Access denied

Seahorse does not have entries for git credentials.

/duplicate of #137183 which is scheduled to be fixed in upcoming recovery release (1.62.3)

I did change from HTTPS to SSH url and it works for me!


Fix missing rust dependency that cause "Failed to install base requirements"

Fixes https://github.com/nlp-with-transformers/notebooks/issues/96
Testing
After patch:

Before patch:

Hi @lvwerra - checking to see if this is okay - this fixes #96 which a few people have been seeing.
Thank you for the book and the code 🙏


loss layers for rpn and new operator for calc rpn bounding box in gpu side

Somehow the commit history is messed up. A lot of previous changes is shown in your PR.
Could you squash your commits with git rebase?

@antinucleon @precedenceguo @winstywang Could you review the rpn part in the last commit?

OK, I will clean my codes.
But could you help me to check my rpn related codes ?

Something might be useful: kCoordLabel are raw input coordinates instead of (dx, dy, dw, dh).

@piiswrong I have been squashed the commits.

@precedenceguo kCoordLabel is a coordinates matrix, it is convenient to do matrix calculation.

seems that no people is interest in it 😞

I'm not familiar with faster rcnn and it's not clear to me what this is for. The guys that did the faster RCNN example seems to be busy at the moment.
It should help if you can integrate this into the faster rcnn exmaple or add doc and tests that explain what these ops do

One comment, this PR seems still contains additional code that has nothing todo with RPN, e.g. load from bytes, which should be deleted


Return cached results too!

The location of the return statement in the recently merged #439 caused cached results for the mod list to never be output to the browser, this was missed by tests because the cached logic isn't covered (we only test these queries once, not twice)

:+1:


Connection reset when submitting transaction and ledger close time exceeds 60 seconds

I think it should return 202 Accepted instead. Developer would have to resubmit a transaction to check it's status.

But I think scott's point was that the person developing against horizon is going to have to handle the time out anyway (in cases of network failure etc) so why bother to have this extra error state?

Because it's confusing. Connection reset may be caused by problems with Horizon but also by problems within the network of organization that's using Horizon. 202 Accepted is a clear indication that the transaction was accepted but another request must be sent to check it's status.

I don't understand why you need to know that horizon accepted the transaction.  The fact that it accepted the transaction is zero guarantee that it will show up in the ledger.  While I might be convinced to presenting a 500-class error on timeout at horizon occurs because of consensus lag, I am adamantly against returning something that is in the 200-class:   Look at most code that checks for "was the request a success" and you'll see something naive like:
if code >= 200 && code < 300 {
  doSomeThingOnSuccess()
}

All that code will start behaving incorrectly if we return a 202 for transactions that will potentially fail.

It's worth noting that the whole point of horizon's transaction posting subsystem is to simplify integration... If you prefer to manage the state yourself, just post directly to stellar core and deal with all the complexity/power.

Yeah, maybe 202 Accepted is not a right choice. But I think returning 500-class error can be a good idea. My point is that connection reset gives you no idea what happened: ledger close delays/internal network issues/external network issues/horizon crash and so on.

Yep, I think that is valid.  It will reduce the instances of feedback-less errors while keeping the integration requirements the same.  I'll look into what it takes to fix later today.  IIRC, the choice to let the golang http layer time the connection out was made because that was the default behavior and required no work.


Prevent canvas of getting smaller than the container

Hi,
I'm migrating the cropper from v0.9.1 to the recent version and I'm am wondering if there is a way to prevent the canvas of getting smaller than the container. Actually its not a problem of size rather the canvas should always fill out the complete container.
In strict mode I'm using right now the image can be moved to the boundaries of the cropper but i would like to expand this behaviour to the container.
In v0.9.1 it was the normal behaviour if I'm not mistaking.
Thank you!

Try this:

var $container = $('#container');

$('#image').cropper({
  minCanvasWidth: $container.width(),
  minCanvasHeight: $container.height()
});

Thanks, thats a step in the right direction. But i can still move the image so far that i can see the checkered background. I never want to be able to do that.
It should always look like the first one and never like the second:

I think i'll somehow need to check the coordinates too and prevent it to happen with the help of the cropmove.cropper event.

@fengyuanchen Any thoughts on this? I would really appreciate some input!

As you say, you can use cropmove.cropper event, why not try it?

@fengyuanchen  great work! man.

I'm having a hard time finding a good result. I'm having a couple of problems, maybe you some input on that.

Setting minCanvasWidth and minCanvasHeight does not work as expected. Instead of setting the exact width and height, it always shows the full picture.  If the image is landscape it will take the width as minCanvasWidth and doesnt respect the minCanvasHeight and vice versa.

The use of the cropmove.cropper and the zoom.cropper events to prevent from the image from moving out of focus is working but have a really bad user expericence.

        'cropmove.cropper': function (e) {

            var CanvasData = $(this).cropper('getCanvasData');
            var DeltaWidth = $container.width() - Math.round(CanvasData.width) ;
            var DeltaHeight = $container.height() - Math.round(CanvasData.height);

            if (Math.round(CanvasData.top) >= 0) {
                CanvasData.top = 0;
                $(e.target).cropper('setCanvasData', CanvasData);
            }

            if ( Math.round(CanvasData.left) >= 0 ) {
                CanvasData.left = 0;
                $(e.target).cropper('setCanvasData', CanvasData);
            }

            if (Math.round(CanvasData.top) <= DeltaHeight ) {
                CanvasData.top = DeltaHeight;
                $(e.target).cropper('setCanvasData', CanvasData);
            }

            if (Math.round(CanvasData.left) <= DeltaWidth) {
                CanvasData.left = DeltaWidth;
                $(e.target).cropper('setCanvasData', CanvasData);
            }

        }

I made a fiddle for that http://jsfiddle.net/tilocgn/u28b2xfd/3/

@tilocgn

Why not use the e.preventDefault?
Can you just call the setCanvasData method once in each event handler rather than several times?
What is this: CanvasData.left <= DeltaWidth(left < width)?!
Are you sure use >= and <= is fine? Why not > and <?

Thank you for your reply. e.preventDefault() does not work because after it is set once the canvas and cropper stops moving.
CanvasData.left <= DeltaWidth is the maximum offset the image should be able to moved to if it is bigger than the canvas. It works like this but not very good. When you try to move the image away from the canvas it starts shaking and oddly enough when you pull the image fast enough you can create a space between image and canvas.
Do you have any other idea what i could do here? And do you have a suggestion about the other problem i had:
"Setting minCanvasWidth and minCanvasHeight does not work as expected. Instead of setting the exact width and height, it always shows the full picture. If the image is landscape it will take the width as minCanvasWidth and doesnt respect the minCanvasHeight and vice versa."
Thanks again for your help and the awesome tool you built!


Error code 9

What does the error code 9 mean?
get request URL : https://api.qwant.com/api/search/images?q=stackoverflow
response: {"status":"success","data":{"result":{"items":[]},"error_code":9}}
What does the error_code 9 mean ?

@lohithpro I think this is related to this issue. I guess you need to pass some more params to your query.

got rid of the error by adding a Mozilla user-agent to the request.
		'User-Agent': 'Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:69.0) Gecko/20100101 Firefox/69.0',
	}```
with a home made user-agent i was stuck with error 9

@kokmok You piqued my curiosity! You don't have this issue at all by adding this User-Agent header?  Definitely need a lil rewrite of my program now. THANK YOU


Unable to specify localStoragePath

The path passed to QWebSettings->setLocalStoragePath() in webpage.cpp is hardcoded to QDesktopServices::DataLocation
This prevents setting any localStoragePath in a config file or specifying --local-storage-path on the cmdline.
The pull request #12042  I have created allows specifying localStoragePath properly

Fixed in https://github.com/ariya/phantomjs/commit/bc294d38d2618a24acc420f2da7a49061da82f5d


Compressed Templates - fail to upload a bz2 file

ISSUE TYPE

Bug Report

COMPONENT NAME

Management, Templates

CLOUDSTACK VERSION

4.16.0-Snapshot

CONFIGURATION

Basic Zone
KVM
OS / ENVIRONMENT

Ubuntu 20.04
SUMMARY

I wanna upload a compressed *.qcow2.bz2 template from my local http server.
bku@cloud:~/webserver$ file ubu-server.qcow2.bz2
ubu-server.qcow2.bz2: bzip2 compressed data, block size = 900k

But when I try to upload it thru a web GUI, I'm getting the following:
"status": "Failed to install: Template content is unsupported, or mismatch between selected format and template content. Found  : POSIX tar archive (GNU) (bzip2 compressed data, block size = 900k)"
I selected KVM, QCOW2, Extractable, Public, HVM, Featured checkboxes
STEPS TO REPRODUCE

Upload a *qcow2.bz2 file

EXPECTED RESULTS

Successfully upload a *.qcow2.bz2 file

ACTUAL RESULTS

"status": "Failed to install: Template content is unsupported, or mismatch between selected format and template content. Found  : POSIX tar archive (GNU) (bzip2 compre
ssed data, block size = 900k)"

@tetra12 are you trying the browser based upload or registering the http url? If it is browser based upload can you try to uploading a tar.bz2 or use qemu-img to compact the qcow2 and upload it?

Closing as can't reproduce


Update sqlmap-xml.xml

correct wrong typing

coverage: 86.597%. remained the same
when pulling 0beec9249a04c89a55249f8752571744ccb452fe on wizneo:patch-1
into 311575892b9ff1d222a28376ed747cf349f391a9 on mybatis:master.


Add Youtube url instead of username

I would like to add my youtube url instead of my username in the config.yml file. Because I'm not the only one that uses mine on YouTube.
So how could I do that?

It doesn't work. It shows the url of the channel but it redirects to a 404 page on my site.


isInline() started to fail since v2.10.27

Package
filament/forms
Package Version
v2.10.27
Laravel Version
v8.83.4
Livewire Version
No response
PHP Version
PHP 8.1
Bug description
After upgrading to v2.10.27 we got some failing code when checking if a field is inline with isInline(). I assume it might be related to https://github.com/laravel-filament/filament/pull/1883.
Steps to reproduce
$toggle = Toggle::make('is_admin')->inline();

$toggle->isInline();

or
$toggle = Checkbox::make('is_admin');

$toggle->isInline();

Error : Typed property Filament\Forms\Components\Component::$container must not be accessed before initialization

Relevant log output
Error : Typed property Filament\Forms\Components\Component::$container must not be accessed before initialization
 /Users/niels/dev/spatie/***/vendor/filament/forms/src/Components/Concerns/BelongsToContainer.php:21
 /Users/niels/dev/spatie/***/vendor/filament/forms/src/Components/Concerns/HasInlineLabel.php:20
 /Users/niels/dev/spatie/***/vendor/filament/forms/src/Components/Concerns/CanBeInline.php:20
 /Users/niels/dev/spatie/***.php:120

Yeah, why are you calling isInline() like that?

Yeah, this would have broken pre-v2.10.27 if a closure was used inside inline().

I'm just interested to see why isInline() was being called directly in the first place, we can recommend a more robust solution.

The public method broke in a patch release, but tbh it shouldn't have been working before anyway.
Those methods need to be marked as public for us to interact with them correctly, but should likely be marked as @internal and used at your own risk since we don't document them as part of the public API for the forms package.
All fields need to be bound to a ComponentContainer, so I'd recommend taking this approach in your test. You can create a fake Livewire component for the container too.


Overwrite functionality is missing

https://github.com/emc-mongoose/mongoose-storage-driver-fs/blob/05c1e8e2fcc96a34790c1a916c483aee1046ac8a/src/main/java/com/emc/mongoose/storage/driver/fs/BasicFileStorageDriver.java#L275

fixed in v0.1.7


log/http: Update Logger API and http to use base/log

log: Add Send() to Logger API
In some cases, you'll want to log an event without providing a message. For example, in http/response.go:62:

		w.log.With(log.Fields{
			"method":    w.request.Method,
			"path":      w.request.URL.Path,
			"status":    code,
			"duration":  diff,
			"requestID": requestID,
		}).Send()

In this case, Send() just calls Log("") implicitly; this is similar to zerolog's implementation

log: Set Context to map to interface{} instead of a string, fixes #110
Our logger implementation calls  writeValue which will properly encode a value to the underlying io.Writer. This let's us do the following without having to cast the value to a string:

logger := log.NewDefaultLogger()
logger.Set("status", 200) // status=200
logger.Set("duration", time.Since(start)) // duration=2020-10-28T...

http: switch from go-kit/log to base/log, fixes #117

Codecov Report

Merging #118 into master will increase coverage by 0.19%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
+ Coverage   84.45%   84.65%   +0.19%
==========================================
  Files          22       22
  Lines         624      632       +8
==========================================
+ Hits          527      535       +8
  Misses         63       63
  Partials       34       34

Impacted Files
Coverage Δ

log/model_fields.go
100.00% <ø> (ø)

http/response.go
100.00% <100.00%> (ø)

log/logger_impl.go
97.72% <100.00%> (+0.10%)
:arrow_up:

log/model_levels.go
100.00% <100.00%> (ø)

log/model_stacktrace.go
100.00% <100.00%> (ø)

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 0fd04bf...5cf8ed4. Read the comment docs.


Compile Error

I'm gettting a compile error trying to build this for iOS:
src/ios/CardIOCordovaPlugin.m
line 200
"matchedString" is undeclared.
Changing it to "hexColor" fixed the problem for me

Thanks!


Incorrect URL

https://github.com/Informasjonsforvaltning/catalog-frontend/blob/36e88e291689ff7e4361d98b2139202325454120/deploy/prod/concept-catalog-frontend/concept-catalog-frontend-ingress.yaml#L13

Fixed by https://github.com/Informasjonsforvaltning/catalog-frontend/commit/5851e864b1749d2821b00c2b379b17a15630fbc5


Pre-release Download Link not working - Not Found

Hey Team.
The download link is not working:
https://github.com/vincode-io/Zavala/releases/download/1.0.1/Zavala.app.zip

Best regards, Flo.

Thanks! I've updated the README to point people to the App Store now that Zavala 1.0 is out.

@vincode-io thx!


Pixel jitter when animation comes to rest

There's a 1px left-right-jitter glitch happening on the moved items when the animation settles down. They are already at their destination, but glitching a bit to the left and to the right until they finally stop. I'm pretty sure this is happening, because the spring is still oscillating, but the values become too small to render (< 1px) causing it to look like a bug.
You can actually see that behavior in the demo gif in the README.
Any ideas how this could be fixed? I was thinking of a custom Dynamic Item or Behavior, but perhaps there's a simpler approach. Help is greatly appreciated!

🤔 I will look into it. Thank you for pointing this out.

Thanks a bunch!


Knowledge Training for cQube

KT has been given to sharmashbee , Ramya R and Tinu NS.
We have completed
Architecture,
Each UI report wise data source explanation,
Configuration in cQube,
File emission process,
Validation process.

Training completed, assessment tasks WIP.

Need to complete below concepts in KT.

[Installation of cQube ].
[data track after processing in Nifi]
[Understanding of Business Logic each datasource wise]

Invalid moved to JIRA

Training is completed


fixed an incorrect relative directory that pointed into wrong location

applocal.ps1 was unable to copy dependent libraries to output directory because installedDir parameter pointed into incorrect location

Hi @jhruby, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks for catching this! I'll fix the other relative reference in this file.


Drop custom logger

There's no need to ship a custom logger.
Replaced custom logger with logrus which is a quite flexible and popular logger.

@amitbet please review


Rest server allows proper wasm code download

Closes #64

Targeted PR against correct branch (see CONTRIBUTING.md)

[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.

[ ] Wrote tests

[ ] Updated relevant documentation (docs/)

[ ] Added  relevant godoc comments.

[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md

[ ] Re-reviewed Files changed in the Github PR explorer

For admin use:

[ ] Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
[ ] Reviewers assigned
[ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

Codecov Report

Merging #65 into master will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   63.62%   63.44%   -0.18%
==========================================
  Files          14       14
  Lines        1116     1116
==========================================
- Hits          710      708       -2
- Misses        339      341       +2
  Partials       67       67

Impacted Files
Coverage Δ

lcd_test/helpers.go
75.53% <0%> (-0.72%)
:arrow_down:

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 678d0d7...5e67261. Read the comment docs.


Training stuck at logging?

Greetings,
I've run into this issue multiple times. In my script, I load, train, and save the model multiple times. I do this routine about 10 times. However, at some random point, the script just stucks. And it always stuck at this point after the missing file logger.

In addition, I can no longer do anything at this point. The Ctrl+C will not terminate the process either. Neither does the programme give me any error message. Do you have any clue about this?
I am running on version 1.7.7, using strategy ddp. I am on Linux with 2 1080Ti, cuda 12.0.
Thank you very much.

@SpaceHunterInf It looks like there might be a race condition of some sort. I suggest to add a trainer.strategy.barrier() at the end or beginning of your loop when a new stage of training begins. Also, you could double check that the logging folder is correct by printing trainer.log_dir and making sure this always is the same folder across all processes per training stage.


[FEAT]: single line card, but you have to type the answer

Is your feature request related to a problem? Please describe.
I would like to have to write the answer instead of thinking it.
It is very usefull for learning languages (kind of like Duolingo), where the spelling is very important
Describe the solution you'd like
I would like to declare a flashcard with a new operator:
appearance@@wygląd (@@@ for reverse or something like that)
and then have to actually type the answer. Then have a UI that shows me a diff between what I typed with what was the real answer
possible you could use something like jsdiff to do the hard part
from their demo app:

then I would decide as usual whether it was hard or not, regardles of the diff
Additional context
No response

Hi @qrzychu
Nice idea and does have some similarity to:
 Text inputs for clozes #644
That PR allows for multiple text inputs in the one flash card, so perhaps could be considered a superset of your suggestion.
From your perspective, is there anything missing from that?
Cheers
Ronny


Folding visualizations and UNAfold differences

Hello and thanks so much for writing accessible code! It's well needed.
I am coming from using UNAfold online, which is luckily still around on IDT's website right now, but I worry that it won't necessarily be there forever. I'm also interested in coding with these programs so UNAfold is out, since the owner of it hasn't responded to anything and I can't find their code anywhere. But when I was using seqfold, I noticed a bunch of differences. Is there a good explanation for this? I trust you, but I'm just not familiar with exactly where all the ambiguity may be coming from. Do you believe that your numbers are more accurate?
Back onto using UNAfold, is there a feature, where I can see the folding that occurs? When I run the fold command in seqfold I just see the base pairing that will occur. Is this for the most likely structure? When I run UNAfold, I get the dG and base pairing for multiple different folded structures.
I also think a brief manual explaining what i and j are would definitely help a bit. Again thanks for all your code and have a nice day!

Do you have some examples of the differences, like which results were surprising? If not I pasted some in the examples section for ref too: https://github.com/Lattice-Automation/seqfold/blob/master/examples/dna.csv
At the highest level the differences are likely the result of different energy functions. There are these numbers, determined from experiments, that are fed in the ddGs: https://github.com/Lattice-Automation/seqfold/blob/master/seqfold/dna.py. Small variations in those add up over the course of the fold.

When I run the fold command in seqfold I just see the base pairing that will occur. Is this for the most likely structure?

Yes, that's for what seqfold things is the most likely/frequent/lowest energy structure
Re: showing the folds/dGs of multiple structures, I don't really have the bandwidth to add that feature, but it would involve traversing the energy matrix with some kind of heuristic to find other paths than the "optimal" one. IIRC the unafold authors published a paper where they described how they were doing it

I also think a brief manual explaining what i and j are would definitely help a bit

i and j are just two common variable names for when looping. My guess is that i was the most popular var name, short for index, and then j was used because it's alphabetically after. k is then used for triply nested loops. It also shows up one of the seminal papers in this space:
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC350273/
At a high level though re: whether to use UNAfold or seqfold, I'd say that it comes down to what you're using it for. If you're doing research on single RNA/DNA molecules and you have access to UNAFold, I'd just go with that because everyone knows what it is. But both it and seqfold are making high-level, super imperfect guesses at secondary structure. The estimates are very ballparky and mostly useful (imo) for understanding the magnitude of self-annealing. For example, both can help you weed out primers that self-anneal

Lmk if you want to talk more about whether seqfold fits you use-case, happy to talk/write more about it


Fix semicolon termination of results in ARC-Authentication-Results

Ensure results are always terminated correctly in the ARC-Authentication-Results header when multiple Authentication-Results headers are present.
This fixes issue #47.

Commit 495aff7 should take care of this.  It's based on your idea (thanks!); I opted to do it without a second header field search.

Ah, that's neater than mine and does the trick. Thanks!


Correctly revoke Celery tasks.

@johngian r?

Coverage remained the same at 91.417% when pulling 54d24f808b22b6121c2a515be64d978daceec0e5 on akatsoulas:fix-celery-revoke into b25f80bf9e3499bbb4cc8f3cbce7f903df176928 on mozilla:master.

Although this is the correct way to do this according to the documentation it's still not clear to me if the previous code is incompatible with the current celery version.
Code looks OK. Let's test it to see if it fixes the issue.
r+


:tada: New parser: Wazuh #7683

Description
This pull request helps to integrate the results of:
https://documentation.wazuh.com/current/user-manual/api/reference.html#tag/Vulnerability

@damiencarol could you please review this PR?

@damiencarol please review again

@damiencarol kindly reminder

It's been a while.  Re-running tests and looking for more approvals @cneill @blakeaowens

@damiencarol kindly reminder

Tests were red, so I was expecting to fix this before reviewing.

@damiencarol Tests are currently all green/passing


Change the ENTRYPOINT in docker.run

Just a quick question:
How can I specify/change the entrypoint when running a docker image with docker.run?

I figured it out.
Just for future references: The entrypoint can be changed like this:
docker.run(imageName, ['-c', command], process.stdout,{entrypoint: '/bin/sh'}, function(...), {}

My biggest mistake was that I kept calling it like:
docker.run(myDockerImage, ['sh', '-c', myKrakenCommand], process.stdout,{entrypoint: '/bin/sh'}, function(...), {}

which will not work as you are already executing sh


Allow inline declarations to reference schemas that are already defined

The aim is to get things like this working:
schema ExternallyDefined
  ...

schema LookAtMe
  field: inline ExternallyDefined

It turns out this is an extremely small change because the processing of TypeName in manifest.ts is universal (we always replace a TypeName with the actual schema, regardless of the position in the AST). Sometimes we can have Nice Things!

cl/322721734 shows this passing g3; not sure why this isn't showing up in the checks automatically.


Method output parameter renders incorrectly - js code does not compile

C# code
public static bool TryParse(object o, out int i)
{
     i = 3;
     return true;
}

javascript output:
tryParse: function (o, i) {
    i.v = 3;
    return true;
}

what is incorrect in that output? Can you provide full test case?

TryParse has the second parameter as int so that the code i.v = 3 fails

ity should not be failed, it is out parameter. Bridge will transform passed argument to such format

Can you provide test case when out parameter is failed?

ok, reproduced with new test case

Fixed


[IA-3305] Adding .R syncing

Changes:

Modified tools object to contain a list of extensions for RStudio which caused a ripple effect of changes
Changed from widespread usage of getDisplayName (which removes the file extension) to getFileName (which does not) in order to keep file extension with the file name throughout the codebase
Added .R pattern and modified helpers to add .R file syncing
Modified Analyses tab language to accommodate for .R files

Changed language from R Markdown files to simply R Files, but open to suggestions! Would changing the language to RStudio be better?

Testing .R syncing:

Since the Welder changes are in the dev version of leo, testing syncing through the UI needs to be run against a FIAB. Start a FIAB and set your local terra UI to it through the instructions here.
Start up terra locally in your browser and create a billing project/ workspace
Create an RStudio runtime from the Analyses tab
Open RStudio and create an R Script (.R) file
Open the workspace bucket (linked under Dashboard -> Cloud Information -> Open Bucket) and check that the file you created appears in the notebook folder 30s after creation
Share the workspace with another user (ie<EMAIL_ADDRESS>BroadDec1) and open the workspace from that user's account
Create an RStudio runtime and open RStudio
Open the file you created from the second user and edit
Go back to your original user and edit the same file and save
Verify that the generation mismatch error modal appears and that it handles the error appropriately

Other helpful testing:
Since I made a lot of changes to file/ analysis names in the UI, it would be beneficial to continue poking around at copying, renaming, and otherwise changing files to see if there are any places that break. I have a list below of a lot of the specific things I tested and changed.
Tested items (a running list, please add new conditions if you think of them):

[x] Upload .R and .Rmd and .ipynb files from desktop
[x] Make .R file copy when generation mismatch error occurs
[x] Make copy of R notebook
[x] Share file and cause generation mismatch error and modal with .Rmd file
[x] Share file and cause generation mismatch error and modal with .R file
[x] Copy .R analysis to another workspace
[x] Rename .R and .Rmd analysis
[x] Copy .R analysis to current workspace
[x] Checked that name validation doesn’t let multiple files have the same name even if they have different file extensions (all extensions)
[x] Copied .R file from one workspace to another and changed extension from .R to .Rmd (should this be allowed??)

Were notebooks also tested? Since we changed the logic around displayName and extensions. We should test things like having to create a copy of a notebook and making sure the copy gets synced to the bucket as it does today

Were notebooks also tested? Since we changed the logic around displayName and extensions. We should test things like having to create a copy of a notebook and making sure the copy gets synced to the bucket as it does today

Yes, notebooks were also tested for copying both to the same workspace and another workspace, renaming, etc

Will take a second look once @nawatts's comments are addressed - +1 to pulling the file extensions out to constants and trying 1 API request to welder instead of two by using a regular expression like .*\\.R(md)
In terms of wording changes for this PR - it might be worth getting Tiffany on a call and running the proposed changes by her. We can probably shorten The selected file is not a .ipynb notebook file or an .Rmd or .R RStudio file. Ensure your file has the proper extension. to something like The selected file is not a .ipynb notebook file or an RStudio file. Ensure your file has the proper extension. Or maybe display all extensions in a list as the supported extensions

I pushed my changes as per @nawatts's comments and reached out to Tiffany about the wording. Should be ready for a second look!

Since the Welder changes are in the dev version of leo, testing syncing through the UI needs to be run against a FIAB. Start a FIAB and set your local terra UI to it through the instructions here.

Is there anything here that's incompatible with the current production APIs? Does release of this have be coordinated with a Leo release?


Listing all verb forms

Listing all verb forms created by<EMAIL_ADDRESS>on 2017-01-04 10:20:29

<EMAIL_ADDRESS>replied, In another thread we discussed how we include the word okedii but not the
word okedau.  Here's a bit of discussion about why this is true.  Please
chime in with suggestions and comments about this if you have any.
In terms of which verb forms are added, here's an email I sent recently to
Jim about how difficult this would be in addition to how it might make the
website harder to use.  The problem is that there are way too many forms of
each Palauan verb!
We have thought about adding all forms beyond just the 3rd person forms.
Currently we only add when they're irregular.  I wanted originally to add
all of them but Lewis Josephs talked me out of it. Correctly I think.  Each
verb has the base verb and then 5 hypothetical prefix forms (k-, l-, m-,
d-, ki-) and 10 suffix (none for imperfect, then nine perfective: -k -au
-ii -d -mam -miu -ir and one for plural inanimate objects that sort of
doesn't have a suffix) so that's a total of 60 forms for each verb.  Then
double that to 120 forms for each verb if you want to add plural forms.
Adding them to the displayed list would hurt the usability of the site and
how do we find them all anyway?   Because there are also variant forms of
each of the above; for example, sometimes it's -mam and sometimes it's
-emam.
If we could somehow gather them, it would be useful to add them to the
database as hidden words that aren't displayed but can be searched to find
the root word.  Although, if they're regular, it'd be better maybe to teach
the rules to the website and have it say, 'that looks like the hypo 3rd
person singular of mengang' or something.

<EMAIL_ADDRESS>replied, what do you think about extracting of basic forms which have v.inch. and v.pred?

<EMAIL_ADDRESS>replied, Can you give two examples please?  I think most of us don't know the Josephs terminology.

<EMAIL_ADDRESS>replied, Can you give two examples please?  I think most of us don't know the Josephs terminology.

<EMAIL_ADDRESS>replied, omkobk: basic form: obkobkinchoative: obkobkangpredictive: obkobkungi think about the basic forms too much, i guess the are derived, but not inflected, maybe we should separate them. but i'm not sure about it yet

<EMAIL_ADDRESS>replied, omriid:basic form: obriidinchoative: obridangpredictive: obridungi think, basic forms having inchoative, predictive and reduplicated forms can be extracted.

<EMAIL_ADDRESS>replied, i think we can choose the most unpredictable verbs and figure out how to deal with them? OMES group looks monstrous now.

<EMAIL_ADDRESS>replied, Maybe we could make '-UES' be a root word and call it v.hyp.suffix or
something.  Link it to OMES, and then list the kues,mues,lues,etc. under it?

<EMAIL_ADDRESS>replied, I like the idea of "roots," but I'm afraid that the cost/benefit tradeoff is the hard part...Finding the root is often an exercise in morpho-phonological analysis.  You have to look at multiple forms (not just verb forms but also derived nouns, etc.) to find a common root that can be used as a basis for all of those surface forms -- each one is like a puzzle.  So despite the usefulness of roots, my opinion (though maybe I'm missing something) is that it would be too time-consuming/error-prone to do this on a large scale.That being said, I've been giving some thought to trying to come up with an automatic morphological analyzer (using some tools from Xerox/PARC research center).  This could aid in automatic identification of roots.  But this would be quite a large project (I'd love some help), and success is not guaranteed.  Anyone (who knows somebody) with both Palauan language and computer programming expertise and wants to discuss further, feel free to open another thread on Debugle and I'm happy to provide some pointers to the documentation/tools.  :)


how to refresh layout after change textsize?

Report

[x] I have searched existing issues and this is not a duplicate

Issues and Steps to Reproduce
I use YogaLayout to add a TextView as a child:
YogaLayout yogaLayout = new YogaLayout(this);
yogaLayout.getYogaNode().setFlexDirection(YogaFlexDirection.ROW);
rootLayout.addView(yogaLayout);
TextView textView = new TextView(this);
textView .setText("test");
yogaLayout.addView(textView1, -1, new YogaLayout.LayoutParams(ViewGroup.LayoutParams.WRAP_CONTENT,
                ViewGroup.LayoutParams.WRAP_CONTENT));

after 5s,I modify the textSize of the textview,the text became larger,but the layout of YogaLayout and TextView don't change~.
new Handler().postDelayed(new Runnable() {
            @Override
            public void run() {
                textView.setTextSize(30);
            }
        }, 5000);

i try to use these codes after change textSize ,but not work:
yogaLayout.invalidate(textView);
yogaLayout.requestLayout();

so,how to change the layout after i change child's LayoutParams in YogaLayout?

hey, how do you solve this problem @icemanstudy ?

The following code will relayout at pod 'YogaKit', '~> 1.18.1'
textView.yoga.markDirty()
textView.superview?.yoga.applyLayout(preservingOrigin: true)

We're deprecating the com.facebook.yoga.android.YogaLayout ViewGroup in the next major release of Yoga. We intend to still push out a new version aligned to the core Yoga 2.0. I have bulk closed ViewGroup issues to reflect that we are not planning to invest time into functional changes of the ViewGroup. and eventually intend to remove it from the repo.


The witch -key does not work if there is no buffers open

You can't use SPC f f when no files are open. It breaks the user flow a little

Yeah, I know, but I haven't found a workaround. PR welcome, if anybody finds a fix 👍


bug: HA - mqtt topics hardcoded

The state topics sent to home assistant discovery are hard coded, they should be set the to <topic prefix> variable
For example
https://github.com/rvdbreemen/OTGW-firmware/blob/379e7358c4c52b71fdab894246ed27f7d5ad2e54/data/mqttha.cfg#L3
"state_topic": "OTGW/fault"

should be:
"state_topic": topic_prefix_variable +"/fault"

This is also true for the home assistant discovery topic. It defaults to homeassistant but a user has the option to change this. A config option/var should be available to set it.

Yep, this can be fixed. Give me time, or do a PR.

@ikkemaniac fix in dev stream of v0.7.8

hardcoded state topic now flexible
added homeassistant prefix at the same time
adding setting in webUI

nice, i'll have a look and test next weekend!


Add parameter to customize the number of glanceAPI workers

why should we add a parameter for this and not use the config customization if we need it? just wondering as we not add parameters for each of the settings in the service config, while this can be customized using the customServiceConfig.

Mostly to keep consistency with other storage operators, but we can use customServiceConfig in this case.

@konan-abhi @stuggi should we simply document how to do override that value?

manila and cinder PRs are different. they customize the httpd processes. this should align with what was already done in  keystone PR https://github.com/openstack-k8s-operators/keystone-operator/pull/500 . in my opinion, if glance is different and does not use httpd process to control this, we should not replicate this here for the glance config. a user should use the default interface for this

manila and cinder PRs are different. they customize the httpd processes. this should align with what was already done in keystone PR openstack-k8s-operators/keystone-operator#500 . in my opinion, if glance is different and does not use httpd process to control this, we should not replicate this here for the glance config. a user should use the default interface for this

I'm ok to remove this parameter for Glance, and parse customServiceConfig to make sure we stay under certain limits. I will update this patch accordingly! Thanks @stuggi for the input.
BTW I can align Manila and Cinder with the keystone patch: I simply don't like the naming and we didn't agree on something we can use and spread across the operators. I think we should have periodic API reviews to keep consistency and agree on the parameters in advance.

manila and cinder PRs are different. they customize the httpd processes. this should align with what was already done in keystone PR openstack-k8s-operators/keystone-operator#500 . in my opinion, if glance is different and does not use httpd process to control this, we should not replicate this here for the glance config. a user should use the default interface for this

I'm ok to remove this parameter for Glance, and parse customServiceConfig to make sure we stay under certain limits. I will update this patch accordingly! Thanks @stuggi for the input. BTW I can align Manila and Cinder with the keystone patch: I simply don't like the naming and we didn't agree on something we can use and spread across the operators. I think we should have periodic API reviews to keep consistency and agree on the parameters in advance.

agreed we nee to do better on that + also to get those things across all operators. Its not efficient that this is done by each group

manila and cinder PRs are different. they customize the httpd processes. this should align with what was already done in keystone PR openstack-k8s-operators/keystone-operator#500 . in my opinion, if glance is different and does not use httpd process to control this, we should not replicate this here for the glance config. a user should use the default interface for this

I'm ok to remove this parameter for Glance, and parse customServiceConfig to make sure we stay under certain limits. I will update this patch accordingly! Thanks @stuggi for the input. BTW I can align Manila and Cinder with the keystone patch: I simply don't like the naming and we didn't agree on something we can use and spread across the operators. I think we should have periodic API reviews to keep consistency and agree on the parameters in advance.

agreed we nee to do better on that + also to get those things across all operators. Its not efficient that this is done by each group

For the record: this is a conversation about scalability we had yesterday within the storage group. The result about the control around scalability, placement, and setting upper limits involved this workers parameter for Glance, processes for Manila and Cinder API. I didn't plan to put these set of patches up but considering how simple is adding this parameter in the picture I started it. Being that said I'm ok to keep consistency across the operators, I simply think we should plan to either periodically review or centralize the API for k8s-operators, and don't risk to do to different directions because I simply thought about a different name for a struct.

@stuggi I'm closing this patch for now, I will reopen in case we need some validation around workers parameter.


Fix gather_grass and gather_limestone

Addresses two issues with gather_resources.lua

If the screen is left on picking coordinate direction for grass gathering for longer then the timeout, the script immediately quits
Limestone gathering will stop because the mouse is over the icon causing Automato not to detect the icon

I am running Linux, so this may be a linux only issue. I've seen that a lot of things on Automato with Linux actually move the mouse around, not just do phantom presses like they do with Windows.

Ahhh @Limb I have heard about that. I removed the mouse movement because it's going to cause problems for anyone running multiple macros. We generally try to avoid mouse movement unless strictly necessary. However that is a legit problem you've got. I think a solution that might work for you, without breaking things for other users, would be to find an offset from the detected image coordinates that moves the mouse low enough that the tooltip doesn't get in the way of the gathering icon.

Alternatively @Limb , there is an Interface Option to "Hide Action Bar Tool Tips"


Feature: rest/json api for generation sla/pdf

It would be very useful to run this as a web service.

Thanks for the suggestion. I won't have the time to look into this request in the near future, but feel free to push a code improvement if you feel like it. I would review with pleasure.

I have no idea how to do this and no experience with python. But still relevant and would be very convenient!

I don't really see the added value VS additional troubles (eg to upload users SLA templates)... but if you wanna give it a try you can either expose the current command line Scribus Generator installed on your web server (or cloud), such as:
https://medium.com/cmd-io/shell-scripts-run-via-secure-web-api-9a59642ee460
or create a native Python API:
https://linuxhint.com/rest_api_python/

any progress / projects in this direction do exist already?

nope, and it's not on my plan for the near future, but I would gladly review a PR if you want to give it a try.

With https://github.com/berteh/ScribusGenerator/issues/228 in place, it would be rather simple to make a tool doing that and indeed, I would love to see such a service, ideally dockerized.

it's all readily available from the CLI, per the links provided in #228
I've got no docker skills and don't know how a full Scribus install would work in a server settings (I think a screen is needed... but not sure) so I won't be able to help, but if you wanna give it a shot I'd gladly test it out and/or integrate PRs here if needed.

forgot to mention: there's already an embryo of docker for ScribusGenerator at https://github.com/berteh/ScribusGenerator/wiki/Docker-image-for-ScribusGenerator
Kindly test/adapt/improve directly in the wiki... and then document in the README in a PR ;)


Added sensible default root location for MKL on Windows

MKL on windows works with this change. Tested with MKL 2017 Update 3 (https://software.intel.com/en-us/articles/intel-math-kernel-library-intel-mkl-2017-release-notes).
Should fix #544
With MKL 2017 Update 3 #514 should not happen too.
Note: I used Anaconda which ships with its own MKL, so I had to make sure that the MKL 2017 Update 3 version was loaded by replacing the .dll in the %AnacondaPrefix%\Library\bin folder. Otherwise, numpy would load it's own version and I would have all sorts of missing procedures errors. Now that the same version is available through conda this is easily fixed with conda install mkl==2017.0.3

Thanks!


um okkur

fix the layout and add text,contack info and links

Site mostly ready can be improved over time


[RFP] New Checkout structure

Summary
[summary]: We need to refactor the Checkout page to be more flexible, easy to extend and get rid of the current way of adding payment/shipping modules which are solely based on the Event Bus (#3020)
This is the open request for proposals/feedback on how the best possible Checkout feature should work like from the developers perspective.
Motivation
[motivation]: We're moving towards https://storefrontui.io to be used in the themes. Current checkout is pretty messy, based on events - hard to extend and to event understand to be the new-comers. The idea is to make the payment/shipping modules less error prone plus increase the extensibility (adding/removing/changing) of the checkout steps.
Guide-level explanation
It would be great to focus on the following features of the checkout with the proposals - on which we'll form the final design for the Checkout feature:

How to hook-in into checkout process - currently there are events like https://github.com/danrcoull/vsf-payment-braintree/blob/d42a2e6f2ccc227aad6c183ea8c089da032a73cb/hooks/beforeRegistration.ts#L25 that modules hook in. How should it be handled?

How to override the Checkout components? Should it be kind of API like dispatch('checkout/addStep'),dispatch('checkout/removeStep') ... ?

How to modify/extend the Checkout UI - especially the payment modules should have an option to hook in and add some forms / inject JS code. For example, the stripe module is expecting us to add the UI manually (probably it's not an optimal way): https://github.com/develodesign/vsf-payment-stripe#integration-the-stripe-component-to-you-theme

How to handle the validation? Should we stick to the Vuelidate that we're using currently or switch to another library? How to hook in the validation process for the custom steps?

Happy to hear Your comments / code snippets / requests on this feature./

@dimasch @lukeromanowicz @lukeromanowicz @filrak @patzick would be great to get Your feedback on that :)

Hi, it would be great if the checkout redesign utilizes a dedicated confirmation page, since most external payment gateways need a page to redirect to, where the order is finally placed and confirmed - something which is not possible with the current checkout.

I will probably extend this suggestion soon but generally everything should be Vuex-driven and as simple as possible.
Ideally we should have Vuex state fields for:

personal details
shipping
payment methods

Also getters + setters/validators (actions) for all above + placeOrder method. Then we can simply consume all of those in a frontend no matter if it's a one-step checkout, multi-step checkout etc.
Also we should have actions related to cache regarding shipping and personal details.
The Vuex implementation shouldn't force any form of checkout steps. People can either validate let's say personalDetails after filling part fo the form or validate all of the fields at the end at once (by calling all validators)
Also in state we should a place called "custom" which is just an object containing custom data. Let's say we are adding new field called Foo so it will be held under custom.foo and there should be a getter called getCustom('foo') (and mutation) that return/set custom fields.

here are my 2-cents on this topic; I would expect VSF not to care about what goes into an order, and as a consequence not to have any required fields for the checkout.
Whatever fields are there defined by the developer are posted to the backend platform.
We can introduce some interfaces that help with suggesting the fields either Shopware, Magento or whatever other system is expecting, but again it's a suggestion.
Payment is separate. Methods are fetched, a choice is made by the user.
Once the order is dispatched we take the chosen payment method and send the authorization request to the PSP. This gets us a response that triggers a visual notification for the user.
The actual payment registration in the order in the backend platform is outside of the scope of Vuestorefront. The PSP can use callback and notification URLs for that.

Thanks for Your feedback. Having this in mind I would say we need to:

Propose an interface for the payments method to comply with.

To change the current way they register:
    // Update the methods
    let paymentMethodConfig = {
      'title': 'Cash on delivery',
      'code': 'cashondelivery',
      'cost': 0,
      'costInclTax': 0,
      'default': true,
      'offline': true,
      'is_server_method': false
    }
    rootStore.dispatch('payment/addMethod', paymentMethodConfig)

to something like:
    // Update the methods
    let paymentMethodConfig = {
      meta: {
      'title': 'Cash on delivery',
      'code': 'cashondelivery',
      'cost': 0,
      'costInclTax': 0,
      'default': true,
      'offline': true,
      'is_server_method': false
},
handler: { // kind of state-machine for payments
    init: (checkoutData) => {},  // called whtn user selects this method
    create: (checkoutData) => { }, // called when the order has been placed and we can create the payment
    authorize: (checkoutData) => { }, // called back on the approval
    cancel: (checkoutData) => {} // called back on canceling the payment flow
}
    }
    rootStore.dispatch('payment/registerMethod', paymentMethodConfig)

.. so the handler lifecycle methods are called instead of current events:
checkout-before-placeOrder

and then
checkout-do-placeOrder

which is in fact called back by the payment method. So we need to take over the control of the process from payment method fully to the Checkout.

Make the checkout fields really dynamic - manageable 100% in the theme. The Vuex store should be just for placing an order (prepared in the Checkout theme component and validated beforehand). Kind of current prepareOrder.

The Vuex store of order and checkout should be merged all into one single checkout module.

The checkout Vuex state should be not typed - to not have static "persionalDetails" whatsoever. The User's checkout should be able to register these sections like: dispatch('checkout/addSection', { name: 'personalDetails' }) of course there should be also dispatch('checkout/removeSection').

The idea we discussed was to make Vuex completely unaware of the checkout state @patzick but it's not probably something we can do as the for example cart/syncTotals requires the data from checkout to estimate shipping costs.
So I agree with @filrak that we should have some defaults + getters for them (as we currently have in the checkout module) + extend it for a user to let them register another section.

The question is WHO is going to prepare a PoC? :)

Handlers for payments are necessary but I would suggest providing actions to dispatch rather than methods because it gives us a little more flexibility and extensibility.
Sander's mention about no field being required is quite important. Some online shops require only the e-mail e.g. for shipping virtual products. We have to keep in mind that some shops may use only one address while others use even 3+ addresses in b2b cases.
Shipping methods and Payment methods should be optional - many shops have single shipping or single payment methods and don't need to give the customer to choose one method from the set containing only one of them.
Shipping and Payment methods should have a possibility to easily write custom components to handle them (e.g. a payment method that needs additional data from the customer). It could be easily implemented with for instance a renderless component.
Checkout should be more modular because enforcing users to extend our default checkout rather than creating their own from ready to use components leads to lots of unused but still bundled code.
I still vote for validation ONLY in frontend client and in a backend. I consider additional form validation in API unnecessary and troublesome.


Fixed issue-137: Gray out icons of not yet implemented features && Make rooms list sidebar narrower

Fixed : #137

[x] Gray out icons of not yet implemented features
[x]  Make rooms list sidebar narrower

Actually most of these issues were already fixed here https://github.com/project-robius/robrix/pull/138, maybe the todo list in the issue wasn't up to date (should've updated/closed when the PR was merged), we agreed on that sidebar size as narrow enough after those fixes.
Besides that, I also agree on graying-out the disabled items further, thanks for also including menu items and for the color naming 🙌🏽

I have  restored the login.toml


Median feature

This solves #21

@srinivasu619 Thank you very much for your effort and continuous support implementing the median method! Looks good now, merged!


Add the fall penality only once

Fix https://github.com/omichel/robotbenchmark/issues/327
When a the robot falls, the 30s time penalty is added every milliseconds. Soon enough, the timer (supporting 100 minutes) is overloaded causing weird results.
Adding the penalty only once fix this issue.

👍

I'm not sure that adding the penalty only once will be fair and will prevent cheating: a user could make the robot fall to exploit instabilities and jump to a further location.
What do you think?
What about setting a maximum time to 100 minutes?

Yes, setting the penalty to the maximum amount of time is also a solution.

In this case, the penaliy becomes more a disqualification criteria. I'm changing this PR target to develop.

Migrated to #829


"code": "InvalidInput",   "statusCode": 400,

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.28.1
Regression From
No response
Architecture
i86
Storage Explorer Build Number
20230309.1
Platform
macOS
OS Version
No response
Bug Description
Impossible d’importer le lot commençant à l’entité avec la clé de partition 'FORMSUBS’ et la clé de ligne 'FORMSUBS1’ et se terminant par l’entité avec la clé de partition 'FORMSUBS’ et la clé de ligne 'FORMSUBS100'.
Steps to Reproduce

launch
Import a csv utf8
All fields validated
Element not imported

Actual Experience
"code": "InvalidInput",
"statusCode": 400,
"request": {
"url": "https://csb10032000e5a7b49c.table.core.windows.net/$batch",
"body": "--batch_e465f1ca-a81b-4976-b18d-d879a5ebbd6d\r\ncontent-type: multipart/mixed; boundary=changeset_4213004f-5a21-444c-86d3-3ad7f4e48af0\r\n\r\n\r\n--changeset_4213004f-5a21-444c-86d3-3ad7f4e48af0\r\ncontent-type: application/http\r\ncontent-transfer-encoding: binary\r\n\r\nPUT https://csb10032000e5a7b49c.table.core.windows.net/FORMSUBS(PartitionKey='FORMSUBS',RowKey='FORMSUBS1') HTTP/1.1\r\nContent-Type: application/json\r\nDataServiceVersion: 3.0\r\nAccept: application/json\r\n\r\n\r\n{"PartitionKey":"FORMSUBS","RowKey":"FORMSUBS1","name":"Bleu","name@odata.type":"Edm.String","firstName":"Jean-Philippe","firstName@odata.type":"Edm.String","level":"Senior 7","level@odata.type":"Edm.String","Height":"175","Height@odata.type":"Edm.Int32","since":"C'est ma première année au CEA","since@odata.type":"Edm.String","next_continue":"Oui","next_continue@odata.type":"Edm.String","next_level":"Le même niveau","next_level@odata.type":"Edm.String","advice":"Non","advice@odata.type":"Edm.String","recommandation_rating":"7","recommandation_rating@odata.type":"Edm.Int32"}\r\n--changeset_4213004f-5a21-444c-86d3-3ad7f4e48af0\r\ncontent-type: application/http\r\ncontent-transfer-encoding: binary\r\n\r\nPUT
Expected Experience
it should be more interesting to have a direct issue and correction suggestion, because the 400 error doesn't tell me :

for which field
if it's coming from property fields or from row infos
where is it located in the partition
how to mainly correct it

Additional Context
No response

@FocusAid the error comes straight from the server, so unfortunately we don't have much more information to expose to you regarding these types of errors. To narrow down what row/entity is causing the problem, you'll need to split up your data, import the smaller parts 1:1, to figure out which row/entity is causing the problem.

Closing due to lack of response. If you are still experiencing this issue we recommend opening a support ticket from the Azure portal. Alternatively you can open a new issue here.


[WIP][Feature] Port MySQLScanNode to BE connector

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #5735
Problem Summary(Required) ：

Consider to insert a logical exchange operator at the end of scan operators.

LGTM. I'll run a pr regression for you.

pr regression looks ok. Myabe you can put another commit to remove mysql scan node/operator.


Symbol.observable is missing from the Observable class on the type level

Describe the bug
RxJS can consume InteropObservables that implement Symbol.observable method but it doesn't allow for reverse - it doesn't make its own observables interop (at the type level) so they can't be consumed by other libs that would like to leverage Symbol.observable for this purpose.
Expected behavior
It should be possible to create a from method that would rely on the input argument having a Symbol.observable method and convert Rx observable to a different observable type.
Reproduction code
import { Observable, interval, InteropObservable, Subscribable } from "rxjs"; // types: 7.5.2

declare const obs1: Observable<number>;
const obs2 = interval(100);

type Test1 = typeof obs1 extends InteropObservable<infer T> ? T : never;
//   ^?
type Test2 = typeof obs2 extends InteropObservable<infer T> ? T : never;
//   ^?

declare class MyObservable<T> {
  subscribe(cb: (a: T) => void): void;
  // just make this class nominal to avoid relying on structural subtyping with the Observable from RxJS
  private tag: string;
}

// strategy based on the one in RxJS itself
// https://github.com/ReactiveX/rxjs/blob/5d0e41ee46337f56dba6d9a1ef938002b261ccac/src/internal/types.ts#L90
type ObservableInput<T> = MyObservable<T> | InteropObservable<T>;

// https://github.com/ReactiveX/rxjs/blob/5d0e41ee46337f56dba6d9a1ef938002b261ccac/src/internal/types.ts
type ObservedValueOf<O> = O extends ObservableInput<infer T> ? T : never;

// https://github.com/ReactiveX/rxjs/blob/888c753e272443bbb879f9ffb1a8145a7773c865/src/internal/observable/from.ts#L6
declare function from<O extends ObservableInput<any>>(
  input: O
): MyObservable<ObservedValueOf<O>>;

// actual
const converted1 = from(obs1);
const converted2 = from(obs2);

// expected result (without having to cast this)
const converted3 = from(obs1 as typeof obs1 & { [Symbol.observable]: () => Subscribable<number> });
//    ^?
const converted4 = from(obs2 as typeof obs2 & { [Symbol.observable]: () => Subscribable<number> });
//    ^?

Reproduction URL
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbzgeQEYGcCmUBuBDVAG0wBo5gA7GbfQsgSSuwjDS1wOLIGUBXDAMZRgqTpjgBfOADMoEEHABEUAB4ArdIoDccAPS64MAJ5hM6AFxwA7ADoArDYBMAKGcATTAMJ4o4gRAp0eAgMAEZLNhoxAB4KXhBUbAA+LWd-QOCMRzgAXnImDkIAClCABlKASlTnY1M4ABUzGFDcwxNMCGk4EPQWzBVqCjd0OEZqOVYMKKJMaMppbAakuAB+BrhLCkwcbFT9OAOAPRWa9oam7Lzajq6e7P7B4dGClkiOGbmKBagl1fXN7a7Zz7I4ndyeby+OBePDoEYAWSMb3wH3qywQzgO6H46CEIkwRQEqEsRTwlnqFVyyxwEGAbgqlhpdNSB32al4QTgIDwAGtxDAABbAEYwuFwCjySh4QiGCBwPBMtxwXyEIyUADm3QocCCUF4AhgvCg0p1-FqGrgAHdgILDALxMixDI5AoAEoqABS3ExcDAwnw1EMeHVll1GtSElc+11eGo6qMcFEWCVATt4gC4kocHdXvIMCwhGkwIMApgMDAFn06ptAv4Nn8IF0rsweANwB2AA1dKoNLoiCFdHY3KVMAAWUKYMcANgAzDOrNI7FO3KJlwBOPAT6RrmcADnKjlQjinoQEAlbunQUAEuko4wo0t013QNnzAGIADJr0qnOqOmaMGAvAwNEaKtIi-7EKBywAD7POMrxTO8UFotU+yluWla6NWgp1g2TYtm2nbduo6B9oQA5DiO46TqOs7zouy6rm4G5bju+6lIex6nueN5Xjed7YA+hBPu0L75r+DpITsbgAGrSrwmDINI0TIMseTIHADyYEMIyQZggHAZ83y-Gs9QbOKgJQGhJZlhW5hVjWeHyARrYwO2mBdj2ZH9qgui7gFAhWHYM6YI4ViOKOo4zqgsW7lYa7btI0ioKEeC7qEo52HgVi5TOAi7lOdiXtet4FMJug9NMxC6LI8ivugn5TuCMJQtIvAUG2qZ1SAqlaQMOlPPphkgXgFBGEkSRFD6lBATAETOAycAQdJMTIpgckKUpKlqSkUYGG5vDSmkAScukOywBtLR5D1RQ9KEVQnRk0IBBd1BuJczryHdWSPcW-WmAaG3KmYvCEPARTWoKEDAXAAoKhaMByuenKCsKFRPWdr3YO9M6tLd93yiM1ydN0YRwAAZIgcAANrcEYCQQIQNhVchmAALokpSOTLHwgjCKIHxxAkySSI9IKgpj8DnTjG2jvjLo-eg2Swm0pik3clPU3TDOoEzLOrTMnNwEU3O8zieKC1BwuJFAywSOLBgHHAxzOEAA
Version
7.5.2
Environment
No response
Additional context
No response

Unfortunately, TypeScript doesn't really support this right now. If you export a unique symbol from a module, then try to use it in another module, the type system can't figure it out just yet. I know this is something the TypeScript team was looking into, but I don't think this has ever been resolved.
@cartant touches on that in his blog here.
Sorry for the inconvenience, but we can't fix this until TypeScript can somehow support it.

In such a case... wouldn't it be better to agree on more versatile interop? I mean... a string property like $$observable would work almost as well as a symbol. I know there are some subtle semantic differences but most of the time they don't matter.
I also think that this can still be fixed on your side - this "only" requires some build-level shenanigans. I've, sort of, worked around this issue by post-processing runtime code with such Babel plugin:
https://github.com/statelyai/xstate/blob/8d375ac4bd551cf9e9b415d075c03f99b9b9cf3a/packages/core/rollup.config.js#L7-L32
This allows me to keep Symbol.observable method for the time when TS emits .d.ts but I strip it from the runtime and I only keep the "ponyfilled" method:
https://github.com/statelyai/xstate/blob/8d375ac4bd551cf9e9b415d075c03f99b9b9cf3a/packages/core/src/interpreter.ts#L1359-L1362
I understand that this is a little bit crazy but I think that the means justify the ends here because Rx observable should be compatible with the interop strategy proposed by it.


Release new plan before exchange

I advise you to release your plan of monyx moving forward, And what monyx can offer for their investors before you take the sell limit off on the exchange. Right now people aren't seeing much value in the coin because there's no offer to make money from the coins, So people will dump the coins on the exchange when they get that opportunity.
Maybe you should do something very similar to thorn coins setup, Their idea is what looks promising for lending platforms to do well and survive.

I absolutely hope people dump their coins on the exchange so I can buy them up for cheap!

@animalmode I'm not responsible for the team's plans, I think they were planning to release a new whitepaper/statement describing the new direction on the next week, not sure when exactly.


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2024-08-22)
Bug Fixes

#105 disabling default prompts still creates slash cmds (d92baf0)
setup slash cmds even if use_default_prompts is false (ea727c2)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.2
:sunflower:


Wrong documentation

The documentation should make it clear that the class needs to be static as well. Otherwise this doesn't work.

The method needs to be static not the class and that’s explicitly called out in the docs 🤔

Debugger.Launch(); in the method does not kick off a debugger, unless the class is static itself, leading me to believe that method is not invoked at all, unless the class is static.

As confirmed by a “working” example:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/8090cf8f2c9f46e7a35119ba5d6fe170b7985a5e/test/WebSites/CliExampleWithFactory/SwaggerHostFactory.cs


Revert "upadte docstring cross reference for documentation"

Reverts faneshion/MatchZoo#209
delete source dir mat make bugs

Codecov Report

Merging #210 into 2.0 will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            2.0   #210   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files        17     17
  Lines       410    410
===================================
  Hits        410    410

Impacted Files
Coverage Δ

matchzoo/engine/base_model.py
100% <ø> (ø)
:arrow_up:

matchzoo/preprocessors.py
100% <ø> (ø)
:arrow_up:

matchzoo/engine/hyper_spaces.py
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 efb871c...e2e4189. Read the comment docs.


add disconnected auth flow

We may have use cases where we want to keep ARENA management completely in a closed system, like a digital twin system for factories. In that case we'd need to support a side loaded user account create/recovery locally. So we could keep the allauth definitions, but need to then support an option where users are performing standard logins. From the UI perspective, we'd add "Login with Email" and remove "Login with Google", but that could be deployment configurable.

Somewhat tangential is the option to "Login as ARENA", where we could support our own OAuth: https://github.com/openauthjs/openauth.


following nativewind v4 setup guide and getting CocoaPods is only supported on darwin machines

Describe the bug
Hi, I've tried to install and configure my react native app like what was written in the official doc but got this error instead:
npm install<EMAIL_ADDRESS>react-native-reanimated tailwindcss

up to date, audited 1209 packages in 11s

78 packages are looking for funding
  run `npm fund` for details

5 moderate severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.
$ npx pod-install
Scanning for pods...
CocoaPods is only supported on darwin machines

To Reproduce
Steps to reproduce the behavior:

Go to https://www.nativewind.dev/v4/getting-started/react-native#installation
follow the steps
See error

Expected behavior
No error

Side info
OS: 6.1.71-1-MANJARO
Node: v20.10.0
package.json:
{
    "main": "node_modules/expo/AppEntry.js",
    "scripts": {
        "start": "expo start",
        "start:clean": "clear && expo start -c",
        "format": "prettier -w . -u",
        "android": "expo start --android",
        "ios": "expo start --ios",
        "web": "expo start --web",
        "prepare": "husky"
    },
    "dependencies": {
        "expo": "~50.0.5",
        "expo-status-bar": "~1.11.1",
        "nativewind": "^4.0.23",
        "react": "18.2.0",
        "react-native": "0.73.2",
        "react-native-reanimated": "^3.6.2"
    },
    "devDependencies": {
        "@babel/core": "^7.20.0",
        "@types/react": "~18.2.45",
        "husky": "^9.0.10",
        "prettier": "^3.2.5",
        "pretty-quick": "^4.0.0",
        "tailwindcss": "^3.4.1",
        "typescript": "^5.1.3"
    }
}

Its simply a documentation error, its missing an (osx only) tag line.
The OSX operating system is named 'darwin`. The error is saying that your not using a mac so you don't need to run it.

OK, then can we add it to the doc?


Fixed bug when passing a list into route decorator's host argument #1120

Fixing #1120
There happens to be a bug in a recursive call to Router._add that tries to apply the strip slashes  logic wrongly. Normally when the host param passed to Router._add is a string, the method runs smoothly (the easy, good case). However, setting host to a list like
@app.route('/post', methods=['POST'], host=['test.com', 'test2.com'])
def handler():
    return text('OK')

triggers a recursive call to Router._add. More details are stated in the commit message.

Thanks for the fix!


[Python] Table / RecordBatch repr displays the wrong timezone for non-UTC timestamps

Describe the bug, including details regarding any error messages, version, and platform.
Printing a Table or a RecordBatch containing timezone-aware timestamps displays the time values in UTC but shows the original (possibly non-UTC) timezone in the schema header.
import datetime as dt

import pyarrow as pa
from pytz import timezone

# Create a table from a non-UTC timestamp
pacific_tz = timezone('US/Pacific')
mapping = [{'datetime': pacific_tz.localize(dt.datetime(2023, 11, 7, 22, 0, 0))}]
print(mapping)
# [{'datetime': datetime.datetime(2023, 11, 7, 22, 0, tzinfo=<DstTzInfo 'US/Pacific' PST-1 day, 16:00:00 STD>)}]
table = pa.Table.from_pylist(mapping)

# Table repr displays the timestamp values as UTC but shows the original timezone in the schema header
print(table)
# pyarrow.Table
# datetime: timestamp[us, tz=US/Pacific]   <-- incorrect timezone
# ----
# datetime: [[2023-11-08 06:00:00.000000]]   <-- timestamps in UTC

# Confirm the underlying table data is correct
print(table.to_pylist())
# [{'datetime': datetime.datetime(2023, 11, 7, 22, 0, tzinfo=<DstTzInfo 'US/Pacific' PST-1 day, 16:00:00 STD>)}]

# Create a table from a UTC timestamp
mapping = [{'datetime': dt.datetime(2023, 11, 7, 22, 0, 0, tzinfo=dt.timezone.utc)}]
table = pa.Table.from_pylist(mapping)

# Printing the table shows the correct timezone in the schema header
print(table)
# pyarrow.Table
# datetime: timestamp[us, tz=UTC]
# ----
# datetime: [[2023-11-07 22:00:00.000000]]

See also this related DuckDB issue
Pyarrow Version:
13.0.0
Platform:
macOS 12.7
Component(s)
Python

Thank you for reporting the issue @nph!
Yes, this is a known issue with timezone aware timestamps. The problem is the dependency on finding a timezone database which is not yet supported on Windows at the moment for example.
I will be adding an alternative solution which would keep the printed value in UTC but adding an indication at the end of the string to make it a bit clearer that UTC times are printed.
See https://github.com/apache/arrow/issues/30117.
Will close this issue as a duplicate, please follow the linked one for future fix.

Duplicate of https://github.com/apache/arrow/issues/30117

(I was by coincidence replying on exactly the same moment ;), keeping a small part of my answer)

datetime: timestamp[us, tz=US/Pacific]   <-- incorrect timezone

Just one additional note on this: it doesn't show the "incorrect" timezone in the schema. Arrow does support keeping track of a timezone parameter on the timestamp type, and therefore when creating the table from tz-aware values, we preserve that information in the schema.
But regardless of which timezone you have, tz-aware Arrow data is always stored as UTC values. But of course showing this original timezone, in combination with the data repr showing those UTC values, is rather confusing.

Thanks @jorisvandenbossche, @AlenkaF.


Playfair Cipher Cryptography in Python

Implementation of Playfair Cipher Encryption Algorithm in Python

I wish to contribute. Kindly assign this to me.


Add installation instructions to read me

Thanks, always good to let people know how to install the thing :smile:

No doubt most will already know how, it's just sometimes nice to have a link to the NuGet page to save you having to manually search for it, I'm lazy like that :)


How to accelerate the training phase.

Hello, Thanks for this nice works. I am wondering how to speed up the training phase. It takes quite a long time (a week for my device) for one run.
or if it is possible to decrease some amount of the parameters (total_trails, steps_per_trails) and still has a decent result.
Thanks for advanced.

Hi @Linsooon, are you running the hyperparameter optimization? In case yes, you can skip it. About the training phase, you can check the tensorboard logs to verify if the agent is performing satisfactorily before completing all the pre-specified episode numbers. You can use a Callback to perform this early stop automatically using SB3 as described here. Another alternative is trying to migrate Stablebaselines3 to  SBX (stablebaselines using Jax).

I am running run.py only. thanks for your response. I will try those advice.


CC #98 Bug fixes: Quad Capacity & Mouse out of bounds

Fixed two bugs.
Bug: A quad could contain more than 4 points when the parent's points overlaps with it
Fix: The parent now moves all of its points to the appropriate quad when subdividing.
Bug: Moving the mouse cursor out of range for the root quadtree's boundary would throw a TypeError when qtree.query() returns null.
Fix: Only query the quadtree when the mouse is within the root's boundary.

It seems like Git messed something up here since it shows over 700 file changes. Could you please resubmit a fix if the bug is still there? Thank you!


Parent restriction not respected in clipboard

The CMS toolbar does not respect the plugin parent restrictions when the plugin is copied (into the clipboard).
This was first reported in https://github.com/divio/django-cms/pull/4559

[ ] Create frontend tests - @vxsx
[ ] Merge #4914 into 3.2.x
[ ] Forward port to 3.3.x

frontend tests are a thing in a fork and they will not be merged for some time, so I'd suggest mergin/forwardporting and i'll do the tests separately


gradle j2objcTranslate Task Configuration

I tried to add the realtime-json lib to J2Objc https://github.com/goodow/realtime-json
realtime-json-0.5.5-SNAPSHOT-sources.jar is under ${projectDir}/lib/
Here is my build.gradle config:
j2objcConfig {
j2objcHome "/Applications/J2Objc/j2objc-0.9.6"
destDir "${projectDir}/../number-incrementer-ios/Classes"
translateFlags "--no-package-directories --prefixes prefixes.properties --mapping method-mappings.properties -use-arc -classpath ${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT-sources.jar"
compileFlags "-ObjC -ljre_emul"
translateJ2objcLibs "guava-14.0.1.jar", "javax.inject-1.jar", "jsr305-3.0.0.jar"
filenameCollisionCheck = true
testExecutedCheck = false
}
Running translation:
gradle j2objcTranslate

Leats to the following errors:
error: /Users/mg/Documents/Grails/GGTS3.6.2/TestJ2Objc/src/main/java/com/example/incrementer/shared/JSNIExample.java:3: The import com.goodow.realtime.json.JsonObject cannot be resolved
error: /Users/mg/Documents/Grails/GGTS3.6.2/TestJ2Objc/src/main/java/com/example/incrementer/shared/JSNIExample.java:14: JsonObject cannot be resolved to a type

This looks like my configuration is wrong.
Given an external library mylib-source.jar
What is the minimal configuration needed to make this lib working with gradle j2objcTranslate?

Okay I used javac first and found the error a lib was missing:
translateFlags "--no-package-directories --prefixes prefixes.properties --mapping method-mappings.properties -use-arc -classpath ${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT-sources.jar:${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT.jar"

Is is possible to split the translateFlags over more than one line?

The answer is only a Google search away ;-)
http://mrhaki.blogspot.com/2010/11/groovy-goodness-string-continuation.html
On Wed Feb 11 2015 at 1:32:55 PM Confile<EMAIL_ADDRESS>wrote:

Okay I used javac first and found the error a lib was missing:
translateFlags "--no-package-directories --prefixes prefixes.properties --mapping method-mappings.properties -use-arc -classpath ${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT-sources.jar:${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT.jar"
Is is possible to split the translateFlags over more than one line?
—
Reply to this email directly or view it on GitHub
https://github.com/brunobowden/j2objc-gradle/issues/7#issuecomment-73969248
.

Thank you. Is there a way to set the classpath lib from dependencies section. For example if you have:
dependencies {
   compile 'com.google.dagger:dagger:2.0-SNAPSHOT'
}

do I have to link to the local jar file or is there any shortcut?

I tried this:
translateFlags "--no-package-directories --prefixes prefixes.properties --mapping method-mappings.properties -use-arc\
		-classpath ${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT-sources.jar: ${projectDir}/lib/realtime-json-0.5.5-SNAPSHOT.jar"

but it did not work so I switched to the """ notation for multi row Strings.

Fixed now.


Fetch before checking out last good commit

Quoting from a comment in the test:
The setup is when the last-good-commit is a commit "in the
future" from the point of view of the locally checked-out copy
of the repo.  In that case we get an exception with "fatal:
reference is not a tree" when we attempt to checkout the l-g-c
Simply fetching will resolve the issue.

Thanks!


The following minimal example function
func F[T error](x T) {
	if any(x) == nil {
		panic(0) // or whatever
	}
}

triggers a "impossible condition: non-nil == nil (nilness) cond" warning by the nilness analyser.
I'm using vscode with its own gopls integration (which, as opposed to calling go vet (by default, unless opted-in) actually performs nilness analysis, as far as I understand).
Of course, the nilness check is actually not bogus:
Calling F[error](nil) will happily run panic(0).
I suspect the analyser somehow thinks T is a non-interface type.
What did you see happen?
An impossible condition: non-nil == nil nilness (cond) warning flagged by my IDE
What did you expect to see?
no warning.

There's an additional situation when this warning happens:
var myPtr *someStruct

// code which may or may not set myPtr

// bogus impossible condition nil != nil
if myPtr != nil {
}

@oakad Can you provide more details than that? A minimized reproducer is usually best.

That's what I'm getting in my VSCode window. How do I package it?

A minimized copy of the function (or the relevant snippet) is usually best. This can be shared on a github issue or go playground. Along with the diagnostic message and version info.


using GITHUB_TOKEN for writing repository settings

Hey, we've been implementing this action over the last days and have found that the actions' GITHUB_TOKEN is not permitted to read/write most of the settings, that probot needs to access.
Do we have to use a PAT (which is somewhat tricky to share across an organization) or is there a way to achieve this with a repository token?

What a great question!
While the built-in GITHUB_TOKEN will allow you to set repository labels, it doesn’t have powers to set branch protections, much less rename repositories or set contributors. For that, you’d need a token that has repository administrator powers. Of course that could be done with a PAT — but sharing a PAT with adminstrator powers across an organization is genarally A Very Bad Idea.
A better way is to use a GitHub Application to generate tokens with just the powers you need. While the setup for that is a little complicated, it app tokens are much more flexible in the permissions they offer — and probably a better fit for this use.
And of course, there’s an action for that:
https://github.com/peter-murray/workflow-application-token-action

Ha thank you so much. Are you open for a PR to rename the GITHUB_TOKEN in the readme to something else ? Was a Little confusing at First (but of course obvious once you find out what permissions the token has)


Question about your mlm mask

I notice that in the pretrainng process, the mlm words do not be masked? Is this a bug of this version code?

No, we preprocess the data including masking words and constructing the ROP task before the pretraining. See this


test: add test for missing child node

This test is providing a regression test for #12165. As promised in #12203.
It verifies that a child link is correctly made in the case of a step with outputs.
Verification
This test fails if #12203 is reverted.

Consider merging multiple PRs into one?

Consider merging multiple PRs into one?

It's not usually good practice, why would you prefer it?

Just reduces the review burden. Currently, you have like 3 PRs for 3 different tests. Just a personal preference for changes like these


Add tokens to opinions in the leaderboard page

Little icons on the right of the opinion that show its progression from creation to implementation. Like scout badges, each one represents an achievement (approved, scheduled, reserved, bill/resolution).

I'm not sure there's space on the leaderboard page for these opinion tokens.

Considering the little amount of space on the leaderboard page, we can wait until after launch to reevaluate.


update hyperlink hover color and cta size

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #188, #189
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
update hover color implementation and cta button size to be consistent with auro-button
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: