[CLOSED] Segmentation fault after 5 requests when creating a subshell

 Issue by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT
Originally opened as https://github.com/glenna-m/rdf-pipeline/issues/9

After several requests (even just to static node http://localhost/max ), the
Apache2 error log shows: "[Sun Jan 01 16:38:34 2012] [notice] child pid 18330
exit signal Segmentation fault (11)" and subsequent requests may fail.  It does
not always happen after the same number of requests, but with a script I am
able to reproduce it predictably.  I tried changing backticks `date` to using
system("date > /tmp/date") but the problem persists.  We could do without that
particular use of a subshell (just to get the current date), but in general it
will be needed in order to run a FileNode updater.  I'm guessing the problem
has something to do with Apache2 threads and subshells, but I have not
confirmed this.  No known workaround yet.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 1 Jan 2012 at 10:01

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

This is reproducible by:

1. Under root, truncate the Apache2 error log and restart Apache:
root@dbooth-laptop:~# apache2ctl stop ; sleep 5 ; truncate -s 0
/var/log/apache2/error.log ; apache2ctl start

2. Run:
./crash.perl http://localhost/max

That should cause the seg fault after 5 requests.

Original comment by<EMAIL_ADDRESS>on 1 Jan 2012 at 10:05

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

I tried using system("date > /tmp/date") instead of `date` but this did not
help.

I might try using Apache2::SubProcess
http://perl.apache.org/docs/2.0/api/Apache2/SubProcess.html

Original comment by<EMAIL_ADDRESS>on 1 Jan 2012 at 10:58

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

Or maybe try this technique:
http://stackoverflow.com/questions/471681/how-do-i-fork-properly-with-mod-perl2

Original comment by<EMAIL_ADDRESS>on 2 Jan 2012 at 4:12

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

Or maybe try using this:
http://search.cpan.org/~rybskej/Apache-forks-0.03/lib/Apache/forks.pm

Original comment by<EMAIL_ADDRESS>on 2 Jan 2012 at 5:06

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

Or perhaps this technique (though this was written for mod_perl version 1):
http://modperlbook.org/html/10-2-5-A-Complete-Fork-Example.html

Original comment by<EMAIL_ADDRESS>on 2 Jan 2012 at 4:31

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

I have isolated this bug.  It occurs when Test::MockObject is use.  I saved a
snapshot of minimal code example in svn rev 25 and reported the bug at
https://rt.cpan.org/Public/Bug/Display.html?id=73723

I still need to change my code now, to avoid using Test::MockObject.

Original comment by<EMAIL_ADDRESS>on 4 Jan 2012 at 3:20

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

See my plea for help from perlmonks at
http://www.perlmonks.org/?node_id=945947

Original comment by<EMAIL_ADDRESS>on 3 Jan 2012 at 3:12

 Comment by GoogleCodeExporter
Tuesday Oct 06, 2015 at 00:42 GMT

Fixed.  The code no longer uses Test::MockObject.

Original comment by<EMAIL_ADDRESS>on 20 Jan 2012 at 10:51

Changed state: Fixed


How-to doc for instrumenting existing systems

In conversations with OT users, various people have said that the function-level documentation and semantics are clear, but the big picture of how to approach instrumentation for large codebases is less so.
The goal of this document is to help individuals and teams that are looking to instrument existing environments with OpenTracing. It is written as a 'how-to' format with a simple example in an effort to be more approachable to developers lacking prior experience in distributed tracing.
Feedback is welcome.

Just wanted to gently bump this thread to see if we could merge these docs. I wanted to make sure these were done before submitting another PR for additional updates.

I basically think this is an improvement and would like to get it in... I may make a few nitpicks, but if there are no great objections I'd propose we merge this in 36h or so.

Please note that without a style change adding more items to the menu will make some menu item unreachable. Right now, on my screen at least, the menu takes all of the vertical space, and it's not scrollable with the rest of the page

Hi Yuri - I am working with a designer to redo our website (as I shared in
the OT Gitter) so that should take care of it :-)
Priyanka Sharma
Co-founder, WakaTime http://www.wakatime.com/ - Quantify your coding
http://www.twitter.com/pritianka
www.twitter.com/pritianka
650-796-7125
On Thu, Sep 22, 2016 at 12:17 PM, Yuri Shkuro<EMAIL_ADDRESS>wrote:

Please note that without a style change adding more items to the menu will
make some menu item unreachable. Right now, on my screen at least, the menu
takes all of the vertical space, and it's not scrollable with the rest of
the page
[image: image]
https://cloud.githubusercontent.com/assets/3523016/18762629/92e507b8-80d7-11e6-8fb6-70d98ecdf881.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opentracing/opentracing.github.io/pull/118#issuecomment-249001290,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACTTmI8_KkummZ9at7zw6954F8UIBpUMks5qstQ7gaJpZM4J5jEb
.

@yurishkuro what browser are you using?

RFOMM (Renders fine on my machine... 😕 )

@azimman it's not a browser issue, it's a window-height issue... I think @yurishkuro's point is that the leftnav doesn't scroll. @pritianka's redesign comment is certainly relevant, but we don't want the leftnav to be busted in the interim. I can try to fix this with css or something.

@bhs got it. I'll take a look tomorrow at the current template and see if there is an easy fix.

@yurishkuro can you take a look at the latest commit and see if it works for your default window size. This is how is renders for a 1280x800 resolution:

Easy to reduce it further if desired... but will obviously start to look small at higher resolutions.
@pritianka It would also be good to provide the designers with minimum resolution and browser requirements as part of the redesign (happy to help you with this).

@azimman based on your screenshot looks like it will be ok.

Any other updates needed before merging this?

added new docs to site redesign.


Is supported window auto size?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

please elaborate, I do not understand what you are asking for.

When I narrow or enlarge the window, the resolution can be adaptive

When I narrow or enlarge the window, the resolution can be adaptive

Does it support this function?

Sorry, your question is still too vague.
Here a few hints that might help:

Which client? We have a couple of clients for different systems (windows/X11/mac/android/...)
Did you check the help output of your client (just run without arguments)?


uid not found

Some block could not be found
Could you please  add the “If  the UID block not found, skip it” function?
Some blocks that can’t be converted are accepted.
Thank you very much!

Can you clarify what exactly you want it to do? Does it break upon encountering missing UIDs or just print a lot?

I have a same problem.

Thank you!

I'm having the same issue here...

this isn't a fix to the actual python script but I'm using the UID that shows up in the terminal within Roam search to delete all empty block references
a bit brute force but it'll fix the issue; hopefully you don't have too many empty block references (I have a handful I'm manually deleting)

Ah I also added a break line after the println so it stops spamming and you get a neat list of what empty references you have to delete


some computer doesn't randering text in firefox

https://drive.google.com/file/d/0B3KHRk10beerOElhbDQzNXIzQ1k/view?usp=sharing
here is screenshot
https://drive.google.com/file/d/0B3KHRk10beerZkdEZ29wNzBTZDQ/view?usp=sharing
left is firefox and adobe

Unable to reproduce on Firefox for Mac OSX:

Could you provide more details Firefox and PDF.js version (the latter can be found in the javascript console)? Also can you paste Graphics section from the about:support?

Firefox is the latest version that i can search in google.
and...
the problem is that only a "few" of the computers.
there is a total of 10 computers in my office, but However, the computer that happens is just 2.

Closing as incomplete. More information needed to re-open the issue, e.g. link to the PDFs and OS version, and Graphics section from the about:support. See also https://github.com/mozilla/pdf.js/blob/master/CONTRIBUTING.md


Update UG for order system

Updated user guide for order system.
Currently using AB3 style prefixes.

Codecov Report

Merging #50 (f892d66) into master (3402fed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #50   +/-   ##
=========================================
  Coverage     69.01%   69.01%
  Complexity      477      477
=========================================
  Files            81       81
  Lines          1517     1517
  Branches        165      165
=========================================
  Hits           1047     1047
  Misses          401      401
  Partials         69       69

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 3402fed...f892d66. Read the comment docs.


PDF ReArrange Script.

Description
The script to re arrange out of order pages in  a PDF file.
Created script,Readme file and added a Test PDF file along with output images.
Fixes #231
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

This is how it looks to me.

I have added relative path for the images.

This is how it looks to me.

I have added relative path for the images.


(6.0.0-beta8) ArgumentNullException caused by model validation

My view model that gets POSTed to the controller is raising an ArgumentNullException in beta8 (was not happening in beta7.)
The source of the exception is a read-only property on the view model that calls Where on another property of the view model which has not yet been set (it gets set in the controller.)
System.ArgumentNullException was unhandled by user code
  HResult=-2147467261
  Message=Value cannot be null.
Parameter name: source
  ParamName=source
  Source=System.Core
  StackTrace:
       at System.Linq.Enumerable.Where[TSource](IEnumerable`1 source, Func`2 predicate)
       at MyProject.Web.ViewModels.MyViewModel.get_MyProperty() in C:\path\to\source.cs:line 31
       at Microsoft.Framework.Internal.PropertyHelper.CallNullSafePropertyGetter[TDeclaringType,TValue](Func`2 getter, Object target)
       at Microsoft.AspNet.Mvc.ModelBinding.Validation.DefaultComplexObjectValidationStrategy.Enumerator.MoveNext()
       at Microsoft.AspNet.Mvc.ModelBinding.Validation.ValidationVisitor.VisitChildren(IValidationStrategy strategy)
       at Microsoft.AspNet.Mvc.ModelBinding.Validation.ValidationVisitor.VisitComplexType()
       at Microsoft.AspNet.Mvc.ModelBinding.Validation.ValidationVisitor.Visit(ModelMetadata metadata, String key, Object model)
       at Microsoft.AspNet.Mvc.ModelBinding.Validation.ValidationVisitor.Validate(ModelMetadata metadata, String key, Object model)
       at Microsoft.AspNet.Mvc.ModelBinding.Validation.DefaultObjectValidator.Validate(IModelValidatorProvider validatorProvider, ModelStateDictionary modelState, ValidationStateDictionary validationState, String prefix, Object model)
       at Microsoft.AspNet.Mvc.Controllers.DefaultControllerActionArgumentBinder.<BindModelAsync>d__5.MoveNext()
  InnerException:

Can you post your code around source.cs line 31

Here's the gist of it:
public class MyViewModel
{
    [FromForm]
    public MyOtherViewModel MyPropertyThatGetsBoundFromFormAndValidated { get; set; }

    public IEnumerable<SomeClass> MyPropertyThatGetsSetInTheController { get; set; }

    public IEnumerable<SomeClass> MyComputedProperty
        => MyPropertyThatGetsSetInTheController.Where(x => x.SomeCondition).OrderBy(x => x.SomeOrder);
}

@tuespetre the stack trace refers to get_MyProperty() and your gist doesn't include a property named MyProperty. What were the original names?

I think [FromForm] cannot be used here. Does it can be used only in Controller?

I think [FromForm] cannot be used here.

[FromForm] is fine on any model-bound property -- including those in your controller. Might be confusing in a few cases but not disallowed.

Can I use [FromServices] in anywhere?

@Kagamine yes, you can use [FromServices] on any model or controller property or action parameter.

@dougbu sorry for the inconsistency. I have updated the example in the comment above.

@rynowak looks like DefaultComplexObjectValidationStrategy is attempting to validate get-only properties. If that makes sense, should PropertyHelper.CallNullSafePropertyGetter() be "safe" w.r.t. additional Exceptions?
@tuespetre I suggest changing the MyProperty to detect MyPropertyThatGetsSetInTheController == null and return null in that case. Probably a good idea in general to set up your model types so they're complete after model binding.

I think you could put [BindNever] on the two properties that are really not intended to be bound using model binding. Or maybe move those properties to a separate class so that you have separation between you binding model and your other logic.

FWIW, just tried [BindNever] and it did not help.


Change time in painting when space width is determined.

Some fonts cannot be correctly drawn or measured in SWT when advanced graphics mode is set on. Example of such font is https://github.com/belluzj/fantasque-sans. So this patch measures space width before advanced mode is turned on when alpha is set.
Before:

After:

Thanks!


CRASH

https://pastebin.com/R3TkvdJa
AND new version ScoreBoardStats doesnt work (compatibilityMode)

Outdated

new verison doesnt work

when compatibilityMode (true)
plugin dont work


Remove unused code

Motivation:
Cleanup PlatformDependent* and remove unused code.
Modifications:
Code cleanup
Result:
Removed unused code

Cherry-picked into 4.1 (1e5d33f) and 4.0 (1342300)


[DCOS-44086] Bump snapshot version for 0.42.5 release

Bump the version in build.gradle for the SDK release.

not related to this, but, do we make sure that fixes that were merged to 0.40.x directly (if at all there are any) are also present in master?

not related to this, but, do we make sure that fixes that were merged to 0.40.x directly (if at all there are any) are also present in master?

We generally backport any fixes to sdk-0.40 after they have landed in master (if applicable).


update etag and kind type

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[x] SDK of Go
[x] PowerShell
[x] CLI
[x] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

how can there be a LintDiff error on changing the description on a new api-version? does this means that the description can never be changed?

how can there be a LintDiff error on changing the description on a new api-version? does this means that the description can never be changed?

It may caused by different model definition which has same name.  @jianyexi  to confirm

@akning-ms and @jianyexi, anything we can do to push this along?


Implement when conditional.

from stories import I, Story, when

class Action(Story):
    I.one
    when("foo") > I.two
    I.three

[ ] Should be supported in story and context representations.

[ ] Update: We will support only one context variable boolean or string. Everything without predicate functions. Otherwise, this will lead to complex expressions in story definition.

If the user wants complex comparisons, he should write an extra step before comparison and prepare this variable there.

[ ] Should be compatible with https://github.com/dry-python/stories/issues/49

[ ] Different branches of the Compare statement should be allowed to assign the same variable in the context. Context contract should take this into consideration.

[ ] If the context contract set the comparable variable to the Enum, we should check chat when blocks present for all Enum members. Or when.any expression was provided.

from stories import story, switch, case

class Action:
    @story
    def do(I):
        I.one
        with switch('foo'):
            case(X.member1) > I.two
            case(X.member2) > I.three
        I.four

X is an enum. If not every member is covered the error will be raised.

stories should be linear. We deny writing the case statement pointing to the step. A substory only could be used inside the switch. Otherwise, logic supposed to be encapsulated inside steps will polute story definition.

from stories import story, switch, case

class Action:
    @story
    def do(I):
        I.one
        with switch('foo'):
            with case(X.member1):
                I.two
                I.three
            case(X.member2) > I.four
        I.five

from stories import story, when

class Action:
    @story
    def do(I):
        I.one
        when('foo') > I.two
        I.three

from stories import story, when

class Action:
    @story
    def do(I):
        I.one
        with when('foo'):
            I.two
            I.three
        I.four

from stories import story, unless

class Action:
    @story
    def do(I):
        I.one
        unless('foo') > I.two
        I.three

from stories import story, unless

class Action:
    @story
    def do(I):
        I.one
        with unless('foo'):
            I.two
            I.three
        I.four

branch

Superseded by #723


Better fix for Issue #1 support for Swift compiler < 5.5

More complete fix for Issue #1.  Note commit comment about not including #if swift(<5.5) as a personal preference that you’re free to add if you prefer it be there.

Ah darn it. Client required tab setting for Xcode foiled me again.  Let me fix those tabs -> spaces.

Note that the changes to Test+Float.swift are to handle the floating point precision comparison issue (0.000000 != 5e-324 that CGFloat = 0 produces ( XCTAssertEqual failed: ("0.000000") is not equal to ("5e-324")).
Since the code includes a default of 6 for precision it seemed appropriate to test against that same level of precision.

FYI, I added some more tests and had to fix a compile issue (you missed the CoreGraphics import).
This is now available in v1.0.1 – thanks again 👍

Welcome.  Yay for more tests!  I saw that more would be good but didn't want to expand the PR outside the focused task/goal (swift<5.5 support).
I'm not seeing an import of CoreGrpahics needed here under Xcode 13.0 or Xcode 12.5.1 (I verified the PR branch before I checked it in and I just took your main branch and changed import CoreGraphics back to import Foundation and it still builds and all tests still pass in both versions of Xcode listed above).
So I figured out why you're seeing this and I'm not:  You're building for iOS and I'm building with a destination of macOS and macOS does not need the CoreGraphics import 🤦♂.  Sorry about that.
This points out the need to get some CI hooked up for every project so CI can auto-test building for various sdks and with various swift versions on a PR when one is created.
Course for this project, we'd have to address issue #2 before that can happen…

(That makes me wonder if it all compiles for the other SDKs the Package.swift file claims it supports - tvOS, watchOS, applecarOS (😂)… )

It does but only because I manually tested those myself. Agreed on the CI front.


Test result: not detected

The pass generated using the qr code with negative covid test result shows:
Test result: not detected
instead of negative

Not detected is correct, CWA also shows "not detected".
It means that the virus wasn't detected. I know, kinda confusing 😅

We may change to "Positive" / "Negative" if that's more intuitive.

Same issue. Standard on test results is "negative"/"positive". "Not detected" seems like "inconclusive" which is an issue if we want to use this app.

Fixed with #141


Support for Source Maps?

Right now all the code we get through in sentry is minified, is there support for source maps at all?
I couldn't find anything about it in the documentation?

Hi @DiagoSmith, as part of the monorepo migration, this issue has been moved to new issue. Our engineers have been notified and will prioritize and work on it ASAP. Thank you!
For more information, see README.md.


Allow pane_grid to be empty

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
in pane_grid, there is this method:
/// Returns `true` if the amount of panes in the [`State`] is 0.
pub fn is_empty(&self) -> bool {
    self.len() == 0
}

but rn, it will never return true, because the close method doesn't remove the pane if there is only one.
What is the expected behavior?
close method should remove the pane, even if there is only one.
For that, there should have some more methods:

new_empty to allow the creation of an empty pane_grid
add_pane because split method will not work on an empty pane.

Version
crates.io release
Operating System
Windows
Do you have any log output?
No response

I agree, I just added this widget to my application. How my widget works is if my data struct is empty the widget doesn't show. This all works correctly expect pane grid shows a empty pane, and the only way to get around this is to just add a conditional in my view.


Create pt.json

Description
Create pt.json
Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Enhancement (changes that improvement of current feature or performance)
[ ] Refactoring (changes that neither fixes a bug nor adds a feature)
[ ] Code style optimization (changes that do not affect the meaning of the code)
[ ] Docs (changes that only update documentation)
[ ] Chore (changes that don't modify src or test files)

Self Check before Merge

[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.

Screenshots (if appropriate):

Shouldn't senha start with a capital? And like 34, before mijn miwa there are 2 spaces.
Thanks!

Ups :)
Fixed


MobileNetv3-large baseline accuracy

Hello Manuel! I have read your CVPR2020 paper, and your method is effective on ConvNets.
While following your work, I have problems to reproduce MobileNetV3-large cifar100 baseline, which has 75% accuracy. However, with the following setting,
epoch=200; SGD with momentum 0.9; weight decay of 10−4; lr=0.1 and decayed by a factor of 0.1 at epochs 100, 150, and 180.
I can only get accuracy around 70%.
I also change the first two stride=2 to stride=1 for MobileNetV3.
Can you share your parameter setting? Or is there anything wrong? Thanks for helping me.

Finally change weight decay to 5e-4 and get the same result.


The script counts the uploaded files so I can't upload another files !!!

Hello,
When using Show previous uploads feature and setting maxFileCount:3 for example and try uploading more files I got error message "... is not allowed. Maximum allowed files are:3" It seems the script counts the already uploaded files so I can't upload another files !!!

I have a similar problem. When I upload an invalid file the counter doesn't decrease.
My solution is that:
var uploadAttachment = $('#attachment2').uploadFile({
          onSuccess: function(files,data,xhr,pd)
            {
                // it was a failure
                if (!data.success) {
                    pd.statusbar.hide();
                    pd.statusbar.remove();
                    uploadAttachment.selectedFiles--;
                }
            },
});

You must reset the counter:
  var uploadAttachment = $('#attachment2').uploadFile();
  uploadAttachment.selectedFiles=-1;

don't set  maxFileCount. it will work


Install Fluxbox window manager in VNC images

This PR is related to #29. It basically installs Fluxbox window manager in the VNC images of Selenoid. The normal images (i.e. without Selenoid) are nor affected, and the windows manager is only available for the VNC flavor of Selenoid images.
I have compared the the resulting images to the original ones, and as you can see the change has a little effect. I checked it for latest version of Chrome, Firefox, and Opera. I built these images using the following commands:
./automate_chrome.sh 64.0.3282.186-1 2.35 64.0
./automate_firefox.sh 58.0.2+build1-0ubuntu<IP_ADDRESS> 1.5.1 58.0 0.19.1
./automate_opera.sh 51.0.2830.40 2.33 51.0

The resulting images are as follows:
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
selenoid/vnc        chrome_64.0         5b592a2c44f7        4 seconds ago       886MB
selenoid/vnc        firefox_58.0        01cfbc1552e3        9 seconds ago       915MB
selenoid/vnc        opera_51.0          6d155108ed9c        7 seconds ago       884MB

After installing Fluxbox in the images, they are as follows. Notice that the difference in terms of size is only 5 MB:
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
selenoid/vnc        chrome_64.0         9c60e605ffaf        25 seconds ago      891MB
selenoid/vnc        firefox_58.0        c0bee659410e        10 seconds ago      920MB
selenoid/vnc        opera_51.0          31c9e5d2730c        8 seconds ago       889MB

Of course the CPU/memory consumption is now a bit higher since the windows manager is running in the container, but it should not be big deal IMHO.
As main benefit of the change, now we now use the browsers exactly how they are out of Docker. This is quite useful in the case of using noVNC (which is my case), in which I interact with the browser using an HTML5 Canvas, and now I can maximize, open preferences, and so on (without a windows manager this was not possible). For example as follows:

Please consider to merge it to the branch master to be available in next VNC Selenoid images. Thanks in advance.

@bonigarcia ok, discussing this with guys.

@vania-pooh because you said in #29 that windows manager is not useful for automated browsers. To minimize the consumption, I just installed it in the VNC images, in which the windows manager can be useful. In any case, if you prefer I can change the PR installing Fluxbox in the base image.

Looks good to me.

Ok, Fluxbox is 10 megabytes unpacked.
root@9e53ede34327:/# apt-get install fluxbox
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  feh libcurl3 libfribidi0 libgif7 libid3tag0 libimlib2 libjpeg-progs libjpeg9 menu xfonts-terminus
Suggested packages:
  fbpager fbdesk fbautostart menu-l10n gksu | kde-runtime | ktsuss xfonts-terminus-oblique
The following NEW packages will be installed:
  feh fluxbox libcurl3 libfribidi0 libgif7 libid3tag0 libimlib2 libjpeg-progs libjpeg9 menu xfonts-terminus
0 upgraded, 11 newly installed, 0 to remove and 73 not upgraded.
Need to get 3217 kB of archives.
After this operation, 9353 kB of additional disk space will be used.
Do you want to continue? [Y/n]

@bonigarcia two points:

To have the same behavior on both images - please add installing Fluxbox to regular (non-VNC) images (it will then automatically be installed for VNC images). Docker build files are like this: https://github.com/aerokube/selenoid-images/blob/master/selenium/opera/blink/Dockerfile.driver.tmpl
As you can see in entrypoint.sh X server is being started the last. So to avoid an error like this...

# fluxbox -display :99
Error: Couldn't connect to XServer:99

... we have to somehow try to start Fluxbox until the display is open. For example for x11vnc we add -loop500 flag to loop until display is open. Not sure how to do this reliably for Fluxbox. But I'm afraid that without such retries depending on user's Docker speed Fluxbox will sometimes start and sometimes will not (just because X server is started after Fluxbox and this is a race condition).

@bonigarcia btw, guys in support chat proposed to add a feature to Selenoid that will convert /maximize call to setSize() call with current screen resolution. What do you think about it? Wouldn't it be more reliable?

@vania-pooh
Regarding 1 (install Fluxbox in regular images), I have just committed new patches, first reverting the original patch, and then installing Fluxbox in base_1 and base_2.
Regarding 2 (race condition etc), yes, you are right. In the tests I made my solution was working the 100%, but it is not the more accurate solution (start Fluxbox before the actual DISPLAY is started). In fact, I think there is an additional problem related to the VNC server. Let me explain.
In my project I am using the VNC Selenoid images. It happened to me several times that when I connect to the VNC sessions (which I always do at the beginning), sometimes the VNC session is not available. If I reload the VNC session, the second time is actually working. I have this pending for further investigation, but now I can see it is related to the wait with the loop500 flag, which seems to be not 100% reliable (at least for my use case in which I require the VNC session right away after the container is started).
All in all, I have looked for a solution for both issues. I found it in the docker container images created by the Selenium team. See for example this entrypoint. Their strategy is to launch first xvfb-run, then wait for it, and finally start fluxbox and finally x11vnc. I think that's the proper way, to avoid race conditions both in fluxbox but also in x11vnc (which I have experienced).
I have change all the entry points for the images implementing this strategy. I tested it and seems to work properly. Please take a look.
Regarding your last comment (the /maximize thing), this is not my use case. I use the VNC session to actually interact with the browsers (through noVNC), not only watching it. So I would need the browser in Docker to behave as a real browser, meaning that maximize, minimize, move the window, open preferences, etc.

@vania-pooh Unfortunately the /maximize, even much more simpler, it is not enough for me. Regarding the other requests, I have added some comments and committed a new patch.

@bonigarcia what else is covered by Fluxbox in addition to /maximize?

I already explained it. Without a windows manager you cannot control the control the browsers, meaning that you cannot maximize, minimize, move, or open preferences. I use noVNC to interact with the VNC browser session (it's a bidirectional communication based on WebSocket). It's not only watching the browser with a given size.

@bonigarcia sorry for the delay. I reviewed again in detail your PR. Everything seems to be ok. I would request one more change. Please move installing fluxbox and x11-utils from base image to the webdriver layer before actually copying the binary. For example insert something like this here:
RUN \
        apt-get update && \
        apt-get -y  install fluxbox x11-utils && \
        apt-get clean && \
        rm -Rf /tmp/* && rm -Rf /var/lib/apt/lists/*

I understand that this is about copy-pasting and I will move these packages to base_3.0 image when needed to create it. Currently this would be easier and faster to use layers with browsers and install additional 10 megabytes on top.

Thank you for contribution! Will rebuild images during next weeks.

@vania-pooh Thanks to you! Keep up the great work with Selenoid!

@bonigarcia just tried to create an image for Chrome 65 using new Dockerfiles. Window maximization does not seem to work (getting an exception when doing maximize call). As usually these images are available as: selenoid/chrome:65.0 and selenoid/vnc:chrome_65.0. Any ideas what's the reason?

@vania-pooh Yes, it seems that the Flux windows manager is missing in the new image for Chrome 65.
We can check it starting the Chrome VNC container as standalone and checking the traces:
$ docker run --rm --name chrome --cap-add=SYS_ADMIN -p 4444:4444 -p 5900:5900 selenoid/vnc:chrome_65.0

 --- x11vnc loop: 1 ---

Starting ChromeDriver 2.36.540471 (9c759b81a907e70363c6312294d30b6ccccc2752) on port 4444
All remote connections are allowed. Use a whitelist instead!
 --- x11vnc loop: waiting for: 34

PORT=5900

As we can see in the latest entrypoint.sh, the VNC Chrome container is supposed to wait for xvfb-run before starting fluxbox. Notice that the traces Waiting xvfb... and the traces for fluxbox are completely missing.
Maybe the Chrome 65 image has been generated without this PR?

Ok, let me investigate more.

@bonigarcia just with driver.manage().window().maximize(); (Java client).


Uncaught TypeError: Cannot read property 'style' of undefined

Installed the gem and added the require files to both application.js and css. I am getting several  Uncaught TypeError: Cannot read property 'style' of undefined on the js file eg (line 85, 	this.tooltip.addClass('top')[0].style.top = -this.tooltip.outerHeight() - 14 + 'px';) etc.
I am following example 1: My view just shows me a blank input due to the error.
   HTML

<   input id="ex1" data-slider-id='ex1Slider' type="text" data-slider-min="0" data-slider-max="20" data-slider-step="1" data-slider-value="14"/>
JavaScript

$('#ex1').slider({
formater: function(value) {
return 'Current value: ' + value;
}
});
   CSS

#ex1Slider .slider-selection {
background: #BABABA;
}

How did you solve it eventually?

initialize the slider on document ready

@manasa91 Help me out here, because I'm having the same issue. I tried putting the slider initialization in document ready ($('#price-slider').slider();), but it doesn't seem to change anything and I still get the javascript error.


ansible-lint: yaml errors unclear

Summary
YAML-errors are not displayed properly in vscode problems panel.

Instead of the actual issues:

Extension version
v1.0.90
VS Code version
1.72.2
Ansible Version
$  ansible --version
ansible [core 2.11.12]
  config file = /workspaces/redacted/ansible/ansible.cfg
  configured module search path = ['/home/vscode/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/vscode/.local/lib/python3.10/site-packages/ansible
  ansible collection location = /home/vscode/.ansible/collections:/usr/share/ansible/collections
  executable location = /home/vscode/.local/bin/ansible
  python version = 3.10.7 (main, Oct  5 2022, 14:33:54) [GCC 10.2.1 20210110]
  jinja version = 3.1.1
  libyaml = False

OS / Environment
Docker Container based on Debian 11, with Python 3.10.7. Remote Extension Version v0.255.4
 $  ansible-lint --version
WARNING: PATH altered to include /usr/local/bin
ansible-lint 5.4.0 using ansible 2.11.12```

### Relevant log output

```shell
"diagnostics": [
        {
            "message": "Rule violations reported by YamlLint when this is installed.\n\nYou can fully disable all of them by adding 'yaml' to the 'skip_list'.\n\nSpecific tag identifiers that are printed at the end of rule name,\nlike 'trailing-spaces' or 'indentation' can also be be skipped, allowing\nyou to have a more fine control.\n\nBy default this rule is not used when yamllint library is missing. If you want\nto make its absence a runtime failure, please add 'yaml' to 'enable_list'\ninside the configuration file.\n",
            "range": {
                "start": {
                    "line": 10,
                    "character": 0
                },
                "end": {
                    "line": 10,
                    "character":<PHONE_NUMBER>
                }
            },
            "severity": 1,
            "source": "ansible-lint",
            "codeDescription": {}
        },

@max06 Does updating the version of ansible-lint change the behavior? (v6.8.1 or later)

That's a great hint - it does!
Guess I need to update ansible so we can use the newer version.
(closing) Thx!


Prevent resource leak by closing the PreparedStatement

fix(jdbc): avoid resource leak when using batching, by closing the PreparedStatement
close  #321

Yes I did. I modified the test BatchTest::namedInsert to insert 2.000.000 rows into a table instead of 5.
As expected, the test fails:

Then, I applied the fix, and now the test passes:

@lburja great! Thanks for the confirmation


Account for new brief withdraw and delete process

https://trello.com/c/JhIzksR3/116-2-replace-banners-with-actions-with-confirm-action-page-pattern
I'm not sure of the best way to make these work before the change and after it, or the exact process for releasing bit by bit.

In this situation it's probably best to make a copy of the original scenario with @skip-local @skip-preview tag, and add a @skip-staging tag to the new scenario. Then as the changes get released you move the tags around as required.
It's a convoluted process, but it does work. Here's an example of it in the past:

Add the new tests https://github.com/alphagov/digitalmarketplace-functional-tests/pull/717
Update tags with release to staging https://github.com/alphagov/digitalmarketplace-functional-tests/pull/720
Remove old tests with release to production https://github.com/alphagov/digitalmarketplace-functional-tests/pull/721


dubbo2.6.x以上版本javassist无法动态加载class

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.6.0及以上
Operating System version: mac
Java version: 1.8

Steps to reproduce this issue

2.5.3版本通过javassist动态加载class，没有任何问题
2.6.0及以上版本通过javassist加载class，在idea内启动项目没有问题，但是通过springboot java -jar启动后会报javassist.NotFoundException异常。

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
Caused by: javassist.NotFoundException: com.alibaba.dubbo.common.bytecode.Wrapper
at javassist.ClassPool.get(ClassPool.java:452)
at com.alibaba.dubbo.common.bytecode.ClassGenerator.toClass(ClassGenerator.java:263)
What do you expected from the above steps？
Actual Result
无法正常加载业务class
What actually happens?
If there is an exception, please attach the exception trace:
Just put your stack trace here!

Try it with the latest version, if you still have problems, you can reopen the issues


raysession: Fix issues with qt not being patched (can’t find platform, qt version mismatch error etc)

Description of changes
Fix https://github.com/NixOS/nixpkgs/issues/228223
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, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/help-fixing-raysession-package/28606/6

Just tested with nixpkgs-review pr 236762 and everything passed.


test: update BarChart visual test to remove flaky test

[FX-NNNN]
Description
More and more we start seeing flaky test of BarChart

I have removed the tooltip and labels from the test as it is already tested in different scenario.
How to test

Check and Approve the Happo change

Screenshots
n/a
Development checks

n/a Add changeset according to guidelines (if needed)
n/a Read CONTRIBUTING.md and Component API principles
n/a Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
n/a Annotate all props in component with documentation
n/a Create examples for component
n/a Ensure that deployed demo has expected results and good examples
n/a Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests

Breaking change

n/a codemod is created and showcased in the changeset
n/a test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers


[SPARK-17238][SQL] simplify the logic for converting data source table into hive compatible format

What changes were proposed in this pull request?
Previously we have 2 conditions to decide whether a data source table is hive-compatible:

the data source is file-based and has a corresponding Hive serde
have a path entry in data source options/storage properties

However, if condition 1 is true, condition 2 must be true too, as we will put the default table path into data source options/storage properties for managed data source tables.
There is also a potential issue: we will set the locationUri even for managed table.
This PR removes the condition 2 and only set the locationUri for external data source tables.
Note: this is also a first step to unify the path of data source tables and locationUri of hive serde tables. For hive serde tables, locationUri is only set for external table. For data source tables, path is always set. We can make them consistent after this PR.
How was this patch tested?
existing tests

cc @yhuai @gatorsmile @liancheng @clockfly

Test build #64425 has started for PR 14809 at commit 915d2b5.

Test build #64425 has finished for PR 14809 at commit 915d2b5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

https://github.com/apache/spark/pull/13060 This change is based on another PR. If users specify the location in CREATE TABLE, we always set the table type to EXTERNAL.

Condition 2 is not always true when condition 1 is true. I found an exception.
          val schema = StructType(StructField("b", StringType, true) :: Nil)
          sparkSession.catalog.createExternalTable(
            "createdParquetTable",
            "parquet",
            schema,
            Map.empty[String, String])

I think this is a bug. Do you want to fix it in this PR? Or I can fix it in another PR?

@gatorsmile can you explain more about this example? I think we will throw exception in CreateDataSourceTableCommand when we create a DataSource and call its resolveRelation.

If we do not specify the schema, it will behave like what you said. For example,
          sparkSession.catalog.createExternalTable(
            "createdParquetTable",
            "parquet",
            Map.empty[String, String])

Unable to infer schema for ParquetFormat at . It must be specified manually;
org.apache.spark.sql.AnalysisException: Unable to infer schema for ParquetFormat at . It must be specified manually;

However, if we specify the schema, we will not get an error.

ah i see, btw in your example when will we throw exception? when we read it? a file-based external table without path is invalid.

Yeah, that is a bug. We did not get an exception when we read it, but we can get the error when trying to write it. The error message is confusing
Can only write data to relations with a single path.;
org.apache.spark.sql.AnalysisException: Can only write data to relations with a single path.;
	at org.apache.spark.sql.execution.datasources.DataSourceAnalysis$$anonfun$apply$1.applyOrElse(DataSourceStrategy.scala:167)

Test build #64974 has started for PR 14809 at commit 7caf2fb.

Test build #64974 has finished for PR 14809 at commit 7caf2fb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #64981 has started for PR 14809 at commit 7caf2fb.

Test build #64981 has finished for PR 14809 at commit 7caf2fb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #64988 has started for PR 14809 at commit 7caf2fb.

Test build #64988 has finished for PR 14809 at commit 7caf2fb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

retest this please

LGTM pending test

Test build #65002 has started for PR 14809 at commit 7caf2fb.

Test build #65002 has finished for PR 14809 at commit 7caf2fb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

thanks for the review, merging to master!


can i remove androidx?

if androidx is enabled, building time is so long and big apk size so i want remove androidx but i cannot remove androidx because used androidx this project
how to remove androidx or close this issues if cant remove androidx
android.useAndroidX=false

sorry for bad english

we use androidx's annotation @Keep for R8 and Fragment support for androidx.
Dependencies are automatically wired, you cannot control what dependencies gradle needs to import, you can fork this project and replace it with minimal dependencies such as androidx.annotation and androidx.fragment, but this may still cause some problems, we do not recommend removing it fundamentally, because the compilation speed is related to gradle, which cannot be controlled.

OK， but support library was outdated

Alright🤪


Hello tycomo, I got an error related compatible issue

when I installed ssrs report viewer latest version 3.1.1 and my project angular CLI is @13.2.6
I got an compatible issue like
'ReportViewerModule' does not appear to be an NgModule class.(-996002)
reportviewer.module.d.ts(1, 22): This likely means that the library (ngx-ssrs-reportviewer) which declares ReportViewerModule is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.
could please tell me, how to overcome this issue

This repo does not support the Angular Ivy compiler. So only angular versions < 13 are supported.
I made a fork that adds support to more recent angular versions here.


NetworkImage not working

latest version of flutter, dart
 Container(
            height: 155,
            width: double.infinity,
            decoration: BoxDecoration(
              // color: Colors.blue,
              image: DecorationImage(
                  image: NetworkImage(
                      "http://images.fineartamerica.com/images/artworkimages/mediumlarge/1/purple-mountains-joye-ardyn-durham.jpg"),
                  fit: BoxFit.cover,
                  repeat: ImageRepeat.repeat),
            ),
          ),

Logs:
.
.
.
Reloaded 2 of 436 libraries in 1,184ms.
Reloaded 2 of 436 libraries in 1,041ms.
I/flutter (11955): ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
I/flutter (11955): The following SocketException was thrown resolving an image codec:
I/flutter (11955): OS Error: Connection reset by peer, errno = 104, address = images.fineartamerica.com, port = 52436
I/flutter (11955):
I/flutter (11955): When the exception was thrown, this was the stack:
I/flutter (11955): #0      NetworkImage._loadAsync
I/flutter (11955): <asynchronous suspension>
I/flutter (11955): #1      NetworkImage.load
I/flutter (11955): #2      ImageProvider.resolve.<anonymous closure>.<anonymous closure>
I/flutter (11955): #3      ImageCache.putIfAbsent
I/flutter (11955): #4      ImageProvider.resolve.<anonymous closure>
.
.
.

D/EGL_emulation(11955): eglMakeCurrent: 0x72f41a24f9a0: ver 3 0 (tinfo 0x72f40852f9e0)

Solved by using VPN !


The game will spoil secret to you as a game admin if you let it automatically deadmin you

Description

Join after a round ends as a game admin, or join and wait for the round to end
Ready up
You will be sent what gamemode is actually chosen a few seconds before the game starts
The game starts and deadmins you, and now you have to pretend like you didn't see anything

The admin message for secret gamemode should be sent after the automatic deadmin (probably easier), or the deadmin should happen sooner.

skill issue on behalf of admins who don't deadmin when they want to play
However maybe the solution is just to have clicking the ready up button automatically deadmin you instead of waiting for shiftstart?


Write basic documentation for http://wookmark.readthedocs.org/

Also change to a nicer theme.

Discontinued


I hope to support trace.

Is your feature request related to a problem? Please describe.
If an interface is very slow, but by looking at the SQL, HTTP, and other IO calls, I can't find where it's slow.
From the flame chart is not always found, I have encountered this scenario before, it is actually a single IO call is not slow, but there is 1 loop inside the interface that will loop many times. At that time, I turned on the flame chart monitoring through async-profiler and did not find the problem.
Describe the solution you'd like
I hope that if we can implement trace based on ebpf, it would be perfect.
Describe alternatives you've considered
The alternative would be to integrate other trace systems, but I don't think this is a good solution, it would require the installation of other components and might cost extra.

Hi @asasas234 ,
Thank you for the comments. I will discuss this with some of the team members. I cannot say (at this time) that anything new would be added to the roadmap, but if I understand what you are talking about, it is philosophically something that would be interesting and useful to support.
Could you possibly describe more about how you might want to see the information? Would your ideal flow be to:

identify a slow request/response in protocol tracing (e.g. http events, sql events) and then,
see the flamegraph associated with that?

Would you want:
a. a stack trace for one certain message/request/response sent by the protocol, or
b. a flamegraph that aggregates stack traces for all events/messages on a certain connection?
Or possibly I have not described your request (and you can write about it some more).
Thank you!

The trace feature I'm looking for is similar to something like Jaeger. A few weeks ago, I encountered a performance issue. At that time, I looked at the flame graph on async-profiler but couldn't identify the specific reason for the slowness. I later discovered through the application logs that the problem was caused by the pagination query only fetching 10 rows at a time, resulting in many pages being fetched in a loop. Because there were too many looped queries, the overall interface was slow. In a situation like this, I couldn't identify the issue using the flame graph on async-profiler, and I think it would be difficult to pinpoint the problem on Pixie as well. As for SQL queries, I can only see if a single SQL query is slow, but I have no way of knowing whether these SQL queries belong to the same request. Therefore, in scenarios like this, a trace feature similar to Jaeger is essential.


Add JS build. Fix multiplatform test platform

Use common load function in commonTest
Implement common load function in JVM
Implement common load function in JS. Use mocha for testing

This PR has been open for too much.


possible show tooltip all the time?

Like this?
    $('.tooltip').tooltipster({         trigger: 'custom'     }).tooltipster('open');
http://jsfiddle.net/m9she00L/2/

yes acactly :) thnks BTW i have a list with much keywords and some needs a littel explpainateion (halt transparent, and 0 margin, half over the original text). less space as posible. thats reasion whey i searching.


Use time.perf_counter instead of time.time

See https://docs.python.org/3/library/time.html#time.perf_counter; time.perf_counter should be used for performance measurements.
These changes are entirely untested.
Performance changes
python3.8 -m timeit -vvvvv 'time.time() - time.time()'
1 loop -> 2.731e-06 secs
2 loops -> 1.617e-06 secs
5 loops -> 2.275e-06 secs
10 loops -> 3.926e-06 secs
20 loops -> 7.546e-06 secs
50 loops -> 1.842e-05 secs
100 loops -> 5.9536e-05 secs
200 loops -> 7.883e-05 secs
500 loops -> 0.000194123 secs
1000 loops -> 0.000410747 secs
2000 loops -> 0.000706704 secs
5000 loops -> 0.001761156 secs
10000 loops -> 0.003544081 secs
20000 loops -> 0.009731159 secs
50000 loops -> 0.01932015 secs
100000 loops -> 0.03682307 secs
200000 loops -> 0.07023627 secs
500000 loops -> 0.1915937 secs
1000000 loops -> 0.3505069 secs

raw times: 340.1403 msec, 347.3075 msec, 336.8322 msec, 336.1656 msec, 338.9545 msec

1000000 loops, best of 5: 336.1656 nsec per loop

python3.8 -m timeit -vvvvv 'time.perf_counter() - time.perf_counter()'
1 loop -> 2.081e-06 secs
2 loops -> 2.551e-06 secs
5 loops -> 4.011e-06 secs
10 loops -> 7.107e-06 secs
20 loops -> 1.332e-05 secs
50 loops -> 3.2143e-05 secs
100 loops -> 6.3795e-05 secs
200 loops -> 0.00010457 secs
500 loops -> 0.000293773 secs
1000 loops -> 0.000447542 secs
2000 loops -> 0.000861884 secs
5000 loops -> 0.002183934 secs
10000 loops -> 0.003578807 secs
20000 loops -> 0.007985933 secs
50000 loops -> 0.02508519 secs
100000 loops -> 0.05550567 secs
200000 loops -> 0.07348392 secs
500000 loops -> 0.1892336 secs
1000000 loops -> 0.3787163 secs

raw times: 361.4678 msec, 349.8074 msec, 362.9233 msec, 368.9578 msec, 354.8154 msec

1000000 loops, best of 5: 349.8074 nsec per loop

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   37.42%   37.42%
=======================================
  Files          12       12
  Lines         481      481
=======================================
  Hits          180      180
  Misses        301      301

Impacted Files
Coverage Δ

jetpack/timepiece.py
61.29% <0.00%> (ø)

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 c9169ef...855ccbf. Read the comment docs.


Remove more self-publishing platforms from Awario News/Blog reach

What is the problem you are seeing? Please describe.
Other self-publishing platforms are being categorised as News/Blogs on Awario, and have incorrect reach for individual posts. So far I've identified:

dev.to
Indie Hackers community forum: https://www.indiehackers.com
ekşi sözlük: https://eksisozluk.com

(In the past we already removed medium and meetup)
How is this problem misaligned with goals of app mining?
The incorrect Awario reach of self-published posts invalidates the news/blogs reach metric, that should translate the effort to reach out to more traditional media outlets.
What is the explicit recommendation you’re looking to propose?
Automatically remove self-publishing platforms with incorrect reach from the Awario score. Ideally, without the need to create a new App Mining issue every time.
What is the dry run period (if any)
Could be applied already to the current audit period.
Describe your long term considerations in proposing this change. Please include the ways you can predict this recommendation could go wrong and possible ways mitigate.
It would keep the news/blogs awario reach focused on what it should measure.

Beat me to it, I generally support this, though people aren't getting an insane advantage from it and it's generally good folks are posting in these places. Unfortunately, I do think if we leave it available, someone will abuse it.
I was thinking about something where we could limit to one per month in places like this. So people are still incentivized to publish in these valuable places, but it can't be spammed for ridiculous Reach scores.

@cuevasm these platforms are definitely nice for promoting apps. But a 50K reach is enough to skew all other news/blogs mentions. If anyone could do that once per month, it would turn the reach into another binary score.

I'm not seeing how that would make it binary. Everyone would have the opportunity to self-publish on a set of sites such as these and then the Reach they earn would be limited to one of them in a period. Their Reach scores would all be different and many wouldn't take advantage of it. If anything it gives more dedicated teams more of means to differentiate themselves.
I would agree about outright removing this site: https://eksisozluk.com/ as it seems to be 99% gibberish. I've reached to Awario as to why that even made the cut as a News/Blog in the first place.

Taking out the big whales like medium and eksisozluk out, it would level things a bit more. But the News/Blogs reach score would become a race to find the best self-publishing platforms with an incorrect reach score on Awario. That would become more fruitful than actually reaching out to news and blogs pitching products.

Just for clarification, platforms/sites are removed when Awario can't determine a realistic reach. "Publishing yourself" is not a criteria that would disqualify an article.

I think we're generally trying to remove sites that a) have unrealistic Reach numbers like Medium and/or b) can be abused because they are open to user-generated content or c) combinations of the two.
I generally think publishing yourself should disqualify something, though it currently does not under the rules. The idea is 'earned media' not 'owned content' and ultimately, when people find self-publishing channels that they can freely post on and garner News/Blog mentions, they end up getting abused.

I want to clarifying that eksisozluk thing.
That entry belongs to a friend i recently met on a cryptocurrency event and introduced blockstack and mentioned my apps on it. A couple days later i noticed she posted lander on eksisozluk. Without my information. (she posted an entry about runkod also but awario deleted that somehow) Hereby that post is 100% organic.
Additionally, eksisozluk is not good place for self-publishing. You can't just signup and post content right away. They have a very long account verifying mechanism.

There haven't been any objections and the next period is coming up, based on this ticket, previous conversation, and the call before last, it sounds like most people are supportive of removing these self-publishing platforms.
Will proceed with the 3 found in this ticket:

dev.to
https://www.indiehackers.com
https://eksisozluk.com

Reach from these sites will not be counted for the January run. (Treated the same as medium.com)

Based on this ticket, a lack of objections, the previous discussion, and the call before last, it seems most people are supportive of removing these types of sites.
Proceeding with removing dev.to, indiehackers.com, and eksisozluk.com for the next run (January), they will now be treated the same as medium.com.

@cuevasm  I do have an objection: There should be sufficient notice if a change is being made. It appears from your statement that you intend to apply this to the January App Mining period, which is based on December reach. This means that anyone who used these sites to promote during December will not have them counted toward reach for January App Mining. I used some of these sites for December reach because they were acceptable during December App Mining period. Changes to what qualifies for reach should be applied at the beginning of the next App Mining period.


Spotify Auth

Add the driver to handle Spotify OAuth2, also with some tests (same as the other drivers).
Also add an example that work well.
I'd also would like to take this PR to thanks you for Adonis, great framework! Love playing and working with it 👍

Closing since tests are failing. Feel free to reopen when all works fine


Please explain this action in the readme?

What does it do?
Are there dependencies?
How to use it?

Hi @geoidesic currently this project and action are under development during my spare time. It's not meant for real usage.

What does it do?

It's a github action to run votes inside of github issues, similar to https://github.com/tfadeyi/oss-governance-test/issues/41.
Any issues opened with a specific label, that you can set, will have a voting comment.

Are there dependencies?

It requires a github token, since it uses the github api. It doesn't require any other dependency as it runs on as a github action.

How to use it?

I would suggest not using it for now has it's still not ready.
Would this type of github action be something you would be interested in?

Not knowing much about it, it's hard to say. I have been looking for a feature voting thing in GitHub. Yours is the closest that popped up on search. Considered writing my own but it's a bridge far too far. It seems like a big, missing, no brainer for open source dev.

I'm planning to continue the work on this. But there are a couple other projects available like https://github.com/cncf/gitvote that do something similar

Ok tx.


Add default LookHorizontal/LookVertical axes to control camera rotation with gamepad.

Overview
Add default axes in the project input configuration to control head rotation with gamepads.
Changes

Fixes: #3892 .

shouldn't LookVertical have Invert property to true ?
not sure about the standard, may be a personal preference.

Yes, i also prefer inverted, but it seems arbitrary.

We've already setup axis 3 & 4, why not just utilize what's already there and fix the profile to utilize the correct axes?

Yes, i've been discussing this with Luis. The thinking was to add new input axes with a specific meaning instead of the the generic "AXIS_", but i see the point of using the profile as the point of configuration rather than changing the project settings. I'll change this.
That being said, using input actions as you suggested on the bug report is probably a better solution altogether. I'd like to make the input simulation profile consistent too, and use the same method for assigning actions for camera control and hand simulation.

/azp run

/azp run


Error in uploaded report from GitHub Actions

I've been trying to debug this. I should be missing something in my workflow.
Report that is uploaded correctly from GitHub Actions CI, see log:
| |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| |  __/ (_| (_) \ V /
 \_____\___/ \__,_|\___|\___\___/ \_/
                              Bash-20200430-d757c17

==> GitHub Actions detected.
    project root: .
    Fixing merge commit SHA
    Yaml found at: codecov.yaml
    -> Found 1 reports
==> Detecting git/mercurial file structure
==> Reading reports
    + ../ncio-build/meson-logs/coverage.txt bytes=904
==> Gzipping contents
==> Uploading reports
    url: https://codecov.io
    query: branch=coverage-bug&commit=aef44e701df1435a28a6f60037da2ff266626013&build=121461088&build_url=http%3A%2F%2Fgithub.com%2FORNL%2Fncio%2Factions%2Fruns%2F121461088&name=&tag=&slug=ORNL%2Fncio&service=github-actions&flags=&pr=25&job=
    -> Pinging Codecov
https://codecov.io/upload/v4?package=bash-20200430-d757c17&token=secret&branch=coverage-bug&commit=aef44e701df1435a28a6f60037da2ff266626013&build=121461088&build_url=http%3A%2F%2Fgithub.com%2FORNL%2Fncio%2Factions%2Fruns%2F121461088&name=&tag=&slug=ORNL%2Fncio&service=github-actions&flags=&pr=25&job=
    -> Uploading
    -> View reports at https://codecov.io/github/ORNL/ncio/commit/aef44e701df1435a28a6f60037da2ff266626013

coverage.txt supported formats
------------------------------------------------------------------------------
                           GCC Code Coverage Report
Directory: /__w/ncio/ncio
------------------------------------------------------------------------------
File                                       Lines    Exec  Cover   Missing
------------------------------------------------------------------------------
bindings/CXX17/ncio/cxx17/cxx17NCIO.cpp        9       0     0%   9-10,14,16,18,21,23,26,28
source/ncio/core/NCIO.cpp                     10      10   100%
source/ncio/core/NCIO.h                        1       1   100%
testing/unit/ncio/core/unitNCIO.cpp            6       6   100%
------------------------------------------------------------------------------
TOTAL                                         26      17    65%
------------------------------------------------------------------------------

codecov shows there is an Error:
https://codecov.io/gh/ORNL/ncio/commit/08e67baf2c0cd4677ecd10dabc0e918d0e089551
I looked into fixing the file paths using this codecov.yaml but no luck:
# From CI for coverage
  - "/__w/ncio/::"

I'd appreciate any help on this. Either a working example or better error debugging message (something that could dump the right fix-path line) after uploading would be very helpful.  Thanks!

Hi @williamfgc, could you open a ticket in our community boards?

@thomasrockhu thanks for the reply. Actually I found the issue was that the coverage.txt gcov generated file is not supported. Using coverage.xml solved it. I'll stick with coverage.xml as the important thing is to have the report in codecov. Thanks!

Awesome @williamfgc!


Plugin doesn't work unless manually setup

Description
The plugin doen't work (does not auto pair my "(") unless manually doing :lua require("nvim-autopairs").setup() in command (or adding an extra line after the plugin manager).
NVIM: 0.9.1
Steps to reproduce

open nvim
enter insert mode
type "("

Minimal config
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)
require("lazy").setup({
        "windwp/nvim-autopairs"
})

If I append an extra line
require("nvim-autopairs").setup()

it works.

Most of neovim plugin require to call a setup function.
so it can easy add a custom config or lazy load.

but is it called by the plugin manger, i.e. lazy.nvim here?

I have the same problem here, idk why setup isn't called automatically by lazy

If you will not pass arguments to setup through the opts option of lazy, you should add the config = true option so that lazy knows to do the require('...').setup() call.
return {
  'windwp/nvim-autopairs',
  config = true,
}

Alternatively you can set opts = {}
See here: plugin spec


CRC error when uploading a specific content to Google cloud storage

The following code results in error message Provided CRC32C "..." doesn't match calculated CRC32C "...":
Storage storage;
String bucketName = "bucket";
String blobName = "gtest";
String GOOGLE_CREDENTIALS_SCOPE = "https://www.googleapis.com/auth/cloud-platform";
String APPLICATION_TYPE = "application/octet-stream";
String googleCredentialsJson = null;

ArrayList al = new ArrayList();
al.add(GOOGLE_CREDENTIALS_SCOPE);
String s = String.format("\r\n--__END_OF_PART__--\r\n01234");

googleCredentialsJson = new String ( Files.readAllBytes( Paths.get("google.json") ) );
GoogleCredentials credentials = GoogleCredentials.fromStream(new ByteArrayInputStream(googleCredentialsJson.getBytes()))
                .createScoped(al);
storage = StorageOptions.newBuilder().setCredentials(credentials).build().getService();

BlobId blobId = BlobId.of(bucketName, blobName);
BlobInfo blobInfo = BlobInfo.newBuilder(blobId).setContentType(APPLICATION_TYPE).build();

storage.create(blobInfo, s.getBytes());

The issue is reproducible using example programs on Linux:
printf "\r\n--END_OF_PART--\r\n01234\n" > rr
java -jar ... com.google.cloud.examples.storage.StorageExample  upload rr bucket

@dk823832 I have tried it and able to reproduced.Contrary to your claim \r\n--__END_OF_PART__--\r\n01234, \r\n It's calling BufferedWriter.newLine() which uses the current platform's default line separator (unless you explicitly set the line.separator system property). On Unix systems this will be "\n" so you just add \n instead of  \r\n error will gets disappear.
String s = String.format("\n--__END_OF_PART__--\n01234");

This is not about platform specific line separator. The issue is reproducible on Linux, see my second comment how to reproduce it. Storage API accepts a byte array so it shouldn't matter what content that array contains and it should be able to upload it.

Any news on this issue? Another reproducer on Linux without using String.format:
replace String s = String.format("\n--END_OF_PART--\n01234");
with byte s[] = {13, 10, 45, 45, 95, 95, 69, 78, 68, 95, 79, 70, 95, 80, 65, 82, 84, 95, 95, 45, 45, 13, 10, 48, 49, 50, 51, 52};
and storage.create(blobInfo, s.getBytes());
with storage.create(blobInfo, s);

The problem appears only if the given content includes "--END_OF_PART--" string which is used as a part delimiter by com.google.api.client.http.MultipartContent.java
(https://github.com/googleapis/google-http-java-client/blob/master/google-http-client/src/main/java/com/google/api/client/http/MultipartContent.java)
Such content being sent is recognized by the server side as multiple parts and handled in a different way: CRC32C/MD5 calculated only for the first part.
If the content includes "--END_OF_PART" (without two dashes at the end) the server will reject:
Caused by: com.google.api.client.googleapis.json.GoogleJsonResponseException: 400 Bad Request
Malformed multipart body.


Fix a missing character in documentation

It needs a second 'm' in 'command'.

Good catch, thank you 🍰


Correspondence - Email Template for Sending Fees

As an IAO analyst
I want to select an email template for Sending Fees
so that I can easily send an email to the applicant

Assumptions & Scope
What are the assumptions for this story?
IAO will need to send new fee estimates when a request is narrowed, the email failed or the applicant cannot find the email.
Letter templates will continue to be created in AXIS.
What is IN scope?
Only email templates for fees: new fee estimate or outstanding fees.
Email templates will include links for the latest CFR.
What is NOT in scope?
Applicant Correspondence Log is covered in #2684.
Other email or letter templates.
Acceptance Criteria
Scenario 1: Email Template for sending Fee Estimate (no deposit or amount has been paid)
•	GIVEN the IAO analyst is on the Applicant Correspondence Log https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/941001462
•	WHEN the analyst clicks the template drop-down
•	THEN there will be 2 options: 'Fee Estimate Email' or 'Outstanding Fee Email'
Scenario 2: Email Template for sending Fee Estimate (no deposit or amount has been paid)
•	GIVEN the IAO analyst is on the Applicant Correspondence Log https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/941001462
•	AND the has opened the template drop-down
•	WHEN the user selects 'Fee Estimate Email'
•	THEN an email fee estimate template will populate in the comment with the Applicants NAME, the Analysts NAME, the Team NAME, the date and time, and a URL link
•	AND the URL link will have the amount owed from the latest CFR Form
Scenario 3: Email Template for sending Outstanding Fee (deposit or amount has been paid)
•	GIVEN the IAO analyst is on the Applicant Correspondence Log https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/941001462
•	AND the has opened the template drop-down
•	WHEN the user selects 'Outstanding Fee Email'
•       THEN an email fee estimate template will populate in the comment with the Applicants NAME, the Analysts NAME, the Team NAME, the date and time, and a URL link
•	AND the URL link will have the outstanding 'Balance Due' from the latest CFR Form
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@m-prodan - For your review - latest CFR for sending fees.

Thanks, @KaraBeach. I have made some edits.
@m-prodan, @arielleandrews, please review.

One question - should the analyst be able to edit the Template content?

Content will be from the design.

Yes, I think the analyst should have the ability to edit the template content. Perhaps we can add in the assumptions that it will be a RT editor? So it would populate the default text, and analysts can edit as they need to?
Also just thinking, we may want a "save draft" feature - but it may not be necessary at this point since the letters will still be generated in AXIS for now, since this is just an email function.

As agreed, we will not having editing on the first release of the email template.

Estimate includes rich text editor so analyst can edit the template.
Rich text editor will work the same as comments.
Will write in a separate story and re-estimate.

Passes QA however there are a couple of concerns with "scenario 2". Specifically:

WHEN the user selects 'Fee Estimate Email'
THEN an email fee estimate template will populate in the comment with the Applicants NAME, the Analysts NAME, the Team NAME, the date and time, and a URL link

Analysts may find this confusing because the actual applicant data is not displayed in the email template, it shows the field title the data is being drawn from (e.g. {{firstName}}).
These parameters are live and because the email is editable, there is room for analyst error (deletion of all or a portion of a required field).


insane heisenbug with WASM FM patches

The web example basically crashes (but AMY still responds to debug messages) if you send a l0 to a certain FM patch, #0 is one. but it only does this if it's running on the web, on a remote server (local hosting the JS/WASM works fine!) I expect some memory corruption but valgrind came back clean.

The issue was my check for how many ops were alive to compute the amplitude
but during note offs, the ops were set to 0.. but shouldn't they still be active during note off / releases?
NaN issue is fixed but we should look into note off behavior

(also, super weird that local WASM doesn't crash on nans but remote does -- a security thing?)


3.2.0 is not sem-ver compatible with 3.1.0

Yesterday valico released 3.2.0 with updated dependencies. One of the dependencies updated is url which went from 1 to 2.
Sadly, this seems to have been a breaking change in the public API as url::Url is exposed in the pubic API such as the Schema::resolve method.
And so code like the following will fail to compile after a cargo update:
extern crate url; // 1.* series
extern crate valico; // 3.* series

Scope::resolve(unimplemented!(), url::Url::parse("hello").unwrap());

// error[E0308]: mismatched types
//    --> src/test.rs
//     |
// 403 |             .resolve(&msg_schema_url)
//     |                      ^^^^^^^^^^^^^^^ expected struct `url::Url`, found a different struct `url::Url`
//     |
//     = note: expected reference `&url::Url` (struct `url::Url`)
//                found reference `&url::Url` (struct `url::Url`)
//     = note: perhaps two different versions of crate `url` are being used?

We should yank 3.2.0 and consider releasing 4.0.

Oops, I overlooked this. I'll fix it as soon as I get a chance, thanks!


docs: new section "Next Generation Repositories (NGR) support"

What is COAR NGR?
Supported:

COUNTER
Creative Commons Licenses
ETag
International Image Interoperability Framework
ORCID
OpenID Connect
Signposting
Sitemaps
Social Network Identities

Planned:

ResourceSync
SUSHI
SWORD

Unsupported (currently):

Activity Streams 2.0
HTTP Signatures
Linked Data Notifications
IPFS
Web Annotation Model and Protocol
WebID
WebID/TLS
Webmention
WebSub

The content can easily be adapted from http://ngr.coar-repositories.org/ which is available at https://github.com/coar-repositories/ngr (without licence).


Native image backwards compatibility

Discussed in https://github.com/microsoft/ebpf-for-windows/discussions/3839

Originally posted by lmb September 17, 2024
I recently tried to load a .sys file compiled against roughly v0.18 against current main. This failed due to a mismatch of the hash of the program info. It seems like upgrading the runtime broke the old native image.
Was this breakage intentional? What is the plan for backwards compatibility in the future? Having to always load the .sys against the correct version of the runtime would be very limiting.

Was this breakage intentional? What is the plan for backwards compatibility in the future? Having to always load the .sys against the correct version of the runtime would be very limiting.

We aren't aware of any backward compatibility breakage. Can you please share your native elf or program file to help repro from our side or the build error observed?

@lmb Any update?

Something very weird is going on which I can't explain.

EbpfGenericMessage       Program information provider module ID mismatch.

EbpfGenericMessage       Program information provider module ID mismatch.

EbpfGenericMessage       Program information provider module ID mismatch.

I believe the error is in EbpfGenericError  ebpf_program_set_program_info_hash returned error  Error=6 generated here: https://github.com/microsoft/ebpf-for-windows/blob/d712e97fd7baf62b13b6270fbdf3a78c69038d93/libs/execution_context/ebpf_program.c#L1881-L1888
The problem goes away after:

Uninstall runtime
Blow away ./x64/
Rebuild on command line (debug build)
Install runtime
Use printk.sys from new build

Here is a zip containing the two modules: printk.zip

93 [1]16A4.07C4::2024/10/08-12:02:02.834495500 [NetEbpfExtProvider]{"Enter":"_net_ebpf_extension_program_info_provider_attach_client","meta":{"provider":"NetEbpfExtProvider","event":"NetEbpfExtGenericMessage","time":"2024-10-08T19:02:02.8344955Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
94 [1]16A4.07C4::2024/10/08-12:02:02.834505200 [NetEbpfExtProvider]{"Message":"_net_ebpf_extension_program_info_provider_attach_client returned success","meta":{"provider":"NetEbpfExtProvider","event":"NetEbpfExtGenericMessage","time":"2024-10-08T19:02:02.8345052Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"keywords":"0x2"}}
95 [1]16A4.07C4::2024/10/08-12:02:02.834525700 [EbpfForWindowsProvider]{"Message":"Program information provider module ID mismatch.","*guid1":"{608c517c-6c52-4a26-b677-bb1c34425adf}","*guid2":"{92ec8e39-aeec-11ec-9a30-18602489beee}","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8345257Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":2,"keywords":"0x80"}}
96 [1]16A4.07C4::2024/10/08-12:02:02.834531000 [EbpfForWindowsProvider]{"Message":"Program information provider module ID mismatch.","*guid1":"{608c517c-6c52-4a26-b677-bb1c34425adf}","*guid2":"{43fb224d-68f8-46d6-aa3f-c856518cbb32}","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8345310Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":2,"keywords":"0x80"}}
97 [1]16A4.07C4::2024/10/08-12:02:02.834539500 [EbpfForWindowsProvider]{"Message":"eBPF object initialized","object":"0xFFFFB70848A16DC0","object_type":3,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8345395Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"keywords":"0x2"}}
98 [1]16A4.07C4::2024/10/08-12:02:02.834626500 [EbpfForWindowsProvider]{"Message":"ebpf_program_create returned success","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfSuccess","time":"2024-10-08T19:02:02.8346265Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"keywords":"0x2"}}
99 [1]16A4.07C4::2024/10/08-12:02:02.834627500 [EbpfForWindowsProvider]{"Entry":"ebpf_program_get_program_info","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8346275Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
100 [1]16A4.07C4::2024/10/08-12:02:02.834642300 [EbpfForWindowsProvider]{"Message":"ebpf_program_get_program_info returned success","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfSuccess","time":"2024-10-08T19:02:02.8346423Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"keywords":"0x2"}}
101 [1]16A4.07C4::2024/10/08-12:02:02.834643300 [EbpfForWindowsProvider]{"Entry":"ebpf_serialize_program_info","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8346433Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
102 [1]16A4.07C4::2024/10/08-12:02:02.834646800 [EbpfForWindowsProvider]{"Message":"Output buffer is too small","value1":2024,"value2":2706,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8346468Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":3,"keywords":"0x2"}}
103 [1]16A4.07C4::2024/10/08-12:02:02.834647900 [EbpfForWindowsProvider]{"ErrorMessage":"ebpf_serialize_program_info returned error","Error":24,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericError","time":"2024-10-08T19:02:02.8346479Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":2,"keywords":"0x2"}}
104 [1]16A4.07C4::2024/10/08-12:02:02.834705900 [EbpfForWindowsProvider]{"Entry":"_ebpf_program_zero_ref_count","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8347059Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
105 [1]16A4.07C4::2024/10/08-12:02:02.834706600 [EbpfForWindowsProvider]{"Entry":"_ebpf_program_detach_links","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8347066Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
106 [1]16A4.07C4::2024/10/08-12:02:02.834712300 [EbpfForWindowsProvider]{"Exit":"_ebpf_program_detach_links","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8347123Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":2,"keywords":"0x1"}}
107 [1]16A4.07C4::2024/10/08-12:02:02.834713100 [EbpfForWindowsProvider]{"Exit":"_ebpf_program_zero_ref_count","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2024-10-08T19:02:02.8347131Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":5,"opcode":2,"keywords":"0x1"}}
108 [1]16A4.07C4::2024/10/08-12:02:02.834716400 [EbpfForWindowsProvider]{"ErrorMessage":"_ebpf_core_protocol_get_program_info returned error","Error":24,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericError","time":"2024-10-08T19:02:02.8347164Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":2,"keywords":"0x2"}}
109 [1]16A4.07C4::2024/10/08-12:02:02.834720100 [EbpfForWindowsProvider]{"Api":""ebpf_core_invoke_protocol_handler"","status":"0x80000005(NT=Buffer Overflow)","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfApiError","time":"2024-10-08T19:02:02.8347201Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":2,"keywords":"0x4"}}
110 [1]16A4.07C4::2024/10/08-12:02:02.834798300 [EbpfForWindowsProvider]{"Api":"DeviceIoControl","last_error":"234(WIN=More data is available.)","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfApiError","time":"2024-10-08T19:02:02.8347983Z","cpu":1,"pid":5796,"tid":1988,"channel":11,"level":2,"keywords":"0x100"}}

C:\Users\TDPUser\Downloads>netsh ebpf add program printk.sys
Loaded with ID 4
C:\Users\TDPUser\Downloads>

@lmb : Please let us know if we can close this issue. There is no backward compatibility issue with printk.sys. Pasted the logs above.


feat(RHTAP-868): use-different-component-repos

added componentRepoUrl as the repo source for the component source
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue ticket number and link
https://issues.redhat.com/browse/RHTAP-868
Type of change

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

How Has This Been Tested?
By running load tests
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/cc @pmacik

/retest

/retest-required

/retest

/retest

/retest

/retest

/test redhat-appstudio-e2e

/retest

/approve cancel

/retest


update_engine: bugfix for missing COREOS_BOARD

We rely on COREOS_BOARD field in os-release file to determine the
board name, but this field is not present in ancient releases
(1298.7.0 stil does not have it).
This introduces a hardcoded fallback logic for such cases.

/cc @squeed @crawford @euank


Remove all @param alignment style in header docs

Keeping param descriptions aligned (adding whitespace) is too difficult to maintain and we end up in weird states like this over time. We need to do a pass over all headers and clean this up.

Might be a good opportunity to explore Orta's suggestion of including SpaceCommander to handle this for us in the future.