Filter Ordering

Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

Some users would like to ratelimit request before authenticating them
Some would like to first authenticate requests, retrieve claims based on the JWT, and ratelimit based on that

To address such cases, we need to introduce the ability to specify a order of preference in case the default order needs to be overridden (lives in https://github.com/envoyproxy/gateway/blob/3f37499cef2c76589ccbcf83a77fef92257cf5da/internal/xds/translator/httpfilters.go#L87 )
[optional Relevant Links:]

Any extra documentation required to understand the issue.

This is blocking EnvoyExtensionPolicy.

closing as completed with https://github.com/envoyproxy/gateway/pull/3273


Fee changes are logged, but only that the change happened, not any more specific data about the fee

e.g. the value of the new fee. so fee config cannot be derived from the event of the transaction alone.
https://github.com/etherisc/gif-next/blob/develop/contracts/shared/ComponentService.sol#L474

found it https://github.com/etherisc/gif-next/blob/feature/logging-improvement/contracts/shared/ComponentService.sol#L595


Nothing to repeat at Regex (*SKIP)(*F)

I am using Regex this way
  var str = "r\(.*?;.*?\)(*SKIP)(*F)|;";
  RegExp _regExp = RegExp(str);
  html = html.replaceAllMapped(_regExp, (Match m) => ";\n");

Tried by putting
"r\\(.*?;.*?\\)(*SKIP)(*F)|;";
"r\\(.*?;.*?\\)(*SKIP)(*FAIL)|;";
but cannot be resolved.
working well while testing see here: https://regex101.com/r/x63m3p/1
how to resolve it. any bug or I couldn't figure out.
I want to figure out this thing: https://stackoverflow.com/questions/66076472/selecting-semi-colon-except-of-small-brackets
one person gave the answer but that regex not working on a dart.
I think (*SKIP) & (*FAIL) is not supported on dart it what.
can anyone resolve it?

You're using PCRE syntax in the regex101.com link. Dart use JavaScript regular expression syntax. It does not have something corresponding to *SKIP or *FAIL.
To avoid semicolons inside parentheses, I'd use a negative lookbehind instead.
Try:
var str = r"(?<!\([^)]*);";

(regex101)
That's probably not particularly efficient, so what I'd really do, to match your original behavior, is:
var re = RegExp(r"\(.*?\)|;");
html = html.replaceAllMapped(_regExp, (Match m) => m[0].startWith("(") ? m[0] : ";\n");

(That is: Just match the parenthesized groups, and then ignore them in the replace).

var html =
"""//exampleint g; String a="random";int j=0; for (int i = 1; i <= 10; ++i){
printf("Random");
}""";
so i want to break on before int except small brackets ()
Expected output:
//example
int g; String a="random";
int j=0; for (int i = 1; i <= 10; ++i){
    printf("Random");
}

except inside small brackets ...

Breaking before int, except inside parentheses. I'd do that as:
var re = RegExp(r"\(.*?\)|(?=int\b)");
html = html.replaceAllMapped(re, (Match m) => m[0].startsWith("(") ? m[0] : "\n");

I wanted to add \b before int too, but the "exampleintprevented that. I added it after to avoid breaking insideprintf, but that's just getting lucky (if it had been print`, it wouldn't have worked).

Sorry to bother you .. and maybe the last question from mine
#include <stdio.h>\n

int main() {
  int number1, number2, sum;
  printf("Enter two integers:\n ");
  scanf("%d %d", & number1, & number2);
  sum = number1 + number2;
  printf("%d + %d = %d", number1, number2, sum);
  return 0;
}

I want to replace the \n that are inside small brackets  this ->   printf("Enter two integers:\n ");  with \\n  and left others as it is.
actually, I don't know regex much.

You have the same issue of recognizing matching pairs of quotes, made worse by not being able to distinguish start and end quotes.
I'd probably do something like:
// Match string literals. (Not completely safe, might match `"` inside comments.
var re = RegExp(r'"(?:[^"\\\n]|\\[^])*"');
input = input.replaceAllMapped(re, (m) => m[0].replaceAll(r"\n", r"\\n"));

That is, try to find string literals, from " to the next matching ".
Then replace the string literal with itself, except replacing \n by \\n.
The input looks like C code, so I won't assume that string literals can be ' or """ quoted like in Dart.
They can be multiline, but only if a line ends in \.
There are still lots of ways that that can fail: A " inside a comment looks like it starts a comment.
The way to avoid that is to also detect comments:
// Match string literals and comments (// to EOL, /* to next */).
var re = RegExp(r'"(?:[^"\\n]|\[^])"|/(?:/.\n|*(?:[^]+*+?)*/');
input = input.replaceAllMapped(re, (m) {
var c = m[0];
if (c.startsWith("/")) return c; // it's a comment
return c.replaceAll(r"\n", r"\n"));
});

Probably massive overkill. :)

int main(){ int array[] = {2,3,4,5}; }
this is the code and i want to put {\n so after { line breaks
i want to put on only those
now i am doing
  html = html.replaceAll(") {", ") {\n");
  html = html.replaceAll("){", "){\n");
  html = html.replaceAll("}", "}\n");
  html = html.replaceAll("{", "{\n");

so I want to break line on ){ or ) { or {
except for these curly braces
{2,3,4,5};

Challenge accepted.
Your problem is that { and } can occur in expressions like {2,3,4,5} as well, so you have to somehow make a distinction between those that are statement blocks and those that are expressions. That's non-trivial, so let's go with a heuristic approach (something that usually works, but not always. I'm again assuming this is C or C++ syntax (so I won't try to match Dart syntax).
I think you actually want a newline before a final }, not after.
If it's C, then {2,3,4,5} is an array literal, and those can only occur after a =, and can therefore only be followed by , or ;,
except that they can be nested and comma separated, so a literal { can also follow a { or , and a } can be followed by a } or ,.
Since valid block can also have }}, like for (int i = 0; i < 10; i++) { while (true) {}} (notice last two chars), we can't generally treat } followed by } as an expression }. Heuristically, let's go with splitting at those.
// Any `{` not preceded by `=`, `{` or `m` and any `}` not followed by `;` or `,`..
var re = RegExp(r"(?<![={,]\s*)\{|\}(?!\s*[;,])");
// Add `\n` after matched `{`, or before matched or `}`.
var replacement = const {"{": "{\n", "}": "\n}"};
html = html.replaceAllMapped(re, (m) => replacement[m[0]]);

I'd probably also check that the brace isn't already followed/preceded by a newline.
// `{` not following `=`, `,` or `{` and not already followed by `\n`,
//  or `}` not followed by `;`, `,` or `}`, and not already preceded by `\n`.
// Or: whitespace at beginning of line.
var re = RegExp(r"(?<![={,]\s*)\{(?!\s*?\n)|(?<!\n\s*?)\}(?!\s*[;,])");

And then it's might be worth considering indentation, but that's a much bigger can of worms.


Implement Standalone 'bogl' Executable

Bring back the standalone bogl executable, without interfering with the existing build process.

Commit 4a4b72 added back in a simple command line interface for BoGL, allowing BoGL programs to be evaluated with a REPL separate from the web-based setup. To achieve this, 2 separate build targets were added: bogl and boglserver. Running stack build & stack install is sufficient to build and install both of the resulting binaries locally.
Running bogl from the terminal is likely to be the most helpful with a bogl program like so:
$ bogl Example.bgl
* parsed & typechecked TEMP.bgl
* Running BoGL Repl
>>>
let x = 24 in x + 12

 36

>>>

I'll put a note about how to use this more thoroughly in the documentation, when that's complete.


tools/svg2tvgt: handle scientific notation in float

This updates the svg2tvgt to handle scientific notation in floating-point values, which can exist, apparently. ;)

Backport of #20.


Draw Initial Hand Proof Integration

Losing my absolute sanity on this.
The problem: I can generate a zk proof for drawing the initial hand locally, I can also verify it locally.
However, on-chain the verification fails systematically.
I've setup an event that relays all the public inputs and the proof used on-chain. They are absolutely identical to the ones used off-chain.
It's to the point where I'm seriously suspecting some kind of snarkjs-side bug.

Fixed — I was sampling random values from [0, 256) instead of bounding them to the field prime. There probably is a bug in Snarkjs that makes it possible to prove & verify with values that are too large: https://github.com/iden3/snarkjs/issues/436


i18n: update dt cn translations

fix missing ex1 & dungeon translations, minor changes on ex2, r3n, add r4n translations. add relative directions output to current dawntrail triggers.

I think it would be better to add dedicated option to automatically translate the directions respectively though.
This PR looks fine now, but I would clean all of the text with directions later.

I think it would be better to add dedicated option to automatically translate the directions respectively though. This PR looks fine now, but I would clean all of the text with directions later.

I think so too, need several options for users to choose is better.


refactor: removed all the froms and made the pid maps

Closes #665

We'll be going a different route with this change. Thanks for taking care of this!


victims link

victims link is not generation on blackeye

https://github.com/UndeadSec/SocialFish/issues/360#issue-1899853898

Dm seve.services on instagram and we will help you with your issue

**Hi 👽 ! If you are interested in ⚙Advanced Phishing⚙ and the ✨Evilginx3✨ tool then I offer you Google Gmail and Custom Phishlets (Paypal, Cryptos...) with a ✅Free Tutorial✅ which arrives if the 📈Telegram reaches 50 Subscribers📈
⏳ GOOGLE GMAIL PHISHLET 2024 WITH PROOF ⏳
->
⛔https://www.youtube.com/watch?v=nA47UknIE6A⛔
🔓 Ask your questions and a free tutorial from 50 subs on telegram !!!
->
📩https://t.me/Noremorse1011📩**


Add capability to write boot.firm to NAND as well as SD

Merging this commit will allow lumaupdate to write the latest boot.firm to both NAND and SD.

Might need some work to make backups of older boot.firm before trying to update it.

Did you test this first? I just made the CIA and tested it, and I got "Reason for failure: CTR-NAND Failure". Thanks for the code, though!

Tried.Works fine for me.

Might be some permission issue. Try adding this in the rsf under FileSystemAccesss:-

Core
CtrNandRo
CtrNandRw
CtrNandRoWrite

That didn't work unfortunately. I'm still getting the same error.

Sorry for the late reply

Do you happen to know how to go about backing up and restoring the boot.firm on CTRNAND?

We could backup the boot.firm in CTRNAND to memory, and if download failes due to some reason we could just write the backed up boot.firm in memory to CTR-NAND.(I'm trying to do this, will submit a PR if successful)

Would you also know anything about making writing to CTRNAND be optional?

Would you also know anything about making writing to CTRNAND be optional?
Can be done. Will pressing a combo of buttons be ok or do you want a separate option?

I like the button combination idea. BTW, I've disabled the CTRNAND code for the time being as people were unsure of writing directly to CTRNAND. Do you want me to enable it?

Do you want me to enable it?
Its better disabled until I get that backup thing to work.

I like the button combination idea

Excellent so which all keys should it be?

Should we make it be like Decrypt9 (and maybe GodMode9) with Left, Up, Right, Up, A or should we go with a smaller button combination like Left, Right, Down?

I would personally like a small key combination. How about L + R?

Update:  Backing up payload on CTR-NAND works now.

Alright Everything done.


update tests for latest node.js version

do not hard code error messages!!! they do change based on node.js vesion!

@millermedeiros, which version of node.js are the tests supposed to work on? I'm running node v4.2.2 and 5 tests fail on master with zero local changes.


Unbound: migrate misc/dot to grid view

This should be it without the migration php script.

ok cobow. if I can manage to get an inline grid working that would be my preference on top

Up to you, its from a year old branch when it was Unbound plus :)


UI defect: ideally, entire bingo card should be in view at smallest breakpoint, while having legible font size

Bingo card is cut off at 320/lower end of smallest break point. The user has to pinch/zoom or swipe to see end of card. Additionally, it's causing weird problems with the white header above it as it is not as wide as the card.
I played around with the css and this appeared to help the design at 320-767 (please don't judge me if this is incorrect way of doing this--this is my first time doing responsive/css :D)-
@media (min-width: 320px)
.bingoTile, .free-square {
max-width: 18vw;
font-size: .875em;
word-wrap: break-word;
}
The downside of this is that words will break into multiple lines at the lower end of the break point, but that's the trade off for having legible font and the card in a single view. : )
Preview of work using inspector:

decreased font so breaking doesn't happen as much as in screenshot.


Wrong sha3 ?

I am guessing it is due to usage of pysha3-1.0.2 rather than pysha3==0.3 (as I read in requirements.txt), but to get namehash to work properly I changed line 4 as follows:
from sha3 import keccak_256 as sha3_256
Not sure why sha3_256 and keccak_256 differ though

@TallJanSmit  you are a hero without a cape.
The requirements are
pyhsa >= 0.3 < 1.0.0
I think that from python 3.6 the pysha3 versions are greater than 1.0.0, so, for example, i tried the examples of the Readme and found this:
  File "test.py", line 1, in <module>
    from namehash import namehash
  File "/home/marcos/sandbox/venv/lib/python3.6/site-packages/namehash.py", line 24, in <module>
    assert codecs.encode(sha3(b''), 'hex') == b'c5d2460186f7233c927e7db2dcc703c0e500b653ca82273b7bfad8045d85a470'  # noqa
AssertionError

I guess there was a breaking modification on the codecs package or sha3 implementation

Is there any way to remove this assertion and make it work properly from a box? Have the same issue.


Expose method to remove a specific alert

In some case (when alert timeout is 0) it may be usefull to be able to remove a particular alert when an event occurs.
These changes allow to do so.

Coverage remained the same at 100.0% when pulling 944bfeeb423a504a2f3ea1bd02a0690689117cf7 on anthonyraymond:patch-1 into 5b93482193269968e43241b6fb53173ec9c91e30 on schiehll:master.

@schiehll any new ont this?

Sorry about the delay @anthonyraymond , I was really busy.
The code looks good. Thank you. I will be releasing a new version with it later today.

Np problem ;)
Thanks for the merge.


Clipping square and line by square causes crash

This code will cause a crash at line 1511 of clipper.engine.cpp because op_front is nullptr.
Clipper64 clipper;

Paths64 squares0;
Path64 square0 = MakePath( "6, 8, 4, 8, 4, 6, 6, 6" );
squares0.push_back( square0 );
clipper.AddClip( squares0 );

Paths64 squares1;
Path64 square1 = MakePath( "5, 5, 7, 5, 7, 7, 5, 7" );
squares1.push_back( square1 );
clipper.AddSubject( squares1 );

Paths64 lines;
Path64 line = MakePath( "5, 7, 5, 5" );
lines.push_back( line );
clipper.AddOpenSubject( lines );

PolyTree64 clippedPolygons;
Paths64 clippedLines;
clipper.Execute( ClipType::Difference, FillRule::EvenOdd, clippedPolygons, clippedLines );

Also, I noticed that clipper.h is dated 16 May 2022 even though it was updated recently.

Hi again Michael. I've found the problem and fixed it but upload tomorrow (as I'm off to bed now).

Hopefully fixed now 🤞😁.


Added hook for Ajax updateCartFormResponse

@adrian-stein Could you explain how you set up an AJAX enabled CartForm? For me, only the first update to the form works and subsequent updates won't work (eg. quantity values will never be applied), because fields are named differently. This is because the rendering of the AJAX response doesn't happen within the context of a CartForm and is thus missing the CartEditField.
I've got it working by changing the context to be the CartForm itself and by using $forTemplate as content in the CartTable template. But maybe you've got a setup that's better and works with the current implementation?

I have it with subsequent updates working. The only issue I am having which I am yet to fully address and find a solution is that on first load the table row uses CartEditField to create the the quanity and remove fields but after AJAX it renders with ShopQuantityField. Both work it just looks strange on update it adds extra features and changes things.
The main change I made was to Cart.ss as below
<% require themedCSS(cart,shop) %>

    <%include CartTable Editable=$Editable  %>

CartTable is the same I just extracted it into its own file as when it did the pushRegion method the whole cart was replaced inside itself. Passing Editable makes sure stays editable and I can keep the text boxes and remove checkboxes.
Due to the issue of the way the text boxes are rendered different on update I am open to a better solution that removes this issue. I just have not had the time to get around to finding one.
I am not qute sure what you mean when you say by "using $forTemplate as content in the CartTable template"?

@adrian-stein Thanks for the reply. I think the whole thing could be simplified quite a bit. In my opinion it would be optimal if the HTML output stays the same, whether it's an AJAX call or a normal HTTP request.
I even found a way that makes the CartTable template obsolete. My current solution looks like this:
public function updateCartFormResponse(&$request, &$response, $form=null) {
    if ($request->isAjax()) {
        if (!$response) $response = $this->owner->getAjaxResponse();
        $this->setupRenderContexts($response);

        $data['Editable'] = true;
        $response->pushRegion('Cart', $this->owner, $data);

        if(self::config()->show_ajax_messages) {
            $response->triggerEvent('statusmessage', array(
                'content'   => $form->Message(),
                'type'      => $form->MessageType(),
            ));
            $form->clearMessage();
        }

        ShoppingCart::curr()->calculate();
    }
}

So instead of using $this->owner->cart as context (which is the current Order), I think it makes sense to use the CartForm instead. So just $this->owner. Then you can simply use 'Cart' as the template which is already used by CartForm for rendering.
Seems to work really well…

Looks like I was too quick with my assumption. The CartTable is still needed.
$response->pushRegion('CartTable', $this->owner, $data);

And then in CartTable just put the following:
$forTemplate

I can get behind this idea. Although if we are now returning CartForm and not just the table we need to move the clear message before the pushRegion so the rendered form HTML does not contain the message, as below.
	public function updateCartFormResponse(&$request, &$response, $form=null) {
		if ($request->isAjax()) {
			if (!$response) $response = $this->owner->getAjaxResponse();
			$this->setupRenderContexts($response);

			if(self::config()->show_ajax_messages) {
				$response->triggerEvent('statusmessage', array(
					'content'   => $form->Message(),
					'type'      => $form->MessageType(),
				));
				$this->owner->clearMessage();
			}

			$data['Editable'] = true;
			$response->pushRegion('CartTable', $this->owner, $data);

			ShoppingCart::curr()->calculate();
		}
	}

Also maybe we should add the cartTable.ss into the project so that people can use it as there is no documentation about how it works. Thoughts?
I am not sure about you but I have the following as the cart section on CartPage.ss. People will need to know where to add the data-ajax-region="CartTable" part as it is not obvious when I did it. I am not sure if we should include this or update the docs or request a PR somewhere. Ideas?

    <% if Cart %>
        <% if CartForm %>
            $CartForm
        <% else %>

                <% with Cart %><% include Cart Editable=true %><% end_with %>

        <% end_if %>
    <% else %>

<% _t('CartPage.ss.CARTEMPTY','Your cart is empty.') %>

    <% end_if %>

Sounds good. I think adding the CartTable.ss to the project is a good idea.
Regarding the status-message: You're right. I would even go as far and remove it even if show_ajax_messages is false. Eg.
public function updateCartFormResponse(&$request, &$response, $form=null) {
    if ($request->isAjax()) {
        if (!$response) $response = $this->owner->getAjaxResponse();
        $this->setupRenderContexts($response);

        if(self::config()->show_ajax_messages) {
            $response->triggerEvent('statusmessage', array(
                'content'   => $form->Message(),
                'type'      => $form->MessageType(),
            ));
        }

        $form->clearMessage();
        $response->pushRegion('CartTable', $this->owner, array('Editable' => true));

        ShoppingCart::curr()->calculate();
    }
}

I think the cleanest way to add the data-ajax-region is to simply add it to the form in the ShoppingCartAjax Extension. Like so:
public function updateCartForm(&$form, $cart) {
    $form->addExtraClass('ajax');
    $form->setAttribute('data-ajax-region','CartTable');
}

If we did it that way, there's not much to do for the user except installing the module to get an AJAX enabled cart.

There's another issue with the current updateCartFormResponse implementation. Recalculation of the cart should happen before pushing the region.
Here's my current version. I've also renamed CartTable to CartFormAjax:
public function updateCartFormResponse(&$request, &$response, $form=null) {
    if ($request->isAjax()) {
        if (!$response) $response = $this->owner->getAjaxResponse();
        $this->setupRenderContexts($response);

        if(self::config()->show_ajax_messages) {
            $response->triggerEvent('statusmessage', array(
                'content'   => $form->Message(),
                'type'      => $form->MessageType(),
            ));
            $form->clearMessage();
        }

        ShoppingCart::curr()->calculate();

        $response->pushRegion('CartFormAjax', $this->owner, array('Editable' => true));
    }
}

This is great, guys. I'll trust your judgment on where to put this stuff. If it's strictly related to the ajax implementation lets keep it in shop-ajax though.

All the changes could go into shop-ajax. If Adrian agrees with the changes, I'd gladly create a PR for this.

It all sounds good except one problem. I have a quick add option on my cart page where people can enter a product number into a text box and add the product directly into the cart from the cart page it self. The issue with putting data-ajax-region onto the form is that if there is nothing in the cart the cart page does not render the form and thus not the attribute and then the cart with the added product does not have a region to put its html.
I was just testing your code with my setup so not sure if it is my set up, me mucking something up or the new code but im noticing now that the element with the attribute on is not getting replaced but instead having the html that is returned from the AJAX call being appending to it. https://github.com/markguinn/silverstripe-ajax/blob/master/javascript/base.js#L153 it appending so I get the complete rendered form with tags as a child of the original form tags which are not overwritten.

I guess the quick-add option isn't part of the form itself then? So it won't work with a form-submit (eg. Update cart) when JS is disabled? If that's the case, I'd treat it as a separate element all together and trigger updates of the cart with it?
In my setup it replaces the contents of the form with new content just fine (no duplicates or anything). I'll test with a vanilla install of shop too to make sure. I think the primary goal here should be to provide a drop-in module for an AJAX enabled shop. Users can then still dig deeper and customize/add functionality.

I just tested with a vanilla shop install, using:
"silverstripe/cms": "^3.2"
"burnbright/silverstripe-shop": "dev-develop"
"markguinn/silverstripe-shop-ajax": "dev-master"
AJAX updates of the cart work out of the box with the changes applied that were discussed in this thread (eg. added a separate template to render the form and updated the methods in ShoppingCartAjax).
I suggest we go ahead with this change and then try to figure out where Adrians problems arise?

Ok sounds good to me.

@bummzack Finally back on this cart thing.
Just checking what happens with your project when you remove all the items from your cart. Does the Update cart button still appear?

@adrian-stein Yeah, the button is still there, with an empty cart message. When I refresh the page, the button is gone.

Cool. I was just making sure I was not going crazy. That still doesn't sit well with me so I might see if I can fix it if I get the time.

@adrian-stein It's not optimal, but I don't consider it a big flaw either. You get the "empty cart" message and when you try to enter checkout you'll get a "Your cart is empty" reminder as well.
There are several ways to fix this though. You could check the number of items in the cart and remove all unwanted form actions as soon the item-count reaches zero.
Here's an example that simply removes all form actions (to be put into updateCartFormResponse):
if($items = $this->owner->Fields()->fieldByName('Items')){
	if($items->getItemsList()->count() == 0){
		while($this->owner->Actions()->count() > 0){
			$this->owner->Actions()->pop();
		}
	}
}

This will still render the "Proceed to Checkout" link, since this is part of the outer template (CartPage.ss). Fixing all this would probably require quite a lot of template refactoring and is probably not a good idea for backwards compatibility… but it could be done in the 2.x branch.


SQLite's WITH RECURSIVE trips up the explain parsing in the macros

Based on this discussion on Discord: https://discord.com/channels/665528275556106240/665528275556106243/867867977591160841
The macros panic on a query using WITH RECURSIVE at this line in explain.rs: https://github.com/launchbadge/sqlx/blob/master/sqlx-core/src/sqlite/connection/explain.rs#L215
With the panic message being no entry found for key, which means for some reason that p1 doesn't exist in r_cursor.

A minimal repo to reproduce this is available here : https://github.com/Leon-MI/sqlx_run_with_panic.git


refactor: registry & deploy changes

Added TRAEFIK_PORT and TRAEFIK_TLS_PORT to dagent.
Fixed a bug which caused dagent failing to pull an image which is not in the default library on Docker Hub.

Codecov Report
Base: 7.64% // Head: 8.67% // Increases project coverage by +1.02% :tada:

Coverage data is based on head (8e9c938) compared to base (6fba038).
Patch coverage: 0.93% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop    #239      +/-   ##
==========================================
+ Coverage     7.64%   8.67%   +1.02%
==========================================
  Files           44      44
  Lines         3675    3771      +96
==========================================
+ Hits           281     327      +46
- Misses        3390    3432      +42
- Partials         4      12       +8

Impacted Files
Coverage Δ

agent/internal/grpc/grpc.go
0.00% <0.00%> (ø)

agent/pkg/crane/crane.go
0.00% <0.00%> (ø)

agent/pkg/crane/crux/deploy.go
0.00% <0.00%> (ø)

agent/pkg/crane/k8s/secret.go
0.00% <0.00%> (ø)

agent/pkg/dagent/dagent.go
0.00% <0.00%> (ø)

agent/pkg/dagent/utils/container.go
0.00% <0.00%> (ø)

agent/pkg/dagent/utils/docker.go
11.61% <0.00%> (-0.70%)
:arrow_down:

agent/pkg/dagent/utils/labels.go
64.28% <0.00%> (-35.72%)
:arrow_down:

agent/pkg/builder/container/utils.go
34.67% <100.00%> (+26.54%)
:arrow_up:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Closing, changes are merged with another task.


New package: Wesang.Yogiyo version 4.0.6

Resolves #182550

Manually verified in Windows Sandbox instance.

Verify:

The settings in the upper right are only printer settings. I couldn't find how to have the application display its version before login.


Trying to deploy to google functions but failed to find modules actions-on-google

Cd /googlecloud/functions
then
npm install actions-on-google
I got this

<EMAIL_ADDRESS>updated 1 package in 5.395s
And glcoud compents update  is already latest updated.

then I tried to deploy the code
gcloud beta functions deploy myfunctionname --stage-bucket myfunctionapibucket.appspot.com --trigger-http
Then It is response like this
Deploying function (may take a while - up to 2 minutes)...failed.
ERROR: (gcloud.beta.functions.deploy) OperationError: code=3, message=Function load error: Code in file index.js can't be loaded.
Did you list all required modules in the package.json dependencies?
Detailed stack trace: Error: Cannot find module 'actions-on-google'
at Function.Module._resolveFilename (module.js:476:15)
at Function.Module._load (module.js:424:25)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object. (/user_code/index.js:20:6)
at Module._compile (module.js:577:32)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
in the index.js code
I just did simple
const {
dialogflow,
Image,
} = require('actions-on-google')
// Create an app instance
const app = dialogflow()

if I remove those line esp require('actions-on-google') then my function work correctly.
Please let me know what is the exactly issues?
Thank

If you just type npm install, it will go to the package.json file and install the dependencies and save them in a local directory.

I did exactly like you said but when I deployed it is giving me the same errors
gcloud beta functions deploy myfunction --stage-bucket mybucket.appspot.com --trigger-http
Deploying function (may take a while - up to 2 minutes)...failed.
ERROR: (gcloud.beta.functions.deploy) OperationError: code=3, message=Function load error: Code in file index.js can't be loaded.
Did you list all required modules in the package.json dependencies?
Detailed stack trace: Error: Cannot find module 'actions-on-google'
at Function.Module._resolveFilename (module.js:476:15)
at Function.Module._load (module.js:424:25)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object. (/user_code/index.js:20:6)
at Module._compile (module.js:577:32)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
And here is my package.json
{
"private": true,
"repository": {
"private": true
},
"dependencies": {
"actions-on-google": "^2.1.3",
"update": "^0.7.4"
}
}
but it is locate in /my/project/package.json
and my index.js is located in /my/project/DialogFlowNodeJS/dev/index.js
Thank you

While you are in /my/project/ you should run npm install and then it should install everything. You should then see a /my/project/node_modules directory which will contain the downloaded code.

Yes I saw that node_modules so should I looking into actions-on-google folders inside node_modules?

more over I can cd node_modules/actions-on-google and do ls and show me all this packages.
LICENSE		README.md	dist		package.json	src		typedoc.json	yarn.lock

Do you need your index.js file to be nested within /my/project/DialogFlowNodeJS/dev/? If you keep everything in the top-level directory you should be able to deploy the index.js without issue.

Ok thank flex let me try to keep it in the same level and I will get back to you
Thanks

Yes it is working thank you.

Looking into account linking I just want to get user information from google assistant and I used this code but It never work
const {dialogflow} = require('actions-on-google');
const app = dialogflow({debug: true, clientId: "xxxxx"});
app.intent('ask_for_sign_in', (conv) => {
conv.ask(new SignIn());
});
// Create a Dialogflow intent with the actions_intent_SIGN_IN event
app.intent('ask_for_sign_in_confirmation', (conv, params, signin) => {
if (signin.status !== 'OK') {
return conv.ask('You need to sign in before using the app.');
}
return conv.ask('Great! Thanks for signing in.');
});
I already created ask_for_sign_in Intent and Action it as Signed In Required? Can you explain? and give me some full example?

You may want to make a new issue or ask your question on StackOverflow. I'm going to close this issue as the original issue has been resolved. Thanks.

Dear @chantrea999,
Have you resolved this issue ?
Please let us know how did you resolve it.
We are getting "TypeError: smarthome is not a function" error while accessing smarthome.


Move spark 3.0.1-shims out of snapshot-shims

Signed-off-by: gashen<EMAIL_ADDRESS>In shims/pom.xml, spark301 is already moved out of snapshot-shims profile.
In shims/aggregator/pom.xml, it should also be moved out.
Otherwise, the running on spark 3.0.1 will not find shims.

build

Thanks, I missed this on my other PR


highlighted row in kb obj

Hi,
I created a keyboard object and I want to have predict dictionary. since I am using hardware keyboard and I only have navigation and select key , to select the predicted text I want to use up/down keys as if the first row of the keyboard is selected and user presses the up key ( or the last row and down key) the predict list is selected but I don' know how can I do that and where this is handled ? maybe if I know that which row in the keyboard has been highlighted I can do this.

Hi,
It's a quite special thing which is difficult to handle by only external buttons.
Exactly which keys do you have?

I have only up/down/left/right and OK key. now I want to have a keyboard and once user typed a character I should search from a table and display a list of options that matches with the character ( same as address book) , see the below photo:
predict

once options are displayed user may wanted to selected from them , but all the keys are assigned to the keyboard. One solution can be using up/down key if one of the btns from the first row of the keyboard or last row have been selected ( if for example "`1#" is selected form keyboard pressing UP key do nothing) . I need to know which btn from the keyboard or at least which row currently has been selected and if it's the first row and user presses UP arrow key , I can focus other object rather than keyboard and user can move among them using Left/Right arrow keys and also retuns back to the keyboard.

Just to keep the threads consistent:
Similarly to Button matrix you can set a new map for the keyboard as well. Your new map should have an extra row on the top with the suggestions.
Maybe it would be better to use a Button matrix instead of the Keyboard because the Button matrix is more flexible.

Ok Thanks  I'll try this.

As there wasn't activity here for months I close this issue.
If you have a similar question feel free to comment here and reopen the issue!


[BUG] Multipliers -> Dino Multipliers, "Dino Character Health Recovery Multiplier" is listed twice

Describe the bug
Multipliers -> Dino Multipliers, "Dino Character Health Recovery Multiplier" is listed twice.
To Reproduce
Steps to reproduce the behavior:

Go to 'Multipliers' tab.
Expand 'Dino Multipliers'
2nd and 3rd slider and textbox have the same label. And the second "row" controls the value of the first row. The 3rd row controls both 2nd and 3rd though.
See error

Expected behavior
2nd row shouldn't be there.
Screenshots

Desktop (please complete the following information):

OS: [Windows 10]
App Version [e.g. 0.1.3]

Thanks for the report 👍


cache access token for the cloud token.

Store JWT access token instead of requesting it for every request.
We have encountered that the terraform client is requesting multiple access tokens when calling the public-api, A possible solution would be to store the token and only request a new one when its about to expire.

@sago2k8 #157 should help. You can now pass in a token directly instead of a client ID and secret, which gives you the ability to request and cache the token outside of Terraform.

@sago2k8 Could we close this issue now that #157 is merged?

I think so, closing now, I think we should maybe communicate to the customer so they start using it.


I know this sounds like a duplicate of https://github.com/UnwrittenFun/svelte-vscode/issues/39 that was closed a while ago, but I've implemented svelte.config.js and I really still do have this issue!
// svelte.config.js
const sveltePreprocess = require('svelte-preprocess');

module.exports = {
  preprocess: preprocess()
}

Here are my versions so we can try to narrow down why i'm facing this:

VSCode version:  1.40.0-insider
<EMAIL_ADDRESS><EMAIL_ADDRESS>macos sierra 10.12.6

as of time of writing, these are all latest versions except the operating system
what other info can i offer you to help? or can you point me to some suggestions to investigate why this doesnt work? im happy to spend a bit of time digging into this.

Hey @sw-yx 👋 A sample project would definitely help to understand the source of the problem 🙏

hey @kaisermann thanks for taking a look! my sample repo is here: https://github.com/sw-yx/sveltepreprocesstest (this is not the real repo i have problems with, but is a small reproduction to hopefully find the error.
what is really weird is that when i clone and install https://github.com/tonyrewin/svelte3-ts-boilerplate.git, i can see that typescript is working in there so it really just looks like something is going wrong only when i use svelte-preprocess (which seems to be the more popular one)?
i know this can be confusing to debug, please let me know what else i can do to debug so we can identify the source of the problem.

Thanks for your great description! Could it be related to your svelte.config.js using a undefined preprocess() method instead of sveltePreprocess() as you defined on the require() line? https://github.com/sw-yx/sveltepreprocesstest/blob/master/svelte.config.js#L6
I just cloned your repo, fixed this line and vscode stopped complaining 😁

ohhh. 🤦♂
erm ok i made a stupid mistake. i wonder if this vscode plugin can surface the obvious syntax error here?

Hehe 😆 I think that would be up to a linter or even vscode default js diagnostics 🤔


Encoding not specified

It says {encoded-sharing-url}. It doesn't work with urlEncode.
It should indicate which encoding to use. Base64, Base64Url, UrlEncoding, UrlFragmentEncoding, ... ?

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

ID: 09b8d398-f3d4-9808-2214-886fc1b42d86
Version Independent ID: c722986c-9322-301b-2ac9-c5b202803d9f
Content: Grant permission - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/permission-grant.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @learafa
Microsoft Alias: MSGraphDocsVteam

Hi,
Thanks for your feedback. There is now an example about how to encode the URL. I'm going to close this item. Thanks for your understanding.


Change cleverdox viewer

Hello Everybody.
There are anything possibilty change cleverboxview for another pdfviewer, because the new version of android not support this app.
thanks

you could play arround with two undocumented options:
options.android.viewerAppPackage = 'de.sitewaerts.cleverdox.viewer';
options.android.viewerAppActivity = 'DocumentViewerActivity';

Thanks regnete.
Sorry for the big delay.


Fix input variable name

Running the action right now causes a warning:

##[warning]Unexpected input 'githubToken', valid inputs are ['github-token']

I guess the input should be adjusted to githubToken, even though the action still works fine without that. But they may change that at some point. ^^

Thanks :-)


add express.js guard middleware

Note to self in prefer yo implement the middleware:
res.once('finish', () => exit)

done.


Make file cap/log persistance options public

Pardon my Swift-awkwardness - I just realized I forgot this when trying to integrate them into Buildasaur 😰 I suppose this requires another release...

Result of Integration 1
Duration: 1 hour, 24 seconds
Result: 0 error, failing state: internal-build-error

Result of Integration 2
Duration: 10 seconds
Result: Perfect build! All 14 tests passed. :+1:

No problem, making a release is just one fastlane command away :)

Done, here's the release: https://github.com/czechboy0/BuildaUtils/releases/tag/v0.2.6

Thanks. :+1: I'll submit a patch for Buildasaur hopefully later today when I get home from work.


IRGen: Only mark field offset globals as constant if they won't be updated at runtime

Even if we have a constant value, we might be emitting a legacy layout
that can be updated in place by newer runtimes. In this case, clients
cannot assume the field offsets are constant, and the globals cannot
be constant either.
Part of rdar://problem/17528739.

@swift-ci Please smoke test

@swift-ci Please smoke test


Updates to IXO chain

Is this just supposed to add a new grpc? @demondayza  or is Notional's meant to be removed?

Nice catch @JeremyParish69 and @ankurdotb , this has been corrected.


A test failure with a moose submodule update

The failure can be seen here: https://civet.inl.gov/job/1435246/. The test is using PJFNKMO, without using constant_matrices = true. I suspect the recent change I made in MOOSE (https://github.com/idaholab/moose/pull/23873) is causing this. The tolerance of that test is not very tight and the CSV diff is not high. If we confirm indeed it is that change causing the failure. I think we can safely regold the test.

Would recomputing the matrices too often have this effect? The matrices should have been constant every time?
anyhow I think looking at after / before the PR is enough to diagnose here, and regold is indeed the way to go

Evaluating matrices on every linear iteration is not good for sure in my opinion. I just verified that undoing the change, I can get
Users/wangy2/projects/gitlib/virtual_test_bed/htgr/pbmr400/steady.neutronics_only ........... RUNNING [82.73s]

I think we should just regold the file. I will tight the tolerance and do two runs again to make sure they will eventually converge to the same.

sounds good.
Careful with the runtime if you tighten the tolerance, that s usually why I limited it

I will not use the tight tolerance in the PR but will just to verify. I will also report the cpu time of the two runs. The one with fix should run much faster.

Closing as addressed by #217


Spigot

Hello there!
We are making a server for my old school and im wondering how i can install spigot??
Thanks!
Jaco.

There's a section for running Spigot type servers:
https://github.com/itzg/docker-minecraft-server#running-a-bukkitspigot-server
Let me know if you have any follow up questions or join the discussion over at Discord
https://discord.gg/DXfKpjB


How can I Improve on performance

I'm using Astro Airflow locally.
So I'm trying to execute DBT transformation models using airflow and find it to be extremely slow.
I'm looking for help to improve the performance of my airflow dag.
I'm trying to run 56 transformation models, I have done the following test:

DBT cloud: it completes my model transformation in 4 minutes

Testing Profile Locally: using  dbt run --profiles-dir /usr/local/airflow/include/dbt/ it completes my transformations in 6 minutes

AIRFLOW:: it takes  44 minutes and i get a fail

I'm looking for a way to run my models much faster.

Having very similar performance issue.

Some progress: #1014.

Hey Good day,
Sorry for the late reply but I was able to speed up my transformation workflow models.
In my case I used GCP - Composer. Things I done to speed up transformations was to update Airflow Configuration:

Configuration
New Value

dagbag_import_timeout
5000

Parallelism
32

max_active_task_per_dag
16

PR #1014 is significantly improving the performance with LoadMode.DBT_LS.

If using LoadMode.DBT_LS, please, could you try Cosmos 1.5.0a9, which will be released as a stable version this week?

https://github.com/astronomer/astronomer-cosmos/releases/tag/astronomer-cosmos-v1.5.0a9
https://pypi.org/project/astronomer-cosmos/1.5.0a9/

Some ways to improve the performance using Cosmos 1.4:
1. Can you pre-compile your dbt project?
If yes, this would remove this responsibility from the Airflow DAG processor, greatly impacting the DAG parsing time. You could try this by using and specifying the path to the manifest file:
DbtDag(
    ...,
    render_config=RenderConfig(
        load_method=LoadMode.DBT_MANIFEST
    )
)

More information: https://astronomer.github.io/astronomer-cosmos/configuration/parsing-methods.html#dbt-manifest
2. If you need to use LoadMode.DBT_LS, can you pre-install dbt dependencies in the Airflow scheduler and worker nodes?
If yes, this will avoid Cosmos having to run dbt deps all the time before running any dbt command, both in the scheduler and worker nodes. In that case, you should set:
DbtDag(
    ...,
    operator_args={"install_deps": False}
    render_config=RenderConfig(
        dbt_deps=False
    )
)

More info:
https://astronomer.github.io/astronomer-cosmos/configuration/render-config.html
3. If you need to use LoadMode.DBT_LS, is your dbt project large? Could you use selectors to select a subset?
jaffle_shop = DbtDag(
    render_config=RenderConfig(
        select=["path:analytics"],
    )
)

More info: https://astronomer.github.io/astronomer-cosmos/configuration/selecting-excluding.html

@tatiana, thank you for detailed explanations. They're helpful. Is using the manifest a recommended mode in terms of performance? If so will it still be the case after 1.5.0 is released?

@kzajaczkowski, it really depends on your team's needs!
Using manifest is the safest approach in production from the perspective you're fully off-loading the Airflow DAG processor from ever having to run the dbt ls command. As you know, the dbt ls command can take some time to run, and it is proportional to the size of the project.
We understand that it is handy not to have to pre-compile the dbt project for many teams. If it is acceptable for the team that from time to time, the Airflow DAG processor will have to run dbt ls and that the cache will be stored in Airflow Variables, then performance-wise, the 1.5 release is the best performance we managed to achieve with LoadMode.DBT_LS.
In the past weeks, we've collaborated with customers who tested every iteration of the 1.5 alphas. They will deploy them live once we release the production version, giving us confidence in the stability of what we've built.


Add files via upload

Project: IRIS Flower Classification
Time to Build: 3-4 hrs
Tech Stack: Machine Learning, Data Science

Please review my PR as this is one of my first Open Source Contributions. Sorry for the issue caused @rakshaa2000 .
Hope u understand.


Increase number of branch arms for either! macro

Increased number of possible branch arms for the either! macro from 6 to 8.

@gbj It seems that these changes haven't yet been included in the leptos release. It seems like leptos is still using 0.1.1 instead of 0.1.2.

@bicarlsen Ah thanks. I've just published 0.1.2, so cargo update should pull it in for you.


Encoded HTML ssr/browser rendering

Really enjoying the new SSR capabilities with bel! I wanted to render some Markdown output on the server recently, and with @yoshuawuyts’s help was able to get this working in both the browser and SSR by doing something like this:
var example = format('
some example

markdown output
')

function format (str) {
  if (typeof window !== 'undefined') {
    var wrapper = html`<div></div>`
    wrapper.innerHTML = str
    return wrapper
  } else {
    var wrapper = new String(str)
    wrapper.__encoded = true
    return wrapper
  }
}

And a little interactive example using Glitch. Super awesome, but a little verbose. Maybe there is a way to do this using a single unified method? #42

this is a really useful snippet! thanks for sharing ✨
just incase anyone takes this for a spin with choo, make sure you're using v6, as it won't be compatible with choo v5 (found this out the hard way 😅)

brilliant! spent a while digging into this issue tonight until I got down to pelo & saw what was happening. this works perfectly, should def go on the working with html section of the wiki :)

Would this be worth exporting as a bundled-but-separate utility function?

Yeah, think so - something like require('bel/unescape') or bel/raw
might make sense I think. Having it be a completely separate module might
make it trickier to find / adds more deps to projects which is not great
On Fri, Aug 4, 2017 at 7:14 PM Bret Comnes<EMAIL_ADDRESS>wrote:

Would this be worth exporting as a bundled-but-separate utility function?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shama/bel/issues/84#issuecomment-320303125, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACWlelu3VUSPqOl5tOw8EIikwFNr6Gckks5sU1DVgaJpZM4OV0mi
.

@yoshuawuyts Yeah, echoing that thought—was discussing this with @louiscenter as I’m using this method within several projects right now. Keeping it within the bel package but separate from the core would be a great move. I could make a PR for it?
Also, pretty fan of bel/raw. 3/3.

Think we all agree then. PR would be grand!
On Mon, Aug 7, 2017, 15:29 Jon-Kyle<EMAIL_ADDRESS>wrote:

@yoshuawuyts https://github.com/yoshuawuyts Yeah, echoing that
thought—was discussing this with @louiscenter
https://github.com/louiscenter as I’m using this method within several
projects right now. Keeping it within the bel package but separate from the
core would be a great move. I could make a PR for it?
Also, pretty fan of bel/raw. 3/3.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shama/bel/issues/84#issuecomment-320663561, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACWlelKnLA7u8-tnY3HYBZtMBVmYQNytks5sVxE4gaJpZM4OV0mi
.

https://github.com/choojs/nanohtml/blob/master/raw.js !


2 dotnet run command cannot work on ubuntu 14.04

First of all sorry my bad English.
My question is.. I cannot 2 dotnet run command on ubuntu. Please help me. I have a web project, running on ubuntu works perfect but now i added web api project but i cannot start 2 project at once.
Web api project program.cs file:
namespace KatmanWebAPI
{
    public class Program
    {
        public static void Main(string[] args)
        {
            var host = new WebHostBuilder()
                .UseKestrel()
                .UseContentRoot(Directory.GetCurrentDirectory())
                .UseIISIntegration()
                .UseStartup<Startup>()
                .UseUrls("http://localhost:5021")
                .Build();

            host.Run();
        }
    }
}

And then webui project Program.cs file:
namespace KatmanWebUI
{
    public class Program
    {
        public static void Main(string[] args)
        {
            var host = new WebHostBuilder()
                .UseKestrel()
                .UseContentRoot(Directory.GetCurrentDirectory())
                .UseIISIntegration()
                .UseStartup<Startup>()
                .UseUrls("http://localhost:5020")
                .Build();

            host.Run();
        }
    }
}

I enter in KatmanWebUI project folder and run dotnet run command works properly and going fine. Also i run this server 70 days without restart. But when runs KatmanWebUI project and then i am starting KatmanWebAPI project and then interfering each other. For example when starting a page from KatmanWebUI page and css is going down. Or some icons not showing and web site in not going properly.
My nginx file about server backend:
upstream backend {
        server <IP_ADDRESS>:5020;
        server <IP_ADDRESS>:5021;
        server <IP_ADDRESS>:19920;
}

So how can i start 2 projects run properly. System is Ubuntu 14.04 LTS Trusty.

/cc @danroth27 who can route this to the correct folks.

@tratcher @joshfree It sounds like you are having issues specifically with serving static files. Static files will by default be served from a wwwroot folder that is relative to the content root of the app. Both these projects are using the "current" directory for their content root. Perhaps the problem is that both apps are trying to serve static files from the same directory? That should in theory be just fine, unless there is some issue with the lower level file APIs.

Side note: Move UseUrls before UseIISIntegration.
I think your nginx upstream configuration is the problem. http://nginx.org/en/docs/http/ngx_http_upstream_module.html "By default, requests are distributed between the servers using a weighted round-robin balancing method." Nginx is routing requests randomly to your two different applications, so sometimes it cannot find the correct resources. You need to change your nginx config, or run a separate instance of nginx for each application.

Yess. Only KatmanWebUI project runs 70 days perfectly. So only 1 project is going properly. i am stopping KatmanWebUI project and run KatmanWebAPI web api project going perfectly. So 2 projects not work same time. So what will do set about nginx config?

Thank you very much all. I change a little change for nginx configuration and i solve this problem.
My nginx conf file is:
        location / {
        proxy_pass http://backend;
        proxy_set_header        Connection "";
        proxy_set_header        Host $host;
        proxy_set_header        X-Real-IP $remote_addr;
        proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_http_version 1.1;
        proxy_buffering off;
        proxy_buffer_size 8k;
        proxy_busy_buffers_size 8k;
        }

And i added ahother location name about webapi with this:
        location /api {
        proxy_pass http://<IP_ADDRESS>:5021;
        proxy_set_header        Connection "";
        proxy_set_header        Host $host;
        proxy_set_header        X-Real-IP $remote_addr;
        proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_http_version 1.1;
        proxy_buffering off;
        proxy_buffer_size 8k;
        proxy_busy_buffers_size 8k;
        }

And i deleted server <IP_ADDRESS>:5021; from backend:
upstream backend {
        server <IP_ADDRESS>:5020;
        server <IP_ADDRESS>:19920;
}

And problem was solved. Thanks.


remote validation does not work when server returns string

I follow official doc to use remote method:
http://jqueryvalidation.org/remote-method
and a demo of it:
http://jqueryvalidation.org/files/demo/milk/
input<EMAIL_ADDRESS>into email field should prompt me "That's already taken.", but it does not work.

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Update link.py and mpi.py to use Algorithm object

Summary
The link.py and mpi.py modules expect a Dict for the algo_config parameter, update these to be an instance of the Algorithm class in models.py.  We may also need to rewrite how the simple_mpi.py and simple_link.py use the algo_config, but only if those modules have been committed to main by the time this issue is worked on.
Acceptance Criteria

[ ] Update all references to the algo_config, and its parts, in link.py to use Algorithm
[ ] Update all references in mpi.py to using parts of an algo_config to using either an Algorithm or AlgorithmPass
[ ] update any test cases that specify the use of an algo_config Dict to an instance of Algorithm
[ ] Update the API in main.py to pass in an instance of Algorithm to the link_record_against_mpi function

Dependencies
#15

@cbrinson-rise8 I've been doing some testing on the new schema this week, https://github.com/CDCgov/RecordLinker/issues/62.  Its looking good so far, performance is ever so slightly better, we're addressing some edge cases that the old code doesn't (eg last name O'Reilly or O'Connor), and its going to be way easier to make some of the changes NBS is asking for down the road.
At this point, I'd say don't worry about making changes to recordlinker.linkage, just focus on the changes needed in recordlinker.linking.  The former packages is likely to be deleted soon.


Refactor common::TestCase::assertLocalizedHolidays

Proposed change

Refactor common::TestCase::assertLocalizedHolidays

Simplify argument passing requirements
Optimize tests execution
Update Monaco holidays l10n

Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

coverage: 100.0%. remained the same when pulling 107b4f0edfa99895d14bceb1840b4b82106f1687 on arkid15r:refactor-common-tests-assert-localized-holidays into bf54ecdeb526e082d90e076a204c162f4e4f1e61 on dr-prodigy:beta.


--train.epochs paramater not working

Bug description
Hi there 👋
First thanks for this awesome library you guys built!
I just have what I believe is a bug when using the litgpt pretrain command when you want to set a fixed number of epochs
I want to run a pretraining job on a custom dataset using the following command:
litgpt pretrain Phi-3.5-mini-instruct \
   --initial_checkpoint_dir checkpoints/microsoft/Phi-3.5-mini-instruct \
   --tokenizer_dir checkpoints/microsoft/Phi-3.5-mini-instruct \
   --out_dir ./new_pretrained_checkpoint \
   --data TextFiles \
   --data.train_data_path Text_files \
   --train.max_tokens 1_000_000 \
   --train.max_seq_length 1000 \
   --train.epochs 25

Here is the error we get:
ValueError: /home/zeus/miniconda3/envs/cloudspace/lib/python3.10/site-packages/litgpt/pretrain.py doesn't support the 'epochs' argument. This is set in TrainArgs(save_interval=1000, log_interval=1, global_batch_size=512, micro_batch_size=4, lr_warmup_steps=2000, lr_warmup_fraction=None, epochs=25, max_tokens=1000000, max_steps=None, max_seq_length=1000, tie_embeddings=False, max_norm=1.0, min_lr=4e-05)

I'm currently using Version: 0.5.3
What operating system are you using?
Linux
LitGPT Version
Version: 0.5.3

Apologies, I see that it hasn't been tackled yet and in the "TO DO"

Thanks for reporting, and yes, the support for fixing the number of epochs is not supported yet but will hopefully be added in the future.


Got 403 error when try to verify transaction

when I got the webhook response in production I try to verify the transaction using this
const flw = new Flutterwave(
FLW_PUBLIC_KEY,
FLW_SECRET_KEY
);

        const payload = { id: body?.data?.id };
        const response = await flw.Transaction.verify(payload);

verify transaction give me 403 forbidden error. I attached the screenshot
it's happening for the last two days. I use the test key in the production server
i use SDK
"flutterwave-node-v3": "^1.0.12",
version

i am closed because now i t's working fine after tomorrow server update.


Pesto demo for the Flutter Gallery app.

LGTM

Uploaded a new version with build functions split up a bit:

_buildAppBar and _buildDrawer to split up _PestoDemo's Scaffold
_RecipeCard is a widget instead of a _buildCard method
_RecipePage's build function split into a _RecipeSheet widget and a _buildContainer method.


Endless change of focus of trace viewer tabs when tabs are beside each other

After opening 2 traces, the trace viewer tabs are stacked behind each other. When moving one of the traces beside the other so that both are visible, then each tab gets focus and loses focus endlessly (see below). Because of this endless change of focus it's hard to  interact with one tab. The focus on the tab should be set by selection to one tab.

This needs to be fixed as per-requisite for #660.

I've been trying to Assign this issue to me, but GH seems like not caching or setting this properly so far.

I've been trying to Assign this issue to me, but GH seems like not caching or setting this properly so far.

Fixed by now it seems.


docker and sudo

The README.md describes how to run prometheus and friends using docker (I wish there was a way without it...) and mentions installing docker as a prerequisite but lacks some of the details on how to use it properly. Installing docker does not, out of the box, allows the user to user it. One also needs to start it, but this is a forgivable omission.
The bigger problem I had is that out of the box, without creating a "docker" group and adding yourself to it and restarting both docker and your login, one needs to run docker under "sudo". So the instructions to run ./start-all.sh will not work. One option is to run sudo ./start-all.sh and that should work. It would have been even nicer if the start-all.sh script itself could figure out whether docker works without sudo, and if it doesn't, try to run it with sudo (sudo is also not guaranteed to work, of course, maybe the user doesn't have sudo - but we can try).

@nyh running docker with sudo considered bad practice, among other it will create files as root.
About using docker, it's not a must, you can install prometheus, grafana and the alertmanager by yourself and just load the dashboard to them.
I do not want start-all.sh to use sudo, not all system that we are going to run on even have sudo.

@amnonh using sudo from scripts is always a bad practice, but it never stopped us before :-)
I think README.md should be more helpful for people who are not experts on everything under the sun. In particular, if you already tell people they must install docker, it is likely that if they do they won't know they need to do something special to actually run it.

I agree, I rather change the README than the script

@amnonh will you update the README?


Df.append() add index handling

@akharche  @densmirn  Does this PR affects documentation?

@AlexanderKalistratov Yes, I'm waiting for merging Maria's PR that fixes append documentation and then I do rebase with improving Limitation block in docs


testing: a second, better take on the call stack view for tests

This borrows some ideas (though no code, yet) from the multi diff
view to show call frame context inline. I think it still needs a little
polish, but I really like the concept. I also designed it as a reusable
component that I plan to use to enhance the debug exception widget.

https://github.com/microsoft/vscode/pull/222126


Stops working after rebuilding kextcache

The driver stops working after installing a kext to L/E and rebuilding kextcache. Reactivating the driver fixes the issue.

This is expected as the driver is locally signed.


fix: Use relative paths in manifest.json for compatibility with Github Pages

When deploying applications using Github Pages, and other scenarios, absolute paths don't work because the website is served under a subdirectory. But using relative paths should work in all scenarios where manifest.json, index.html and the icons folder are on the same location. Which should be the case with the generated files if they are not modified.
In case this is not accepted, it should at least be documented because the current setup is generating PWAs that don't have proper icons in Github Pages. This same thing happens with vuepress, so once this is resolved I'll do the same on the other repo.

Looks good to me.
By the way what do you think about prefixing relative paths with ./? It would be more clear no?

@Kocal If you think that's better I don't mind. For web resources I am used to declare relative paths without the ./, but it's true that it is more explicit so I guess it's an unfunded habit (writing less I guess, which is no real justification).
Should I commit that change then? If you confirm it I'll go ahead.

Well it's just my own opinion, as you said it's more explicit (and Explicit is better than Implicit :eyes:).
If it's improve readability and if your fix is still working, so yes it would be fine to commit that change, thank you :slightly_smiling_face:

@Kocal Ok done :+1:


Topicos Dezembro

Swan privacy update
https://www.nobsbitcoin.com/swan-bitcoin-privacy-update/

Blixt v0.6.9
https://github.com/hsjoberg/blixt-wallet/releases/tag/v0.6.9

Sparrow v1.8.0
https://github.com/sparrowwallet/sparrow/releases/tag/1.8.0

Buy eSIMs via LN
https://www.lnesim.com/

Mailing list sunsetting
https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2023-November/022134.html

HTLC aggregation with covenants
https://lists.linuxfoundation.org/pipermail/lightning-dev/2023-October/004181.html

Durabit - A Bitcoin-native Incentive Mechanism for Data Distribution
https://github.com/4de67a207019fd4d855ef0a188b4519c/Durabit/blob/main/Durabit - A Bitcoin-native Incentive Mechanism for Data Distribution.pdf

Fee Estimator updates from Validation Interface/CScheduler thread
https://github.com/bitcoin/bitcoin/pull/28368

bitcoin core v26.0rc-2
https://bitcoincore.org/bin/bitcoin-core-26.0/

BitEscrow: Non-Custodial Escrow for Bitcoin Transactions
https://www.nobsbitcoin.com/bitescrow-beta-launched/

85% Of Whirlpool Users are using self-custodial solutions
https://www.nobsbitcoin.com/85-of-whirlpool-users-are-using-self-hosted-solutions/

Mutiny Wallet adds Onion Message support
https://www.nobsbitcoin.com/mutiny-wallet-v0-4-28/

Kollider is shutting down
https://www.nobsbitcoin.com/kollider-is-shutting-down/

Replacement Cycling Attacks on the Lightning Network
https://github.com/ariard/mempool-research/blob/2023-10-replacement-paper/replacement-cycling.pdf
https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2023-October/021999.html

já discutimos em alguma edição sobre o fato de recentemente de >50% transações (às vezes mais) serem de ordinals? o tópico é polêmico e acho que pode gerar debates interessantes.

STARK proofs for BitVM execution:
https://github.com/neocarmack/STARK/blob/main/STARK proof for BitVM circuit execution.pdf

https://twitter.com/sunitram/status/1726263927783452955
https://github.com/bitcoin/bitcoin/pull/28913

sha256 function for bitvm
https://twitter.com/super_testnet/status/1726772975544807913

miningpool-observer project recently reported six OFAC-sanctioned transactions as missing from blocks
https://twitter.com/0xB10C/status/1726964430460588201

Opcodes explained
https://opcodeexplained.com/opcodes/

BBQr: Better Bitcoin QR
Coinkite proposed BBQr protocol and is looking for feedback. BBQr encodes larger files into a series of QR codes so they can cross air gaps.
https://www.nobsbitcoin.com/bbqr-better-bitcoin-qr/

bdk-ffi v1.0.0-alpha.2a release
https://github.com/bitcoindevkit/bdk-ffi/releases/tag/v1.0.0-alpha.2a

@a-moreira

já discutimos em alguma edição sobre o fato de recentemente de >50% transações (às vezes mais) serem de ordinals? o tópico é polêmico e acho que pode gerar debates interessantes.

@jaonoctus

miningpool-observer project recently reported six OFAC-sanctioned transactions as missing from blocks
https://twitter.com/0xB10C/status/1726964430460588201

a pool nova do Luke-jr censurando inscriptions ta jogando lenha nessa fogueira 🔥
https://twitter.com/sethforprivacy/status/1730048309622886827?t=_HYPzs8g8kpF9VA0ZXFidA

IPv6 Bitcoin-Certified Addresses
[...] In this paper, we present Bitcoin-Certified Addresses (BCA), a new technique to bind IPv6 addresses to hosts' public keys. BCA reduces the computational cost of generating addresses by using the PoW computed by Bitcoin nodes to secure the binding. Compared to CGA, BCA provides better protection against spoofing attacks and improves the privacy of hosts. Due to the decentralized nature of the Bitcoin network, BCA avoids reliance on a trusted authority, similar to CGA. BCA shows how the PoW computed by Bitcoin nodes can be reused, which saves costs for hosts and makes Bitcoin mining more efficient.
https://arxiv.org/abs/2311.15842


Use PyScaffold to set up a project structure

This is a WIP that is untested as of now.
Closes: #7
Closes: #8
Closes: #18

@VallariAg this should be ready for review! Let me know what you think.


Added Camera feature

See issue #29 for details

Do we need that separate "CameraShakeNames" class? I would do the conversion right inside Camera.cpp, less jumping around between source files.

Ya, you're right. I moved the names lookup to the Camera class, pull request is updated

Once this is finished, I want this to be merged as well. It would be awesome to combine vehicle mods and cameras.


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name (First, Last): Neal Robben
Where is your GitHub Repo (with your challenge solution): https://github.com/nealrobben/ColdStartChallenge2021_Challenge4
What Challenge is this for (challenge number between 1 and 6): 4
(Optional) Anything else we should know? e.g., language used, location, blog post?

Thanks for participating in our coding challenges ❤! To claim your Raspberry PI device, please DM me on Twitter to get the link for providing your shipping details. https://twitter.com/nicktrog


"100% reads removed: mapping on the negative strand" caused by discordant transcript IDs.

Thanks for developing this helpful toolkit. I found the message printed by this section of code is not very helpful for some special cases:
    if(transcript_align == TRUE | transcript_align == T){
      nreads <- nrow(dt)
      cat(sprintf("reads (total): %s M\n", format(round((nreads / 1e+06), 2), nsmall = 2)))
      dt <- dt[as.character(transcript) %in% as.character(annotation$transcript) & strand == "+"]
      cat(sprintf("%s M  (%s %%) reads removed: mapping on the negative strand\n",
                  format(round((nreads - nrow(dt))/ 1e+06, 2), nsmall = 2),
                  format(round(((nreads - nrow(dt))/nreads) * 100, 2), nsmall = 2) ))
    }

For example, I have ensembl transcript id in annotation files like ENSMUST00000195858. But the bam file has transcript ids with version numbers like  ENSMUST00000195858.1. This will caus all reads being filtered out with a message like "100% reads removed: mapping on the negative strand". However, the true reason is that the transcript ids don't match.
This sounds really critical but I hope you could consider this. A possible way is do this fitering in two steps and print more helpful message for users to debug.

Hi,
thank you for using riboWaltz and for reporting the issue.
I'm just wondering how come you have such a discrepancy between the transcript IDs in the annotation and the BAM file.
This issue shouldn't arise neither i) using Bowtie directly on a transcriptome FASTA and the corresponding (i.e. downloaded from the same repository) GTF file in downstream analyses, where the transcript IDs are supposed to match, nor ii) using STAR on a standard FASTA of chromosome sequences where an annotation file is required, and the same GTF is given as input to the create_annotation function.
Which tool did you use for aligning your data and where you get the FASTA and GTF you are using? Or are you employing txdb objects instead?
To solve the problem, I think I'll split bamtolist moving the filtering step to a dedicated function allowing, if needed, the removal of the version of the transcript from the IDs in the BAM file (assuming the transcript IDs would match after this additional step).
I really appreciate your tip about the two-steps filtering. Thus, if the change in the code described above is not what you meant, I ask you to elaborate the concept a bit (e.g. what the steps should be)  so I might try and modify the script in line with your suggestions.
Bests,
Fabio

Hi,
Thanks for the timely response.
how come you have such a discrepancy I downloaded the transcript sequences and gtf file both from here. The gtf file was supplied to create_annotation. The reads were mapped to Mus_musculus.GRCm38.cdna.all.fa.gz with bowtie2 and the transcript ids in the final bam file contains version numbers.
My suggestion is more like the simple code below:
dt <- dt[as.character(transcript) %in% as.character(annotation$transcript)]
cat('xx% reads are removed because their refence sequences are not included in annotation table')
nreads <- nrow(dt)
dt <- dt[strand == "+"]
...

i.e, tell the use how many reads were removed due to as.character(transcript) %in% as.character(annotation$transcript) and how many reads were removed due to strand == "+".  I don't think it is necessary for you to fix the ids. There are other ways to produce the same error. For example, I once provided a human gtf for mouse data and got the same error. I think a message telling user how many reads are removed due to  transcripts not found in annotation data would be enough. I hope I have made my points clear :).
Bests,
Hong

Hi Hong,
thank you for the clarifications.

I downloaded the transcript sequences and gtf file both from here.

I usually download the FASTA and GTF from GENCODE, where the two files refer to the mRNAa through the same IDs (that include the version of the transcripts). Thus, I didn't account for potential discrepancies in different repositories, but I will now.

telling user how many reads are removed due to transcripts not found in annotation data.

This is undoubtedly a good idea, I will add a dedicated message in a while.

I don't think it is necessary for you to fix the ids.

I thought to this option to avoid the user to modify either the FASTA or the GTF so that the IDs match (assuming them to refer to the same repository/release/organism).
I'll proceed with the last point as soon as possible. Meanwhile I am going to close the thread.
If other issues arise, please let me know.
Bests,
Fabio


[Bug]: Spotify Music Player Window hovers Appears Unexpectedly.

Issue Fixed.
Now when cursor approaches Spotify icon, then music player appears.

Closes #172

Good Work @faiz66388


Pull requests?

Hi @forty9er ,
Are you accepting PRs on this project? I'd love to get involved!
James

Hi @Jameskmonger, good question! I hadn't expected this to attract any such attention - I'm just building this to improve my JS/TDD and design skills.
What is the content of your proposed PRs? Improvements or contributing towards building the game?
I am keen to collaborate, and also interested in your motivation.

Hi @forty9er, thanks for your reply! I'm currently learning JS (and TypeScript) with TDD and so I'd love to get involved on a lightweight project like 21s to help me improve my skills.
I'd be happy to submit PRs with either improvements or content contributions.

OK, why not.
I've been slowly implementing features based on this description: https://github.com/guardian/pairing-tests/tree/master/21s.
It's probably not ideal, but I'm using the Jasmine Node package described here: http://jasmine.github.io/2.3/node.html. To run tests, run jasmine from the root directory.
Feel free to contribute however you feel and we can discuss.

Thanks! Will send a PR your way when I can! :smile:


Large BankIdUiSignState cookie

When the BankIdSignProperties contains large byte-arrays for UserVisibleData and UserNonVisibleData there is a chance that the properties are not persisted in the BankID UI state cookie.
What area is it related to
ActiveLogin.Authentication.BankId.AspNetCore.Sign.BankIdSignService
Describe the solution you'd like
Better documentation for the properties in BankIdSignProperties, that the size of the byte-arrays matters.
A validation of the cookie value before appending it to the response.
Additional context
Using theese props will fail the sign-request:
var props = new BankIdSignProperties(userVisibleContent)
{
    Items =
    {
        {"scheme", provider}
    },
    UserVisibleDataFormat = BankIdUserVisibleDataFormats.SimpleMarkdownV1,
    UserNonVisibleData = Encoding.UTF8.GetBytes(GenerateString(30000))
};

While the props with a hash works just fine:
var props = new BankIdSignProperties(userVisibleContent)
{
    Items =
    {
        {"scheme", provider}
    },
    UserVisibleDataFormat = BankIdUserVisibleDataFormats.SimpleMarkdownV1,
    UserNonVisibleData = BitConverter.GetBytes(GenerateString(30000).GetHashCode())
};

#425

#425


Feature/Filter-Results-ak-crystal

Proposed changes

Added two dropdown filter menu for Image & Video results on reportPage
Allows you to filter results by defect type

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] DevOps related

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the project guidelines doc
[x] I locally linted my code
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
Before Image/Video Filter

During Image/Video Filter

After Image/Video

Hey @idomo1 Damien, thanks for the feedback, made all changes mentioned above. Had a question regarding the 'classification', if no defect is found, is the 'classification' tag just empty?

@Akshith-Patil From what I understand, for video results "Can't place in space" = No defect. Classification will always be one of the 10 options


Add keys dates to our build to check if specs fail or not

Context
I am creating this PR to check if the specs will fail in important dates.
Running the tests simulating we are on these key dates:

1 day after Apply 2 deadline
1 day after Find opens (my birthday!)
1 day after Apply opens

Closing this because the purpose was to check if tests fails and they will. So there are cards to fix them


Contact Gallery button fade animation feels slow when it comes in the first time

To be clear, this is the first time after a cold start of the application. Once the application is running, the button shows up a lot sooner.

wait, what? why would the animation be slower on the first start?

is there any reason not to remove the animation all together? i don't think it adds much.

Think removing the animation is fine IMO

fixed by #479


Fix logic

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (yarn build) was run locally in root directory and any changes were pushed
[x] Lint (yarn lint:fix) was run locally in root directory and any changes were pushed

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: fix #451
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

@vimwitch do you think it can be merged?

Yes, lgtm


Issue with Lsp Snippet jump to next placeholder (clojure-lsp)

Describe the bug
Using Clojure-lsp, jumping to the next placeholder does not. When confirming a snippet, the first placeholder is highlighted okay but then tab just post 2 spaces.
The server returns
"(defn%s ${1:name} [$2]\n  $0)"

but after is selected and modified nothing works
Minimal config
https://gist.github.com/tami5/5da1dae368202898f8d408bef6ad60b3
To Reproduce

select a snippet item
overwrite first placeholder
tab

Expected behavior
recognize snippet placeholder

I think you want to follow this to set it up according to what snippet engine you are using

@jethrosun I believe I have set up this in the minimal config + this issue seems to happen in clojure-lsp only

I think this issue can be fixed https://github.com/hrsh7th/nvim-cmp/issues/513.

I think this issue can be fixed https://github.com/hrsh7th/nvim-cmp/issues/513.

Thanks @hrsh7th I will test it out in a bit

I think this issue can be fixed #513.

Thanks @hrsh7th, I will test it out in a bit

Nice no longer an issue. @hrsh7th  you are the best


ArcRest "arcrest.agol.services.FeatureService.createReplica" works on one ArcGIS Online organization but not another

ArcRest
I'm using the same exact script to create a replica and download a file geodatabase.  All works well on one organization, but as soon as I modify to hit a different organization, it fails with the following error.
Runtime error
Traceback (most recent call last):
File "", line 24, in
File "C:\Python27\ArcGIS10.5\lib\site-packages\arcrest\security\security.py", line 1173, in init
urlInfo=urlInfo._replace(netloc= "%s.%s" % (portalSelf.urlKey, portalSelf.customBaseUrl))
File "C:\Python27\ArcGIS10.5\lib\site-packages\arcrest\manageorg_portals.py", line 331, in urlKey
self.__init()
File "C:\Python27\ArcGIS10.5\lib\site-packages\arcrest\manageorg_portals.py", line 235, in __init
proxy_url=self._proxy_url)
File "C:\Python27\ArcGIS10.5\lib\site-packages\arcrest\web_base.py", line 686, in _get
context=ctx)
File "C:\Python27\ArcGIS10.5\Lib\urllib2.py", line 154, in urlopen
return opener.open(url, data, timeout)
File "C:\Python27\ArcGIS10.5\Lib\urllib2.py", line 429, in open
response = self._open(req, data)
File "C:\Python27\ArcGIS10.5\Lib\urllib2.py", line 447, in _open
'_open', req)
File "C:\Python27\ArcGIS10.5\Lib\urllib2.py", line 407, in _call_chain
result = func(*args)
File "C:\Python27\ArcGIS10.5\Lib\urllib2.py", line 1241, in https_open
context=self._context)
File "C:\Python27\ArcGIS10.5\Lib\urllib2.py", line 1198, in do_open
raise URLError(err)
URLError: <urlopen error [Errno 11001] getaddrinfo failed>
I can't figure out what's different between these organizations.  All settings look the same.  I can create replicas manually from these feature services.  Is there an ArcGIS Online setting that would cause this issue?

Bet it requires https.  Ensure the url to your org is https://www.arcgis.com
Sent from my Verizon Wireless 4G LTE DROID

Thanks for the response Mike.  I appreciate it very much.  Neither organization's url is https, so I didn't try that before, but I did try now just in case, but I get the same error.  On feature services... same thing.  I've tried them with http and https, but the error message does not change.

Did you ensure the org URL is https, not just the feature service.

I believe so, but the only way I know to do that is to check the box to only allow https communication, and then it changes the url of the organization from http to https?
From: Michael Miller<EMAIL_ADDRESS>Sent: Wednesday, August 23, 2017 2:31 PM
To: Esri/ArcREST<EMAIL_ADDRESS>Cc: Alison Walker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Esri/ArcREST] ArcRest "arcrest.agol.services.FeatureService.createReplica" works on one ArcGIS Online organization but not another (#356)
Did you ensure the org URL is https, not just the feature service.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Esri/ArcREST/issues/356#issuecomment-324453629, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ad1pGztbbvUI2YWOQjXcDOgegyoNFaR6ks5sbIwTgaJpZM4PAV-P.

Could you post your script?

Does this work?
AGOLpics - Copy.zip

Think you need to pass in the agolsecurityhandler the org url as https.
Sent from my Verizon Wireless 4G LTE DROID

Thank you Michael.  I’m going to try that.
… pardon my ignorance, do you have a sample of what that might look like?
From: Michael Miller<EMAIL_ADDRESS>Sent: Wednesday, August 23, 2017 6:51 PM
To: Esri/ArcREST<EMAIL_ADDRESS>Cc: Alison Walker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Esri/ArcREST] ArcRest "arcrest.agol.services.FeatureService.createReplica" works on one ArcGIS Online organization but not another (#356)
Think you need to pass in the agolsecurityhandler the org url as https.
Sent from my Verizon Wireless 4G LTE DROID
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Esri/ArcREST/issues/356#issuecomment-324501573, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ad1pG2vR--EW3-pGzVQtAKYExog5HCaJks5sbMjXgaJpZM4PAV-P.

Here is the line - https://github.com/Esri/ArcREST/blob/master/src/arcrest/security/security.py#L1112
but looking at it, it is defaulting to https.  So my next guess is the user name and password is wrong.  ArcRest is case sensitive for both the username and password.  Log into the organization with a browser and verify the username.  That produces this error.  This has caught a number of people.
I would also install fiddler and monitor the calls, looking at the response can help you figure out what is going on.

  You are absolutely right.  I had one letter lowercase when it needed to be capitalized, and voila!  Fixed.  But thank you so much for taking the time to take a look and point me in the right direction.  I probably would never have given it a second look.
From: Michael Miller<EMAIL_ADDRESS>Sent: Thursday, August 24, 2017 12:03 PM
To: Esri/ArcREST<EMAIL_ADDRESS>Cc: Alison Walker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Esri/ArcREST] ArcRest "arcrest.agol.services.FeatureService.createReplica" works on one ArcGIS Online organization but not another (#356)
Here is the line - https://github.com/Esri/ArcREST/blob/master/src/arcrest/security/security.py#L1112
but looking at it, it is defaulting to https. So my next guess is the user name and password is wrong. ArcRest is case sensitive for both the username and password. Log into the organization with a browser and verify the username. That produces this error. This has caught a number of people.
I would also install fiddler and monitor the calls, looking at the response can help you figure out what is going on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Esri/ArcREST/issues/356#issuecomment-324711703, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ad1pG0qIooUaq-wCttYZrWuEMERSD6Tqks5sbbrigaJpZM4PAV-P.


Add method for packages to extend command-line interface

I see a lot of issues and suggestions around people wanting new command-line options. It would be great if people could write their own command-line option handler. Perhaps something like this:

keymaps
lib

options

standalone.coffee

spec
styles

standalone.coffee contains:
module.exports = (arg) ->
  # Do something with the argument here

See: https://github.com/atom/atom/issues/1433#issuecomment-76732628

@lee-dohm Could you share more details and add an example + motivation?

Yeah, sorry ... I accidentally hit ENTER and was typing all that out :laughing:

:+1: Thanks, should have guessed that :smile:


Migrate devicelab tests running on VMs

Some of the devicelab tests are running using linux vms. These tests can run directly using luci linux bots once we have functionality to upload metrics from the test runner.

FYI for https://github.com/flutter/flutter/issues/71749 I created linux_web_benchmarks_canvaskit on LUCI for test purposes. After that issue is fixed, we'll need to migrate the upload_metrics property to be set based on the branch (like what wbh does)

I am going to monitor infra logs to look for linux vms still being used. If not, I am going to free the vm instances from flutter-infra. Also, they will need to be removed from agents datastore.

I turned off all vms in flutter-infra GCP. I am going to give soak time before clearing them and wiping them.
Initial results show nothing needing them now other than the agent datastore.

Linux VMs turned down


Podcast page

Fixed #200

please resize thumbnails and position as per the design

We do not need the drop downs on this page (select podcast and date)

This works thanks

Lgtm. @mrjamieeast Please check this works

Yup great

On 30 May 2022, at 10:05, Chris Nesbitt-Smith @.***> wrote:

Lgtm. @mrjamieeast Please check this works
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Overlay - Ask for permission before enabling service

This PR adds a layout in order to display whether the user has granted permission to the app in order to draw on top of other apps.
Screenshot

:+1: good stuff!

nice gif! :+1:

Comments have been addressed @xrigau @blundell

:shipit:


Confusion with css in project

In project we have 3 css files:

src/css/simplemde.css
debug/simplemde.css
build/simplemde.min.css

I'm doing my automated build and I want full (not minified) version. So I included 'src/css/simplemde.css'. That should work as it does in assortment of different libraries, right? No, it doesn't. Cursor doesn't appear and editor becomes mostly unusable.
The point is, you probably want to either toss both minified and full versions into build folder, or have full css in src folder. Because debug folder is not the place other people will be looking for stable bundle in the package.

@WesCossick I figured where files are located. The point is it's confusing for many people.
Take a look on where css is located in other libraries (e.g. font-awesome, bootstrap and many others).

I agree, would save us some time if the source files and their use were explained in the readme or so. Debug makes me think of incomplete files used for testing or as scratch files.


Improve blockquote linting

Ensure that blockquote linting is correct. See LBF22. Also address nested blockquotes (bountonw/translate#174).

I believe this was addressed with #19. Feel free to reopen if you notice any issues/regressions.


VS2019 Customise/override Typescript .exe location (using msbuild?).

Microsoft.NodejsTools.targets, Microsoft.NodejsToolsV2.targets, msbuild, VS2019, Ttypescript, Typescript, node_modules.
Suggestion
So I just switched over to using Visual Studio 2019 (from 2017) and found that creating a new Typescript web project uses Microsoft.NodejsToolsV2.targets instead of Microsoft.NodejsTools.targets which requires using a Typescript install in node_modules instead of the built-in SDK. This got me excited because I thought I'd be able to overwrite a variable to point to the custom wrapper for tsc that I'm using (ttypescript), but unfortunately not the case.
Currently it is possible to switch the build & clean version of tsc over to a custom location by creating your own .targets file and c# script to run tsc with the provided configuration and providing all the settings as appropriate (as I have done), however the tsc used by CompileOnSave is hardcoded into the SDK, and cannot be changed using msbuild. So I've lost all compile on save support (if I turn CompileOnSave on my output is overwritten by javascript that doesn't run).
So I'm requesting the ability to specify explicitly where tsc is located, so that I can use a custom wrapper instead of the default tsc in VS2019. I'm perfectly fine for this to be only available within msbuild or the project properties.
Use Cases
This will allow me to use the experimental typescript transforms prior to the official implementation within vs 2019 across my projects while still being able to use compile-on-save.
Examples
I don't know what to put here aside from what I've already said.
Checklist
My suggestion meets these guidelines:

[y] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[y] This wouldn't change the runtime behavior of existing JavaScript code
[y] This could be implemented without emitting different JS based on the types of the expressions
[y] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[?] This feature would agree with the rest of TypeScript's Design Goals.

Final note:
Whenever I raise these issues people tell me to switch to VS code. Please don't. I'm too dependant on the VS->Azure workflow to make the switch.

@PranavSenthilnathan @minestarks @jessetrinity?

I think there are a couple of points of confusion to clear up here.

Compile On Save in Visual Studio actually uses tsserver.js, not tsc.js - so unless you have a custom wrapper for tsserver too, it's not going to be possible to wire up your custom bits to Compile On Save
Sounds like you've succesfully hacked/modified the MSBuild files to pick up a custom compiler. But (except in a few supported scenarios) the contents of MSBuild project don't matter to how VS locates tsserver. If you do have a custom tsserver.js, you can use this unsupported registry key to point VS to it.

From the VS developer command prompt:
CD "%VSINSTALLDIR%"
vsregedit set . HKLM TypeScriptLanguageService CustomTSServerLocation string "full_path_to_my_custom_tsserver.js"

I'm going to re-emphasize that this setting is unsupported and its behavior may change in future versions of VS - but then so is using a custom tsc wrapper 🙂

Thanks for the quick reply, that was a lot faster than I was expecting. I do in fact have a custom tsserver..js! (I'm using ttypescript which generates both)
It's not clear from the key itself, but I'm guessing that changing that value changes the tsserver used for all projects?
I have two projects on the same PC which I'm actively working on and only one of them uses the changed tsc. Is there a way for me to change the tsserver for only that project?

@minestarks any chance on getting some clarification on this?

I would just try it, but I'm pretty sure you should be able to lock yourself to a specific TS version at a project-level granularity. https://github.com/Microsoft/TypeScript/wiki/Updating-TypeScript-in-Visual-Studio-2017

@DanielRosenwasser  the question was does using the custom registry key override all typescript versions, just one of them or create a new entry?
I don't like editing my registry on a whim, so I'd prefer to know in advance whether or not I can use it, particularly since I use this machine for work on a project that doesn't have an altered compiler, so it's pretty important that I know that that's not going to break.

The reg key should be removable if it doesn't work but I won't pressure you into changing it if you're not comfortable with that.
I'm not sure, @PranavSenthilnathan or @minestarks may be able to answer this.

Setting the key clobbers. I think you can also set the CustomTSServerLocation in the command line before launching VS a la https://github.com/microsoft/JSTSdocs/blob/master/articles/troubleshooting/index.md.
Since you have a custom tsserver though, you should be able to create your own "SDK version. Below is what I just tried and I would normally not recommend doing this, but since you have a custom tsserver I don't think I can stop you:

Back up your existing TypeScript SDK folders in C:\Program Files (x86)\Microsoft SDKs\TypeScript.
copy your latest SDK folder, probably 4.1.
Rename the copy to something ludicrous that is not likely to collide with an actual TS version anytime soon, say 6.6.
Copy your tsserver files into the 6.6 folder. I don't know what ttypescript does, but if you're working with the TypeScript repo you can just copy the contents of typescript\built\local.
Open a project in VS and you should see a 6.6 version when you select the version for your project. You're likely to see something break like localization, but I do see the right node process starting on my machine.

I would really like to add a menu to VS that just lets you restart the server, select version, etc...

Ooooh! Thank you @jessetrinity for the answer and the workaround. I'd love to see a menu like that too, but making a custom SDK works too.
Could I name it 0.1 since 0.8 was the first public build, so it shouldn't be able to ever conflict?


More subscription options

Not having subscription options for Apple/Stitcher/etc seems like a usability deficiency given how popular those platforms are

we made this choice based on several discussions with what our priorities are, and how we are working with RP on the player.


Added football2 and football3

Hey @dumbmatter , I added a few jersey designs for football jerseys. Haven't done code yet for the actual football-gm to support multiple jerseys, but will do so if you're still open to the idea.
Jersey 2

Jersey 3

Jersey 3

This is cool, thanks!
Even without adding support directly in BBGM/FBGM, someone could still use it for a custom player. But yeah, at some point, it would be good to allow teams to pick jerseys. There's already multiple basketball jerseys, but no support for picking one for a team in BBGM.


chore(main): release 2.3.2

:robot: I have created a release beep boop
2.3.2 (2023-06-04)
Bug Fixes

gradle: publishing contains wrong artifacts fixes #367 (e49f881)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/joke/spock-mockable/releases/tag/v2.3.2 :sunflower: