fix(module/vmseries)!: update private IP address option name

Description
Fix discrepancy between documented and actual network interfaces option to specify static private IP address.
Motivation and Context
Resolves #121.
How Has This Been Tested?
Apply common_firewall example.
Types of changes

Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

looks correct :+1:


Use Cell rather than RefCell in unix check_signal

Cell has less overhead, and we don't require a ref into the cell.

Good catch, thanks :)


'MQTTnet.Exceptions.MqttProtocolViolationException: Received packet 'PubAck: [PacketIdentifier=1] [ReasonCode=]' at an unexpected time.

Description:
Ver: 3.1.2
I am using MQTTnet to create MQTT Broker (Server). When I public a message. A client will get message and public the results.
However, after I public the message, I got this error
Client 'MQTTnet.Exceptions.MqttProtocolViolationException: Received packet 'PubAck: [PacketIdentifier=1] [ReasonCode=]' at an unexpected time.    at MQTTnet.Server.Internal.MqttClientConnection.ReceivePackagesLoop(CancellationToken cancellationToken)': Error while receiving client packets.
After that, the client is disconnected.

I have tried create a client using MQTTnet and connect to my MQTTnet server. Which will not occur this error.

Do you mean PublishAsync() API? If yes, I have already awaiting

This is about creating server
`private async void MqttAsync()
{
MqttServerOptionsBuilder optionsBuilder = new MqttServerOptionsBuilder()
.WithConnectionBacklog(100)
.WithDefaultEndpointPort(1883)
.WithPersistentSessions()
.WithMaxPendingMessagesPerClient(1000);
        mqttServer = new MqttFactory().CreateMqttServer();
        await mqttServer.StartAsync(optionsBuilder.Build());

        mqttServer.ApplicationMessageReceivedHandler = new MqttApplicationMessageReceivedHandlerDelegate(e =>
        {
            ReceivedMessage(e.ClientId, e.ApplicationMessage.Topic, Encoding.UTF8.GetString(e.ApplicationMessage.Payload ?? new byte[0]));
        });

        mqttServer.ClientSubscribedTopicHandler = new MqttServerClientSubscribedTopicHandlerDelegate(e =>
        {
            Debug.WriteLine(e.TopicFilter.Topic);
        });

        mqttServer.ClientConnectedHandler = new MqttServerClientConnectedHandlerDelegate(e =>
        {
            Debug.WriteLine(e.ProtocolVersion);
        });

        mqttServer.ClientDisconnectedHandler = new MqttServerClientDisconnectedHandlerDelegate(e =>
        {
            Debug.WriteLine(e.ClientId);
        });

    }`

And this is about sending message.
`
string topic = "Test_topic";
string payload = "Test_message";
            MqttApplicationMessage applicationMessage = new MqttApplicationMessageBuilder()
                .WithTopic(topic)
                .WithPayload(payload)
                .WithAtLeastOnceQoS()
                .Build();

            if (mqttServer != null)
            {
                if (mqttServer.IsStarted)
                {
                    try
                    {
                        await mqttServer.PublishAsync(applicationMessage);
                    }
                    catch (Exception ex)
                    {
                        Debug.WriteLine(ex);
                    }
                }
            }`

Please test with the latest version 4. There is also a sample available how to startup a simple server.

Closing this due to inactivity.


Rust-only methods in #[pymethods] impl

Hello, please add some functionality that will make methods in #[pymethods] impl don't visible in python

You should be able to put those methods into a additional/separate impl block without #[pymethods] attached. These methods are not visible to Python and are not processed by PyO3 in any way.

Note that you can have multiple impl blocks, unlike in python where you can only have one class definition.


replace lock files with two-phase compaction

Restic currently uses lock files to prevent dangerous race conditions when performing non-additive operations (basically, prune).
The lock file system works quite well across all supported backends, although there's some downsides. A forgotten / crashed machine will not clear the exclusivelock file; you can clear it manually, but maybe the other machine was a laptop that had only gone to sleep, and when it re-awakes it will wreak havoc.
There's an alternative: two-phase fossil compaction.
The project "duplicacy" is shared-source (not FOSS) but has an interesting design document available on this topic:

https://github.com/gilbertchen/duplicacy/blob/master/DESIGN.md

Can these ideas be applied to restic at the pack level?
Can we fossilize a pack by only making changes to the index files?

At the time of writing, the duplicacy project has this to say about restic on its README.md file:

restic is a more recent addition. It is worth mentioning here because, like Duplicacy, it is written in Go. It uses a format similar to the git packfile format. Multiple clients backing up to the same storage are still guarded by locks. A prune operation will therefore completely block all other clients connected to the storage from doing their regular backups. Moreover, since most cloud storage services do not provide a locking service, the best effort is to use some basic file operations to simulate a lock, but distributed locking is known to be a hard problem and it is unclear how reliable restic's lock implementation is. A faulty implementation may cause a prune operation to accidentally delete data still in use, resulting in unrecoverable data loss. This is the exact problem that we avoided by taking the lock-free approach.

I've seen what duplicacy does, but I have a hard time understanding how to apply this to restic without changing the repository format completely (and basically doing exactly what duplicacy does).

Here is a proposal:
During a repository prune, the prune binary should:

Find currently-running jobs ("X,Y,Z")
Determine which packs can be removed ("A,B,C")
Find currently-running jobs
If they do not match $1, return to step $1 (or abandon)
Upload a new index with all known packs still, but two additional messages:
"A,B,C packs can be deleted after X,Y,Z jobs complete without using them"
"this index is provisional until confirmed"
Find currently-running jobs
If they match the jobs from $1 and $3, then
upload an extra index file, containing no content except "confirming $4".
else if they do not match:
delete the index from step $4, return to step $1 (or abandon)

During a backup job:
Any new backup jobs will load indexes at first. The backup job should not use any of those packs A,B,C at all, regardless of whether the mark is provisional. The only jobs that will use packs A,B,C are those X,Y,Z jobs that were currently running.
During a successive prune:

When a prune next runs,

If backup jobs X,Y,Z are no longer running (no lock files for them) then

it can check if X,Y,Z needed to use packs A,B,C.

If they were unused, it is safe to remove the packs without needing an exclusive lock.
If they were used, then prune can check the indexes to determine exactly which packs were used, and only delete the packs that were unused.

Comments:

Since it requires cooperation from all restic clients, this should be part of a breaking repository version number bump.
This proposal excludes repacking, and simplifies prune to only delete entire packs. Honestly, this is more realistic when cloud storage is involved.
Lock files still exist, but, none of them need to be exclusive.

I am interested in discussing this proposal, and implementing a PoC to test performance and correctness, but since this work is related to my $DAYJOB i cannot legally submit nontrivial changes.

I'll need to think about this, but I like the general direction. We can certainly options so that an index file is able to mark pack files as deprecated.
The lock file ID btw is changed regularly, the lock is refreshed (recreated) every five minutes.

I have yet another proposal, which boils down to

Introduce new lock types to lock individual blobs and snapshots
Use double-check locking to detect and retry conflicting locks

This approach does not require new repository format and only needs strongly consistent repository list operation (which can be approximated with multiple eventually consistent lists). I have a google doc with little more details (just request access if you want to comment).
I am happy to work on the implementation but obviously won't do anything until there is agreement about the approach.

I made a suggestion which would remove the need for a lockfile altogether in #1544 without the need to change the backend storage format and individual calls to the backend for additional locking checks.
Summary:

Defered prune: purge just writes the objects it wants to delete to a file and is given this list on the next run and only delete objects it has seen in the previous run.

Defered prune:
As for parallel prunes, why not use a two-phase "commit' for prunes?
restic prune only writes the objects it would delete out to a file, on the next run it would run normally meaning first create a list of objects to prune from the whole repository, and after that only delete files found in the list written on the run before. All objects not in the list are written out into a new list. This would remove the need for any locking with the safety guarantee to not delete objects from running backups which where shorter than the prune interval (which if prune only runs every month is quite a good safety margin).
This idea would have the additional advantage as to not need a new repository format too.
The list might be possible to store in the backend store itself, though i'd prefere not to as the client should not assume that he can delete files from the storage.
I'd prefere to not have any finer grained locking as this might influence the performance pretty negatively, especially for cloud storages with high latency.

purge just writes the objects it wants to delete to a file

This file tells backup, check, etc, what objects to avoid. I called this file "object lock file", but I think otherwise we are talking about the same approach.

Thanks for all the suggestions! I feel this is a productive discussion which will (eventually) result in an implementation :)

I made a suggestion which would remove the need for a lockfile altogether in #1544 without the need to change the backend storage format

I'm not sure which suggestion you're talking about, would you mind repeating it here?
Just "not create lock files" does not work for all users with the current implementation, and since the current repo format definition states that there will be lock files, not creating them is also a "change" in this sense. While this would allow us to keep data in the repo (we don't have to rewrite everything), it would still require incrementing the repo version number in the config. Which is a breaking change for older clients (for good reasons).

@fd0 i've already repeated the suggestion here in my post from 15.Feb, sorry if i was not clear enough.
basically it's a two phase commit but with other tradeoff's, eg. that the two phase commit is done "offline" and in the sole responsibility of the client doing the prune (not for the client doing the backup/write of the objects).
the core idea is that the client stores locally a list of object it want's to prune, and only on the next prune run really deletes them, which gives a safety margin against corruption for every backup that has not taken longer than the interval of the two prune calls.
Though this idea changes the contract in the repo format about lock files.
Thus there is an additional idea, just write the lock file, but never delete them.
This would change the handling in such a way that restic would write a lock file for the current instance to be available at least once. Thus all "dumb" clients doing prune without two phase commit for deletion of object would error out and be safe in that regard. All clients with two phase commit for prune would just ignore the lock file as they don't need them.
For creating backups with restic, this code would just ignore the lock file as it's not needed on the backup side, the only thing that can happen as i can see is that objects would possible be duplicated in the backend store, but no data being corrupted.

I think there are many ways to skin this cat, but in the end we will need to prove the new implementation guarantees snapshots can't reference deleted blobs under all concurrency and failure scenarios. The secondary goal is to guarantee all unused pack and index files are eventually deleted. Right now I don't see how to do that without backup knowing the list of blobs being deleted by concurrent prune, which I think in turn requires strongly consistent datastore to share the list of blobs between backup and prune. I still believe we are thinking about the same basic idea, just want to emphasize the guarantees the new implementation should provide.

@ifedorenko "Right now I don't see how to do that without backup knowing the list of blobs being deleted by concurrent prune"... please read my suggestion from above again and feel free to ask questions.
But right now i feel that you have neither read nor understood the suggestions i've given (now even a second time with additional information).

I did read, multiple time, honestly ;-)
When prune selects blobs it wants to delete, backup must not reuse to-be-deleted blobs. Likewise, when backup selects blobs it wants to reuse, prune must not delete to-be-reused blobs. Do you agree these are required properties of any prune/backup implementation?
I think you propose to:

remove to-be-delete blobs from the index, so they are not reused by subsequent backups.
wait long enough for all concurrently running jobs to finish (and I assume for eventually consistent backends like S3 to resolve all inconsistencies).
validate blobs selected in phase 1 are not used by any snapshot
delete blobs selected in phase 1

Did I get your proposal more or less right or completely misunderstood your idea?

@gebi Thanks for the additional information. I think I understood what you've proposed, I need to think about this for a bit. Would you mind confirming that @ifedorenko's summary (thanks!) is accurate?

Hi,
thx for going through all my ideas about a lockless prune operation!
@ifedorenko, yes you are on the right track
ad 4. you understood correctly.
ad 3. exactly correct too.
ad 2. is done with eventual consistency yes, but the time intervals thought about are way above anything eventually consistency delays you know of any common cloud storage (eg. i thought about a time delay of a month or more, but the safety margin depends on the the difference to the time it takes for your longest backup job to run).
ad 1. this is indeed a point that needs a seconds thought and might it necessary to use a three phase commit (idea below).
With your point 1 there is an additional step needed for prune operation:

prune checks which backups it want's/is instructed to remove, this results in a list of objects to remove from the shared datastore.
prune stores this list locally on the machine running the prune, let's call this list the list of objects indended to be deleted.
Prune takes the list of objects intended to be deleted, creates the set difference of this list with all objects referenced by backups in the timeframe between 1. and 3. this creates a new list of objects to be deleted but this time prune "proposes" this list into the datastore as a new index.  (this is an additional step for the missing handling of your point 1).
On the 3. invocation prune can now finally delete the difference between the "old" index from before point 3 and the one created in 3. subtracted for objects that are referenced in backups that the initial prune should not have been deleted (those are objects from backup runs that where executed in parallel to the prune operations and are then again valid objects and need to be added to the index again).

For context, what operations are we wanting to support in parallel? Are we scoping to support only one client per repo (but we want that client to be able to issue backup and prune operations at once), or could one machine be creating a snapshot while another with the same source data be issuing a prune?
From @ifedorenko's earlier comment:

I think there are many ways to skin this cat, but in the end we will need to prove the new implementation guarantees snapshots can't reference deleted blobs under all concurrency and failure scenarios
I agree, especially if this is going to make a breaking change to backwards compatibility (after all, if there's a breaking change then why not break it to the two-step design?).

from @gebi's comment:

which gives a safety margin against corruption for every backup that has not taken longer than the interval of the two prune calls.
If I am understanding your proposals above, this is correct but it still would not handle the case if a new backup command was issued just after the second prune operation started (that process that would do actual deletion). Is that correct?

I want to run backup on one computer and prune on another concurrently. I want to run backup on multiple systems concurrently too. I also want to be able to start prune on multiple systems concurrently, but I would not mind if only one instance did any work while other instances waited.

I've tried to outline and describe what I can imagine doing here: https://gist.github.com/fd0/d24abf8301724988919c80e65aba9b6d
There are a few open questions at the end of the document, and it feels to me I forgot something. So, please have a look and let me know what you think :)

I can imagine a simple two-step prune that has an exclusive lock for a shorter amount of time.  It would still disallow some other operations (prune, rebuild-index, check) while running.
It would require the introduction of a new "upgradeable lock" type.  This kind of lock prevents other concurrent upgradeable locks and exclusive locks, but allows concurrent shared locks.  It can be upgraded to an exclusive lock.

Prune acquires an upgradeable lock on the repository.
All of the standard prune operations are completed, through the object-marking garbage collection but the process stops short of rewriting or deleting any packs.  (We do "mark" but not "sweep.")
After this completes, prune upgrades its lock to an exclusive lock.  This should fail if there are any other locks in the repository.  (If there are, prune should wait until there are no other locks.)
Once the exclusive lock is acquired, prune will look for any new snapshot objects.  If any are found, it reloads the indexes and does object-marking against those snapshots.
The process continues as normal, deleting/rewriting packs, and rebuilding indexes.

This approach allows other clients to write to the repository while prune is busy marking objects in memory, which in my experience is the longest part of prune's operation.
I think the new lock type can be added in a backwards-compatible way.  The lock would otherwise look like a shared lock but with a new "upgradeable" flag.  This will allow other shared-lock operations (like backup) to operate correctly with the new prune operation even if those clients do not understand the flag.  Existing exclusive-lock operations will also work correctly since they will refuse to operate in the presence of any lock.

I've seen what duplicacy does, but I have a hard time understanding how to apply this to restic without changing the repository format completely (and basically doing exactly what duplicacy does).

That may actually be the best way to go to make it work better long term, but would obviously be a massive rewrite of so much of the code :-/
I personally had to switch my backups to duplicacy as the restic design impacts its memory use and has left me with some old backups that I can't easily restore from :-(

Just want to note that I implemented a two-phase compaction in rustic.
The idea is the following:

in the index format additionally to packs there is a stuct  packs-to-delete which is basically the same as packs but also has an extra manditory attribute time for each pack to save the time when that pack was marked for removal.
The prune works basically the same as now, but instead of removing packs, they are kept and saved in the index as packs-to-delete.
Commands like backup don't ever use packs-to-delete. This also allows restic versions which don't know about this struct to backup to a repo while another two-phase pruning is running in parallel.
prune also handles packs-to-delete: those packs are either recovered, i.e. moved to the packs struct (when prune realizes that they in fact are needed, e.g. by backup which ran parallel to a previous prune ) or kept in the state to-delete (when the mark is not old enough) or really removed (when the mark is old enough).

Feel free to try it out at www.github.com/rustic-rs/rustic.

"The lock file system works quite well across all supported backends"
I think that's a wrong assumption. Lock files are not as trivial as many think. There is a reason why NFS needs special lock servers and why distributed systems' bottlenecks are usually the coordinators (~lock servers). Just putting a lock file on an arbitrary backend does NOT prevent race conditions as that backend usually does NOT guarantee synchronized visibility to different remotes.
IMHO the easiest way to fix this is to use a tiered locking system: primarily use a local lock-file and if you want, put an additional lock-file on the remote side which indicates the current client locking the repository. However you can also just omit the remote file since that will never work reliably. If you use more than one client, there is only one way: make sure to use separate repositories (ie put the client name/address into the repository path/id etc.).

@jjYBdx4IL You are right - eventually consistency is what most cloud storages do guarantee. And this is actually a bad idea to use for locking. Now, restic operations like prune usually run for a pretty long time and the used cloud storages usually are able to synchronize the (lock) file information pretty fast - so the probability of getting an error due to this is actually quite low. But note I wrote "usually" - we don't know what happens if the cloud storage gets under stress for some reason...
There is the alternative of using delayed check for files. This is how a two-phase prune actually works, but it could also be a way to get locks really reliable. Of course, this depends on good and valid assumptions for sync times - under all circumstances.

I really don't care (the delayed check is just meddling with something that just isn't really possible so IMO it shouldn't even be tried in the first place - if you run multiple clients against the same repo, it's the user's responsibility and you should simply display a big fat warning). I'd prefer a local lock/semaphore option so aborted operations can automatically recover and do not produce a stale lock file that blocks further operations as I have the backup running in the background.


Cannot compile vim with MinGW on Windows10

I am trying to compile vim with MinGW on Windows10:
 cd src
 mingw32-make -f Make_ming.mak gvim.exe

I get the following error message when the compiler tries to compile beval.c:
In file included from beval.c:11:
beval.c: In function 'find_word_under_cursor':
vim.h:1705:18: error: 'INT_MAX' undeclared (first use in this function)
 # define MAXCOL  INT_MAX  // maximum column number

Which version of MinGW do you use? How do you install it?

I am not sure how I can determine the version of MinGW. I download the (what I believe to be) newest version from MinGWs site and installed it under C:\MinGw.
When I execute mingw32-make -v, it prints
GNU Make 3.82.90
Built for i686-pc-mingw32
Copyright (C) 1988-2012 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

gcc -v prints
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=c:/mingw/bin/../libexec/gcc/mingw32/8.2.0/lto-wrapper.exe
Target: mingw32
Configured with: ../src/gcc-8.2.0/configure --build=x86_64-pc-linux-gnu --host=mingw32 --target=mingw32 --prefix=/mingw --disable-win32-registry --with-arch=i586 --with-tune=generic --enable-languages=c,c++,objc,obj-c++,fortran,ada --with-pkgversion='MinGW.org GCC-8.2.0-5' --with-gmp=/mingw --with-mpfr=/mingw --with-mpc=/mingw --enable-static --enable-shared --enable-threads --with-dwarf2 --disable-sjlj-exceptions --enable-version-specific-runtime-libs --with-libiconv-prefix=/mingw --with-libintl-prefix=/mingw --enable-libstdcxx-debug --with-isl=/mingw --enable-libgomp --disable-libvtv --enable-nls --disable-build-format-warnings
Thread model: win32
gcc version 8.2.0 (MinGW.org GCC-8.2.0-5)

Hm, I thought we need either mingw-w64 or msys2 compiler.

If this is the case, it should imho be changed in the build instructions under src/INSTALLpc.txt. They currently read
3. MinGW
========

(written by Ron Aaron<EMAIL_ADDRESS>
This is about how to produce a Win32 binary of gvim with MinGW.

First, you need to get the 'mingw32' compiler, which is free for the download
at:

    http://www.mingw.org/

or you can use 'MinGW-w64' compiler.

    http://mingw-w64.sourceforge.net/

... etc ...

I created a PR: #5248.

Just to make sure I interpret the thread correctly: "win32: Drop support for old msvc" also means that Mingw32 is not supported anymore (and I have to use Mingw64).

Yes, the header files and libraries come with the original mingw is too old to compile Vim.

I created a PR: #5248.

Please close this issue if that patch fixes the problem.
--
I AM THANKFUL...
...for all the complaining I hear about the government
because it means we have freedom of speech.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I think we can close this now.


currently open ended learning doesn't work.

https://github.com/MicrosoftDocs/azure-docs/issues/107548#issuecomment-1510607435
An error occurs if the prompt is blank.
Of course, the completion is filled.

@taktak1
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

@taktak1
Thanks for your feedback! We will investigate and update as appropriate.

Hi @taktak1
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an Azure support request.
Thank you for your time and patience throughout this issue.

@taktak1
We are going to close this thread if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Add Me Gusta by AYNES Ft Marco Mares

Song Name
Me Gusta
Artist Name
AYNES Ft Marco Mares
Source
Youtube
Youtube Link
ShKI3lp58Ro
Direct File Link
No response
Song ID
321
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 98607976
Downloaded: No

it still relevant
El vie, 1 nov 2024 a la(s) 8:31 p.m., github-actions[bot] (
@.***) escribió:

This submission is stale because it has been open for 14 days with no
activity. If it is still relevant, please leave a comment and we will
reopen it.
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/5149#issuecomment-2452808399,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGY253YSBAWYJWRHYOTYASLZ6Q2P5AVCNFSM6AAAAABQFFL4WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJSHAYDQMZZHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Testing: Add tests for createList

Description
There are currently no tests for the createList functionality. Some should be added.
Libraries to use

Jest
Have a look at createContact.test.ts (Code) to see an example how to test

Files involved

src/lists/createList.ts - File to be tested
src/lists/createContact.test.ts

Hello, I would like to resolve this issue

Can you assign this issue to me?

Assigned to you! Feel free to start working on this! Thanks for considering to contribute.

Thanks @igeligel, I'm starting at hacktoberfest, can I consider submitting the resolution tomorrow or is there a deadline for signed activities?

This will be my first contribution

There is no deadline of course. Take your time!

@igeligel good night!
With the pull request I add the solution to the issue, but I have some questions, I don't see the need to test all cases of handleApiGlobalAxiosErrors in the validation of my controller, as handleApiGlobalAxiosErrors must have its isolated test.
The rule for throwing the EmailOctopusError exception is the return of an unknown error, so theoretically only all mapped errors will be caught by the handler and only unmapped errors will return EmailOctopusError, correct?


[wip] swav

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Adapts official FAIR implementation to bolts.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @ananyahjha93! Thanks for opening this PR.

In the file pl_bolts/models/self_supervised/swav/swav_module.py:

Line 255:21: E251 unexpected spaces around keyword / parameter equals
Line 366:1: W293 blank line contains whitespace

Do see the Hitchhiker's guide to code style

please add

cifar10 baseline weights
loading instructions
table of results
charts with results
tensorboard links


Update dates in Changelog for 2023

The newest dates in CHANGELOG.md reference 2022, but I believe it should be 2023.

Coverage: 92.392%. Remained the same when pulling df6cdb60ab64230a1b0e17f38b558bc94acf4f9b on rudidykman:master into 0d27bcb60b9bfd57fabf896715350089e0b1050c on knex:master.


Open in folder_browser instead of file_browser

Is your feature request related to a problem? Please describe.
I am looking for a way to use telescope to create files/directories from a keymap. The folder browser mode fulfills this, but I haven't found a way to start in that mode from a keymap.
Describe the solution you'd like
The ability to launch the folder_browser with something like :Telescope folder_browser
Describe alternatives you've considered
I'm open to alternatives, but not sure what they would be offhand
Thanks for your attention

You can you :Telescope file_browser files=false :+1:

I am looking for a way to use telescope to create files/directories from a keymap.

For this though, you can still be in the file_browser "mode". Unless I'm confused with what you're trying to do.

That works for me. Thanks!


Multiply degrees in vmount/DO_MOUNT_CONTROL

Describe problem solved by the proposed pull request
Gimbals from Gremsy expect DO_MOUNT_CONTROL messages with the degrees multiplied by 100. This pull request adds the possibility to specify this multiplier, which defaults to 1.
Test data / coverage
Tested with Gremsy Pixy F gimbal. Responds correctly to /mavros/mount_control/command (see https://github.com/mavlink/mavros/pull/1257) with mode 2 (DO_MOUNT_CONTROL).
Describe your preferred solution
Some gimbals vary a bit from the standard MAVLink messages. Even though I think it's bad to deviate from the MAVLink standard, this was the only feasible solution in one of our drones as we had no UART ports left.
Describe possible alternatives
For this type of gimbal, create a UART connection to the gimbal from a companion computer and send those custom MAVLink commands not from Pixhawk. This requires a companion computer though.

Thanks @dayjaby. Please do make format to fix the style issue.

I am using a Gremsy S1 V2 and Gremsy is not following the convention listed here for angles in Mavlink messages:
http://ardupilot.org/dev/docs/code-overview-adding-support-for-a-new-mavlink-gimbal.html
If I command pitch: -45 (which is actually entered as 45 in QGC but gets mapped to -45 before being stored in the mission plan) and yaw: -45, the gimbal should look down and to the left. What the gimbal actually does is look up and to the right.
The generalized solution offered at the beginning of this PR would allow me to fix the problem. It would also allow me to mount the gimbal inverted and then also invert the angles sent to the gimbal using PX4. It would be great if the gimbal would offer settings to correct for all of these things but it currently does not.
Also, other than the Ardupilot convention that I previously linked to, I have not seen any "universal" convention of angular definition for Mavlink gimbals so flexibility on the PX4 end of things would be nice to have.
https://mavlink.io/en/messages/common.html#MAV_CMD_DO_MOUNT_CONTROL

@mwiatt thanks for your insight here. It's indeed a bit messy and - from what I can tell - the problem goes back to the fact that the MAVLink message did not have the frame/signs defined for a long time leading to implementations. I then came around and documented the signs in the usual NED frame as used normally with MAVLink.
I suggest that we get this pull request in and then you can make another pull request where you add an option for the Gremsy S1 so that it works correctly for you.

@dayjaby could you run make format here and rebase on master and then we can get this merged.

@julianoes formatted and rebased.

@julianoes Agree on getting the current version in.
I am currently working with Gremsy to fix their angular definition. They seem very willing to fix and update their firmware. This is great but should also serve as a bit of a warning that this PR may become obsolete with a future Gremsy firmware release for the Pixy_F.

@dayjaby Have you checked to make sure that the sign convention of the Mavlink angles matches between this PR and the current version of the Pixy_F firmware? It doesn't on the current version of the S1 firmware. This isn't super obvious in RC mode but is glaringly obvious when you use a ROI waypoint where PX4 is calculating the angles.
Also note that this PR is tied to a specific version of Pixy_F firmware that is likely to change as Gremsy starts fixing their FW. Would you please post your Pixy firmware version so that it is documented for the future. I wonder if a note should be added to the params file.

I suggest that we get this pull request in and then you can make another pull request where you add an option for the Gremsy S1 so that it works correctly for you.

@julianoes I agreed a few days ago but now I am not so sure. This PR is tied to not only a specific version of HW, but also a specific version of firmware. This will probably end up causing confusion when Gremsy updates their firmware. The current version of PX4 will end up chasing the latest and greatest from Gremsy.
Arbitrary scaling factors for roll/pitch/yaw would not only work in this case but also for any other version of firmware that Gremsy releases. Or any other gimbals that dont follow the convention for that matter.

@mwiatt If gremsy decides to update their firmware to the proper mavlink implementation, the px4 user can easily change the new parameter back to default. I think we could just make it explicit in the parameter description for which firmware versions it is (probably) working

Jenkins complains:
[2019-11-06T17:16:02.410Z] ../../src/modules/vmount/output_mavlink.cpp: In member function 'virtual int vmount::OutputMavlink::update(const vmount::ControlData*)':
[2019-11-06T17:16:02.410Z] ../../src/modules/vmount/output_mavlink.cpp:94:28: error: 'math' has not been declared
[2019-11-06T17:16:02.410Z]    vehicle_command.param1 = math::degrees(_angle_outputs[1] + _config.pitch_offset);
[2019-11-06T17:16:02.410Z]                             ^~~~
[2019-11-06T17:16:02.410Z] compilation terminated due to -Wfatal-errors.

@julianoes Thank you! I included mathlib/mathlib.h now. I thought this was included via math.h already.

I changed the description strings.
I probably have to do another pull request later for vmount regarding this specific gimbal.

Hm. If Gremsy if fixing this anyway, then I'm reluctant to merge this and would rather wait for an update from their side. Otherwise it will get confusing.

Hm. If Gremsy if fixing this anyway, then I'm reluctant to merge this and would rather wait for an update from their side. Otherwise it will get confusing.

I know by experience that waiting for third-party to solve issues can be a real developer blocker. I would rather have it fixed the way we are able to and then adjust it to the third-party updates when it makes sense.


Should it escape more control and unusual characters?

I noticed that js-stringify replaces low ASCII control characters with Unicode hex escapes. Should it do similar for some others, like U+0090 device control or U+FEFF zero width no-break space?
My Firefox has no problem with them, as it has no problem with U+0002, so that's probably not the criterion. How do you decide?
PS: In case it helps, here's a list of characters I consider suspicious.

We escape them because they can possibly contribute to an XSS attack, when the source is directly embedded in an HTML <script> tag.

Yes, there are two types of character we escape in this way:

Characters that are not valid in a JavaScript string (\u2028 and \u2029) - needed to prevent bugs
Characters that could have special meaning if in an inline script in HTML (<, >, /) - needed to prevent XSS

We don't escape characters that are already escaped by JSON.stringify, or don't fit into one of the two categories above.


Common Arabic text filter

With cached result used within and reset in the seeder, plus a decent heads-up in both config AND README

Added in #34


Can not update the now cli nor remove it

Hello,
I wanted to use the now-cli on the windows subsystem linux called "bash on ubuntu on windows" (it's like an ubuntu linux, hidden in windows 10 distribution).
I can't remove my global package now globally because a script install.js is missing. Can someone help me to understand what I'm missing ?
/mnt/c/Users/me ⌚ 23:14:11
$ npm install -g now
/root/.nvm/versions/node/v10.9.0/bin/now -> /root/.nvm/versions/node/v10.9.0/lib/node_modules/now/download/dist/now

><EMAIL_ADDRESS>postinstall /root/.nvm/versions/node/v10.9.0/lib/node_modules/now
> node download/install.js

module.js:487
    throw err;
    ^

Error: Cannot find module '/root/.nvm/versions/node/v10.9.0/lib/node_modules/now/download/install.js'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Function.Module.runMain (module.js:605:10)
    at startup (bootstrap_node.js:158:16)
    at bootstrap_node.js:575:3
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `node download/install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-08-23T21_14_57_844Z-debug.log

Did you try to install Now Desktop?
Doing so is recommended and easiest way to use Now CLI as it comes bundled with it.
I am not sure but this bash on ubuntu on windows is rather something which tries to be Bash than Linux subsystem. IMO that's the main issue.

Please try which now to get the patch and rm <path> to remove it. After that, try installing it fresh using yarn or npm.

Hello 👋
Here some news :

@robertgrzonka as I want to develop on the subsystem, having the windows client won't suit my needs, that is why I wanted to keep the npm package. Yes my OS is Windows, but I work on a unix subsystem 😄.
@leo I used your solution and it worked in my case ! thank you 👍

I'd like to share a list of everything I did to make it work. It is still in wip, but the look and feel to have a bash env with hyper.js / now on windows is really sweet !
Have a look => https://gist.github.com/Ugarz/2f2a4bdb724e9ae03d915d2ecc6ae661


getExifOrientation does not work for some Uris

My source Uri for image file is:
file:/storage/emulated/0/Android/data/com.***/files/private-pictures/JPEG_20160720_113440_54943315.jpg
But it does not get orientation because of this code in library:
if(sourceUri.startsWith("file:///") && !sourceUri.startsWith("file:///android_asset/"))
Notice the additional // used in the if statement.
I constructed the Uri by using this:
imageViewTouch.setImage(ImageSource.uri(Uri.parse(file.toURI().toString())));

Another related issue is when I try using this:
imageViewTouch.setImage(ImageSource.uri(file.toURI().toString()));
Uri is not parsed correctly and I get this exception:
java.io.FileNotFoundException: /file:/storage/emulated/0/Android/data/com.intermedix.eics/files/private-pictures/JPEG_20160720_114307_-1897635745.jpg
Note that 'file' is from java.io.

Solved this by:
imageViewTouch.setImage(ImageSource.uri(file.getAbsolutePath()));

@JonasPrap
Any idea if I want to load image from url and then apply scale effect as like this library?
Sincerely,
Shreyash

@ShreyashPromact
I'm not sure I completely understand the question, it seems unrelated to this issue. I think you need to download the file and save it to file system, then use this library to open that file. To accomplish that I used Retrofit2 (https://futurestud.io/blog/retrofit-2-how-to-download-files-from-server), but there are other solutions to download files and save them to disk.

@JonasPrap
Thanks for quick reply. Here in my project, I have so many images on server. I am displaying that image in the app. Now, I want to add your library as I have zoom in image functionality in the app.
But as the image coming from server, I can't use your library. Even its normal thing that No user will store such big size of image in device for there app. So, is there any support available for such condition where I can direct display image from the server (URL of the image)?
Thanks.
Sincerely,
Shreyash

@ShreyashPromact
First, this is not my library :) Just reported an issue in it when I started using it.
This library requires you to store image in a file....
If the images are not too huge and you are not running into memory problems I suggest trying Picasso library for image loading and this library or ImageViewZoom library for an imageView with zoom/pan capability.

@JonasPrap
You are right. I was doing same thing. I have used glide to load image. But i want to apply zoom on same loaded imageview. While here it is view. So glide is not working as I want. Same thing will be happen with Picasso. So, I have requested to have support for that.
Thanks for your reply and support. Let me know if you have any trick to load image from url and apply such zoom effect on same.
Thanks.
Sincerely,
Shreyash

@JonasPrap looks like you found the solution, can I close this now?


Touch events are not working while page is blocked?

Are you requesting a feature or reporting a bug?
I'm not sure if this is a missing feature, or actually a bug...
What is the current behavior?
I've been able to simulate touch gestures using hammer-touchemulator here: https://www.boxfactura.com/pulltorefresh.js/demos/trigger.html
With that enabled the demo page is also working on desktop browsers.
What is the expected behavior?
I expect my tests run with this page the same way.
How would you reproduce the current behavior (if this is a bug)?
First try to move the trigger to enable the PTR, it'll work.
Now, run the test below to drag the same trigger.
Also I noticed that running the same test with -d enabled, and then unlocking the page, allowed me to pull the trigger as usual...
Provide the test code and the tested page URL (if applicable)
Tested page URL: https://www.boxfactura.com/pulltorefresh.js/demos/trigger.html
Test code
fixture('Trigger test')
  .page('https://www.boxfactura.com/pulltorefresh.js/demos/trigger.html');

test('try to drag', async t => {
  await t
    .drag('.trigger', 0, 100, { speed: 0.1 })
    .drag('.trigger', 0, 200, { speed: 0.1 });
});

I'm doing wrong or there's a preferred way to perform this kind of test?
Specify your

operating system: OSX 10.13.3
testcafe version: 0.19.2
node.js version: v9.9.0

Hi, I've tried to reproduce your scenario on the provided page (https://www.boxfactura.com/pulltorefresh.js/demos/trigger.html).
But after I pulled the trigger nothing happened. I've tried in Chrome on Windows and in Safari on Mac. However, when I entered Device Mode in chrome developer tools I was able to reproduce the required behavior and saw the 'Refresh' alert dialog. So, could you please clarify if this script is supposed to work on desktop browsers or only on mobile devices or emulators?

The library pulltorefresh is intended to be working only on mobile devices, we're using only touch* events for handling this which is working great.
We've updated the page, it was missing the hammer-touchemulator plugin, but the behavior remains the same: just pull the trigger to refresh, etc.
Adding this plugin is not part of PTR itself, but it allows us to demonstrate the behavior to users using desktop browsers.
My task is adding some e2e tests to that library, so we can ensure their behavior but I was not able to use chrome:emulator:device:iphone 6 to make my tests, it just behave the same.

I've reproduced the described behavior. It looks incorrect, so I need additional time to analyze it.


pktgen: build with the same CFLAGS as dpdk

Motivation for this change
This is related to PR https://github.com/NixOS/nixpkgs/pull/15962, cherry-picked from master branch.
Unbreak the master build failure https://hydra.nixos.org/build/36613719
pktgen requires SSE3 instructions - same as DPDK.
Interestingly build wasn't failing on release-16.03 with gcc-4.9.3 but fails with gcc-5.3.0.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

cc @domenkozar


how to disable shortcuts?

Hi,
IE has a lot of built-in shortcuts and I would like to disable all of them but not sure how to add multiple keys? can you show me an example with multiple keys please?
I know that I also have to define the virtual keycode from Win32 API
VK_F5 := 0x74 but I am confused about this code that looks like JSON:
disabled_shortcuts :=
( Join ; ahk
{
0: {
this.VK_F5: true
},
this.MODIFIER_BITMAP[this.VK_CONTROL]: {
GetKeyVK("F"): true,
GetKeyVK("L"): true,
GetKeyVK("N"): true,
GetKeyVK("O"): true,
GetKeyVK("P"): true
}
}
)
Does each object need it's own key?
Do I need to increment the object's number like
	0: {
		this.VK_F5: true
	},

	1: {
		this.VK_F6: true
	},

disabled_shortcuts :=
( Join ; ahk
{
    0: { ; this object will contain keys that need to be blocked when pressed without a modifier
        this.VK_F5: true
    },
    this.MODIFIER_BITMAP[this.VK_CONTROL]: {  ; this object will contain the keys that should be blocked when pressed with control (like ctrl+f)
        GetKeyVK("F"): true,
        GetKeyVK("L"): true,
        GetKeyVK("N"): true,
        GetKeyVK("O"): true,
        GetKeyVK("P"): true
    }
}
)

So you just need to add the key's VK to the correct object inside disabled_shortcuts

I don't understand, maybe I'm missing something but it looks like the EXACT same thing I posted? I don't see the difference? Please show me an example with multiple keys. VK_F5 AND VK_F6, for example

I've updated the comment to make it more clear

Ahh I think I understand, thanks for clarifying.
So to add another key VK_F6 without a modifier I just add that to the 0 object like
0: {
this.VK_F5: true,this.VK_VKF6: true
},
?
That seems easy enough, thanks for clarifying!

Thanks! that seemed to work and was so easy..one more question Is it possible to disable the modifier keys without an additional key? so I can have the alt (vk_menu) and control keys themselves disabled/ignored and handle them in AHK itself?


buck run :app is not working as it should. When I run the command (after quickstart) parsing of buck files and  building is successful. But after that instead of showing the output it shows "file not found : link"

Not able to reproduce, can we have more info about your system? -
eg.:

OS
buck version
how did you install buck
jvm version

I suspect that this is an issue with your buck installation

The OS is windows 10
buck version: 2<IP_ADDRESS>
I installed buck using Chocolatey ( choco version 0.10.15 )

Okay, thanks.


Same player can appear multiple times on a leaderboard

Usually, leaderboards only show the best performances of players, however this game seems to show all player performances.

This is a big one! I will need to implement authentication, which is pretty high on my priority list.
Unfortunately the timeline for this is long, since I want to set up a general game website that includes games other than Pentris, and I hope to add authentication to that and not just Pentris. I'll update this issue when things start rolling.


Use WalletClient instead of ethers signer

@Da-Colon can you please give some more context to this issue?

Closing this, it has already happened


Setup Neo4j Database to store graph data

Added at least a docker compose file

Store data in docker volumes


Fixed bug when converting negative zero to string.

This PR fixes, converting a negative zero -0.0 to string. previously we would give "-0" as the converted string and this is wrong according to spec step 2 in https://tc39.es/ecma262/#sec-numeric-types-number-tostring

If x is +0 or -0, return the String "0".

Made zero number printing more efficient because we don't use unnecessary ptr::copy_nonoverlapping, instead we directly put the 0 in the buffer

Also added some more ECMAScript compliance tests and a NOTE to format32 because it is not spec compliant.

Lets merge this :)


Csrf token is regenerated in the session after several Ajax requests in a short time

Hi,
i'm working on a Laravel 5.1 app where the user can upload many images (one by one).
the problem i have is the token is regenerated in the session after uploading couple images via ajax,
so the next image wont be uploaded because Laravel tries to compare the old csrf token (when i first opened new content form) with the new one regenerated in the session.
i dug into Laravel files and i figured out that the token is being regenerated  by this function in
vendor/laravel/framework/src/illuminate/session/store.php file :
public function start()
    {
        $this->loadSession();

        if (! $this->has('_token')) {
            $this->regenerateToken();
        }

        return $this->started = true;
    }

here is an example :
click the images for a full screen
test part 1

test part 2

what about laravel 5.2?

Sorry, this is a duplicate.

@GrahamCampbell Which is it a duplicate of? #8172? That issue was closed, but not actually resolved. The same issue started occurring for me a week ago, and perusing the issues, it appears there's longstanding subtle concurrent session bugs for at least a year now. If this has actually been solved somewhere, I'd love to know the solution. Even a Laracast or forum post directing how to avoid it would be useful.
My personal guess is it's some sort of concurrency bug involving saving large amounts of session data in the terminate() method, which is after data has been sent out to client, so incoming requests are already hitting the server again. In my case, at least, it has nothing to with driver (tried memcache, apc, and file).

Before i opened this issue i've googled it and i didn't find any convenient answer. just hacking ways.
so if this issue is duplicated and SOLVED, plz i'd love to know

I also experience this error.
I have a dynamic resize url for image, when the images are loaded my session is changed for every request and in the end not valid anymore.
Any solutions would be great.

to solve that in my situation i just added the post URI to $except attribute of VerifyCsrfToken Middleware  (App\Http\Middleware\VerifyCsrfToken) .
see https://laravel.com/docs/master/routing#csrf-protection

So nobody want to solve this important issue. Also happening on redis or database session driver.

maybe that has something to do with the browser, it works fine on Mozilla Firefox !

to solve that in my situation i just added the post URI to $except attribute of VerifyCsrfToken Middleware (App\Http\Middleware\VerifyCsrfToken) .
see https://laravel.com/docs/master/routing#csrf-protection

This is not supposed to be a good solution, as that will open up many possibilities. For any post to db, a csrf token ought to be verified. I feel the best solution would be to create a route to refresh the tokens and pass the refreshed token to the meta header dynamically upon each Ajax request.


Make public generated mapToInsertQuery, mapToUpdateQuery,...  methods

For instance in the : https://github.com/pushtorefresh/storio/blob/master/storio-sqlite-annotations-processor/src/main/java/com/pushtorefresh/storio/sqlite/annotations/processor/generate/PutResolverGenerator.java
Modifiers are put as public for mapToContentValues, but not for mapToInsertQuery, mapToUpdateQuery, ...
It would be useful for when using storIOSQLite.lowLevel().insert() or .insertWithOnConflict() to be able to retrieve them.
I can do the PR for that if you want.

Hi, @gbero! I think it make sense.
@artem-zinnatullin what do you think?

I'm ~ok with that, but I'd like to leave them final


Who’s to blame?  #reopening of the government

#@grimen

#@grimen

- @grimen

Do people trust their source of information on social media?


feat: Add support for env.NYC_CONFIG_OVERRIDE

This allows a process to spawn a child process with an environment
variable NYC_CONFIG_OVERRIDE.  This is a JSON string which overrides any
values in the NYC_CONFIG env.  It is not deleted in the child process,
so will be contagious unless unset, and it is the responsibility of the
caller to manage.
The first intended use case of this feature is to allow node-tap to map
tests to the specific portion of a system under test that they cover.
In this way, unit tests can be more focused, and only re-run when their
specific unit has changed.
There are, of course, many other uses that this could be put to, and it
did not seem appropriate to add a special hook just for overriding
the include list.

I'm concerned about our expanding environmental variable surface.  Not specifically against this but I'd like to hear what @JaKXz and @bcoe have to say / if they can think of any alternative methods of accomplishing the goal.

@coreyfarrell #1078 removes some environs that are no longer load-bearing, if you're concerned about that.

@isaacs I think removing unused environs in the next major release will be good but my concern is more about adding environmental variables which can be set from outside NYC to manipulate how it runs.


Feature/rtodling/gsibclim to gsibec

This renames the library (w/ the help of Matt) and adjusts all internal files and subdirectories to be consistent w/ the new name.

All looks new due to name change, but it's zero diff to previous develop.


detach the badges plugin from the catalog plugin

This puts a preliminary plan in place for how to be able to inject extra menu items into the entity three-dots kebab menu. We wanted to get this in place before the release so we don't make a release with the previous state which was to have a dependency between the catalog plugin and the badges plugin.
The interface is marked with UNSTABLE, and can change at any time without prior notice. We will perhaps need more time to get this into the final shape, but wanted to have some some fix in place before the release.

@benjdlambert  DONE

OH NO TEST FAILURES!

Hi, I'm using the new EntityLayout component. Is there plans to support adding the badges plugin there?

Hi, I'm using the new EntityLayout component. Is there plans to support adding the badges plugin there?

That's actually what this PR is doing to the builtin example app. https://github.com/backstage/backstage/pull/5107/files#diff-be7251ebe458c47f2b1f8691db3216b5b9e4fa66177033f8dcbaec2eef0939b8R183

Oh wait. You mean EntityLayout as opposed to EntityPageLayout. Hm, now I am unsure - should we have it there too?

My understanding was that I thought the EntityPageLayout will be eventually deprecated in favor of EntityLayout with the new composability system (which is why we updated to use EntityLayout) so I assumed new features should have forward compatibility?

Yep, you are quite right that this should be in both places! Added it now

@benjdlambert OK this turned out to actually be very involved. This will actually have to go into the release for more than one reason now :)
The issues we saw were ultimately because the new search backend stuff pulled in the wrong things, leading to type collisions. And this was ultimately because our templates weren't updated during recent dependabot bumps. Sooo there was a series of things to fix :)

I also want to note that I manually tested the badges functionality by grabbing the badge content from hitting the backend badge api directly and it looks like the formed entity url is not compatible with the new EntityLayout:
https://github.com/backstage/backstage/blob/995522142711c89005b895681f2f2e11755fa4b1/plugins/badges-backend/src/badges.ts#L25-L32
The entity url it returns is in the form: <kind>/<namespace>/<name>. However, the actual format of the entity url in my instance is: <namespace>/<kind>/<name> since I'm following the new composability setup.
Let me know if I should file a separate issue for this.

The issues we saw were ultimately because the new search backend stuff pulled in the wrong things, leading to type collisions. And this was ultimately because our templates weren't updated during recent dependabot bumps. Sooo there was a series of things to fix :)

Is this why my branch is complaining about duplicate typedefs in express-promise-router?

@kuangp can you open another issue for your comment

Is this why my branch is complaining about duplicate typedefs in express-promise-router?

Yeah. Should be fixed by this PR. The actual release will go out later today.


Add blocked_signals context manager

Description
Tiny little PR here that adds a blocked_signals context manager.  There are a few places (and probably will be more) where we temporarily block qt signals using:
obj.blockSignals(True)
# do something
obj.blockSignals(False)

this turns that into
with blocked_signals(obj):
    # do something

(also, it's in #675 and I'm trying to break that down into smaller bits)
Type of change

[x] New feature (non-breaking change which adds functionality)

How has this been tested?

[x] example: all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works

of course.  Funny, I actually had it renamed blocked_qt_signals and removed the qt.  So back it goes!  I like qt_signals_blocked


Fast Pixelize function

Is your feature request related to a problem? Please describe.
It would be great if it was possible to censor images without additional apps.
Describe the solution you'd like
Sections of the image either pixelate or black out - without people being able to recover it.
Thanks for your work and app.

I recently checked the most popular free photo editors (cf wiki), and obfuscation (by pixelating or blurring) is surprisingly rare. I only found it in the specialized app PrivacyBlur. Do you know of others?
As for your request, I'll close this issue because I consider this feature part of that issue:

#17

Feel free to upvote/subscribe to that issue.

Unfortunately, I don't know of any lightweight (not bloatware) FOSS apps.
LineageOS 20, however, has a built-in simple app called "Markup" that lets you paint over parts of the image - just as if you were painting over it with Windows Paint. I'll use that for now.

It's probably not to your liking (because it's bloated and not open-source) but B612 has an interesting brush variant that can pixellize the painted area.
As for Markup, i believe it's com.google.android.markup and was extracted from some Android Pie source. Minimalist but works well indeed.


[INFRA] Build schemacode package from SCHEMA_VERSION

Rather than using versioneer and tying the schema package to the repository version, this proposes using SCHEMA_VERSION. This will allow us to upload to pypi without relying on the specification release schedule, and the SCHEMA_VERSION semver rules should be more informative.
Thoughts?

Should drop Python 3.6, and I guess we'll need to require importlib_resources for Python <3.9.


Support IngressClass changes in IngressController  v1.0.0 (post 1.22 upgrades)

It looks like https://github.com/kubernetes/ingress-nginx/releases/tag/controller-v1.0.0 doesn't play nicely with Aergia after the k8s 1.22 upgrades in PR #7
https://kubernetes.github.io/ingress-nginx/#what-is-an-ingressclass-and-why-is-it-important-for-users-of-ingress-nginx-controller-now
I've pinned it to the last 0.x release for now, but we'll need to add support in to this.  It may just be a test anomoly.

Error message when loading ingress under 4.0.1
W0831 00:49:33.918625 8 controller.go:1007] Service "example-nginx/example-nginx-svc" does not have any active Endpoint.
I0831 00:49:33.942946 8 main.go:101] "successfully validated configuration, accepting" ingress="example-ingress/example-nginx"
I0831 00:49:33.945102 8 store.go:361] "Ignoring ingress because of error while validating ingress class" ingress="example-nginx/example-ingress" error="ingress does not contain a valid IngressClass"```

I'm currently working around it in testing with the following config in ingress-nginx-values.yaml
controller:
  watchIngressWithoutClass: true
  ingressClassResource:

Pretty sure this is no longer required


Add missing frontmatter to some docs (release-v0.18.x patch)

:information_source: This PR targets a release branch, not main.
Changes
Patch a previous release branch with the changes introduced in #3958, i.e:
Some documentation files had no frontmatter block, making them invisible
in the sidebar navigation on the website (since they were missing the
linkTitle property).
Update weights to adjust the order of links:

Installation to the top
Deprecations at the bottom
Migration guides just before Deprecations

See tektoncd/website#271 for more background.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[:no_good_man:] Docs included if any changes are user facing
[:no_good_man:] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

/kind documentation
/assign @afrittoli

looks good 👍

/ok-to-test


Add tests

For reference, how mention-bot is tested: https://github.com/facebook/mention-bot/blob/master/tests/


TIBCO

https://stackoverflow.com/questions/24936259/posting-a-message-in-jms-queue-using-java

http://tutorialspedia.com/tibco-ems-how-to-send-and-receive-jms-messages-in-queues/
https://stackoverflow.com/questions/24936259/posting-a-message-in-jms-queue-using-java

https://www.tibco.com/products/tibco-messaging/downloads

https://www.jbrandsma.com/news/2019/01/25/use-ems-via-the-java-api/


ceedling:test all - ERROR: Test executable failed.- rake aborted

test_lights.zip
Hi,
I am trying to do a simple unit test project for a PIC18 microcontroller using XC8 compiler. When I run ceedling test:all I get the following errors.
Running test_lights.out...

Shell executed command:
'build/test/out/test_lights.out 2>&1'
Produced output:
sh: 1: build/test/out/test_lights.out: not found
And exited with status: [127].

ERROR: Test executable "test_lights.out" failed.

Produced no final test result counts in $stdout:
sh: 1: build/test/out/test_lights.out: not found
And exited with status: [0] (count of failed tests).
This is often a symptom of a bad memory access in source or test code.

rake aborted!
/home/usha/Software/Test5/vendor/ceedling/lib/ceedling/generator_helper.rb:36:in test_results_error_handler' /home/usha/Software/Test5/vendor/ceedling/lib/ceedling/generator.rb:162:in generate_test_results'
/home/usha/Software/Test5/vendor/ceedling/lib/ceedling/rules_tests.rake:47:in block in <top (required)>' /home/usha/Software/Test5/vendor/ceedling/lib/ceedling/task_invoker.rb:74:in invoke_test_results'
/home/usha/Software/Test5/vendor/ceedling/lib/ceedling/test_invoker.rb:104:in block in setup_and_invoke' /home/usha/Software/Test5/vendor/ceedling/lib/ceedling/test_invoker.rb:63:in setup_and_invoke'
/home/usha/Software/Test5/vendor/ceedling/lib/ceedling/tasks_tests.rake:11:in block (2 levels) in <top (required)>' /var/lib/gems/2.3.0/gems/ceedling-0.28.2/bin/ceedling:287:in block in <top (required)>'
/var/lib/gems/2.3.0/gems/ceedling-0.28.2/bin/ceedling:274:in <top (required)>' /usr/local/bin/ceedling:23:in load'
/usr/local/bin/ceedling:23:in `'
Tasks: TOP => build/test/results/test_lights.pass
(See full trace by running task with --trace)

OVERALL TEST SUMMARY
No tests executed.
I am not sure what I am missing. Any help would be appreciated. Thanks,
R

I am having a similar issue in which my runner.o is not being created. Would like to find a solution for this.


Lyrics button not showing in latest version

🧐 Have you read the FAQ and viewed similar issues?

[ ] I have read the FAQ
[ ] I have checked older issues, open and closed
[X] I did a clean install of SpotX as described in the FAQ

Сountry of your account
Philippines
What is your Spotify Plan?
Free
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify: <IP_ADDRESS>1.gbf202593
- Windows: Windows 10 Home Single Language Version	22H2
- PowerShell: 5.1.19041.3031

📝 Description

Open Spotify
Have lyrics button (mic logo) on the bottom left

📸 Screenshots

What should be shown

read FAQ


Rewrite ChainerX activation tests

Part of #6401

jenkins, test this please.

Jenkins CI test (for commit a22a2d5ca6e4cf4ebe3fecbb24f4e8acc7a8bddc, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Can you check the Travis time-out failures?

The ReLU test gets stuck in an infinite loop trying to generate differentiable points. You need to make generate_inputs stochastic. How about simply using numpy.random.uniform?

jenkins, test this please.

Jenkins CI test (for commit 520737d19b4041302442da45bb7f9afd2d6fe513, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit 520737d19b4041302442da45bb7f9afd2d6fe513, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins and AppVeyor failures seem unrelated but Travis is raising a NumPy error for tests including shapes with dimensions of length 0, e.g. (5, 0) on the numerical gradient computation.

Fixed.

jenkins, test this please.

Jenkins CI test (for commit 5f4f18aaf18f8f114c139535b4fd9cc1ea1846f7, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Hm, CI is failing. Can you take a look?

jenkins, test this please.

Jenkins CI test (for commit 15498231fe2d931de1f5b5342d5a0a4170db68cd, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

CI is still failing.

I'd like to rerun the tests after https://github.com/chainer/chainer/pull/6599.

jenkins, test this please.

Jenkins CI test (for commit 668dc07013834995d90def0c55dc14736abd3802, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

CI failures seem unrelated. LGTM!


Update docker-ce 19.03.6

Alright, the errors we saw some weeks ago seem to be temporarily. I've triggered a build today and it's green now 👍


Make a slight modification to classic em

This PR makes a slight modification to classic em calculation.
@mn2007 san, could you review this PR?

I understand, @tt-ims san. I'll check.

@mn2007 san, Thank you.

Calculated results obtained on FX100 are same as previous. So I merge this PR. Thank you, @tt-ims san.

Thank you for merging, @mn2007 san!


CompositeAuth requires auth methods to be ActionFilters

What steps will reproduce the problem?
Prior to #19418  it was allowed to create an auth implementation that implemented AuthInterface.
What is the expected result?
Minor changes shouldn't break backwards compatibility.
Based on the docblock from CompositeAuth.php:
    /**
     * @var array the supported authentication methods. This property should take a list of supported
     * authentication methods, each represented by an authentication class or configuration.
     *
     * If this property is empty, no authentication will be performed.
     *
     * Note that an auth method class must implement the [[\yii\filters\auth\AuthInterface]] interface.
     */
    public $authMethods = [];

I'd expect that the only requirement is to implement the mentioned interface.
What do you get instead?
#19418 blindly calls a function isActive which is not in defined in AuthInterface, without checking whether the concrete authentication method implemention actually extends ActionFilter.
Proposed solution
Fix it so that it checks whether the auth method extends ActionFilter before blindly calling a function on it:
if ($auth instanceof \yii\base\ActionFilter) && isset($this->owner->action) && $auth->isActive($this->owner->action)) {
                $identity = $auth->authenticate($user, $request, $response);
                if ($identity !== null) {
                    return $identity;
                }
            }

This will fix the issue without breaking stuff. It'll even leave the "bugfix" intact.
Note that the original bugfix still leaves room for changed behavior; for example when an auth method is used inside CompositeAuth, no events are fired since the filter's beforeFilter and afterFilter methods are never called.
Additional info

Q
A

Yii version
2.0.46

PHP version
N/A

Operating system
N/A

if (
    $this->owner instanceof Controller
    && (
       !isset($this->owner->action)
       || (
           $auth instanceof \yii\base\ActionFilter
           && !$auth->isActive($this->owner->action)
       )
    )
) {
    continue;
}

That's not the code that's in master AFAIK? Or are you proposing an alternative fix?

@SamMousa it's combine master with your fix

could you maybe release this as a fix? <IP_ADDRESS>? we could not switch to 2.0.46 because of this and https://github.com/yiisoft/yii2/issues/19534


De-genericify lexer and clarify std.range imports

This should address points 1 and 3 from #153. Working on point 2.

ScopedBuffer is stack/heap-allocated and its data is destroyed out of the scope.
The 1. is about to assume that the input is a string.
For example, this should be:
const(char)[] readTimestamp(T)(ref T t)

 //@nogc is optional. See also MirException and DIP1008
const(char)[] readTimestamp(return ref const(char)[] text) @safe pure @nogc

This allows to do not allocated the result. Note, that ScopedBuffer is destroyed on exit. So, the valid expression is return buffer.data.idup. The idea is to use the buffer only for strings and only if the string contains escaped characters. In the example above, readTimestamp returns the slice of the input text and doesn't need to use a buffer.
Furthermore, it doesn't need to make a full timestamp or number validation because this can be done on the next stage.
This simplifies the code and makes it one-two orders of magnitude faster.

Re: full timestamp/number validation. That sounds good to me! I was more or less trying to keep functionality inline with upstream Ion implementations, but I'm open to refactoring the reading primitives and moving the validation to higher-levels.
Re: ScopedBuffer going out of scope, I wasn't aware of that! I guess it is in the name, but I wasn't entirely okay with the idea of idup being thrown all over the place. I'll add it back in.

I do have one question with making the code less generic: does that come with ditching the entire idea that the tokenizer will take in a (theoretically infinite) input range? Since slicing is not guaranteed for all input/forward ranges, I tried to avoid it and go the safer route of reading/parsing lazily, which is not too dissimilar to how it is done upstream.
So I'm a little wary on this - I'd love some more clarification.

Hm. That is very true, I hadn't considered that.
How about a compromise here? I have two ideas:

Maintaining the current API, but changing the functions to be @nogc and refactoring them to return exclusively slices from the internal input range
Changing the API, and refactoring the readers to be more generic. This would mean that these readers would exclusively operate on a const(char)[] buf (similarly to what you posted) and return the matching item. These readers would ideally be wrapped within the tokenizer (so you don't need to deal with maintaining state)

How about a compromise here? I have two ideas:

Could you please give few API examples? I am not sure I got it right.

I was thinking about just (as you suggested):
const(char)[] readTimestamp(return ref const(char)[] text) @safe pure @nogc

and then within the IonTokenizer class:
const(char)[] readTimestamp() @safe pure @nogc

I've been attacking this for the past few days and I've run into one huge snag: normalization of CRLF.. How do we normalize CRLF to just a NL without invoking GC? Replace all instances of CRLFs with NLs?

I've just tackled this issue with replacing any instances of CRLF or just plain CR with LF/NL (which should be spec). The tokenizer itself does no GC allocation (other then the ctor and corresponding helper functions -- this should be easy to strip out later), and I'm working on getting the readers to be the same.

It’s possible to make it nogc if it represent string by parts.

Can you clarify what you mean by "parts"? Do you mean like, creating a struct with the matched text? I think that would still require an allocation regardless (as we'd have to allocate the buffer to hold the matched text for further transformation)

BTW, scoped-buffer uses malloc if it is out of stack space.
However, with a @nogc design, we don't need it in the readers and tokenizer.
The readers may return a string by parts. It is fine because ion binary tape construction API allows it.
If the reader/tokenizer (it all the same thing in my head) find CRCF sequence or escaped sequence it encodes it to Tokenizer.encodedEscapedSequence buffer and returns a slice of it. We don't need the whole encoded string.  It is fine to return it part-by-part.
struct IonTextToken
{
     // slice of the IonTokenizer.input OR
     // a slice of the IonTokenizer.encodedEscapedSequence
      const(char)[] matchingText;
      // true if it is a not fiunal string part.
      bool notFinal;
}

// Not a generic struct!
struct IonTokenizer
{
    const(char)[] input;

    // Encoded escaped sequence in UTF-8
    char[6] encodedEscapedSequence;

    bool finished;

    size_t position;

    IonTokenType currentToken;

    this(const(char)[] input) @safe pure nothrow @nogc
    {
        this.input = input;
    }
...

@hatf0 any help I can do for MR?

Sorry - life has been a little hectic recently. I'll take a look at this and try and get it all figured out (by this weekend, hopefully).

Sorry about the delay. I'll be taking a look at this PR and have it return bit-by-bit.

Hey! I have an example working for this. Let me know if this is okay:
  void testMultiPart(string ts, string expected, string after, char last) {
      auto t = tokenizeString(ts);
      assert(t.nextToken());
      assert(t.currentToken == IonTokenType.TokenString);

      auto str = t.readString();
      assert(str.matchedText == expected);
      assert(!str.isEscapeSequence);
      assert(!str.isFinal);

      auto str2 = t.readString();
      assert(str2.matchedText == after);
      assert(str2.isEscapeSequence);
      assert(!str2.isFinal);

      auto str3 = t.readString();
      assert(str3.matchedText == "");
      assert(!str3.isEscapeSequence);
      assert(str3.isFinal);
      assert(t.readInput() == last);
  }

testMultiPart(`"Hello! \U0001F44D"`, "Hello!", "👍", 0);

In this example, you are expected to consume until readString().isFinal == true

Great! One nitpick: str2 is already final:
  void testMultiPart(string ts, string expected, string after, char last) {
      auto t = tokenizeString(ts);
      assert(t.nextToken());
      assert(t.currentToken == IonTokenType.TokenString);

      auto str = t.readString();
      assert(str.matchedText == expected);
      assert(!str.isEscapeSequence);
      assert(!str.isFinal);

      auto str2 = t.readString();
      assert(str2.matchedText == after);
      assert(str2.isEscapeSequence);
      assert(str2.isFinal);
  }

Great! I've fixed that. One thing that this does break is the implicit concatenation of long-strings, though.  ''''''\U0001F44D'''`, "Hello! ", "👍", 0);
  test(`'''0xFOOBAR''',`, "0xFOOBAR", ',');
  test(`'''Hello, 'world'!'''`, "Hello, \'world\'!", 0);
  testMultiPart(`'''Hello,'''''' world!'''`, "Hello,", " world!", 0); // note the use of testMultiPart here
  testMultiPart(`'''Hello,'''     ''' world!'''`, "Hello,", " world!", 0); // note the use of testMultiPart here

Awesome.
The tokenizer can work with a const(char)[] directly, instead of ubyte array. Also, we don't need to duplicate the data. const(char)[] should be used everywhere here for input or its slice or slice of a tokenizer buffer.

You can ignore CI failures on old compilers

Sounds good. I have the vast majority of the tokenizer converted to @nogc @safe pure. The only time that a GC allocation is required is with the replacement of CRLF to LF (and CR to LF), and I doubt this is avoidable. Everything else has been converted over (and returns by parts!)

Sounds good. I have the vast majority of the tokenizer converted to @nogc @safe pure. The only time that a GC allocation is required is with the replacement of CRLF to LF (and CR to LF). This is avoidable - the user should know if their data is normalized to Unix-endings, but for compatibility sake it remains. Everything else has been converted over (and returns by parts!)

Ion specifies newlines normalization only for long strings. For other strings, newlines are either aren't allowed by spec or we can do not care about them. Could you please add built-in @nogc long string line CRLF to LF (and CR to LF) chunking the same way as for escape sequence?

Sounds good. I have the vast majority of the tokenizer converted to @nogc @safe pure. The only time that a GC allocation is required is with the replacement of CRLF to LF (and CR to LF). This is avoidable - the user should know if their data is normalized to Unix-endings, but for compatibility sake it remains. Everything else has been converted over (and returns by parts!)

Ion specifies newlines normalization only for long strings. For other strings, newlines are either aren't allowed by spec or we can do not care about them. Could you please add built-in @nogc long string line CRLF to LF (and CR to LF) chunking the same way as for escape sequence?

Hm. According to the ANTLR spec, CRLF is treated as a regular new-line, so I've removed all cases where we normally fail. This makes the core function @nogc, with the only exception being the UTF-16/UTF-32 overloads.
This previous commit does normalization for long strings (and explicitly long strings). CRLF/LF is consumed, and a new-line is placed in a result. The semantics are virtually identical to how escape sequences are handled - it will return isFinal == true if a CRLF/LF precedes the end of the string. See:
https://github.com/libmir/mir-ion/blob/60180c6bf1fa6536821feecaa411008f505bbbb1/source/mir/ion/deser/text/readers.d#L684-L688

Thanks, merged!


Different format for times on user statistics

Hi,
on the user stats page it displays total times in dd:hh:mm format. So if I cycled for 30 hours it will show 1:06:00, which is basically quite confusing as it could also mean 1 hour and 6 minutes.
Maybe you should "humanize" these values with a postfix, like 4d 10h 3m which is obvious and easy to understand.
Thx!

Hey Igor,
That's a good point!
I'll take a look at how we could change it to something more readable.
Thanks for using the app :)
Greets
Glenn

Hi Glenn,
just got the latest update and it looks much better now. Good work!
But depending on the combination of totals, the string might get too long and it will cut the last bit out. Check the image attached (the "all time / moving time"). A slightly smaller font will do the trick :-)

Ha good catch!


under js.experimental-foreign-object-prototype, pushing into Java array violates ECMA spec (instead of failing)

Summary: When using the js.experimental-foreign-object-prototype flag and passing a Java array (not List) to JS, the JS object violates the specification for Array.prototype.push. The JS method invocation succeeds and returns the incremented array's length, but the array is not in fact incremented.
Expectation: Given the semantics of Java arrays, push operations on a proxied (?) Java array cannot be expected to modify the array. Instead, the JS method invocation should fail. Producing JS behaviour inconsistent with basic specs is much worse since the Java host object type (array vs. List) is not visible from within JS and resulting problems would be very difficult to debug.
Many thanks for your work!!!
Version: 21.1.0 (Java 11)
To Reproduce:
Context.Builder builder = Context.newBuilder("js")
	.allowHostAccess(HostAccess.ALL)
	.allowExperimentalOptions(true)
	.option("js.experimental-foreign-object-prototype", "true");

Context context = builder.build();
Value b = context.getBindings("js");

b.putMember("array", new Object[] { "a" });

Value o = context.eval("js", "[array.push('b'), array.length, array]");
System.out.println("length returned by push (in js): " + o.getArrayElement(0L));
System.out.println("array.length (in js): " + o.getArrayElement(1L));
System.out.println("getArraySize(): " + o.getArrayElement(2L).getArraySize());
System.out.println("the array: " + o.getArrayElement(2L));

Output:
length returned by push (in js): 2
array.length (in js): 1
getArraySize(): 1
the array: [a]

Hi @MatthiasHild
interesting find, thanks for bringing this up. I can confirm this is happening.
The root cause of this is not around js.experimental-foreign-object-prototype but more fundamental: even code like array[2] = 'test'; fails in similar fashion, by NOT failing ;-) Internally, an ArrayIndexOutOfBoundsException is occurring of course, everything is fine according to the semantics. But we happen to drop that exception on the interop boundary, so the JavaScript world does not learn that an error occurred.
On the JavaScript side, we thus have no way of knowing something went wrong, and do what the spec says: return the expected new length of 2 (spec does not say to actually check the new length, but just return the counter variable, that's what we do: https://tc39.es/ecma262/#sec-array.prototype.push
So on the JavaScript side we are correct; but of course, we need to catch the error and provide that to the user. There might be a good reason why we don't do that at the moment, I am checking with our engingeers; but on first sight, I agree with you that I would expect an error.
-- Christian

Christian,
This answers my question "How active is GraalJS?". The obvious answer is "Very!"
Many thanks for looking into this.
M.

Fixed in https://github.com/oracle/graaljs/commit/0a0e04d8d13a1a309e073a8020127a03e13154ca (21.2.0).


fake(engine): add cached checkpoint in job master

What problem does this PR solve?

Issue Number: close #6598
What is changed and how it works?
Cache checkpoint in job master, if part of workers are offline, keep the old value of them.
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/run-check-issue-triage-complete

/run-check-issue-triage-complete

passed chaos test in: https://github.com/pingcap/tiflow/actions/runs/2789584041

@amyangfei  Could you explain more about this unstable test? I think I don't real get the point.

@amyangfei Could you explain more about this unstable test? I think I don't real get the point.

The issue was explained in the original issue.

Codecov Report

Merging #6599 (df023db) into master (6290709) will decrease coverage by 0.1064%.
The diff coverage is 58.8899%.

:exclamation: Current head df023db differs from pull request most recent head 33b3b89. Consider uploading reports for the commit 33b3b89 to get more accurate results

Flag
Coverage Δ

cdc
65.8001% <60.6289%> (-0.3129%)
:arrow_down:

dm
52.0799% <54.4117%> (+0.0436%)
:arrow_up:

engine
63.5019% <53.0303%> (-0.2398%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6599        +/-   ##
================================================
- Coverage   59.4214%   59.3149%   -0.1065%
================================================
  Files           763        777        +14
  Lines         87418      88197       +779
================================================
+ Hits          51945      52314       +369
- Misses        30894      31241       +347
- Partials       4579       4642        +63

https://github.com/pingcap/tiflow/runs/7666668553?check_suite_focus=true
Another case for this issue.
Seems this case becomes unstable after pr https://github.com/pingcap/tiflow/pull/6436

/merge

After offline discussion with amyangfei and CharlesCheung96,
we need to investigate whether the new merged pr causes the behavior that job is dispatched to an offline executor.


FINCN-165: Implement bash scripts to provisioner fineract cn services

Great work @Anh3h . I will test it out tomorrow the latest.

Thanks for the review @aasaru. I have made the suggested updates. Please, can you review it again?


Where to download the pretrained model?

Is there a way to get checkpoint_15500 in inference file?

Checkpoint files are save to the output_directory that is specified when running the train.py command. The example code on our repo saves to outdir

@rafaelvalle
When I run train.py with default hparams on 8GB 1080 gpu, I get this error. I change batchsize to 24 but the error still exists!
src/tcmalloc.cc:278] Attempt to free invalid pointer 0x100000009
Traceback (most recent call last):
  File "train.py", line 291, in <module>
    args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
  File "train.py", line 209, in train
    for i, batch in enumerate(train_loader):
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 330, in __next__
    idx, batch = self._get_batch()
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 309, in _get_batch
    return self.data_queue.get()
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/multiprocessing/queues.py", line 335, in get
    res = self._reader.recv_bytes()
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/multiprocessing/connection.py", line 216, in recv_bytes
    buf = self._recv_bytes(maxlength)
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/multiprocessing/connection.py", line 407, in _recv_bytes
    buf = self._recv(4)
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/multiprocessing/connection.py", line 379, in _recv
    chunk = read(handle, remaining)
  File "/data00/home/caoyuetian/share/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 227, in handler
    _error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 125263) is killed by signal: Aborted.
Segmentation fault (core dumped)

This is probably related to your CPU not being able to load the data.
Try changing the number of workers or smaller batch size.

@rafaelvalle
Can you share your pretrained Tacotron2 model and hparams that generated the sample audio?

@OptimusPrimeCao The model uses approximately 300MB per sample. Try reducing your batch size to 16.

@MXGray We can share the hparams.
Please do not post the same message on multiple issues. I deleted your message from the Audio Examples issue.

Please share the hparams.

Is there a way I can continue training my model from a particular point?
To be specific, my training crashed at checkpoint_32000 because of memory issue which I have fixed. Can I now somehow resume training from this point or should I again begin from the start? If yes, how do I do it?
I wasn't sure if opening a new issue for this was required hence posting my comment here.
Any help is appreciated. Thanks!

Never mind, figured it out.
python train.py --output_directory=outdir --log_directory=logdir --checkpoint_path='outdir/checkpoint_32500'

Pre-trained model has been made available on our README page.

Never mind, figured it out.
python train.py --output_directory=outdir --log_directory=logdir --checkpoint_path='outdir/checkpoint_32500'

@vijaysumaravi  Mine was also stopped after 32.5 epochs, did you figure out the reason?

Reducing my batch size helped. I was training it on a single GPU.

Tutorial: Training on GPU with Colab, Inference with CPU on Server here.


what is LSNCD in the code? is it wnorm?

Hi, When I see your code, I can't find the LSNCD and How did you make style space in your code?

loss: https://github.com/caopulan/GANInverter/blob/433356122fcdd15de0a855192661a65a97310952/training/encoder_trainer.py#L332
calculate $\mu$: https://github.com/caopulan/GANInverter/blob/433356122fcdd15de0a855192661a65a97310952/training/encoder_trainer.py#L80

Some codes are not organized well now, which may be confusing to read. But scripts (training/inference) and configs of each method are available to reproduce the experiments. Have fun with GANInverter!
We are going to clean and re-organize the codes (v2.0) and release the weight after our paper is accepted.

We are working on the implementation of SAM/HFGI/Hyperstyle now, especially their inference phase. SAM may be implemented in GANInverter in February or March.
Yes, they only release the optimization codes. I report an issue to ask whether they will release the encode but have not received any replies.
However, if they don't release their official encoder codes, we may not implement them in GANInverter, because we are working on a new inversion method, "Domain-Specific Hybrid Refinement", which gains significant improvement to all previous SOTAs. But it is only an optimization-based method. We may study its encoder version later.

thank you so much. I have another question, I want to know how to get style space from w space. and do you try to use w+space? do you know how to get w+space from w?


Reference v3 Location docs in Glossary.md

Link was to master but master does not have a docs folder

Thanks, @emp823 😄


Dependency graph resolution: Equivalent excludes can cause un-necessary graph mutations [backport 7.x]

Backport of #23521, see details there

Done through #25060


dotnet-sdk_6.0 6.0.407 -> 6.0.408

Description of changes

https://github.com/dotnet/core/blob/main/release-notes/6.0/6.0.16/6.0.16.md
https://github.com/dotnet/core/blob/main/release-notes/6.0/6.0.16/6.0.311.md

Fixes CVE-2023-28260
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 225835 run on x86_64-linux 1

3 packages failed to build:

BeatSaberModManager
audiobookshelf
tone

39 packages built:

alttpr-opentracker
azure-functions-core-tools
boogie
btcpayserver
btcpayserver-altcoins
commitizen
commitizen.dist
depotdownloader
discordchatexporter-cli
dotnet-aspnetcore
dotnet-runtime
dotnet-sdk
eventstore
formula
galaxy-buds-client
github-runner
inklecate
jackett
jellyfin
jetbrains.rider
lidarr
marksman
msbuild
nbxplorer
netcoredbg
nuget-to-nix
omnisharp-roslyn
opentabletdriver
osu-lazer
pre-commit
pre-commit.dist
prowlarr
ps3-disc-dumper
radarr
readarr
roslyn
scarab
wasabiwallet
xivlauncher

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

The others are already broken for other reasons.


site_cfg is not defined

When I tried to use CPATH environment variable to pass include dierctory, I got this:
  File "generate.py", line 99, in create_module
    site_cfg['include_dirs'] += [os.path.join(p, 'nanomsg')
NameError: global name 'site_cfg' is not defined

The reason is this line. It should probably be set_source_args instead of site_cfg

Ah, nice find. Would you be able to contribute a pull request for this?

Fixed with #39.


expert review: endocrine system disorder'

spreadsheet. Address https://github.com/monarch-initiative/mondo/issues/6651
ROBOT Reviewer-Curator Agree - Stay
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vTv9CrYrbsEG2hZ3PfXCiN5Wt1icJvlKe1fyBbCM1BOVs941fikGu3P4PCKSZIVLg/pub?gid=2066654847&single=true&output=tsv" merge_template -B
ROBOT Reviewer-Curator Agree - Leave
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vTv9CrYrbsEG2hZ3PfXCiN5Wt1icJvlKe1fyBbCM1BOVs941fikGu3P4PCKSZIVLg/pub?gid=294969719&single=true&output=tsv" merge_template -B
ROBOT Reviewer-Stay Curator-Leave
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vTv9CrYrbsEG2hZ3PfXCiN5Wt1icJvlKe1fyBbCM1BOVs941fikGu3P4PCKSZIVLg/pub?gid=672834167&single=true&output=tsv" merge_template -B
ran normalization command

I need to manually add a comment to a term


added secondary vm backup

Related Issue

#566

Work Done

Adapted the rsync section of the file transfer guide for the nextcloud deployment, as asked by a farmer/user from the community
This ensures that users can deploy a nextcloud instance, have a daily borgbackup on the same VM, and then have a daily backup to another VM. This ensures great redundancy.

worked fine but ran the script manually


Add a file existence check

I hope this check will be useful for stuff like "this box needs rebooting" tracking and other fun file-based notifications.

A concise and simple solution to the problem. :shipit: and well done.


Mobileraker.conf cannot appear

hi i just finish the companion but after i restart klipper or  restart the servise thae Mobileraker.conf is not there
is there any way to add it mantually

Hey,
AS i stated in the readme.
A config is only required if you want to configure multiple printers.
If that is the case you can create a Mobileraker.conf file in:
~/Mobileraker.conf
<mobileraker_companion DIR>/Mobileraker.conf
~/klipper_config/Mobileraker.conf

Sorry to ask stupid questions, but where exactly do I need to create the config?

Hey, AS i stated in the readme. A config is only required if you want to configure multiple printers. If that is the case you can create a Mobileraker.conf file in:

~/Mobileraker.conf
<mobileraker_companion DIR>/Mobileraker.conf
~/klipper_config/Mobileraker.conf

Sorry to ask stupid questions, but where exactly do I need to create the config?
I have 2 Printers on my Pi

Just add it to the installation directory of mobileraker_companion

Okay, I did this, but I still don't get any notifications.


How to use with Remix

Any idea how to incorporate this into Remix? I don't think it exposes the underlying bundler in the same way Next.js does.
https://remix.run/docs/en/v1/api/conventions#remixconfigjs
Adding unplugin-icons to serverDependenciesToBundle leaves the build hanging and provides no opportunity for config.
Any ideas?

Would love to know how to use this too especially with Remix moving over to Vite!

would love to see it as well