Doesn't track startup errors

I don't think the SDK is currently tracking ASP.NET Core startup errors. These are exceptions that occur during the startup phase of the application before the server has started receiving requests, e.g. a line in the applications' Startup.ctor or Startup.Configure methods fail.
ASP.NET Core has configurable behavior to either catch startup errors and allow the server to start, then show a canned 500 error page, or not catch startup errors and let the exception propagate, usually to the applications Program.Main and thus causing it to exit with an error condition.
It would be useful to track these startup errors in AI for cases where the app is deployed, successfully configured to talk to AI, but fails elsewhere during startup. The timing may be problematic however, as typically the AI hookup doesn't happen in the application until the Configure phase (via an IStartupFilter) which would mean that any error occurring before this wouldn't get tracked without us rethinking this a bit. It might be possible to hook something up earlier exclusively for tracking startup errors and have this feature be controllable via a separate flag in the SDK.

Closing this as this is now mostly addressed with ILogger integration. AI can capture exceptions occurring from program.cs/startup.cs itself, (though with a minimal config).


Nice

[Enter feedback here]

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

ID: 0bffdd7e-ea18-32e8-6aa3-1f24e531414f
Version Independent ID: 053c490b-4078-a18c-cc22-38e8e3329dc1
Content: Reactivate a disabled Azure subscription - Microsoft Cost Management
Content Source: articles/cost-management-billing/manage/subscription-disabled.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

Thanks for the feedback. Closing this issue for now. If you have any feedback related to referred Azure document, please share it elaborated context so we can improve the document as appropriate and continue to discussion if required. Thanks again!


Content Security Policy HTTP header script-src directive requires unsafe-eval

When setting the Content Security Policy HTTP header, the script-src directive causes an error when 'unsafe-eval' is not given as a value.
How to reproduce the bug

In superset_config.py include Content-Security-Policy in DEFAULT_HTTP_HEADERS like

DEFAULT_HTTP_HEADERS = {
    "Content-Security-Policy": "default-src 'none'; script-src 'self'; style-src 'self' 'unsafe-inline'; connect-src 'self'; img-src 'self'; font-src 'self';"
}

Deploy superset

Expected results
The Superset webpage loads correctly
Actual results
I get the error: Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".
Screenshots

Environment
(please complete the following information):

browser type and version: Chrome, Version 98.0.4758.102
superset version: 1.4.1
python version: 3.8
node.js version:
any feature flags active: ROW_LEVEL_SECURITY, DASHBOARD_RBAC, DASHBOARD_NATIVE_FILTERS, VERSIONED_EXPORT, ENABLE_TEMPLATE_PROCESSING

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

Hat tip to @sfirke who notes this is now explicitly noted here https://superset.apache.org/docs/security/#csp-requirements
Closing accordingly.


Rescript jest

@glennsl I noticed one more typo this morning after you merged 97.
BTW, I don't see 0.8.0 showing up in the releases list on GitHub.  It's also not available on npmjs.com.  I don't see any actions associated with the repo.  Do you manage npm releases manually?

BTW, I don't see 0.8.0 showing up in the releases list on GitHub. It's also not available on npmjs.com. I don't see any actions associated with the repo. Do you manage npm releases manually?

Oh, forgot about this. I use np, and it seemed to succeed, but maybe it failed in some stealthier than usual way. I'll try again.

There, now it's released. Thanks for the heads up!

Thanks Glenn,
I see it now.  Thanks also for fixing the one last PR.

On Nov 30, 2021, at 2:35 PM, Glenn Slotte @.***> wrote:
There, now it's released. Thanks for the heads up!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/glennsl/rescript-jest/pull/98#issuecomment-982954899, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACA4DOB7BOUWTWXOSPJWFV3UOURRJANCNFSM5I5IN47A.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Animated: useNativeDriver was not specified. This is a required option and must be explicitly set to true or false

Animated: useNativeDriver was not specified. This is a required option and must be explicitly set to true or false

Hi, I also have the same error. Were you able to solve it?


feat: support ptr of ptr of ... in mapping

Codecov Report

Merging #2779 (5798f30) into master (bae061a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted Files
Coverage Δ

core/stores/redis/hook.go
82.88% <ø> (ø)

core/mapping/unmarshaler.go
88.55% <100.00%> (-0.16%)
:arrow_down:

core/mapping/utils.go
91.40% <100.00%> (+0.23%)
:arrow_up:

core/stores/mon/trace.go
100.00% <100.00%> (ø)

core/stores/sqlx/trace.go
77.77% <100.00%> (ø)

rest/handler/tracinghandler.go
100.00% <100.00%> (ø)

rest/httpc/requests.go
100.00% <100.00%> (ø)


Unable to select date in the past

Hey @reecelucas. Looking at dateIsSelectable() in helpers/date.js, am I right in assuming you can't select a day that is in the past?
Combinations of initialDate,  minDate and maxDate don't seem to allow it either.

Hi @thisoldbear,
This should be fixed in v2.0.0, thanks to @Malven . Past dates are now selectable by default.  The minDate prop can be used to override this behaviour. Hope this helps!


As a coder, I want to be able to view the README.md file to learn more about the app's purpose.

Complete README file on main page - Adding more information to the README file such as the application's purpose, what problem it intends to solve, and how it will go about solving that problem will be helpful to others who review your code.

updated readme file for better understanding of the app.


Promote recorded events on What's On

What is it and who's it for?
People who missed out on attending an in-person or livestreamed event can catch up on the most recent past events
User story
Given that I wasn't able to attend an event
I want to see what's available online to watch
So that I can watch what I missed
Implementation

Most recent recorded events promoted from What's On

Acceptance criteria

New slice added below Events for recorded events - section header should be Catch up
Recorded events display reverse chronological of the date the original event occurred

Testing and tracking

[ ] Setting up additional tracking and monitoring - talk to Tacey

Measurement

Related KPIs (e.g. breadth of collection accessed, reading time, etc.)

3 December 2020
Order of What's on:
Exhibitions, Events, catch up events
Catch up events on What's On
catch up events will be a new slice
what do Editorial want to call this section?
order of those past events: reverse chronological of date of original event

Danny wants to use Catch up for the section header

@jennpb couple of questions:

Do we want these segmented by date as we do events?
If not how many do we want in the list?

Do we want these segmented by date as we do events?
No, just newest to oldest order

If not how many do we want in the list?
Max of 6, link to /events/past to find additional

This is behind a toggle on the site. Awaiting further instruction.

@jamesgorrie Is the further instruction you're awaiting the question about what to do when there are no catch-up events? Or is there something additional you're waiting for?

@jennpb what I was looking for was the messaging when there are no events, which is currently "There are no upcoming catch up events". There is an event now - so we are going to remove the toggle and make this live.

Super - from now on we'll always have catch up events 🥳


pull request for e2e test at 01/21/2017 16:26:18

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Improve clean command message

This makes it clear when there are still command pending if some commands are taking too long
Before:

After:

Thanks


Home page - Visual design

Home  is complete with images. @amy-jung @twblack88 how is content doing on this page?

all done and staged.


Jenkins PR unit tests checks failing to create a new baseline

Impact of the bug
Jenkins
Describe the bug
It's been weeks that I see PR jenkins reports like:
Python3 Unit tests: succeeded
* 7 tests added

however, those 7 new unit tests have been added by a PR that has been merged many weeks ago.
So, something in our PR vs baseline unit test setup is not persisting the new unit tests, and it keeps considering those as new over and over.
How to reproduce it
Not sure
Expected behavior
There are 2 baseline jenkins releases created every day. So, whenever a PR gets merged, the unit tests that it adds should be persisted in the next baseline jenkins release, and those should stop being reported as new tests.
Additional context and error message
None

The 7 unit tests added for MSUnmerged were failing in the Jenkins job DMWM-WMAgentPy3-TestAll due to the following openssl RuntimeError.
RuntimeError: You are linking against OpenSSL 1.0.2, which is no longer supported by the OpenSSL project. To use this version of cryptography you need to upgrade to a newer version of OpenSSL. For this version only you can also set the environment variable CRYPTOGRAPHY_ALLOW_OPENSSL_102 to allow OpenSSL 1.0.2.

Resolved by setting this environment variable in the Jenkins tests (which was already set for the PR tests).
export CRYPTOGRAPHY_ALLOW_OPENSSL_102=1

Now PR tests are coming back with proper counts for added tests.
Jenkins results:

Python3 Unit tests: succeeded
 * 1 changes in unstable tests

Wonderful! Thank you very much for pin-pointing this problem and getting it fixed!


Get the proper exception logs of a task when the task is retried

Looking at the code at https://github.com/Parsl/parsl/blob/185283452bafdd477a638c5882f259a962abf5d3/parsl/dataflow/dflow.py#L254,
It does not seem to work. It is always None.
It would be great to get the exception so that we can log that to monitoring.db.
Probably related to #111.

I suspect what is happening here is that the exceptions you are getting are returned inside RemoteExceptionWrapper as a regular result of the future, rather than as the future going into an exception state.
The code around line 245,
            res = future.result()
            if isinstance(res, RemoteExceptionWrapper):
                res.reraise()

ensures that if an exception arrives by the regular Future exception way, or by the RemoteExceptionWrapper mechanism, it will be raised as an exception; but only in the first way will future be storing an exception.

Possibly catching the exception using except Exception as e: instead of just except Exception and using the newly bound variable e instead of future._exception will give you the value that you want in either exception handling path.


Add automatic migration for accordion classes

This automatic migration can not be written with our actual migration setup.
CWF v4 accordions are integrated with card classes.
As long as we're not able to select a node by a querySelector (parent > child) like approach, it's not possible to select the right elements only in accordions.

This is now possible to do, with the new dom-migration-utils.


Invalid Api-Key ID in v5.0.9 but not in v5.0.8

I updated the xchange-binance version from 5.0.8 to 5.0.9 (and changed nothing in my code). After updating the lib, the following error happens:
Caused by: org.knowm.xchange.binance.dto.BinanceException: Invalid Api-Key ID. (HTTP status code: 400)
Switching back to 5.0.8, no error and it works. Is there a changelog I can look what or why this breaks?

I have the same problem.
On Fri, Jun 25, 2021, 12:34 PM ts-00 @.***> wrote:

I updated the xchange-binance version from 5.0.8 to 5.0.9 (and changed
nothing in my code). After updating the lib, the following error happens:
Caused by: org.knowm.xchange.binance.dto.BinanceException: Invalid Api-Key
ID. (HTTP status code: 400)
Switching back to 5.0.8, no error and it works. Is there a changelog I
can look what or why this breaks?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/knowm/XChange/issues/4137, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIWQ7Y4RVXGASPT2Q5IZUDTURESRANCNFSM47JQ6JCQ
.

Any news on this one?

I have found the issue. The problem is on the getAssetDetails call on the remoteInit. If you complete delete it, it will work

Fixed here : https://github.com/knowm/XChange/pull/4198


Add better UI for adding podcasts

@ollicle @cilice
This is a place to discuss the "add podcast" UI enhancement that you talked about in #1.

hehe I wanted to assign you both but only one can be assigned :)

A thought for editing podcasts – click the My podcasts button to toggle:
http://output.jsbin.com/muhosu/3
Clearly doesn’t address the immediate need to adding podcasts.
Thinking about that I reckon a nice UI for adding a podcast:

A single field for entering the iTunes podcast url. (perhaps some instruction on how to find it)
From the url the id can be extracted with simple regex.
Pull the feed for the corresponding podcast
Display an appropriate error if it fails
If successful populate a podcast name field and id field with data from the feed
Name field may be customised

Although the above makes me wonder if there is any benefit for exposing the id field for editing?

Idea for adding podcasts:
have an option in the podcast dropdown that says "add new podcast", when you chose that it triggers a dialog that expects the URL or name and ID and then adds the podcast to the dropdown
OR
we use the new ui that ollie did and have the side slide down and reveal the form to add the podcast.

Clearly doesn’t address the immediate need for adding podcasts.

I can just show a special element that asks to enter the first podcast when there is none entered yet. It's easy to do.

Progress thinking along the lines of there is no need to edit a podcast once it is added:
http://output.jsbin.com/muhosu
For adding a podcast the next step depends on whether there is a need to customise the title before it is added?

I think the mockup is going in a good direction, what do you think @cilice?
We also need a new review list view with the artwork + name on top of the list, yes?

Oh, i like this one a lot. I'll try to implement it as soon as i can.


navd: rewrite in python

TODO:

[x] Testing
[x] Ensure bearing works correctly
[x] Ensure end of route behavior matches old navd
[x] Load last gps position from param
[x] Resend route on manager restart

I keep receiving this message. Does it have sth to do with these recent changes?
Process navd:
Traceback (most recent call last):
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/openpilot/selfdrive/manager/process.py", line 43, in launcher
    getattr(mod, 'main')()
  File "/openpilot/selfdrive/navd/navd.py", line 243, in main
    route_engine = RouteEngine(sm, pm)
  File "/openpilot/selfdrive/navd/navd.py", line 50, in __init__
    self.mapbox_token = Api(self.params.get("DongleId", encoding='utf8')).get_token(expiry_hours=4 * 7 * 24)
  File "/openpilot/common/api/__init__.py", line 13, in __init__
    with open(PERSIST+'/comma/id_rsa') as f:
FileNotFoundError: [Errno 2] No such file or directory: '/root/.comma/persist/comma/id_rsa'

This is running on the PC?

Is the renderer gone?


New version: NaturalNeighbours v1.2.3

Registering package: NaturalNeighbours
Repository: https://github.com/DanielVandH/NaturalNeighbours.jl
Created by: @DanielVandH
Version: v1.2.3
Commit:<PHONE_NUMBER>16e45b705ecf50eb59c86d5bc201d1
Reviewed by: @DanielVandH
Reference: https://github.com/DanielVandH/NaturalNeighbours.jl/commit/370939073616e45b705ecf50eb59c86d5bc201d1#commitcomment-130580749
Description: Natural neighbour interpolation methods for scattered data interpolation and derivative generation of planar point sets.

#93837


healthcare: change ListMessagesResponse to account for new fields

If anyone can provide insight into the failure I'd appreciate it. It just says "Process completed with exit code 1." on the goimports test.

Run goimports -w . (from the "Instructions" one right after).

Run goimports -w . (from the "Instructions" one right after).

Done, thanks.


Fix: #317 Get config data with network mode

Summary (Please recap what you changed or fixed)

Add useConfiguration hook

Checklist (If you won't pass this checklist please comment why)

[x] I removed all commented or unnecessary code.
[ ] Provide the screenshots due to UI changed or bug fixed
[ ] My code has covered these test cases (please list down your tested cases):

wait for next release


for<> is erased from macro calls

Given this input:
a!(for<> A);
a!(for<A> A);
a!(for<A: A> A);
a!(for<A: ?A> A);

rustfmt produces the following output:
a!(A);
a!(A);
a!(A);
a!(A);

Which is incorrect since macros can depend on the for<...>. (playground)

Thanks for the report!
Seems like this could be related to #4631 as both deal with dropping tokens in macro calls.

Doing some testing and it seems like #5301 will fix some of the examples you outlined, but the first case a!(for<> A); is still an issue even after testing this snippet on that branch.

I believe all but the first issue were resolved by #5848. @compiler-errors do you have any interest in digging into the empty for<> case?


Handling "ogc_fid" column

When gdal creates a pg_dump, it creates a serial pk column "ogc_fid", which was not present in the original source dataset. These are not generated for csv/parquet outputs.
Because unique, simple ids are useful, many product builds have logic that uses these ogc_fid columns, and therefore will break when naively importing parquet files. So we need a fix. I propose we have our parquet/csv to_sql methods handle this. Can have an optional id arg which can specify the name of the id column if present, and if not have it create a serial pk column. We could simply keep it as "ogc_fid", or just call it "id". "ogc_fid" is nice in that it seems unlikely that we'd ever have a collision, I could imagine we have a dataset that has an "id" column that's not just a serial db pk.

This was implemented, presumably we'll have to phase out these columns in some way post overhaul, but I don't think this really needs to be scoped right now


MEross plugins starts and stops and so on

What issue do you have? Please be as thorough and explicit as possible.
The Meross bridge with Hoobs starts and stops and starts and stops
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?

Hoobs

Which version of Homebridge/HOOBS do you have?

4.1.9

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.

Raspberry Pi

Which version of this plugin (homebridge-meross) do you have? Has the issue started since upgrading from a previous version?

71.0

Which Meross devices do you have that are causing issues? Please include product models if applicable.

msg100
Please paste any relevant logs below.

I can't paste any logs because there is no bug report. It only starts and stops after prompting user and passwort

With a popup of hoobs and i can the the logs that there are alwasy the same messages

anyone here with an MSG100 device? would you have any insight into this issue?
https://github.com/bwp91/homebridge-meross/issues/512


Not being able to download and continue installation

Hello! My wabbajack is not being able to download these 3 mods and so it doesn't let me continue :(
Thanks!
-High_Poly_Head_v1.4_(SE).zip
-Remastered Ash Piles Of Skyrim-44971-1-1-1611883469.7z
-OBIS SE - Headhunter - patch-52520-1-05-1648483442.zip
Downloading OBIS SE - Headhunter - patch-52520-1-05-1648483442.zip
34.2 - Downloading High_Poly_Head_v1.4_(SE).zip
34.2 - Downloading Remastered Ash Piles Of Skyrim-44971-1-1-1611883469.7z
34.27 - File hash check (-42) 4f04c55d2bc0afe4ccb523a180fb8d508b120d370b1aee66a8e80c6c8234dc2c
34.33 - Loading for alternative to eFtZWCVdD/U=
34.33 - Loading for alternative to ZpTIdCj0cTc=
34.33 - No alternative for eFtZWCVdD/U= could be found
34.33 - No alternative for ZpTIdCj0cTc= could be found
35.07 - IPS4 Request Error InternalServerError Internal Server Error -
https://vectorplexus.com/api/downloads/files/283
{
"errorCode": "EX6",
"errorMessage": "UNKNOWN_ERROR"
}
35.07 - Loading for alternative to gsvUHbjw0js=
35.07 - No alternative for gsvUHbjw0js= could be found
35.11 - Hashing Remaining Archives
35.11 - Looking for files to hash
35.54 - Found 2168 total files, 2097 matching filesize
36.17 - Unable to download High_Poly_Head_v1.4_(SE).zip (VectorPlexusOAuthDownloader+State|283|High_Poly_Head_v1.4_(SE).zip|False)
36.17 - Unable to download OBIS SE - Headhunter - patch-52520-1-05-1648483442.zip (NexusDownloader+State|SkyrimSpecialEdition|52520|273294)
36.17 - Unable to download Remastered Ash Piles Of Skyrim-44971-1-1-1611883469.7z (NexusDownloader+State|SkyrimSpecialEdition|44971|182506)
36.17 - Cannot continue, was unable to download one or more archives

I had an issue getting the obis and ash piles mods too. you can download them manually from the nexus
https://www.nexusmods.com/skyrimspecialedition/mods/44971?tab=files
https://www.nexusmods.com/skyrimspecialedition/mods/52520?tab=files (update file)
High poly head you can get from here https://drive.google.com/drive/folders/1V_jcYzwTiKnSv8Dbv-7Z0hh9SWbkn6Bi

I had an issue getting the obis and ash piles mods too. you can download them manually from the nexus
https://www.nexusmods.com/skyrimspecialedition/mods/44971?tab=files https://www.nexusmods.com/skyrimspecialedition/mods/52520?tab=files (update file)
High poly head you can get from here https://drive.google.com/drive/folders/1V_jcYzwTiKnSv8Dbv-7Z0hh9SWbkn6Bi

Thank you for your help with this mods!! I truly appreciate it!


Setting for disabling "Show more workflows..." in Actions tab

Description
I have like 20+ workflows defined in GitHub repo and it always collapse workflows list and show this Show more workflows.... To see multiple workflows runs I have to click it multiple times just to see full list of workflow and choose workflows I want. It is pretty irritating tbh.

I could not find any setting related to this in Refined GitHub settings so this is my feature request. Thank you for this great tool.
Example URLs
https://github.com/refined-github/refined-github/actions

Example URLs
refined-github/refined-github/actions

Can you provide a URL where there's a "Show more workflows" button?

Atm I can't because it is inside private repository, but I will prepare some dummy public repo with this and will link it here.

@fregante here it is https://github.com/piotrekkr/workflow-test/actions

They added workflow pinning now, which means you might be able to just pick the important ones now.
If not enough, this feature can be implemented in one line in your userscript manager ($('that link')?.click())

Okay thanks. Will use userscript then.


Added Undertow thread pool parameters

Fixes #1375
Changes proposed in this pull request:

Added Undertow thread pool parameters

Check mailbox configuration when submitting. Contributor Guide


DS-1996: Embargo Support in ItemImport

https://jira.duraspace.org/browse/DS-1996
Add the ability to embargo submissions using the ItemImport command line interface. We propose adding the following option to contents file in order to embargo bitstreams: \tembargo:. Dates take the form of YYYY-MM-DD, YYYY-MM, or YYYY.

Closing as outdated/obsolete (no activity in several years). Please resubmit if you are interested in helping move this forward. Thanks!


Create dbSchemaChallenge_12.ts

Added Create dbSchemaChallenge_12.ts

🎉 Snyk hasn't found any issues so far.
✅ security/snyk check is completed. No issues were found. (View Details)
✅ license/snyk check is completed. No issues were found. (View Details)


Wrong type argument error

When I tried running this in Spacemacs I got the following error when using the C-c s t p command:
(wrong-type-argument char-or-string-p nil)
  write-resp-val-to-buf(nil #<buffer *sayid-traced*>)
  sayid-setup-buf(nil t nil)
  sayid-req-insert-meta-ansi(("op" "sayid-show-traced" "ns" nil))
  sayid-show-traced()
  sayid-trace-ns-by-pattern()
  call-interactively(sayid-trace-ns-by-pattern nil nil)
  command-execute(sayid-trace-ns-by-pattern)

I was using the newest development version (develop branch) with the default clojure layer. Sayid was added as an additional package using a melpa recipe from github:
(sayid :location (recipe :fetcher github                                                                :repo bpiel/sayid                                                                :files ("src/el/*.el")))
How should I fix this?

Do you have cider connected to nrepl?

I did, I used cider-jack-in

@zane have you gotten sayid working with spacemacs at all? thanks

@bpiel Seems to be working now. Not sure what was going, sorry to bother you. Maybe I just needed to restart emacs (I thought I did that).

@DonyorM No problem. I like hearing that sayid is being used (or attempted) and appreciate having an official report of it working with spacemacs. thanks

@bpiel I'm working on integrating it with the clojure layer. Not working perfectly yet, I may be back later :)

@DonyorM Keep the issues coming. Thanks for your interest and effort.

Ok I figured out the original error. On my own project (which has only one namespace) it seems to trace fine. However, if I try and trace the contrived example (either with a wildcard or without) it gives the error I said above. I have no idea why.

@DonyorM that is weird. Do you get any exception output from the repl? I'll think about how to tackle this.

I am actually, I didn't notice that. It says
FAIL in (test1) (machine_test.clj:23)
expected: (= (-> test-vending-machine (m/insert-coin :quarter) (m/insert-coin :dime) (m/insert-coin :nickel) (m/insert-coin :penny) (m/press-button :a1)) expected-vending-machine)
  actual: (not (= {:inventory {:a1 {:name :taco, :price 0.85, :qty 9}}, :coins-inserted [], :coins-returned [:quarter :quarter :nickel], :dispensed {:name :taco, :price 0.85, :qty 10}, :err-msg nil} {:inventory {:a1 {:name :taco, :price 0.85, :qty 10}}, :coins-inserted [:quarter :dime :nickel :penny], :coins-returned [], :dispensed nil, :err-msg true}))

However, I get the same error when I try to trace the functions in the non-test namespace (contrived-example.machine) I get the same emacs error, but no output on the repl. It may not be related

@DonyorM How are you including sayid in the clj project? Is it in your profiles.clj? The contrived-example project doesn't include it directly (in project.clj), so maybe sayid isn't loaded at all?

That would do it. Sorry I for some reason didn't think to check if it was loaded in the project. I'll add that and try it again, though I imagine that's the issue.
A better error message for this situation would be very helpful, if that's possible (I don't know elisp at all).

There should definitely be a better error message.

Made an issue
https://github.com/bpiel/sayid/issues/11

Confirmed the issue was the missing sayid plugin in the project.


engine: remove two levels of indentation

(see commit message)

Yes :) I thought I should get this in sooner than later.


[ci] Adds test for web hot restart with const App.

This PR adds a test that reproduces the problem described in the linked issue: hot restart on the web seems to not update if the app being run is const.
The new test is expected to fail, until the const issue with hot restart in the web is resolved.
Expected failure mode is a 15s timeout in the following test:
02:31 +3 ~1 -1: Hot reload (index.html: Default) (with `const MyApp()`)): newly added code executes during hot restart [E]
  TimeoutException after 0:00:15.000000: Future not completed
  dart:async  _startMicrotaskLoop
  ...

(And then a bunch of output that I'm not 100% sure is intended :))
Issues

#141588

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Seems to be failing as expected: https://ci.chromium.org/ui/p/flutter/builders/try/Mac web_tool_tests/36148/overview

Thanks David, LGTM! I have not idea what the process would be to land this change if it is failing currently. I'll try using it locally though to verify my changes in the Dart SDK.

Probably the best way would be to manually verify the dart sdk changes running this test locally, and then once the fix lands in the Dart SDK main and rolls to flutter/flutter, we can rebase this PR and the presubmits should go green, which would verify the fix.

Probably the best way would be to manually verify the dart sdk changes running this test locally, and then once the fix lands in the Dart SDK main and rolls to flutter/flutter, we can rebase this PR and the presubmits should go green, which would verify the fix.

Sounds good, can you help me identify the right command to run this test locally if I already have my engine built with my changes to the embedded dart sdk?

Probably the best way would be to manually verify the dart sdk changes running this test locally, and then once the fix lands in the Dart SDK main and rolls to flutter/flutter, we can rebase this PR and the presubmits should go green, which would verify the fix.

Sounds good, can you help me identify the right command to run this test locally if I already have my engine built with my changes to the embedded dart sdk?

Ooh, that's a good question actually....

@nshahan this is the command I've been using to run this test in my local checkout of the flutter framework:
dit@dit:/work/flutter/flutter/packages/flutter_tools$ flutter test test/web.shard/hot_reload_web_test.dart

(But what @christopherfujino might help with overriding the SDK)

[...] the process would be to land this change if it is failing currently. I'll try using it locally though to verify my changes in the Dart SDK.

[...] once the fix lands in the Dart SDK main and rolls to flutter/flutter, we can rebase this PR and the presubmits should go green

This is the idea, yes; this will need to be rebased before it can land (or be landed at the same time as a the engine roll that contains the fix).

This should pass, now that https://github.com/flutter/flutter/pull/142098 has rolled into the framework.

However, they don't :/
04:28 +8 ~2 -1: /Volumes/Work/s/w/ir/x/w/flutter/packages/flutter_tools/test/web.shard/hot_reload_web_test.dart: Hot reload (index.html: Default) (with `const MyApp()`)): newly added code executes during hot restart [E]
  TimeoutException after 0:00:15.000000: Future not completed

186s            Spawning flutter [run, --machine, --no-serve-observatory, -d, chrome, --web-run-headless, --verbose, --web-renderer=html] in /Volumes/Work/s/w/ir/x/t/flutter_hot_reload_test.1XfRrI

https://ci.chromium.org/ui/p/flutter/builders/try/Mac web_tool_tests/36340/overview

Hot restart seems to work in my Linux machine, though, so I'm starting to think that this test is SUS :/

(Or maybe the "update with master" button didn't do what I wanted :/

This seems to work locally, so the test must be bad :/

@nshahan suggested a fix for the test (related to an old, linked issue) and it seems to work as expected now! (I'll verify the test does fail by rewinding master to before Nic's fix landed)

I've just checked that if I apply this PR on top of beta, the test fails as expected. See this branch.

Do your thang, autosubmit!


Support complex keys in mappings

Currently, we assume that the mapping node consists of (scalar node, node) pairs, but YAML spec allows complex keys to be used (AFAIK it could be arbitrary YAML node) in mappings.

for reference, this looks like a bit more esoteric feature, I've tried to parse an example from spec with few online yaml parsers, only this one worked: https://jsonformatter.org/yaml-parser
? - Detroit Tigers
  - Chicago cubs
:
  - 2001-07-23

? [ New York Yankees,
    Atlanta Braves ]
: [ 2001-07-02, 2001-08-12,
    2001-08-14 ]


Note what happens if you modify bidderSettings

If you modify bidderSettings, you can't use the setPriceGranularity
method (it doesn't work).
Addresses https://github.com/prebid/prebid.github.io/issues/88

@mkendall07 added notes to Publisher API reference in both setPriceGranularity and bidderSettings sections.
Let me know if this language works for you or if a different way of putting it would be clearer.


8 digits border color property disappear when ng build --prod

Versions
    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/

Angular CLI: 1.7.4
Node: 8.11.1
OS: darwin x64
Angular: 5.2.10
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.7.4
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.3.2
@angular-devkit/schematics: 0.3.2
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.2
@schematics/angular: 0.3.2
@schematics/package-update: 0.3.2
typescript: 2.5.3
webpack: 3.11.0

Repro steps

ng new test-app
Set css border property with 8 digit color, like border: solid 5px #00000050;.
ng build --prod
border: solid 5px; generated

Mention any other details that might be useful (optional)
Tried pattern is here.

background-color: #00000050; and ng build --prod => OK
border-color: #00000050; and ng build --prod => OK
border: solid 5px #00000050; and ng build => OK
border: solid 5px #00000050; and ng build --prod => NG

my project is also getting css with errors after prod compilation with 1.7.4 (http://github.com/orizens/echoes-player)

I'm sorry, but we can't reproduce the problem following the instructions you provided.
Remember that we have a large number of issues to resolve, and have only a limited amount of time to reproduce your issue.
Short, explicit instructions make it much more likely we'll be able to reproduce the problem so we can fix it.
If the problem persists, please open a new issue following our submission guidelines.
A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.


Return 404 for unknown facet fields

Bug Report
Description
Probably due to some old search urls that are still out there, a query for a facet field that doesn't existing anymore will cause ActionView::Template::Error: undefined method 'facet_config' for nil:NilClass
We should return 404 for these.
To Reproduce
Steps to reproduce the behavior:

Create a link such as /catalog?f%5Bbased_near_sim%5D%5B%5D&f%5Bresource_type_sim%5D%5B%5D&sort

@srerickson This does not actually raise the error that Adam describes—perhaps this was fixed previously, or perhaps we installed an update to blacklight. Either way, the example url does not bomb out, it simply renders a list of search results with no facets selected, which in my humble opinion is probably a better solution than 404ing


Add BETTER_ENUMS_IGNORE_ATTRIBUTES_* for clang

The build errors in Travis CI seem to be unrelated to the change here. Could you check this, @aantron?

@janten Thanks. Yes, the Travis errors are unrelated, and we should ignore them :) I should probably disable Travis for now, as I don't have time to fix this the "right" way at the moment.
Can you say what error this PR addresses? These macros were added because a GCC version had broken handling of -Wattributes (#79). Is there a similar problem with some versions of Clang?

I am using Xcode 11.3 and was getting warnings about BETTER_ENUMS_IGNORE_ATTRIBUTES_BEGIN being undefined, which I fixed here. A previous version of Better Enums, which did not have BETTER_ENUMS_IGNORE_ATTRIBUTES_BEGIN all, also worked fine.

Thanks. I think these macros should be defined to expand to nothing on Clang. Can you try that?

It works with empty #defines, too. I updated the source accordingly.

Thanks again!


Skip features or scenarios by @Pending annotation

According to the issue: https://github.com/curit/ember-cli-yadda/issues/17
It's now possible to skip any feature or scenario by adding @Pending annotation. It's now supported when tests are running on QUnit. I've used native QUnit skip function, so in test reports it will be shown how many tests were skipped by @Pending annotation ;).

PS. Fell free to edit, @Pending annotation can be renamed to @Ignore

Thanks for the PR!
Can you do this for mocha too? 😄 And maybe add a little doc in the README.md?

@Dejavs see #38


chembench 2.0 - collecting ideas

Frameworks

https://github.com/stanfordnlp/dspy

Some Introductory notes on prompting
https://www.promptingguide.ai/

Another library https://github.com/outlines-dev/outlines/tree/main

another framework https://ukgovernmentbeis.github.io/inspect_ai/

another framework that is built for comparing models https://promptfoo.dev/

another framework https://ukgovernmentbeis.github.io/inspect_ai/

this one looks quite robust, but I see that we cannot add new models if using some of their code

@lamalab-org/chem-bench I think our current direction is to keep this codebase here very independent of what the actual system does.
I guess the ideal outcome for us might be that the chembench-core repo does not have any model-related dependencies


notice issue showing 500 error

Laravel Version: 5.7.4
PHP Version:7.2.10
Database Driver & Version:Mysql 5.7.23

Description:
When notice occured in laravel it showing status code 500 (internal server error).
And execution stopped.
    public function update(Request $request, $id)
      {

	$error = array();

	$service_data = $request->all();

	try {
		$data = Service::findorfail($id);
	} catch (ModelNotFoundException $exception) {

		$error = array(

			"name" => "Data not found",
			"message" => "Requested ID not found.",

		);

	}

	if(empty(trim($service_data['name']))){

		$error = array(

			"name" => "Missing fields",
			"message" => "Requested fields cannot be empty for updation.",

		);

	}

	if(empty($error)){
		return response()->json(['data' => $data, 'status_code' => '200'],200);
	}else{
		return response()->json(['error'=> $error, 'status_code' => '404'],404 );
	}

  }

Above code showing notice when APP_DEBUG = TRUE in .env and app.php but status is 500.
Notice is : Undefined index 'name'.
When I set APP_DEBUG = FALSE then it is not showing notice but showing 500 error.
This should not happen when APP_DEBUG is FALSE. Why it is showing 500 for notice.
If I am not checking any variable then it's working fine like
empty(trim($service_data['name'])

then it is not showing notice and also 500.

This is expected. Laravel throws 500 errors for notices.


cdcusb.h: No such file or directory

Best I could find to solve this was to include: Adafruit_TinyUSB.h library with no success.
Arduino: 1.8.19 (Mac OS X), Board: "ESP32S2 Dev Module, Enabled, Enabled, Enabled, Internal USB, Disabled, Default 4MB with spiffs (1.2MB APP/1.5MB SPIFFS), 240MHz (WiFi), QIO, 80MHz, 4MB (32Mb), 921600, None"
In file included from /Users/Choodyp/Downloads/RubberNugget-main/RubberNugget/RubberNugget.ino:4:
RubberNugget.h:6:10: fatal error: cdcusb.h: No such file or directory
#include "cdcusb.h"
^~~~~~~~~~
compilation terminated.
exit status 1
cdcusb.h: No such file or directory
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

Hey!
You can find the TinyUSB we're using here: https://github.com/chegewara/EspTinyUSB.
This should include the CDC USB file referenced.

The RubberNugget also depends on a few other libraries including our custom graphics library, which I'll statically upload to the repository when I get around to it.

Yea now its getting stuck on Nugget_Interface.h
Thank you! Look forward to trying this out when its ready!

You can actually try out the v1.0 beta binary here!
https://github.com/HakCat-Tech/RubberNugget/releases/tag/v1.0-beta
you just need to upload payloads under a subfolder for each os (os->category->payload.txt)

Still having some issues. Tried to unpack the .zip v1.0 beta and open the .ino. but its then getting stuck on the same issue when uploading to the board. base64.hpp : no such file or directory. I think I need a walkthrough to get this going. Should I close this issue?

I was able to flash the binary to the nugget using esptool but the nugget appears dead after the flash. Reset does nothing. Am I missing something?
/Users/ben/Library/Arduino15/packages/esp32/tools/esptool_py/3.3.0/esptool --chip esp32s2 --port /dev/cu.usbmodem01 --baud 921600 --before default_reset --after hard_reset write_flash -z --flash_mode dio --flash_freq 80m --flash_size 8MB 0xe000 /Users/ben/Downloads/v1.0-beta-RubberNugget.bin

Try flashing here in Chrome: https://nabucasa.github.io/esp-web-flasher/

Excellent! That worked. Thank you. I think I may have a defective nugget though. It appears my nugget's buttons are not working except for the bottom button. This was happening as well with the Rubber Nugget Circuit Python version. I've tried shorting the pins for the other buttons and different cables. When I press the bottom button it works but on the next screen I have to press the up button to run my payload and I can't because the up button isnt working. Can I try and fix this on my own? I didnt assemble my own Nugget but am willing to self diagnose the problem with some instruction.

@btasteel636 Open a new bug about the buttons and we'll work with you to diagnose this. @AlexLynd can close this one when the libraries needed are added here.
First though, check if using a jumper wire to jump the buttons gives you a button press. If the button presses when you jump the connection, the button may have a bad solder joint. If it doesn't it might be an issue with the PCB or the microcontroller. But let's discuss in a new bug thread.


[samples]Remove defaulted containerImage fields

We have defaulting webhook to set the containerImage fields so to actually test that webhook in the e2e job this patch removes the explicit values from the containerImage fields from our samples for those services where the webhook integration is already done in openstack-operator.

Does sub CRs in Glance fails to default?
+ oc kustomize /alabama/install_yamls/out/openstack/openstack/cr/
+ oc apply -f -
The OpenStackControlPlane "openstack" is invalid:
* spec.glance.template.glanceAPIInternal.apiType: Unsupported value: "": supported values: "internal", "external"
* spec.glance.template.glanceAPIExternal.apiType: Unsupported value: "": supported values: "internal", "external"

Does sub CRs in Glance fails to default?
+ oc kustomize /alabama/install_yamls/out/openstack/openstack/cr/
+ oc apply -f -
The OpenStackControlPlane "openstack" is invalid:
* spec.glance.template.glanceAPIInternal.apiType: Unsupported value: "": supported values: "internal", "external"
* spec.glance.template.glanceAPIExternal.apiType: Unsupported value: "": supported values: "internal", "external"

It is probably fixed by https://github.com/openstack-k8s-operators/glance-operator/pull/212 but we need to bump that glance version into openstack-operator first.

the latest glance-operator is now used in openstack-operator so this PR is rebased and expected to be green.


The example project is bitrotting.

Bit rotting = source code that stops working because of language or package dependency changes and depreciations.
For a beginner, having to solve broken dependency issues on top of learning the language is a tall order.
Some package dependencies have evolved so much that the minimum version available is above teh version required by the example project.
Example project should be reviewed and updated to be a smooth installation for the beginner to test.
Ste´ps to reproduce:

follow the instructions.
when you reach the step "build install", dependency errors arise.
Project can't run.

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

ID: b9d5437c-5b4f-bf5b-4877-e810d5fefb4f
Version Independent ID: eea67277-74ba-c23c-bee5-af4cf6eb14db
Content: Quickstart: Create a Ruby app - Azure App Service
Content Source: articles/app-service/quickstart-ruby.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@BrazilianJoe we'll review your feedback and get back to you shortly. Thanks.

I totally understand the concern about package dependencies. To better assist you on this, are you referring to 'bundle install' step?
As I don't seem to hit the dependency issue/As far as the doc flow is concerned, hope the 'Prerequisites' are taken care of.
If it's about Rails and Ruby version compatibility, please check this doc and thread for additional info.

@BrazilianJoe,  Just checking in to see if you have had a chance to see the previous response. Hope the above clarifies. Kindly let us know if you need further assistance on this topic.

@BrazilianJoe, We will close this out, but if you feel you need more information please just let us know.


[BACKLOG-8272] Added MDI Support to Multiway Merge Join

@mdamour1976

@mdamour1976 @jmwoliver This change broke meta-injection for the refinery steps.  We get a NPE at MetaInject:121.  Simple fix is to wrap in an if != null, but I don't really know the purpose behind this change, so don't want to commit it.  Would really love to have something in by eod so that QA can do a complete test of our new stuff

Also see MetadataInjectionModelTest.testJobEntry in the pentaho-data-refinery project.  It has a failure due to this change


endless loading of Avito task list and some others

Describe the bug
endless loading of Avito task list and some others
To Reproduce
Steps to reproduce the behavior:
https://leetcode.com/problemset/
find "Avito" in "Trending Companies"
click
Screenshots

Describe the bug
endless loading of Avito task list and some others
To Reproduce
Steps to reproduce the behavior:
https://leetcode.com/problemset/
find "Avito" in "Trending Companies"
click
Screenshots

Yes sadly the database was last updated in October, i'll try to update it.

The problem is still relevant (checked for Avito)

@Reidak18 i know, thats why the issue is open, i will try to update the database, it's just complicated.


python310Packages.gaphas: 3.7.0 -> 3.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.gaphas is: "GTK+ based diagramming widget"
meta.homepage for python310Packages.gaphas is: "https://github.com/gaphor/gaphas"
meta.changelog for python310Packages.gaphas is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.8.0 with grep in /nix/store/45ch6zrawlsmlacyrchkp47g7cf4851i-python3.10-gaphas-3.8.0
found 3.8.0 in filename of file in /nix/store/45ch6zrawlsmlacyrchkp47g7cf4851i-python3.10-gaphas-3.8.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
gaphor
python310Packages.gaphas
python39Packages.gaphas

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/45ch6zrawlsmlacyrchkp47g7cf4851i-python3.10-gaphas-3.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.gaphas https://github.com/r-ryantm/nixpkgs/archive/c13481eefc158d5d6757636a56e66133b31c3589.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/45ch6zrawlsmlacyrchkp47g7cf4851i-python3.10-gaphas-3.8.0
ls -la /nix/store/45ch6zrawlsmlacyrchkp47g7cf4851i-python3.10-gaphas-3.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

gaphor
python310Packages.gaphas
python39Packages.gaphas

Maintainer pings
cc @WolfangAukang for testing.

Result of nixpkgs-review pr 193626 run on x86_64-linux 1

3 packages built:

gaphor
python310Packages.gaphas
python39Packages.gaphas


Can't create a file in the root of a project if there are no files already there.

VSCode Version: 1.3.1
OS Version: 10.9 Mac

Steps to Reproduce:

Add a new project
Add a folder
Select that folder
Try to add file in the root.

Works for me on MacOS with 1.3.1


Modal dispatch actions but not showing other scene

Version
Tell us which versions you are using:

react-native-router-flux v3.36.0
react-native v0.35.0

Expected behaviour
Modal to dispatch action to next scene.
Actual behaviour
Action PUSH and FOCUS dispatched but view scene doesn't change. Even the back button stop responding unless i click on it a few times (usually works after once, Action.pop() )
Did a console.log on scene that modal is suppose to transit to, logs at render and constructor successfully. But scene still stuck at modal.
Steps to reproduce

Create scene with modal
Transit to Modal scene
Trigger to dispatch action to another scene

Hi, @jomaint did you figure out how to fix it?


fix: Improve error handling in the plugin config model parser of hgctl

Ⅰ. Describe what this PR did
Improve error handling in the plugin config model parser of hgctl.

Fix typos in variable and method names.
Return field name and related info when unable to parse a specific model field.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report
Attention: Patch coverage is 40.00000% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 38.15%. Comparing base (08e5678) to head (c7c782a).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #891      +/-   ##
==========================================
+ Coverage   38.12%   38.15%   +0.02%
==========================================
  Files          61       61
  Lines       10438    10438
==========================================
+ Hits         3980     3983       +3
+ Misses       6158     6156       -2
+ Partials      300      299       -1

Files
Coverage Δ

pkg/cmd/hgctl/plugin/types/model_parser.go
69.20% <40.00%> (ø)

... and 1 file with indirect coverage changes


couldn't install cli via git bash on windows 11

I tried a lot but couldn't install cli via git bash on windows 11. Am I doing anything wrong? Need help

I think it's still not fixed properly,


Feature/new test parameter default

support for default parameter files

Doesn't seem like the ui is communicating with the webservice properly in tests

Codecov Report

Merging #130 into develop will decrease coverage by 0.02%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##           develop     #130      +/-   ##
===========================================
- Coverage    64.91%   64.89%   -0.03%
===========================================
  Files           95       95
  Lines         2554     2555       +1
  Branches       322      323       +1
===========================================
  Hits          1658     1658
  Misses         723      723
- Partials       173      174       +1

Impacted Files
Coverage Δ

...-workflow-modal/register-workflow-modal.service.ts
91.3% <33.33%> (-2.03%)
:arrow_down:

Continue to review full report at Codecov.

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


Seek position in stream (Python)

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Is there any way to seek audio position in youtube_dl (Python) for a stream?

No.


base directory cannot exec  env.sh ,case env.sh script dir not right,

base -> run.py ->start->start_tasks_in_cluster function, input params "case " `s  value not right.
env_dir = os.path.join(
config.FLAGPERF_PATH, "benchmarks", case, config.VENDOR)
env_dir ：cannot find dir ,

thanks for report, you can see if https://github.com/FlagOpen/FlagPerf/pull/710 can be a solution


Plotting intervals do not work

Tried to go through Tutorial, just copy/pasted code, plot function for SampleForecast entity  doesn't seems to work the way it should.

Originally posted by @ivanvoid in https://github.com/awslabs/gluonts/issues/2333#issuecomment-1501392375

Hi @ivanvoid,
I created a new issue from your comment. Let's discuss this here.
The plotting issue you mentioned happens sometimes with me as well. As far as I understand, it's due to conflicts between how pandas plotting interacts with plt.fill_between when using a datetime index on the x-axis. Can you share your GluonTS, Matplotlib and Pandas version?
As a workaround, you can go to this function in your GluonTS installation and replace all instances of self.index.to_timestamp() in the function to self.index. This has worked for me in the past and we can merge this as a bug fix.

In the future we ideally would not want to rely on pandas.
Another thing to do right with Zebras!

Fixed by #2800


Explicitly set rabbit flags for MN-CI upgrades

See https://github.com/stackhpc/stackhpc-kayobe-config/blob/7daaead629796270f183f801c4ec5929eff7796d/etc/kayobe/environments/ci-aio/kolla/globals.yml#L18
for the same format in ci-aio

"Set durable queue flags" not quite a correct description, but not worth blocking over

Oh but it is


Remove usages of ComponentResolveMetadata

The primary motivation here is to simplify LocalComponentMetadata (now renamed to LocalComponentGraphResolveMetadata), by reducing the type hierarchy. This will make it much easier to heavily modify the way local component metadata works.
ComponentResolveMetadata was a legacy type, previously used to represent a component when resolving artifacts. Now, the modern type, ComponentArtifactResolveMetadata, has been propagated to all prior places that used the legacy type.
All methods on the removed type have been moved to its remaining subinterface, ExternalComponentResolveMetadata. This subinterface is not heavily used, but removing it would cause more work than was desired for this change.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


Property "region" not yet implemented on Android

See tabris-maps-example app page "property: region".
When tapping the button on that page the console shows an error:

TypeError: Cannot read property '0' of null
...
pages/regionPage.js:24:56

region has been implemented on Android and iOS.


BAEL-2274 DDD Bounded Contexts and Java Modules

test this please

test this please

Hi, it seems that now it is okay, I haven't done anything :)


Shape problems in LSTM; Configuring LSTM properly in Keras

Issue type
Support
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
tf 2.8
Custom code
Yes
OS platform and distribution
Windows 11
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
So I wanted to make a neural network that predicts weather on python. It should take data on previous 30 days as input and make a prediction on how many % (+ or -) will the air temperature change. Neural network will compute 2 inputs, so data on a single day looks like this:
1-st input:
[20, 72, 1]
First entry is temperature, second is humidity and third is atmospheric pressure.
2-nd input:
[Weather observations]
[Weather observations] array contains long strings of weather observations. In this array there may be one observation, may be more than one or none.
One complete unit of training data for 30-ty days looks like this:
1-st input:
[ [20, 72, 1], #1-st day
[15, 89, 1.03], #2-nd day
[19, 79, 1.01], #3-d day
[24, 67, 0.98], #...
...
... ]
2-nd input:
[ ["Many cloudy spots...", "Today we had experienced..."], #1-st day
["Rain has been going for..."], #2-nd day
[""], #3-d day
["In following regions...", "Cause of geomagnetic...", "Today is one of the sunniest...], #...
...
... ]
So both of those input units contains thirty entries like this (becuase of mentioned 30-ty days), and the label for this unit is air temperature on the day that is a week ahead of the last entry.
Summarizing, the inputs will be lists of lists, where each list contains device readings/weather observations for 30 days with daily readings/observations placed in another list inside. So, first element in the second group of 30 days will be a set of observations which is second element in the first group of 30 days (aka 3-rank list, I hope you got it).
I wrote the following model:
Standalone code to reproduce the issue
embedding_dim = 128
vocab_size = len(words_to_int)  # number of unique words

num_input = keras.Input(shape= (30,3), name="nums")
nmlstm = layers.LSTM(64)(num_input) #num input goes to num lstm
nmdense = layers.Dense(64)(nmlstm) #num lstm goes to num dense

text_input = keras.Input(shape= (30, max_sequence_length), name="text") #basically length of the longest string observation sequence

text_vec = layers.Embedding(vocab_size, embedding_dim, mask_zero=True)(text_input) #text input goes to text embedding
txlstm = layers.LSTM(64)(text_vec) #text embedding goes to text lstm

united = layers.concatenate([nmdense, txlstm]) #concatenating text lstm and num dense
almostlast = layers.Dense(64)(united) #computing united input
last = layers.Dense(2, name='prediction')(almostlast) #output dense layer

model = keras.Model(inputs=[num_input, text_input], outputs=last)
model.compile(optimizer='adam', loss=keras.losses.CategoricalCrossentropy)
model.fit(
    {"nums": X1_train, "text": X2_train},
    {"prediction": y_train},
    epochs=8,
    batch_size=10)

### Relevant log output

```shell
When I try to run the code, I get this error:

ValueError: Input 0 of layer "lstm_1" is incompatible with the layer: expected ndim=3, found ndim=4. Full shape received: (None, 30, 2, 128)

I tried changing the input shape of text_input, but I kept on getting the same error.
To mention, I didn't have any problems on encoding data, padding it etc. Also it's better if there's a way of fixing it without changing the shape of 2-nd input data because the main goal is to make the neural network treat 30-day data as one input and providing only one output (how many % (+ or -) the air temperature will change) while learning on text observations to make predictions also based on them. Thank you in advance.

@flexorcist,
Your input needs to be reshaped.
input_shape: A 3D tensor with shape [batch, timesteps, features]
Keras' convention is that the batch dimension (number of examples (not the same as timesteps)) is typically omitted in the input_shape arguments. The batching (number of examples per batch) is handled in the fit call.
Also please take a look at this SO link and the https://github.com/keras-team/keras/issues/7403 with the similar error. Thank you!


Bug: Cannot use variable x because the declaration either comes later or was skipped.

export type Order = {
  x: number;
  y: number;
}
a: Array<Order> = [];
f() {
    let a_i = this.a.slice(-1)[0];
    a_i.x += 1; // that's ok
    a_i.y -= 1; // that's ok
    assert( a_i.x - a_i.x < 1 );
    if (true) {
        assert(a_i.x < 1); // that's not okay? why?
        // "Flow Error: Cannot use variable a_i because the declaration either comes later or was skipped."
        let a_i = this.a.pop(); // I do realize it's bad practice to declarate it again
        // but supposly that's not agsinst ES6 standards
        // if it's not - the error message is ambiguous and misleading.
    }
}

see screenshot https://www.dropbox.com/sh/m6vm307dj4267dc/AAD_AAlqUS6kyXzEeqb1t_z4a?dl=0

I think I misunderstood JS standard for that, sorry.


Class overrides no longer working after change to classpath

Describe the bug
Traditionally, the additions module has been used to override base DSpace classes. However, due to a recent change, overrides within additions (and likely other modules) are no longer functioning correctly.
The PR in question is PR 2544, and specifically the change which moved away from
JARS=`echo $DSPACEDIR/lib/*.jar | sed 's/ /\:/g'`

To Reproduce

Override a class from dspace-api in dspace/modules/additions (I found this when trying to override MediaFilterCLITool)
Call overridden class in a command-line context
Notice that the class within additions is not the one being used.

Expected behavior
Overrides within additions should take classpath priority to classes within dspace-api
Related work
PR 2544

Note: The reason this probably broke is because the wildcard classpath capability of Java says the order is "not specified and may vary from platform to platform", whereas the previous approach relied on globbing by the shell, which sorts alphanumerically.
From https://docs.oracle.com/javase/8/docs/technotes/tools/unix/classpath.html

The order in which the JAR files in a directory are enumerated in the expanded class path is not specified and may vary from platform to platform and even from moment to moment on the same machine. A well-constructed application should not depend upon any particular order. If a specific order is required, then the JAR files can be enumerated explicitly in the class path.

Accidentally made this in angular instead of REST. closing
See https://github.com/DSpace/DSpace/issues/3248


Recommend removal of music track with voice

The track "Trammell Starks - All That Jazz" has some of the original TWC narration in it. It starts by saying "Currently the temperature is 38 degrees with light snow". This is a bit jarring when it's 65 degrees and sunny outside :)

"Trammell Starks - Island Groove" has a similar artifact.

@nivex Thanks for noticing that!  I'll fix that right away :)

Done!  Website updated.  Thanks again!!


⚠ Bofrost.es has degraded performance

In 2a68400, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1593 ms

Resolved: Bofrost.es performance has improved in 73101a7 after 6 minutes.


Vagrant minion setup never completes

On latest pull, the Vagrant kube-up.sh scripts completes successfully, but the Salt setup in the minion keeps going forever. End result is a cluster that is not useable. Looking at '/var/log/salt/minion' on the minion box, see lots of entries like:
2015-03-04 20:09:11,952 [salt.minion      ][INFO    ] Starting a new job with PID 18901
2015-03-04 20:09:11,954 [salt.minion      ][INFO    ] Returning information for job:<PHONE_NUMBER>0911935191
2015-03-04 20:09:12,027 [salt.crypt       ][DEBUG   ] Decrypting the current master AES key
2015-03-04 20:09:12,028 [salt.crypt       ][DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem
2015-03-04 20:09:12,217 [salt.minion      ][INFO    ] User vagrant Executing command grains.items with jid<PHONE_NUMBER>0912208260
2015-03-04 20:09:12,217 [salt.minion      ][DEBUG   ] Command details {'tgt_type': 'glob', 'jid': '20150304200912208260', 'tgt': '*', 'ret': '', 'user': 'vagrant', 'arg': [], 'fun': 'grains.items'}
No other help looking at the master's salt log.
This is Vagrant 1.7.2, with VirtualBox 4.3.20 on Mac OSX (Yosemite).

Is this repeatable for you if kube-down and kube-up?

Yes, this is reproducible on my laptop. But, I was able to get it up by
doing a kube-push after the failed kube-up.
On Wed, Mar 4, 2015 at 1:11 PM, Eric Tune<EMAIL_ADDRESS>wrote:

Is this repeatable for you if kube-down and kube-up?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/5043#issuecomment-77248161
.

--
-- Fabio Yeon

Self assigning and will try to reproduce.
Sent from my iPhone

On Mar 4, 2015, at 4:18 PM, Fabio Yeon<EMAIL_ADDRESS>wrote:
Yes, this is reproducible on my laptop. But, I was able to get it up by
doing a kube-push after the failed kube-up.
On Wed, Mar 4, 2015 at 1:11 PM, Eric Tune<EMAIL_ADDRESS>wrote:

Is this repeatable for you if kube-down and kube-up?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/5043#issuecomment-77248161
.

--
-- Fabio Yeon
—
Reply to this email directly or view it on GitHub.

I was able to reproduce.
It appears that the Kubelet fails to start properly on first start:
Mar 05 14:55:21 kubernetes-minion-1 systemd[1]: kubelet.service: main process exited, code=exited, status=2/INVALIDARGUMENT
Mar 05 14:55:21 kubernetes-minion-1 systemd[1]: Stopping Kubernetes Kubelet Server...

Mar 05 14:55:03 kubernetes-minion-1 kubelet[2730]: I0305 14:55:03.448591    2730 util.go:37] Trying to create cadvisor client.

Mar 05 14:55:03 kubernetes-minion-1 kubelet[2730]: I0305 14:55:03.441032    2730 docker.go:794] Connecting to docker on unix:///var/run/docker.sock
Mar 05 14:55:03 kubernetes-minion-1 kubelet[2730]: I0305 14:55:03.440906    2730 server.go:163] Using root directory: /var/lib/kubelet
Mar 05 14:55:03 kubernetes-minion-1 systemd[1]: Started Kubernetes Kubelet Server.

The push basically caused the Kubelet to restart, and then it does work.
Without the push, systemd would have eventually restarted the Kubelet.
We need more investigation as to what was causing the Kubelet to fail on initial startup.


Guardian.Phoenix.Socket is not available

The ONLY place that it's mentioned is in the CHANGELOG.md
Seeing as MOST elixir developers are looking at the current Hex Docs, not scouring the github, it took me an absurdly long amount of time to figure out that Guardian.Phoenix.Socket was moved to a completely separate package.
The Hex Docs severely need to be updated to the latest version
Not even Issue #216 mentions that it was ACTUALLY DONE, the wording of the thread makes it look like that this separation was not going to happen

This just saved me a lot of time. I've been trying to get it working with phoenix live views, but of course it wants a Conn, not a Socket.
Didn't even know there was a separate package for guardian phoenix. See:
https://hex.pm/packages/guardian_phoenix
https://github.com/ueberauth/guardian_phoenix

@SampsonCrowley / @aaron-price pull requests to improve the documentation are always welcome.

And if I find time to do so, I absolutely will. But as I'm not a maintainer of this project, there are other projects that will get that time first. Regardless of who finds the time to do the work, the issue still stands that this is a glaring gap in documentation, that should have been updated with the change being made

Thanks for the report @SampsonCrowley

Eh, it's all free, you can't have high expectations. You guys still do great work!
If I ever figure it out, I'll make a PR, but more likely I'll just give up and do my own auth from scratch.

Thanks @aaron-price! It’s unfortunate when people see open source (and maintainers) as something they’re entitled too without compensation or contributions. Thankfully they’re few and far between!
Those of us who’ve contributed and work to maintain these packages want them to be perfect but alas we have our own obligations (both in work and personal lives) and we all have different backgrounds and levels of experience so what some may see as “glaring” deficiencies are rather obvious to others.
Constructive feedback and contributions are always welcome 😁


Update Hyperledger logo

Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

Fixes https://github.com/coreinfrastructure/best-practices-badge/pull/1749

The method length warning is no big deal, but the other warning suggests I need to re-look at this. I don't think the width/height settings are actually working, so a short debug session will be needed.

As noted in my commit, I did it the hard way & then discovered I didn't need to. The system can just use SVG files the same way it uses PNG files - just give the name in a normal "img" (image) tag. I made a mistake in my earlier tests that made me think it didn't work, then followed complicated directions that turn out to solve a more complicated problem... which worked but we're trying to minimize complexity here!!
This one seems to work well  though, nice clean lines. I still put the new Hyperledger logo at the end; it's really wide, and thus doesn't cuddle well with other logos.


Example launch file with overlying params.

FYI @remod

Thanks, looks good! Do you think we should comment on this somehere in the wiki?

Yes, good point!
I'm happy to add a small section about overlying parameters here: https://github.com/ethz-asl/ros_best_practices/wiki#parameter-organisation

👍


$.javascript examples

Can you add some docs / examples on the $.javascript helper?
The other helpers are fairly straightforward but I'm not sure what can or can't be used with $.javascript.
Can I do {{$.javascript Session.get('someKey').length}}?
Can I do {{$.javascript console.log('someLog')}}?
Can I do {{$.javascript Session.set('someKey', 'someValue')}}?

That part of the documentation may be cryptic at first. What it means is, there is a new $ helper, whose scope can be extended with any javascript function or namespace object you can directly access.
Meteor and Session are already in that scope so that not only you could do {{$.Session.get 'someSessionKey'}} or {{$.Meteor.isCordova}} but you could also
testCollection = new Meteor.Collection('test');
testCollection.insert({name: 'test 1', createdAt: new Date()})
Helpers.addScope('testCollection', testCollection);

{{#each $.testCollection.find}}
  	{{name}} - timeStamp: {{createdAt}}<br/>
{{/each}}

@merlinpatt I'm guessing your question has been answered. Please make not if you need further assistance.


Diagram is hard to see with dark theme

It seems that the diagram has a transparent background. This makes the black text and lines almost invisible when the dark theme is used. I suggest that you use a white background or even better keep the transparent background but change the text and lines to be visible when the theme is dark. This avoids creating a big mostly white area on the otherwise dark page.

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

ID: 0f2b93a5-28b8-db4a-ab64-6342d3aafc63
Version Independent ID: 245586f2-63e1-1e78-a5dd-21e8fa2cc5d3
Content: Provision throughput for Azure Cosmos DB
Content Source: articles/cosmos-db/set-throughput.md
Service: cosmos-db
GitHub Login: @aliuy
Microsoft Alias: andrl

@Liversage Thank you very much for the detailed feedback. I have assigned this to the content author to evaluate and update as appropriate.

@SnehaGunda Bringing this to your attention.

@Liversage thanks for pointing out the issue, we are in the process of redesigning the images. We will get to this soon.

@Liversage We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Missing callout numbers and admonition images if PDF is opened with Adobe acrobat reader

Dear Asciidoctor-pdf team,
I am having the following problem:
PDFs are not displayed correctly when opened in Adobe acrobat reader and Adobe acrobat reader DC on windows 10.

Admonition images are not visible (like for note, warning ...) in the PDF.
Callout numbers (in code segments) are not visible in the PDF.

The following error message appears:

Cannot extract the embedded font '635a1d+SegoeUISymbol'. Some characters may not display or print correctly.

I have tried different fonts and added fallback fonts to resolve this:

M+ 1mn
M+ 1p
SegoeUISymbol

(the named font in the message was different in those cases.)
In all other programs to view PDF-files everything worked fine! I have tested Firefox, Edge, pdf24Reader.
(Additional note 1: This was not an issue in the past. I can still open older generated PDFs without this problem.)
(Additional note 2: The Adobe reader had the same problems with notoserif-italic-subset.ttf.)
Thank you in advance!
Joerg

Check whether you are using ttfunk 1.8 instead of 1.7. If you installed the Asciidoctor PDF gem from rubygems.org, you should have ttfunk 1.7 since Asciidoctor PDF locks to that version (see https://github.com/asciidoctor/asciidoctor-pdf/blob/v2.3.x/asciidoctor-pdf.gemspec#L38). ttfunk 1.8 has known issues (see https://github.com/prawnpdf/ttfunk/issues/102) and should not be used.
In the future, please ask questions about usage in the project chat as the issue template requested. Issues should only be filed one a problem is verified. This helps make the project more sustainable. You can find the policies here: https://docs.asciidoctor.org/about/get-help/

Thank you a lot. This solved the problem instantly! :)


Support authenticating to HTTPS repositories via client certificate

Currently Gradle does not support 2 way SSL for authenticating to https:// maven repositories (https://docs.gradle.org/4.10.2/userguide/repository_types.html#sub:authentication_schemes). Support should be added for mutual SSL auth via client certificates.

Any updates on this?

Hello, we are still interested in using this and would like to keep this open. Are there any updates?

We also would love to see this feature implemented. Any update on this?

Just for reference (and i found it while searching a solution for it), it was already reported also in the old Jira https://issues.gradle.org/browse/GRADLE-3473

Sorry for the late reply.
This feature request is in the backlog of the relevant team and is prioritized by them.

Implementing this is currently blocked by ongoing work on credentials:

https://github.com/gradle/gradle/issues/30823


(#231) Change test task association with role through name

https://github.com/howtohireme/give-me-poc/issues/231
Description
Change test task association with role through name
Testing steps
Explain how to test your PR manually

Update Developer::TestTask.first assotiation to role :developer
check Developer::TestTask.first.role -> "developer"
Delete role :developer
Check Developer::TestTask.first.role -> nil
create role :developer -> User.first.add_role :developer
Check Developer::TestTask.first.role -> "developer"

Features PR url
https://github.com/howtohireme/give-me-poc-features/pull/11
Checklist
Make sure that all steps a checked before merge

[x] RSpec tests are passing on CI
[x] Cucumber features are passing localy
[x] bin/cop -a does not return any warnings
[x] Tested manually

Screenshots

пофиксил фичи https://github.com/howtohireme/give-me-poc-features/pull/11


Why don't we have a develop branch?

Here we use a simplified "git-flow" approach: master is essentially a "develop" branch, and releases are just tagged commits from master (where in git-flow you have a dedicated branch for releases).
I'm sorry for my responsiveness on latest developments in a couple of months, I'll become more active soon (likely in about a week from today).

That’s not git-flow at all, it’s just the standard model. :-)

I believe you got the answer. Feel free to ask more!


Fixed some spelling/grammar

@jackmoody11 You can rebase from master and then resubmit this PR if you want to :smile:


Make AbstractTracer a subtype of Real, not Number

Increases compatibility with existing code

Would solve https://github.com/control-toolbox/CTDirect.jl/issues/88#issuecomment-2120736903

@gdalle thank you for feedback. well, if this indeed solves the issue, this is most welcome. given the current type hierarchy, this seems to be no problem provided that no complex numbers are involved 👀.

Sounds reasonable. We won't touch differentiation of complex numbers anyway, so this would only have been relevant for connectivity_pattern on complex functions, which isn't our priority.
I'll tag a last patch release and add this breaking change.

Some weirdness resulting from this: our local tracer type Dual can contain a Complex primal value but is Real.
Maybe the most sane fix is to constrain the primal to Real as well.

I agree, and we'll be forever banning complex autodiff (but so did ForwardDiff)

Based on the way tracers are constructed, if the function signature is f(::Vector{<:Real}), the primal values will never be anything but Real anyway

Reopening until #95 is merged

I agree, and we'll be forever banning complex autodiff (but so did ForwardDiff)

Indeed, replacing $\varepsilon^2=-1$ by $\varepsilon^2=0$ is all about ForwardDiff.jl 🤭


Add legacy dat:// rendering/browsing

Currently dat:// URLs just render the converter interface. We should bring in readonly page-rendering and move the converter to another part of the UI.

🙏 👍🏻

Okay we've got browsing and viewing, now I need to hit the converter and we'll be good to go

We're in business!


Make microSD card benchmark script compatible with non-Raspbian/non-Pi distros

Currently, there are a few hardcoded assumptions in the microSD card benchmark script, like the existence of the /home/pi directory. I'm doing some testing on an ODROID-C2, and the script tries to cd into that nonexistent directory (the user's home is /home/odroid), and the experience isn't ideal.
It should be abstracted slightly so the script works on any Debian/debian-like distro on a single-board computer (e.g. ODROID-C2, Raspberry Pi, Orange Pi, etc.).

Also, make isn't always guaranteed to be present, so the script should account for that and install apt-get install -y build-essential if necessary.


Add maximum iframe width feature

Image width can be limited, but I ran into an issue where iframe was rendered too wide. Here is a patch to add iframeMaxWidth in the same style as imageMaxWidth.

Hi, thanks for the PR.
I've already added a similar feature in the development branch named staticContentMaxWidth. For now, it's only used with <iframe>, but I can consider using it for future renderers.
If you have suggestions regarding this feature, please comment on #101.


ESLint and Prettier introduced

Closes #17.
ESLint and Prettier have been added to vsf-capybara. ESLint is configured to use eslint, vue and prettier recommended rules. Prettier is configured with own defaults (out of the box) but empty .prettierrc has been included in this repo to provide place for custom rules, if needed in the future.
I fixed all errors and warnings in capybara aka default theme basing on current ESLint and Prettier configuration:

most of them have been automatically fixed by calling eslint --ext .js,.vue ./ --fix:

attributes order and placement
component properties order
single quotes replaced by double quotes
code formatting

100+ of them have to be investigated and fixed manually:

unused variables have been removed
all v-html have been replaced with v-text
all v-for used together with v-if have been replaced with equivalent computed properties
one issue has been temporarily "fixed" in pages/Category.vue by disabling require-atomic-updates rule due to pending fix in ESLint https://github.com/eslint/eslint/issues/11899
fixed dependencies in package.json for current capybara theme to match its real dependencies used in source code - it should be updated as soon as storefront-ui components are integrated and legacy code becomes unused

The lint script has been added to package.json and dev script now points to main repo (your vue-storefront) since there is no sense to just build theme without main vue-storefront project.
Now vsf-capybara is error and warning free and it just waits for actual start of development   😎
I have a lot of problems and conflicts with integration ESLint 6.x and it seems that it is quite common: https://github.com/microsoft/vscode-eslint/issues/696 (it is in vscode repo but it looks like this is quite "global" issue). Finally, I had to downgrade to the latest 5.x version which works perfectly.

ESLint works:

Prettier works:

Respect for description and screenshots :)


Pivoting nested array

Been using react-table for a while now, but im having trouble displaying an array of object and each object there's a nested array of objects.
ex. data structure:
const arrOfObj = [
  {
    name: 'Janey Doe',
    points: 10,
    nestedArr: [
    	{
      	month: 'January',
        points: 2
      },
      {
      	month: 'February',
        points: 5
      },
      {
      	month: 'March',
        points: 3
      }
    ]
  },
  {
    name: 'Johny Cash',
    points: 12,
    nestedArr: [
    	{
      	month: 'January',
        points: 5
      },
      {
      	month: 'February',
        points: 5
      },
      {
      	month: 'March',
        points: 2
      }
    ]
  },
]

Expected output
Name                   Month                      Points
Janey Doe                                           10
                       January                       2
                       February                      5
                       March                         3
--- ETC

I can toggle 'Janey Doe' or the name value to show the months.
How can i do or display this with pivot without using aggregation? since the data is already aggregated in the back end.

any updates on the same issue ?


Shoule be an exception when debugtoolbar can't insert itself

The debugtoolbar uses the </body> tag as the marker when inserting itself. In html5, both the opening and closing body tags can be implicit, thus valid documents may omit them.
Patch raises a RuntimeError if there is no  tag in the html.

Thanks. I've pushed this after making a small fix so it still matches </body> with a case-insensitive search.


Password Recovery Feature #201

Description:
This pull request introduces the Password Recovery feature, allowing users to reset their passwords through a simple and user-friendly flow.
Related Issue
Fixes #201
Changes Made
Created the Password Recovery component with React.js.
Added a three-step flow:
Input email to simulate OTP sending.
Verify OTP.
Reset password.
Used alerts to simulate sending/verification of OTP and password reset.
Provided a basic structure for future backend integration.
Checklist
I have tested my code and it works as expected.
Documentation has been updated where necessary.
Tests for the new feature are included.
The code follows project standards.
Screenshots (if applicable)
(Include any relevant screenshots that illustrate your changes.)
https://github.com/user-attachments/assets/1c3ec137-d327-48d3-8d25-a5ff9835e324
Example:
Title: Password Recovery Feature
Description: Adds a password recovery feature with a simulated OTP flow.
Related Issue: Fixes #201
Changes Made:
Created components for password recovery.
Implemented a flow for email input, OTP verification, and password reset.
Added alerts to simulate OTP-related actions and password reset.

Hey @saras-69,
Can you add the feature to change the password?

Don't create a PR to master branch. Create a new PR to develop branch, with suggested changes.

Thanks.
Closing this PR as did not meet the PR requirements.


add gflags to switch octal escape char support

Fixes https://github.com/vesoft-inc/nebula-graph/issues/1033

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


Plaid link iframe is not removed

Hi there!
I've noticed that iframe that is created by usePlaidLink hook is not removed on successful scenario, so when I start plaid linking process multiple times  - a new iframe is created each time. The only case when iframe is not recreated is when I close plaid link window and onError callback is triggered.
I've noticed that this behaviour happens because usePlaidLink effect fires an exit method on unmout, but destroy callback is not triggered:

React component to reproduce:
import React, { useEffect } from 'react';
import { PlaidLinkOptions, usePlaidLink } from 'react-plaid-link';

interface PlaidLinkProps {
  options: PlaidLinkOptions;
}

export const PlaidLink: React.FC<PlaidLinkProps> = ({ options }) => {
  const { open, ready } = usePlaidLink(options);

  useEffect(() => {
    if (ready) {
      open();
    }
  }, [ready]);

  useEffect(() => {
    return () => {
      console.log('unmount');
    };
  }, []);

  return (<></>);
};

I think this issue is related to opened PR https://github.com/plaid/react-plaid-link/pull/226


PartitionKey is required for input binding

Using the single-doc sample shown here will result in an HTTP 500 saying PartitionKey is required.
I would advise re-assessment of all usages of the Cosmos binding shown in this doc.

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

ID: fa5bcb79-2af8-3b0e-c78c-0c89a2535e15
Version Independent ID: 31b87687-7841-f46f-8c97-38898e7b97db
Content: Azure Cosmos DB bindings for Functions 2.x
Content Source: articles/azure-functions/functions-bindings-cosmosdb-v2.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

cc @sarah-aly

@brandonh-msft Thank you for your feedback! We have assigned this issue to the author to review further and take the right course of action.

+1. Agreed with @brandonh-msft .

It appears that the input binding examples have moved to https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-cosmosdb-v2-input?tabs=csharp.  The examples include the partition key, and a note indicating that if the collection is partioned, then a partition key is required.
Is there more to do on this feedback?  @mike-urnun-msft @craigshoemaker

So actually I've just tried it just yesterday. I have a CosmosDb collection partitioned on /nameOfField, and no-matter what I do in an output binding:

specify partition key with "/nameOfField"
specify partition key with "/nameNonExistentField"
specify partition key with "a_value_of_the_partition_key_field"
leave partition key out

It always inserts the new record.
Interestingly, when inserting with LogicApps, there has to be an header with the VALUE of the partition key (not the path), and it has to be the same as the one on the request payload. As to functions, I have no idea how this works.
Also the documentation page says: "PartitionKey | When CreateIfNotExists is true, it defines the partition key path for the created collection." but this could be made clearer. I'm seeing several examples on the web and even in a book where the pk value is passed here instead of the "key path". For example:
https://books.google.de/books?id=KB2IDwAAQBAJ&pg=PA43&lpg=PA43&dq=CosmosDB+output+binding&source=bl&ots=AXr_QtltBw&sig=ACfU3U1FOIgp_oryPNkwqG03lK9kdjHzpA&hl=en&sa=X&ved=2ahUKEwjIwoegk6rpAhUQQ0EAHYLYCkgQ6AEwF3oECBwQAQ#v=onepage&q=CosmosDB output binding&f=false
and
https://dontcodetired.com/blog/post/Writing-Azure-Cosmos-DB-Data-from-Azure-Functions

#reassign: @cachai2 followup with CosmosDB