Initial import of vs-kubernetes and vscode-helm extensions

This PR brings the two existing extensions by @brendandburns and @technosophos into one repo.  At this stage I haven't done much to improve consistency or rationalise away duplicates or different approaches to doing things - just getting the thing all building in one package.
@technosophos We had two implementations of Draft support.  I stuck with my implementation because I was a bit happier with the structure, but I'd welcome a review, and I'm happy to make changes or to adopt your implementation instead.  Let me know!

cough so I just issued a PR into my own fork cough
That was an auspicious start.


ci: upload native libraries to huggingface when release new version.

I still don't understand why the binaries need to be uploaded from huggingface for autodownload to work - why can't they be pulled from this repo? That way no separate step is needed, we just need to know the git tag/commit ID to download e.g. https://github.com/SciSharp/LLamaSharp/blob/c325ac91279428e8d88719357b93e43cd9c5ed52/LLama/runtimes/deps/avx512/llama.dll

Using github is absolutely a good option and I'd like to accept such a PR if anyone want to add it as a second resource.
Here're the reasons why I chose huggingface.

AFAIK there's no mirror site for github with official mechanism. IIRC you're living in UK so visiting github is never a problem for you. However, people in some countries such as China, India and Iran are restricted to visiting github (and huggingface.co). Though some of them could use VPN on their PC, it might be impossible for them to use VPN in the production environment. If a developer in these countries is distributing an APP, it's even worse because most of the users with no CS background don't have a VPN. As a website built from an open-source project, huggingface has many mirror sites, such as hf-mirror.com. This means a lot for people in these countries. All they need to do is to set a mirror site url when using this feature.
File cache is significant in this feature. It's OK to do so with github as what I did in HuggingfaceHub. But the model downloading is also in my plan so using huggingface is obviously a big time saver for me. :)
Privacy: Either using huggingface or github, it's absolutely possible for users to switch to their own native libraries with a single string modification. However, I believe some people would not like to make their own native libraries public. While huggingface support downloading files with token, I didn't figure a way to download files from a private github repo with http request.

In conclusion, because of (1) and (3), I don't think we should drop the huggingface and take github instead. I'd like to discuss with you about which one to be used as default if both of them are supported. However, right now I don't have extra time to add support for it (that's why (2) is a reason!).

restrictions/mirroring are a good point, I'd never have thought of that!


Increase abstractassert test coverage

Check List:

Fixes: NA
Unit tests : YES
Javadoc with a code example (on API only) : NA
PR meets the contributing guidelines: NA

Following the contributing guidelines will make it easier for us to review and accept your PR.

Thanks, @joel-costigliola, for the feedback, I'll fix the style issues asap. I was also planning to add tests for the more complex classes that I've left out for now, should I add it to this PR or make a new on to make reviewing a bit easier?

thanks for the quick reply, I would prefer another PR so that we can finish this one and it's easier to review shorter PRs.

I've fixed the commented things, hopefully this is what you meant!

Integrated thanks @sarajuhosova!
I noticed when integrating some unused imports or * imports and I tool the liberty to reformat other tests not initially in the PR and squashed all the commits (keeping you as the commit author obviously)


KAFKA-13858; Kraft should not shutdown metadata listener until controller shutdown is finished

When the kraft broker begins controlled shutdown, it immediately disables the metadata listener. This means that metadata changes as part of the controlled shutdown do not get sent to the respective components. For partitions that the broker is follower of, that is what we want. It prevents the follower from being able to rejoin the ISR while still shutting down. But for partitions that the broker is leading, it means the leader will remain active until controlled shutdown finishes and the socket server is stopped. That delay can be as much as 5 seconds and probably even worse.
This PR revises the controlled shutdown procedure as follow:

The broker signals to the replica manager that it is about to start the controlled shutdown.
The broker requests a controlled shutdown to the controller.
The controller moves leaders off from the broker, removes the broker from any ISR that it is a member of, and writes those changes to the metadata log.
When the broker receives a partition metadata change, it looks if it is in the ISR. If it is, it updates the partition as usual. If it is not, it stops the fetcher/replica. This basically stops all the partitions for which the broker was part of their ISR.

When the broker is a replica of a partition but it is not in the ISR, the controller does not do anything. The leader epoch is not bumped. In this particular case, the follower will continue to run until the replica manager shuts down. In this time, the replica could become in-sync and the leader could try to bring it back to the ISR. We rely on https://github.com/apache/kafka/pull/12181 to ensure that does not happen.
Committer Checklist (excluded from commit message)

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

Good post. Someone stole $70,000 from me once (crypto). It took me
1 month with the help of Wizesafetyrecovery @ gmail com and a bunch of stress to finally get it back but it was a learning experience. Ever since then I have had to tighten up my online presence. If anyone reading this has a similar issue, I recommend using them to recover your lost funds.


A new sortable table component

Loosely based on the sortable table in Trade elements but with many improvements and made simpler. Adopts the new component framework.
The example shown isn't in the project (we all know why) but it demonstrates the control.
If this works it could be used to replace existing tables.

The js data hook could just be js-SortableTable. Doesn't need to include BEM style class names.
Sent from my iPhone

On 2 Jun 2017, at 13:24, Zac Tolley<EMAIL_ADDRESS>wrote:
@ztolley commented on this pull request.
In src/sass/components/_sortable-table.scss:

@@ -0,0 +1,17 @@
+.data-table--sortable {
Happy to hear views on naming. Think of it as a regular table (which inherits the table styles from GDS) that is made sortable

data-table{% if sortable %} js-data-table--sortable data-table--sortable {% endif %}
_data-table.scss
Would this be better?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.


yAxis scale issue

Version
4.2.1
Reproduction link
https://jsfiddle.net/gob7t3w5/
Steps to reproduce
Please review the commented code in the minimal reproduction link.
Basically add two bar series and a y axis with scale = true.
The first series should have an element that is larger that the second series element.
What is expected?
I am expecting both series to appear on the chart regardless of setting the scale for the yAxis to true and regardless of the order of the series.
What is actually happening?
Somehow the second series is hidden when scale of the yAxis is set to true and the series with the large element is added first to the chart.

Hi there, this looks like a bug. Can you add the bug label again.

Hello there @100pah ,
is there an update on this bug ?
Best regards,
Sergios.

Hi there @100pah @Ovilia ,
I just checked with version 4.7.0 and this issue has been resolved.
You can see this jsfiddle that verifies the fix.
Thanks,
Sergios.


memset large memory appear stack message

log message：
[UNISOC] [WRN] main: Unisoc fdl 0x11
do_download:enter
1 is 2 is 104720two is 104720three is 0four is 104720two is 108734three is 1four is 108734two is 10c748***** USAGE FAULT *****
Executing thread ID (thread): 0x00110c88
Faulting instruction address:  0x0
Illegal use of the EPSR
Fatal fault in essential thread! Spinning...
code ：
for (i = 0; i < PACKET_MAX_NUM; i++) {
printf("two is %x", &packet[i]);
memset (&packet[i], 0, sizeof (dl_packet_t));
printf("three is %x", i);
packet[i].next   = &packet[i+1];
printf("four is %x", &packet[i]);
}
dl_packet_t size 16K

Enable CONFIG_INIT_STACKS=y
and then use the STACK_ANALYZE() macro to get the thread stack usage.


external form block controller action not firing

When I put an external form block on the page and choose the test_form which is included in the core, everything works fine. When I now copy the controller file and the view file to
/application/blocks/external_form/form/my_test_form.php
/application/blocks/external_form/form/controller/my_test_form.php
and change the namespace of the controller to
namespace Application\Block\ExternalForm\Form\Controller;
and the class name of the controller to "MyTestForm", the submission of the form leads to a page not found and the controller action test_search is not firing. The same happens, when I try to put the external form in my own package.

Are you actually using the Github version ? I believe this has been fixed in github for awhile. If this is specific to <IP_ADDRESS> it should be posted in the bug report forum on concrete5.org (although as I mentioned I believe it should be fixed.)

Well, first I tried with <IP_ADDRESS> and then with a fresh install of the develop branch from github. In both cases I get a 404 error when submitting the form and the controller action is not firing.

The structure of the files may have changed since <IP_ADDRESS> so you'll want to make sure you're following the structure of the files in Github.  Here's how mine is set up.
Here's my file system in application/blocks/external_form/

My form submits to $view->action('submit');
My controller class and namespace is this:
`
namespace Application\Block\ExternalForm\Form\Controller;
use Concrete\Core\Controller\AbstractController;
class SweetForm extends AbstractController
{
`
And my action_submit($bID = false) method is this:
public function action_submit($bID = false)     {         if ($this->bID == $bID) {             $this->set('response', t('Thanks!'));             return true;         }     }
Note the $bID parameter. This is new 5.7.4 and let's multiple blocks be on the same page.

I just cannot reproduce it. I am using the following version:

My file structure is exactly like you wrote:

My controller is:
<?php
namespace Application\Block\ExternalForm\Form\Controller;
use Concrete\Core\Controller\AbstractController;
class SweetForm extends AbstractController
{
    public function action_submit($bID = false)
    {
        if ($this->bID == $bID) {
            $this->set('response', t('Thanks!'));
            return true;
        }
    }
}

And my view is:
<? defined('C5_EXECUTE') or die("Access Denied.");
$form = Loader::helper('form');
if (isset($response)) { ?>

<?=$response?>

<? } ?>
<form method="post" action="<?=$view->action('submit')?>">

<?=$message?>

        <?=t('Test')?>
        <?=$form->text('test_text_field')?>

        <input type="submit" name="submit" value="submit" class="btn btn-default" />

</form>

Here is the url of the test install with just an external_form block:
c5-latest.whitelabel-muenchen.de
It is not too dramatic, if I cannot get this resolved. In the meantime, I have built my custom block to handle our ajax forms with it's own block controller, which is working fine. However, I am sure others will have the same problems, if it is not my stupidity that causes the problem but a bug.

Getting exactly the same issue

i have exactly the same issue

I am trying to set up my first <IP_ADDRESS>  site, and compared to my past experience with earlier version 5.7 has been an uphill battle. I am having the same issue of getting a 404 error on submit.  Also how do you add the recipient email address on the external form. Any help will be greatly appreciated

you can create a new block rather than overwrite external block and put on it your form. it will work. i try to do that and it worked with me

Hi elgreatly do you have a template that I could look at?

Anyone find the problem? I'm getting this issue with <IP_ADDRESS>. The external form submits fine on my local dev machine, but I get a 404 on the live site.

Ok, ignore my post. I got the directory structure borked on the live site - hence the 404.


Feature/location geopoint

At the moment, a location object only has the individual parts of the address, and no type of verification on the backend. This branch will add some verification through geocoding, in that if mapbox or google maps is unable to find a location for the address, then the object will not be created. In addition, the inclusion of latitude and longitude adds additional specificity and clarity.

@jamesfneyer this PR now has conflicts from the #6 merge, please rebase this branch from master


[neteasemusic] Fixes #8239

<IP_ADDRESS> is a reverse proxy for the netease cdn [via https://github.com/Unblocker/Unblock-Youku/issues/488#issuecomment-162314538]

If I understand it correct, users in Mainland China can download netease music without problem. We shouldn't enforce proxies on all users.

@yan12125 If that's the case, will you accept a PR that is something like this:
song_url = 'http://m5.music.126.net/%s/%s.%s' % (
    self._encrypt(details['dfsId']), details['dfsId'], details['extension'])
if not self._is_valid_url(song_url, info['id']):
    song_url = 'http://<IP_ADDRESS>/m1.music.126.net/%s/%s.%s' % (
        self._encrypt(details['dfsId']), details['dfsId'], details['extension'])

Sorry but it's not a good idea to use hardcoded proxies in the codebase. You can add a new option and use this option in neteasemusic.py.

@yan12125 Even if the reverse proxy apparently belongs to netease? (as claimed by the poster in the in the linked comment).
I'm curious what you mean by "add a new option", do you mean an extractor-specific option? Do you have example of an extractor doing this?

Well I thought 黄易 is different from 网易. As a record, <IP_ADDRESS> belongs to the same company as netease.com. [1][2]
This fix is OK. Just eliminate duplicated codes and add some comments.
[1] http://whois.domaintools.com/<IP_ADDRESS>
[2] http://whois.domaintools.com/netease.com

Thanks merged.

I guess this proxy-feature is broken again.
Test:
http://<IP_ADDRESS>/m1.music.126.net/9Oq39CoHx4H4W2i438mBjQ==/2888417047551516.mp3
=> Access Denied


Autostart doesn't work for me

Host Wily AMD64, LXD stable repo from Ubuntu containers team enabled and updatet. Host running as a KVM guest that I rent.
2 Guests: Debian 8 + Ubuntu Wily
For both:
boot.autostart: true (I also tried boot.autostart: 1 at first, but then changed it to "true")
I tried without boot.autostart.priority and boot.autostart.delay and I used different integers between 1 und 90 and a couple different combinations.
No matter what I do, the guests do not start automatically. Reboot or cold start, guests running or stopped before the reboot. Once I start using lxd commands on the host, they start up. lxc list will show the guests as stopped, but will show them as running a short while later. lxc list seems to start the guests. lxc exec will also start them.

This has been fixed by https://github.com/lxc/lxd/commit/baa728088e2323270470eb36b178f151e32d275f


chore: Replace glob with tinyglobby in @jest/reporters, jest-config, and jest-runtime

Summary
I made this change as a part of the e18e initiative. tinyglobby is a minimalist alternative to various other packages handling glob patterns. .replaceAll('\\', '/') is exactly what's done by glob when the windowsPathsNoEscape: true option is passed: https://github.com/isaacs/node-glob/blob/feaf0a85c1d1bcc7c992f45edf51aaab8ed729ac/src/glob.ts#L470-L472.
Initially I also wanted to replace slash usages in these three packages, but I decided it'd be better to leave that for another PR as to separate concerns.
Test plan
Ensure the test suites are passing for the changed files.

Closing as I mistakenly assumed tinyglobby could work as a drop-in replacement for glob (which turned out not to be the case), and I won't have enough time to ensure its proper integration into the codebase.


devourment, atmans

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,d]

/trunk merge


fix: host less JDBC URI ignores the database name

This PR is to fix the JDBC host less URI ignores the database name, related issues: #2544 #4121

Hi @eddumelendez, could you help to have a look on this change, thanks in advance.


Unable to run tests with mock on iOS

I will shortly share a sample project on this, however when running tests for iOS in a multiplatform project, tests are passing for android but failing for iOS with the following error:
Uncaught Kotlin exception: org.kodein.mock.Mocker.MockingException: <My mocked class>.itsFunction() has not been mocked
Invalid connection: com.apple.coresymbolicationd

Anyone has seen that before or know how to resolve this?

How are you applying MocKMP ?
Can you share a reproducer ?

@SalomonBrys  Sorry for the delay, I was trying to figure out what's going on. So it seems like an issue with maybe scope in tests. Here is the setup:
interface Foo {
    fun doSomething(): Boolean
}

class Bar(private val foo: Foo) {
    suspend fun bar(): Boolean {
        return foo.doSomething()
    }
}

@UsesMocks(Foo::class)
class MyTest : TestsWithMocks() {
    override fun setUpMocks() {

    }

    private val myFoo: Foo = MockFoo(mocker)
    private val sut: Bar = Bar(myFoo)

    @BeforeTest
    fun setup() {
        every { myFoo.doSomething() } returns true
    }

    @Test
    fun `this is a test`() = runTest {
        val actual = sut.bar()
        assertTrue { actual }
    }
}

The above test passes for android but fails for iOS. I'm assuming it's something to do with runTest{}. But explicitly calling setup() inside the test passes:
@Test
    fun `this is a test`() = runTest {
        val actual = sut.bar()
        assertTrue { actual }
    }

Update: Moving everything I have above in setup() to setupMocks() from TestsWithMocks solved the issue.

This needs to be documented

The problem came from:
    override fun setUpMocks() {

    }

You're supposed to override setUpMock with the injectMocks method that was generated:
override fun setUpMocks() = injectMocks(mocker)


How mutable should PKEYs be after initialisation?

The possibility was raised first here: https://github.com/openssl/openssl/pull/14062#issuecomment-773112106 :

I see another potential race condition:
thread 1 has an ongoing operation (digestverify, say)
thread 2 makes a change to the key (calls EVP_PKEY_set1_encoded_public_key(), say) and starts another operation (digestsign)

Thread 2 would invalidate the operation cache, because the dirty count doesn't match up, which would free all the keydata, including the one that thread 1 is currently using.
Whoops.

Some discussion follows from: https://github.com/openssl/openssl/pull/14062#issuecomment-773225594

For me it looks like the question is not about mutability but about thread-safety

There is also discussion about that in https://github.com/openssl/openssl/pull/14059#discussion_r569793419

For me it looks like the question is not about mutability but about thread-safety

It's really about both.  Like I've said elsewhere (I forget where), there was a discussion years ago (long before 2014) around the idea of declaring a key "finished".  That would be a state where a key wouldn't mutate any more, and could be used for sign, verify, etc operations.  And yes, that is a way of fixing the threading problem, but it's also a conceptual shift of what you're permitted to do with a key at diverse moments of its life span.

This would go back to the state machine logic restricting which calls can be made on the various EVP contexts.

My proposal is:

Keys are mutable only up to a point where they are set into an operation context (this would be new in 3.0 but would really simplify things). We do not need to necessarily enforce this at all the possible places but major calls should be blocked and error returned instead of doing something undefined.
Mutating keys from multiple threads at once is undefined operation (this is already in 1.1.1).
Using keys for different operations (read-only) from multiple threads at once must be possible (this worked in 1.1.1, should keep working).

I agree with 1 and 2, but not sure which operations are really read-only.

Number one is fairly easy.  If there's anything in the operation cache, no more modifications are allowed.  The check for this should be in EVP_PKEY_set_params() (which is becoming the universal function called from everything else that wants to set params, as off PR #14056, which I just noticed has been merged).

It seems that the only reason why keys now need to be mutable is
to do a key derivation. We should probably consider a new API
where it's easy to avoid that, but that can wait until after 3.0.

I agree with 1 and 2, but not sure which operations are really read-only.

That's fairly well define by 1, isn't it?  When any operation starts using a key, it becomes unmutable.
(do we want it to be unmutable for the rest of its life span, or is it mutable again when no operation is using it?  Is that the question you're asking, @beldmit?)

I agree with 1 and 2, but not sure which operations are really read-only.

All operations except setting key parameters and a key generation (including generation of a public key from a private key) are read-only.

Not sure. Implementations can do some caching on-demand.

Not sure. Implementations can do some caching on-demand.

If you mean to do some precomputations and cache the values for further operations that could find those values useful, then this is a purely provider side thing and the provider should be fully responsible for properly locking the cache accesses. It is not really a mutability of the key as seen from the application side.

What sort of caching are we talking about here?  Of course, providers can do whatever they want with their own data, but it sounds absolutely unwise to cache something operation related into the keydata itself...

Ok, agreed.

A clarification question.  Is pre-computing montgomery or CRT values considered to be the caching in an RSA key? When is that done, and does it properly discard and recompute when the key is modified?  Because to an application signing is a read-only use.

A clarification question. Is pre-computing montgomery or CRT values considered to be the caching in an RSA key?

Isn't that done early, when the key is created, i.e. before any operation uses the key?

Isn't that done early, when the key is created, i.e. before any operation uses the key?

I don't know, which is why I asked.  I looked at ossl_rsa.c and got too deep into code I don't know.

I haven't checked closely again in some time, but I am fairly sure that both RSA and EC tended to create internal context to accelerate some operations only on-demand if that operation is called or if the "owner" of the key object called a precomputation function explicitly (probably to avoid precomputing stuff that consumes time and space for every instantiated key object, as maybe that key object is just being duplicated and passed around for different protocol contexts or to convert it to other forms and shapes).
This precomputation usually has its full potential only if it is reused many times for the same key, so it made (and makes) no sense storing it into the short lived operation ctx, which explains why it is altering the contents of the key object.
Now that I think more about it, during the last year I encountered at least one place that had this behavior while I was trying to constify key parameters in EVP or provider functions, but I am failing at finding it!

There is additional discussion in #14391

OTC: We will not make EVP_PKEYs fully immutable for 3.0

Changing EVP_PKEYs to be fully immutable would be a quite big API break now. Moving to 4.0.0 milestone.


konnection:src@jmw:fix incorrect connection

Change null type network, to unknown. Add search contains by short name.
Issues#9

If I found an active network. It means it works. I'd rather not know what network it is than get information that there is no network.

A different approach for this Issue was introduced in version 1.4.2


Allow removal of next/previous button

Is it possible to expose a parameter to html_theme_options like prev_next_buttons_location: None?
I would like to disable the next/previous buttons that show up at the bottom of the page.
I can use JS to remove prev-link and next-link since it's unique.  Was hoping there was something I missed in the docs at https://sphinx-book-theme.readthedocs.io/en/latest/configure.html.
Thank you for the amazing theme!
Phi

Hmm, it's not currently an option in the theme, but could you just add this CSS:
a.right-next, a.left-prev {
    display: none;
}

?

Totally overlooked that solution.  I'll take it since I didn't realize the option I mentioned is specific to a few themes.
Thank you for the snippet.


[BUG] Ios can't find "from my mac" folder

Sync photos between mac and ios,
In sytem album can saw a folder name "from my mac",
But  use this lib not see this folder,
Is lib supporting this, or how can i resolve it?
environment
Flutter 2.0.3
photo_manager 1.2.5

If albums are in other folders, getAssetPathList can not get those albums. This method seems only get those albums in gallery folder. Any other folders created by user. This method will not return albums in other folders.

Fixed by #601 .


Don't format XML of the NixOS manual

Motivation for this change
Fix the horribly broken workflow.
Formatting has been neglected. Running make would format dozens
of files, which is a great way of scaring away newcomers and those
with less git experience. It would also annoy the heck out of
regular contributors.
The purpose of formatting is to avoid a small annoyance, so it
should not become a big annoyance that makes people give up on
their work.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Okay, sure. I guess we probably won’t need this like this for markdown.
On March 22, 2021, GitHub @.***> wrote:

@roberth https://github.com/roberth requested your review on:
#117273 https://github.com/NixOS/nixpkgs/pull/117273 Nixos dont
format.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/117273#event-4492368138, or
unsubscribe <https://github.com/notifications/unsubscribe-
auth/AAASXLAP4NB545BPAEGLC33TE6ROLANCNFSM4ZT2KOLQ>.


[TPU] For XLA Strategy, added function arg to control broadcast_master_params()

What does this PR do?
Adds boolean broadcast_master_params to XLAStrategy to control whether or not to call broadcast_master_params(). Broadcasting master parameters is not always needed. For example, when initializing random weights, if the seed is defined and the same on all devices, then broadcast_master_params() is not needed.
The motivation for this change is that broadcast_master_params() adds additional time to training and if not needed should be skipped.

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[X] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[X] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

We should add a test like this one: https://github.com/Lightning-AI/lightning/pull/17370/files#diff-664848c0cba1d81c012e047af0b592829a44870a55e75c24884569e81e3e62beR41

I added a test for the Fabric XLAStrategy.


change an exclamation mark to a dot

Godot is kind to it's users and don't want to add stress with an exclamation mark, the warning message is sufficient by himself.

The change is done.

Thanks!


Implement Deref and DerefMut traits for File

Better than implementing methods for file to get a immutable or mutable reference to the underlying fs::File type.
Requires a major bump but seems better to me.

Thanks for the PR!
My concern with this change is that it introduces ambiguity between methods defined on fs_err::File and std::fs::File. I think fs_err methods would normally take priority, but I can imagine someone accidentally writing (*file).metadata() and then wondering why their error doesn't include extra information.
It might also be a concern for backwards compatility - for example if people start calling a new std::fs method like set_times on a fs_err::File, and then we wish to add a wrapper with a slightly different signature fn set_times(&self, times: fs_err::FileTimes), then existing code willbreak.

Makes sense


add ability to indicate PriorityClass for agent in jaeger operator crd

hi,
please add ability to indicate PriorityClass for agent(parameter in deployment or daemonset) in jaeger operator crd.

@jpkrohling pls review
https://github.com/jaegertracing/jaeger-operator/pull/1392

@jpkrohling hi
pr #1392 was merged
where can we expect the release?
thank you!

@jpkrohling hi
pr #1392 was merged
where can we expect the release?
thank you!

We should be having a new release in the next couple of weeks, following the next release of Jaeger itself.


how to use Circle-map to get counts for a specific ecDNA?

Dear iprada
After using Circle-Map Realign, I can get many potential ecDNAs in .bed file. But can I get counts of a specific ecDNA by Circle-Map? or I need other software? Thanks a lot

Hi,
Could you elaborate a bit more?
Do you wanna get the read counts for every ecDNA in your sample? If so, do you program in python?

exactly, I wanna get the read counts for every ecDNA in my sample, but I don't know how to do it...

Do you program in python?

sorry... I only know it's a programming language. if it helps, I'd love to learn it

If you program in python, you could use pysam (https://pysam.readthedocs.io/en/latest/api.html)  to get the counts. I also thing that "bedtools coverage" does exactly what you want. If you do not know what I am talking about, I encourage you to find some bioinformatics collaborator that could help you.
Best,
Iñigo

Thanks a lot~~~ these clues are really valuable ~~~

You are very welcome.
Sorry I cannot be more helpful. I am finishing my PhD and I am quite busy. Feel free to open a new issue if you have more questions

Dear iprada
Hi, I used bedtools coverage to get read counts for a specific ecDNA. But how to use these reads to trace back the copy number of a specific ecDNA?
Besides, is it appropriate to directly use Circle-Map realignmy_unknown_circle.bed file (split reads + discordant reads, or just split reads) to indicate ecDNA copy number? Because my understanding is that these reads contain breakpoints. For instance, if this putative ecDNA have 5 split reads, then I can say the raw data contain 5 copies of this ecDNA?

Hi @shaka-emperor,
This outside my scope and it heavily depends on your experimental setting. If you do not have anybody with a rigurous statistics/quantitative bioinformatics background, I encourage you to find a collaborator that could help you there.
I am sorry I cannot help more on this.
Best,
Iñigo

Thank you anyway, you have helped me a lot~~


Zulip won't start

Hi first time user,
PLease help me with this error: Fail to start after restart server.
Traceback (most recent call last):
File "./manage.py", line 44, in
log_management_command(" ".join(sys.argv), settings.MANAGEMENT_LOG_PATH)
File "/home/zulip/deployments/2020-10-09-16-43-23/zulip-py3-venv/lib/python3.7                                                                                /site-packages/django/conf/init.py", line 79, in getattr
self._setup(name)
File "/home/zulip/deployments/2020-10-09-16-43-23/zulip-py3-venv/lib/python3.7                                                                                /site-packages/django/conf/init.py", line 66, in _setup
self._wrapped = Settings(settings_module)
File "/home/zulip/deployments/2020-10-09-16-43-23/zulip-py3-venv/lib/python3.7                                                                                /site-packages/django/conf/init.py", line 157, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/lib/python3.7/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 677, in _load_unlocked
File "", line 728, in exec_module
File "", line 219, in _call_with_frames_removed
File "/home/zulip/deployments/2020-10-09-16-43-23/zproject/settings.py", line                                                                                19, in
from .configured_settings import *  # noqa: F401,F403 isort: skip
File "/home/zulip/deployments/2020-10-09-16-43-23/zproject/configured_settings                                                                                .py", line 8, in
from .default_settings import *  # noqa: F401,F403 isort: skip
File "/home/zulip/deployments/2020-10-09-16-43-23/zproject/default_settings.py                                                                                ", line 13, in
from .prod_settings import EXTERNAL_HOST, ZULIP_ADMINISTRATOR
File "/home/zulip/deployments/2020-10-09-16-43-23/zproject/prod_settings.py",                                                                                line 427, in
ENABLE_GRAVATAR = false
NameError: name 'false' is not defined
Traceback (most recent call last):
File "/home/zulip/deployments/current/scripts/restart-server", line 32, in
subprocess.check_call(["./manage.py", "send_stats", "incr", "events.server_r                                                                                estart", str(int(time.time()))])
File "/usr/lib/python3.7/subprocess.py", line 347, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['./manage.py', 'send_stats', 'incr', 'e                                                                                vents.server_restart', '1603993691']' returned non-zero exit status 1.

Nevermind. It was the ENABLE_GRAVATAR line casing the issue.


Mouse events not firing correctly?

I'm completely befuddled. The demos on the main site and the react example (cloning and running) work just fine, but when I try the first "Getting started" example my own using the latest release my animations are not working at all.
Codepen reproduction: https://codepen.io/lilactown/pen/WKLGea
Strangely, closing and opening the devtools causes the animation to fire?? 😖
Confirmed in Chrome 68.0.3440.84 (Official Build) (64-bit) and Firefox 61.0.1 (64-bit).

After reading the docs, I figured out that changing space.playOnce(200) to space.play() would run the animation continuously. This has fixed my basic example.
Still trying to figure out exactly how this should interact with React... but that's another story.
I think that the example code should be updated to reflect how the API should actually be used?

Sorry about that. You're right, I need to add comments to getting started example's space.playOnce(200) because it's confusing. (It's there so that the guide's demos stop until mouse is hovered)
And will fix React soon!

Added comments in the guides' demo code to note the difference between playOnce and play https://github.com/williamngan/pts/commit/6fb106b78c0f28dd548051a27d018d7d7ff99a01
Thanks for catching this!


[Feature]: Need a dedicated contact us page for sending queries

Is your feature request related to a problem? Please describe.
Yes, this problem is related to the problem as there is no a direect connection between the conatct of user to the organisation. Thus it needs to be present of the dedicated contact us page for the users that can be used to send the required queries as well. @pranshugupta54
Describe the solution you'd like
My solution to this problem is to create a new contact us page with proper styling and responsiveness of it. Where there are required inputs taken from user to create or raise a query or send a direct message to the organization.
Alternatives
Any other alternative is that of adding the google forms of the contact us form link.
Additional Context
No response
Are you interested in contributing?

[X] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
gssoc

We have a new contact page on contact-form branch, we need to fix the styling for it - #862


autosavebans != autosaveBans

Fixing an Case error :)

Thanks for keeping the project maintained! Do you know what is wrong with the ReadMe markdown?

Well I have an admin system that depends on this so, kinda have to :)
plus had to update last night, could not unban after lastest arma update /GG ARMA


Milestone 4 | add join table for visits

🚩I have made the following changes to complete Vet Clinic - Milestone/Project 4
Here is a summary of what has been done
Type of files changed/added

In this new Branch, I have ✏MODIFIED✏ the succeeding *📂 folders and/or **📄 files to complete this milestone's requirements
*📂 root/

**📄 data.sql ✏

I have inserted data into the vets, specializations and species tables.

**📄 queries.sql ✏

New queries have been made to the animals, owners and species tables.

**📄 schema.sql ✏

I have added new tables to the schema:

vets
specializations
visits

I have also fixed the missing FOREIGN KEY constraint in the animals table.

📸 Screenshots of queries made to the database

SQL Query Collection- Click to expand!

Thank you for taking the time to review this PR ⭐

If you require additional information or have any questions, please contact me on Slack as Arturo (Arthur) Emanuel Guerra Iturres. I'll be happy to help you out. 🎯

Hi  @AbayomiOlaoye! Abayomi, thank you for reviewing this PR partner😄🙏🏼.  Wish you a great Thursday/Friday and happy coding! 🎯


Manipulate headers to downstream hosts

Hi is was playing with Ocelot and was wondering if it was possible to add information in the headers for the downstream hosts
I'm trying to add the subdomain to the headers so my back end logic can switch things based on a tenant domain.
I found this in the closed issues https://github.com/ThreeMammals/Ocelot/issues/1011
        var configuration = new OcelotPipelineConfiguration()
        {
            PreQueryStringBuilderMiddleware = async (context, next) =>
            {
                var host = $"{context.HttpContext.Request.Host.Value}";
                context.DownstreamRequest.Headers.Add("OriginalHost", host);
                await next.Invoke();
            }
        };

but the solution gives a error on the HttpContext and the DownstreamRequest
when i try to fix the code i can't find the DownstreamRequest
Ocelot version: 16.0.1

I reverted back to 15.0.6 and found out the above code is working.
Sinds version 15.0.7 the solution is broken.

@rh101 Thanks ! this is working perfect.


Only show most recent (QA) stamp

Right now, a separate QA stamp is shown on a pull for each person who has ever QA'd it regardless of whether it is still valid.
What happens now:

Pull gets stamped by tester A.
QA stamp box is green. QA stamp is green or blue depending on who is looking.
Change made in the code that invalidates the stamp.
QA stamp box is no longer green, and the stamp is orange or red depending on who is looking.
Pull gets tested and stamped by tester B.
The QA stamp box is green. The QA stamp that is orange/red (invalid stamp) remains, and a new QA stamp is added for tester B.

What should happen:

Pull gets tested and stamped by tester B.
The QA stamp box is green. The now-invalid stamp is removed and replaced with the new valid stamp that is green/blue.

Note: The number of stamps should reflect the number of required QA's, so if it's a req 2, there should be two.
Second note: I haven't checked if the same thing is happening with CR's, but I image it's likely.

The now-invalid stamp is removed and replaced with the new valid stamp that is green/blue.

@sterlinghirsh, @masonmcelvain and @evannoronha and I came up with a different but similar spec. I'd like your thoughts:

Only ever show 1 bubble per required signature

Signatures beyond the requirements would not be shown except in the hover state as specced below

If one of those is out of date

Show it orange
Show it red if you are one of the users that has a previous signature

On hover:

Show all QA / CRs
Show their statuses and times
Make them link to their respective comments / reviews

Thoughts?

Make them link to their respective comments / reviews

Oooo. I like this. I believe the other aspects will meet our needs and won't hurt anything, so I'm good with it, but it would be good to give other QA team members a chance to chime in.

I second this: https://github.com/iFixit/pulldasher/issues/298#issuecomment-1111355565
or is it "third"? Anyways, I vote in favor.

@danielbeardsley's suggestion above is very similar to what @erinemay was requesting in https://github.com/iFixit/pulldasher/issues/284. Seems like another vote in favor.


Test & build artifacts

Description
__
Please include a summary of the change and which issue it addresses. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #78
Type of change
Please mark the options that are relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
[Not needed for CI PR]
Checklist:
__

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] No new warnings are generated by my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@okdas Is this a draft or should I review it?

@Olshansk not a draft - please review / let me know if you'd wish to change something.

@okdas Appreciate the additional explanations 👌


Two python errors when running interactively

Hi!
I'm trying to get a monitor up on my rpi to understand when it's down, so I can understand what the root cause is. imon seems like a great fit for this.
I've gone through the configuration (see output below). But when I try to run the script I get the following error. Mind helping me troubleshoot?
pi@raspberrypi:~$ python3 --version
Python 3.11.2
pi@raspberrypi:~$ pip3 list | grep icmp
icmplib             2.1.1

pi@raspberrypi:~$ sudo /usr/local/bin/imon --nosyslog --instance main
2024-08-01 15:46:49 [main] Starting
2024-08-01 15:46:49 [main] Version V2.96
2024-08-01 15:46:49 [main] Command line: /usr/local/bin/imon --nosyslog --instance main
2024-08-01 15:46:49 [main] Configuration file /etc/default/imon-main
2024-08-01 15:46:49 [main] ** [global] **
2024-08-01 15:46:49 [main]     action
2024-08-01 15:46:49 [main]     datefmt           %Y-%m-%d %H:%M:%S
2024-08-01 15:46:49 [main]     instance          main
2024-08-01 15:46:49 [main]     note
2024-08-01 15:46:49 [main]     imon-configure-version V1.0
2024-08-01 15:46:49 [main]     imon-configure-created 2024-08-01 15:46:25
2024-08-01 15:46:49 [main] ** [ddns] **
2024-08-01 15:46:49 [main]     enable            False
2024-08-01 15:46:49 [main]     interval          10
2024-08-01 15:46:49 [main]     pingcount         2
2024-08-01 15:46:49 [main]     pingwait          1
2024-08-01 15:46:49 [main] ** [updown] **
2024-08-01 15:46:49 [main]     enable            False
2024-08-01 15:46:49 [main]     interval          10
2024-08-01 15:46:49 [main]     pingcount         2
2024-08-01 15:46:49 [main]     pingwait          1
2024-08-01 15:46:49 [main]     monitorip
2024-08-01 15:46:49 [main]     addping
2024-08-01 15:46:49 [main]     dnsserver
2024-08-01 15:46:49 [main]     dnslookup
2024-08-01 15:46:49 [main] ** [failover] **
2024-08-01 15:46:49 [main]     enable            True
2024-08-01 15:46:49 [main]     interval          10.0
2024-08-01 15:46:49 [main]     pingcount         2
2024-08-01 15:46:49 [main]     pingwait          1
2024-08-01 15:46:49 [main]     prefer-primary    False
2024-08-01 15:46:49 [main]     metric            201
2024-08-01 15:46:49 [main]     primary-if        emil
2024-08-01 15:46:49 [main]     primary-name      emil
2024-08-01 15:46:49 [main]     primary-gw        <IP_ADDRESS>
2024-08-01 15:46:49 [main]     primary-ping      weatherapp.sammanfatta.se
2024-08-01 15:46:49 [main]     primary-postping  weatherapp.sammanfatta.se
2024-08-01 15:46:49 [main]     standby-if        emil_ext
2024-08-01 15:46:49 [main]     standby-name      emil_ext
2024-08-01 15:46:49 [main]     standby-gw        <IP_ADDRESS>
2024-08-01 15:46:49 [main]     standby-ping      weatherapp.sammanfatta.se
2024-08-01 15:46:49 [main]     standby-postping  weatherapp.sammanfatta.se
2024-08-01 15:46:49 [main] ** [ping] **
2024-08-01 15:46:49 [main]     enable            True
2024-08-01 15:46:49 [main]     interval          10.0
2024-08-01 15:46:49 [main]     pingcount         2
2024-08-01 15:46:49 [main]     pingwait          1
2024-08-01 15:46:49 [main]     monitorip         weatherapp.sammanfatta.se
2024-08-01 15:46:49 [main-failover] Start Network Failover Monitor
2024-08-01 15:46:49 [main-ping] Start Ping Monitor
Exception in thread Thread-2:
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/usr/local/bin/imon", line 428, in run
    host.packet_loss, host.jitter))
                      ^^^^^^^^^^^
AttributeError: 'Host' object has no attribute 'jitter'
2024-08-01 15:46:50 [main-failover] Network Failover Initial State for Primary Network emil (emil): online
2024-08-01 15:46:50 [main-failover] Network Failover to Primary Network emil (emil)
2024-08-01 15:46:50 [main-failover] Network Failover Primary Route emil (emil) is active
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/usr/local/bin/imon", line 243, in run
    self._dofailover(self.primaryx, self.standbyx)
  File "/usr/local/bin/imon", line 209, in _dofailover
    self._ckping(ito)
  File "/usr/local/bin/imon", line 180, in _ckping
    self._logit("Warn: {} Network Ping Target {} Not Responding After Failover".format(ifdat['ename'], ifda['postping']))
                                                                                ^^^^
NameError: name 'ifda' is not defined. Did you mean: 'ifdat'?

Python is correct, that ifda should be ifdat. Not sure about the first one yet.
Will spend some time on this today after breakfast.
Thanks for giving imon a go, and for the feedback. Stay tuned.

Both of your issues (icmplib doc and python errors) are corrected with updates on this GitHub.
You should be able to rerun imon-install, which will download the updated files and configure the python venv that imon now uses.
Please LMK how your testing goes.

Hi, I had the same issues when I tried this a week ago. For me, the latest commit works! Thank you!
However, I would like to note that on my rpi (3 Model B+, running Raspberry Pi OS Bookworm), i needed to manually install pip3 using sudo apt install python3-pip (not sudo apt install pip3).
Also, in order to test-run imon after setting up the config file with sudo imon configure, I could not simply use sudo /usr/local/bin/imon --nosyslog --instance MY_INSTANCE_NAME, as it doesn't seem to use the venv configured during install:
Traceback (most recent call last):
  File "/usr/local/bin/imon", line 7, in <module>
    from icmplib import ping, multiping
ModuleNotFoundError: No module named 'icmplib'

Instead, I copied the command from the service and ended up with the following:
sudo /root/.imon-venv/bin/python /usr/local/bin/imon --nosyslog --instance MY_INSTANCE_NAME
This did work to run imon interactively!

@minimusubi Thx for your report. I updated the README to have the correct example command for testing (using the venv), and corrected the installer to install python3-pip rather than pip3.

No activity. Closing issue.


Update README links and add a middleware section to local-sample.py

When I tried to do a local install using vritualenv, the database migration gave me a SystemCheckError about the missing debug toolbar middleware.  The migration worked after adding a middleware section to my local.py file, so I added a section to the local-sample.py file as well.
Also, the README had a couple of broken links.   I hope guessed right as to the destinations you wanted.

Thank you for your contribution and updating the README file. However, would you mind making the same pull request to the upstream repo at https://github.com/toladata/TolaActivity/tree/new_dev.
We're a fork of that upstream repo and do not want to diverge from the upstream repo.

Just created the PR on the toladata/new_dev branch.  I mistakenly created a PR on toladata/master before I reread your request.  That one is now closed.


Help research a analogue timer

Make something like https://www.w3schools.com/graphics/canvas_clock_start.asp for the timer on the page.

Found something that might work, but not enough time to implement at the moment


[incubator/mysqlha] Add the ingress to mysqlha.

What this PR does / why we need it:
Add the ingress to mysqlha chart
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

Oh, there may have some considerations on Ingress. Ingress have to manage by other controllers, like Nginx ingress controller and Traefik etc and  Ingress can only match HTTP traffic with service port and HTTP path, so, it can not match RPC or TCP traffic. In other words,   Ingress controller support to exposure TCP/UDP service by using Service, Ingress itself can not do that.  E.g, Nginx ingress controller  is exposure service port by using the ConfigMap.


Tweak tagline on elsewhere pages

We've been using this version lately.
Should be an easy review, @mattbk @nobodxbodon @JessaWitzel et al.

"Merge," he whispered urgently.


Added test case for Tree class

@yegor256 please, pay attention to this pull request

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 8min)


fit_generator issue

Hi, I tried to modify the logic according to this suggestion but haven't managed to run training procedure successfully.
Here is the code:
import os
os.environ['TF_KERAS'] = '1'

import tensorflow as tf
from tensorflow.keras.layers import *
from tensorflow.keras.callbacks import TensorBoard, ModelCheckpoint, Callback
import dataloader
from heatmaps import *
import numpy as np
from keras_radam import RAdam
import neural_structured_learning as nsl

image_size = 200
## output shape is the same as input
n = 32 * 5
nClasses = 6
nfmp_block1 = 64
nfmp_block2 = 128
batch_size = 64

IMAGE_ORDERING = "channels_last"

model = tf.keras.Sequential([
    tf.keras.Input(shape=(image_size, image_size, 3)),
    # Encoder Block 1
    Conv2D(nfmp_block1, (3, 3), activation='relu', padding='same', name='block1_conv1', data_format=IMAGE_ORDERING),
    Conv2D(nfmp_block1, (3, 3), activation='relu', padding='same', name='block1_conv2', data_format=IMAGE_ORDERING),
    MaxPooling2D((2, 2), strides=(2, 2), name='block1_pool', data_format=IMAGE_ORDERING),
    # Encoder Block 2
    Conv2D(nfmp_block2, (3, 3), activation='relu', padding='same', name='block2_conv1', data_format=IMAGE_ORDERING),
    Conv2D(nfmp_block2, (3, 3), activation='relu', padding='same', name='block2_conv2', data_format=IMAGE_ORDERING),
    MaxPooling2D((2, 2), strides=(2, 2), name='block2_pool', data_format=IMAGE_ORDERING),
    ## bottoleneck
    (Conv2D(n, (int(image_size / 4), int(image_size / 4)),
                activation='relu', padding='same', name="bottleneck_1", data_format=IMAGE_ORDERING)),
    (Conv2D(n, (1, 1), activation='relu', padding='same', name="bottleneck_2", data_format=IMAGE_ORDERING)),
    ## Decoder Block
    ## upsampling to bring the feature map size to be the same as the input image i.e., heatmap size
    Conv2DTranspose(nClasses, kernel_size=(4, 4), strides=(4, 4), use_bias=False, name='upsample_2',
                             data_format=IMAGE_ORDERING),
    ## Reshaping is necessary to use sample_weight_mode="temporal" which assumes 3 dimensional output shape
    ## See below for the discussion of weights
    Reshape((image_size * image_size * nClasses, 1))
])

# Wrap the model with adversarial regularization.
adv_config = nsl.configs.make_adv_reg_config(multiplier=0.2, adv_step_size=0.05)
adv_model = nsl.keras.AdversarialRegularization(model, adv_config=adv_config)

radam = RAdam(total_steps=10000, warmup_proportion=0.1, min_lr=1e-5)
adv_model.compile(optimizer=radam, loss='mse', sample_weight_mode="temporal")

data_folder = 'data'
id2filename, filename2id, annotated_images = dataloader.get_image_annotations(data_folder)
df = dataloader.get_annotation_dataframe(id2filename, annotated_images)
msk = np.random.rand(len(df)) < 0.8
train = df[msk]
test = df[~msk]
encoding = MultiPointHeatmapEncoding(image_size, df, batch_size=64)

model_name = 'stacked_hourglass_adv'
log_dir = "logs/{}".format(model_name)
model_filename = "saved-models/{}.h5".format(model_name)

train_gen = encoding.generator(train, batch_size)
test_gen = encoding.generator(test, batch_size, get_weights=True)

steps_per_epoch = len(train) // batch_size
validation_steps = len(test) // batch_size
if validation_steps == 0:
    validation_steps = 1
if steps_per_epoch == 0:
    steps_per_epoch = 1

cb_tensorboard = TensorBoard(log_dir=log_dir, histogram_freq=1, write_graph=True, write_images=False)
callback_save_images = CallbackHeatmapOutput(model, get_generator(test_gen), log_dir, encoding)
checkpoint = ModelCheckpoint(model_filename, monitor='val_loss', verbose=1, save_best_only=True, mode='min')

history = adv_model.fit_generator(
            get_generator_dict(train_gen),
            validation_data=get_generator_dict(test_gen),
            steps_per_epoch=steps_per_epoch,
            epochs=5000,
            validation_steps=validation_steps,
            verbose=2,
            use_multiprocessing=True,
            callbacks=[checkpoint, callback_save_images, cb_tensorboard]
        )

before trying to use Neural Structured Learning, I used get_generator; the code is the following:
def get_generator(in_gen, should_augment=True):
    if should_augment:
        image_gen = ImageDataGenerator(fill_mode='reflect',
                                       data_format='channels_last',
                                       brightness_range=[0.5, 1.5])
    else:
        image_gen = ImageDataGenerator(fill_mode='reflect',
                                       data_format='channels_last',
                                       brightness_range=[1, 1])
    for items in in_gen:
        if len(items) == 2:
            in_x, in_y = items
        else:
            in_x, in_y, weights = items
        g_x = image_gen.flow(255 * in_x, in_y, batch_size=in_x.shape[0])
        x, y = next(g_x)
        yield x / 255.0, y

Then, according to suggestion, I modified the above generator to output the dictionary:
def get_generator_dict(in_gen, should_augment=True):
    if should_augment:
        image_gen = ImageDataGenerator(fill_mode='reflect',
                                       data_format='channels_last',
                                       brightness_range=[0.5, 1.5])
    else:
        image_gen = ImageDataGenerator(fill_mode='reflect',
                                       data_format='channels_last',
                                       brightness_range=[1, 1])
    for items in in_gen:
        if len(items) == 2:
            in_x, in_y = items
        else:
            in_x, in_y, weights = items
        g_x = image_gen.flow(255 * in_x, in_y, batch_size=in_x.shape[0])
        x, y = next(g_x)

        yield {'feature': x / 255.0, 'label': y}

Nevertheless, I get the following error:

Traceback (most recent call last):
File "/home/niko/programs/pycharm-community-2019.2.1/helpers/pydev/pydevd.py", line 2060, in
main()
File "/home/niko/programs/pycharm-community-2019.2.1/helpers/pydev/pydevd.py", line 2054, in main
globals = debugger.run(setup['file'], None, None, is_module)
File "/home/niko/programs/pycharm-community-2019.2.1/helpers/pydev/pydevd.py", line 1405, in run
return self._exec(is_module, entry_point_fn, module_name, file, globals, locals)
File "/home/niko/programs/pycharm-community-2019.2.1/helpers/pydev/pydevd.py", line 1412, in _exec
pydev_imports.execfile(file, globals, locals)  # execute the script
File "/home/niko/programs/pycharm-community-2019.2.1/helpers/pydev/_pydev_imps/_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "/home/niko/workspace/pupil-detection-v2/fcn-adv.py", line 92, in
callbacks=[checkpoint, callback_save_images, cb_tensorboard]
File "/home/niko/workspace/advanced-computer-vision-course/venv_python3/lib/python3.5/site-packages/tensorflow_core/python/keras/engine/training.py", line 1296, in fit_generator
steps_name='steps_per_epoch')
File "/home/niko/workspace/advanced-computer-vision-course/venv_python3/lib/python3.5/site-packages/tensorflow_core/python/keras/engine/training_generator.py", line 209, in model_iteration
callbacks.on_epoch_begin(epoch, epoch_logs)
File "/home/niko/workspace/advanced-computer-vision-course/venv_python3/lib/python3.5/site-packages/tensorflow_core/python/keras/callbacks.py", line 283, in on_epoch_begin
callback.on_epoch_begin(epoch, logs)
File "/home/niko/workspace/advanced-computer-vision-course/venv_python3/lib/python3.5/site-packages/tensorflow_core/python/keras/callbacks_v1.py", line 382, in on_epoch_begin
self.model._make_test_function()
File "/home/niko/workspace/advanced-computer-vision-course/venv_python3/lib/python3.5/site-packages/tensorflow_core/python/keras/engine/training.py", line 2145, in _make_test_function
self._feed_targets +
AttributeError: 'AdversarialRegularization' object has no attribute '_feed_inputs'

@nikogamulin, thanks for the question.
The TensorBoard callback object in TensorFlow 1.x doesn't work with subclassed models, and unfortunately AdversarialRegularization is one. Could you please try the code in TensorFlow 2.0?

Thanks, @csferng! I have migrated to TF 2.0 and everything works fine now.


get('branch') when run locally returns hidden chars in beta.17

This commit: https://github.com/deployphp/deployer/commit/b6b8eacef51a7d7014ce5cbe8cba96a488ce5a29 seems to have caused an issue in our deployments.
I'm running zsh (not bash) in phpstorm terminal and the value returned for runLocally to get the branch contains ansii escape sequence and hidden characters:
(Bmain

The solution for me was the manually set a default branch set('branch', 'main'); in my config.
For reference: https://en.wikipedia.org/wiki/ANSI_escape_code#CSI_(Control_Sequence_Introducer)_sequences

What dep version are you using?

What dep version are you using?

beta.20 but anything from beta 17 causes the issue.

Can you try to run same command in same Symfony Process component?

You could also check that your .profile does not output anything.


Issue with GROUPS of elements, which are not placed on each page…

When I have a group of elements (e.g. two grouped text frames) in the template, then these groups are not child elements of each but, but all groups belong to the "top" page and therefore get deleted from all pages when the top page get's deleted.
Is this reproduceable or does anyone get groups to work properly?
Thanks in advance and thanks your work!
edit: To be clear; the grouped elements appear on every site, just as you would expect, but they all "belong" to one site only.

I cannot reproduce. Could you please attach a problematic SLA file or describe the steps to create one ?

closing issue for lack of answer. @something-strange-to-eat kindly reopen if you still experience the problem and want help fixing it.


Added Ruby version to Gemfile

Description
Adds the current Ruby version to the Gemfile so that dependencies are solved, taking into consideration the current Ruby version. Don't know why this was so hard to find, but I finally ran across bundler PR 4651 that described this workaround.
Issues Resolved
Fixes #85
Check List
N/A. Just an improvement for dev

Any questions/concerns with this @tmiotto or @hdiomede ?


Use cleanupOutdatedCaches and clientsClaim at the same time

Library Affected:
workbox-core, workbox-precaching
Issue or Feature Request Description:
Is it possible to use workbox.core.clientsClaim() and workbox.precaching.cleanupOutdatedCaches() at the same time ? I wonder if it will not create two activate event listeners, and one could override the other. This would be an unwanted result.
const { clientsClaim } = workbox.core;
const { cleanupOutdatedCaches } = workbox.precaching;

/* ... */

cleanupOutdatedCaches();
clientsClaim();

Ok, this should not be a problem. I thought cleanupOutdatedCaches wasn't working because of clientsClaim, but cleanupOutdatedCaches just cleanup precache caches, not old runtime caches or any other cache.

Yeah, deleting a cache is something that can be done using the Cache Storage API directly (as you're doing in that example), so wrapping it in a Workbox method didn't seem necessary.


Feature request: Better to have conversation saved

Thanks for your brilliant project! I really appreciate its simplicity and power.
In regards to start conversation from last context, would it be possible to add a feature that allows for saving conversations? This would be very helpful.

Great suggestion, I will implement this feature as soon as possible.

I have now updated with a new feature that supports the "New Chat" function. The historical conversation data of the topic will be stored in the IndexedDB database, as this is a storage method with relatively large capacity on the browser-side. We hope that you will find it more comfortable to use.


Close edited buffers if threshold is reached

I've set a pretty small threshold(7) so I can always navigate with tabs across buffers. However, when I edit many files in one go, I simply can't open buffers till I close ones.
My proposition is to add an option to close edited buffers if they are saved and there is no place for opening a new buffer.
I can try implementing the feature, but first say pls, if there is any alternative already presented already..

Hi! Thank you for bringing this up. I do see value in this idea, however, I wouldn't use it myself, since I use autosave. Implementing this should be easy. Create an autocommand for the BufWritePost event and call toggle_pin action. I think you will have to also check the state so smth like
state = require("hbac.state")
actions = require("hbac.actions")

if state.is_pinned(buf) then
  actions.toggle_pin(buf)
end

Thanks for the proposition! I think you are right, it is possible to do on the client side, and I'm satisfied with the solution!
Resulting autocmd for anyone who is interested, really easy:
autocmd("BufWritePost", {
  callback = function(ev)
    local state = require "hbac.state"
    local hbac = require "hbac"
    if state.is_pinned(ev.buf) then
      hbac.toggle_pin(ev.buf)
    end
  end,
})


Require transactions to refer to recent blocks

Currently, we have no restriction on how old a block referenced by a transaction needs to be (it could be a genesis block, for example). It may be beneficial to make this a bit more restrictive (and this is done in some other chains - e.g., Solana). For example, we could require that a transaction references a block which is within 256 blocks of the chain tip. This would be a global setting applicable to all transactions. Individual transactions would still be able to reduce this window via individual expiration settings as they can now.

Is there any specific reason why we would not want to initialize the expiration delta to 256 instead of u32::MAX on the kernel prologue?

Good point! Though, we'd set the block expiration time to block_ref + 256 rather than the delta directly. I'll transfer this issue to miden-base.


Appium Python

I'm already using Appium and have Python test code.
Can Python (Appium) be used with Test Cloud?

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

ID: ed7c0d11-e60a-f088-e983-7465aca41515
Version Independent ID: b2c6a308-1fcb-a385-239d-862b2c14114c
Content: Preparing Appium Tests for Upload - Visual Studio App Center
Content Source: docs/test-cloud/preparing-for-upload/appium.md
Service: vs-appcenter
GitHub Login: @glennwester
Microsoft Alias: glwest

Hi @gremito, @Oddj0b would be happy to help with the above.

Currently we only support Appium Java tests as noted in the doc.
You can file and vote on this or other feature requests here: https://github.com/Microsoft/appcenter/issues


Highlight search results in minimap

This would indeed be a nice feature! PR welcome!

vimscript skills are at a level of zero, and currently I am more into kakoune

I would also love this feature :-)

+1


Support stake/unstake transaction for TRON

Current Functionality
List of transactions doesn't contain stake/unstake transaction. When a wallet imports those transactions doesn't appears in the wallet but It would be nice to have.
Possible Improvement
Return stake/unstake transaction

Part of https://github.com/trustwallet/blockatlas/issues/1122

Duplicated


Add custom annotations and labels to tetragon operator

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
I encountered a limitation with my ability configuring annotations and labels for the tetragon-operator workload. Currently helm-chart doesn't support custom annotations and labels, however I would like to have this ability, due to the fact that in my infrastructure, I can configure prometheus scraping and log shipment only by adding annotations and labels.
Describe the feature you would like
I would like to have an ability to add custom annotations and labels to the tetragon-operator pods and deployment, the same as it already implemented for the daemonset pods:
https://github.com/cilium/tetragon/blob/v1.0/install/kubernetes/values.yaml#L8
https://github.com/cilium/tetragon/blob/v1.0/install/kubernetes/values.yaml#L21
Describe your proposed solution
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks, above make sense to me.
I think the best approach for this would be to:

Apply the podAnnotations in both the daemonset and the operator deployment pods
Have specific annotations for tetragon daemonset set under tetragon in the chart (tetragon.podAnnotations)
Have specific annotations for the operator deployment set under tetragonOperator in  chart (tetragonOperator.podAnnotations)

Thoughts? Would doing just (1) solve your specific issue?
Also, Would you be interested in implementing this?
Thanks,
Kornilios.


.NET Core / Standard API docs: Hyperlinks in  XML doc tag render incorrectly in VisualStudio IntelliSense tooltips

(I am not sure I am posting in the correct repository—if not, please direct me to the right place.)
I've noticed an issue with .NET Core & .NET Standard API documentation. Hyperlinks end up being rendered incorrectly in Visual Studio IntelliSense tooltips.
I'll demonstrate this by example of the  for the System.FormatException class.
Documentation in the .NET Framework (correctly rendered)
This is what the IntelliSense tooltip for System.FormatException will look like when targeting the .NET Framework (4.x):

This is what the relevant part of the XML documentation file looks like (additional line breaks and indentation added by me):
<member name="T:System.FormatException">

    The exception that is thrown when the format of an argument is invalid, or when a
    composite format string is not well formed.

</member>

Documentation for .NET Core and .NET Standard 2.x (incorrectly rendered)
And the same when targeting either .NET Core or .NET Standard 2:

And again, the relevant part of the XML documentation file (additional line breaks and indentation added by me):
<member name="T:System.FormatException">

    The exception that is thrown when the format of an argument is invalid, or when a
    <see cref="~/docs/standard/base-types/composite-formatting.md">composite format string</see>
    is not well formed.

</member>

VisualStudio IntelliSense does not render <see cref="link">link text</see> correctly. It'll display the cref instead of the text between the two tags.
However, using link text instead might work properly.

@mairaw Should we move this to the MicrosoftDocs/Feedback repo?


chore: fixed links in docs / issues

Fixing broken links and refs left over from old project and repo names.

Merged, thanks!


unix: factor out reusable hrtime implementation for POSIX platforms

Create a dedicated source file to share among platforms on which we
implement hrtime using POSIX clock_gettime.  De-duplicate our
existing copies of this implementation on such platforms.

Tiny commit log nit: can you make sure the first line is <= 50 columns and the ones after that <= 72?  Cheers.

I've updated the commit message.  Thanks!

Landed in https://github.com/libuv/libuv/commit/2ba39be67e948f8b99af7cf408828f1507371fd5. Thanks!


led matrix fps limited to 60hz and no way to send all pixels in 1 frame.

I was trying to create some animations for the led matrix, however I noticed that the commands are only received every frame (at 60fps) even at high baud rate. This means updating all 9 columns at once results in a maximum of 60fps. Black and white animations can be run at 60fps sending everything in the 39 bytes, but this is impossible for grayscale animations at the moment. Would it be possible to add a command for sending all 306 bytes of grayscale image data?
An easier improvement would be to reduce the delay time for sending the whole column and not clear the column buffer by default.
Here is a code example of what I am trying to accomplish, note the FPS counter
import serial
import time
import random
FLUSH_BUFFER = 0x08
SEND_COL = 0x07
def send_command(s, command_id, parameters, with_response=False):
      s.write([0x32, 0xAC, command_id] + parameters)
      if with_response:
          res = s.read(32)
          return res

s = serial.Serial("COM3", 115200)

while(1):
    start = time.time()
    # loop 10 times to get a good avg fps
    for _ in range(10):
        for i in range(9):
            arr = [i]+[random.randint(0, 255) for q in range(34)]
            send_command(s, SEND_COL, arr, with_response=False)
        send_command(s, FLUSH_BUFFER, [], with_response=False)
    end = time.time()
    print("FPS: ", 1/((end-start)/10))

Output:
FPS:  5.92586648213136
FPS:  5.925065363114918
FPS:  5.88382407939697
FPS:  5.9254545945151
FPS:  5.927377226756621
FPS:  5.880792332361544
FPS:  5.923988337622522
FPS:  5.926013000679308
FPS:  5.881560904169057

Would it be possible to add a command for sending all 306 bytes of grayscale image data?

Yes that is absolutely possible, but not right now. The buffer in the firmware is not large enough to fit it all.

An easier improvement would be to reduce the delay time for sending the whole column

Which delay is that? On the host side or the module side?


Algorithm 2 inconsistent with code

Thanks for the work :)
I have read the code, and corresponding issues #10, but there are some places I still feel inconsistent with the paper. Please correct me if I am wrong.

In the Algorithm 2 line 7, we are calculating the norm between local model and global model. But the code is using l2 norm for local model without considering global model. Taks mnist/mclr.py line 40 for example.

I also checked the NLP experiments Shakespare, but I didn't find the regularization part in create_model. shakespare/stacked_lstm.py create_model

Thank you!

The updates of FedProx is defined here: https://github.com/litian96/FedProx/blob/master/flearn/optimizer/pgd.py#L29-L36
where var is the current local model you are optimizing for, and vstar is the current global model which is the starting point of solving for the local subproblem.

Thank you, It makes sense now!

Hi Tian,
Thanks for previous reply, but there is one place I still don't figure out. The paper shows objective function is optimizing Fk(w) and ||w - w^t|| simultaneously, but the code shows that we are calculating loss and ||w - w^t|| seperately.
Take mnist task as an example, the code first calculate the cross entropy loss in
grads_and_vars = optimizer.compute_gradients(loss), this part does not consider ||w - w^t||.
By calling var_update = state_ops.assign_sub(var, lr_t*(grad + mu_t*(var-vstar))) in train_op = optimizer.apply_gradients(grads_and_vars, global_step=tf.train.get_global_step()) we consider the dissimilarity between w and w^t.
Therefore, it seems that the code is calculating the gradient from Fk's perspective, and then subtract a dissimilarity part to get the final gradient. I also debuged the code to check who has called _apply_dense(), it shows that this method is only used by apply_gradient(), instead of compute_gradient().
Thank you in advance :)

You are exactly right that we first need to obtain the gradients of \nabla F_k (w_local) + \mu (w_local - w_global), and then apply the gradients to w_local.
What is your question?

My question is compute_gradient() only calculates \nabla F_k(w_local). How can we find argmin_w h_k without considering \nabla (w_local - w_global)?
I know we considered \mu (w_local - w_global) in apply_gradient(), but this proximal term not necessary leads to argmin h_k.

The gradient of h_k is \nabla F_k (w_local) + \mu (w_local - w_global), which is what we apply to update w_local (specifically, grad + mu_t * (var-vstar) ).

Ohhhhh, I see, really appreciate for the clarification :)


Don't use withVariantReselection for Jacoco report aggregation with only the classes attribute present

Fixes #25618

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


Enabled light_profile_template.xlsx download

The "touchy" part of this is that the user probably has to "allow" the site to download.  This may be something we change.  This is MVPish.

Totally fine for the MVP.


DBAAS-368 MongoDB Database instance Provision failed

This PR fixes the issue DBAAS-368:

After an atlas project is created and before starting the creation of a new cluster, the project should be ready.
The project name (and parameter values) should have leading and trailing spaces trimmed to avoid confusion to the users.

@jianrongzhang89 couple of lint issues that need to be fixed, please. The job has added comments in the changed files tab


leaderboard project Hit the API part

This is the leaderboard project. The leaderboard website displays scores submitted by different players. It also allows you to submit your score, fetch scores from the API and render to the web.

done

done


Trim called without arguments throws index out of range

Version: alpha 4
When using toLower(), I was getting an IndexOutOfRange exception during query generation. When debugging the source, I found there was an issue in the JetStringMethodTranslator where the Trim clauses are formed. It checks if the passed arguments contain a null value on index 0, but it doesn't check if there actually is a value to begin with. That was the reason for the exception.
I've adjusted the clauses to now first do that check before attempting to access the arguments.
I noticed a couple of tests failing with the same exception. After adding the fix, there was still an issue, where UPPER was expected in the generated query, while UCASE was used. The latter is the correct MS Access Function so I've adjusted the expected result both the upper and lower case test.
Fixes #78

@Veflow Thanks for your contribution!


feature: compare after measure

Summary
Automatically outputs reassure compare output after reassure measure iff baseline file already exists.
In addition, this makes reassure measure useful as the default CLI command, so it can be run using just reassure.
Resolves #57
Test plan
Tests run successfully.

I wonder if we actually should remove the compare command altogether. User will basically have to run only these two commands in his perf script:

reassure --baseline
reassure (will also output compare, as baseline file is available.

Instead of reassure measure  --baseline + reassure measure + reassure compare.
@thymikee @Xiltyn wdyt?

For reference V8 does this:
Baseline
$ test/benchmarks/csuite/csuite.py sunspider baseline out.gn/master/d8
Wrote ./_results/master.
Run sunspider again with compare mode to see results.

Current + compare
$ test/benchmarks/csuite/csuite.py sunspider compare out.gn/x64.release/d8

                               benchmark:    score |   master |      % |
===================================================+==========+========+
                       3d-cube-sunspider:     13.9 S     13.4 S   -3.6 |
                      3d-morph-sunspider:      8.6 S      8.4 S   -2.3 |
                   3d-raytrace-sunspider:     15.1 S     14.9 S   -1.3 |
...

See: https://v8.dev/docs/benchmarks#example-usage

Check for #78 for implementation of the above concept

@mdjastrzebski I've merged the followup PR here. Can you adjust the description and changeset message?


usb: fix CDC ACM breakage with CONFIG_USB_DEVICE_SOF

Commit e4c447aac3 ("usb: add SoF event") added support for calling the
status callback with the USB_DC_SOF value for each SoF when
CONFIG_USB_DEVICE_SOF is enabled. The CDC ACM driver saves the latest
received status and compares it USB_DC_CONFIGURED to decide if it can
send data to the host. This therefore doesn't work any more when the
status callback is called regulary with USB_DC_SOF.
Fix that by ignoring USB_DC_SOF when saving the latest received status.
Signed-off-by: Aurelien Jarno<EMAIL_ADDRESS>

Codecov Report

Merging #12115 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #12115   +/-   ##
=======================================
  Coverage   48.07%   48.07%
=======================================
  Files         284      284
  Lines       43328    43328
  Branches    10389    10389
=======================================
  Hits        20828    20828
  Misses      18363    18363
  Partials     4137     4137

Continue to review full report at Codecov.

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


Invalid checksum

Hi,
I've followed all the great samples provided and actually I'm facing the following issue:
Invalid checksum     at Kerberos.NET.Crypto.AES.AESDecryptor.DecryptWith(Byte[] workBuffer, Int32[] workLens, Byte[] key, Byte[] iv, KeyUsage usage)    at Kerberos.NET.Crypto.AES.AESDecryptor.Decrypt(Byte[] cipher, Byte[] key, Byte[] iv, KeyUsage usage)    at Kerberos.NET.Crypto.AES.AESDecryptor.Decrypt(Byte[] cipher, KerberosKey key, KeyUsage usage)    at Kerberos.NET.Crypto.AESDecryptedData.Decrypt(KeyTable keytab)    at Kerberos.NET.KerberosRequest.Decrypt(KrbApReq token, KeyTable keytab)    at Kerberos.NET.KerberosValidator.Validate(Byte[] requestBytes)    at Kerberos.NET.KerberosAuthenticator.Authenticate(Byte[] token)    at Kerberos.NET.KerberosAuthenticator.Authenticate(String token)    at elearningapp.MessageHandlers.KerberosMiddleware.ParseKerberosHeader(OwinEnvironment env, HttpContext httpContext)
Does anyone has already encountered that before ?
Thansk in advance !

This often indicates either the provided key isn’t correct, or the SPN doesn’t match. AES explicitly requires an SPN be set on the provided KerberosKey if you are providing a raw secret. You don't need to provide the SPN if you're using a key tab file.

Possibly. Depends on how the file was created. You can try reviewing the contents of the file (using the debugger or some external tool) and see what the PrincipalName is and make sure it matches the SPN on the service account in AD.
You can also try disabling AES on the service account in AD so it forces RC4 to make sure the key itself is correct.


Hololens pose sync

I'm looking for a way to combine WebRTC with HoloLens synchronized pose data. Would peercc-unity example help me to achieve this purpose? where can I get your latest WebRTC HoloLens code running?
Regards,
Danilo.

Hi Danilo,
Here's a sample that does the above: https://github.com/phongcao/webrtc-mrvc-sample
The latest working branch is still this one, which requires older Unity tooling: https://github.com/ortclib/ortclib-sdk/tree/Mosa/20180320-peercc-unity
We are currently very close to having M66 running in Unity with with a 2018 toolset.  I'll update you when that's ready, or you can keep an eye on the issues here which is where the work is taking place: https://github.com/webrtc-uwp/webrtc-uwp-sdk
Thanks,
James

Thanks for you answer

@jamescadd Is M66 working with Unity at this point? I am working on updating an old application of mine that uses the HoloLens with WebRTC and want to know what is the known best/current branch to work off of.


Use sbt-compat lib

So the ad-hoc compatibility wrappers would not clash with other plugins that use sbt-compat.

Relevant sbt-compat change: https://github.com/dwijnand/sbt-compat/pull/1

sbt-compat 1.1.0 now should have this change, so this PR could be updated :+1:

Cool, thanks, I'll have a stab.

Hmm strange it doesn't seem to compile for sbt 0.13. Did that work @2m?

Nope. I only tried 1.0.


Can not create developer token ?

I can't create dev token anymore?
I get following message:

Update: the creation of developer tokens is temporarily disabled.

So how do I authorize ?

You need to contact their customer support to get your account whitelisted for dev token creation.

@mikaelho Customer support? Can you provide a link? I tried using forums, but it didn't help.

I just used Google and followed the links to:
https://help.evernote.com/hc/fi/requests/new
Your language will be different and you will need to sign in.

@khurshid-alam
have you solved it ?


Ukrainian translation

Description
This PR translates application to Ukrainian following our script - https://github.com/decent-dao/fractal-interface/blob/develop/TRANSLATIONS.md
Notes
n/a
Issue / Notion doc (if applicable)
n/a
Testing

Open the app
Switch the language to Ukrainian
Verify that you see Ukrainian language
Thoroughly walk through the app - check that all the strings are translated (some names like "Fractal", "DAO", "NFT", "Gnosis", "Safe" are not translatable).
Glory to Ukraine! Slava Ukraine!

Screenshots (if applicable)

I found a couple of small things which I do not have anything do to with your translations, but were not apparent until I could turn a legit one on. I'll make issues for them. So Thumbs Up!! looks awesome.

I'll approve, but is this PR going to be waiting on the current open PR updates?

I think we can merge this in, then after all the other updates are done and copy changes are made we can easily export the English / Ukrainian comparisons to CSV again and make updates


[BUG]CMS Components- products in 'Commerce' block are are not shown in the right places

Describe the bug
Block category 'Commerce' - the products are not shown in the right places. Products in 'Three columns, product boxes' block are shown one after another, not one next to another.
How to reproduce
Initial conditions: the category 'Movies & Toys' is active
Steps to reproduce the behavior:

Go to 'categories' from Shopware
Select 'Clothing'-'Movies & Toys' from category structure
Go to 'Layout assignment'
Click 'Edit in designer' button
Click 'Settings' icon
Click 'block' icon
Select block category 'Commerce'
Add 'Three columns, product boxes'
Click 'setting' icon and add products on each product boxes
Click 'Save' button in the upper right corner
Go to subcategory 'Movies & Toys' on our website
See error

Expected behavior
Products in 'Three columns, product boxes' block are shown one next to another on the website.
Actual behavior
Products in 'Three columns, product boxes' block are shown one under another on the website.
Screenshots
'Three columns, product boxes' block in Shopware 'Shopping Experiences' layout:

'Three columns, product boxes' on the website:

Devices (please complete the following information):
All devices

@akrajna can you rechek it?

I have rechecked it and it seems the 'Three columns, product boxes' block is not implemented. Because there is the information: 'Block product-three-column is not implemented yet!' on the page.

A new issue for it is created #897


Adds admin game command

Description
Adds the !game command which can let admins create ad-hoc games between users.

Resolves #38

Checklist

[x] Add tests for these changes
[x] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #40 into master will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   96.38%   96.68%   +0.29%
==========================================
  Files           8        8
  Lines         637      694      +57
  Branches       98      110      +12
==========================================
+ Hits          614      671      +57
  Misses         19       19
  Partials        4        4

Impacted Files
Coverage Δ

src/spellbot/__init__.py
95.58% <100.00%> (+0.60%)
:arrow_up:

src/spellbot/data.py
97.79% <100.00%> (+0.06%)
:arrow_up:

Continue to review full report at Codecov.

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


Restore Git bash utilities to the PATH

This is a quick fix for issue https://github.com/actions/virtual-environments/issues/282
In #211 C:\Program Files\Git\usr\bin was removed from the PATH caused to using Windows tar(C:\Windows\System32\tar.exe) instead of the tar comes along with Git. Windows doesn't have xz support by default, tries to use mingw xz utility and hangs.
Changes:

Restore "C:\Program Files\Git\usr\bin" in the PATH, until MSYS2 is added(https://github.com/actions/virtual-environments/issues/30)

Is any regression expected after these changes?
In scope of #211 , this line was removed so, probably, we can bring some issues back again

Is any regression expected after these changes?
In scope of #211 , this line was removed so, probably, we can bring some issues back again

@maxim-lobanov complains were mostly about mingw64 directory in the PATH. And after #211  changes some of the tools are broken as explained in this comment
https://github.com/actions/virtual-environments/pull/211#pullrequestreview-346292514


[Decentral Games] Decentral Games Strategy

Strategy for Decentral Games added

Hi @SatoshiNaoki This looks equal to multichain strategy. why not use it instead?

for some contracts, voting power should be x1000.
I was going to do it by setting decimals as 15 instead of 18, but it seems it's not good.
so I just updated the calculation

Hi @SatoshiNaoki

for some contracts, voting power should be x1000.
I was going to do it by setting decimals as 15 instead of 18, but it seems it's not good.
so I just updated the calculation

Which line is this change? I see it is the same as multichain? 🤔
Do you mean in examples.json? if so you can change the params from your space settings directly

Closing this as multichain strategy works for you. feel free to reopen if needed


[Feat] Review Section Design at the landing page

Description
Issue: Creation of a Review Section for "AI Content Generator" Website
Hi @iamkanhaiyakumar @GauravKesh
Project Overview
The "AI Content Generator" website is a platform built to deliver AI-powered content generation tools tailored to various professional needs. This site leverages modern development frameworks to ensure efficient, scalable content workflows, ultimately enhancing productivity across diverse user groups. The website caters to multiple user personas by providing specialized toolsets, each designed to streamline specific tasks for coders, content creators, and social media influencers.
Objective
To develop a comprehensive review section for "AI Content Generator" that captures user feedback on the various AI tools available. The review section will allow users to share insights, rate features, and provide suggestions based on their experiences. This addition will not only foster community engagement but also provide valuable feedback for iterative improvements.
Targeted Tools for Review
The review section should allow users to provide feedback on specific tools within the platform, including but not limited to:

For Coders:

Write Code: AI-assisted code generation with support for multiple programming languages and styles.
Debug: Automated debugging tool to identify and suggest solutions for common coding issues.
Run Test: Tool to execute test cases and provide feedback on code quality, performance, and optimization.

For YouTubers:

YouTube SEO: Optimizes video metadata for better search visibility and engagement on YouTube.
Script Writer: Assists in generating and structuring engaging video scripts based on topic input.
Hashtag Generator: Suggests relevant hashtags to boost video discoverability and reach.

For Instagram:

Post Caption Creator: Generates captions tailored to the post’s theme and tone.
Hashtag Generator: Identifies trending hashtags relevant to the post to enhance engagement.

Post Idea Generator:Generate ideas which creators can use for their own purpose.

Proposed Structure for the Review Section
The review section should be structured to capture user feedback in a way that is both organized and actionable:
Ratings and Reviews:

Rating: A star-based or numerical rating system (e.g., 1-5 stars).
Commentary: Text boxes allowing users to elaborate on their experience with each tool.

Benefits of the Review Section

User Engagement: Encourages active participation, fostering a sense of community among users.
Product Improvement: Collects valuable feedback for developers to enhance tools based on real user input.
Transparency and Trust: Demonstrates a commitment to user satisfaction and continuous improvement.

This review section will not only enhance user engagement but also provide a roadmap for the continuous enhancement of the AI Content Generator platform.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hi @GauravKesh , I just completed the design let me know what you think about it

@dev129 It's looking nice but need to add relevant data there. and do you have different idea also is it reponsive for all devices

@GauravKesh
Responsiveness is still to be tested, its a matter of few minute and as for ideas what we can do is tag give somewhat famous faceless YT creators who use AI, some software engineers will in MAANG, I can also add an auto play button between left and right button which plays comment automatically

Sure thing, @dev129 ! Instead of tagging famous YouTube creators without their okay, we can use titles like 'AI Enthusiast' or 'Tech Expert.' We can also highlight comments from real users and smaller creators to make the content more engaging and fun!

Hi @GauravKesh , here is the updated version with changes you suggested, any more changes you wanna see
https://github.com/user-attachments/assets/08816e2a-08b0-40b2-b7a1-518d2ee1b80c

Hi @GauravKesh @iamkanhaiyakumar do you like the design ?

@dev129 It's looking good. Just make the buttons round

Very well @GauravKesh , you please assign the level label.

Hi @GauravKesh @iamkanhaiyakumar , this is the final design , I hope you will like it.
https://github.com/user-attachments/assets/4da4aa02-da5e-4b88-a311-374b2001367e
I have made the button round, setup autoplay, applied images.

Hi @GauravKesh , shall I launch the PR ?

@dev129 yeah you can make PR