[Doc Issue][Silent authentication update adal]

added Important note.

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/tabs/how-to/authentication/auth-silent-aad.md
82
18
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 14f1c6c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.md
:bulb:Suggestion
View
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.md

Line 11, Column 223: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/msal-migration' will be broken in isolated environments. Replace with a relative link.

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/tabs/how-to/authentication/auth-silent-aad.md
82
18
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 9444f3c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.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

Our samples should also get updated to no longer use ADAL.

https://github.com/OfficeDev/Microsoft-Teams-Samples/search?q=ADAL
https://github.com/OfficeDev/TeamsFx/search?q=ADAL

Hi @v-ypalikila , is there a public announcement from MS that already contains the ADAL EOL(?) date? If yes, then let's link to it. If not, are our docs the source of truth to announce this date?

Our samples should also get updated to no longer use ADAL.

https://github.com/OfficeDev/Microsoft-Teams-Samples/search?q=ADAL
https://github.com/OfficeDev/TeamsFx/search?q=ADAL

Hi @v-ypalikila , are we tracking this? In ADO?

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/tabs/how-to/authentication/auth-silent-aad.md
73
27
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 a5b0d16:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.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/tabs/how-to/authentication/auth-silent-aad.md
73
27
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 74e1c49:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.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/tabs/how-to/authentication/auth-silent-aad.md
80
19
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 f449aa1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.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

Our samples should also get updated to no longer use ADAL.

https://github.com/OfficeDev/Microsoft-Teams-Samples/search?q=ADAL
https://github.com/OfficeDev/TeamsFx/search?q=ADAL

Hi @v-ypalikila, @v-habibaumme are we tracking this? In ADO?

Hi @ashishguptaiitb, we have created a separate ADO to track and update the sample codes.
ADO: 2180226

Docs Build status updates of commit cdf26e0:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.md
:warning:Warning
View
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.md

Line 139, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/active-directory/develop/msal-overview' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 139, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/active-directory/develop/msal-overview' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9dbb045:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/authentication/auth-silent-aad.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


Not working for Arguments

Thanks a lot for great library.
If I have argument of some type, which has properties with JsonProperty attribute, formatting are not appied.

Thanks @bantikyan. Can you give me a code sample?


Trying to load old cmc result files do not color muscles

Observed using Gait2392 and Chand simulation results. Fails even if the file is converted to a states file offline

This boils down to needing to call realizeDynamics to populate the color cache entry (per comments in GeometryPathy::generateDecorations), does that seem justified? @aseth1 @chrisdembia ?
Would that cause other side effects or excessively slowdown playback?

Where would the call be?
It's weird that dynamics is necessary, because activation is a state and is available at stage Model. Perhaps generateDecorations is trying to use the activation stored in MuscleDynamicsInfo.

Why would realizeDynamics cause issues on a model that was used to run CMC? If you ignore activation dynamics or use any other path type actuator, expecting dynamics to know what color it should be seems reasonable to me.

I can't tell from the discussion if this is ready to be verified or not. @aymanhab can you comment.

I have a build with a fix that I can share but I don't want to claim it's fixed and have testers spend time verifying until we have example files to test with. (e.g. any model with setup files to run CMC out of the box with activations that clearly color the muscles) to check live animation and playback from inside GUI and to load precomputed results from files). Let me know if you have these around.

If I load gait2354 and load the states file from CMC that is found in the Output reference, the muscles color as expected.
Is this what I'm supposed to be testing for this issue?

@jenhicks Becaus eof some ambiguities in issues descriptions, for each of the tools I check that:

Runnning the tools colors the muscles as expected
Playback of results colors the muscles.
Load pre-made/saved results colors the muscles.

If all that works then we're golden.  I assume in this case results wee pre-made (Chand's simulations)


Move production content to public folder

As a developer I would like the production content to be inside of the public folder.
Need to change:

Webpack configuration build directory
Location of index.html
Repositories publicly hosted directory.

Resolved via commit https://github.com/jfehrman/decision_json_editor/commit/d4664887464b01777904085506672d7647819d54.


[Feature] load/save safetensor in bfloat16

Currently, there are some issues with fine-tuning and training using float16. We need to convert some weights to float32 during the forward pass and then convert them back to float16 when saving the weight. Since the latest Apple Silicon already supports bfloat16, would it be possible to load/save safetensor in bfloat16 for supported devices? This way, we wouldn't have to convert the weights from float32 back to float16 during or after training and fine-tuning.

@mzbac can you be more specific with the issues you are describing? We already allow bfloat16 to be loaded directly from safetensors, so I am not sure exactly what you are asking for?
>>> a = mx.array([1.0], mx.bfloat16)
>>> mx.save_safetensors("a.safetensors", {"a": a})
>>> mx.load("a.safetensors")
{'a': array([1], dtype=bfloat16)}

@mzbac can you be more specific with the issues you are describing? We already allow bfloat16 to be loaded directly from safetensors, so I am not sure exactly what you are asking for?
>>> a = mx.array([1.0], mx.bfloat16)
>>> mx.save_safetensors("a.safetensors", {"a": a})
>>> mx.load("a.safetensors")
{'a': array([1], dtype=bfloat16)}

Hi @awni, I will double-check because I checked a few models and it seems that they have float16 weight after being loaded. In Lora, we use fp32 weight. We have to cast it back to float16 during the merge of Lora. So, I was thinking of enabling bfloat16 for Lora training to avoid the casting. I thought load safetensors doesn't support float16, which is why I am getting model weight as fp16.

So our load_safetensors will load the type unchanged. Different models come with different types out of the box, we'll just use the given type. I don't think we do anything in mlx-lm to change the loaded type (but we should verify).
One thing to keep in mind is that even though MacOS supports bfloat16 it is a lot slower than fp16. It is purely a memory saving device not speed. That why I usually default to float16 when it makes sense to explicitly specify a default.

So our load_safetensors will load the type unchanged. Different models come with different types out of the box, we'll just use the given type. I don't think we do anything in mlx-lm to change the loaded type (but we should verify).
One thing to keep in mind is that even though MacOS supports bfloat16 it is a lot slower than fp16. It is purely a memory saving device not for speed. That why I usually default to float16 when it makes sense to explicitly specify a default.

Thanks for the detailed explanation. I will investigate further. The only thing that brought me this issue was the weight of Lora using fp32, and during merge we cast it to fp16. I am not 100% sure if this is causing the issue, but sometimes when  fusing Lora for a non-quantized model, there is some performance degradation.

The only thing that brought me this issue was the weight of Lora using fp32, and during merge we cast it to fp16.

Interesting. I think it's more likely the degradation comes from the fusion itself rather than casting the adapters. I noticed this in the past. One thing to check is: cast but no fuse vs cast + fuse.
When i was encountering this, it was because the adapters were small which especially when you add them to the large weight matrix values you lose precision. Using them outside the matrix can be numerically more stable in that case since you get the reduction first. In that case increasing the scale during training helped.

Thanks for the suggestion, I will check them out.

@mzbac let me know if we can close this. It sounds like if there may be something to fix but it is likely specific to LoRA / mlx-lm


Inject Container?

I would like to register an object, whereby I inject the container itself into a factory (factories can use whatever they need to to create the object they will, including a DiC).
However, trying to inject the container using .register provides:
   self.container!.register(MyFactory.self) { r in
        return MyFactory(self.container!)
    }

Closure cannot implicitly capture a mutating self parameter

Any ideas on how I can do this otherwise?

Ignore this, it was a swift question ;) I got around this with: let c = container! before using c instead.


The .del method is not accepting callbacks

Hello,
I'm trying out ioredis and I got an issue. I'm using the .del method like this:
redis.del(key)
.then(function() {
  console.log('No errors');
})
.catch(function(error) {
  console.log('Got an error');
});

But I'm not getting logs at all, while there is no problem for the .setex method.
Do you know if the .del method actually supports callbacks?
Thank you,
Flavio

Every command (including del) accepts a callback. Could you please enable the debug mode (DEBUG=ioredis:* node yourapp.js) and post the logs here?

Thank you @luin and forgive me for opening this issue, I found that it wasn't related to redis but to the gulp task. The process.exit() prevented the callback from executing. Thank you anyway!


Create  AnandThakur2001.html

already done ...
kindly look at it again
and accept the changes requested . Thankyou
On Sun, Oct 9, 2022 at 12:37 PM SNEHA AGARWAL @.***>
wrote:

@.**** requested changes on this pull request.
Kindly update your details in Your Portfolio HTML File.
—
Reply to this email directly, view it on GitHub
https://github.com/DecodersCommunity/portfolio-builder/pull/24#pullrequestreview-1135309699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWDE3DJ7376CARO2G27PYVLWCJVLBANCNFSM6AAAAAARAHFNBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


[build] es5 dist version throws error.

macro-carousel.es5.min.js throws the following error:
Uncaught TypeError: Failed to construct 'HTMLElement': Please use the 'new' operator, this DOM object constructor cannot be called as a function.

False alarm, I did not check out the demo.
@ciampo Perhaps we should also mention this process in the docs?

It is mentioned in the README.
Going to close the issue for now, but feel free to re-open it in case you still see errors after following the instructions


Add license (#1996)

Pull request opened by the merge tool on behalf of #1996

Merged into branches iso3 in f8c1ae2a3180e2882ccf168aa34029ac3f112a9d


warn instead of error for SHOULD

When providing a custom property, it errors about it (prints in red), instead of warning about it.
[!] course-of-action--d6e1dbd5-104f-4a09-b76c-3531d3398ecb: {103} Custom property 'obs_ref' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by dashes), a dash and then the name.

Thanks, @jmgnc.
Right now we don't distinguish "errors" from "warnings". Specific checks can be turned on and off using --enable or --disable. Additionally, all of the SHOULD checks can be turned off with --lax.
Do you think the current implementation is sufficient, or do we need to distinguish errors from warnings (besides being able to selectively turn them on and off)?

I think supporting warnings would be a good thing.  It lets those warnings
be annoying as opposed to turning them off prevents you from seeing when
new warning happen.  It's also not clear from the table which of the errors
are SHOULDs versus MUSTs.
On Tue, Nov 22, 2016 at 7:19 AM, Greg Back<EMAIL_ADDRESS>wrote:

Thanks, @jmgnc https://github.com/jmgnc.
Right now we don't distinguish "errors" from "warnings". Specific checks
can be turned on and off using --enable or --disable. Additionally, all
of the SHOULD checks can be turned off with --lax.
Do you think the current implementation is sufficient, or do we need to
distinguish errors from warnings (besides being able to selectively turn
them on and off)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oasis-open/cti-stix-validator/issues/5#issuecomment-262268538,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALCthQN7DfgIO5wyDq0U_mnC-FK8VPgtks5rAwftgaJpZM4K418h
.

It'd be great to see this same warning behavior applied to open vocabularies. According to the spec, it's acceptable to use custom values in an open-vocab field (which consumers are free to ignore).

The value of the property SHOULD be chosen from the suggested vocabulary but MAY be any other string value.

The validator treats this as an error right now.
Example:
malware--031e8443-3a44-5068-b897-7a242b69b0b6: {216} labels contains a value not in the malware-label-ov vocabulary.

Sounds good! We'll take a look.


Fix mina.service logging

Thank you for contributing to Mina! Please see CONTRIBUTING.md if you haven't
yet. In that doc, there are more details around how to start our CI.
These changes will alter the default settings for the system journal to prevent the mina service from flooding /var/log/syslog with messages. It is not needed to log all data to /var/log/syslog the client already creates its own logs in .coda-config.
By allowing syslog to be flooded with messages we are wasting system resources especially disk writes. I have not went through all of the settings this file can contain I just wanted it to stop abusing my syslog
I tested my changes by creating the file in the correct location with these settings then restarting the server.
I have no idea if this will work in your CI but this is more a change to the system defaults than to the actual operation of the code. I just added the conf file in the correct location in the source code the scripts folder then changed the only file where i found mina.service to include copying the new file in the build process
I would have someone more familiar with the code than myself test this to ensure it does not cause any issues.
Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Closes #0000
Closes #0000

Submitting new PR to correct branch with update to remove extra information in the mkdir command

Submitting new PR to correct branch with update to remove extra information in the mkdir command

https://github.com/MinaProtocol/mina/pull/7486

https://github.com/MinaProtocol/mina/pull/7486


Add libuv version 1.40.0

I have fixed and merged this patch. https://github.com/xmake-io/xmake-repo/pull/282


Passing extra data from client to server

Hi
is it possible to passing extra-data to server, for example the destination folder?
thanks

yes, it is. you can send it inside the metadata object:
new tus.Upload(file, {
    // ...
    metadata: {
        filename: file.name,
        filetype: file.type,
        destination: '/path/to/folder',
    },
    // ...
})

I found the same solution, the problem was reading this data from tus-php but I've found how to get this param.
Thank you
Ivan


Add templates for issues

Issue templates are very helpful for a collaboration repo. When users identify a bug or want to add a new feature, you can provide templates so you can collect all the pertinent information you need to fix a bug or add a new feature.
We recommend creating a “Report Bug” and “Feature Request” issue template.
you can refer this: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository
Some suggested prompts/questions you can add to a “Report Bug” template are:
Briefly describe the bug
What is the expected behavior?
Please provide step by step instructions on how to reproduce the bug
Some suggested prompts/questions you can add to a “Feature Request” issue template are:
Briefly describe your feature request
What problem is this feature trying to solve?
How do we know when the feature is complete?
Reference: https://github.com/Recode-Hive/Stackoverflow-Analysis/issues/new/choose
Click on Get Started button and copy the template to your issue.

could you please assign me this issue I would like to work on it @anusha7530 @AbhiDiva96

hey @nagajaideep
I assign it to you. you can contribute

@AbhiDiva96 No PR was raised for this issue can i work on it ?

@AbhiDiva96 you can close this issue @yashksaini-coder
this issue is closed I've just not referenced this you can take a look at closed pull request and add more functionality
happy coding


WMTS support for time-dynamic and static dimensions

TO DO:

[ ] Needs #5536 to be merged first
[x] Unit tests for WebMapTileServiceImageryProvider class
[x] Unit tests for TimeDynamicImagery class
[x] Cleanup Sandcastle example

@kring any chance you could look at this in addition to #5536?
I can do a quick review for the standard stuff but it would be good for someone more familiar to also look at this.

Ok, this should be good to go for another review, then just waiting on #5536 to be merged.

Looks good, just those minor comments.


Nx Expo -  "path" argument must be of type string. Received undefined

I just tried adding a new expo app to an existing nx mono repo and I am getting the error
The "path" argument must be of type string. Received undefined

I decided to  try creating a new project and followed this blog
https://blog.nrwl.io/introducing-expo-support-for-nx-3fffb1849a6f
However, this gives me a similar error
TypeError: Cannot read property 'endsWith' of undefined\n" +
    '    at createProjectRootMappings (nx-expo-poetry/node_modules/nx/src/project-graph/file-map-utils.js:12:38)\n' +

Anyone got any ideas on where I can put the path so that it is not undefined?
Mac OS 12.3.1
NX version: 13.10.6

Also running into this

I thought it might be because of this update but I made sure I had project and watchFolders defined correctly.

@leggomuhgreggo @mikerudge i added a migration script https://github.com/nrwl/nx-labs/commit/aa1392d5285c560cefab096721dccf9fa033057c#diff-25ec7d9495695fda92b26961d7285c1f2661842d49dd9217a152246dc9a3a564R13.
if the migration script did not apply, you might manually update in your app's metro.config.js, in return withNxMetro..., add
   projectRoot: __dirname,
    watchFolders: [],

so it becomes:
return withNxMetro(defaultConfig, {
    // Change this to true to see debugging info.
    // Useful if you have issues resolving modules
    debug: false,
    // all the file extensions used for imports other than 'ts', 'tsx', 'js', 'jsx'
    extensions: [],
    projectRoot: __dirname,
    watchFolders: [],
  });

close this issue with above solution.


Completing javascript code before going into background mode.

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
I have a post function which posts (a video + a video poster image) / (no video but an image) to AWS S3 via Parse (now AWS hosted parse server) then returns the ULR which is passed into firebase along with any accompanying text for the post (similar to Facebook etc). This functionality works great when in normal use, but if the user hits post and pauses the app by hitting the home button (IOS) to do something else the upload fails. This is rather essential functionality as most users won't wait for an upload to complete or an upload complete dialog before moving out of the app and indeed compared to native apps this looks rather amateur.
What behavior are you expecting?
In light of the above I tried to use cordova-background-plugin to allow the javascript to complete on pause, the result is the same, the code fails with no error. I am now thinking that maybe uploading with cordova-file-transfer may allow the app to complete the upload and the javascript in the callback? I'm waiting for Apple to help out on developer tech support for a custom cordova plugin to save base64 to file and return device path to cordoba-file-transfer to do this.
The workflow is:
1/ Save image / video poster image (Parsefile.save/cordova-file-transfer)
2/ (In case of video) save video (Parsefile.save/cordova-file-transfer)
3/ Post details and text to firebase (JS SDK)
I'm looking for a way to allow this process to complete if the app is paused mid flow.
Steps to reproduce:

Upload poster image jpeg file
Upload video mp4 file
Post to firebase with previous 2 URLs.

insert any relevant code between the above and below backticks

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Which Ionic Version? 1.x or 2.x
Ionic 1.7.11
Plunker that shows an example of your issue
For Ionic 1 issues - http://plnkr.co/edit/jnOrQRtE8rN2WCXDQYQY?p=info
For Ionic 2 issues - http://plnkr.co/edit/sSFR9jbfTsxgAmrm9hUR?p=info
Run ionic info from terminal/cmd prompt: (paste output below)

Hello! As this issue is more of a question and not a bug with the framework i would suggest that you hit up our forum( https://forum.ionicframework.com/ ) or our slack channel (http://ionicworldwide.herokuapp.com/) and the awesome community on there should be able to help you! Thanks for using ionic!

Hi jgw96, is it expected behaviour with ionic that the upload should fail when paused? I wasn't sure if this was a bug or not.
Thanks Anthony

Hey! So yeah, by default javascript is not gonna run in the background without using a plugin of some kind or a service worker.

Thanks @jgw96 it's quite hard to find any info on this, (how to keep processes going that started in app open mode and continue in background when closed, like an upload), what is a service worker? do you have a link I could read up on to save you explaining? Thanks for your time. Cheers Ant

@antonfire Definitely, no problem at all! So service worker is a new spec that is currently supported in chrome and firefox (i think safari and edge are working on it) that basically allows a javascript thread to continue running even after you close a website or web app using it. Its mainly used for caching for offline progressive web apps. Not sure it would help you with what your trying to do, as its not really built for that. But, still a pretty cool thing to read up on. https://developer.mozilla.org/en-US/docs/Web/API/Service_Worker_API


When will logos be shown in Metamask?

Sorry if you've repeated the answer to this question. My PR got merged a couple of weeks ago but it's still not shown in Metamask. I'm assuming it will be shown in the next update?

It should be available when this repo and new MM extension will be released. It can take 2-3 weeks


infer-linux64-v0.14.0 build error in Ubuntu 16.04.3

I use vmware machine to build the infer-linux64-v0.14.0
$ cat /etc/issue
Ubuntu 16.04.3 LTS \n \l
run as
./build-infer.sh
[ERROR*][47539] *** ERROR 'Building native(opt) Infer'
[ERROR*][47539] File "atd/clang_ast_j.ml", line 1:
[ERROR*][47539] Error: I/O error: as -o 'atd/.InferGenerated.objs/InferGenerated__Clang_ast_j.o' '/tmp/camlasm02a83c.s': Cannot allocate memory
[ERROR*][47539] make[1]: *** [_build/opt/infer.exe] Error 1
Makefile:172: recipe for target 'src_build' failed

See #914, which looks similar. It looks like you're running out of memory. In particular you can try the workaround suggested here.


Switch to UI-Router

Fix #189, base for #238
This PR is not ready to merge since there are a couple of bugs/limitations. See the list below for an overview. If you encounter any bug or weird behaviour during testing, please add a comment to this PR. Please test @eScienceCenter/spacialists
Known Bugs/Problems

[x] Save & Continue in the unstaged changes popup doesn't work
[x] Sources & certainty in the certainty popup are stored, but only displayed after a page refresh
[x] Linked files are not displayed
[ ] After drawing a new geo object, the selected context is no longer selected (by design? bug?)
[x] Editing marker properties doesn't work (e.g. color or lat/lng)
[x] Linking contexts to geodata objects is not possible and may be falsely restricted
[x] Unlinked geodata has empty popup and displays empty context detail page
[x] When one opens a context detail page (/context/<id>) directly, the popup of the linked geo object doesn't get opened. When one opens a link to a geo object (/geodata/<gid>), the page stays empty due to several errors.

Beautify

[ ] Replace unused fields/methods
[ ] Replace unnecessary views in db
[ ] Rename controllers/services/etc. (e.g. image/photo => file)

External deps

[ ] Edit/delete geodata (depends on Leaflet.draw)
[x] Selecting a context in the tree sometimes displays the wrong context in the detail page (after necessary switch to npm #276, seems to be a bug in the sticky-state plugin) disabled sticky for now

Functionality should be almost done. Could you please all test this branch and have a look at the open bugs/questions in OP? Thanks! @eScienceCenter/spacialists

I fixed most of the remaining bugs/todos. There are still things that don't work 100%, but nothing critical. Maybe something for a 0.3.1 release :wink:
Could please anyone test and confirm this so we can finally release Cannae? Thanks! @eScienceCenter/spacialists


feature/prepare ingress for kubernetes v1.22

The extensions/v1beta1 and networking.k8s.io/v1beta1 API versions of Ingress is no longer served as of v1.22.
This PR make ingress object ready for kubernetes versions >= v1.22.
see: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22 for more details.
In addition it is tested on openshift 4.7 and can be used as a drop in replacement for the route feature branch if wanted.

@dzabel Thank you very much for the PR.
Can you please check if this can also work with .Capabilities.KubeVersion.Version or .Capabilities.APIVersions.Has ?
It seems that .Capabilities.KubeVersion.GitVersion is deprecated as it is mehtioned here: https://github.com/helm/charts/issues/20918

@bakito I was not aware of the deprecation, thank you for pointing it out.
I added a fix commit removing deprecated capabilities. Please have a look at it.


Resolve-DnsName prefers a failed IPv4 query to a successful IPv6 query

Steps to reproduce
Query DNS for the name myip.opendns.com against the OpenDNS server resolver1.opendns.com, it should echo your connection's public IP address.
NB. it will only retun an A record on an IPv4 connection, an AAAA record on an IPv6 connection; mixing them and querying for an A record using an IPv6 connection returns a "name not found" lookup fail. On my connection which supports both IPv4 and IPv6, this happens:
PS C:\> Resolve-DnsName -Name myip.opendns.com -Server resolver1.opendns.com
resolve-dnsname : myip.opendns.com : DNS name does not exist

A WireShark packet capture shows that this goes out over IPv6 and queries both A and AAAA records. The A record lookup gets a "DNS name does not exist" response - lookup fail. The AAAA record gets an IPv6 address back. PowerShell is showing me the fail, rather than the success.

NB. that I can specify -Type AAAA and get a valid result, but I can't see a way to choose whether Resolve-DnsName uses IPv4 or IPv6 for its connection, so I have no way of knowing whether I should specify -Type A or -Type AAAA. One will match the connection it uses and one will not, so I would have to try both. Which is annoying, given that it defaults to trying both for me already - the only problem is it throws away the more useful result in favour of the less useful result.
Expected result
The data from the successful query, because it's more useful.
Actual result
The IPv4 lookup fail takes precedence, I see an exception, and the successful query data is lost to the void.
Environment data
Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@HumanEquivalentUnit The cmdlet is not in the repo. It is in DnsClient module. Please use other channels to communicate with the module owners (Windows 10 feedback tool, UserVoice site, forums and so on).


Yum_package constraints

I'm not sure if this is a bug or a clarification of intended behavior.
If package foo ver 1.2 is installed on a client and 1.3 is available on the yum server and I have a resource like:
yum_package 'foo >= 1.2'

Running chef-client 12.2.1, this would be considered "up to date" if either 1.2 or 1.3 was installed.
After updating to chef-client 12.4.1, the chef-run is upgrading to 1.3, even if 1.2 is installed.  I'm guessing this change came from PR #3155.  My question is - which is the DESIRED behavior, the old or the new?
I would think the old way since anything greater than OR EQUAL should be considered acceptable.

Yeah, having :install act like an :upgrade is wrong.  If the constraint is satisfied by the installed packages, nothing should be done.  Probably all its doing is checking the candidate_version against the installed_version when it needs to be checking the installed_version against the version constraint before doing the candidate_version check.

I'm guessing this is the same root problem, but I could be wrong.
I've recently run into a slightly different situation.  Again this was not a problem with chef-client 12.2.1, but becomes a problem in 12.4.1.  using the same resource definition:
yum_package 'foo >= 1.2'
where version 1.3 is installed and both 1.2 & 1.3 are available from the yum server.
Ver 1.2 is identified as the "candidate" version and since I do not have allow_downgrade set to true, it errors saying the candidate verison is older ... even though 1.3 should meet the ">= 1.2" constraint.

Forgot to include log sample
Chef::Exceptions::Package: yum_package[glibc >= 2.12-1.7.el6] (cars-oraclient::11g line 9) had an error: Chef::Exceptions::Package: Installed package glibc-2.12-1.149.el6_6.5 is newer than candidate package glibc-2.12-1.129.el6

Manual yum search:
[root@test]# yum list glibc --showduplicates

Installed Packages
glibc.x86_64                                                              2.12-1.149.el6_6.5                                                               @extras
Available Packages
glibc.i686                                                                2.12-1.129.el6                                                                   base
glibc.x86_64                                                              2.12-1.129.el6                                                                   base
glibc.i686                                                                2.12-1.149.el6_6.5                                                               extras
glibc.x86_64                                                              2.12-1.149.el6_6.5                                                               extras
```

Any updates?

should be fixed by #6540


[RESTEASY-1709]

Create proxy for injected Application.

@ronsigal , is this PR still needed? If not, please close it, so that we cleanup the PR list a bit. Thanks!

@ronsigal , if it's needed, instead, does the "Do not merge" label still apply, or we can consider/review it and possibly merge?

ok, I'm closing this, it can be reopened if needed.

@asoldano, sorry, I missed this. The issue is still open and pretty thorny, as I recall. I'll try to get back to it.


[Feature request] Support multiple directories for preserveColor

technically the same as #10 issue but for preserveColor field
Since i have svg's in icons folder and public i need to specify both, i tried giving it a regex for both folders but it didnt work.
And also it would be nice if we could set it just to true so it would preserveColor for all svg's

@Nikjee
if preserveColor is Regx, it will match the SVG path, such as(in windows):
D:\project\unplugin-svg-component\examples\vue3-vite\icons\common\icon-addx.svg
so in my understanding, it should be able to meet your needs, you just need to find the correct Regx.

@Nikjee
if preserveColor is Regx, it will match the SVG path, such as(in windows):
D:\project\unplugin-svg-component\examples\vue3-vite\icons\common\icon-addx.svg
so in my understanding, it should be able to meet your needs, you just need to find the correct Regx.

Sorry my bad, problem is not reproducible anymore, my regex was working only on half of the files but now it is working correctly. Closing it.


Feature psalm static analysis

add psalm static analysis support

Psalm doesn't like the naming of the TempFileGZIPWriter class; I think that's just because it's not TempFileGzipWriter ... but I don't want to change that ...

@DavidGoodwin is that possible to suppress it?

Thanks!


Public site papers needs to be a list with a limited amount of entries

The list of papers needs to be limited to say 5, with a ... more(x) that when clicked will display the other x papers. In addition papers should appear above downloads.

Always show all papers (for now).


Credit Card endless loading loop

Describe the bug
Dear Adyen Team we have found an issue with the Credit Card where sometimes fall in an endless loading loop because cannot load the iframes.
To Reproduce
We do not have a specific "follow to reproduce" but we have a stack trace of the error:
edp-checkout-page-module-es2015.bc67bb5be7048eccdc49.js:1 GET https://checkoutshopper-live.adyen.com/checkoutshopper/securedfields/{CLIENTKEY}/3.2.6/securedFields.html?type=card&d=aHR0cHM6Ly93d3cuYWxsaWFuemRpcmVjdC5kZQ== net::ERR_CACHE_WRITE_FAILURE 200 (OK)
stacktrace noise....
edp-checkout-page-module-es2015.bc67bb5be7048eccdc49.js:1 GET https://checkoutshopper-live.adyen.com/checkoutshopper/securedfields/{CLIENTKEY}/3.2.6/securedFields.html?type=card&d=aHR0cHM6Ly93d3cuYWxsaWFuemRpcmVjdC5kZQ== net::ERR_CACHE_WRITE_FAILURE 200 (OK)
stacktrace noise....
securedFields.html?type=card&d=aHR0cHM6Ly93d3cuYWxsaWFuemRpcmVjdC5kZQ==:149 The devicemotion events are blocked by feature policy. See https://github.com/WICG/feature-policy/blob/master/features.md#sensor-features
Additional context
We know that it is happening most of the times in Google Chrome but we could not find any way to solve it or to reproduce it based on a pattern. The people that fall in the endless loading loop they refresh the browser and they are able to use the credit card normally.
The library version is 3.17
Thank you very much.

Hi @emidev98,
Thanks for opening this issue and our apologies for the delayed response! Are you still experiencing this issue? I haven't been able to reproduce it.
If this is still the case, could you please confirm your Google Chrome's version? It would also be relevant to know if you have any extension installed (and if this still happens with those disabled).

Hi @emidev98,
Thanks for opening this issue and our apologies for the delayed response! Are you still experiencing this issue? I haven't been able to reproduce it.
If this is still the case, could you please confirm your Google Chrome's version? It would also be relevant to know if you have any extension installed (and if this still happens with those disabled).

Hello @marcperez,
No problem, thanks for your time. Our engineering team investigated the topic but seems like there is no easy way to reproduce the bug (we even disabled all the extensions in the browser).
Our conclusion was to check the code of the securedfields but when we asked to the Adyen Support Team they declined the access to that code.
Regarding the Google Chrome version I do not remember the specific version but should be some from 80 to 88 version.

Hello @marcperez,
No problem, thanks for your time. Our engineering team investigated the topic but seems like there is no easy way to reproduce the bug (we even disabled all the extensions in the browser).
Our conclusion was to check the code of the securedfields but when we asked to the Adyen Support Team they declined the access to that code.
Regarding the Google Chrome version I do not remember the specific version but should be some from 80 to 88 version.

@marcperez one of our colleagues reproduced the issue with Google Chrome Versione 87.0.4280.66 (Build ufficiale) (a 64 bit).
We asked to her a lot of question but seems like she was not doing anything "strange" and also she had no extensions in the browser.

@marcperez one of our colleagues reproduced the issue with Google Chrome Versione 87.0.4280.66 (Build ufficiale) (a 64 bit).
We asked to her a lot of question but seems like she was not doing anything "strange" and also she had no extensions in the browser.

We have another colegue that falled in that issue with Google Chrome 88.0.4324.104 (Offizieller Build) (64-Bit)

I could reproduce it myself. I felled in this error with the next warning in the console

I followed the link but I could not find very much relevant info

We have detected that this bug was created because of a cache policy that some of our laptops have installed.
I'll procede to close the issue. Thank you for support!

That's good to know, thanks @emidev98!


Bugfx/revert 2783 2883

revert #2783
revert #2883

Revert documentation as well.
https://github.com/apache/incubator-weex-site/pull/491


[WIP] Switch to base toolkit package

Resolves #6

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

I've seen this error before. I forget how I fixed it. I'll come back to this later.
2023-06-07T19:48:44.3180221Z import: 'openff.nagl'
2023-06-07T19:48:46.9120512Z Traceback (most recent call last):
2023-06-07T19:48:46.9121926Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/test_tmp/run_test.py", line 2, in <module>
2023-06-07T19:48:46.9122469Z     import openff.nagl
2023-06-07T19:48:46.9169519Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/__init__.py", line 6, in <module>
2023-06-07T19:48:46.9170137Z     from openff.nagl.nn._models import GNNModel
2023-06-07T19:48:46.9170845Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/nn/__init__.py", line 7, in <module>
2023-06-07T19:48:46.9171273Z     from ._containers import ConvolutionModule, ReadoutModule
2023-06-07T19:48:46.9171963Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/nn/_containers.py", line 6, in <module>
2023-06-07T19:48:46.9172395Z     from openff.nagl.molecule._dgl import DGLMolecule, DGLMoleculeBatch
2023-06-07T19:48:46.9173107Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/molecule/__init__.py", line 5, in <module>
2023-06-07T19:48:46.9173490Z     from ._dgl.molecule import DGLMolecule
2023-06-07T19:48:46.9174188Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/molecule/_dgl/__init__.py", line 1, in <module>
2023-06-07T19:48:46.9174594Z     from .batch import DGLMoleculeBatch
2023-06-07T19:48:46.9175269Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/molecule/_dgl/batch.py", line 5, in <module>
2023-06-07T19:48:46.9175667Z     from .molecule import DGLBase, DGLMolecule
2023-06-07T19:48:46.9176376Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/nagl/molecule/_dgl/molecule.py", line 4, in <module>
2023-06-07T19:48:46.9176799Z     from openff.toolkit.topology.molecule import Molecule as OFFMolecule
2023-06-07T19:48:46.9177726Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/toolkit/topology/__init__.py", line 1, in <module>
2023-06-07T19:48:46.9178230Z     from openff.toolkit.topology.molecule import (
2023-06-07T19:48:46.9178949Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/toolkit/topology/molecule.py", line 57, in <module>
2023-06-07T19:48:46.9179367Z     from openff.toolkit.utils.constants import DEFAULT_AROMATICITY_MODEL
2023-06-07T19:48:46.9180083Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/toolkit/utils/__init__.py", line 9, in <module>
2023-06-07T19:48:46.9180488Z     from openff.toolkit.utils.toolkits import (
2023-06-07T19:48:46.9181173Z   File "/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/toolkit/utils/toolkits.py", line 134, in <module>
2023-06-07T19:48:46.9181620Z     from openff.toolkit.utils import all_subclasses
2023-06-07T19:48:46.9182488Z ImportError: cannot import name 'all_subclasses' from partially initialized module 'openff.toolkit.utils' (most likely due to a circular import) (/home/conda/feedstock_root/build_artifacts/openff-nagl_1686167106821/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.11/site-packages/openff/toolkit/utils/__init__.py)

Thanks Matt :) That's the best commit message I've ever seen!

all chatGPT - there were a couple of edgier ones I passed on 🙃

I think this has been superseded by #11. #11 does include rdkit as a required dependency, which we could remove in the future -- looks this error was the same as the one in https://github.com/conda-forge/openff-fragmenter-feedstock/pull/7#issuecomment-1668505500 and is therefore resolved by the new toolkit release.

Aha ... this must be what I was thinking of!


Provide get_setup_version function

Currently, autover requires that projects duplicate the get_setup_version() and embed_version() functions in their own setup.py files. Before pip 10 arrived, pip did not support "build time" dependencies, so having the functions in each project's setup.py and including a way to download version.py if it wasn't already available made sense. But now with pip 10, projects can declare that the appropriate version of either autover or param must be available before running setup.py. This PR takes advantage of build time dependencies to move get_setup_version() into version.py itself, and to eliminate embed_version().
In any case, setup.py dependencies won't affect users installing via wheel or conda packages (i.e. this change won't affect the majority of end users). Packagers and users installing from git source will be affected, but pip>=10 and conda build both support declaring build time dependencies. But to avoid confusion for users installing from git source who do not yet have pip 10, projects could include something like the following in their setup.py:
try:
    import param
    assert param.version >= whatever
except:
    raise "Building xyz requires param >= whatever. Please upgrade to pip>=10 and try again. Alternatively, install the build dependencies manually first (e.g. `pip install --upgrade "param>=whatever"` or `conda install -c pyviz "param>=whatever"`)')

Note that I have not updated tests etc, but will go on to do that if the idea of this PR is accepted in principle.
The alternative approach of bundling version.py in a project will also be simplified by these changes, because projects will be able to import the get_setup_version function from the bundled file, rather than having to define it in setup.py.
I have also added get_setup_version2(), which is the same as get_setup_version(), except that it's configured via setup.cfg. This allows the following kind of thing in setup.cfg:
[metadata]
name = xyz
version = attr: param.version.get_setup_version2
...

as an alternative to the following kind of thing in setup.py:
setup_args = dict(
    name='xyz',
    version=param.version.get_setup_version(__file__, "xyz", ...),
    ...

There are some to-dos:

should get_setup_version fns be available via Version rather than as standalone fns?
rename the get_setup_version fns?

This all sounds good though if you want to offer both versions of get_setup_version, I would call get_setup_version2 something like get_setup_version_from_config  (or similar). As they do the same thing I don't see why we shouldn't offer both versions and let autover users decide.

@jlstevens: once the tests pass, I'll merge this PR unless you object.
To summarize, the important changes are:

use build time dependencies rather than auto downloading/embedding
provide get_setup_version() and get_setupcfg_version() functions in autover itself

After autover's released, I'll need to add some minimum version specifications in this repository.
After param's updated, I'll need to update the param tests in this repository.

Looks good though I am seeing this comment about the place:
# TODO: should these be available via Version instead?

I would either implement it or file an issue instead of leaving TODO items in comments. Also, do remember to squash and merge!


is there a way to measure usage of QAT?

I wonder get a way to monitor usage of QAT .
Just like  usage of CPU or bandwidth of  Ethernet card .

Hi @lioncfliu ,
There is no explicit way to measure the usage of QAT card like CPU but instead we can check the count for number of requests over time via cat /sys/kernel/debug/<QAT_device_id>/fw_counters
for QAT1.7 devices  and cat /proc/icp_<device_number>/qat for QAT1.6 devices which is answered in
the below thread in terms of QAT utilization.
https://github.com/intel/QAT_Engine/issues/78
Example Commands:
For QAT1.7 :
cat /sys/kernel/debug/qat_c6xx_0000:05:00.0/fw_counters
For QAT1.6 :
cat /proc/icp_dh895xcc_dev0/qat
Please check and let us know if you need further information.

thanks a lot .
I am going to try it out .

Hi @Yogaraj-Alamenda ,
Do I have to install any additional components to view "fw_counters" .
I observed only the following under /sys/kernel/debug
ls /sys/kernel/debug/qat_dh895xcc_0000:03:00.0/
dev_cfg  transport

@sonyarpita  Normal make install will do,  Additional components installation not needed.  Am able to see the below folders when installed with root using the  latest driver (4.14) .Could you please let us know what is the driver version you are using ?

 ls /sys/kernel/debug/qat_dh895xcc_0000\:05\:00.0/
cnv_errors  dev_cfg  fw_counters  heartbeat  heartbeat_failed  heartbeat_sent  transport  version

I have inboxed driver -
modinfo qat_dh895xcc
filename:       /lib/modules/5.13.0-rc3/kernel/drivers/crypto/qat/qat_dh895xcc/qat_dh895xcc.ko
version:        0.6.0
description:    Intel(R) QuickAssist Technology
firmware:       qat_895xcc_mmp.bin
firmware:       qat_895xcc.bin
author:         Intel
Could you please point me to the latest driver location .

Hi @Yogaraj-Alamenda ,
I want to know conclusion of this thread.
not fw_counter, is there any other way to measure usage like cpu?
If not yet, do you have support plan in future? or if fix code, is it possible?

@jhseodev  At present, QAT Engine doesn't have any facility within to measure CPU usage. We rely on other CPU Usage tools to measure.  There is no plan for adding it within QAT Engine in near future.  However if you can submit any PR that supports within QAT Engine we will be able to test and integrate it.
Latest Driver version can be found here

@Yogaraj-Alamenda
Sorry, I asked a question about to measure usage of QAT.
"not fw_counters, any other way to measure usage of QAT."
Thank you for comment.


[GUI] invalid device name: the text has not been translated

The text has not been translated into Portuguese.
Steps to reproduce the problem:

Click "Novo dispositivo"
Select a template
Click "Voltar"
Fill in device name with invalid data
Click "Salvar"

Affected Version: 58.2-20190321

Test OK.
The text has been translated.

Tested on Version: 62.3-20190603


Add option for paneling by groups

Clinical Summary
Add an option to create multiple eDish panels for a categorical variable (e.g. treatment). This could be instead of, or in addition to, the current color-by-group functionality.
Details
Probably possible to set this up via webcharts.multiply, but definitely not trivial with all of the custom interactivity, so leaning towards skipping this for now.

Consider for v2.0.

The more I think about this, the less I like the idea. Going to close this for now. We can re-open and discuss further if/when users start requesting this functionality.


Branch sync

🔥 Summary
Just a sync of release branch

replaced by the new PR
https://github.com/casper-network/casper-blockexplorer-frontend/pull/274


api: static bootstrap proto definition.

This patch adds a static bootstrap proto that is expected to be provided
on the filesystem or command-line. This should enable Envoy to then
either fetch the rest of config from disk or reach out to the various
management servers for the rest of the APIs.
Fixes #93.

@mattklein123 @tschroed (for #93 and other G considerations) @lizan/@rshriram (for Istio)

@mattklein123 My thinking is that ClusterSource will be a parameter to a factory method that generates Subscription objects for a given API. For SDS/EDS, it will replace SdsConfig (it's fairly easy to see how the existing SdsConfig fields can map into ClusterSource). WDYT?

Sure sgtm.

If there's no additional feedback, can we approve this so I can merge, would like to use the ClusterSource proto in my next EDS PR. Ta.


TabView breaks Navigation and causes memory leaks.

Describe the bug
Just by adding the TabView control to a page, it breaks Navigation and causes memory leaks.
Steps to reproduce the bug
App2.zip
Steps to reproduce the behavior:

Build App attached above.
Run App.
Select Page navigation buttons and observe Page deconstructors being called.
Close App.
Uncomment TabView control on Page 1 Xaml.
Run App.
Select Page navigation buttons and observe Page 1 deconstructor not being called causing memory leaks.

Expected behavior
Page 1 deconstructor should be called so its memory resources can be reclaimed.
Screenshots
n/a
Version Info

NuGet package version:
[Microsoft.UI.Xaml.2.6.0-prerelease.210315002]

Windows 10 version
Saw the problem?

October 2020 Update (19042)
Yes

Device form factor
Saw the problem?

Desktop
Yes

Additional context
This issue affects every release of the TabView control, including those in the Community Toolkit, so the problem predates integration into WinUI.

https://github.com/microsoft/microsoft-ui-xaml/issues/3597
Could this be related to the issue I reported last year?

@duke7553 , I wouldn't be able to speculate on that.  Just adding the control and not using it was enough to cause Navigation to leak.  Have a look at the code.  I don't have much experience with the control, I was looking into an issue someone else had posted.  Would be helpful to have other eyes on the code samples.
There are curious interactions with Navigation Frame and NavigationCacheMode.  NavigationCacheMode="Enabled" seems to trigger a memory leak regardless of what controls you are using.  TabView seems to have additional problems.
For additional context, you can look here: https://github.com/microsoft/microsoft-ui-xaml/issues/4713

I have refined the test so that it can be run outside of Visual Studio 2019, since it adds its own memory leak artifacts.
This version of the test also manifests a previously closed issue: https://github.com/microsoft/WindowsTemplateStudio/issues/2726
TabView1.zip

The following testing tool is mostly self explanatory if you have a close look at the code: https://github.com/Noemata/RosettaNavigation
The tool may be used to test both performance and Navigation Frame memory issues for both WinUI and UWP.


Projection issue with Equi7 and GeoTiff source

Describe the bug
It looks like there's an issue with projection detection or reprojection for the Equi7 projection when I load a COG.
To Reproduce
Sandbox: https://codesandbox.io/s/geotiff-reprojection-forked-p3utjd?file=/main.js
The url for the epsg4326 variable is shown over the OSM baselayer.
If you change to the url for the equi7 variable, it's shown "somewhere" but definitively not over the baselayer.
The files though are at the same location, just one COG is in EPSG:4326 and one is in Equi7 (see https://github.com/TUW-GEO/Equi7Grid).
Loading them in QGIS shows them at the same location.
Expected behavior
The Equi7 COG should be shown at the correct location (and over the baselayer). Not sure what the best way to achieve this is.
The baselayer should stay in the default projection (EPSG:3857) and the COGs should be reprojected.
I have the correct proj string (+proj=aeqd +lat_0=53 +lon_0=24 +x_0=5837287.81977 +y_0=2121415.69617 +datum=WGS84 +units=m +no_defs), but I'm not sure how I can assign it to the source. In other OL sources such as GeoJSON it seems I can set a source projection, but it seems for GeoTiff source I can't do that?!
fyi @tschaub :-)

view: source.getView() will set the view to to COG projection and reproject the baselayer.
To display the COG in the baselayer projection see https://codesandbox.io/s/geotiff-reprojection-forked-21zp8q
or for multiple COGs see https://openlayers.org/en/latest/examples/multiple-cogs.html
The Equi7 COG is returning EPSG:4326 as its projection but the coordinates are not EPSG:4326

I tried overriding that https://codesandbox.io/s/geotiff-reprojection-forked-umgdey but there is no RGB data visible

Thanks!
Ah yes, indeed, I shouldn't have set the View from the source. I guess I was too tired.
@SerRichard can you say why the COG is returning EPSG:4326 as projection?

How are you setting the projection?  OpenLayers is looking for either ProjectedCSTypeGeoKey or GeographicTypeGeoKey which is the numeric part of a projection code which follows EPSG:.  For custom projections that should be 32767.

The geokeys in the equi7 file are:
{
  GTModelTypeGeoKey: 1,
  GTRasterTypeGeoKey: 1,
  GTCitationGeoKey: 'Azimuthal_Equidistant',
  GeographicTypeGeoKey: 4326,
  GeogCitationGeoKey: 'WGS 84',
  GeogAngularUnitsGeoKey: 9102,
  GeogSemiMajorAxisGeoKey: 6378137,
  GeogInvFlatteningGeoKey:<PHONE_NUMBER>63,
  ProjectedCSTypeGeoKey: 32767,
  ProjectionGeoKey: 32767,
  ProjCoordTransGeoKey: 12,
  ProjLinearUnitsGeoKey: 9001,
  ProjFalseEastingGeoKey: 5837287.81977,
  ProjFalseNorthingGeoKey: 2121415.69617,
  ProjCenterLongGeoKey: 24,
  ProjCenterLatGeoKey: 53
}

Using https://github.com/matafokka/geotiff-geokeys-to-proj4 it results in +proj=aeqd +lat_0=53 +lon_0=24 +x_0=5837287.81977 +y_0=2121415.69617 +a=6378137 +b=6356752.314245179 +datum=WGS84 +no_defs
This is similar to the original projection. a + b are not meant to be there, but units is missing.

GeogSemiMinorAxisGeoKey <=> b
GeogSemiMajorAxisGeoKey <=> a (is the additional a and b an issue? not sure)
GeogLinearUnitsGeoKey or ProjLinearUnitsGeoKey <=> unit -> ProjLinearUnitsGeoKey ist set to 9001, which is meters so it might be a issue in the js lib that units is not generated

So to me it looks like the projection detection in OL (or geotiff.js?) is just not capable to cater for this, right?

OpenLayers simply checks ProjectedCSTypeGeoKey and GeographicTypeGeoKey (in that order) for values which are truthy and not 32767, which explains the EPSG:4326.  It does not build proj4 definitions, and 32767 or no projection specified in each setting defaults to the view projection.
Some documents suggest 32768 to 65535 can be used for "Private User Implementations".  If the COG specified 32768 and OpenLayers hardcoded the required proj4 definition is should reproject.

We are in a setting where we can't change the GeoKeys in such a way, so I need to look for another option.
Do you think a PR for an option in the GeoTiff to override the projection (similar to GeoJSON) would be accepted? Then I'd try to work on that, I think.

A projection option in the constructor could be used to override anything found (or not found) in the file.  It would only be needed if the COG contains no values other than 32767 or one which is inappropriate such as 4326 in your case.


FailedInflightCheck prevents node lifecycle?

Version

Karpenter Version: v0.24.0

Server Version: version.Info{Major:"1", Minor:"25+", GitVersion:"v1.25.6-eks-48e63af", GitCommit:"9f22d4ae876173884749c0701f01340879ab3f95", GitTreeState:"clean", BuildDate:"2023-01-24T19:19:02Z", GoVersion:"go1.19.5", Compiler:"gc", Platform:"linux/amd64"}
Expected Behavior
karpenter does remove stale nodes after ttlSecondsAfterEmpty or ttlSecondsUntilExpired threshold reached.
Actual Behavior
We have stuck nodes that seem to not get deleted after the expiry time. It is set to
      ttlSecondsAfterEmpty: 30
      ttlSecondsUntilExpired: 7200 # 2 hours

We had similar issues when the GPU was not recognized: https://github.com/aws/karpenter/issues/2737
What we can see with those nodes, they all have one thing in common, a event of type FailedInflightCheck of different nature:
kubectl describe nodes | grep FailedInflightCheck
  Warning  FailedInflightCheck  5m8s (x30 over 4h55m)  karpenter  Expected 200Gi of resource ephemeral-storage, but found 20959212Ki (10.0% of expected)
  Warning  FailedInflightCheck  5m15s (x30 over 4h55m)  karpenter  Expected 200Gi of resource ephemeral-storage, but found 20959212Ki (10.0% of expected)
  Warning  FailedInflightCheck  5m16s (x1504 over 10d)  karpenter  Expected resource "nvidia.com/gpu" didn't register on the node
  Warning  FailedInflightCheck  5m18s (x30 over 4h55m)  karpenter  Expected 200Gi of resource ephemeral-storage, but found 20959212Ki (10.0% of expected)
  Warning  FailedInflightCheck  5m19s (x30 over 4h55m)  karpenter  Expected 200Gi of resource ephemeral-storage, but found 20959212Ki (10.0% of expected)
  Warning  FailedInflightCheck  5m24s (x1504 over 10d)  karpenter  Expected resource "nvidia.com/gpu" didn't register on the node

Is it possible, that this check prevents karpenter from recognizing the node as "up" and thus skips any other cleanup steps?

Name
Status
Version
Pods
Age

ip-10-23-16-174.us-west-2.compute.internal
Ready
v1.25.7-eks-a59e1f0
19
27d

ip-10-23-100-154.us-west-2.compute.internal
Ready
v1.25.7-eks-a59e1f0
23
27d

ip-10-23-189-119.us-west-2.compute.internal
Ready
v1.25.6-eks-232056e
10
27d

ip-10-23-234-114.us-west-2.compute.internal
Ready
v1.25.7-eks-a59e1f0
11
27d

ip-10-23-249-252.us-west-2.compute.internal
Ready
v1.25.7-eks-a59e1f0
11
27d

Steps to Reproduce the Problem
see above
Resource Specs and Logs
see above
Community Note

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

It looks like the GPU didn't register again. Are you running the nvidia device plugin daemonset?
Yes, the nvidia device plugin daemonset is deployed, and works 95% of the time.

Can you post your Provisioner and AWSNodeTemplate. I'm curious why the ephemeral storage isn't matching.
An engineer will post them in a few minutes, but we have recently made some changes for the bottlerocket AMI (changing the deviceName: /dev/xvdb size from 200G to 500G. The default disk size for deviceName: /dev/xvdb, is 20G, see https://github.com/bottlerocket-os/bottlerocket/blob/develop/tools/testsys/src/aws_resources.rs#L276
That being said, it looks like the nodes with the small disk size, coming from a previous karpenter configuration, confuse the current carpenter setup and it does not proceed with the cleanup. IMPORTANT the nodes are still being used by the cluster workloads, so karpenter.sh/initialized apparently has no effect on the READY state of the cluster, on what is being scheduled or not.

Do you see the karpenter.sh/initialized label set to true on the nodes?
No, it does not show up. So I assume FailedInflightCheck prevents karpenter to progress in the lifecycle (i.e. applying karpenter.sh/initialized)

linking this issue here, we're having a problem in the same area: https://github.com/aws/karpenter-core/issues/341
Our use case is slightly different tho: Karpenter is spinning inf1 instances to satisfy cpu and memory requests (no workloads is requesting neuron gpus, so we suspect the choice is made on a price basis) and  we're not supplying any neuron kubernetes plugin, so the resource fails to register.
However, the node from a kubernetes perspective is ready, and the kubernetes scheduler is able to schedule pods on it; Karpenter node lifecycle, on the other end, is heavily disrupted as karpenter.sh/initialized is missing.

@cebernardi For now, the workaround for the inf1 issue is to install the neuron device plugin or exclude inf1 types from your provisioner.
https://awsdocs-neuron.readthedocs-hosted.com/en/v1.16.2/neuron-deploy/tutorials/tutorial-k8s.html

We try to not take action on nodes that don't initialize so that the root cause can be fixed.

Ok, I take this as: FailedInflightCheck (and similar) will prevent karpenter.sh/initialized. This is intentional, and will allow kubernetes administrators to take action.

Expiration may be a reasonable thing to take action on even if the node doesn't initialize, but it would likely run into the same issue on the next launch... It's probably best to monitor nodes that do not initialize and investigate.

What about letting the customers decide this: How about we add a new ttlSecondsUntilExpiredUninitialized, which would, if not set, behave as today, but would allow us to specify a timeout for these cases?

@blackliner We have updated the initialization logic starting in #3408. This change means that we would only wait for the extended resources to register that we have simulated based on pod requests. e.g. if you have a set of pods that don't request neuron and then launch an inf instance, we won't wait for the neuron extended resource to register anymore.
This new logic was released in the release candidate v0.28.0-rc.1 which should get promoted to stable v0.28.0 sometime this week. If you want to try the release candidate in the meantime, you could see if it solves the issue you are running into with inf instances.

I am the one with g4dn instances and GPU issues, inf was @cebernardi , but I get the gist :D thanks

Ack, sorry, I skimmed through the thread super quick, but yeah, the same concept applies, any extended resource that isn't asked for won't block initialization anymore.

Closing for now, will report back once we upgrade the version to 0.28 and it happens again


error flutter firestore 0.12.9

Just switch to AndroidX -> https://flutter.dev/docs/development/packages-and-plugins/androidx-compatibility

Thank you so much for sending email to me And i used androidx but does not help maybe i missed something
Sent from Yahoo Mail on Android
On Tue, Aug 6, 2019 at 5:29 PM, Emanuele<EMAIL_ADDRESS>wrote:
Just switch to AndroidX -> https://flutter.dev/docs/development/packages-and-plugins/androidx-compatibility
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[REVIEW] Project Zone

Project link
https://github.com/rockingrohit9639/project-zone

some thoughts:

consider removing the wiki if not used
maybe change the screenshot so a gif or demo video

some thoughts:

consider removing the wiki if not used
maybe change the screenshot so a gif or demo video

Thanks for your review, I will definitely look into it.

Looks good

make banner a little small in README.md
Add a security policy

Reviewed on live stream https://www.youtube.com/watch?v=T14I0ECfPlU
Looks great some thoughts...

great to see releases being used
great README
is the .github/.workflows needed?


⚠ Node - sg-67539fc1.yalaso.top has degraded performance

In 33474cc, Node - sg-67539fc1.yalaso.top (http://sg-67539fc1.yalaso.top/api/v1/ping) experienced degraded performance:

HTTP code: 200
Response time: 23179 ms

Resolved: Node - sg-67539fc1.yalaso.top performance has improved in cc0da7a.


Recursive link in README

The README has a link to http://damienclarke.me/code/posts/writing-a-better-noise-reducing-analogread which redirects back to GitHub hence is of little use. Is the text of the referred article available? Maybe it could be added to the GitHub wiki.

Agreed. Regarding reading the article, the internet archive comes to the rescue: https://web.archive.org/web/20170409184051/http://damienclarke.me/code/posts/writing-a-better-noise-reducing-analogread


corrected command for installation of cuda command-line tools

docs correction for 16214

Nagging Assignee @protoget: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 60 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 90 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

This file has conflicts. Can you update the PR or close it if it is obsolete. Thanks!

new install_linux.md takes care of it


Marked packages and ActivityHandler as deprecated

@tracyboehrer what is the up2date way to implement the handling of activities? docu links or some hints are welcome 🙂

@GeBeater The same way will work.  This was marked as deprecated simply to help draw attention to the entire Java SDK end-of-live plan.  Java SDK is in sustained engineering and will be retired November 2023.

@tracyboehrer thanks for the info. we are started some month ago to develop a new product that partly based on this Java SDK 🙈😬 do you have a recommendation, which sdk will receive the most support in the long term?

@GeBeater Both the DotNet and Javascript SDKs will continued to be developed.


locking functionality

hi there, is there a locking functionality to ensure e.g. write once or queued/ structured writing to the table from multiple processes?

Hi. Tables are immutable. There is no way for multiple processes to write to the same memory region. If they manipulated a table, like to change the values in a column with set_column, then they would be producing a copy of the Table instance. All the methods on Table that appear to modify a Table return a new Table instance for that reason.
Only the modified column's array data would be copied - the other arrays would be unchanged.


'By stage' page layout adjustment

I'd like to summarize the new features and changes we are gonna implement on the 'by stage' page.
First of all, we can provide an expandable panel on top of the drop rate table. It may contain:

Sample size filter (#181)
Drop rate filter
Stage availability filter (#323)
...

If more than one filter is applied (which means that not all records are shown), we can show something to remind the user.
Second, under the drop rate table, we can show some other statistical data for this stage, such as:

Drop Pattern Results (#208)(https://github.com/penguin-statistics/backend/pull/43)
Mark as Favorite Stage (#370)
Maybe more graphs?
More stage statistical data? For example, sample num in 24 hrs
Export CSV (#261 )
Stage Info, such as AP cost and min clear time (#342)

Third, there are some other UI changes.

Switch to adjacent stages using small arrows (#344)
any others?

New release candidate 3.3.7 has implemented large portion of this issue except the following items:

Export CSV
More stage statistics

New release candidate v3.3.7 has implemented a large portion of this issue except the following items:

Export CSV
More stage statistics

For 'Export CSV', I think we may not need this...? Since the graph library you are using has this feature.
For 'More stage statistics', now it is under 'Site Stats', so it's good now.
I think this issue can be marked as done after the release.


conformance: enable HTTPRouteBackendRequestHeaderModifier

https://github.com/kubernetes-sigs/gateway-api/pull/2820
xref: https://github.com/envoyproxy/gateway/pull/3336

blocked on https://github.com/envoyproxy/gateway/issues/2572


Accept header fix!

This is tough example of codegeneration issues;

I could not fix problem on the backend as from backend side things working fine
I could not fix problem in generator (as generator doesn' support anything else than json and they force things intentionally)

This fix removes silly override that forces JSON accept header where others like zip file can exist

Question is how we handle this fix. We need to provide custom templates :/ in long run as it will be overridden later.

Question is how we handle this fix. We need to provide custom templates :/ in long run as it will be overridden later.

Propose the fix with a pull request to OpenAPI Generator. Clearly state the reason why this is a bug and how we should not be overriding headers like this.
Use template locally in the meantime and hope the PR gets accepted.

Done fix but very low hopes on getting it merged:
https://github.com/OpenAPITools/openapi-generator/pull/10337
Having issues with the templates (it is not possible to isolate templates to some small chunk of code so I need to push whole template which is going to case issues.

Instead of pushing tons of template files I settled with more smooth approach by removing 500 error code. API will still return errors normally and they will be parsed just fine but this time Accept header problem is mitigated.


Equal Sum Partition Problem

Description
I want to add popular data structure and algorithms problem named :"Equal Sum Partition" which is a very famous problem and can enhance one's problem solving skill. So I want to add this problem in this project with proper comments with mentioning time complexity of every step in well mannered way ,so please assign me the issue of this problem.
The problem is to tell if a given array can be divided into 2 subsets such that sum of both the subsets is same.
For Example:
Input: 1 5 11 5
Output: True     (as there can be two subsets (1,5,5) and (11) such that sum of both are same.)
I am a participant at GSSOC'22 .
@Lakhankumawat  I want to work on this issue , can u please assign this to me ?
Thank you!
Domain
Data Structure
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

I would like to work on it under Gssoc22 @iamgunjan


McMurdo data has gotten a bit munched in data_files

See specimens line 1358.
I will re-download this to fix the repo, then try to figure out where it is getting messed up.

I re-downloaded McMurdo and ran through all the tests.  The weird extra column didn't show up again, so hopefully it is fixed!


[spi_device] Write enable (WREN) and write disable (WRDI) should be handled in hardware

The write enable command (WREN, 0x06) sets the write enable latch (WEL) bit, while the write disable command (WRDI, 0x04) clears it. The host sends WREN every time before "writing" to the device, e.g. before chip/sector erase or page program. Since handling this in software would introduce additional latency; WREN & WRDI commands should be handled in hardware.
cc @cfrantz @a-will

I remember @a-will @cfrantz and @tjaychen and I had discussion about this. The conclusion was to not implement the feature in HW at the first silicon due to the schedule issue and assumed it is OK to be processed in SW (no latency issue at all here).
If WE is 0, and host sends WREN and PROGRAM commands, SW can process WREN first then PROGRAM, as CMD_FIFO has multiple entries. Then PROGRAM can set BUSY 1. Host will poll the BUSY status to send next PROGRAM commands.
I don't see any side effect on this.

I remember @a-will @cfrantz and @tjaychen and I had discussion about this. The conclusion was to not implement the feature in HW at the first silicon due to the schedule issue and assumed it is OK to be processed in SW (no latency issue at all here).

That is not strictly true. Hosts can check the status register immediately following a WREN command, and that would be too fast for firmware to make the observable change. We're playing fast and loose without hardware handling of WREN, but I can't claim to have seen the host that behaves in the way described.

What happens if the host system issues WREN then read back the STATUS but it shows still WE bit is 0?
Honestly, I don't want to revise the design as we are approaching the RTL freeze. If the status read is rare and host is able to handle the incorrect status bit , then I think it is ok to move on?

On Apr 4, 2022, at 4:37 PM, Alex Williams @.***> wrote:
I remember @a-will https://github.com/a-will @cfrantz https://github.com/cfrantz and @tjaychen https://github.com/tjaychen and I had discussion about this. The conclusion was to not implement the feature in HW at the first silicon due to the schedule issue and assumed it is OK to be processed in SW (no latency issue at all here).
That is not strictly true. Hosts can check the status register immediately following a WREN command, and that would be too fast for firmware to make the observable change. We're playing fast and loose without hardware handling of WREN, but I can't claim to have seen the host that behaves in the way described.
—
Reply to this email directly, view it on GitHub https://github.com/lowRISC/opentitan/issues/11869#issuecomment-1088123369, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJDG3ROPMIZRFRL5GKIAJTVDN4KTANCNFSM5SPN3NTA.
You are receiving this because you were assigned.

What happens if the host system issues WREN then read back the STATUS but it shows still WE bit is 0? Honestly, I don't want to revise the design as we are approaching the RTL freeze. If the status read is rare and host is able to handle the incorrect status bit , then I think it is ok to move on?

 I can't exactly give guidance about what response the host application would have, since the severity and action to take would depend on the context.
I would also advocate for us getting this done. If you're concerned about schedule, shall we chat about options?

@eunchan i think is out for a few days, so just to summarize:

if we have a well behaving host that always does WEN then PROGRAM, I think the claim is that this will be okay since we can rely on the second operation setting "BUSY=1" to stall the operations.

However if we have a host that sets the status and reads it back, we would be a non-compliant device since it will look like we "ignored" the host's request to set the status.  Is that correct?

I understand the reason of the HW-processed WREN/ WRDI commands. @domrizz0 may comment about the schedule :)

https://docs.google.com/document/d/1ogKkYK3G9XCXBwhdflMmyRE4SBvUXvBRhb3zHY7QyvQ/edit
Here's the summary doc. Please take a look.

Recent PRs addressed this feature request. Can we close this issue?


Update various README files to be less specific about the supported JavaScript features

By being less specific about which exact JavaScript features are required for the default vs legacy build, we don't need to worry about keeping multiple README files up-to-date.
These README files will now refer back to the FAQ for current browser/environment support information.

Sounds like a good idea; thanks!


Tenacity not able to correctly adapt to different threadpool and command key property

Raised an issue : https://github.com/yammer/tenacity/issues/36 4-5 months back.
Even though tenacity allows us to give separate threadpool and command keys to the command , it does not allow us to set a TenacityPropertyKey's timeout without giving threadpool and circuit-breaker properties for the command(thus effectively creating a new threadpool for each command key).  But timeout is a command key property in hystrix and hystrix originally honours timeout at command level rather than threapool level .
This may also lead to problems in circuit breaker.

There's another constructor to TenacityCommand which should help which allows you to select a different threadpool. By default, TenacityCommand uses a threadpool by the same name a your name. This should allow you to share threadpools if that is what are you trying to do.
protected TenacityCommand(TenacityPropertyKey commandKey,
                              TenacityPropertyKey threadpoolKey) {
        super(HystrixCommand.Setter.withGroupKey(tenacityGroupKey())
                .andCommandKey(commandKey)
                .andThreadPoolKey(threadpoolKey));
    }

https://github.com/yammer/tenacity/blob/master/tenacity-core/src/main/java/com/yammer/tenacity/core/TenacityCommand.java#L18-L23
Threadpools and Circuitbreakers are only created on-demand, meaning if you use them in a TenacityCommand that is invoked or executed.
If you want you can change a TenacityPropertyKeys timeout on the fly by doing something like this:
ConfigurationManager
   .getConfigInstance()
   .setProperty(TenacityPropertyRegister.executionIsolationThreadTimeoutInMilliseconds(TenacityProepertyKey), NEW_TIMEOUT_IN_MILLIS);


Repeated query string keys are transformed into bracket notation

I'm dealing with an API which requires me to add the same querystring key multiple times, where the order is significant.
var sa = require('superagent')
sa.get('http://my-api.com/user?sort=lastName&sort=firstName').end(...);

But the request is sent to 'http://my-api.com/user?sort[0]=lastName&sort[1]=firstName'. However, the API does not accept this format.
The issue is caused by passing a hard-coded true to url = parse(url, true); in Request.prototype.request where parse is defined: var parse = require('url').parse;.
Is it a viable solution to surface this boolean option?

Which version of superagent?

Sorry, I somehow had 0.21.0 installed. Only started this project last week, so not sure how/why npm did that.
I've got 1.1.0 now and it works just fine :)


ContactView

Platforms Impacted

[X] iOS
[ ] macOS

Description of changes
ios/FluentUI/People Picker/ContactView.swift
This is a new View that will display a "Contact" which consists of an AvatarView and either 1 or 2 UILabels. There are two methods to create a ContactView. Both methods allow the consumer to provide an AvatarView. The difference between the two methods is the text being passed in. The first method allows a first name and a last name which will create two UILabels. The second method allows for a more generic single string which I expect to be either a phone number, email address, or a first name. Consideration for Right-To-Left locales was made and deemed unneccessary due to the fact that we are center-aligning the text when provided a first and last name.
ios/FluentUI/Core/Colors.swift
Added a struct to specify the colors being used for the labels within a `ContactView".
ios/FluentUI.Demo/FluentUI.Demo/Demos/ContactViewDemoController.swift
This is a demo within the demo app that contains 4 possible ContactViews. It includes:

ContactView with image
ContactView without an image and using initials instead
ContactView without an image and using an email address
ContactView without an image and using a phone number

Things to address

Use a pawn / placeholder image instead of a "#" for emails or phone numbers.
Is it fine to use a grey colour background since the background of a ContactView is white in order to see the bounds?
ContactView is currently accepting an AvatarView as a parameter for it's constructor, but after some discussion with Mathieu, we would prefer to have the consumer pass in an image and have ContactView call the constructor for AvatarView.
Pressed state implementation has not been started.
The height of the labels are slightly different between a ContactView that uses 1 label and a ContactView that uses 2 labels. This is due to the intrinsic size of the text because of the font being used (i.e. subhead and footnote vs subhead). When a long email or phone number is used, the singular label will extend to a second line and both lines will have height of two subhead rather than a single subhead and single footnote.
The ContactView for a phone number when using extra extra extra large setting for accessibility text looks awkward as the first line only shows "+1" and then a large white space.

Verification
Testing was done via manual verification of height and other properties by using "Debug View Hierarchy" with the iOS simulator.

Before
After

Demo app with accessibility setting for text set to extra small

Demo app with accessibility setting for text set to large (default)

Demo app with accessibility setting for text set to extra extra extra large

Demo app with accessibility setting for text set to large (default) and dark mode

Pull request checklist
This PR has considered:

[X] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[X] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[X] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

please update your screenshots of your PR as well.


Deprecate toil restart (resolves #807)

I didn't change any documentation in the attic directory, I assumed we are not trying to keep those files updated.

I didn't change any documentation in the attic directory, I assumed we are not trying to keep those files updated.

Your assumption is correct. Attic is like a trash can that gets never emptied. Even though git lets you recover deleted files, obviously, finding the commit where a file is deleted can be hard, AFAIK. That's why I tend to us an "attic" instead.

Do we mention toil restart anywhere in the Sphinx documentation?

We mentioned it in cli.rst but I think thats it. I looked each result returned by searching 'toil restart', 'toilrestart' (case insensitive), and just plain 'restart' so I'm fairly sure I got all references to it.


[Question] upstream的 jit 内存泄漏问题 polardb-for-PG 有处理吗

Describe the problem
看到polardb-for-pg 也会使用LLVM，所以 JIT 也在用对吧？
开启JIT 之后，build_inline的过程会llvm的一些types有内存泄漏问题，upstream 15 版本之前其实都有这个问题。
https://www.postgresql.org/message-id/20201001021609.GC8476%40telsasoft.com
现象就是 开启JIT，在一个backend上跑大量的sql，这个backend 进程物理内存会持续上涨，polar这边
有没有处理这个问题呢？
...

您好，您的邮件我已收到，愿您身体健康，万事如意！

Hi @ZhangHuiGui :
PolarDB for PostgreSQL does use JIT, but does not modify the code of this part. As the issue you have mentioned has not been patched on upstream, the same issue should exist in PolarDB, too.

Hi @ZhangHuiGui :
PolarDB for PostgreSQL does use JIT, but does not modify the code of this part. As the issue you have mentioned has not been patched on upstream, the same issue should exist in PolarDB, too.

Thanks for your reply! Get it.


test: add doc example to insomnia requests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
611

Relevant Lines:
611

💛  - Coveralls


Two runtimeclass with same property name but different type cause cannot convert argument error

Describe the bug
Same with #8994, but WinUI3 C++ project gives a different error.
Steps to reproduce the bug

Create a C++ WinUI3 project

Update every nuget to latest, I have

Create two runtimeclass

    [bindable]
    [default_interface]
    runtimeclass MyClass1
    {
        MyClass1();
        Int32 MyProperty;
    }

    [bindable]
    [default_interface]
    runtimeclass MyClass2
    {
        MyClass2();
        Object MyProperty;
    }

Build project, you get

error C2665: 'winrt::_5_Function_Template_Conflict_WinUI3::implementation::MyClass2::MyProperty': no overloaded function could convert all the argument types
1>(compiling source file '/MyClass2.cpp')
1>C:\Users\Peter\Desktop\WinUI-Bug\5-Function-Template-Conflict\5-Function-Template-Conflict-WinUI3\MyClass2.cpp(14,20):
1>could be 'void winrt::_5_Function_Template_Conflict_WinUI3::implementation::MyClass2::MyProperty(int32_t)'
1>	C:\Users\Peter\Desktop\WinUI-Bug\5-Function-Template-Conflict\5-Function-Template-Conflict-WinUI3\Generated Files\winrt\_5_Function_Template_Conflict_WinUI3.h(94,25):
1>	'void winrt::_5_Function_Template_Conflict_WinUI3::implementation::MyClass2::MyProperty(int32_t)': cannot convert argument 1 from 'const winrt::Windows::Foundation::IInspectable' to 'int32_t'
1>		C:\Users\Peter\Desktop\WinUI-Bug\5-Function-Template-Conflict\5-Function-Template-Conflict-WinUI3\Generated Files\winrt\_5_Function_Template_Conflict_WinUI3.h(94,37):
1>		No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called

Repo here
Expected behavior
Builds fine
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.5.0: 1.5.240227000
Windows version
Windows 10 (1809): Build 17763
Additional context
No response

It looks like your repro specifies Object MyProperty; as the property in the idl file but still specifies int32_t as the type in the .h and .cpp files:
        int32_t MyProperty();
        void MyProperty(int32_t value);

Fixing the code to match the changed property type in the idl should fix the compile issue.

My bad. I have updated the code, and it seems this issue does get fixed for WinUI3. And UWP's issue remains. Please also take a look at #8994 if possible, thanks!


add pipeline function

Add pipeline function.
Pipeline works well when using one detection.

Function pass, merge first.


Add publisher, location, ... to some more types

I compared the current Dar spec with the CSL and suggest to add some more fields:

For confproc we can add additionally (a paper in a conference proceeding is IMO just a special case for a chapter and all fields from there can also be used here):

isbn
volume
series
publisher-name
publisher-loc

For data we can additionally:

publisher-name
publisher-loc
version

For newspaper we can add additionally:

publisher-name
publisher-loc

For report we can add additionally:

page-count
(report number is also often used, but I don't know which field would best fit for this)

There might also be some fields for patent which could be added, but I don't know them well. Moreover, for report, thesis, datasat CSL has a field "genre". But let us maybe concentrate on the above list of easy additions.

No problem for us, we might restrict for eLife specific.

No problem for us either.

Neither for us!!!