Command line usage: defaults

From looking at the postgrest command line usage output, it appears that some settings are optional (those surrounded in brackets) and others are required (those surrounded in parentheses).  For the optional settings, I'm not sure what defaults will be used if I omit them.  Can those defaults be documented?
Meanwhile, if I specify all of the settings that seem to be required, I get the following. Any assistance would be greatly appreciated!
$ postgrest -d $USER -U $USER -a $USER
WARNING, running in insecure mode, auth will be in plaintext
Listening on port 3000
<IP_ADDRESS> - - [10/Mar/2015:16:36:37 -0700] "GET / HTTP/1.1" 500 - "" "Wget/1.15 (linux-gnu)"

And a client request shown in the log above:
$ wget http://localhost:3000/
--2015-03-10 16:36:37--  http://localhost:3000/
Resolving localhost (localhost)... ::1, <IP_ADDRESS>
Connecting to localhost (localhost)|::1|:3000... failed: Connection refused.
Connecting to localhost (localhost)|<IP_ADDRESS>|:3000... connected.
HTTP request sent, awaiting response... 500 Internal Server Error
2015-03-10 16:36:37 ERROR 500: Internal Server Error.

And from postgresql-9.4-main.log:
2015-03-10 16:41:44 PDT [23859-1] $USER@$USER LOG:  could not receive data from client: Connection reset by peer

Here's another data point.  An OPTIONS request fares better, at least resulting in a response body. The error says I haven't provided a password, but I can't provide one since I'm using trust authentication over a UNIX socket in a development environment.  Perhaps postgrest is attempting to connect over a TCP socket; if it does, then Postgres will want a password.  So this comes back to another request I opened (https://github.com/begriffs/postgrest/issues/169).
Here is the request/response pair:
$ telnet localhost 3000
Trying ::1...
Trying <IP_ADDRESS>...
Connected to localhost.
Escape character is '^]'.
OPTIONS / HTTP/1.1

HTTP/1.1 500 Internal Server Error
Transfer-Encoding: chunked
Date: Tue, 10 Mar 2015 23:44:14 GMT
Server: postgrest/<IP_ADDRESS>
Content-Type: application/json

05B
{"details":"fe_sendauth: no password supplied\n","message":"Can't connect to the database"}
0

Also, for what it's worth, if there is an approach to resolving this and #169 that we can both be happy with, I would probably be able to write up patches and submit them.

Thanks for submitting this issue and for volunteering to help with patches down the road.
As for the original question, you can pass --help to get more info about default values. However the catch is that this option is itself not documented. :stuck_out_tongue: Maybe a quick note in the readme or in the program description would suffice. https://github.com/begriffs/postgrest/blob/master/src/Main.hs#L31-L33
Care to send a pull request to document this?
I'll look into the Unix socket problem and take that up in issue #169

Regarding --help: I see! I think that's surprising behavior, since usually any time a program shows help, it shows all of it unless there is just too much to show, in which case it shows a synopsis with more detailed help commands provided.  Would you be willing to accept a patch that changes this to emit the same help output no matter what?  That would also sidestep the issue of documenting the more detailed help option since it'd be the default.

Another way in which this could be improved would be to use <...> instead of (...) to indicate required parameters, and to put them first in the usage output so they're noticed up front.  Right now the (...) syntax makes them harder to spot, as does their interleaving with optional parameters.  I also suggest <...> since that's the syntax I'm used to seeing when things are required.

This all sounds great. I'd love a pull request if you've got the time to put one together.

Looking into this now, I see that to get most of the things I've asked about would require patching optparse-applicative. I'd rather not do that since it'd affect more users than might be appropriate, so I'll live with the option ordering and choice of syntax. There are a couple of things left: whether required options are grouped together, and whether the default output is the more verbose output with defaults.
As for the first issue, I could reorder the fields of the AppConfig type to reorder the options in the help output, but that's unpleasant because it means that the presentation issues drive the type declaration, and that seems wrong to me. So unless you see a better path forward there, I'll live with that as-is, too, since the alternative is worse.
The second issue of showing better help output by default might be feasible, but I don't know for sure because I don't understand how optparse-applicative decides whether to show the long or short forms of the usage output, and I don't know how to get it to show the long form by default. I've only ever used the Getopt-style option parsing in Haskell; if you know how to achieve this already, it might be more expedient for you to make this change.
So: as much as I am happy to help out, these issues might be hitting a wall.

There was a pull request two weeks ago where @adambaker made a friendlier command line parser. It actually worked really nicely but had problems with linting and building on Circle CI. Maybe with a little more work it could be revived: https://github.com/begriffs/postgrest/pull/139

Oh, great!  Aside from the change to use record, it looks like those patches essentially take the approach I prefer, i.e., use System.Console.GetOpt.  As far as I can tell, the test failures are build infrastructure issues - perhaps dependency resolution problems?  In other words, as far as I can tell, they aren't issues with the code per se, but perhaps I missed the details.
If you could point me at what needs more work, I can take a look!

Basically it would be great if you could make the GetOpt way work without using Record. The build/linter problems happen because of the record library, and that library is by no means necessary. I think a more vanilla haskell data type would work just fine.
Improving the command line interface would be valuable. The current interface can be confusing.

@jtdaugherty have you thought about this issue since since the comments above?

Hi! Pardon my long absence; the work project I was working on decided it needed something more customized than postgrest and so I haven't returned to this project since then. Although I'd like to see the change described here, I don't have the resources to contribute it right now. I would be happy to have this ticket closed if you would.

I'll close the ticket for now.


option to have JsonSerializer ignore fields with null values

currently the JsonSerializer will output json like this:
{
    people: [
        {
            "id": 10,
            "name": "Dave",
            "email": null
        },
        {
            "id": 11,
            "name": "Susan",
            "email"<EMAIL_ADDRESS>        },
        {
            "id": 12,
            "name": "Dave",
            "email": null
        }
    ]
}

it would be very handy for me if there was a setting to get to ignore the fields with nulls in them:
{
    people: [
        {
            "id": 10,
            "name": "Dave"
        },
        {
            "id": 11,
            "name": "Susan",
            "email"<EMAIL_ADDRESS>        },
        {
            "id": 12,
            "name": "Dave"
        }
    ]
}

Is there something in vibe-d to do this or will I need to write my own serializer?

Nothing built-in, but it probably makes sense to add something, as this has been requested before if I remember right. What you can do is to use custom Json Person.toJson() const; and static Person Person.fromJson(Json); methods that perform the conversion to Json manually. It means some manual serialization work, but it still integrates with the serializer, so that it can still be used in complex structures like above.

It's this request #1541

I need this feature too, but I think we have better type - candidate to skip by default: undefined. Ideally it will be good to have some annotation to skip fields with defined values, for example @toJsonSkip(Null, Undefined) string a; or @toJsonSkip(ZeroLength) string[] b;
I don't know how to implement complete skip json nodes with current implementation of toJson(), I can only make undefined, so I made the next: struct JsonOptional with Nullable inside, generating undefined type node if value isNull, and then function jsonOptionalString() with ugly hack inside: generating json string and then removing undefined nodes with ctRegex. if here will be option in serializeToJson to skip undefined nodes completely it will be reasonable solution, still requiring class like JsonOptional, but simple to implement.
JsonOptional:
struct JsonOptional(T)
{
	import std.typecons : Nullable;
	import vibe.data.json;

	Nullable!T value;

	this(T init){ value = Nullable!T(init); }

	// can't use alias this because of magic method toJson
	T get(){ return value.get; }
	bool isNull(){ return value.isNull; }
	void nullify() { value.nullify; }
	void opAssign(T newValue){ value = newValue; }

	Json toJson() const
	{
		import std.traits;
		static if(isArray!T) // skip empty array if applied to array
		{
			return value.isNull || value.get.length==0 ? Json.undefined() : serializeToJson(value.get);
		}
		else
			return value.isNull ? Json.undefined() : serializeToJson(value.get);
	}

	static JsonOptional!T fromJson(Json src)
	{
		return JsonOptional(Nullable!T(deserializeJson!T(src)));
	}
}

usage:
	struct Email
	{
		string email;
		JsonOptional!string name;
	}

	Email e = {email<EMAIL_ADDRESS>	assert(e.jsonOptionalString == "{email:\"example@example.org\"}");

Skipping undefined nodes needs to be done anyway (currently a bug), so that solution should definitely be possible. I think instead of explicitly telling the serializer what to skip, using the value of the default initializer is the cleaner solution, because it ensures that serialization is always properly reversible. However, there is some ambiguity in that definition for nested types...
I'll close this as a duplicate of #1541 for now (we can continue to discuss there).


404 at /desktop/previous-versions/3.x-windows/undefined

I found a broken link : https://docs.docker.com/desktop/previous-versions/3.x-windows/undefined
I arrived on this page through: https://docs.docker.com/desktop/previous-versions/3.x-windows/404 at /desktop/previous-versions/3.x-windows/undefined
at /desktop/previous-versions/3.x-windows/undefined

Thanks for sharing the issue. Closing this ticket in favor of https://github.com/docker/docker.github.io/issues/13553, but if you're able to provide more details that could help us find the problem, feel free to comment on that ticket.


Bug: HScrollbar slider cannot be grabbed when at leftmost side.

Godot 2.1.2 Stable for Linux
Issue description:
When the slider of the scrollbar is at the far left side, which
is the min value, I cannot grab the slider with the mouse.
I insured that no other control node or selectable node was
obstructing selection of the slider.
The slider is still highlighted when the mouse hovers
over the bar but the moment the mouse pointer is within
range of grabbing the slider the slider stops being highlighted.
The issue caused by the bug is alleviated by setting the
sliders page value to a higher setting. Doing so lets the mouse
pointer grab the right portion of the slider but not the middle
or left side of the slider.
I apologize if I formatted this incorrectly as this is my first
time reporting a bug for Godot.
**Node: HScrollbar
Node Settings:
Custom Step: -1
Min: 0
Max: 255
Step: I set it to many different steps. 0-123
Page: 10 or less
Exp Edit: False or True
Rounded: False or True
Anchor
Left: Begin
Top: Begin
Right: Begin
Bottom: Begin
Rect
Size (154, 12)
Min Size(0,0)
Rotation 0
Scale(1, 1)
Ignore Mouse: True or false
Stop Mouse: True or False
Size Flags
Horizontal: Fill
Vertical: null
Stretch Ratio: 1
I have no theme installed
Margin
Left 0
top - 68
right 154
bottom -56
no custom icons or styles
Visible true
Opacity 1
Self Opacity 1
Behind Parent false
Blend Mode: Mix
Light Max: bit 0
No materials
process Inherit
Script null.
When the slider is small, I cannot grab the slider with the pointer when
the slider is on the far left slide of the bar**
Link to minimal example project:

Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.

As there was no update since the previous post, we close this issue.
If it is still relevant in the way it was described and discussed above, please comment to ask for it to be reevaluated. If it is only partly relevant, or if the original issue has changed, it would be better to open a new issue (potentially referring to this one) focused on the current state.


fix(datepicker): don't fall back to parse using new Date(...)

Fixes #2508

+1 (haven't seen this PR and prepared mine https://github.com/angular-ui/bootstrap/pull/3288). After some searching I found a lot of issues/PRs with some solutions. Would be nice to finally have one of them merged into master.
Cheers.

This fails tests when rebased off of master.

@wesleycho Travis passes?

That is probably because it was valid when the PR was open.  It does not pass currently though.

As I commented on #3294, this needs tests for validity of the date if it doesn't follow a specified format.


Jose to create "searchpage" HTML with CSS framework

Style this page using Bootstrap and CSS
Refer to the Bootstrap docs here https://getbootstrap.com/docs/5.3/getting-started/introduction/


😈 Send Notification to Main Channel on Failed Terraform Applies

User Story
As an AP Engineer, I would like to be prompted when a terraform apply fails after a merge so the code can be fixed as quickly a possible.
Value / Purpose
This will allow us to maintain our main repository in a deployable state always.
Useful Contacts
Analytcal Platform Team
User Types
Ops Engineers
Hypothesis
If we are promptly notified of failures this will improve the overall posture of our platform
Proposal
Add a slack notification when a terraform apply fails. If we limit it to applies only we should minimise spam.
Additional Information
No response
Definition of Done

[ ] Webhook implemented
[ ] Noisiness tuned
[ ] Another team member has reviewed
[ ] Tests are green

Draft PR created. Documented the process for future use, will share this in #github-community when I've confirmed this process works.


Should Validator validate() keep using ValidationException or perhaps RuntimeException

I've just hooked up the validation to avaje-nima and looking at the difference between the avaje-validation ConstraintViolationException and avaje-http ValidationException.
When mapping from ConstraintViolationException -> avaje-http ValidationException we can lose details that ConstraintViolationException has and to me that could be problematic (and problematic for other validators that someone might wish to use).
I'm pondering if we should effectively remove / deprecate / not use the avaje-http ValidationException and instead allow the ConstraintViolationException to be thrown and then the error handler has full control over how the ConstraintViolationException is handled.
That is, the API requirement for avaje-http validate() becomes that it throws some RuntimeException (a ConstraintViolationException) and it's up to the http routing to have a error handler registered to handle that specific exception.
Another alternative is for ValidationException to have a cause which in the avaje-validation case is the ConstraintViolationException.

I like the consistency and plug and play between the different validators, maybe just add cause to Validation Exception?

maybe just add cause to Validation Exception?

Let's go with that and see how it plays out in 2.0-RC.  My gut also likes the consistency and plug and play between the different validators. If/when an error handler needs the underlying cause exception it can be accessed so that should be good enough and simple/elegant enough.


Fix Request Headers gatherer

Use GET instead of HEAD as some servers don't respond to the HEAD request. For example a  HEAD request to https://www.funda.nl/ results in 404 causing audits to fail:

Should also be ResponseHeaders instead of RequestHeaders.


Rename the master branch to main

After releasing v0.8.0, I'll rename the master branch to main (I have several repositories with main branch locally and it confuses me when typing git commands.)

Done.


Linux installed without my consent

Thanks for the debloater, but now I have Linux listed in File Explorer.

When trying to open Properties to see details on it, I get an error "The properties for this item are not available".

I did not consent to this.  So...  a debloater added bloat to my PC?
How do I remove the whatever iteration of Linux installed from running BloatyNosy?  It is not listed in partitions.  It is not listed in hardware devices.  It is not listed in installed programs.  It is not listed in Features.  There doesn't seem to be a simple way to remove what I didn't ask for.
(yes...  I created an account JUST to complain about this...)

You've enabled "Microsoft Windows Subsystem for Linux" feature in BloatyNosy. Many find it useful, which is why it has made its way into BloatyNosy. You can easily disable it by checking the box next to "Feature Recommendation" and then restoring the default settings (button with downward arrow next to the analysis button).

That...  worked?
I think it worked.  But I had restored defaults...  so I rebloated Windows?
Anyhow...  the Linux drive/folder/subsystem//icon/whatever is gone.  Luckily, this PC isn't important enough to bother further.
I recommend streamlining the GUI so it's simple to follow.  Having a feature listed at the bottom of a list (accessible via a small-print link-type sentence at the bottom of an oversized window with too much blank space) and describing the feature as to making a user unsure if I should check or uncheck the box to activate or deactivate the feature makes it confusing.
This is one of the reasons I regularly have trouble with open-source software and the like.  I enjoy them, but I have to filter through lots of badly interfaced choices until I find a good one.
Thanks, though!!!

@Belim, why is WSL considered part of debloating, and why can't it be toggled off so it doesn't have to be manually turned off after 'fix'-ing?


Perhaps we should bump our MSRV

I am running into this error:
$ cargo metadata
warning: please specify `--format-version` flag explicitly to avoid compatibility problems
error: failed to download `bumpalo v3.11.1`

Caused by:
  unable to get packages from source

Caused by:
  failed to download replaced source registry `https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to parse manifest at `/home/steve/.cargo/registry/src/mirrors.ustc.edu.cn-61ef6e0cd06fb9b8/bumpalo-3.11.1/Cargo.toml`

Caused by:
  failed to parse the `edition` key

Caused by:
  supported edition values are `2015` or `2018`, but `2021` is unknown

$ rg "bumpalo"
Cargo.toml
31:[target.'cfg(target_arch = "wasm32")'.dependencies.bumpalo]

This is because bumpalo bumps their MSRV to 1.56, so perhaps we should also bump our MSRV.
Since whoami won't bump MSRV until a major version update, an alternative approach is to use an older version of bumpalo.

If you run these commands the error goes away.  An alternative would be to remove the rust-toolchain file (which I think would be fine).
Some context: https://github.com/ardaku/whoami/issues/43

Thanks for your help


Render each word as its own  element (fix #1, #5)

Testing revealed that wrapping each line in a  and rendering individual words as <tspan> elements did not have any effect on the text selection behavior. Due to an incomplete SVG implementation in Firefox, it would not stretch <tspan> elements to the appropriate width.
So, with no ill effects to expect (to my current knowledge), this PR removes the rendering of <tspan> elements and renders every word as an individual, unwrapped <text> element.

Codecov Report

Merging #21 into master will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
- Coverage   71.86%   71.79%   -0.07%
==========================================
  Files          10       10
  Lines         430      429       -1
  Branches      135      134       -1
==========================================
- Hits          309      308       -1
  Misses         99       99
  Partials       22       22

Impacted Files
Coverage Δ

src/components/PageTextDisplay.js
95.83% <100.00%> (-0.09%)
:arrow_down:

Continue to review full report at Codecov.

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


How long is a stisfied MFA prompt valid?

Hi,
I successfully implemented Point-to-Site VPN via an Azure Virtual Network Gateway.
Also, MFA is working.
When I disconnect the session and reconnect, I do NOT get prompted for MFA again.
My MFA Service Settings do NOT allow saving the persistent coockie (i.e. don't prompt for the next 14 days or so).
When deleting the saved user account within the VPN Client, the user login and MFA is required again, which is totally clear and makes sense.
So my question is: How long will the MFA satisfaction be valid, until a reconnect will prompt again for MFA? And is there any other way to influence this, besides the Session Lifetime in the respective Conditional Access policy?
Thank you!

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

ID: 77476799-7803-afb8-c69f-d537af04c2f3
Version Independent ID: 5e1056a6-61b1-585b-5f01-d6cd942600f3
Content: Enable MFA for VPN users: Azure AD authentication
Content Source: articles/vpn-gateway/openvpn-azure-ad-mfa.md
Service: vpn-gateway
GitHub Login: @KumudD
Microsoft Alias: alzam

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@joeakim
Apologize for the delayed response. Answering your question-
-How long will the MFA satisfaction be valid, until a reconnect will prompt again for MFA? And is there any other way to influence this, besides the Session Lifetime in the respective Conditional Access policy?
MFA session tokens are valid until explicitly revoked as mentioned in document-MFA session tokens are valid until explicitly revoked as mentioned in document- https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-configurable-token-lifetimes#configurable-token-lifetime-properties.
Other than session lifetime in the Conditional access policy, you can use the following- "Remember Multi-Factor Authentication feature". This feature lets users bypass subsequent verifications for a specified number of days, after they've successfully signed-in to a device by using Multi-Factor Authentication. The feature reduces the number of authentications on web apps, which normally prompt every time. The feature can increase the number of authentications for modern authentication clients that normally prompt every 90 days, if a lower duration is configured.
Please let us know if you have any further questions/concerns and we will be glad to assist you further. Thank you!

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@SaiKishor-MSFT I don't think your comment is accurate. The lifetime of the Azure token does not determine how long the Azure VPN client's connection lasts. For example, if you create a conditional access policy with a 1hr sign-in frequency, this does not terminate the user's VPN connection after 1 hour.
You even mention the same in another thread: https://docs.microsoft.com/en-us/answers/questions/560291/disconnection-of-vpn-after-inactivity.html


[Proposal] Ability to specify resource controller's wildcard place holder.

A resource controller's wildcard segment is currently named after the resource itself. I'd prefer to specify my own sometimes.
Benefits:

Share regex route constraints by utilizing the same wildcard across different resources.
Less typing. A long resource name parameter is a bigger hassle to write than id or similar.
Allows users to be more semantic. If I have a companies resource controller, I feel weird specifying a single ID into a {companies} parameter. That might just be me, but it's a benefit in my book. :grinning:

Implementation
In Router::resource()
// Was a custom wildcard place-holder specified?
if (isset($options['wildcard']))
{
    $base = $options['wildcard'];
}
else
{
    // We need to extract the base resource from the resource name. Nested resources
    // are supported in the framework, but we need to know what name to use for a
    // place-holder on the route wildcards, which should be the base resources.
    $base = $this->getResourceWildcard(last(explode('.', $name)));
}

Added and working great for me already from what I can tell.

Pull request reference
https://github.com/laravel/framework/pull/7654

Here's the solution: https://github.com/laravel/framework/pull/7654#issuecomment-221854648


Need to show the code for MyMiddleware

This page is fantastic in showing how to unit test a middleware.  But to replicate it, I need to see the code for MyMiddleware that it the code under test.  (It causes a 404 in some situations).
It would be nice to include the code for this file.

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

ID: 9421feee-5207-e379-00fb-847cc880014c
Version Independent ID: 8415adee-793e-2ba7-b97b-672b493ba42b
Content: Test ASP.NET Core middleware
Content Source: aspnetcore/test/middleware.md
Product: aspnet-core
Technology: aspnetcore-test
GitHub Login: @Tratcher
Microsoft Alias: riande

Hello @Vaccano ... This isn't meant to be a tutorial. It's a reference doc that shows in a general way how to compose a test in a mostly arbitrary way. You would add/subtract from the tests as needed for whatever middleware that you want to test. My recommendation is for you to use an existing middleware that you have or just whip one up to try out the features described, then tailor the tests to that middleware.


Embrace parseable output to stdout

It is common to print smth parseable for the sake of pipe chaining among POSIX utils.
However, ansible-review doesn't seem to care about this:
✦ ➜ ansible-review rebuild-world.yml -q 1>/dev/null
Couldn't parse task at rebuild-world.yml:8 (no action detected in task. This often indicates a misspelled module name, or incorrect module path.

The error appears to have been in '<unicode string>': line 8, column 5, but may
be elsewhere in the file depending on the exact syntax problem.

(could not open file to display line))
{ 'eix': {'__file__': 'rebuild-world.yml', '__line__': 9, 'action': 'sync'},
  'name': 'Sync portage/layman trees and index ebuilds'}

I'm building a GitHub App integration demo, which would really benefit from not having to use heuristics and manual conditional parsing to retrieve structured result data.

@willthames ?

FWIW here's my playground: https://github.com/webknjaz/ansible-gentoo-laptop/pull/4/checks?check_run_id=35656446
And the source code: https://github.com/webknjaz/ansiwatch-bot

If there is a PR that would fix this, let me know. I've had a rereview of #28 but that needs a rebase and I'm still not convinced by the custom option parser.

Yeah, I'm also not sure whether that solution is a good fit.
OTOH I'd probably prefer just one switch to emit everything as JSON or similar, maybe JSONL.
The problem is that I don't know whether it needs architectural changes to unify output format as well.
And I also think that for my purposes using Python API would be even better which would probably require a slight change like putting some chunk of logic from main() into its separate function.


Support Custom Annotations

thanks for butterknife.
I found butterknife anntations managed by ButterKnifeProcessor class
-private static final List<Class<? extends Annotation>> LISTENERS
I want to make some custom annotation with butterknife.
let me know how I add custom annotation myself.
is it possible?

Hey,
To make custom annotations are not related with ButterKnife. It is built by using annotations. ButterKnife's annotations are also custom ones. So, yes, it is possible. ButterKnifeProcessor class extends AbstractProcessor which implements Processor interface. All of the annotations are
parsed by the Processor. You can check this and this
for further information.

@JakeWharton would you be willing to take a look at a PR for something along these lines if I were to work on it?

Sure!
On Mon, May 9, 2016 at 11:48 AM Jay Newstrom<EMAIL_ADDRESS>wrote:

@JakeWharton https://github.com/JakeWharton would you be willing to
take a look at a PR for something along these lines if I were to work on it?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JakeWharton/butterknife/issues/497#issuecomment-217904065

HI @JakeWharton
I need to know if i can implement this feature in ButterKnife.
That's what I'm thinking about

Add new Annotation which is called AnnotationAdapter with ANNOTATION_TYPE target under butterknife-annotations module, take one attribute which is Class<?>, this annotation used for annotate custom annotation and pass implementation class to it as attribute e.g (@AnnotationAdapter(ViewPager2Listener.class)) the ViewPager2Listener is a class for deal with ViewPager2 and register listener to it
Add new interface which is called ButterKnifeAdapter for example and this interface take one generic parameter which is the view you need to create custom annotation for it to use it with AnnotationAdapter
Add another interface which is called ButterKnifeListener for example this interface to link custom listener with code generation

Full example
AnnotationAdapter
@Target(ElementType.ANNOTATION_TYPE)
public @interface AnnotationAdapter {

    Class<? extends ButterKnifeAdapter> value();
}

ButterKnifeAdapter
public interface ButterKnifeAdapter<T> {
}

ButterKnifeListener
public interface ButterKnifeListener<T> {
    void invoke(T t);
}

OnPageChangeListener2 (My custom annotation)
@AnnotationAdapter(ViewPager2Listener.class)
public @interface OnPageChangeListener2 {
    int value();
}

ViewPager2Listener(My custom listener)
public class ViewPager2Listener implements ButterKnifeAdapter<ViewPager2> {

    public ViewPager2Listener(ViewPager2 viewPager2, ButterKnifeListener<Integer> listener) {
        viewPager2.registerOnPageChangeCallback(new ViewPager2.OnPageChangeCallback() {
            @Override
            public void onPageSelected(int position) {
                super.onPageSelected(position);
                listener.invoke(position);
            }
        });
    }
}

MainActivity
public class MainActivity extends AppCompatActivity {

    @OnPageChangeListener2(R.id.viewpager2)
    void onPageChange(int position) {

    }
}

I do not know if there is a better way to do that but that's what I thought of, also i think i need another annotation like @SupportCustomListeners for example to annotate activity, fragment etc... to scan classes and search for custom annotations which is annotated with @AnnotationAdapter

I worked on something for this a while back. https://github.com/JakeWharton/butterknife/pull/573
I'd be happy to help if there's anything we can do to get support merged.

@JayNewstrom After i saw your pull request i have changed my thought about custom annotations i think it will be implemented like this
OnPageChangeListener2 (My custom annotation)
@ListenerClass(
    targetType = "androidx.viewpager2.widget.ViewPager2",
    setter = "registerOnPageChangeCallback",
    remover = "unregisterOnPageChangeCallback"
    type = "androidx.viewpager2.widget.ViewPager2.OnPageChangeCallback",
    method = @ListenerMethod(
        name = "onPageSelected",
        parameters = "Integer"
    )
)
public @interface OnPageChangeListener2 {
    int value();
}

MainActivity
public class MainActivity extends AppCompatActivity {

    @OnPageChangeListener2(R.id.viewpager2)
    void onPageChange(int position) {
    }
}

And ListenerClass must be added to LISTENERS in ButterKnifeProcessor and generate code based on ListenerClass attrs


Web Map "ObservationsMap" references services/layers that do not exist

Item id 3ca2b8e0e7eb4f8893143c11b00e42fc references the following services/layers that do not exist:
NationalIntelligence/ObservationReports/FeatureServer/0
This service only contains a single layer "SaluteReports" layer id 1.

@elinz fixed

@topowright Layer 0 still does not exist.

Observation report Layer 0 exists. Closing.


Add test showing that inheriting a non-existent document summary give…

…s NullPointerException
Today we give a warning and plan to fix this in Vespa 9. This test shows that we might as well throw as we get a NullPointerException and config server returns and internal server error when this happens, which is worse.
@baldersheim the fix is easy, but it is marked for change in Vespa 9, any objections to changing it now?

I now see that this statement will fail if we do this:
   document-summary content-summary inherits default {


Update examples in README

Ref #46

Hey, I see this is a pull request with good documentation changes but it looks like it's to your own repo. If you submit it as a PR to mine, I'll happily accept it.

Haha, oops!  Thanks for pointing that out, one sec...


fix(circleci): update ci

Comparable to the other CI update PRs, but I will test this before marking it ready for review. I will also see what we want to do with vue-mapbox since that does not work with new versions of mapbox-gl and causes npm audit to fail

Partially superseded by #959
Rest will be done later in another new PR


Fixed open bug

If the user gives a path that does not exist and must be created, open(file, output_dir) fails.
I've fixed the problem with 2 additional lines.
I think it is worth to merge.

Thanks !


C# - GC.Collect causes the game to crash

Godot version
4.0 beta 3
System information
Windows 10, RTX2080 Vulkan
Issue description
Calling "GC.Collect()" causes the game to crash. The window freezes fore a second and closes.
There's no error message thrown inside the editor nor in VisualStudio while debugging.
"GC.Collect()" works in 3.x
Steps to reproduce
Call GC.Collect() from code.
Minimal reproduction project
No response

@Xyotic Please upload a minimal reproduction project to make this easier to troubleshoot.

I opened PR https://github.com/godotengine/godot/pull/69194 that may fix this issue, can you try it out to confirm?
Linux users can download the artifact built by CI: https://github.com/godotengine/godot/suites/9514119809/artifacts/451640372
For other platforms, you'll have to build Godot yourself.

Can you check if this is fixed in beta 8?

I tried my old code with GC.Collect() and it seems to be stable now. No random crashes anymore.
Closing this as it seems to be fixed.
Thanks for the awesome work. 👍


[Theme] Suggestion: Bootflat Admin

I came across this https://github.com/silverbux/bootflat-admin

I think this would fit perfectly in Flask-Admin, maybe I will start porting the template, does anybody want to help on this?

Reminds me of some themes on https://bootswatch.com/
I don't think it's necessary to make this the default, but maybe you could improve the layout examples with it? https://github.com/flask-admin/flask-admin/tree/master/examples/layout_bootstrap3

Yes, If you could make an example that runs with that, I'd love to put it up at http://examples.flask-admin.org/.

One thing though - need to check license. If it is not BSD, we can't include it to the main repo, but can reference it from layout example or docs.

:+1:  ;)

Hi - just curious - I would love to use a layout to possibly use for a flask-admin dashboard with d3.js - did an example ever get created with it?

Bootswatch has been included. To select a theme, use:
app.config['FLASK_ADMIN_SWATCH'] = 'cerulean' and have a look at http://bootswatch.com/3/ for the available themes.
It would be great to have an example of a dashboard for monitoring / analytics. If you've implemented one, please contribute.


branch feature/best_matching: simply doesn't run through

On branch "feature/best_matching" simply doesn't run through. The problem lies in actors.py,  lines 94 and 95:
if cfg.show_prints:
    print(f"Cleared Volume: {round(m.cleared_volume[t], cfg.round_decimal)}")

t starts with 0 while the time is counted starting with 1.
Suggestion:
if cfg.show_prints:
    print(f"Cleared Volume: {round(m.cleared_volume[t+1], cfg.round_decimal)}")

already merged in dev


Simplify job forms (UI)

As ...
Austin - Network Automation Engineer
I want ...
to be able to hide "irrelevant" form fields Task queue and/or Execution type from the job form UI and just trigger jobs with parameters (and default settings).

So that ...

the form is not unnecessarily longer,
it doesn't confuse people who don't work with Nautobot that often and trigger some jobs very sporadically,
I can decide instead of user what worker should be used within the code.

I know this is done when...

I am able to hide these options in UI.
I am able to set default settings within the job code.

Optional - Feature groups this request pertains to.

[X] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[X] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
No response
External Dependencies
No response

Maybe making less used panels collapsed by default would also be a good idea.


findspam.py: Pattern for http://%20 in href

This is a common formatting error in spam currently (386 TP hits, 1 FP, 1 NAA)

Remember confirmation bias. Have you checked this against normal posts too?

I tried to search in SEDE but it times out as soon as I get a query which I hope will actually work.
On a smaller site like English, I can run this, and get zero results: https://data.stackexchange.com/english/query/616177/urls-with-http-20-pattern

Zero on Mathematica (too small to be useful, I guess), Ask Ubuntu, Server Fault, Super User, and Mathematics.  Still times out on Stack Overflow. (The SEDE front page arranges sites by size, so I picked the top four after SO, and a smaller one by mistake -- Mathematica autocompleted when I wanted to select Mathematics.)

The onsite search shows 1,400+ posts with the pattern http://%20
Search Link


Add indexing job solution to documentation

Autorun search and navigation indexing job
Environment synchronizer is unable to trigger this job using "AutoRun: true". This is because the job requires access to HttpContext, and the IScheduledJobExecutor runs jobs as background tasks, where HttpContext does not exist.
This can be solved by modifying the job via a custom synchronizer.

Solve problem with issue #22


Adding functions to support dynamically change Aerospikeconfig

Codecov Report
Attention: 647 lines in your changes are missing coverage. Please review.

Comparison is base (08273f7) 33.58% compared to head (96da5b0) 30.22%.

Files
Patch %
Lines

deployment/as_setconfig.go
0.00%
242 Missing :warning:

asconfig/utils.go
14.80%
206 Missing and 7 partials :warning:

deepcopy.go
0.00%
79 Missing :warning:

asconfig/schema.go
1.53%
64 Missing :warning:

deployment/cluster.go
0.00%
32 Missing :warning:

deployment/aeroinfo.go
0.00%
6 Missing :warning:

deployment/host.go
0.00%
6 Missing :warning:

info/as_parser.go
66.66%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   33.58%   30.22%   -3.36%
==========================================
  Files          19       21       +2
  Lines        4758     5326     +568
==========================================
+ Hits         1598     1610      +12
- Misses       3052     3608     +556
  Partials      108      108

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


Format issue

Template
Please write your blade template that occurs issues.
Context (Environment)

write your environment

Description

Any other informations

Closed by #372


Fix error on webview first load

Fixes https://github.com/ukwa/npld-player/issues/28 by preventing changes to webview src attribute on first load, and some minor cleanup.
See https://github.com/electron/electron/issues/24608
Manual testing
Run app with yarn start. Verify that webview loads as expected and error does not show in dev tools console log.

The fix is never changing the src attribute of the webview, right?


feat: add custom assets to the release

closes #95

:tada: This PR is included in version 1.1.0-dev.12 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0-dev.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add package.json metadata for npmjs.org publication

This is to make some metadata available to the npmjs.org scripts, namely the license and the relevant project urls (code, bugs, homepage) that are suggested when you npm init a new module.
The keyword section of the package.json changes should help make our project more visible to people browsing the various statsd backend plugins.

+1


Create README.md

Description

[x] I have added a folder(Named SHASHANK0712 as my GitHub user id)
[x] I have added a Readme.md file inside the folder
[x] I understand the GitHub Forking Concept fully

Hello  Sir,
It is a humble request to you to please help me by accepting my PR.
I will be very grateful to you.
Thanks


Is TabBar can scroll horization?

When tabs are too many, it will so crowded to render in full width screen

Can you share a screenshot?

I'm assuming this is what is referred to. Adding a lot of tabs when needed, it would be good to have a minimum and make the tabs themselves scroll hortizontally

This is just the example snack adding 10 tabs:
https://snack.expo.dev/@morten-backflip/react-native-tab-view-lazy-load


1.0.0dev5 release

This is the tracking issue for version 1.0.0dev5 of text_formatter.
Release process jobs:

[x] Make GitHub release and tag
[ ] Upload package to http://pypi.org/project/text_formatter
[ ] Make note on project text_formatter - Project board

I don't know if we must release this. Even when most of our issues are closed, we are not ready for a consistent release at all.

There are just a lot of things to do. I'll cancel this process.


chore: avoid command panic

chore: avoid command panic

Codecov Report

Merging #44 (d1ee116) into main (c587a29) will decrease coverage by 1.30%.
The diff coverage is 17.64%.

: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             @@
##             main      #44      +/-   ##
==========================================
- Coverage   37.27%   35.97%   -1.30%
==========================================
  Files          12       12
  Lines         491      492       +1
==========================================
- Hits          183      177       -6
- Misses        283      290       +7
  Partials       25       25

Impacted Files
Coverage Δ

pkg/account/create.go
57.50% <14.28%> (-8.36%)
:arrow_down:

pkg/account/delete.go
66.66% <20.00%> (-7.25%)
:arrow_down:

pkg/account/get.go
76.47% <20.00%> (-5.35%)
:arrow_down:


Browser panel does not fill entire space

There is an empty space below browser panel when the application is in maximized.

Removal of vbox in BrowserPanel.fxml should solve the issue.


Add Chinese (Simplified) translations.

Thank you!

Hey @jellybean13 , I am currently in the process of relicensing Controllable to a more suitable license. I would like your permission to relicense your contributions to Controllable under MIT.


Improve the README

To make it easier for people to know what's going on here!
Add that it uses github pages when deployment is done.
Film template pages
History

We've improved this now!


breaking: Remove NetworkIdentity Server Only

This has become a corner case feature that only applies in practice to networked scene objects. Such could just be instantiated by Network Manager from OnStartServer with MonoBehaviours.
WIP: If this will be approved, I'll need to fix the examples.

that only applies in practice to networked scene objects.

why is that?
and are you sure that nobody needs this feature?

Closing - more hassle than it's worth.

Maybe a tooltip will help, as i've seen more people accidentally use it, than have needed it.
Explanation + example use-case scenario.
(i remember a few have have found a use for it in the past)


Error when creating osz

I get this error when exporting:
Traceback (most recent call last):
File "fnf_converter.py", line 672, in exportAsOsz
File "fnf_converter.py", line 200, in exportOsuFile
File "fnf_converter.py", line 345, in getNotesFromPlayer
ValueError: invalid literal for int() with base 10: 'Show Song'
Any solutions?

Which song? Can you put the song and the chart file here?

demise.json
the chart from mario madness v2

Never seen something like this -_-
					[
						16105.2315789474,
						1,
<PHONE_NUMBER>63158
					],
					[
						15157.8947368421,
						-1,
						"Show Song",
						"",
						""
					]

Fixed in version 1.3.1. Download it, it should work


Review of course material

p.5  "are very populare" -> popular

Currently having permission issues on macOS when installing sct, I've ran as instructed by the installer :

sudo chown -R $(whoami) /usr/local/var/homebrew
sudo chown -R $(whoami) /usr/local/lib/pkgconfig /usr/local/share/info /usr/local/share/locale /usr/local/share/man/man3 /usr/local/share/man/man5 /usr/local/share/zsh /usr/local/share/zsh/site-functions /usr/local/var/homebrew/locks
chmod u+w /usr/local/lib/pkgconfig /usr/local/share/info /usr/local/share/locale /usr/local/share/man/man3 /usr/local/share/man/man5 /usr/local/share/zsh /usr/local/share/zsh/site-functions /usr/local/var/homebrew/locks

But I still get the following error message :
Updating Homebrew...
touch: /usr/local/Homebrew/.git/FETCH_HEAD: Permission denied
touch: /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/.git/FETCH_HEAD: Permission denied
fatal: Unable to create '/usr/local/Homebrew/.git/index.lock': Permission denied
fatal: Unable to create '/usr/local/Homebrew/.git/index.lock': Permission denied
error: could not lock config file .git/config: Permission denied

Adding /usr/local/Homebrew to the suggested chmod commands fixes it

@olix86 could you please copy/paste the entire terminal output (so i know what part of SCT install process you went through, what was the verbose info you got, etc.) thx!

p.s. i've created a card in project for typos specifically (easier for me to go through all typos at once): https://github.com/neuropoly/spinalcordtoolbox/projects/11#card-31469746

(ivmed) pass:sct olix86$ ./install_sct

*******************************
* Welcome to SCT installation *
*******************************

Checking OS type and version...

Darwin pass.neuro.polymtl.ca 18.6.0 Darwin Kernel Version 18.6.0: Thu Apr 25 23:16:27 PDT 2019; root:xnu-4903.261.4~2/RELEASE_X86_64 x86_64
ProductVersion:	10.14.5

Checking requirements...

WARNING: "gcc" is not installed.

Do you want to install it now? (accepting to install "gcc" will also install "brew" in case it is not installed already)? [y]es/[n]o: y
Error: Can't create update lock in /usr/local/var/homebrew/locks!
Fix permissions by running:
  sudo chown -R $(whoami) /usr/local/var/homebrew
Error: The following directories are not writable by your user:
/usr/local/lib/pkgconfig
/usr/local/share/info
/usr/local/share/locale
/usr/local/share/man/man3
/usr/local/share/man/man5
/usr/local/share/zsh
/usr/local/share/zsh/site-functions
/usr/local/var/homebrew/locks

You should change the ownership of these directories to your user.
  sudo chown -R $(whoami) /usr/local/lib/pkgconfig /usr/local/share/info /usr/local/share/locale /usr/local/share/man/man3 /usr/local/share/man/man5 /usr/local/share/zsh /usr/local/share/zsh/site-functions /usr/local/var/homebrew/locks

And make sure that your user has write permission.
  chmod u+w /usr/local/lib/pkgconfig /usr/local/share/info /usr/local/share/locale /usr/local/share/man/man3 /usr/local/share/man/man5 /usr/local/share/zsh /usr/local/share/zsh/site-functions /usr/local/var/homebrew/locks

ERROR: Installation of "gcc" failed. Please contact SCT team for assistance.

Installation failed

(ivmed) pass:sct ovincent_admin$ sudo chown -R $(whoami) /usr/local/var/homebrew
Password:
(ivmed) pass:sct olix86$ sudo chown -R $(whoami) /usr/local/lib/pkgconfig /usr/local/share/info /usr/local/share/locale /usr/local/share/man/man3 /usr/local/share/man/man5 /usr/local/share/zsh /usr/local/share/zsh/site-functions /usr/local/var/homebrew/locks
(ivmed) pass:sct olix86$ chmod u+w /usr/local/lib/pkgconfig /usr/local/share/info /usr/local/share/locale /usr/local/share/man/man3 /usr/local/share/man/man5 /usr/local/share/zsh /usr/local/share/zsh/site-functions /usr/local/var/homebrew/locks
(ivmed) pass:sct olix86$
(ivmed) pass:sct olix86$ ./install_sct

*******************************
* Welcome to SCT installation *
*******************************

Checking OS type and version...

Darwin pass.neuro.polymtl.ca 18.6.0 Darwin Kernel Version 18.6.0: Thu Apr 25 23:16:27 PDT 2019; root:xnu-4903.261.4~2/RELEASE_X86_64 x86_64
ProductVersion:	10.14.5

Checking requirements...

WARNING: "gcc" is not installed.

Do you want to install it now? (accepting to install "gcc" will also install "brew" in case it is not installed already)? [y]es/[n]o: y
Updating Homebrew...
touch: /usr/local/Homebrew/.git/FETCH_HEAD: Permission denied
touch: /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/.git/FETCH_HEAD: Permission denied
fatal: Unable to create '/usr/local/Homebrew/.git/index.lock': Permission denied
fatal: Unable to create '/usr/local/Homebrew/.git/index.lock': Permission denied
error: could not lock config file .git/config: Permission denied
Error: Your Xcode (8.2.1) is too outdated.
Please update to Xcode 10.2.1 (or delete it).
Xcode can be updated from the App Store.

Error: Xcode alone is not sufficient on Mojave.
Install the Command Line Tools:
  xcode-select --install

ERROR: Installation of "gcc" failed. Please contact SCT team for assistance.

Installation failed

Permission issue is fixed by sudo chown -R $(whoami) /usr/local/Homebrew/, currently looking at xcode issue

hum... that's an annoying issue. I'm not sure we should add an "if case" in the installer to update Homebrew permissions (a bit too invasive). Hopefully this is an isolated case (pass is an old station, with a lot of software installed/uninstalled). What do you think?

Indeed, it's probably not worth it if I'm the only one who experienced the issue...

I have the same Xcode issue (which is just an update to install) and afterward, it ran fine on my computer. However, I am not sure if changed the permission on the homebrew folder previously ...

gcc comes with xcode-- so if your xcode is up-to-date, the installer will not try to install homebrew (hence, no issue with permissions).
so maybe we should suggest to update xcode... could you please copy/paste the message you got wrt. xcode version from the installer?

* Welcome to SCT installation *
*******************************

Checking OS type and version...

Darwin marsalis.neuro.polymtl.ca 19.2.0 Darwin Kernel Version 19.2.0: Sat Nov 9 03:47:04 PST 2019; root:xnu-6153.61.1~20/RELEASE_X86_64 x86_64
ProductVersion:	10.15.2

Checking requirements...

WARNING: "gcc" is not installed.

Do you want to install it now? (accepting to install "gcc" will also install "brew" in case it is not installed already)? [y]es/[n]o: y
Updating Homebrew...
==> Downloading https://homebrew.bintray.com/bottles-portable-ruby/portable-ruby-2.6.3.mavericks.bottle.tar.gz
######################################################################## 100.0%
==> Pouring portable-ruby-2.6.3.mavericks.bottle.tar.gz
==> Homebrew has enabled anonymous aggregate formulae and cask analytics.
Read the analytics documentation (and how to opt-out) here:
  https://docs.brew.sh/Analytics

xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
==> Auto-updated Homebrew!
Updated 3 taps (homebrew/core, homebrew/cask and homebrew/cask-drivers).`

[ removed part that detailed the new Homebrew update. Full error available here ]
xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
Error: The following formula
  gcc
cannot be installed as binary package and must be built from source.
Install the Command Line Tools:
  xcode-select --install

ERROR: Installation of "gcc" failed. Please contact SCT team for assistance.

Installation failed

Following this I ran xcode-select --install which opened Xcode installer to update it !

Same for me + updating xcode from the app store

ok, as long as Homebrew installer clearly indicates the solution (Install the Command Line Tools:
xcode-select --install), i think we don't need to do anything else. If people struggle, hopefully they will contact us...


Jobs disapper after completed/stopped

after a job is completed/stopped the job completely disappears and i can not review my results

Check this out: https://github.com/jmmcatee/cracklord/issues/82
If you used the repo to install cracklord, just grab the two files (jobsController.js, jobsView.html) that were changed and update them locally until they update the repo.  Alternatively, you could just grab the lines that were changed in the commit that fixed the issue (https://github.com/jmmcatee/cracklord/commit/bcec1348c9e37d81871b33e0eff5bfb6d1377034)

I'm going to go ahead and push a build so that this issue is taken care of for folks.  You should see the debs update in just a little bit.


#207 | create FAQs page

``/etc/hosts`` is not properly updated when I got errors, after ``$ vagrant destroy`` and ``$ vagrant up``. I end up having multiple same entry that the wrong one is one the top of the file. This leads to wrong DNS to point to the right VM IP. What should I do?
----------------------------------------------------------------------
WARNING: /opt/app/faqs.rst:47: (WARNING/2) Title underline too short.
``/etc/hosts`` is not properly updated when I got errors, after ``$ vagrant destroy`` and ``$ vagrant up``. I end up having multiple same entry that the wrong one is one the top of the file. This leads to wrong DNS to point to the right VM IP. What should I do?
----------------------------------------------------------------------
WARNING: /opt/app/faqs.rst:73: (WARNING/2) Title underline too short.
What should I do when meeting other  errors, and the problems cannot be solved with ``vagrant reload`` or ``vagrant reload --provision``?
-------------------------------------------
WARNING: /opt/app/faqs.rst:73: (WARNING/2) Title underline too short.
What should I do when meeting other  errors, and the problems cannot be solved with ``vagrant reload`` or ``vagrant reload --provision``?
-------------------------------------------``` from https://travis-ci.org/teracyhq/dev/jobs/255610597

@hoavt don't forget to fix these kinds of problems before sending new pull requests.

@hoatle Applied comments. Please check.

@hoatle applied your feedback. Please check.


Modify min required transformers version

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


BUILD FAILED - Task 'installDebug' not found in root project 'android'.

ExpoKit ejected app won't BUILD
* What went wrong:
Task 'installDebug' not found in root project 'android'. Some candidates are: 'installDevDebug'.

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>node v8.9.4
npm 5.7.1
yarn 1.5.1
watchman 4.9.0

i have hacked node_modules/react-native/local-cli/runAndroid/runAndroid.js with
    } else {
      gradleArgs.push('installDevDebug');
    }

so it runs in installDevDebug instead of installDebug.


Colored output for unittest command line.

This issue was originally filed by<EMAIL_ADDRESS>
It would be really great if unittest could output colored output. Red for fail, green for pass. That'd be really awesome.

This issue has been moved to dart-lang/test#208.


Fix Index recovery path for split(put)

1- fix old root change when new root buffer is being discarded and old root is dirtied
2- fix the wait down when the down_wait of a node for children is 0 and up buffer of the should not wait for this node to be repairedase enter the commit message for your changes. Lines starting

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 45.80645% with 84 lines in your changes missing coverage. Please review.

Project coverage is 65.75%. Comparing base (1a0cef8) to head (0701695).
Report is 102 commits behind head on master.

Files with missing lines
Patch %
Lines

src/lib/index/wb_cache.cpp
52.17%
36 Missing and 8 partials :warning:

src/include/homestore/index/index_table.hpp
20.00%
19 Missing and 1 partial :warning:

src/lib/index/index_cp.cpp
0.00%
9 Missing and 2 partials :warning:

src/lib/index/index_service.cpp
66.66%
8 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #609      +/-   ##
==========================================
+ Coverage   56.51%   65.75%   +9.24%
==========================================
  Files         108      109       +1
  Lines       10300    10922     +622
  Branches     1402     1502     +100
==========================================
+ Hits         5821     7182    +1361
+ Misses       3894     3034     -860
- Partials      585      706     +121

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


Build script also updates the vsix manifest version number.

Hows that?

One last thing:
node.Attributes.GetNamedItem("Version").Value <- version

should be
let currentVersion = node.Attributes.GetNamedItem("Version").Value
node.Attributes.GetNamedItem("Version").Value <- sprintf "%s.%s" currentVersion AppVeyor.AppVeyorEnvironment.BuildNumber

As it is, users have to uninstall builds from AppVeyor when the official one is released.

Would they? The build number for the released on would also be incremented.
Either way its done.

@FeodorFitsner Is there any bug on AppVeyor deployment? We see builds from other users and some pushes are not built at all.

Looking in that. Can you drop a link to a build please?

Feodor

On Wed, Oct 28, 2015 at 8:32 AM, Anh-Dung Phan<EMAIL_ADDRESS>wrote:

@FeodorFitsner Is there any bug on AppVeyor deployment? We see builds from other users and some pushes are not built at all.
Reply to this email directly or view it on GitHub:
https://github.com/fsprojects/VisualFSharpPowerTools/pull/1218#issuecomment-151883163

https://ci.appveyor.com/project/dungpa/visualfsharppowertools/history

and https://ci.appveyor.com/project/dungpa/visualfsharppowertools/build/1.0.1753

Cancel queued builds and start new one - it should be fine now. Please let me know about the results.

@FeodorFitsner Everything is ok now.

@TWith2Sugars Great, thanks.


Client Display: Level Breakdown's alert text is not visible when a theme is applied

Client Display: Level Breakdown's alert text is not visible when a theme is applied

Looks OBE:


Fix issue with payloads when using Krill on Java 8

The current master branch won't successfully execute the tests and will be unable to perform searches if Krill is executed on a Java 8 VM.
This seems to be related to a change in the semantics of splitting on empty strings which is explained in more detail in http://stackoverflow.com/a/27477312. The current code expects an empty string at the beginning of the array and skips it by using a static offset in the index. This pull request changes the RegEx so that both under Java 7 and Java 8 there is no empty string as first element in the splitted string. It also changes the indexes used to access the data to reflect the changes made to the RegEx.
I executed all tests cases both under OpenJDK 7 and OpenJDK 8 (both on Ubuntu Linux) and all test-cases succeed with these changes.

Thank you very much! That's great!


Enable DCO due to CNCF project requirements

Please install and enable for all repos: https://github.com/integration/dco

Done


[crashpad]Upgrade crashpad to latest version

Describe the pull request

What does your PR fix?

Updates crashpad. This fixes some bugs and also implements attachments for crash reports

Which triplets are supported/not supported? Have you updated the CI baseline?

No changes. Windows/mac. I've tested it on linux by also applying #14211 which sadly isn't getting merged.

Does your PR follow the maintainer guide?

Yes

@omartijn ,Please take a look :
ERROR at //third_party/mini_chromium/mini_chromium/build/config/BUILD.gn:435:15: Unknown substitution pattern
    command = "$cc -MMD -MF $depfile {{defines}} {{include_dirs}} {{framework_dirs}} {{cflags}} {{cflags_c}}${extra_cflags}${extra_cflags_c} -c {{source}} -o {{output}}"
              ^----------------------------------------------------------------------------------------------------------------------------------------------------------
Found a {{ at offset 57 and did not find a known substitution following it.
See //build/test.gni:43:7: which caused the file to be included.
      executable(target_name) {
      ^------------------------

Hmm, it looks like latest crashpad uses some newer ninja functions and the ninja coming with vcpkg is outdated.

@JonLiu1993 Turns out it was GN in need of updating, but only on Windows. I've added this change as a separate commit (so it can easily be cherry-picked if desired).
Please let me know if you would like me to split up the PR so that one PR updates GN only. I'm a bit conflicted on this since it's only a dependency, not an actual port.

@omartijn ，As you said, I suggest a separate PR to update GN, because in this regard, we will solve this update step by step.

@omartijn ，Are you still working on this pr?

Well, I am a bit stuck on upgrading gn in a different PR. That rebuilds almost the entire catalogue and that results in some errors I believe are unrelated.

Well, I am a bit stuck on upgrading gn in a different PR. That rebuilds almost the entire catalogue and that results in some errors I believe are unrelated.

@omartijn ，If it is not feasible to upgrade gn with a different pr, we still upgrade in the original way, as you said, it is not a port

@JonLiu1993 Shall I mark this PR as ready for review, then?

@JonLiu1993 Shall I mark this PR as ready for review, then?

It seems that CI was not successful. After I rerun, we will look at the specific errors.

@omartijn ,Thanks for your pr,I will close this pr for now, if you need to repair this pr, you can reopen it


Error when creating db using MariaDB

I was able to create the database without a problem using MySQL 5.7, but when using MariaDB 10, I keep getting the same error below. Not sure if it's a timeout issue or an actual bug causing the disconnection.
File "/root/.local/share/virtualenvs/ZeroScraper-NYgLjCjb/lib/python3.7/site-packages/pymysql/connections.py", line 707, in _read_bytes
CR.CR_SERVER_LOST, "Lost connection to MySQL server during query")
sqlalchemy.exc.OperationalError: (pymysql.err.OperationalError) (2013, 'Lost connection to MySQL server during query')
[SQL: ALTER TABLE Site ADD COLUMN airtable_id VARCHAR(256)]
(Background on this error at: http://sqlalche.me/e/e3q8)

To followup, I've found the issue consistently occurs in the same 3 files as documented below. (I've included only parts of the error read-out since it's quite lengthy, but can include the entirety if needed).
Error 1:
  File "/home/ubuntu/ZeroScraper2/migrations/versions/da6f10c8ebf4_add_site_airtable.py", line 21, in upgrade
    "Site", sa.Column("airtable_id", sa.String(256), nullable=True, unique=True)

Error 2:
  File "/home/ubuntu/ZeroScraper2/migrations/versions/95e1de28f5ba_add_info_columns.py", line 22, in upgrade
    op.add_column("Site", sa.Column("site_info", sa.JSON, nullable=False))

Error 3:
  File "/home/ubuntu/ZeroScraper2/migrations/versions/2f07cdaab83d_add_site_crawl_time.py", line 20, in upgrade
    op.add_column("Site", sa.Column("last_crawl_at", sa.Integer))

The lines in the error messages seem to finish before the process fails, so one workaround I've found is running pipenv run alembic upgrade head, commenting out the line in the error message and the preceding ones in the file that already ran, running it again, and so on for each error. (though for Error 1 almost all of the upgrade function seems to need to be commented out)
Based on searching online, I tried doing some basic attempts at troubleshooting what might be causing connection timeouts, though I wasn't able to successfully get any of them to work:

Increasing the max_allowed_packet size in /etc/mysql/my.cnf
Using set global thread_pool_idle_timeout to adjust the value
Adjusting the pools/pool recycling for SQLAlchemy

Again the issue doesn't occur when using MySQL, but it seems like using MariaDB may be necessary for this project due to issues using MySQL with the ArticleParser: https://github.com/disinfoRG/ArticleParser/issues/43


rqt_plot crashes when resizing the window

Steps to reproduce.

Type rqt_plot in a terminal window
Grab the bottom edge of the rqt_plot gui window with the mouse
Drag to resize.

I get this error:
qurrent@qurrent-HP-Z820-Workstation:~/catkin_ws$ rqt_plot /vrx/debug/wind/speed Traceback (most recent call last):   File "/usr/lib/python2.7/dist-packages/matplotlib/backends/backend_qt5agg.py", line 77, in paintEvent     self.resizeEvent(event)   File "/opt/ros/melodic/lib/python2.7/dist-packages/rqt_plot/data_plot/mat_data_plot.py", line 110, in resizeEvent     self.figure.tight_layout()   File "/usr/lib/python2.7/dist-packages/matplotlib/figure.py", line 2031, in tight_layout     self.subplots_adjust(**kwargs)   File "/usr/lib/python2.7/dist-packages/matplotlib/figure.py", line 1881, in subplots_adjust     self.subplotpars.update(*args, **kwargs)   File "/usr/lib/python2.7/dist-packages/matplotlib/figure.py", line 241, in update     raise ValueError('bottom cannot be >= top') ValueError: bottom cannot be >= top
OS: Ubuntu 18.04
Version: ros-melodic-rqt is already the newest version (0.5.1-1bionic.20200320.111301)
Any help here would be great!
Thanks,
Jonathan L Clark

Possibly related to this and #35, I see rqt_plot crashes for me on resize with python3.8, matplotlib-3.2.1, and ros2 eloquent. The error message I am seeing is:
Traceback (most recent call last):
  File "${ROS_WS}/install/rqt_plot/lib/python3.8/site-packages/rqt_plot/data_plot/mat_data_plot.py", line 109, in resizeEvent
    super(MatDataPlot.Canvas, self).resizeEvent(event)
  File "/usr/lib/python3.8/site-packages/matplotlib/backends/backend_qt5.py", line 397, in resizeEvent
    self.figure.set_size_inches(winch, hinch, forward=False)
  File "/usr/lib/python3.8/site-packages/matplotlib/figure.py", line 902, in set_size_inches
    raise ValueError(f'figure size must be positive finite not {size}')
ValueError: figure size must be positive finite not [9.33 0. ]

I am also seeing the issue described by Johnathan with the following setup

OS: Ubuntu:18.04,
Package: ros-melodic-rqt bionic,now 0.5.1-1bionic.20200320.111301 amd64
Backend: matplotlib

The error message was:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/matplotlib/backends/backend_qt5agg.py", line 77, in paintEvent
    self.resizeEvent(event)
  File "/opt/ros/melodic/lib/python2.7/dist-packages/rqt_plot/data_plot/mat_data_plot.py", line 110, in resizeEvent
    self.figure.tight_layout()
  File "/usr/lib/python2.7/dist-packages/matplotlib/figure.py", line 2031, in tight_layout
    self.subplots_adjust(**kwargs)
  File "/usr/lib/python2.7/dist-packages/matplotlib/figure.py", line 1881, in subplots_adjust
    self.subplotpars.update(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/matplotlib/figure.py", line 241, in update
    raise ValueError('bottom cannot be >= top')
ValueError: bottom cannot be >= top

I found that a workaround is to right-click on the rqt_plot window banner and maximize the window. Then right click again and un-maximize. Afterwards, the rqt_plot no longer seems to crash upon resizing.
This "fix" appears to persist across multiple re-launches.

After installing pyqtgraph via pip, rqt_plot is no longer crashing on resize (and is correctly displaying a graph at startup). I am still using the matplotlib interface for presentation.

After installing pyqtgraph via pip, rqt_plot is no longer crashing on resize (and is correctly displaying a graph at startup). I am still using the matplotlib interface for presentation.
Thank you very much. This fixed the issue for me also!

After installing pyqtgraph via pip, rqt_plot is no longer crashing on resize (and is correctly displaying a graph at startup). I am still using the matplotlib interface for presentation.

This also worked for me! I was running Ubuntu 18.04 and ROS melodic in a virtual machine in Oracle Virtual Box

This is a duplicate of #35, which was actually fixed in #52 nearly a year ago, becoming part of release >= 0.4.11.
However, this fix is still not released into Kinetic (stuck at 0.4.8) or Melodic (stuck at 0.4.9).
@dirk-thomas: Could you please prepare new releases, such that rqt_plot works out of the box with the system-installed matplotlib? Thanks.

ROS 1 branch with the fix in #69 has been released to kinetic, melodic, and noetic. Will be in the next respective syncs.
https://github.com/ros/rosdistro/pull/28396
https://github.com/ros/rosdistro/pull/28397
https://github.com/ros/rosdistro/pull/28398
I haven't looked at the ROS 2 fix yet but will try to soon.

This fix was released to ROS 1 quite a while ago, and I also just retested this on the latest ROS 2 Rolling.  It seems to not occur there.  With all of that said, I'm going to close this out, but please feel free to reopen if this is still occurring for you.


zkVM: enable proof of fault

This pull request enables proof of fault for the zkVM. When the executor is about to run into a failure (i.e. store/load from an out of bounds address), it terminates the program with a new exit code and executes a separate guest code called the fault checker as the next segment. The fault checker guest code takes the risc-v machine state and executes the next instruction to ensure that the instruction will indeed fail.
I marked this as a draft PR for now because I realized that I have a failure in the prover. I'll mark it as ready once I fix this but feel free to take a look in the meantime!

@flaub from talking to Jeremy, I made a mistake. I think the over all structure looks good but I need to add some changes to the fault checker guest and make slight adjustments to the verifier logic. I'll finish this and mark it as ready for review after that

Shoot - I realized that I need to edit the verifier logic for flat receipts but will do later tonight

The change for the fix resides mostly in https://github.com/risc0/risc0/blob/d3aaf32cacfd467981dc5382c456a89acafbaf2b/risc0/zkvm/src/mini_monitor.rs where I create the merkle tree of pages using the page containing the next instruction indicated by the pc and SYSTEM.start()

I think we should consider doing a smaller PR that is scoped to just the fault checker without any of the extra memory checking inside. Otherwise seems OK to evolve. I think there's potentially some cleanup to do afterwards but it's hard to tell from this side.

splitting this out into separate PR's as @flaub suggested. Closing


More tags for each place

Enhancement
As of V1.4, each place only has one tag. specifying whether it is a tourist attraction or a Michelin starred restaurant. We would like to introduce more tags to highlight the unique attribute each place provides.
Example draft

Chicken_Rice
Noodle
Top_Visited_2016
Newly_Opened

Also maybe

Cuisine
Nature/Shopping
Indoor/Outdoor


Mobile searches aren't starting

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

[X] I've cleared the sessions folder.

Title

[X] The title is no longer "Title" and I edited it with the right error name.

Branch
master
Commit
Latest
Describe the bug
Whenever i execute the script without the -v parameter, which i require considering i run the script in the background and the browser always pops up at the worst moments, the desktop searches are executed but the mobile searches throw an error.
Copy and paste your error
←[38;5;39m2024-08-14 15:53:06,880 [INFO] [BING] Finished Desktop Edge Bing searches !←[0m
DevTools listening on ws://<IP_ADDRESS>:51065/devtools/browser/a0823679-6fb8-4d71-97e3-ad30b17e30ac
[33560:1776:0814/155327.807:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.808:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.810:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.810:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.811:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.811:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.812:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.813:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.813:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.814:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.815:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.816:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.816:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.818:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.819:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.821:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.821:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
[33560:1776:0814/155327.822:ERROR:shared_image_manager.cc(223)] SharedImageManager::ProduceSkia: Trying to Produce a Skia representation from a non-existent mailbox.
←[38;5;226m2024-08-14 15:54:37,667 [WARNING] Failed to send to Discord notification: Bad Request - Unsupported Parameters., error=400.←[0m
←[38;5;196m2024-08-14 15:54:37,668 [ERROR] SessionNotCreatedException: Message: session not created: cannot connect to chrome at <IP_ADDRESS>:51084
from chrome not reachable
Stacktrace:
GetHandleVerifier [0x008D8923+23283]
(No symbol) [0x0089E934]
(No symbol) [0x007D05AD]
(No symbol) [0x007BF99C]
(No symbol) [0x008048F2]
(No symbol) [0x007FBBC0]
(No symbol) [0x007FBA17]
(No symbol) [0x0083D9ED]
(No symbol) [0x0083D21A]
(No symbol) [0x008341B6]
(No symbol) [0x00808017]
(No symbol) [0x0080890D]
GetHandleVerifier [0x009CA5F3+1013699]
GetHandleVerifier [0x009D3E4C+1052700]
GetHandleVerifier [0x009CD4B4+1025668]
GetHandleVerifier [0x008FEA2B+179195]
(No symbol) [0x008A6833]
(No symbol) [0x008A3198]
(No symbol) [0x008A3337]
(No symbol) [0x0089B4BE]
BaseThreadInitThunk [0x75EA7BA9+25]
RtlInitializeExceptionChain [0x7730C10B+107]
RtlClearBits [0x7730C08F+191]
←[0m
Traceback (most recent call last):
File "C:\Microsoft Rewards\MS-Rewards-Farmer\main.py", line 42, in main
earned_points = executeBot(currentAccount, args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Microsoft Rewards\MS-Rewards-Farmer\main.py", line 243, in executeBot
with Browser(mobile=True, account=currentAccount, args=args) as mobileBrowser:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Microsoft Rewards\MS-Rewards-Farmer\src\browser.py", line 42, in init
self.webdriver = self.browserSetup()
^^^^^^^^^^^^^^^^^^^
File "C:\Microsoft Rewards\MS-Rewards-Farmer\src\browser.py", line 92, in browserSetup
driver = webdriver.Chrome(
^^^^^^^^^^^^^^^^^
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\seleniumwire\undetected_chromedriver\webdriver.py", line 61, in init
super().init(*args, **kwargs)
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\undetected_chromedriver_init_.py", line 466, in init
super(Chrome, self).init(
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\selenium\webdriver\chrome\webdriver.py", line 45, in init
super().init(
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\selenium\webdriver\chromium\webdriver.py", line 66, in init
super().init(command_executor=executor, options=options)
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\selenium\webdriver\remote\webdriver.py", line 212, in init
self.start_session(capabilities)
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\undetected_chromedriver_init_.py", line 724, in start_session
super(selenium.webdriver.chrome.webdriver.WebDriver, self).start_session(
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\selenium\webdriver\remote\webdriver.py", line 299, in start_session
response = self.execute(Command.NEW_SESSION, caps)["value"]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\selenium\webdriver\remote\webdriver.py", line 354, in execute
self.error_handler.check_response(response)
File "C:\Users\user\AppData\Local\Programs\Python\Python312\Lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 229, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.SessionNotCreatedException: Message: session not created: cannot connect to chrome at <IP_ADDRESS>:51084
from chrome not reachable
Stacktrace:
GetHandleVerifier [0x008D8923+23283]
(No symbol) [0x0089E934]
(No symbol) [0x007D05AD]
(No symbol) [0x007BF99C]
(No symbol) [0x008048F2]
(No symbol) [0x007FBBC0]
(No symbol) [0x007FBA17]
(No symbol) [0x0083D9ED]
(No symbol) [0x0083D21A]
(No symbol) [0x008341B6]
(No symbol) [0x00808017]
(No symbol) [0x0080890D]
GetHandleVerifier [0x009CA5F3+1013699]
GetHandleVerifier [0x009D3E4C+1052700]
GetHandleVerifier [0x009CD4B4+1025668]
GetHandleVerifier [0x008FEA2B+179195]
(No symbol) [0x008A6833]
(No symbol) [0x008A3198]
(No symbol) [0x008A3337]
(No symbol) [0x0089B4BE]
BaseThreadInitThunk [0x75EA7BA9+25]
RtlInitializeExceptionChain [0x7730C10B+107]
RtlClearBits [0x7730C08F+191]
Screenshots
Can't take screenshots as the issues is only obtained without the -v parameter
Value of dashboard variable
NA

Close all Chrome instances (or restart computer and try again)

I already imagine this wouldn't work as i always taskkill all chrome process at the beginning and at the end of the script, but i wanted to restart the pc anyway and try it but it didn't change anything :(

Try something for me - go to chrome://flags and search for the word open - try the option d3d9 and see if it helps

Nothing changed, same error :(

same here but not smart enough to know if its for the same reason

Try the develop branch, should be working there


Formatting is not idempotent with Cpp11BracedListStyle

To reproduce:

Unselect Autoformat on changes (I don't know if this is related, it just makes it easier to separate the steps).

Paste in this code:
#include <stdbool.h>
#include <stdlib.h>

typedef struct {
    int kind;

    struct {
        bool is_low;
        double level;
    } power;
} event_t;

event_t* do_thing(bool flag, double voltage)
{
    if (flag) {
        event_t* power_event = malloc(sizeof *power_event);

        *power_event = (event_t) {
            .kind = 1,
            .power = {
                .is_low = true,
                .level = voltage,
            },
        };

        return power_event;
    }

    return NULL;
}

Click Config File and replace the entire contents with:
BasedOnStyle: WebKit
Language: Cpp
AlignAfterOpenBracket: BlockIndent
AlignOperands: AlignAfterOperator
BinPackArguments: false
BinPackParameters: false
BreakBeforeBinaryOperators: NonAssignment
Cpp11BracedListStyle: true

Click Load Config.

Click Format. Note that the brace after .power = gets put on a new line and indented.

Click Config File and replace the entire contents with the same config except the Cpp11BracedListStyle line. Click Load Config and Format.

Note that the .power = { brace is now back on the same line.

Click Config File and replace the entire contents with the same config as step 3, including the Cpp11BracedListStyle line. Click Load Config and Format.

The .power = { brace remains on the same line.

I would except that with the same source and the same config, steps 8 and 3 should produce the same results.
This happens with clang-format Version 16.0.6 and 17.0.6 selected.
Browser is Firefox 124.0 (64-bit) installed via Snap on Ubuntu 23.10.

@detly
I failed to reproduce step 5 on Manjaro both on Firefox 124.0.1 (64-bit) and on Chromium Version 123.0.6312.58 (Official Build) Arch Linux (64-bit) with a freshly loaded page:
The brace after .power =  stays on its line

GIF

How often can you reproduce this bug? It would be great if you could take a video demonstration.
Are you sure Autoformat on changes was off when you encountered this bug or pasted the code from step 2? If Autoformat on changes is on, code gets formatted with LLVM style and the brace after .power =  actually gets put on a new line and stays there when code is formated with your config file (with Cpp11BracedListStyle: true). This is closest scenario that follows your steps, and that's how clang-format works, I've double checked it on my local clang-format version 16.0.6

Anyway, I think the statefulness requires more than two or three steps, which is why I could reproduce it but you couldn't. I suspect it's actually related to the "penalty" parameters but haven't been able to narrow it down.

@detly

I must have done this about 20 times in a row last night, and it did what I described. I tried just now, and it did not.

I haven't updated anything on the server and frontend, so my only two guesses here are that either autoformat was on or this bug is state-dependent and is not reproducible on a freshly loaded page

Here's a video that's a little bit more laboured than I'd usually like to provide for a bug report, but it shows the issue.

Counting from the end is not very convenient, so I haven't followed the steps in the video, however this statement of yours is not true, and it might be the source of your misunderstanding:

The last set of settings should be the same as the third last, but do not change the position of the braces.

It should not produce the same result, since the formatted code is different. With Cpp11BracedListStyle: true the brace ALWAYS stays on the line it was before formatting, it just gets indented.

GIF

Ah I see what you mean. My mistake then was in leaving the result of previous formatting runs in the code pane, rather than pasting it in again in its original state. I had the idea that clang-format was a more "canonicalise every aspect of formatting" tool, like rustfmt, but it's not as aggressive as that. So, not an actual issue.


Unable to pair with iPhone

Hub shows in hue app and notification appears in HA but when enable link mode it doesn’t pair and then the hub is no longer found on searches after that.

Seems to work now with the latest update!

Doesn’t work for me with the latest update, bridge appears in hue app but will not pair.
I think it might be due to running nginx though as I think this requires port 443

the latest version should work better with connecting/pairing the app to the virtual bridge.
@timknowlden be aware that the virtual bridge really needs a dedicated port 80 and 443 to work.


[Office Hours] ZavaX Bridge October

Supported Kit and team during Avalanche Summit.

Oct 22nd sync. 1hr

Oct 29th Sync. 1hr


Support text along lines

Description
In utility GIS, there is a great need for text along pipelines while point labeling is unusable.
Example
Current:

With labels along line:

Alternatives
I currently use the following code, with which I add setPlacement('line') to the openlayers style:
setLineTextStyle(style) {
    const olStyleX = (feature, resolution) => {
      const applyTextPlacement = (x) => {
        if (x.getText() && feature.getGeometry().getType() === 'LineString') {
          x.getText().setPlacement('line');
        }
      };

      if (typeof style === 'function') {
        const s = style(feature, resolution);
        if (Array.isArray(s)) {
          s.forEach(applyTextPlacement);
        } else {
          applyTextPlacement(s);
        }
        return s;
      }

      if (Array.isArray(style)) {
        style.forEach(applyTextPlacement);
        return style;
      }

      applyTextPlacement(style);
      return style;
    };

    return olStyleX;
  }

Additional Context
I don't know how it would be possible to add setPlacement('line') for Mapbox, qGIS and SLD but for openlayers it is possible to do it with code similar to previous.

Hey @AlenKelemen,
thanks for your issue.
The support for text along lines was generally added to the GeoStyler with version 7.5.0.
@jansule added the support for the sld-parser just yesterday with version 5.3.0.
I guess the openlayers-parser should already support this since version 4.3.0
If it is not working for you as expected please provide some geostyler-style code.

Thanks for the quick reply, you are absolutely right.


Amplitude Logger [Error]: Converting circular structure to JSON errors occurring after updating to version 2

Expected Behavior

Amplitude Logger [Error]: Converting circular structure to JSON errors do not occur.
Current Behavior

On version 2.1.3, interacting with form elements that have the id attribute equal "name" will cause Amplitude Logger to log "Converting circular structure to JSON" errors when the user interacts with the form. Changing the id to any other string will resolve the issue.
<input id="name" type="text" maxlength="95" value="Name">

Possible Solution

It appears this issue only happens if the id is the string "name" on v2.1.3. This does not seem to be an issue on v1.11.0.
Steps to Reproduce

Using @amplitude/analytics-browser v2.1.3
Create a form that has an input where the id="name".
Add text to the input
Notice when the input is no longer focused the following error will appear:

Environment

JS SDK Version:
Installation Method: NPM
Browser and Version: Chrome 115

Hi @keegan-kelly-prometheum ,
Thanks for using amplitude, I tried the form interaction with this example. However, I'm not able to reproduce the circular issue. Do you mind sharing with me your code snippet related to the form?
Thanks.

@keegan-kelly-prometheum
I'm not able to reproduce with <input type="text" id="name" name="name" maxlength="95" value="Name"> neither.
I just merged the example. Please try to run the example here.

I think the issue is when you have an onSubmit handler which then fires an amplitude request.

@keegan-kelly-prometheum  @daniel-covelli  this issue should be fixed with the latest release, v2.2.3 with this fix.
I will close this issue now. Feel free to reopen the issue if still happening.
Thanks.


How should Turbo Navigator expose underlying delegate callbacks?

I sort of hinted at this in #62 but wanted to bring up a bigger discussion on the same issue.
Turbo Navigator is the SessionDelegate of each Session it manages because it needs to handle session(_:didProposeVisit:) to visit pages. It also provides sensible defaults for error handling, opening external URLs, etc. via a protocol extension. This gives developers the option to override which, if any, of these function they want to handle with custom logic.
However, not all of SessionDelegate is currently exposed. For example, I just found a need for sessionDidFinishFormSubmission(_:).
Should TurboNavigationDelegate expose all of the functions from SessionDelegate?
At first I thought that it definitely should. But then I started to wonder if we purposely shouldn't and provide some higher level API. Perhaps never even expose Session to the developer when using Turbo Navigator.
I ask the same question in #62 where Turbo Navigator assumes the role of the WKUIDelegate to handle confirmation dialogs. Should it also expose all functions of that protocol to the developer?
@olivaresf and @jayohms, I'd love your feedback on this. I feel like this is the last big question before I can open an upstream PR into turbo-ios.

A path forward was figured out in #74.


Prevent page from shifting down 3px when loading bar appears

It's all in the small details. Thank you very much.


deploy agents by SSH

It will be great to have a feature to deploy kontena agents by ssh, Any Linux OS with correct docker version must be valid.
Rancher has a similar feature, is very handy and save couple of extra steps.
As bonus, mostly any distro will work out-of-the-box
This will allow too, to deploy agents without internet connection (as in a internal network)
I find quite frustrating tha tI am "forced" to use a very specific ubuntu version + very specific docker version

Actually you can install master & agent just by starting docker containers with right run options.

@jalberto you can try with these commands (taken from cloudinit files)
https://gist.github.com/nevalla/c4b3362623c0e3e9f03d3122e530a4ff

Just noticed that docker run are missing -d options. Will update gist.

Docs have now instructions how to deploy Kontena with docker-compose:
https://www.kontena.io/docs/getting-started/installing/docker-compose
Using these instructions it should be pretty easy for users to setup Kontena anywhere.
Feel free to re-open in case something is missing.


Allow restore wallet with seed at anytime

Hey, could you please explain what you mean by this? I am not sure about what you are asking.

I am given a seed passphrase at the beginning of setup. I would like to
import my existing account into your wallet.
Regards,
Michael Sheldon Jr.
Uncle Mantis
I accept Ethereum and Bitcoin as payments through
unclemantis.crypto
On Wed, Feb 10, 2021 at 3:24 AM Léo Pradel<EMAIL_ADDRESS>wrote:

Hey, could you please explain what you mean by this? I am not sure about
what you are asking.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pradel/blockstack-wallet/issues/90#issuecomment-776531321,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFPO3P3RSXXXSQRJHPDB63S6I7CDANCNFSM4XMBEF7A
.

When you start the application for the first time, you should see 2 buttons:

First one "Create a new wallet": will generate a new seed phrase for you.
Second one "I already have a wallet": will allow you to import any seed phrase.

I would like the ability to enter a new phrase afterwards without the need
of Uninstalling/installing the APK. Just a thought.  Would be cool to have
multiple accounts too.
On Wed, Feb 10, 2021, 3:35 AM Léo Pradel<EMAIL_ADDRESS>wrote:

When you start the application for the first time, you should see 2
buttons:

First one "Create a new wallet": will generate a new seed phrase for
you.
Second one "I already have a wallet": will allow you to import any
seed phrase.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pradel/blockstack-wallet/issues/90#issuecomment-776537256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFPO3N56WLTQAMQOSLORSDS6JAOBANCNFSM4XMBEF7A
.

Maybe for now (as multiple accounts will be pretty hard to implement) I could add something to "reset the app" from the settings, removing all the config so you can import a wallet after this step.


Update of parts DB fails

I guess we reached the limits of GH actions runner disk space 😬
https://github.com/Bouni/kicad-jlcpcb-tools/actions/runs/9772117526/job/26975987280#step:6:519
sqlite3.OperationalError: database or disk is full
So, what can we do about this?

Let this run on another server and fetch the database from there. Quick fix
Refactor and build a API as discussed in #450 . Maybe the best way, but a lot of work

It seems that @yaqwsx 's jlcparts has the same issue, an that's the source for our parts db ...
https://github.com/yaqwsx/jlcparts/issues/132

Fixed by #498

@ukd1
Unfortuantely this still fails with no space left on the device ...
You can see it in the annotations here: https://github.com/Bouni/kicad-jlcpcb-tools/actions/runs/10005331382

I just searched for a solution for the disk space problem and stumbled upon this post: https://github.com/actions/runner-images/issues/2840#issuecomment-790492173
I gave it a try and it did the trick, the database files are built again successfully 🥳


REJECT-CLAIM-NOTIFICATION tasks are failing on test

env: test
Steps:

Submit and reject a claim

Expected:

receive rejection notification

Actual:

no notification

Seeing failed tasks in Test due to error calling Notify.
Kibana log:
{"@timestamp":"2017-02-01T15:20:36.519Z","message":"","tags":["bunyan"],"source":"test-wrk-1/node /usr/src/app/apvs-asynchronous-worker/start.js","level":"error","name":"asynchronous-worker","hostname":"test-wrk-1","pid":65410,"error":{"message":"400 - {\"message\":{\"template\":[\"Missing personalisation: first_name, requestInfoUrl\"]},\"result\":\"error\"}","name":"StatusCodeError","stack":"StatusCodeError: 400 - {\"message\":{\"template\":[\"Missing personalisation: first_name, requestInfoUrl\"]},\"result\":\"error\"}\n    at new StatusCodeError (/usr/src/app/apvs-asynchronous-worker/node_modules/request-promise-core/lib/errors.js:32:15)\n    at Request.plumbing.callback (/usr/src/app/apvs-asynchronous-worker/node_modules/request-promise-core/lib/plumbing.js:104:33)\n    at Request.RP$callback [as _callback] (/usr/src/app/apvs-asynchronous-worker/node_modules/request-promise-core/lib/plumbing.js:46:31)\n    at Request.self.callback (/usr/src/app/apvs-asynchronous-worker/node_modules/request/request.js:186:22)\n    at emitTwo (events.js:106:13)\n    at Request.emit (events.js:191:7)\n    at Request.<anonymous> (/usr/src/app/apvs-asynchronous-worker/node_modules/request/request.js:1081:10)\n    at emitOne (events.js:96:13)\n    at Request.emit (events.js:188:7)\n    at IncomingMessage.<anonymous> (/usr/src/app/apvs-asynchronous-worker/node_modules/request/request.js:1001:12)\n    at IncomingMessage.g (events.js:291:16)\n    at emitNone (events.js:91:20)\n    at IncomingMessage.emit (events.js:185:7)\n    at endReadableNT (_stream_readable.js:974:12)\n    at _combinedTickCallback (internal/process/next_tick.js:74:11)\n    at process._tickDomainCallback (internal/process/next_tick.js:122:9)"}}

The reason for this issue is changes that are currently being made to most of the email templates to include first name and the request info URL (see https://github.com/ministryofjustice/apvs-asynchronous-worker/pull/135).

Issue is caused by trying to use data module get-first-name-by-claimId with reference, so result is empty.
apvs-asynchronous-worker/app/services/workers/send-rejected-claim-notification.js ln8

Verified