[GitHub Linguist] @media rules?

In AdGuard Filters maybe all usage was removed - maybe as outdated (or I can not remember files with ~13 rules).
I see these is now invalid at end:
rootblog.pl#$#@media all and (min-width: 1200px) { a[href*="news.google.com/publications/"] { visibility: hidden !important; } }
rootblog.pl#$#@media (min-width: 1200px) { a[href*="news.google.com/publications/"] { visibility: hidden !important; } }

rootblog.pl#$#@media all and (min-width: 1200px) { a[href*="news.google.com/publications/"] b { content: "Chcesz być na bieżąco?..." !important; font-weight: bold; position: absolute; left: 19%; color: var(--cs-color-site-background); background: var(--cs-color-site-background); visibility: visible !important; } }
rootblog.pl#$#@media (min-width: 1200px) { a[href*="news.google.com/publications/"] b { content: "Chcesz być na bieżąco?..." !important; font-weight: bold; position: absolute; left: 19%; color: var(--cs-color-site-background); background: var(--cs-color-site-background); visibility: visible !important; } }

In uBo run only as:
rootblog.pl##a[href*="news.google.com/publications/"]:matches-media((min-width: 1200px)):style(visibility: hidden !important;)
rootblog.pl##a[href*="news.google.com/publications/"] b::after:matches-media((min-width: 1200px)):style(content: "Chcesz być na bieżąco?..." !important; font-weight: bold; position: absolute; left: 19%; color: var(--cs-color-site-background); background: var(--cs-color-site-background); visibility: visible !important;)

Filters is useful for articles on rootblog.pl to hide nasty Google news promotion. For now only covered wide PC.
(I don't see opened issue in FiltersCompiler about this parole by gorhill - still rejected will be "key frames" or "supports")

Maybe should fixed by #50 if cover GitHub.

@krystian3w @Alex-302 Please try the temporary fix here
Commit reference: https://github.com/scripthunter7/VscodeAdblockSyntax/commit/b815876ad1e0c1b5e45d0c37f6cb02401bb12674

@krystian3w It seems that the new version has been uploaded to GitHub, but it will take some time before it will work properly everywhere:
#$#@media (a: b) { .selector { a: b; } }


Update llvm-version.txt

Update to a version that has SPIRV backend working. Fixes a problem with #202 build.

FYI @leshikus, seems like there's a missing dependency between the llvm build and compiler build jobs.

IMEX (our dev fork) can no longer be built with the new LLVM, should we pin older LLVM version for -DGC_USE_IMEX=ON builds or update our fork of IMEX to be compatible with the new LLVM ver? cc @kurapov-peter @Menooker


ignore Inertia::share() executing for some routes or groups?

I'm developing a laravel project with inertia + vuejs + fortify. i have some queries in in inertia share method. but when I
run some Ajax get requests, the inertia share method
also executing . i detected this when checking queries with laravel debug bar package. simple Ajax request also execute inertia share method. is there any way to ignore inertia
share method to some routes? thank you.

Hello, I just fixed this problem creating another route file and another middleware group same as web except inertia middleware.
app\Http\Kernel.php

app\Providers\RouteServiceProvider.php


wordpress puppeteer

i read in wordpress docs than gutenberg uses puppeteer, i also want to use puppeteer in wordpress but unable to find a way to use it,can you tell me the way or link to place where step by step procedure is held to use puppeteer on my wordpress site.
thanks

Hi there 👋  This project isn't officially affiliated with WordPress or Gutenberg, and it doesn't use Puppeteer.
You might have more luck at the WordPress Gutenberg project:

https://github.com/WordPress/gutenberg
https://developer.wordpress.org/block-editor/

Specifically, you might want to check on @wordpress/scripts, and its included features for using Puppeteer to do end-to-end testing:

https://developer.wordpress.org/block-editor/packages/packages-scripts/
https://www.liip.ch/en/blog/how-to-test-your-gutenberg-blocks


Navigation from topic api

What
Navigation information is now taken from dp-topic-api, with the previous markup provided if the navigation data is not supplied.
How to review
No special instructions.
Who can review
Anyone.

Built and unit tests are failing in CI with this error
/go/pkg/mod/github.com/!o!n!sdigital/log.go/v2@v2.2.0/log/log.go:16:2: missing go.sum entry for module providing package github.com/ONSdigital/dp-net/v2/request (imported by github.com/ONSdigital/log.go/v2/log); to add:

	go get<EMAIL_ADDRESS>
/go/pkg/mod/github.com/mattn/go-isatty@v0.0.14/isatty_tcgets.go:7:8: missing go.sum entry for module providing package golang.org/x/sys/unix (imported by github.com/fsnotify/fsnotify); to add:

	go get<EMAIL_ADDRESS>

Audit is failing because of new vulnerabilities rather than the build error mentioned above


LDAP support

Dear all,
I tested portainer and this is really good. However, can you add Ldap support? Or maybe it would be simplier if if we configure pamd for Ldap, and Portainer would rely on Pamd?
Thank you,
Antoine

Use access control is really new (1.12.0)
I think we have to implement groups and role base access before implementing external auth

We can separate Ldap tasks into 2: one for Ldap support for admin users only, and one when groups/roles has been defined and implemented. The first one is faster.

LDAP/AD support will be available later in the year as a paid plugin.
See the discussion here for more info: https://github.com/portainer/portainer/issues/550#issuecomment-287138803

LDAP support will be part of Portainer. We will start working on it when #750 is implemented.

personally I consider LDAP itself a very basic function while secondary LDAP is more an ee feature

Note that it'd be possible for someone to write a plugin for LDAP management and open source it :)

I'm currently working on the LDAP support feature.

Any ETA for this? As I am planning to run a portainer demo at QCRI?

Its in progress now... @alapenna can advise more accurate eta
Rgds,
Neil Cresswell
On 1/08/2017, at 7:14 PM, Ahmad Amr <notifications<EMAIL_ADDRESS>wrote:
Any ETA for this? As I am planning to run a portainer demo at QCRI?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_677-23issuecomment-2D319288258&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=XqryRPDPmos7ybi77AOA4GME5zvszoPMuu-C6G0W57M&s=5nlX44ntFW1GVmtJEoJ74Lm8C8o3p8f3RpiANwk7Xjs&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlfPz-5FFjmg8iWbQribsDvOXzugk1yks5sTtBKgaJpZM4MdwUq&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=XqryRPDPmos7ybi77AOA4GME5zvszoPMuu-C6G0W57M&s=qyeiif7JP8WEX9WeTWMUZf4Fvb99gs-bQ9oQOyCIaAE&e=.

@socrat3000 this will be available within 2 weeks. A test version will also be shared on our Slack channel soon.


Add Documentation 😏

No need to remind me this given any opportunity. Documentation is a major feature of the v3 and subsequent releases. The internal implementation changes too fast to document it though, although sadly I didn't really have motivation to do much coding in the last couple of days - was playing factorio instead.

Knowing what is the public API would help.  I mean, does ccmod.patchList really need to be exported ? Because if it isn't, that's less API documentation to write :wink:


[Excel] (Custom Functions) Update overview article

Fixes #2830 and #2829.
Re: the shared runtime note. Because this article includes a link to the Shared Runtime article in paragraph following this bullet point, I opted to remove the obsolete browser note entirely.

Docs Build status updates of commit 101d108:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9f0cc40:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 6cab967:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Added New Connectivity Tutorial

Configuration of fiori reference apps on an ABAP Backend system.

We reviewed this tutorial in a call today, and it looks good to me.  Ready for publication.

I have added the validation steps. The tutorial can go public from my point of view.
Thanks


Fix for #834

Description of Changes
Fix for #834
The canvas of the background was drawn over the children widgets.
fixed background canvas layer
Screenshots
Code example:
from kivymd.app import MDApp
from kivy.lang import Builder

kv = """
<TheCard@MDCard>:
    orientation: "vertical"
    padding: "8dp"
    size_hint: None, None
    size: "280dp", "180dp"
    md_bg_color:1,0,1,0.0

    MDLabel:
        text: "Title"
        theme_text_color: "Secondary"
        size_hint_y: None
        height: self.texture_size[1]
        canvas.before:
            Color:
                rgb:1,0,0
            Rectangle:
                pos:self.pos[0]-20,self.pos[1]-20
                size:self.size

    MDSeparator:
        height: "1dp"

    MDLabel:
        text: "Body"

Screen:
    TheCard:
        pos_hint: {"center": [0.5, 0.5]}

"""

class TestApp(MDApp):
    def build(self):
        return Builder.load_string(kv)

TestApp().run()
with the alpha of md_bg_color set to 1:

Before:

After:

@podraco Something strange is happening with your changes. Your pull requests contain changes that were already made in the previous pull request ...

Merge master into my personal branch, I'm still unable to do a direct git pull to the master branch

Just included your last changes to the docs

Still having this problem, using kivymd master branch.  i can see the children widget of the card need help pls


Submit reviews responder

This PR adds a responder listening to the following command: submit review <REVIEW-URL> time <REVIEW-HOURS>
The review URL, duration and date are logged in Airtable.
The responder can also add or remove labels when the number of reviews equals the number of reviewers.
Re: #9
Here's a screenshot of this responder's docs:

@xuanxu Thanks a ton!
Will it work if people mistakenly write "3.5h", "3.5 h", "1hour", "3 hours" etc?

Note: currently the review_date column in the reviews table is only present in the test Airtable database, it is missing from the production database.

Thanks, now added to the production database.

Also, does the "all reviews" completed depend on the reviewers assigned? (as opposed to an arbitrary number of reviews)

Also, does the "all reviews" completed depend on the reviewers assigned? (as opposed to an arbitrary number of reviews)

That is critically important, because there definitely are, and will be, cases with num_reviewers != 2.

Will it work if people mistakenly write "3.5h", "3.5 h", "1hour", "3 hours" etc?

Currently it would only accept "3", "3.5", "3,5", with no text after that: "submit review URW time 6.5".
Should I change it to accept also the [h/hour/hours] part?

Also, does the "all reviews" completed depend on the reviewers assigned?

Yes, the responder queries the airtable API for the reviews of all the assigned reviewers, and if they are all complete (review_url is not empty), then it consider all reviews are in and add the label. It works with any number of reviewers in the list. In the example in the docs there's only two, but could be any other number of reviewers.

Thanks @xuanxu!

Should I change it to accept also the [h/hour/hours] part?

Yes I think it'd be safer (with or without the space before the h).

Should I change it to accept also the [h/hour/hours] part?

Yes I think it'd be safer (with or without the space before the h).

@maelle Done!

@mpadge right now that format is not accepted and the command will be ignored. Once we have an "I dont understand that" responder, this will happen automatically. Do still want a specific error message for this case?

Only  because i though that would be easy to do, and then useful to have a specific response telling people that their time values aren't properly formatted. Your call, but if it's not too difficult, then yes, please!

I'll wait for @xuanxu's answer before merging and testing this.

I've added a simple validation: if the time value includes a colon : the response will be an "Invalid time format" comment.

Thank you @xuanxu!!


Update README.md

Thank you for contributing to stackforge/fuel-plugin-ha-fencing!
stackforge/fuel-plugin-ha-fencing uses Gerrit for code review.
Please visit http://wiki.openstack.org/GerritWorkflow and follow the
instructions there to upload your change to Gerrit.


divide posts into semesters and a sidebar

divide posts into semesters and use a object to wrap around the semesters
e.g  {fall 2016:[post1, post2..], spring 2016:[ ], .....}
pitfall: the order of the semesters can't be sorted, because there are the properties of a object.

when the browser's width reduces to a certain amount (e.g a portrait ipad), the side would get under the posts. I have made use of grid.

3. is there a body tag? how can I edit the body tag? I need it for scrollspy.
bootstrap scrollspy  Angularjs might not have native support for bootstrap scrollspy?
what I have so far:

If you want to sort the dates you can get a list of the keys, sort that and then use that as a lookup to order the way you want, for now reverse chronological order seems fine though

The semesters/keys list  does not the "reverse chronological order". it's pure insertion order. It might never cause any problems, but I just think someone should know.

This fails
Error: [ngRepeat:dupes] Duplicates in a repeater are not allowed. Use 'track by' expression to specify unique keys. Repeater: sem in listOfsem, Duplicate key: string:Fall 2016, Duplicate value: Fall 2016


[Enhancement]: Search box, Previous and Next button effect

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
When hovering on the previous and next step button it shows a hover effect which is not that much visible and catchy. Also the size  of the both box is not same which is looking like a odd .
When someone leaves the search box the typed game gets disappeared so that's not giving a Good impact. So having my expertise in Frontend i would like to work on this enhancement and will give a very valuable result.
@kunjgit kindly issue me this task under gssoc24.

Describe the solution you'd like
Solution steps
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @rajvaidyanath27 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @rajvaidyanath27 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Refactor AccountID:

All AccountID functionality is removed from RippleAddress and
replaced with free functions. The AccountID to string conversion
cache is factored out as an explicit type with an instance in
the Application object. New base58 conversion functions are used,
with no dependence on OpenSSL.
All types and free functions related to AccountID are consolidated
into one header file, ripple/protocol/types.h which will hold all
the types and functions that deal with Ripple protocol "tokens".
A token one of the cryptographic primitives used in Ripple:
Secret Seed
Server Public Key
Server Secret Key
Account ID
Account Public Key
Account Private Key

and these deprecated primitives:
Account Family Seed
Account Family Generator

Second to last commit fails npm tests:
"[FOLD] Consolidate AccountID in types.h"

So what's wrong with that second-to-last commit?  It looks fine!

Yeah....good question. I'm going to delete the last 2 and retry.

(I verified your issue btw...)

I rewrote the first commit to reflect Tom's feedback

Pushed a fix for secure_erase and rewrote the branch

:+1:


Cosmos - update readme, partition key on delete.

I've found it problematic to make delete item code work due to the fact that I've misinterpreted the value of the partition code in the example code. I've added a small note for this operation on how to fill the partition key parameter. Should help for people with the same problem in the future and save some of their time.

Can one of the admins verify this patch?


Have a way to share mock data between multiple requests

Use case
A clear and concise description of what the problem is.
I'm trying to figure out a robust way to share mock data between multiple requests. A lot of our requests query different things about the currently logged in user. So ideally we would be able to do something like this:
val user = Builder(__CustomScalarAdapters).run {
    buildUser {
        contactMobile = "+2217730634835"
        // etc, lots of setup
    }
}

addMockResponse(LoginMutation, LoginMutation.Data(fakeResolver) { user = user })
addMockResponse(PermissionsQuery, PermissionsQuery.Data(fakeResolver) { user = user })
addMockResponse(AnnouncementsQuery, AnnouncementsQuery.Data(fakeResolver) { user = user })
// ^ not actually this simple, but this gives the idea

But this code has a problem: anything generated by the fake resolvers will be different for the different responses even though it's supposed to be the same user.
If we specified every field in the builder then there would be nothing generated by the fake resolver so this would work.
Describe the solution you'd like
We came up with two ideas for solutions:
Seed arguments
Builders have an optional seed argument which is passed to the fake resolver, fake resolvers should generate the same value given the same seed
e.g.
val user = Builder(__CustomScalarAdapters).run {
    buildUser(seed = "42") {
        contactMobile = "+2217730634835"
        // etc, lots of setup
    }
}

This has the advantage that it fits quite well into the current API. But the disadvantage is that if you want to use fixed seeds you'd need to be careful when writing fake resolvers.
Run fake resolvers at data object build time
Currently fake resolvers run when generating the response, but we could also/instead run them at the end of e.g. buildUser and fill the generated objects into the map which buildUser outputs.
e.g.
val user = Builder(__CustomScalarAdapters, fakeResolver).run {
    buildUser {
        contactMobile = "+2217730634835"
        // etc, lots of setup
    }
}

The advantage of this is that it's simpler: the fake generated values are simply stored alongside the other values. Unfortunately it would require a breaking change to the current fake resolvers API because at this point we don't know what the path will be (and the path is currently provided to the fake resolver when it runs).
(Note: I don't fully understand this part, this is from discussion in slack with Martin) Another potential problem is around aliases, consider:
{
  viewer {
    personalPhone: phone(PERSONAL)
    professionalPhone: phone(PROFESSIONAL)
  }
}

Here we don't know about the alises personalPhone and professionalPhone at the time when we run buildUser (we only know that there is a phone field), so we can't set fake values for them there.

Turns out both our options would not work 🙃
Seed arguments do not work because depending the queries, the selections might be different and the number of calls to resolveLeafFields would differ. For an example:
val user = Builder(__CustomScalarAdapters).run {
    buildUser(seed = "42") {
        contactMobile = "+2217730634835"
    }
}

# query1
{
  user {
    field1
    contactMobile
    field3
  }
}

# query2
{
  user {
    # not querying field1
    contactMobile
    # because a call to resolveLeafField is missing, field3 will actually have the value of field1
    field3
  }
}

Run fake resolvers at data object build time does not work because objects can have cycles. For an example:
type User {
  contactMobile: String!
  # Oopsie, if we try to fake this one, we're looping forever. Only actual queries break that cycle
  friend: User
}

I made another attempt there: https://github.com/apollographql/apollo-kotlin/pull/4468
It exposes cacheKey in FakeContextResolver and makes DefaultStateResolver stateless so that the values are derived from cacheKey (mainly uses hashCode() at the moment). It works with the @typePolicy directive so that if you specify id, all the child fields of this object will have deterministic values:
val user = Builder(__CustomScalarAdapters).run {
    buildUser {
        // specifying the id makes sure that all sibling fields will have deterministic values
        id = "42"
    }
}

Writing this, I realize that not everyone will use @typePolicy so we'd need another API to do things more pragmatically. I'll revisit shortly but wanted to get your feedback. Let me know how that sounds to you.

Hi, thanks so much for thinking about and working on this! :pray:
Yeah I think using things derived from the cacheKey sounds like a good approach: since the cacheKey is already used to define "object identity" it seems sensible to reuse it here for the same thing.
I can't think of any kind of data that couldn't be generated deterministically from a hash of the cache key. Numbers are obviously easy, but anything can be done by having an array of possible values and then doing something like values[hash(cacheKey) % values.length()]. Might be good to document or even provide a default implementation which does this though if we expect most people to use that approach? It's a fairly subtle thing to figure out on your own.
It might be good to have an (opt-in) way to prevent it from generating new values where the dev expects to be the same, e.g. by enforcing that the cacheKey field(s) is/are explicitly set in every call to a builder. I think for Wave's version of this we can just make the fakeResolver throw if asked to generate an ID (because all of our cacheKeys are IDs and all/almost all of our IDs are cacheKeys). I'm not sure if this is solvable in more general cases though since the CacheKeyGenerator can run arbitrary code to create a key.
Re typePolicy: will code that uses CacheKeyGenerator work in the same way? We use a CacheKeyGenerator which returns the field named id.


Fix problem with shared volume + refact

When using a shared volume from host its better to touch the .mongodb_password_set file into the VOLUME directory so the marker file is saved on the host, and in case of reattaching it to another container the database wont be populated again.
Also done some refact:

no need the use the /usr/bin prefix with the mongod, since the binary is already placed into the $PATH upon installation
DRY-ing the code in the run.sh, by making the /data/db env var
method addUser is depricated in mongodb-2.6,  in favor of the new createUser

Thank you

You're welcome :)


Divide 1 controller into 2 controllers

Analyze if the current controller submitted_content_controller, can be divided into 2 different controllers:

submitted_files_controller
submitted_hyperlinks_controller

If yes refactor it, and also refactor the test file for the same

Dividing the controller into 2 different controllers isn't ideal in this case
More info on why is mentioned in the Wiki


Remove bam_to_bqsr merge inputs after the merge.

Once the merge step is complete. The inputs, per instrument data aligned BAMs, should no longer be necessary. We could remove those inputs at this point in the alignment workflow.
https://github.com/genome/analysis-workflows/blob/e509210450e4c62aecb99a228fc97f0eae2d9580/definitions/subworkflows/bam_to_bqsr.cwl#L58

Summary of all intermediate BAMs that can be removed:
After the merge, the align/tagged_bam inputs could be removed.
After the name_sort, the merge/merged_bam input could be removed.
After mark_duplicates_and_sort, the name_sort/name_sorted_bam can be removed.
After apply_bqsr, the mark_duplicates_and_sort/sorted_bam can be removed.

I can confirm that BAM files are no longer created many times for downstream steps. However, the alignment step itself, creating intermediate copies of the BAMs, is still using a lot of unnecessary space. For example:
792G    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc
30G     /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-bam_to_cram
21G     /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-detect_variants
30G     /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-index_cram
12G     /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-variant_callers
882G    total

Within the alignment step the breakdown of storage usage looks like:
256G    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-align
155G    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-apply_bqsr
1.6G    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-bqsr
75G     /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-index_bam
153G    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-mark_duplicates_and_sort
153G    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-merge
256K    /gscmnt/gc2648/wgs/model_data/aa00989be76243d6bc544ce891335da6/build6bf88831c26145849cca97635c3e967e/tmp/cromwell-executions/germline_wgs.cwl/adf00e15-8017-43a7-8455-0a8de0a48697/call-alignment_and_qc/wgs_alignment.cwl/d7086dba-3c19-434a-9884-7a9c55d5c809/call-alignment/bam_to_bqsr.cwl/265110be-6c72-4b24-bd86-d218d525fdc6/call-name_sort
791G    total

Are there thoughts on whether this is possible within the CWL workflow? Does it have to do something janky like call a script that traverses the workflow tree and finds the right bams to remove, or is there some CWL function that allows for "cleanup"?

I'm sure there's some way to do it, but it would be outside the intended bounds of CWL.  How to do it would be backend and implementation dependent--that is, different for cwltool than for Cromwell and different depending on your environment (local filesystem, GCP, etc.).  This would be better implemented as a feature of the workflow runner.  (The GMS cleans up intermediate files at the end, for instance, and Cromwell has an experimental delete_intermediate_output_files option for GCP.  But both operate at the end of workflows--there's an assumption you have enough working space to hold all the intermediate files.)
For the case of these alignment files, moving to a speedseq-ish giant single step that does the alignment would reduce the duplication.

okay, given that limitation, I'll suggest that we close this issue.  Thoughts on whether we should create a new issue that references this and raises the possibility of a speedseq-like integrated alignment step (outside the bounds of our 2.0 milestone)?


Remove unmanaged containers, don't stop them

The biggest problem with stopping unmanaged containers is that they can
still impact scheduling decisions.  For example anti-affinity to a
container label.

https://github.com/rancher/rancher/issues/8364


Grease Pencil to take notes in the editor

Hey guys...I was thinking (inb4 here we go again) what if Godot had a Grease Pencil to write some notes in the scene. I really like to use it in Blender to take some notes about a specific thing.
and I think it would be awesome to have this in Godot to sketch some game bahaviours and systems, or note comment something someone did (good or bad).
It could be a node that only renders on editor. Is it viable?

Is there another way to take notes in Blender?
I ask, because if it's the only way, then it's sure useful to be able to take notes at all in Blender.
As of Blender, I am a noob, but from my experience I noticed, that most of the time notes should be rather taken in code - something that mostly does not exist in Blender, unless you are creating python scripts, I guess, right? In Godot 2d and 3d views are just used to arrange visual elements in the scene.
When I really needed to remember specific thing in the visual scene editor in Godot, I just used label node, that I was able to see and delete whenever I wanted.
One could even create a special parent for notes taking and put them all there, easy to find and delete.
But if someone will be willing to create grease pencil feature, probably some people will use it.

Yeah, but how about the overall design process? I mean, drawing gives much more freedom to artists take notes, highlight something, save a scene with notes of how an animator could animate the sprites. Or even game designers could take notes about a mechanic, so that a programmer opens the scene and implement the changes, or try to design the UI, et cetera.
Notes in code is just to comment the code, how it works and stuff, artists are not intented to even open a script, scripting and coding is aimed to coders/programmers. With this kind of feature the team communication can have a much higher level, easing the rework time and making the workflow more fun by improving the usability and design process.

I don't know -- in all the time I've used blender so far (for lots of things actually, but mostly player around with cycles), I never found a single use of grease pencil (and when I did, it was not the best too to use -- guess I was overthinking it at the time).

I think it could be interesting, but it likely doesn't need to be implemented in the engine directly. It could be an editor plugin distributed in the asset library.

Honestly it depends, I see that as a really powerful collaboration tool. By the day we integrate Git and all, it would be handy to have this built-in.

Well I also don't see much point in integrating "Git and all" out of the box when it can be done perfectly via plugins. What if you want to use Mercurial, or, more common in big video game studios, Perforce? Plugins is the way to go for all those.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


ConsensusLoop race condition

if (this.Tip.ChainWork > this.Chain.Tip.ChainWork)
{
    // This is a newly mined block.
    this.Chain.SetTip(this.Tip);
    this.Puller.SetLocation(this.Tip);

    this.logger.LogDebug("Block extends best chain tip to '{0}'.", this.Tip);
}

Here after the if is evaluated to true it is possible that a peer sends us a better chain and we switch to it and after that inner body of the if statement is executed.
Also a race condition is possible for line 400: this.Chain.SetTip(this.Tip); (ask @Aprogiena about it)

I think this code is only executed when the node is mining.
Whats the consequence of such a race condition?

Whats the consequence of such a race condition?

Yes, it is kind of a problem that maybe is not critical, because it is quite likely that in most cases the node will recover from it.
But it is immediate sign of wrong design, specifically concurrent chain. So it is just another reminder that concurrent chain should be redesigned. In the last PR that Ivan submitted, we tried to move all logic that changes the concurrent chains tip into the new component. But it was not entirely possible because of these guys in consensus loop.
Unfortunately it is not possible to fix these properly without some update to concurrent chain, but better redesign it completely with sessions and blackjack.

@Aprogiena can we close this or not yet? Or is there any other issue we should link this to?

not yet, we have it on the list

Fixed with activation merge.


Methods naming :smiling_imp:

https://github.com/Dayver3/blog/blob/74d31977693388c9827764144fb0eca3440b9c62/src/application/controllers/CommentController_.php#L5
ex: actionIndex

done


Kam bootstrap --push-to-git flag

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
Adding e2e test for --push-to-git flag
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #127
How to test changes / Special notes to the reviewer:

/retest

/retest

/retest

/retest

In the CI build process OpenShift (or is it container runtime? not sure) puts / as a home to /etc/passwd for the user the container runs for. Test platform tries to compensate by setting up a writable location and exporting it as $HOME which is fine for most cases but openssh does not respect $HOME and uses the item in /etc/passwd.

In a conversation with platform team we guess a workaround might work but unfortunately it did not work
I am closing this pr for now. When we will get a proper working documentation steps, i will open up a fresh pr to implement --push-to-git=true flag in CI

/close


Unable to get transfers

I believe I've found a bug with the new default backends that prevents me from fetching transactions on the wallet if no explicit backend is provided.
AttributeError: 'NoneType' object has no attribute 'transfers_in'
On first glance it looks like an easy fix. Also, it looks like it was missed in tests due to the nature of the mock backend and transactions.
$ python3 -m venv .venv
$ .venv/bin/pip install monero==0.7.6
$ monero-wallet-rpc --rpc-login "test:xxxxxx" --rpc-bind-port 55013 ..........other cli args........
>>> from monero.wallet import Wallet
>>> w = Wallet(user='test', password='xxxxxx', port=55013)
>>> w.incoming()
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "......project path...../.venv/lib/python3.9/site-packages/monero/transaction.py", line 113, in __call__
    fetch = self.backend.transfers_in if self.direction == 'in' else self.backend.transfers_out
AttributeError: 'NoneType' object has no attribute 'transfers_in'

Merged. Thanks!

Merged. Thanks!


Ion Icons 5 update

Fixes:  #312
PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Demos changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #313
What is the new behavior?
Does this PR introduce a breaking change?
[x] Yes
[ ] No

Other information

Is this PR going to be reviewed and merged ?

@Tonio-lavanda Not sure, I think repo maintainer is not active. You can use my fork if you need the change.

@itsLucario how to use your fork ? i have tried : npm i itsLucario/ion2-calendar  but it doesnt work

+1 merge

+1 mege

I have published my fork to @lucario/ion5-calendar
You can use it with npm i @lucario/ion5-calendar


config command not working

Hi,
Just installed 1.3.7 from pip, and can't use config
[@Fredxxx]: config HIDE_PROMPT = true
Just can not set the key.

Nothing shown here (at first not json file, neither in my home dir, nor root's)
[@Fredxxx]: config
Detailed explanation can be found at http://rainbowstream.readthedocs.org/en/latest/#config-explanation

Thanks,
fred

same here, any updates?


Specify source of SolidQueue to ErrorReporter

Because SolidQueue uses the Rails executor, exceptions raised in a Job are automatically handled by ActiveSupport::ErrorReporter.
This is very convenient.
I want to make this even more convenient.
Proposal
Simply put, this is what I propose.
diff --git a/lib/solid_queue/app_executor.rb b/lib/solid_queue/app_executor.rb
index da0976f..0580213 100644
--- a/lib/solid_queue/app_executor.rb
+++ b/lib/solid_queue/app_executor.rb
@@ -4,7 +4,7 @@ module SolidQueue
   module AppExecutor
     def wrap_in_app_executor(&block)
       if SolidQueue.app_executor
-        SolidQueue.app_executor.wrap(&block)
+        SolidQueue.app_executor.wrap(source: "application.solid_queue", &block)
       else
         yield
       end

Currently, the default value application.active_support is applied for source.
This does not identify it as an exception raised inside SolidQueue.
This is a quote from the rails guide.

source: a String about the source of the error. The default source is "application". Errors reported by internal libraries may set other sources; the Redis cache library may use "redis_cache_store.active_support", for instance. Your subscriber can use the source to ignore errors you aren't interested in.

Again, examples of the use of source were explained.
https://github.com/rails/rails/pull/44999
What do you think?

Ohh, this is a great suggestion! I didn't know this, somehow missed it! 😳 But it makes complete sense. Would you like to open a PR with this change?

Thanks for the response.
I'll do it today👍


Initial support to enabled/disable each sensor input

To reduce power consumption, I add these methods to support enabling/disabling each sensor input

@andriyadi
Sounds like an interesting addition to the library. I will run some tests and if there are no problems we can include it into the next release.

Ok thanks.
I can see the current consumption reduction about 100 uA.


feat: add support for client side rules

Summary
Add support for customer to mutate a batch before it gets uploaded.  This is exposed via the API mParticle.config.onCreateBatch.  The batch is passed to it before uploaded so that the developer can mutate it.  If the developer sets onCreateBatch, we set batch.mutated = true.
Testing Plan
Add unit and integration tests.  Tested this manually via local app as well.
Master Issue
Closes https://go.mparticle.com/work/SQDSDKS-4036

:tada: This PR is included in version 2.16.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Block domain flare-network.ai

Domain: flare-network.ai
Requested by @usser0134

.

@usser0134 Thank you. We have been very busy with other more recent scams. Your help is great.


refresh_from_db doesn't update the initial state

Hi!
When calling refresh_from_db() on a model instance, the values are reloaded from the database. However, the _initial_state stored on a LifeCycleModel instance is not recalculated. If the maintainers consider this a bug, I can provide a fix by overriding the refresh_from_db method in the library Mixin.

That makes sense to me


[Bug Report] When trigger a mouseEnter event on an empty table cell, it consoles err

Element UI version
2.3.8
OS/Browsers version
Win10 Chrome 66.0.3359.139 x86
Vue version
2.5.17-beta.0
Reproduction Link
https://jsfiddle.net/coolucifer/xdcbhtxt/
Steps to reproduce
1.create a table with empty table cells
2.press 'F12' to open console
3.move mouse into an empty cell
4.it consoles an error
What is Expected?
no error happens
What is actually happening?
error occurs

Yes, it's fixed in #11137. Please wait for our next release. Or downgrade Element to 2.3.7 for now.


Fixed ArgumentNullException from GetLockoutEndDateAsync

We encountered an issue when logging in. We were using the SPA template in Visual Studio 2013 and followed your directions.
Basically there is no lockout end date set so when it goes to parse it and the raw value is null or empty, it throws an ArgumentNullException.  We think the correct behavior is to check if it exists first.  This fixed our issue.

Sorry for answering to this pull request 29 days ago...
And thank you for advance for the fix. Let me review it and I'll merge it as soon as possible!

I've already review the code.
Actually I believe I found the same issue some time ago and I've fixed it using the TryParse approach instead of string.isNullOrWhitespace.
Let me know if I'm not mistaken and my fix already solved the same issue:
    public virtual async Task<DateTimeOffset> GetLockoutEndDateAsync(TUser user)
    {
        string rawLockoutEndValue = await Database.HashGetAsync(UserLockDateHashKey, ((IUser)user).Id);
        long lockoutEndValue;

        if(long.TryParse(rawLockoutEndValue, out lockoutEndValue))
        {
            return DateTimeOffset.FromFileTime(lockoutEndValue);
        }
        else
        {
            return DateTimeOffset.MinValue;
        }
    }

I think the problem might have been the value being null, which was causing an exception in TryParse.
Cody Rigney
Software Engineer
PH:<PHONE_NUMBER> x109
FX:<PHONE_NUMBER>
www.sensourceinc.comhttp://www.sensourceinc.com/
<EMAIL_ADDRESS>Better Decisions Start Here
3890 Oakwood Ave. Youngstown, OH 44515
NOTE: The information contained in this message is intended for the recipient/s only. It is privileged and confidential and should be protected from disclosure. If the reader of this message is not the intended recipient, or an employee or agent responsible for delivering this message to the intended recipient, you are hereby notified that any dissemination, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify us immediately by replying to the message and deleting it from your computer. Thank You.
From: Matías Fidemraizer<EMAIL_ADDRESS>Sent: Thursday, May 21, 2015 9:08 AM
To: mfidemraizer/stackredis.aspnet.identity
Cc: Cody Rigney
Subject: Re: [stackredis.aspnet.identity] Fixed ArgumentNullException from GetLockoutEndDateAsync (#1)
I've already review the code.
Actually I believe I found the same issue some time ago and I've fixed it using the TryParse approach instead of string.isNullOrWhitespace.
Let me know if I'm not mistaken and my fix already solved the same issue:
public virtual async Task<DateTimeOffset> GetLockoutEndDateAsync(TUser user)

{

    string rawLockoutEndValue = await Database.HashGetAsync(UserLockDateHashKey, ((IUser)user).Id);

    long lockoutEndValue;

    if(long.TryParse(rawLockoutEndValue, out lockoutEndValue))

    {

        return DateTimeOffset.FromFileTime(lockoutEndValue);

    }

    else

    {

        return DateTimeOffset.MinValue;

    }

}

—
Reply to this email directly or view it on GitHubhttps://github.com/mfidemraizer/stackredis.aspnet.identity/pull/1#issuecomment-104269985.

@codyrigney92  In previous commits, I believe the code was using Parse instead of TryParse, which caused the exception.
Check this DotNetFiddle: https://dotnetfiddle.net/jOus2Q
AFAIK, if you provide null to TryParse it won't throw an exception but return false.
Anyway I'm glad you're using this project and I enjoyed a lot that someone provided me a fix! Sadly I can't merge this pull request because the issue is already fixed, but since I'm going to check notifications from now, I hope any other pull request that provides improvements or fixes will be merged, because I published this project to improve it together!

Ah, I've edited your comment to remove email signature and other sensitive info from you to preserve your privacy ;)

Thanks for that!  I can't seem to remember the exact problem.  Maybe we had an old version of the code.  Thanks anyway!


Don't wrap with React.createElement unless needed

https://github.com/rackt/react-router/blob/master/modules/createRouter.js#L513
Currently causes react to emit a warning if you give a factory or existing ReactElement class as a handler in react-router. Very confusing for people who don't use JSX and are told by the React docs to ALWAYS wrap your components with createFactory if you want to use them without JSX, only to find out that react-router is doing wrapping you can't opt out of.
Warning: This JSX uses a plain function. Only React components are valid in React's JSX transform.

Only problem is I don't think we can detect if you give us a ReactElement or a factory. That's why we don't tell you anywhere in our docs to give us a factory.
Maybe we could fix it with a FAQ?

@mjackson What about an option for disabling the wrapping? If not I will probably have to fork and maintain a separate repo that doesn't do this. Dealing with a situation where I have to support the result of createFactory

Also causing warnings to be logged when using RouteHandler https://github.com/rackt/react-router/blob/master/modules/RouteHandlerMixin.js#L43

What about an option for disabling the wrapping?

The only way we could unwrap your factory to get at the underlying component class would be to access the deprecated type property on the factory. The type property is also exposed on component classes, but it's deprecated there as well.
Why can't you give us component classes instead of factories? Are you using a lib that only generates factories?

I guess we could have a <Route handlerFactory> prop. Then you could be explicit about the fact that you're giving us a factory, not a component class.

@mjackson Yep - using a lib that generates factories. A handlerFactory param seems like a good solution

people who don't use JSX and are told by the React docs to ALWAYS wrap your components with createFactory if you want to use them without JSX

Looking at this doc and related ones, they always tell you to do the wrapping inside the module that consumes the component (after import). So if you have a module or library that gives you a factory, it seems an anti-pattern to me and a step backwards.
Anyway, I'm not opposed to handlerFactory as a band-aid solution. But without factory.type there's no way to support transition hooks, or is there?

That lib should not generate factories, I don't want to add API to be defensive for people using anti-patterns defined by React itself.
You could make your own version of RouteHandler though. Or even monkey patch render in your app.

@rpflorence Is it a blessed pattern to wrap components into elements when components are given as properties instead of letting the user control it?

I think users should always get a component, not a factory, no matter where.


Variable Explor

Description
What steps will reproduce the problem?

spyder 3.3.1 - variable explorer not refreshing...
Versions

Spyder version: 3.3.1
Python version: 3.7.0
Qt version: 5.9.6
PyQt5 version: 5.9.2
Operating System: Windows 10

Dependencies
pyflakes >=0.6.0  :  2.0.0 (OK)
pycodestyle >=2.3 :  2.4.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
sphinx >=0.6.6    :  1.7.9 (OK)
rope >=0.9.4      :  0.11.0 (OK)
jedi >=0.9.0      :  0.12.1 (OK)
nbconvert >=4.0   :  5.4.0 (OK)
pandas >=0.13.1   :  0.23.4 (OK)
numpy >=1.7       :  1.15.1 (OK)
sympy >=0.7.3     :  1.1.1 (OK)
cython >=0.21     :  0.28.5 (OK)
qtconsole >=4.2.0 :  4.4.1 (OK)
IPython >=4.0     :  6.5.0 (OK)
matplotlib >=2.0.0:  2.2.3 (OK)
pylint >=0.25     :  2.1.1 (OK)

Please post a screenshot of what you're seeing because your description is not enough.

Here is the screenshot
[image: image.png]
On Wed, Oct 24, 2018 at 4:49 PM Carlos Cordoba<EMAIL_ADDRESS>wrote:

Please post a screenshot of what you're seeing because your description is
not enough.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/8141#issuecomment-432822612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiTmWAZC3_P-pEgAqqSgP6scGOMXwWEFks5uoNJDgaJpZM4X4m00
.

Please upload it directly through Github's web interface. You can't send it via email, sorry.

Here is the screen shot ..

Please close Spyder, open an Anaconda prompt, run there
spyder --reset

and try again. That should fix your problem.

Still the same ...   Still I don't the dataframe df in the Variable Explorer..

I don't know what's happening, sorry. Please try to modify the options in the gear menu of the Variable Explorer:


There is no obvious command to upgrade or downgrade a program to an specific version

How would this feature be useful?
Sometimes one wants to use a program in a specific version or version range, instead of the latest one. For instance, when the upgrade would cause breaking changes or the most recent version has some bug that affects one's work.
Describe the solution you'd like
pipx upgrade command should have a clear way to tell which version to upgrade to. To me, the most intuitive option is to mimic the pipx install command by accepting package_spec (package name or pip installation spec) instead of just the package name as it currently does.
Example: pipx upgrade "poetry<1.1" to upgrade or downgrade Poetry to its latest 1.0.x version.
Describe alternatives you've considered
The only way I found that I could downgrade a package was by uninstalling it and then installing it again with the spec I wanted.
If there is already a way to do it, it is not obvious and it is not exemplified in the documentation.

Thanks for the vote for this, it's probably something we need to do.
I'm closing this because it seems to be a duplicate of #356


Disposing Vlc object hangs Application

From more than one week i am scratching my head as vlc object doesn't dispose even after Unregistering all Events and calling its Dispose() method.
Note: I am using WindowsFormHost to Host Vlc object in a WPF Application.
what i tried so far:

Adding this.Dispose method in Another Task with Thread.Sleep of 100 ms.
calling this.pause before disposing
and more.
But none of this worked for me.
Can you please solve this issue or atleast provide me a workaround with it.
ill be so thankful to you.

Could you try with latest version from NuGet and provide more info like:

The version of libvlc you are using
A sample code to reproduce (at what time do you call Dispose? Is it on the same thread that created libvlc?)

Hi jeremy i asked the same question on stackoverflow you can find the link below:
here
i dont know about LibVlc but
Vlc.DotNet.Forms version: 2<IP_ADDRESS>

Closing this since an answer has been provided on stackoverflow. Feel free to reopen if needed.


Fix typos in code comments

Description
Recently, I found out that there were 2 typos remaining in this project.

Here, my should be may.

https://github.com/TraceMachina/nativelink/blob/d37bd90a314890fe901235e0432d263faa66d221/deployment-examples/docker-compose/Dockerfile#L21

Here, the code comments for simple_cache_test.sh and simple_prometheus_test.sh are the same currently.
For simple_prometheus_test.sh, the comment should be changed.

https://github.com/TraceMachina/nativelink/blob/d37bd90a314890fe901235e0432d263faa66d221/integration_tests/simple_cache_test.sh#L16
https://github.com/TraceMachina/nativelink/blob/d37bd90a314890fe901235e0432d263faa66d221/integration_tests/simple_prometheus_test.sh#L16
Fixes #1189
Type of change
Please delete options that aren't relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] This change requires a documentation update

Checklist

[x] Updated documentation if needed
[x] bazel test //...  passes locally
[x] PR is contained in a single commit, using git amend see some docs

This change is

Just caught simple issues. cc: @allada, @MarcusSorealheis
Waiting for reviews on my previous submitted issues and wanna work on more issues. :)

Rebased. cc: @aaronmondal, @MarcusSorealheis

Hope you to merge this. cc: @allada, @MarcusSorealheis, @aaronmondal


feat: add hertz code generator

What type of PR is this?

feat: Add hertz code generator
What this PR does / why we need it (English/Chinese):

en: add hertz code generator
zh: 添加 hertz 代码生成器
Which issue(s) this PR fixes:

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


Use new binary build recipe in E2E cleanup make target

Missed a refactor spot in #4732.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Merging to get the hourly periodic passing.


octopus should handle data with nans

One of the eight arms of octopus should handle nans.
For example, this bit of code below does not play well when data have nans. Excluding the nans with something like data[data == data] doesn't  do the trick because mean is defined everywhere, hence data.shape and mean.shape differ, which leads to runtime errors.
class PoissonLikelihood(Likelihood):
    def evaluate(self, params):
        return (self.mean(*params) - self.data * np.log(self.mean(*params))).sum()

cc @barentsen @gully

fixed in 1b7ec6d


Masternode sync improvements

Masternode sync improvements:

add simple helpers for few more sync states (use them where appropriate instead of old code + rpc output)
use new helpers to avoid meaningless message processing
actually fail if sync shouldn't continue due to lack of info, make sure Reset is used to quit failed state

Have these changes been tested on testnet ?
I'd feel better about this PR knowing that the changes have been tested and don't obviously break syncing.

Works both for testnet and mainnet for me at least for a couple of days, testers are welcome :)

Let's hit testnet - utACK https://github.com/dashpay/dash/pull/969/commits/da406e3d23d4e86d2a27d4c9d3e5f818ef0df15f


Block Cover spams nei

I noticed the block cover that looks like stone spans multiple pages across nei

ya that is normal, they are basically like facades in buildcraft


HA 0.95.4

Hey guys
Just having issues after updating HA to 0.95.4
HA seems to be having a cry about country being in the climate configuration
climate:
- platform: smartthinq
refresh_token: YOUR_TOKEN_HERE
country: AU
language: en-AU
Has anyone had the same problem or has a solution
Thankyou

What is the error, exactly?

Hey mate thanks for all your hard work this component is better than the app.
So i downgraded back to 0.94.4 because smarthinq stoped working
The error Im getting before the update on 0.94.4 is
2019-06-30 10:37:22 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for smartthinq which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-06-30 10:37:22 WARNING (MainThread) [homeassistant.helpers.config_validation] Your configuration contains extra keys that the platform does not support.
Please remove [country], [language].  (See /config/configuration.yaml, line 312).
Offending data: {"platform": "smartthinq", "refresh_token": "", "country": "AU", "language": "en-AU"}
Your configuration contains extra keys that the platform does not support (but were silently accepted before 0.88). Please find and remove the following.This will become a breaking change.
[country], [language]. (See /config/configuration.yaml, line 312).
Offending data: {"platform": "smartthinq", "refresh_token": "", "country": "AU", "language": "en-AU"}
Then when i Upgrade to 0.95.4 the Component stops working. I will upgrade again then post log. Im on hassio.
Thankyou

I get this error on 0.95.4 and entity not available
2019-07-01 08:45:31 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for smartthinq which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-07-01 08:45:31 ERROR (MainThread) [homeassistant.config] Invalid config for [climate.smartthinq]: [country] is an invalid option for [climate.smartthinq]. Check: climate.smartthinq->country. (See /config/configuration.yaml, line 312). Please check the docs at https://home-assistant.io/components/climate.smartthinq/

Thanks! That's just a warning, so you can safely ignore it. But I've just pushed a fix that should silence the warning even so.
But that doesn't solve your problem, sorry! Maybe there is some other problem going on too.

Thanks mate
That fixed the warning ended up deleting the python cache and now its working again.
Sent from my iPhone

On 1 Jul 2019, at 12:41 pm, Adrian Sampson<EMAIL_ADDRESS>wrote:
Thanks! That's just a warning, so you can safely ignore it. But I've just pushed a fix that should silence the warning even so.
But that doesn't solve your problem, sorry! Maybe there is some other problem going on too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Feature/99 style property and mlt fields

Q
A

Branch
master

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
yes

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#99

License
MIT

This branch is based on: https://github.com/chameleon-system/chameleon-base/pull/168

Thanks @pixeljunkie


Prepare for 1.1.2 version

cc @mlorbetske @Eilon

Closing this and recreating against the right branch.


only use crypt related vars within FEAT_CRYPT

Signed-off-by: BlackEagle<EMAIL_ADDRESS>

Not quite right, the #ifdef comes too early.  Anyway, just sent out a patch that does something similar.


deprecation warning in confusion_matrix.py

This template is for miscellaneous issues not covered by the other issue categories.
For questions on how to work with TensorFlow, or support for problems that are not verified bugs in TensorFlow, please go to StackOverflow.
If you are reporting a vulnerability, please use the dedicated reporting process.
For high-level discussions about TensorFlow, please post to<EMAIL_ADDRESS>for questions about the development or internal workings of TensorFlow, or if you would like to know how to contribute to TensorFlow, please post to<EMAIL_ADDRESS>OS: Ubuntu 18.04.2
Tensorflow-gpu: 2.0.0alpha0
WARNING: Logging before flag parsing goes to stderr.
W0423 15:59:53.297839<PHONE_NUMBER>71264 deprecation.py:323] From /python3.6/site-packages/tensorflow/python/ops/confusion_matrix.py:194: to_int64 (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.cast instead.
W0423 15:59:53.298299<PHONE_NUMBER>71264 deprecation.py:323] From /python3.6/site-packages/tensorflow/python/ops/confusion_matrix.py:195: to_int32 (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.cast instead.

These have been fixed on master, next update of alpha/tf2.0 will have them fixed. Thank you for notifying us about them


📃: JS Intermediate Project Addition: Makeover Game

:red_circle: Title : Makeover Game
:red_circle: Aim :  To dress the character as per your wish.
:red_circle: Tech Stack Used :  JS
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Khushi Kalra
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.) GSSOC'24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@abckhush no games project and don't raise more than 1 issue


MDMang produces incorrect demangling of hashed symbol names

I have a symbol file that contains two unusual symbols:
?catch$0@?0???@f4873c94f485cd6716c2319fc51ac714@@4HA
??@f4873c94f485cd6716c2319fc51ac714@

(from: http://msdl.microsoft.com/download/symbols/perf_pofx.pdb/86dc62bc8c6e50836f2f1414bcf3f8fe1/perf_pofx.pdb )
From which MDMang produces:
`int f4873c94f485cd6716c2319fc51ac714::ERROR UNKNOWN OPERATOR TYPE'::`1'::catch$0
f4873c94f485cd6716c2319fc51ac714::ERROR UNKNOWN OPERATOR TYPE

Conveniently, Clang can once again provide us a good explanation of what's happening! https://github.com/llvm/llvm-project/commit/febb27283d8209d31b80d4b575aa4381931fcb75 explains that when a mangled symbol name would be longer than 4096 characters, it is instead emitted as a ??@, @ wrapped MD5 hash.
I don't know what would be a good way to present them, but at the least dropping the ::ERROR UNKNOWN OPERATOR TYPE would be an improvement.

Thank you once again for providing details on MDMang issues.
The second symbol is a nested item in the first.  I'm considering putting the single quotes (backward and forward ticks) along with a special identifier as such (for the second symbol):
`HashedObjectf4873c94f485cd6716c2319fc51ac714'

Since the first symbol has a nested name (which happens to be the second symbol object), it also gets these single tick marks, and would look like this:
int ``HashedObjectf4873c94f485cd6716c2319fc51ac714''::`1'::catch$0

What do you think about these?
I supposed they would be subject to change once I get more time to take a bigger look these symbols and others (including the other unfinished MDMang issue that still exists) that could make use of insights from LLVM Clang.
Another component to processing these symbols is in determining how Ghidra should apply them to a program.  For this, a PDB with symbols is not necessarily enough to follow the possibly processing paths.  This happens in MDMangGhidra, which currently assumes there is an MDQualifiedBasicName, which will no longer be the case.  I have an approach where I'm modifying MDObjectCPP, adding new methods such that those assumptions do not need to be true.  I'm not extremely happy with how I've approached it, but think I have something that will hopefully work without breaking things.  I'll have to see what additional testing I can do to prevent unforeseen breakage.

I'm not a fan of the "Object" part of that - If I came across that, I would assume that "HashedObject" is referring to the actual data at that address rather than the label itself. "HashedSymbol" or "HashedLabel" might be better, but I would favor simply giving less to [mis]interpret - such as `Hashed_f4873c94f485cd6716c2319fc51ac714' or even just `f4873c94f485cd6716c2319fc51ac714'

I understand.  I had misgivings of saying Object, since that is such an overloaded term... I only put it because that is how I have these symbols depicted internally at the point where the modifications would need to take place (these objects have names and types, etc. in their hierarchy).
Of the suggestions you have given, I'm working on using the latter.


MP Talk @ Code One 2019

Conference Name:
Oracle CodeOne 2019
Conference website and social media links:
https://www.oracle.com/code-one/
@OracleCodeOne
Talk name:
Take advantage of MicroProfile benefits on your monolithic application.
Talk Link:
https://events.rainfocus.com/widget/oracle/oow19/catalogcodeone19?search=DEV1151
Speaker Names:
Victor Orozco, CTO, Nabenik (@tuxtor)
Jorge Cajas, CTO, inin.global (@cajasmota)
Tweet Message:
Come to learn with @tuxtor and @cajasmota how to take advantage of Eclipse Microprofile benefits on your monolithic applications without refactoring them to microservices.
Thursday, September 19, 11:15 AM

@Motojo scheduled for 2 days to go out on twitter


Create helm cluster failed because of secrets already exists

2018-09-28 06:49:20.78698 -ERROR- Failed to handle subtask in runtime [kubernetes]: rpc error: code = Unknown desc = release wxh-alex-wordpress-k8s-test failed: secrets "wxh-alex-wordpress-k8s-t" already exists (controller.go:425)(t-Lo4DLRj1REyo|j-M2NvDq81qOZv)

Maybe related to https://github.com/helm/helm/issues/2947.
Try to upgrade helm to fix it.

I think this is fixed by https://github.com/openpitrix/openpitrix/pull/694.


Use build script to build custom resources in build-custom-resources.ts

Description of changes
instead of invoking node_modules/.bin/tsc directly to build custom resources (via path.join(targetDir, 'node_modules', '.bin', 'tsc')), we should instead use the defined build script in the resource’s package.json. this script is automatically created when a custom resource is added with amplify add custom.
using ${packageManager.executable} run build invokes the build script using the current package manager and works no matter what kind of package manager configuration is currently set up (e.g. with PnP). resolves #11889.
Issue #, if available
#11889
Description of how you validated changes
i verified that npm run build and yarn run build both work to build the cdk-stack.ts file into build/cdk-stack.js in the same way as executing the node_modules/.bin/tsc path works. i then used amplify-dev build and amplify-dev push locally in my repo where i first ran into #11851 and my custom resource built successfully and deployed with no errors.
Checklist

[x] PR description included
[x] yarn test passes

note about the tests that not all of the tests actually passed, but i believe all the failures are 100% unrelated to my changes. there were a few tests that failed with Exceeded timeout errors (e.g. should attempt setting up local instance of opensearch with custom configuration, which timed our at 90224 ms!), and the amplify-dynamodb-simulator:test command, which i believe failed because i don’t have the correct java setup (max retries hit for starting dynamodb emulator) and to be honest, i’d rather not deal with that. as a general note, running yarn test made my M1 MacBook pro unusable while they were running, causing the macOS “Your system has run out of application memory” dialog box to show up, despite me only having my terminal and safari open, so i’m not surprised that there were some timeouts. here was the final log result:
 >  NX   Ran target test for 28 projects and 40 task(s) they depend on (1h)
    ✔    64/68 succeeded [49 read from cache]

    ✖    4/68 targets failed, including the following:
         - nx run amplify-dynamodb-simulator:test
         - nx run @aws-amplify/amplify-opensearch-simulator:test
         - nx run @aws-amplify/amplify-appsync-simulator:test
         - nx run @aws-amplify/cli-internal:test

the test that failed the @aws-amplify/cli-internal:test command is src/__tests__/test-aborting.test.ts:
       Summary of all failing tests
        FAIL  src/__tests__/test-aborting.test.ts (56.958 s, 1206 MB heap size)
         ● test SIGINT with execute › case: run

           thrown: "Exceeded timeout of 5000 ms for a test.
           Use jest.setTimeout(newTimeout) to increase the timeout value, if this is a long-running test."

              6 |   });
              7 |
           >  8 |   it('case: run', async () => {
                |   ^
              9 |     const input = { argv: ['/usr/local/bin/node', '/usr/local/bin/amplify-dev', '-v'], options: { v: true } };
             10 |     const mockExit = jest.fn();
             11 |

             at src/__tests__/test-aborting.test.ts:8:3
             at Object.<anonymous> (src/__tests__/test-aborting.test.ts:3:1)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #11854 (5b5ab24) into dev (ec9a2ba) will increase coverage by 47.59%.
The diff coverage is 45.83%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##              dev   #11854       +/-   ##
===========================================
+ Coverage    0.00%   47.59%   +47.59%
===========================================
  Files        1296      842      -454
  Lines      149743    38320   -111423
  Branches     1296     7834     +6538
===========================================
+ Hits            0    18239    +18239
+ Misses     148447    18458   -129989
- Partials     1296     1623      +327

Impacted Files
Coverage Δ

...ify-category-function/src/events/prePushHandler.ts
33.33% <0.00%> (+33.33%)
:arrow_up:

...ib/S3AndCloudFront/helpers/configure-CloudFront.js
87.06% <ø> (+87.06%)
:arrow_up:

...lify-category-hosting/lib/S3AndCloudFront/index.js
89.65% <ø> (+89.65%)
:arrow_up:

...s/amplify-cli-core/src/errors/amplify-exception.ts
82.35% <ø> (+82.35%)
:arrow_up:

...ackages/amplify-cli-core/src/help/commands-info.ts
100.00% <ø> (+100.00%)
:arrow_up:

packages/amplify-cli-core/src/types.ts
100.00% <ø> (+100.00%)
:arrow_up:

packages/amplify-cli/src/commands/console.ts
0.00% <0.00%> (ø)

packages/amplify-cli/src/commands/pull.ts
0.00% <0.00%> (ø)

...es/amplify-provider-awscloudformation/src/index.ts
60.20% <ø> (+60.20%)
:arrow_up:

...c/extensions/amplify-helpers/auth-notifications.ts
41.00% <4.34%> (+41.00%)
:arrow_up:

... and 24 more

... and 1271 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Test cleanup: simplify prepareProject usage

Now takes String instead of File.
Discussed at https://github.com/azkaban/azkaban/pull/1151#discussion_r120014278.

This appears to be another random failure
I am concerned that there may be more.
azkaban.execapp.FlowRunnerTest2 > testCancel FAILED
java.lang.NullPointerException
at azkaban.executor.ExecutableFlowBase.isFlowFinished(ExecutableFlowBase.java:398)
at azkaban.execapp.FlowRunnerTestBase.lambda$assertThreadShutDown$0(FlowRunnerTestBase.java:32)
at azkaban.execapp.FlowRunnerTestBase$$Lambda$9/1157486615.apply(Unknown Source)
at azkaban.execapp.FlowRunnerTestBase.waitFlowRunner(FlowRunnerTestBase.java:42)
at azkaban.execapp.FlowRunnerTestBase.assertThreadShutDown(FlowRunnerTestBase.java:31)
at azkaban.execapp.FlowRunnerTest2.testCancel(FlowRunnerTest2.java:729)

Valid concern. I'll try to replicate the random failures by running the test multiple times in a row.

Could you take a look at #1156 so that we can stabalize the master branch?

Looks like that error came from a synchronization issue, the way how ExecutableFlowBase#isFlowFinished works. I'm going to fix this by not calling that method but checking the flow finishing in another way.

Thanks. While you are investigating, could you approve #1156 so that the master is stable?

Fix for that test failure is here: https://github.com/azkaban/azkaban/pull/1158

I thought more about this change.
I am not convinced that on balance this change is better.
I understand that the caller of this method has to do less work.
The benefit, however, is that it is more explicit to the readers what the contract is. i.e. this string is a path to a file.

Yeah, I was thinking the same thing. Closing.

Thanks. I appreciate that you have an open mind.
I hope my objection didn't cost you too much time.


Fix Emoji Copy (# 📋 pb4wsl  --> # ≡ƒôï pb4wsl)

cat README.md | pbcopy

will produce the issue when paste(ctrl+v) to vscode if it contains an emoji.
instead if paste with
pbpaste > output.txt

everything works just fine.

change engine to xclip https://www.reddit.com/r/wsl2/comments/12icvgu/comment/jg7tn42/?utm_source=share&utm_medium=web2x&context=3


"Radomirești", "Călinești", "Crăciunei", "Poiana" are listed in the drop-down menu

Description:
When user writes "Radomirești", "Călinești", "Crăciunei" and "Poiana" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Radomirești"

Expected results
The location appears in the drop-down menu as "Sat Radomirești, OLT (Radomirești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Călinești"

Expected results
The location appears in the drop-down menu as "Sat Călinești, OLT (Radomirești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Crăciunei "

Expected results
The location appears in the drop-down menu as "Sat Crăciunei, OLT (Radomirești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Poiana "

Expected results
The location appears in the drop-down menu as "Sat Poiana , OLT (Radomirești)".


Migrate hard-coded JSON data into JSON assets files

JSON minification is no longer necessary since assets files are already gzipped during APK/AAB packaging.
Fixes #391

This solution tries to modify as less code as possible.
It requires enabling isIncludeAndroidResources instead of overriding sourceSets between Android and JVM.

@SimonMarquis Thank you for your contribution Simon! Great idea, I was meaning to do this myself. Could you fix the failing test please? Thanks again!

Thank you for your contribution Simon!


Upgrade jenkins version to latest LTS version

Is your feature request related to a problem? Please describe
We are currently deploying jenkins version jenkins-2.263.4
However, the latest LTS has many security fixes including the plugins. Jenkins is smart enough to install latest compatible plugin and hence we can also remove hard coded version of plugins.
Since the configuration is loaded from jenkins.yaml, the plugin specific version should not matter.
Describe the solution you'd like
Upgrade to latest LTS version and remove plugin hard coded versions.
Describe alternatives you've considered
No response
Additional context
The upgrade is not a direct change as jenkins has changed the way it installs on CentOS.
A quick check concluded that changes like https://github.com/opensearch-project/opensearch-ci/blob/main/lib/compute/jenkins-main-node.ts#L213 does not take effect on jenkins.service anymore.
Needs deep dive into configuration.

We need to disable set up wizard for jenkins. This is done by passing java opt of -Djenkins.install.runSetupWizard=false.
When we try to upgrade version(2.332.1), the jenkins.service process loads differently than current jenkins version(2.263.4).
Current version:
● jenkins.service - LSB: Jenkins Automation Server
   Loaded: loaded (/etc/rc.d/init.d/jenkins; bad; vendor preset: disabled)
   Active: active (running) since Thu 2022-03-24 19:22:49 UTC; 6min ago
     Docs: man:systemd-sysv-generator(8)
  Process: 4601 ExecStart=/etc/rc.d/init.d/jenkins start (code=exited, status=0/SUCCESS)
   CGroup: /system.slice/jenkins.service
           └─4665 /etc/alternatives/java -Djava.awt.headless=true -DJENKINS_HOME=/var/lib/jenkins -jar /usr/lib/jenkins/jenkins.war --logfile=/var/log/jenkins/jenkins.log --webroot=/var/cache/jenkins/...

Upgraded version:
● jenkins.service - Jenkins Continuous Integration Server
   Loaded: loaded (/usr/lib/systemd/system/jenkins.service; disabled; vendor preset: disabled)
   Active: active (running) since Thu 2022-03-24 02:04:06 UTC; 15h ago
 Main PID: 12037 (java)
   CGroup: /system.slice/jenkins.service
           └─12037 /usr/bin/java -Djava.awt.headless=true -jar /usr/share/java/jenkins.war --webroot=%C/jenkins/war --httpPort=8080.

We are using latest jenkins via docker now.


How?

Hi there!
Maybe I'm to burned out by everything, but I can't find for the love of me how to apply what is wrote within the README.md
I'm interested, more than everything, on HD subtitles (something that I truly envy to HD Remake).
Just to be sure and now overlook on anything: there's something concrete I can try or this is a POC?
If former, can I have some little guide?

POC, still need a way to allow arbitrary sized textures. Thinking of taking a look at the SH2 enhanced edition patch to see how it was done there, but I've never messed with texture injection, so it is probably a long way out.


Format docs for import

Codecov Report
Patch and project coverage have no change.

Comparison is base (4d0a83e) 95.37% compared to head (402d3f1) 95.37%.

:exclamation: Current head 402d3f1 differs from pull request most recent head ecc6779. Consider uploading reports for the commit ecc6779 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #48   +/-   ##
=======================================
  Coverage   95.37%   95.37%
=======================================
  Files           5        5
  Lines         303      303
  Branches       78       78
=======================================
  Hits          289      289
  Misses         12       12
  Partials        2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Remove PUT and DELETE methods from an OPTIONS request if readOnly is true

Currently DefaultServlet is returning all HTTP methods for an OPTIONS call even when the readOnly flag is true.
Example:
$ curl -v -X OPTIONS localhost:8080
....
< Allow: GET, HEAD, POST, PUT, DELETE, OPTIONS
....

It should be the following instead if the readOnly flag is true.
$ curl -v -X OPTIONS localhost:8080
....
< Allow: GET, HEAD, POST, OPTIONS
....

As a side note I don't understand the following.

Why is a POST allowed when readOnly is true?
Why does a POST call GET internally?

Patch looks good to me.
Regarding your questions:

Why is a POST allowed when readOnly is true?

Probably because the DefaultServlet just delegates POST -> GET, but ...

Why does a POST call GET internally?

Good question. I suspect because that's what httpd does when making a POST request to a static resource.
That code has not changed since 2006 (the initial import) so at this point, it's "legacy" and "backward-compatible" and extremely unlikely to change.

The current OPTIONS behaviour is intentional.
OPTIONS lists the valid methods, not the permitted methods, for a resource.
The GET/POST issue is before my time with the project. There might be some explanation in the mail archive. I agree with Chris it is unlikely to change.

Thanks for the quick responses.
I modeled this change after the changes for TRACE in the next if statement below. Also looking at issue this change seemed to be similar to that one.
It would make more sense to me if a user could actually change the request some how to invoke PUT or DELETE (by authenticating for example), but in this case there is nothing the user could do to ever get these 2 methods to work without changing the readOnly flag to false.
I am not adamant that this change should be made. I am just trying to understand what appears to be an inconsistency to me.

The inconsistency stems from the different status codes used. If TRACE is disabled at the connector, level then a 405 is returned for TRACE requests. If readOnly is set in the DefaultServlet, a 403 is returned for PUT and DELETE. Methods that return a 405 should not be included in an OPTIONS response.
Doing some mailing list archaeology, the readOnly parameter started out in the WebDAV servlet ( http://markmail.org/message/vls7an5nhf7eigwc ) and was refactored to the Default Servlet shortly afterwards ( http://markmail.org/message/vls7an5nhf7eigwc ). All this was back in 2000.
Looking at RFC 2616 (since this code pre-dates RFC7231) there isn't much to choose between 403 and 405. Both look equally applicable. One obvious difference for this use case is that a 405 MUST include an Allow header so on that basis using 403 looks to be the simpler implementation.
RFC 7231 adds some detail on the differences between 403 and 405. In short 405 means 'method never allowed' whereas 403 means 'request not allowed but might be allowed with different credentials'.
In this case it is arguable that 405 is the better response code since if readOnly is true, PUT and DELETE are never going to be allowed (with a similar argument applied to many of the WebDAV methods). We have made changes in 9.0.x to better align with RFC 7231 so I'm not against considering changing the status code used. That would also mean changing the OPTIONS response.
As I have been researching this I noticed that the is scope to clean up the existing code. HttpServlet.doOptions()  could use a StringBuilder and ALLOW_OPTIONS seems unnecessary. DefaultServlet.doOptions() appears to be unnecessary as well. This clean-up assumes the current behaviour is not changed. A switch to using 405 is likely to impact some of that clean-up.
If there is interest in switching the DefaultServlet to use 405 rather than 403 when readOnly is true, I'd suggest that a discussion on the dev@ list is required to ensure there is consensus on this approach.

@markt-asf I'd second the 405 instead of 403.

I started this thread on the mailing list to look for consensus.
If consensus is reached would you like me to update this PR or start a different one?

I haven't had any responses in thread I started for this issue.
I have updated this PR to return a 405 rather than a 403. With this 405 response an Allow header will also be returned.
I also updated HttpServlet.doOptions() to use a StringBuilder like you asked. I modeled it after this  HttpServlet.doOptions()

Why is POST not part of the writable commands?

@michael-o I don't understand your question. Are you asking why PUT and DELETE are not allowed when the readOnly flag is true but POST is allowed?

@kdombeck Correct.

@michael-o I had the exact same question in my original post. The only thing I saw was that the POST delegates to the GET, so they both have the same effect.

Alright, did not notice that. Mark has to tell wether he is willing to change this in 9.0. The other version likely won't change.


Pin package:vm_service_client to 0.2.6+2

Description
package:vm_service_client as of 0.2.6+3 is marked as deprecated.  To
avoid future analysis warnings when upgrading packages, pin it to
0.2.6+2 until we can fully migrate to using package:vm_service
instead.
Tests
None.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

PR #34252 includes this.


Regula server docker image is still stuck at v0.5.0

This issue has mutliple parts:

the docker image in the repository is still set to buld v0.5.0, and for me doesn't even build with a current version of docker.
the ./.ci/docker.sh script that is run by travis on this repository is failing every time it runs, so we're not pushing any new builds to quay.io.  The last succesful push is a year old!

... this is causing issues in development because aww pulls in the docker image and people are testing rules against v0.5.0 instead of v0.6.1, which means valid rules including the Percentile and FNV expressions are being rejected.

Context:
@sixstone-qq  was testing a new rule in his local environment and got an "unknown expression kind" error back, that he shouldn't have seen.   That's because he was running the docker image as a server (v0.5.0) but using v0.6.1 as the client code.   The result of this is that he can't test his new rule locally and it makes it hard for him to have confidence about putting it into production.


Grows size of the string buffer for plain text registers dump in x64

Grows size of the string buffer for plain text registers dump in x64 configuration. See #22

Hi @NeoAnomaly ,
Thank you for your PR. We appreciate it. I'm going to wait a few days to give others a chance to review your PR. We'll probably merge your changes next week.
@rush2ko, are you available to test this PR? Does this look like it will solve your issue in #22?
Thanks,
Brian


Ensure proper regex matching

When ensuring a file name is valid should also ensure we're getting the regex options right.

Looks good for me


Furnace fails to render anything on Ubuntu

I usually work on Solus, but I've temporarily been using Ubuntu on a different machine as well. Thought I'd do some work with furnace, but found it wouldn't do anything useful, including giving an error message. This needs to be fixed.

First problem: pcregrep was not installed, and was not checked for. Have fixed that as of 61c038aa867e9f3de594c7b0906df2b3ab6b7584

Second issue was not hurting anything, but was attempting to optimize when no valid optimizer was available.


Add Sync'ing of User Codes Across Locks

Is your feature request related to a problem? Please describe.
I have a handful of ZWave locks and in order to manage codes across all of them need to repeat the same additions/removals of codes for each one.
Describe the solution you'd like
I'd love a way to manage my user codes and have ZWave JS propagate/synchronize them to all nodes.
Describe alternatives you've considered
Manually updating all of them.
Additional context
Add any other context or screenshots about the feature request here.

What about a driver function?

What about a driver function?

Sorry - I'm not sure what that means. Is there a doc you can point me at?

Was thinking of something similar to https://github.com/FutureTense/keymaster, FYI

Driver function allows you to write custom async functions and give you access to all the instances created in the backend.
You can call it from Control Panel > Advanced > Driver Function
This is an example function you can use:
// replace `undefined` with the id of the node you want to clone
const node = driver.controller.nodes.get(undefined);

// list here all the nodes ids you want to copy configuration to
const clonedNodes = [].map(n => driver.controller.nodes.get(n));

const values = node.getDefinedValueIDs()

for (const clonedNode of clonedNodes) {
  for(const v of values) {
  	if(v.commandClass === 0x63) {
  		await clonedNode.setValue(v,
  		  node.getValue(v)
  		)
  	}
  }
}

Then just press on RUN to run it

I don't understand your question, user codes are valueIds, in this snippet I simply clone all writeable values associated to Command Class UserCodeCC

As you imagine you can edit and improve this snippet as you want but it should do the trick as it is (locks should be of the same manufacturer/model btw otherwise you could end up writing value that doesn't exist and that would throw error). Also you could skip write of empty user codes. Anyway let me know if it works


Wrong collectionViewContentSize

I use UICollectionView with NHBalancedFlowLayout inside another UICollectionView because I must get the height of content in NHBalancedFlowLayout by this code:
[((NHBalancedFlowLayout *) self.filesList.collectionViewLayout) prepareLayout];
CGFloat collectionSize = [((NHBalancedFlowLayout *) self.filesList.collectionViewLayout) collectionViewContentSize].height;

But this function returns 0. How can I get the real size of content?

I located this issue. Wrong content height gets when one of sections has 0 count of cells.


Add the EPGCNDS model

Also better to close issues using magic commit messages in PRs

Added the PR to close the issue.


⚠ Turnos 24x7 has degraded performance

In 349c61f, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1309 ms

Resolved: Turnos 24x7 performance has improved in 5b16fca.


Transmission

Environment
Docker
Version
0.7.0
Describe the problem
With 0.7.0 we can now connect to Transmission !
When adding Transmission it requires us to fill the password field, however, one can set up Transmission without any password, like I do.
Thanks a lot for your work. I really like this project.
Additional info
No response
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

Does it still work if you add a value to the password field?

At first, thought it didn't work (error 500 in console).
But now that I try to add a random password it still shows up in green with no error in console so i may have made a mistake at first, I would say it work.

At first, thought it didn't work (error 500 in console).
But now that I try to add a random password it still shows up in green with no error in console so i may have made a mistake at first, I would say it work.

Well, can you view the downloads ?

Yes :) !

Just added a DL to try ; and Yes :) !

Nice 😊 I will now mark this as a 3 point issue if anyone wants to fix it so a password is not required for transmission (if none is provided the server returns 500)

Can confirm this same behavior is applicable to Deluge as well.

@Write @LarveyOfficial Can you test if https://github.com/ajnart/homarr/pull/223 fixes your issue please ?


python3.3 -c "from Crypto.Cipher import ARC4" problem

(venv) krystian@Lenovo-X240:~$ sudo ./extractor.py 8227L-8.bin --system-dir-output wyp
Traceback (most recent call last):
File "./extractor.py", line 29, in
from Crypto.Cipher import AES
ModuleNotFoundError: No module named 'Crypto'

Did you perhaps install the requirements without sudo and then ran the command with sudo?


Collections.Array Module documentation does not have the signature for Array.contains

This page here is missing Array.contains:  https://msdn.microsoft.com/visualfsharpdocs/conceptual/collections.array-module-[fsharp]
Array.contains : 'T -> 'T [] -> bool
This page is missing List.contains:
https://msdn.microsoft.com/visualfsharpdocs/conceptual/collections.list-module-[fsharp]
List.contains : 'T -> 'T [] -> bool
This page is missing Seq.contains
Seq.contains : 'T -> 'T [] -> bool

@KevinRansom Hey Array/List/Seq.contains all are on the module pages now after pull request #217. However, the links don't work yet because the GUIDs in the meta data for the individual documentation aren't linked yet. Not sure what needs to be done to fix that.

I just checked, and as far as I can tell, the links are correctly displayed. So perhaps this issue could be closed?


doc(CHANGELOG.md): add entry for v2.2.0

changelog for v2.2.0

@smothiki This looks good to go, but CI seems borked.


Packagers always say that it's an ACL package, even when it isn't

This file is shared across all packagers, so calling it an ACL package is misleading. There is probably a better way of doing this so that it indicates the type of package based on the packager used, but going with quick and dirty for now.

Pull Request Test Coverage Report for Build 6403

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.469%

Totals

Change from base Build 6396:
0.0%

Covered Lines:
16681

Relevant Lines:
29540

💛  - Coveralls

THanks @shsteimer !


update loader-utils dependency to 0.2.7

since stringifyRequest was introduced in that version

Thanks


"No symbols found" for Markdown files

Since v 1.74.x I have not been able to view outlines of Markdown files.
Steps to reproduce:

Open Markdown file.
Look at Outline panel

Expected behavior

Outline appears in Outline Panel

Actual behavior

Panel shows message "No symbols found in document [DOCNAME]"

VS Code: v 1.75.0
Mac OS: Ventura 13.1 running on Apple M1
(Same issue using Universal and Apple Silicon versions)
Note that this issue does not occur when using the "Visual Studio Code -- Insiders" version. But has been happening for me on the "production" branch for months.

I suspect you've disabled the built-in markdown extension that provides these. Search @builtin markdown  in the extensions view to check  that Markdown Language Features is enabled

@mjbvz your suspicion was spot on. Enabling that setting solved the issue straightaway. Thanks much for this!


Add controller mapping migration tests

Overview
Adds some tests to catch the bug reported in #7405.
Also:

Updates a test script with the test icon
Fixes a quirk I found while testing this fix, where the mapping profiles aren't automatically saved when the update menu item is clicked
Removes an extra line from the controller mapping profile definition

Changes

Follow-up to #7406

/azp run


the project is permanently running on port :3000 - not able to kill the process

After a successful setup - the boilerplate is permanently running on localhost:3000. How can I cancel it ?

OK, I found a way. After unregistering the project on chrome://serviceworker-internals, now I am able to run different project on port:3000.


Update cacheDir configuration with "getresource" usage

This PR updates the cacheDir setup to utilize getresource instead of getjson, aligning with recent changes steering us away from 'getJSON' for remote data retrieval.
Notably, this enhancement now enables correct refreshing of data on the Download Page and CVE feed.
Useful Links
Preview Refreshed CVE Feed Link  | Current Stale CVE Feed
Preview Download Page with v1.30 | Current Empty Download Page
/area web-development

/area web-development

Very small change but huge postive impact on the website. Expectional find and quick fix.
/lgtm

/lgtm

Thanks
/priority critical-urgent
/approve


add "more questions about your landlord" transition

It jumps into subsidized housing - I think it would be easier to understand if it said there were more questions about the landlord
id: subsidized housing identification

I think the navigable sections on the left replace this need, but we can keep the idea open to see how it feels a bit longer.


Play music in time with the action spinner

If I want to use bevy_kira_audio directly, blocked on this issue: https://github.com/NiklasEi/bevy_kira_audio/issues/23

Done!


Recovering schedule doesn't repurpose existing schedule

the previous schedule was actually not touched in Rock, instead a new schedule was linked to the persons account

Note still need to fix the name of the fund


Bug Report: modchart error ( expected near "end")

im trying to do some modchart test but im stuck in this error and i cant do anything bc i dont know what it is

here is the code, im having this error since yesterday

i just realized i didnt need to put end to "for i=0,7 do", the big bruh moment


Remove HttpAction in dotnet

ref: https://github.com/Microsoft/AdaptiveCards/issues/618
note: this is NOT backward compatible

@EricDahlvang,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


ECC-1916 - Update design of UTR screen

Approved, but when Germans was doing this in Reg I think it was, one of the message keys was being used by the CYA page - can you check to make sure that the CYA pages still show the previous content and weren't re-using the same keys?

Checked and confirmed we aren't using the same keys for the CYA page