Poor lighting differentiation between chip & background

Low lighting sometimes seems to work best on some samples to differentiate between the chip being present and the background.
For instance, the chip and the background will both 'turn blue' (register as color entropy) at the same threshold values when a user can clearly see a difference still.
It's hard to describe the issue well, but the SEYR color analysis is sometimes able to differentiate between when it's plotting Blue vs Purple in the threshold map, but it'd be better if maybe the regions are separated a bit more somehow.
Very hard to send a sample image set for you to see what we mean.

hard to know exactly what to test - it seems like the feature might have too many different possible states - meaning it is no longer "present" and "not present", but it is "empty" and "mostly empty". am I on the right track?

okay - I made a simple but massive change
there is now a tool in the composer called "Entropy Balance" where you pick an RGB color
that color controls the levels of the image that go into the scoring routine:
default and legacy is always red (255, 0, 0).
if you make it black (0, 0, 0), you get no data.
if you make it white (255, 255, 255) you get a much smoother curve than legacy red.
you can make it any color in between (no transparency control, though)

Scroll bars for finetuning with default primary option buttons? (Red, Yellow, Blue, Green, etc..).
Need a way to test and compare when setting the recipe up.

standard windows color picker

Yeah! That works!

now the question... do you need one entropy balance per .seyr or one per feature?

much better values with a bit more blue

click the chroma button to change color, then border color reflects it

yes, but that reminds me I need to expose the default color so your team can change it in the XFER-SEYR panel
the exiting color is always Red, and that is the current default. But please test this in the demo .exe (new one coming soon)

For the color selector, is there a way to add an eyedropper so a user could click on a color on the screen to choose the color of the inspection chip? That could be really useful so they don't have to guess at the colors.

yes, that is possible - but I am not sure you will want the chroma to be the color of the device itself. will know more after testing the newest demo .exe

Changing the chroma color setting does not change the threshold map image on the left. Wouldn't the image on the left change if the color that was being scanned for changed?

Chroma Setting is set to purple by default, can we change it back to whatever color it was originally by default? I thought it was Red (255, 0, 0).

ya, that was not supposed to make it into the .exe, was just testing

Changing the chroma color setting does not change the threshold map image on the left. I feel like it should. Wouldn't the image on the left change if the color that was being scanned for changed?

ah, yeah, I cheat and don't actually use the scoring method to generate that image - it is kind of an estimation. I can update it, don't know how challenging/slow it will be

ah, yeah, I cheat and don't actually use the scoring method to generate that image - it is kind of an estimation. I can update it, don't know how challenging/slow it will be

It's a must have, otherwise the user has no clue if they're capturing/scoring the chip, or capturing the background instead.

ya - okay I got it to work and found a big error in the main calc as well
good news is that is looks really good with your sample pics and a gross color (111, 111, 0) with threshold 41
will make a new demo .exe with fixes

threshold map does not match the score values being made.
It's saying there's a valid chip but the map suggests the area should be null/missing

Vs this:

the opposite-ness is sort of an indicator of how the magic wand got there, but not exactly
current magic wand is c = Color.FromArgb(Math.Min(255, c.R + c.B), Math.Min(255, c.G + c.B), 0); but that is from limited testing
it is important to know that the threshold is only used for null detection, which will kick the score to -10

I think something is wrong with the yellow color, It's not displaying the threshold map correctly compared to the other colors

there are 2 types of visualizations:

threshold
chroma
the final score is either chroma or -10, depending on the threshold

The main reason for confusion is the current released version's Threshold button. It is a farce as has nothing to do with the final scoring. All it does it apply a 3rd party threshold function using the user's value on all channels - hence the hue that you see.
The new threshold button uses the very same calculations that are used at runtime. You get a greyscale image indicating the chroma image after the threshold is applied.

Sure, but the current display is incorrect...
The score values are correct, the map/image that's being displayed is not.

The map is showing an image that should be failing, but the scores are passing.
And it's only happening with yellow color

hmm okay - I will isolate the functions and do some debugging. please continue to test the other additions as well as how the chroma performs regardless of the threshold preview. I have a lot of meetings today, so I will be slower.

This should not be failing - there's a filled square, in the image, but a lot of white/unfilled space around it - enough to be more than 10%... so it should be a valid count. Idk, there's definitely something going on.

yeah - I will figure it out... but the contrast looks much better! once I fix it, it should help a lot

so weird - I am not seeing that issue... consistently getting scores of 20-40

so weird - I am not seeing that issue... consistently getting scores of 20-40

Yes! That's part of the problem!
When it's unfilled, it's seeing a base score for some reason, it's like if there's unfilled there's noise. It should be failing it instead if the map is being displayed correctly.

my bad - I meant consistent 20-40 scores where there is a chip
consistent -10 score for no chip

maybe sched a teams call w/ me? idk if XDC is open to that
that way we can verify this stuff after I fix all the other new features and find bugs with this one

took me a while to figure this out - but the duplicate ABC indices in the images dir mess up the image scroller
my guess is that the same grid was run with different origins for a total of 8 times? if not, then I am confused... so many coordinate systems...
anyway these chroma changes are looking good!

new demo here!!!

Okay, you gotta explain what the different colors in the map now mean.

Something is off with the color Green:
It's like green doesn't get counted for any score values.

When I land on a decent color, I get such better separation now!
Super cool to see!

Any issues or it just needs some documentation?

Dark green is messed up

this is the logarithmic linear color space of one of the brighter pics from GIMP

the color you select is like a chroma key, but not exactly. you select a color distribution to emphasize in the image pre-threshold. For these images, a bright yellow (255, 255, 0) looks pretty good at the lows and highs and will likely mask out most of the blurry blue blackground.

it might be best to restrict chroma to a third of the color wheel (> 300 deg or < 60 deg)

I liked the magic wand button, kinda sad it went away

If user selects Red chroma (same as default, but actually selecting it) it should fill red to show, not same as background like default does

I'm thinking I will create a new wizard window with a teaser v1.4 format that will replace the chroma button and threshold slider as well as show the steps it takes to get the score. Might take a bit longer, so let me know if you'd rather I just make documentation about what it does. I can do the manual red thing, but it will mean an additional serialized boolean for each feature.

Whatever doesn't cause issues with performance probably.
I actually like the chroma button a lot - as it stand it really does work really well. Maybe the magic wand button could be replaced with a wizard hat and that could open the 'color wizard'. But leaving the chroma button as is.

I've got great pass/fail results with v1.3.28
White/Grey/DarkGrey, and for other samples, gold/orange, and light green, purple/blue, and red all have good/interesting results.
I do need a definition of what the color and scoring means.
In the threshold map:

black = empty
white (and any color except purple or green?) = filled.

But fundamentally, this works amazingly well! I really can't be more excited with it in how much it helps improve yield fidelity.

ok - I will create a graphic showing how it works

infographic added to readme


Problem of float precision in the data

Hi,
When i retrieve the data to the server in the mysql database : i go this :

In the mysql database i have 1.8701 in a double(11,4) and when the ajax return me the value i have 1.8700999999999999.
But i does like that for some rows and not all why ?
Code snippet of problem
To render on 4 decimal i do like this :
{data: 'superficie', name: 'sous_parcelles.superficie', render: DataTable.render.number(' ', ',', 4), className: "text-right"},

How i can limit to 4 decimal in the result of the ajax dataTable ?
For the moment i add this in javscript to have good precision but it's a temporary solution :
Math.round(table.row('#' + row_id).data().superficie * 10000) / 10000;

System details

Unix
PHP Version 8.1
Laravel Version 10.13.5
Laravel-Datatables Version 10.4.3

Regards

Thanks it works.
But, why i need to do that because in the database i got a field with 4 decimals, and it does it only on some rows not all.

It depends, do you have casting, mutators or maybe a computation occurs? Or the value is fetch directly from DB?

Also, if you have to format several columns, you can use formatColumn.

I think it's directly from the DB because i do like that :
   $userData = DB::table('sous_parcelles')->select(.....)->where(....)
return Datatables::of($userData)
                ->editColumn('superficie', fn($model) => number_format($model->superficie, 4))
                ->filter(function ($query) {})->make(true);

I have no cast or  others in the model of SousParcelles.
And yes i have to format several columns.


Cannot import name 'Network' from 'controlnet_aux.hed'

When I run:
from controlnet_aux.hed import Network
hed_network = Network(f"{local_fold}/lllyasviel/ControlNet/annotator/ckpts/network-bsds500.pth")
HEDdetector(hed_network)

I got this error:
ImportError: cannot import name 'Network' from 'controlnet_aux.hed'
(I find init.py of "hed" folder has no 'Network' class or function.

I think on new version (0.0.6) they have changed pth file.
(network-bsds500.pth is gone now ...)
try this,
from controlnet_aux import HEDdetector
from controlnet_aux.hed import ControlNetHED_Apache2

# download the new file via .from_pretrained()
# this will download the file on huggingface cache dir
# usually ${HOME}/.cache/huggingface/hub/models--lllyasviel--Annotators
hed = HEDdetector.from_pretrained(model_repo)

model_dir = "...."  # <-- find 'ControlNetHED.pth' file's directory from the cache dir
model_path = os.path.join(model_dir, "ControlNetHED.pth")
network = ControlNetHED_Apache2()
network.load_state_dict(torch.load(model_path, map_location="cpu"))
network.float().eval()

hed_detector = HEDdetector(network)

copied from,
https://github.com/patrickvonplaten/controlnet_aux/blob/1e6bdc07c88d0358ae3c84862adb043ffd30c27c/src/controlnet_aux/hed/__init__.py#L72


Subcollection as Indexable Field?

Hello,
I am indexing a few document-level fields and would also like to include data from a subcollection within the documents being indexed. Is this possible? I can't find any documentation.

No, this is not possible at this time.  The extension only monitors the defined collection.  However, you might be able to use the transform function to enrich the record.


drawing issues on Mapbox

T-rex is awesome. Everything seems to be working very well. However i'm having some issues with high-level zoom (above 12).  I'm attaching a few screenshots. Was wondering if it was an issue with zoom level. But not sure

Looks strange indeed! Are these maps rendered with a built-in viewer? Which version?

I'm trying with the latest version 0.33.1. However, happens on the built in as well. Also noticed that no issues with built in OpenLayers and XRay

Then I'm supecting degerated geometries caused by clipping or simplification (see #5). Can you turn off simplification and play with the clipping parameters? I'm very interested in a dataset working with OL, but not with MB GL js. Is your datasource public or can you make it available for testing?

Thank's for the reply. I will test without simplification later today. I also can share this dataset. It's a query on our postgis server. However, I can export as shape file or geojson.

Tried to remove clipping and simplification = false, and the result is the same. Some polygons have holes. Maybe that's the issue. However, on OL they are correctly drawn. I can send you the dataset. What's the best format?

Best option would be a PG dump (pg_dump --file=out.dump --format=c --table=...). A file export would maybe do it as well.

Find attached the dump file. Let me know if you able to test it. Thank you very much!.
out.zip

Thanks for the data. I couldn't reproduce your output, but also saw some glitches when displaying it with Maputnik. A first look at the data shows some invalid geometries:

gid
st_npoints
st_nrings
st_isvaliddetail

162620
16348
11
(t,,)

162621
17451
9
(t,,)

162622
16006
6
(t,,)

162623
11674
8
(t,,)

162624
809
1
(t,,)

162625
12193
1
(t,,)

162626
22191
14
(f,"Too few points in geometry component",0101000000FBA45F828DCA53C1BED636691E7644C1)

162627
14718
2
(t,,)

162628
8661
2
(t,,)

162629
11419
9
(f,"Too few points in geometry component",01010000002E4D0730DEDB53C10198F0D8160045C1)

162630
8195
5
(f,"Too few points in geometry component",0101000000417405DD7AE553C132B7F7E9B3DC44C1)

162631
8441
9
(t,,)

162632
12311
4
(t,,)

162633
5055
1
(t,,)

162634
27981
10
(t,,)

162635
10026
2
(t,,)

162636
6761
2
(t,,)

162637
5083
1
(t,,)

162638
2663
1
(t,,)

162639
3961
1
(t,,)

162640
2929
1
(t,,)

162641
5122
1
(t,,)

162642
2513
1
(t,,)

162643
20223
10
(f,"Too few points in geometry component",010100000046B04456ECE453C1926402CE9EA644C1)

162644
6420
3
(t,,)

162645
3559
1
(t,,)

162646
2835
1
(t,,)

Thank you @pka. I've published the layer and removed the invalid geometries, however, mabox still is drawing the lines.  OpenLayers still fine.

These artifacts are caused by unclipped polygons or lines. Clipping has been fixed for reprojected layers in 7801bb8 and in a6889b7 for config files written with versions 0.7.0-0.7.2.
Always use buffer_size in your configuration!


Update Kiali to 1.40.1

Kiali released version 1.40.1

[x]  Mirror images in rancher/image-mirror repo
[x]  Pull changes into rancher-kiali-server chart
[x]  Update Istio Chart.yaml and Values.yaml to use the most recent version of Kiali

With 1578 we have updated the Kiali chart to 1.41.0

Kiali v1.41.0, istio v1.11.4
Create a downstream cluster with 2worker, 1etcd, 1 cp

Install istio version 1.11.4 by enabling kiali
Verified the version of kiali installed is rancher/mirrored-kiali-kiali:v1.41.0
Deploy a demo bookservice app in the istio-injected namespace and verified the app has installed by nodeIP:port/productpage
From cluster explorer --> istio --> click on the kiali --> from graph [select namespace where the app is installed], the traffic is shown for the book demo app
Install the weighted demo bookservice app and verified traffic generated from kiali --> graph --> verified the traffic going through ingress gateway
Enable egress gateway and verified the traffic via egress gateway


Upgrade postgres to v1.3.4

The postgres driver was throwing error:
panic: runtime error: invalid memory address or nil pointer dereference
See: https://github.com/tektoncd/results/issues/184

/ok-to-test

/kind bug


SqlBuilder Where(string, args) makes object null

Using this code:
var finalArgs = args.ToArray();
var sql = new NPoco.Sql(string.Empty, finalArgs);

sql.Select(parser.SelectClause);

if (!parser.FromClause.IsNullOrEmpty())
   sql.From({$" from {parser.FromClause} ");

if (!parser.WhereClause.IsNullOrEmpty())
   sql.Where(parser.WhereClause, finalArgs);

When the Where method is executed, it makes the object null. During the .Select call and .From call the object updates normally. It also doesn't seem to import the "finalArgs" I'm passing in. This is a blocker for me as I cannot get proper Sql object back from this call.

Its working as expected.
var sql = new Sql();

sql.Select("blah, boo, bar");
sql.From("mytable");
sql.Where("test = @0", "1");

Console.WriteLine(sql.SQL);
Console.WriteLine(sql.Arguments);

SELECT blah, boo, bar
FROM mytable
WHERE (test = @0)

["1"]


Added Memory Parameter to SQL-Chain (Langchain Experimental)

Description:  SQL Database Chain and Agent does not allow to pass in memory, therefore, added this feature with additional tests.
I had created a PR #7546 for this but  chain has moved to langchain_experimental lib therefore created a new PR.
Issue:  #6918
Tag maintainer: @hwchase17

@hwchase17 and @baskaryan please have a look.

@keenborder786 i changed it a bit here: https://github.com/langchain-ai/langchain/pull/8550

Have any of the PRs have been merged relating to add chat history memory to. the SQL agent or SQL chain?


prow.sh: avoid creating paths unless really running tests

verify-go-version.sh sources prow.sh to determine the desired Go
version. This had the unintended side effect of creating one
$GOPATH/pkg/csiprow.* directory per make invocation and triggered
error messages when GOPATH was not set.
Also, go env GOPATH is now used as fallback when not set explicitly.
Fixes: https://github.com/kubernetes-csi/csi-release-tools/issues/142

/release-note-none

/assign @bswartz

/lgtm


Add the instructions for installing TorXakis on macOS.

Ask Axel Belinfante to do a user test?

Good idea to involve Axel!

I discovered when building the brew recipe for torxakis:
Sometimes the ghc build from source with stack fails.   Jan Tretmans said he
also had this problem (under cygwin If I remember correctly), and the solution was just to retry  the build.
The reason is that ghc has some makefiles which do not handle parallel
builds well. Often it will go ok, but sometimes
it fails. You can prevent this failure by disallowing parallel running the
makefile builds by setting the environment
flag :
export MAKEFLAGS=" -j1 "

The problem only appears when stack runs a makefile, but within stack itself
doing parallellisme is handled fine. Stack by default chooses the right
number of parallellisme depending on the number of cpu cures. The MAKEFLAGS
environment variable doesn't effect the parallellisme within stack itself.
So the installation instruction on the torxakis website for linux:
stack setup
stack install

should be changed into:
export MAKEFLAGS=" -j1 "
stack setup
stack install

this will prevent some build failures.
If you look at the homebrew recipe at
https://github.com/TorXakis/homebrew-TorXakis/blob/master/Formula/torxakis.rb
then you can see I do something similar. (homebrew uses ENV.deparallelize ).
The reason that I had to really solve this problem was that the build had to
be reliable for the brew bot. Just restarting the build was not an option.
Note: something similar is done by
https://github.com/Homebrew/homebrew-core/blob/master/Formula/haskell-stack.rb
greetings
Harco

@pjljvandelaar @tretmans I added a short version of the instructions on how to install on MacOS.
Axel found an error when running TorXakis I'm gonna try and see if it occurs on Linux and Windows.


Update rendering driver name on fallbacks. Fix rendering driver/method in the editor system info.

Fixes wrong names used in Help → Copy System Info (was using project settings, instead of real current driver/method).
Add missing Direct3D 12 to the list.
Set saved driver name if it was changed by fallback

Might fix some issues with shaders when D3D12 ↔ Vulkan fallback is used, since saved name is used to set defines: https://github.com/godotengine/godot/blob/3b3d6226c5cd3f9897160815990e2e234f165605/servers/rendering/renderer_rd/shader_rd.cpp#L202
Fixes #96380

Thanks!

This breaks compiling with MSVC, it seems to introduce bad declares by introducing platform code, suspect platform/windows/platform_gl.h
Trying to create a fix

This breaks compiling with MSVC, it seems to introduce bad declares by introducing platform code, suspect platform/windows/platform_gl.h

Strange, CI runs MSVC and passed.

Starting with a simple undefine series but might be something that can be configured for the include to block the defines

simple undefine series but might be something that can be configured for the include to block the defines, can alternatively build editor_node.cpp separately in scu

Or we can add another internal OS method like set_current_rendering_method() and remove the include, actual value is set in the same init code that sets set_current_rendering_method.

Sounds like a more coherent solution, making a PR to split up the SCU as a band-aid while that solution can be implemented


Add SSL termination to bare metal k8s setup

update bare metal setup docs to support ssl termination.
Can we use let's encrypt for the cert?

Completed in a previous PR.


Limitations that was solved in 10.0.17

I think it is save to remove described limitations that was solved in 10.0.17 :-)

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

ID: 12a6fc14-5802-043a-299c-4c3a7f6ac438
Version Independent ID: 68118450-ee3b-7e3e-8d6a-d27ada70bd26
Content: Revenue recognition reallocation - Finance | Dynamics 365
Content Source: articles/finance/accounts-receivable/rev-rec-reallocation.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @bking
Microsoft Alias: bking

Bug 793925 has been logged to address this issue.

@kurthatlevik-inspirit365, thank you for the feedback. The topic has been updated.


ko: Fixing outdated Korean contents

Fixed the _index.html.

[x] M1. content/en/_index.html
#28746

@niteshseram Thanks !
/lgtm
/approve


Add soft 404 detection

From GitLab repo urls2warc/-/issues/2#note_15998, Mark says:
"If we decide to try and do soft 404 detection something like this might be useful - https://github.com/dogancanbakir/soft-404
We would use it on pages that were recognized as html that have a status code of 200."
From slack, Mark says (April 13, 2022):
"If we can give the soft-404 a try https://github.com/dogancanbakir/soft-404 (just because we have the framework to try and make that happen) that could be cool.
If it seems like it will be quite a bit of effort to really make it work I do not have strong desires to go down that road right now.
But if it plugs in and seems to work ¯_(ツ)_/¯ "

@vphill In adding soft404 detection, what output would you expect?
Here is a starting point for the sidecar payload:
Identified-Payload-Type: {'fido': 'application/xhtml+xml', 'python-magic': 'text/html'}
Preservation-Identifier: fmt/102
Charset-Detected: {'encoding': 'ascii', 'confidence': 1.0}
Languages-cld2: {'reliable': True, 'text-bytes': 1804, 'languages': [{'name': 'ENGLISH', 'code': 'en', 'text-covered': 99, 'score': 1189.0}]}
Soft-404-Detected: 0.6963556617840959

Also, do we only want to keep track of those with a probability of 50% and higher?

This looks reasonable to me.
As for keeping track of things 50% and higher I think that can be done easily by a downstream client if they are interested in that information.


Runtime error with arrays

Copy/paste into http://elm-lang.org/try
import Array exposing (Array)

type alias Box =
  { value : String }

convertTo2D : Array Box -> Int -> Array (Array Box)
convertTo2D boxArray numRows =
  let
    nextRow = Array.slice 0 numRows boxArray
  in
    if Array.length nextRow > 0
    then Array.push nextRow (convertTo2D (Array.slice numRows (Array.length boxArray) boxArray) numRows)
    else Array.empty

oneDimArray = Array.initialize 100 (\index -> { value = toString index })

temp = convertTo2D oneDimArray 10

This is boiled down from a larger application. The relevant error is:
"Uncaught TypeError: Cannot read property 'height' of undefined  everything.js:831"

This is more likely relevant for the https://github.com/elm-lang/core repository, so I suggest moving the issue over to there. In fact, first please check whether there is already a relevant issue open there, to avoid duplication. If there is, just close the one here.


fix: ensure that Origin header is rewritten as necessary

The wrangler dev command puts the Worker under test behind a proxy server. This proxy server should be transparent to the client and the Worker, which means that the Request arriving at the Worker with the correct url property, and Host and Origin headers.
Previously we fixed the Host header but missed the Origin header which is only added to a request under certain circumstances, such as cross-origin requests.
What this PR solves / how to test:
This change fixes the Origin header as well, so that it is rewritten, when it exists, to use the origin of the url property.
A nice way to test this is to simply clone the reproduction from #4761 - https://github.com/Le0Developer/wrangler-host-header-repro, then replace the local Miniflare with the pre-release one from this PR.
Fixes #4761
Author has addressed the following:

Tests

[x] Included (updated worker-app fixture tests)
[ ] Not necessary because:

Changeset (Changeset guidelines)

[x] Included
[ ] Not necessary because:

Associated docs

[ ] Issue(s)/PR(s):
[x] Not necessary because: no user facing change

Hey @petebacondarwin and @mrbbot, I have a question regarding this PR and would be very grateful if you could clarify it for me :slightly_smiling_face:
I am building an application that has a React frontend making requests to Cloudflare workers. Whenever any request is sent, the Origin header is correctly set in the browser (in my case to localhost:5173). However, when the worker starts processing it, that header has the same value of the request url (in my case localhost:8787). I checked with wireshark that the request is actually being sent with the correct Origin header, so I believe this change took place between the request being received and the worker starts processing it. The issue with this behavior is that without the original Origin header value, I cannot set the correct Access-Control-Allow-Origin header, making the frontend reject every response.
After looking at this PR, I noticed that the Origin header is being set in the following lines:
if (rewriteHeadersFromOriginalUrl) {
    request.headers.set("Host", url.host);
    // Only rewrite Origin header if there is already one
    if (request.headers.has("Origin")) {
        request.headers.set("Origin", url.origin);
    }
}

I'm having trouble figuring out why the Origin header needs to be set to the request url, and not to its original value (request.headers.get("Origin"), or no change at all, since the original request was used to build the new one). I would be very grateful if you could clarify that for me!

@afonsocrg - happy to try to help. Can you confirm what version of Wrangler you are using, and exactly what your setup is here? It seems from what you are saying that you are running wrangler pages dev and testing this locally?
Also when you say that the request in the browser has the Origin set to localhost:5172, what is the actual request you are making? Are you setting the url to localhost:8787 but the Origin to localhost:5172?

@petebacondarwin thanks for the fast reply!! This behavior happens only in versions 3.25 and 3.26. The worker was running using wrangler dev. If I ran with the --remote flag, I did not have this issue, but I'd rather not have to use that flag.
The React app is running on localhost:5173, and when it sends any request (OPTIONS, POST, etc.), the same happens: The browser sends the correct Origin header (localhost:5173, which is the endpoint the frontend is running), but the worker receives Origin: http://localhost:8787, which is incorrect. I changed the worker code to only print the headers and return an empty response, so the headers must have been modified before the worker code has been called.
I expected the headers to be the same as the ones sent by the browser, so that the wrangler layer is transparent to the worker

How does the request get to the Worker if it is running at localhost:8787 and the browser is making a request to locahost:5173?

Sorry, the frontend is running at localhost:5173, and is sending the requests to localhost:8787.
According to MDN, I believe the origin header should be set to localhsot:5173, and the Host header should be the one set to localhost:8787
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Origin
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Host

This stuff is so confusing but here is my take on how this should work.

Host is where the request is going to, it is very likely going to match the request.url property since this is also what is used to route the request to the correct end point.
Origin (as noted in MDN) is effectively the server where the response being processed by a client originated from.

So as I understand it, if you have a web page hosted at localhost:5173 and you make a client side fetch() request to a server at localhost:8787, then the Request should have localhost:5173 as its Origin and localhost:8787 as its Host headers.
With that in mind, you might be right that something in our implementation is a bit off in our implementation of this fix, since we are reconstructing the Origin header from the request url origin, rather than the referring URL...


Fixed some minor things in the readme.

So I added a comma before and(Run it, Wait for it... sentence) due to having three independent clauses.
Change downloads to releases. Unneeded, but I find it better.
Made client arguments to client’s arguments

thanks!


polyml: 5.7.1 -> 5.8

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/polyml/versions.
meta.description for polyml is: '"Standard ML compiler and interpreter"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 5.8 with grep in /nix/store/xmsf9vfzxgzkgysj7c0sb8r3whr9673x-polyml-5.8
directory tree listing: https://gist.github.com/5a79692202d5a3d5f6453ed8b63e2a2f
du listing: https://gist.github.com/c6d8c3b1d1dade25e72ec65f665363bf

Rebuild report (if merged into master) (click to expand)

14 total rebuild path(s)
4 package rebuild(s)
4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
hol
isabelle
polyml
tlaps

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xmsf9vfzxgzkgysj7c0sb8r3whr9673x-polyml-5.8 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A polyml https://github.com/r-ryantm/nixpkgs/archive/99783a87dbea712a2035996a60e9e88149e5a861.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xmsf9vfzxgzkgysj7c0sb8r3whr9673x-polyml-5.8
ls -la /nix/store/xmsf9vfzxgzkgysj7c0sb8r3whr9673x-polyml-5.8/bin

cc @maggesi @yurrriq for testing.

It's been a minute since I've done much ML programming, so I'm not sure how much help I'll be.

When I run nix-review locally, tlaps builds fine. Let's try ofborg:
@grahamcofborg build polyml isabelle tlaps

reverse dependencies build fine.


tl.dot maybe has issues with first argument transposed

Hi all, I am learning triton and in the course of writing a kernel, I think I have found an issue.
A reproducible example of what I think is the root cause of the bug can be found below.
It is a simple modification of https://triton-lang.org/main/getting-started/tutorials/03-matrix-multiplication.html so that instead of calculating A @ B --> C we instead calculate A.T @ B --> C.
I've indicated which lines are changed via #NOTE.
import torch

import triton
import triton.language as tl

"""
Modification of the tutorial https://triton-lang.org/main/getting-started/tutorials/03-matrix-multiplication.html

except instead of calculating A @ B = C, calculate A.T @ B = C

That is instead of shapes:
   A: [M, K]
   B: [K, N]
   C: [M, N]

We
   A: [K, M]
   B: [K, N]
   C: [M, N]

"""

# `triton.jit`'ed functions can be auto-tuned by using the `triton.autotune` decorator, which consumes:
#   - A list of `triton.Config` objects that define different configurations of
#       meta-parameters (e.g., `BLOCK_SIZE_M`) and compilation options (e.g., `num_warps`) to try
#   - An auto-tuning *key* whose change in values will trigger evaluation of all the
#       provided configs
# NOTE: only a single config for faster running
@triton.autotune(
    configs=[
        # triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 256, 'BLOCK_SIZE_K': 64, 'GROUP_SIZE_M': 8}, num_stages=3, num_warps=8),
        # triton.Config({'BLOCK_SIZE_M': 64, 'BLOCK_SIZE_N': 256, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4, num_warps=4),
        # triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 128, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4, num_warps=4),
        # triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 64, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4, num_warps=4),
        # triton.Config({'BLOCK_SIZE_M': 64, 'BLOCK_SIZE_N': 128, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4, num_warps=4),
        # triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 32, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4, num_warps=4),
        # triton.Config({'BLOCK_SIZE_M': 64, 'BLOCK_SIZE_N': 32, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=5, num_warps=2),
        triton.Config({'BLOCK_SIZE_M': 32, 'BLOCK_SIZE_N': 32, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=5, num_warps=2),
    ],
    key=['M', 'N', 'K'],
)
@triton.jit
def matmul_kernel(
    # Pointers to matrices
    a_ptr, b_ptr, c_ptr,
    # Matrix dimensions
    M, N, K,
    # The stride variables represent how much to increase the ptr by when moving by 1
    # element in a particular dimension. E.g. `stride_am` is how much to increase `a_ptr`
    # by to get the element one row down (A has M rows).
    stride_am, stride_ak,
    stride_bk, stride_bn,
    stride_cm, stride_cn,
    # Meta-parameters
    BLOCK_SIZE_M: tl.constexpr, BLOCK_SIZE_N: tl.constexpr, BLOCK_SIZE_K: tl.constexpr,
    GROUP_SIZE_M: tl.constexpr,
    ACTIVATION: tl.constexpr,
):
    """Kernel for computing the matmul C = A x B.
    A has shape (M, K), B has shape (K, N) and C has shape (M, N)
    """
    # -----------------------------------------------------------
    # Map program ids `pid` to the block of C it should compute.
    # This is done in a grouped ordering to promote L2 data reuse.
    # See above `L2 Cache Optimizations` section for details.
    pid = tl.program_id(axis=0)
    num_pid_m = tl.cdiv(M, BLOCK_SIZE_M)
    num_pid_n = tl.cdiv(N, BLOCK_SIZE_N)
    num_pid_in_group = GROUP_SIZE_M * num_pid_n
    group_id = pid // num_pid_in_group
    first_pid_m = group_id * GROUP_SIZE_M
    group_size_m = min(num_pid_m - first_pid_m, GROUP_SIZE_M)
    pid_m = first_pid_m + (pid % group_size_m)
    pid_n = (pid % num_pid_in_group) // group_size_m

    # ----------------------------------------------------------
    # Create pointers for the first blocks of A and B.
    # We will advance this pointer as we move in the K direction
    # and accumulate
    # `a_ptrs` is a block of [BLOCK_SIZE_K, BLOCK_SIZE_M] pointers # NOTE: SWAPPED DIMS
    # `b_ptrs` is a block of [BLOCK_SIZE_K, BLOCK_SIZE_N] pointers
    # See above `Pointer Arithmetics` section for details
    offs_am = (pid_m * BLOCK_SIZE_M + tl.arange(0, BLOCK_SIZE_M)) % M
    offs_bn = (pid_n * BLOCK_SIZE_N + tl.arange(0, BLOCK_SIZE_N)) % N
    offs_k = tl.arange(0, BLOCK_SIZE_K)
    a_ptrs = a_ptr + (offs_k[:, None] * stride_ak + offs_am[None, :] * stride_am) # NOTE: SWAPPED DIMS
    b_ptrs = b_ptr + (offs_k[:, None] * stride_bk + offs_bn[None, :] * stride_bn)

    # -----------------------------------------------------------
    # Iterate to compute a block of the C matrix.
    # We accumulate into a `[BLOCK_SIZE_M, BLOCK_SIZE_N]` block
    # of fp32 values for higher accuracy.
    # `accumulator` will be converted back to fp16 after the loop.
    accumulator = tl.zeros((BLOCK_SIZE_M, BLOCK_SIZE_N), dtype=tl.float32)
    for k in range(0, tl.cdiv(K, BLOCK_SIZE_K)):
        # Load the next block of A and B, generate a mask by checking the K dimension.
        # If it is out of bounds, set it to 0.
        a = tl.load(a_ptrs, mask=offs_k[:, None] < K - k * BLOCK_SIZE_K, other=0.0) # NOTE: SWAPPED DIMS
        b = tl.load(b_ptrs, mask=offs_k[:, None] < K - k * BLOCK_SIZE_K, other=0.0)
        # We accumulate along the K dimension.
        accumulator += tl.dot(a.T, b) # NOTE: SWAPPED DIMS
        # Advance the ptrs to the next K block.
        a_ptrs += BLOCK_SIZE_K * stride_ak
        b_ptrs += BLOCK_SIZE_K * stride_bk
    # You can fuse arbitrary activation functions here
    # while the accumulator is still in FP32!
    if ACTIVATION == "leaky_relu":
        accumulator = leaky_relu(accumulator)
    c = accumulator.to(tl.float16)

    # -----------------------------------------------------------
    # Write back the block of the output matrix C with masks.
    offs_cm = pid_m * BLOCK_SIZE_M + tl.arange(0, BLOCK_SIZE_M)
    offs_cn = pid_n * BLOCK_SIZE_N + tl.arange(0, BLOCK_SIZE_N)
    c_ptrs = c_ptr + stride_cm * offs_cm[:, None] + stride_cn * offs_cn[None, :]
    c_mask = (offs_cm[:, None] < M) & (offs_cn[None, :] < N)
    tl.store(c_ptrs, c, mask=c_mask)

# We can fuse `leaky_relu` by providing it as an `ACTIVATION` meta-parameter in `_matmul`.
@triton.jit
def leaky_relu(x):
    x = x + 1
    return tl.where(x >= 0, x, 0.01 * x)

def matmul(a, b, activation=""):
    # Check constraints.
    assert a.shape[0] == b.shape[0], "Incompatible dimensions" # NOTE: updated shape stuff
    assert a.is_contiguous(), "Matrix A must be contiguous"
    assert b.is_contiguous(), "Matrix B must be contiguous"

    # NOTE: updated the shape stuff
    K, M = a.shape
    _, N = b.shape
    # Allocates output.
    c = torch.empty((M, N), device=a.device, dtype=a.dtype)
    # 1D launch kernel where each block gets its own program.
    grid = lambda META: (
        triton.cdiv(M, META['BLOCK_SIZE_M']) * triton.cdiv(N, META['BLOCK_SIZE_N']),
    )
    matmul_kernel[grid](
        a, b, c,
        M, N, K,
        a.stride(0), a.stride(1),
        b.stride(0), b.stride(1),
        c.stride(0), c.stride(1),
        ACTIVATION=activation
    )
    return c

if __name__ == "__main__":
    torch.manual_seed(0)

    # NOTE: this is changed to faciliate new shapes
    a = torch.randn((1, 512), device='cuda', dtype=torch.float16)
    b = torch.randn((1, 512), device='cuda', dtype=torch.float16)
    triton_output = matmul(a, b)
    torch_output = torch.matmul(a.T, b)
    print(f"triton_output={triton_output}")
    print(f"torch_output={torch_output}")
    if torch.allclose(triton_output, torch_output, atol=1e-2, rtol=0):
        print("✅ Triton and Torch match")
    else:
        print("❌ Triton and Torch differ")

I would expect this code to work,  however the inputs do not match.
Output:
triton_output=tensor([[-0.1672,  0.5107, -0.8545,  ...,  1.9160, -0.1917, -0.3193],
        [ 0.0327, -0.0999,  0.1670,  ..., -0.3748,  0.0375,  0.0624],
        [ 0.0000,  0.0000,  0.0000,  ...,  0.0000,  0.0000,  0.0000],
        ...,
        [ 0.0000,  0.0000,  0.0000,  ...,  0.0000,  0.0000,  0.0000],
        [ 0.0000,  0.0000,  0.0000,  ...,  0.0000,  0.0000,  0.0000],
        [ 0.0000,  0.0000,  0.0000,  ...,  0.0000,  0.0000,  0.0000]],
       device='cuda:0', dtype=torch.float16)
torch_output=tensor([[-0.1672,  0.5107, -0.8545,  ...,  1.9160, -0.1917, -0.3193],
        [-0.0769,  0.2349, -0.3928,  ...,  0.8813, -0.0881, -0.1469],
        [-0.4780,  1.4600, -2.4434,  ...,  5.4805, -0.5483, -0.9131],
        ...,
        [-0.0576,  0.1760, -0.2947,  ...,  0.6606, -0.0661, -0.1100],
        [ 0.2534, -0.7744,  1.2959,  ..., -2.9062,  0.2908,  0.4841],
        [ 0.1185, -0.3621,  0.6060,  ..., -1.3594,  0.1359,  0.2263]],
       device='cuda:0', dtype=torch.float16)
❌ Triton and Torch differ

Some information:
CUDA: 12.1
Driver: 530.30.02
Card: RTX 3070
Triton: git @ 9820899b3845e461d9031dba66062efade65d420
Note you can write an analogous kernel for (A @ B.T) and it will work. This makes me think it is an issue with tl.dot with the first argument transposed.
Any ideas? I have reviewed what I have done carefully, and from my understanding of triton semantics I believe this should work.

That does look like a bug. This is not the way we usually do transpositions so it's possible this code path isn't very well tested

What's the suggested way for doing transpositions in this case?

inverting a.stride(0) and a.stride(1) when you launch the kernel, or alternatively just passing a.T to the op

I see, but it won't work if we need to use both a.T and a in the triton kernel.

I looked at your kernel more carefully, and I think it is actually wrong. stride_am and stride_ak should be swapped inside the kernel

Ah yeah, thanks.


problem with config

I'm brand new to this space. Trying to use the Polar Flow integration. When I try to configure I get a direct to the website. On the website I get the following on my account page.
"OOPS, SOMETHING WENT WRONG SOMEWHERE ALONG THE WAY. LET'S TRY AGAIN."
what did I do wrong ??

+1. I have the same issue.

+1. I have the same issue.

see https://github.com/burnnat/ha-polar/issues/6

see https://github.com/burnnat/ha-polar/issues/6

Please check the URL, I've found that redirect was coming from my internal IP address while in the auth I gave the Dickens address; in case go on the authapp and modify the address. Check also http Vs https.

I have set in polar http://myexternalip:8123/api/polar_auth but then I get the oops page....

Until we will get some correct feedback from the maintener try to:

check exactly the string of your redirect, you'll find exactly what the script will put out for you
crosscheck what you put in polar auth. In this example I assume you put http://myexternalip:8123/api/polar_auth
modify the string of your redirect accordingly
It worked for me. TBH I don't remember if I've deleted 8123. I remember for sure I had to add http"S"

Thanks! The authentication worked. But the next problem is that no data is synchronised..... I see the sensors VO2max etc but no data....

I had same issue, It took 2 days but now it's working correctly giving physical data. Strange thing is that it updated "batch", meaning the first time I saw my phisical data, then after other 1 or 2 days I saw daily steps and after I think 2-3 days more my lastworkout.
If after a couple of days you have no updates post your config

Thanks for replying. I installed and authenticated it last week. No data is synchronized.
in my config I have the following:
polar:
client_id: 21ecb43f-xxxxxxxxxx
client_secret: d01404fa-xxxxxxxxxx
unit_system: metric # Defaults to global unit_system if not specified
monitored_resources:
daily_activity:
- "calories"
- etc. etc. etc.


Bugfix render_login_html_to_string (refs: #53)

Hello and thanks for djangosaml2idp!
While reviewing this software for use in our company, I found some bugs and I decided to make a couple of PRs for this. I'll try to keep the PRs small and simple, so they are easy to review :)
There was also an issue reported for this already (#53), I believe it can be closed by merging this PR.
If there are comments, please let me know and I'll see what I can do :)

Thanks for the contributions @mjholtkamp! I'll get through them as soon as possible. There are indeed some issues that crept in during refactoring in the latest release, as you noted in one of the MR's. We ourselves are running v0.5.0 in production and have been for several months, should you want to to try a more tested version.

Thanks for the quick reply @mhindery and thanks for the tip of 0.5.0. In the documentation it is being said that this project is production-ready, but I was a bit surprised to find these bugs (no offence meant, I mean that it's likely they would be found if used in production).
That confused me a little bit. I don't know if you have a 'stable' channel or something, but maybe mention somehow in the documentation that the 0.5.0 is considered stable and the 0.6.x is being worked on (or whatever the status is)? How do you feel about that?


⚠ Bofrost.at has degraded performance

In c7420de, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2531 ms

Resolved: Bofrost.at performance has improved in 828c4eb.


Cannot Configure CORS on Module

Describe the bug
I configured CORS on Module with this configuration :
apiVersion: v1
kind: Service
metadata:
  annotations:
    getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind: Module
      name: tls
      config:
        server:
          enabled: True
      ---
      apiVersion: ambassador/v1
      kind:  Module
      name:  ambassador
      config:
        origins: "*"
        methods: POST, GET, OPTIONS
        headers: Content-Type, XXX, YYY

But it didn't work and got result :

Failed to load resource: net::ERR_CONNECTION_REFUSED
Failed to load resource: the server responded with a status of 403 (Forbidden)
Access to XMLHttpRequest at 'https://' from origin 'https://' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.

But it worked if I configured CORS on the mapping with this configuration :
apiVersion: v1
kind: Service
metadata:
  annotations:
    getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind:  Mapping
      name:  xxx
      prefix: xxx
      service:xxx:8080
      cors:
        origins: "*"
        methods: POST, GET, OPTIONS
        headers:
            - Content-Type
            - XXX
            - YYY

Versions (please complete the following information):

Ambassador: [0.53.1]
Kubernetes environment [1.11.5]

I'm experiencing a similar issue

+1 also faced with this problem.

+1 same issue.
is this issue solved by anyone?

+1 same issue.
is this issue solved by anyone?


Broken with multiple authentication methods

Attempting to test this out with the examples I get the following error:
Obviously this is due to having both Github and Basic Auth enabled but that seems like a reasonable use case (Github for users, basic auth for bots). fly login supports --username and --password so that seems like a reasonable fix?
Downloading fly executable...
Logging into concourse...
1: GitHub
2: Basic Auth
choose an auth method: [redacted-username]
invalid selection (not a number)
[redacted-password]
invalid selection (not a number)
/usr/lib/ruby/gems/2.2.0/gems/process_helper-0.0.3/lib/process_helper.rb:117:in `handle_exit_status': Command failed, pid 17 exit 1. Command: `/tmp/d20160914-7-w1zdsp/fly --target=concourse login --concourse-url=http://concourse-web.build.svc.cluster.local:8080 `. Command Output: "1: GitHub (ProcessHelper::UnexpectedExitStatusError)
2: Basic Auth
choose an auth method: [redacted-username]
invalid selection (not a number)
[redacted-password]
invalid selection (not a number)
"
	from /usr/lib/ruby/gems/2.2.0/gems/process_helper-0.0.3/lib/process_helper.rb:22:in `block in process'
	from /usr/lib/ruby/2.2.0/open3.rb:199:in `popen_run'
	from /usr/lib/ruby/2.2.0/open3.rb:188:in `popen2e'
	from /usr/lib/ruby/gems/2.2.0/gems/process_helper-0.0.3/lib/process_helper.rb:12:in `process'
	from /tmp/build/0e9e5171/concourse-branch-manager/tasks/lib/cbm/pipeline_updater.rb:32:in `set_pipeline'
	from /tmp/build/0e9e5171/concourse-branch-manager/tasks/lib/cbm/branch_manager.rb:49:in `run'
	from concourse-branch-manager/tasks/manage_branches.rb:7:in `<main>'

Feel free to submit a PR for this.  It seems like we should support three independent cases, either Github, basic auth, or both?

Yeah, that makes sense (though I'm not a huge fan of parsing an interactive CLI...).
Sanity check this for me?
fly -t blah login

if "choose auth method" prompt:
  match each line like /(\d)\. (\w+)/
  if github and github token:
     select github option
     reply to token prompt
  else basic auth flow:
     select basic auth option
     reply to username and password prompts

else if "username" prompt:
  assume basic auth
  reply to username and password prompts

else if "enter token" prompt:
  reply to token prompt

@philk That looks right - but I don't have access to a server configured with github auth, so I can't test any of this out.

@philk I looked at refactoring it in the way you're describing (using --username and --password), there isn't a reason you can't do it. However, you need to be careful because if a command fails, it prints the entire command to stderr, which would mean you'd be able to see the username and password in the build log.

Heh, it's currently doing that anyway (see my [redacted] above) but I agree if we can avoid that it'd be good.

@philk  I would like suppressing the credentials in output to be a separate issue, if it's pre-existing.  Feel free to open one.  If you do, we should be able to do something about it with the proper [process_helper](https://github.com/thewoolleyman/process_helper options) - specifically :include_output_in_exception and/or :puts_output


Nota traducida

La nota al pie de la página no estaba traducida en los archivos de español.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


change group apikey

Description
Please provide a summary of the pull request and the issue it fixes. Please add necessary details, context, dependencies, explanation of when review is needed (see next section), etc.
Fixes #(add issue number here and remove parentheses)
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[ ] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[ ] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

I think the config.template.py needs to be updated too. Thanks.

@clav1d, Could you please do a quick review of this PR when you get a chance? Thanks.

Thanks, David. I will merge this then.


Search Formatter for Specific Cell (Phone Number)

Question
Hello,
I have a table with a column that contains a phone number. Since the data is stored unformatted, column uses a formatter to properly format the phone number. So for example<PHONE_NUMBER>8 -> 1<PHONE_NUMBER>.
For this table, searchFormatted is set to true to search over the formatted data. However, I have noticed that users search for phone numbers in many ways:
14632222408
1<PHONE_NUMBER>
1-463-222-1408
1-(463)-222-1408
+14632222408
Is there a way I can change the search value for the column to only compare the raw data? So for example, if the user searches 1<PHONE_NUMBER> it should remove all non numeric characters and only search against the raw value. Keep in mind this is the only column where the value would be compared like this searchFormatted needs to be enabled for the rest of the table. I have tried onColumnMatch but have not had any luck.
I appreciate any assistance with this.
Screenshots

codesandbox
Please give a simple and minimal example on https://codesandbox.io so that we can reproduce it easily.

hi @AdamJakiela boss, try this: https://react-bootstrap-table.github.io/react-bootstrap-table2/storybook/index.html?selectedKind=Table Search&selectedStory=Custom match function&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel
I just found I this custom search didn't documented. will patch it... sorry


Refactor runtime.go file

/area runtime
Describe the proposal
Currently the pkg/runtime/runtime.go is a humongous file >2800 lines of code.
As new code gets added and new building blocks are considered and added or old code is modified, it becomes really hard to maintain. Similarly the runtime_test.go file is also huge >4900 lines of code.
The proposal in this issue is to split the runtime.go file into its component building block code, with them residing in different files while only the init, orchestration and shutdown are performed in the runtime.go file.
cc @dapr/maintainers-dapr thoughts on this?

Hey
I would like to work on this issue.
Please assign.

Hi, I also encountered the same problem when designing the runtime api of the SDK model: https://github.com/dapr/dapr/issues/3261.
runtime.go is too big, both from a code perspective and a developer maintenance perspective.
I mentioned the split idea before here: https://github.com/dapr/dapr/issues/3600. Currently java-runtime-api runtime api is split into 4 parts:

core: core, stable API, such as invoke, pubsub, state, etc.
enhance: Enhanced, incubating apis, such as dislock, sequencer
native: APIs that support native protocols, such as redis, s3
saas: support extensible saas api, such as email, sms

Looking forward to the follow-up design of the community


Problem with random_matrix in conftest

The random_matrix fixture is a nice idea but it has an issue:  the seed is set in conftest (which gets called once per test module) which means that the results of the tests depend on the order that the tests are executed. This means that tests which depend on the matrix's value will fail if you try to run them individually e.g.
pytest tests/data/test_HOA_tools_cec2.py::test_P
Or will fail if a new test is added anywhere than at the end.
I propose an improved implementation that will allow both the current behaviour and one with a test-dependent seed
@pytest.fixture
def random_matrix():
    """Generate a random matrix for use in tests."""
    # The fixture returns a function that can be called to generate a random matrix
    # >>> def my_test(random_matrix):
    # >>>     matrix = random_matrix(seed=1234)
    # or use the global seed
    # >>>     matrix = random_matrix()
    # or
    def _random_matrix(seed: int | None = None) -> np.ndarray:
        if seed is not None:
            # Seed is supplied so use a generator with that seed...
            rng_to_use = np.random.default_rng(seed)
        else:
            # ... else use the global generator (defined in conftests's global namespace)
            rng_to_use = rng
        return np.asarray(rng_to_use.random((100, 100)))

    return _random_matrix

The  random_matrix fixture is currently only being used in tests/data/test_HOA_tools_cec2.py so this issue can be fixed while dealing with issue #81

This issue will be closed by PR #254


Operation lifecycle tweaks

Summary

Changes how we are handling -init for objects that are declaring NS_DESIGNATED_INITIALIZER on a method aside from -init (and don't subsequently use -init in any way, marked as NS_UNAVAILABLE), across the entire Operative framework.
Some general clean up around compiler warnings and -init
Documents OPLocationConditions designated initializer
Documents OPGroupOperations designated initializer
Documents NS_UNAVAILABLE -init methods with appropriate @see reference to designated initializers across the entire Operative framework.
Updates .gitignore with typical Carthage stuff as well as AppCode stuff. Tidying.

References

Similar changes to OPOperationQueue as #16, however some additional "guarding" (carry over of Swift optionals) and potential for referencing a nil delegate. Ultimately better alignment with Advance NSOperations implementation.

I went through this PR and it looks good to me. This changeset supersedes #16.


Live demo white screen

When trying to open the live demo http://sdc.beda.software/ a white screen is displayed.
White screen should be replaced with a text information about launch mechanism, default login and password and a link to run ide on top of EMR https://sdc.beda.software?client=sdc-ide

Looks it's not relevant anymore, closing the issue


public certificate needs to be rotated

The current TLS/SSL certificate for https://superknowa.tsglwatson.buildlab.cloud/ has expired and needs to be updated.

Updated the certificate.


affected/package:

What version of Go are you using (go version)?

$ go version

go version go1.20.6 windows/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\48940\AppData\Local\go-build
set GOENV=C:\Users\48940\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=D:\global\golang\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=D:\global\golang
set GOPRIVATE=
set GOPROXY=https://goproxy.cn,direct
set GOROOT=D:\software\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=D:\software\go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.20.6
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=NUL
set GOWORK=
set CGO_CFLAGS=-O2 -g
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-O2 -g
set CGO_FFLAGS=-O2 -g
set CGO_LDFLAGS=-O2 -g
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=C:\Users\48940\AppData\Local\Temp\go-build3248466755=/tmp/go-build -gno-record-gcc-switches
What did you do?

What did you expect to see?
I'd like to see the commands actually executed, i.e. escaped.
What did you see instead?
The output is not the real command, the real command will be spliced by some symbols will be escaped, and this command does not reflect the escape.

Duplicate of #15566


Search actor from within collection still only returning start matches

Seems as no bug, but tomcat misconfiguration or malfunction -> update tomcat


Added Norwegian Nynorsk Locale

Added Norwegian Nynorsk Locale

Thanks for the contribution -- may I ask why you closed this PR? There are a few places to update to add a new locale. Since this one is a sub-locale of Norway, it needs to be treated like pt-BR in that, for example, there won't be auto-translation. I can help get things in order if you need help.

Hi. This is my first PR ever so help would be much appreciated

Going to merge this and then apply changes in my own branch. Thanks for your contribution.


HIVE-27555: Upgrade issues with Kudu table on backend db

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

Is the change a dependency upgrade?

How was this patch tested?

@dengzhhu653 thanks for the patch! please fill the summary


Modify CI to build and release both with and without the raspberry feature

Presently the raspberry feature is not enabled in our CI workflow. Since we're unable to test builds for this without actually doing so on a Pi (or messing around with cross-compilation) it'd be best if we could check this automatically.

Hi! After some evaluation I found the following options:

Have the package and rust checks into the same file for raspberry targets.

This felt weird as raspberry targets had a single file for rust and package while the others targets had two.

Use a single file for Raspberry targets as in the first option, but merge the current rust.yml with the current package.yml, so that non-raspberry targets have also a single file.

Files become harder to read.

Split raspberry stuff into two files: One for rust checks and another for creating release packages, as we do for other targets.

It maintains the way files were organized and keeps the files relatively simple. There are still some differences between Raspberry targets and others, for example, rust checks for raspberry targets are executed in a matrix, while for the other targets they are not.
I went for option 3 in #255, but I am happy to change to any of the other option(it may not be in my list).


Dig 94445 dependabot

I suspect you didn't mean to open this PR upstream?  Looks to be a lot of local changes, though maybe some of the individual changes might be worth upstreaming?  Also really cool to learn that TotalWine is using imageproxy!


Do not wrap result into string

This will break yaj-rpc clients, but I will merge an push a fix for them ASAP


Add paroduscl and Infinite-File-Curtailer

There didn't seem to be an appropriate category for Infinite-File-Curtailer.  Please advise if I should create a new category for it.

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


✨Unique traces of technology

Description
Hello guys, how to determine master css through traces in page sources. Are there any unique traces?

Hey mate, сould you please share links to a couple of projects, otherwise I don’t even have examples, I wanted to add this technology to the Technology Lookup project https://www.wappalyzer.com/technologies/ui-frameworks/

@ceopeo https://rc.css.master.co/guide You can determine whether Master CSS technology is used by whether the page contains style#master.


Assignments

things that need to be fixed before I can accept this PR:

assignment 3 needs to have its content moved out of "analysis" and to the right level.
assignment 3 needs a video
assignment 4 needs to fix the table
assignment 4 needs two videos
there are no screenshots after assignment 3 -- they're needed for all assignments


Not working with NextJS/CommonJS (fix provided)

Describe the bug
Page on app will load, but once I refresh it complains about the import syntax
To Reproduce
Steps to reproduce the behavior:

Use NextJS, CommonJS, server side rendering
Implement this library
Refresh the page

Expected behavior
There should be support for cjs if possible, there are a few easy solutions available
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: iOS
Browser Chrome
Version 121.0.6167.160 (Official Build) (x86_64)

Solution

To avoid any other issues, an easy way is to npm init in the src folder to put a bare minimum package.json, and add "type": "module" at the top level
Alternatively, putting "type": "module" in your main package.json can also fix it, I just don't know if you'll run into any other issues?
Some of the other loaders will be able to handle it as well, there's lots of resources out there

In my experience, locally, I was able to do the first solution (npm init in src folder) and it's all working just fine now.
Thank you so much in advance! Let me know if I can help in any way

Hi @aafarian
Will you be able to introduce the issue in online sandbox environments? Like stackblitz

@bumbeishvili hi, thank you for the response! I built it on StackBlitz with NextJS: https://stackblitz.com/edit/stackblitz-starters-astamr?file=app%2Fsrc%2FOrganizationalChart.js
It seems to be working on StackBlitz, but it's basically re building the entire thing if I refresh the page. It works on first build for my application as well, and breaks when I refresh the screen. If you look into the error message above, you'll see that it has to do with the import structure not recognizing that it's a module until you move a package.json in the same directory.
I've done this locally with your project and it's working just fine for me now, no more errors


Improvements for Result tool tip

This PR implements the following features and changes related to the result tool tip:

Improved settings name and description including the German translation.
Support for showing path and main window title in tool tip.
Improved tool tip layout and showing command line only if it is enabled.

New tool tip layout (replacing the command line value):
Main window: xxxxx
Path: xxxxxx
Command line: xxxx

How was this tested
Unfortunately I have not enough disk space to test it yet and can not test it locally on my system. Please test everything before merging this in.

@8LWXpg
Here is the PR. Unfortunately I can't test it yet. (If you can make some screenshot on testing would be nice.)

@8LWXpg
Here is the PR. Unfortunately I can't test it yet. (If you can make some screenshot on testing would be nice.)

Unfortunately I have not enough disk space to test it

If you want to test it, you can run ProcessKiller/copyLib.ps1 which creates symlink under ProcessKiller/Lib and follow the instructions in README.

@8LWXpg
I did the requested changes.


Migrate configuration to use schema fields

Description

[ ] Bug fix
[ ] New feature

Useful links:

Baton SDK coding guidelines
New contributor guide

This is weird, why is CI failing? Was the integration test succeeding without an API key?


Escape/turn-off GO helm templating for a section of config file present in templates folder

Helm Version -> 3.2.0
Kubectl Version -> 1.17.0
Kubernetes Version -> 1.16
Cloud Provider -> AWS
Description ->
I am using Helm 3 and I am looking for way by which in the same YAML file I can turn Helms GO Templating off. For example, in the following snippet, I want that helm should not try to substitute {{ $value }} and let it pass as it is, so that Prometheus can actually fill in the $value as per the value of the alert expression. And also I want {{  .Values.global.namespace  }} to follow helm templating.
apiVersion: v1
kind: ConfigMap
metadata:
  name: prometheus-rules-conf
  labels:
    name: prometheus-rules-conf
  namespace: {{ .Values.global.namespace }}
data:
  prometheus.rules: |-
    groups:
    - name: demo.rules
      rules:
      - alert: TESTING NGINX POD STATUS
        expr: sum(kube_pod_info{namespace="test", pod=~"nginx.*"}) < 3
        for: 1m
        labels:
          severity: critical
        annotations:
          description: Nginx Trouble In Harmony Demo
          summary: The Nginx Pod count is {{ $value }} which is less than expected value of 3.

The error I get while running the command helm lint alert_manager is as follows

[ERROR] templates/: parse error at (alert-manager/templates/prometheus-rules-conf.yaml:20): undefined variable "$value"

duplicate of https://github.com/helm/helm/issues/2798. Use {{ "{{" }} to escape template syntax.


API: number-only values incorrectly rejected on string GET parameters

Hello,
Playing with the API (thanks again), I noticed some weird 400 Bad Request errors on apparently valid requests. It appears that parameters' type inference is way too strict.
All parameters in the API are typed, like, in the Hello world entry point, the name parameter type is string. But when we pass a GET parameter with a number-like value (lets say, ?name=128), the type inference system reads it as an integer and the request is… rejected with a 400 Bad Request, because the type is wrong (int is not string, right?).
Example and expected response
As example, let's say I send this HTTP request:
GET /api/demo?name=21 HTTP/1.1
Accept: application/json
Accept-Encoding: gzip, deflate
Authorization: Bearer TOKEN
Connection: keep-alive
Host: rcl.re
User-Agent: HTTPie/2.5.0
X-Signature: SIGN

This is the response:
HTTP/1.1 400 Bad Request
Cache-Control: no-cache, private
Content-Length: 103
Content-Type: text/html; charset=UTF-8
Date: Wed, 29 Sep 2021 09:48:17 GMT
Server: Google Frontend
X-Cloud-Trace-Context: b209146f958fe319a28a4f288a685977

{
    "code": "0001",
    "context": [],
    "message": "An error occurred, check response status code.",
    "success": false
}

This should be the expected response:
HTTP/1.1 200 OK
Cache-Control: no-cache, private
Content-Length: 103
Content-Type: text/html; charset=UTF-8
Date: Wed, 29 Sep 2021 09:50:06 GMT
Server: Google Frontend
X-Cloud-Trace-Context: ef94241edcf0679a6fb97e77f4508067

{
  "success": true,
  "payload": {
    "hello": "21"
  }
}

And as soon as I add a letter to the request parameter, everything work as intended:
GET /api/demo?name=21B HTTP/1.1
Accept: application/json
Accept-Encoding: gzip, deflate
Authorization: Bearer TOKEN
Connection: keep-alive
Host: rcl.re
User-Agent: HTTPie/2.5.0
X-Signature: SIGN

HTTP/1.1 200 OK
Access-Control-Allow-Origin: https://rcl.re
Cache-Control: no-cache, private
Content-Encoding: gzip
Content-Type: text/html; charset=UTF-8
Date: Wed, 29 Sep 2021 09:52:48 GMT
Server: Google Frontend
Transfer-Encoding: chunked
Vary: Accept-Encoding
X-Cloud-Trace-Context: 30fe1c268cbfe7c8d40c39520ac2db8b;o=1

{
    "payload": {
        "hello": "21B"
    },
    "success": true
}

Implications
This is problematic because some Pegass records identifiers are numbers, so requests like GET /api/admin/pegass?page=1&type=department&identifier=75 (to obtain the Parisian local units, just so I don't scan the whole list of volunteers) fail.
I tried to look at the source code but I don't really have experience with Sf4 (I used to work with Sf2 and Silex but that's pretty old now). This being said, if you have pointers, I would be happy to work on this myself.
Thank you very much!

Thank you Amaury,
During normalization, symfony serializer uses the @var annotations on the model object property in order to know whether the value is valid or not, in the demo case:
https://github.com/minutis-redcall/app/blob/master/symfony/bundles/api-bundle/Model/Facade/HelloRequestFacade.php#L16
But during the conversion of the query string into a flat json in order to map it to a model object, there's a type conversion on numeric values:
https://github.com/minutis-redcall/app/blob/master/symfony/bundles/api-bundle/ParamConverter/FacadeParamConverter.php#L37-L46
I need to find a way to only reject strings that do not contain numeric values when an integer is expected but a string value is given.
This issue only affects endpoints that use the GET verb.

Thank you Amaury,
During denormalization, symfony serializer uses the @var annotations on the model object property in order to know whether the value is valid or not, in the demo case:
https://github.com/minutis-redcall/app/blob/master/symfony/bundles/api-bundle/Model/Facade/HelloRequestFacade.php#L16
But during the conversion of the query string into a flat json in order to map it to a model object, there's a type conversion on numeric values:
https://github.com/minutis-redcall/app/blob/master/symfony/bundles/api-bundle/ParamConverter/FacadeParamConverter.php#L37-L46
A fix could be to allow float or int into the demo model object:
@var string|float

You may do that as a quick fix in the Pegass model object.
But it looks super verbose and quite not comfortable in the long run for all other model objects (ex: all "criteria" parameters in resource searches).
We better need to find a way to only reject strings that do not contain numeric values when an integer is expected but a string value is given.

I've found the root cause, parameters were strong typed only leaving strings as a possible value for these properties.
Removing that strong typing (until Google App Engine supports PHP 8) is the solution.
I'm pushing a fix.

Found the right fix (later is better than never 😅 )
https://github.com/minutis-redcall/app/commit/51cb17b940b772537cb54ea1b2ee50c2e0cf6659

Oh nice, thanks again!


Implementing very long drive cycle to solve DFN model

I'm trying to implement a drive cycle to solve the basic DFN half-cell model. There are around 5000 data points in the drive cycle, but whenever I try to use the current interpolator and then solve the model it gives a runtime error.  I am discretizing the model and then using the Casadi solver in the fast mode. The model can only be solved when I compress the data to around ~1300 points, but then the current interpolator is very inaccurate. Any suggestions on fixing the runtime error so I can pass the entire dataset into the current interpolator and then solve the half-cell model?

Can you try using casadi solver in safe mode?

Yes, I tried it in "safe" mode a couple times and it keeps crashing. The drive cycle is around 50,000 seconds long, if that helps.

Can you write a SPM half cell model and see how the drive cycle does? Would be good to make sure it doesn't go outside the voltage limits with the parameters you are using.

Ok, could you help with what parts of the DFN half cell code I have to change to do a SPM half cell model?

Basically DFN half cell is to basic DFN what SPM half cell would be to basic SPM. Maybe @molel-gt can help with this?

I still have not been able to run long cycles in Basic DFN half cell. Without measuring the time, I noticed that I could not run a simulation for 10 hours using the DFN half cell, but could do that with the full DFN model. The other thing would be to check the current density. The model fails during discharge for large current densities due to polarization.

Ok, I will check current density. Have you ever tried running a long cycle in a simpler half-cell model (SPM or SPMe)?

@sful can you try this with the recently updated models in https://github.com/pybamm-team/PyBaMM/blob/develop/examples/scripts/compare_lithium_ion_half_cell.py (without the SEI option)

Not enough details to fix, closing


Can I use this in a browser?

So it seems rust can easily be compiled to WebAssembly, at least that's what I read. Is there anything stopping me from throwing this in a compiler and having it generate an svg in the browser by throwing my dot source into the module? I don't really understand how to build or integrate WASM yet, this is like my first try of understanding that.

Apparently, you can.
See this demo: https://barafael.github.io/wasm-dot-renderer/
Code is here: https://github.com/barafael/wasm-dot-renderer
Please close the issue if this answers your question.

Ha nice, thanks a lot!


Custom Header Component Not Collapsed

Hi, I have a header with custom navigationOptions component. I am able to make header collapsed, however, its component is  not collapsed. Am I missing something?

Hi, can you check if you can reproduce the problem with the /example in the library?


Merge code from Trust

Fix eth_sign and personal_sign
Support eth_signTransaction and eth_sendTransaction
Expose error handler

close for this now, will prepare new pr


GraphQL API connection to local Dynamo DB

Note: If your question is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

** Which Category is your question related to? **
GraphQL and Dynamo DB
** What AWS Services are you utilizing? **
Amplify CLI / AppSync / DynamoDB
** Provide additional details e.g. code snippets **
Hi, sorry if this is a basic/misguided question, I'm just getting started.
I've managed to setup the backend of my React app using Amplify and have deployed via the Amplify Console. I created a GraphQL API using the Amplify CLI and have been able to connect to the created DynamoDB table. I want to continue developing the front-end locally (running yarn start) and would like my GraphQL queries to be against a local DynamoDB as I'm pulling data on every refresh and don't want this taking up my monthly calls. I'm not really sure where to begin, and any guidance is much appreciated! Thanks in advance.

@s-hood We currently don't have support for local testing but we're actively working on it.
cc: @yuth

Thanks for your reply! That's good to hear.
For now is there a global variable set or some other way I can determine if the app is running locally or not (i.e. from 'yarn start' or from publishing to aws)?

@s-hood take a look at AWSAppSyncClient and/or Apollo Client. You shouldn't need to query over the wire on every refresh -- that implies to me that your query is just initializing your application -- because of Apollo caching. https://www.apollographql.com/docs/react/advanced/caching
Are you querying your Dynamo table directly or via a GraphQL query?

For now is there a global variable set or some other way I can determine if the app is running locally or not (i.e. from 'yarn start' or from publishing to aws)?

I suppose you could check to see if window.location.origin includes "localhost" or is exactly equal to your test URL (including port). https://www.w3schools.com/jsref/prop_loc_origin.asp Not sure if this opens up any security concerns for your particular application.
You could also just check whether or not your application is online or offline, but it's a different question and may lead to unintended behavior or just not work the way you want it to. https://developer.mozilla.org/en-US/docs/Web/API/NavigatorOnLine/onLine

Okay thanks, I've managed a hacky work around for my local testing but I'll take a look at AWSAppSyncClient and try to cache that data instead.

Are you querying your Dynamo table directly or via a GraphQL query?

Via GraphQL

@s-hood mind sharing your local solution? I'm curious.
I asked b/c I initially got tripped up on AppSync subscriptions, which are not fired when Dynamo is directly manipulated instead of through AppSync mutations.

@jkeys-ecg-nmsu It's literally just a matter of running different Javascript to load in some local JSON files rather than running a GraphQL query in React's ComponentDidMount lifecyle method. I need to work on some front-end functionality so that does the job for now, when I need to work on my DB structure and querying I might run a local Dynamo and query that.

Oh, so you're mocking your data.
I think that the fact that AppSync subscriptions are not fired on direct mutations of underlying data sources is (1) theoretically sound and secure and (2) a minor hassle for developers. It means a developer has to convince Dynamo/NoSQL/RDS devs in their shops (as do I in my own organization) to learn GraphQL and the intricacies of AppSync in order to affect my designed front-end. It would be nice if Dynamo mutations took an AppSync API ARN as an optional parameter and broadcast subscriptions if it was passed with proper IAM authorization.

Okay, apologies for the misunderstanding. Agreed, that would be a useful feature.
Thanks for your time and help!

https://github.com/aws-amplify/amplify-cli/issues/1433

@s-hood We released support fo local testing of GrpahQL API with a local DynamoDB table. Please take a look at our launch blog to know more - https://aws.amazon.com/blogs/mobile/amplify-framework-local-mocking/


Change "top left corner" to "top right corner" in note on landing page

The landing page suggests switching the basemap to satellite imagery using a toggle in the upper left corner, but that toggle is in the upper right corner:

Closing this PR, as it seems this has been fixed elsewhere :)


Static content from libraries not working with Blazor dev server or default hosted project

The new "static content from libraries and projects" feature works great if you have a app.UseStaticFiles() call in your Startup.cs. However:

The Blazor DevServer (for the standalone template) doesn't contain such a call, and developers have no way to add one. So static content just doesn't work with this project type.
The Blazor hosted-on-ASP.NET-Core template also doesn't contain such a call, although developers can easily add one to their Startup.cs.

Both fixes could be trivial one-liners.
Better fix: Also delete the now-obsolete embedded content feature from Blazor, and update the E2E tests to use the new feature.

@SteveSandersonMS I am not able to fix the problem in a client-side Blazor app using the hosted-on-ASP.NET-Core template even after the call to app.UseStaticFiles(). Here is a repo with the project: https://github.com/Stamo-Gochev/blazor-wasm-static-assets
Am I missing something here? The same test works with a server-side Blazor app that contains the call to app.UseStaticFiles().

@Stamo-Gochev I'm investigating. I'll keep this issue posted.


Fix Omnisci dbe import for Windows

@igalink Why it was closed and removed?..

@vnlitvinov I asked for the branch to come from personal fork, not main fork. @igalink kindly made that change.


Topic/mnaveau/noetic packaging

First shot at ROS1 packaging
The docker used to develop is here:
https://gitlab.laas.fr/kwojciecho/gepetto-dev-container

Sorry this was not finished I should I have put a wip.
I will re-create it from the noetic-devel branch when it's a bit cleaner.


PDK_STRUCTURE.md: fix typo.

The wrong name has been deprecated.

@tgingold, we're well aware of this typo. But, the variable name in the open_pdks used currently has this typo. So, we're waiting for updating it there along with some other stuff and then we'll correct that typo here as well (merge your PR).
Thank you!

@tgingold: it's finally updated :) , so I'll merge this. Thank you!


drop queries when processing takes to long (> 6s)

Just filed https://github.com/coredns/coredns/issues/2663 to think about dropping queries when forwarding. This may make sense to do in this lib (although I'm not yet convinced it should)

This feels like a good idea for it to be in CoreDNS, but I don't think it makes sense in this library. Timeouts can already be set on the Client if desired, and I don't think there's a clean place for them in the server. Also, isn't it completely valid for queries in certain configurations (things like zone transfers) to take quite a long time?

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] drop queries when p..." ]

This feels like a good idea for it to be in CoreDNS, but I don't think it makes sense in this library. Timeouts can already be set on the Client if desired, and I don't think there's a clean place for them in the server. Also, isn't it completely valid for queries in certain configurations (things like zone transfers) to take quite a long time?

Yeah, I think this is too hard to do in a generic way.


create_modal or create_view ? path to custom create modal ?

Hi,
thank you for this package.
In My project, I have purchases linked to supplier (1-n).
I have inserted an instant field into Purchase to trigger Suppliers.
But, my suppliers  views (create and edit) are custom views (I needed to add some js here).
The create_modal and edit_modal options are made for me, if I understood correctly, but I can't figure out how to use it.
In my SupplierCrudController, I have these lines
$this->crud->setCreateView('vendor.backpack.crud.supplier-create');
$this->crud->setEditView('vendor.backpack.crud.supplier-edit');

So I tried
'on_the_fly'           => [
                'entity'        => 'supplier', // e. g. user, contact, company etc...
                'create_modal'  => 'vendor.backpack.crud.modal.supplier-create',
                'edit_modal'    => 'vendor.backpack.crud.modal.supplier-edit'
]

Result:
GET http://myproject.localhost/admin/purchase/vendor.backpack.crud.modal.supplier-create 404 (Not Found)
I tried many others path types, without best results.
In the doc, you write also
'on_the_fly' => [
    'entity' => 'entity',
    'create_view => 'route/to/modal/html'
]

Is create_view  another option, or is this a typo ?

Thanks for the hint - I just updated the inconsistency of the readme.
I will try to strip down some things:

IMO the modal should be automatically generated (even if you have set a custom create or edit blade in your SupplierCrudController). Did you try without create_modal and edit_modal attributes?

If you really need to use custom modals:
Please be sure to provide a valid url (relative or absolute)! You just used a view-string and this does not work. On loading the modal a request (to the provided url) is fired and the resulting HTML will be filled in the modal.

Thank you for your reply.
Without create_modal or edit_modal, the form loads perfectly.
But if I created custom create_view and edit_view for this model, it's because I needed to add some javascript (some fields must be visible / hidden according to a checkbox).
And this javascript is not included in the modal, which is normal I think.
If I add javascript directly to your code (\vendor\webfactor\laravel-backpack-instant-fields\resources\views\modal\create.blade.php or edit.blade.php), it works as expected. But it's bad practice.
So I copy your files into my resources folder:
\vendor\webfactor\laravel-backpack-instant-fields\resources\views\modal\create.blade.php (and edit) =>
\resources\views\vendor\backpack\crud\modal\supplier-create.blade.php (and edit)
and I add the javascript in my files.
I'm now trying to use create_modal and edit_modal to call my modal, but I can not find what I have to put as a path for these variables.
Do I have to create a method that returns the code of my modal, and a route to call that method. And this is the route I should put in create_modal?
I'm going to try. But then it would be nice to be able to indicate in create_modal the path to the view, and not the path to a method that returns the view, if possible.

Can you please post an example of the javascript you want to use in the fields? I would like to test whether there is a way to make it work out of the box.
I got your point on defining the view, I will think about it.
For now you could overwrite the ajaxCreate() and ajaxEdit() methods in your SupplierCrudControlller (s. Line 72/91 of HandlesAjaxRequest.php in our package. There you would just have to switch the used view.

Here is the code of my custom create view. No change in the html part comparing to backpack view, only added @push ('after_scripts') ... @endpush
@extends('backpack::layout')

@section('header')

        {!! $crud->getHeading() ?? $crud->entity_name_plural !!}
        {!! $crud->getSubheading() ?? trans('backpack::crud.add').' '.$crud->entity_name !!}.

{{ trans('backpack::crud.admin') }}

route) }}" class="text-capitalize">{{ $crud->entity_name_plural }}

{{ trans('backpack::crud.add') }}

@endsection

@section('content')
@if ($crud->hasAccess('list'))
	route) }}" class="hidden-print"> {{ trans('backpack::crud.back_to_all') }} {{ $crud->entity_name_plural }}
@endif

getCreateContentClass() }}">

		@include('crud::inc.grouped_errors')

		  <form method="post"
		  		action="{{ url($crud->route) }}"
				@if ($crud->hasUploadFields('create'))
				enctype="multipart/form-data"
				@endif
		  		>
		  {!! csrf_field() !!}

		      @if(view()->exists('vendor.backpack.crud.form_content'))
		      	@include('vendor.backpack.crud.form_content', [ 'fields' => $crud->getFields('create'), 'action' => 'create' ])
		      @else
		      	@include('crud::form_content', [ 'fields' => $crud->getFields('create'), 'action' => 'create' ])
		      @endif

                @include('crud::inc.form_save_buttons')

		  </form>

@endsection

@push('after_scripts')

@endpush

@DonDiegoAA
I just released v2.2 where you can define create_modal_view, edit_modal_view and delete_modal_view, so you don't have to overwrite ajaxCreate() (and the other methods) anymore or even custom routes.
Hope this will work, I didn't make detailed tests.


unsupported proxy type

tun2socks.exe  -tunAddr <IP_ADDRESS> -tunMask <IP_ADDRESS> -tunGw <IP_ADDRESS> -tunName "SSTAP" -tunDns <IP_ADDRESS>,<IP_ADDRESS> -proxyType socks
2020/07/29 10:09:20 TAP device name: SSTAP
2020/07/29 10:09:20 Set SSTAP with net/mask: <IP_ADDRESS>/<IP_ADDRESS> through DHCP
2020/07/29 10:09:20 Set SSTAP with DNS: <IP_ADDRESS>,<IP_ADDRESS> through DHCP
2020/07/29 10:09:20 unsupported proxy type

I believe you're not running binaries download from the releases page.


couchdb uses shared adapter main

Proposed Changes

consolidate couchdb adapter main code.

/assign @vaikas-google

/lgtm

@vaikas-google can you approve?


[jvm-packages] Feature Constraints param in XGBoost4J-Spark

I am training xgboost model on the spark in distributed manner. I want to use interaction_constraint and monotone_comnstraint, but I am unable to find a way to do so.
I am aware that it is supported in in-memory python, but It will be helpful if it is supported in spark.

@CodingCat Do we have distributed hist in 0.82-SNAPSHOT?

Yes,

@akhilbatra898 You should try using 0.82-SNAPSHOT. The 0.82 release is currently in the work

@hcho3
Is there any documentation how to use feature constraints in spark. As per my understanding one cannot pass them along with general params.

you can...check https://github.com/dmlc/xgboost/blob/master/jvm-packages/xgboost4j-spark/src/test/scala/ml/dmlc/xgboost4j/scala/spark/XGBoostGeneralSuite.scala#L126

Hi @CodingCat
I tried the examples from test case. Apparantly initialising Classifier like this ignores extra params like interaction constraints and monotone constraints and only use general params for training.
Can you check?

which versiok are you using?

I have tried on both 0.82 and 0.81 snapshot

I validated by instrumenting in https://github.com/dmlc/xgboost/blob/master/src/tree/split_evaluator.cc#L240-L247
and https://github.com/dmlc/xgboost/blob/master/src/tree/split_evaluator.cc#L413-L421
I can see the specified constraints is well received in native layer
Can you compile the latest master branch and try?


Active submenu

Maybe we need active submenu to know which menu we chose? Maybe we can add underline or different color text or something

Hi,
what do you mean exactly?
I would say its all there. Screenshot from the demo page.

Hi,
what do you mean exactly?
I would say its all there. Screenshot from the demo page.

Hmmmm look like i have problem with my browser. My browser does'nt


Bindings in oracle have different format

There is need to handle oracle bindings: :p0 instead of @p0.
Solution 1
40fd8a899a0ad5873ab8a294dd3deb04067d498f, then in type OracleSqlResult overload method NamedBindingFormat.
Solution 2
Move NamedBinding mechanism to Compiler class.
Solution 3
26536011de6a947faf68b0c74880ace929d7ea0b
Which one would you prefer?

I think the 2nd solution is more appropriate, leaving this to the compiler is easier to maintain and extend.
but I am thinking why this is not handled by Dapper?

The SQL is passed through as-is, and must contain the correct :param1 or @param1 etc.

~Marc Gravell source: https://stackoverflow.com/questions/20962405/dapper-net-oracle-parameter

In SqlResult class there is + operator overload which, in compiler solution, is problematic. All solutions that come to my mind end up with storing db type information in SqlResult. In my opinion this is acceptable, because being created by db-specific compiler, SqlClass is by definition db-specific.
+ operator implementation has to somehow regenerate named bindings. It can be done for example by concatenating 0 and 1 to the end of parameter names, but this will generate bad looking and unnecessary long names when summing many SqlResults. Rewriting numbers inside names would be not efficient.
If we want to have single SqlResult class we could create NamedBindingParameters class to SqlQuery which different implementations would contain all logic required.
Considering all, I think that solution 1 is the most clean and easy one.

Ok go with it, if it's the most approachable way to do that.
Maybe this part need to be redesigned in a better approach, but this not a priority right now.

I tried to move binding mechanism to compiler, and adjust "+" operator. I had to move SqlResult combining logic to compiler as well.
Please look at commit fb08756b334073507ccffce7d7a32672c05b1854

@tlaguz, sorry for the late reply, but I was a bit overloaded, I still need some time to review it, I will get back to you the soonest.

@ahmad-moussawi Have you found time to review it? I would like to continue with oracle compiler

@tlaguz, sorry for the late reply, I agree going with the first option is acceptable, in 2.0 we can replan if there is a more appropriate choice.


The LoadingState.hx Problem

Describe your problem here.
source/LoadingState.hx:49: characters 89-93 : Expected , or )
I Just Trying To Add The Radnom LoadingScreen On LoadingState.hx, But It's Still Showing The Error.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Just LoadingState

Just put a , or ) whatever it's on that line

Just put a , or ) whatever it's missing on that line

Nevermind, I Fixed


App Crash on Ios when using MergeResourceDictionary

I create a app using your styles. I have a problem with MergeResourceDictionary. My app crashed on Ios. Android still Ok. When I remove MergeResourceDictionary and copy all code to file whitetheme.xaml. My app not crash on all version. What's problems with MergeResourceDictionary on Ios ?

Hello @nhdanh, in this application I have not had those problems, send me a capture or the path of your project to verify the problem and solve it.


Need centralize Logging for the A2HA

Acceptance Criteria

Logs aggregation should works for both cloud and non cloud solution.
In Case customer don't want centralize, they can still use the current approach.

Definition of Done

Need to ship the log from a2ha cluster to the centralised place.
We need to keep logs separated automate, chef-server, PG and ES logs
We need to do Logs rotation at the centralised place (to make sure that disk goes out of space).

Testing Steps


Correct typo

In the Dutch interface, it says 'woorsoort' somewhere, instead of 'woordsoort'. Please correct.

Fixed

@Woseseltops @vanlummelhuizen Could you check why this is not online yet?

I actually already knew without checking, @ocrasborn :) ... The translation was there, but it was not 'compiled yet' (see https://github.com/Signbank/NGT-signbank/wiki/How-to-translate-texts-for-Signbank#making-changes-visible-online) . I ran the command, and now it is!


Generate better API documentation

/cc @yurydelendik  @brendandahl
Initiating a thread discussing ways to improving the docs and feedback for the same.
Starting with missing comments on the API docs page

/botio-windows preview

Nice!
Can you also fix double entry of PDFDocumentLoadingTask (see http://<IP_ADDRESS>:8877/824a96518b07a1c/api/draft/PDFDocumentLoadingTask.html)? Interesting that PDFDocumentProxy (see http://<IP_ADDRESS>:8877/824a96518b07a1c/api/draft/PDFDocumentProxy.html) has no such issue.

I agree, this is looking much better now! Fortunately the update to JSDoc 3.3.3 also included a better lay-out, so it is more pleasant to look at too.

@timvandermeij Update JSDoc for PDFJS too?

No, we already use JCDoc 3.3.3 as can be seen on the last text line of http://<IP_ADDRESS>:8877/824a96518b07a1c/api/draft/PDFDocumentLoadingTask.html. I just meant to say that the general look of the documentation has improved with that update from recently. No need to do any updates, sorry for the confusion :)

@timvandermeij Oh! I thought you were referring to the version in package.json which is 3.3.0-alpha9 Thank you :)
Anyways, about adding more examples within the docs, how do I get started?

Back to how it was before along with the fix

/botio-linux preview

Looks good. Thank you for the patch.


Syntax Error

I am getting a syntax error now that I am using 2.0.0 in models.py Line 47
.env/lib/python3.5/site-packages/bloop-2.0.0-py3.5.egg/bloop/models.py", line 47     abstract: bool             ^ SyntaxError: invalid syntax
Any ideas?

Thanks for opening an issue, and sorry you're running into this problem.  This is because bloop 2.0 uses new syntax from python 3.6+ (PEP-526).

Closing this out since it's likely a python version issue.  Feel free to re-open with additional debugging info.


Update search options according to available trainings

I've been working on Plone Sphinx Theme and released it, which will incorporate Nuclia Search in a future release. I'm afraid that this work will be superseded by that work in a couple of weeks. I've been posting in Community, Discord, team meetings, and issue trackers this plan.

I am looking forward to your new beautiful theme with Nuclia search. It will be great to have all resources included in the search. I set it up locally and it looks very nice.
In the meantime the search filter options could be updated. By now the options do include trainings that are not anymore available on docs.plone.org.
I am aware that this search will be replaced by a Nuclia search.

Excellent. FYI, I plan to work remotely on integrating Nuclia search at Buschenschanksprint, in case you want to join in. We also need to take a snapshot of 2023 trainings and prepare for 2024.


Ensure DocDB is encrypted at rest (default is unencrypted)

Possible check to target the following resource

[ ] aws_docdb_cluster_instance

i can take this one. is there an auto-assign bot or do you have to manually assign it?

i can take this one. is there an auto-assign bot or do you have to manually assign it?

Just manually assign it for now, thanks

closing, tracking in spreadsheet

closing, tracking in spreadsheet