Missing target while using actor_stream

hi.
I'm expecting troubles with actor_stream(request.user).
for example:
from actstream.models import actor_stream, Action
print actor_stream(request.user)[0].target # returns None
print Action.objects.all()[0].target # returns Target object

BUT:
from actstream.models import actor_stream, Action
print request.user.actor_actions.all()[0].target  # returns Target object

User model - this is a custom user model, configured as described in djangoproject tutorial.
Target model looks like this:
class Task(models.Model):
    id = models.UUIDField(primary_key=True, default=uuid.uuid4, editable=False)
    user = models.ForeignKey('core_auth.User', blank=True, null=True)
    ... < more fields >

Let me show you:
>>> from actstream.models import actor_stream, Action
>>> from core_auth.models import User
>>> u = User.objects.get(username='fynjah')
>>> actor_stream(u)[0].target # None
>>> x = actor_stream(u)[0]
>>> x
<Action: fynjah changed status DONE 58 minutes ago>
# BUT:
>>> x.target_content_type
<ContentType: Task>
>>> x.target
>>> x.target_object_id
u'f08ae5ab-8c07-4929-9021-62c867f0d081'
# AND:
>>> u.actor_actions.all()[0].target
<Task: wdstrm compositing>

What am i doing wrong :\

what is your exact issue? i think it's "target returns None when using actor_stream" but i want to make sure
also can you please provide a more complete example that illustrates the problem? i dont see how the Action and Task models are being created

Hi. Yes, the exact issue is "target returns None when using actor_stream".
Here is how actions are being created.
@receiver(pre_save, sender=Task)
def detect_task_status(sender, instance, **kwargs):
    try:
        obj = Task.objects.get(pk=instance.pk)
    except Task.DoesNotExist:
        # Object is new, so field hasn't technically changed, but you
        # may want to do something else here.
        pass
    else:
        if not obj.status == instance.status:  # Field has changed
            logged_in = LoggedInUser()
            # LoggedInUser is a singleton, which processed in middleware
            # and returns request.user
            user = logged_in.user
            action.send(user,
                verb='changed status',
                action_object=instance.status,
                target=instance,
                old_status={
                    'name': obj.status.name,
                    'id': obj.status.id,
                    'color': obj.status.color
                    },
                action_type='change_status')

Once action is created, i'm trying to receive actor_stream
@login_required
def get_stream(request):
    from actstream.models import actor_stream, Action
    mdict = {'actions':actor_stream(request.user)}
    return render_to_response('user/stream.html', mdict, context_instance=RequestContext(request))

And then target returns None.
But if i'm doing like this, everything is ok:
@login_required
def get_stream(request):
    from actstream.models import actor_stream, Action
    mdict = {'actions': request.user.actor_actions.all()}
    return render_to_response('user/stream.html', mdict, context_instance=RequestContext(request))

are you using the Action.public bool at any point? the actor_stream should return essentially request.user.actor_actions.public(**kwargs)

No, i'm not using public anywhere. Actually, all code i'm using at this moment i've posted above.
I've just checked actions in DB - they are all marked as public=true

hrm very weird. i notice you are using the pre_save signal, can you try switching to the post_save? i believe in pre_save the object will not have an id yet which throws off GFK relationships. that's the only thing that seems out of place to me. if that doesnt help then im not sure what else to do. luckily you have a working implementation so just the actor_stream is broken but you can complete your view.

Hi. In pre_save signal i verify does object exist or not.
here:
    try:
        obj = Task.objects.get(pk=instance.pk)
    except Task.DoesNotExist:
        # Object is new, so field hasn't technically changed
        pass
    else:
     .......

So if object already exists and status field is changed - than create action.
And even more, in the first post i've showed this:
>>> actor_stream(u)[0].target # None
>>> x = actor_stream(u)[0]
>>> x # An action exists
<Action: fynjah changed status DONE 58 minutes ago>
>>> x.target_content_type
<ContentType: Task> # Content type exists
>>> x.target # None (!!!)
>>> x.target_object_id # target_object_id exists
u'f08ae5ab-8c07-4929-9021-62c867f0d081'

So, as you can see, target_content_type and target_object_id exists, but target field retuns None
About issue #246 - I've tested request.user.actor_actions.all().fetch_generic_relations() and target again returns None !
Here is log from shell:
>>> from core.models import User
>>> u = User.objects.get(username='fynjah')
>>> u.actor_actions.all().fetch_generic_relations()
[<Action: fynjah changed status APPROVED 1 day ago>, <Action: fynjah changed status COMMENTED 4 days, 3 hours ago>, <Action: fynjah changed status DONE 4 days, 3 hours ago>, <Action: fynjah changed status HOLD 4 days, 3 hours ago>, '...(remaining elements truncated)...']
>>> x = u.actor_actions.all().fetch_generic_relations()
>>> x = x[0]
>>> x.target # None (!!!)
>>> x.target_content_type # Exists
<ContentType: Task>
>>> x.target_object_id # Exists
u'1174029a-3cf8-4e93-aaed-1080a3fee417'

I think it's all about uuid as primary key in Task model

I'm having this same problem with using actor_stream or any_stream.  Getting none for action_object.  Verified action_object exists in the DB and I can access it directly (without using streams).  Have not tried target or other streams.
Setting FETCH_RELATIONS to false (mentioned in other thread with similar issue) did not seem to fix things.
Streams basically unusable currently.

I am facing a similar issue with using the user_stream via the template tag as follows:
{% activity_stream 'user' request.user %}
{% for action in stream %}
    {{ action.pk }} : {{ action.target }} : {{ action.action_object }}
{% endfor %}

The above template renders as follows:
7 : None : None
6 : None : None
5 : Amyth Arora : None
3 : Amyth Arora : None
1 : None : None

And when I do the following in the shell for the above actor object:
In [15]: for action in actor.actor_actions.all():
   ....:     print action.pk, action.target, action.action_object
   ....:
7 Flowers By Irene None
6 None Central Perk
5 Amyth Arora None
3 Amyth Arora None
1 None None

So you see for action object 7, the target exists and I can see it in the shell and for action object 6, target is none but action_object exists. But in the template for actions 6,7 both target and action_objects is none.

Same issue here, in the same context (using uuids as primary keys). This kind of issue is frequent in django extensions, uuids are not a smooth ride.

Word for word the same problem i'm facing only with models that have UUIDs as PKs.

Yup this happens to me too

Are you guys using django polymorphic in your projects?

are you facing it while using django-polymorphic with act-stream?

Yes, but it's a polymorphic bug.
In class GFKQuerySet, method fetch_generic_relations the queryset returned by
qs.prefetch_related(*[g.name for g in gfk_fields])

sets all the three generic relation to None.
django-polymorphic has some strange behaviours when using prefetch_related (eg. https://github.com/django-polymorphic/django-polymorphic/issues/68).
A workaround is to set the FETCH_RELATIONS setting to False.

It's not only django-polymorphic's bug, it's probably bug related to django itself, @oaubert have already mentioned prefetch_related is not working when using UUID as pk. This what i found so far – https://code.djangoproject.com/ticket/24912

Does anybody know why django still have this bug for postgres, but for non-postgres  db it was fixed?

I've created ticket for django for that case – https://code.djangoproject.com/ticket/30343#ticket.

thanks for reporting timur!! the issue is fixed in Django https://github.com/django/django/pull/11211

@auvipy Even though the change is made in Django, based on a brief look at the code, it appears as though the Action model would need to be changed to accomodate UUIDs. I ended up using UUID's as secondary keys for my models, and having actstream hook up with my standard autoincrement pks. If you want to use UUID's and actstream, this is probably the best you can do until the code fixes are in place. Unfortunately I am not competent enough to make said code changes.


Main image is not a valid URL

Trying to cross-post from hashnode to dev.to, but received the following error:
Error occured while cross posting to DEV: Main image is not a valid URL

The image is base64 encoded: Error: Could not load img: "data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw=="

Thank you for reporting this. I guess the approach should be to detect if an image URL is a data URL and then get a URL for it. Another approach would be to allow an option in the CLI to enter an image URL to be used instead of scanning the article for the image.

Thank you so much for the great tool! I had gotten this problem as well. I had received these errors
Error: Could not load img: "data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw=="

and
Error occured while cross posting to DEV: Main image is not a valid URL

I suppose that for now, I'll be posting on Dev.to and crossposting to Hashnode. Thank you again.

Hello @sjkorvin thank you for your comment on this. Unfortunately, I haven't been able to get into this due to my busy schedule, but I'm aware of what the problem is.
The tool currently only takes the src of the first image it encounters in an <article> element. Then, that src is used as the URL of the image when posting to the different platforms. It does not check what the type of image is, however, DEV is very particular about the type of images accepted, which causes the error.
I'm hoping that soon either someone will want to be assigned to work on this or I'll get some time to look into it. Maybe adding a few options here and there would more helpful, as well.

This issue has now been resolved by providing different options to the tool. You can check the README for more info


Translation string for keyboard shortcuts for search operators

It would be helpful for translation of certain languages to have the red text be a variable within the translation string.
Keyboard shortcuts modal, for search operators

Found in this file https://github.com/zulip/zulip/blob/master/templates/zerver/app/keyboard_shortcuts.html#L159
Related discussion topic: https://chat.zulip.org/#narrow/stream/58-translation/topic/keyboard.20shortcuts.20for.20search/

Thanks for the report!  I can confirm the strings are not constructed correctly.
We may want start with porting this template to handlebars, like we already did for the "message formatting" tab of this widget, since our frontend i18n library has a better syntax for passing HTML elements into translated strings, which we'll want to do for the highlighting here.

@zulipbot claim.
(Will be relocating the other modals as well to /static/template folder)


[UI] button to expand the menu is not visible when the menu is minimised

Current Behavior

The button to expand the menu is not visible when the menu is minimised as shown below.
The users have to scroll to find the button to expand the menu.

Desired Behavior

The button to expand should be visible and users should be able to click it without having to scroll.
Environment

Browser: Chrome
Host OS: Mac
Meshery Server Version: edge
Meshery Client Version: edge
Platform: Kubernetes

Contributor Guides and Resources

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@sudo-NithishKarthik can I do this issue? Never done anything on meshery UI 😅

IMPLEMENTATION:
Make the chevron fixed at the bottom right corner and increase the z-index

alrighty @sudo-NithishKarthik


[BUG] Randomly using double movement on non-difficult terrain

Describe the bug
While Drag Ruler (Traumi's Patch) [1.12.13], Enhanced Terrain Layer [10.4] and Terrain Ruler [1.7.1] active tokens randomly need twice the movement for non-difficult terrain squares.
To Reproduce
Steps to reproduce the behavior:

Go to Configurations and to Enhanced Terrain Layer.
Check "Include Live Tokens" and "Include Dead Tokens" to Any.
Test dragging a token over the canvas. It counts difficult terrain but randomly also uses double movement on non-difficult terrain...

Expected behavior
Dragging over the canvas should only count as difficult terrain, when moving over difficult terrain, dead or live tokens.
Screenshots

Please complete as much of the following information as possible:

Chrome
Foundry Version 10 (Build 288)
System D&D 5e (2.0.3)
Drag Ruler (Traumi's Patch) [1.12.13], Enhanced Terrain Layer [10.4] and Terrain Ruler [1.7.1]
see the errors in console in the screenshot. Unfortunately, i couldn't make a screenshot of the bug itself.
Error only occurs when "Include Live Tokens" and "Include Dead Token" are active. Tested both in new and existing worlds...

I believe this is fixed with the latest version.  If it is not, please re-open this ticket.


Set requiredSlipSpeed default value to 0.1 to avoid slipping while still.

Fixes #2569
Sets the default slip speed to 0.1 in order to fix the character slipping if a SlipperyComponent spawns under its feets while not moving.

Unexpectedly easy fix, nice.

Nicely done! I guess this is a big "oops" since I should've set the required slip speed in the slippery prototypes, buut...


Virtual Overloaded Problem about qpoases in balance control

Happy new year.
I am trying to write my own controller, but when I intend to include the header files that are related to BalanceController and qpoases, I encounter virtual overloaded problems which happen in the third- party codes instead of mine. Like this,

I could add the "-Wno-overloaded-virtual" flag to the Cmakelist, and the whole building progress ran smoothly. But when I activated the simulation and my own controller, the simulation crashed with segfault.

Did you guys meet the same problem when writing the codes about balance standing or trotting? If so, how did you solve it?

I don't know for sure, but based on the error message, I'm guessing you have some bug related to a std::vector (or our vector class for eigen stuff) in your code.  I would guess it is unrelated to the virtual override warning.
You could try using a debugger to find out exactly where it is crashing - I believe debug symbols are enabled by default in our build configuration.
You may also find these useful:
https://eigen.tuxfamily.org/dox/group__TopicStlContainers.html
https://eigen.tuxfamily.org/dox/group__DenseMatrixManipulation__Alignement.html

Thank you for the response, I will give it a try.


Move trait definition into ty module.

This PR is one of several in a process to address https://github.com/FuelLabs/sway/issues/2951. This PR moves definition of trait to the ty module. Currently, things are a little awkward organizationally, as the definitions and impl blocks are in different modules, but this is done in (quick) iterations to reduce merge conflicts overall.

Closing in favor of #3024.


Proposal - Vuego: a WASM-based Vue.js wrapper written in Go

What problem does this feature solve?
This is a very unusual feature request for Vue.js.
I propose a sub-project for Vue.js, called Vuego. Vuego is a WASM-based Vue.js wrapper written in Go.
It uses the Go 1.11  Web Assembly compiler to compile the wrapper.
The immediate benefit of Vuego is that it allows Vue.js users to deliver their Web applications in binary (Web Assembly) forms. So your companies would have a better way to protect IPs for Vue.js applications.
The project is just started and I would love to see if Vue.js maintainers and users want to have a look. In eventually, if you're OK with the concept, the Vuego project will be donated as a subproject of Vue.js.
https://github.com/chanwit/vuego
What does the proposed API look like?
Something like this:
	New(&Vue{
		El: "#app",
		Data: &Data{
			Items: []*Item {
				{Text: "Bananas", Checked: true},
				{Text: "Apples", Checked: false},
			},
			Title: "Vuego Framework 0.1",
			NewItem: "",
		},
		Methods: &Methods{
			"addItem": func(this interface{}) {
				this.(*Data).AddItem()
			},
		},
	})

Hey @chanwit , thanks for sharing the project. This is an interesting idea for sure, although it seems the main benefit is just IP protection?
In terms of WASM, we are more curious about leveraging it for performance improvements (either runtime or SSR). While IP protection may be a topic that particular user groups are interested in, unfortunately the team currently does not have the bandwidth or expertise to provide meaningful feedback / collaboration on this project. We appreciate your effort though, and would encourage you to keep exploring the idea.

Thank you for your reply @yyx990803.

In terms of WASM, we are more curious about leveraging it for performance improvements (either runtime or SSR).

SSR would be greatly simplified as Go already has a lot of server-side stuffs to help archive this, including Go templates, or protobuf + gRPC over WS, for example.
The runtime performance would be a bit challenging as it requires to port the whole Vue.js to WASM. It's also a midterm goal of the project too.

I surely continue to explore in this space. Thank you for your encouragement. Hopefully we could collaborate somehow in the future when the project becomes more mature.


Migrate stable types from Generator Shared source to public types in Azure.Core

This tracks the work to migrate internal shared source from autorest.csharp repo into azure-sdk-for-net repo.
Additionally, we may move some of them into the public Azure.Core API as part of the related efforts in https://github.com/Azure/azure-sdk-for-net/issues/28369.
Files live in autorest.csharp/feature/v3/src/assets/Generator.Shared

Per @m-nash, please involve generator team in any decisions here.

Address Pageable helpers as part of this move, per https://github.com/Azure/azure-sdk-for-net/issues/9896


Tritium

any news about the next Nexus wallet update?
atm when u lose connection the trust built will start over from 0.5 %
hope that this issue is addressed

That's one thing that's addressed, yes


testcentric: Change download URL to Choco

Github still hosts the .nupkg. Why the switch?
Please note a .zip file is also provided. See https://github.com/TestCentric/testcentric-gui/blob/master/README.md#installation

It is recommanded by author, and switching to Choco will have extracted file hash so as to provide better integrity check.

Have we asked the chocolatey admins if we can do this? I wonder if their terms of service allow this use? We are a competitor of sorts, and they offer a paid service.

I never heard back from Rob @ Chocolatey.


add link to manage stripe account on bank accounts page

It would be something like this: https://dashboard.stripe.com/acct_1JWtL8RJfXdDzvgQ/
They can update payout method, etc here.

Allows people to see their transactions

Closed by https://github.com/OdyseeTeam/odysee-frontend/pull/2291


fix auth token

#4870  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


Add global Stanford header to all pages

Note: This PR assumes that the related Stanford Footer PR has already merged;
this depends on CSS that it adds to all pages.

@caesar2164 It's basically just symlinks/includes...
Note: the styling looks weird here, how the top Stanford logo doesn't really line up like it does on the LMS; I hacked a line or two of CSS and then stop putzing with this.
You're going to need some custom styling to make this look nicer.

This depends on https://github.com/Stanford-Online/edx-platform/pull/875 and #70

Closing in favor of https://github.com/Stanford-Online/openedx-themes/pull/73


I18n fields: first getAttribute is empty

I have several actAs I18n definitions in my schema.yml
Article:
  tableName: article
 actAs:
    I18n:
      fields: [name]
 columns:
    name: string(255)

in my schema.yml
the first call for those i18n attributes are always empty, it works from the second call.
$article = Doctrine::getTable('Article')->find(1);

$artricle->getName(); // first call returns ""
$article->getName(); // second call returns "foo"
$article->getName(); // and all further calls return "foo"

the code worked fine with symfony 1.4.20

This repo isn't really maintained. You might get some help in friendsofsymfony1/symfony1.


Tutorial won't compile:   Meet The Composable Architecture - Testing your Feature, Testing Effects, Step 7 has an error

Description
The tutorial has an error on line 14.   This won't compile for me with Xcode 14.3.1 despite the store defined on line 7 being an instance of TestStore.
You can see the autocompleted doesn't provide the expected receive method.  When I paste it in the code from the tutorial the test won't compile

This is a screen grab of the tutorial:

Checklist

[X] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
Copying and pasting code from a tutorial should compile
Actual behavior
Multiple compiler errors
This screen grab shows if you simply instantiate a test store with TestStore() then autocomplete works and the test compiles

But using the store constant doesn't work. 🤷♂
The root cause appears to be that the tutorial doesn't tell us to add the following line to the CounterFeature.swift file.
extension CounterFeature.Action: Equatable {}

this fixes the problem
I got lucky and got an error about Action needing to be equatable and remembered the trick the tutorial required to make State equatable.
But this is a frustrating experience and difficult to resolve.
Steps to reproduce
Copy paste tutorial example code, try to compile, observe compiler error
The Composable Architecture version information
0.55.0
Destination operating system
ios 16.3
Xcode version information
14.3.1
Swift Compiler version information
awolf@Lins-iPhone-5s ios % xcrun swiftc --version
swift-driver version: 1.75.2 Apple Swift version 5.8.1 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>.100)
Target: arm64-apple-macosx13.0

Hi @awolf-foreflight, a few steps before the step you highlight it was mentioned that the actions need to be made Equatable:
https://github.com/pointfreeco/swift-composable-architecture/blob/af16494b9e6ccd306605aa328f5deea3d75cd33a/Sources/ComposableArchitecture/Documentation.docc/Tutorials/MeetTheComposableArchitecture/01-Essentials/03-TestingYourFeatures/01-03-TestingYourFeature.tutorial#L157-L158
Maybe this could be improved, but it's not really an issue with the library so I am going to convert this to a discussion where we can further discuss ways to improve this.


switch to new version of industrial_ci

https://github.com/ros-industrial/industrial_ci/issues/390

https://github.com/ros-industrial/abb_librws/pull/56/commits/af1639f04b9bddf3692eac6cc991d35ce84e893b could get reverted later

@ipa-mdl: would you recommend we merge this PR in here?
Are you satisfied with the stability of the new version or are there changes you'd like to integrate before switching over?

would you recommend we merge this PR in here?

for now it is a beta version, it's up to the maintainer

Are you satisfied with the stability of the new version

I would say it is stable, I don't expect any breaking changes.

are there changes you'd like to integrate before switching over?

I'd like to refactor it even further, but mostly under the hood.

Some of this seems to be included in #69.
After merging that PR, we should see which parts of this one are still relevant.

Seems the changes in this PR were merged in parts by #57 and #69.
We're using master of industrial_ci here now.
Thanks @ipa-mdl for the initial PR and thanks for your continued support for industrial_ci :+1: :hamburger: :beers:


Present the Terms of Service in a scrollable window for users when creating an account

1. What's the name of the policy?
https://github.com/github/site-policy/blob/master/Policies/github-terms-of-service.md
2. Is this issue related to a specific section within one of our policies (e.g. the Terms of Service)? If so, please include a link to the section or subsection.
Related to the account creation process, not the contents of the document.
5. Why do you think this section or language needs improvement?
Currently, during the account creation process users do not actually have to acknowledge that there is a Terms of Service and they agree by clicking a button or checkbox (there is just text that says by clicking sign up you agree). It would be beneficial for legal purposes for GitHub to make this change and make users confirm their agreement, but it would also give a better opportunity for users to look at the Terms of Service. Rather than having to find the page, it would be put in front of them automatically, increasing their likelihood of reading the terms.

Thanks for the suggestion, @eknox6.
We'll give that some thought. However, the site-policy repository is intended for discussions about the content of the policies, not for proposals about additional functionality for GitHub. At this time, we don't have the ability to create new pop-up windows for the Terms of Service on GitHub.


Packet Injection Tools Not Working

I have been trying to use your FW image, with your branch of Killerbee, but have been running into some issues while trying to run the active tests.
ZBReplay and ZBStumbler works as intended and I am able to retransmit packets that have already been captured. However, when I try to run ZBAssocflood I am met with this error:

ZBPANIDConflict continuously gives me the error that the resource is busy, but the dongle can handle running zbid, zbwireshark, and zbstumbler.
Aside from that, the passive tools all seem to be working correctly aside from zbdump. I can run zbdump but none of the packets are collected in the .pcap file.
I would really appreciate any help I could get!
I am running:
Ubuntu 20.04
Python 3.8.10
PyUSB 1.0.2-1build1
Installed Killerbee from your branch github

RiverloopSec merged my branch and have improved Python3 code since then, I guess it may worth it to give their main branch a try as killerbee is now using Python 3 and is compatible with this Bumblebee firmware :)


Rooted UrlDispatcher / Mini Applications

Currently, each Application has a single router instance (UrlDispatcher by default). This works fine for small applications.
Express (of node.js) has the convenient router class which expands on this functionality. Specifically, Express allows the user to create "mini applications" which can be assigned (re-rooted) to a given path. I think that aiohttp could benefit from similar functionality.
That is, instead of having the following calls:
    app.router.add_route('GET', '/blog/posts', posts_handler)
    app.router.add_route('POST', '/blog/authors', authors_handler)
    app.router.add_route('*', '/blog/about', about_handler)
    ...

One could have something like a rooted UrlDispatcher / mini application:
    Class Blog(RootedUrlDispatcher):
        def __init__(self):
            super().add_route('GET', '/posts', self.posts_handler)
            super().add_route('POST', '/authors', self.authors_handler)
            super().add_route('*', '/about', self.about_handler)
            ...

and then later simply:
    app.router.add_router('/blog', Blog())

Note that all the calls to /blog/* will be forwarded to the Blog class. Thus one can keep all the resources (e.g., database, template files) belonging to the blog inside the Blog class. Also, one can easily re-root the entire Blog "mini application" by changing only a single line of code.
The RootedUrlDispatcher is just a suggestion. Any way of implementing mini applications would be a nice addition to aiohttp.

I thought about problem like this for a long time.
Latest aiohttp 0.21 is one step closer to solution.
Eventually I'll implement nested applications.
Stay tune.

@asvetlov Thanks for the quick reply! I'm glad to hear that you are already working on a solution. Looking forward to the next couple of releases.
I'd like to add that if the user could modify the Request object (#682) then the user could implement this feature himself.

Do you mean request.path replacing?
I doubt if it good idea, better to support nested routes with unmodified paths given from request.
Otherwise printing debug info with unprefixed paths makes a mess.

Do you mean request.path replacing?
Yes, that is what I had in mind.

I doubt if it good idea, better to support nested routes with unmodified paths given from request.
I completely agree. I'm just impatient and want a workaround. It's a much better idea to support nested routes directly (and leave the request object immutable).

Hi, it would be great to have ability to include many routes in one (like in Django :).
I tried to write my own realization of this feature (https://github.com/Gr1N/wuffi/blob/master/wuffi/core/urls.py#L14):
class UrlDispatcher(web_urldispatcher.UrlDispatcher):
    def include_routes(self, path, router):
        if not path.startswith('/'):
            raise ValueError('path should be started with /')
        elif path.endswith('/'):
            raise ValueError('path should not be ended with /')

        router = import_string('{}.router'.format(router))

        # include `resources`
        self._resources.extend(self._to_include(path, *router._resources))

        # include `named_resources`
        for name, resource in router._named_resources.items():
            if name in self._named_resources:
                raise ValueError('Duplicate {!r}, '
                                 'already handled by {!r}'
                                 .format(name, self._named_resources[name]))

            self._named_resources[name] = resource

    def _to_include(self, path, *resources):
        for resource in resources:
            if isinstance(resource, web_urldispatcher.PlainResource):
                resource._path = '{}{}'.format(path, resource._path)
            elif isinstance(resource, web_urldispatcher.DynamicResource):
                resource._pattern, resource._formatter = self._compile_path(
                    '{}{}'.format(path, resource._formatter))

        return resources

    def _compile_path(self, path):
        pattern = ''
        formatter = ''
        for part in self.ROUTE_RE.split(path):
            match = self.DYN.match(part)
            if match:
                pattern += '(?P<{}>{})'.format(match.group('var'), self.GOOD)
                formatter += '{' + match.group('var') + '}'
                continue

            match = self.DYN_WITH_RE.match(part)
            if match:
                pattern += '(?P<{var}>{re})'.format(**match.groupdict())
                formatter += '{' + match.group('var') + '}'
                continue

            if '{' in part or '}' in part:
                raise ValueError("Invalid path '{}'['{}']".format(path, part))

            formatter += part
            pattern += re.escape(part)

        try:
            compiled = re.compile('^' + pattern + '$')
        except re.error as exc:
            raise ValueError(
                "Bad pattern '{}': {}".format(pattern, exc)) from None

        return compiled, formatter

@asvetlov what do you think about this solution? To my mind it is very ugly and buggy, but it works for my case. If you have idea how to write it in right way, but have no time for realization, so tell us, maybe we can try to write smth and send PR.

May I ask what the status on this "nested"-route feature is?

I'm working on it.
The nested route is a bit challenge but please keep fingers crossed.
I really want to have the feature in aiohttp 1.1

Fixed by #1301

@asvetlov thanks for realization, as I can see from documentation we need to create additional application and add those application to existing route, to my mind it's a complex solution, what about just creating a new UrlDispatcher and then adding to existing route? Something like this:
router = UrlDispatcher()
router.add_route('*', '/view1', View1)

sub_router = UrlDispatcher()
sub_router.add_route('*', '/view2', View2)

router.add_subrouter('/sub', sub_router)

You solution very useful in cases when we need something big with middlewares, signals, etc. But in many cases we need something simple like in my example.

No. Subapplications are required exactly because app is an container for its global state, signals and middlewares.
If you need just resource grouping you always could write a helper like
def fill_routes(app, prefix):
     app.router.add_get(prefix+'/getter', handler1)
     app.router.add_post(prefix+'/setter', handler2)

fill_routes(app, '/prefix')

Sub-routers make overcomplication.

Looks like solution, but not very useful to my mind. Let's imaging big routing configuration, this is example using django realization, but it pretty clear show what I'm talking about:
url(r'^top/', include([
    url(r'^$', view1, name='view1'),

    url(r'^sub1/', include([
        url(r'^$', view2, name='view2'),
        url(r'^view3/$', view3, name='view3'),
    ], namespace='sub1')),

    url(r'^sub2/(?P<sub2_id>[0-9a-z-]{36})/', include([
        url(r'^$', view4, name='view4'),
        url(r'^view5/$', view5, name='view5'),
        url(r'^sub3/(?P<sub3_id>[0-9a-z-]{36})/', include([
            url(r'^$', view6, name='view6'),
        ], namespace='sub3')),
    ], namespace='sub2')),
], namespace='top'))

It's easy to read and I can reverse urls using: top:sub1:view3 and top:sub2:sub3:view6.
Using your way I need to write additional helpers (sometimes very complex) in each project.
I know aiohttp not a django, but for now routing in aiohttp very simple (and not useful for big projects) and requires improvements to my mind, but maybe I'm wrong.

Are you really asking for nested routes or namespaces for url reversing?
What the principal difference between router.add_subapp() and proposed router.add_subrouter()?

Why I want router.add_subrouter(), for example I have some project:
/users
   views.py
   router.py
/news
   views.py
   routes.py
main.py
routes.py

In top level router I just want something like this:
# routes.py
from users.routes import router as users_router
from news.routes import router as news_router

router = UrlDispatcher()
router.add_subrouter('/users', users_router, name_or_namespace='users')
router.add_subrouter('/news', news_router, name_or_namespace='news')

# main.py
from routes import router
app = Application(router=router)
...

In this case I don't want to create additional apps for news and users modules, because I don't need additional middlewares or signals, I just want to include routes. And yes if we can include routes I want to use namespaces for url reversing.
My proposal is more about project organization and how aiohttp router can help me with this.

Well, I need to sleep on it.
Please create a new issue with your proposal.

Done https://github.com/KeepSafe/aiohttp/issues/1343


Update ucSpinnerCogs.xaml

Changed the bottom cog from 344 to 374, this resolves a popping issue.
I like these, thanks again!

Thanks for the fix!


travis: auto build site

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Rebase before creating a PR to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you PR.

万一被人改了自动往 master 分支推代码就好玩了 🌚


Fix interstitial sprout anchors check

Motivation
There was an issue with the interstitial Sprout anchor check, described in https://github.com/ZcashFoundation/zebra/issues/3236
Specifications

https://zips.z.cash/protocol/protocol.pdf#joinsplit
Designs

Solution

Change state to add the matching note commitment tree with each Sprout anchor
Also add it to non-finalized state (Chain)
When checking the anchor, build the interstitial trees starting from the tree that matches the anchor (and not from the tree of the finalized tip)
Update state documentation

Closes #3236 and #3210.
Review
Not particularly urgent, but it's the final consensus check. Usually @dconnolly and/or @upbqdn could review but @teor2345 might want to take a look.
After accepting but before merging we need to update the cached state in CI (or disable the CI job until we do that).
There is no specific test - I tested by syncing to the tip:

Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

I can regenerate the cached state once someone else approves this PR.

@upbqdn Conrado isn't available this week, so can you push all the changes you think are needed to this PR's branch?
I can re-approve after you're done.

I can regenerate the cached state once someone else approves this PR.

I think it's ready to go, after you make whatever changes you think are needed.
Just checking, are you going to regenerate the cached state, update the image in this PR, then merge it to main?
(That way, the tests won't fail on main, because there's no gap between the version and cached state update.)
The hash in this line needs to be updated:
https://github.com/ZcashFoundation/zebra/blob/454897bc330633246ab2ba714a17c97d8c823bc5/.github/workflows/test.yml#L57

Just checking, are you going to regenerate the cached state, update the image in this PR, then merge it to main?

Yes, exactly. I'm going to follow the steps in the onboarding doc. I refined them recently, so it should be smooth.

The last bit that's missing here is the new state. I will generate it once we fix the other issues.

The last bit that's missing here is the new state. I will generate it once we fix the other issues.

Can you review PR https://github.com/ZcashFoundation/zebra/pull/3168 ?
It might fix the hangs by spawning the syncer as a separate task.
Or we might also need to fix #3234 and #3254.

This PR also partially documents the consensus rules for #3210 and #3211.

The following statement should not be a consensus rule

This PR documents all consensus rules mentioned in #3210 except for the statement above, so it also closes #3210.
See https://github.com/zcash/zips/issues/582 for more details regarding the statement above.

I managed to recreate the cached state, updated it and it works :tada:
(But now CI is failing :cry: , see https://github.com/ZcashFoundation/zebra/pull/3366)

@dconnolly I don't know why it dismissed your review, I just accepted your suggestion :confused:

@dconnolly oops, fixed formatting :sweat_smile:


[Minor] [ML] [Doc] Cleanup dots at the end of some sharedParam

What changes were proposed in this pull request?
Remove duplicated dot at the end of some sharedParams in ScalaDoc.
cc @mengxr @srowen
How was this patch tested?
Documents change, no test.

Test build #51870 has started for PR 11344 at commit a12b1ac.

@yanboliang this isn't worth changing. The period isn't hurting anything

Test build #51870 has finished for PR 11344 at commit a12b1ac.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  final val thresholds: DoubleArrayParam = new DoubleArrayParam(this, \"thresholds\", \"Thresholds in multi-class classification to adjust the probability of predicting each class. Array must have length equal to the number of classes, with values >= 0. The class with largest value p/t is predicted, where p is the original probability of that class and t is the class' threshold\", (t: Array[Double]) => t.forall(_ >= 0))

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51870/
Test FAILed.

Merged build finished. Test FAILed.

@srowen Some ScalaDoc will end with two dots if we don't fix, you can refer here.

Jenkins, test this please.

Test build #51881 has started for PR 11344 at commit a12b1ac.

Test build #51881 has finished for PR 11344 at commit a12b1ac.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  final val thresholds: DoubleArrayParam = new DoubleArrayParam(this, \"thresholds\", \"Thresholds in multi-class classification to adjust the probability of predicting each class. Array must have length equal to the number of classes, with values >= 0. The class with largest value p/t is predicted, where p is the original probability of that class and t is the class' threshold\", (t: Array[Double]) => t.forall(_ >= 0))

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51881/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please.

Test build #51970 has started for PR 11344 at commit a12b1ac.

Test build #51970 has finished for PR 11344 at commit a12b1ac.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  final val thresholds: DoubleArrayParam = new DoubleArrayParam(this, \"thresholds\", \"Thresholds in multi-class classification to adjust the probability of predicting each class. Array must have length equal to the number of classes, with values >= 0. The class with largest value p/t is predicted, where p is the original probability of that class and t is the class' threshold\", (t: Array[Double]) => t.forall(_ >= 0))

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51970/
Test PASSed.

Merged build finished. Test PASSed.

May not be worth it, but I think it's fine to fix.  LGTM  I'll merge this with master


Trying on Hackintosh OS X 10.11.6

I disabled system integrity protection (info) and ran from sudo, but I still get the "try from sudo" error. Any ideas what I might check?
The gui does come up, btw.

This is not designed or supported for non Linux systems. The application does not find the needed mapper to the USB HID device.

So no thoughts on where I might look to patch/hack it?

You would need to get a list of the HID devices attached to the machine.
http://stackoverflow.com/questions/6477385/raw-access-to-hid-devices-in-os-x

Of course, yes. I can look into that further... not insurmountable, by any mean... Can you point to a particular part of your code that deals with the device Ids?
Much appreciated.

The main line for retrieving and initiating the keyboard is here: https://github.com/stevelacy/msi-keyboard/blob/master/lib/findKeyboard.js#L7
And sending the feature report to the hid device: https://github.com/stevelacy/msi-keyboard/blob/master/lib/setColor.js#L24

Right, okay. So you instantiate an HID object with the Ids... I'll see what I can do. Thanks!

Quick question, could you do this:
var HID = require('node-hid');
var devices = HID.devices();
console.log(devices);

And post the output?

[ { vendorId: 1452,
productId: 782,
path: 'IOService:/AppleACPIPlatformExpert/PS2K/ps2controller/ApplePS2Controller/ApplePS2KeyboardDevice/ApplePS2Keyboard/IOHIDKeyboardDevice',
serialNumber: '',
manufacturer: 'Apple',
product: 'Keyboard',
release: 0,
interface: -1,
usagePage: 1,
usage: 6 },
{ vendorId: 1452,
productId: 613,
path: 'IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC@14/XHC@14000000/HS01@14100000/Magic Trackpad 2@14100000/Actuator@2/AppleUSBTopCaseHIDDriver@14100000,2',
serialNumber: 'CC260450KWDG61CA0',
manufacturer: 'Apple Inc.',
product: 'Magic Trackpad 2',
release: 2052,
interface: -1,
usagePage: 65280,
usage: 13 },
{ vendorId: 1452,
productId: 613,
path: 'IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC@14/XHC@14000000/HS01@14100000/Magic Trackpad 2@14100000/Trackpad / Boot@1/AppleUSBTopCaseHIDDriver@14100000,1',
serialNumber: 'CC260450KWDG61CA0',
manufacturer: 'Apple Inc.',
product: 'Magic Trackpad 2',
release: 2052,
interface: -1,
usagePage: 1,
usage: 2 },
{ vendorId: 1452,
productId: 613,
path: 'IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC@14/XHC@14000000/HS01@14100000/Magic Trackpad 2@14100000/Accelerometer@3/AppleUSBTopCaseHIDDriver@14100000,3',
serialNumber: 'CC260450KWDG61CA0',
manufacturer: 'Apple Inc.',
product: 'Magic Trackpad 2',
release: 2052,
interface: -1,
usagePage: 65280,
usage: 3 },
{ vendorId: 1452,
productId: 782,
path: 'IOService:/AppleACPIPlatformExpert/PS2K/ps2controller/ApplePS2Controller/ApplePS2MouseDevice/ApplePS2SmartTouchpad/IOHIDPointingDevice',
serialNumber: '',
manufacturer: 'Apple',
product: 'Trackpad',
release: 4,
interface: -1,
usagePage: 1,
usage: 2 },
{ vendorId: 1452,
productId: 613,
path: 'IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC@14/XHC@14000000/HS01@14100000/Magic Trackpad 2@14100000/Device Management@0/AppleUSBTopCaseHIDDriver@14100000,0',
serialNumber: 'CC260450KWDG61CA0',
manufacturer: 'Apple Inc.',
product: 'Magic Trackpad 2',
release: 2052,
interface: -1,
usagePage: 65280,
usage: 11 } ]

The keyboard is not listed. What MSI laptop is this?

Yeah, it's a Ghost Pro GS60 6QE, but I think I gave up on the backlight control when it appeared that the Hackintosh uses ApplePS2Keyboard.kext to run it... so it's not USB as far as the system is concerned. Honestly, I don't understand what's up with that. Maybe some kind of emulation?... not sure.

Nice, I have that laptop as well. It sounds like apple uses a keyboard interface for all raw hid devices with a keyboard type.
If your keyboard stays on after reboot you could turn it on in linux or windows and then reboot into apple. Quite a hassle, but it might work

hmm... can't remember, but I think it might stay on. Will check it out at some point. Thanks for looking into it.

Thats good, can you write up a install/setup doc?

The gui version doesn't work with OS X (I'm looking into it), but msi-keyboard is.

I'm guessing it's due to compiling for a different version of node, or node-hid mismatch.

so far, it's because of cp for mac being a bit different than it is for linux, where there is no -t option.

yup, its a node-hid mismatch

Tested the latest changes on macos :smile:
Let me know if the latest works for you, if not I'll re-open

msi-keyboard works on MacOS by using SmartTouchPad
This does not affect anything thats related to the colors since MSI has the rgb colors separated into this USB called "MSI EPF USB". Also, I used the latest release of msi-keyboard-gui but the colors don't change. The "node-hid mismatch" still happens. I will see if I can fix for macOS. I'm on node 8.14.1.

node-hid mismatch" still happens

Nevermind. It does not say "node_module_mismatch" anymore, but the colors does not change.


integration-test iso5/master timeout

The master and iso5 builds for integration-tests/performancetest fails often with a timeout.

I believe this is related to #4059 and it should be fixed once the iso5 release is out. I'll leave this ticket open for now so I can double check afterwards.


404 at /desktop/mac/release-notes/archive/undefined

I found a broken link : https://docs.docker.com/desktop/mac/release-notes/archive/undefined
I arrived on this page through: https://docs.docker.com/desktop/mac/release-notes/archive/

Looks the same as https://github.com/docker/docker.github.io/issues/13553 (and some other reports), but so far we haven't been able to find the cause for these broken links (it may be a JavaScript issue)
Do you know what link you clicked on that page? What browser are you using, and do you have any browser extensions installed?

Closing as duplicate off https://github.com/docker/docker.github.io/issues/13553


New Touch Party Game (Europe) missing title screen

from Desmume site
OK, here's the scoop. This game seems to be achieving the raster FX on the logo by rendering the 72px tall logo with 72 polys, one for each line. This brings out a typhoon of sub-pixel precision issues which can vary between opengl devices, and cause some lines to get distorted or vanished seemingly randomly. This is not going to get fixed any time soon, it will need to be a test case for future work on major precision improvements in the softrasterizer. Also note that it might possibly never work in HD due to it being unclear how 1px details drawn possibly with <1px line heights should be scaled.


Question: Is it possible to trigger the call back when the app is in terminated state?

I want to detect incoming & outgoing calls, even when the app is in the terminated state?
How do I do it?

Same problem is here

@sodipto any solutions are there?


Refactor wrapSymbolTrackerToReportForContext to improve perf

In the same vein as #51682, this refactors a function that was extremely polymorphic/megamorphic that seems to account for a significant percentage of emit time in some of our benchmarks.

@typescript-bot perf test

I merged this into #51682, while testing for knock-on effects from the two changes. I'm going to close this PR in favor of the combined version in #51682.


[charts] bmrg-flow-apps-flow ingress definition is invalid

Describe the bug
bmrg-flow-apps-flow ingress template is invalid. annotations.kubernetes.io/ingress.class and spec.ingressClassName cannot be used at the same time
What project does this issue relate to?
Boomerang Flow
To Reproduce

Have a Kubernetes platform
Deploy bmrg-flow-4.13.3 helm chart

Environment
Kubernetes platforms used:

k3d (k3s versions 1.21 and 1.22)
IBM Kubernetes Service v1.21

Additional Contenxt
No response

Hey @aiciobanu I checked with @morarucostel and @gchickma on the removal of the annotation in the last release 3.10. It was determined it was no longer needed to be supported as support for the annotation has been dropped many versions before, and I think has been supported since Kube 1.19 or 1.21
It would be preferable to not have to have support for it.

@tlawrie Closed the first PR and made a new one that removes the annotation


New version: Zoom.ZoomRooms version 6.1.0

Pull request has been created with komac v2.3.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-07-10 10:35:44.818 [CLI ] ShellExecute installer failed: 1
Installation failed with exit code -1978334957
2024-07-10 10:29:45.646 [CLI ] ShellExecute installer failed: 1633
MSI (s) (00:BC) [10:29:45:263]: Product: Zoom Rooms Installer -- Installation failed.
This installation package is not supported by this processor type. Contact your product vendor.

Automated error analysis suggests -1978335226 may mean Install successful but system restart may be required.
Automated error analysis suggests 1633 may mean ERROR_INSTALL_PLATFORM_UNSUPPORTED - This installation package is not supported by this processor type. Does this manifest have Architecture: x86? It might need to be x64.
(Automated response - build 886.)

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-07-10 10:35:44.818 [CLI ] ShellExecute installer failed: 1
Installation failed with exit code -1978334957
2024-07-10 10:29:45.646 [CLI ] ShellExecute installer failed: 1633
MSI (s) (00:BC) [10:29:45:263]: Product: Zoom Rooms Installer -- Installation failed.
This installation package is not supported by this processor type. Contact your product vendor.

Automated error analysis suggests -1978335226 may mean Install successful but system restart may be required.
Automated error analysis suggests 1633 may mean ERROR_INSTALL_PLATFORM_UNSUPPORTED - This installation package is not supported by this processor type. Does this manifest have Architecture: x86? It might need to be x64.
(Automated response - build 886.)

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-07-10 10:35:44.818 [CLI ] ShellExecute installer failed: 1
Installation failed with exit code -1978334957
2024-07-10 10:29:45.646 [CLI ] ShellExecute installer failed: 1633
MSI (s) (00:BC) [10:29:45:263]: Product: Zoom Rooms Installer -- Installation failed.
This installation package is not supported by this processor type. Contact your product vendor.

Automated error analysis suggests -1978335226 may mean Install successful but system restart may be required.
Automated error analysis suggests 1633 may mean ERROR_INSTALL_PLATFORM_UNSUPPORTED - This installation package is not supported by this processor type. Does this manifest have Architecture: x86? It might need to be x64.
(Automated response - build 886.)

Edit: Sorry for spam

All good, I accidentally left the 86 installers which have been removed on this version going forward.

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-07-11 00:21:04.980 [CLI ] ShellExecute installer failed: 1

Automated error analysis suggests -1978335226 may mean Install successful but system restart may be required.
(Automated response - build 886.)

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-07-11 00:21:04.980 [CLI ] ShellExecute installer failed: 1

Automated error analysis suggests -1978335226 may mean Install successful but system restart may be required.
(Automated response - build 886.)

I think previous updates had this issue to?


installation error!

When i install "react-native-custom-switch" on my android device showing error like this,
2 | import { View, StyleSheet, Text, TouchableWithoutFeedback, Animated, LayoutAnimation, Platform, UIManager } from 'react-native';
3 |

4 | import hexToRgb from './hexToRgb';
|                       ^
6 | if (Platform.OS === 'android') {
7 |   if (UIManager.setLayoutAnimationEnabledExperimental) {
at ModuleResolver.resolveDependency (/home/nachu/Desktop/todo_app/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:137:15)
at DependencyGraph.resolveDependency (/home/nachu/Desktop/todo_app/node_modules/metro/src/node-haste/DependencyGraph.js:260:43)

and i rebuilded my app and cleared cache with npm start --reset-cache command, still showing this error
package-version = "^1.0.7"

I am getting the same error, looks like missing dependency. yarn add hex-rgb didn't help.

https://github.com/arshigtx/react-native-custom-switch/blob/main/src/customSwitch.js#L4
import hexToRgb from './hexToRgb';
I think it should be
import hexToRgb from './utils';

Guys, this lib is abandoned, someone published a fixed version of this:
https://www.npmjs.com/package/react-native-custom-switch-new


Can Percona Everest work for SQL Server?

Hi Community,
Our company using SQL Server and its a pain to maintain it on Prem.
We are wondering if its possible to use Percona Everest with SQL Server?
Thank you!

Hello @btzq - there are no such plans. We at Percona are focused on open source technolgoies, whereas MSSQL is a great database, but it is far from being open source. Also Everest is based on Kubernetes, running MSSQL on k8s would be a pain (at least I don't see any shortcuts).
Please let me know if you still have questions.


chore: bdd-spring-1575383435 to 0.0.1

chore: Promote bdd-spring-1575383435 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Format code using 'cargo fmt'

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

Merging this in #215.


Rename [tuplN] into [tupleN] because it's nicer

With all the respect I have for https://github.com/coq-community/coqffi/commit/a21b10db55a5ae47ffc14f3826693bfd6a2bfa6d. tuplN is not good

:ok_hand:


A proposal to alter the builder order

Summary
A proposal to alter the builder order and move Node.js lower.
Use Cases
This prevents Node.js buildpack from incorrectly detecting true for applications that also bundle frontend code with backend code.
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Personally, I'd like to do a bit more testing of the updated order before merging this in – in particular, run against more sample apps and test a reordering of the full builder, not just the base builder.
But on a meta level, I also wonder: how much are users relying on the builders rather than defining their own? Which types of users are relying on the builder? The answer to these questions will influence how important it is to support the frontend-backend combo use-case described here and how much we should actually worry about breaking other users' builds.

@dmikusa-pivotal This sounds reasonable to me. In addition, it seems to me that most Paketo users, right now, are using CNB platforms other than kpack, which has its own builder abstraction.
I'd still like to see more validation that the proposed reordering is minimally breaking for the types of apps we already support.

If we're going to re-order the base and full builders, should we cut minor releases upon this change?


Document undocumented declarations

Strange, I'm able to comment on old threads that reference commits that have been force-overwritten.

You're able to comment on them at the top level of the conversation view, but if you try to reply to multiple at once (by clicking into the file above the comment) you get this:

Noted, will favor multi-commit and squash-merge in the future 👍


Refactor add_track/remove_track

From Sudara:
+1 to removing the model callback though! We can do that in add_track/remove_track (which ideally get moved into a new tracks_controller as create/destroy as those actions haven't been updated since REST was a thing!!).

@sudara, thoughts, considering that currently all of our tracks' behavior is handled through assets_controller per routes:
    resources 'tracks', controller: :assets do
      member do
        get :share
        get :stats
      end
      collection do
        get  :latest
        post :search
        get  :mass_edit
      end
      resources :comments,  only: [:create, :destroy]
    end

should we move all actions that make sense to new tracks_controller or come up with another way?

Correct me if I'm wrong, but I think what you are asking is "we already have a tracks route, representing everything that happens on an asset under the hood. Wouldn't a tracks_controller conflict with that?"
Let's continue this discussion in your other issue #882 and close this one out. You can see my comment for suggested controller tweaks.


how to set a global traffic permission so that k8s namespace pods cannot access k8s namespace pods

how to set a global traffic permission so that k8s namespace pods cannot access k8s namespace pods ，pods of A k8s namespace cannot access pods of B k8s namespace 。how to set in k8s yml file

apiVersion: kuma.io/v1alpha1
kind: TrafficPermission
mesh: default
metadata:
name: allow-all-traffic
spec:
sources:
- match:
kuma.io/service: ''
destinations:
- match:
kuma.io/service: ''

Closing this, you should be able to use autogenerated k8s tags with TrafficPermission.


calling yaml.load() without Loader=... is deprecated

Hi Andrei,
Thanks a lot for your code and videos :)
when i executed the main.py i got below error. could you please help me here.
I downloaded the file from --> https://github.com/CyberPunkMetalHead/binance-trading-bot-new-coins
Hope this the latest code.
After download i only changed  binance_tld:  to "COM"
Error as below:
C:\Anaconda\envs\api_test\lib\site-packages\dateparser\conf.py:45: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
cls.yaml_data = load_yaml(data).pop('settings', {})
C:\Anaconda\envs\api_test\lib\site-packages\dateparser\languages\loader.py:38: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
data = load_yaml(data)
Traceback (most recent call last):
File "main.py", line 4, in
from new_listings_scraper import *
File "C:\ADM\Test_binance\binance-trading-bot-new-coins-main\binance-trading-bot-new-coins-main\new_listings_scraper.py", line 10, in
from send_notification import *
File "C:\ADM\Test_binance\binance-trading-bot-new-coins-main\binance-trading-bot-new-coins-main\send_notification.py", line 4, in
config = load_config('config.yml')
File "C:\ADM\Test_binance\binance-trading-bot-new-coins-main\binance-trading-bot-new-coins-main\load_config.py", line 5, in load_config
return yaml.load(file, Loader=yaml.FullLoader)
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml_init.py", line 114, in load
return loader.get_single_data()
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\constructor.py", line 49, in get_single_data
node = self.get_single_node()
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\composer.py", line 36, in get_single_node
document = self.compose_document()
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\composer.py", line 55, in compose_document
node = self.compose_node(None, None)
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\composer.py", line 84, in compose_node
node = self.compose_mapping_node(anchor)
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\composer.py", line 127, in compose_mapping_node
while not self.check_event(MappingEndEvent):
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\parser.py", line 98, in check_event
self.current_event = self.state()
File "C:\Anaconda\envs\api_test\lib\site-packages\yaml\parser.py", line 439, in parse_block_mapping_key
"expected , but found %r" % token.id, token.start_mark)
yaml.parser.ParserError: while parsing a block mapping
in "config.yml", line 2, column 3
expected , but found ''
in "config.yml", line 18, column 22

This package issue was automatically fixed, once i created a new environment. Thanks !!


[BUG] Faucet request to substrate address fails

Problem :
Requesting tokens on faucet for tangle with a substrate address fails with error
Example :

Debugging output:

Fixed


version number not correct

Hello,
it seems the version number for the August 2019 Update (2.71.5523.641) release as stated on this page is the same as the version number for the July 2019 Update (2.71.5523.641) on page https://docs.microsoft.com/en-us/power-bi/desktop-latest-update-archive. We are taking these version numbers to report on in our Power BI  reporting. The duplicates are causing us a problem in the reporting. Thanks for rectifying. :-)

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

ID: ba6a76e2-8d94-5e76-bfb6-971c3b2c7c85
Version Independent ID: f7bd2111-3bb0-fb08-0f23-a31efd407f66
Content: What's new in the latest Power BI Desktop update? - Power BI
Content Source: powerbi-docs/desktop-latest-update.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks @lcallewi. I'm assigning to the author to fix. My current install is at 2.73.5586.421, but it's possible that's ahead of the August release.

Sorry about that, @lcallewi ! I've updated the article, which will be live in the next ~24 hours.
Meanwhile, the correct number for August is: 2.72.5556.801


Automate document generation

We have been manually refreshing documentation (generated using pdoc) till now.
However, this can be automated using Github actions.

Open questions:

Do we want to update github pages, too? This may require us to monitor the quality of docstrings closely. This can be a deterrent to a first time contributor.
What do we do with runtime warnings?
Is generation of documentation a pre-push or post-push item? If it's post-push, what if we get errors?

I am new to open source and would love to work on this issue, can I contribute?

Welcome @cyberShaw! I assigned this issue to you.

Welcome @cyberShaw! Great questions @KranthiGV, thanks for raising this.
We had originally intended to use Sphinx for documentation, but that setup required a bit more  time than originally planned, so for now we are using pdoc.  (Definitely open to revisiting that decision!)
In terms of order of refreshing content, there's a nice list on the sphinx page that we could use for our pdoc doc gen flow for now.
Do we want to update github pages, too?
My gut reaction is yes; hopefully the docstrings aren't too much of a burden on new contributors. We should definitely write some text about this in CONTRIBUTING.md, as I see that's missing!
What do we do with runtime warnings...Is generation of documentation a pre-push or post-push item? If it's post-push, what if we get errors?
I think this is something we might have to experiment with a bit.  I am not familiar with how strict the error tolerance is, I don't want to discourage users.  For example, it's probably ok if the doc is "slightly off-format" vs "totally broken", and I'm not sure where the error threshold falls.

I did a simple PR (#223) that run pdoc as a pre-commit. In this way, if the code ends up in changing the documentation, the html page with the doc is update automatically. I guess we can go with this solution until we don't have better options. We still need to fix the pipeline for updating the gh-page. That is still manual at the moment.


Optimize piVPN - Easy-RSA is not updated + Suggestion

PiVPN is no longer maintained, see the README

PiVPN Issue Template

Easy-RSA stuck on version 3.0.4 (newer is 3.0.7). The install script is set to install the 3.0.4.
Suggestion: Update command should be available from pivpn menu,like pivpn -update.
Have you searched for similar issues and solutions?
Yes

Have you taken any steps towards solving your issue?
 Renamed the older easy-rsa folder, git the new easy-rsa, and copied the PKI folder inside the older easy-rsa to the new easy-rsa

could you please re-adit your issue to skip unnecessary template data? that way it's hard to read....
anyhow, i'm also curious why pivpn brings it's own version of easy-rsa

could you please re-adit your issue to skip unnecessary template data? that way it's hard to read....
anyhow, i'm also curious why pivpn brings it's own version of easy-rsa
also see #181

Done mate, thanks for the tip. The issue is quite simple, the install script of pivpn is searching specifically for version 3.0.4 of easy-rsa instead of the latest. I'm not an expert by all means, i'll leave that to the pros. But i checked the change log of 3.0.4 and 3.0.7 (if im not mistaken) and there were a couple of issues resolved by the latest version. It's just that.
Oh and the suggestion is in pivpn command menu, like "pivpn -c" checks for connected clients, it would be quite easier to update pivpn with a similar command like "pivpn -up" to update from the repo and other openvpn dependencies (like easy-rsa perhaps). Just my 2 cents on the subject, hope it's helpful.

yes, i think it's helpful and it's not "just that" because it is a problem if security related programs are being forked and remain unmaintained and don't get their updates.
so, it's important to know why pivpn still uses 3.0.4 and if there is no real reason anymore, it should get rid of that and use upstream project

I concur. Don't know if it helps but on the install.sh i found this:
"easyrsaVer="3.0.4"
easyrsaRel="https://github.com/OpenVPN/easy-rsa/releases/download/v${easyrsaVer}/EasyRSA-${easyrsaVer}.tgz"
"

whoops, apparently i did oversee that pivpn does use upstream easy-rsa - but why is it in https://github.com/pivpn/easy-rsa then ? that could/should be deleted then....

@devZer0 https://github.com/pivpn/easy-rsa is no longer used anymore by the script. See #597.

yes, i recognized that in the meantime. things would be easier (and not so confusing) if that repo would be dropped then. when getting into pivpn, the first thing people do is browsing the web/git repo or reading into things - so why should they come across things which are obsolete already? this is why many software projects simply suck - there is so much oudated information/references everywhere...  life could be so much easier....

https://github.com/pivpn/pivpn/issues/700

Closing this... as #700 will take over this issue.


doc-3502 fix source path description

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Static Position on Dropdown from mobile list UI

Basically what the title says.
Is there a way to display results from select2 in a list where you can view in static way, like a list just below the search field?

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


Syntax Error: Unexpected token '?'

On a fresh install I'm getting the following error

This is an error because your node version is lower than 14.0.1, can you tell me what is your node version?
use
node -v

in your terminal  to get the version
basically this app requires you to have a node v >= 14.0.1
@richawo
should I add this into documentation?

Solved in #13 , thanks


testing fix for h5py error

removed mhkit load from circular function. If this works it can be applied to all other functions- but a solution will need to be found for the WPTO hindcast function

@ssolson can you pull down this PR and test the call to the circular function?

Getting error with h5_py although it is now slightly different than before.
Error using h5>init h5py.h5 (line 1)
Python Error: ImportError: DLL load failed while importing defs: The specified procedure could not be found.

Error in version><module> (line 15)

Error in __init__><module> (line 33)

Error in multi_file_resource><module> (line 5)

Error in __init__><module> (line 8)

Error in hindcast><module> (line 3)

Error in __init__><module> (line 3)

Error in __init__><module> (line 2)

Error in __init__><module> (line 1)

Error in <frozen importlib>_call_with_frames_removed (line 219)

Error in <frozen importlib>exec_module (line 783)

Error in <frozen importlib>_load_unlocked (line 671)

Error in <frozen importlib>_find_and_load_unlocked (line 975)

Error in <frozen importlib>_find_and_load (line 991)

Error in <frozen importlib>_gcd_import (line 1014)

Error in __init__>import_module (line 127)

Looking at the h5py source code where is fails on line 15 of version.py I believe it is trying to load a pyd file.

My guess is there some mismatch between the local DLLs and the DLLs these were complied with and they are not being matched correctly when called through MATLAB. I don't know much about this though so I am just taking a guess here.

I do not know anything about DLLs either. It does look like h5py is up to version 3.1.0. Can you try updating the package?

The error above was received with h5py-3.1.0

I doubt this will work, but try one more time with this change. If this does not work, we may need to consider removing the WPTO hindcast function temporarily until we can find a solution

Same error as before. Yeah, I think we should remove this until h5py and anaconda resolve this issue.

Can you take care of removing it as you can more quickly test the Matlab side to make sure everything that needs to be removed to make Matlab work has been done?

ohh you know I was thinking we would remove it from Matlab not python...Now I see that doesn't make sense. I will see if a non-anaconda version of python works with MATLAB.

Well I can report that this is not an anaconda issue.

Okay so you cannot change the MATLAB version HDF5 because it is closely tied to the MATLAB executable. Currently Matlab uses an outdated as far as I can tell version of HDF5 library 1.8.12.
At a surface level to me, it seems to me that that HDF5 version 1.10 breaks some compatibility.
h5py brings precompiled HDF5 libraries that uses up to date HDF5 version 1.12.0.
It is possible to compile hypy using a custom compiled source but this is far from a plug and play solution. This will require each MATLAB user to do custom compilations. I have not yet confirmed that this will solve the issue either.

Thanks for continuing to dive into this. This issue brings to light a couple underlying issues with the Python/Matlab development. I plan to have this as a discussion item for the broader team to try and determine a path forward with development in general.

This issue is fixed by https://github.com/MHKiT-Software/MHKiT-Python/pull/104


Safe State introduced with 0.39 isn't working anymore

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.40.0
Safe State which was introduced in Version 0.39 doesn't work anymore with 0.40. After Restart my Input_boolean is back to default setting.

douplicate with #6463


WebServer: refactor based on development guidelines

Refactor the webserver module to follow the development guidelines in PR #135.
The following tasks are part of this issue ("Validate" means check and if not compliant, fix):

[ ] Change builders that accept other builders to accept a supplier instead
[ ] Change scope to "provided" where reasonable to have a cleaner dependency graph
[ ] Validate bundle dependencies to not bring in too many third party libraries
[ ] Move SE bundles to a /bundles root module and change group ids accordingly??
[ ] Validate module names and package names to ensure they adhere to dev guidelines)
[ ] Validate naming convention of configuration keys used by components
[ ] Fix inconsistencies in getter and setter names
[ ] Validate we do not create instances using "new" and use factory methods
[ ] Refactor factory method names (use create)
[ ] Refactor config methods in builders (use config)
[ ] Validate java module files are in each module

See issue #193  for webserver test refactoring.

Validate we do not create instances using "new" and use factory methods

What is the point of using static factory methods instead of new operator?

static factory methods are provided as convenient shorthands on top the builders. I believe the only reason to enforce this over "new" is for consistency.
@tomas-langer feel free to comment.

A constructor is more restrictive - you lose the flexibility to use whatever implementation you need and are stuck with the implementation when designing the API.
We have encountered this already - we needed to change a class into an interface. As we were using static factory methods and a builder, there was no impact on the API.
Some other advantages:

Factory methods have names
Factory methods may return the same instance or a new instance as needed
Factory methods may return a subtype of the expected type (see example above with interface)

In general, this is about an opinion - either approach is usable. We have decided for the factory methods and must be consistent across all our modules.


Possibly incorrect docs for Control.VerticalContentAlignment

        public class C : Control
        {
        }

Do new CC().VerticalContentAlignment returns Center. The docs say:

The default is Top.

If this is correct, wouldn't the above code return Top?

Also, the apparent "default value" of each individual control used in UI can be different. For example, a Button control starts with the value Center.

It seems that all controls have default value of Center, so button isn't an exception.
Anything I'm missing?
Same goes to HorizontalContentAlignment as well

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

ID: 886f9354-7ad6-1ed2-93ed-01dbebe2f68f
Version Independent ID: 57a4468a-f0c6-e339-7372-236f851b2408
Content: Control.VerticalContentAlignment Property (Windows.UI.Xaml.Controls) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.UI.Xaml.Controls/Control.xml
Product: uwp
Technology: winui
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

Thanks for the feedback. This documentation has been updated to reflect the current default values.


tests: add riscv cpu_tests

Hoa, can you remove the makefile then push this to the mainline?
Thanks!

The commit is pushed to the upstream: https://gem5-review.googlesource.com/c/public/gem5/+/16889.
I think I should close this PR.

Thanks!
On Wed, Apr 24, 2019 at 11:17 AM Hoa Nguyen<EMAIL_ADDRESS>wrote:

Closed #97 https://github.com/darchr/gem5/pull/97.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/darchr/gem5/pull/97#event-2298083554, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAA2YHGXKHZE3P2IFRWTCXTPSCP23ANCNFSM4GXSMM2Q
.

--
Jason Lowe-Power
Assistant Professor, Computer Science Department
University of California, Davis
3049 Kemper Hall
https://arch.cs.ucdavis.edu/


Regen after Readonly -> ReadWrite flatten updates

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.14265765,Cognitive - Form Recognizer:0.024802485,Search:0.0225287'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Mgmt:0.58176696,Client:0.39135036,Service:0.020512953'


Post locations to firebase without overriding HTTP services

Is it possible to post locations to a different server via HTTP while simultaneously posting locations to firestore?

No, that would consume too much battery.  Use firebase functions to relay to your other server.  It's easy.


SCB-571 Decouple OmegaContext in the Application

Now the demo application can use enableOmega or not.

Coverage decreased (-1.0%) to 94.199% when pulling 944e745c1b61d80afe916841f8e637ff7bb13a3c on SCB-571 into d8ad37c3acc8a4e35bf2e346314a91e9f2e496d3 on master.


ModelSpeedUp support for aten::mul?

When I ran ModelSpeedup on a model which uses an operator "*" in the model definition, I ran into the following error:
RuntimeError: Has not supported infering output shape from input shape for module/function: aten::mul, conv2.0.gate.aten::mul.418
My question is:

Will aten::mul be supported by ModelSpeedup anytime soon?
How much effort it requires to write custom function to infer input/output shape associated with aten:mul? If I want to implement it by myself, where should I start?

Thanks in advance!

Hi~ thanks for the feedback! Could you please give us the code snippet (looks like it's in the conv2) that has the aten::mul so that I can add the support for aten::mul and test on it?

Thanks @zheng-ningxin for your prompt feedback. I am using the OSNet model implemented in the torchreid package. Here is the code snippet for speed-up. To run the following snippet, you need to install the torchreid package.
import torchreid

n_classes = 751

device = torch.device("cuda:0")

model = torchreid.models.build_model(
    name='osnet_x1_0',
    num_classes=n_classes,
    loss='softmax',
    pretrained=True
)

model.load_state_dict(torch.load('pruned_AGP_l1.pth'))
model = model.to(device)

dummy_input = torch.randn([1,3,256,128]).to(device)
dummy_input = dummy_input.to(device)

masks_file = 'mask_AGP_l1.pth'
m_speedup = ModelSpeedup(model, dummy_input.to(device), masks_file)
m_speedup.speedup_model()

I have attached the OSNet model definition file from torchreid, pruned_AGP_l1.pth, and mask_AGP_l1.pth. The aten::mul operator seems to result from the ChannelGate class in osnet.py. In the ChannelGate.forward function, there is an operation input * xused.
osnet.zip

Thank you for adding the suppor for aten::mul. I tried the newly added feature, and it seems that the ModelSpeedup function does not report any error on my previously attached example. However, when I tested the model after speed up (model_speed_up) on a test dataset, the latency even increases a bit compared to the model without speedup (which is the pruned_AGP_l1.pth in the previous attachment). Also, the accuracy of the model_speed_up drops a lot compared to pruned_AGP_l1.pth. It seems that the zero filters are now removed correctly by ModelSpeedup. Could you take a further look into the added support for aten:mul? Thanks.

@guojunyao419 Hi~ I also found this problem when testing with the example you gave. First of all, there is no acceleration effect because the dependence between the channels of the submodules is very heavy. As for why the channel dependency will reduce the acceleration effect of the model, you can refer to ChannelDependency tools for more details.
Basically, you can try to enable the dependency-aware mode for the pruners you used(only Filter Pruner supported), or you can align the pruned channel manually in the pruning algo for the channel-dependent layers(you can get the layers that have channel dependencies at ChannelDependency tools )
FYI: I also visualize your model:

I have the same problem, but is the Mish activate function! I use the easy model with mish ,then speedup will have error RuntimeError: Has not supported infering output shape from input shape for module/function: Mish, Mish
my model is this
`class Mish(nn.Module):
def forward(self, x):
return x * F.softplus(x).tanh()
class NaiveModel(torch.nn.Module):
def init(self):
super().init()
self.conv1 = nn.Conv2d(1, 20, 5, 1)
self.conv2 = nn.Conv2d(20, 50, 5, 1)
self.bn1 = nn.BatchNorm2d(self.conv1.out_channels)
self.bn2 = nn.BatchNorm2d(self.conv2.out_channels)
self.fc1 = nn.Linear(4 * 4 * 50, 500)
self.fc2 = nn.Linear(500, 10)
    self.Mish = Mish()

def forward(self, x):
    x = self.Mish(self.bn1(self.conv1(x)))
    x = F.max_pool2d(x, 2, 2)
    x = self.Mish(self.bn2(self.conv2(x)))
    x = F.max_pool2d(x, 2, 2)
    x = x.view(x.size(0), -1)
    x = self.Mish(self.fc1(x))
    x = self.fc2(x)
    return x`

can you tell me if i want to do this , how can i do?

I have the same problem, but is the Mish activate function! I use the easy model with mish ,then speedup will have error
RuntimeError: Has not supported infering output shape from input shape for module/function: Mish, Mish
my model is this
`
class Mish(nn.Module):
def forward(self, x):
return x * F.softplus(x).tanh()
class NaiveModel(torch.nn.Module):
def init(self):
super().init()
self.conv1 = nn.Conv2d(1, 20, 5, 1)
self.conv2 = nn.Conv2d(20, 50, 5, 1)
self.bn1 = nn.BatchNorm2d(self.conv1.out_channels)
self.bn2 = nn.BatchNorm2d(self.conv2.out_channels)
self.fc1 = nn.Linear(4 * 4 * 50, 500)
self.fc2 = nn.Linear(500, 10)
self.Mish = Mish()

def forward(self, x):
x = self.Mish(self.bn1(self.conv1(x)))
x = F.max_pool2d(x, 2, 2)
x = self.Mish(self.bn2(self.conv2(x)))
x = F.max_pool2d(x, 2, 2)
x = x.view(x.size(0), -1)
x = self.Mish(self.fc1(x))
x = self.fc2(x)
return x
`
can you tell me if i want to do this ,how can i do

@guojunyao419 Hi~ I also found this problem when testing with the example you gave. First of all, there is no acceleration effect because the dependence between the channels of the submodules is very heavy. As for why the channel dependency will reduce the acceleration effect of the model, you can refer to ChannelDependency tools for more details.
Basically, you can try to enable the dependency-aware mode for the pruners you used(only Filter Pruner supported), or you can align the pruned channel manually in the pruning algo for the channel-dependent layers(you can get the layers that have channel dependencies at ChannelDependency tools )
FYI: I also visualize your model:

close the issue as the answer had been provided.


David/wallet erc1155

Includes functionality to work with ERC1155 tokens in the wallet, as well as adding tests for ERC721 and ERC1155 to sdk testing.

In order to have ERC1155 working correctly, we have removed restrictions to Shield contract about tokenId for ERC1155 and ERC721. Now, tokenId 0 is allowed.
Calculation of balances in commitments-storage.mjs from commitment value and tokenId now for an ercAddress. If it's an ERC20 tokenId will be 0. (Ex.: [ { balance: 60, tokenId: 0 } ])
Fix nonce error when changing deposit from ERC20 to ERC721. Approve transaction with missing encodeABI was the cause.

From wallet now you can

Visualize ERC1155 tokens
If you select an ERC1155 token, details will be displayed
You can add new ERC1155 tokens (new ERC1155 address), or add a new token ID to an existing ERC1155 token

To test SDK, npm run test-e2e
To test functionality, launch wallet tests by ./start-nightfall -wt -s and search for Selenium tests PASSED in wallet-test container
Closes #351, #385 , #384 , #381 , #379, #365

Tested with E2E, wallet tests and UI.

E2E passed well

UI did not show 1155 functionality to test it

Wallet tests (./start-nightfall -wt -s) are failing with error

Additional changes are required but we can handle that it in a different issue to have this PR merged soon.
I created a new issue for this

@ChaitanyaKonda , thanks for the thorough review. Regarding why ERC1155 tokens don’t show up, my guess is that you probably are working with a saved password, and the configuration is stored in local storage. I think if you clear the local storage, the the default configuration with mock erc1155 tokens will show.
Regarding second issue, I think there is a problem in your laptop. Try to  prune dockers and do an new setup. I know it works because that what GitHub actions are running.


[5.0] Fixed Resource Method Name Generator

Fixed invalid resources names if 'as' parameter defined and parent group has autoprefixed
For example
  $router->group(['namespace' => 'ApiControllers\v1'], function($router) {
    $router->group(['middleware' => ['oauth']], function($router) {
			$router->resource('/v1/user', 'UserController', [
			    'as' => 'api',
			]);
			$router->resource('/v1/user.webSite', 'WebSiteController', [
			    'as' => 'api',
			]);
    });
  });

Before trimming the dots this code produce route names like this "api..v1.user.index"

This is actually a fix that's been a long time coming. Thanks for the PR and the merge!


Support API v4

V3 is deprecated; https://about.gitlab.com/2017/03/22/gitlab-9-0-released/

Linking to GitLab CE API v3 to v4 Changes for more context.

Thanks @tomasbjerre. Would you be willing to submit a PR to upgrade to v4?
Should be a relatively easy change based on the link from @jsorah

Perhaps. I cannot promise anything. But yes, if I find the time.

As I wrote in #201, I suggest to bump the major of java-gitlab-api to 4, so it corresponds to the API level.

@mfriedenhagen 👏 👏 great work!
More than happy for you to bump the version to v4.

This is now in master. Once we've worked out any issues I'll create a release.

Thanks! :)

Will you also release 4.0 soon?


remove dead code

so I ran https://github.com/remyoudompheng/go-misc/blob/master/deadcode/deadcode.go
on docker just to see and cleaned some things up
lets see what breaks

Please note that concept of execdrivers is being replaced with OCI compliant binaries executed through containerd. There is an ongoing effort for switching to containerd in #20662 . Please consider porting the changes in your PR to this branch instead.

thx @GordonTheTurtle

LGTM

LGTM


Fixed Hex Viewer writing white text on white background when in light mode

Objective
The Hex Viewer would write the raw data in white, on white background, when in light mode. This fixes that.
Solution
The Color of the text is now not hardcoded anymore, but inferred.

The latest commit makes the raw bytes in the hex viewer black or white depending on the background, changes the color of the selected packet in the packet list and adds a hint text to the filter bar to make it more obvious

LGTM


color of semantic highlighting is not rendered after pasting code

I can stably reproduce it in both stable and insiders editions. See below screenshot, the color #4ec9b0 was not rendered.   (There's another bug, if theme-selectors is too long, I cannot see the token type in the inspector window due to overflow, that's why I cut them in the screenshot)

I created the original issue in https://github.com/redhat-developer/vscode-java/issues/1421 when we testing the RC build of vscode-java v0.61.0 .
Steps to reproduce:

Open an empty folder and create a new file foo/bar/apiImpl/Hello.java:

package foo.bar.apiImpl;
public class Hello { }

Semantic highlighting should be working well, you can observer apiImpl.
Cut the first line (package.....), type some whitespaces, and paste it back.
Boom! Now you see the color fallbacks to syntax highlighting.
If you modify the content, e.g. appending a whitespace to any line, the color would get back to work.

VSCode version:
Version: 1.44.2
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T17:07:18.473Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.4.0

and
Version: 1.45.0-insider
Commit: a0fe37870c42e0704a492cdc6b8550b4cf23f63c
Date: 2020-04-30T05:36:26.465Z
Electron: 7.2.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.4.0

Similar issues occur after text edits are being applied (eg. organizing imports: https://github.com/redhat-developer/vscode-java/issues/1423) or commenting  code (https://github.com/redhat-developer/vscode-java/issues/1424)

I can reproduce in a Java container (Remote Container: Try a sample). I verified it's not a color lookup issue.
I see the following exception a lot in the log, not sure if related.
  ERR n.subarray is not a function: TypeError: n.subarray is not a function
	at Function._convertToEdits (/home/vscode/.vscode-server-insiders/bin/829230a5a83768a3494ebbc61144e7cde9105c73/out/vs/server/remoteExtensionHostProcess.js:695:355)
	at /home/vscode/.vscode-server-insiders/bin/829230a5a83768a3494ebbc61144e7cde9105c73/out/vs/server/remoteExtensionHostProcess.js:694:771
	at o (/home/vscode/.vscode-server-insiders/extensions/vscjava.vscode-java-test-0.22.3/dist/extension.bundle.js:1:255660)
	at /home/vscode/.vscode-server-insiders/extensions/vscjava.vscode-java-test-0.22.3/dist/extension.bundle.js:1:256157

https://github.com/microsoft/vscode/blob/ddff80b3e6dee4c0aa85261bd1e20540091d1e93/src/vs/workbench/api/common/extHostLanguageFeatures.ts#L754

FYI, I just disabled the extension vscjava.vscode-java-test and tried, the issue was still there. Adding @jdneo maintainer of extension for awareness.

It looks like the SemanticTokens.data being returned from the extension is not a Uint32Array, but an Array.


[SYM-4787] Update Aptible example to use runtime_connector module 2.x

Description

Updates the basic/aptible_access_strategy to use the new runtime_connector v2.0 module
Updates the usages of main to reference the local.environment_name variable instead, for consistency.

Testing

Since the 2.0 connector is not published yet, I tested this by overriding the module in override.tf with

module "runtime_connector" {
  source  = "git@github.com:symopsio/terraform-aws-runtime-connector.git?ref=leslie/sym-4763-create-runtime-connector-v2x"

  environment_name = local.environment_name

  sym_account_id = "906191524257"
}

Current dependencies on/for this PR:

main

PR #61   👈

This comment was auto-generated by Graphite.


Add CI for Github Action

So, when there are error in PR we can see, which part broke and make Web Apps error.
similar issues / PR for CI Github Action is
PR Github Action

Hi @naufaldi, assign this to me.
You want CI on every PR by running npm build ?

#13 Have added github action file.


podman logs: fails on created (but not started) container

Minor corner case:
# podman create --name foo fedora date
7486b62d2316429c94bd547b19f2b809e9905d83c709cc0ba478768548003fe2

# podman logs foo
unable to read container log file: open /var/lib/containers/storage/overlay-containers/7486b62d2316429c94bd547b19f2b809e9905d83c709cc0ba478768548003fe2/userdata/ctr.log: no such file or directory
# echo $?
125

(works as expected after podman start foo).
FWIW docker simply emits no output and exits 0, which seems like reasonable behavior to me.

We should detect when the container is ContainerStateConfigured and exit cleanly instead of trying to read the log file to prevent this.

@mheon sometimes it seems like you reply before I've even hit the Submit button

@edsantiago I'm pretty on the spot with github emails. Otherwise they start multiplying in my inbox.

Closed via #434


Add kokoro test configs

Also fixes the Java version reported in the header for Java 9. (which was inadvertently introduced in #1146)

This library does not currently have system tests.

Ah gotcha. Carry on!
On Fri, Aug 24, 2018 at 10:10 AM Jeff Ching<EMAIL_ADDRESS>wrote:

This library does not currently have system tests.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-java-client/pull/1149#issuecomment-415822378,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUcyse6T6n7ZlRDhzHVe2_UW4vNRDOks5uUDOPgaJpZM4WKcKQ
.


Adding the last bug fix commit to CPAN

Hi,
I think the latest bug fix commit a5c5f0626df7ad70b4fa5a8db2d07d5090e0078e is not on CPAN, causing Mira to error out on build:
Undefined subroutine &Text::Markmoredown::Header2Label called at C:/Strawberry/perl/site/lib/Text/Markmoredown.pm line 1825.

Installing the repository version fixes the issue.

:(
wow, such a big mistakes

Fixed now, thank you.