Licensing Required for Editing Entity Store Reports

It is not clear here, nor in the Licensing Guide, what Power BI license is required for editing entity store based reports (those that stay in Dynamics, not the ones from powerbi.com). Do users need a Power BI Pro license to logon to the VM and edit the existing reports (more specifically the datasets)?

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

ID: 630034c6-dbcc-d365-d58b-36bef4277582
Version Independent ID: ce5941a7-67bb-9c0a-c1ab-230fe8aff69f
Content: Power BI Embedded integration - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/analytics/power-bi-embedded-integration.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @TJVass
Microsoft Alias: tjvass

That would also go for creating any NEW entity store based reports that are subsequently uploaded to LCS.

Thanks for the feedback. @TJVass can you please take a look?

BUG 440097 created to track

Hello. Is there an update to the docs that provide this information?

@sericks007, please review.

@RichdiMSFT or @MilindaV2, can one of you answer this question?


chore: updated nuget packages

Issue number:
Summary
Updated dependencies for Powertools libraries and sample projects
Changes

Please provide a summary of what's being changed
Nuget pages updated

User experience

Please share what the user experience looks like before and after this change
No change

Checklist
Please leave checklist items unchecked if they do not apply to your change.

[x] Meets tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Codecov Report
Base: 56.35% // Head: 56.29% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (a7764b1) compared to base (810dd7f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #172      +/-   ##
===========================================
- Coverage    56.35%   56.29%   -0.06%
===========================================
  Files           41       41
  Lines         1778     1778
===========================================
- Hits          1002     1001       -1
- Misses         776      777       +1

Flag
Coverage Δ

unittests
56.29% <ø> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...da.Powertools.Logging/Internal/PowertoolsLogger.cs
90.36% <0.00%> (-0.61%)
:arrow_down:

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

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


Incorrect path on Part 3 Section IV Step 6

The guide says the folder to delete is in Nintendo 3DS/(32 Character ID)/(32 Character ID)/extdata/, but the contents are only a single folder named 00000000. The folder to delete is actually inside the 00000000 folder. (or at least in my case)

Fixed


Conditional tab or conditional tab-headline

Hi there,
is it possible to use the "conditional card" together with the "tabbed card"?
Basically: Could one have a given tab only appear if a given condition is met? Alternatively, I assume one could use conditional cards within a tab. In that case, could one have the tab's headline be formated based on a certain condition? For example: have its background be green if a certain condition is met and red if it's not met.

you can use the conditional card to some degree as shown here:
https://user-images.githubusercontent.com/16073360/210274795-96eb10e6-eff8-4f9a-a1b8-8ca091241ead.mp4
there was already a feature request along these lines for the tabs/tab labels be conditional: https://github.com/kinghat/tabbed-card/discussions/49. you can make suggestions and follow along there.

Thanks @kinghat ! I'll have a look at the video tomorrow. Also added this feature request / suggestion to #49 . I had seen it before posting my "report" but thought mine was sufficiently different to write this new feature request.
Thanks anyways :-)


[Feature] Add supportfor  array data type

Currently array data type are not supported. we need to find best way to support it.
        case "array":
          throw new RuntimeException("'" + fieldName + "' has Array type, Array type not supported!");
          //schemaColumns.add(Types.NestedField.optional(columnId, fieldName, Types.ListType.ofOptional()));
          //break;

implemented by #63


Support passing mongoose specific params to queries.

Similar to this PR https://github.com/feathersjs/feathers-sequelize/pull/27

+1 !
yeah I totally agree that supporting more mongo-specific query operators would be super good, especially for $regex$push as other issues have covered. $regex is crucial in many cases.
In fact, for mongoose and mongo adapters, I suggest that we could just pass the query options to backend database without twisting it. That might be the easiest way to make us able to freely use any query/update operators in the new version of mongodb, such as the logical ones ($and, $nor...), the geolocation ones ($near, $center...) and the array ones ($each, $pull, $push...).

I tested and found basically all mongo query/update operators like $push, $and, $elemMatch... are already available. Amazingly the only operator that's not working is $eq, the most simple one.

@beeplin $eq is supported through our querying mechanism by just passing the object. Most other MongoDB query operators should work (but only with Mongoose and MongoDB) but if they are not part of our querying spec they will probably not work with other database adapters.
This issue will address passing options (like multi: true) to a Mongoose query.

Thanks for the explanation. I understand that mongo's finding options -- sort, limit, skip and fields selection are already supported in all adapters. For mongo's other two writing options, upsert and multi, since feathers' update\patch\remove only allows id as the first parameter, I don't think multi is necessary, but it would be nice to have an $upsert option.

+1 for $upsert support.  I was looking for a way to use it last month.  While the new Wired Tiger engine in MongoDB does make for really amazing, ridiculous performance charts, if I can save a single query to the database, it will potentially have a big pay off... once the app reaches 40 thousand requests per second.  😃

@beeplin Feathers database adapters do support updating multiple resources if you set the id to null (see http://docs.feathersjs.com/services/readme.html#service-methods), e.g. app.service('todos').patch(null, { complete: true }, { query: { complete: false } }) will set all incomplete todos to be completed. That's why it sets the multi option internally here.

oops missed that. Smart design! Thanks~

Also closed via #87

Hi @marshallswain I know it has been long ago, but did you figure out how to use upsert in feathers-mongodb or feathers-mongoose? Recently I am trying to reproduce the behavior of upsert in feathers-mongodb by following @daffl 's suggestion in https://github.com/feathersjs/feathers-mongoose/issues/121, but I found there are some edge cases that can hardly be handled in this way. Maybe it's time to design a special upsert key in params?

Silly thought for upsert. Is there any value to do an optimistic update first, then a create should the update fail?

I am using the update-then-create-if-failed method. It can save a get in most times, and what's more important is that, theoretically, the get-then-update-or-create method may possibly fail in some edge cases. Here is an example:

get id='aaa',  and fails, which means the id 'aaa' does not exist
create id='aaa'
but between 1 and 2 someone else create a new doc with id='aaa', so the create in 2 fails since the id is duplicate.

So at least the get-then-update-or-create method should be improved like get-then-update-or-create-then-update-if-creation-fails.
However, that's still problematical coz it may return wrong results if two users are trying to 'upsert' the same absent id -- we cannot guarantee the final result is determined by the 'upsert' called later  since there are so many async writings to db and it is hard to maintain the proper sequence.
That's why I think an atomic operator like 'upsert' is necessary in this case.

I would think only the DB itself could provide an atomic upsert.

Yes, and mongoDB does have this atomic upsert already -- just pass {upsert: true} as the third parameter when doing update.
The problem here is the feathers-mongodb/feathers-mongoose adapter does not have any interface to call this upsert (maybe for compatibility among different dbs).

Setting params.mongodb or params.mongoose to { upsert: true } doesn't work?

@daffl yeah I just saw this line https://github.com/feathersjs/feathers-mongodb/blob/master/src/index.js#L36. Sorry that I didn't check earlier.

I was having a hard time putting all these notes and docs together so in case anyone else is in the same boat here is the line from above with the upsert included.
app.service('todos').patch(null, { complete: true }, { query: { complete: false }, options: { upsert: true } });

Are you sure this is working? params.options does not seem to be used anywhere (unless you are using a pretty old version). It should be params.mongoose which is documented here.

Oh sorry, I meant to post this in the mongodb version of this issue! (Too many tabs open!)
This would be the proper way to use upsert for mongodb, right?

It should be params.mongodb (although params.options is still supported for backwards compatibility reasons).

So to summarize (for mongoose this time):
app.service('todos').patch(null, { complete: true }, { query: { complete: false }, mongoose: { upsert: true } });


refactor(oci-collector): google/go-containerregistry and cosign libraries to fetch artifacts

Description of the PR
This PR switches from regclient library to google/go-containerregistry (short GGCR) and cosign library to fetch artifacts.
Cosign helps downloading attestations and sboms with their oci package which integrates tightly with GGCR.
To ensure future artifact collection from oci registries, this PR also introduces a Collector interface inside the oci collector package to add more collectors like buildkit attestations mentioned in #1370.
PR Checklist

[x] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[x] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
(not applicable)
[ ] If collectsub protobuf has been changed, make proto has been run
(not applicable)
[ ] All CI checks are passing (tests and formatting)
[x] All dependent PRs have already been merged

Thanks for opening the PR @hown3d! We will take a look at this

thanks @hown3d for the PR! My last PR was around the OCI collector so I'm interested in this work. Could we open a GitHub issue to document and discuss the decision to move away from regclient? It would be great for the community to discuss this as it is a substantial change. Another option that I would like to put up for consideration is oras-go which is a CNCF sandbox project.

thanks @hown3d for the PR! My last PR was around the OCI collector so I'm interested in this work. Could we open a GitHub issue to document and discuss the decision to move away from regclient? It would be great for the community to discuss this as it is a substantial change. Another option that I would like to put up for consideration is oras-go which is a CNCF sandbox project.

Thanks @ridhoq, Agree that we should open an issue to determine the best path forward.

That's a good idea @ridhoq.
I've created an issue regarding the discussion: #1456

@hown3d added a comment to the issue. Please take a look and we can continue the discussion there.


Update QGIS to 2.8.1-1

(Version currently offered at source kyngchaos.com)

Looks good, thanks! :+1:

No problem -- glad to help!


Chromatic

Fixing Storybook stories enough to be able to deploy to Chromatic.
Now deployed to Chromatic.
Next steps

build it with Buildkite(?)
ask Chromatic to mark our instance as open-source
point cardigan.wellcomecollection.org to the Chromatic-hosted instance
~profit~

The shared link in the description 404s for me; has it moved somewhere else?

Good point – updated now (new link)


Merge 2.5 changes to 2.x

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #537 (04666c7) into 2.x (423db21) will increase coverage by 9.92%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x     #537      +/-   ##
==========================================
+ Coverage   49.67%   59.59%   +9.92%
==========================================
  Files         195      256      +61
  Lines        6225     8719    +2494
  Branches      975     1559     +584
==========================================
+ Hits         3092     5196    +2104
- Misses       2910     3144     +234
- Partials      223      379     +156

Impacted Files
Coverage Δ

...oards-plugin/public/pages/Indices/utils/helpers.ts
83.87% <0.00%> (-16.13%)
:arrow_down:

...ublic/pages/Indices/containers/Indices/Indices.tsx
82.50% <0.00%> (-3.62%)
:arrow_down:

...ement-dashboards-plugin/public/pages/Main/Main.tsx
6.32% <0.00%> (-1.08%)
:arrow_down:

...-dashboards-plugin/public/index_management_app.tsx
6.25% <0.00%> (-0.42%)
:arrow_down:

...p/containers/CreateRollupForm/CreateRollupForm.tsx
69.86% <0.00%> (-0.36%)
:arrow_down:

...ndex-management-dashboards-plugin/public/plugin.ts
0.00% <0.00%> (ø)

...ex-management-dashboards-plugin/utils/constants.ts
100.00% <0.00%> (ø)

...gement-dashboards-plugin/public/utils/constants.ts
100.00% <0.00%> (ø)

...ic/components/ContentPanel/ContentPanelActions.tsx
100.00% <0.00%> (ø)

...atePolicy/components/FlyoutFooter/FlyoutFooter.tsx
66.66% <0.00%> (ø)

... and 73 more

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

Code review has been done in below PRs for easy reading, Common component: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/508
Index creation/update/delete: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/509
split index: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/512
template management: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/511
alias management: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/510
Reindex: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/513
Shrink index/open/close: https://github.com/opensearch-project/index-management-dashboards-plugin/pull/516


DEV: Add DictionaryAuthenticator.

Adds an authenticator that accepts a pre-defined dictionary from user -> hashed password.  This is useful for defining user lists that aren't tied directly to unix users.

this does not merge cleanly anymore, and seem to have stalled.
I'm going to close, but fell free to reopen.


Fix some formatting

The formatting around Dockerfile was a little wonky.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks good. Thanks Phil!


Added SwaggerParameterAttribute and related filter

Added a new swaggerparameterattribute and related operationfilter to the annotations (and related tests)
Made one of the existing ApiDescriptor extension methods public as it was "nice to have" for the filter.

Sorry for the delay - anyway i went ahead and changed the implementation so it resembles the xml comment filter a bit more (not entirely though). I think this also fixes the above mentioned issue with parameter names not being matched properly.


Update README.md

Updated the read me conform newer standard due to documentation not being correct.

Thank you for reaching out. @Slyvan25
Could you please be more specific about the change you are recommending in the readme file?
Thanks.

The current documentation tells users to use the text prop, but the text prop doesn't exist anymore due to the widget expecting a string instead of
text: "this is a string"

Hi, @Slyvan25
Thank you for your contribution. Good find. 👍🏻


Correction of discrepancies for gte-Qweb model

#1600

Checklist

[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.

@KennethEnevoldsen, I'm not sure if a rerun is required. The current results from MTEB are almost identical to the leaderboard. It is unlikely that the model will change its position in the leaderboard after the rerun. Recalculating the scores makes sense if we want the implementation in MTEB to match the leaderboard 100%

alright then I would just leave it.


bug: diff not possible if left dict is empty

see this line
suggested approach:

when both dicts are empty just return empty results
when no common keys is there:

try to check if we at the lowest level by checking a type of a dict element from left or right side, whichever works:

yes - raise error, it looks like we are comparing to different data types
no - common keys are used only to go down one level here, simply return an empty set so that the loop that loop based on keys_to_check ends

Merged with #77


detect pending PVCs

see how to check if storage class is valid or any other info in the status?

solved in #49


Note's duration is too short.

When I export ABC from the attached midi using track "Strings 1", I get the error "Note's duration is too short." in lotro (Lonely Mountain Fiddle).
I did not transpose the track or change the tempo or meter.
When playing in ABC Player it plays fine.
alw-M.zip

I did some tests of this. It looks like the shortest note LOTRO will play is:
0.125 seconds
So this will work:
Q: 60
C1/8
This wont work:
Q: 60
C1/16
Any chance Maestro could somehow adapt notes/rests shorter than 0.125s into longer notes/rests?

Actually 1/16 at Q:60 works.
The shortest note possible in LOTRO is 0.06 seconds ( /8 at 125 bpm )
The longest note is 8 seconds ( 8 at 60 bpm )
Tried and tested for years. :)

I just retested and you are wrong. (At least for the 64bit client)
Q: 60
E1/16
Do not work.

Try this file, can you play it? I cannot..
%abc-2.1
%%song-title       cit
%%song-composer    dp
%%song-duration    1
%%song-transcriber al
%%abc-creator myself
%%abc-version 2.1

Z: al
C: dp
X: 1
T: part-1
M: 4/4
Q: 60
K: C maj

	E1/16

Oh, that's interesting! It wouldn't play as is, but when I added an extra line in the field to specify L: 1/4 it played fine.
L stands for the unit note length, and in the above case it's specifying one beat a quarter note.
It's usually how it is by default, so I'm not sure why specifying it makes a difference. Hmm.

Thats interesting!
I also just found out that it will not complain about note length being more than 8 seconds unless L:1/4 is specified.

@digero I guess to solve this issue for Maestro then it would need to:

Include L: in the ABC
Stop making rests longer than 8 seconds. Split it into smaller rests. Like two z240/59 instead of one z480/59.

Ah, we made a mistake. The default for 4/4 is not L: 1/4, its L: 1/8.


Allow customize DefaultThreadsPerController value

Expected Behavior
The DefaultThreadsPerController value can be customized through passing flags like  #1700 suggested or environment variable.
Actual Behavior
https://github.com/knative/pkg/blob/master/controller/controller.go#L53
var (
	// DefaultThreadsPerController is the number of threads to use
	// when processing the controller's workqueue.  Controller binaries
	// may adjust this process-wide default.  For finer control, invoke
	// Run on the controller directly.
	DefaultThreadsPerController = 2
)

Currently, this value is fixed and can not be modified without changing the code.

/kind feature

/area API

It seems we had that before, but this seems reasonable in general.

/assign
I'll try to look at it.

@vagababov thank you for looking at this.
I see now the QPS and burst value can be modified by changing kubeconfig file, but it will make these values have influence on every containers within cluster.
We have a shared cluster and just don't want to influence others, is it reasonable to customize the value using env variable, or by passing flags in deployment yaml : containers. args ?

/remove-lifecycle stale

/unassign

Is there any progress on this issue?

Is there any progress on this issue?

no progress I think, since no current assignee


Fix code coverage

Also remove obsolete uses of the Hosted variable

Something seems to have gone weird with the codecov upload. I don't see the results anywhere.

Ya, this PR doesn't actually fix it ... yet.


[ROCKETMQ-249] Do not attempt to clear disk even if disk store path i…

…s found
https://issues.apache.org/jira/browse/ROCKETMQ-249

Coverage increased (+0.1%) to 38.836% when pulling 7b587210ff691fdaaecfe1a85f9fb3cc33b98ce2 on shroman:ROCKETMQ-249 into 118bdec96005ce695295bcf0d9082e0230e69bf7 on apache:develop.

Coverage increased (+0.1%) to 38.836% when pulling 7b587210ff691fdaaecfe1a85f9fb3cc33b98ce2 on shroman:ROCKETMQ-249 into 118bdec96005ce695295bcf0d9082e0230e69bf7 on apache:develop.

Coverage increased (+0.1%) to 38.836% when pulling 7b587210ff691fdaaecfe1a85f9fb3cc33b98ce2 on shroman:ROCKETMQ-249 into 118bdec96005ce695295bcf0d9082e0230e69bf7 on apache:develop.

Coverage increased (+0.1%) to 38.817% when pulling a4e9eb84f7ae4a68be59aae3e38b45c4bb595444 on shroman:ROCKETMQ-249 into 118bdec96005ce695295bcf0d9082e0230e69bf7 on apache:develop.

Coverage increased (+0.1%) to 38.817% when pulling a4e9eb84f7ae4a68be59aae3e38b45c4bb595444 on shroman:ROCKETMQ-249 into 118bdec96005ce695295bcf0d9082e0230e69bf7 on apache:develop.

Coverage increased (+0.1%) to 38.817% when pulling a4e9eb84f7ae4a68be59aae3e38b45c4bb595444 on shroman:ROCKETMQ-249 into 118bdec96005ce695295bcf0d9082e0230e69bf7 on apache:develop.

@vongosling @zhouxinyu @lizhanhui @vsair Anyone's willing to review the PR?

It seems that broker can't work properly if disk store path is not found?

@zhouxinyu The problem is described in the JIRA issue -- "If disk usage estimates fail because the disk store path is not found (doesn't exist), message store falsely decides the disk is full and tries to delete."
No failure, just a wasteful operation. I corrected the execution path, plus wrote the unit tests for it.

@dongeforever Modified as you suggested, thanks!

Coverage increased (+0.3%) to 38.964% when pulling ba0348b972cbab3f7c4f296039c85c27c5c1c1e2 on shroman:ROCKETMQ-249 into 845830865fc37d0364a19cbd89ceaf8a30b37e1c on apache:develop.

@shroman I will close the pr, if you happened to the same question, please let me know.


Make BTC the default deposit/withdrawal type on OpenLedger gateway

Currently the default deposit type is AGRS, but AGRS addresses are the same format as BTC addresses, so it's a common error for users to accidentally send BTC to an AGRS deposit address. The best solution is probably to make BTC the initial default for depositing coins.

Better fork the project and make new modifications there.

I've assigned this issue to Adam. Adam, Matta can show you where the code for this page is, he's worked on this code in the past.

Pretty sure this is fixed now, closing.


Error while launching scanner acivitiy which contains CompoundBarcodeView

Description of the problem:
We are using this following code to open QR code scanning screen
IntentIntegrator _intent = new IntentIntegrator(this).addExtra(Intents.Scan.CHARACTER_SET, "ISO-8859-1");
_intent.setCaptureActivity(ScannerActitivity.class);
_intent.initiateScan();
& inside ScannerAcitivity
barcodeScannerView = (CompoundBarcodeView) findViewById(R.id.zxing_barcode_scanner);
capture = new CaptureManager(this, barcodeScannerView);
capture.initializeFromIntent(getIntent(), savedInstanceState);
capture.decode();
we facing this error -(refer below image URL)
It won't allow accessing the camera of the device.
https://imgur.com/a/cPnZuud
Kindly help us with any solution.
Which library version are you using? E.g. com.journeyapps:zxing-android-embedded:3.4.0
Which phone/tablet are you using, and which Android version does it run? (e.g. Samsung Galaxy S5,
Android 5.0) Specific to Android 5.0+ (It works on 6.0)
Does the same happen on other devices or an emulator?
Can you reproduce the issue in the sample project included with the library? If not, can you
provide your own sample project or sample code that produces this error?
In the case of an error do you have a stack trace or adb logs?
Failed to open camera
java.lang.RuntimeException: Fail to connect to camera service
at android.hardware.Camera.(Camera.java:637)
at android.hardware.Camera.open(Camera.java:371)
at com.google.zxing.client.android.camera.open.OpenCameraInterface.open(OpenCameraInterface.java:83)
at com.journeyapps.barcodescanner.camera.CameraManager.open(CameraManager.java:133)
at com.journeyapps.barcodescanner.camera.CameraInstance$3.run(CameraInstance.java:181)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:135)
at android.os.HandlerThread.run(HandlerThread.java:61)

Does the same happen on those devices if you use the sample application? Or is it specific to your application? (Download the sample app here: https://github.com/journeyapps/zxing-android-embedded/releases)


Update sbt-scalafix, scalafix-core, ... to 0.9.21

Updates

ch.epfl.scala:sbt-scalafix
ch.epfl.scala:scalafix-core
ch.epfl.scala:scalafix-testkit

from 0.9.19 to 0.9.21.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala" } ]

labels: library-update, semver-patch

This looks like a backport candidate.


[UI] Alerting - Better message for expressions that do not return graph data

Version: master-head (v2.3) (9/9/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Enable alerting
Edit the "A set of alerts for node" alert rule group.
Look at the node disk rules

Result:
There is no data returned in Prometheus so the UI shows a message "No metric graph data."
Expected Result:
We expected the UI to either show this in a more friendly manner (no red text making it look like there is an error) or to show something on the chart (e.g. we could render the chart but show a message on the chart itself like "No metric graph data."
Additional Comments:
This isn't really a big deal -- the UI is doing what it should. But to a new user it may look like their expression is not set up correctly because of the red text. It threw me off at first -- I had thought the expressions were wrong but they are not - there is just simply no data to show.
I propose that at a minimum we make the text no longer red.

This bug is reproduced in rancher\rancher:v2.3-head version b7e6f7b52.
Create a cluster.
Enable monitoring.
From Global > Tools > Alerts
Edit the "A set of alerts for node" alert rule group.

The bug fix is validated in rancher\rancher:master-head version.
Message does not show up.


multi console rancher interface feature request

I want to contribute to rancher with a multi-console interface, will show essential metrics, filter console feed, alerts, etc.
I don't know if this feature is planned in the roadmap or community will like it, but I'm willing to make it happen if there is an interest for it.

This would be very useful!  👍


Fixes #116

Collapses sibling container when removing element from DOM.

Landed in https://github.com/angular-ui/ui-layout/commit/298ff5bfb03b41332a6220a223607b434e90ae58
Welcome to the contributors list!


Cannot make it work in Oracle Cloud VM

I have Wireguard VPN set up in Oracle Cloud VM (Ubuntu 18.04). Wireguard client (Windows) connects, but there is no internet. 0 bytes received, hundreds sent every other second.
Here are the things I've done and tried:

Created and updated/upgraded Ubuntu VM
Executed Angristan's scripts.
Added the port number assigned by Angristan's script in Ingress Rules (firewall)
Tried to check/uncheck "Skip source/destination check" in VNIC setup
Tried to set MTU size to 1360 (the default for wg0 was 8920) and added it in client config
(Update) Tried to open the firewall another way "sudo ufw allow 54321/upd", I have no idea why I should do this, but still no result.

One thing I didn't understand in the process is why "net.ipv4.ip_forward=1" is still commented out inside "/etc/sysctl.conf" file after executing Angristan's script. Isn't it supposed to be enabled? Anyway, I tried to manually enable it in the file, still no internet.
I'm not good with Linux, Wireguard and networking stuff, so I'm hoping to get some help here on how to troubleshoot this issue.

Here is "/etc/wireguard/wg0.conf" file content:
[Interface]
Address = <IP_ADDRESS>/24,fd42:42:42::1/64
ListenPort = 54321
PrivateKey = PRIVATE_KEY
PostUp = iptables -A FORWARD -i ens3 -o wg0 -j ACCEPT; iptables -A FORWARD -i wg0 -j ACCEPT; iptables -t nat -A POSTROUTING -o ens3 -j MASQUERADE; ip6tables -A FORWARD -i wg0 -j ACCEPT; ip6tables -t nat -A POSTROUTING -o ens3 -j MASQUERADE
PostDown = iptables -D FORWARD -i ens3 -o wg0 -j ACCEPT; iptables -D FORWARD -i wg0 -j ACCEPT; iptables -t nat -D POSTROUTING -o ens3 -j MASQUERADE; ip6tables -D FORWARD -i wg0 -j ACCEPT; ip6tables -t nat -D POSTROUTING -o ens3 -j MASQUERADE
Client WireguardClient1
[Peer]
PublicKey = PUBLIC_KEY
PresharedKey = PRESHARED_KEY
AllowedIPs = <IP_ADDRESS>/32,fd42:42:42::2/128

Here is the Wireguard Windows client config:
[Interface]
PrivateKey = PRIVATE_KEY
Address = <IP_ADDRESS>/32, fd42:42:42::2/128
DNS = <IP_ADDRESS>, <IP_ADDRESS>
MTU = 1360
[Peer]
PublicKey = PUBLIC_KEY
PresharedKey = PRESHARED_KEY
AllowedIPs = <IP_ADDRESS>/0, ::/0
Endpoint = <IP_ADDRESS>3:54321

Hello,
I am also facing the issue in setting up the Wireguard VPN on Oracle Cloud VM.
I followed the instruction and created the VPS and their clients but not working for me here are the details:
===========================================
interface: wg0
public key:
private key: (hidden)
listening port: 64399
peer:
preshared key: (hidden)
allowed ips: <IP_ADDRESS>/32, fd42:42:42::2/128
peer:
preshared key: (hidden)
allowed ips: <IP_ADDRESS>/32, fd42:42:42::3/128
====================
This is the Desktop client conf:
[Interface]
PrivateKey =
Address = <IP_ADDRESS>/32, fd42:42:42::2/128
DNS = <IP_ADDRESS>, <IP_ADDRESS>
[Peer]
PublicKey =
PresharedKey =
AllowedIPs = <IP_ADDRESS>/0, ::/0
Endpoint = <IP_ADDRESS>:64399
====
Do you think it has something to do with the Firewall?

Oracle cloud has a firewall that has to be configured in their portal.
Once you allow it, you should be able to use it just fine.
I'm able to run it fine on Ubuntu 20.04.

@FireMasterK I have opened ports for Wireguard by adding ingress rules. Is there anything else to be done to configure the firewall on the Oracle side? By the way, I've also tried it with Ubuntu 20.04, same issue.

The script works perfectly for me like 6 months ago but not work for my new instances on Oracle cloud VM recently.
Allowing the udp port on Oracle dashboard is not the only work to make it works.
After adding the accept rule for udp port into iptables, you will see your client device start receiving and sending data with the VM.
But your client is still not able to access Internet.
It seems there is a prebuilt FORWARD rule on Oracle VM blocking the traffic.
The generated config /etc/wireguard/wg0.conf is using "-A" to append the rules to the bottom of FORWARD chain, you may modify it using "-I" instead so it will add the rules to the top, and after it, run "sudo wg-quick down/up wg0" for resetting the iptables.
It should work then. At least it works for me.


Add separate optional "email subject" and "email body" fields

This is an issue for sites which expect account deletion request emails to be formatted in a certain way. This is already an issue for Zynga, at minimum. (I believe some other entries mention specific verbiage to use in contact emails as well)

Sounds doable, but I wonder how many people will know how to use them

It would admittedly be a low-use feature, but currently there's no way to provide an alternative subject/body for the email link.

I'm ok with the idea, if you're up for implementing it.


Error while trying to download extracted files

Here's how I'm having the bug.
I'm just creating a simple zip file, not protected by any passwords and submiting it to AL with the service "extract" checked.
The files in the archive are being extracted but when i click the button "Download File" after clicking on an extracted file an error occurs.
{"api_error_message":"File 29cb482e07497a5b2d806df9a2b242440c189ed55086836762f4010f2cbaed69 is not associated to submission 4iLG6DPeArmCnmmLKvWpwK.","api_response":{},"api_server_version":"4.0.0","api_status_code":403}

Can you tell us more?

Is on docker or kubernetes
Are you using the default filestore settings
is this happening for all extracted files, or only ones from extract with this context

Hello !
It was on Docker, i didn't change any settings and I tried creating a simple service that basically just add a file with add_extracted(), the same problem happened

With a totally new install, the problem does not seems to be there anymore.


Additions

I'm not sure if this is too detailed and too specific for Python. But maybe there could be a section for Python in the guidelines?

I'll update and make them better.


"SCRIPT0: secure random number generation not supported by this browser use chrome, FireFox or Internet Explorer 11"

Hi, I am using webpack and its loading crypto-browserify 3.11,
when using IE explorer or safari on mobile it throws the error in the title.
did this happen to anyone else? and if yes, what was the fix?

if you could answer the questions I asked over at crypto-browserify/randombytes#6 that would go a long way towards figuring this out

Closing in favour of https://github.com/crypto-browserify/randombytes/issues/6


Use SmallVec for binding references

(Checking to see benchmarks.)

Do we have any evidence that this improves performance? if not, then using a vec is fine.

(Need to find a free moment to benchmark.)

Closing until I have time to benchmark this.


Commit f272517 is a breaking change to 0.33.0 (Update MD024/no-duplicate-heading to remove duplicate parameter allow…)

This change removed allow_different_nesting as an allowed parameter of MD024.
This is a logic change that not only deprecates allow_different_nesting, but also changes the underlying logic. This has recently caused issues for our team as we had our configuration set to:
MD024:
  siblings_only: false
  allow_different_nesting: true

This logic change breaks backwards compatibility, and as such should institute a semver major version change.
The behaviour should be patched on existing 0.x.x releases, and either be reverted on main or induce a major version change.

I'm sorry for the inconvenience, but the change you link to does not represent a change of behavior for the rule, merely the removal of one of two redundant names for a parameter. The remaining parameter name behaves the same as it did before the change. Your scenario should work the same as before once you set siblings_only to true.
For what it's worth, semantic versioning rule 7 (that you link to) does not apply here. Rule number 4 is the relevant one and allows changes such as this one.

I definitely do not agree that this doesn't represent a breaking change in behaviour. If in 0.32.0, I have a perfectly valid configuration file and everything works fine, but upon upgrading to 0.33.0, suddenly markdownlint starts erroring on docs that it previously didn't, how could that be anything other than a change in behaviour?
However, as you mentioned (and as I forgot 🤦), 0.x.x is an experimental version and doesn't guarentee stable API, so I'll close this ticket and freeze our version of markdownlint that we use.


[WIP] Fix findDOMNode error

A WIP fix for the error that is thrown during classification.
Attempted to test it out on PFE using npm link and it's throwing compiling errors that I'm still working on:

I'm going to come back to this after fixing the refs.


Custom paths not working

When setting custom paths they don't seem to work, I think it's because of AOT but I'm not sure. It breaks both when I build the project and when I test it. I use the same paths in my CLI project and it works fine so I guess there's some config missing in this project but I haven't been able to figure out what.
I use this tsconfig:
{
  "compilerOptions": {
    "baseUrl": ".",
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "noEmit": true,
    "sourceMap": true,
    "declaration": false,
    "noImplicitAny": false,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "lib": [
      "es2015",
      "dom"
    ],
    "typeRoots" : [
      "./node_modules/@types"
    ],
    "types": [
      "jasmine",
      "karma",
      "node",
      "webpack",
      "webpack-env"
    ],
    "paths": {
      "@shared/*": ["src/*"],
      "@components/*": ["src/components/*"],
      "@directives/*": ["src/directives/*"],
      "@modules/*": ["src/modules/*"],
      "@pipes/*": ["src/pipes/*"],
      "@validators/*": ["src/validators/*"]
    }
  },
  "include": [
    "src/**/*.ts"
  ],
  "exclude": [
    "src/**/*.spec.ts",
    "node_modules",
    "demo",
    "dist"
  ]
}

@chrillewoodz I'm not using custom paths there and all's fine. Why do you need them?


Added support for nested @each iteration

Fixes https://github.com/outpunk/postcss-each/issues/6

Great, thank you!


Unable to see Trace Id in Access Logs when request is rate limited (429)

%REQ(traceparent)% is not working when request is rate limited
cc @wbpcode

Yeah. We can extend the substitution formatter to output the trace id and span id (downstream request span id) from the active span directly. As far as I know, the active span is part of the HttpFormatterContext now?

ok, will add a new formatter %TRACE_ID%.


Add ScottRooneys file

Commit this slide SR - added a second slide

Merging back to main

Merging 2

deleted branch after successful merge


clang 10.0: error: unknown warning group '-Wsuggest-destructor-override' #145

[x] I've read CONTRIBUTING.md
[x] I've rebased my changes against the current HEAD of origin/master (if necessary)
[ ] I've added new test cases to verify my change
[ ] I've regenerated toml.hpp (how-to)
[ ] I've updated any affected documentation
[ ] I've rebuilt and run the tests with at least one of:

[ ] Clang 6 or higher
[ ] GCC 7 or higher
[ ] MSVC 19.20 (Visual Studio 2019) or higher

[ ] I've added my name to the list of contributors in README.md

Hi, thanks for the contribution! Looks good to me, but before I can merge you'll need to also regenerate the single-header version of the library (see CONTRIBUTING.md for info).

Single header toml.hpp generated and provided!

Also I can confirm that now toml++ module compiles fine on all latest major platforms in github-actions:

Also provided a fix for MinGW platform

MinGW compiles fine as well

Thanks!


chore: release 1.0.10

📦 Release PR
1.0.10 (2024-01-05)
🪲 Bug Fixes

added missing exports to use interfaces etc in other packages (1ed064d)

🤖 This pull request was automatically generated. To publish the version, please merge the pull request.

:robot: Release is at https://github.com/Vertretungsapp/indiware-api/releases/tag/v1.0.10 :sunflower:


DropdownMenu overflow with large helper text

Steps to reproduce
Use DropdownMenu with large helper text.
The size and helper text which causes helper text to crop is tricky and need magic number
Expected results
input region ("hint text") should be able to tapped to select.
helper text should not be cropped.
Actual results
input region cannot be able to tapped to select.
text is cropped
Code sample
Code sample
import "package:flutter/material.dart";

void main() => runApp(const MyMenuApp());

class MyMenuApp extends StatelessWidget {
  const MyMenuApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: SizedBox(
          width: 327,
          height: 658,
          child: Center(
            child: Container(
              color: Colors.purple,
              child: DropdownMenu<int>(
                hintText: 'Hint text',
                helperText: 'This is a long helper text that need 3 lines',
                inputDecorationTheme: InputDecorationTheme(
                    helperMaxLines: 3, helperStyle: TextStyle(fontSize: 30)),
                dropdownMenuEntries: <DropdownMenuEntry<int>>[
                  DropdownMenuEntry<int>(
                    value: 0,
                    label: 'MenuEntry Item 1',
                  ),
                ],
              ),
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel master, 3.22.0-14.0.pre.67, on macOS 14.4 23E214 darwin-arm64, locale en-AU)
    • Flutter version 3.22.0-14.0.pre.67 on channel master at /Users/mainj/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1a905d508d (30 hours ago), 2024-04-21 06:44:23 -0400
    • Engine revision 75ca2195c9
    • Dart version 3.5.0 (build 3.5.0-83.0.dev)
    • DevTools version 2.35.0-dev.8
....

Reproducible using the code sample provided above.
https://github.com/flutter/flutter/assets/88313112/7f4397ed-9153-4f47-9fcc-8cfbf0daf96e

flutter doctor -v
[✓] Flutter (Channel stable, 3.19.6, on macOS 14.4.1 23E224 darwin-arm64, locale en-GB)
    • Flutter version 3.19.6 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 54e66469a9 (5 days ago), 2024-04-17 13:08:03 -0700
    • Engine revision c4cd48e186
    • Dart version 3.3.4
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Users/nexus/Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode-15.3.0.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Users/nexus/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.5)
    • IntelliJ at /Users/nexus/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 77.2.2
    • Dart plugin version 232.10286

[✓] VS Code (version 1.88.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.86.0

[✓] Connected device (5 available)
    • Pixel 7 (mobile)     • adb-28291FDH2001SA-5Lv71w._adb-tls-connect._tcp. • android-arm64  • Android 14 (API 34)
    • Nexus (mobile)       • 00008020-001875E83A38002E                        • ios            • iOS 17.4.1 21E236
    • Dean’s iPad (mobile) • 00008103-000825C811E3401E                        • ios            • iOS 17.4.1 21E236
    • macOS (desktop)      • macos                                            • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)         • chrome                                           • web-javascript • Google Chrome 124.0.6367.62

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.22.0-15.0.pre.21, on macOS 14.4.1 23E224 darwin-arm64, locale en-GB)
    • Flutter version 3.22.0-15.0.pre.21 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1be28aa61a (2 hours ago), 2024-04-23 13:26:09 +0800
    • Engine revision 9c0d24ff1c
    • Dart version 3.5.0 (build 3.5.0-88.0.dev)
    • DevTools version 2.35.0-dev.8
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Users/nexus/Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode-15.3.0.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Users/nexus/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.5)
    • IntelliJ at /Users/nexus/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 77.2.2
    • Dart plugin version 232.10286

[✓] VS Code (version 1.88.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.86.0

[✓] Connected device (6 available)
    • Pixel 7 (mobile)                • adb-28291FDH2001SA-5Lv71w._adb-tls-connect._tcp. • android-arm64  • Android 14 (API 34)
    • Nexus (mobile)                  • 00008020-001875E83A38002E                        • ios            • iOS 17.4.1 21E236
    • Dean’s iPad (mobile)            • 00008103-000825C811E3401E                        • ios            • iOS 17.4.1 21E236
    • macOS (desktop)                 • macos                                            • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                            • darwin         • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)                    • chrome                                           • web-javascript • Google Chrome 124.0.6367.62

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.


SelectBox doesn't close

VSCode Version: code-oss-dev 1.20.0 (a83a46d9a7356e955ed9a3dcc75193040588d35c)
VSCode Version: Code - Insiders 1.20.0-insider (f9115349ef09cfef2bed669680d10ac58b490dce, 2017-12-21T16:36:58.030Z)
OS Version: Windows_NT x64 10.0.16299
Extensions: none

Steps to Reproduce:

Open OUTPUT  or TERMINAL view
Show SelectBox
Close the Panel
bug

I used Code-OSS because Insider has not been updated for over a week

I can reproduce it with the same version.

@EbXpJ6bp
@maxibanki
Nice catch, and thanks for reproducing !
I have been able to reproduce this, no blur event is fired on panel close which is usually used to close the drop-down.  I have a fix will post soon.

Fixed with: #40974


Safari 17.2: Fullscreen API on iOS & relaxed CSS Nesting

Via https://github.com/Fyrd/caniuse/issues/6873 & https://developer.apple.com/documentation/safari-release-notes/safari-17_2-release-notes

Added support for the Fullscreen API on iOS. (58946263)

Added new relaxed parsing behavior for CSS Nesting. (113475843)

Fixes https://github.com/Fyrd/caniuse/issues/6873.
I think those two are actually all that are required for caniuse-entries 🤔

Thanks!


Create hloworld.py

I create a new python file for my understanding.

a pull request from my friend


rune/libenclave/skeleton: interactive with ra-tls-server

Signed-off-by: Shirong Hao<EMAIL_ADDRESS>

This is not a RFC any more from now on. Plz associate this pr with https://github.com/alibaba/inclavare-containers/issues/229.

This is not a RFC any more from now on. Plz associate this pr with https://github.com/alibaba/inclavare-containers/issues/229.

@haosanzi Looks like no CI/CD pass.

@haosanzi Looks like no CI/CD pass.


Upgrade to Pro button in frappe top bar - redirect to an upgrade page in react

Feature Request: Redirect Functionality for 'Upgrade to Pro'
As part of enhancing the user journey within our platform, we need to establish a redirect for the 'Upgrade to Pro' button located in the Frappe top bar. The button should lead users directly to a dedicated upgrade page within our React-based dashboard.
Current Behavior
Currently, the 'Upgrade to Pro' button does not redirect users to the intended upgrade page, which is likely affecting our conversion rates for upgrades.
Expected Behavior
Upon clicking the 'Upgrade to Pro' button, users should be taken to an upgrade page on our React dashboard where they can view the benefits of upgrading and complete the process.
Goals

To streamline the upgrade process for users.
To improve the visibility and accessibility of the upgrade options.
To increase the conversion rate for users upgrading to the Pro version.

Action Items

Implement the redirect functionality for the 'Upgrade to Pro' button.
Ensure the destination page clearly communicates the benefits and process of upgrading.
Test the redirect to ensure it works seamlessly across various browsers and devices.

Importance
This update is critical for providing a clear path for users considering an upgrade, which is an essential step in our funnel for revenue growth.
Request for Action
Please prioritize this task as it directly impacts our business model and user satisfaction.
Thank you for your prompt attention to this request.

I have changed the link when clicking the 'Upgrade to Pro' button on the top bar to https://zaviago-dashboard.vercel.app/dashboard/compare-plan, please check whether I set the correct link or not. Thank you.

Review close


Tambah kondisi production order manager

Menambahkan kondisi di production order manager pada function get report

Coverage decreased (-0.003%) to 75.555% when pulling 200af280cbd3270ce5498e24d178e1b2c7e90ff2 on baguswidypriyono:dev into 1ad0f477569e66ddcf5d8b4c17b70e84c9f06443 on danliris:dev.

Coverage decreased (-0.003%) to 75.555% when pulling 3714b8a3c3e0f1f03d135bd7dafdb6abde649ead on baguswidypriyono:dev into 1ad0f477569e66ddcf5d8b4c17b70e84c9f06443 on danliris:dev.

Coverage decreased (-0.003%) to 75.555% when pulling 3354e8afa362c5939575cac40a0d37a92d7236e1 on baguswidypriyono:dev into 1ad0f477569e66ddcf5d8b4c17b70e84c9f06443 on danliris:dev.

Coverage decreased (-0.003%) to 75.555% when pulling e2b78477f80a4f99c3ac10717ffbe68bf81776a0 on baguswidypriyono:dev into 1ad0f477569e66ddcf5d8b4c17b70e84c9f06443 on danliris:dev.

Coverage decreased (-0.003%) to 75.555% when pulling 8b8b72f2613faaa8333159ec7c3efd932734ca46 on baguswidypriyono:dev into 1ad0f477569e66ddcf5d8b4c17b70e84c9f06443 on danliris:dev.


CITE-seq with inDrop V3

Hi All,
We generated CITE-seq data for 4 samples using the inDrop V3 chemistry. I have the following question: How to run CITE-seq-count on demultiplexed fastq files from our experiment. Using the raw BCL files, I am demultiplexing using a standard pipeline to process inDrop data (https://github.com/indrops/indrops). After demultiplexing, I get reads R1-R4, where

R1: biological read (Antibody or CITE-seq barcode)
R2: first half of the gel barcode
R3: library index
R4: second half of the gel barcode, the UMI, and a fraction of the polyA tail

I wonder how I can obtain protein expression as I have part of the cell barcode in R2 (8bp) and another part (8bp) in R4 while the schematics in the Cite-seq-Count documentation seems to expect the cell barcode + UMI in read 1. Moreover, how can I estimate protein expression in each of my samples? What am I missing?
I used this command to check how the output looks like but it is taking a really long time to finish (2 days running so far, so I am guessing I am doing something wrong).
CITE-seq-Count -R1 read1/Undetermined_S0_R4_001.fastq.gz -R2 read2/Undetermined_S0_R1_001.fastq.gz -t tags.csv -cbf 1 -cbl 8 -umif 9 -umil 14 -cells 10000 -o citecounts
I very much appreciate your help!
Thanks!

Hi,
I am so sorry to ask again but can anybody please provide me some advice?
Thanks!

Hello @edroaldo,
I would need to see some reads to help you out. The structure that you listed above is definitely not compatible directly with CSC.
Would you mind sending me 1000 reads from each file so that I can try to figure out a solution for you?

Hi @Hoohm ,
Thank you so much for help! Please, find in the following link the information is asked for. This is typically what I get after demultiplexing the raw BCL files for a inDrop V3 run.
https://drive.google.com/open?id=1aoyPaKU5J-Di-fTIQyhZiGoXsB9_tvv1
Thank you much!

Hi @Hoohm ,
Just checking if by any chance you had the time to take a look at the files above.
Thank you so much!

Hey @edroaldo I've looked a bit into it.
Here is what is needed.
For this read: @NB551687:153:HL3GHBGXC:1:11101:20020:1055
R1: ACCCTACTTCTAACCTCCCTGAANAAANAAAAAAANAAAAAAAATAGGGAGTNGGGGGGGG
R2: GACACCTG
R3: Not needed
R4: TCCCGNNNNNNNNN
R1 is our regular R2, nothing to do there. Just use your R1 files as input for R2 in CSC.
R2 and R4 combined, is our usual R1.
So, for this particular read, R1 would hold the concatenation of R2 and R4: GACACCTG TCCCGNNN NNNNNN
I'm assuming the gel barcode is GACACCTGTCCCGNNN, 16 bp, and the umi NNNNNN, 6bp
The easiest way to deal with this is to concatenate each line of R2 and R4.
So using paste -d"\0" R2.txt R4.txt > catted.txt and use this as an input for R1 in CSC.
Let me know if you need more help.

Fantastic! Thank you so much for your help! Truly appreciate it!
On Fri, Feb 14, 2020 at 7:49 AM Patrick Roelli<EMAIL_ADDRESS>wrote:

Hey @edroaldo https://github.com/edroaldo I've looked a bit into it.
Here is what is needed.
For this read: @NB551687:153:HL3GHBGXC:1:11101:20020:1055
R1: ACCCTACTTCTAACCTCCCTGAANAAANAAAAAAANAAAAAAAATAGGGAGTNGGGGGGGG
R2: GACACCTG
R3: Not needed
R4: TCCCGNNNNNNNNN
R1 is our regular R2, nothing to do there. Just use your R1 files as input
for R2 in CSC.
R2 and R4 combined, is our usual R1.
So, for this particular read, R1 would hold the concatenation of R2 and
R4: GACACCTG TCCCGNNN NNNNNN
I'm assuming the gel barcode is GACACCTGTCCCGNNN, 16 bp, and the umi
NNNNNN, 6bp
The easiest way to deal with this is to concatenate each line of R2 and R4.
So using paste -d"\0" R2.txt R4.txt > catted.txt and use this as an input
for R1 in CSC.
Let me know if you need more help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Hoohm/CITE-seq-Count/issues/108?email_source=notifications&email_token=ACNJIR6YSORG3A6F4XCZY7DRC2HNRA5CNFSM4KOQJKW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELY5MMA#issuecomment-586274352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNJIR6PM3DCQBKFWGE37A3RC2HNRANCNFSM4KOQJKWQ
.

--
Edroaldo

Hi @Hoohm
I am getting this error message while running the following command:
I combined the L001-L004 fastq files into a single one for each read. After combing reads R2 and R4 reads with paste -d"\0" Undetermined_S0_R2_001.fastq Undetermined_S0_R4_001.fastq > Undetermined_S0_CS_R1_001.fastq
Unequal number of read1 (40) and read2(37) files provided
What does this error mean? I have 4 samples.  I am trying to troubleshoot...
CITE-seq-Count -R1 read1/Undetermined_S0_CS_R1_001.fastq.gz -R2 read2/Undetermined_S0_R1_001.fastq.gz -t tags.csv -cbf 1 -cbl 16 -umif 17 -umil 22 -cells 6000 -o citecounts
Thanks!

I used bcl2fastq with --mask-short-adapter-reads 0 --minimum-trimmed-read-length 0. Could that be a reason?
Thanks!

Have you provided a samplesheet to bcl2fast for the demultiplexing?

Yes. I also tried with a samplesheet over the weekend. I was able to get the fastq files for each sample, repeated the steps to combine reads R2 and R4 with paste but still get a similar error message as before:
Unequal number of read1 (31) and read2(28) files provided
Any clues? Thank you!

It seems that you are providing more R1 files than R2 files. 31 doesn't make sense since it's not a multiple of 4.

Yes, but I am providing only one file for each read. My command looks like this:
CITE-seq-Count -R1 read1/sample1_S1_CS_R1_001.fastq.gz -R2 read2/sample1_S1_R1_001.fastq.gz -t tags.csv -cbf 1 -cbl 16 -umif 17 -umil 22 -cells 3000 -o output
Sorry for so many questions but I am stuck.

This is really strange, because the number of files depends on how many commas are files are given with a comma for splitting them.
Try renaming your samples and see if the numbers in the error change.
Also, double check if your data is compressed or not.

Very interesting! Renaming the files to something like sample1_R1.fastq.gz and sample1_R2.fastq.gz did the trick! It is running now.
Thank you so much!


gocql does not re-resolve DNS names (issue 831 from https://github.com/gocql/gocql/issues/831)

DNS remains the same, but IP changes for the control connection node. Need to understand the behavior when a node is replaced:
### Tasks
- [ ] The node is not the one the control connection is with.
- [ ] The node is *the* one the control connection is with.

Since https://github.com/gocql/gocql/commit/b9737ddcadbbe8092b27df4f6ab2e6e9f3cf4c72 was merged and we fetched it from the upstream, this can be closed.


Add application credentials to openstack provider

Signed-off-by: Happy2C0de<EMAIL_ADDRESS>What this PR does / why we need it:
Add support for openstack application credentials
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes #899
Special notes for your reviewer:
Addition of getConfigAuth function in provider.go limits the cyclomatic complexity that was reached with a direct integration into getConfig function. From my point of view, excluding the retrieval of authentication information makes sense.
Please to proper e2e tests.
Optional Release Note:
Add application credentials support to the Openstack provider

/assign @xrstf

@Happy2C0de thank you for the contribution. Really great to have it!

@Happy2C0de thank you for the contribution. Really great to have it!

@kron4eg No problem.
Thanks for the valuable change request. 👍

/test pull-machine-controller-build

@Happy2C0de please see build log from build failure

@kron4eg
Thanks for reviewing and I apologise for the delayed reply.
/test pull-machine-controller-build

/test all

@Happy2C0de webhook is not happy
verify.go:159] Creation of "ubuntu-upgrade-1365061762797277184" failed, retrying: admission webhook "machinedeployments.machine-controller.kubermatic.io" denied the request: failed to default machineSpec: failed to get a openstack client, due to You must provide a password to authenticate

/test pull-machine-controller-e2e-ubuntu-upgrade

@kron4eg
Does your Openstack test cluster already support ApplicationCredentials?
See https://pkg.go.dev/github.com/gophercloud/gophercloud#AuthOptions
I think authentication should work now but it fails with the given error: "You must provide a password to authenticate" that comes from the provider. Username and Password are empty in provider.go at authentication time. LGTM
Local e2e tests are not possible for me such that I have to rely on your test pipelines.

It looks like the error comes from here
https://github.com/gophercloud/gophercloud/blob/v0.14.0/auth_options.go#L285
means that no ApplicationCredentialID nor ApplicationCredentialName are set (or empty).

/test all

/approve

@Happy2C0de thanks for contributing!

/cherry-pick release/v1.27


Fix bug

Unit tests unit tests unit tests
—
Created via Raycast

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:


add multi-module gen support

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #392
Special notes for your reviewer:

This is failing.
In a scenario that in theory it should work:
  src:
    path: test/fixtures/multi-modules
    context: {}
    generate:
      - gomod:
          paths:
            - module1
            - module2
        subpath: project

Also looks like this needs a go generate ./... so it updates the json schema.
Otherwise this seems like its working.


feat: Optionally update VPC Route Tables for attached VPCs

Description

When attaching a VPC to a Transit Gateway, you also need to update the VPC's Route Table(s) to direct traffic to the TGW. Currently this needs to be done outside of the module. With this change it is now possible to provide 2 extra values on the vpc_attachments object.
For example:
vpc_attachments = {
  some_vpc = {
    ... other values

    # optional new values
    vpc_route_table_ids  = module.some_vpc.private_route_table_ids # using https://github.com/terraform-aws-modules/terraform-aws-vpc
    tgw_destination_cidr = "<IP_ADDRESS>/0"
  }
}

Motivation and Context

Motivation: currently we are using this module and additionally defining our routes outside the module.
There's also this issue open: https://github.com/terraform-aws-modules/terraform-aws-transit-gateway/issues/17
Breaking Changes

If you have defined routes outside the module you cannot redefine them with these new parameters (you need to move them with terraform state mv or delete and recreate them).
Since the new parameters are optional there is no breaking change.
How Has This Been Tested?

In own environment, with and without the new values.

@jmcorallo would you mind rebasing this against master to update your branch?

@tfhartmann ready

v2.4.0 has been just released.
PS: @tfhartmann Could you please update the example code (in the examples folder) to show this and all other features which will be added into this module in action. Without examples, we will simply forget about this one. If updating example is not feasible, please make a note in README about this, at least.
Thank you!


[Doc Improvement][Update conversation events code format]

Updated C# format and indentation for Install update event C# code sample

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
95
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 1a91f0d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
95
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit ca08152:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
95
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 25d2e67:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


NullPointerException

Frequently call addview() and removeAllViews().
java.lang.NullPointerException
at com.google.android.flexbox.FlexboxLayout.drawDividersHorizontal(FlexboxLayout.java:2095)
at com.google.android.flexbox.FlexboxLayout.onDraw(FlexboxLayout.java:2047)
at android.view.View.draw(View.java:15114)
at android.view.View.updateDisplayListIfDirty(View.java:14048)
at android.view.View.getDisplayList(View.java:14071)
at android.view.View.draw(View.java:14838)
at android.view.ViewGroup.drawChild(ViewGroup.java:3404)
at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)
at android.view.View.updateDisplayListIfDirty(View.java:14043)
at android.view.View.getDisplayList(View.java:14071)
at android.view.View.draw(View.java:14838)

Thank you for reporting this. Looks like something unexpeted happened with a divider. Could you tell me how you set your divider? (Either XML or Java code)

app:dividerDrawableHorizontal="@drawable/flex_divider"
app:dividerDrawableVertical="@drawable/flex_divider"
flex_divider.xml

app:dividerDrawableHorizontal="@drawable/flex_divider" app:dividerDrawableVertical="@drawable/flex_divider"
flex_divider.xml
<shape xmlns:android="http://schemas.android.com/apk/res/android">     <size         android:width="8dp"         android:height="10dp" />     <solid android:color="@android:color/transparent" /> </shape>

Thanks. What is your showDivider setting? (or showDividerHorizontal or showDividerVertical)

I do not set showDivider setting, may be I should set "none"?

In what orientation, you want to show your divider?
If you set none it means no dividers will be drawn. So setting the dividerDrawable isn't useful.
To show a divider at least you need to set one of the showDivider , showDividerHorizontal,  showDividerVertical.

Sorry , I missed sth, I do not set showDivider , but I have set showDividerHorizontal and app:showDividerVertical
app:showDividerHorizontal="middle"    app:showDividerVertical="middle"

showDivider is a shorthand for setting showDividerHorizontal and showDividerVertical at the same time.
So it's totally valid if you set showDividerHorizontal and showDividerVertical separately.
However I couldn't reproduce the issue even if I called addView and removeAllViews extensively.
What version of the FlexboxLayout do you use? Current latest stable version is 0.2.5.

I made the sample ,  please check it , https://github.com/zhangxu1005/flexbox_test.git

Thank you very much for the sample code.
Hmm, but I still couldn't reproduce the issue with the sample.
Could you tell me what operations are needed to reproduce if it's consistently happening?

Sorry, I do not make the operations clear, please repeatedly click the "click here" quickly will reproduce the issues.

Ok I was able to reproduce the issue, thanks.
Actually the issue was already fixed in the master by #163, but not released as a library.
I'm going to release a new version anytime soon.
Thanks for reporting this!

Thank you so much!

FYI, released a new version as 0.2.6.
The issue should be fixed.


CLOUDSTACK-9618: Load Balancer configuration page does not have "Source" method in the drop down list.

If we create an isolated network with NetScaler published service offering for Load balancing service, then the load balancing configuration UI does not show "Source" as one of the supported LB methods in the drop down list. It only shows "Round-Robin" and "LeastConnection" methods in the list. However, It successfully creates LB rule with "Source" as the LB method using API.

Changed the base branch to 4.9.

@jburwell : As this is Netscaler specific. Yes, you need a Netscaler to verify.

ping @sateesh-chodapuneedi @rajesh-battala @karuturi

LGTM based on code review
@karuturi Since this is a network plugin related fix and requires Netscaler, I don't think there is a need to run the BVTs.


This seems to be an excuse not a resolution

Telling us why memory usage is unacceptably high doesn't address the fundamental problem that the memory is unacceptably high.
Can you please add a link to means to lobby/request for Microsoft to address this issue?  Can I suggest linking to the link below, that is hitting the 4-year anniversary and gets dozens of votes and comments a week?
https://microsoftteams.uservoice.com/forums/555103-public/suggestions/17219609-teams-uses-600mb-and-is-super-slow
Thank you.

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

ID: fc21902c-d522-3851-8587-177f01bc085c
Version Independent ID: 590bcc64-a86d-b61e-7d94-4e4322c4e71d
Content: How Microsoft Teams uses memory - Microsoft Teams
Content Source: Teams/teams-memory-usage-perf.md
Service: msteams
GitHub Login: @msdmaguire
Microsoft Alias: dmaguire

@PhilLeitch thank you for reaching out. Just for clarification, are you suggesting adding a link to uservoice in this article?

Yes, but to be honest I don’t understand even creating an article explaining why resource use is unacceptably high.
What is the purpose of the article?  What was it attempting to achieve?  What did you expect someone who searched for and found this article would leave with?
Is it designed to explain that Microsoft is aware that the resource footprint is unacceptably high and explain why it can’t do anything about it?  It’s almost as though the article explains the mistakes Microsoft has made that lead to the situation I’m in when I use teams.
Can you please re-read the article to see what I mean – it offers no usable information to IT professionals.  We either add more ram to devices or upgrade devices, right?
The article doesn’t provide any actions I can do to reduce the excess usage, like a setting or option to minimise the massive footprint.
This frustration comes from money and time we are burning on teams.  We’ve just had to retire 20 still new Microsoft Surface because 8GB is simply insufficient to run teams.
By insufficient, I mean that if teams is quit the surfaces operate normally, but with Teams running the surface is all but unusable.
And the flood of messages sharing this sentiment that appear on uservoice are simply going unanswered….
From: Reece Peacock<EMAIL_ADDRESS>Sent: Thursday, 1 October 2020 6:55 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Philip Leitch<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] This seems to be an excuse not a resolution (#5688)
@PhilLeitchhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPhilLeitch&data=02|01|philip.leitch%40alphacompanies.com.au|7caab73adc9d4d45228308d865e7b287|480e4b71395b4dd6b9094f754acfdd01|1|0|637371393101687157&sdata=p9oPBZWtwjxPh295HWcHUuPKohWm7XN4dOzFwiCZLjo%3D&reserved=0 thank you for reaching out. Just for clarification, are you suggesting adding a link to uservoice in this article?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fissues%2F5688%23issuecomment-701992157&data=02|01|philip.leitch%40alphacompanies.com.au|7caab73adc9d4d45228308d865e7b287|480e4b71395b4dd6b9094f754acfdd01|1|0|637371393101697151&sdata=HCOmprpE8c5kfa3wBoZahMJyN3NyM7fqcfAt93oZJI4%3D&reserved=0, or unsubscribehttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPR7H7VQUA5AG4AVAVGKYCDSIQ7WTANCNFSM4QSSCXKQ&data=02|01|philip.leitch%40alphacompanies.com.au|7caab73adc9d4d45228308d865e7b287|480e4b71395b4dd6b9094f754acfdd01|1|0|637371393101697151&sdata=pc%2F2hiDziFr8D%2FXSCyI2sDI%2FKYW7830XEIfYUgwAkac%3D&reserved=0.
Philip Leitch
Business Solutions Architect
t: (07) 3185 3055 m: 0414 861 252 f:
e<EMAIL_ADDRESS>a: Level 18, 444 Queen St, Brisbane QLD 4000 AU [Embedded Image]

@PhilLeitch The intention of the article was to reduce the great many calls and questions Microsoft was receiving by admins and customers who saw how resource-intensive Teams can be in some environments. It was meant to explain that Teams uses whatever resources are available to it, but not at the expense of other apps and services running in the same environment. Teams "gives back" when there is a demand for resources elsewhere. Certainly we will pass along your feedback that current memory usage is unacceptably high. We appreciate the feedback.

Sorry, but for clarity, can you confirm that the purpose of this article is to reduce the calls Microsoft was receiving by admins and customers when Teams is excessively resource intensive?
Can you confirm that this knowledge base article does not providing admins and any ability to address or take any action on Teams actually being so resource intensive that customers would otherwise contact Microsoft?
Also, I can repeatedly demonstrate that Teams does not “return” sufficient resources to Microsoft Surface and other environments in order to be used in an office environment, as also shown by many others over about 5 years on the community thread. Am I able to provide you telemetry to demonstrate this article is in fact wrong or misleading (as insufficient resources are returned by teams)?
Thank you for your patience on this
From: David Maguire<EMAIL_ADDRESS>Sent: Friday, 2 October 2020 2:01 AM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Philip Leitch<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] This seems to be an excuse not a resolution (#5688)
@PhilLeitchhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPhilLeitch&data=02|01|philip.leitch%40alphacompanies.com.au|eb6253cbbf194a7e474808d8662325e3|480e4b71395b4dd6b9094f754acfdd01|1|0|637371648435789160&sdata=tC70y7thgYDb3Z41Bt3ObTLafn62V8ULh26NmuCnrxQ%3D&reserved=0 The intention of the article was to reduce the great many calls and questions Microsoft was receiving by admins and customers who saw how resource-intensive Teams can be in some environments. It was meant to explain that Teams uses whatever resources are available to it, but not at the expense of other apps and services running in the same environment. Teams "gives back" when there is a demand for resources elsewhere. Certainly we will pass along your feedback that current memory usage is unacceptably high. We appreciate the feedback.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fissues%2F5688%23issuecomment-702234629&data=02|01|philip.leitch%40alphacompanies.com.au|eb6253cbbf194a7e474808d8662325e3|480e4b71395b4dd6b9094f754acfdd01|1|0|637371648435789160&sdata=SjVZvgcpzH1HYEkZIPAOi%2BwHl3rNjuw0NgwNVOotkVg%3D&reserved=0, or unsubscribehttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPR7H7S7CGAAYZ5NQKFVXZDSISRSPANCNFSM4QSSCXKQ&data=02|01|philip.leitch%40alphacompanies.com.au|eb6253cbbf194a7e474808d8662325e3|480e4b71395b4dd6b9094f754acfdd01|1|0|637371648435799158&sdata=1GUsDQRXWQ3lQhJ5L0Wit9f3sky5K0oKNmBNwq2OXL0%3D&reserved=0.
Philip Leitch
Business Solutions Architect
t: (07) 3185 3055 m: 0414 861 252 f:
e<EMAIL_ADDRESS>a: Level 18, 444 Queen St, Brisbane QLD 4000 AU [Embedded Image]


Interest in the release date!

Hi! First of all congratulations on this work. It looks really relevant.
I have seen that the paper got accepted in ICLR, so congrats on that. Is there any news regarding the release date of the dataset?
Best

Hi！We are delighted that you are interested in our work. As it is currently under review, we will release our dataset as soon as the peer-review process is completed.
We hope you can continue to follow our work.
Thank you


linuxPackages.isgx: mark as broken for kernels >= 6.4

One of the common failures in the kernel package sets is the out-of-tree isgx module (upstream repository here). The maintainers note:

This driver is deprecated and no longer maintained by Intel. We recommend the SGX community to use the SGX driver that was upstreamed into the Linux kernel.

There are some old, unmerged PRs to bring the module compatability to recent kernels but it largely seems like Intel has officially deprecated the out of tree module.
I ran a few builds against the versioned kernel package sets for the module and it won't succeed on anything up to or past 6.4, so this PR should properly highlight when the module won't work as expected.
Maybe Intel will merge fixes for recent kernels and the bounds on the broken attribute can be updated, but for the time being, this change should accurately reflect when the module will and won't work.
(NB I'm treading into kernel sets for the first time so feel free to correct anything I'm doing wrong here)
Things done

Built on platform(s)

[x] 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.

ZHF: https://github.com/NixOS/nixpkgs/issues/352882


V001 rewrite ts microbundle

This is the initial refactor of client.js. It should be backward compatible. Note, the examples and tests will be updated in a new pull request.
Here are a summary of the changes:

Implemented Typescript & rewrote client in Typescript
Implemented Microbundle. This takes care of polyfill, minification, cross environment compatibility, TS declaration generation, etc. etc. Also, it will immediately add functionality, such as producing the UMD files necessary for browser

I'll submit an issue on the main repo to discuss these changes

Whoops.. meant to pull this into my fork, not the main repo


Removing imports from same namespace is not loading Eloquent relationships

Laravel Version: 9.19
PHP Version: 8.1.11
Database Driver & Version:
Ubuntu 22.04

Description:
In current Laravel version (v9.19) I am having this issue. In my model classes, Pint removes imports from same namespace i.e. models imported from same namespace. And because of this the eloquent relationships are not loading even when I query my model using with clause. Here is the code I have written in my controller function.
$item = Item::with([
    'grade' => function ($q) {
        $q->select('id', 'title', 'slug', 'description', 'org_search_key', 'isActive');
    },
    'subject' => function ($q) {
        $q->select('id', 'title', 'slug', 'description', 'org_search_key', 'isActive');
    },
])->where('id', $request->itemId)->first();

In the model class if I import Grade and Subject classes, these relationships work fine. If classes are not imported (i.e. removed by Pint), the relationships turn out to be null.
Steps To Reproduce:
Create an API call that loads the Item model from database. You only need one relationship with another model like Grade or Subject in this example. If you use above PHP code, and remove imports from same namespace, you might see this happening.

I think I closed a similar issue in the Pint repo? Please don't open duplicate issues and try a support channel.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC


Release 7.0.5 has problems downloading ODBC Bridge drivers

System information:

Operating system (distribution) and version
Windows 10 1709
DBeaver version
7.0.5
Additional extensions

Connection specification:

Database name and version
Netezza <IP_ADDRESS>-P2
Driver name
ODBC
Do you use tunnels or proxies (SSH, SOCKS, etc)?
No

Describe the problem you're observing:
When attempting to set up ODBC-JDBC bridge, DBeaver wants to download drivers. Cannot download driver because it is getting a 403 error on https://dbeaver.io/files. Once it hits the 403 it aborts, not attempting the Maven repo.
<<The problem does not exist in 7.0.3>>
Steps to reproduce, if exist:
Database menu -> Driver Manager -> Double-click on ODBC -> Click Download/Update -> Click Download
Error message:
Driver file download failed.
Do you want to retry?
Reason:
Can't open 'https://dbeaver.io/files/drivers/odbc': Not Found
#################################################
By contrast - in 7.0.3, the same path of operations gets the driver properly downloaded
#################################################
Include any warning/errors/backtraces from the logs

Seeing the same issue here. Any help appreciated!

Windows 10 Pro v1903
DBeaver v<IP_ADDRESS>005171753 (5/18/2020)

Thanks, Neil

Hello,
here is a picture with the difference between 6.3.0 and 7.0.5 maybe it is of help.
The first entry is a folder on the webserver, so the "Forbidden" error is ok I guess, as you cannot usually target an entire folder for downloading.
I've tried to add the missing entry for the jar file but have no idea how to. Neither "Add file" nor "Add artifact" seem to work.

hi, issue is already fixed in #8732 Fixed will be released in scope of 7.1.0
thanks for the bug report

Thanks uslss!


Add namespace param to the Helm charts

What do these changes do?
Add namespace param to the Helm charts yaml files.

@haoxins Thanks for your interest in GraphScope and contributing to the code. I think you may want to deploy GraphScope in the specified namespace by Helm, and follow Helm Install Doc:

Options:
--create-namespace: create the release namespace if not present
-n, --namespace string:  namespace scope for this request

So, you can do it by following command:
helm install graphscope graphscope/graphscope --create-namespace --namespace your_namespace
# Delete instance
helm -n your_namespace uninstall graphscope
# Don't forget to delete the namespace manually cause helm doesn't do it
kubectl delete ns your_namespace

@lidongze0629 I already did this and I don't ask for the instructions :)


Copy and Paste support for DateField and TimeField

Provide a general summary of the feature here
Looking for an easy way for users to copy and paste values to/from DateField and TimeField
🤔 Expected Behavior?
Copy and paste much like inputs
😯 Current Behavior
Currently does not support selecting all DateSegment values for copy and paste.
💁 Possible Solution
No response
🔦 Context
There are cases when users must fill in multiple Date/Time fields in a single form. Copy and paste seems like an intuitive way to provide a starting value before making additional changes.
💻 Examples
Two quick examples that allow for copy and paste in a DateField:
MUI: https://mui.com/x/react-date-pickers/date-time-picker/
Kendo: https://www.telerik.com/kendo-react-ui/components/dateinputs/datetimepicker/
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

I think the question for copy and paste is "in what format"? This goes back to the whole reason we use individual segments instead of a single text field in the first place: because it's close to impossible to parse dates from a string correctly in an unknown/arbitrary format.
We could support copying and pasting only between React Aria date components using our own custom data format, but then you couldn't paste from an external location. We could choose a standard format like ISO 8601, but it's less likely for a human user to be copying a date in a computer format. But when it comes to dates a real user would actually want to copy and paste from outside a form into it that will be really difficult to do.
If we want to do something here we will need to decide on a way forward and probably try to scope it down to something doable.

I would assume the user expects the value to be copied as shown in the field (containing the separators) and to be able to paste a value with the separators as shown in the field. No conversions, not any difficult parsing.
The field should behave as much as a normal textbox although it enforces the user to input the values in a a specific format.

@ttsirkia - I agree.  What is pasted would be expected to be in the format of the datefield.
This has been open for well over a year, so I guess this is not something that is acceptable?   Copying a date is a common use case for us.   Not sure if we can implement this support ourself, but paste support (add on by us or directly in the headless) we would be unable to use this datefield component.
@devongovett Is this something that would/could be considered?


[1.15.2] Error on loading the client on Forge 31.2.0

Just updated to the latest Forge 31.2.0, getting this error on every launch of the client:
crash-2020-05-17_18.35.01-client.txt

This isn't caused by Flux Networks, looks like you're issue is caused by BiomesOPlenty
BiomesOPlenty-1.15.2-<IP_ADDRESS>8-universal.jar Biomes O' Plenty<EMAIL_ADDRESS>ERROR} - I think Biomes O' Plenty released a fix for this yesterday.
Due to the parallel loading forge added, it causes crashes during the loading phase to be ignored.
Forge Issue:
MinecraftForge/MinecraftForge#6374

Yes, it it looks like this, now I see. There definitely was a new problem with Biomes o' Plenty that might have caused this.
And wow that bug has a long history on Forge.. hmmm. Actually I think that a number of other crashes I've encountered can also be explained by this. Well, let's hope it gets resolved one day!
Thank you for the explanation :)


Configure Sentry exceptions reporting

install sentry/browser pkg
call Sentry.init in index.tsx
current DSN<EMAIL_ADDRESS>is a personnal one from glitchtip.com. We may want a global one accessible from all of us ? cc @tiero ?
Sentry only enabled in case of process.env === 'production'

it closes #411
@tiero please review

Any reason to not use the React package of Sentry?
//...
import * as Sentry from "@sentry/react";

Sentry.init({
  dsn<EMAIL_ADDRESS>  integrations: [
    new Sentry.BrowserTracing({
      // Set 'tracePropagationTargets' to control for which URLs distributed tracing should be enabled
      tracePropagationTargets: ["localhost", /^https:\/\/yourserver\.io\/api/],
    }),
    new Sentry.Replay(),
  ],
  // Performance Monitoring
  tracesSampleRate: 1.0, //  Capture 100% of the transactions
  // Session Replay
  replaysSessionSampleRate: 0.1, // This sets the sample rate at 10%. You may want to change it to 100% while in development and then sample at a lower rate in production.
  replaysOnErrorSampleRate: 1.0, // If you're not already sampling the entire session, change the sample rate to 100% when sampling sessions where errors occur.
});

const container = document.getElementById(“app”);
const root = createRoot(container);
root.render(<App />);

Any reason to not use the React package of Sentry? @louisinger

It looks like a wrapper or the Sentry class, not sure the react components add a real value 🤔

Solve conflicts


libjack related link error

During the final link on an ARM Linux system, I'm getting this error even though I've got libjack installed:
note: ../rustzx/target/release/deps/libportaudio-cf78f0b5104b5853.rlib(pa_jack.o): In function `BuildDeviceList':
pa_jack.c:(.text+0x844): undefined reference to `jack_client_name_size'
pa_jack.c:(.text+0x868): undefined reference to `jack_client_name_size'
pa_jack.c:(.text+0x89c): undefined reference to `jack_get_ports'
pa_jack.c:(.text+0x9f4): undefined reference to `jack_client_name_size'
pa_jack.c:(.text+0xc0c): undefined reference to `jack_get_sample_rate'
pa_jack.c:(.text+0xe00): undefined reference to `jack_get_ports'
pa_jack.c:(.text+0xe58): undefined reference to `jack_port_by_name'
pa_jack.c:(.text+0xe64): undefined reference to `jack_port_get_latency'
pa_jack.c:(.text+0xef8): undefined reference to `jack_get_ports'
pa_jack.c:(.text+0xf50): undefined reference to `jack_port_by_name'
pa_jack.c:(.text+0xf5c): undefined reference to `jack_port_get_latency'

and so on. What are the required dependencies again? (static versions?)

@petevine, thanks for the report!
Hmmm, interesting. I not even tried to build it on ARM. My PortAudio grabs ALSA automatically, and it just works. Have you installed libportaudio2 and portaudio19-dev or portaudio was built during installation? Because I had some problems also, when dev package was not installed and portaudio crate compiling it manually. Or try to install jack dev package.  Btw, what distribution are you using?
If you will archive some success, tell me.

I mentioned ARM cause I was expecting OpenGL issues - otherwise it doesn't matter, the Linux system is based on Ubuntu 14.04 and I made sure all dependencies were installed. Maybe I made a typo in a symlink or something (/usr/lib/arm-linux-gnueabihf/ path is sometimes not considered by the linker)

Maybe there will be also some problems with OpenGL, but your log points to jack audio library. In first line we can see that rust crate portaudio tries to compile C-library, but can't find some Jack function definitions. So, if rust-portaudio tried to compile it, this means that  it can't find system portaudio package with pkg-config utility.

Let me repeat, this happened during the final link - the rust portaudio crate had compiled just fine earlier.

Following on that idea, I added -ljack to the final link command line and it managed to build this time.
Not sure why it was never a problem on your platform but modifying the build script along these lines is probably a good idea.


ID token in URL after callback

Just a question not an issue, in my app after the user is authenticated and sent back through the callback component the 'callback/id_token' stays in the URL. I'm not doing anything out of the ordinary, but I can't figure out how to get it the URL back to the root.
The routing works perfectly fine. Let me know if you have any ideas, and awesome library thanks for creating and sharing!

Hi, and thanks for the kind words!
The <CallbackComponent /> accepts a successCallback prop, which is normally used to redirect the user after a sucessful login. See this line in the example app.

Yep I am using the success callback exactly as described in the example and still seeing this. The redirect to home page works perfectly fine but the URL stays with the full ID token in it. Not sure if I'm just missing something basic, but I am actually just using an exact copy of the callback component in example.

May not even be related to the library but this is my first time working with oidc in a react project so I'm not sure where else to ask

I'm not sure then why this happens. Are you using react-router's hash history? This could cause issues.

Yeah I'm using the BrowserRouter, haven't had much success yet still but I will keep this question updated if I figure it out.

I was able to resolve by using the withRouter higher order component from react router dom then using this.props.history.push("/") in the ''
import React from 'react';
import { connect } from 'react-redux';
import { CallbackComponent } from 'redux-oidc';
import userManager from '../../utils/userManager';
import { withRouter } from 'react-router-dom';

class CallbackPage extends React.Component {
    successCallback = () => {
        this.props.history.push("/");
    };

    render() {
        // just redirect to '/' in both cases
        return (
            <CallbackComponent userManager={userManager} successCallback={this.successCallback} errorCallback={this.successCallback}>
                <div>

                </div>
            </CallbackComponent>
        );
    }
}

function mapDispatchToProps(dispatch) {
    return {
        dispatch
    };
}

export default withRouter(connect(null, mapDispatchToProps)(CallbackPage));

Thanks for the help pointing me to router issue, appreciate your library!


AKS deployment

[Enter feedback here]

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

ID: 597ba957-9501-5919-73a0-6d47b1ca2c6d
Version Independent ID: 6345fa19-b3f2-ffd6-f526-908f845b51f6
Content: Use your TLS certificates for ingress - Azure Kubernetes Service
Content Source: articles/aks/ingress-own-tls.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

I could work with this sample. I could get the response using the curl command. However, when trying the URL in the browser, I was still not able to get the response. Not sure what could be wrong.

This code sample is designed to be used with curl so you can use the --resolve parameter to map the demo.azure.com name to the public IP address of your ingress controller. The demo is not intended for your use case. If you are having an issue with TLS certificates in your AKS cluster please open a support case with support.


Ward<PHONE_NUMBER>43

@ileaderx
Please remove colon (:) after your name.
This branch has conflicts that must be resolved.
Could you resolve it locally and then push again?
Thanks
Hui

@ileaderx
You removed other team members to resolve the conflict???
Why?
Hui