What is the technical difference between Azure Kinect and Kinect V2's body tracking?

After using the body tracking of Azure Kinect. We found most of the time, the Kinect V2 has better performance (speed, efficiency, and reliability) than it. So what is the technical difference between Azure Kinect and Kinect V2's body tracking except the Azure Kinect uses a DNN model to derive the skeleton, the Kinect v2 uses a ‘random forest’ model? In which situation we choose to use the Azure Kinect's body tracking instead of that of Kinect V2?

From what I understand, the Azure Kinect's current Body Tracking SDK uses a machine learning model trained on a large data sets to extrapolate poses from the IR feed of the sensor. This approach has been quite successful for Google, Intel, WrenchAI, whereas Microsoft's attempt leaves a lot to be desired.
You can learn about the Kinect v2 Body Tracking random forest model here.
In this moment the Kinect v2 Body Tracking is superior to it's successor in pretty much all respects but I'm rooting for the developer team to get the DNN body tracking model up to speed with the other companies - it has the potential to work even better than the others because it is trained on IR rather than RGB which is superior in dark environments.
Azure Kinect was built for industrial applications, most of which rely on small form factor computers without discreet GPUs running passively all day.. so the developer team have the incentive to make body tracking work in these scenarios, but I think it'll take them a long time to get a fast, accurate and efficient DNN model up and running. Until then we're stuck with extremely latent and slow body tracking that only runs on Nvidia gaming rigs.
Much of the developer community have  put their projects on hold because of this so I've started a feature request to implement a Legacy Body Tracking option. This would give developers the choice to to use random forest Kinect v2-style body tracking over DNN if they want - the feature request has already amassed a lot of support so hopefully we'll have a solution sooner than later.

@Jessica-Yeung the team debated porting the previous Kinect for Windows body tracking algorithm to Azure Kinect vs leveraging 10+ years of research into pose estimation since we developed the original algorithm. The team opted to develop a new model built around a DNN and plan to evolve this model to improves its performance and accuracy.

@fractalfantasy I think the first thing that needs to be requested is the Body Tracking SDK team to be replaced with another one, more skilled, experienced and responsive to customer requests.

The current Body Tracking SDK is almost unusable on most of the devices where Kinect V2 was running smoothly. The result of this, especially in these Covid-19 times, is that we almost cannot use the new Kinect for Azure for all those applications home-based that are currently desperately needed to keep people active and in shape. Let me humbly suggest to the development team to consider opening a development branch of the previous SDK for Kinect v2 to be used with the new Kinect for Azure, of course paying the price of a less accurate tracking. Which for most of the interactive applications was already fairly good (we are ahppily using Kinect v2 + old sdk in a rehab center for children).
Other solutions (i.e. based on Intel Realsense D425 or D415), much simpler and less effective from many points of view, are already in place, and if there will be no changes in the current Kinect for Azure SDK development plans regards the use of GPU-intesive DNN for the BT, I think many will switch to those ones .

@dotslinker I agree with you on all fronts - this topic got closed but body tracking performance is now being tracked in @rfilkov’s thread #514 so feel free to copy paste your response there if you‘d like the sdk team to see it.
Also the idea you mentioned is on the feedback forum so please upvote if you’d like to see it happen:
https://feedback.azure.com/forums/920053-azure-kinect-dk/suggestions/39945454-legacy-body-tracking-like-kinect-v2

@fractalfantasy thanks a lot. I did it right now. It is really unbelievable that I may reach less than 20 fps for the BT SDK on a Dell i9 32 GB Ram just because I have a low rated Nvidia GTX 1650.
And the other thing I really don't understand is that usually the newer versions of the SDKs are set on the top of the previous ones, while the new K4W SDK one seems to be rather built from scratch and definitively less usable for many aspects (i.e. source examples just to name one) if compared to the Kv2...

Is it possible that the same virtual / ai modeling that the Azure uses for presentations could be achieved by the xbox one kinext?


haxe-api theme

I used the latest flambe theme, and made it a bit more in line with haxe.org styles, to be used as new api.haxe.org theme.

Can you double-check that https://github.com/dpeek/dox/pull/99 is in there?

Yess that's in too.


How to Add PEFT to PPO Trainer or PPO Config

I am trying to realize RLHF through PPO.
May I ask how can I realize PEFT in RLHF/PPO. I can see this parameter in DPOTrainer. However, I cannot see that in PPOTrainer.

PPO does not yet support PEFT. But it may be a good enhancement.

Hi @qgallouedec, I'd love to contribute to this enhancement! I noticed that this issue has been open for three months, and I'd like to help bring it forward.
Is there anything specific I should be mindful of as I start on this? Any guidance or notes on the current status would be greatly appreciated.
Thank you!


Built-in themes conflicting with existing ones (One Dark & Material Icon)

When starting VSC with Material Icon Theme as enabled real extension, it conflicts with the one included in yours.
Custom icons doesn't work and a few other configs...
Is it possible to disable built in themes and icons? Or at least locally
Have to remove that every update
"themes": [
	{
		"label": "One Dark Modern",
		"uiTheme": "vs-dark",
		"path": "./theme/OneDark-Modern.json"
	}
],
"iconThemes": [
	{
		"id": "material-icon-theme",
		"label": "Material Icon Theme",
		"path": "./dist/material-icons.json",
		"_watch": true
	}
],```

You can change theme manual.

I think you think the name conflicts, I will change his name in the next version.

I know i can change it, but looks like it still conflicts, you can try that, i will tell you

The update LGTM, no more conflicts, thanks!


feat: add timeout error

handle timeout error in server mode

please implement TimeoutError in every piece of code where a timeout can occur (look for timeout= in methods args)

done


Allow chained search through canonical properties

Example search
Questionnaire?_has:EvidenceVariable:derived-from:identifier=<myIdentifier>
The issue is that EvidenceVariable.derived-from refers to an element of type canonical(Any), which is not currently indexed in our reference tables for use in chained search

Fixed in #5457


Pack TOTP into plugins

Part of https://github.com/wso2/product-is/issues/10998

Remove totp connector from dropins
Remove totp webapps from being added (TOTP page are now in authentication portal)

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/496045542

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/496045542

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/496045542
Status: cancelled

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/496045542
Status: cancelled


[i18n-CN] Translating docs to Simplified Chinese

Hi!
Let's bring the documentation to all the -speaking community 🌐
Who would want to translate? Please follow the 🤗 TRANSLATING guide. Here is a list of the files ready for translation. Let us know in this issue if you'd like to translate any, and we'll add your name to the list.
Some notes:

Please translate using an informal tone (imagine you are talking with a friend about huggingface_hub 🤗).
Please translate in a gender-neutral way.
Add your translations to the folder called <languageCode> inside the source folder.
Register your translation in <languageCode>/_toctree.yml; please follow the order of the English version.
Once you're finished, open a pull request and tag this issue by including #issue-number in the description, where issue-number is the number of this issue. Please ping @Wauplin @stevhliu for review.
🙋 If you'd like others to help you with the translation, you can also post in the 🤗 forums.

README

[ ] README.md

Get Started section

[x] index.md
[x] quick-start.md
[x] installation.md.

How-to guides section

[ ]  community.md
[ ]  download.md
[ ]  hf_file_system.md
[ ]  inference.md
[ ]  integrations.md
[ ]  manage-cache.md
[ ]  manage-spaces.md
[ ]  model-cards.md
[ ]  overview.md
[x]  repository.md
[ ]  search.md
[ ]  upload.md
[ ]  webhooks_server.md

Conceptual guides section

[x]  git_vs_http.md

Yay! :tada: Very excited about this! Thanks @2404589803 for starting with this work. I started to review your PR to make sure it is good from a technical point of view.

I want to complete the translation work for the "search.md" document~ 😊


Removing umlaut limitation in secrets

Intro:
Secrets with umlauts were forbidden due to #131.
Change:
I removed the limitation by converting secrets to base64 and back.
The idea was mentioned by @freddydk in #131.
I am looking for a codereview.

Yeah, I tried something similar, but was unable to find a mechanism where the secret isn't exposed.
The big problem is really that when the secret is used later - it might be exposed in the log - and since we need the secret as non-base64. In your case - it might be that it doesn't show up exactly in this area - but then it might show up in a different area.
And since exposing secrets might reveal classified information, I found no other way than blocking secrets with special characters and I am afraid that this PR doesn't solve the problem fundamentally.
The problem (as I see it) is a bug in GitHub.

I also think it is definitely a bug in GitHub. I tracked down the problem to yaml.

SecretsJson: ${{ env.RepoSecrets }}

The secrets are correctly formatted in env.RepoSecrets. After they are passed to the actions it breaks. I dont think i can fix the Github bug soon.
Another idea would be handling the secrets withoutputs. Also just a workaround. Other actions for example Azure/get-keyvault-secrets have taken this way. I think it is quite unhandy to manage secrets like this and it is lacking extensibility.
Note: I obviously have secrets with umlauts that i can not change anymore and it is not the licensefileUrl.

Which secret are we talking about?

My codeSignCertificatePassword contains umlauts.
Meanwhile i submitted the bug to GitHub. I will post updates here?

That is a real concern - I will identify a solution asap

Update from GitHub support:

Thank you for reaching out to GitHub Support! I apologize for the delayed response here.
I did a bit of testing with this scenario today and noticed that the behavior you are experiencing with umlauts is specific to Powershell. The following Actions workflow outputs characters in powershell, which looks to return various symbols instead of the umlauts - just like you are seeing in your workflow:
on: [push]

defaults:
  run:
    shell: powershell

jobs:
  output:
    runs-on: windows-latest
    steps:
      - run: echo 'aäoöuü'

> Run echo 'aäoöuü'
aÃ¤oÃ¶uÃ¼

From some other posts I found regarding this interaction, it appears that powershell's InputEncoding and OutputEncoding need to be configured to UTF-8, which looks to be the default on the Actions runner:
>$OutputEncoding = [console]::OutputEncoding
>$InputEncoding = [console]::InputEncoding
>
>echo $OutputEncoding
>echo $InputEncoding

BodyName          : utf-8
EncodingName      : Unicode (UTF-8)
HeaderName        : utf-8
WebName           : utf-8
WindowsCodePage   : 1200
IsBrowserDisplay  : True
IsBrowserSave     : True
IsMailNewsDisplay : True
IsMailNewsSave    : True
IsSingleByte      : False
EncoderFallback   : System.Text.EncoderReplacementFallback
DecoderFallback   : System.Text.DecoderReplacementFallback
IsReadOnly        : True
CodePage          : 65001

BodyName          : utf-8
EncodingName      : Unicode (UTF-8)
HeaderName        : utf-8
WebName           : utf-8
WindowsCodePage   : 1200
IsBrowserDisplay  : True
IsBrowserSave     : True
IsMailNewsDisplay : True
IsMailNewsSave    : True
IsSingleByte      : False
EncoderFallback   : System.Text.EncoderReplacementFallback
DecoderFallback   : System.Text.DecoderReplacementFallback
IsReadOnly        : True
CodePage          : 65001

I'm still going to look a bit into what can be done at the workflow-level to potentially set the necessary encoding, but this doesn't look to be exclusive to Actions, just Powershell.

If I try this in various PowerShell "runners" on my laptop:

and in PowerShell ISE:

and in VS Code

an interesting observation in a workflow:
Write-Host with a literal in code does not work:

If the literal is Base64 - then the output works:

So, it doesn't seem to be an output problem - the string seems to have a wrong value in the inline script

Now I think I know what is wrong...

Whenever you use string literals in PowerShell, then the string becomes the binary representation of the UTF8 string.
The string contains 9 bytes (as chars) which are the 9 bytes, which the UTF8 string consists of.
I might be able to create a fix for this and allow secrets with special characters.

I actually think that your original suggestion of carrying the secrets as base64 encoded is good and likely the right approach.
It just couldn't stand alone, we also had to escape some characters.
I was then led off in all kinds of directions due to other challenges with encoding/decoding.

I also went full cycle and I am not yet satisfied.
I will keep in contact with GitHub support. Perhaps a solution will emerge there.
Your investigation was very helpful to mine. I'll probably sink a few more hours into it.
Meanwhile I updated the pullrequest.

You found most places:-)
I have submitted PR to main with support for this - running end 2 end tests now to see if anything breaks.
Will deploy this to preview if/when all tests are passing.

Implemented with #166

Thanks for your investigations, persistence and ideas - as you can see - your original PR was the right idea:-)


Get today's permits instead of yesterday's

I'd like to see how often the permits change in a day.
Gonna update the frequency in the AWS console manually to every 6 hours.

:construction:


View Invariance

Hi Yanjie,
Exciting work, I am curious how you achieved View invariance? Did you collect data from different views during data collection or using data-aug in 3D for variant views? What's the action space frame and robot state frame(for EE), are you using camera frame for EE?

Hi, thank you for your interest. The view invariance is achieved via using our 3D representation. We do not collect data from different views. We use joint positions as action/state.

when you collect data, does the robot move?

We do not use camera pose; but yea, the camera is moving (as shown in our videos) due to the head move.


Forum Category Dropdown values are not full width

Describe the bug
When trying to reclassify a category using the Category dropdown menu, the text content does not take up the full width of the dropdown.   This seems to have changed in the recent migration.
To Reproduce
Steps to reproduce the behavior:

Go to any preexisting topic.
Click on the Category dropdown to change the category,
See issue.

Expected behavior
I expect the text to take up the full width so that you do not have to scroll down as far to find the desired category.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows 2010 Home Edition
Browser: Chrome
Version: 83.0.4103.116 (Official Build) (64-bit)

@ahmadabdolsaheb Hi! Have you had any progress on this bug? I see you self-assigned. I was wondering if you'd need any help? I'm trying to look into it right now.

@RandellDawson Could you describe more in details how to reproduce the screenshot you have?

Hello~!
I can confirm this bug - it appears when trying to modify the categories of a topic after it has been posted.

@ptkpyitheim, please go ahead if you have a solution in mind.

I have updated the issue body to reflect what @nhcarrigan stated (which I should have done originally).

@RandellDawson For some reason I cannot reproduce the bug. Would you be able to send a link? I'm using MacOS and I'm using Chrome

@ptkpyitheim If you go to any topic that you have already created and try to change the category, you should see the issue.  Can you show a screen shot of what you see when you attempt to change an existing topic's category?

Okay I see the issue now. I wish I knew I had to sign up for an account and create a new topic in order to see the bug. The original description assumed that and going to a preexisting topic doesn't necessarily means it's a topic I created.
I'll take a better look.

It seems like there's a max-width that's being set. In my inspector I see that a max-width has been set for each li tags. Unselected that stretches out.

I reported the issue to Discourse and they acknowledged the issue and merged a PR to fix it about an hour after reporting.
@raisedadead What do we need to do to get this update from Discourse?  Will it just get added to their next release?
If it is just a matter of waiting till they ship to production, should I go ahead and close this issue and add the status: resolved/shipping label to it?

Okay I see the issue now. I wish I knew I had to sign up for an account and create a new topic in order to see the bug. The original description assumed that and going to a preexisting topic doesn't necessarily means it's a topic I created.
I'll take a better look.

@ptkpyitheim Sorry for any confusion in the issue description.  I forgot about the fact that moderators can go in and modify anyone's category.  I wrote the description without considering that normal users can only change their own topic.

if we are not planning to update our discourse, I can make the necessary changes from our theme.

This bug was fixed by Discourse and has been applied to our instance.


time unit not match in ThreadMetricUtil.cpp

Signed-off-by: Lloyd-Pottiger<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #5595
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-check-issue-triage-complete

/build

/run-all-tests

/merge


ItemsRepeater with UniformGridLayout takes very much space

Describe the bug
A clear and concise description of what the bug is.I am new to ItemsRepeater with UniformGridLayout so maybe my expectation ist wrong. However when I use an ItemsRepeater with an UniformGridLayout with Orientation set to Horizontal it creates a giant container without need.
To Reproduce
See attached example
EvenlySpaved.zip
Expected behavior
My expectation is something like this:

But I get this:

Environment

OS: Windows
Avalonia-Version: 11.0.7

It's the combination of ItemsStretch="Uniform", a large MinItemHeight, and very small width elements.
Uniform stretch maintains aspect ratio. This means your tiny elements are stretched 50x+ times to fit the window width. This multiplies the item height during layout which is already quite large with MinItemHeight.

ItemStretch="Fill" solves it for me


Unable to get teams from the store after i restart haaukins

This might be already fixed when we deploy the new changes made @mrturkmencom

The problem is not about getting teams instead the problem is that when haaukins restarted, it is not possible to login with existing credentials of users who are registered from event page.

Error fixed in this PR 25dc39be79b57a7b53e4466541df9736e4bf70e5


PMID:19075108

New term :  delayed orthogonal actomyosin contractile ring assembly from randomly oriented actomyosin filaments
A cell phenotype observed in the vegetative growth phase of the life cycle in which a cell exhibits randomly misoriented actin and myosin II filaments, eventually forming a regular orthogonal actomyosin contractile ring. Although the onset of filaments assembly is not delayed, the cell takes more time complete the formation of an orthogonal actomyosin contractile ring.

If "rate of actomyosin contractile ring assembly" accounts for the increased duration of the assembly. sure. I just feel like "delayed" more clearly conveys the temporal aspect of the phenotype than "decreased rate".

Yes OK delayed is better here.
v

@kimrutherford  we need to add Pascale somewhere as PomBase:PC
so I can  add him as definition sources.

we need to add Pascal somewhere as PomBase:PC so I can add him as definition sources.

What is the process for that?

I thibk I found the file
fypo/src/ontology/fypo-idranges.owl
probably if I add a new ID range, I will be able to use PomBAse:PC as support for definitions.
I tried to do this but it would not let me save.
Will do it on next call.


Bugs when the labels is empty during inference.

Hi,
I noticed that if labels is empty during inference, there will have a bug in max(labels) as shown below.
============================
mmdet/models/detectors/base.py
row:204
raw:
color_masks = [
np.random.randint(0, 256, (1, 3), dtype=np.uint8)
for _ in range(max(labels) + 1)
]
fixed:
max_labels = max(labels) if len(labels) > 0 else 0
color_masks = [
np.random.randint(0, 256, (1, 3), dtype=np.uint8)
for _ in range(max_labels + 1)
]

Fixed in #2763


Proposal: Form-level onChange implementation

This code is very simple, and it should be easy for anyone to understand. The unit testing framework in place, however, is daunting. If this proposal is acceptable, I would appreciate help getting coverage for this behavior. It is based off of https://github.com/erikras/redux-form/issues/883#issuecomment-215732308

In practice, it seems like onComponentWillReceiveProps only gets called for certain fields. For example, in my application, I have several Field wrapped components that invoke this.props.input.onChange directly with their value. For simple component wrappers like WrappedInput it works fine, but for more complex ones like WrappedFileUpload or WrappedSelect (which I'm sorry, I can't share the implementation for yet), componentWillReceiveProps does not get invoked. It could be an implementation problem on my end, but I don't think so.

ping @erikras

@gustavohenke if you can hack the unit tests in for this, I would be grateful.

Another thought - it's probably advisable to always cache this.previousState, as adding an onChange prop during some point in the component lifecycle will dispatch an errant onChange invocation from undefined to currentValues

This is not a bad idea, but you've implemented it in the wrong place.
I've implemented it in #2576.

Fix published in v6.6.0.


Add hook for custom configuration of the child process in uv_spawn

libuv has the possibility to do some configuration of the child process created in uv_spawn using the fields in the uv_process_options_t struct. However there is no way to cover all possible configuration that a user of libuv might want to do to the child process.
Therefore I propose to add a function pointer in the uv_process_options_t struct that will be called early in uv__process_child_init, before all other configurations are applied, to allow the user of libuv to do custom configurations of the child process. Also, to make it possible to pass custom configuration options to that function, a void *user_data can be added to the struct as well.
For example:
typedef void (*uv_spawn_cb)(uv_process_t*);

typedef struct uv_process_options_s {
  uv_spawn_cb spawn_cb; /* Called from the child process, before the command is executed. */
  void *user_data; /* User data associated with this process */

  ...
}

This will only work on unix systems that does forking.

That's one reason not to do it.  Another is that only a handful of functions are safe to call after forking.

@netrounds-fredrik what kind of processing would you want to do in this function?

I have a use case where I need to change the network namespace of the child process. It can not be done in the parent process before forking since it would terminate all open network connections.

That's one reason not to do it.

Are you referring to the fact that it will only work on UNIX systems? There are already other options that are only valid for some platforms, for example uid/gid and UV_PROCESS_WINDOWS_HIDE.

Yes, but those are common options that you can't really do without on those platforms.  The rule of thumb is that we only implement what can be implemented on all platforms, everything else is "no, unless."
On to practical matters: extending the uv_process_options_t struct changes the ABI, meaning it can't land in the v1.x release branch.

Still a valid feature request. But if you see that it won't be accepted you can close it.

It's been open for over two years and no one else requested it so I'll go ahead and close it.


Fixing Tensor.backward's function signature

Fixes #692
TLDR: Tensor.backward has a different parameter order compared to PyTorch and also swaps retain_graph and create_graph in its internal function call.
See https://pytorch.org/docs/stable/generated/torch.Tensor.backward.html for backward's function signature:
Tensor.backward(gradient=None, retain_graph=None, create_graph=False, inputs=None)
The current TorchSharp version's function signature is:
Tensor.backward(grad_tensors=null, create_graph=false, retain_graph=false, inputs=null)
Note the difference between the ordering of retain_graph and create_graph. Tensor.backward is just a wrapper to torch.autograd.backward which has a function signature of:
autograd.backward(tensors, grad_tensors=null, retain_graph=null, create_graph=false, inputs=null)
This means calling Tensor.backward(retain_graph: true) in TorchSharp is actually Tensor.backward(create_graph:true) in PyTorch. Same thing for Tensor.backward(create_graph: true) actually being Tensor.backward(retain_graph:true).
The proposed fix is breaking and would change the Tensor.backward function signature to match PyTorch. However, nobody noticed for like 2 years anyway and imo retain_graph should actually mean retain_graph (and same for create_graph) 🙂.

Please add an entry in the RELEASENOTES.md file, and note that this is a breaking change (if you are passing the parameters by position rather than name).

No problem, I added the change to the release notes to NuGet Version 0.103.1, though I am not sure if that's the correct place to put it.
Let me know if there are any problems and I'll fix it,

@dotnet-policy-service agree


Missing pictures in documentation

Pictures are missing on this page in documentation
https://doc.babylonjs.com/tutorials/particles

Thanks for reporting. Seems link to #2125

Fixed

Was fixed : https://github.com/BabylonJS/Documentation/pull/435
Just not deployed.
You may find other images missing due to babylon.js website changes.


product.js error

On a freshly made theme, product.js is throwing an error.
I ran "yarn create slate-theme my-new-theme", modified my .env file, and ran "yarn start". Site loads fine. In the console, there's errors.

The reason error appears is that on a given page there's no DOM element with data-product-form attribute. Mainly because on a dev env you may not necessarily have a product added yet.
If you go go src/scripts/sections/product.js and replace the following:
this.product = await this.getProductJson(
  productFormElement.dataset.productHandle,
);
this.productForm = new ProductForm(productFormElement, this.product, {
  onOptionChange: this.onFormOptionChange.bind(this),
});

with:
if(productFormElement !== null) {
  this.product = await this.getProductJson(
    productFormElement.dataset.productHandle,
  );
  this.productForm = new ProductForm(productFormElement, this.product, {
    onOptionChange: this.onFormOptionChange.bind(this),
  });
}

It will fix the problem.

Thanks. Can I make a pull request to fix this issue?

Sure, go ahead :)


[WIP] Reword and rename “Managing Compute Resources for Containers”

Follows on from closed PR #18425
Reword Managing Compute Resources for Containers:

change the title to “Managing Resources for Containers”, because there are other resource that aren't compute

maybe “Requests and limits” would be a better title. What do people think?

redo the page introduction
move troubleshooting to the end of the page
rewrite the section about local ephemeral storage
tweak the What's next? section

Mark monitoring / management of local ephemeral storage as beta since v1.10 (rather than just the current version)
I really didn't understand the existing text for local ephemeral storage, so I've tried to reword it to clarify how I think it works. I've aimed to state things clearly so that if I have made an incorrect assumption, that shows up and we can fix it.
I also moved the troubleshooting section to the foot of the concept page. Really I think that might work better as a Task, but for now moving it to the end is at least a small improvement.
This might want tech review from someone familiar with the local ephemeral storage feature, as well as with this overall area of Kubernetes.
/sig storage
/sig node

@celestehorgan thanks for the review—really good feedback.

/unassign

/retitle [WIP] Reword “Managing Compute Resources for Containers”

/retitle Reword “Managing Compute Resources for Containers”

It would be great to get one or two tech lgtms from related sigs.
Page to review: https://deploy-preview-18940--kubernetes-io-master-staging.netlify.com/docs/concepts/configuration/manage-compute-resources-container/
/cc @kubernetes/sig-storage-pr-reviews  @kubernetes/sig-node-pr-reviews
Lastly, @sftim do you think it's worth while to update file / links / redirects for manage-compute-resources-container.md to manage-resources-container.md?

do you think it's worth while to update file / links / redirects for manage-compute-resources-container.md to manage-resources-container.md?

Good idea
/retitle [WIP] Reword and rename “Managing Compute Resources for Containers”

/retitle Reword and rename “Managing Compute Resources for Containers”

/retitle Reword and rename “Managing Compute Resources for Containers”

/retitle [WIP] Reword and rename “Managing Compute Resources for Containers”

/retitle Reword and rename “Managing Compute Resources for Containers”

Revised; @dashpole could you take another look?

Also relevant: https://github.com/kubernetes/website/issues/20134

looks good to me after the remaining open comments are addressed.

I've marked a few things resolved. How do these latest changes look?

lgtm

/label tide/merge-method-squash
/lgtm
/approve


Optimize packet struct in PacketParser plugin

investigate size reduction

can you take this https://github.com/microsoft/retina/pull/624 into account? since im making changes to that struct 🙂

Commented on the PR

Resolved in recent conntrack PRs


Add Default VPC Requirement for SageMaker Studio

Is your idea related to a problem? Please describe.
N/A
Describe the solution you'd like
As of dataall's current state, Sagemaker Studio requires the default VPC to exist in the Environment. Can this be added to documentation/can we extend dataall's flexibility to provide custom VPCs for studio
P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

Solved in #409


Profile and optimize new VM.validate_header()

That method takes 1.4s to complete when I call it passing the DAO fork header (1920000). If it takes as long to validate any header, that will be a huge problem for the light client and fast-sync
Need to profile it and optimize. Unless, of course, it's only when validating the DAO fork header that it takes that long

I heard a rumor that geth doesn't check the PoW on every header. Would be worth checking out how they do that safely.

Indeed it doesn't; I describe their approach in https://github.com/ethereum/py-evm/issues/330#issue-292753647

So, as shown by the profiling data above, doing PoW validation makes light sync much slower, but it's mainly because of pyethash.hashimoto_light, which I imagine won't be easy to optimize, so I think our best bet is #932

Should we:

move this issue to the following milestone, or
add kth header validation to Grace Hopper

I think I'm pro 1

I guess we could do both. 2 is already on its way (#1010)

With #1010 merged, this issue seems done enough to close.


Support 1.16.5

Sorry to bother you, but I was wondering if you could add support to 1.16.5? I searched online and found that Player Animator is forge 1.16+, meaning it should be able to support 1.16.5. Is there a specific reason why it doesn't support it?
Again, sorry if I'm bother you.

Because I chose not to

Well, I looked through your plugin and found that the code for the procedures is the same for the different versions of forge, so do you think 1.16.5 would also have the same code? I'm just asking so I could try it out myself.
Thank you for your time. Sorry to bother you again.


fix typo

Fix typo of README.md

Since we failed to fix it, we will close it.

Hi,
I merged your changes. Thanks.
Regards
Bruno

On 30. May 2020, at 12:16, moritalous<EMAIL_ADDRESS>wrote:
Since we failed to fix it, we will close it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/aws-samples/arduino-aws-greengrass-iot/pull/4#issuecomment-636310006, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHMR7VAW6ERA2Z443YUKM2TRUDMGFANCNFSM4NOUHOJA.


Cow: Watch damyno.com

Cow requests the watch of the watch_keyword damyno\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

damyno\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Update documentation regarding PCI compliance

The section "Is App Service compliant with PCI Standard 3.0 and 3.1?" is incorrect and out of date.  Please update to accurately reflect PCI compliance and the ability with App Services to set a minimum TLS version (or perhaps just link to the current compliance documentation in Trust Center).

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

ID: b4968f0a-740d-c541-4b34-457231666ff2
Version Independent ID: c7d64806-1b38-fe4c-d351-ab445cf51905
Content: Configuration FAQs - Azure App Service
Content Source: articles/app-service/faq-configuration-and-management.md
Service: app-service
GitHub Login: @genlin
Microsoft Alias: genli

Hi @jftl6y, thanks for the feedback! We are currently investigating and will update the doc shortly.

@Grace-MacJones-MSFT Unfortunately that link is also completely out of date and inaccurate (was last updated over two years ago).  Per our compliance documentation we are currently compliant under PCI DSS version 3.2.1.  This is why I suggest rather than providing guidance to a specific point in time compliance, simply link to our compliance page:  https://docs.microsoft.com/en-us/microsoft-365/compliance/offering-pci-dss, so that the current information is always available.

Thanks for the clarification John.  I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.


Add Bing Ads + branch.io icon (redo)

This has the same changes as #46, just rebased on top of master.
This will hopefully: (a) make the diff cleaner; (b) resolve conflicts; and (b) shut googlebot up.

@googlebot CLAs are approved for commit authors @jeffjose<EMAIL_ADDRESS>and @philkrie (philkrie@google.com); see discussion in #46.
@jeffjose and @philkire: can you confirm that this PR is what you are expecting?

LGTM. Please wait for @philkrie to confirm his changes before merging. Thanks!

Looks good to me. Should we merge now or wait to add the full update to use the latest Wappalyzer JSON?

This PR is now obsolete. We moved readiness-tool from this repo onto ampproject/amp-readiness.
Please feel free to close this PR out WITHOUT merging.


datetime editor quickfilter

i have a datetime column and want to have a datetimepicker in the quickfilter
I only see a date picker how can i get a datetime picker ?
[QuickFilter,DisplayFormat("dd/MM/yyyy HH:mm:ss"),DateTimeEditor(),Width(150)]
public DateTime Timestamp { get; set; }

See source of addDateFilter, you need to write similar one

@volkanceylan
related to this question (I also have a Date filter), in initializing a default value like 3 months from now for Date Start field. I have seen InitialValuesForQuickFilters grid sample. I want to improve it by using moment.js which is already in Serene instead of Javascript Date().
but how do I include it in Grid.ts so that I can use moment() functions?

See a TypeScript guide please.


Magenta performance-rnn training not using GPU

I am trying to train the performance-rnn from scratch on a new dataset, but it is using the cpu only. Is there a way I can specify which gpu to use from the command line? Here is the script I'm running:
python performance_rnn_train.py \
--config='performance_with_dynamics' \
--run_dir=/tmp/performance_rnn/logdir/run1 \
--sequence_example_file=/tmp/performance_rnn/sequence_examples/training_performances.tfrecord

Update: I didn't touch anything, but it seems to have "decided" to start working... now it's using gpus. However, it would be helpful if I could specify which one(s) to use in the configuration without modifying magenta source or exporting CUDA_VISIBLE_DEVICES. Does anyone know if this is possible?

have you install tensorflow-GPU,you can't install tensorflow-CPU，task will take precedence using tensorflow-CPU，so you should uninstall tensorflow-cpu first if you have both version.

One other thing that may help with this is using the magenta-gpu pip package, which depends on tensorflow-gpu. The magenta package just depends on tensorflow, which is the CPU version.
After installing the GPU-enabled version of tensorflow, the CUDA_VISIBLE_DEVICES variable should work as expected.


KS scraper broken

yarn start --location US/KS fails with:
Begin scraper for iso2:US-KS, iso1:US
  🐢  Cache miss for scraper: /US/KS/index.js; url: https://public.tableau.com/views/COVID-19Data_15851817634470/CountyCounts.pdf?:showVizHome=no; filepath: coronadatascraper-cache/2020-4-30/ac48861604ff931fc999c34e7558251b.pdf
  🚦  Loading data for https://public.tableau.com/views/COVID-19Data_15851817634470/CountyCounts.pdf?:showVizHome=no from server
  ❌ Got error 404 trying to fetch https://public.tableau.com/views/COVID-19Data_15851817634470/CountyCounts.pdf?:showVizHome=no
  ❌ Error processing iso2:US-KS, iso1:US:  TypeError: Cannot read property 'filter' of null

The front page (https://www.coronavirus.kdheks.gov/) has a link to a PDF doc, which we appear to have parsed in the past.  Link example:

			3,738 Confirmed Cases in Kansas (PDF)

The underlying https://www.coronavirus.kdheks.gov/DocumentCenter/ doesn't appear easily scraped itself, so I'll get the file from the above.

Fixed in #917, waiting for merge.


Latest version cannot be installed

Hi,
I am using Windows 10.
I uninstalled angular-cli and wanted to install again the latest. After the installation is complete, I check the version and it is always stuck to 1.6.5. Please check logs below for more info:
Binary found at ><EMAIL_ADDRESS>install ...\AppData\Roaming\npm\node_modules\@angular\cli\node_modules\node-sass
> node scripts/install.js

Cached binary found at ...\AppData\Roaming\npm-cache\node-sass\4.7.2\win32-x64-57_binding.node

><EMAIL_ADDRESS>postinstall ...\AppData\Roaming\npm\node_modules\@angular\cli\node_modules\webpack\node_modules\uglifyjs-webpack-plugin
> node lib/post_install.js

><EMAIL_ADDRESS>postinstall ...\AppData\Roaming\npm\node_modules\@angular\cli\node_modules\node-sass
> node scripts/build.js

Binary found at<EMAIL_ADDRESS>Testing binary
Binary is fine
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\@angular\cli\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

+<EMAIL_ADDRESS>added 938 packages in 112.825s

D:\TO_BE_DELETED>ng -v

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

Angular CLI: 1.6.5
Node: 8.9.4
OS: win32 x64
Angular:
...

D:\TO_BE_DELETED>

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.


chore: fix data races

PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[ ] ~Describes the source of new concepts.~ <- I don't add new concept
[x] References existing implementations as appropriate.
[ ] ~Contains test coverage for new functions.~ <- I don't add new function
[x] Passes all golangci-lint checks.

What
Hello.
Thank you for maintaining this great module!
I would like to:

fix https://github.com/tmc/langchaingo/issues/281#issuecomment-2510861762
add -race flag in go test CI to avoid data race.
fix data race in test case https://github.com/tmc/langchaingo/actions/runs/12115516980/job/33774019172?pr=1080

There is at least one more race, ernieclient.Client.accessToken field is written in autoRefresh() through a race. This is pretty much a syntactically-catchable thing, I wonder why go vet or other tools don't catch this.

@leventov
Hello.
You mentioned about https://github.com/nktks/langchaingo/blob/fix/data-race/llms/ernie/internal/ernieclient/ernieclient.go#L178 ?
I think, the normal use case is to call ernieclient.New() only once for one *ernieclient.Client instance.
In this case, autoRefresh() is called only in ernieclient.New(), and the accessToken is written in only one goroutine in autoRefresh(), so rata races won't occur.

@nktks yes, only one goroutine. But isn't writing a plain field c.accessToken from one goroutine and then reading from others (any goroutines that use the Client for making LLM calls) a data race? Shouldn't such value hand over happen either via atomic.Value, or a channel, but not just plain field write/read?

Here's from https://go.dev/ref/mem:

Reads of memory locations larger than a single machine word are encouraged but not required to meet the same semantics as word-sized memory locations, observing a single allowed write w. For performance reasons, implementations may instead treat larger operations as a set of individual machine-word-sized operations in an unspecified order. This means that races on multiword data structures can lead to inconsistent values not corresponding to a single write. When the values depend on the consistency of internal (pointer, length) or (pointer, type) pairs, as can be the case for interface values, maps, slices, and strings in most Go implementations, such races can in turn lead to arbitrary memory corruption.

@leventov Thank you!
Indeed reading/writting accessToken happens data race.
I added test case to reproduce.
https://github.com/tmc/langchaingo/actions/runs/12176399051/job/33962066313
Then I fixed it by using sync.RWMutex.


Combining dotnet watch and a Vagrant synced_folder does not work

Environment: Running Red Hat Enterprise Linux (RHEL) in a VM on my Windows 10 machine, using Vagrant to manage a VM in Virtual Box.
Using the Vagrant "synced_folder" option, I can share a folder between the host (Windows 10) machine and the guest (RHEL) machine.
If, after starting dotnet watch run on the RHEL machine, you make changes from the Windows machine (using, for example, Visual Studio Code), dotnet watch does not see the changes.
Making changes from within the VM (using, for example, VI) does work.

 THANK YOU!
Seriously; thanks.

Ah shucks, thanks :).
Question: would a command line switch have been more discoverable? e.g. did you ever do dotnet watch --help?

foo --help is pretty much how I learn everything, so yeah ... that would be great.

What if we simply add the environment variable stuff to the --help output?

What if we simply add the environment variable stuff to the --help output?

You read my mind. We started doing this recently (https://github.com/aspnet/DotNetTools/pull/218).


Depreciation of @gcornut/valibot-json-schema

👋 Hello,
I'm here to warn you that @gcornut/valibot-json-schema is now deprecated as Valibot now has an official JSON schema converter 🎉 !
It's not extremely urgent to migrate to @valibot/to-json-schema as my lib will continue to work on Valibot schema as long as no breaking changes are introduced in Valibot.
Some of the features not been ported yet (like bigint and date conversion) because they are controversial (they imply non-standard JSON serialization/deserialization). But please open issues on the Valibot repository if you need them re-implemented.

Thanks for the info !  I will do it when I have time but if someone want to do a PR, it will help me ^^


Jobs never stop running

Informations

Qiskit version: 0.8.2
Python version: 3.6.0
Operating system: Ubuntu 18.04

What is the current behavior?
I'm submitting simple circuits to ibmq_20_tokyo and the jobs get stuck in the running state forever.
Steps to reproduce the problem
Example jobs ids:
5d20bb14e5ac32007041e4be
5d20c0d820a135006fa8478b
Backend status:
BackendStatus(backend_name='ibmq_20_tokyo', backend_version='1.2.2', operational=True, pending_jobs=0, status_msg='active')

Running is the default mode on those devices for a variety of reasons. However it likely means that you are in queue waiting. This behavior will be changed in the near future.

Thanks. It's good to know you'll change that, it's been almost a year that I see pending_jobs=0 in tokyo. In any case, these jobs seem to be running longer than usual. For instance, 5d20c0d820a135006fa8478b was created 2 hours ago already. Could I be in the queue for so long?

Yes you can, depending on what others are running, and how much allocated time you have already used up.

Finished after more than 8 hours. I used tokyo in the past and it never took so long. How can I control how much allocated time I have available?

The allocation amounts are set by your hub admin.

Tokyo is now offline. Is this reopened for a different device?

I reopened it for Tokyo. This problem is happening again since yesterday. I'm not the only user having trouble getting results for jobs submitted more than 20 hours ago. Running status just now prints: BackendStatus(backend_name='ibmq_20_tokyo', backend_version='1.3.1', operational=True, pending_jobs=0, status_msg='active'). Will it be back online again soon? How can we check?

Sorry, but Tokyo is retired, and will not be back online. It still exists as a backend solely for users to retrieve saved jobs. The August 29 sunset date was communicated to those with access over a month ago, and reiterated in the announcement of the Boeblingen device.

You are correct. Thank you for the reminder.


feat(@angular/cli): adding option to define own karma config for app

Fixes: https://github.com/angular/angular-cli/issues/6008

Adding a new flag is a feature, please mark it as such. Thanks :)

Is this still good to be merged? I need this feature.

@laheadle this feature never got in and at this point needs to be reworked... You can use ng test --config=other-karma.conf.js in the command line though.

Build Facade is a more long term solution to this PR.

sorry to have posted something after issue is closed here!
 The application is ready and works fine.

I want to run test cases only for a particular featured module. I have only 1 apps config in angular-cli.json file.
how can i achieve this. Their will be one build ONLY.


tests.plasma5: Fixes OCR in sddm by providing a custom theme

The custom theme just sets the old background colour that OCR works
again.
Motivation for this change
Fixes: https://github.com/NixOS/nixpkgs/issues/42315
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Tested it, makes the test do the thing again in the most logical way. :+1:

@grahamcofborg test plasma5

Success on x86_64-linux (full log)
Attempted: tests.plasma5
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 119.12s
cleaning up
killing machine (pid 593)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/vmynkx6wbnbg0jss9sx3f6fpmwyq5msk-vm-test-run-plasma5

Success on aarch64-linux (full log)
Attempted: tests.plasma5
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 401.64s
cleaning up
killing machine (pid 627)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/awifdndpdxfaylw5kvaya1z282y61p86-vm-test-run-plasma5


Dark theme of site doesn't automatically sync with OS/browser theme

Example URL(s)
https://developers.cloudflare.com/
Actual Behavior
The dark theme of the site does not automatically sync with the OS/browser theme. For example, if you use the dark theme on macOS and use the Chrome browser, the Cloudflare Docs site still appears in the light theme.
Expected Behavior
The dark theme of the site automatically syncs with the OS/browser theme. For example, when using the dark theme on macOS and the Chrome browser, the Cloudflare Docs site appears in the dark theme automatically.
Additional information
No response

Thanks so much for this feedback, @rakleed! We'll raise an internal ticket to track this and close out the Github Issue for now.


Fix a flaky test NullEnabledQueriesTest.testQueriesWithDictFloatColumnNoNullValues.

The test is flaky because we store the column as float and do a SELECT WHERE column = Float::doubleValue() query.
We can reproduce a failed test by changing line 224 to Float baseValue = 0.05102920532226562568f.
Tested:

Changing line 224 to Float baseValue = 0.05102920532226562568f and running NullEnabledQueriesTest.testQueriesWithDictFloatColumnNoNullValues.
Running NullEnabledQueriesTest with invocationCount = 200.

Codecov Report

Merging #10997 (9b90159) into master (82252ed) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #10997     +/-   ##
=========================================
  Coverage    0.11%    0.11%
=========================================
  Files        2191     2137     -54
  Lines      118037   115532   -2505
  Branches    17883    17581    -302
=========================================
  Hits          137      137
+ Misses     117880   115375   -2505
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration1temurin20
?

integration2temurin11
?

unittests1temurin11
?

unittests1temurin17
?

unittests1temurin20
?

unittests2temurin11
?

unittests2temurin17
?

unittests2temurin20
0.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 56 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Issue #224: Added additional symbol route for Visual Studio 2017

Summary of the changes (in less than 80 chars)

Added a route proposed by @ivony tested with a local build and VS 2017 pro.

Addresses https://github.com/loic-sharma/BaGet/issues/224

Thanks for opening this, looks great! 👍


Scope CLI updates Supporting Edge Integration

Tasks that are expected to be required for a summer release. Specifically for Edge integration.

cmd add/delete filter entries
start cmd without filter file
start cmd does not update profile
start cmd updates ld.so.preload
cmd looking for new procs run on interval
eBPF from a separate repo
eBPF events from new processes
inspect cmd expanded to support all procs
filter file entries need to include a GUID, hostname(?)
get filter cmd
JS expression defining which apps to add to the filter
respect the deny list in the filter
add cmd line option to output in json to stdout
accept pipe input for all commands used by Edge
re-order the filter list on command
add a field to AppScope config so Edge can name them/give them ideas

Overall Goal:
Configure a system to automatically scope a set of processes/services
Retrieve the status of; and reconfigure currently scoped processes

In the Edge UI, a user will be able to visualize:
a) The set of processes to be scoped/not scoped
b) Connection status and errors
c) Current process configuration

From the Edge UI, a user will be able to :
a) Define a set of processes / services to be scoped / not scoped
b) Add or remove a process / service to be scoped / not scoped
c) Update configuration of a scoped process.

Internal specification: https://docs.google.com/document/d/1IfP8CtGVEEalk9q6PmfeiovMdVhNfe86XUcUCr0_s4o/edit?usp=share_link

CLI changes proposed:
(all commands)

json output

scope start

remove ability to attach, modify services, install filter file, install scope sh
remove ability to pipe in filter input

scope attach

add a --status argument to wait and retrieve status via ipc
add a --prefix to specify path to host FS to enable container to container attach from a --privileged container
add ability to pipe in scope config file
display a helper menu when run without an argument (priority 2)

scope inspect

add a --all argument to retrieve status of all scoped processes
display a helper menu when run without an argument (priority 2)

scope update

add a --status argument to wait and retrieve status via ipc
add ability to pipe in scope config file

scope filter (migrate some of this from scope start)

add ability to retrieve filter file from /usr/lib/appscope or /tmp/appscope (from the host?)
scope filter --add
scope filter --remove
scope filter --update
scope filter --update --reconfigure
scope filter --add/--remove --deny
scope filter APP_NAME or piped input
respect the deny list

scope ps

add a --prefix to specify path to host FS to enable container to container process listing (priority 2)

scope detach

display a helper menu when run without an argument (priority 2)

Proposed changes to the scope start and scope filter commands
CLI changes
scope start
is just going to install /usr/lib/appscope/ver/libscope.so or /tmp/appscope/ver/libscope.so
with --prefix we will put it in another ns/the host
	requires *cli constructor to change ns and write files
		--install --namespace

scope filter
is going to retrieve filter file from
/usr/lib/appscope/scope_filter
/tmp/appscope/scope_filter
with --prefix we will get it from another ns/the host
	no ns switch required for read

scope filter --summary
shows allows without configs
with --prefix we will get it from another ns/the host
	no ns switch required for read

scope filter --delete
will delete the filter files from
/usr/lib/appscope/scope_filter
/tmp/appscope/scope_filter
unsets ld.so.preload if set
detaches from procs
with --prefix we will delete from another ns/the host
	requires *cli constructor to change ns
		--unconfigure --namespace

scope filter --add nginx (optional < scope.yml) (optional --allow-child) (optional --source_id)
adds nginx to the scope filter file with default scope config or piped config
sets ld.so.preload if not already
attaches to existing nginx procs (from the current container, using --prefix if supplied)
if nginx is already in place, if there are no changes, nothing happens
if nginx is already in place, if there are changes, filter changes and re-attach/ipc update happens
with --prefix we will overwrite it on another ns/the host
	requires *cli constructor to change ns
		--configure --namespace

scope filter --remove nginx (optional --source_id) (optional --nodetach)
remove nginx from the scope filter file
detaches from procs
with --prefix we will overwrite it on another ns/the host
	requires *cli constructor to change ns
		--configure --namespace

CLI constructor changes
--install (optional --namespace)
install the library
--configure (optional --namespace)
truncate and overwrite a filter file
set ld.so.preload to lib path
--unconfigure (optional --namespace)
remove the filter file
unset ld.so.preload
need to check what these commands are currently used for. scope start/stop only?
Other changes
remove the deny list
support allow-child in filter file?
filter commands no longer modify service entries. they only affect proc names. if you want to scope a service via a filter, use its proc name.
Other notes
since a lot of the work must be done in the constructor, create most of the functionality in the loader code.
the cli is however responsible for detecting changes between a current and a new filter file, and determining which actions to take.

user caveat:
because of the requirement to support multiple appscope sources
it must be possible to have multiple entries for one proc in the filter file
the library will only respect the first entry it finds that matches
so, if a user defines the same proc in multiple appscope sources, that is a user error. it's on them and we're ok with that

when a container is defined it can start procs later - set ld.so.preload and put filter file in those containers too?

not doing this (we dont want used to know what a filter file is. if they really want, they can drop one in the right path)
scope filter < example_filter.yml
	replaces scope filter file with the piped filter file
	sets ld.so.preload if not already
	attaches to existing procs
	if there are no changes, nothing happens (detect changes)
	if there are changes to procs, re-attach/ipc update happens

put filter file, a lib, and set ld.so.preload in all containers running
attach to dockerd to enable any new containers that get started to also get the same update
mount all this stuff into the containers using a mount from the host so all files can live on the host
^these two are configurable on the cli

for all filter commands if you’re running in a container, you need to pass --prefix to interact with the host filters
	if rw hostfs, you can modify the file but we are not supporting this scenario
	if privileged and readonly hostfs, you can change ns (in scope constructor)

use the scope custom config instead of a filter file? later

would be easy if no one had to know what a filter file was, including edge. configs and args only.
	pass in configs to cli instead of filters

multiple sources. what happens when you shut down a source, need to remove those filter entries
	does a remove and passes the --source_id 56gf48 (unique source name?)
	each filter proc gets an source_id field also
	when you remove a source, those entries get removed from the filter file

in edge there is a section to modify the filter table
then an apply button afterwards
what happens every time you click the apply button?
the source is stopped and started, so old config and new config are available to edge
the scope filter --add command is run and the scope filter --remove command is run

Proposition for scoping processes in containers:

mount into existing containers = scope new procs in containers
attach to existing procs in containers = scope existing procs in containers
mount into new containers (dockerd interposition) = scope new procs in containers

...where mount is a path to the host file system container libscope, filter, unix socket
We would need to be in the host perspective to execute the mount commands

either use cron to do this
or in our thread in the dockerd process running in the host use our periodic thread to create mounts in existing containers.

...we first need to validate that we can attach to dockerd running in the host, from a container

Original approach to container-to-container
running on the host, install library on the host:
no ns switch required

running on the host, install in a container:
scope start --prefix /proc/9394/root
- read /proc/9394/root/proc/1/status to get host pid 9394
- run scope --install --namespace 9394
- loader switches namespace using /proc/9394/ns/mnt (OK)

running in a container, install on the host (or another container):
scope start --prefix /hostfs
- read /hostfs/proc/1/status to get host pid 1
- run scope --install --namespace 1
- loader switches namespace using /proc/1/ns/mnt (wrong! should be /hostfs/proc/1/ns/mnt)

Proposed approach to container-to-container
running on the host, install library on the host:
no ns switch required

running on the host, install in a container:
scope start --prefix /proc/9394/root
- run scope --install --rootdir /proc/9394/root
- loader switches namespace using /proc/9394/root/1/ns/mnt (OK)

running in a container, install on the host (or another container):
scope start --prefix /hostfs
- run scope --install --rootdir /hostfs
- loader switches namespace using /hostfs/proc/1/ns/mnt (OK)

5/4/23 discussion with edge:

scope start runs when first source is enabled
scope stop runs when last source is disabled

It was discovered that we are unable to patch libscope for musl, without the musl environment present. In order to patch, we need to know the path to the musl shared library on the system. Therefore, we will only patch once we have access to the environment; i.e. in the Filter command instead of the Start command.

5/11/23 discussion with edge:
When a filter is added to a system by Edge, via the AppScope CLI:

AppScope will read the CRIBL_HOME environment variable to determine where to install the library and filter. This directory will be mounted into all containers.
AppScope will also receive a path that determines where the AppScope source unix socket lives, and this directory will also be mounted into all containers (if different from CRIBL_HOME).
It is proposed that a $CRIBL_HOME/state/appscope directory should be created by Edge by default.

6/4/23 Remaining list of work to support Edge integration:

review the attach PR #1490
whenever --rootdir is specified with scope inspect, provide the requested pid in a hostPid field in the inspect output
add the scope filter command

support --add argument
support --remove argument
support --rootdir argument
support --sourceid argument
support a config via stdin, or use default run config
successful output goes to stdout, warnings and errors go to stderr

allow duplicate entries, but only unique entries per sourceid
if process is already scoped, perform a scope update
if process is not scoped, perform a scope

when scope filter is run
retrieve a filter file if it exists
when scope filter --add APP_NAME is run
check the library is installed. if not, install it.
create a filter file if it doesnt exist.
set ld.so.preload if it is not already set.
add an entry to the filter file. if process is already in the filter list, update the entry and perform a scope update for all exi.
perform a scope update on all matching processes that are already scoped
perform a scope attach to all matching processes that are not already scoped
run scope ps. anything that doesnt match the filter, should be detached.
when scope filter --remove APP_NAME is run

remove proc from the filter file
detach from all matching procs
if it's the last entry in the filter file

unset ld.so.preload
remove the filter file

in the library

@seanvaleo

whenever --rootdir is specified with scope inspect, provide the requested pid in a hostPid field in the inspect output

Done via 9ea021b

EXISTING CONTAINERS

need ld_preload set
need a scope_filter in place # must be first
need a unix socket
need to attach to existing procs

determine <unix_socket_dir>
look in the filter for the unix socket path

	scope --mount (/usr/lib/appscope/, /usr/lib/appscope) --rootdir /hostfs     # ensure `scope_filter` is present in container
	scope --mount (<unix_socket_dir>, <unix_socket_dir>) --rootdir /hostfs  # ensure the unix socket is present in the container

	scope extract /opt/appscope --rootdir <container root>                             # ensure the library is present in /opt/appscope/<ver>/libscope.so

	scope --preload /opt/appscope --rootdir <container root>                        # set ld.preload.so to point to /opt/appscope/<ver>/libscope.so
	scope attach <proc> --rootdir <container root>                                        # attach to existing procs

NEW CONTAINERS

need ld_preload set
need a scope_filter in place
need a unix socket

the assumption is that a filter file and a library exists on the host.
so, when runc starts up, we interpose a function and create a config file
filter file has to live in a known dir (usr/lib)
unix socket can and will live anywhere
therefore we need 2 mounts

determine <unix_socket_dir>
while on the host, open and read /usr/lib/appscope/scope_filter and look for the unix socket path

config file defines mounts:

filter file
unix socket
scope executablemount(/usr/lib/appscope/, /usr/lib/appscope/)                                      # ensure `scope_filter` and `scope` is present in container
mount(<unix_socket_dir>, <unix_socket_dir>)                                    # ensure unix socket is present in container

config file defines a hook that (after creation but before run)
/usr/lib/appscope/<ver>/scope extract /opt/appscope/                         # second copy of the library - this is the one to use in the container, relevant to the environment

config file sets LD_PRELOAD=/opt/appscope/libscope.so           # set ld.preload.so to point to /opt/appscope//libscope.so

Documentation for the final set of CLI changes here: https://github.com/criblio/appscope/issues/1536#issue-1766342446


Proper monaco typing for Typescript

Currently the monaco ref returned via monaco.init() is an "any" type.
https://github.com/SurenAt93/monaco-react/blob/182da7e727caa7bfc07bcb7c84115a17e73f897d/src/index.d.ts#L172
Would it be possible to return the proper typing as is described here?
https://github.com/microsoft/monaco-editor/blob/master/monaco.d.ts

Sure. It will be ready in the next version. I’ll keep you up to date here. Thank you!

Hi @stekyne. Please see the updated version of index.d.ts file. I think it will cover your needs. Won't it? (the updated version is 2.1.1).

Yeah looks good!


Adds Gradle Release plugin

Adds release plugin to build.gradle.
Sets path to gradle.properties.
Adds version property used by gradle release plugin.
Syncs VERSION_NAME property used by android-maven plugin and gradle-mvn-push.gradle.
Triggers uploadArchives after successful release build.

https://github.com/researchgate/gradle-release
Closes #157

👍 This really cleans up our release process!


Check mixins support

Mixins with @transition decorator

Hi, Do you mean adding support for implementing transition methods as a Mixin?  For example:
class WorkflowMixin:

    @transition(field=state, source="*", target='draft')
    def draft(self):
        pass

    @transition(field=state, source="draft", target='published')
    def publish(self):
        pass

Then on a Model, you could mix in as follows:
class BlogPost(WorkflowMixin, models.Model):
    title = models.CharField()
    content = models.TextField()
    state = FSMField(default="draft")

In a previous project (using python 2; I haven't checked this lately), I could get around this by doing:
class WorkflowMixin:
    state = None
    # ....

However, this currently does not work (in python 3.4) because the transition decorator sets the field prior to runtime (leading to a "AttributeError: 'NoneType' object has no attribute 'change_state'" exception, here).
I'd love to use fsm with mixins, but right now this doesn't seem possible.


Some schemas are invalid against JSON Schema draft 4

A couple of the observable schemas fail validation against the latest draft4 schema. The linter returns the following errors:

observables/process.json
Field: .definitions['startup-info-dictionary'].patternProperties['^lpReserved|lpReserved2$'].type
Error: should be equal to one of the allowed values
Value: null
Field: .definitions['startup-info-dictionary'].patternProperties['^lpReserved|lpReserved2$'].type
Error: should be array
Value:  null

observables/file.json
Field: .definitions['file-extensions-dictionary'].patternProperties['^pdf-ext$'].properties['document_info_dict'].patternProperties['description']
Error: should be object
Value: "Specifies details of the PDF document information dictionary (DID), which includes properties like the document creation data and producer, as a dictionary."

The second one looks to be a miss-placed bracket, the section document_info_dict should be:
        "document_info_dict": {
          "type:": "object",
          "patternProperties": {
            "^[a-zA-Z0-9_-]{3,256}$": {
              "type": "string",
              "description": "Specifies details of the PDF document information dictionary (DID), which includes properties like the document creation data and producer, as a dictionary."
            },
            "pdfid0": {
              "type": "string",
              "description": "Specifies the first file identifier found for the PDF file."
            },
            "pdfid1": {
              "type": "string",
              "description": "Specifies the second file identifier found for the PDF file."
            }
          }
        }

@clenk is going to take a look at this 😀


Fix Port numbers in docs

Fix VIP without port number; For other VIP example, use different example port than 5050

@sargun can you resolve the conflicts pls?


FIX: move pr template to proper place

In .github/workflows/, it doesn't work.
Also, removed trailing white space and added newline to end.

@skops-dev/maintainers ready for review


When opening preferences via CMD+,, the entire page doesn't load at first.

Just exploring this repo and xi-editor, going to get my feet wet soon. I got it open and figured I'd try out the preferences panel, and this is what happened.

Only the first few lines are displayed, and then the rest is filled in as I scroll.
If, however, I type into the document and then hit CMD+, it opens it in a new frame which works just fine. So this is only when it opts to open it in the existing frame.

Interesting, this was a known issue but I thought it had been resolved. I'll see if I can reproduce.

@zachdaniel I don't imagine there's any chance your xi-mac is a bit behind master? I'm not able to reproduce.

Says I'm up to date on master. Perhaps I have outdated tooling?

Looks like I was still running on nightly rust. Let me see if switching back to stable helps.

I don't think it should be related; what this looks like is that the front-end isn't sending the "scroll_to" RPC, which indicates the size of the viewport, but I'm not sure why that's happening. One possibility is that EditViewController.visibleLines isn't getting reset; we check that before sending the scroll notification to avoid duplicate events. It should be getting reset in XiDocumentController.openDocument. If you feel like firing up Xcode, that's at least somewhere to look.
My guess is that we're racing on resetting visibleLines, but very much a guess.

@cmyr I'll use this issue as an excuse to just dive in and get my hands dirty. All I know about swift I learned when it was announced and I did a tutorial, but I might be able to piece something together :) I mostly came to this project for the rust, but if I pick swift up too, NBD.

From what I can see, it's sending visible lines explicitly as currentDocument.editViewController?.visibleLines = 0..<0

So it looks like we go down a separate path if the current document is empty, where we basically simulate the current document as the new file but freshly opened.
if let currentDocument = self.currentDocument as? Document, currentDocument.isEmpty
Is that an accurate assessment?

I should probably not clutter up these github issues with "me learning swift"!

@zachdaniel ping me on irc if you want some real-time help with this<EMAIL_ADDRESS>

I also see this issue.
It looks like EditViewController.willScroll is not invoked if you open the preferences file from an empty document.
I added the following and it seems to fix it, but I don't know if it's the correct approach.
// In EditViewController
func willScrollToOrigin() {
    willScroll(to: scrollView.contentView.bounds.origin)
}

// In XiDocumentController L63-71
Events.NewView(path: url.path).dispatchWithCallback(currentDocument.dispatcher!) { (response) in
    DispatchQueue.main.sync {
        currentDocument.coreViewIdentifier = response
        currentDocument.editViewController?.visibleLines = 0..<0
        currentDocument.editViewController?.willScrollToOrigin() // Here
        currentDocument.fileURL = url
        self.setIdentifier(response, forDocument: currentDocument)
        completionHandler(currentDocument, false, nil)
    }
}

I hope this helps.

@Garee I have a patch inbound, but this is very close to my approach. Were you also seeing this issue?

@zachdaniel when you're back around, you can see if this patch fixes this for you? https://github.com/google/xi-mac/pull/131

@cmyr I could reproduce the issue and #131 fixes it for me.

@Garee okay cool, I'm going to assume this was some behaviour change between 10.12 (which I'm on) and 10.13 (which I'm assuming you're on); the correct-seeming behaviour on 10.12 was accidental. 😁

Fixes for me as well.


fix: call UIImageView initializer on main thread

ABImageView is instantiated outside the DispatchQueue.main.async callback in a background thread. This results in the following warning: runtime: UI API called from background thread: UIImageView.init(image:) must be used from main thread only.
I've moved the image and imageView instances into the async callback/block. I'm also calling imageView.setupGestures() before completion() instead of after (is that okay or is there a reason for calling it after completion()?)

Any update on merging this PR.


Lustreinfiniband

lustre-ipoib - This is a created implementation of Lustre using ip over infiniband (IPoIB)
lustre-rdma - This is a created implementation of Lustre using native Remote Direct Memory Access (RDMA)

lustre_rdma_nvmedrives:
Changes to files to enable Infiniband functionality:
lfsmaster.sh
lfsoss.sh
lfsclient.sh
lfsrepo.sh
lfspkgs.sh
Addition for the installation of new Mellanox OFED (MOFED) for the Lustre kernel : installMOFED.sh
Addition for correct drives placement of OSSes : installdrives.sh
*installdrives.sh takes about 15 minutes to run so please either remote this entity, or wait it out.
Additions for correct Lustre kernel :
lustreinstall1.sh
lustreinstall2.sh
Addition for pause after MDS/OSS reboot : waitforreboot.sh

I have reduced the size of the headnode to a 'Standard_D8s_v3' since there is no infiniband connectivity with the Lustre servers anyway.
These are not HPC images, hence the need to install OFED. MOFED never worked for IB Lustre when we tried to use the HPC image. This may be something to look at.
The waagent restart comes from another Lustre script.
What do you suggest to actively check if the Lustre kernel has been installed and the node is up? (ssh -q hpcuser@hostname exit ??)


Amber/adding exported models controller to conversations typespec

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

Hi Pranshu, this is the PR to bundle my changes to yours. Could you approve this? Thank you!

Hey Amber - going to close this one - lets make your branch based off mine, then target that branch for merge instead of targetting main


Unable to import class com.datadog.api.client.Configuration after upgrading to com.datadoghq:datadog-api-client:2.28.0

Describe the bug
I was previously using com.datadoghq:datadog-api-client:1.1.0 After upgrading to version 2.28.0 I'm unable to import the class com.datadog.api.client.Configuration. See this error message:

Expected behavior
Maybe this class has been removed, however in the Datadog documentation it gives an example for this version of the datadog-api-client that has the Configuration import: https://docs.datadoghq.com/api/latest/?s=Configuration.getDefaultApiClient()#client-libraries

Environment and Versions (please complete the following information):
I'm working with Kotlin Gradle app.

FWIW: This may just be a documentation error, but I'm not sure how to add the documentation label


Update to the OpenVR 1.0.6 driver interface.

The new API removes a lot of boiler-plate code. The major changes are:

Removal of the IClientTrackedDeviceProvider interface, most of the info it provided is now part of a manifest.
A new IVRProperties interface was added that handles most of the boilerplate code around device properties. We can now simply set the relevant properties upon activation.
The ITrackedDeviceServerDriver no longer needs to implement query functions for the device list, it can now simply add devices through the IVRServerDriverHost interface.

cc @jfrank-razer @JeroMiya

Thanks for the PR!
I think I have most of this covered already in the update-v1.0.6 branch, but I will review your changes more closely soon to see if you caught any that I missed.
I also like your manifest and the mock code in the test_hmd_driver program and will incorporate that in as well.

The master branch now builds against OpenVR 1.0.7.


toggle value as string

This will allow us to have a toggle that can hold more than true / false.
The use case for this is that we want a toggle on multiple query types that the API can now accept.
I'd like to run through any alternative options before merging this, this is just a "How does this feel" PR.

Don't think this is the way...


[tests-only] Refactoring unshare step definition code

Description
This PR refactors the step definition code relating to unshare scenerio.
Related Issue

Part of  https://github.com/owncloud/ocis/issues/6325#issuecomment-1590750636

How Has This Been Tested?

test environment:
locally
ci

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[x] Acceptance tests added
[ ] Documentation ticket raised:

:boom: Acceptance test Core-API-Tests-ocis-storage-2 failed. Further test are cancelled...


'make' Error

I ran the 'make'file, but it came an error like this:
python setup.py build_ext --inplace
Traceback (most recent call last):
File "setup.py", line 11, in
include_dirs = [np.get_include(), '../common'],
AttributeError: module 'numpy' has no attribute 'get_include'
Makefile:3: recipe for target 'all' failed
make: *** [all] Error 1

same problem here.

same problem here.Could you help me?


Crash when starting game

I'm playing in 1.12.2 and have created 6 materials with the mod
link to the crash: https://pastebin.com/5yHWVVHU

I had the same issue and after a bit of troubleshooting I found out downgrading infini-tic to 0.1.35 which is marked as 0.1.36 on the curseforge page for some reason made it work again

Really? I will try it, also I think this has to do with Insane96 mods since I created only 3 materials with Carbonado, Vulcanite, and Galaxite (materials from his mods) and it crashes the game but when I create another material it works (maybe it's another bug, I'm no really sure)

I havent created any materials but I got the same error code as you, it didnt matter what combination of the newer version of infini-tic I was using with what version of tconstruct, eventually downgrading fixed it (for me)

Ok thanks

#12
Reviving this issue, after doing a bit of looking at the code it seems that the issue is caused by an error in a .json with a gem attached.
Would you mind posting your JSON files?

Ok, right now I can't post them but I'll try to get them here as soon as I can

Thanks a lot, after doing some more testing im like 99% sure that it's a bug with the registering of gems for material use.
Which I sadly dont know how to fix because I cant code at all

Well, that confirms it.
Adding anything with a gem breaks Infini-TIC
what a shame :(

New version released for MC1.12.2.  This should work correctly now.  If not, please post a link to the mod you are using and a pastebin of your json config file.


Support role/claim-based link rendering in ASP.NET MVC applications

In many cases we would not like to render links which points to an action that would drop HTTP403 for the user. Sometimes it is enough to override the Html.ActionLink() function, but if we use for example icons in link, than we need to write down the 'a' tag, and generate the href with Url.Action().
With this the best you could do is return an empty string with Url.Action, but it is also not on option, because it will render a link - a broken link.
My goal would be that somehow the Razor parser would check the link's target, and if it is a forbidden action, it hides the link completely.
For example there is a controller:
public class MyController : Controller
{
   [Authorize(Roles = "Admin")]
   public ActionResult Index() {}
}

And a link in a cshtml file:
 link to somewhere

If the user is not in the Admin role, this link should not rendered into the response.
How could I achieve this behaviour? Can I override the parser to handle this? Thanks a lot!

This repository is not related to ASP.Net. And we don't override the parser in any way (we just use it).
Please take a look at: https://github.com/aspnet/Mvc and https://github.com/aspnet/Razor/
Just one hint, which is not what you asked for but may help you:
In RazorEngine you can extend the Razor Syntax by providing your own Template base class.
https://antaris.github.io/RazorEngine/TemplateBasics.html#Extending-the-template-Syntax
Something similar might be possible in the ASP.net world and it might be exactly what you are searching for.


Allow printing before launch() with the XLAStrategy in Fabric

Description & Motivation
This is currently not possible:
fabric = Fabric(accelerator="tpu", devices=8)
fabric.print("error")  # <- problematic
fabric.launch(main)

Pitch
Detect that we are running with XLA and that processes haven't launched (before accessing local_rank), to enable printing in the main process (equivalent to just builtin-print).
My quick dirty sketch:
def print(self, *args: Any, **kwargs: Any) -> None:
    local_rank = 0 if isinstance(self._strategy, XLAStrategy) and not self._strategy._launcher._launched else self.local_rank

    if local_rank == 0:
            print(*args, **kwargs)

(It might be better to move this check to the fabric.local_rank definition directly though)
Alternatives
No response
Additional context
This was raised originally in https://github.com/Lightning-AI/lit-gpt/pull/317
cc @borda @carmocca @justusschock @awaelchli @JackCaoG @steventk-g @Liyang90

Lets do it in the local_rank definition of xla strategy. We already have self._launched defined

See also: https://github.com/Lightning-AI/lit-gpt/pull/317#issuecomment-1657358874


Implement and use UserTestOutputAdapter

Add a basic implementation of UserTestOutputAdapter. This will send on SYSTEM_OUT and STATUS messages in a project test run, and only STATUS messages in a validation run. It also has TODOs for sending on TEST_RESULT messages once we add that message type, and for parsingNEIGHBORHOOD messages in validation runs. This output adapter is now used when the execution type is TEST.
Links

spec
jira

Since we're just getting started with this, can you clarify the definitions of "project test run" and a "validation run"? Just want to make sure I understand the basics :).

Since we're just getting started with this, can you clarify the definitions of "project test run" and a "validation run"? Just want to make sure I understand the basics :).

@bencodeorg the definition I'm using it project run is student provided tests, while validation run is levelbuilder provided tests. Open to suggestions if there's better terminology ideas.

Since we're just getting started with this, can you clarify the definitions of "project test run" and a "validation run"? Just want to make sure I understand the basics :).

@bencodeorg the definition I'm using it project run is student provided tests, while validation run is levelbuilder provided tests. Open to suggestions if there's better terminology ideas.

Ok that's what I thought :). I guess my confusion is the two are one execution (right? ie, one click of the Test button) and I think of "run" as synonymous with "execution"? I like the second wording you used ("student-provided tests", and "levelbuilder-provided tests")


Responsively Style Radio Buttons

Hello Team,
I believe this lesson is only for Nesting radio button within
However i am able to nest the check boxes as well within the
Please check and confirm if this is expected.

Please supply the code you used.

Going above and beyond in a challenge shouldn't fail the tests necessarily. Please use the Help chat room for getting challenge related help. Happy Coding!


Build custom-ops for more TensorFlow versions

Right now we are only building custom-ops for one version of TensorFlow, meaning we're (or should be) forcing users to only use that specific version.
There are two parts to this issue:

Determine to what extent custom-ops compiled for one version of TensorFlow works for other versions (say, compiled for 1.13.1 but used on 1.12 or 1.14).

If not needed per 1. then adapt current loading of custom-ops to not be so picky about TensorFlow version. If needed per 1. then make sure the deploy process builds and packages more versions of the custom-ops.

Related #513: this, this, and this

When we were first doing the custom op I did experimentation related to this. Most custom op builds only worked with the version of tensorflow they compiled against. This had to do with the ABI/API changing within Tensorflow.

We need to decide what versions of TensorFlow we're going to support if we're not going to just support the latest.

Note for future: python_requires='>=2.6, !=3.0.*, !=3.1.*, !=3.2.*, <4'


why remove SPP module?

Thanks for your code. I also found you removed SPP module in your feature extraction module. To be honest, I think it is useless for matching especially those pooling layers with large kernel size. On the other side, do you think low feature will be helpful?


CLI command help is no help

it would be really helpful if there were 1-2 simple command examples on how to download an installer
i'm sure reading all the documentation and help would reveal that info, but i gave up at
mist help download installer

Yeah reading is much harder than taking the time to create an issue in the wrong repo. Try again after a rest. https://github.com/ninxsoft/mist-cli/blob/main/README.md#examples

:-D lol & peace !
it wasn't my day today ;-)

On 12 Oct 2023, at 21:20, Rory Murdock @.***> wrote:
Yeah reading is much harder than taking the time to create an issue in the wrong repo. Try again after a rest. https://github.com/ninxsoft/mist-cli/blob/main/README.md#examples https://github.com/ninxsoft/mist-cli/blob/main/README.md#examples
—
Reply to this email directly, view it on GitHub https://github.com/ninxsoft/Mist/issues/96#issuecomment-1760237181, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGFLPGQAQ4RRWAJH5V5SBGTX7A7JNAVCNFSM6AAAAAA555L7ESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRQGIZTOMJYGE.
You are receiving this because you authored the thread.


Query feature table

Description
PR to add a new Jetpack Compose sample "Query feature table" in Search and Query category. This sample branches off the Analyze Hotspots PR(https://github.com/Esri/arcgis-maps-sdk-kotlin-samples/pull/114) to utilize the message dialog component.
Feel free to only review the query-feature-table module as part of this PR.
Links and Data
Sample Epic: runtime/kotlin/issues/2574

Thanks @hud10837, added in the feedback. I've updated the branch with v.next, ready for your review.


Added "should" delegate methods for hide/show.

These delegate methods apply both to tap gestures and to pan gestures for hiding/showing.

Why leftViewDisabled and rightViewDisabled are not enough?


test change

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[x] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@Alexandr-Nedomets can you elaborate more on change or issue you are fixing here?
Thanks

@chirag-madlani
Hi, it is testing pull request.
I'm sorry, I don't wanted to worry
I describe task in support  chat in slack


raft: Add applied index as an argument to newRaft and RestartNode.

This PR replaces #2121 and is based on comments there.
Closes cockroachdb/etcd#6.
@xiang90 @yichengq

lgtm


Feature Request: Simplify connection to twitch

I'd like to see a simplified connection to twitch, like streamer.bot for example does, just click the button, a browser tab opens, you click authorized and it all works.
(I've had no success, with the described method while streamer.bot worked flawless.....)

That's not possible for the overlay, as I don't run a server/website.
Keeping the client ID save somewhere in the code is not possible, so then it could be extracted and used as phishing attack against users.
What issues did you have with the method? Maybe I can help there :)

The required 2FA for getting the key.

another option is to use irc, which can be used without 2FA..


[release-1.13] Fix and skip istiodless istioctl tests

This is an automated cherry-pick of #37052

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.13
2ee21998b6b6c49b7d23d9769d15c6ad31ab6dad
link
unknown
/test unit-tests_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio_release-1.13
2ee21998b6b6c49b7d23d9769d15c6ad31ab6dad
link
unknown
/test integ-cni_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Where can I get a backup URI to restore devices on oneview ?

I am trying to run the example script to restore devices on oneview:
https://github.com/HewlettPackard/python-hpOneView/blob/release/v4.8.0/examples/restores.py
This script says to use a backup uri. Here is the example one given:
uri_of_backup_to_restore = "/rest/backups/example_backup_2017-04-25_195515"
I am currently on oneview now and I'm trying to find a backup uri to use but I do not know how I can get this uri. I am able to create a backup under settings -> backup, but where can I find a uri to use on oneview ? Thank you for your time.

Use the oneview_client.backups.get_all() and filter out on the name of your backup to find the uri

@saturnsbelt703 , as Dung suggested use get_all() to get the current backup URI.
From the API doc: Only a single backup can exist on the appliance at a time - it is overwritten each time a backup is requested.


TypeScript

It would be great if this package had types for using with TypeScript.

I completely agree! I thought about adding types multiple times but haven't had the time so far. Feel free to add types via DefinitelyTyped. Should I have more time at some point in the future, I'll rewrite it using TypeScript!

@italodeandra TypeScript is now finally supported! I just published a new version to NPM xD


WIP: Update Axom and MFEM

Updates axom and mfem
Clean spack repo after successful TPL build. Fixes #800
I'll be skipping updating docker containers in here since that will be done in this PR: https://github.com/LLNL/serac/pull/1074

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (939c95d) 88.79% compared to head (99cd71d) 88.80%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1079      +/-   ##
===========================================
+ Coverage    88.79%   88.80%   +0.01%
===========================================
  Files          154      154
  Lines        12511    12511
===========================================
+ Hits         11109    11111       +2
+ Misses        1402     1400       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@jamiebramwell In this new MFEM version, the strumpack ctor being used is deprecated. Looks like there is another one that takes in a matrix object that could be used instead.
   // Constructor with STRUMPACK matrix object and command line arguments.
   STRUMPACKSolver(STRUMPACKRowLocMatrix &A, int argc, char *argv[]);

In file included from /usr/WS2/meemee/serac/repo/src/serac/numerics/equation_solver.cpp:7:
/usr/WS2/meemee/serac/repo/src/serac/infrastructure/../../serac/numerics/equation_solver.hpp:208:53: error: 'STRUMPACKSolver' is deprecated [-Werror,-Wdeprecated-declarations]
  StrumpackSolver(int print_level, MPI_Comm comm) : strumpack_solver_(0, nullptr, comm)
                                                    ^
/usr/WS2/smithdev/libs/serac/toss_4_x86_64_ib/2024_02_15_11_25_36/clang-14.0.6/mfem-4.6.2-rc0-nsazjwq3gfybuqduzgsph4ygf5gq5re5/include/mfem/linalg/strumpack.hpp:234:4: note: 'STRUMPACKSolver' has been explicitly marked deprecated here
   MFEM_DEPRECATED STRUMPACKSolver(int argc, char *argv[], MPI_Comm comm)
   ^
/usr/WS2/smithdev/libs/serac/toss_4_x86_64_ib/2024_02_15_11_25_36/clang-14.0.6/mfem-4.6.2-rc0-nsazjwq3gfybuqduzgsph4ygf5gq5re5/include/mfem/mesh/../fem/../general/globals.hpp:120:40: note: expanded from macro 'MFEM_DEPRECATED'
#define MFEM_DEPRECATED __attribute__((deprecated))
                                       ^
In file included from /usr/WS2/meemee/serac/repo/src/serac/numerics/odes.cpp:7:
In file included from /usr/WS2/meemee/serac/repo/src/serac/infrastructure/../../serac/numerics/odes.hpp:20:
/usr/WS2/meemee/serac/repo/src/serac/infrastructure/../../serac/numerics/equation_solver.hpp:208:53: error: 'STRUMPACKSolver' is deprecated [-Werror,-Wdeprecated-declarations]
  StrumpackSolver(int print_level, MPI_Comm comm) : strumpack_solver_(0, nullptr, comm)
                                                    ^
/usr/WS2/smithdev/libs/serac/toss_4_x86_64_ib/2024_02_15_11_25_36/clang-14.0.6/mfem-4.6.2-rc0-nsazjwq3gfybuqduzgsph4ygf5gq5re5/include/mfem/linalg/strumpack.hpp:234:4: note: 'STRUMPACKSolver' has been explicitly marked deprecated here
   MFEM_DEPRECATED STRUMPACKSolver(int argc, char *argv[], MPI_Comm comm)
   ^
/usr/WS2/smithdev/libs/serac/toss_4_x86_64_ib/2024_02_15_11_25_36/clang-14.0.6/mfem-4.6.2-rc0-nsazjwq3gfybuqduzgsph4ygf5gq5re5/include/mfem/mesh/../fem/../general/globals.hpp:120:40: note: expanded from macro 'MFEM_DEPRECATED'
#define MFEM_DEPRECATED __attribute__((deprecated))

Updating spack and fixing this mvapich2 error I've been getting in LiDO just to get it out of the way. Once tpls have been rebuild this pr should be ready.


feat: Add preemptible template example

Description
Fixes b/216660863
Note: If you are not associated with Google, open an issue for discussion before submitting a pull request.
Checklist
Readiness

[x] Yes, merge this PR after it is approved
[ ] No, don't merge this PR after it is approved

Style

[x] My sample follows the rules described for Terraform in the Effective Samples style guide
[x] My sample follows the other requirements and best practices in the Contributing
guide

Testing

[x] I have performed tests described in the Contributing guide:

[x] Tests pass: terraform apply
[x] Lint pass: terraform fmt check

Intended location

[x] Yes, this sample will be (or already is) included on cloud.google.com
Location(s): https://cloud.google.com/compute/docs/instance-groups/create-mig-with-preemptible-vms#gcloud

[] No, this sample won't be included on cloud.google.com
Reason:

API enablement

[x] If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

[x] If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

/gcbrun

/gcbrun


Implement rollbackCreateTable in hive connector

I'm in the process of rewriting the Hive insert code, and the rewrite implements the rollback methods.

Ok, I'll close it. Looking forward to the insert code.