NotificationReceiverController - Should use custom RouteScope so it works with headless sales channels as well

Describe the bug
With multiple sales channels the NotificationReceiverController would require a corresponding Storefront sales channel with a correct domain configuration. It will not work for headless sales channels and it might also be tricky regarding which URL to use if the sales channel has multiple or one uses the same adyen configuration for multiple sales channels.
Instead if would make more sense to either use the admin API ACL definitions with a corresponding permission
or a custom route scope defintion.
The custom route scope is probably the way to go as you can define it for the adyen prefix, and can create your own authentication which you probably need for the web hooks. It should optionally be possible to specify the sales channel ID in the URL, e.g. as query parameter or suffix.
It would probably be necessary
Versions
Shopware version: <IP_ADDRESS>
Plugin version: 3.4.1

Hi @AndreasA ,
Thank you for reaching out to us. We have received your ticket and will review it promptly. Our team will analyze the details and get back to you with further updates as soon as possible.
If you have any additional information or questions in the meantime, please feel free to share them.
Thank you for your patience and understanding.
Best regards


Fix some leftover references to idle_frame

#44593 left some references to "idle_frame" signal, one of them was spamming errors in the animation editor.

Thanks!


MarketMaker cluster not used

Market maker always uses default grid fee as it is in cluster None, which is ok but produces a lot of warning even though it is a quite common use case to have a Market Maker
As the market maker is not part of the power network to not clutter the graph (and as it does not actually sit in one place in the power network), the generated gridfee matrix does not include the market maker right away.

Theoretically market maker could be added to the network, but it would mean to add an edge to every other cluster if the grid fee should be the same for all clusters.

The market maker has a hidden (inside **kwargs parameter) market_maker_cluster parameter which is only known to the market maker but can be accessed by the scenario. I.e. it could be added to the grid fee matrix after it was generated from the network.

Current working solution is that market maker uses default_grid_fee as market maker cluster is None by default, But the market maker is not added to the grid fee matrix with the default_grid_fee
The Market Maker could also be in a specific Cluster when using the wrapper function while explicitly stating this cluster in the grid fee matrix.


[wip] Clarify peer finder docs

I wrote some text but I don't know where to put it, or if we should elaborate with examples/split out into its own doc etc.
Thoughts?

This change is

I think Eric's doc covers this material (https://github.com/kubernetes/contrib/pull/2082#pullrequestreview-10430199), please re-open if that's not the case


Add menu choosing helper from pokey_talon

Nice find. I didn't even realize this existed in community already. I'm okay with this as I prefer "choosing" wording as well, as to me drop down actually selecting the entry isn't so intuitive.
That said I forget what we generally do about adding aliases to existing commands just because some people want them, so someone will have to remind me. Maybe something to write down as a general policy for the repo, if it's not already somewhere.

Personally, I'm not a fan of needing to fork the repo to customize so I'm hoping there's a solution that doesn't require it!
Regarding limiting scope: I was paying with the Homeric app on Mac last night and this command can navigate menus system wide once you have "clicked" with it so I'd like it to work everywhere.

I haven't seen any more feedback on this one. I think no feedback means it's probably not too controversial...?
Are you able to merge @fidgetingbits?

I actually think, for the same reason I already said, that this shouldn't be merged unless we come up with an official policy of dealing with this type of request. That said, I'd rather leave it to others to decide on what the goal of the community repo is, and I don't particularly enjoy debating stuff like this lol.
But fwiw the way I see it, and my understanding of PRs that we generally (there have been exceptions) accept, is that community generally defines one standard way of saying things, and if someone doesn't like it they need to customize, which atm usually means fork. There are too many people customizing stuff (thus so many community forks) to sanely merge in everyone's chosen alias for a given command, especially given the current design.
I know you don't like forking, which is fine, but I'd also like it to be clarified (who I dunno, maybe aegis needs to) if we want one of the goals of the community repo to be preventing people from having to fork (I don't believe it is atm). If we did actually want to go that route, then I'd still want to reject this PR and suggest it instead it should be done in a way that abstracts the spoken command through an action so that anyone can specify whatever they want (similar to how pokey uses <user.teleport> in his repo iirc).
The problem with going with this latter abstraction route is, as far as I know, one of the goals of the talon language is ease of use and readability for non programmers. If every command becomes abstracted in a way to prevent forking, imo the talon files will become unreadable to the common user. Maybe this aegis needs to clarify too.
To me this PR is merging something because you personally find it convenient and don't want to fork. Hypothetically, if I also didn't like to fork, and already had choosing used for something else, and we merge this, then who wins? Now I have to fork because you didn't want to? It seems not feasible to avoid this problem, and so we just generally select a single thing that's "good enough" for general users and if people want something else they have to customize. A further example is @nriley uses pick for this exact same thing in his pick.talon plugin iirc. He'll presumably never use choosing, so should we now have (drop down | choosing | pick)? The more of these aliases we add for personal preference the more it restricts the available words to use for other commands people want. imo its easier to just create a choosing.talon file in your .talon/user (no need to fork for this particular one even ) and not merge this.
All that said, I'm not actually against the idea of a repo (ironically perhaps a fork of community) that has a main goal which is to not have users fork in order to customize, and maybe I'd even use it myself, but my issue here is that I don't think that's actually one of the goals of community in its current form.
Maybe I'm thinking too much and should just merge this, but I'd rather wait for someone that's been around as long or longer than me to comment, because I'm quite sure this isn't the first time we've not merged something like this for the same reason.
@knausj85 @rtnz @pokey @phillco @lunixbochs @AndreasArvidsson @timo

It seems thatmy desire to avoid forking was communicated by my reason for avoiding it hasn't... I definitely understand that there are personal preferences that may differ and that negotiating those differences is challenging. I'm of the mind that where there are significant differences it ought to be configurable or not in this repository at all. As it is, earlier preferences have more weight than anything that comes later because there's no easy way to agree to adopt a change. There are various ways to do many things and there are quite a few that I don't like but have been dealing with because I don't want to maintain a fork. I'm not trying to be difficult here, and I can solve this on my own if I'm out of line here but I think everyone loses out if we choose not to change it just because the existing language got there first even if another word choice is better for more people. (Not saying this choice is, but if it were then it would be a bad outcome).

It seems that my desire to avoid forking was communicated by my reason for avoiding it hasn't... I definitely understand that there are personal preferences that may differ and that negotiating those differences is challenging. I'm of the mind that where there are significant differences it ought to be configurable or not in this repository at all. As it is, earlier preferences have more weight than anything that comes later because there's no easy way to agree to adopt a change. There are various ways to do many things and there are quite a few that I don't like but have been dealing with because I don't want to maintain a fork. I'm not trying to be difficult here, and I can solve this on my own if I'm out of line here but I think everyone loses out if we choose not to change it just because the existing language got there first even if another word choice is better for more people. (Not saying this choice is, but if it were then it would be a bad outcome).
Fwiw - I also like "pick" as much as "choosing" - I'm just not a big fan of the current "drop down (number)" because I find it hard to say accurately. I'm interested though in hearing out other opinions on this because there could be a really good reason why it is the way it is that would persuade me to just keep using it but with a better appreciation of why it's a good design.

You aren't out of line and I'm not trying to be difficult either fwiw. I also don't use
drop down, and likely never will, and don't know off hand why it was chosen, aside from I guess it is related to drop down menus.
I'm not opposed to changing commands like this if they don't make sense, and agree just because things are there first they shouldn't be favored for no reason. If things do change , they should be removed using deprecation vs just adding in more aliases imo.
But, I don't know how to reasonably select what is actually "better", aside from usually fewer syllables and thematic words are widely considered preferable. I like pick the most of the 3 and have actually been testing using it lately, but I also use tick for ' and I suspect that will occasionally conflict which will inevitably lead me to using something else anyway. That doesn't mean it's not best for everyone else shrug Maybe most people already like drop down :D
I recall knausj85 already mentioned why he doesn't want to break out into multiple repos to solve some of these problems, which I guess can be continue to be debated elsewhere.
fwiw aegis has some vision for a more modular plugin based setup with a super minimal base grammar in the future (no idea on time frame), that might address a lot of your concerns. Not sure, but this may also play into why people aren't rushing to change community's current form. May be worth asking him more about that.

I haven't seen the reasoning for not breaking out into separate repositories. I have heard about the modular plugin system idea. I could be wrong but I think that even a modular plugin system would require a lot of work from community to be usable. For example, I'd expect that many users wouldn't want to add support for every programming language. There's currently a global registry of programming languages that doesn't allow for registration without modifying the base Python code, so it couldn't be used without redesigning it to support languages registering themselves if you wanted to have different language configurations dependent on what plugins were installed. (Maybe not the best example, but this one comes to mind). I think this is resolvable now if there is the desire to do so and the benefit is that anyone could add languages to their own configurations without changing the base repository.
My point is that the problem doesn't go away or change by adding a packaging system, to my understanding. It actually puts more pressure to resolve the problem since it adds additional expectations about how that would work.

I wanted to share one example where drop down can cause issues with cursorless terminology. I often use drop down when I'm selecting a file from file hunt. Sometimes talon thinks I said chuck down <n> instead of drop down. This deletes the line n lines below the cursor position while I'm trying to select a file. I can't undo without first closing the file select dialog, and if I pick a file instead vscode will auto save with that change. This is a pretty unpleasant use experience that would be nice to fix for users of vanilla community and cursorless.

The main problem with adding arbitrary aliases is potential conflicts, so I personally prefer to keep aliases to a minimum. This is more problematic for any global commands such as this one.
I don't believe this would play well with this, for example https://github.com/talonhub/community/blob/main/core/homophones/homophones_open.talon#L3
As-is, depending on context, people would need to remember "choose" or "choosing" which seems less than ideal. Plus, choose and choosing likely too close to be reliably distinguished
We could implement an overridable action here to provide a consistent "choose" command

I like the overridable idea!

I'm not sure how to approach it though. Can you help me understand how to implement it?

I'm not sure how to approach it though. Can you help me understand how to implement it?

search for eg code_comment_block (whole word only). You'll see how it appears in a .talon file, a generic .py file, and context-specific .py files

I understand how to use indirection to call into a python file. I'm confused about is how you would override it without modifying community. Can you override python files by simply including some additional files that override them?

If some wants to take a stab at this (in case it's easier to demonstrate than explain) I'd be happy to review.

I think @knausj85 is proposing that there be one choose action, and if homophones are open, it picks the homophone, otherwise it runs the generic action defined in this PR. So if I'm understanding correctly it's not about users overriding the action, but us overriding the action when homophones are open

Also, looks like something strange happened to the commit history here 🤔. The diff is including a bunch of recent mainline changes for some reason

it prominently features my name there but i'm pretty sure i didn't do anything to this PR

Does it look better now?

LGTM

I think I'm confused. I thought that by adopting the same wording, we were addressing the comment https://github.com/talonhub/community/pull/1393#issuecomment-2041064896. I also don't understand how to use the deprecation infrastructure. Can you help me out there?

The problem with this approach is that you know have a conflict between the "choose" here and the one when homophones is up. Could hack it by adding a context here but better to have an overridable action
Re deprecation, grep for "deprecate" in the code base; if still not clear how to use it lmk and we can try to improve docs in that file

Thank you for that context. I'm still not sure what an "overrideable action" is. Is there an example I could follow?

I'd have another look at https://github.com/talonhub/community/pull/1393#issuecomment-2041451646; that should help figure out how to do an overridable action. If that's still not making sense maybe we could connect briefly before Tuesday cursorless slot like before?

Thanks! Yeah I took another look but I'm still not quite clear. Sure-I'm glad to come at 7:30am pacific before the Tuesday cursorless meetup to chat. Thanks!

I'm starting to get it, I think. Is there any way to define a default and only override in the one case?

How's this approach?

That still leaves the user with two places to change the spoken form if they want to change it, but may be a reasonable compromise? @knausj85 wdyt?
Also, any reason to delete the old file and make a new one? It's still doing the same thing; just a new spoken form
And still needs deprecation for old spoken form; there should be instructions if you search for "deprecation" or "deprecate"

Re: deletion-I thought it made sense since the old file name also matches the command name. If I understood the override idea, I think it would still have the same effect (I.e. needing to change in more than one place). I think you could make a talon list and add a spoken form that can be changed in one place there but since there as only two examples so far it seems like too much indirection to me. Happy to restore the original file name.

The talon wiki covers how to create and override actions pretty well: https://talon.wiki/Customization/Talon Framework/actions
I still think this is a preferable approach, as there are similar mechanics in many places.
for example in gaze-ocr
https://github.com/wolfmanstout/talon-gaze-ocr/blob/main/gaze_ocr_disambiguation.talon
with the action approach, we can consolidate to single overridable action and potentially define a more consistent grammar.
The downside of this approach is we would no longer be able to interact with drop downs while the other stuff is open. It's rare, but I've personally seen it.

I prefer this over an overridable action because I often interact with drop downs in applications that have no talon configuration at all. I could be misunderstanding here, but wouldn't it require every application to define an action for the override approach to work? Or is there a way to set a default behavior?

The thought is the drop down implementation would be the default action implementation, and the command would still be global.
And eg Homophones and gaze ocr could override the action when active.
This allows us to define one command that does the right thing depending on the active context.

Ok. That makes sense to me. If you have an existing example you could share for setting up a default action please do. I'll look into this later or also please feel free to push to this branch with what you think makes sense.

I'm struggling with how to do this a little bit. This is what I tried:
from talon import Context, Module, actions

default_dropdown_select = Context()
mod = Module()

@mod.action_class
class Actions:
    def choose(number_small):
        """Choose the nth item"""
        actions.key("down:{number_small-1} enter")

    def choose_up(number_small):
        """Choose the nth item up"""
        actions.key("up:{number_small} enter")

@default_dropdown_select.action_class("user")
class ChoosingActions:
    def choose(number_small):
        actions.key("down:{number_small-1} enter")

    def choose_up(number_small):
        actions.key("up:{number_small} enter")

It's not working and I'm not sure why. This is the error I see in the talon log:
2024-05-12 13:39:14.927 ERROR cb error topic="phrase" cb=SpeechSystem.engine_event
   16:      lib/python3.11/threading.py:995 * # cron thread
   15:      lib/python3.11/threading.py:1038*
   14:      lib/python3.11/threading.py:975 *
   13:                    talon/cron.py:156 |
   12:                     talon/vad.py:23  |
   11:                     talon/vad.py:129 |
   10: talon/scripting/speech_system.py:369 |
    9:             talon/engines/w2l.py:742 |
    8:      talon/scripting/dispatch.py:134 | # 'phrase' main:EngineProxy._redispatch()
    7: talon/scripting/speech_system.py:66  |
    6: -------------------------------------# [stack splice]
    5:      talon/scripting/dispatch.py:134 | # 'phrase' main:SpeechSystem.engine_event()
    4: talon/scripting/speech_system.py:442 |
    3:      talon/scripting/executor.py:111 |
    2:  talon/scripting/talon_script.py:707 |
    1:  talon/scripting/talon_script.py:610 |
talon.scripting.talon_script.TalonScriptError:
 in script at /Users/tkma6d4/.talon/user/community/lang/tags/operators_math.talon:15:
 > user.code_operator_greater_than()
NotImplementedError: Action 'user.code_operator_greater_than' exists but the Module method is empty and no Context reimplements it

You're on the right track!

I don't think that error message is related. There are no others in your log?
What happens when you say "choose three"?
I don't think you need the second class; it should just work with the first class. You should only need the second class when you want to override it, eg when homophones is open, where you'd do what you have there but add a matcher to the context
Looks like you're missing argument+return value type hints and a docstring on the action definition. Talon is probably rejecting it for that reason, tho usually you'd see an error message

I got this to work now. Thanks for your help! 🙇

We should use deprecate_command to deprecate the commands.

I'm getting a new problem that I don't understand. The homophones work now but now the original 'choose' helper is throwing an error:
2024-05-28 21:17:08.035 ERROR cb error topic="phrase" cb=SpeechSystem.engine_event
   16:      lib/python3.11/threading.py:995 * # cron thread
   15:      lib/python3.11/threading.py:1038*
   14:      lib/python3.11/threading.py:975 *
   13:                    talon/cron.py:156 |
   12:                     talon/vad.py:23  |
   11:                     talon/vad.py:129 |
   10: talon/scripting/speech_system.py:369 |
    9:             talon/engines/w2l.py:742 |
    8:      talon/scripting/dispatch.py:134 | # 'phrase' main:EngineProxy._redispatch()
    7: talon/scripting/speech_system.py:66  |
    6: -------------------------------------# [stack splice]
    5:      talon/scripting/dispatch.py:134 | # 'phrase' main:SpeechSystem.engine_event()
    4: talon/scripting/speech_system.py:442 |
    3:      talon/scripting/executor.py:111 |
    2:  talon/scripting/talon_script.py:707 |
    1:  talon/scripting/talon_script.py:610 |
talon.scripting.talon_script.TalonScriptError:
 in script at /Users/tkma6d4/.talon/user/community/plugin/dropdown/dropdown.talon:2:
 > user.choose(number_small)
TypeError: object of type 'NoneType' has no len()

I'm kind of lost as to what could be going on here.

I verified that the addition of the homophones user override seems to cause this error. If I remove the override it works.

I checked this a few times and I think it's working correctly now.

This is still not quite right. You currently have two rules of the formchoose <number>. That will cause Talon to randomly pick one. There should only be one rule of this form, have it mapped to a single action, and then override that action depending on context

got it. Removed the redundant rule.

I'm not seeing any additional warnings beyond what I normally do. It works based on my testing it several times.

Just took it for a spin locally; works for me! One last thing: the deprecation warning in the log points the user to BREAKING_CHANGES.txt, so we should drop a line there about this deprecation. Otherwise it's good to go!

Curious if we should take the opportunity to change the plugin name from dropdown to something something like "menu_choose" since it's more descriptive?
Also just a thought, homophones are "core" functionality, and now rely on an action defined by this plugin (which sort of implies it's not core? I dunno, seems weird anyway). I wonder if we should move to to core/menu_choose/?

Also just a thought, homophones are "core" functionality, and now rely on an action defined by this plugin (which sort of implies it's not core? I dunno, seems weird anyway). I wonder if we should move to to core/menu_choose/?

this is a great point. I think we should have a hard rule that core cannot depend on anything from plugins

moved it into core.

I think all of the issues have been addressed now. Thanks @jaresty for your contribution, effort, and patience.

Yes I'll second @fidgetingbits's comment here. I'm quite happy with how this one turned out


NETOBSERV-1568: reintroduce finalizer check to remove any remaining finalizer

Description
Since removing ipfix, we don't use finalizers anymore. However users upgrading from a 1.5 will still have one set in their CR. So we need, for a release, to continue checking for the finalizer, and remove it if found.
Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[x] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[x] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/ok-to-test

@jotak - I tried to upgrade test with pre-merge catalog, but unable with downstream 1.5 and upstream pre-merge, QE testing would have to occur on downstream bundle for this change.

@jotak - I tried to upgrade test with pre-merge catalog, but unable with downstream 1.5 and upstream pre-merge, QE testing would have to occur on downstream bundle for this change.

What I did to test was installing downstream 1.5, then downstream 1.6 (the one that is buggy) and then I manually changed the operator image from the installed CSV to the upstream one.
Otherwise, to switch from a downstream catalog to an upstream one, IIRC there's a couple of more things to do like replacing the "replaces" key in CSV

What I did to test was installing downstream 1.5, then downstream 1.6 (the one that is buggy) and then I manually changed the operator image from the installed CSV to the upstream one. And then I was able to delete the FlowCollector.

thanks @jotak . I was able to verify with this approach.
/label qe-approved

/lgtm

/approve


Remove unused content from logging site extension

applicationHost and scmApplicationHost are now packed in https://github.com/aspnet/AspNetCore/blob/master/src/SiteExtensions/src/Microsoft.AspNetCore.AzureAppServices.SiteExtension/Microsoft.AspNetCore.AzureAppServices.SiteExtension.csproj

Build failures:

D:\a\1\s.dotnet\x64\sdk\2.2.100-preview2-009404\Microsoft.Common.CurrentVersion.targets(4533,5): error MSB3030: Could not copy the file "D:\a\1\s\extensions\Microsoft.AspNetCore.AzureAppServices.SiteExtension\applicationHost.xdt" because it was not found. [D:\a\1\s\test\Microsoft.AspNetCore.AzureAppServices.SiteExtension.Tests\Microsoft.AspNetCore.AzureAppServices.SiteExtension.Tests.csproj]

There were some tests using the XDT file, moved them in https://github.com/aspnet/AspNetCore/pull/3726

Btw, I noticed this PR targets master, but https://github.com/aspnet/AspNetCore/pull/3726 targets release/2.2. Any reason for the difference?

I'm too used to having release/2.2 as default branch.


Search with a longer time interval than 7 days

Whenever I try to search for anomalies, the maximum duration is around 7 days. Is there any way to increase the interval on the frontend?

You mean "Detector output" on "Model definitions" page? That's sample result for historical data. We limit the date range as 5 days currently. These are sample data which show how the model will look like if applied to historical data.  You can check real anomaly result on "Anomalies" page.

Thank you for the clarification @ylwu-amzn . Just to verify, you are saying that that real anomaly detection uses all data, not just in the data range of 5 days?

I've found the answer in the documentation. All data is used. Thank you again @ylwu-amzn for the explanation.


pie chart having Error

pie chart having Error - Cannot read properties of undefined (reading 'getValueForPixel')
To Reproduce
I have implemented this on pie chart , while dragging any point it's showing below error
TypeError Cannot read properties of undefined (reading 'getValueForPixel')
here is a codesandbox where i have wrapped the chartjs pie chart with draggable link: https://codesandbox.io/s/chartjs-draggable-bmnzog

Hey @fazlay,
Thanks for raising an issue!
Pie charts are unfortunately not supported as of right now. I'd happily accept a (well tested) PR though 😊


zh, en: update online ddl doc

First-time contributors' checklist

[x] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)

Change content related to online ddl. Because we want to use online-ddl instead of online-ddl-scheme.
Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version)
[ ] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from: https://github.com/pingcap/dm/pull/1904.
Other reference link(s): none.

Do your changes match any of the following descriptions?

[x] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/merge

/cherry-pick release-2.0


Hacktoberfest Examples: Full-Text Search within an Index

It would be pretty cool to see some examples of how to use Full-Text search within Redis OM.NET indexes, these mostly need to be decorated with Text attributes, and then more or less queried as normal. For more information on how to do full-text search in RediSearch (and probably some inspiration for this topic) check out the RediSearch docs, probably the topic on FT.SEARCH will be most instructive.
See #217 before getting started.
If you’d like to work on this please mention me @slorello89 or @SuzeShardlow in a comment and we’ll assign this issue to you.
This is part of our participation in Hacktoberfest 2022. Learn how to complete this issue here: https://redis.io/community/hacktoberfest/
Need help or want to talk to us? Join us on Discord where we have a dedicated Hacktoberfest channel

Hi, @slorello89/ @SuzeShardlow  I would like to work on this.

Go for it @Jeevananthan-23. Please check out our guidelines and have fun! We look forward to seeing your pull request.
https://redis.io/community/hacktoberfest/

Thank you for giving me this opportunity @SuzeShardlow / @slorello89 . A pull request created please look into it.

Thanks @Jeevananthan-23, we will check it out :)


how to add tricks to max 10 and min 0 in polar chart

hi,
I am using polar chart in Angular with ng2-charts.
I want to display tricks always show min 0 and max 10.
I have added this code, but it is not working.
public polarChartOptions: (ChartOptions) = {     responsive: true,     legend: {       position: 'top',       display: false     },     scales: {       ticks: {         min: 0,         max: 100       },     }   };

@chiraggmodi the key needs to be scale not scales. The following should work
public polarChartOptions: (ChartOptions) = {
  responsive: true,
  legend: {
    position: 'top',
    display: false
  },
  scale: {
    ticks: {
      min: 0,
      max: 100
    },
  }
};

I have added scale, but it shows error, that scale does not exist in type 'ChartOptions'.
 ERROR in src/app/dashboard/dashboard-passions/dashboard-passions.component.ts(65,5): error TS2322: Type '{ responsive: true; legend: { position: "top"; display: false; }; scale: { ticks: { beginAtZero: boolean; min: number; max: number; }; }; maintainAspectRatio: false; }' is not assignable to type 'ChartOptions'.       Object literal may only specify known properties, but 'scale' does not exist in type 'ChartOptions'. Did you mean to write 'scales'?


Merge remove_drawable into dev

I started to merge everything together into the dev branch.

Nevermind.  I'll need to do this in VSCode.


[Feature]: Add support for 44.1khz audio

Description
I was wondering if it would be possible to add support for 44.1khz audio files without having to resample. It's also a common sample rate, so it might be good to have as an option. Thanks for your efforts!
Problem
Currently, 44.1khz audio options are not present in Applio. Given that it's a common sample rate for CDs, I would think it should be an option.
I have a ton of audiobooks on CDs that I would like to work with, hence the request to support 44.1khz audio. I would like to take the narrator of one audiobook and use their voice to replace the narrator's voice of another audiobook.
Proposed Solution
44.1khz audio should be supported without having to resample.
Alternatives Considered
n/a

we don’t plan to do that, 40khz and 44.1khz are mostly the same, if you want more quality you can go with 48khz

additionally there are no 44.1KHz pretrains

It seems that using 48khz causes the voice to become robotic, distorted, and choppy. 40khz sounds better, but sacrifices quality. It would be a real shame if there's truly no plans for such a common sample rate.
What would it take to make a 44.1khz pretrain?

It seems that using 48khz causes the voice to become robotic, distorted, and choppy. 40khz sounds better, but sacrifices quality. It would be a real shame if there's truly no plans for such a common sample rate. What would it take to make a 44.1khz pretrain?

Well, this sample rate isn't that common, the most common is 32k because of YouTube. As for making a pretrain, it would cost about 40 dollars, BUT the code for making pretrains is currently broken, so even if you had the money, it wouldn't work.

It seems that using 48khz causes the voice to become robotic, distorted, and choppy. 40khz sounds better, but sacrifices quality. It would be a real shame if there's truly no plans for such a common sample rate. What would it take to make a 44.1khz pretrain?

Well, this sample rate isn't that common, the most common is 32k because of YouTube. As for making a pretrain, it would cost about 40 dollars, BUT the code for making pretrains is currently broken, so even if you had the money, it wouldn't work.

It's a very common sample rate; every single CD that has ever been released has been 44.1khz, and CDs have existed decades before YouTube. There are billions of hours more of CDs than there are YouTube videos.
Okay, why would it cost 40 dollars? And what code is it?

Okay, why would it cost 40 dollars? And what code is it?

He meant the cost of training a pretrained model from scratch at 44.1KHz on 4090 x 100-200 hours. The training from scratch unfortunately produces garbage results right now due to a noise introduced by the model.

Okay, why would it cost 40 dollars? And what code is it?

He meant the cost of training a pretrained model from scratch at 44.1KHz on 4090 x 100-200 hours. The training from scratch unfortunately produces garbage results right now due to a noise introduced by the model.

I have a 4080S; I'll do the work. Aside from the working code, what is needed to be done?

Okay, why would it cost 40 dollars? And what code is it?

He meant the cost of training a pretrained model from scratch at 44.1KHz on 4090 x 100-200 hours. The training from scratch unfortunately produces garbage results right now due to a noise introduced by the model.

I have a 4080S; I'll do the work. Aside from the working code, what is needed to be done?

you need a new config for v2/44100.json with appropriate settings
you need to change config.py to include this file
you need to change tabs/train/train.py to include the new option for sampling rate
then you need to figure out a big data set to train the pretrain on. It is unknow which data set was used for original RVC training or how it was actually made. Safe to say, multiple hours of varying audio.
train a new model without using pretrained (from scratch) until the model reaches an optimal state and no longer produces horizontal lines on the spectrogram.

Okay, why would it cost 40 dollars? And what code is it?

He meant the cost of training a pretrained model from scratch at 44.1KHz on 4090 x 100-200 hours. The training from scratch unfortunately produces garbage results right now due to a noise introduced by the model.

I have a 4080S; I'll do the work. Aside from the working code, what is needed to be done?

you need a new config for v2/44100.json with appropriate settings
you need to change config.py to include this file
you need to change tabs/train/train.py to include the new option for sampling rate
then you need to figure out a big data set to train the pretrain on. It is unknow which data set was used for original RVC training or how it was actually made. Safe to say, multiple hours of varying audio.
train a new model without using pretrained (from scratch) until the model reaches an optimal state and no longer produces horizontal lines on the spectrogram.

the base pretrained is vctk

Awesome, thanks for the info, @AznamirWoW and @blaisewf ! Once the issue with the code is resolved, I will get it done. Is there an issue open to track that problem, or an ETA for when/if the code will be fixed?

Awesome, thanks for the info, @AznamirWoW and @blaisewf ! Once the issue with the code is resolved, I will get it done. Is there an issue open to track that problem, or an ETA for when/if the code will be fixed?

There's no ETA, since we don't know what's causing the problem, about open an issue, there are already two open
https://github.com/fumiama/Retrieval-based-Voice-Conversion-WebUI/issues/87
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI/issues/2319

Alright, I will keep checking in on those issues to see when it's resolved and proceed after. Thanks for the links, @ShiromiyaG !

@blaisewf Why close? It seems odd to not support the most common sample rate.

It seems that using 48khz causes the voice to become robotic, distorted, and choppy. 40khz sounds better, but sacrifices quality. It would be a real shame if there's truly no plans for such a common sample rate. What would it take to make a 44.1khz pretrain?

Well, this sample rate isn't that common, the most common is 32k because of YouTube. As for making a pretrain, it would cost about 40 dollars, BUT the code for making pretrains is currently broken, so even if you had the money, it wouldn't work.

@ShiromiyaG YouTube's sample rate is either 44.1khz or 48khz, not 32.

@blaisewf Why close? It seems odd to not support the most common sample rate.

It seems that using 48khz causes the voice to become robotic, distorted, and choppy. 40khz sounds better, but sacrifices quality. It would be a real shame if there's truly no plans for such a common sample rate. What would it take to make a 44.1khz pretrain?

Well, this sample rate isn't that common, the most common is 32k because of YouTube. As for making a pretrain, it would cost about 40 dollars, BUT the code for making pretrains is currently broken, so even if you had the money, it wouldn't work.

@ShiromiyaG YouTube's sample rate is either 44.1khz or 48khz, not 32.

It doesn't mean that the audio is 44.1k because the audio has that sample rate. If you want to see for yourself, download the “Spek” application and see with him an audio from YouTube.

blaisewf Why close? It seems odd to not support the most common sample rate.

It seems that using 48khz causes the voice to become robotic, distorted, and choppy. 40khz sounds better, but sacrifices quality. It would be a real shame if there's truly no plans for such a common sample rate. What would it take to make a 44.1khz pretrain?

Well, this sample rate isn't that common, the most common is 32k because of YouTube. As for making a pretrain, it would cost about 40 dollars, BUT the code for making pretrains is currently broken, so even if you had the money, it wouldn't work.

@ShiromiyaG YouTube's sample rate is either 44.1khz or 48khz, not 32.

It doesn't mean that the audio is 44.1k because the audio has that sample rate. If you want to see for yourself, download the “Spek” application and see with him an audio from YouTube.

YouTube typically resamples audio to 48kHz, but this is the first time I've heard of YouTube using 32kHz, and can't find anything on Google. Their documentation recommends 48kHz: https://support.google.com/youtube/answer/6039860
I opened a YouTube video in spek like you said, and the max frequency is higher than 32kHz.

Get Lucky (YouTube)

CD downsampled to 32kHz:

Original CD:

I also tried listening to the YouTube video downsampled to 32kHz, and it sounds noticeably worse.
I'm not sure why the YouTube audio doesn't cover the entire spectrum. It could be that the audio was originally a lower sample rate, a low pass filter was used, or a side effect of lossy compression/encoding. I'm too lazy to upload a video to test this.
44.1kHz is the most common audio sampling rate: https://en.wikipedia.org/wiki/44,100_Hz
40kHz outside of voice AI isn't common.


What is the workflow for moving a local dev site to a new computer?

Hello,
I'm hoping I can access the collective brain power of this group to clarify this:
I want to move my local dev site to a new computer:
I have MAMP stacks on both machines with wordpress installs. Do I:
set up a fresh build of FoundationPress on my new machine via the command line and then copy over (overwrite) the entire directory of my new build with the old FoundationPress theme (including the massive node_modules dir)?
would this work? is there a better workflow?
Both machines are macs running El Capitan
Thanks in advance
Andi

Hi there,
I just recently went through this myself a few months ago when I switch from my iMac to a MBP.
What I ended up doing, after I had a running WordPress installation on my new computer, was copy over my theme directory SANS node_modules and bower_components. Then run npm install like you normally would with a new FoundationPress project, and you're good to go. This was also mentioned in issue #797
Alternatively, if you're using Git and your remote is all up to date, you could just clone your repo into your themes directory, which will be cloned in its own brand new theme folder.

thanks, I'll give it a try and let you know how it goes!!!!

btw, I have node_modules removed, but I can't find bower_components, can you point me in the right direction? I see a few hidden files with "bower" in the file name and "bower.json" in the root. did you mean to write node_modules OR bower_components? excuse my Noob question...

I have a dir called "FoundationPress/assets/components" that seems to be similar to bower_components -- it has "fontawesome", "foundation_sites", "jquery", "motion-ui" and "what-input" in it. Is that the one I should delete?
thanks in advance

Yes, that would be your bower components.
It might be named different or in a slightly different location depending on if you have a more recent or older version of FP. I think I'm just used to calling it bower_components because of working with older versions of FP and Foundation.

thanks for clarifying!

ok, that didn't work, here is the npm-debug.log
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'run', 'watch' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prewatch', 'watch', 'postwatch' ]
5 info prewatch<EMAIL_ADDRESS>6 info watch<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec watch script
9 verbose stack Error<EMAIL_ADDRESS>watch: gulp
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter. (/usr/local/lib/node_modules/npm/lib/utils/lifecycle.js:214:16)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at EventEmitter.emit (events.js:172:7)
9 verbose stack     at ChildProcess. (/usr/local/lib/node_modules/npm/lib/utils/spawn.js:24:14)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at ChildProcess.emit (events.js:172:7)
9 verbose stack     at maybeClose (internal/child_process.js:821:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd /Applications/MAMP/htdocs/wp-content/themes/FoundationPress
12 error Darwin 14.5.0
13 error argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "watch"
14 error node v4.2.6
15 error npm  v2.14.12
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>watch: gulp
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>watch script 'gulp'.
18 error This is most likely a problem with the foundationpress package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     gulp
18 error You can get their info via:
18 error     npm owner ls foundationpress
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]

I had basically the same error when I was trying to move to a new computer as well, check out issue #773. Does deleting node_modules again and re-running npm install do anything?

I've done it the hard way, I did a clean install and moved all my old files EXCEPT node_modules and assets/components and that seems to have done the trick, everything seems to be working,  but I see some subtle changes in styles, I think maybe foundation has changed since my original build, that would be in assets/components I guess. I'm thinking that the potential to destroy my old site is too great. I can't develop too much longer on my old machine. I wish these dev repros were more portable.... blahhh

I tried installing a new version of foundationPress in my themes folder, deleting it and then replacing it with the original theme from my old machine, the code wouldn't complile. (kept getting errors that sass couldn't work with the new foundation.css,etc...) Is there a quick way to get the new install of npm to work with my old code?

That difference in versions could possibly be contributing to your problems. Have you ever used Node Version Manager? With that you can manage a number of different Node versions. I remember having to switch back and forth when I'd work on various projects at the same time.

@andreaarcher what version of Foundation was installed in your old computer? You can find this in assets/components/foundation-sites/bower.json.
Additionally, when you moved the project to the new computer (omitting node_modules and assets/components) and you ran npm install, what was the version in assets/components/foundation-sites/bower.json? I suspect that the reason you saw a style difference is because it installed a new bower package of foundation-sites and the rest of your project didn't wasn't updated to match.
I haven't had to do this before, but this is the process I would suggest:

Backup entire project somewhere safe
Copy all files from project to the new computer, except for the node_modules directory.
In package.json remove bower install from the "postinstall" script so it is now: "postinstall": "gulp build"
npm install
npm run start

This keeps all of your bower packages in assets/components at the same version, while getting the appropriate node_modules for the node installation on your new computer. Removing bower install from the "postinstall" script prevents bower from updating the components when you install the npm modules. This should make it build the same on the new computer as it did on the old.
And as previously mentioned, using a Node version manager will make your life a lot easier in times like these.

looks like the old site was built on 6.2.0 and the fresh install on my machine is running 6.2.1. I'll try this new approach, THANKS!!!
just to clarify, if I follow this workflow do I still need to be running the same versions of npm and node.js as my original machine or will that not be necessary?

I just did what @colin-marshall suggested and everything went perfectly!!
So to summarize for my future self, other team members or anyone else who can benefit:
In order to move a local repo from one machine to another (or replicate it for another team member) and keep everything the same, do the following:

find out what version of node.js and npm you are using on the original machine and make sure they are installed on your new machine and available in the shell (use node version manager to do this).

Copy the old repo sans the node_modules dir, move to the new location on your new machine

If the theme is named "FoundationPress" I like to make sure to rename it at this point to make sure I can do fresh installs in the same theme directory.

In package.json (in root) remove "bower install" from the "postinstall" script so it is now: "postinstall": "gulp build"

npm install
npm run start
npm run watch

open your site and go!

thanks to @swthate and @colin-marshall  and the rest for all the great input!!!
now I'm going to back up the live remote site and then start pushing my new code live, wish me luck!

@andreaarcher that's awesome news!
Quick question though - what about dragging the database over to a new machine? It's great that we have the theme itself, but the db would be great too.

I use backup and transfere plugins for that-- I like wp-clone

@andreaarcher glad we got you up and running!
A pretty sweet command line solution I found for syncing WP is Wordmove. You can sync any combination of plugins, themes, uploads, db, and/or wordpress core. It goes super fast over SSH and you can have multiple environments (staging, production, etc.). It was a little confusing/intimidating at first but it's pretty awesome once you get the hang of it.


Transaction Problem.

Hi guys, I really need your help on this. It has stated in my wallet that i've received my bitcoins of 1.182691BTC on 29/05/18. This's my wallet address: muFjdWSy2DKStSP2gUH2aHZq6kRBva8ZDV it stated on the insight.bitpay.com. It stated confirmed. However, i can't send out to another wallet address as it stated insufficient funds. Please advise me what should i do? Is it a scam? I really need the money soon. My mum is in icu & needed the money soon. I got into mining because i want to earn more money to help my mum.
My transaction ID is b2898b8c5f5f01f573e768ec21fc4f15c1e433e7255960fb71551f06cec36651. It seems that my transaction confirmed. What should I do next? I’m stuck.

https://test-insight.bitpay.com/ should show a big "TEST NET" label with warning messages.

Hi Patrick, I bought the bitcoins from localbitcoins. And I send this bitcoins over $2k SGD to the miner where he sent me the address to be send the bitcoins too.

Kayley, I am sorry to tell you this but the trader/miner that you bought the bitcoins from scammed you. What is his username/profile URL? Your only hope is to contact localbitcoins.com and attempt to dispute the trade.

Patrick, I purchased my bitcoins from localbitcoins Cryptotil (https://localbitcoins.com/accounts/profile/cryptotil/) & Getbelu (https://localbitcoins.com/accounts/profile/getbelu/) And the bitcoins did sent to me to my localbitcoins wallet. From there, I send my purchased bitcoins to my coinbase wallet. The “Miner” gave me a wallet address to sent all this bitcoins over to him so he could help me on mining. This is the “miner” (https://m.facebook.com/blessing.ezelie?xts%5B0%5D=12.%7B%22unit_id_click_type%22%3A%22graph_search_results_item_in_module_tapped%22%2C%22click_type%22%3A%22result%22%2C%22module_id%22%3A0%2C%22result_id%22%3A100016657001547%2C%22session_id%22%3A%22e1ccc720b1c327d9ca4cf660086356f7%22%2C%22module_role%22%3A%22NONE%22%2C%22unit_id%22%3A%22browse_rl%3A15d74bb5-0568-df16-b8e5-1de04adca8a8%22%2C%22browse_result_type%22%3A%22browse_type_user%22%2C%22unit_id_result_id%22%3A100016657001547%2C%22module_result_position%22%3A0%7D) that scammed by $2k plus SGD, and blocked me on Facebook when I decided to publish him out everywhere.
Does it help to get back my money? I’m very upset about it as I just want to earned and help my mum for her weekly injection that costs a bomb.

No, this doesn't help sorry. Once someone has your bitcoins there is no way to reverse the transaction. I am still confused as to why you posted a testnet transaction. If you sent your money to him through coinbase I don't see how you would get a testnet transaction at the end. I am really willing to help, however, beyond this point there's nothing that can be done from a technical side. I am really sorry that you got scammed. I personally understand what it feels like and it's not a good feeling. I took at look at his Facebook profile, it looks like he's living in the US; perhaps you could threaten to contact the police as this is blatant theft.

Patrick, the “miner” told me to create a BitPay account & wallet as the earnings is a bit to large. This is what he said, “This time it would be sent to you through bitpay then you send out to your coinbase or I will show you where to sell it and get cash straight to your bank. Now because this amount is bit large the use of bitcoin payment is the only option and there are recent rules against money laundering using crypto currency so hence they have devised a means to pay a fee to aviod this rule
But first they want you to be assured before you pay this fee that you are going to earn, so they have come up with a new system
Go and download bitpay app on your phone
Create a wallet open it and click on “CREATE PERSONAL WALLET” on the next page enter mining as the name and BITCOIN as the coin then hit the advance options and tick testnet and single address, then create the wallet, look for recieved and then copy out the address send it to me and I will give them, they will send the bitcoins to you, you will receive them on that wallet but you will not be able to use them because they are being held in escrow by the bitpay servers, the company told bitpay servers to hold it till you pay the money laundering fee which is $1000,
So once you pay it, they will ask bitpay to release it so you can use it, after that then you can send it to coinbase or any wallet and cash out to your bank account, it’s simple
Sorry about the fee you have to pay it’s a new development but atleast you are assured of your earnings
I was just informed of this now.”

<EMAIL_ADDRESS>

Sorry, Kayley. There's nothing that can be done. You got scammed. I am sorry that this happened to you. I would try to contact the proper authorities.

Really very upset. 😭thank you Patrick for the help.

My wallet key:3BoFcMLSJDH5oc3gPeE4F8JuqTAmvA7B4
You can withdraw in my account!
Thank you
2018年6月9日 下午5:43，"KayleyWong"<EMAIL_ADDRESS>Hi Patrick, I bought the bitcoins from localbitcoins. And I send this
bitcoins over $2k SGD to the miner where he sent me the address to be send
the bitcoins too.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bitpay/insight/issues/942#issuecomment-395955615, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ak3MKWvm-F7zm7qcwyyr87w5FDRpdu0Dks5t65imgaJpZM4UWApK
.

Danny. what do you mean?

You can pay them right from this email.
Google Pay
You received a money request from MY MiCkY
$ 200 .
Send
Decline
img
MY MiCkY
<EMAIL_ADDRESS>Time
Sep 04, 2018 11:50 AM PDT
Method
ID
L.1B98-DF27-9601-005C
Google
Google Play App Store
You can also try new features at pay.google.com
How to recognize suspicious emails | Terms of Service
Help Center | Privacy Notice
Funds subject to processing by Google Payment Corp. Google receives
consumer report info to protect you from fraud and inform transaction
decisions.


Add batching UXS transport

Add a transport UXS to complement the UX and UXF transport. UXS would use SOCK_STREAM instead of SOCK_SEQPACKET. This, in combination with XCM-internal buffering (which the API already allows), will allow several messages to be sent/receive with a single syscall. This in turn might yield a performance benefit for applications that send multiple messages on the same connection, before doing a context switch.

This issue is supersedes by issue #63.


[FRONTEND] - Update README

the readme must be updated every third day to show the status of the project

got it


Tips for training with randomly initialised word embeddings

I'm working with Fairseq's transformer model for a seq2seq translation-style task where the source language is the same as the target language (e.g. de-de). However, when I initialise my word embeddings randomly (i.e. without specifying a path to a pre-trained embedding file), my model fails to train.
After following the example https://github.com/pytorch/fairseq/tree/master/examples/translation#iwslt14-german-to-english-transformer, I applied the same preprocessing steps to my own data:
1. tokenize
2. learn and apply bpe
3. binarize with `fairseq-preprocess`

Then, for training, I'm using the command:
fairseq-train $DATA_BIN \
--dataset-impl mmap \
--arch transformer \
--task translation \
--lr 5e-4 --optimizer adam \
--dropout 0.3 --attention-dropout 0.3 --activation-dropout 0.3 \
--max-epoch 10 --max-tokens 3360 --update-freq 2
--share-all-embeddings \
--skip-invalid-size-inputs-valid-test \
--save-dir $CHECKPOINTS

This command differs slightly from that used in the IWSLT14 example, but I figured overall functionality should be the same.
According to the output training logs, for my data, train loss fails to improve, while validation loss jumps all over the place. Meanwhile, for the IWSLT14 example data (with the train command from the example), both train and validation loss and perplexity begin to quickly decrease.
If I initialise the embedding matrix for my data from pre-trained embeddings, say BPEmbs (https://nlp.h-its.org/bpemb/), training/validation loss and perplexity decrease nicely over the first epoch.
As far as I understand, the default behaviour is to learn the embedding matrix along with all other model weights, so I'm not sure why it's failing to learn them when training on my data. Am I missing something regarding training with randomly initialised embeddings vs. pre-trained embeddings?
Thanks in advance!

Hi @tannonk
I wonder how you used pre-trained embeddings for training a machine translation model. I have a trained XLM-roberta encoder model and I would like to know if I can use it is weights as embedding for a machine translation task.
Thanks in Advance

Hi @ahmed-gamal97
I'm not sure how relevant this is to your situation, but I was providing static pre-trained embeddings using the --encoder-embed-path  and --decoder-embed-path arguments and providing the path to locally stored embeddings as text files, e.g. de.wiki.bpe.vs10000.d200.w2v.txt which can be downloaded from https://github.com/bheinzerling/bpemb


Change libvo_aacenc to aac

VisualOn AAC encoding library (libvo_aacenc) is no longer supported by ffmpeg: https://trac.ffmpeg.org/wiki/Encode/AAC#libvo_aacenc

I need to check this first

Yup

FFMPEG is being compiled locally from now and includes the aac native library, much more performant.


Some problem

I think that the signal was collected correctly. (CF = 5.7965GHz), sometimes I could show the target signal(drone-id burst)
Because The output of drone_id_test.grc (Found burst @ xxxx/xxxx)  and I was checked that Burst was detected at regular intervals.
After that, I saved the output file collected by filesink block and ran updated_scrits/process_file.
However, only the output shows that burst is not found. I tried several times, but I couldn't check the result.
I used the "gr-droneid-update" branch.

I'm not sure I quite follow the question.  It sounds like you recorded samples with the file sink in GNU Radio and then processed those samples with the process_file.m script.  It's quite likely that there are no bursts in the file you recorded.  DroneID doesn't appear very often (~ once every 600 milliseconds) and frequency hops which doesn't give you much of a chance of actually recording a burst.  If you saved off the exact same samples that you saw burst detections for in GNU Radio then it should work.  If you post a screenshot of the recorded file in Baudline then I might be able to tell you if what you have likely contains DroneID bursts that for some reason were not processed properly.
Also, what SDR did you use to collect the samples?


[Frontend, Tensorflow, Tensorflow2] Tensorflow frontend op refactor

This PR refactors the current tensorflow.py parser and separates the code for the ops into a separate file tensorflow_ops.py. The new parser for handling tensorflow2 models in TVM, ie tensorflow2.py, imports the existing frontend op codes from tensorflow.py. With the current PR both of these frontend codes will import the ops from a common file for the ops in tensorflow_ops.py.
This PR is in line with the original plan of supporting tensorflow2 parser in TVM frontend as in the discussion #4102  Exact commit plan here.
Co-authored-by: Rohan Mukherjee<EMAIL_ADDRESS>Co-authored-by: Xingyu Zhou<EMAIL_ADDRESS>Co-authored-by: David Huang<EMAIL_ADDRESS>Co-authored-by: Srinidhi Goud<EMAIL_ADDRESS>Co-authored-by: Xiao<EMAIL_ADDRESS>Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

@trevor-m @yongwww  @weixa @zxy844288792 @dvhg @srinidhigoud @zhiics

@mbrookhart @srkreddy1238 @kevinthesun @masahi

Thanks @rohanmukh @trevor-m TF2 is one important frontend for TVM users. Great work and let us go on!


Enhancement Request: hierarchical key-submission / covid-shield service

Great work on these well-structured covid alert app repos! This request was motivated by the additional info requests from the BC gov, and perhaps others, which have delayed their adoption. I believe it may also address the fact that while many may distrust sharing their data with the Canadian government, they may be more inclined to trust their regional health authority.
If I understand the app correctly, currently all key-submission activities are made to a Canada wide Bulletin Board (BB) like service. These are pushed back to client apps across the nation.
The problem:
Even though the data being shared is minimal, the public may be concerned that contact tracing could be abused. The Singapore gvt has recently said that they would allow law enforcement to use it in prosecuting crimes after initially saying they wouldn't for example. Although the CAN Covid-19 app does not allow this mechanism of abuse, the general public does not have the technical expertise to verify this for themselves.
Proposed solution:
Since most device-code pairs will occur in a limited geographic region, most pairs from B.C would  not need to be shared with PEI for example. If the key-submission service was running at Provincial or even regional levels, then locals could be promised that their contact tracing data will not leave the health-district/province unless they travel outside that boundary. Only codes generated during their time outside their home region ( + a 2 week buffer) would be shared with a parent key-submission server. This might significantly improve adoption as trust for local government is usually higher than that of the national gvt.
This may also support provincial authorities wishing to track more data as they could do this at the regional/provincial level but only share CAN-Covid19 compliant data with national servers. Finally, in the event of some breach, there would be less data exposed in any tier/pool of servers.

In keeping with our newly really release cds-snc/covid-alert-documentation#31 practices, I've moved this up into the covid-alert-documentation repo as our central point for taking input from folks outside the team, and am converting the issue into a Discussion.


What is the meaning of this line in angular universal?

I'm submitting a...

[ ] Bug report
[ ] Feature request
[ ] Documentation issue or request

In package

[ ] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[ ] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

Expected behavior

Minimal reproduction of the problem with instructions

Example repository

What is the motivation / use case for changing the behavior?

Environment

Angular version: X.Y.Z

- Node version: XX
- Platform:

Others:

EDIT FROM COLLABORATOR
What is the meaning of this line in angular universal. and how to achieve it.You won't be able reference browser-only native objects such as window, document, navigator or location. If you don't need them on the server-rendered page, side-step them with conditional logic. Is there any way to implement angular universal on specific components or if we can make two modules and implement seo on only one or how to avoid browser-only native which are not in my app but comes from node modules in server.js while compiling.

Well phrased question though.
But that simply means that you won't be able to access window variable like window.localStorage window.innerHeight and You can make two module one app.browser.module.ts and app.server.module.ts this way your server-side code won't break due to client-side dependencies.

Well phrased question though.
But that simply means that you won't be able to access window variable like window.localStorage window.innerHeight and You can make two module one app.browser.module.ts and app.server.module.ts this way your server-side code won't break due to client-side dependencies.

Do you have an example for this? I created two moduels, one for borwser, one for client and declared the components inside this modules. Now I get the error
'xxx-test' is not a known element:
1. If 'xxx-test' is an Angular component, then verify that it is part of this module.
2. If 'xxx-test' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message. ("

This SO question describes the problem pretty well, I just don't know how to resolve it.

⚠ Closed as invalid issue. Please, fill the issue template when opening new issues in order to help us track the bug.
Cheers!


session hangs when running df

resolution_fixed   |    by ibc599

Running the command df or listing files in /projects causes the current session to hang.
There seems to be a problem with one of the mount points?

Issue migrated from trac:64 at 2024-01-31 17:29:24 +1100

ibc599 changed resolution from `` to fixed

<EMAIL_ADDRESS>commented

From raijin:/etc/motd:
Jan 23 (1045 - ) /g/data NOTICE
A hardware module in /g/data1 has failed and is being replaced.
Some data stored on the affected hardware may be temporarily unavailable.
Storage administrators are currently working on resolving the fault.

<EMAIL_ADDRESS>commented

Same thing is happening on Raijin ...
So not Accessdev specific.
Robin

ibc599 changed status from new to closed

<EMAIL_ADDRESS>removed milestone (was Future)

<EMAIL_ADDRESS>commented

Milestone Future deleted

ibc599 commented

NCI seems to have fixed this now.


One more madness engine game

Hi, there is one more Madness Engine game, World of Speed, which seems to be similar to PC1, but it has a slightly different folder structure and for some BFFs isn't unpacked full content, probably because of using files of other type/extension. Would you be compliant to add this game to your tool?

This game is not supported because its config file (Languages/languages.bml) is slightly different.
First of all, it's not encrypted (it's just a regular .bml).
Secondly, it has an encryption patterns strings, but for some reason pattern data are also not encrypted.
BFF archives are also not encrypted at all and uses XCompress compression method, if it's interested.
So as a temporary solution you can use Languages.bml file from other games on this engine and then try to unpack the archives - I used that way for myself and it works.


Dimmer animation gets stuck

On the latest beta, I had an issue with an image that dims on hover. The image would sometimes remain stuck in the dimmed state, with further mouse hovers doing nothing.
I was able to reliably reproduce the problem by very quickly moving the mouse across the picture several times.
The JS of Semantic UI bugs out at line 16972:
return module.cache.animation || false;

The message:
Uncaught TypeError: Cannot read property 'animation' of undefined

I didn't dig into the code, but adding the following to the currentAnimation function seems to solve the issue:
if (module.cache === undefined) {
    return false
}

+1 get this same problem

I'll look at this tomorrow.

I've gone ahead and fixed the JS error. I don't have a reproducible test case so closing for now.


Error: Cannot read property 'properties' of undefined

I'm trying to run batarangle on my Angular2 app (found here https://github.com/robianmcd/tag-trends) but I get the following output in the console and none of my components are shown in the batarangle frontend
Backend: Message Received: Object {name: "init", tabId: 39} Object {id: "cppbdcpcfamjjmdinkfmddacbdgpdboa"}
Backend: Message Received: Object {name: "message", tabId: 39, message: Object}message: ObjectactionType: "START_COMPONENT_TREE_INSPECTION"__proto__: Objectname: "message"tabId: 39__proto__: Object

Uncaught TypeError: Cannot read property 'properties' of undefined
    Angular2Adapter._getComponentProperties	@	angular2.ts:273

I modified my bootstrap file to look like this
import {bootstrap} from 'angular2/platform/browser';
import {bind} from 'angular2/angular2';
import {AppViewListener} from 'angular2/src/core/linker/view_listener';
import {DebugElementViewListener, inspectNativeElement} from 'angular2/src/platform/dom/debug/debug_element_view_listener';
import {App} from "./components/app/app";
import {UrlUtil} from './services/urlUtil';

bootstrap(App, [
    bind(AppViewListener).toClass(DebugElementViewListener)
]).then(applicationReference => {
    window['ng']['probe'] = inspectNativeElement;
});

I'm running Alpha 48 so maybe this is happening because of the mismatched angular versions. I can run batarangle with the example app just fine. Any idea?

Looks like we didn't take into account that metadata isn't available in some providers, hence the bug. It should be fixed shortly. Thanks for bringing that to our attention!

Sweet it's working for me now. Thanks @vanessayuenn


Wiki should point to /blob/master/docs/readme.md

The Wiki link in GitHub currently points to the root README.
It would be great if it pointed to https://github.com/DarqueWarrior/vsteam/blob/master/docs/readme.md for better discoverability of the cmdlets documentation.

Not sure what you mean. Currently, there is no wiki at all here. Can you clarify. Maybe I am missing something. :-)

When I click on Wiki as a non-collaborater, I am transported to the front page (aka readme)
I‘d suggest to create a simple single page that just has a link to the docs... (the readme is quite „large“)

I was even thinking to create GitHub pages rendered to HTML from markdown pages. Then the documentation could be a bit more elaborate and visible.

That also sounds like something that could work, yep

@DarqueWarrior changed it to have a Wiki with basic info. I think that's good enough for now :)


(python3-streams) Also lockdown sub-folders

Description

Fixes https://github.com/chocolatey-community/chocolatey-packages/issues/1840
Switch to Write-Debug for per-folder messages, or the console will be spammed.
Motivation and Context

Close the security hole; make permissions consistent
How Has this Been Tested?

All sub-folders are limited to administrators, so pip installs to user site-packages as expected if it's run as a normal user. I tested Python 3.9 and 3.12.
PS> python3.9.exe -m pip install pyreadline3
Defaulting to user installation because normal site-packages is not writeable
Collecting pyreadline3
  Using cached pyreadline3-3.4.1-py3-none-any.whl (95 kB)
Installing collected packages: pyreadline3
Successfully installed pyreadline3-3.4.1
PS> python3.12.exe -m pip install pyreadline3
Defaulting to user installation because normal site-packages is not writeable
Collecting pyreadline3
  Using cached pyreadline3-3.4.1-py3-none-any.whl (95 kB)
Installing collected packages: pyreadline3
Successfully installed pyreadline3-3.4.1

Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[x] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment. - I got several issues during setting up the test environment, and I prefer not investing time into those issues
[x] The changes only affect a single package (not including meta package).

Is there a reason we're applying this to every level, rather than fixing inheritance? ("it's just the way I did it" is a fine answer, I'm just curious)

I didn't try it as I'm not familiar with permission inheritance. I will check it out when I'm back to my Windows machine next week.

Somehow an unwanted ACL rule for NT AUTHORITY\Authenticated Users is left with GetAccessControl/SetAccessControl, as I described in #1840, and the issue is gone with Get-Acl/Set-Acl.

:white_check_mark: Package verification completed without issues. PR is now pending human review

@JPRuskin Mind to have another look?

@AdmiringWorm @Windos @JPRuskin @gep13 @corbob I don't use Python. I don't know enough about this issue to review it. Could one of you pick it up?

@yan12125 - just double checking, does this PR include all of your intended changes?
Just asking as you mention in the description about switching to Write-Debug, but that change isn't included any more. I note a force push in the history here that got rid of the Write-Debug change and replaced it with the switch to Get/Set-Acl cmdlets only.

@Windos thanks for the notice. Yes, this pull request includes all intended changes.
The Write-Debug change is not needed for the new Get/Set-Acl approach. I updated pull request description to avoid confusion.

In my testing of this the resulting ACLs on sub directories appear to remain the same as indicated in the issue (authenticated users having write and modify rights.)
Is it possible that
Set-Acl -Path $folder -AclObject $acl

should be recursive?
Get-ChildItem -Path $folder -Recurse -Force | Set-Acl -AclObject $acl

Thanks for testing. Coud you share more details about sub directories that still have the issue? I checked permissions with the following command and didn't find a file/directory where Authenticated Users can write/modify.
Get-ChildItem -Path C:\Python39\ -Recurse -Force | Get-Acl | Format-List > acl.txt

JPRuskin and I had a discussion about the non-recursive approach (see comments above), and after that I feel the non-recursive way better in terms of performance of chocolateyInstall.ps1.

@Windos Could you have another look? I have been using the new solution with various Python versions for several months, and it's always working fine for me.

:white_check_mark: Package verification completed without issues. PR is now pending human review

Just a trivial rebase, as my branch is more than 1000 commits old.

Apologies for the delay @yan12125, just wanted to confirm this is on my list to re-review. I should be getting back to you in a couple of days.

I finally had a chance to re-test this today and did so against a handful of operating systems. Pleased to say the change worked across the board.


add a designated initializer to pass certificate data

Thanks for you PR! The codebase has actually under gone a major refactor and completely changed how we handle certificate pinning. Feel free to check it out here #653. Feedback welcomed!

Thanks for your feedback. This PR is about Two-way SSL (using a client certificate). I didn't see this feature on the new branch : do you plan to report it ? Is it possible to review this PR to eventually merge it on this version if you plan to report it later in the new version ?


Use noarch

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

can someone from @conda-forge/core could merge this ?

@jschueller - @jakirkham is the maintainer here, so there is no need to ping core :wink: I'd also suggest adding yourself as a maintainer, if this is a package you depend on :smile:

LGTM. Are you interested in helping maintain?

not really, this one is not likely to move a lot

Thanks @jschueller :)


Can't get any of the examples to compile

I must be doing something terribly stupid but I can't seem to get any of the examples to work. I am rather new to ReasonML.
I tried to make a minimal example
<EMAIL_ADDRESS>let make = () => {
  <div>
    <MaterialUi.List>
    </MaterialUi.List>
  </div>;
};

When I try to compile it it tell me this:

4 ┆
5 ┆
6 ┆   <MaterialUi.List>
7 ┆   </MaterialUi.List>
8 ┆ ;
This has type:
'a =>
ReasonReact.component(ReasonReact.stateless, ReasonReact.noRetainedProps,
ReasonReact.actionless)
But somewhere wanted:
React.component('a) (defined as 'a => React.element)
The incompatible parts:
ReasonReact.component(ReasonReact.stateless, ReasonReact.noRetainedProps,
ReasonReact.actionless)
(defined as
ReasonReact.componentSpec(ReasonReact.stateless, ReasonReact.stateless,
ReasonReact.noRetainedProps,
ReasonReact.noRetainedProps,
ReasonReact.actionless))
vs
React.element

So I am a bit unsure what this says because I suspect there are some type aliases at work here, but it seems to suggest I need to pass in a function instead of a value somewhere. I can't really make it out though.
I installed the @jsiebern/bs-material-ui version 1.1.1, the @material-ui/core version 3.9.2, reason-react 0.7.0 and bs-platform 6.2.1.  Is there some clash between those versions or so?

Exactly the same error here. I'm discovering Reasonml and I wanna give ReasonReact a shot.

I'm sorry guys, I desperately need to publish the hooks bindings onto the master branch.
For now, just run
yarn add @jsiebern/bs-material-ui@hooks

@jsiebern wow that was a really fast answer. I just tried that and everything compiles fine, but if I use a MaterialUi component the page becomes blank and I got this error in the browser console
/src/Dashboard/Dashboard.bs/__mod/@material-ui/core 404 not found.
Still really thanks for the good work here.

Okay this seems to work locally on my machine, but I cannot find a version for this and cannot add it to my package.json it seems. It always complains that the version I try to install is not present. Bit annoying for my CI ;-)

@KasperJanssens
The current version to use would be 1.1.0-hooks.14 instead of 1.1.1!
So the command yarn add<EMAIL_ADDRESS>should install the right one.
@infodevart
You need to install MaterialUI separately, it's listed as a peer dependency only.
yarn add @material-ui/core @material-ui/styles
That should do it 👍

yeah so I am using npm and this is my dependenies list :
"dependencies": {
"@jsiebern/bs-material-ui": "^1.1.0-hooks.14",
"@material-ui/core": "^3.9.2",
"bs-webapi": "^0.15.6"
},
I need to specifically do npm install --save @jsiebern/bs-material-ui@hooks on top of that if not I get an issue during run time 'TypeError: "u.makeProps is not a function"'.
If I then do this : npm install --save @jsiebern/bs-material-ui@hooks it seems to run. No idea why. Some transitive dependency coming in? Also I keep on getting the warning
'├──<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>'
Even though material ui is installed. Any ideas or an npm specific issue you think?

@jsiebern I just did a new project and  installed the dependencies but now I got the problem with the Popper
http://localhost:8000/moduleserve/mod/node_modules/@material-ui/core/Popper/Popper/__mod/popper 404 (Not Found)

@KasperJanssens Material UI should be version v4.x, that's what the latest hooks version uses.
@infodevart What are you using to serve it? Is that an HTTP/2 server? Never experimented with it before to be honest.

First of all, thanks for the amazing work. It is such a helpful library.
I also get
Error: http://localhost:8000/moduleserve/mod/node_modules/@material-ui/core/Popper/Popper/__mod/popper: Not Found.
I'm quite new to this, so this could be not what you mean, but I guess it is served with moduleserve https://github.com/marijnh/moduleserve as provided by the bucklescript react-hooks template https://github.com/BuckleScript/bucklescript/tree/master/jscomp/bsb/templates/react-hooks.
I think it fails in node_modules/@material-ui/core/Popper/Popper:18 which looks like this
var _popper = _interopRequireDefault(require("popper.js"));
So it seems like something related to the popper.js module.


1.9.3-p0 build failed on OS X Lion 10.7.2

$ rbenv install 1.9.3-p0
Downloading http://pyyaml.org/download/libyaml/yaml-0.1.4.tar.gz...
Installing yaml-0.1.4...
Installed yaml-0.1.4 to /Users/cjbottaro/.rbenv/versions/1.9.3-p0
Downloading http://ftp.ruby-lang.org/pub/ruby/1.9/ruby-1.9.3-p0.tar.gz...
Installing ruby-1.9.3-p0...

BUILD FAILED

Inspect or clean up the working tree at /var/folders/98/83mjk36d4ts49lpbjvbz48f80000gn/T/ruby-build.20111030161956.28281
Results logged to /var/folders/98/83mjk36d4ts49lpbjvbz48f80000gn/T/ruby-build.20111030161956.28281.log

Last 10 log lines:
ossl_x509store.c: In function ‘ossl_x509stctx_set_flags’:
ossl_x509store.c:520: warning: ‘X509_STORE_CTX_set_flags’ is deprecated (declared at /usr/include/openssl/x509_vfy.h:464)
ossl_x509store.c: In function ‘ossl_x509stctx_set_purpose’:
ossl_x509store.c:532: warning: ‘X509_STORE_CTX_set_purpose’ is deprecated (declared at /usr/include/openssl/x509_vfy.h:460)
ossl_x509store.c: In function ‘ossl_x509stctx_set_trust’:
ossl_x509store.c:544: warning: ‘X509_STORE_CTX_set_trust’ is deprecated (declared at /usr/include/openssl/x509_vfy.h:461)
ossl_x509store.c: In function ‘ossl_x509stctx_set_time’:
ossl_x509store.c:561: warning: ‘X509_STORE_CTX_set_time’ is deprecated (declared at /usr/include/openssl/x509_vfy.h:466)
linking shared-object openssl.bundle
make: *** [build-ext] Error 2

I did a brew installation of both rbenv and ruby-build about 15 mins ago, so I should be all up to date.

Hi @cjbottaro -- check /var/folders/98/83mjk36d4ts49lpbjvbz48f80000gn/T/ruby-build.20111030161956.28281.log to see the full build log and track down the error that occurred.
The last ten lines look like warnings only.

To build on 10.7.2, I had to set CONFIGURE_OPTS="--with-gcc=clang", as described in the 1.9.3 NEWS file.

I got it to compile by using the readline installed by brew:
LDFLAGS="-L/usr/local/Cellar/readline/6.2.1/lib" CPPFLAGS="-I/usr/local/Cellar/readline/6.2.1/include" rbenv install 1.9.3-p0

GCC is required now, so builds won't get as far as hitting errors like this. We can revisit when Ruby supports clang.

Will today announcement (Command Line Tools for Xcode) change things?

Yes it will!


Segfault on develop branch in python environment

Problem description
Running simulation in python results in a segmentation fault in develop branch and not master.
Steps to reproduce
(Explain how to reproduce the behavior, e.g.,)

Clone https://github.com/slacgismo/TESS.git -b simulation_model
cd simulation
Run python3 main.py

PC95555:simulation alyona_slac$ python3 main.py
Segmentation fault: 11

Expected behavior
Running simulation
System information

GridLAB-D version: HiPAS GridLAB-D 4.2.1-200226 (develop) Darwin
Operating system: Darwin PC95555.local 19.3.0 Darwin Kernel Version 19.3.0: Thu Jan  9 20:58:23 PST 2020; root:xnu-6153.81.5~1/RELEASE_X86_64 x86_64

Supporting information
Running gridlabd model.glm doesn't produce an error
Running on a master install doesn't produce an error
Attachments
NA

The simulation still fails later on (on the 1 year run):
PC95555:simulation alyona_slac$ python3 main.py
WARNING  [INIT] : EIA 2015 implicit enduse data is experimental
Segmentation fault: 1117:42:48 PST...


Attachments can't be displayed

Many attachments can’t be displayed.
o	Steps to Reproduce:

Select show attachments for<EMAIL_ADDRESS>Sort by type
Scroll down and select link for Brett Signature.jpg
Result: new tab displays a 404 Not Found error message.

Closing as OBE.  GUI was completely redone.


[embeddingapi] Add usecase to check saveState&restoreState performanc…

…e in XWalkView
-Add usecase to check saveState/restoreState performance in XWalkView
-With these two api in same XWalkView it failed to restore
Impacted tests(approved): new 1, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 0, fail 1, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-4393

@jennycao please help to review it.
Thanks.

LGTM


Improve JS enablement logic/flow

Random thought:
https://github.com/elkarte/Elkarte/pull/3612/files#diff-aed017bb41b71b3f6cd77616ea41ebfe859afe51a7f1c5e2e6585ddf3a81a550R278
wouldn't make sense to toggle these types of behaviours via a variable?
Something like:
theme()->addJavascriptVar(['enable_video' => !empty($modSettings['enableVideoEmbeding']) ? 'true' : 'false');

that way we don't have to assume much on the "source" side and we can load the videos the way we want on the javascript directly?
I guess there is a possible slightly overhead on the client-side, but....is it worth considering?
Originally posted by @emanuele45 in https://github.com/elkarte/Elkarte/issues/3612#issuecomment-1065869141

On that one, I did a little bit, but... yeah, facing the issue that it feels right to just move everything into some template call... and that may lead to a little bit of troubles. :laughing:


fix(anvil): block mining in place

Motivation
Closes #2282

Solution
block reading env in place

rip Compute units budget...

ProviderError(JsonRpcError(JsonRpcError { code: -32000, message: "Unable to complete request at this time.", data: None }))', evm/src/executor/fork/backend.rs:263:33

https://github.com/foundry-rs/foundry/runs/7306376203?check_suite_focus=true#step:8:169
either alchemy is down or keys rekt -.-


Release notes for version 2.4.1.

Fixes #1422

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.19%. Comparing base (f073184) to head (87cc104).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1423      +/-   ##
============================================
+ Coverage     92.11%   92.19%   +0.08%
- Complexity     3158     3161       +3
============================================
  Files           320      320
  Lines          6178     6178
  Branches        592      592
============================================
+ Hits           5691     5696       +5
+ Misses          342      341       -1
+ Partials        145      141       -4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[feat] data augmentation

We should be able to augment data (as opposed to noise data) as a way to allow for constructing better datasets (should improve training vs decrease training power).
Data augmentation should only happen on the train set
Data augmentation should happen after noise and after data has been joined (in the event of column splitting)

[x] add an add_augmentation method to the csv class in python that adds data (duplicates).
[x] add augmentation methods in augmentation/augmentation_generators.py
[x] check if the experiment class can handle data augmentation
[x] write tests for both experiment and augmentation classes
[x] allow json schemer to handle user input augmentation method

Python part is completed in #96

nextflow part is completed in #118


annatwo.toml

Please provide some contact info provided. Not merged for now due to no contact info


Hub and spokes ACM functionality

Includes:

New CRD
New controller
New logic

Everything (except for filter package, which is modified to accomodate both controllers) is completely separated from previous CRD handling.

/hold

/unhold

/hold
Just to prevent the PR from being merged while I am reviewing it 🙂

/retest

PTAL @qbarrand @yevgeny-shnaidman @pmtk @ybettan @enriquebelarte

@pacevedom Can you please squash all the commits?

@pacevedom Thanks for squashing.
The commit message still is a long list of all older commits titles.
Can we please make it a logical commit message describing the change? (Same as you did fo the PR description).

/retest

/lgtm

/lgtm


[Content]: GCP Cloud Functions doc doesn't explain request format

What article on segment.com/docs is affected?
https://segment.com/docs/connections/destinations/catalog/google-cloud-function/
What part(s) of the article would you like to see updated?
When writing a GCP cloud function to handle inbound Segment data, I need to know what the body of the request is going to look like so that I can write handlers for it. Without that, I have to find out by trial and error, creating an event that just dumps everything to the console. That's very silly!
Additional information
No response

Hi @borenstein, thanks for submitting this issue.
Requests to destinations vary widely, depending on the way the Source is configured, and the type of event that is sent.
I think your best bet here will be to use a combination of the Source Debugger, and Destination Event Tester as explained in Testing Connections. The Source Debugger, in particular, shows the raw content of the event data received by Segment, which is then sent on to the destination.
I will add this detail to the doc, for future reference.

Great. Yes, clarifying that the request body will be identical to what you
see in the source debugger (if that is true, which it seems to be), would
have saved me a couple hours today.
On Thu, Sep 30, 2021, 12:53 PM markzegarelli @.***>
wrote:

Hi @borenstein https://github.com/borenstein, thanks for submitting
this issue.
Requests to destinations vary widely, depending on the way the Source is
configured, and the type of event that is sent.
I think your best bet here will be to use a combination of the Source
Debugger, and Destination Event Tester as explained in Testing Connections
https://segment.com/docs/connections/test-connections/. The Source
Debugger, in particular, shows the raw content of the event data received
by Segment, which is then sent on to the destination.
I will add this detail to the doc, for future reference.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/segmentio/segment-docs/issues/1929#issuecomment-931495800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABB6BH5CFZC6KE2I5YPCT3TUESIXZANCNFSM5FCN3AWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thanks for confirming, and apologies for the lack of clarity.
We will add this detail to the doc.


Add support for commas and semicolons in email name and add test

This PR fixes issue #368 and replaces #370. It wraps the email name in double quotes when commas or semicolons are present (without, the mail would be rejected by the API).
I've added unit test to check the behavior of the Sendgrid\Email->setName() method. I've also sent real emails with various names to verify that they are delivered.
I've also added a comment explaining the issue because the patch should be reverted if the API is modified to support commas and semicolons.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@qligier
We have not been able to merge your Pull Request, but because you are awesome - we wanted to make sure you could still get a SendGrid Hacktoberfest shirt.
Please go fill out our swag form before Nov 5th and we will send the shirt! (We know that you might have tried this before and it didn’t work, sorry about that!)
You have till Nov 5th to fill out this form in order to get the Hacktoberfest shirt!
Thank you for contributing during Hacktoberfest! We hope to see you in the repos soon! Just so you know, we always give away a SendGrid shirt for your first ever non-Hacktoberfest PR that gets merged.

Hey,
Thanks for the shirt!
Why haven't you been able to merge it? Does it require some changes?

Hello @qligier,
I'm not sure if your PR will require any changes just yet. We are processing all of the PRs from oldest to newest and we had over 1,000 PRs this hacktoberfest, so we are a bit (OK, a lot) behind :)
Thank you for your contribution and patience!
With Best Regards,
Elmer

Hello @qligier,
Could you please resolve the conflicts so that I can merge this? Thanks!

Hello @qligier,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


too long igc line skipped

SeeYou competition adds competition details in the L records (comments) of the IGC file. These records may exceed the maximum line length and are thus skipped in the parsing. This information is sometimes very useful and not indicative of a file error. For instance the task rules are presented like this. The TaskTime=03:30:00 tells that this task is an AAT, but this can never be recovered after parsing, because the line is left out:
LSEEYOU OZ=-1,Style=2,R1=5000m,A1=180,Line=1
LSEEYOU OZ=0,Style=1,R1=20000m,A1=180
LSEEYOU OZ=1,Style=1,R1=20000m,A1=180
LSEEYOU OZ=2,Style=1,R1=40000m,A1=180
LSEEYOU OZ=3,Style=1,R1=2000m,A1=180
LSEEYOU OZ=4,Style=3,R1=500m,A1=180,Move=1,Reduce=1,Line=1
LSEEYOU TSK,NoStart=13:29:00,TaskTime=03:30:00,WpDis=False,MinDis=True,NearDis=0.5km,NearAlt=200.0m,MinFinAlt=0.0km

Maybe it would be an idea to add a flag for relaxing the maximum line length requirement on L records?

Maybe it would be an idea to add a flag for relaxing the maximum line length requirement on L records?

yep, I have no idea why we have line length limits for this at all... 🤔

This is implemented following the IGC spec (FAI doc, section A2.1):

An IGC-format file consists of lines of characters, each line giving a set of data such as for a GNSS fix. Each line starts with an upper-case letter denoting one of the Record types listed in para A2.2., and ends with CRLF (Carriage Return Line Feed). Each line is limited to 76 characters in length, excluding the CRLF which is hidden and does not appear in text form.

The idea was to skip invalid B records, such as the following (from this IGC):
B1023084948920N01509065EA0183501916006000
B1023134948940N01509187EA0184701922006000
BÿÿÿøÄÀÈÌÄàÐäÐàààÁ8ÀÄÔÀäÈÌåÀÄàÔäÀÄäÌØÀÀØÀÀÀ4)ÄÀÈÌÈÌÐäÐààÐÙ8ÀÄÔÀäÄÐÝÀÄàÜÌÀÄäÔàÀÀØÀÀÀ4)ÄÀÈÌÈàÐäÐàààÑ8ÀÄÔÀäÀÔÉÀÄàààÀÄäÜÈÀÀØÀÀÀ4)ÄÀÈÌÌÌÐäÐàäØÅ8ÀÄÔÀäÀÜáÀÄàääÀÄäÜÐÀÀØÀÀÀ4)ÄÀÈÌÌàÐäÐàäÜÕ8ÀÄÔÀäÈÄÁÀÄäÀÀÀÄäÜÀÀÀØÀÀÀ4)ÄÀÈÌÐÌÐäÐàäÄÝ8ÀÄÔÀäÈàáÀÄäÀÌÀÄäàÈÀÀØÀÀÀ4)ÄÀÈÌÐàÐäÐààØÉ8ÀÄÔÀäÈÈÙÀÄäÄÔÀÄääàÀÀØÀÀÀ4)ÄÀÈÌÔÌÐäÐààäÉ8ÀÄÔÀäÄÈÙÀÄäÌÄÀÈÀÄàÀÀØÀÀÀ4)ÄÀÈÌÔàÐäÐàäØá8ÀÄÔÀäÄÐÙÀÄäÌäÀÈÀÄÐÀÀØÀÀÀ4)ÄÀÈÐÀÌÐäÐàäÜå8ÀÄÔÀäÈÜÝÀÄäÌØÀÈÀÀÐÀÀØÀÀÀ4)ÄÀÈÐÀàÐäÐàäÄÉ8ÀÄÔÀäÌÐÁÀÄäÐÀÀÈÀÈÐÀÀØÀÀÀ4)ÄÀÈÐÄÌÐäÐààÐÕ8ÀÄÔÀäÈäÙÀÄäÔÈÀÈÀÌÐÀ0
B1024184948844N01509182EA0196402050006000
B1024234948914N01509131EA0197602058006000

What is your opinion? Should a possible relax_line_rule flag be applicable to L-records (comments as added for instance by SeeYou competition) or to all lines?

The idea was to skip invalid B records, such as the following

I guess that B record wouldn't validate anyway, even without this check, right?

What is your opinion? Should a possible relax_line_rule flag be applicable to L-records (comments as added for instance by SeeYou competition) or to all lines?

IMHO we should remove this line length check completely instead of making it optional

I guess that B record wouldn't validate anyway, even without this check, right?

I don't know. That's definately something that needs to be verified before the line length limitation is removed.

IMHO we should remove this line length check completely instead of making it optional

Wouldn't it be nice to have the funcionality to check an IGC file on its correct implementation of the spec? (line length is explicitly mentioned in the official FAI doc)

I don't know. That's definately something that needs to be verified before the line length limitation is removed.

sure, we should certainly have a unit test for something like the broken record you mentioned in https://github.com/Turbo87/aerofiles/issues/28#issuecomment-371269755

Wouldn't it be nice to have the funcionality to check an IGC file on its correct implementation of the spec? (line length is explicitly mentioned in the official FAI doc)

in theory yes, but I don't see the practical advantages here. I'd prefer a parser that's close to reality instead of close to the spec.


Doesn't work with subscriptions

Describe the bug
I was able to set this up and get it to work for queries, but it doesn't work for subscriptions.

Reproduction
Use a subscription with gql.tada
gql.tada version
gql.tada: 1.4.2
Validations

[X] I can confirm that this is a bug report, and not a feature request, RFC, question, or discussion, for which GitHub Discussions should be used
[X] Read the docs.
[X] Follow our Code of Conduct

You can see I have a subscription in my schema, but it isn't created in the type definitions file.


Merge main to feature/triggers

@msftbot do not merge unless @jander-msft approves

Build completed successfully.


Investigate  testability support for spring developer

how can our customer easily write test cases against azure services?
similar to spring-boot-test, we should have a test lib that help user to write test easiler, maybe some integration with spring-boot-test
also, AWS has a local mock service, which is really nice , more info can be found here

Any updates on this?

@amuthanarulraj not at the moment, do you have any specific ask for it? scenario is appreciated


Update ElasticBeanstalk platform to v2.0.6

Changelog includes:

new Amazon Linux AMI
Python 2.7.10
Apache and mod_wsgi minor upgrades

:+1: