feat: revise apis

Applied fit and predict-style APIs instead of generate_code

@kimusaku Currently this doesn't work when save_datasets_format='csv' because you removed csv_delimiter, which is used in adaptation. I want to know your intention.

@kimusaku Currently this doesn't work when save_datasets_format='csv' because you removed csv_delimiter, which is used in adaptation. I want to know your intention.

@AkiraUra this is because I observed the test regarding csv_delimiter https://github.com/sapientml/sapientml/blob/main/tests/sapientml/test_sapientml.py#L504 did not raise any error and then thought there is no impact whatever I assign as a csv_delimiter. But thanks to your opinion, I rethink csv_delimiter must be required. I will revert the code related to csv_delimiter. Thanks.


Karsten - WeekChanger

Request to merge fully functional and tested WeekChanger component into the main branch.

This branch likely contains a slight navbar error, which is causing the PageRoutes conflict. Would like someone else to look it over before approving the request.

Requested closing. New Pull request will be created


Bare bones needs for file attachment functionality

[x] Need to toggle which attachments agreement holder can see
[x] Ideally some way to group them (folders/naming)
[x] file upload
[x] file delete
[x] file download
[x] tab or similar simple UI (Oamar to help figure out) off a RUP
[x] only staff can upload*  if its easy to allow AH too, then go for it
[x] AH can see files marked that they can see

List of allowable statuses https://github.com/bcgov/range-web/blob/dev/src/utils/helper/plan.js#L247


[Bug] Clockcount field does not show/update correct numbers

Description

Done number does not change and display when completing one or more clocks. There is data in the sqlite db.
Version

c4e3fa3b26601b305974fce70ea6654a402c9845 (latest master branch)
Steps to Reproduce

launch the clock
star
wait for the clock ends. answer all questions with yes
wait for the break clock ends

Expected Behavior

Done number bumps
Actual Behavior

Done number is always 0.
More Information
By closing and reopen the application, Done number is 1.

fix-up https://github.com/zztin/ctimer/pull/64


[#127062871] Track account creation using GA virtual page views

Currently, we are not able to tell using GA how many user accounts have been created. We could try to use a page view of the URL after an account is created to indicate this, however the URL is also accessible from other user flows and will not therefore be an accurate count of how many user accounts have been created.
Therefore, as part of this pivotal story, we wish to send a GA virtual page view every time a supplier account is successfully created. By embedding
data-analytics="trackPageView" data-url="/suppliers/vpv/?account-created=true"
into our template, we will send the required virtual page view on the success page(functionality already exists to do this automatically when the required data attributes are provided).
This tracking tag will only be embedded if they have come through the account creation process (indicated by setting a flash message).
Note, a similar process has been created for the creation of buyer accounts.

Cool, looks good and the existing JS tests already cover the new thing we've added here.
I'm going to hold off on thumbsing this one up until we've got a more definitive answer about sending double pageviews, which I described in more detail in a comment in the buyer app.
It seems a bit hypocritical of me because I already gave the other one a thumbs up, but I kind of think they'll both be fine.

Also 👍


Design data reporting APIs

Design Data Reporting - API, Service & Repository Layers.
Requirements: https://github.com/microsoft/ccf-app-samples/blob/main/data-reconciliation-app/docs/data-schema-data-flow.md#data-reporting
Members will query for report summary. All data returned will be JSON. Voting Threshold should be configurable.
Design APIs:

[x] by individual record (unique ID)
[x] for ALL data

Design Service Layer

[x] Design returned summary object. Table: https://github.com/microsoft/ccf-app-samples/blob/main/data-reconciliation-app/docs/data-schema-data-flow.md#data-results-schema. Summary object can be used for both APIs.
[x] Pseudo Code: how we will manipulate the domain model (K-V store) and create summary object?

Repository Layer:

Query K-V store by ID
Query K-V store for ALL

Domain Model according to this ADR

[ ] Design
[ ] Implement
[ ] Test

Acceptance Criteria:

ADR
Review ADR with team (Sync or Async)

All the api design and service layer connections are implemented on this branch: https://github.com/microsoft/ccf-app-samples/commits/canga/85-data-reporting-api
Remaining work is to

implement this ADR: https://github.com/microsoft/ccf-app-samples/blob/main/data-reconciliation-app/docs/adr/03-reconciliation-logic.md
Unit testing the domain model
Updating demo scripts to have e2e integration testing using curl

I'm closing this issue and remaining work will be tracked on #86


Spelling

Thank you for contributing to the CF CLI! Please read the following:

Please make sure you have implemented changes in line with the contributing guidelines
We're not allowed to accept any PRs without a signed CLA, no matter how small.
If your contribution falls under a company CLA but your membership is not public, expect delays while we confirm.
All new code requires tests to protect against regressions.
Contributions must be made against the appropriate branch. See the contributing guidelines
Contributions must conform to our style guide. Please reach out to us if you have questions.

Does this PR modify CLI v6, CLI v7, or CLI v8?
Not meaningfully afaict
Description of the Change
This PR corrects misspellings identified by the check-spelling action.
The misspellings have been reported at https://github.com/jsoref/cloudfoundry-cli/commit/b700f2163153a9ac80525141d48919cbfb8c4b0f#commitcomment-87631508
The action reports that the changes in this PR would make it happy: https://github.com/jsoref/cloudfoundry-cli/commit/dfec2652cfa3c22fb63c9a3eaf6524f6f4c46e89
Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.
Why Is This PR Valuable?
Typos make products harder to use and code harder to debug
Why Should This Be In Core?
Explain why this functionality should be in the cf CLI, as opposed to a plugin.
Applicable Issues
List any applicable Github Issues here
How Urgent Is The Change?
This will be the least urgent PR you'll see this year or next. Although the longer it takes to process the more merge conflicts will accumulate.
Other Relevant Parties
Who else is affected by the change?

@jsoref can you please prepare PRs for v8 and v7 based on f773611

@a-b done


search_path order not honoured for pg_catalog functions

CrateDB version
5.1.0
CrateDB setup information
No response
Steps to Reproduce
CREATE FUNCTION public.version() RETURNS TEXT LANGUAGE JAVASCRIPT AS 'function version() {return "PostgreSQL 10.5";}';
SET search_path TO public, pg_catalog;
SELECT version();
Expected Result
"PostgreSQL 10.5"
as in PostgreSQL
(tested with
CREATE FUNCTION version() RETURNS TEXT AS $$ BEGIN RETURN 'This is from the public schema'; END $$ LANGUAGE 'plpgsql';
)
Actual Result
CrateDB 5.1.0 (built 5c9617d/2022-10-11T09:39:39Z, Linux 5.19.0-23-generic amd64, OpenJDK 64-Bit Server VM 18.0.2+9)
Please note this works for pg_catalog tables, it seems to be an issue only for functions.
Repro'ed also with a mock generate_series.

CrateDB
CONNECT OK
cr> SHOW search_path;
+-----------------+
| setting         |
+-----------------+
| pg_catalog, doc |
+-----------------+
SHOW 1 row in set (0.013 sec)
cr> SET search_path TO public;
SET OK, 0 rows affected  (0.006 sec)
cr> SHOW search_path;
+--------------------+
| setting            |
+--------------------+
| pg_catalog, public |
+--------------------+
SHOW 1 row in set (0.006 sec)

PostgreSQL 14
gtraar=# SHOW search_path;
   search_path
-----------------
 "$user", public
(1 row)

gtraar=# SET search_path TO public;
SET
gtraar=# SHOW search_path;
 search_path
-------------
 public
(1 row)

makes sense, if you explicitly define the order you expect to be respected.

Afaik it's intentional that you cannot shadow built-ins with user defined functions


Fixed misspelled Sprite in docs

Simple fix, happy to help!

Coverage remained the same at 100.0% when pulling c982d3771c5520634c294d44b36ce74bbbda73ae on programmeramera:fix/spelling into 66df8254a6ea5e2ae30424f9d4bd501f9f56a14b on straker:master.

Thanks for the pr. So the docs are auto generated from the JSDoc comments in the code. You'll need to fix the comment here for it to take affect.

Awesome thanks.


future proof the wireshark source path

the src/ directory only has the release version, so the ebuild breaks upon every release
the all-versions sub-dir won't have the same problem

Looks good, thanks. Travis says "The log length has exceeded the limit of 4 MB" so I'll try locally.


JSX spread children issue

Trying to parse, and my computer ran out of memory.
Steps to reproduce

esprima.parse('<div {...c}> {...children}{a}{...b}</div>', { jsx: true})

Expected output
Should parse
Actual output
Throws and run out of memory

It should not consume any memory at all. It should throw an exception "Unexpected token ..." because the parser does not known the syntax (adding support for JSXSpreadChild will be tracked as #1819)


Topic factory

ATopicFactory interface and a default implementation.  This interface defines what is needed when a topic is automatically created.  Users can define different implementations.
This change documents the KMProducer and KMConsumer interfaces.

Forgot to rebase before submitting this PR.


Add Kerbal Galaxy Revamped  from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of TheJangleMan, to add Kerbal Galaxy Revamped  to CKAN.
Mod details:
name = /mod/872/Kerbal%20Galaxy%20Revamped%20
author = TheJangleMan
abstract = Adds cool and unique planets to your game! Pictures: http://imgur.com/a/d7zNF
license = Creative Commons
Homepage =
description = This is your mod listing! You can edit it as much as you like before you make it public.
To edit this text, you can click on the "Edit this Mod" button up there.
By the way, you have a lot of flexibility here. You can embed YouTube videos or screenshots. Be creative.
You can check out the SpaceDock markdown documentation for tips.
Thanks for hosting your mod on SpaceDock!
Please direct questions about this pull request to TheJangleMan.

Closed in favour of #4481


Test Cases Needed

[x] Test for summarizenegs
[x] Test for setsegmentqcflags matches the qc after all other segment QC set
[x] Test that if no nuclei or area does not perform those qc without error
[x] Add test Spec 3 for probe qc, targs with less than 3 probes get no grubbs flags
[x] #115
[x] checkQCFlags test (pushed back a release)
[x] all utils functions need test
[x] add countsshiftedbyone to accessors test cases (#109)
[x] when the feature type is "Target"
[x] check error message in agg counts when there are not "Negatives"
[x] check error message in agg counts when there are not multiprobe targets
[x] Add tests to check single probe counts are correctly handled in agg
[x] Update test for rewritten background subtraction function (#110)

GxT v. DA script to be added as attachment in QB:

[x] Test results match for segment QC
[x] Test results match for probe QC

Test cases for DA comparison were added in a separate test file and added to quickbase


[TASK] Add ability to maximize panel

To allow users the ability to maximize a panel, upgrade to the latest version of flexlayout-react

Dependent on #359


Removed Connect method from IDistributedCache

This change is going to need changes in other repos, for example Session which calls the Connect method.

cc @lodejard @muratg @Tratcher

LGTM

Will you also be removing ISessionStore.Connect? This will move the cache initialization from startup (https://github.com/aspnet/Session/blob/e91ce99c70614efab26825f685dd026e721b0fa0/src/Microsoft.AspNet.Session/SessionMiddleware.cs#L65) to first access and make issues harder to debug.

Rebased and merged.


Updating buildToolsVersion to minimum supported by Gradle plugin 3.2.1

Gradle plugin 3.2.1 (used by ad-accounts) requires buildToolsVersion > = 28.0.3

Need to check MSAL buildToolsVersion for compat

Travis-CI config must be updated

Thanks @kreedula - merged in latest, will build and merge if all is 👍


Phoneme and Viseme Recognition

Hi, I think the addon is super cool and very similar to what I am looking for. Key word tho "similar".
The addon seems to be able to detect full sentences, but I don't see any option to detect the "dominant" phoneme or letter that is being current pronounced. This could be very helpful in adding a live lip-sync to the project. Another issue I had is how long it takes to detect the words, even when playing with the parameters of the detection node. I was originally going to try using the output words in some way instead, but the delay is too much for what I need.
I hope if phoneme/ viseme support gets added, that massive delay would be way less. The addon is super cool and really would love to use it further in my projects!
Thanks :D

I made a few new friends from the whispering tiger project. Will be trying their suggestions of large v2 and large v3. https://github.com/Sharrnah/whispering-ui

I made a few new friends from the whispering tiger project. Will be trying their suggestions of large v2 and large v3. https://github.com/Sharrnah/whispering-ui

Very interesting, what is the difference between the version and how would they help implement the feature? 👀


Does not compile under Windows

Lots of errors:
position.cpp: In member function 'bool Clovis::Position::do_move(Clovis::Move, bool)':
position.cpp:303:9: error: jump to label 'nullmove'
303 |         nullmove:
|         ^~~~~~~~
position.cpp:208:22: note:   from here
208 |                 goto nullmove;
|                      ^~~~~~~~
position.cpp:213:19: note:   crosses initialization of 'Clovis::Piece piece'
213 |             Piece piece = move_piece_type(m);
|                   ^~~~~
position.cpp:212:20: note:   crosses initialization of 'Clovis::Square tar'
212 |             Square tar = move_to_sq(m);
|                    ^~~
position.cpp:211:20: note:   crosses initialization of 'Clovis::Square src'
211 |             Square src = move_from_sq(m);
|                    ^~~
position.cpp: In member function 'void Clovis::Position::undo_move(Clovis::Move)':
position.cpp:388:5: error: jump to label 'nullmove'
388 |     nullmove:
|     ^~~~~~~~
position.cpp:350:18: note:   from here
350 |             goto nullmove;
|                  ^~~~~~~~
position.cpp:353:16: note:   crosses initialization of 'Clovis::Square tar'
353 |         Square tar = move_to_sq(m);
|                ^~~
position.cpp:352:16: note:   crosses initialization of 'Clovis::Square src'
352 |         Square src = move_from_sq(m);
|                ^~~
search.cpp: In function 'void Clovis::Search::init_lmr_tables()':
search.cpp:36:60: error: 'log' was not declared in this scope; did you mean 'long'?
36 |                     lmr_table[depth][ordered] = int(0.75 + log(depth) * log(ordered) / 2.25);
|                                                            ^~~
|                                                            long
search.cpp: In function 'Clovis::Move Clovis::Search::start_search(Clovis::Position&, SearchLimits&)':
search.cpp:107:9: error: jump to label 'end'
107 |         end:
|         ^~~
search.cpp:59:22: note:   from here
59 |                 goto end;
|                      ^~~
search.cpp:69:17: note:   crosses initialization of 'int beta'
69 |             int beta = POS_INF;
|                 ^~~~
search.cpp:68:17: note:   crosses initialization of 'int alpha'
68 |             int alpha = NEG_INF;
|                 ^~~~~

Should be fixed in release 1.1!

Yes, it is all right now.
However:

If I leave the makefile where it is (outside of src) then I get
File not found - *.cpp
File not found - *.cpp
g++: fatal error: no input files
compilation terminated.
mingw32-make: *** [Makefile:11: build/Clovis] Error 1
If I move the makefile to the src folder then I get
FIND: Parameter format not correct
FIND: Parameter format not correct
g++: fatal error: no input files
compilation terminated.
mingw32-make: *** [Makefile:11: build/Clovis] Error 1

So I had to compile it directly via the command "g++ *.cpp -O3 -march=native -flto -oClovis_1.1-x64-SzG.exe"
Please note that I am a complete stranger to makefiles so I don't know how to edit the makefile to fix the above issue.


The assemble workflow should have a function to wrap build artifacts into a RPM package

Tasks
Estimate
Status
Notes
Dependencies

Just like build workflow, add a “--distribution” parameter, so that the assemble workflow knows which distribution to create based on the input artifacts from build workflow 20220220: Design change, since build workflow already have --distribution ready, assemble workflow can directly take distribution info from build manifest. Then assemble the new rpm after zips installation.
3
In Progress

The existing distribution generating code of TAR and ZIP should still function after the above change
3
In Progress
20220220: #1659

20220220:

https://github.com/opensearch-project/opensearch-build/pull/1659

20220228: Improvements on maps: https://github.com/opensearch-project/opensearch-build/pull/1676 ✅


Add expiration and message retention options to Subscription

We create many dynamic subcriptions during a day
(because of deployments) for our search mechanism.
It left some long lived subcriptions that we don't use. It can
be dangerous because we have limit of how many subcriptions we
can have. The default expiration of a inactive subcrption is
31 days, we want to reduce it to the minimum value that is 1 day.
However, according to the docs, the subcrption expiration should
always be greater than message retention duration. The default
message retention is 7 days, so we need to reduce it, I think
a good one would be the minimum again, 10 minutes.
Disclaimer: I didn't tested yet against against the real lived
system. I coded only using the docs aviables. So my next steps:

[x] Test pubsub emulator
[x] Test in staging with real pusub

@cachemoi Our plan is to merge it back to the library author and seems he doesn't put any link to the pubsub docs, so I'll keep it without, and ask him if we should add.


Add apistatus online/offline intregration

Here's an integration for my new apistatus service, which I think will be great for adding to API readme's and docs. The last URL segment is a base64 encoded URL which my API makes a request to and returns structured data about the response.
I also made a simple website that does the base64 encoding, sends a request with a preview of the response and generates the badge URL.

Thanks for contributing!
Could you remove commit 76dfd4d? Part of it is irrelevant to this PR, the other part is inaccurate (https://apistatus.org isn't a service that produces images that comply with the standard we define).
Adding your service requires maintenance on my part, so I'd like to make sure you believe your API to be frozen for at least 18 months.
As an aside, http://api.apistatus.org/?url=aHR0cHM6Ly9tb2NrYmluLmNvbS9zdGF0dXMvMjAw returns {"online":false}, while your website shows that it should be online.
I believe, in the current state of your project, that you should download the badge you need (either https://img.shields.io/badge/API-online-brightgreen.svg or https://img.shields.io/badge/API-offline-red.svg) from shields.io, and deliver it to an endpoint on your server, eg. api.apistatus.org/badge/<base64 of the API endpoint to test>.

Could you remove commit 76dfd4d? Part of it is irrelevant to this PR, the other part is inaccurate (https://apistatus.org isn't a service that produces images that comply with the standard we define).

Done, my mistake.

Adding your service requires maintenance on my part, so I'd like to make sure you believe your API to be frozen for at least 18 months.

What maintenance on your part are you referring to? In any case the service will be online for at least the next 18 months as it's going to be integral to another product I'm working on.

As an aside, http://api.apistatus.org/?url=aHR0cHM6Ly9tb2NrYmluLmNvbS9zdGF0dXMvMjAw returns {"online":false}, while your website shows that it should be online.

That's because it works by checking the actual URL, not a base64 encoded version on one. I originally had the shields.io implementation do the same but hit a snag with the regex routing so I added the base64 decoding into shields.io the server.

I believe, in the current state of your project, that you should download the badge you need (either https://img.shields.io/badge/API-online-brightgreen.svg or https://img.shields.io/badge/API-offline-red.svg) from shields.io, and deliver it to an endpoint on your server, eg. api.apistatus.org/badge/.

I could do that as well, would this then make me eligible for point one?

What maintenance on your part are you referring to?

Shifting APIs, deprecation, anything that requires a change in shields to continue providing the same services.

it's going to be integral to another product I'm working on.

Out of curiosity, what product is that?

That's because it works by checking the actual URL, not a base64 encoded version on one.

Ah, I see. That's smart.

would this then make me eligible for point one?

Could you detail what you mean by "point one"?
My understanding of the way apistatus.org works is that it simply performs a HTTP HEAD and returns some JSON to give exactly the information that the HEAD request gives. It may be more efficient to perform the HEAD request from shields.io, instead of performing a full POST to apistatus.org, waiting for the answer, and parsing the JSON. What do you think?

Out of curiosity, what product is that?

PublicAPIs.com and the next generation of Mashape.com's API directory.

Could you detail what you mean by "point one"?

In the original reply your first paragraph mentioned "https://apistatus.org isn't a service that produces images that comply with the standard we define.", so if I added the SVG to produce the image then it should satisfy that point.

My understanding of the way apistatus.org works is that it simply performs a HTTP HEAD and returns some JSON to give exactly the information that the HEAD request gives. It may be more efficient to perform the HEAD request from shields.io, instead of performing a full POST to apistatus.org, waiting for the answer, and parsing the JSON. What do you think?

Actually it makes a GET request (since some servers may not accept HEAD requests) and then takes that response and turns it into the structured JSON you see. I'll probably also add more properties like content-type and body size but won't be changing any existing properties. The logic could of course also be done in shields.io itself, but I don't see any reason to duplicate the code.

I'm sorry, but I don't think it will be useful to people in the proposed form. You can certainly provide shields.io badges from your website.

In the original reply your first paragraph mentioned "https://apistatus.org isn't a service that produces images that comply with the standard we define.", so if I added the logic to produce the badges myself then it should satisfy that point.

If that's what floats your boat, I suppose I'd accept. If your motivation is fame, that page is hardly ever read, even amongst contributors, let alone the rest. I don't really see why you'd want to do it, and I can't think of any valid motive, but if you have one, why not.

I'm sorry, but I don't think it will be useful to people in the proposed form. You can certainly provide shields.io badges from your website.

Thanks for your feedback, I will add an endpoint that proxies the badges so people can get them from the apistatus api directly from something like http://api.apistatus.org/badge?url=https://restcountries.eu/rest/v1/all. The whole point is really to provide a live snapshot of an APIs current status. We'll be implementing them into our API documentation, github repos and API management tools so I figured others would make use of them as well.

If that's what floats your boat, I suppose I'd accept. If your motivation is fame, that page is hardly ever read, even amongst contributors, let alone the rest. I don't really see why you'd want to do it, and I can't think of any valid motive, but if you have one, why not.

I was only replying to your comments, I'm not sure why you chose to assume I had an ulterior motive.

I added some labels to the try.html and badge labels themselves to hopefully make it more clear. Here's a screenshot of what an implementation could look like:

This project looks to be dormant: https://github.com/Mashape/apistatus
apistatus.org no longer goes anywhere, nor publicapis.com


How to force rerender of model when stlModel changes ?

Describe the bug
Hello, i am building a small app where the main feature is to display .stl Models uploaded by the user. I can successfully display a single uploaded model, but when the stlModel property value changes (it is binded to a string which represent a served file from a server) the viewer doesn't change the displayed object). Is there a way to force - render on the viewer in order to display the latest uploaded model ?
To Reproduce
Steps to reproduce the behavior:

Create a small app with a stl viewer, in which the [stlModel] property is binded to an input.
Change input to a valid stlModel
stl Viewer does not render the correct Model, continues to display the first binded model

Expected behavior
Viewer should render a new model everytime the input (stlModel property) changes
Desktop (please complete the following information):

Ubuntu 18.04
Firefox & Chrome (latest)

I'm looking for the exact same thing. :(

I have found a workaround to this by using arrays.
the template looks like this
 <stl-model-viewer *ngFor="let stl of STLs"  [stlModel]=stl.file [material]=material></stl-model-viewer>

And in the ts I have an array of objects.
STLs = [{file : this.selectedFile, update: 0}]

where I update the STL path itself. The update key is being used to force updates when the filepath stays the same but I need to update the material.

Thank you for the comment, i will try your method and see if it suits my needs.
Just to inform other users in the future:
For reasons i cannot comprehend, if i use a *ngIf directive on the stl-viewer with binded model property, the first time it "shows" it can successfully render the selected stlModel. So, for now, the "workaround" is simply to set the variable controlling *ngIf to false and after a small amount of time to re-set it to true (using a setTimeout(..., 10ms) or something), thus triggering a rerender.
Disgusting code, but it works :)


Updated elgohr/Publish-Docker-Github-Action to a supported version (v5)

elgohr/Publish-Docker-Github-Action@master is not supported anymore

Closing this PR because we simplified our workflows some time ago, so we aren't using elgohr/Publish-Docker-Github-Action anymore.


New topic: AI new project with VS and Azure

Create new topic that is similar to Publish an ASP.NET Core web app to Azure App Service using Visual Studio
This was first requested in Issue 3373 "AI info and links. Explain how AI is hooked and show how to use it."
@wadepickett @scottaddie @rachelappel all agreed this should be a new topic, as it doesn't make sense to inundate the user with AI information in a level 100 basic how to on using Visual Studio. It's outside the scope of that doc.

There's a related issue from back in February. There's an outline there that may or may not help; but either way, there seems to be some overlap here or there: https://github.com/aspnet/Docs/issues/2800.
Also possibly related: https://github.com/dotnet/docs/issues/2716

Looks like the Azure side has a doc covering App Insights https://docs.microsoft.com/en-us/azure/application-insights/app-insights-asp-net
@guardrex  I'm not sure where AI is specifically mentioned in issue 2800. Looks like just Azure App Services - not App Insights.

Azure Integration lol ... sorry. My acronym-fu failed me.

Well "AI" is pretty vague. Let's just blame Rick since he started it. :D :D

@rachelappel @guardrex  so the goal is to build a doc for Azure AD Integration for a Core App? There are a few code samples for this. https://azure.microsoft.com/en-us/resources/samples/active-directory-b2c-dotnetcore-webapp/ for instance.

@isaac2004 AI = App Insights https://docs.microsoft.com/en-us/azure/application-insights/app-insights-asp-net

@rachelappel sorry confused, there is a comment by @guardrex that AI means Azure Integration

@isaac2004  So many acronyms it is impossible to keep track of what people mean. Anyway looks like I can close this.


MDX update to Plate 9

A long laundry list of things that we've left out of the initial POC and are now revisiting. This is all being worked on as a batch of updates and can be seen here.

Use our own components instead of the out-of-the box ones from Plate
Fix handling of block/inline embeds

Issues with tabbing through these elements
Inconsistencies with deleting and adding elements for uncollapsed collections

For links, instead of a window prompt to get the value, use a Tina form (like the image field)
Toolbar should be sticky so that it moves down the screen as you scroll
Support for quick action slash command to MDX elements

Toolbar should be sticky so that it moves down the screen as you scroll

Will this include the "embed" button? When I was testing out the documentation starter, it took me a second to realize I needed to scroll up to embed something:


internal/provider: expose the internal_dns field within the region model

Previously, there was no way to obtain the internal_dns field via the
terraform provider; the CC API does return the internal_dns field. This
commit updates the provider to include that field so that users could use it
for VPC Peering or AWS PrivateLink.

TFTR!


[Tests] Fix BotFrameworkAdapter.getTokenStatus() test

This test has been throwing an uncaught promise exception. I rewrote the test so that it properly fails, and skipped it because it was already failing in master. I will file an issue for someone to look into this test.

Originally posted by @stevengum in https://github.com/microsoft/botbuilder-js/pull/1337
This test has been failing for a while now, but the test was also incorrectly written and throwing uncaught promise exceptions.
This issue is to track fixing and reenabling the test.

Fixed via https://github.com/microsoft/botbuilder-js/pull/2414


mindustry: use libs from nixpkgs

Motivation for this change
Currently we use binary shared libraries from maven. These include statically linked variants of SDL2 and glew from the author's build machine. To run mindustry on wayland without Xwayland it is necessary to link against a different glew library.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This works now, but the whole file definitely needs a cleanup because it is currently very messy.
There are way too many fixed-output derivation hashes that need to be redone on every update. We should automate that with an update script (think nix-universal-prefetch) and also add a check that ensures the isn't outdated (like buildRustPackage does with the cargo vendor hash).

This works now, but the whole file definitely needs a cleanup because it is currently very messy.
There are way too many fixed-output derivation hashes that need to be redone on every update. We should automate that with an update script (think nix-universal-prefetch) and also add a check that ensures the isn't outdated (like buildRustPackage does with the cargo vendor hash).

I wonder if this fixes the aarch64 build too

I wonder if this fixes the aarch64 build too

Partially: libsdl-arc64.so can be built, but libarcarm64.so, which is also used for sprite packing at build-time, is still missing. On x86_64-linux this is blob is downloaded from maven. All of these binary downloads that are not easily packagable all the way down are so ugly I can't even, but this is a generic Java problem.
I'm also wondering how this affects non-linux platforms, but it's difficult for me to test this.

Partially: libsdl-arc64.so can be built, but libarcarm64.so, which is also used for sprite packing at build-time, is still missing. On x86_64-linux this is blob is downloaded from maven. All of these binary downloads that are not easily packagable all the way down are so ugly I can't even, but this is a generic Java problem.
I'm also wondering how this affects non-linux platforms, but it's difficult for me to test this.

https://github.com/Anuken/Mindustry/issues/1704#issuecomment-599093145
:frowning_face:

https://github.com/Anuken/Mindustry/issues/1704#issuecomment-599093145
:frowning_face:

I can now build both libarc64.so and libsdl-arc64.so on x86_64-linux and inject them into the downloaded maven deps before starting the build. This should also make some of the LD_LIBRARY_PATH hacks unnecessary, since libarc64.so now has the correct rpath with alsaLib and so on.
However, porting this to aarch64 will be a bit more difficult: Upstream has some hard-coded compiler flags that can not work with aarch64.

I can now build both libarc64.so and libsdl-arc64.so on x86_64-linux and inject them into the downloaded maven deps before starting the build. This should also make some of the LD_LIBRARY_PATH hacks unnecessary, since libarc64.so now has the correct rpath with alsaLib and so on.
However, porting this to aarch64 will be a bit more difficult: Upstream has some hard-coded compiler flags that can not work with aarch64.

That's unfortunate.
While you're at it then could you set meta.platforms to x86_64 and remove aarch64 from meta.broken, since it's unsupported upstream?

That's unfortunate.
While you're at it then could you set meta.platforms to x86_64 and remove aarch64 from meta.broken, since it's unsupported upstream?

While you're at it then could you set meta.platforms to x86_64 and remove aarch64 from meta.broken, since it's unsupported upstream?

Done.

Okay, so this has turned into a complete refactor:

I didn't like how we had to build the whole project twice each time (once for the -deps), so I investigated a bit and found a solution to only prefetch all dependencies in the -deps derivation, without building anything.
By accident I discovered that the Mindustry gradle project will discover an Arc checkout if it's placed in ../Arc (probably mostly intended for development) and use that instead of attempting to download prebuilt versions. So I used this to combine the Arc and Mindustry builds into one derivation, going back to one -deps derivation as well.

This might have broken some more things on darwin, but since it didn't work before and it doesn't look like that problem will be solved soon, I would like to ignore that for now.

While you're at it then could you set meta.platforms to x86_64 and remove aarch64 from meta.broken, since it's unsupported upstream?

Done.

Okay, so this has turned into a complete refactor:

I didn't like how we had to build the whole project twice each time (once for the -deps), so I investigated a bit and found a solution to only prefetch all dependencies in the -deps derivation, without building anything.
By accident I discovered that the Mindustry gradle project will discover an Arc checkout if it's placed in ../Arc (probably mostly intended for development) and use that instead of attempting to download prebuilt versions. So I used this to combine the Arc and Mindustry builds into one derivation, going back to one -deps derivation as well.

This might have broken some more things on darwin, but since it didn't work before and it doesn't look like that problem will be solved soon, I would like to ignore that for now.

Conflict

Conflict

Conflict

is resolved

Conflict

is resolved

Sorry I forgot to push earlier.

Sorry I forgot to push earlier.

thank you! one binary less in nixpkgs :tada:

thank you! one binary less in nixpkgs :tada:

There will probably be a followup PR with some minor refinements and aarch64 support in the next few days (also planning to add aarch64 support upstream)

There will probably be a followup PR with some minor refinements and aarch64 support in the next few days (also planning to add aarch64 support upstream)


Browserpass on Firefox is not showing the pinentry

General information

Operating system + version: Arch Linux (up to date)
Browser + version: Firefox 66.0.1 (64-bit)
Information about the host app:

How did you install it?

Installed via a package manager.

If installed an official release, put a version: 2.0.21

Information about the browser extension:

How did you install it?

Installed via webstore.

Browserpass extension version as reported by your browser:  Browserpass <IP_ADDRESS>

Exact steps to reproduce the problem

Try to have browserpass autofill a login form without having already filled a pinentry recently.
No pinentry shows up.
The form ends up blank.

What should happen?
A pinentry should show up. In my case pinentry-gnome3.
What happened instead?
Nothing showed up and the form was not filled.

Hi, please try out Browserpass v3 and let me know if the issue is still present: https://github.com/browserpass/browserpass/issues/331
This has been previously reported (https://github.com/browserpass/browserpass/issues/281), but I don't know what I can do on Browserpass side to help, the extension is not responsible for opening pinentry dialog, it's the gpg itself.
The only suggestion I can think of is make sure a proper pinentry application is configured in gpg-agent.conf, see this section of the documentation for more info: https://github.com/browserpass/browserpass-native/#hints-for-configuring-gpg
Closing as I don't really know what I can do on Browserpass side, but do let me know if you solve the issue. Good luck!

I was able to solve a similar sounding issue on my laptop by pointing gpg towards the pinentry program
# in ~/.gnupg/gpg-agent.conf, replace '/usr/local/bin/pinentry-mac' with the path to 'pinentry-gnome3'
    pinentry-program /usr/local/bin/pinentry-mac


Pock Button from control fails to launch pock strip

Describe the bug
When we close the pock strip from the x button on left and try to reopen it from the pock button it does not open.
Expected behavior
Expected to toggle pock strip.

Seen this as well.

I also found this problem, which puzzled me

It seems somehow restarting the app resolves the issue


OpenMP errors when running nosetests

I've found that when I run nosetests on test_solvers.py, the test fails once it gets to test_scs_options (test 16 of 19). The failure is preceded by this warning message from OpenMP:

OMP: Error # 15: Initializing libiomp5.so, but found libiomp5.so already initialized.
OMP: Hint: This means that multiple copies of the OpenMP runtime have been linked into the program. That is dangerous, since it can degrade performance or cause incorrect results. The best thing to do is to ensure that only a single OpenMP runtime is linked into the process, e.g. by avoiding static linking of the OpenMP runtime in any library. As an unsafe, unsupported, undocumented workaround you can set the environment variable KMP_DUPLICATE_LIB_OK=TRUE to allow the program to continue to execute, but that may cause crashes or silently produce incorrect results. For more information, please see http://www.intel.com/software/products/support/.

The console then displays

Process finished with exit code 134 (interrupted by signal 6: SIGABRT)

and the test fails.
HOWEVER, if I run test_scs_options separately from all other tests, then it passes. This seems to suggest that the OpenMP warning (and eventual test failure) is due to nosetests running in parallel. Has anyone seen this issue before, or have advice on how to investigate further?

To do: add OpenMP support to TravisCI builds.


Restoring from trash is not possible after changing the path of the parent folder

ownCloud Web UI 6.0.0-rc.2
Infinite Scale 2.0.0-beta.8+f442970f1 Community
Steps:

create folder f1
create file file.txt into folder f1
deletes file file.txt
rename folder f1 or move folder in another place or delete
try to restore file.txt

Actual: PROPFIND request of the trash bin https://localhost:9200/remote.php/dav/spaces/trash-bin/uuid contains original parent path in response:
oc:trashbin-original-locationfol/file.txt</oc:trashbin-original-location> and it does not change after changing the parent path so:

if we click to parent folder of the file in trash we get 404 not found

if we try restore file we get 409 conflict:

Expected: probably as implemented in os10. if we have no actual path -the file is restored to the home directory

Issue can be closed - not actual more

user restores file to non existing folder
Actual:
web checks if the f1 folder exists or not
if it doesn't exist -> the f1 folder is created automatically.
and the file is restored  to f1

https://github.com/owncloud/ocis/assets/84779829/70b31a2d-0c12-4cdf-869b-4b8cb881d69e


[Feature Request] Read from database and automatically cast to camel case

Most of the developers use sneak case in database systems but use camel case in java or kotlin source code.
I hope if there is a convenient way to do auto cast when read data from database.
e.g. if a table has these properties
create table demo(
  id int,
  first_name text,
  last_name text
)

after using val df = DataFrame.readSqlTable(config, "demo", 10, caseConvertor=Convertors.sneakToCamelConvertor) we can access property with camel case df.firstName[0]

Good suggestion, thanks

While we cannot do it on read atm, we do have the DataFrame<T>.renameToCamelCase() function. This will also convert snake case to camelCase automatically

Thanks, this worked for me.
Please close this issue if this is one is not planed since it's not important. Or remain this issue if it's worth to implement. Thanks again.


Remove the default value "" from delim of SimpleTokenizer

Removing the default value "<NB>", which used to be used for Stanford CoreNLP.
Because Fonduer now mainly uses spaCy instead of Stanford CoreNLP, "<NB>" should not be the default.
You can still specify delim="<NB>" if they need it.

@HiromuHota let me when you think this can be merged into master?

This cannot be merged now as the test fails.
The test fails because we are not passing anything to delim, meaning delim becomes None.
According to https://docs.python.org/3/library/stdtypes.html#str.split,

If sep is not specified or is None, a different splitting algorithm is applied: runs of consecutive whitespace are regarded as a single separator, and the result will contain no empty strings at the start or end if the string has leading or trailing whitespace.

This behaviour may not be what we want for the SimpleTokenizer.
I think this is not just a bug fix, but more like a design question if delim is not <NB>.
How should SimpleTokenizer split text?

@HiromuHota, good point!
The design goal for SimpleTokenizer is to parse content and output (yield) parsed all sentences, where the delim is to split the content in to sentences and then for each sentence it uses .strip() to do tokenization.
Sen

So the question is "what would be the default delimiter to split text into sentences?"
I'd say that is "." (period), and changed the patch accordingly.
I'd admit that this sentencizer is too simple. For example, "U.S.A." will be split into three sentences.
@SenWu any thought?

@HiromuHota This is a good point. SimpleTokenizer is designed to help parse simple data input without NLP attributes. On usage is plain text where each line is a sentence and the goal is to extract information for such sentences. So in the SimpleTokenizer, I think Fonduer will take a delimiter as input to split the data into sentences.

So... is the current patch good to go or needs some change?

Yes, good to merge.


rds-broker-passwd tool: Allow passing inputs as cli flags

What
I wanted to pass in the "seed" and "id" arguments as cli flags rather than interactive input so I can use in a little script.
This makes it possible to do rds-broker-passwd --seed x --id y and also keeps the original interactive method for any flags you don't set.
How to review
See if you like it
Who
Anyone who likes cli flags

I made a note in ab515b99077ec13b9fcc4f48e6272515e1b34409:

I've made it interactive, rather than taking CLI arguments, to prevent someone from storing the master seed password in their shell history.

I'm a little bit nervous about it because leaking the seed for production would give away a lot of power.
What do you think about having ID as a flag and seed as STDIN? Would that work for your scripting?

I could argue that by having it interactive the seed gets left around in my terminal's scroll back history, whereas with the cli args it's usage is hidden behind scripts so doesn't get left anywhere ... but I'll concede to your wisdom .... I hadn't considered it might be used in a non-dev way... but of course if it can be abused it will be!, so fair play leave it as is and close me :)

I'm totally up for making ID a CLI flag if that helps your use case?

Closing for now. Feel free to re-open in future with just --id flag if it's useful.


Dagre control points for drawing more accurate edges

It would be nice if thie cytoscape-dagre layout could make the edges "points" attribute available for the cytoscape graphs, giving the possibility to draw more accurate edges with segments or unbundled-bezier styles.

I was having same thought ...
From example http://cpettitt.github.io/project/dagre-d3/latest/demo/tcp-state-diagram.html ,
Is it possible to have edges like above example in cytoscape ?
Edges that go straight and bend as per location ..

see cytoscape.js vs dagre-d3.js for same set of data.

cytoscape : https://jsfiddle.net/bababalcksheep/m40m5mmx/6/
dagre-d3: http://cpettitt.github.io/project/dagre-d3/latest/demo/tcp-state-diagram.html

How can I behave cytoscape edges same like dagre ?
Any help will be much appreciated

Not easy to do until 2.7 is released with style properties for configuring where the segment-weights start and end: https://github.com/cytoscape/cytoscape.js/issues/1236

@maxkfranz 2.7 is released. Waiting for this feature. Thankx

@djangosdk There are a lot of other tasks for Cytoscape.js that have to take priority.  If this feature is important to you, we'll gladly accept a pull request.  It should be straightforward to implement in 2.7.  The segments property values should go in user specified node.data() fields so the user can map to them in their stylesheet.

@maxkfranz
Can you still help with some basic example or point me in right direction. I am in dire need for Dagre control points for Cy.
A description of what to do , where to do will also help ( I did not understand fully how to utilize  segments property values )

I spent some time trying to convert Dagre's control points to Cytoscape's segment edges (distances and weights). Unfortunately, the rendered result is not that beautiful, but maybe it's a start to get this issue done?
[https://github.com/newlukai/cytoscape.js-dagre/commit/3a0a7c41e37cc4e2fd00ba7068746c1627a37935](A first version of implementing this issue).

@newlukai I have improved your implementation, make it more beautiful. please check it out.
3bef946c348d419366c8b75b0bd3a2ee3ac28e34

and fix when rankDir is "LR":
af46725944e8d294179fe6f5d344679f7d025694

@knownact
Is it like this?http://jsfiddle.net/5u9mzfse/47/

Now, the result is like this:

By the way , I think default result is good.
https://jsfiddle.net/knownact/usL3ayxn/3/

@knownact
If it is drawing a flow chart, Klay shows better.
dagre:

klay:

@yansenlei yep, but can you give me the test data? i think dagre is not bad like this:

sure, like this: https://jsfiddle.net/leiys/e2rc66b8/3/

this is what a get:

yes, this layout is not like a flowchart.

yep, Klay is better!

Since I was in the need to find a solution, I switched to mxGraph. It's not as light-weight as cytoscape, but the provided layouts do fine and I'm now able to make very specific changes to the graph.
@knownact : Your changes look good.
Regarding the discussion started shortly: Yes, Klay is better.

can we get this feature ?

Dagre can only inform where the edges should be routed for the node positions that Dagre specifies.  As soon as the nodes are moved, Dagre's assumptions are no longer true and the use of Dagre's edge points would give bad results.
For trees -- and probably some types of DAGs as well -- specifying the points from Dagre itself isn't really needed in cytoscape>=3.5.   The curve-style:taxi edge style allows for automatic routing of an edge using taxicab geometry.  This gives a very similar result as Dagre's edge points, except the native Cytoscape taxi feature works interactively whereas Dagre does not.
See https://github.com/cytoscape/cytoscape.js/issues/2306.  The example graph in that issue uses a Dagre layout with taxi edges.

The example graph:

was anyone able to make the this work? I am facing the similar issue, why doesn't cytoscape.js-dagre follow the dagre layout for edges?

Is there any future plan to make edges follow dagre layout?

hello stale bot, did microsoft send you to break opensource? It's not hard to see how changing the meaning of "closed" is perfectly counterproductive, is it?

I do not understand what the solution to overlapping paths is. Has this been addressed in any of the cytoscape versions ?

this is what I get:

How did u manage to draw an edge saperately in my case difference edges shares one i mean it merges down at some point

Hey, @knownact -- I just wanted to thank you for the hard work you did on https://github.com/cytoscape/cytoscape.js-dagre/issues/5#issuecomment-373244016
It's lovely for my purposes.


Add ECS deployment config template

What this PR does / why we need it:
Suspend this error 👇

Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/cc @cakecatz

/lgtm

/approve


error when run go generate ./...

I got an error:
ModuleNotFoundError: No module named 'ctypes'
when run comand:
go generate ./...
system: widows 11
go version: 1.20.1
image error:

how to fix it.
thank you so much.

Windows as build system is untested and I unfortunately can't support this (I have no Windows)

I'm closing this. If someone wants to add support for Windows as build system, I'm happy to re-open it.


Remove a digit?

The missed calls and voicemail emails from Office 365 have an extra 1 for international calls.  We've submitted a ticket with Support to try and get the email changed, but in the meantime, is there a way to detect numbers based on the +1011 pattern and replace with +011?

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

ID: 940f3a9e-f4cb-4c60-c57a-3d064e79677a
Version Independent ID: 52842215-3cd7-c8c8-6556-d166a92a7b48
Content: New-CsVoiceNormalizationRule (skype)
Content Source: skype/skype-ps/skype/New-CsVoiceNormalizationRule.md
Service: skypeforbusiness-powershell
GitHub Login: @kenwith
Microsoft Alias: kenwith

@dbetlow Thank you for this feedback
I am not sure on the answer for this. The best place to get this answered would be Support. I apologize for sending you back to them since it sounds like you were already there but I want to make sure you get a solution and support will be able to handle any solution.
Here is a quick video on opening a ticket: https://www.youtube.com/watch?v=puuOy0sEg1Y
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


fix: direct indirect links text

Overview

Checklist

[ ] I have run this code and it appears to resolve the stated issue
[ ] README.md updated and added information about my change

@AdiGajbhiye : Can you take care of the ids, this looks good to merge otherwise.


V1 contributor console - Gerrit - Errors when contributor sends request to cla manager to be approved.

A user created a ticket for this issue. Their company signed a CCLA but had not added the user to the approved list. The user used the contact form to send the cla manger a notification that they'd like to be approved and ran into issues. They report:
When I click "contact" button nothing happens and when I click somewhere else, the contact form comes and goes very quickly.
This happens on both Chrome and Firefox.
I found this error in Firefox console:
ERROR Error: Uncaught (in promise): TypeError: user.user_emails is null

@WillsonHG I have gone through the code and looks like either the issue is with cache or in the user record (as per the error mentioned in the ticket user object is null).
I have tested on my side and it is working fine with me.
Can you please share the project and userId or simply a contributor console URL?

@amolsontakke3576  the linked ticket with the problem is in my comment. It's a user from Gerrit - check his comment in #1670


We need a root README.MD

Please add Travis CI indicator as well

Already done. (Link with master build status)


[server] Pass version role change information to AggKafkaConsumerService.

Summary, imperative, start upper case, don't end with a period

This PR introduced a new type of consumer action: VersionRoleChange, which is triggered by a new store listener: StoreVersionRoleChangedListener.  Once the store's current version changed and store version's role changed, it will notify the StoreIngestionTask to trigger consumer action for all its partitions on that server host to execute on this action.
Regarding this new consumer action, there will be graceful un-subscription/ re-subscription for all partitions to let bottom KafkaConsumerService to find corresponding consumer thread pool for them.
To let KafkaConsumerService aware of store version role during subscription, there will be more information with TopicPartitionReplicaRole  passed with aggKafkaConsumerService#subscribe. Here TopicPartitionReplicaRole is a bundle of information about particular topic partition:

Leader or Follower role
Version role: future/current/backup

In the future, we could also bundle more information to this object to pass into KafkaConsumerService to let it do better consumer pool allocation. With proper allocation of consumer thread pool, we could achieve prioritization on certain type of traffic (e.g. we could allocate larger consumer thread for current hybrid leader ingestion)
A unit test was added to verify the subscription/subscription happened for local and remote ingestion.
How was this PR tested?

Does this PR introduce any user-facing changes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

For batch-only stores, there is no need for resubscription. When future ingestion is done and it becomes current version, it doesn't need any consumer from the current version pool.
I assume SIT would unsubscribe right after reporting completion for batch-only store, so this action for batch-only stores will be an NO-OP as there won't be active subscriptions.

For batch-only stores, there is no need for resubscription. When future ingestion is done and it becomes current version, it doesn't need any consumer from the current version pool.

I assume SIT would unsubscribe right after reporting completion for batch-only store, so this action for batch-only stores will be an NO-OP as there won't be active subscriptions.

@gaojieliu I believe the store change listener would still send out the new consumer action (haven't checked the latest diff yet). Could you please also take a look at my comment of the listener?  https://github.com/linkedin/venice/pull/1034#discussion_r1690588199


sql: allow references to top-level WITH from apply-join

PR: https://github.com/cockroachdb/cockroach/pull/65550
From release notes:

References to WITH expressions from correlated subqueries are now always supported. [#65550][#65550] {% comment %}doc{% endcomment %}

Our WITH, correlated subqueries , and known limitations docs do not explicitly disallow WITH expressions in correlated subqueries.
Closing as there doesn't seem to be doc impact.


Slow Bitmap Speed

When reading a 320x240 bitmap image on an ESP32 (just reading, not writing it over Serial to my computer), it takes a long time, around 2 seconds. I’m guessing this is the limitations of sending the image over SPI from the ArduCAM to the ESP32. I was wondering there was a method to increase the speed of transmission. I might not need a 3-channel image, and a grayscale would work. The main goal is to do real time image processing (something quite simple) on board the ESP32. For reference, I am using a mini module camera shield with OV2640 camera module.

Hi @BillyBee123. Could you post the one for 80x60 too? Thanks so much!

Sure.  These are the steps to change the BMP resolution from QVGA (320x240) to QQQVGA (80x60):

In the ov2640_regs.h library registry file, make a copy of the struct sensor_reg OV2640_QVGA[] definition, and rename it OV2640_QQQVGA. In the copy, change the hex pairs from {0x5a, 0x50} to {0x5a, 0x14}, and {0x5b, 0x3c} to {0x5b, 0x0f}.

In the ArduCAM.cpp library file, replace wrSensorRegs8_8(OV2640_QVGA) with wrSensorRegs8_8(OV2640_QQQVGA).

In the sketch, replace the bitmap header definition with:
const char bmp_header[BMPIMAGEOFFSET] PROGMEM =
{
0x42, 0x4D, 0xB6, 0x25, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x42, 0x00, 0x00, 0x00, 0x28, 0x00,
0x00, 0x00, 0x50, 0x00, 0x00, 0x00, 0x3C, 0x00, 0x00, 0x00, 0x01, 0x00, 0x10, 0x00, 0x03, 0x00,
0x00, 0x00, 0x80, 0x25, 0x00, 0x00, 0xC4, 0x0E, 0x00, 0x00, 0xC4, 0x0E, 0x00, 0x00, 0x00, 0x00,
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xF8, 0x00, 0x00, 0xE0, 0x07, 0x00, 0x00, 0x1F, 0x00,
0x00, 0x00
};
This header is specific to a 80x60 image height, width, image size, and file size.

Again in the sketch, find the nested loop where the image is transferred to serial, and reduce the i and j loop size from 240 and 320, to 60 and 80, respectively.

Lee -- I'm actively developing machine vision for ArduCAM to recognize solid colored quadrilaterals, similar to PixCam.  To date, I've done the following:

Read and change pixel red, blue, and green values in the image buffer using masking and bit-shifting bitwise operations.

Average the red, green, and blue values (0 to 31) to manually produce a grayscale image.

Store the grayscale image shades in a matrix, then populate a histogram from 0 to 31 with a count of each shade.

Separate foreground shades from background shades by a threshold analysis on the histogram to minimize the sum of the light and dark shade variance (Otsu's Method).

Implement a row-by-row connected components algorithm to numerically label foreground pixels, then resolve a list of equivalent labels from adjoining labels using a quick union algorithm.

At this point, shapes of a common color can be separated from background pixels.  Still to do is assigning each shape to an object class, then defining the axis-aligned or oriented bounding box, and calculating the RGB color of the shape.  So far, I've been able to do all this from YouTube tutorials.

@BillyBee123
Hi,
Can you help me with changing the resolution to 32x32 bmp image?


having an issue with resolving and rejecting $q promises

This all works fine if I use native promises, but with Angular's $q the chai-as-promised methods just timeout:
Here is my service method:
  svc.test = function(foo){
    var def = $q.defer();

    if (!foo){
      return $q.reject(new Error('foo is required'));
    } else {
      // get data via ajax here...
      // def.resolve({});
    }

    return def.promise;
  };

Here is my unit test. The promise is resolved, but the resolveValue.should.eventually.... does not work (the test just times out):
  it.only('should error ', function(){
    var resolvedValue = MyServices.test();
    $rootScope.$apply();
    return resolvedValue.should.eventually.be.rejectedWith(TypeError, 'foo is required');
  });

@anthonyettinger I think that transferPromiseness does the trick:
chaiAsPromised.transferPromiseness = function (assertion, promise) {
  assertion.then = promise.then.bind(promise);

  inject(function ($rootScope) {
    // $q promise duck typing, Angular 1.3+
    if ('$$state' in promise && !$rootScope.$$phase) {
      $rootScope.$digest();
    }
  });
};

@domenic Can we add this recipe to the manual? Using Chai as promised with Angular is existing problem on the internets, no straightforward solution has been provided so far.

I'd rather link to a StackOverflow answer or similar so it's not something I'm responsible for maintaining, since I don't have any experience with the solution itself. E.g. "If you are trying to use Chai as Promised with Angular promises, [some people have ideas on how to do that on StackOverflow]" with the bracketed part being a link.

@domenic Sure, I suppose that this one should cover the issue.
I've been somewhat discouraged by the fact that there's no alternative to Angular-ish jasmine-promise-matchers in Mocha world, but fortunately, chai-as-promised is flexible enough.

Sorry bout the necro, but $rootScope.$digest is not 'the way' to go about flushing asynchronous tasks in Angular.
$timeout.flush is what you want, and no additional setup is necessary:
function foo () {
  return $q((resolve, reject) => {
    setTimeout(function () {
      resolve('foo');
    }, 250);
  })
}

it('resolves with foo', () => {
  expect(foo()).to.eventually.become('foo');
  $timeout.flush();
})

Also, the code in the OP is not generating the expected behaviour because the expectation comes before the execution. If anything, I would add a note about that in the README.
// works!
.to.eventually.do.something();
flush();

// does not work!
flush();
.to.eventually.do.something();

@kasperlewau Considering that there is 250ms $timeout, it is a good thing to test blackboxed $timeout with
$timeout.flush(250);
$timeout.verifyNoPendingTasks();

and treat $timeout.flush as a part of the spec.
And surprise, $timeout.flush won't work on setTimeout. setTimeout specs are still async and should be treated accordingly - with Jasmine done, or Sinon promise return, or Sinon fake timers, or Jasmine clock, or whatever. It is out of the scope of the issue, also not a concern for chai-as-promised.
The issue concerns synchronous tasks that cannot be completed because then callback is not fired on fulfilled promises until the digest is triggered manually.

@bisubus Alright, let's scrap the timeout.
function foo (bool) {
  return $q((resolve, reject) => {
    if (bool) {
      resolve('foo!');
    } else {
      reject('boo!');
    }
  });
}

This will still work with the correct order of execution. You don't have to involve the done callback when using chai-as-promised. That's the beauty of it as far as I'm concerned.
As soon as you start using chai-as-promised w/ Angular and you dont want to involve the done callback, you need to switch positions of your expectation & execution. That is it.

The issue concerns synchronous tasks that cannot be completed because then callback is not fired on fulfilled promises until the digest is triggered manually.

You will (and should) always need to trigger a $digest cycle in your tests. If the code under testing depends on that sort of stuff. $q promises, does.
I'm going to disregard from the $timeout best practices for now, feels like that's besides the point.
We can continue this on StackOverflow, if need be.

@kasperlewau If there are no non-Angular asynchronous operations like setTimeout, the spec is synchronous then.
Calling $rootScope.$digest() in transferPromiseness hook saves the one from calling it after each expect...eventually. To make the hook workable when accidentally calling expect...eventually on digests (namely, inside another then callback), it should be foolproofed with
$rootScope.$$phase || $rootScope.$digest();

I guess it's as simple as that.

Hi sorry about bump again, I tried using the solutions provided by the link on the readme and here, but it's not looking good, made a plunker with examples that would be nice to have documented imho


Missing docstring for Separate()

Hi,
thanks for the awesome struct, it's very very useful!
Using it i noticed that there is a missing docstring here leaving an eerie
// Separate removes the set items containing in t from set s. Please aware that
that bodes doom.

LOL. That does rather feel like
i think i wrote the original incomplete sentence. if i can remember what that warning was supposed to be, i'll put up a PR...


[Bug]: Unable to see the autocomplete options for Data fields in JSON Form

Is there an existing issue for this?

[X] I have searched the existing issues

Description
For the source data, when you get to bind the default value in field configuration, we are unable to see the actual fields in autocomplete dropdown. Instead , we are getting options from the dummy JSON form which we get to see on the first drop
Steps To Reproduce

Drop JSon form
add source data from some query
Go to field config of any field and see the auto complete options for the fields that can access the current form using {{Sourcedata.fieldName}}
https://www.loom.com/share/5a195d26088648089ca138c3b5d61b16

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Prod

@bharath31 This is not related to evaluations.
@kamakshibhat-appsmith Could you please confirm if this is a regression ?

@arunvjn I could see this in older apps as well(~3 months old). So can't confirm when this started happening

@ashit-rath are you aware if this was the case from the very beginning?


Update to Go  1.14.3 and Go 1.13.11

https://groups.google.com/forum/#!msg/golang-announce/-9KWN-OUSl0/Jro9NDu8CgAJ
Looking at the issue lis, I suspect I've been annoyed by https://github.com/golang/go/issues/37833

Fixed in 3cc41523bef802d1942f3d31018547a18cc55923


.data() / .exists() are not recognized functions

Hello. Beginner here. I've tried your sample code and it doesn't even work. The functions data() and exist() in the sample aren't recognized in Web version 9. I was working with Web version 8 last year and the sample code works.

Disregard this issue. VSCode just solved my problem for me. Neat!


Question - Character limits to Management Group Ids

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Is there reasoning behind limiting the character length of the root management group at all?

Have a customer that would like the ability to import an existing MG as their level_1 or root management group and this is blocking them.

Thanks Kevin. In short the MG exists and they want to continue to to use this to avoid a request for changes at the tenant root group (its quite onerous but ultimately political in some larger orgs to get this done), it already has an id which is 11 characters in length. I thought cool, we'll just import it to state removing the need to maintain a custom landing zone hierarchy but if thats what is required then so be it.


Databricks workspace service fails to deploy to East US with DNS error

Resolves #3515
What is being addressed

Add missing entries to databricks-udr.json
Add notes to the docs

/test-force-approve


Tab complete should "just work" for builtin commands

Tab complete should know about shell-lambda options and flags and offer completions accordingly.

I am interested in this project and would like to help, but have effectively no experience with Common Lisp beyond working through two books (Gentle Introduction to Symbolic Computation and Practical Common Lisp). This issue seems (from a naive view) to potentially be a good place to start (fairly well-defined and of limited scope). Would you agree with that, or would you recommend I start on a different issue?

@qwertystop I agree, from a programming point of view this should be straightforward.  It might be hard to figure out where the appropriate hooks are in the codebase, so chime in here if you need help!


Add printf wrapper function to ExampleTypes

Makes debugging easier, as only one line needs to be commented to disable printf output, which makes the example run much faster.

Using a #define instead also works to make a one line change to disable to all the printfs :-)
I will close this PR and I will just make the define change locally when I need to debug the types example :-)


CPO / DPO: Fix red CI

I accidentally pushed the fix of the red CI on main.
https://github.com/huggingface/trl/pull/1734 overrides TrainingArguments __post_init__ method without returning super().__post_init__().
https://github.com/huggingface/trl/commit/4e23d958f20fd4fdd795cb06c2cdb7ebea704855 combined with this PR should make the CI happy
cc @kashif @vwxyzjn

LGTM! Thanks @younesbelkada!

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


archive logs step

we had 3 canary build failures due to archive logs steps, there are not enough information about why it happened, I am separating it so that we have one zip per folder, hope it will make it easier to spot the issue. I am not sure whether this will actually fix the problem

Pull Request Test Coverage Report for Build 432997679

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 43.12%

Totals

Change from base Build 431379576:
0.01%

Covered Lines:
24159

Relevant Lines:
51340

💛  - Coveralls

@aasimkhan30 , I saw the canary build failed again yesterday, I am going to merge this to see whether it helps.

@alanrenmsft, Thank you! I will keep a check.


fix(autocomplete): preventDefault on popover-icon mousedown instead of click

PR Checklist
Please check if your PR fulfills the following requirements:

Tests for the changes have been added/updated: N
Docs have been added/updated: n/a?
Does this PR introduce a breaking change? N
I have linked any related GitHub issues to be closed when this PR is merged? n/a

Describe the new behavior?
Fixes issue where autocomplete with text-field + popoverIcon (but not chip-field) would currently have issues with the dropdown when the icon was clicked directly:

Additional information

Meant put this up as Draft yesterday, marking it now, but looking for confirmation on the approach.


Shouldn't getReact search only among descendents of the resq node passed by the previous command?

Reading the docs I understand that getReact searches only among the descendants of the resq node passed by the previous command. So for example cy.getReact('A').getReact('B') should search for B only among A's descendants. I got this right?
If yes, I added this test to calculator.spec.js, which is failing
  it('getReact should search only among descendants of the resq node passed by the previous command', () => {
    cy.getReact('t')
      .nthNode(2)
      .getReact('t')
      .nthNode(9)
      .getProps('name')
      .should('eq', '5');
  });

It should get the element at index 2, then among its descendants get the element at index 9 (the button '5'). Instead it gets the button '9', as if the second getReact searched for elements starting from the root component.

Hi @matfer , you are correct. This is always the desired outcome in any test automation scenario. Currently, We are blocked due to this: https://github.com/abhinaba-ghosh/cypress-react-selector/issues/41
Happy to get help.

Hi, this is now fixed with version 2.3.13 ( #251 )
Example - https://github.com/abhinaba-ghosh/cypress-react-chain-queries
Feel free to re-open the ticket if you find any issues.


Added Contact form

Added a responsive contact form under contact folder

This is in the Figma File? @mandiraghosh783

@Rajdip019 this is a html file

Please read the README.md file  and CONTRIBUTING.md before contributing to any repo. And raise an issue before making an PR. @mandiraghosh783


Upgrading the project to use either .NET 4.0 or 4.5 framework

Would it be a breaking change if we updated this solution to use the .NET 4.0 or even 4.5 framework?
There are a few language enhancements that were introduced in .NET 4.0 that I would like to use in some refactoring of the MailChimpExport.cs class.
This is not a big deal either way, but it would be nice to take advantage of if the solution could be upgraded.

Please see #176 for an attempt to update this to .NET 4.5


Just a quick question about scanning speed

V0.2.1 work fine for me, but it's much much slower than when it was like 0.1.5. This is just because Niantic throttle the request rate to 10s, there's nothing I can do right?
PS. SO GLAD THIS WORK AGAIN. THANKS DEVS FOR YOUR HARD WORK!!!!!!!!!!

I was actually wondering the same thing and about to make a post myself.

Can someone help me to download it and shit? :dancer:

What is your exact problem?

scanning is veeeeery slow

Are you using a Java API? If you are then I did hear a rumor that they are making new accounts accept the ToS on Pokemon go before being able to see the API files or something like that. So if that's the case try logging onto that gmail on pogo and agreeing to everything and making an account THEN try the app again and see if it changed anything.  If it's none if these make sure that you have "show Pokemon" turned on in the filter area of the UI.

change nothing scan is really slow and show not many pokemon


Api for upload trigger on torrent.

I use this trigger but it's not referenced into the api docs !

I'm guessing you closed it because it was in this pending PR https://github.com/feross/webtorrent/pull/750
Thank you for the PR nonetheless.


Should dnx use framework from project.json

I have removed clr support as I have a dependency that needs full framework
My project.json extract
"frameworks": {
"dnx451": {
"dependencies": {
"hiqpdf": "8.10.0"
}
}
},
If I run migrations I get
C:\Users\mikes\Projects\Reports2\Main\Reports2\src\Reports2>dnx database update
-c Applicationdbcontext
System.InvalidOperationException: The current runtime target framework is not co
mpatible with 'Reports2'.
Current runtime target framework: 'DNXCore,Version=v5.0 (dnxcore50)'
Version:      1.0.0-rc1-16231
Type:         CoreClr
Architecture: x86
OS Name:      Windows
OS Version:   6.1
Runtime Id:   win7-x86
Please make sure the runtime matches a framework specified in project.json
Should dnx use framework from project.json automatically to avoid this error and save me setting the default manually with dnvm?

I have removed clr support as I have a dependency that needs full framework

Should read
I have removed core support as I have a dependency that needs full framework

Then you should use the right dnx. Look at your command line. IT looks like you're running the dnx based on .NET Core instead of the one based on .NET Framework.

Ah OK I thought dnx was purely a bootstrapper and common to all frameworks and it could somehow change frameworks based to project.json.
But I see now it is a different binary for core vs clr thanks for your help.


Instruction to uninstall

My Mac keep the nord colors when ls even though I have deleted the .dir_colors file. How can I uninstall it completely?

The script only copies the theme file named as .dir_colors to your home directory (~/.dir_colors), it won't create a .dir_colors directory. You can uninstall the theme by just delete the file.

I updated the comment, please re-open this issue.

Have you started a new shell session? Otherwise the theme is still loaded in the process memory.
Also check the value of the $LSCOLORS environment variable. It might only look like the theme is still used, but maybe the default dircolors definitions are making use of the terminal colors in range 1-15 which are the colors of the Nord terminal theme.

Then there's a $LSCOLORS environment variable I assume it your're running macOS, right? :smile:
Unfortunately there's a difference between Apple's BSD ls and GNU ls which is the default on Linux systems. There are many threads, guides, tutorials and issue reports regarding the incompatibility between both and many macOS users switched to GNU ls via homebrew and a shell ls alias to override the default system ls. This difference also applies to dircolors themes (dircolors comes from the Linux coreutils package).
Again, you have to differentiate between this theme, read by the dircolors binary which result is used by ls, and the terminal theme (e.g. Nord iTerm2 or Nord Terminal.app that provides the actual values. The dir_colors file doesn't contain any color values at all (e.g. HEX or RGB(A) color values), but only uses the predefined color palette numbers which are resolved by the terminal theme to the actual color value.
There are many resources to help you to fix the problem by just searching for something like "macOS ls dircolors". All I can say for now is that the theme install script only copies the file to ~/.dir_colors, it's just a simple shellscript. There are no other changes neither the terminal or any other shell specific configuration files.


Fix typos

Motivation and Resolution
I just like fixing typos!
...
RPC version (if applicable)
...
Usage related changes
Better readability

Change 1.
...

Development related changes
Better readability

Change 1.
...

Checklist:

[x] Performed a self-review of the code
[x] Rebased to the last commit of the target branch (or merged it into my branch)
[x] Linked the issues which this PR resolves
[x] Documented the changes in code (API docs will be generated automatically)
[x] Updated the tests
[x] All tests are passing

The generated API signatures shouldn't be changed.


feat: WT-2076 - expose root raw error

Summary

It is currently extremely hard to debug the root cause of a CheckoutError because the raw error isn’t always exposed.
https://imtbl.slack.com/archives/C02UK3GC2N4/p1705887314679759
Add the root error to the CheckoutError payload. Example:
    throw new CheckoutError(
      'An error occurred while signing the message',
      CheckoutErrorType.SIGN_MESSAGE_ERROR,
      { error: err },
    );

Customer Impact

Allow partners to access to root error and stack trace.

Changed

CheckoutError data payload
Things worth calling out

Before submitting the PR, please consider the following:

[ ] Prefix your PR title with feat: , fix: , chore: , docs:, or refactor:.

Nice work @andrearampin! 👍 Just a minor nit I'm not a fan of abbreviations like err but I can see it's how the code was originally written.

yep me too, I just tried to avoid as many changes as possible in this PR


License ?

Hello, under what license is this project released under ? I would like to study it so I can learn how to use the Rhino API. Thank-you.

Hi there,
I added a license file. Please note that this project is several years old and I do not know if it is still working with a current version of Rhino.


Make test failing

Please help me resolving this error while installing openssl 1.1 on Ubuntu 16.04
./test/recipes/80-test_ssl_new.t                (Wstat: 256 Tests: 29 Failed: 1)
  Failed test:  12
  Non-zero exit status: 1
Files=155, Tests=1466, 89 wallclock secs ( 1.89 usr  0.23 sys + 74.98 cusr  4.29 csys = 81.39 CPU)
Result: FAIL
Makefile:207: recipe for target '_tests' failed
make[1]: *** [_tests] Error 1
make[1]: Leaving directory '/home/vivek/openssl-1.1.1g'
Makefile:205: recipe for target 'tests' failed
make: *** [tests] Error 2

Please provide some more details about how you configured OpenSSL during your build.
Also please provide the output from this command:
make TESTS=test_ssl_new V=1 test

Thank you for the quick reply.
To configure Openssl, I used the following command:
./config --prefix=/home/username/openssl --openssldir=/home/username/openssl no-ssl2
and on running make TESTS=test_ssl_new V=1 test
I get :
Test Summary Report
..```
/test/recipes/80-test_ssl_new.t (Wstat: 256 Tests: 29 Failed: 1)
Failed test:  12
Non-zero exit status: 1
Files=1, Tests=29, 13 wallclock secs ( 0.34 usr  0.06 sys + 10.82 cusr  0.41 csys = 11.63 CPU)
Result: FAIL
Makefile:207: recipe for target '_tests' failed
make[1]: *** [_tests] Error 1
make[1]: Leaving directory '/home/vivek/openssl-1.1.1g'
Makefile:205: recipe for target 'tests' failed
make: *** [tests] Error 2

I need the full output from the make TESTS command.

make_test.txt

Looks like a problem with expired certificates in the test suite. You are are using an old version of 1.1.1. Please try the latest version 1.1.1t.

Thanks Matt. True that, it was with the version. Now it seems it is working. Thank you so much for the help. :)

No problem.


Stop using implicit style differentiating

It seem the style used  here is being deprecated and  won't work with Flux 0.14: https://github.com/FluxML/MLJFlux.jl/blob/452c09d7dc13914f4057c661448bc310c9362d3d/src/core.jl#L37

Relatedly, it would be nice if the MLJFlux models listed here  https://github.com/FluxML/model-zoo#examples-elsewhere  could be updated to use latest Flux, and avoid implicit gradients.
Examples of similar upgrades: https://github.com/FluxML/model-zoo/issues?q=is%3Aclosed+label%3Aupdate+explicit
In the end, Flux 0.14 did not drop support for implicit gradients, but 0.15 should.

@pat-alt Would you have any time and interest in addressing this issue?

That actually syncs well with some of my other outstanding issues and I think I'll have to address this very same thing in CounterfactualExplanations.jl soon. So yes, please feel free to assign to this one to me and I'll look at it in the coming weeks 👍

I have added a draft for this with very minor changes here #230:
function train!(model::MLJFlux.MLJFluxModel, penalty, chain, optimiser, X, y)
    opt_state = Flux.setup(optimiser, chain)
    loss = model.loss
    n_batches = length(y)
    training_loss = zero(Float32)
    parameters = Flux.params(chain)
    for i in 1:n_batches
        batch_loss, gs = Flux.withgradient(chain) do m
            yhat = m(X[i])
            pen = penalty(parameters) / n_batches
            loss(yhat, y[i]) + pen
        end
        training_loss += batch_loss
        Flux.update!(opt_state, chain, gs[1])
    end
    return training_loss / n_batches
end

Currently, the following test fails:
[ Info: regularization has an effect:
[ Info: acceleration = CPU1{Nothing}(nothing)
regularization has an effect (typename(CPU1)): Test Failed at /Users/patrickaltmeyer/code/MLJFlux.jl/test/integration.jl:25
  Expression: !(loss2 ≈ loss3)
   Evaluated: !(0.8354643267207931 ≈ 0.8354643267207931)

I'm not quite sure what's happening. @mcabbott can you spot anything obviously wrong this?

Thanks both!

What is penalty? For L2 it will be better to use WeightDecay like this: http://fluxml.ai/Flux.jl/stable/training/training/#Regularisation

Currently, penalty functions are explicitly defined callable objects in MLJFlux (see [here]). I saw the note on WeightDecay in the Flux docs and was wondering if it's worth changing that.
In any case, I can't really get either of the approaches you suggest to work in this particular case, so we may indeed want to rethink the implementation of the penalty functions, for example by using WeightDecay instead. Will require a little extra work, but should be doable. @ablaom what do you think?

I can't really get either of the approaches you suggest to work in this particular case

Can you elaborate? I'm not sure I understand why/how they wouldn't work.

Sure!
Moving the params call inside as follows
function train!(model::MLJFlux.MLJFluxModel, penalty, chain, optimiser, X, y)
    opt_state = Flux.setup(optimiser, chain)
    loss = model.loss
    n_batches = length(y)
    training_loss = zero(Float32)
    for i in 1:n_batches
        batch_loss, gs = Flux.withgradient(chain) do m
            yhat = m(X[i])
            pen = penalty(Flux.params(m)) / n_batches
            loss(yhat, y[i]) + pen
        end
        training_loss += batch_loss
        Flux.update!(opt_state, chain, gs[1])
    end
    return training_loss / n_batches
end

the tests just seem to get stuck at some point. I may try and commit this now, but at least locally on my machine things get stuck.
Alternatively, using the approach in https://github.com/FluxML/Flux.jl/issues/2040#issuecomment-1535535892 as follows
function train!(model::MLJFlux.MLJFluxModel, penalty, chain, optimiser, X, y)
    opt_state = Flux.setup(optimiser, chain)
    loss = model.loss
    n_batches = length(y)
    training_loss = zero(Float32)
    for i in 1:n_batches
        batch_loss, gs = Flux.withgradient(chain) do m
            yhat = m(X[i])
            l = loss(yhat, y[i])
            reg = Functors.fmap(penalty, m; exclude=Flux.trainable)
            l + reg / n_batches
        end
        training_loss += batch_loss
        Flux.update!(opt_state, chain, gs[1])
    end
    return training_loss / n_batches
end

I get the following error:
[ Info: acceleration = CPU1{Nothing}(nothing)
┌ Warning: Layer with Float32 parameters got Float64 input.
│   The input will be converted, but any earlier layers may be very slow.
│   layer = Dense(5 => 15)      # 90 parameters
│   summary(x) = "5×20 Matrix{Float64}"
└ @ Flux ~/.julia/packages/Flux/n3cOc/src/layers/stateless.jl:60
fit! and dropout (typename(CPU1)): Error During Test at /Users/patrickaltmeyer/code/MLJFlux.jl/test/test_utils.jl:38
  Got exception outside of a @test
  TypeError: non-boolean (NamedTuple{(:layers,), Tuple{Tuple{Dense{typeof(identity), Matrix{Float32}, Vector{Float32}}, Dropout{Float64, Colon, Random.TaskLocalRNG}, Dense{typeof(identity), Matrix{Float32}, Vector{Float32}}, Dense{typeof(identity), Matrix{Float32}, Vector{Float32}}}}}) used in boolean context

Perhaps it has to do with the fact that the penalizers aren't Functors?

Yeah I wouldn't try the first version you have there, was referring to the second one or @mcabbott's suggestion about moving things to the optimization step.

I get the following error: ...

Pretty sure that's due to a typo in the original example code snippet. See https://github.com/FluxML/Flux.jl/issues/2040#issuecomment-1660776696

hmm in that case I get the following error: MethodError: no method matching Dense(::Float32, ::Float32, ::typeof(identity)). Any ideas?

Thanks @pat-alt for this work!

In any case, I can't really get either of the approaches you suggest to work in this particular case, so we may indeed want to rethink the implementation of the penalty functions, for example by using WeightDecay instead. Will require a little extra work, but should be doable. @ablaom what do you think?

WeightDecay only provides a mechanism for L2 regularisation, but the current implementation provides for a combination of both L1 regularisation (good for feature selection) and L2 regularisation. It seems a pity to drop support of a feature to accomodate the new explicit syntax.
I don't know what the source of your current issue is.

@pat-alt I don't think your use of Functors.fmap is valid here. The penalty function takes a tuple of matrices, as returned by Flux.params(chain),  and returns a single aggregate number.
Your first suggestion (with params) actually works but is 3600 times slower than the implicit style code on the dev branch, when tested on a small model / dataset.
@ToucheSir To implement mixed L1/L2 penalties (not just L2 ones) I don't really see how to avoid the params in the withgradient block. (And this is after all a suggestion in the Flux documentation (second code block here). Am I to conclude that explicit-Zygote style AD is just no good on this problem?

To implement mixed L1/L2 penalties (not just L2 ones) I don't really see how to avoid the params in the withgradient block. (And this is after all a suggestion in the Flux documentation - second code block here). Am I to conclude that explicit-Zygote style AD is just no good on this problem?

It's arguably better, but it requires some helper functionality that isn't currently nicely packaged up in a library. https://github.com/FluxML/Optimisers.jl/pull/57 is one example of how to do this and how we're thinking about packaging it up going forwards, but the problem with general solutions is that they take time. For this work, you may be better served by implementing a similar but more constrained version on top of Functors.jl and Optimisers.jl which only includes as much as MLJFlux needs for regularization. If you do, feel free to ping me for input.

@ToucheSir Thanks for the prompt response and offer of help.
So, with the apparatus you describe (Functors.jl, etc ) what code replaces the following to avoid the params call, working for a generic Flux model, chain, and so that differentiating chain -> penalty is free of issues?
# function to return penalty on an array:
f(A) = 0.01*sum(abs2, A) + 0.02*sum(abs, A)

f(ones(2,3))
# 0.6000000000000001

chain = Chain(Dense(3=>5), Dense(5=>1, relu))
penalty = sum(f.(Flux.params(chain)))

Or if you prefer, how should the regularisation example in the documentation be re-written?

f(A) = ...
penalty = mytotal(f, chain)

Where mytotal is a simplified form or direct copy of Optimisers.total as I mentioned earlier.

...(without the weight-decay trick , which does not work for L1 penalty)?

Side note, but I remembered looking into this a few months back and coming across https://stackoverflow.com/questions/42704283/l1-l2-regularization-in-pytorch/66630301#66630301, which suggests that L1 could be implemented using a similar trick. Whether that would be compatible with MLJFlux's API I'm not sure, but we could consider adding it to Optimisers.jl.

Thanks for the help @ToucheSir . Unfortunately, Optimisers.total is not working for me. I've tried some variations on that approach but without any luck.
I suggest we wait on the WeightDecay extension referenced above and switch that approach, which is likely more performant anyhow.


Warning when changing boolean values - Communication result 503 and 503

When changing the plenticore.0.devices.local.battery.SmartBatteryControl, the result is
WARN: PUT to settings devices:local / Battery:SmartBatteryControl:Enable (1) resulted in code 503: {"message":"internal communication error, try again later"}
When changing the plenticore.0.devices.local.battery.DynamicSoc, the result is:
WARN: PUT to settings devices:local / Battery:DynamicSoc:Enable (1) resulted in code 503: {"message":"internal communication error, try again later"}
Numeric values like MinSoc or MinHomeConsumption work without any problem.

Hm, I am quite sure it worked earlier. Maybe they changed the handling in the firmware at an earler update.


[SR-4240] Remove erroneous drop(while:) optimization

Calling drop(while:) after prefix() loses the
prefix, because in the internal drop(while:) override grabs the
underlying base iterator from _PrefixSequence and wraps it in a
_DropWhileSequence.  Instead, fall back to Sequence's implementation.

Resolves SR-4240.

@swift-ci please smoke test

If it isn't too late to nominate this for 3.1, I'd like to get it in.

This was totally wrong, and seems to be copy-pasted from its counterpart into it.

@practicalswift How does it look now?

@swift-ci please smoke test

@CodaFi You meant @airspeedswift ? :-)

@CodaFi great, thanks! except I think smoke test won't run that test since it's a validation-test...

@swift-ci Please test

⛵


KOGITO-4687 Fix Quarkus Ecosystem CI with Quarkus master

https://issues.redhat.com/browse/KOGITO-4687

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

The Linux check is successful.


unicode method for form.fields

As django templates and models primarily translate all strings to unicode.  This may be preferred and less confusing over the status quo u'field_01', u'field_02', etc.

The __str__ method is used by Django when the @python2_unicode_compatible decorator is used. No need to define __unicode__. https://docs.djangoproject.com/en/1.9/ref/utils/#django.utils.encoding.python_2_unicode_compatible


[C++] util/small_vector_test.cc reports a self-move warning with MinGW

Describe the bug, including details regarding any error messages, version, and platform.
https://github.com/apache/arrow/actions/runs/5006155150/jobs/8971042580#step:7:1154
D:/a/arrow/arrow/cpp/src/arrow/util/small_vector_test.cc:417:22: warning: moving 'moved_moved_ints' of type 'arrow::internal::TestSmallStaticVector<arrow::internal::VectorIntLikeParam<arrow::internal::StaticVectorTraits, arrow::MoveOnlyDataType> >::IntVectorType<5>' {aka 'arrow::internal::StaticVectorImpl<arrow::MoveOnlyDataType, 5, arrow::internal::StaticVectorStorage<arrow::MoveOnlyDataType, 5, true> >'} to itself [-Wself-move]
  417 |     moved_moved_ints = std::move(moved_moved_ints);
      |     ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
D:/a/arrow/arrow/cpp/src/arrow/util/small_vector_test.cc:417:22: note: remove 'std::move' call

Component(s)
C++

I got the same error on macos with gcc 13.1
/Users/zhjwpku/zhjwpku/arrow/cpp/src/arrow/util/small_vector_test.cc:418:22: error: moving 'moved_moved_ints' of type 'arrow::internal::TestSmallStaticVector<arrow::internal::VectorIntLikeParam<arrow::internal::SmallVectorTraits, arrow::MoveOnlyDataType> >::IntVectorType<5>' {aka 'arrow::internal::StaticVectorImpl<arrow::MoveOnlyDataType, 5, arrow::internal::SmallVectorStorage<arrow::MoveOnlyDataType, 5> >'} to itself [-Werror=self-move]
/Users/zhjwpku/zhjwpku/arrow/cpp/src/arrow/util/small_vector_test.cc:418:22: note: remove 'std::move' call
I checked the GCC 13 Release note, this is a newly added warning, I guess the warning with MinGW might be using
gcc 13 too.


[transducers] feature async generators

As a big fan of async generators I would love to see some support from the @thi.ng/transducers package.
Conceptually they are not too far from normal iterators anyway so I hope this feature is possible.

Iterators
Async iterators

Object method to provide iterator
Symbol.iterator
Symbol.asyncIterator

next() return value is
any value
Promise

to loop, use
for..of
for await..of

Example usage: NodeJS - simple directory recursion
import { opendir } from "node:fs/promises";
import { extname, join } from "node:path";

async function* walk(dir) {
  for await (const d of await opendir(dir)) {
    const entry = join(dir, d.name);
    if (d.isDirectory()) yield* walk(entry);
    else if (d.isFile()) yield entry;
  }
}

const collectMarkdownFiles = await transduce(filter(p => extname(p) === ".md"), push(), walk('.'))

Example usage: Alternating URL fetch
async function* getResources() {
  yield fetch('https://someurl.org')
  yield fetch('https://someFallback.org')
}

// Imperative usage
for await (const res of getResources()) {
  if (res.status === 404) continue;
  // response handling
  break;
}

// Declarative usage
const payload = await transduce(/* response handling */, push(), getResources())

Just a quick one: There's a 3.5 year old branch for async transducers. I stopped with it due to widespread lack of async/await support back then. I've been meaning to revisit it for a while, just no bandwidth so far. If you're interested in taking a peek:
https://github.com/thi-ng/umbrella/blob/feature/async-transducers/packages/async/src/index.ts

Closing this since there's a thi.ng/transducers-async package for a few months now...


Add support for disallowing plugins via CLI

Pull Request Checklist

[x] implement the feature
[x] write the documentation
[x] extend the test coverage

Where is this documented? I see just litte note in for provision --help. Could it be copied into https://tmt--2598.org.readthedocs.build/en/2598/spec/plans.html#provision. How about 'versionadded' ?

Where is this documented? I see just litte note in for provision --help.

For me, the help text was good enough. But I guess a story might be a fitting documentation tool?

Could it be copied into https://tmt--2598.org.readthedocs.build/en/2598/spec/plans.html#provision.

I'd prefer not to copy it anywhere. Plus, that particular page will become much shorter, not documenting individual plugins, and instead, I'd like to include help for the step from its sources, which would, eventually, propagate --allowed-how into this place as you suggested. Would that be acceptable?

How about 'versionadded' ?

I'd include it in the story.

Code looks good but I'm missing more documentation + examples in the docs (especially with several methods allowed)

Added a story in 7089c290

@lukaszachy seems can be re-reviewed

Tried it in Testing Farm: https://artifacts.dev.testing-farm.io/0dfa742b-424f-4844-878e-f97ff30d5ec7/
I think these hints are misleading as tmt+provision-virtual is actually installed:
summary: Basic multihost test
    warn: Suitable provision method 'virtual.testcloud' disallowed by configuration.
    hint: Install 'tmt+provision-virtual' to run tests in a virtual machine.
    hint: Use the 'local' method to execute tests directly on your localhost.
    hint: See 'tmt run provision --help' for all available provision options.

@happz Thank you for the story.
Personally I read the help as the 'true docs' however generated docs are easier to link when describing the feature for others (or when searching for features).

/packit test --identifier full

/packit test --identifier full


Searchbar - closes #270

Component Description
A Searchbar with an inline svg search icon

Can you move the search icon to the left


Unity 5.4 Support

I had to delete the InternetManager.cs for this to work. To many changes so I'm just local networking at the moment. Awesome. Thanks.
I've swapped out with prefabs works great. I'll see if I can get Networking under the belt and help.

I've updated for Unity 5.4 now.