Unable to resolve service for type 'Amazon.AspNetCore.Identity.Cognito.CognitoSignInManager`1[Amazon.Extensions.CognitoAuthentication.CognitoUser]' while attempting to activate 'WebAdvert.Web.Controllers.AccountsController'.

Hi,
In the README.md, it says to inject CognitoSigninManager, however, I believe it should be CognitoSignInManager (capital I) as that's how it's written in the source code.
However, although there is no error in the editor, it shows this error after running it:
InvalidOperationException: Unable to resolve service for type 'Amazon.AspNetCore.Identity.Cognito.CognitoSignInManager1[Amazon.Extensions.CognitoAuthentication.CognitoUser]' while attempting to activate 'WebAdvert.Web.Controllers.AccountsController'.
Microsoft.Extensions.DependencyInjection.ActivatorUtilities.GetService(IServiceProvider sp, Type type, Type requiredBy, bool isDefaultParameterRequired)
lambda_method(Closure , IServiceProvider , object[] )
Microsoft.AspNetCore.Mvc.Controllers.ControllerActivatorProvider+<>c__DisplayClass4_0.b__0(ControllerContext controllerContext)
Microsoft.AspNetCore.Mvc.Controllers.ControllerFactoryProvider+<>c__DisplayClass5_0.g__CreateController|0(ControllerContext controllerContext)
Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(ref State next, ref Scope scope, ref object state, ref bool isCompleted)
Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeInnerFilterAsync()
Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.g__Awaited|24_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, object state, bool isCompleted)
Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Rethrow(ResourceExecutedContextSealed context)
Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Next(ref State next, ref Scope scope, ref object state, ref bool isCompleted)
Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.InvokeFilterPipelineAsync()
Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
Microsoft.AspNetCore.Routing.EndpointMiddleware.g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)`
Anyone else run into this issue?

Seems like we can't inject it directly in the constructor.
To use the class, I have to cast it from SignInManager instead.
await ((CognitoUserManager) _userManager).CreateAsync(user, model.Password, validationData);


[Monaco Editor React Component] unavailable language server throws unhandled error

When the component mounts and the language server is not available, an error get thrown but not caught, which then also seems to cause the onLoad not to fire. It would be nice if the error simply got caught and logged by the component.
The error thrown:
'''
{
"message": "languageClientWrapper (Python Language Server Example): Websocket connection failed.",
"error": "No error was provided."
}
'''

Hi @Stainless2k thank you for reporting this. Would you like to contribute a fix? Otherwise, I will fix it at some point (next week or so).

@kaisalmen thank you for your quick answer, was on vacation the last weeks so i couldn't respond. I would love to contribute at some point, but sadly i don't have the time atm. I  assume since this is still open it has not been fixed yet? Just asking because I have trouble finding the release notes for the repo :sweat_smile:

@Stainless2k I haven't done it, yet. But, it will come...

@Stainless2k fix will be available in next release. I just committed the fix. It is part of https://github.com/TypeFox/monaco-languageclient/pull/600 due to this: https://github.com/TypeFox/monaco-components/issues/66

thank you :+1:


Multiple things could slowdown drastically TcpSyslogMessageSender

TcpSyslogMessageSender has three things that need to be change in order for that class to be more reliable regarding performance

Connection timeout should be configurable and have a default value of 5ms (or even less)
Inner retry mechanism should be remove or be configurable
sendMessage should not be a synchronize method (actually one thread at a time can sent a log and could lead to huge slowdown when there is high volume of event being log)

my recommendation is to start using non-blocking socket and remove the synchronization in order to avoid any performance issue

Connection timeout should be configurable and have a default value of 5ms (or even less)

Please see TcpSyslogMessageSender#setSocketConnectTimeoutInMillis(int socketConnectTimeoutInMillis)
https://github.com/CloudBees-community/syslog-java-client/blob/syslog-java-client-1.1.7/src/main/java/com/cloudbees/syslog/sender/TcpSyslogMessageSender.java#L256

Inner retry mechanism should be remove or be configurable

Please see TcpSyslogMessageSender#setMaxRetryCount(int maxRetryCount)
https://github.com/CloudBees-community/syslog-java-client/blob/syslog-java-client-1.1.7/src/main/java/com/cloudbees/syslog/sender/TcpSyslogMessageSender.java#L260

sendMessage should not be a synchronize method (actually one thread at a time can sent a log and could lead to huge slowdown when there is high volume of event being log)

2 key ideas that motivated this choice are:

For reliability & scalability, users can introduce a syslog forwarder on their infrastructure, close to the emitter.
The UDP sender could solve some scalability problems ... with some downsides


mobile header redesign

@chrislorch redid the mobile nav after we talked yesterday. Be sure to pull these changes in locally before deving.


Conflict between camel-http / camel-http4 components

I made a brief reference to this back in issue #157.


Report error when bluetooth is turned off.

We are using the DFU library, but our testers found out that when in the middle of ota if user turns bluetooth off the process gets stuck and nothing is reported from the library to the client.
We added this line to report the error to the client when bluetooth is turned off. When bluetooth is turned off the error is reported to the client and the cyclic reference is released. Then in the client is responsible to prompt the user to turn off bluetooth again and restart the ota process.

Thank you for the PR. Nice catch.


Thumbnail overflow on Firefox

A page has several thumbnails in a column. This column is in a row, which row is in a panel. The panel itself is last in a container.
For some reason which I cannot understand, the thumbnails render properly in Chrome (Opera etc) but not in Firefox 36. In this case the thumbnails take up a large size overflowing the row.
This is the markup of the page (from Django):
http://www.bootply.com/ZOS8y2vHPE
I was wondering if there is something wrong with my markup (most possibly) or there is something off with twbs.

Hi @Wtower!
You appear to have posted a live example (http://s.bootply.com/render/ZOS8y2vHPE), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 432, column 23: Element style not allowed as child of element body in this context. (Suppressing further errors from this subtree.)
line 435, column 84: Element link is missing required attribute property.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Removed irrelevant django-toolbar markup which seemed to be causing a validation issue:
http://www.bootply.com/DcSCcJxJzV

Hi @Wtower!
You appear to have posted a live example (http://s.bootply.com/render/DcSCcJxJzV), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 9, column 9: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.3.1
line 437, column 9: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.3.1

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Now using latest bootstrap version:
http://jsfiddle.net/7wfmnbbq/

Removed some irrelevant markup from tests that might have been confusing (apologies for that); latest fiddle:
http://jsfiddle.net/fdab1jgc/

Which OS are you using?

I can confirm the thumbnail images being giant on medium-and-larger screens on OS X Firefox.
This is because you are using <fieldset>s, which have weird legacy behavior (min-width: min-content) in HTML. Here's a version with <div>s instead that works fine: http://jsfiddle.net/5xj1hpc9/1/show/
For more info and another workaround, see the "Browser compatibility" & "Bugs" sections of https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Fieldset
Closing since it's not feasible to do anything about this at the Bootstrap level.

Great. Many thanks for your accurate and swift response.


Add images to collections

I'd consider collections the proper way to group related models, so it would be nice to also add preview images to collections to see what this collection is about.

I definitely want to do this! We have an issue for it already in #1114, so I'll close this one if you don't mind! 😃


HTML syntax isn't colored correctly inside markdown headers

Not only tags, none of the markdown features are highlighted in headers:
# `code` **bold** _italic_ [link](https://www.google.com)

Fixed by https://github.com/microsoft/vscode-markdown-tm-grammar/commit/399ff6f608a7bef3f68713be23cdcb4c6d475804
We can see what the feedback is. It generally looks ok in the default dark+ theme, but in other themes it makes the header more difficult to tell apart from normal text

Theme authors can still make 1-color headers with the new grammar:
{
    "scope": [
        "entity.name.section.markdown",
        "markup.heading punctuation",
        "markup.heading entity.name.tag",
        "markup.heading markup.inline.raw",
        "markup.heading markup.bold",
        "markup.heading markup.italic",
        "markup.heading string",
        "markup.heading markup.underline",
    ],
    "settings": {
        "foreground": "#EE9900"
    }
},

It might look better in the future if vscode were to support background color #3429

@usernamehw I think that one unfortunately has some architectural issues in monaco blocking it ☹


Consolidate country list endpoint usage to one #254

Brief summary of the change made

Are there any other side effects of this change that we should be aware of?
Describe how you tested your changes?
This PR enables the removal of redundant usage of multiple of country list endpoint. The goal is to only have one country list endpoint i.e /countries
Partial work on  #254

Pull Request checklist
Please confirm you have completed any of the necessary steps below.

[ ] Meaningful Pull Request title and description
[ ] Changes tested as described above
[ ] Added appropriate documentation for the change.
[ ] Created GitHub issues for any relevant followup/future enhancements if appropriate.

In subsequent PRs we can then remove the usage of /country-list endpoint entirely.


"Internal Server Error" when connecting via Github Actions

We're seeing a consistent error the last 24 hours with our Github Action ZeroTier connection:
⏁  Installing ZeroTier
⏁  Authorizing Runner to ZeroTier network
Internal Server Error
⏁  Joining ZeroTier Network ID: ***
200 join OK

At this point the step just hangs until timeout. Is this something we can address on our side somehow? We aren't detecting any other problems with our ZeroTier connection apart from our Action, which we've been using for a long time now. Any feedback / ideas appreciated.

same.

Update: I made a fork to do debugging - https://github.com/twfahey1/github-action - Oddly, with only echoing out and debugging parameters in place... it succeeded... Confirming it wasn't a fluke...

Ha, interestingly... That seems to be working now! Will update if anything changes, but, I just wired up my fork across our suite of jobs that implement several back-to-back ZT connections, and it went off as normal.
I noticed that in the output with my debugging, I'm now seeing an initial failure, then success...
⏁  Authorizing Runner to ZeroTier network
DEBUG: API URL: https://my.zerotier.com/api/v1
DEBUG: Network ID: ***
DEBUG: Member ID: [the right data]
DEBUG: Auth Token: ***
DEBUG: Constructed URL: https://my.zerotier.com/api/v1/network/***/member/****
DEBUG: GitHub Member: Zerotier GitHub Member 'the right data here'
DEBUG: GitHub Server URL: https://github.com/
DEBUG: GitHub Repo: The right repo
DEBUG: GitHub Run ID: The right num
Authorization failed. Retrying in 2 seconds... (Attempt 1 of 10)
{All expected JSON appears now}
Member authorized successfully.

I'm at this point assuming something is happening along the lines of the initial authentication is failing, and perhaps the Linux environment is such that the retry logic is not working as expected... and somehow by echoing things out, we're working around this error?
When I have more time, I could test out a simple echo of "Initiating..." or something, which might be more mergeable if a better fix isn't available....

To be clear, I replaced instances of zerotier/github-action@v1 with my fork and branch I made the updates on: twfahey1/github-action@main

I made a PR recently and now they have a 1.0.3 version of the action.
@twfahey1 would you mind testing it?
I also see that there's an initial Authorization failed but I don't know why.
Also, maybe it would be useful if you could make a PR with your debug strings with the conditional ${{ runner.debug }} variable?
So if the action workflow is run in debug mode we can check what is happening?

Thanks @orensbruli ! I realize now that I was using v1... so with your implementation in main, I was seeing the fix in my fork! It looks like using v1.0.3 all is well. I have moved our implementation back to this explicit version


Rotate more parameters during HexBlock.rotate

A follow on from #1533
when Block.rotate is called, the only thing it really does are

Updates the orientation parameter according to the rotation
Updates the pinLocation parameter providing "1-d spiral-based" location of each pin
Updates the displacementX and displacementY parameters

(Aside: why does rotation change block/assembly displacement?)

Any parameters stored on blocks or their pin-like components that should be rotated are not. We should rotate some parameters so the data layout matches the orientation of the block
What to rotate
The motivation for a block being rotated is its parent assembly is being rotated. Presently, ARMI rotates assemblies via a fuel handler module, e.g., armi.physics.fuelCycle.assemblyRotationAlgorithms.buReducingAssemblyRotation
https://github.com/terrapower/armi/blob/f5267e0cc332ac9c97e8fe1c4e49e28076455399/armi/physics/fuelCycle/assemblyRotationAlgorithms.py#L50-L52
In nuclear reactors, assemblies are moved or shuffled during reloading outages. The logic above emulates the rotation of assemblies, potentially during a shuffling operation, but still during an outage. One would expect a physics solve to occur after rotation, and before someone does something with the assemblies (e.g., plugins accessing the data model). Data like pin fluxes, linear pin power, etc. would be overwritten prior to being called after rotation. However, data like composition, displacements per atom (DPA), are cumulative and would need to be handled such that the data for the rotated pins rotates correctly.
I think we will need some logic to identify parameters on blocks and components that are rotatable and should be rotated. My current plan is to leverage Category.pinQuantities and Categories.cumulative and develop a way to rotate those data arrays.

I'm going to add the rotatable category for parameters. My rationale is this is an explicit way to embed the logic of what parameters need to be rotated. Most parameters with the cumulative and pinQuantities will be rotatable, but some non-cumulative block parameters will also need to be updated (e.g., Block.p.orientation).
Additionally, some corner and edge parameters must be updated (like pointsCornerDpa) but not all corner and edge parameters should be updated. The logic that exists right now is any parameter on the edge or corner gets rotated
https://github.com/terrapower/armi/blob/5a41b25fe9f262acb03347923443c4a2808837db/armi/reactor/blocks.py#L2041-L2042
This is probably fine. You're likely to have a new physics calc soon after rotating that updates and overrides things like pointsCornerDpaRate that are more instantaneous and not cumulative.

Things we need to rotate / update

Block.p.orientation - triplet (x,y,z) for rotation of the block around the x, y, and z axis
Block.p.displacementX and displacementY - pair of floats that may or may not exist but will both exist for the "Assembly displacement in the X/Y direction"
Duct corner / edge parameters - vectors of six scalars for values that exist at the corners and edges of hexagonal blocks.
Pin-like parameters - things like percentBuByPin and linPowByPin and other data that are defined on blocks and components that should have data for every pin in the hexagonal block. Could be scalar data per block but could be multi-dimensional (e.g., pinMgFluxes)

We have unique callouts and methods for 1-3 already. All that remains, I think, is to handle logic for 4. Which could be similarly handled in a separate method

Closing in favor of the approach from #1939


edge cases in matlab handles nan, and in python i have to cast them to 0

this creates edge differences

To elaborate, the issue is that the Matlab implementation explicitly sets to "badval" any location that is outside of valid locations.  The Python implementation may be more forgiving than this


More items to be added in application inventory filter list

This might be in progress so just a placeholder bug .

As mentioned in https://issues.redhat.com/browse/TACKLE-8
In Application Inventory page :
"The list must be filterable by Name, Description, Business Service, Tag, Assessment status, Review status and Tag count. Comments is not a filterable field."

@sshveta that's right.
So far we can add only Description and Business Service filtering (since we have no other columns):

Description is in https://github.com/konveyor/tackle-ui/issues/68
Business service is in https://github.com/konveyor/tackle-ui/issues/51

I close this issue because we will keep trace of the work to be done in the issues mentioned above.


a fix for #14 and some minor tweaks i think might be nice

fix(#14): playback in private window
fix: remove external font nunito dependencies
feat: add shortcuts to show current song info, toggle favorite, toggle radio type, save song info

My bad. I looked over this PR and meant to test it locally but ended up forgetting. Looks good at first glance. Give me a day or two to test and I'll get back to you.

Thanks for re-opening and taking a look at this.
Hope you find it at least mostly ok.
By the way, i also have some local modifications to the preferences (aka. options) page ... which i think look a little nicer than the current one.
If you want, i can make a PR for that too.
Preview:

Sadly i found an issue myself with the jpop/kpop type switching, but because i already delete the original fork repo (because i wasent expecting a reply) now i can not modify this PR.
So i guess the simplest solution is if i'll just close this PR and open another with the fix.
Sorry for the trouble.

@pilar6195
You can preview and test the new fixed and modified version here:
https://addons.mozilla.org/en-US/firefox/addon/listen-moe-player/

I like to test my stuff under real conditions, since i've had some suprises with web-ext debug builds and sideloaded extensions.

I might have overdone it a bit with the modifications, since besides the styling and some of the core aspects of the radio object not much has been left untouched.

Sorry if some of the UI stuff looks a bit off. I am not that good with CSS.

After you give me some feedback and we make a couple of rounds preliminary changes and bugfixes then i'll see how to best make a PR out of it.

Could you create a PR with your new changes so I can look it over?

You can look it at it over here:
https://github.com/igorlogius/listen-moe-player


checkValidity() method on forms

checkValidity() method on forms

This issue was imported from Google Moderator
Moderator votes: +4
Issue added by Jeffrey on 2015-01-14
To vote this issue up or down, simply include +1 or -1 in your comment.

Part of http://caniuse.com/#feat=form-validation


Exception while applying catalog rule "adjust to percentage"

Bug report
Got an exception while applying a catalog rule, that should adjust the price to a given percentage:
Symfony \ Component \ Debug \ Exception \ FatalThrowableError (E_ERROR)
Class 'Webkul\Discount\Actions\Catalog\AdjustToPercent' not found
Open: /var/www/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php > Line 432

Preconditions
Framework Version: Bagisto v.0.1.8
Latest commit 3eb7d86
Steps to reproduce

Create Catalog Rule

Customer Group: General
Channel: Default
Is Active: Yes
Start / End: left empty
Condition: Select by Categories -> select any
Actions: Apply -> Adjust price to percentage
Discount Amount: 10% (or any)
End other rules: No

Save the Rule
On Catalog Rule Overview: Click Button "Apply"

Expected result
After clicking the Apply button, a success message should appear.
The catalog rule should be applied to the chosen category, i.e. the price of that category's products should be decreased by the given percentage.

Log: environment_and_details.txt

Actual result
After clicking the Apply button, the rule is not applied, but an exception occurs:
Symfony \ Component \ Debug \ Exception \ FatalThrowableError (E_ERROR)
Class 'Webkul\Discount\Actions\Catalog\AdjustToPercent' not found
Open: /var/www/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php > Line 432

@AnnikaWolff
We are revamping the entire cart rule and catalog rule. By this month we will release new version with more stable cart rule and catalog rule.

Hello @jitendra-webkul
Alright, thanks for the answer. I am looking forward to it :)
If you want to close this issue, it is fine by me.
Best regards!


Cant change quality settings youtube

Here is an example where I want to force set the quality of a youtube video: https://codepen.io/anon/pen/MPpMqa.
It totally ignores the set default value of the quality option. I really need plyr to force it to play in 1080p despite of the speed of my connection.

Yep. It's a bug with the YouTube API unfortunately. There's a plan to remove support for it as their API is useless. You request a quality level but they never actually set it.


Cannot render  with SSR

Background
One of the big reasons to use server side rendering is to make the site accessible to crawlers and bots, which often look at the the metadata in <head> to determine the site's contents. It is crucial to have the <head> fully rendered when HTML is returned on first render
Problem
Right now, there's no (ergonomic) way to render the <head>. Our examples only show how to render the body. Trunk also interferes because index.html is generated by it and not us.
Possible Solution
Trunk generates the artifacts like:
dist
├── index-8b7906476da3648f.css
├── index.html
├── ssr_router_hydrate-bb65b3ba22cdbb94.js
└── ssr_router_hydrate-bb65b3ba22cdbb94_bg.wasm

The server would:

Read the index.html
Modify the head to keep all the link tags and such (for assets) and add all the elements generated from the ServerRenderer
Send the updated HTML

Drawbacks
We can't stream the response. We would need the server to obtain the entirety of HTML from ServerRenderer, parse it, take the <head> from it and modify the trunk generated index.html accordingly.
Additional Requirement
It should be possible to set the <head> from anywhere in the component tree, in a vein similar to next/head

There is already solutions like: Bounce Helmet that provides this functionality (No SSR support yet).
To implement this, you need a state management logic to deduplicate / merge head state and diffing rendered elements.

If we do not provide the state management logic to public, we have some code that can be useful to users kept for ourselves.
If we provide a state management to public, then it will become the official state management library which will hinder any third party state management libraries (yewtil::store).

Personally, I do not see an issue with this remaining as a third party solution at the moment.

I don't mind it staying a 3rd party library but I would like it if the ssr_router example and docs described how to set <head>. It's an important piece that is currently missing from Yew

This would definitely be nice to have, has any progress been made on this addition so far or does it need someone to take the lead?

I think Bounce Helmet already supports rendering head with SSR?


NPS: avoid showing survey during tutorial

@xpalaum I think this is already done. It was covered by this PR: https://github.com/decentraland/explorer/pull/678

Oh, I had agreed with frami and Boris to add the issue to the next sprint. No problem, but unfortunately, after revieweing the code, it is not what I was expecting :S
Expected:
Keep triggering the survey from setLoadingScreenVisible(), but add the condition that tutorial has to be finished. Basically avoid showing the survey on the loading screen within the tutorial, and instead show it on a subsequent loading screen after the tutorial is well over.
Current:
The survey is now out of loading screen and into the end of the tutorial which is not ideal.
Unless I'm missing something let's change it one more time, this time for good.
@BrianAmadori @2fd @moliva

Covered by: NPS disabled.


Create a BIDMat dict for moodids

Can someone generate a BIDMat dict which maps the moodids to their strings? Should be 135 entries. Zero should correspond to "not specified" in someway, while 50 and 94 are not used by LiveJournal,  @anasrferreira or @lambdaloop willing to volunteer?

Done. See sbmat file: /var/local/destress/moodDict.sbmat


Does this library support react native?

I want this work for RN, so cool~

Don't suppose so because it works on HTML Canvas only at the moment.
I have plans to add native SVG rendering - maybe there's a way to integrate SVG into native.

Oh, thanks a lot! Really awesome work you have done!


Fix bug in get_weights when using a2c loss.

Calling get_weights() when using the a2c loss in policy_gradient currently throws: "AttributeError: 'PolicyGradient' object has no attribute '_q_values_layer'". This is because there is no _q_values_layer when using a2c loss, this is instead replaced with _baseline_layer. Updated the get_weights() function to check which loss is being used and append the appropriate weights. Also, added a2c loss into the enum in the kuhn_policy_gradient example.

Thanks! @fzvinicius , can you take a look and import when satisfied?

@fzvinicius this seems pretty straight-forward: I will import it and add you as a reviewer.


Stops Updating

The display stops updating and it does not appear to be at any specific time. The code is still running, but the display as stopped getting the information. A quick ctrl+c and then the up arrow, enter and the code starts all over again and the display updates.
Do you have any idea what might be happening?
Thanks

I haven't had that issue. I'd recommend modifying the code so it only runs once and stops. Then set up a scheduled/cron task to have it run at a certain time. I've been meaning to do that, but I've just not had the time.


Determine a name for our bot

This will be something to sign controllers with so other players can easily identify when the bot is being used by NCPs (non-coding players)

Piky


Legalize generated Android package names so they don't contain '-'

To prevent this error when building an app with the '-' character in its name:
./build/Development-23/AndroidManifest.xml:2: Tag <manifest> attribute package has invalid character '-'.

See http://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html

OK, I confirmed a specification about package name in AndroidManifest.xml

package
A full Java-language-style package name for the Android application. The name should be unique. The name may contain uppercase or lowercase letters ('A' through 'Z'), numbers, and underscores ('_').
https://developer.android.com/guide/topics/manifest/manifest-element.html

I think If accept '-' letter, we will also need to accept another letters such as '!'
gsub(/\W/, '_') is better for this issue, I think.
>> name = 'foo test-android!'
=> "foo test-android!"
>> name.downcase.gsub(/\s /, '').gsub('-', '_')
=> "foo test_android!"
>> name.downcase.gsub(/\s /, '').gsub(/[^a-zA-Z0-9_]/, '_')
=> "foo_test_android_"
>> name.downcase.gsub(/\s /, '').gsub(/\W/, '_') # \W == [^a-zA-Z0-9_]
=> "foo_test_android_"


Add censoring to debug logs

Agreed
Found a link here describing what the different parts of the vin describes https://en.wikipedia.org/wiki/Vehicle_identification_number
We can probably reuse the censor tools in #94


fix: convert bytes to megibytes

there is issue that default code convert bytes to megabytes, but says it is megibytes
it is quite misleading and hard to reason about capacity

Thanks for fixing this! Not sure how I missed this one for so long.


read motion setup in Maxwell2D

read MotionSetup when getting boundaries data in Design

check "ModelSetup" in Q2D -> test_30_Q2D.
Check other solvers.


Older version prior to commits work. I think for the newest pull I made for a different computer, I am getting an error regarding resources in lib. It fails to import for the the latest pull.

OS:
PyQt version:

According to the latest change, you have to call make, make qt4py2, or make qt5py3

Okay thank you very much! I am doing this in anaconda and will try this now.


Issue when uploading data (using formdata)

In what area(s)?

/area runtime

What version of Dapr?

0.3.x

Expected Behavior
I build an upload function on NodeJS (using simple multer middleware). Everything is running fine while I'm not plugging dapr in.  I hope that dapr sidecar would forward through the request to my nodejs app without any issues.
Actual Behavior
Then I plug dapr sidecar into the Kubernetes deployment, it was throwing error 413 request entity too large in my nodejs application.
Steps to Reproduce the Problem

Build a simple nodejs application with upload function.
Bind sidecar to the app
Try uploading using any tools (such as Insomnia) then it will throw the error  request entity too large in nodejs app

Are you calling a Dapr API to invoke your app? If so, can you post here the details? (URL, payload).

Hi Yaron,
I've created a sample application to show the error while uploading (multi-form data) if using Dapr.
The sample is at repo: https://github.com/trumhemcut/dapr-upload. You can just deploy the app by
$ kubect apply -f ./node.yaml

After that, expose the dapr service to public port via LoadBalancer:
kubectl patch svc nodeapp-dapr -p '{"spec": {"type": "LoadBalancer"}}'

Node application is running at port 3000, while Dapr service is at port 80. Then we can quickly test the app:
Without Dapr:
$ curl -X POST -F 'dataField=A data field' -F<EMAIL_ADDRESS>http://localhost:3000/upload
{"fileName":"14cbdefd870772a654bd313bbf72b6a7","dataField":"A data field"}

With Dapr:
curl -X POST -F 'dataField=A data field' -F<EMAIL_ADDRESS>http://localhost/v1.0/invoke/nodeapp/method/upload
<!DOCTYPE html>

<meta charset="utf-8">
<title>Error</title>

TypeError: Cannot read property 'filename' of undefined
    at app.post (/app/index.js:10:38)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/app/node_modules/express/lib/router/route.js:137:13)
    at multerMiddleware (/app/node_modules/multer/lib/make-middleware.js:18:41)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/app/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/app/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at /app/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/app/node_modules/express/lib/router/index.js:335:12)

If any unclear info the just let me know. Thanks.

@trumhemcut Is this still repro with latest versions of Dapr. Please feel free to reopen if its repro now.


Add Registry Deserialization

Draft PR for deserialization of Registry.
This is nearly done, just need to finish deserializing into Registry and then write some tests. I will wait until after #36 is merged to take out of draft state, since that's dealing with re-organizing the JSON format and may effect how deserialization is done. I may rebase to fix some of the commits before pulling out of draft mode (and remove serde_json dep).

Oh good job you put this draft up - I was in the middle of doing it too! Looks good.

the  deserialization works. I also added a couple methods returning iterators on Registry and one on Interner, so that the  type definitions can be accessed when using type-metadata as a library


⚠ Slack has degraded performance

In 52f53a4, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 169 ms

Resolved: Slack performance has improved in d172dd8 after 2 hours, 53 minutes.


[shardformer] Unit test

📌 Checklist before creating the PR

[x] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[x] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number
fixed #3927
📝 What does this PR do?
Add unit test of shardformer
💥 Checklist before requesting a review

[x] I have linked my PR to an issue (instruction)
[x] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[x] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

Local test of shardformer:


fix: make sure the result from the conv1d is the same shape coming out

This PR makes the output size of conv1d's that have even filter sizes the same shape as the input. It does this by padding one less on the front end to match tensorflow.
Tested by training a tensorflow model, converting it into a pytorch model by extracting weights from the checkpoint and comparing the results (both full prediction results as well as internal activation values) between the models. They match. Also tested in cases where the input time dimension was even and when it was odd

It makes more sense to me to change the behavior of Conv1DSame and then just use that primitive instead on even filters


Minor improvements

Fix linting
Improve portability

Requires ba-st/Hyperspace#44 to be merged upstream

Requires ba-st/Hyperspace#44 to be merged upstream


go.mod: Update to go 1.20

Building with go 1.19 fails as some deps require go1.20. This will also match what is defined in flake.nix.

Majority of changes beyond the go 1.20 line were the result of go mod tidy and a followup gomod2nix.
I had attempted a bump to go 1.21, but that presents other upstream dep problems through the tailscale.com module as the gvisor dep renamed the package bufferv2 to buffer. That's present even in tailscale 1.48.1.


feat(grid-js): Add grid variables as JS

** Please provide as much detail as possible, but feel free to remove irrelevant sections **
The new salary range component for SRP requires the grid variables to be accessible in the component as the library being used is yet to allow className to be passed. Reference: https://github.com/seekinternational/srp-react/pull/309
BREAKING CHANGE:
RFC URL:
MIGRATION GUIDE:
EXAMPLE USAGE:
import { gridBase } from 'seek-asia-style-guide/theme/layout/grid'

Can we use CSS variable in CSS-in-JS environment? I'm wonder if we can have one variable say :root { --gridBase: 4px;} for global scope and override it in component level.

@patrickliu-sa The hard part is to access the variables in JS files. Maybe I can try something like this https://github.com/michaeltaranto/less-vars-to-js which might be able to remove the need to copy the values as raised by @mengtzu

@syakiruzair Actually you can get and set css variables with native javascript method ;)
https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_variables

@patrickliu-sa I saw that earlier but can we use that for less as well?

I realised that there are already implementations using less-vars-to-js in this style guide. So I am just following the approach.

If we use less-vars-to-js in the past, can keep using that. We can explore more possibility of using CSS variable in the future.

@syakiruzair this is good to merge whenever

Now then @mengtzu. Thanks :)


Instructions for citing Predicer?

@sundelld currently, there don't seem to be any instructions for citing Predicer?
I'd recommend creating a VTT pure entry for it, which could be used for the time being. Later, we might want to switch citing to a publication or create a Zenodo repository with a DOI through GitHub-Zenodo integration, if I can get the admins of vttresearch to enable it.
For now, I'll use the following citation (based on what PURE formatting will most likely look like):

Sundell, D.; Pursiheimo, E. (2022). Predicer: Predictive decider for actors making decisions over multiple stages. Software, VTT Technical Research Centre of Finland. https://github.com/vttresearch/Predicer

README.md now has a link to the Optimization and Engineering paper. Close?


Function to get personal access token for Gitlab API

Hello, please review this patch (relates to #265, #267). The introduced solution is based on HTML parsing and may seem dirty. However, as far as I could check it, it works with the most recent versions of Gitlab: 10.1.4 and 10.2.2.
Also, if the proposed solution is acceptable for you, I would like to discuss the way of testing this. Should it be integration tests based on Travis CI? Or shall we confine ourselves to unit-testing only?
@LiangXugang I will appreciate it if you give some feedback on this function (whether it meets your needs).
Kind regards,
Vitaly

Hi @vitalyisaev2, I'm sorry but I don't think I would like to merge this PR.
The more I read up on it, the more I get the feeling you are trying to do something that isn't supported (anymore). Please join the discussion in issue #267 so we have one location were we can talk about this feature and why (and if) it is needed...

Sorry @vitalyisaev2, but I'm not going to merge this. As you mentioned yourself this approach feels very dirty and hacky. I do understand your challenges so let me see if I can come up with another solution to your problem...


python3Packages.pyclip: init at 0.5.4

Motivation for this change
Required by waydroid for sharing the clipboard between host and container.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

pasteboard could be a propagatedBuildInputs for darwin.

Sorry, I'd rather not package pasteboard right now. I'm packaging pyclip because it's an optional dependency of waydroid.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 144031 run on x86_64-linux 1

2 packages built:

python38Packages.pyclip
python39Packages.pyclip


[Example] thenlper/gte-large (a BERT-based model) not working when use-pth flag is set

cargo run  --example bert --release --features cuda -- --model-id thenlper/gte-large --prompt "Here is a test sentence" --use-pth
results in:
Error: cannot retrieve non-contiguous tensors Layout { shape: [512, 1024], stride: [1024, 1], start_offset: 31254528 }
It works fine if the flag is not set. This doesn't seem like intended behavior.
This is on 3.2 of candle-core, candle-nn, and candle-transformers.
My GPU is an RTX 4090
Driver Version 535.129.03
CUDA version 12.2
Please let me know if there's any other information or any other way I can be helpful. Thanks!

Thanks for reporting this issue, I've just merged #1546 that should hopefully help with this (the culprit was that this checkpoint uses tensors stored with an offset which is a bit uncommon and wasn't supported).


向量检索版sdk支持java异步调用

 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.


Fix bug in storing CVs

Resolves #489
Actually this reduces security a little and removes the check for locality. I was not able to reproduce the error so I cannot test if it works now.

It should be trivial to reproduce the error. Make a directory not in the OPS path, like $HOME/my_test_directory. Paste the example from #489 into a file called test_file_locations.py in that directory. Type nosetests. You should get the error.

Thanks will try

Needed to copy some code. But now it seems to work. I just removed the check for locality. Worst thing that can happen is that it will fail for an unknown reason. But blocking it to run working versions does not make sense. I will check if you can load the stored version correctly though

Works. Nice. That was easy. I assume the local stuff had some security and legacy reason. I actually do not remember why I checked for that. Does not seem to make sense.

Restarting...

I remember why I did the local check. Actually the solution was to first try the fall_back save solution and if that is not working try the hard way using marshal. That does the trick.
The original local check was to store the package and the function if it was not local. Now it tries to figure out the package (like mdtraj) and save only the function name. If not a known package then use saving by bytecode.

Remarking again: this is assigned to me for review, but currently has conflicts with master. @jhprinz : can you fix conflicts? Then I'll review it!

Sorry, did not see that. Was a conflict in notebooks that appeared merging other PRs.

No worries. Also, the merge makes it easier to review, since it previously had other PRs mixed in the diff.
It's not clear to me that the test you added would identify this problem. (In fact, I'm not sure that there is an easy test for it, since it would need to be tested outside the test suite.) However, it looks like this should basically fix it -- and you did test it on an external file, right?
A couple line notes coming: should be quick fixes.

ready to review

Looks good! I'll have to keep in mind that we have the word-wrap now... might be useful in the future to also give it an option to indent a certain number of spaces.
Merging!


Fix documents for zip and unzip.

Mismatch with code. And some docs seem to be misplaced.

A follow up to #2159.


Serve React Router

I was able to serve a react app which I created with "create-react-app", but how I serve an app which is created with "create-react-app" and has react router? Is this possible?

@Dominik-Robert from my understanding, you just need to serve index.html using wildcard, routing will be handled on the frontend by react-router. If u're talking about server-side rendering for react app, this is not the library that serves such purpose

You need to have an fallback to index.html in case of direct accessing your SPA routes. If /some/page is handled by react and not by the server, the server needs to serve the index.html and let then react itself decide what to do with the path at the visitors browser.
In case of SEO and initial load times it is a good idea to let a node server pre rendering the SPA. Thats called server side rendering (SSR). This way your go application will only serve the api, not the static files or the SPA.


[Inlets] Inlets guide points to non-existent releases page

The Inlets guide points to the project releases page to download a pre-buit binary for Inlets.
That page does not contain any releases, so we should either update it with the appropriate page, or remove the reference.
cc @alexellis

Hi @radu-matei the binaries now exist here: https://github.com/inlets/inlets-pro/releases
The guide needs a couple of tweaks as inlets is now a paid product, but there is a monthly tier with a lower price for personal users.
I'd be happy to take a look at updating this, it's a relatively trivial change that's needed.
Alex

Is there still the option to build your own binaries?
Regardless, I personally think we should update the guide and definitely keep it, as it is a very helpful addition, but I defer to @thomastaylor312 and / or @bacongobbler.
Thanks!

Yeah, I think it is still useful to have! I would just mention in the doc update that it is a paid tool so it doesn't catch anyone by surprise


Alarmo Card is not loaded

Hi, When I want to configure an alarmo card. I have the following error:
Invalid configuration
type: 'custom:alarmo-card'

You could try to clear your browser cache, for some users this caused the card to show up.
Otherwise, you should assign the entity to the card, see also here: https://github.com/nielsfaber/alarmo#configuration

Great,  it works.
I just added the entity attributes to make it work.
Thanks you
Le dim. 11 avr. 2021 à 05:56, Niels Faber @.***> a
écrit :

You could try to clear your browser cache, for some users this caused the
card to show up.
Otherwise, you should assign the entity to the card, see also here:
https://github.com/nielsfaber/alarmo#configuration
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nielsfaber/alarmo/issues/177#issuecomment-817242997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI3S34MVHCGXH7WOAIQNEETTIEMXFANCNFSM42WZ6SEA
.


Initial change for 7.3

@cvarjao keycloak theme bcgov-v2 has been updated, available for test at https://sso-22-devops-sso-sandbox.pathfinder.gov.bc.ca/auth/admin/test-1/console There are several themes for bcgov now, shall we clean that up as well?

Still need to clean up the OpenShift templates file name and keycloak.version in pom.xml


dat init returns 0 even if it fails

found this while making a local dat client in python
In [64]: call(["dat init --no-prompt"], shell=True)
Skipping dat init because there is already a dat here
Out[64]: 0

was there already a dat in that directory? if so this is by design (similar to git init in a git repo)

was there already a dat in that directory? if so this is by design (similar to git init in a git repo)

oh okay

oh okay


Use the correct class name in the log messages generated by Python runner actions

Previously, we didn't correctly retrieve the name of the class of the action so it was always incorrectly logged as "ABCMeta" (this is the parent class of each action).
Before:
result:
  exit_code: 0
  result: false
  stderr: 'st2.actions.python.ABCMeta: DEBUG    test log message

    st2.actions.python.ABCMeta: DEBUG    value=10

    '
  stdout: ''

After:
result:
  exit_code: 0
  result: false
  stderr: 'st2.actions.python.PrimeCheckerAction: DEBUG    test log message

    st2.actions.python.PrimeCheckerAction: DEBUG    value=10

    '
  stdout: ''

TODO

[x] Changelog entry
[ ] Test

Based on the feedback from a customer, I also decided to add new log_level runner-level parameter to Python runner.
With this parameter, user can control which log messages from Python runner actions are displayed and output to stderr. For backward compatibility reasons it defaults to debug.
The reason for adding it is that some actions depend on external libraries which output a lot of info under DEBUG log level and sometimes you just want to see info or above or similar.


"connecting..." indefinitely

Expected Behavior
With a config that contains
"privateKeyPath": "/some/path",
"passphrase": null

and a private key that is protected by a passphrase, when I SFTP: Sync To Local, I expect to receive a prompt that allows me to enter my private key's passphrase.
Actual Behavior
"connecting..." is displayed indefinitely in the bottom status bar.
Steps to Reproduce the Problem
1, Create an SSH private key

Create a configuration with the above options
Press F1
SFTP: Sync To Local

Requisites (important to diagnose the problem!)
Extension Logs
[set option sftp.printDebugLog to true, then reload vscode, reproduce the problem, get the logs(view -> output-> sftp)]
[info]: config at redacted {
    "host": "redacted",
    "port": 22,
    "username": "core",
    "password": null,
    "protocol": "sftp",
    "agent": "/tmp/ssh-5D4Vv4lQQE4L/agent.7053",
    "privateKeyPath": "/redacted",
    "passphrase": null,
    "passive": false,
    "interactiveAuth": true,
    "remotePath": "/redacted",
    "uploadOnSave": true,
    "syncMode": "full",
    "watcher": {
        "files": false,
        "autoUpload": false,
        "autoDelete": false
    },
    "ignore": [
        "/redacted/**/.vscode/**",
        "/redacted/**/.vscode/**"
    ],
    "configRoot": "/redacted"
}
[debug]: conncet to remote

Specifications

Version: 0.8.5 on VS Code 1.17.2
Platform: OpenSUSE Leap 42.3

Sorry. You must set  passphrase to the config file currently.


[android] onloadmore of scroller is not executed when loadmoreoffset is not set

Brief Description of the PR
On the web and iOS, do not set "loadmoreoffset" can trigger the loadmore event, but not on Android.
This pr solved it.
Checklist

[x] Demo

I have a same pr, but Maven read timed out in CI build.
Now I want to rebase the newest commit, but the repo have too many changes and I have too many conflicts.So I just delete my repo, fork the repo and pull the request again.

Please rebase to master branch, as there is a conflict.


Add "invalid word" popup in addition to the red circles

Some people have been unsure what the red circles mean when they submit an invalid word.
Let's add a small toast for a couple seconds that says "invalid word" when this happens.

Hi. I've fixed this. Please check pull request #66.


⚠ SceneTime has degraded performance

In 0b4f24f, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2186 ms

Resolved: SceneTime performance has improved in c45bddc after 6 minutes.


samples: net: zperf: Remove task_profiler as it is not used

The task_profiler is not needed nor used by zperf so remove
it from CMakefile.
Fixes #11051
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #11068   +/-   ##
=======================================
  Coverage   53.22%   53.22%
=======================================
  Files         217      217
  Lines       26733    26733
  Branches     5926     5926
=======================================
  Hits        14228    14228
  Misses      10086    10086
  Partials     2419     2419

Continue to review full report at Codecov.

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


Fix issue introduced in Change d866b74bc648a56a50909670691b8566cf2a45e3.

Carried the new parameter/field into the kick start parameters where it was
being sent to the underlaying DevAppServerMain. Of course that class doesn't
know about this parameter and doesn't care.
This change removes the addition of the parameter to the KickStartParams and all
related usage of it.
TESTED:

Discovered this by using the new disableDatagram property within an appengine
task declaration on a project that has a functional test that requires the
DevAppServer to be started. Server start would fail with a warning that an
unknown option (--disable_datagram) was being used.
Ran the same project against a local publication of the AppEngine Plugin built
from this patch and the test ran fine.

LGTM, merging in.


RedStone Oracles Addition to integration

I added RedStone Oracles description under the section 'oracles'.

removed the duplicate mdx file


public & test tenant tests in TenantTestCase?

I am looking https://github.com/tomturner/django-tenants/blob/master/django_tenants/test/cases.py and I don't see a public test schema, I only see a tenant test schema.
I understand how to write unit tests to test out the tenant test schema but I don't understand how to write public test schema. Can anyone give some points or help? I looked at the documentation for testing but it doesn't talk about public/tenant unit testing.
Thanks!

Eeee.... public schema is used as default one? So every not tenant test is just public schema test. Also you can explicit set public tenant schema like for normal tenant. It's just matter of changing search_path in db cursor.

This issue was solved in #86. Closing ticket.


Postgres Tuning

As of now our images do little things to tune PostgreSQL, tuning suggestions from https://nominatim.org/release-docs/3.7.0/admin/Installation/#tuning-the-postgresql-database are ignored.
We could add those suggestion as default settings to our postgresql.auto.conf and allow the user to override those with environment variables.
This should be done in start.sh to reflect changes on every startup. In addition we need to revert fsync & full_page_writes instead of deleting the config file after the import.
We could also place two .conf files into the conf.d folder (e.g. import.conf, tuning.conf and remove import.conf after the initial import.

I like @mtmail's approach of putting files into the conf.d folder.

Thanks @philipkozeny I'll try that tomorrow !
I have a couple runs with France data to benchmark against (sort of).

Hi, did a test today with France.
It went pretty well, everything is working fine.
Duration was 402 minutes, previous deployments took 405 and 409 minutes so I don't see a big increase in performance.
But environment variables are a big plus so I was able to increase easily the max_wal_size easily.
Thanks for that @philipkozeny  and @leonardehrenfried .

You can notice that I changed the urls to https because the firewall blocks http requests here.

Perhaps the tuning isn't so effective when you're not using a SSD.
Good catch with the http url. If you want to send a PR to update the documentation, we'd welcome that.

@leonardehrenfried I was not sure that http/https in urls was a bug or a feature ;)
I'll do that today or tomorrow.

I'm not sure the current approach of removing postgres-import.conf after successful import is working as intended. The file is removed at the end of init.sh, but this only removes it from the local filesystem of the container that did the import. If / when the container is restarted then the file is present again when the postgres service is started again.
I've just completed an import, restarted the container sometime later and have noitced that both fsync and full_page_writes are both off.

You're right. This is addressed in https://github.com/mediagis/nominatim-docker/pull/373#discussion_r967670285


Strange formatter behaviour { objectProperty}

Result of atom-strandard-formatter work:

 return Object.assign({}, { row }, common)

return Object.assign({}, { row}, common)

{ row} looks kinda ugly here

If you upgrade to 2.5.0 this should be fixed (assuming you're "Style" setting is "standard")


Turn off "something went wrong" message

I'm seeing the "Something went wrong" error message document in https://github.com/Microsoft/azuredatastudio/issues/3369 much more frequently with the result streaming feature.
As far as I can tell there is no user impacting issue beyond the annoying notifications.  This PR turns off the notification in the release/1.3 branch.  The prompt is still enabled in master so we can root cause https://github.com/Microsoft/azuredatastudio/issues/3369.

We should probably remove this code in general and just pass through the entire promise since this code is functional code not UI code.

@anthonydresser it makes sense to switch this from a notification to a console.log message, or something less invasive in master.  I'd like to understand why the error is occurring more frequently now though to ensure we're not hiding an underlying defect that may have other side-effects we haven't noticed yet.


binary.py usage guide

Hi @mikeboers,
I just discovered this neat library via Google and was hoping to use it to read a binary file as though it were in Maya ASCII format, but I can't quite wrap my head around whether that's actually what it's for?
The end goal is to list each node from a scene file. Would it be possible to provide a snippet of how to achieve something along these lines?
The final example on Read The Docs hints towards the possibility of reading names of shape nodes.

You can certainly parse the raw binary format:
parser = mayatools.binary.Parser(open_file)
parser.parse_all()

However, the binary format does not self-describe, so while you have the structure, you don't really know what tags mean what and what data type they are. You need to make simple scenes to figure that out. And then you will find that there is sometimes quite a bit of ASCII data within the binary file. :stuck_out_tongue:
Good luck!

Thank @mikeboers, I ran the above, but it didn't return anything. I tried .pprint() but only got a binary representation.
How would I retrieve the .notes attribute of something like the persp node in a scene? Is that possible?

Since I wrote this parser, the file format switched to allow for 64bit values (the file will start with FOR8). I'm not reading those correctly.
Try this other project -> https://github.com/agibli/sansapp/blob/master/maya/binary.py

Aah, I see. Thanks for the link, I'll give that a try.


mouse button do not work

Looking

Looking

Check out #11

Check out #11

Still need scroll buttons, but I'm not sure how to send them :)

Still need scroll buttons, but I'm not sure how to send them :)

@hh :+1:

Yeah! I'm happily using a Beagle Bone Black embedded in my keyboard (and using a wireless logitech mouse) with my laptop over bluetooth now.


Subclassing subStep breaks event management

Hello,
I need to maintain some link between my table line and the substep I create in Gantt, to use it when user clicks on it.
As there is no setData on steps, I made a subclass of substep  :
	public class TaskStep extends SubStep 	{ 		Task task;         }
With this kind of substep, Gantt is displaying without any trouble but click-event is broken : event object has no step ( event.getStep() always returns null )
I have the feeling this is something coming from uuid, so I tried to force uuid creation on my TaskStep object, but without any more success.
Regards

Task need to be Serializable, just like SubStep is. I suspect that Task is not serialized and therefore client side will fail and propably throws some exceptions and uses null value instead of TaskStep.
When you add new field into shared state object, which SubStep is, you need to also pay attention on how GWT can serialize it. Prefer using primitive types or their equivalent Object types. LIke String, long or Long, etc.
Putting entity or any other Java bean into shared state is not a good idea because it might lead into unexpected issues when changed entity/bean suddently breaks the serialization. And there will be unnecessary addition to the data being transered to the client browser.

Latest version 0.9.1 has Gantt.setData(AbstractStep,Object) and getData(AbstractStep). Try to use those instead of extending SubStep.


the example doesn't work?

if I run it with elm-reactor I just get some text, no map.

Yes this elm package only put the html tag on browser and the example make sure of it.
elm-reactor doesn't allow to run a package with an npm module.
You have to add the webcomponent https://www.npmjs.com/package/stencil-leaflet-component to make it works.
I don't no more time to work on it. You are free to fork it.


Other resources

In a while there are going to be a lot of resources available, and we should prioritize what to show and what not to show at the front page.
One suggestion is that we both have the following at the front page:

our own course-material.

Scratch, Python etc

combinations.

ex. when clicking Arduino, you currently go to Kodegenets Arduino course, but should go to a page where you can choose between different Arduino-resources, including Kodegenet.

others.

"Andre ressurser" which goes to a page where you can click at ex. "Code.org" and get redirected to studio.code.org etc.

Ideas?

I guess when getting a lot of resources one can combine them in a course, e.g. Arduino. Then "Bygg en radiobil - Kodegenet" is a lesson with specified difficulty. To further differentiate, one can add a icon(like glyphicon-new-window) showing that the resource is external.
One can also create a single course name "External" and put everything there.
Further then those approach, one will need to think about UI and organization, which will be opinionated if one does not have any numbers to lay on (we have x users on the python course, y users leave our page after 30 seconds, etc).
Maybe this discussion more appropriate in kodeklubben/oppgaver?


Wonky screen

Hey guys. Is CodeWars not really meant for iOS? Just have impossible time doing any ‘training’ once I hit the ‘train’ tab. That part of screen goes mental on me and can’t do anything. I wish there was an iOS app but there isn’t. Am I missing a setting somewhere? Thanks very much
Bruce Koken
8th kyu
┆Issue is synchronized with this Clickup  by Unito

Is this #1397 ?

Thanks


Test execution with qaf-3.1.0 is not saving any test-result which can be viewed using dashboard.htm

QAF Version 3.1.0
Steps To Reproduce
1. Clone "qaf-blank-project-maven" repo
2. Go to "pom.xml" and change the <qaf.version>LATEST</qaf.version> to <qaf.version>3.1.0</qaf.version>
3. Go to "/qaf-blank-project-maven/src/test/java/com.qmetry.qaf.example.test.SampleTest.java" and execute
Note:
When <qaf.version>LATEST</qaf.version> is used, it is using qaf-3.2.0b and report is working as expected without making any other changes.
Also qaf-3.0.0 is working as expected, where we just need to provide the chromedriver.exe path in application.properties.
Actual behavior
Is the issue reproducible on runner?

[ ] QAS
[x] Maven
[ ] Gradle
[ ] Ant
[x] Eclipse

Test case sample

Please, share the test case (as small as possible) which shows the issue

Thanks @cjayswal for quick reply. I have downgraded the Eclipse TestNG plugin from 7.4.0 to 6.14.3 and it worked for me as well.


⚠ Jerry Savelle Ministries International Australia has degraded performance

In 496ca72, Jerry Savelle Ministries International Australia (https://jsmiaustralia.org) experienced degraded performance:

HTTP code: 200
Response time: 8919 ms

Resolved: Jerry Savelle Ministries International Australia performance has improved in 95b1fb6.


Python.Runtime.PythonException: 'dict' value cannot be converted to Skender.Stock.Indicators.Quote

What happened?
Python.Runtime.PythonException: 'dict' value cannot be converted to Skender.Stock.Indicators.Quote
The above exception was the direct cause of the following exception:
System.ArgumentException: 'dict' value cannot be converted to Skender.Stock.Indicators.Quote in method Void Add(Skender.Stock.Indicators.Quote)
---> Python.Runtime.PythonException: 'dict' value cannot be converted to Skender.Stock.Indicators.Quote
--- End of inner exception stack trace ---
The above exception was the direct cause of the following exception:
System.AggregateException: One or more errors occurred. ('dict' value cannot be converted to Skender.Stock.Indicators.Quote in method Void Add(Skender.Stock.Indicators.Quote))
---> System.ArgumentException: 'dict' value cannot be converted to Skender.Stock.Indicators.Quote in method Void Add(Skender.Stock.Indicators.Quote)
---> Python.Runtime.PythonException: 'dict' value cannot be converted to Skender.Stock.Indicators.Quote
--- End of inner exception stack trace ---
--- End of inner exception stack trace ---
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\Administrator\e.py", line 29, in
results = indicators.get_zig_zag(quotes, EndType.CLOSE, 3)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\site-packages\stock_indicators\indicators\zig_zag.py", line 39, in get_zig_zag
results = CsIndicator.GetZigZag[Quote](CsList(Quote, quotes), end_type.cs_value,
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\site-packages\stock_indicators_cstypes\list.py", line 35, in new
deque(map(cs_list.Add, sequence), maxlen=0)
TypeError: No method matches given arguments for List`1.Add: (<class 'dict'>)
Code usage
import pandas as pd
import matplotlib.pyplot as plt
from stock_indicators import indicators
from stock_indicators import EndType

def load_historical_data(filename):
    data = pd.read_csv(filename)
    data['timestamp'] = pd.to_datetime(data['timestamp'])
    quotes = []

    for index, row in data.iterrows():
        quotes.append({
            'timestamp': row['timestamp'],
            'close': row['close'],
            'high': row['high'],
            'low': row['low']
        })
    return quotes

quotes = load_historical_data('Price.csv')
results = indicators.get_zig_zag(quotes, EndType.CLOSE, 3)
results_df = pd.DataFrame(results)
print(results_df)

plt.figure(figsize=(14, 7))
plt.plot(results_df['timestamp'], results_df['zigzag'], label='ZigZag', color='red')
plt.plot([q['timestamp'] for q in quotes], [q['close'] for q in quotes], label='Close Price', color='blue', alpha=0.5)

plt.title('ZigZag Indicator with Close Prices')
plt.xlabel('Date')
plt.ylabel('Price')
plt.xticks(rotation=45)
plt.legend()
plt.grid()
plt.tight_layout()

plt.savefig('zigzag_plot.png')
plt.close()

Log output
No response

@Ugroon can you attach the Price.csv file associated with this error?  There's likely some unexpected data being imported from it.  Being able to see it will help us understand the problem and find the right solution.

@Ugroon It's the wrapper of C# library, so the type matters. That means quotes must be a iterable of Quote, which is subclass of Skender.Stock.Indicators.Quote
from stock_indicators.indicators.common.quote import Quote

@Ugroon we'll assume you've resolved the problem and close this issue if we don't hear back from you soon.  Let us know how it's going.

If you have further question, please re-open anytime.


Power Query SDK - Query Results are editable

Is there a point in having editable query results (even check boxes) ?

Might be a good idea for small data sets. Haven't tried embedding this into a data connector output, but it's worth a shot.
Have a look at Table.EnterDataFormula
https://github.com/Hugoberry/PowerQueryExtensions/tree/master/src/Table

I think it's just a side effect of the control we're using. Changing the values doesn't do anything, but at least you can copy and paste the values. (I use this all the time when using the Unit Test code)

It should be possible to mark the grid as read-only. Maybe that blocks copy?


libxml2, python-lxml : upgrade to fix CVE-2022-2309

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

libxml2, python-lxml : upgrade to fix CVE-2022-2309
Change Log

Change
Change
Change

Does this affect the toolchain?

YES
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2022-2309

Test Methodology

Pipeline build id: xxxx

abandoned


Issue #39 - React 16 support

Hey, thanks for this library! I'm working on refactoring/updating another library (react-forms-builder) and want to use this in that library. One of the tasks I'm doing is upgrading the library to support React 16 (this issue: #39), so I figured I'd do the same here. I've also made a number of other tweaks to make development a bit easier, as well as cut down on some devDependencies. Happy to take any feedback/constructive criticism where it's available.

Bumped to version 1.3.1 to reflect React 16 dependency
Moved React/ReactDOM to peerDependencies
-- Example includes CDN-provided React/ReactDOM UMD modules
Updated dependencies (no outdated packages)
Remove Browserify dependency (was able to obviate the need for it with Webpack)
Moved src/StarRatingComponent.jsx to canonical index.jsx; updated "main" field for package to reflect this (this is an opinion, but to me it's more clear when importing the module; happy to revert this bit if it's confusing)
Added sourcemap configuration for Webpack
Added webpack-dev-server
Apply eslint to webpack.config.js

sorry, it was already added today :( you may check and update your PR

Hey thats cool @voronianski. Hopefully the other changes made were helpful also. I also started some basic unit test coverage (just added 2 so far).


CAMEL-10543: New Apache Camel Logo

Here is my proposal, heavily based on @lanwen work.

This is only an icon. Is the other PR from you the one we should include in the list of proposed logos?
In other words I think the other PR is better than this, but maybe you can tell more what this PR is intended to help with.

Hi @davsclaus this is a remix of the work @lanwen did in PR #1369 it changes the look of the background on the logo and the type on the -named SVG file. I cannot be objective but I think this is an improvement on the work @lanwen did. But as I stated it's heavily inspired by his work.
Having done this remix, I was inspired to do another thus the one in PR #1494. I would like this to be included in the list of proposed logos as well.

Hello All, would like to contribute with my two diferent logo options,
in your searching for a new one Camel.
Attached both:

Best regards
Alvaro

...Sorry. If this is not the right place to put them, please tell me where.
Thanks
Alvaro

@AlvaroBus thank you for your contribution, the best way to contribute the logo is to create a new pull request and to do that you need to fork this GIT repository, make your changes in the docs/img subdirectory of the repository, i.e. place your proposal images there. Commit, push to your forked repository and create a pull request, see GitHub docs on that:

Fork A Repo
Creating a pull request from a fork.

If it's easier for you, you can use a graphical GIT client such as GitHub Desktop or SourceTree.
See the details in the call for proposals on the Apache Camel website.

Thank you @zregvart , is open a new issue with the logo.
Best regards
Alvaro

Hi we are voting on the received proposals, the vote is open till 15.5. see all the proposals and cast the vote for the three designs you prefer the most.
Also read the news announcement on Camel blog.

Since the vote ended I'm closing the related PRs


Support @optional directive use on macro edges

Currently, the @optional directive may not be applied on macro edges. This is because definitions of macro edges are allowed to rely on the composition of multiple edges, and there is no general way to know which of these multiple edges the @optional semantics should apply to.
For macro edges whose definitions only rely on a single edge, this is simple -- when the macro edge is marked @optional, the only edge becomes @optional.
When the definition contains multiple edges, we need a way to allow the creator of the definition to specify which edges become @optional and which remain required when the macro edge is marked @optional. This is perhaps best accomplished by adding another directive that can only be used at macro definition time, and relying on it to mark edges in the definition that should become optional.

Problem: The result of an optional traversal is the same as the union of the result with mandatory traversal, and the result with no traversal. If you mark some edges inside the macro as mandatory-under-optional, this property no longer holds.
Proposal: I think defining the semantics such that all edges in the macro are optional by default will buy us a couple of years until someone requests mandatory-under-optional edges. It's also easy to implement and avoids the problem above. The only downside I see is that nested optionals are bad practice in OrientDB. But I think it's time to move on from prioritizing OrientDB.


Code Block, SQL: SQL commands aren't recognized

Code Block, SQL: SQL commands aren't recognized by CodeMirror (javascript library) and hence, they aren't highlighted correctly.
Example mark up:
FROM fe_users u
Comparisons:
1) CodeMirror: SQL demonstration
https://codemirror.net/1/contrib/sql/
key word: "join"
join
2) CodeMirror: SQL Mode for CodeMirror
https://codemirror.net/mode/sql/
key word: FROM
FROM

Thanks for reporting @steffenmaechtel.
Just need to tweak the mode used. Fix will be included in the next release.


Adding format to resourcemanager property ResourcesMoveContent.TargetResourceGroup

Fixes https://github.com/Azure/azure-sdk-for-net/issues/35550
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager


command mute

this needs a way to persistently store mutes and a timestamp to unmute automatically.

for storage of persistent data see https://github.com/zeroBzeroT/0DB/issues/2


i18n

Some of investigating besides the default options in Blazor: https://docs.microsoft.com/en-us/aspnet/core/blazor/globalization-localization?view=aspnetcore-6.0

Sample for WASM only: https://github.com/pranavkm/LocSample
https://github.com/jsakamoto/Toolbelt.Blazor.I18nText
JSON based: https://github.com/xaviersolau/MediumBlazorArticleSeries/tree/with_json/JsonLocalizer, or JSON based with cached: https://github.com/hishamco/My.Extensions.Localization.Json/tree/dev/src/My.Extensions.Localization.Json, and this from another community BootstrapBlazor - Localization-Json
But we think we need the option on the holistic view on both server and client sides so that we can localize model validation, error message...

Db localization: https://github.com/valdisiljuconoks/localization-provider-core


How to import .ckpt file ?

How do I import .ckpt file ? I understand how to import a .pb file but what about a .ckpt file ?

For example inception model v4 from https://github.com/tensorflow/models/blob/master/slim/README.md#pre-trained-models is a .ckpt file

http://www.mattrajca.com/2016/11/25/getting-started-with-deep-mnist-and-tensorflow-on-ios.html
It seems we need to package .pb, and .ckpt into one .pb file to import and run it.

how do I do that in tensorflowsharp ?

1. Make a Frozen Model File(.pb) which both contains graph defs & weights.
First, Save original pb file & checkpoint file
Python Code
# Save checkpoint file
tSaver = tf.train.Saver();
tSaver.save( tSess, "Profile.ckpt"); # It will generate 4 files: checkpoint, Profile.ckpt.index, Profile.ckpt.meta, Profile.ckpt.data-00000-of-00001

# Save pb file
tf.train.write_graph( Session.graph_def, "LogFolder", 'Profile.pb', as_text=True )
Command

Then, Use freeze_graph.py to merge them into a frozen model .pb file
(some argument need to adjust when using)
python "Python\Lib\site-packages\tensorflow\python\tools\freeze_graph.py" --input_graph=.\Profile.pb --input_checkpoint=.\Profile.ckpt --output_node_names=Output/Predictions,Output/Loss --output_graph=frozen.pb
pause

2. Import this pb file in code:
C# Code
using (var graph = new TFGraph())
{

	graph.Import(File.ReadAllBytes("frozen.pb"),"");
	var session = new TFSession(graph);
	var runner = session.GetRunner();
	runner.AddInput(graph["Input/IsTraining"][0], false);

	runner.AddInput(graph["Input/Values"][0], new TensorFlow.TFTensor(TFDataType.Float, new long[]{1,44,60},1*44*60*4));
	runner.Fetch(graph["Output/Predictions"][0]);

	var output = runner.Run();

	// Fetch the results from output:
	TFTensor result = output[0];
}


End to end workflows on GPU

It would be very useful for the GPU data science and research community if Scanpy were able to perform end to end workflows on the GPU, using either Cupy, CuDF or both.
An initial iteration of this feature could include simply swapping out the numpy imports for cupy.

[X] Additional function parameters / changed functionality / changed defaults?
[ ] New analysis tool: A simple analysis tool you have been using and are missing in sc.tools?
[ ] New plotting function: A kind of plot you would like to seein sc.pl?
[ ] External tools: Do you know an existing package that should go into sc.external.*?
[ ] Other?

...

I recall looking at some ScanPy workflows and @davidsebfischer pointed out that many of them rely on Sparse data types.  Is that still the case ?

I recall looking at some ScanPy workflows and @davidsebfischer pointed out that many of them rely on Sparse data types. Is that still the case ?

yes, this still is the case

@quasiben @davidsebfischer. I've been working on using RAPIDS/CuPy to implement a Seurat / Scanpy single-cell RNA workflow. Specifically, I've been finding it quite challenging do w/ CuPy sparse arrays because of the following two issues:
https://github.com/cupy/cupy/issues/2360
https://github.com/cupy/cupy/issues/3178
Currently, I'm having to convert to scipy.sparse to implement filtering.

Do you know how hard it would be to add cuSparse to CuPy for more sparse support ?

@quasiben As far as I know Cusparse is being used under Cupy currently for a lot of the operations.
I’m not quite sure why those slicing strategies aren’t supported yet. I just figured maybe they were less than trivial than the others and weren’t immediately needed so they were pushed off to future feature requests.
The issue #2360 I can’t imagine is too hard- I imagine the output array the size of the selection list could be allocated and a Cuda kernel scheduled to write the selected entries in parallel.
I’m not as sure about the other issue, but what Dask is trying to do seems more like an API compatibility issue than one of performance/compute.

What features in cuSPARSE would be useful for slicing?

@jakirkham, I’m not sure about slicing. @quasiben ’s question seems to imply the use of cusparse in cupy for general sparse operations. Please correct me if I misunderstood.
I believe once the two issues above are resolved, much of the scipy.sparse functionality for the preprocessing in Scanpy should be able to be swapped with cupy.sparse.
The ML stuff is a little but different, and I’ve created a separate issue to track that discussion.

I think our initially identified bottleneck with using sparse arrays was this here https://github.com/cupy/cupy/issues/2359.
The analysis workflows usually have very clear computational bottlenecks, so the translation to GPU should take this into consideration: Is it feasible in terms of available code to keep the array on GPU and actually perform all operations there or will this stay a CPU centric library that deploys particular steps to GPU. Inbatchglm / diffxpy we took the first approach, we build ontop of (a CPU centric scanpy and) deployed GLM fitting to GPU via tensorflow2, we also use estimation code in dask in the same package that we could in principle use with cupy, right now this just sits ontop of numpy.
Happy to be involved with this stuff, I spent some time thinking about this with @quasiben already. I think it is really crucial to figure out where it makes sense to invest time to build pipelines that can be end-to-end be executed on GPU: because of the large number of tools this will not be the entire scanpy tool environment for a long time, so mixed workflows will be necessary.

I would for example restrict all efforts to the submodule sc.tl for now because this contains most potential bottlenecks I think that are frequently used. "end-to-end" doesnt need to go all the way up to analysis graph leaves, such as plotting, in my opinion, as their is little performance gain there.
Nice to have for non-core functionalities would then be some examples of how GPU-based arrays can be used within anndata so that 3rd parties can modify their tools to directly operate on the GPU array rather then starting to copy arrays. I think this is not really clear for most people right now (I have never done that either) and documenting this properly / improving this would help a lot.


🌱 Upgrade to controller-runtime v0.11.2

To keep up with https://github.com/kubernetes-sigs/cluster-api/pull/6360

/test-v1b1-ubuntu-integration


PhpUnit @expectedExceptionMessage may be on multiple lines

@keradus PhpUnitNoExpectationAnnotationFixer source seems not so trivial, if you could take care of the bugfix I'd appreciate it. Btw the priority to me is very low, near zero, just found the case in a project and wanted to report the bug.

Never mind, just found the time to fix it 😛


[PDI-8560] Table Input does not work with lazy conversion when querying Hive: Method not supported

It must be merge with https://github.com/pentaho/big-data-plugin/pull/408

@brosander @mattyb149  please review.

closing since this has merge conflicts and issue has been deprioritized. Reissue when applicable


Fix UI scaling issues on Windows

Description
UI scaling does not look correct if the screen scaling factor is above 100% on Windows.
This patch modifies the Glfw window backend to try and resolve the issue.
Fixes #93
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build and run on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

@aurolou - Could you test out this change and see if it resolves your older issue with GUI scaling? :)


Bug: Deleting partial selection with link turns the following text into link

Lexical version: 0.3.5
Steps To Reproduce

Open https://playground.lexical.dev/
Enter: An extensible italic text editor framework
Select "extensible italic text" -> create a link
Select "italic" -> turn into italic
Select "ible italic text editor" and hit backspace or delete

Link to code example:

The current behavior
The link wraps "extens framework"
The expected behavior
The link wraps "extens"
https://www.youtube.com/watch?v=HRRvrgheBM8

Can't reproduce


improvement: decrease dagstore startup time

Current dagstore time is unsustainable due to a sync startup and slow, unindexed storage backends. The solution space is split up into two kinds of solutions:

Keep a sync startup, but make loading fast enough that loading hundreds of millions of blocks will take under 2 minutes
Develop async startup, allows us to keep the same storage backend

The big question here is how much of a speedup #1 can provide us. This is currently being benchmarked and I will update this issue soon with the results.
If we cannot speed it up enough even with a different backend, we should go for an async startup. This is a bit more complicated and there are many ways to do this, which I will elaborate on once I finish collecting the benchmarks for solution 1. It will be pretty challenging to implement, and add a lot of unwanted complexity, but it may be necessary long term

Updating this issue:

I don't think this will be a relevant issue for long, so no need for complex solutions. Once pruning is implemented, this is a non-issue.
A lot of the startup time came from debug logs (3-4x!), which are now deactivated
Multiple backends have been tested, LevelDB for example cuts the time in half but currently has issues on macos
We should definitely keep the sync startup. If we didn't have pruning, this would be problematic

According to napkin math, we have about 1.5 years of blocks before nodes will start needing 2 minutes to start up. So, we should implement pruning by then. If it isn't implemented, we should increase the default timeout again.

closing for pruning as solution


Export underlying connection

Hi,
It would be good to export the underlying proxyproto.Conn.conn connection.
Since proxyproto.Conn is not an tcp.Conn, and I need tcp.WriterCloser

Where can we find tcp.WriterCloser?

I fixed my comment. We cannot cast a Conn from this package to net.TCPConn that's why I need the underlying net.conn where the cast is possible (if it's a TCP conn)

https://github.com/pires/go-proxyproto/releases/tag/v0.3.1


Html autocomplete not working in php file.

For what more you need more info. Just not working + CSS.
I don't know what you need. Say.
https://github.com/Microsoft/vscode/issues/69737

Hey @octref I can use emmet in PHP with div but not <div, which used to work, can you take a look?

HTML only has <div completion too. Emmet only expands div not <div.
You might want to use https://marketplace.visualstudio.com/items?itemName=bmewburn.vscode-intelephense-client which has good support for HTML in PHP.

Still not working.

@AndRoMarian It's working for me though. Can you make sure Intelephense is the only extension you install, and disable all other PHP extensions?


scdf server should refer to versioned image

Checking github.com/spring-cloud/spring-cloud-dataflow-server-kubernetes/blob/v1.2.0.RELEASE/src/etc/kubernetes/scdf-controller.yml as it uses
image: springcloud/spring-cloud-dataflow-server-kubernetes

Not sure what it resolves as there is no version defined in that image setting but I'd assume that 1.2.0.RELEASE should always fetch same version.

We will be addressing the version visibility via spring-cloud/spring-cloud-dataflow-server-kubernetes#209.


paket.template conformance

Currently, paket.template is not conforming with NuGet changes. Here's a checklist to get into conformance:
License stuff:

[ ] Add support for license (#3475)
[ ] Add support for licenseFile (probably a path to the license file?)
[ ]  Deprecate licenseUrl (deprecated in NuGet)

Icon stuff:

[ ] Add support for icon (probably a path to the icon file?)
[ ] Deprecate iconUrl (deprecated in NuGet)

Pitching in what I've noticed: paket.template also doesn't appear to follow the same behavior when generating nuspecs, as newest nuget always adds exclude="Build,Analyzers".


project checkResult  may NPE

What is the purpose of the pull request
1 word error
2 modify check project perm may NPE
3  modify checkResult  may NPE
4 update project add desc check

Codecov Report

Merging #1606 into dev will decrease coverage by <.01%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             dev   #1606      +/-   ##
========================================
- Coverage   9.07%   9.06%   -0.01%
========================================
  Files        283     283
  Lines      13846   13850       +4
  Branches    2205    2207       +2
========================================
  Hits        1256    1256
- Misses     12490   12494       +4
  Partials     100     100

Impacted Files
Coverage Δ

...e/dolphinscheduler/api/service/ProjectService.java
0% <0%> (ø)
:arrow_up:

...lphinscheduler/api/service/AccessTokenService.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


aixPB: bypass the need for yum.sh to download and install AIXToolbox yum package

Checklist

[x] commit message has one of the standard prefixes
[ ] FAQ.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

@aixtools - YAMLLint (2 v 4 spaces)

From ojdk06, on AIX 7.2 TL2 SP5 - the download, while not what I would consider fast (232.3Kbytes) it seems to be accurate: AND inaccurate was the bigger issue.
ftp> mget rpm.rte
227 Entering Passive Mode (170,225,15,26,126,241)
150 Opening BINARY mode data connection for rpm.rte (40987648 bytes).
226 Transfer complete.
40987648 bytes received in 172.3 seconds (232.3 Kbytes/s)
local: rpm.rte remote: rpm.rte
ftp> dir rpm.rte
227 Entering Passive Mode (170,225,15,26,129,203)
150 Opening ASCII mode data connection for /bin/ls.
-rw-rw-r--    1 35092<PHONE_NUMBER>8 Feb 15 01:03 rpm.rte
226 Transfer complete.
ftp> !ls -l rpm.rte
-rw-r--r--    1 root     system     40987648 Mar 23 08:45 rpm.rte

From my Power6 (in the Netherlands versus Oregon) - also connecting to service.boulder.ibm.com: much faster (factor 10x)
ftp> mget rpm.rte
227 Entering Passive Mode (170,225,15,26,235,161)
150 Opening BINARY mode data connection for rpm.rte (40987648 bytes).
226 Transfer complete.
40987648 bytes received in 14.1 seconds (2838 Kbytes/s)
local: rpm.rte remote: rpm.rte

Further study of my NL partitions:

While getting, comsistently, 2800+ Kbytes from one partition I was getting, consistently, 450+ Kbytes from other partitions. Looking for differences I saw that I had increased the default tcp_buffers (ISNO settings) for the ethernet connection. When I did the same on the slow partitions - speed increased.

Testing on ojdk06 - even with the ISNO change - speed from IBM ftp site remains slow - but DL speed from my NL server increased to an amount that makes me believe my server was the bottleneck - not ojdk06.

Summary: from my NL server ojdk06 has input between 1800 and 2800 - consistently; from the IBM site - for the file rpm.rte it continues to be 150k (and less) - sometimes with error.

Getting back to a question from @sxa - what is the problem. IMHO - not rpm.rte, but yum.sh. And the key issue is that yum.sh is not able to adequately respond to things gone wrong.

My confidence is higher in ansible using get_url to get the two files yum.sh picks up, and then using command: statements to install the downloads.


Make hubot-tell work with Hipchat

In Hipchat, the msg.message.user.room value referenced in robot.enter is different to the msg.message.user.room value referenced in robot.respond.
robot.respond receives a packet like:
{ reply_to<EMAIL_ADDRESS>room: 'stuff' }

whereas robot.enter receives just the<EMAIL_ADDRESS>style room name.
This PR makes the script store against the<EMAIL_ADDRESS>style room name.

Ugh, thanks. Not quite sure but shouldn't the condition read  if msg.message.user.reply_to? then (with a question mark?). @xhochy, my coffeescript is getting rusty, can you check this please? Or maybe something like msg.message.user.reply_to || msg.message.user.room or whatever that would look like in CoffeeScript...

Possibly. My coffeescript isn't great, so I was just doing a ternary.

Yep || works here as we access entries of an object.

This seems to break hubot-slack. The reply_to is like C02EKKBLG instead of devops so looking up the room fails.

Yikes. Seems like either HipChat or Slack is ignoring the hubot spec somewhere (or it's rubbery enough to allow broken behaviour like this...) - but why on earth would reply_to have a value to which you can't reply? That seems plain broken to me.

Yikes. Seems like either HipChat or Slack is ignoring the hubot spec somewhere (or it's rubbery enough to allow broken behaviour like this...) - but why on earth would reply_to have a value to which you can't reply? That seems plain broken to me.

The reply_to value overrides the room name. The reply_to seems to encode both the room and the user sending the message so it changes per user. When the enter hook is triggered, it looks up the room name with no reply_to, which would, of course look different. So the flow is:

Receive a message in #devops that matches regexp for telling.
Extract reply_to (C02EKKBLG) and username (parkr)
Save message to localstorage["C02EKKBLG"]["parkr"].
Later, parkr enters #devops. Receive EnterMessage with room devops and username parkr.
Look up localstorate["devops"]["parkr"], but nothing is there.

Yeah I see what's happening in the case of Slack, I'm just wondering whether it's the hubot plugin for Slack or for HipChat that's misbehaving here, because apparently in HipChat it's exactly the other way around.


Need to add license statement header to code files

Need to add WINNF boilerplate license agreement statement to .py code files

Should add following license (as per CONTRIBUTING file in repository)
=============
Copyright 2017 CBSD Project Authors. All Rights Reserved.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
=============


Add spec file for Fedora distros

Wow!, thanks a lot!
Even documents!!!!
it looks good, can you fix the nit thing I found?

Thanks a lot!!

Thanks to you :)
On Fri, Jul 22, 2016 at 12:54 PM, Miguel Ángel Ajo<EMAIL_ADDRESS>
wrote:

Thanks a lot!!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mangelajo/os-log-merger/pull/28#issuecomment-234515114,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AToCs9UDb7bMgdQYULQ_oyGNzn9TPyr5ks5qYKFvgaJpZM4JR0Pw
.


V1 0 0

To be merged, and properly released, after approval from the technical committee and the board of HL7 Finland ry.

@mikajylha OK for me to review this already, so merging is simple when the time comes.


What is the ValueEstimator of NSM?

I read that REINFORCE algorithm have a ValueApproximator and a PolicyEstimator.
Thank you!

Actor-Critic has value estimator, REINFORCE doesn't.

Thank you, very very much.

Thank you, I read from here:
https://github.com/dennybritz/reinforcement-learning/blob/master/PolicyGradient/CliffWalk REINFORCE with Baseline Solution.ipynb


Remove version: kkclue.GarfieldPlus version 1.01

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: - https://pixeldrain.com/api/filesystem/UgVhcrVc/GarfieldPlus-v1.01.zip?attach
Invalid URI: The URI scheme is not valid.
(Automated message - build 911)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 886.)

For moderators or/and anyone else

[!Caution]
As a result of several personal problems I've had over the last few months + my final exams coming up very soon, I'm currently unable to look at my GitHub notifications every day and maintain my executions correctly.
If you detect a certain problem on a certain pul request:

Either you can close the pull request directly without mentioning me.
Or you can send me a message on X (Twitter): @Exorcism0666

Thank you for your understanding, and I'm sorry if some pull requests can't be approved due to the situation I'm in.

This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！