Attribute title is not correct in ReportPage

Use template: /projectTemplates/GoodSalesDemo/2
Create a report contains:
WHAT: Amount
HOW: Sales Rep
final Metric amountMetric = getMdService().getObj(getProject(), Metric.class, title(METRIC_AMOUNT));
final Attribute saleRep = getMdService().getObj(getProject(), Attribute.class, title(SALES_REP));

ReportDefinition definition = GridReportDefinitionContent.create(SIMPLE_REPORT,
    singletonList(METRIC_GROUP),
    singletonList(new AttributeInGrid(saleRep.getDefaultDisplayForm())),
    singletonList(new MetricElement(amountMetric)));
definition = getMdService().createObj(getProject(), definition);
getMdService().createObj(getProject(), new Report(definition.getTitle(), definition));

Open that report in UI, the title of Sales Rep will be replaced by Owner Name, which is the title of default display form.

Well this is expected -  as described in javadoc of new AttributeInGrid(DisplayForm):

Creates new AttributeInGrid using given DisplayForm's uri and it's title as alias.

Use new AttributeInGrid(Attribute attr) is the same. Is it expected as well?

Please read the javadoc and answer.
Additional info:
Previously there was only possibility to create AttributeInGrid from URI, which resulted in alias set to empty String (this deprecated constructor is broken in 2.0.0-RC.1 but already fixed in 2.0.0-RC branch). My guess is that UI behaves the way, it replaces empty String with Attribute's title.
The question is whether the new AttributeInGrid(Attribute) shouldn't use attribute's title rather than default display form - @martiner what do you think?.

@vhtien would you (as SDK user) expect that constructor taking Attribute, have alias set from Attribute's title? Is it more logical?
Currently the alias is taken from defaultDisplayForm because the AttributeInGrid's uri must point to displayForm - so we took uri and alias from the same object.

well, as SDK user, I need the SDK provide exactly what I've done in UI. And I believe that create a report with attribute have the alias set from that attribute.

Current way to behave as the UI (my guess): new AttributeInGrid(attr.getDefaultDisplayForm().getUri(), "") or better  new AttributeInGrid(attr.getDefaultDisplayForm().getUri(), attr.getTitle()).
More constructors, as well  as change of the new AttributeInGrid(Attribute) to use attribute's title will be in next RC version.


Add Dockerfile and GitHub Workflow

I know there is a Dockerfile under the docker directory, but it is for development.
This Dockerfile is intended more for production. It builds an image with both the frontend and conda environemnent using a multi-stages build to keep the final image size small.
I also added a GitHub workflow to build and push the image:

The image mambaorg/quetz:latest is built and pushed for every commit to the master branch.
When pushing a tag, the image is tagged with it (mambaorg/quetz:<tag>).
For PR, the image is only built, but not pushed.

For the workflow to work, the DOCKERHUB_USERNAME and DOCKERHUB_TOKEN secrets shall be defined for the repository.

looks great!
btw. the commits use a different email address then your GitHub profile which means they might not get associated to your account. I think you can fix that if you want. Let me know then I'll hold off with merging this one.

Thanks for noticing! I changed the email in those commits.
Will add my new work email to my profile as well.

I forgot to mention you should have added DOCKERHUB_USERNAME and DOCKERHUB_TOKEN secrets in the project settings before merging.

yep, adding them right now, thanks!


check new version without renv or download_data

@cassandrayuen I have made some modifications to remove renv as discussed. can you test?

Ran fine on my local and CoESRA, though neither needed to download packages.

Also fine on PC254, did not need to download packages.


Autofix: The "TimeoutTest::testInvalidTimeoutSettingThrowsException" test fails on Selenium 3.x

I updated the setTimeouts and applyTimeouts methods in the Selenium2Driver class to handle exceptions from both Selenium 2.x and 3.x. This should resolve the issue with the testInvalidTimeoutSettingThrowsException test failing on Selenium 3.x.

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!

This bug was fixed for free by Latta AI - https://latta.ai/ourmission

Hehe, it replaced a bunch of code with "previous code remains unchanged"...

Apparently that @lattaai1 isn't as clever as Skynet. Closing.


Is September 2020 content update available in more languages than English?

Hi,
I checked https://docs.microsoft.com/de-de/office365/customlearning/custom_contentupdates yesterday and noticed a content update is available. So i checked on our site and couldn't find any new content in German language. I checked the administration page and for example the new scenario "Security" was available. Next I switched my language to English and now I can see the new scenario "Security" and all the new content in "Planner", etc.
The question now, is the new content available only in English?
We are running Version 4.1.1 and I attach screenshots in English and German.
Thanks, Manfred

Yes, that sub-category does appear to be missing from the manifests, I'll reach out to that team and see what the intent was.

Same in french.
Merci

It looks like (saw a PR) the manifests were just updated for the translations.  See if it's fixed now.

I just tried to refresh and I do not see the new content.

By just tried a refresh I assume you mean went to the CustomLearningAdmin.aspx page, then after that loaded went to CustomLearningViewer.aspx...

Nevermind, i just looked a bit more and see that it hasn't been merged yet... so clearly the fix isn't complete.

Ok, looks like the update was merged in, try it now.

Allmost there. The category is now displayed but the images are only available in English language setting.

Hi,
is fixed, images are now also displayed.
Thank you

Also fixed in French. Thanks!


Add scope test machinery for clojure and go

Checklist

[x] I have added tests
[/] I have updated the docs and cheatsheet
[/] I have not broken the cheatsheet

done


--show desugared-egglog --> --desugar

Pressing F8 results in an error:
Executing task: egglog --show desugared-egglog ~/egglog-playground/tests/herbie-tutorial.egg

error: unexpected argument '--show' found

It seems that in the fresh egglog version the option --show desugared-egglog has been replaced with --desugar.

Hello, I tried to solve the issue.
This is what I did:
Changed the '--show desugared-egglog' option to '--desugar' in the egglog_desugar command to fix the error when pressing F8.
You can review changes in this commit: https://github.com/lattaai12/hatoo-egglog-language-server-11/commit/47bb06e40baed2d7b9af46011e6b09d6a6340c4c.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.

Hmm, it's odd, cargo run -- --show desugared-egglog /home/hatoo/egglog/tests/array.egg works fine on the latest commit ca52ac13cb3c0bbacc8e7cc540789521d1019bd2

Oh! I apologize! The problem is with egglog's installation instruction. It says:
make all

But, actually, it should say
make all
cargo install --path=.

Thus, I was trying to run an old version of egglog. With the fresh version, F8 produces the expected result.


Is it production ready?

When I run cargo run, I get Segmentation Fault: 11.
I started removing some code to understand what's causing it and it all came down to these lines:
use etcd_rs::prelude::*;
use etcd_rs::Client;

@thehappycoder I cannot reproduce the error if you don't paste the code you run. 😂 Is there any problem when you run the example code?


[formrecognizer] Doc updates

Fix docs on begin_build_model() and run black on the code.

API change check
API changes are not detected in this pull request.


fix css arrows to be pointing in the correct direction

Thanks for the great library!
It appears that the css for the aria-sort is creating the arrows in the opposite direction:

The name column is correctly assorted ascending, but the arrow is pointing down instead of up.
This PR changes the CSS so the arrows point in the correct direction.

Disclaimer, I'm not associated with this project in any real way, nor maintaining it, or anything. I'm just reading the PRs because I made one recently too.
It's an interesting matter of perspective. I get where you're coming from, having the arrow pointing up means "ascending" to you, but I've always seen it as indicating the reading order, having the arrow pointing up means A→Z going up, i.e. "descending". It reminds me of the "pagedown" problem, where some people expect the text to go down and some people expect the screen to go down!
In the end, it seems a lot of people agree with you on stackexchange.
LGTM ✔

This seems debatable. I don't have a problem with the current icons. But I wouldn't oppose a merge if authoritative usability sources support your interpretation either.


MFA support: Token request failed: Unauthorized

With Aurora+'s recent push to use MFA, this lib is no longer able to authenticate.
>>> import AuroraPlus
>>> api = AuroraPlus.api(username, password)
>>> api.gettoken(username, password)
>>> api.Error
'Token request failed: Unauthorized'

The most obvious impact is the HA integration LeighCurran/AuroraPlusHA#10

Some quick inspection of the flow from the browser. It looks like it's an OAuth bearer flow. I'm not fully familiar with it, but that looks pretty standard.

The login pages https://customers.auroraenergy.com.au/auroracustomers1p.onmicrosoft.com/b2c_1a_sign_in/oauth2/v2.0/authorize?client_id=<CLIENT_UUID>&scope=openid%20profile%20offline_access&redirect_uri=https%3A%2F%2Fmy.auroraenergy.com.au%2Flogin%2Fredirect&client-request-id=<REQUEST_ID>&response_mode=fragment&response_type=code&x-client-SKU=msal.js.browser&x-client-VER=2.38.2&client_info=1&code_challenge=... sets cookies ASLBSA, ASLBSACORS (same hex value), and a few  x-ms-* (base64 encoded)
Login and password are POSTed to https://customers.auroraenergy.com.au/auroracustomers1p.onmicrosoft.com/B2C_1A_SIGN_IN/SelfAsserted?tx=StateProperties=<BASE64_JSON>&p=B2C_1A_SIGN_IN

The <BASE64_JSON> contains a single TID attribute, which looks like a UUID v4
The application/x-www-form request has an X-CSRF-TOKEN as and contain three fields: request_type: "RESPONSE", signinName and password.
The JSON response contains a status attribute, set to 200 on success, and sets 4 cookies

MFA data is POSTed to https://customers.auroraenergy.com.au/auroracustomers1p.onmicrosoft.com/B2C_1A_SIGN_IN/SelfAsserted?tx=StateProperties=<BASE64_JSON>&p=B2C_1A_SIGN_IN

The <BASE64_JSON> is similar (maybe same?) as before
The application/x-www-form request has an X-CSRF-TOKEN as and contain three fields: request_type: "RESPONSE", otpCode.
The JSON response is similar to before

It hits https://customers.auroraenergy.com.au/auroracustomers1p.onmicrosoft.com/B2C_1A_SIGN_IN/api/SelfAsserted/confirmed?csrf_token=<ANOTHER_CSRF_TOKEN>&tx=StateProperties=&p=B2C_1A_SIGN_IN&diags=%7B%22pageViewId[...]URL%3Ahttps%3A%2F%2Fcustomers.auroraenergy.com.au%2Fmy-cdn%2Fcomponents%2Fapi.selfasserted.totp%2Findex.html[...]

This returns a 302 redirection to the redirection URL given in step 1, with a `#state=<BASE64_JSON>&client_info=<BASE64_CLIENT_INFO_JSON>&code=
The JWT seems to be used as a refresh_token

It hits https://customers.auroraenergy.com.au/auroracustomers1p.onmicrosoft.com/b2c_1a_sign_in/oauth2/v2.0/token

Request: { "client_id": "<CLIENT_UUID>", "redirect_uri": "https://my.auroraenergy.com.au/login/redirect", "scope": "openid profile offline_access", "code": "<JWT>", "x-client-SKU": "msal.js.browser", "x-client-VER": "2.38.2", "x-ms-lib-capability": "retry-after, h429", "x-client-current-telemetry": "5|865,0,,,|@azure/msal-react,1.5.11", "x-client-last-telemetry": "5|0|||0,0", "code_verifier": "<SOME_STRING>", "grant_type": "authorization_code", "client_info": "1", "client-request-id": "<UUID>", "claims": "{\"access_token\":{\"xms_cc\":{\"values\":[\"CP1\"]}}}", "X-AnchorMailbox": "Oid:<UUID>-b2c_1a_sign_in@<UUID>" }
Response: ```{
"id_token": "<OAUTH_TOKEN>",
"token_type": "Bearer",
"not_before":<PHONE_NUMBER>,
"client_info": "<BASE64_CLIENT_INFO_JSON>",
"scope": "",
"refresh_token": "",
"refresh_token_expires_in": 86400
}

5. Hit https://api.auroraenergy.com.au/api/identity/LoginToken with a JSON payload with the `<OAUTH_TOKEN>` as `token` and receive a JSON payload with `"tokenType": "bearer"` and an `"accessToken": "bearer <ACCESS_TOKEN>"`
7. The `accessToken` can be put in an `authorization` HTTP header on subsequent requests to, e.g., https://api.auroraenergy.com.au/api/customers/current or https://api.auroraenergy.com.au/api/usage/day?serviceAgreementID=<ID>&customerId=<ID>&index=-1.

* Step 3 is a way to get a (hopefully long-lived) refresh token
* Step 4 (and 5) seems to be usable for renew a token after expiration.
* Step 7 is what this lib needs.

I've started trying to play around with https://gist.github.com/shtrom/e13b667f8181b05a53761e34a473525e
Findings so far:

Looks like pretty standard OAuth
Can't change the client_id, so we need no pretend we're the website (or, maybe, the Android app)
I can't find the client_secret, which is needed to get a token. This makes sense, as it's a secret, but I suspect it's got to be somewhere so the browser-side code can create the needed code_verifier

Ok, the client_secret has nothing to do with it. The approach is to create a code_verifier, hash and base64urlsafe encode it as the code_challenge https://www.valentinog.com/blog/challenge/
code_verifier = ''.join([random.choice(
    string.ascii_letters
    + string.digits
    + '-_')
    for i in range(43)])
code_challenge = base64.urlsafe_b64encode(
    hashlib.sha256(code_verifier.encode()).digest()
).strip(b'=')

This now works, and I have updated the Gist, but we're facing the next problem of a oauthlib.oauth2.rfc6749.errors.MissingTokenError: (missing_token) Missing access token parameter.
Stay tuned.

The response from /token looks good, but we then need to grab the id_token, and pass it to https://api.auroraenergy.com.au/api/identity/LoginToken as
{ "token": <id_token> }

to get the bearer token to use.

More progress: I'm now getting an access_token... but it's then getting rejected... Gist updated.

Ok, I got it!
I have a function Web application flow that allows a Python script to get a token from a user interaction with MFA to authorise it.
I haven't try refreshing it yet, but the gist is updated and should be functional. This still needs to be properly integrated into the lib, but this is looking good.

It also looks like the Login token doesn't expire (at least not within 24h, so far)


Widget Rule definitions should allow day, day of month, month selection for all frequencies

tl;dr
The dateutil docs give examples like:
list(rrule(YEARLY, interval=4, count=3, bymonth=11,
               byweekday=TU, bymonthday=(2,3,4,5,6,7,8),
               dtstart=parse("19961105T090000")))
[datetime.datetime(1996, 11, 5, 9, 0),
 datetime.datetime(2000, 11, 7, 9, 0),
 datetime.datetime(2004, 11, 2, 9, 0)]

but we can't specify months AND days of week in the same rule in the django-recurrence widget
Detail
I need to specify a range for certain days of the week, but only for particular months of the year (for seasonal activities). I can achieve an example of this in python with:
list(rrule(YEARLY, bymonth=(1,2,3,7,9), byweekday=(SA,SU), count=10))

(list and count included for readability of output)
As awesome as django-recurrence is, each frequency tab (eg Annually, Weekly) only supports options relative to that type, so Annually doesn't let me specify for which weekdays, for example. I can add another rule, but that is a logical OR which means that dates are included if they match the first rule or the second rule.
One way around this would be for me to use an Exclude, but this seems to only be a single day in the month (when I choose Annually and my months). Let me know if I'm missing something here.
Another way would be an enhancement to the widget. It seems from the dateutil docs that the Frequency selected is only relevant for interval and bysetpos. This means that any Frequency can have day of week, week of year, etc. So the widget would need reworking so that regardless of Annually / Monthly / Weekly / Daily selected, the month, day of month, and day of week selectors would always be visible.
Discussion welcome

Hey @humbled - thanks for the bug report.
I think this boils down to the same basic problem as reported in #37 - that there are various rules expressible in dateutil which aren't expressible using the django-recurrence widget. I'm honestly not really sure how to handle this - without a fairly significant JS change (and possibly rewrite, per #4). There are a couple of other issues that require JS changes that we've also punted on (#14 and #39).
I'm open to patches that address any of these - bonus points if anyone can set up some basic JS tests for django-recurrence too (the majority of the code for django-recurrence is JS, but all our unit tests are for the Python).
Dom


Random vs static

Hi @gojko , surprisingly you are the owner of this awesome plugin 👍
I am curious what is the intent of using static values, such us<EMAIL_ADDRESS>instead of random by some template.
I found a usefull case of having random email when testing sign up form
Thanks.

the intent is that they are easily available on right click when you want them, as it is with the whole plugin. random stuff is useful as well, for other things


python312Packages.sunweg: 2.1.1 -> 3.0.1

Changelog: https://github.com/rokam/sunweg/releases/tag/3.0.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

5 packages built:

home-assistant-component-tests.sunweg
python311Packages.sunweg
python311Packages.sunweg.dist
python312Packages.sunweg
python312Packages.sunweg.dist


Configurable graph force strength and link length

The two following parameters can be now be customized through the config object:

link.size (the length of the link)
node.gravity (defines how close nodes are from each other in the graph)

Until now these two values were set as constants (FORCE_IDEAL_STRENGTH and LINK_IDEAL_DISTANCE), but the predefined values could present visual issues for bigger node sizes.
In some scenarios, the links wouldn't even appear on screen, as they would be completely hidden beneath the nodes. Also, it could happen that many of the nodes happened to overlap, making it impossible to view all of them without dragging them to isolated locations.
Having these properties as configurable values will make it easier for the developers to customize the distance between nodes depending on their size, and so ensure the graph looks good regardless of their choice.

Good initiative, linking Trello card that corresponds to this PR, will review asap.
https://trello.com/c/NBdYTyV2

Thanks for the feedback, @danielcaldas !
I've just uploaded another commit with the changes you proposed. Now both FORCE_LINK_STRENGTH and SIMULATION_ALPHA_TARGET are also customizable, and all of these new settings are wrapped into a d3 object within config.
If there's something else you'd like me to change, don't hesitate to let me know. 😃


[SPARK-25299] Yet another attempt to integrate API with scheduler

This is pretty similar to https://github.com/palantir/spark/pull/548 except that we move the port/host logic behind the API. Thanks to @mccheah for the idea!
This gives us a cleaner API, allowing the plugin to define the logic for whether to remove the MapStatus or not based on the implementation and structure of ShuffleLocation and MapShuffleLocations.
We avoid putting the shouldRemoveMapOutputOnLostBlock() logic inside of MapShuffleLocations directly to keep the MapShuffleLocations as an immutable object (instead of something with mutable state, which would further complicate the plugin implementation). Furthermore, placing it in the ShuffleLocationsComponents allows extensibility for cases where the immutable MapShuffleLocations object might not have all the information necessary for whether to remove a MapStatus or not. An example would be in the individual file server case. For example, if you had the following in your MapOutputTracker:

Map task 1
** Block 1: [hostA, hostB]
** Block 2: [hostA, hostB]
Map task 2
** Block 1: [hostA, hostC]
** Block 2: [hostA, hostC]

If we get a FetchFailed([hostA, hostB]), then shouldRemoveMapOutputOnLostBlock() wouldn't remove the MapStatus associated with map task 2 because both blocks can be retrieved from hostC. Suppose that the scheduler gets a FetchFailure([hostC, hostD]) later. It also doesn't mark map task 2's MapStatus as invalid because the MapShuffleLocations object itself doesn't know that hostA has already been invalidated. Therefore, the MapShuffleLocations object itself doesn't have enough state to know whether a certain location should result in deleting that MapStatus entry.
By moving the logic inside of a separate entity like ShuffleLocationComponents, we make it possible for that object to maintain bookkeeping about which hosts have been blacklisted, allowing it to do additional filters for whether the MapStatus should be removed.
Furthermore, we were hesitant about placing ShuffleLocationComponents inside of ShuffleDriverComponents because of the order of initializations within spark. Currently, the SparkContext first initializes the SparkEnv and then calls ShuffleDriverComponents#initializeApplication(). This makes sense because in some implementations, such as the K8s individual file server implementation, it might need to call to retrieve modules from SparkEnv (https://github.com/mccheah/spark/pull/6/files#diff-1a614b54924c44438c4301263685e461R42). However, the MapOutputTracker is initialized inside the SparkEnv, and we'd ideally want to pass the map status removal logic to the MapOutputTracker. If we placed the map status removal logic inside of ShuffleDriverComponents, we would either need to mandate that SparkEnv is never used inside of ShuffleDriverComponents and initialize it before SparkEnv, or keep it initialized afterwards and pass it to the MapOutputTracker afterwards through a MapOutputTracker.initialize() method.

Think @ifilonenko will want to look at this too


Shaitan Ka Saala (From "Housefull 4") by Sohail Sen unsupported.

Check if issue with swaglyrics or whether song lyrics unavailable on Genius.

stripper -> Sohail-Sen-Shaitan-Ka-Saala-From-Housefull-4
version -> 1.1.1

Now this issue is also solved.

We can close all the issue related to indian song.


the converted model get more larger

hi，thanks for your best contributions!
When i converted the trained model on cifar10, i find the converted model get more larger,e.g. 10723->24418, how can i converted the model right, or the fact is that.
the train arguments on cifar10 is :
--model condensenetv2 -b 64 -j 12 --data cifar10 --stages 14-14-14 --growth 8-16-32
thanks!

Did you use this function to convert your model?

yes,i did,
and i tried to converted the model by convert_and_eval.py on cifar10, the result is same.
and then i tried to replace the function of CondensingSFR in layers.py to CondensingConv in projects Condensenet，the models got smaller，but it got wrongs when test.
what i think the question of the converted models get larger is in  the function of CondensingSFR, would you please have a see.
thanks

I test the model configuration which you provide.
The original model's FLOPs/Parameters are 203.17M/1.49M. After converting, the model's FLOPs/Param will be 40.60M/1.22M which are less than original model.
Could you give model details about your experiments? For example, what is the meaning of the number 10723->24418?

To be honest, we do not test what will happen if we convert the pytorch model to onnx model. Our experiments are all conducted on GPU device.  You can send your code to my email(jhj20@mails.tsinghua.edu.cn), I will check your code later(project ddl is coming, sorry) see what I can do to help you.

OK, thanks
i will send my projects to you, and  the converted model on GPU device is also get larger, so i will hope you help me solve the problem,


Empty tables in PDF viewer when using GL renderer

Describe the bug
When using GL renderer, tables in the PDF viewer appear empty.
To Reproduce
Steps to reproduce the behavior:
Must be using GL renderer --use-angle=gl

Go to https://fccid.io/2BDK7-OP75/RF-Exposure-Info/18-OP75-RFExp-r1-7382035
Scroll down to PDF Viewer
Observe table appears empty

Expected behavior
The table should contain text - this can be seen by manually downloading the PDF.
Screenshots
(not attached)
Desktop (please complete the following information):

OS: Windows 8.1
Browser: 124.0.6367.245 (Official Build) (64-bit)

Additional context
N/A

Win: 7 SP1 x64
Supermium: 124.0.6367.245 x64
Angle: any (OGL, DX9, DX11)
Vulkan: Default
Extra: Adobe Acro XI, DirectWrite fix
No issues with on-page viewer nor chrome'

I have added a screenshot to the initial post showing how it looks for me.

eeeem - where the text right before the table? it's also missed on your screenshot. Try to click in any place in PDF viewer and then press Ctrl+A - does text become visible somehow?
probably it's site-related issue. Can you try to set #enable-force-dark to any enabled mode and check does it change anything?

Any information in --enable-logging via CMD?

Any information in --enable-logging via CMD?

I see this line, though not sure if it has any bearing
WARNING:angle_platform_impl.cc(49)] DisplayWGL.cpp:286 (initializeImpl): WGL_ARB_create_context_robustness exists but unable to create a context with robustness.

eeeem - where the text right before the table? it's also missed on your screenshot. Text that is visible for you is an image. Try to click in any place in PDF viewer and then press Ctrl+A - does text become visible somehow?
probably it's site-related issue? (then why OGL only?..) Can you try to set #enable-force-dark to any enabled mode and check does it change anything?
p.s. sry, light mode for me as for vampires, so i prefer to stay on dark-side 🤣

Text is selectable but not visible. Force dark makes no difference.

okay the issue somehow related to <canvas>
just inspected a bit this pdf viewer and find that all text is just fiction for copying purposes, and all you see are drawn on canvas and overlayed with transparent text, somehow sized to visual part. so the issue source probably can be found on inspecting js

I see. I can alter the CSS and make the overlaid text visible, so the issue is with the underlying canvas.

it's only my guesses, not more not less
small digging in pdf.js and seems that issue somehow related to fonts rather than pdf cuz this library seems to read pdf byte-by-byte, creates array of instructions to its internal renderer, then executes those instructions and draw each object by points. And because of table, image and text overlay(that you made visible on your side) is ok - there no any issues with reading pdf and text, as well as with font rendering by browser on DOM level.
But during drawing chars there some stuff with fonts for canvas so i guess issue somehow related to custom fonts loading/drawing (#728) - can you share your console output on page (re)load? exactly part after
https://fccid.io/2BDK7-OP75/RF-Exposure-Info/18-OP75-RFExp-r1-7382035.pdf               pdf.js?screx=1&sxcb=1a:1474
if you read the mentioned issue and see something similar in your console, then read the last message - it may helps you, or you can wait for the next? release where the "fix" will be embedded
anyway give feedback in this thread aswell
btw - going to check issue w/o "fix"...

There is nothing like that in the console, I don't think that issue is related.

workaround: add --disable-accelerated-2d-canvas to command line


6 warnings and 1 error

I think these can  cause a serious problems and must be resolved as soon as possible but how????

@aminsoraya Could you provide more information on your environment?
Visual Studio version: ??
.NET Core SDK version: ??  (You can run dotnet --info on a command prompt for this)

those error  and warning was related to the test libman.cspro inside main projects . my VS and SDK are updated till now. I was removed the test project and re build the all project and all things works right

Addressed, at least in part, in https://github.com/aspnet/LibraryManager/pull/563.

Closing this as it's super old and the MAT dependency was removed a few years ago.


Feature suggestion: Show SSSS highlights for embedded saving throw DCs inside features of NPCs

if you see this issue and think it's a good idea that will be useful to you, please upvote this!  This way I can judge whether this feature is desired enough to add

Closing this until further notice;  I know how to do this programmatically, but the problem is that most of these ability/save check buttons don't even get rendered when the sheet appears;  they only get added to the HTML when the user clicks them, and that's not an event I can hook into with my module.


Unable to import my franz account

Hi!
(1) I startet with opening the link "https://api.getferdi.com/import" in my browser which produced "Could not login into Franz with your supplied credentials. Please check and try again". I do have a working franz account and can login into my meetfranz account without any problems (https://meetfranz.com/login)!
(2) Then I created a new getferdi account through the app itself with the "same" cretentials as my existing franz account.
(3) when opening "https://api.getferdi.com/import" in the browser again I get  "There was an error while trying to import your account: - There is already a user with this email"
Any chance to import my settings from meetfranz and how?
Best regards!

Also getting the same issue

I am having this same issue. I tried to delete and reimport my Franz account to the Ferdi server (because of other issues I'm having with Ferdi) and my problem has exactly the same symptoms.

When Ferdi try to authenticate to the Franz API, it is now receiving a 400 Bad Request reply, with a message "Could not verify request". Seems Franz API has changed its requirements...

same

When Ferdi try to authenticate to the Franz API, it is now receiving a 400 Bad Request reply, with a message "Could not verify request". Seems Franz API has changed its requirements...
For the login request, it is now necessary to add the following header:
x-franz-source: Web

Can someone get this change into a push request? It seems fairly easy to implement.

See https://github.com/getferdi/ferdi/issues/1036 for a likely related issue.

@dgervalle Can you state a source for your suggestion?
Could you provide this change with a pull request? I guess the appropriate place would be app/Controllers/Http/UserController.

This just happened to me. No fix still after 2 months?

This just happened to me. No fix still after 2 months?

That's an unfair statement. If you have the coding skills simply add the fix suggested by @dgervalle to app/Controllers/Http/UserController, and verify whether that helps. A PR is always welcome!
If you're looking for a software that is run by a full-time development team, please head over to Franz. The 3 €/month are worth it, it pays their rent. Note that they don't provide a support contract, but you can freely shout at them on their bug tracker when you pay, I think they don't complain. Also, their releases are stable for what I can tell. Less features, less freedom, but more stability.
You choose. Thank you for your understanding!

I personally use Ferdi quite often (and like it). So in case nobody will have done it until I'll have the time in the coming days, I will then do it if it's really just that little thingy.

Seems they not only changed the required headers.
The only thing I can get out of the Franz API is this 401 response:
{   message: 'User credentials not valid',   code: 'invalid-credentials',   status: 401 }
Maybe they changed the hashing for building the token?

That's an unfair statement. If you have the coding skills simply add the fix.

I don't have the skills @bittner. You launched this app in hopes of getting users, and users need this fix. If you feel that it is not worth your time to fix this issue then either say it clearly so we don't expect a fix, or put a donate link in your website, I'd gladly donate your hourly rate to fix this issue. You choose.

I don't have the skills @bittner. You launched this app in hopes of getting users, and users need this fix. If you feel that it is not worth your time to fix this issue then either say it clearly so we don't expect a fix,

I don't want to go into arguing. I'm just explaining things.
Your profile says, "Product Designer that codes". This statement conveys you can do coding. And even if you can't, this does not give you the freedom to treat people – you don't pay – like contractors that miss a deadline. This is just not how Open Source works. Period.

or put a donate link in your website, I'd gladly donate your hourly rate to fix this issue. You choose.

Thanks for asking. Please, go ahead and donate at https://opencollective.com/getferdi. You don't need to cover an actual hourly rate (you can, though; my employer's customers pay 200 CHF/USD + tax per hour for me, usually with half a day minimum amount to book). Really, any amount will do. Of course, there is no commitment from any of us. But, of course, someone will be motivated when you give (probably with a matching amount of motivation).
In Life, You Always Meet Twice
It's really just to treat each other as we should. Think, what if we meet at the next developer conference? Will we be able to look each other in the eyes? (Note that I'm talking in the names of all maintainers, not just about myself.)

@mcdope It might worth to look into this again once https://github.com/getferdi/ferdi/pull/1033 gets merged. If Franz has changed the authentication protocol, it'll probably be propagated to Ferdi after that. Otherwise, we might need to reverse engineer Franz's authentication, but that would be a bit ridiculous (since Franz is already source available, at least).
On the other hand, this issue falls in line with the worrying trends of trying to ban alternative clients (like Ferdi) from online apps (see Gmail, Facebook login issues, Skype call issues, etc.) Maybe we should set up integration tests for such regressions, but just look at how that ended up for youtube-dl...

As far as I have seen in the Franz source the authentication is unchanged. I'm suspected something like a Domain Whitelist. Though I haven't tried using the exact user agent of Franz, maybe that will help.

As far as I have seen in the Franz source the authentication is unchanged. I'm suspecting something like a Domain Whitelist. Though I haven't tried using the exact user agent of Franz, maybe that will help.
Edit: Just tried again setting all the headers Franz sets in the current version (according to sourcecode), plus this time I've set source to "Desktop" instead of "Web" (in case there is some CORS security magic or domain whitelist preventing it if Web) and the user-agent to the same Franz is using. Still getting the same 401 response.
No clue how to fix this  For "desktop" source there can't be any whitelist, so it must work. I think this is something deeper, maybe the used http library doesn't support the encryption scheme used by the Franz server or something like that. But just guessing here.

Did you have any success with manual export/import (which was also broken last time I tried)? Or this is a separate issue?

Same here.  Cannot import my Franz Account even though I KNOW (and tested to be sure, to be sure) the creds were correct.

Same here.

Same here.

Is there a sequence that is working? I have only done and experienced part (1) of the original issue description:

(1) I startet with opening the link "https://api.getferdi.com/import" in my browser which produced "Could not login into Franz with your supplied credentials. Please check and try again". I do have a working franz account and can login into my meetfranz account without any problems (https://meetfranz.com/login)!
(2) Then I created a new getferdi account through the app itself with the "same" cretentials as my existing franz account.
(3) when opening "https://api.getferdi.com/import" in the browser again I get "There was an error while trying to import your account: - There is already a user with this email"

And if there's a better / functioning sequence with which to proceed, I'd like to do that rather than proceeding with (2) and (3).

I might be mistaken, but I'm now unable to login to Ferdi using the Ferdi server (api.getferdi.com) and I'm wondering if this is related? Login works fine when logging in using the Franz server (api.meetfranz.com) instead.

sepertinya sudah ada yang melakukan PR untuk fix issue ini, bisa kah anda mereview pr ini @getferdi/helping-with-issues
https://github.com/getferdi/server/pull/51

For my case, at least, that has not resolved the issue.

Still seeing this on my end too - valid Franz credentials, but https://api.getferdi.com/import says it can't log in.

The fix has been deployed to production and tested on my side:

Your account has been imported. You can now use your Franz account in Ferdi.

Confirmed that I was able to import my account again and login to it using nightly-78.

@kytwb per my earlier comment; I am continuing to experience this issue.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1736 ms

Resolved: Bofrost.at performance has improved in 011186b after 9 minutes.


Added 188

Added 188
Closes #77

I'm afraid this one's already implemented - just made the merge. Sorry!


[storage] Design for Off-cluster Storage

For the alpha, we need to point a deis installation at off-cluster storage. This ticket is a placeholder for that design.

[ ] design off-cluster storage configuration
[ ] document storage configuration in deis/workflow

@technosophos FYI removed you from the assignee. I'll be taking this later this week

@smothiki are you currently working on this? refer to https://github.com/deis/builder/pull/21 for how it's being done

documentation for storage config should be in builder in addition to workflow.

Workflow Right now looks for Minio and falls back to fetcher I have to add checks for outside storage environment variables.

great, thanks @smothiki


Cancelling an active PipelineRun with TaskRuns that are already complete hits "PipelineRunCouldntCancel" condition on v0.62.0

Expected Behavior
Cancelling a PipelineRun via kubectl patch that has TaskRuns that have completed (pass or fail) should properly cancel the pipelinerun and reflect that the pipeline is cancelled.
This is how i am currently cancelling the pipeline.
kubectl patch pipelinerun <pipeline> --type='merge' -p '{"spec":{"status":"Cancelled"}}'

Actual Behavior
Cancelling a PipelineRun that has taskruns that have already completed results in the pipeline encountering the PipelineRunCouldntCancel condition .
Steps to Reproduce the Problem
See the following gist for a script to reproduce: https://gist.github.com/hrivera-ntap/4376d79825c2b720e76fd109b26bd013
I suspect that this started happening after this was introduced: https://github.com/tektoncd/pipeline/pull/8147 , if possible @l-qing , can you confirm?
Additional Info

Kubernetes version:

Client Version: v1.28.6
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.6

Tekton Pipeline version:

v0.62.0

Your description is very detailed.
It seems that the cancellation request was sent to all TaskRuns here, including those that were already completed.
https://github.com/tektoncd/pipeline/blob/95fbf318460a2f73ac505bbdb13786a788d1c092/pkg/reconciler/pipelinerun/cancel.go#L138-L145
I will address this issue today. I'm very sorry for the inconvenience caused.
/assign l-qing


SEO improvements

Describe the problem
"SEO improvements
– Feed pages are higher indexed

Loading improvements
Page title changes
Review Google analytics account for TransitFeeds"

Proposed solution
See above
Alternatives you've considered
No response
Additional context
No response

Investigation Timebox: 1 day


Library refactoring

[x] Rename _root to elementRef
[x] Make directive and component ElementRef public accessible
[x] Remove directive and component from src/lib file names
[x] Make event registry listen and unlisten public accessible
[x] Add passive event handling to event registry
[x] Prefix private variables with leading underscore
[x] Functions must specify strongly-typed object(s)
[x] Add pointerup + mouseup types to ripple's Interaction Handler. reference: https://github.com/material-components/material-components-web/issues/1124#issuecomment-335647088
[x] Set default property values
[x] Rename MDC adapter files to adapter.d.ts
[x] Refactor MdcFormField
[x] Prefix events with on instead of handle (e.g.: onKeyPress)
[x] Set encapsulation to ViewEncapsulation.None
[x] Refactor mdc-icon-toggle to use mdc-icon
[x] Change MdcButton to Directive
[x] Change MdcList to Directive
[x] Change MdcToolbar to Directive
[x] Change MdcTab to Directive

Updating unit tests.


Bug: Summon app mixing up Share and Loot tokens in init params

Description
Seems like etherscan is mixing up loot token and shares token - maybe just in how it's validating and shoing the contract in the contract tab? not sure if there is something up in baal or the templates the summoner is using?
https://goerli.etherscan.io/address/0x6808653b0ba5386c48da47c9a51646fa82ed4f21#readContract

lootToken function returns:
0x0c84d8411448CB5C0625b5f8E74088Cf4C3605Db

token name is the LOOT name, but the verified contract on etherscan is the shares erc20
https://goerli.etherscan.io/address/0x0c84d8411448cb5c0625b5f8e74088cf4c3605db#code

sharesToken function returns:
0xD1b973C951f503B853A7A8601b5A68B071F73632

token name is the shares name, but the verified contract on etherscan is the loot erc20
https://goerli.etherscan.io/address/0xd1b973c951f503b853a7a8601b5a68b071f73632#code

https://github.com/HausDAO/daohaus-monorepo/pull/356


Github releases

Is it on the roadmap to publish releases of this Revit add-in?
For instance: pyRevit  is using this Github feature.

It would be nice to have a installer, so that a lot of non-technical users can download and install this Revit add-in.
For instance:

https://github.com/buildingsmart-community/bSDD-Revit-plugin/releases/tag/v1.0.0_beta


Chaining method calls with CHANGING parameters is not supported in 702

see https://github.com/abapGit/abapGit/pull/3955
probably easiest to implement this in code instead of the syntax diagrams

from #996,

from #221,

starting this

done in 2.59.0, closing


Virtual envs: ImportError with bin/opensfm_run_all

Building OpenSfM
What I have
Operating System: Manjaro Linux
KDE Plasma Version: 5.26.5
KDE Frameworks Version: 5.103.0
Qt Version: 5.15.8
Kernel Version: 5.15.94-1-MANJARO (64-bit)
Graphics Platform: X11
Processors: 8 × Intel® Core™ i7-8650U CPU @ 1.90GHz
Memory: 31.1 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics 620
Manufacturer: LENOVO
System Version: ThinkPad T580
What I did

[x] cloned recursively the repo
[x] installed opencv and ceres-solver (with paru package manager)
[x] created virtual env using conda called opensfm-env and activated it
[x] runned pip install -r requirements
[x] runned python setup.py build  all went well

What I got
But when I want to test bin/opensfm_run_all   data/berlin I get this error:
Traceback (most recent call last):
  File "/source/OpenSfM/bin/opensfm_main.py", line 9, in <module>
    from opensfm import commands
  File "/source/OpenSfM/opensfm/__init__.py", line 1, in <module>
    from opensfm import pybundle
ImportError: /home/ayoubf/miniconda3/envs/opensfm-env/bin/../lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /usr/lib/libtbb.so.12)

What could possibly have gone wrong ?
Thank you.

Hello, I am getting the same error. ANy workaround you found?
Thanks.

Hi @gautam-sh @ayoubft ,
Have you tried one of the first two results on google?
[1] https://github.com/rstudio/reticulate/issues/1282
[2] https://stackoverflow.com/questions/72540359/glibcxx-3-4-30-not-found-for-librosa-in-conda-virtual-environment-after-tryin

Thanks @fabianschenk and I apologize for not returning back here and mentioning that I already tried this solution and got it working.
So the installation process should look like this:

[x] clone recursively the repo
[x] install opencv and ceres-solver (with paru package manager)
[x] create virtual env using conda with name opensfm-env and activate it
[x] run pip install -r requirements
[x] run conda install -c conda-forge gcc=12.1.0
[x] run python setup.py build


Bug Fixed Class 'Arnovr\OwncloudProvisioning\Command\Assertion' not found

Bug Fixed Class 'Arnovr\OwncloudProvisioning\Command\Assertion' not found,
Added use Assert\Assertion; missing

please, can you accept this update ?


Support deserialising BadRequestError from Rerank

Hey Cohere team, I ran into an issue with the SDK when using the Rerank function. I'd like to be able to handle cases where the client returns a Bad Request, however, the SDK only supports returning core.ApiError and cohere.TooManyRequestsError. I believe this is due to the API documentation only describing responses for 200 and 429 statuses - I was unable to find this documentation online to raise a change to.
It would be great if the Cohere client supports deserialising error responses to all available error types, as it would make handling edge cases a lot nicer, and saves having to dig into the client source code to figure out what is going on.
Many thanks!

Hey @reecerussell funny you should ask this, I've been trying to document our errors this week! I just released v2.8.2 which has a bunch more error types. Let me know if that works better for you and if you have any other feedback for us. Thanks for reporting this!

Legend, thanks @billytrend-cohere 🙌🏻


Surface the failing command's name from the build error dump

Because Cargo's formatting of build errors is very noisy users struggle to understand what is causing the build to fail.
Here's a recent case where the failure reason was simple, but the user could not see it through all the noise.

    Compiling openssl-sys v0.9.103 (~/openssl-sys/openssl-sys)
+The following warnings were emitted during compilation:
+
+warning<EMAIL_ADDRESS>configuring OpenSSL build: Command 'perl' not found. Is perl installed?
+
 error: failed to run custom build command for `openssl-sys v0.9.103 (~/openssl-sys/openssl-sys)`
 note: To improve backtraces for build dependencies, set the CARGO_PROFILE_DEV_BUILD_OVERRIDE_DEBUG=true environment variable to enable debug information generation.

 Caused by:
   process didn't exit successfully: `~/openssl-sys/target/debug/build/openssl-sys-60b6752181d583d2/build-script-main` (exit status: 101)

 [...]

   --- stderr
-   thread 'main' panicked at ~/openssl-src-rs/src/lib.rs:578:14:
+   thread 'main' panicked at ~/openssl-sys/build/find_vendored.rs:18:39:

   Error configuring OpenSSL build:
-      Command: cd "~/openssl-sys/target/debug/build/openssl-sys-8f90d3753aa6037e/out/openssl-build/build/src" && env -u CROSS_COMPILE AR="ar" CC="cc" RANLIB="ranlib" "perl" "./Configure" "--prefix=~/openssl-sys/target/debug/build/openssl-sys-8f90d3753aa6037e/out/openssl-build/install" "--openssldir=/usr/local/ssl" "no-dso" "no-shared" "no-ssl3" "no-tests" "no-comp" "no-zlib" "no-zlib-dynamic" "--libdir=lib" "no-md2" "no-rc5" "no-weak-ssl-ciphers" "no-camellia" "no-idea" "no-seed" "darwin64-arm64-cc" "-nologo" "-MD" "-O2" "-Z7" "-Brepro" "-mmacosx-version-min=11.0"
-      Failed to execute: No such file or directory (os error 2)
+      Command 'perl' not found. Is perl installed?
+      Command failed: cd "~/openssl-sys/target/debug/build/openssl-sys-3ae48db4843d8fdf/out/openssl-build/build/src" && env -u CROSS_COMPILE AR="ar" CC="cc" RANLIB="ranlib" "perl" "./Configure" "--prefix=~/openssl-sys/target/debug/build/openssl-sys-3ae48db4843d8fdf/out/openssl-build/install" "--openssldir=/usr/local/ssl" "no-shared" "no-ssl3" "no-tests" "no-comp" "no-zlib" "no-zlib-dynamic" "--libdir=lib" "no-md2" "no-rc5" "no-weak-ssl-ciphers" "no-camellia" "no-idea" "no-seed" "darwin64-arm64-cc" "-nologo" "-MD" "-O2" "-Z7" "-Brepro" "-mmacosx-version-min=11.0"

Cleverness of the Debug impl of Command obscures the command name that has been run. This change prints the command name alone.
NotFound error is handled with a dedicated message.
Uses cargo:warning to also print the message above the stdout and stderr dumps.

I've also added try_build, so that I can make openssl-sys report the error without a noisy backtrace.

Instead of using panic, maybe using eprintln!() + std::process::exit(1) would work? openssl-src is probably not called in any context where recovering from an unexpected build error is necessary, and you added try_build for cases where build errors are expected to happen.

I've updated it with a clean exit, and converted other panics into errors.

Thanks for this!


Android 12 box recording screen streaming, playing streaming picture garbled

Hello, I used DisplayService on the Android 12 box to record the screen and push the stream, and then used the vlc player to pull the stream, but the picture was garbled, but after I used the Android 9 box to record the screen and push the stream and played the pull address, there was no such phenomenon, may I ask if you have any idea about it?

Maybe it is related with a video encoder bug in you Android 12.
Can you check in logcat if you are using the same video encoder in both cases? The idea is exclude you encoder here:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/java/com/pedro/encoder/utils/CodecUtil.java#L418

Closing as inactive.


remove cf deployment files

What? and Why?
A spike was done a while ago to get SDX working in cloudfoundry. We've since moved away from CF so these files will never be useful.
Checklist

[x] CHANGELOG.md updated? (if required)

Procfile needs to be deleted too

I can't find one!


Publish 1.6.2?

I see this in the history:

But I see this on npm (and I get an error if I try to use 1.6.2 in my package.json)

Is there anything blocking a release of 1.6.2?

FWIW, 1.6.2 just adds the --disable-workspace-trust flag.

Were you going to publish it, or is there something wrong with this version?  The npm registry still doesn't have 1.6.2.


How to store logs in a log file.

Hello there,
I am trying to store all sanic logs in a file, but I am not able to do that.
Please help me out.

custom log.
You can refer to this issue
#1644

Closing now. If you need more help, you can also turn to community forums.


feat: skills page

Description
Add/edit skills in a composer configuration page.
skills will be saved to appsettings.json
looks like
{
  "MicrosoftAppId": "",
  "luis": {
    "name": "",
    "authoringRegion": "westus",
    "defaultLanguage": "en-us",
    "environment": "composer"
  },
  "qna": {
    "knowledgebaseid": "",
    "endpointkey": "",
    "hostname": ""
  },
 // new added
  "skill": [
    {
      "manifestUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/manifest/manifest-1.0.json",
      "name": "Email Skill",
      "description": "The Email skill provides email related capabilities and supports Office and Google calendars.",
      "endpointUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/api/messages",
      "msAppId": "79432da8-0f7e-4a16-8c23-ddbba30ae85d"
    }
  ]
}

Task Item

close #2326
Screenshots

Add by manifest url

manifest url will be fetched in server, and auto-fill fields with the response data

    {
       // add by user
      "manifestUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/manifest/manifest-1.0.json",
      // auto-filled in server
      "name": "Email Skill",
      "description": "The Email skill provides email related capabilities and supports Office and Google calendars.",
      "endpointUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/api/messages",
      "msAppId": "79432da8-0f7e-4a16-8c23-ddbba30ae85d"
    }

Add by app configure

endpointUrl should start with http[s]
msAppId can not be duplicated.

Edit & delete

if created by manifest url, only allowed to edit manifestUrl.
if created by app configure, only allowed to edit app configuration.

@zhixzhan why do we need to serialize the following fields in appsettings.json ?
      "name": "Email Skill",
      "description": "The Email skill provides email related capabilities and supports Office and Google calendars.",
      "endpointUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/api/messages",
      "msAppId": "79432da8-0f7e-4a16-8c23-ddbba30ae85d"

All of these fields can be retrieved from of the manifest url. I like keeping the hosted manifest data as the single source of truth.

@cwhitten Did you mean only save those in appsettings.json
  "skill": [
     // if skill added by manifestUrl
    {
      "manifestUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/manifest/manifest-1.0.json",
    },
     // if skill added by app configuration
    {
      "name": "Email Skill",
      "endpointUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/api/messages",
      "msAppId": "79432da8-0f7e-4a16-8c23-ddbba30ae85d"
    }
  ]

When we need detail data (name/endpointUrl/msAppId..) from manifestUrl,

Option 1. Make a real-time request (to composer server) to fetch them.
Option 2. Fetch them when index botProject, put detail data in getBotProject response

 [
     dialogs:[...]
     lgFiles:[...]
     luFiles:[...]
     settings:{
           ...
           skill: [
                  {
                     "manifestUrl": "not empty if add by manifest url"
                     "name": "Email Skill",
                     "endpointUrl": "https://yuesuemailskill0207-gjvga67.azurewebsites.net/api/messages",
                     "msAppId": "79432da8-0f7e-4a16-8c23-ddbba30ae85d"
                 }
          ]
     }
 ]

By doing so, we reach a balance of single source of truth & usage convenience.

@zhixzhan can you explain the scenario where a skill is provided via "app configuration"? I thought all skills are configured via providing a manifest url and nothing more.

@zhixzhan can you explain the scenario where a skill is provided via "app configuration"? I thought all skills are configured via providing a manifest url and nothing more.

What zhixiang refered is here

We supported two ways to create, do you mean we should just disable the second approach?

We discussed this in standup today and couldn't find the scenario around "Add by App Configuration". The only way to add a skill is by adding it from a manifest url.

Ok, get it

@boydc2014 @cwhitten updated.

Some functional issues:

It's possible to "connect" the same skill over and over again. We shouldn't allow this. If a Skill is already added and one attempts to add it again, it should be a no-op.
Deleting the Skill row does not delete the entry from the skills structure in the settings page.

We can fix this via inspection of the manifest url and the operation the user is attempting to complete.

Speak of this, how we treat a skill is duplicated?

By manifestUrl
One url may be different in [protocol]://[host-path][query][fragment]
should we take [host-path] as an uniq id to avoid duplicate ?

Or by endpoint.msAppId in manifestUrl response.

The endpoints is a list, it may have production/test/.. endpoint,  currently I just take endpoints[0] as a skill's endpoint, it may be not very accurate. then by endpoint.msAppId to judge duplicate seems also difficult.

@zhixzhan Can we reintroduce the "name" field from the manifest into the settings page as well, so:
{
  "name": "Email skill",
  "manifestUrl": "http://contoso.com"
}

I understand that a manifest may have different endpoints. Is this a blocker for you?

@zhixzhan Can we reintroduce the "name" field from the manifest into the settings page as well, so:
{
  "name": "Email skill",
  "manifestUrl": "http://contoso.com"
}

@cwhitten Ok, "name" looks good.

I understand that a manifest may have different endpoints. Is this a blocker for you?

No, here I just use endpoints[0] as skill's endpoint, just want bring it up be aware.

thanks for calling that out. it's important for @tdurnford's knowledge to allow the user to select an endpoint when calling the skill

updated

E2E is failing due a flakey test. Merging.


Dont show cancelled deployments as current prod deployment

If there was a push to master that was cancelled by the build step cancellation, zeitgeist still shows it as the current prod deployment.

App Version: 3.2.2		Device: iPhone16,2
OS: iOS 17.3.1

Thanks for the feedback @motz0815! This issue will be fixed in the next patch version (v3.2.4)


docs: include mermaid.js library

Adds the minified mermaid.js library to the mkdocs extra_javascript.
Inspired by a comment on the original feature request at mkdocs-material for mermaid support.
Using the extra_javascript feature to include the minified mermaid.js library from<EMAIL_ADDRESS>Resolves: #567

Here are some findings from my testing:

this did not work when using the fence_code_format
adding extra_css did not change behaviour

markdown_extensions:
  - pymdownx.superfences:
      custom_fences:
        - name: mermaid
          class: mermaid
          format: !!python/name:pymdownx.superfences.fence_div_format
          # Doesn't work
          # format: !!python/name:pymdownx.superfences.fence_code_format

extra_javascript:
  -<EMAIL_ADDRESS>
# Doesn't change behaviour
# extra_css:
#   -<EMAIL_ADDRESS>


Inserting incorrect value into field when we click on text

Hello,
I am using this plugin into our mobile app. All is working fine except one thing, Please follow the scenario:

Type C into text field
Select second option from dropdown - just click on text instead of blank area
You will get first value instead of selected value.

You have to try this 3 to 4 times to recreate issue and move focus up & down then select. But in mobile browser you can get such issue at every time. You can recreate this issue at below link:
http://textextjs.com/manual/examples/ajax-with-filter-tags-and-autocomplete.html
This should be fix asap. Thanks in advance...

any solution for this?


Please add contrib routingconnector to Splunk distro

We'd like to route logs through different pipelines depending on their attributes. Mainly, we'd like to sample some applications while getting all logs from other applications. To do that, we want to create a pipeline with probabilistic_sampler processor and another pipeline without it - then use the routing connector to decide which pipeline the log goes to based on attributes.
I tried to use the routing processor, but am getting error decoding 'connectors': unknown type: "routing" for id: "routing" (valid values: [count forward spanmetrics])
I suppose the routing processor only needs to be added from contrib into the splunk distro of otel-collector to get this to work: https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/processor/routingprocessor

Please make sure to open a Splunk idea and link it here.

Thank you, we will follow up with the Splunk idea.


Add new Windstream Parser

Raising this Pull Request to onboard a new vendor - Windstream.
All testing I have completed looks to deal with these emails correctly.
Invoke tests --local does report that I have too many local variables but I am not sure how strict we are being on these things as I am seeing messages for other parsers in there as well.

Invoke tests --local does report that I have too many local variables but I am not sure how strict we are being on these things as I am seeing messages for other parsers in there as well.

Don't worry about it

Hey @chadell - I just noticed that my start/end logic was incorrect as it was marking the initial time received from the vendor as UTC from the get-go. I have modified the code to correctly localise it to "EST". Conversions to UTC will then occur correctly. This did mean that I needed to move some things around as I added a bunch of extra local variables to deal with it. I have managed to avoid going over the number of local variables (15) and statements in my script (50) due to the things I shuffled in the last commit. Everything tested correctly now and this should be the last commit as long as you are happy with the changes I made!
Thanks :)


Export unmarshal error type

http://thecodebarbarian.com/importance-of-apis-in-a-full-stack-world.html#comment-3441552620


⬆ Upgrade support for SQLAlchemy 1.4.49, update tests

This addresses the tests failures as seen in #510

Great, thank you @sandrotosi! 🚀 🍰


Fixing Vulnerability

Issue #, if available:
Description of changes:
Updating aws-sdk-go to v1.44.114
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

OSX gives an error, but Ubuntu and Windows do not when a connection cannot be made.
https://github.com/aws/aws-xray-sdk-go/actions/runs/3617649480/jobs/6096682231#step:4:91


[front_end] Consider adding extension mode for compiling private fields in libraries

During debugging, users might want to evaluate expressions that contain references to private fields in other libraries.
Currently, VM's incremental compiler allows that by simply making the parameters to the synthetic procedure, that is created for expression evaluation, have dynamic types.
In web scenario we need the types to generate correct JavaScript code:
Issue: https://github.com/dart-lang/sdk/issues/43469
CL: https://dart-review.googlesource.com/c/sdk/+/166925
This fix actually removes ability to compile such expressions - because the types are available, the compilation just fails.
We may want to consider adding a feature to the compiler where it would allow private fields from libraries as a extension mode, only for expression compilation (need to collect used feedback first to assess priority).
Comment from @johnniwinther
Note that the ability to resolve accesses to private fields in other libraries would be need to be speculative since it goes beyond what the language offers. It might still be a useful/needed feature, though.
Such an extension to the resolution would need to search for potential targets, and only if the search comes out with a single target do we have a stable (sound-ish) semantics.
Consider these cases:
// lib1.dart

class A {
  int _foo;
  int _bar;
}

// lib2.dart

import 'lib1.dart';

class B {
  int _foo;
}

class C extends A {
  int _bar;
}
// main.dart
import 'lib1.dart';
import' lib2.dart';
#syntheticProdecure(A a, B b, C c, dynamic d) {
  a._foo; // Would target A._foo
  a._bar; // Would naturally target A._bar but could have been C._bar
  b._foo; // Would target B._foo
  b._bar; // Would not have a natural target
  c._foo; // Would target A._foo
  c._bar; // Would naturally target B._bar but could have been A._bar
  // Would not have a natural target, could validly be A._foo or B._foo
  d._foo;
  // Would not have a natural target, could validly be A._bar or C._bar
  d._bar;
}

Note that a noSuchMethod implementation in the searched classes would always be a potential target.

It would be helpful to define upfront:  What does it mean to write x._foo in a breakpoint?
I had the impression that the semantics in this case were the same if you were to modify the source file at the location of that breakpoint and insert the expression by hand (similar to writing print('${x._foo}')). Under such interpretation, you would only be allowed to evaluate private fields from the same library that has the breakpoint.
With that in mind, it seems to me that your CL is not adding a regression, but changing something that used to be a runtime error to a compile-time error (I'd say that's a good thing :)):

When parameters were dynamic, the CFE accepted code like d._foo (where d is dynamic). The CFE generated a dynamic invocation where _foo encodes the name of the enclosing library containing the breakpoint, then defers to the backend to resolve the call. When executing that expression, I'd expect the VM and DCC would give a noSuchMethod at runtime if d's runtime type was not defined in that same library where the breakpoint is.
When parameters are typed, the CFE can statically determine whether _foo is in the same library, and if not, issue a compile-time error upfront.

Is that correct?
Re prioritization - my hunch is that accessing private fields is not as highly requested, but I don't have data to confirm this. This also has a viable alternative: users can add a public getter to expose private fields and workaround this limitation. Given that, my inclination is to keep this at a lower priority.


validateGlb version check

Worked on issue #98. Updated validateGlb to check for version, added spec validateGlbSpec, also updated specUtility.

Thanks!


Update dependencies

This PR updates the dependencies required to deal with some reported vulnerabilities.
Also, is it possible to release a new version in npm? The last version 1.0.1 is from 3 years ago.

@diego-binary Thank you. I will try to publish a new version if I still have the rights.


Adapter goes red and has to be manually restarted.

[onReady] Retrieving device failed!
Happens also with new App and firmware. Seems to be cloud related.

Today the adapter again went offline.
Latest version installed 1.7.5
2024-08-01 10:34:22.133 - error: zendure-solarflow.0 (224435) [onReady] Retrieving device failed!


Set Up Integration Suite Trial

Tutorials: https://developers.sap.com/tutorials/cp-starter-isuite-onboard-subscribe.html
Your documentation is out of date and the screenshot and instructions no longer relevant since the systems have been refactored.

@wikushartzenberg - thanks for the feedback. Are you referring to the content that mentions about how to subscribe to the service? If yes, we are in the process of publishing the updated content very soon. If no, could you please let me know which part of the tutorial is irrelevant so that I can fix it.
Thanks,
Karuna

@wikushartzenberg - tutorials are updated now


Another (pure macro) approach to value paramterization

Description
I have written a small macro-only tool to help me parameterize test cases.
Here it is in its entirity:
#define CONCAT(a, b) CONCAT_INNER(a, b)
#define CONCAT_INNER(a, b) a ## b

#define PP_NARG(...) PP_NARG_(__VA_ARGS__,PP_RSEQ_N())
#define PP_NARG_(...) PP_ARG_N(__VA_ARGS__)
#define PP_ARG_N(_1, _2, _3, _4, _5, _6, _7, _8, _9, N, ...) N
#define PP_RSEQ_N() 9, 8, 7, 6, 5, 4, 3, 2, 1

#define PARENS ()

#define EXPAND(...) EXPAND4(EXPAND4(EXPAND4(EXPAND4(__VA_ARGS__))))
#define EXPAND4(...) EXPAND3(EXPAND3(EXPAND3(EXPAND3(__VA_ARGS__))))
#define EXPAND3(...) EXPAND2(EXPAND2(EXPAND2(EXPAND2(__VA_ARGS__))))
#define EXPAND2(...) EXPAND1(EXPAND1(EXPAND1(EXPAND1(__VA_ARGS__))))
#define EXPAND1(...) __VA_ARGS__

#define DEBRACE(...) __VA_ARGS__

#define FWD(func, ...) func(__VA_ARGS__)

#define FOR_EACH(func, constVar, ...) __VA_OPT__(EXPAND(FOR_EACH_HELPER(func, constVar, __VA_ARGS__)))

#define FOR_EACH_HELPER(func, constVar, sub, ...) FWD(func, DEBRACE constVar, sub) \
                                                  __VA_OPT__(FOR_EACH_AGAIN PARENS (func, constVar, __VA_ARGS__))
#define FOR_EACH_AGAIN() FOR_EACH_HELPER

#define SUBCASE_VAR_GENERATE(nested, var, ...) FOR_EACH(SUBCASE_GENERATE, (nested, var), __VA_ARGS__)
#define SUBCASE_UNPACK(nested, var) SUBCASE_VAR_GENERATE(nested, var)

#define SUBCASE_SPECIAL1(a) SUBCASE_UNPACK(, DEBRACE a)
#define SUBCASE_SPECIAL2(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL1(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL3(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL2(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL4(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL3(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL5(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL4(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL6(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL5(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL7(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL6(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL8(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL7(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL9(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL8(__VA_ARGS__), DEBRACE a)

#define SUBCASE_GENERATE(nested, var, sub) SUBCASE("AUTOGENERATED RECURSIVE PARAMETERIZATION " #var " = " #sub) { var = sub; nested; }

#define SUBCASE_PARAMETERIZE(...) CONCAT(SUBCASE_SPECIAL, PP_NARG(__VA_ARGS__))(__VA_ARGS__) (void)0

As opposed to the implementation in #321 this does not require any standard headers.. or code!
Example
Using it is very simple and pretty intuitive:
TEST_CASE("Parameterization") {
	String a; String b; int i;
	SUBCASE_PARAMETERIZE(
		(a, "A", "B", "C"),
		(b, "x", "y", "z"),
		(i, 1, 2, 3, 4)
	);
	std::cout << a << " - " << b << " - " << i << std::endl;
}

The output varies i first, then b, then a, as you'd expect.
Variables lower in the parameterization list can depend on higher up variables (e.g. b could depend on a).
Shortcomings
There are a few shortcomings to my approach.

It requires a standard compliant preprocessor, which means having to change a project setting in MSVC when not using the c14/c17 standard..
The exact form I have posted here requires C++20's __VA_OPT__ for the loop.
The preprocessor is not turing complete. The possible length is both limited in the amount of arguments, as well as in the amount of parameters.

Remedies
1.
I feel like this might very well be the deal breaker.
There might be a way to implement the same behavior with MSVC's legacy preprocessor (which can be checked for via _MSVC_TRADITIONAL).
If there is sufficient interest in my approach I could investigate this further.
It seems like the compliant preprocessor might become standard in MSVC in C++>=20, which helps very little in maintaining compatibility with C++11 and C++17 though.
I could try and post a feature request for a pragma that toggles preprocessor behavior in the Microsoft developer community, but yeah, I'll eat a hat if that's going to happen.
2.
This is trivial to solve, rewriting it to use the "classic" style is as easy as pie, having it toggle with C++20 should be similarly easy. However, that would probably still be slightly problematic, see 3.
3.
For most cases this should be absolutely trivial, utilizing the new for-each means an exponential increase in possible inputs for each line in the EXPAND chain. Luckily, we're able to use the new for-each for the actual arguments, which is usually the one where more input is anticipated. I am unsure whether the modern for-each is possible for the parameters, since it might not be compatbile with their nested nature.
Either way the traditional for-each must be available for both cases anyways, in order to support C++<20.
This raises the question of how much input should be allowed, both for arguments and parameters, since for each additional argument/parameter, we require one extra source line.
I think the most straightforward approach is splitting this into different files, with a #define being able to request the needed amont per header inclusion. Concept:
#if REQUESTED >= 10
  #include "expand10.h" // 0 - 9
  #if REQUESTED >= 100
    #include "expand100.h" // 10 - 99
    #if REQUESTED >= 1000
      #include "expand1000.h" // 100 - 999
    #endif
  #endif
#endif

However, I have not at all measured any compile/include performance (partly, because I don't really know how to!) so I can't really say anything on that aspect.
Conclusion
What I've written is fine enough for my purposes, if there is actual interest in incorporating it into the library I would happily look into solving the problems I've posed though. If the things I mentioned do prevent an inclusion in the main library, perhaps it could be included as an extension? Or perhaps the other approach is preferable in that case, I don't know!

I won't be able to chime in on this and dig deeper, but here are my 2 cents:
An extension header as you have usggested is 100% an option (there's already a folder for such) and that way it won't have to be conformant with old standards and it might even be gcc/clang only.
You could wait to see what the interest is from others before moving forward with committed code because it would then have to be maintained. But it's great to at least have it in an issue for others to find!

Description
I have written a small macro-only tool to help me parameterize test cases. Here it is in its entirity:
#define CONCAT(a, b) CONCAT_INNER(a, b)
#define CONCAT_INNER(a, b) a ## b

#define PP_NARG(...) PP_NARG_(__VA_ARGS__,PP_RSEQ_N())
#define PP_NARG_(...) PP_ARG_N(__VA_ARGS__)
#define PP_ARG_N(_1, _2, _3, _4, _5, _6, _7, _8, _9, N, ...) N
#define PP_RSEQ_N() 9, 8, 7, 6, 5, 4, 3, 2, 1

#define PARENS ()

#define EXPAND(...) EXPAND4(EXPAND4(EXPAND4(EXPAND4(__VA_ARGS__))))
#define EXPAND4(...) EXPAND3(EXPAND3(EXPAND3(EXPAND3(__VA_ARGS__))))
#define EXPAND3(...) EXPAND2(EXPAND2(EXPAND2(EXPAND2(__VA_ARGS__))))
#define EXPAND2(...) EXPAND1(EXPAND1(EXPAND1(EXPAND1(__VA_ARGS__))))
#define EXPAND1(...) __VA_ARGS__

#define DEBRACE(...) __VA_ARGS__

#define FWD(func, ...) func(__VA_ARGS__)

#define FOR_EACH(func, constVar, ...) __VA_OPT__(EXPAND(FOR_EACH_HELPER(func, constVar, __VA_ARGS__)))

#define FOR_EACH_HELPER(func, constVar, sub, ...) FWD(func, DEBRACE constVar, sub) \
                                                  __VA_OPT__(FOR_EACH_AGAIN PARENS (func, constVar, __VA_ARGS__))
#define FOR_EACH_AGAIN() FOR_EACH_HELPER

#define SUBCASE_VAR_GENERATE(nested, var, ...) FOR_EACH(SUBCASE_GENERATE, (nested, var), __VA_ARGS__)
#define SUBCASE_UNPACK(nested, var) SUBCASE_VAR_GENERATE(nested, var)

#define SUBCASE_SPECIAL1(a) SUBCASE_UNPACK(, DEBRACE a)
#define SUBCASE_SPECIAL2(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL1(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL3(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL2(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL4(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL3(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL5(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL4(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL6(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL5(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL7(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL6(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL8(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL7(__VA_ARGS__), DEBRACE a)
#define SUBCASE_SPECIAL9(a, ...) SUBCASE_UNPACK(SUBCASE_SPECIAL8(__VA_ARGS__), DEBRACE a)

#define SUBCASE_GENERATE(nested, var, sub) SUBCASE("AUTOGENERATED RECURSIVE PARAMETERIZATION " #var " = " #sub) { var = sub; nested; }

#define SUBCASE_PARAMETERIZE(...) CONCAT(SUBCASE_SPECIAL, PP_NARG(__VA_ARGS__))(__VA_ARGS__) (void)0

As opposed to the implementation in #321 this does not require any standard headers.. or code!
Example
Using it is very simple and pretty intuitive:
TEST_CASE("Parameterization") {
	String a; String b; int i;
	SUBCASE_PARAMETERIZE(
		(a, "A", "B", "C"),
		(b, "x", "y", "z"),
		(i, 1, 2, 3, 4)
	);
	std::cout << a << " - " << b << " - " << i << std::endl;
}

The output varies i first, then b, then a, as you'd expect. Variables lower in the parameterization list can depend on higher up variables (e.g. b could depend on a).
Shortcomings
There are a few shortcomings to my approach.

It requires a standard compliant preprocessor, which means having to change a project setting in MSVC when not using the c14/c17 standard..
The exact form I have posted here requires C++20's __VA_OPT__ for the loop.
The preprocessor is not turing complete. The possible length is both limited in the amount of arguments, as well as in the amount of parameters.

Remedies
1.
I feel like this might very well be the deal breaker. There might be a way to implement the same behavior with MSVC's legacy preprocessor (which can be checked for via _MSVC_TRADITIONAL). If there is sufficient interest in my approach I could investigate this further. It seems like the compliant preprocessor might become standard in MSVC in C++>=20, which helps very little in maintaining compatibility with C++11 and C++17 though. I could try and post a feature request for a pragma that toggles preprocessor behavior in the Microsoft developer community, but yeah, I'll eat a hat if that's going to happen.
2.
This is trivial to solve, rewriting it to use the "classic" style is as easy as pie, having it toggle with C++20 should be similarly easy. However, that would probably still be slightly problematic, see 3.
3.
For most cases this should be absolutely trivial, utilizing the new for-each means an exponential increase in possible inputs for each line in the EXPAND chain. Luckily, we're able to use the new for-each for the actual arguments, which is usually the one where more input is anticipated. I am unsure whether the modern for-each is possible for the parameters, since it might not be compatbile with their nested nature. Either way the traditional for-each must be available for both cases anyways, in order to support C++<20. This raises the question of how much input should be allowed, both for arguments and parameters, since for each additional argument/parameter, we require one extra source line. I think the most straightforward approach is splitting this into different files, with a #define being able to request the needed amont per header inclusion. Concept:
#if REQUESTED >= 10
  #include "expand10.h" // 0 - 9
  #if REQUESTED >= 100
    #include "expand100.h" // 10 - 99
    #if REQUESTED >= 1000
      #include "expand1000.h" // 100 - 999
    #endif
  #endif
#endif

However, I have not at all measured any compile/include performance (partly, because I don't really know how to!) so I can't really say anything on that aspect.
Conclusion
What I've written is fine enough for my purposes, if there is actual interest in incorporating it into the library I would happily look into solving the problems I've posed though. If the things I mentioned do prevent an inclusion in the main library, perhaps it could be included as an extension? Or perhaps the other approach is preferable in that case, I don't know!

Would u like provide such a macro？
|
v
for (auto a : {"A", "B", "C"}) {
    for (auto b : {"x", "y", "z"}) {
        for (auto i : {1, 2, 3, 4}) {
            std::cout << a << " - " << b << " - " << i << std::endl;
        }
    }
}

|
v
TEST_CASE("Parameterization") {
	SUBCASE_RUN(
	        (std::cout << a << " - " << b << " - " << i << std::endl;),
		(a, "A", "B", "C"),
		(b, "x", "y", "z"),
		(i, 1, 2, 3, 4)
	);
}


#3088, Use ANTLR parse SQL-92

Fixes #3088.

Pull Request Test Coverage Report for Build 457

2 of 16 (12.5%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 65.814%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/spi/database/SQL92DatabaseType.java
2
3
66.67%

sharding-core/sharding-core-parse/sharding-core-parse-sql92/src/main/java/org/apache/shardingsphere/core/parse/SQL92Parser.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-sql92/src/main/java/org/apache/shardingsphere/core/parse/SQL92ParserEntry.java
0
3
0.0%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/metadata/datasource/dialect/SQL92DataSourceMetaData.java
0
9
0.0%

Totals

Change from base Build 419:
-0.05%

Covered Lines:
9730

Relevant Lines:
14784

💛  - Coveralls


ROSA w/ HCP: scale to 500 node clusters

ROSA w/ HCP: scale to 500 node clusters
Safely support clusters with up to 500 nodes as validated by Red Hat load testing.

https://docs.openshift.com/rosa/rosa_planning/rosa-hcp-limits-scalability.html#tested-cluster-maximums-hcp-sd_rosa-hcp-limits-scalability


update installation.md

I have proposed some changes.

After installing uber-juno many peoples face error and not able to use REPL. So, they don't find any quick fix.
I have just added some quick fixes.
I have introduced julia-client packages to avoid an error.

it seems like the lines you added are basically the same as what we describe already, no?

Yeah, some lines are matching but when we face error after installing juno, then what we should do after it. I have described that. I have mentioned about juli-client but I have not found it in our previous installation guide. So, that's why I have added it.

so you mean sometime we need to do re-install uber-juno right ?
If so, it would be better to describe what is the error and why it does happen instead of saying "just reinstalling it". Or, at least it would be better to be described concisely (maybe in !!! tip section or something).

so you mean sometime we need to do re-install uber-juno right ?
If so, it would be better to describe what is the error and why it does happen instead of saying "just reinstalling it".

Yes, right. I can mention the error but till now I am not able to investigate why it does happen.  😒

so you mean sometime we need to do re-install uber-juno right ?
If so, it would be better to describe what is the error and why it does happen instead of saying "just reinstalling it".

Done the required changes.


type_or_die

Your checklist for this pull request
Author name
Jesse Blum aka jblum-coder
About your game
Type out words before the time runs out!
How do you play your game?
Use WASD to move to the characters and J to select the letter. To remove a letter press K.
Code

[x] The game was made using the [Sprig editor]
[X] The game is placed in the in the [/games directory]
[X] The code is significantly different from all other games in the [Sprig gallery]
[X] The game runs without errors.
[X] The name of the file/game contains only alphanumeric characters, -s, or _s.
[X] The game name is not the same as the others from [gallery]

Image (If an image is used)

[X] The image is in the [/games/img directory]
[X] The name of the image matches the name of your file.

Thanks for your PR!

Hello, I heard about Sprig from my computer science teacher. I am a sophomore and i am in AP computer science principles. He showed me this because he knows I am a good coder and thought I might enjoy it. I thought it would be cool to get a Sprig so i developed the game “type_or_die” in hopes of meeting the requirements and getting sent a Sprig of my own.

I increased the time, I get what your saying. It was super hard, but it's now 15 seconds. Is it good?


Add Koji remote repo content consolidation to local hosted repo

This implements a backgrounded / threaded process whenever a new Koji remote repository is created, which will proactively download all available files (based on the Koji build / archive records), and copy that content into a single hosted repository, creating the target repo and adding it to the
group first if necessary. The target hosted repository will be added to the top of the target group's
membership, and will have the same name as the target group. Once all content for the remote repository is transferred into the hosted repository, associated metadata paths will be removed from both the hosted repo and the target group, so these will be regenerated. Finally, the newly-created Koji remote repository will be removed from the group so as to keep the group as slim as possible.
Along with this new feature, the patch also adds a new repair endpoint for koji which will attempt to
go through all of the Koji remote repositories in the membership of some target group and run the same consolidation logic on all of them, in the order in which they appear in the membership.
Consolidation will use checksums available in the Koji build records, along with those calculated during
various transfers (both on download and on copy to the consolidated repo) to ensure nothing is corrupted.
If content is corrupted, the download will be retried up to 3 times per path.
If the consolidation fails because of corruption or some other reason, the remote repo will be retained
but marked as having failed consolidation for later follow-up with some new type of repair feature.
This patch also contains logic to clear metadata caches when the corresponding metadata file is
deleted via the ContentManager component. This should help us de-cache incorrect metadata when it happens.

@ligangty The hosted koji repo is per group IIUIC. e.g. for pnc-build group, there is a hosted:pnc-build in it.


renderRange for torque layers

Exposes torque methods to:

render a range with start and end steps: renderRange(start, end)
reset render range: resetRenderRange()

cc @viddo

:+1:


set whereabout of pet to same position

It is possible to set a pets wherabout to like, inside even it is already inside. The position will be then set with a new time stamp and the schedule of the pet is affected with wrong data.

[x] Check position of pet before send the request to the pet care server

was fixed with https://github.com/andreasvogt89/mauzis/commit/7bd84c68459b0a9082a42cb8956afcf03745c3ef


Feature Request: Provide automatic garbage collection triggering by the {N} Runtimes

Is your feature request related to a problem? Please describe.
When large native objects are created and referenced from JavaScript they do not cause memory pressure for the garbage collection and could remain alive for too long. This sometimes causes memory issues and even app termination.
Describe the solution you'd like
Provide a setting in app/package.json for configuring a heuristics algorithm implemented in the {N} runtime, which will periodically check for the current memory footprint of the application and forcefully trigger JS garbage collections depending on memory usage.
The idea of the algorithm is to check the current memory footprint on each N (configurable number) of native calls. When the application's memory usage has increased by at least M(another configurable value) megabytes it will immediately trigger a garbage collection.
Describe alternatives you've considered
Leave users to manually call __collect whenever they decide in the application logic. This however doesn't create a best practice approach and could lead to other problems. Having one recommended approach is preferable although we know that it may have cases which will not benefit or could even suffer (most likely performance-wise) from it. That's why it should be switched off by default.
Additional context
It turns out that a single garbage collection cycle might not be enough to free a (large) object. This is the case when a long chain of JS -> Native -> JS -> Native ...  references leads to the given object. During the first cycle only the first JS -> Native part of the chain will be released. The reason is that all Native instances keep strong references to JS fields and they cannot be marked for collection until the last Native instance referring them is destroyed. On the other hand, all JS instances also hold strong references to the Native fields and they can't be destroyed until the last JS instance holding a reference is garbage collected.
The GitHub Issue for the Android Runtime is https://github.com/NativeScript/android-runtime/issues/1232

(android only) Can we somehow know if GC ran in native (e.g. event, memory consumtion falls, etc) and simply trigger the V8 GC then? When native memory starts to fill up, the Java GC should run and try to free up some. We register this and call the corresponding JS method to start GC there as well.

I'm not sure but we should implement this in the android-runtime instead of require plugins/apps to handle it themselves.
We will open a separate feature request in the android-runtime repository as well and I suggest that we discuss all details regarding the android implementation there.

How heavy is the _collect statement?    Why not just call _collect four times on navigation away from a page.  That should free up most memory as the UIViewController, and UIView are freed up in passes 1 & 2 and the first two levels of the screen should then get cleaned up.

@NathanaelA we used to do that when navigating away from a page. There are cases where you put several pages in the navigation stack and then discard them. That used to trigger huge delays, like you click "back" and the app freezes for two seconds.
We've discussed with @mbektchiev that there may be a way to keep an eye on the memory usage of the app and hint the JS VM to run async GC more frequently in general, while when the memory pressure is high, to run blocking GC passes. That sounds a little bit better since it will not be blocking apps in cases there is enough free memory.
Please note the approach I described earlier is merely a hint how native APIs can be used in a way that they are more easily collected by the GC and overall reducing both CPU and memory usage. When apps are crashing with OOM it should be possible to attach Xcode instruments, find what native instances cause the memory spike and try to detangle the native things.

@NathanaelA That might be a solution but you never know in advance how many GC passes will be required in order to reclaim the memory. Moreover it might be the last element which occupies the most amount of it. A rather extreme example would be if you have a deeper UI components hierarchy and there's a video player at the lowest level that keeps a video file loaded in memory.
Another peculiarity is that due to the way native instances are being delay-released, calling several __collects  in row doesn't actually manage to get rid of the chain -- rather it's required to queue them on the main loop, by calling setTimeout(__collect, 0); for example.
All in all, it seems that there are a lot of details that need to be considered in order to successfully cope with these memory issues and IMO the runtime has to provide a mechanism for that.

@mbektchiev it’d be nice if developers had a property to set on certain objects which the system could consider when being more aggressive or not to collect. For example a developer could flag a Page object with ‘gc=“auto”’ or ‘gc=“aggressive” if they knew that page contains a lot of memory hungry components that when navigated away they would want the system to aggressively recollect memory from by calling the GC multiple times or as @pana-cc mentioned potentially traversing the tree to agreesively free up the stack up to the top node with one GC pass or whichever strategy ended up being most effective.
Idea would be to allow a view property to be set on nodes which the system would consider when doing GC passes and how aggressive it should be with an apps view hierarchy.


fix: use regionalized sts calls for admin login

Description of changes

configures a regional STS client when making calls for studio login. This is to support opt-in regions where non-regionalized STS tokens are not valid
Issue #, if available

Description of how you validated changes
Manually verified
Checklist

[x] PR description included
[x] yarn test passes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #12054 (3bca071) into dev (984c0f8) will decrease coverage by 3.48%.
The diff coverage is 53.81%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##              dev   #12054      +/-   ##
==========================================
- Coverage   49.45%   45.97%   -3.48%
==========================================
  Files         692      819     +127
  Lines       33424    37689    +4265
  Branches     6840     7724     +884
==========================================
+ Hits        16529    17329     +800
- Misses      15370    18685    +3315
- Partials     1525     1675     +150

Impacted Files
Coverage Δ

...sync-simulator/src/data-loader/opensearch/index.ts
45.45% <0.00%> (ø)

...s/amplify-appsync-simulator/src/type-definition.ts
100.00% <ø> (ø)

...sync-simulator/src/velocity/util/dynamodb-utils.ts
6.55% <0.00%> (ø)

...y-appsync-simulator/src/velocity/util/map-utils.ts
10.00% <ø> (ø)

...tor/src/velocity/util/transform/dynamodb-filter.ts
4.93% <0.00%> (ø)

...ync-simulator/src/velocity/util/transform/index.ts
21.05% <0.00%> (ø)

...appsync-simulator/src/velocity/value-mapper/map.ts
86.88% <ø> (ø)

...ackages/amplify-category-auth/src/commands/auth.ts
0.00% <0.00%> (ø)

...es/amplify-category-auth/src/commands/auth/help.ts
0.00% <0.00%> (ø)

.../amplify-category-auth/src/commands/auth/import.ts
0.00% <0.00%> (ø)

... and 534 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Issue] azd init should also include github workflow file

azd init generates infra and azure.yaml, but doesn't include an azure-dev.yml github workflow file.
I had to hunt that down on my own.
Would be nice if it included one.

I have always wondered if maybe azd pipeline config (or maybe a separate gesture like azd pipeline init?) would be the right place to provide a starter workflow for azd. I could see that:

Maybe some users won't actually need CI/CD, and these extra files aren't well understood.
Maybe some users would benefit from progressive disclosure to build up a better mental model instead of having all these things upfront.

Let's say if we were trying to implement this feature in azd init today. We probably would need to surface a question of whether Azure Pipelines or GitHub should be used if no folders were present.. But would users already understand what that means when azd init runs?

Having it be part of azd pipeline config would be fine:
"We noticed that you don't have a github workflow file, would you like to add one?"


Vite Server did not start - error when zipping and trying on another pc

Describe the bug
I used pmpm pack:zip
This creates a zip file of the contents of the extension directory.
I copied to another pc and tried to load unpacked in developer mode in chrome.
The extension loaded however when I click on the icon, I get a popup saying vite server did not start.
I also tried this on development workstation and found the same issue.
How best can I zip and share?
thanks
Reproduction
See steps above
System Info
windows

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

ensure you run build first


New line parsing not working correctly in 0.6.0

In 0.5.0, following CLRF formatted string:
settings-restart-ask-message =
    Playnite needs to be restarted in order to apply new settings. Restart now?

    Restarting will cancel any active tasks (downloads) currently in progress.

was returned properly as
Playnite needs to be restarted in order to apply new settings. Restart now?

Restarting will cancel any active tasks (downloads) currently in progress.

but 0.6.0 now returns:
Playnite needs to be restarted in order to apply new settings. Restart now?
Restarting will cancel any active tasks (downloads) currently in progress.

Full file I'm adding as a resource via File.ReadAllText : en_US.zip

@JosefNemec Did 0.6.1 fix the issue?

Works properly with 0.6.1, thanks for the fix!


Move to pion/ice@v2

Removed support for trickle ice

Everyone I added to the PR don't feel a need to review! I wanted to give everyone a heads up that Trickle ICE is now the default. This is a MASSIVE breaking change, so would love peoples input.
This had to happen to make ICE Restarts possible. We need the ICEConnectionFailed state so people can handle things the same way they do in the browser.
After this lands I am going to update the README and start a wiki page about v3 changes. There is going to be a lot more happening https://github.com/pion/webrtc/issues/9

@at-wat w/e you think we are ready to tag pion/ice@v2 then this is ready to merge.

May I suggest starting an “upgrading” doc explaining what people need to do to upgrade? I am happy to help copyedit it if you can get the rough material jotted down.

@josharian done<EMAIL_ADDRESS>

@at-wat I tagged rc-1 for ICE! When this is approved I will push updates to the README.
Will put ICE Restarts in another PR. This is big enough already, but had to be done :)


Chore: update dependencies

Update etcd toolchain to v3.5.0
Other dependencies updates

Thanks for this update, it's really prompt.


RI Recommendations per linked account

Currently not supported in the console, would be great to filter recommendations by linked account

Apologies - but currently this can't really be done as: https://docs.aws.amazon.com/aws-cost-management/latest/APIReference/API_GetReservationPurchaseRecommendation.html only supports an AccountScope attribute of PAYER...
I plan to add the PAYER level RI recommendation, and then hopefully a view of average Instance usage over the month by account id if I can do in minimal calls, which would be a similar view..

Actually I'm not certain, as we can specify AccountId... So let me implement the base functionality, and I will see if looping over child accounts works even with scope PAYER. This will mean that a child account RI report would be 1c x N accounts if it works.