Rename Result.init { … } to Result.catching { ... }

Initializers can't have the @_unsafeInheritExecutor attribute, so we'll need to carve out a func instead to get reasonable async behavior.

@jpsim Haha, this has been a journey but with a deeper dive I don't think it's appropriate to add @_unsafeInheritExecutor either. That seems to hide some warnings it shouldn't.
I think what we've concluded is that Result.init we ship should stay the way it is, and it should be on the caller to isolate the closure when needed. So if you're in an isolated actor context, you can do:
await Result { @Sendable in
  …
}

And that should remove the actor warning but still reveal issues with sendability that occur in the closure itself.
So we recommend you remove the custom initializer in your app, use the one in this library, and annotate @Sendable in contexts that need them. Please let us know if you think we're missing something, though!
This stuff is all very complicated, and it's probably a contributing factor to reasync not being a thing. When thinking about all the consequences of sendable closures, actor contexts, and more, it's a lot to get right!

So we recommend you remove the custom initializer in your app, use the one in this library, and annotate @Sendable in contexts that need them. Please let us know if you think we're missing something, though!

This works for our codebase and the mental model for why this is the correct solution fits in my head, but then again I thought the same thing the two previous times, which makes me really question my own understanding and the complexity of the Sendable model in general 😬.


SiteDefinedSharingCapability is a field returned from get-sposite but it's use is not decribed here

It is set to disabled on all our sites, what is it for?

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

ID: 5d1c6b2b-5af6-acce-b7e8-bf784bb0973e
Version Independent ID: 301586d3-ec04-c42d-4681-0bb59443d35c
Content: Set-SPOSite (sharepoint-online)
Content Source: sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
Service: sharepoint-powershell
GitHub Login: @techwriter40
Microsoft Alias: kirks

@tacklers Thank you for this feedback
I want to make sure your question get answered. Support can handle this to resolution.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Here is a quick video on opening a ticket: https://www.youtube.com/watch?v=puuOy0sEg1Y
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


Outdated link to book in starting guide

This repository's issues are reserved for feature requests, and bug reports, and questions.
I'm submitting a ...

[x] bug report
[ ] feature request
[ ] question

Expected Behavior
Link to book under "Getting Started" should link to a copy of "Effective Computation in Physics"
Actual Behavior
Link to book under "Getting Started" links to outdated host link on ProQuest, and upon further investigation by ISBN, the book is no longer available on the service
Steps to Reproduce the Problem

Open starting guide
Navigate to "Getting Started" > "Download the book"
Follow hyperlink from "book" to http://proquest.safaribooksonline.com/9781491901564
Hyperlink no longer leads to copy of "Effective Computation in Physics" on ProQuest

Specifications

Version:
Platform:
Subsystem:

How can this issue be closed?
Switching the link to a current host, such as, at the time of writing, i-share

Closed request due to improper procedure


[#1] Initialize the project using flutter-templates

Close #1

What happened 👀
Generate a project from flutter-templates, with the following parameters:

[x] PACKAGE_NAME: "co.nimblehq.ic.flutter.avishek"
[x] PROJECT_NAME: "flutter_survey"
[x] APP_NAME: "Survey"

Also, update the following files accordingly (after the generation):

[x] CODEOWNERS
[x] PULL_REQUEST_TEMPLATE.md → Update flutter-templates to ic-flutter-avishek
[x] release_template.md → Update flutter-templates to ic-flutter-avishek
[x] README.md → Update flutter-templates to ic-flutter-avishek
[x]  Set up codecov.

Remove the following files:

[x] story_template.md → Remove this, as it's unnecessary.
[x] codecov.yml

Insight 📝

Generated the project using Flutter templates following the instructions.
Used Flutter version 3.10.4 and Dart version 3.0.3 and updated some code accordingly.
Had to downgrade intl to make it work.
The flutter format command doesn't work anymore. Used dart format instead in the test.yml workflow file.
Updated some deprecated classes in some files.

Proof Of Work 📹
https://github.com/nimblehq/ic-flutter-avishek/assets/8093908/ebd4409b-a9c7-4652-8e40-ba69ce428dc3

@AVI5HEK, Can you run the app on the ios side? If yes, Could you please update POW to include ios app 🙏

Updated the PoW.


JSBIN IS DOWN

It's been a few days.
It's impossible to clone or save a jsbin, that is, to use jsbin at all.
Is it going to be fixed any soon?

jsBin.com is down.

Thanks, I caught this shortly after and it's back


Allow extensions to createRoutesFromChildren

I've been working on something that allows users of react-router to add their own transforms to react-router. ex:
addTransformForType(CollectionRoute, function (child, index) { /* ... */ })
addTransformForType(NamespaceRoute, function (child, index) { /* ... */ })

// now we can do this:
var routes = (
  <Route handler={App} path="/">
    {/*
    This gives the following routes
    They will be wrapped by *WrapperComponent*

    /posts
    /posts/:id
    */}
    <CollectionRoute
        name="posts"
        handlers={{
          index: IndexComponent,
          show: ShowComponent,
          wrapper: WrapperComponent
        }} />

    <NamespaceRoute name="admin" handler={AdminWrapper}>
      {/*
      This gives the following routes
      They will be wrapped by *AdminWrapper*

      /admin/posts
      /admin/posts/:id/edit
      */}
      <CollectionRoute
          name="posts"
          handlers={{
            index: IndexComponent,
            edit: ShowComponent,
          }} />
    </NamespaceRoute>
  </Route>
)

I'm wondering if you would be interested in this being added to react-router proper, or if you think this is best suited for an external package.

@mikew I'm afraid I don't quite understand what you're doing here. Can you please explain in a non-technical way what you're trying to achieve?

Is this sorta like rails resources :photos stuff?

@rpflorence that's the influence yeah, and to let people add their own transforms for whatever pattern their app may follow.
Here's the interface I came up with. The idea is people simply register a function that returns one (or an array of) Route components.
Here is a more complete example of CollectionRoute

I think that's pretty cool.

I'm wondering if you would be interested in this being added to react-router proper, or if you think this is best suited for an external package.

I'd say external package. Every bit of API we internalize has an expense on maintenance and we'd like to keep our focus on the minimum feature-set. Thanks for sharing though :)


Support for Chinese?

Hello,
great job with this gem.
Are you going to support Chinese in near future?

As soon as you contribute @gtalkowski! Thanks.


ION DIDs are not anchoring

Updating ion-tools does not make a difference. Monitoring to see if it resolves itself otherwise will switch to unanchored.

Seems to be working now


Update sql-server-ver15-release-notes.md

Removing AG reference

@amitmsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MikeRayMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

checking in @MikeRayMSFT - you pushed a commit after approving. can you sign off if this is good to review and merge? thanks.

#sign-off


Update license year to 2016

If someone is reviewing the pull request, then could they also check -https://opensource.org/licenses/NCSA
This license likely contains placeholder values from the original license which should be replaced with real values - like Copyright changing from University of Illinois at Urbana-Champaign. to  Apple Inc.

Thank you, but we are not diverging from upstream clang unless we have to.  Please submit this change to the llvm.org project.


Update Dokken config

Description
Try fixing errors with cgroupsv2
Issues Resolved
List any existing issues this PR resolves
Check List

[ ] A summary of changes made is included in the CHANGELOG under ## Unreleased
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

Released as: 5.1.2


Fetching specific JSON causes massive memory leak

Environment
React Native Environment Info:
  System:
    OS: macOS High Sierra 10.13.6
    CPU: x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
    Memory: 6.28 GB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.6.0 - ~/.nvm/versions/node/v10.6.0/bin/node
    Yarn: 1.7.0 - /usr/local/bin/yarn
    npm: 6.1.0 - ~/.nvm/versions/node/v10.6.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
    Android SDK:
      Build Tools: 23.0.1, 25.0.0, 26.0.1, 26.0.2, 26.0.3, 28.0.1
      API Levels: 23, 26, 28
  IDEs:
    Android Studio: 3.1 AI-173.4819257
    Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.4.1 => 16.4.1
    react-native: ^0.56.0 => 0.56.0
  npmGlobalPackages:
    react-native-git-upgrade: 0.2.7

Description
Downloading this <1 KiB JSON file once a second, parsing the JSON, and appending the resultant JSON to an array, causes the demo app below to hit 500MiB of memory usage in less than 15 seconds. Each request permanently allocates ~35MiB of memory.
Reproducible Demo
I have created a repo to reproduce the issue.

Will this be fixed by https://github.com/facebook/react-native/pull/19333?

Unfortunately, no. I just updated the test repo to 0.57 and the issue is still occurring. After downloading the tiny JSON file one a second for sixty seconds, memory usage of the Release build of the test project climbs to over 2 GiB.

We're also experiencing the huge memory allocation when fetching JSON. We're fetching a much bigger resource than the example provided here, and I wonder if the memory allocation is proportional because our app gets killed by iOS for memory consumption (spike from 150mb to over 700mb), even before the fetch call completes.

i get a similar problem. i fetch a 15mo Json file. often the app get killed or get stuck in loading it.
console.log(url); // curl this url take less than 5s to download it
try {
  let response = await fetch(url);

  console.log("response", response); // log ok
  let statusCode = response.status;

  console.log(statusCode);   // log 200
  if (statusCode == 200) {
    let data = await response.json();
    // data = _.cloneDeep(data); // doesn't change anything

    console.log("json", data);  // never reach this log ( fast wifi connection ) !!!!!!!!!
  }
}

Seems memory is fine in Release mode but bad in debug mode.

Yes, in our case the memory consumption graph looks extremely different in debug mode compared to release.

Could it be related to this ?

@gablorquet Looking at the reproduction code, it doesn't look like NetInfo is involved.

Does this repro on 0.59? Also, it isn't clear from the issue if this is iOS or Android. If it is Android I could imagine that this might be fixed by the latest JSC.

@TheSavior Yes, I confirmed on iOS, it has leaks, I checked both iOS and Android implementations, they share the same handle logic, so I assume Android may also has this issue. One leak we find is blob not removed, besides this, seems still has other leaks.

If I recall correctly, this issue is iOS-only. My bad on not making that clear initially. The repro I made above is iOS-only.
I have updated the repro repo to RN 0.59.1 and unfortunately the issue remains.

Got it. Thanks for giving it a try. Do you have any sense on whether this is the same problem as #23801? If so, as there is more conversation happening there than I'd like to close this as a duplicate.

Hmm... I took a quick look at the code to the repro in #23801, and on first glance I think they are different. But it's very possible they are caused by some of the same underlying issues.
Pulling from my comment above:

There are two issues going on here, in my judgement:

The bigger issue, which is that a network request for a tiny JSON file is taking so much RAM.
Network requests aren't being garbage collected if the resultant JSON isn't. If I let the resultant JSON get garbage collected, the memory leak doesn't occur. But the network request still needs to allocate a whole bunch of memory.

The first issue doesn't seem to apply in #23801, as the size of their downloaded file is pretty substantial (5 MiB) in contrast with the tiny file here (<1 KiB).
The second issue doesn't seem to apply either. In the repro for #23801, the author actually lets the JSON get garbage collected, whereas I was only able to demonstrate a memory leak by keeping the JSON and appending it to an array.
It appears that there are two distinct memory leak issues: one that none of the native resources for the network request are cleaned up unless the resultant JSON blob is garbage-collected, and the other that some of the native resources aren't cleaned up even if a "resultant JSON blob" is never allocated.

@Ashoat Can you solve this problem?

Fixed in 9ef5107 for ios and I just opened a pr with a fix for android.

Should this be re-opened? It's still an issue and as far as I understand the PR was pulled

I can't find evidence that the PR was pulled, but if you could  demonstrate that the issue is still extant in 0.60 (which should include the PR) I think it would make sense to reopen. I can't seem to get react-native upgrade 0.60.0-rc.1 to work on the repo above, but maybe once 0.60 is released we can try upgrading it.

The fix on android is working for us with 0.61.5 (and presumably some earlier versions too, though we've not tried them). Only for the default JavaScriptCore/jsc engine though, not hermes or v8 (see the above two issues).


Immediately send getnodestate reply on no pending merges edge

@geirst please review
@toregge FYI
Lets the cluster controller update the derived bucket space states
as quickly as possible when merges are done for the global space,
without having to wait for the normal reply timeout period.

Closing this due to conflicts with #5364, will open new PR with rebased branch once that has been merged.


Update main.yml

ports are not supported anymore
Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] 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 not work as expected)
[ ] This change requires a documentation update

Reviews
Please identify developer to review this change

[ ] @developer

Checklist:

[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing tests pass with my changes

bugfix winlogbeat config

Hi @philipp1184
I think it necessary to update the Jinja template for winlogbeat configuration https://github.com/lean-delivery/ansible-role-winlogbeat/blob/master/templates/winlogbeat.yml.j2#L118-L120

We can do it like this:
{% if winlogbeat_output == "elasticsearch" %}
  hosts: ["{{ elasticsearch.host }}:{{ elasticsearch.port }}"]
{% elif winlogbeat_output == "logstash" %}
  hosts: ["{{ logstash.host }}:{{ logstash.port }}"]
{% endif %}


It would be interesting to see correspond eslint-react rules corresponding to the good/bad examples in this repo

@davidfurlong Sounds like a great idea 😄
Would you like to take a stab at it?

@vasanthk I'm afraid I'm very short on time at the moment. Perhaps when things ease off a little

@davidfurlong Closing this for now. Would be really cool to have this though.
Let us know if you ever get on this :)


Get Cluster/Service CIDRs for Helm deploys dynamically from subctl or Armada, don't hardcode

Part of the review of #317 showed the need to remove a hardcoded link between the Cluster/Service CIDR vars passed to Helm deploys and the underlying deployment implementation (ie Armada internal defaults). That connection wasn't introduced by the proposed change, and has a fairly different scope, so it was agreed it should be handled separately.
Ideas so far:

Use subctl info to scrape network details - via @mangelajo

Well-developed, used by Operator path

Scrape network details from Armada output - via @tpantelis

Doesn't make a dependency on subctl

Per feedback from @mkolesnik, and I agree, it seems the implementation (https://github.com/submariner-io/submariner/pull/406) isn't worth the complexity now that we aren't trying to support more generic deploys.


nit: kubectl → kubectl-convert

This is a Bug Report

Problem:
In here,
the step No. 3 is talking about the location of kubectl binary,
but actually in this section ("Install kubectl convert plugin") we should care about the location of kubectl-convert binary, not of kubectl binary.
Proposed Solution:
In L180:
-1. Append or prepend the kubectl binary folder to your `PATH` environment variable.
+1. Append or prepend the `kubectl-convert` binary folder to your `PATH` environment variable.

And optionally, wrap these words with backquotes in this file:

L41: kubectl
L47: kubectl
L62: kubectl
L159: kubectl-convert
L165: kubectl-convert
(L180: kubectl-convert) - Already suggested above.

Page to Update:
https://kubernetes.io/docs/tasks/tools/install-kubectl-windows/#:~:text=convert.exe.sha256)-,Append%20or%20prepend%20the%20kubectl%20binary%20folder%20to%20your%20PATH%20environment%20variable.,-Verify%20plugin%20is

/triage accepted

/assign


Add url/path guessing helper + test

This only guides which part of the code is run, if there is some kind of failure it's done by whichever branch we enter into

Evaluating...

Other than a landingPage example don't seem to notice any errors i haven't seen behavior. Merged for expediency


Players on the server die when vent is working

When I connect vent device with kerosene to powergrid, all players on the server begin to get damage until they die. And there are no messages on the screen or in server logs.
For example:
[00:02:38] [Server thread/INFO]: Taxman[/<IP_ADDRESS>:50159] logged in with entity id 7918 at<PHONE_NUMBER>792094, 72.5625,<PHONE_NUMBER>623531)
[00:02:38] [Server thread/INFO]: Taxman joined the game
[00:02:38] [Server thread/INFO]: Sending config to Taxman (4283edcf-2931-3969-9a83-70905da7e60b)
[00:03:53] [Server thread/INFO]: Taxman died
[00:04:10] [Server thread/INFO]: Taxman died
[00:05:32] [Server thread/WARN]: Can't keep up! Is the server overloaded?


chat: add embed content

A first stab at embed content. @urcades I just did this off the cuff, let me know what you think.
Only supports youtube, twitter, spotify, and direct audio links for now.
https://user-images.githubusercontent.com/1221094/218745164-4965430f-f237-4636-b219-b841eeb5679f.mov
https://user-images.githubusercontent.com/1221094/218745175-f34b96bd-2501-4b87-8c8e-fe50ec8bf3bc.mov

@urcades were you ever able to give this a look?

After merging master into this branch something is up with the audio player component. It's constantly re-rendering after a scroll. Is this a VS issue? @arthyn

Hey @urcades,
Yes, they need to be rendered outside of the virtual scroller to avoid having the scroller redraw and jump around.
I've updated the spacing, looks like this now:

I've also created a new light box component that we could use anywhere in the app, looks like this:

I can add some functionality to show images in the light box pretty easily too, commit incoming.

Ok, added the light box in for images:
https://user-images.githubusercontent.com/1221094/221283899-3fa261e3-23b8-4ac4-ab84-8d3564354307.mov

How does one view the source material each embed points to? Is this possible via the metadata text underneath the embed content? Through the lightbox view of the embedded content?

I made the title for the embeds a clickable link, but I've also now added the "Source" button from your mocks which is a link to the source material.


add cleanup for create failure

Refs: #119
This will automatically call Remove to cleanup a machine when Create fails if DEBUG is not specified and show an error message.  If DEBUG is specified, it behaves as normal:
Default
ehazlett@ejh-mbp ~/d/g/s/g/d/machine> machine create -d digitalocean test-do
INFO[0000] Creating SSH key...
ERRO[0000] Error creating host: POST https://api.digitalocean.com/v2/account/keys: 401 Unable to authenticate you.
WARN[0000] Removing provisioned machine.  Use --debug to prevent removal.
FATA[0001] You will want to check the provider to make sure the machine and associated resources were properly removed.
ehazlett@ejh-mbp ~/d/g/s/g/d/machine> machine ls
NAME   ACTIVE   DRIVER       STATE     URL

Debug
ehazlett@ejh-mbp ~/d/g/s/g/d/machine> ./machine -D create -d digitalocean test-do
INFO[0000] Creating SSH key...
DEBU[0000] executing: /usr/bin/ssh-keygen ssh-keygen -t rsa -N  -f /Users/ehazlett/.docker/machines/test-do/id_rsa

Generating public/private rsa key pair.
Your identification has been saved in /Users/ehazlett/.docker/machines/test-do/id_rsa.
Your public key has been saved in /Users/ehazlett/.docker/machines/test-do/id_rsa.pub.
The key fingerprint is:
dd:2a:e3:88:a2:e7:1d:9c:07:a4:57:ea:22:d1:92:a1<EMAIL_ADDRESS>The key's randomart image is:
+--[ RSA 2048]----+
|                 |
|                 |
|.   . .          |
|.+ o o   . .     |
|E o +   S . .    |
| o + o     .     |
|. . = . o .      |
| .oo + o o       |
|.+..o . .        |
+-----------------+
ERRO[0000] Error creating host: POST https://api.digitalocean.com/v2/account/keys: 401 Unable to authenticate you.
FATA[0000] POST https://api.digitalocean.com/v2/account/keys: 401 Unable to authenticate you.
ehazlett@ejh-mbp ~/d/g/s/g/d/machine> machine ls
ERRO[0000] error getting state for host test-do: GET https://api.digitalocean.com/v2/droplets/0: 401 Unable to authenticate you.
ERRO[0000] error getting URL for host test-do: IP address is not set
NAME      ACTIVE   DRIVER         STATE     URL
test-do            digitalocean   Error

LGTM.  This is pretty much what I was envisioning and a quick local test indicates it works as I expect.  Just a few logging notes.

One pedantic point; LGTM

@nathanleclaire thx!


IronPython.Wpf does not work in 2.7.10 nuget package

It looks like the DLL that was packaged is empty.

Was resolved with 2.7.11-candidate1.


Prevent Event Re-Publishing

There's a bug that causes events to be republished whenever you save them.

When an event is updated and it's unpublished it seems to always republish it.  I think this is probably not intended behavior and it's not intuitive.  I found that it works like this:
Event.EventUpdate is called when an event is edited.  There is a line in this procedure:
-- --------------------------------------------------
-- If this event is published then update Mozu
-- --------------------------------------------------
if @CurrIsMozuPublished = 1 begin
exec Event.EventUpdateIsMozuPublished @EventId, 1
end
I think this predates the addition of the IsMozuProductActive field and that's why it's checking the published flag instead.  In the current model, once something is published it stays published, but its mozu active state can be true or false.  This code ends up always making the active flag true.
The Event.EventUpdateIsMozuPublished procedure has a few different if/else cases, and it's probably not even necessary for it to be that complex anymore, as long as it respects the IsMozuProductActive setting.

Modified Event.Event to looks at the IsActive flag instead of the IsPublished flag.
Passes testing in dev.


More details and examples for tf.image.generate_bounding_box_proposals

URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/api_docs/python/tf/image/generate_bounding_box_proposals
Description of the issue (what needs changing):
The documentation for this function isn't clear. The meanings of the function's arguments aren't given. There are no usage examples. Seeing the code linked in the documentation doesn't make anything clearer.
Clear description
This function can be used to greatly simplify the making of object detection models, as it implements the proposal generation part of the code.
Correct links
Is the link to the source code correct?
Yes
Parameters defined
Are all parameters defined and formatted correctly?
No, the parameter definitions aren't available
Returns defined
Are return values defined?
Returns are defined
Raises listed and defined
Are the errors defined? For example,
No, the errors raised are not defined
Usage example
No
See the API guide: https://www.tensorflow.org/community/contribute/docs_ref
on how to write testable usage examples.
Request visuals, if applicable
There are no visuals available now. However, it will be helpful to add image examples as this is a function from the image module. The bounding box proposals generated by the function can be verified easily with images.
Submit a pull request?
I would be happy to fix this issue, but I would need more information about the parameters used. I am unsure what they mean.
Are you planning to also submit a pull request to fix the issue See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs,
docs API guide: https://www.tensorflow.org/community/contribute/docs_ref and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style

I can take this issue, I will need to change the file posted here with the docstrings I suppose.
I checked the docs repo but  I believe api docs are generated from docstrings so I will modify the changes accordingly in the source file mentioned here itself.
@ymodak Let me know if I am misinterpreting it. Thanks.

@lamberta  I followed the instructions given here yet I am still facing the issue
Traceback (most recent call last):
  File "tensorflow/tensorflow/tools/docs/tf_doctest.py", line 32, in <module>
    from tensorflow.tools.docs import tf_doctest_lib
ImportError: cannot import name 'tf_doctest_lib' from 'tensorflow.tools.docs' (/usr/local/lib64/python3.8/site-packages/tensorflow/tools/docs/__init__.py)

I have also installed tf_nightly over the already installed tensorflow.

[this@that ~]$ python tf_doctest.py /home/that/Github/tensorflow/tensorflow/python/ops/image_ops_impl.py
2020-10-03 22:35:40.483100: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory
2020-10-03 22:35:40.483230: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
/usr/local/lib/python3.8/site-packages/absl/flags/_validators.py:407: UserWarning: Flag --module has a non-None default value. That does not make sense with mark_flags_as_mutual_exclusive, which checks whether the listed flags have a value other than None.
  warnings.warn(
2020-10-03 22:36:01.245212: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory
2020-10-03 22:36:01.245355: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory
2020-10-03 22:36:01.245374: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:31] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
Running tests under Python 3.8.5: /usr/bin/python
[ RUN      ] unittest.loader._FailedTest.Github
[  FAILED  ] unittest.loader._FailedTest.Github
======================================================================
ERROR: Github (unittest.loader._FailedTest)
----------------------------------------------------------------------
AttributeError: module '__main__' has no attribute 'Github'

----------------------------------------------------------------------
Ran 1 test in 0.000s

FAILED (errors=1)
[this@that ~]$ python tf_doctest.py --file /home/that/Github/tensorflow/tensorflow/python/ops/image_ops_impl.py
2020-10-03 22:38:06.502343: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory
2020-10-03 22:38:06.502381: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
/usr/local/lib/python3.8/site-packages/absl/flags/_validators.py:407: UserWarning: Flag --module has a non-None default value. That does not make sense with mark_flags_as_mutual_exclusive, which checks whether the listed flags have a value other than None.
  warnings.warn(
2020-10-03 22:38:08.391603: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory
2020-10-03 22:38:08.391731: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory
2020-10-03 22:38:08.391751: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:31] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
Running tests under Python 3.8.5: /usr/bin/python
FATAL Flags parsing error: flags module=[], file=/home/aphadke/Github/tensorflow/tensorflow/python/ops/image_ops_impl.py: At most one of (module, file) must have a value other than None.
Pass --helpshort or --helpfull to see help on flags.
[aphadke@DESKTOP-TP18Q4P ~]$

I am getting two different results .

@NitishaS-812k
Is this still an issue

@NitishaS-812k
Is this still an issue

some usage examples would be great too, other functions in tf.image have a lot of useful examples.

Hi @NitishaS-812k @ashwin-phadke !
Could you make the necessary changes here and create a PR from your end .
Thank you!

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Port hdfsmongodb as s-c-t module

As a developer, I'd like to move hdfsmongodb module from XD to s-c-t-m repo, so I can use it as task to build batch pipelines.
Acceptance:

hdfsmongodb as task module (Spring Boot App) is available in spring-cloud-task-modules repo
maven coordinates include hdfsmongodb with respective group/artifact/version

This issue was moved to spring-cloud/spring-cloud-task-app-starters#13


IRGen: Correctly set and honor the "is reflectable" bit on structs.

If we only emit an opaque reflection record for a struct, then we can't reflect its fields. We failed both to clear the "is reflectable" bit in the context descriptor for non-reflectable structs, and to check for the bit before trying to present a struct's fields as children in the runtime. rdar://problem/41274260

@swift-ci Please test

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 1943a78e9ab908b5d24af3ea41f25ab94a0fd7a8

Build failed
Swift Test Linux Platform
Git Sha - 1943a78e9ab908b5d24af3ea41f25ab94a0fd7a8


Pt 158586283 use id in txs

Extend the usage of aec_id:id() to the interface in all transactions.
Connect the base58c encoded information in the API to the creation of  transactions.
Resolve name hashes while processing transactions at the correct tree height instead of in the API layer (resolves https://www.pivotaltracker.com/n/projects/2124891/stories/156600645)

Main pivotal tracker ticket: https://www.pivotaltracker.com/n/projects/2124891/stories/158586283
TODO:

[ ] Make tests for spending to different account types
[ ] Make more tests for spending to names
[ ] Implement querying an oracle by name
[ ] Implement transferring a name to a named account
[ ] Add release notes

Among TODO items I would add:

Check whether spec/api doc changes in protocol repo necessary, and if so prepare new PR superseding https://github.com/aeternity/protocol/pull/91

The only thing that bothers me is around previous work in aec_id. We have specialize/1, specialize/2 and specialize_type/1. Somehow it is hard to follow the pattern. Why not specialize_tag/1 and specialize_val/2 ?
``
-spec specialize(id()) -> {tag(), val()}.
specialize(#id{tag = Tag, val = Val}) ->
{Tag, Val}.
-spec specialize(id(), tag()) -> val().
specialize(#id{tag = Tag, val = Val}, Tag) when ?IS_TAG(Tag), ?IS_VAL(Val) ->
Val.
-spec specialize_type(id()) -> tag().
specialize_type(#id{tag = Tag}) when ?IS_TAG(Tag) ->
Tag.
``


fix syntax (README is rst not markdown)

Failure is unrelated to PR and also in main, so will merge.


GraphQL where filter possibilities

As @etiennedi pointed out correctly, two issues arise regarding the where filter implementation of the GraphQL API.
Or + nested NotEqual operators
First, queries like the following, where an Or operator is applied to two NotEqual filters are applied will always return all data. This is just basic logic. The question is whether we should provide this option to the user, because using this filter does not modify the returned results. On the other hand, applying a filter like this is not incorrect, so maybe returning the response enriched with a warning might be an idea to implement? This counts for Local as well as Network queries.
{
  Local{
    Get(where:{
      operator: Or,
      operands: [{
        path: ["Things", "City", "name"],
        operator: NotEqual
        valueString: "Amsterdam"
      }, {
        path: ["Things", "City", "inCountry", "name"],
        operator: NotEqual,
        valueString: "Netherlands"
      }]
    }) {
      Things{
        City{
          name
        }
      }
    }
  }
}

Network where filter paths
Currently, filters in the network are also placed on the Get level. In the path it should be specified to which Weaviate instance the filter operation should be applied to. However, it makes no sense to compare and combine nested filters of multiple Weaviate instances, because they come from different sources and are queried seperately in the next GraphQL query layer. @etiennedi would moving the where filter to the Weaviate instance/name level solve this problem? Then the filter will look (and work) exactly the same as the Local where filter. I currently see no advantage of having the filter on the Get level to compare property values of multiple Weaviates (except for Get(where) consistency), but I might be missing something.. I think one can just query and use filters per Weaviate instance to get desired results I guess.
This makes no sense:
{
  Network{
    Get(where:{
      operator: Or,
      operands: [{
        path: ["WeaviateB", "Things", "Airport", "place"],
        operator: Equal,
        valueString: "Amsterdam"
      }, {
        path: ["WeaviateC", "Things", "Human", "birthday"],
        operator: Equal,
        valueString: "01-01-01"
      }]
    }) {
      WeaviateB{
        Things{
          Airport{
            place
          }
        }
      }
      WeaviateC{
        Things{
          Human{
            birthday
          }
        }
      }
    }
  }
}

So proposed is:
{
  Network{
    Get{
      WeaviateB(where: {
      	path: ["Things", "Airport", "place"],
        operator: Equal,
        valueString: "Amsterdam"
      }) {
        Things{
          Airport{
            place
          }
        }
      }
      WeaviateC(where:{
        path: ["Things", "Human", "birthday"],
        operator: Equal,
        valueString: "01-01-01"
    	}) {
        Things{
          Human{
            birthday
          }
        }
      }
    }
  }
}

@laura-ham

would moving the where filter to the Weaviate instance/name level solve this problem? Then the filter will look (and work) exactly the same as the Local where filter. I currently see no advantage of having the filter on the Get level to compare property values of multiple Weaviates (except for Get(where) consistency), but I might be missing something

Yes, this would be awesome. This would make quite some code that I wrote obsolete, which is always good. Less code => less complexity to maintain. The Network filtering would then really just be a very simple proxying, whereas right now because of having to remove the first element adds the overhead to parse an unknown structure, modify the path array, etc.
So I'm all for this change, both logically and implementation-wise.
There is a minor inconsistence in you example query though - I assume this is just a typo:
Network.Get.WeviateB.where.path does not start with the instance as the first element
Network.Get.WeviateC.where.path does start with "WeaviateC" which is redundant <-- I assume this wasn't intended.
Nevertheless, I'd still propose to first merge what we have now for Network.Get with the old filter structure and then tackle this as a separate concern slightly later to

make sure the branch doesn't deviate fromdevelop to much.
to allow me to make some refactoring on the graphql code first (to make the Network style more consistent with Local)
to then (after refactoring!) split up the work between @JeroenStravers1 and me, so he can tackle the API change and I can tackle the resolver change. (If you're busy with something else or not available, I can also tackle both of course - just looking for most efficient workmode)

tl;dr: ✅ like the proposal, makes sense from a UX perspective, makes the code less complex than before, but want to tackle it only after merging the current implementation for Network.Get

@bobvanluijt @laura-ham @JeroenStravers1
I've thought about this a bit more and actually I don't see any benefits of keeping the where filter at such a high level in any case (also not for Local.Get). What keeps us from moving the where filters all the way down to the individual class, like so?
{
  Local {
    Get {
      Things {
        City (where: {
          operator: And,
          operands: [{
            path: ["population"],
            operator: GreaterThan
            valueInt: 1000000
          }, {
            path: ["inCountry", "Country", "name"],
            operator: Equal,
            valueString: "Netherlands"
          }]
        }){
          name
          InCountry {
            ... on Country {
              name
            }
          }
        }
      }
    }
  }
}

What would we lose from this?

The ability to combine filters across different classes, however, I don't think they ever make sense. All Cities with name not equal to Amsterdam OR all the cars with brand not equal to Mercedes doesn't make sense from a logical perspective. They're just two seperate queries with two separate filters.

What would we gain from this?

The actual filtering is much closer to the Class I'm querying
We don't have to specify redundant info anymore in the path, because the entire filter is already scoped to ["Thing", "City"], so we only need to keep the important stuff, such as ["population"] or  ["inCountry", "Country", "name"]
It would be consistent with the API design for grouping, where we also apply the groupBy on a Things.City level

Am I missing anything?

@etiennedi I've thought about this for a bit more and I think your suggestion seems, for the reasons you mention, the most logical thing to do. I remember when I made the where filter, first I designed this in the way you propose (so the filter on the Class level). But I changed it to allow combinations of filters. Currently I however cannot come up with an example where you should need a combination in the filter of multiple classes, simply because it doensn't make sense from a logical perspective, as you point out.
Changing this would also solve #629.
@bobvanluijt @JeroenStravers1 What do you think? Are we missing something important here or would you approve this change?

I agree with both of you. As a user it would make more sense to me to apply filtering as close to the actual classes as possible, and I'm a big proponent of consistency in where elements are located. I can't think of any use case that hasn't already been discussed above.

First of all, before on of you will do it, the appropriate link regarding my €0.02 on this topic.
I'm perfectly fine with this, however I would like to understand the following implications first;

@laura-ham Can you set this up in the docs and the prototype to keep the tests passing?
@etiennedi do you have any insights on how much work this would be for you to make the actual changes?

@bobvanluijt

@etiennedi do you have any insights on how much work this would be for you to make the actual changes?

If I am not mistaken there are four things to change:

the graphql structure on Local
the graphql structure on Network
the graphql resolvers on Local
the graphql resolvers on Network

My assumption is 1 + 2 is just moving stuff around and making tests pass again, but @JeroenStravers1 would be the export on this, I'd say a few hours max.
For 3+4 we're already resolving at a later place anyway, so right now what we do is consume the filter where it appears and pass it through. All we have to do is move the passing through. I'd say a few hours max.
Overall should easily be achievable within a day unless I'm missing something (@JeroenStravers1).

@bobvanluijt
Same for me, its mainly moving some stuff, renaming a couple of GraphQL names and fields etc. Then the resolvers will take some more time indeed, but since the idea remains the same it won't take too long.

@etiennedi is right, it's a matter of moving code around for me too

All parts are completed now including resolving for Local.Get and Network.Get. I'll submit a PR soon to close this issue.


As a user I want to view the project in different viewports

As a user I want to resize the preview pane horizontally and vertically to check my designs in all possible sizes.

Related to #69, where the side panes are hidden. For the preview content, we should have a look at Chrome's Devtools, as they already have an approach for both, viewports/devices, and side-pane hiding/resizing.


Deploying Flask App with nba_py

Has anyone sucecssfully deplpyed a flask app that uses nba_py? I have used Heroku and pythonanywhere and both times it doesn't work when I try to request game information. It just times out.
On Heroku when I open the command line and type scoreboard = nba_py.Scoreboard(month=2, day=21, year=2015) the command never executes just hangs there.

NBA.com seems to block known IP address blocks from cloud hosting providers, including AWS and Heroku
https://github.com/seemethere/nba_py/issues/88

@bttmly Do you know of anyway to get around this?

Many endpoints allow a JSONP API, which can be accessed directly by a web client. When you do that, the requesting IP address is that of the client's own computer, so you circumvent this restriction. This would mean your app would have to run in the browser though; not sure what your use case is.
Otherwise you can host your application on a provider that NBA doesn't block. I just ran a request to NBA through a Node.js client from a server hosted by Linode and it worked (disclaimer: I couldn't get a similar curl command to work, no idea why).
Shameless plug: I wrote a Node.js/browser NBA client in JavaScript which supports JSONP: https://github.com/bttmly/nba


[WIP] React api

You'll see... wink

Changes Unknown when pulling dfecbe2fefaf577424831c84f63b783301f4c22c on mokhan:react-api into ** on stronglifters:master**.

Changes Unknown when pulling 4c233d14d9037b028b9735f520b542ad6dfc2b00 on mokhan:react-api into ** on stronglifters:master**.


2022.4 | adding podlables

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.rajcool seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Error in README.md

«the config: ln -s $HOME/.i3/config $HOME/.i3/»
This command does not seem possible. You cannot create a link named .i3 when there is a folder with the same name.

That's not what it means. If you put a folder as a destination that already exists, it will create the symlink with the same filename as the target (config). So, in short, it will create a config symlink inside the i3 folder.

This was fixed by https://github.com/nightsh/i3-plasma/commit/30c5ce4abbd9067d65bb21e93b03d1facb87f1a2
Source and target pointed to the same file


Update Ktor

https://youtrack.jetbrains.com/issue/KTOR-2678#focus=Comments-27-5004753.0-0  Would need to undo changes to Setup and buildscripts.


Update introduction.md

🚀 Deployed on https://63645b3ac42e4e168d032a8a--cozy-fox-0defec.netlify.app


Update package dependencies and .NET 5

Updated the NuGet packages to latest version and changed .NET framework to version 5.

Seems good in my opinion (just doing a cursory look), I'll probably try to build it and release a new version later today (if it's not up in a few days just bug me lol).
I plan(ned) on adding Vault Card support but I never really got the motivation to work on this beside DLC/customization updates because I'm pretty well separated from BL3 at this point honestly

Hmm wait, did a bit more reading, it seems that this PR is missing the generated C# when the protobufs were updated.
I think what I used was protogen to generate the C#.
I basically just slapped them into that text box in order of importance OakShared --> OakSave --> OakProfile probably and then split them up into individual C# files.
There's no point in updating the protobuf files if the compiled C# isn't updated otherwise.

Hmm wait, did a bit more reading, it seems that this PR is missing the generated C# when the protobufs were updated.
I think what I used was protogen to generate the C#.
I basically just slapped them into that text box in order of importance OakShared --> OakSave --> OakProfile probably and then split them up into individual C# files.
There's no point in updating the protobuf files if the compiled C# isn't updated otherwise.

Thanks for the heads up, i will close for now


sudo: 1.8.14p3 -> 1.8.15

Updating because I hit a bug in 1.8.14p3.
Release notes: https://www.sudo.ws/stable.html#1.8.15
cc @edolstra

trigger rebuild on travis ... lets see

Travis seems to have killed the build. I've rebased against current master to force a new build.

@fpletz tnx!

Thanks!


#235

Argumant of Type (params: BootFuncParams) #235
Froze the version of the package.

Thank you! Also 2.0.5 was released which fixed this as well 🤣


Link overview pages and suggest order

Addresses #4

Trying out the round-robin thing here... who's getting lucky? Because this is a very lightweight PR!


Format HTML used in examples

Introduces a dependency on pretty
Before:

  <fieldset class="govuk-c-fieldset">
    <legend class="govuk-c-fieldset__legend">

What is your nationality?

      If you have dual nationality, select all options that are relevant to you.
    </legend>

        <input class="govuk-c-checkboxes__input" id="nationality-1" name="nationality" type="checkbox" value="british">

        British

        <input class="govuk-c-checkboxes__input" id="nationality-2" name="nationality" type="checkbox" value="irish">

        Irish

        <input class="govuk-c-checkboxes__input" id="nationality-3" name="nationality" type="checkbox" value="other">

        Citizen of another country

</fieldset>

After:

  <fieldset class="govuk-c-fieldset">
    <legend class="govuk-c-fieldset__legend">

What is your nationality?

      If you have dual nationality, select all options that are relevant to you.
    </legend>

        <input class="govuk-c-checkboxes__input" id="nationality-1" name="nationality" type="checkbox" value="british">

          British

        <input class="govuk-c-checkboxes__input" id="nationality-2" name="nationality" type="checkbox" value="irish">

          Irish

        <input class="govuk-c-checkboxes__input" id="nationality-3" name="nationality" type="checkbox" value="other">

          Citizen of another country

  </fieldset>

https://trello.com/c/7wZdB2pn/766-prettify-html-in-code-examples

This seems to have done some funky things to some of the code snippets…
For example, a lot of the code blocks on the typography page now look like this:

  govuk-body

It's also added extra whitespace to some examples, like [the first example for the select component]:

    Label text goes here

  <select class="govuk-c-select" id="select-1" name="select-1">
    <option value="1">GOV.UK frontend option 1</option>
    <option value="2" selected>GOV.UK frontend option 2</option>
    <option value="3">GOV.UK frontend option 3</option>
  </select>

😭

This wont be trivial so closing this and moving back into the backlog when I have more time.


Media Source Extensions

Does nodee-webkit support media source extensions?

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Hello @viig99,
well it depends on what you actually need: the latest version is based on Node v0.11.13 and Chromium 38.0.2125.104, and there's no much difference between the original Chromium and the merged version you have in node-webkit.
So coming back to our ships, does node-webkit support MediaSource? You can check here on MDN Documentation or you can just create a PoC and then run it on node-webkit :).
However be aware that, as documentation suggests, some APIs are not yet supported on Chromium\Chrome.


feat: add BitcoinAmountOrAll to fedimint-cli

builds on #3573 BitcoinAmountOrAll, moves struct into fedimint_core and adds 'all' option to fedimint-cli clap args

@elsirion see https://github.com/fedimint/fedimint/pull/3574/commits/3a2a50990f27aeeb8736e0af272853b014b21b84 for fix for 'all' serialization from other PR's comments.

Clippy is currently unhappy https://github.com/fedimint/fedimint/actions/runs/6858954816/job/18650509987?pr=3574#step:6:74

Clippy is currently unhappy https://github.com/fedimint/fedimint/actions/runs/6858954816/job/18650509987?pr=3574#step:6:74

Clippy is generally grumpy and uncooperative, so don't take it personally. Just hard character to work with for everyone.

You should be able to run clippy locally: just clippy. Much faster than waiting for CI every time.

And just clippy-fix :)


Add distsql to SET/SHOW vars

Add distsql to SET vars and SHOW vars. Will need to add the 2.0 setting later (#3720).
Note: Conceptual docs about the DistSQL engine / distributed query heuristic will be written later.
Closes #3717 and #2364.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e4352714233d18fa25ae499b96d13e619f8d3a5c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/7bcbcc8ce24aee7a04e797b9060e161513e48694/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d7855ef8f303e495c02d92ba1314a2131e9878cd/


[Bug] Wrong locale after installation, not possible to change

Hi!
Describe the bug
I installed the tool via Run with Power Shell and to my surprise Notepad launched with DE locale. My OS is PL locale. I have DE keyboard installed, but it was not active at the time of installation. There is no way to change it in the options.
Desktop (please complete the following information):
Windows 1909
Thanks! Good job!

Actually I am not sure how to change it, what is the language in your windows settings?

Dear Jason, my language is Polish, but I have two keyboards installed: PL and DE, out of which only PL was active at the time.

Not sure why Windows 10 is not using PL instead... This is enabled and set as default by windows. I don't find any control on my end with my app, sadly...


Expose more data via Prometheus metrics

Environment
Nautobot version: 1.5.13 (to get metrics.py support)
Proposed Functionality
Expose Prometheus metrics for this app including, but not limited to:

[ ]  Average Duration of Maintenances
[ ]  Average Duration of Maintenances per Provider
[ ] QTY Upcoming Maintenances
[ ]  QTY Maintenances upcoming this week
[ ] Average Number of Maintenances Per Month
[ ] Average Number of Maintenances Per Month Per Region
[ ] Average Number of Maintenances Per Month Per Site

More ideas...

Similar info presented today via dashboard view could go directly as metrics (logic is already there)


log_dir and comment not working together in SummaryWriter

As it is in master, you cannot use a specific log_dir with the variable comment.
The current code directly creates the file_writer with the specified log_dir ignoring comment.
if not log_dir:
    import socket
    from datetime import datetime
    current_time = datetime.now().strftime('%b%d_%H-%M-%S')
    log_dir = os.path.join('runs', current_time + '_' + socket.gethostname() + comment)
self.file_writer = FileWriter(logdir=log_dir)

Seems straight forward to add comment even when log_dir is specified:
if not log_dir:
    import socket
    from datetime import datetime
    current_time = datetime.now().strftime('%b%d_%H-%M-%S')
    log_dir = os.path.join('runs', current_time + '_' + socket.gethostname())
self.file_writer = FileWriter(logdir=log_dir + comment)

Similarly, if we want to log the date also:
if not log_dir:
    log_dir = 'runs'
import socket
from datetime import datetime
current_time = datetime.now().strftime('%b%d_%H-%M-%S')
log_dir = os.path.join(log_dir, comment + current_time)
self.file_writer = FileWriter(logdir=log_dir)

Is this behaviour between log_dir and comment intended? Is it worth a PR?

I use time + hostname + comment for simple logging. If users want a customized logging path (e.g. parameter, dataset, special time format ...), they should just compose their log_dir and ignore the comment argument. PRs are welcome as long as the behavior is backward compatible. (For example, your last snippet forcefully append time to the user's dir.)


Cauldron is too fast

Could you possibly make it so brewing in a cauldron isn't instant? So for example have the brewing take some time to happen with a progress bar (something like how ex nihlio's barrel when making witch water is for example)

Nope, that would involve making it a ticking TE and change it from the vanilla behavior. By default the cauldron only brews 2 potions per ingredient instead of 3 so the brewing stand still had a use.


Implement an easy way to check for images existence

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

Thanks @ferpetrelli! Definitely helpful.


fix: typo in warmwassertemperatur sensor

Does changing the sensor name affect the long term statistics?

This is a Dashboard config file. So no changes in the statistics.

Thanks for your contribution.


Room link is invalid

I/flutter (11913): {token: null, msg: Room link is invalid, status: 404, api_version: 2.0.148}

@andim27 which link are you trying?
Try this link: https://yogi.app.100ms.live/meeting/nih-bkn-vek

@andim27 were you able to resolve the issue?

Now it works/The broken link was: Check your codebase please))

yea, that room has been permanently disabled now. On develop branch it's updated.


Failed to create bn kernel on Resnet

Hi, I have test pipecnn on the Terasic OpenVINO Starter Kit (OSK) which can run the Alexnet smoothly. While when I change the pipecnn for RESNET in hw_param.cl file, it will report the "failed to create bn kernel" message. I only change the network definition to RESNET in hw_param.cl and recompile the project. Any extra setting required for RESNET?

Please check the resource utilization in the report. There might not be enough hardware resource for the cyclone device

Thanks. I change the card to another PC and tune some hw params. Now it can run.

@WenyeLiu Facing same issue. Could you please share, what changes you made?

You can change the VEC_SIZE and LANE_NUM to a smaller value


Edit README.md

Is your feature request related to a problem? Please describe.
Edit README.md to reflect the 2FA design doc
To-add

[ ] Project Overview (Shortened)
[ ] Links

[ ]  design document
[ ] outside resources listed in Bibliography

To-remove

[ ] Layout/readme
[ ] Issues
[ ] git specific stuff/linux and everything underneath
[ ] conclusion

I updated the readme to include information from the wiki and was wondering if I should make a pull request for others to review it before anything.

Awesome! Yes, you should make a pull request, be sure to write the phrase closes #1 in the pull request message, so it's linked with this issue


Load modules to support NAT traversal in VR

Description
Load necessary modules so that VPN connection works properly

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[X] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[X] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[X] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@rhtyd can we incorporate this in the new systemvm template?

The scripts run on systemvm start by calling modprobe, no need to change/include in the template.

@ravening if this is useful/needed in 4.15.1, can you change the base branch and milestone?

@rhtyd shouldn't we only include the changes in https://github.com/apache/cloudstack/pull/4777/files#diff-c1dd33dcef66362966d2f8b3598d6b2412c49a048b89c75c02486c5c3d58e351 and not the explicit modprobe? It seems redundant to have both.

@rhtyd shouldn't we only include the changes in https://github.com/apache/cloudstack/pull/4777/files#diff-c1dd33dcef66362966d2f8b3598d6b2412c49a048b89c75c02486c5c3d58e351 and not the explicit modprobe? It seems redundant to have both.

modprobe works with existing template and previous releases.
it is not required if we build new systemvm template
anyway, they are not harmful.

@Pearl1594 can you include this in the 4.15.1 and cks/master PRs for new systemvmtemplates?

@ravening if this is useful/needed in 4.15.1, can you change the base branch and milestone?

@rhtyd rebased it to 4.15

@blueorangutan package

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 206

@blueorangutan test

@Pearl1594 a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-223)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35441 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4777-t223-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 85 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_migrate_VM_and_root_volume
Error
81.56
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
50.14
test_vm_life_cycle.py


Docker Hub builds are failing for most flavours

https://hub.docker.com/r/williamyeh/ansible/builds/

This should have been fixed since https://github.com/William-Yeh/docker-ansible/commit/17b0a870d18c2e66bfcbc93b1b5895e094d611b0


Feature: Add good first issues to the dropdown for the repo

Type of feature
🍕 Feature
Current behavior
TBD
Suggested solution
TBD
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

What does this mean? We do have good first issues now, right?


TB-12: implement scheduling for sending notification about new articles

Implement scheduling for sending notification about new articles.
Acceptance Criteria:

find new articles workflow implemented as expected

corresponds to #5 task


Validate topology.yaml

Things we want to check:

every transform chain ends in a terminating transform - complete
a terminating transform is not followed by a non-terminating transform - complete
transforms are compatible with each other e.g. RedisTimestampTagger -> CassandraDestination would be bad I think?

How should we determine which transforms are invalid with other ones? I was thinking we could specify with two arrays for each transform which transforms are invalid when in front or behind.

I'm closing this since transform compatiblity was largely resolved by https://github.com/shotover/shotover-proxy/issues/520


Q: so for App flow dataset, the only feature is time?

https://github.com/ant-research/Pyraformer/blob/master/preprocess_flow.py#L19-L22
extract: time, weekday, hour, month
and is used here:
https://github.com/ant-research/Pyraformer/blob/master/preprocess_flow.py#L54-L57
I'm just wondering:

why, for example, not using zone (convert to some integer) as extra features, and in that case, how does this model perform?

or: if the train data only contains the single time feature (without weekday, hour, month), will this model still perform?

Sorry for the silly questions, want to hear your insight.
Thanks.

Hi,

The information of 'zone' and 'app_name' is actually used, see https://github.com/ant-research/Pyraformer/blob/master/preprocess_flow.py#L13 and https://github.com/ant-research/Pyraformer/blob/master/preprocess_flow.py#L57. Each 'app_name' in each 'zone' corresponds to a time series, so we convert the 'app_name' and 'zone' information into an integer, namely, the 'seq_id'.
It is also possible to make predictions based solely on historical time series. Following previous works, our implementation introduced these covariates.

I got more questions: in
https://github.com/ant-research/Pyraformer/blob/master/preprocess_flow.py#L55
  single_data[:, 0] = seq_data.copy()

is the real raw input data,
but in https://github.com/ant-research/Pyraformer/blob/master/data_loader.py#L513
        cov = all_data[:, :, 1:]

it's dropped from the training data?
That's my question: so the previous value of the raw input sequence value is not used at all in training?


Private Module Registry

Question/Feedback
First of all, it's great to see the work you have done in the area of ALZ.  It's an area we are actively working on at the organisation I work for.
I'm interested in your thoughts around the development lifecycle and versioning strategy for BICEP module engineering.
We have been following publish bicep modules to ACR with azure devops with interest and have been prototyping using Nerdbank as mentioned in the thread.  Will you be aligning to the outcomes from the work in relation to the MSFT public repo?
I guess it would also help to understand the ACR tagging strategy too - module development versus module release.

Hey @furrm, Thanks for raising this question and for your kind words RE ALZ, it means a lot!
This is something we are definitely considering and reviewing internally in the team and with the Bicep engineering team.
But as you rightly mention the upcoming public Bicep Registry work that the Bicep engineering team are working on and discussing here https://github.com/Azure/bicep/issues/5138 is something we are actively participating in and will align to to ensure there is no conflicts or different approaches, unless absolutely required.
Some other similar and interesting project we are also collaborating on with Bicep also use the Nerdbank versioning approach as you can see here https://github.com/Azure/ResourceModules/blob/main/arm/Microsoft.Network/azureFirewalls/version.json & here https://github.com/Azure/ResourceModules/blob/main/.github/actions/templates/publishModule/action.yml
Summary
Yes will be aligning to what is done upstream on the public Bicep registry, when it lands, and will link to and/or create documentation specific to ALZ-Bicep as required to help customers on their journey 👍

We have no ETAs yet and are dependant on the Bicep work upstream - but this is something we are keeping a close eye on 👍

Ado sync


PR releases release (unwanted) merge state

This is a follow up to https://github.com/leanprover/lean4/issues/2441
The PR releases are great! But I think they don’t release what they should.
Today, I pushed https://github.com/leanprover/lean4/pull/2577/commits/cf120acb2c43f3922ac2a2c9115b3353269060ff to https://github.com/leanprover/lean4/pull/2577/.
The release at https://github.com/leanprover/lean4-pr-releases/releases/tag/pr-release-2577 however isn't built from cf120acb2c43f3922ac2a2c9115b3353269060ff, but rather from a synthetic merge commit with lean’s latest master, namely https://github.com/leanprover/lean4-pr-releases/commit/96a003904b50ebd7e08c6e2228b202ec57cd6668 which does not exist on any branch, and has

Merge https://github.com/leanprover/lean4-pr-releases/commit/cf120acb2c43f3922ac2a2c9115b3353269060ff into https://github.com/leanprover/lean4-pr-releases/commit/44416624904d389f6fb2cddf018594d34624b2bf

as the commit message.
Looking at https://github.com/semorrison/lean4/blob/master/.github/workflows/pr-release.yml it seems the issue isn’t actually with this job; this job simply uses the commit that was tested by CI.  The problem is thus with https://github.com/semorrison/lean4/blob/master/.github/workflows/ci.yml using actions/checkout@v3:
That action has the very annoying default to not check out the tested branch as it is, but rather as it would look like if it were merged into master at that particular point (see https://github.com/actions/checkout#Checkout-pull-request-HEAD-commit-instead-of-merge-commit for docs). This is of course going to be very confusing: The CI results and the toolchain obtained this way describe some rather arbitrary state that depends on both the branch and master at the time it runs, and will be something else at some later time, even if the branch hasn’t changed. (Note that Github does not update the CI status when master evolves, so this logic is surprisingly broken for a default.)
If others agree that this is a problem then my suggested way to fixing this would be to make all CI jobs test the head commit of the feature branch, and not some merge state, and additionally enable “strict merges” so that only PRs that are up-to-date with regard to master can be merged.

I guess I should @semorrison since it is related to #2441.

@nomeata, I agree this is a problem.
We also need to consider how we should construct the Mathlib testing branches lean-pr-testing-NNNN. Currently every time a lean-pr-testing-NNNN toolchain is created, we merge Mathlib's nightly-testing into lean-pr-testing-NNNN, and try that.
Obviously this is an approximation that is only certain to work if:

The PR NNNN is up to date with respect Lean master
Mathlib's nightly-testing compiles against master

Further, of course I don't actually attempt to keep Mathlib's nightly-testing working against master: only against the latest nightly release.
While your proposed change will definitely be an improvement for testing of individual PRs against ad-hoc projects, I'm worried it will make testing Mathlib worse, because we will be further away from the approximation that the PR is always update to date with master.
So far I'm only coming up with really complicated solutions, such as:

for each new Lean PR, work out where it departs from the master branch, and find the two nightly release on either side of that. (Perhaps there isn't yet one after that.) Select either the one before or after. (Which is better?)
look through Mathlib's nightly-testing branch to find some commits that use that particular nightly, and succeed (Which of these?)
make the lean-pr-testing-NNNN branch by starting from that commit on nightly-testing, and setting the toolchain to leanprover/lean4-pr-releases:pr-release-NNNN.
if the Lean PR is updated, and its point of departure from the master branch changes ... ?

It's clear that the integrated testing with Mathlib is both fragile and really useful. We need a plan that includes making it more robust.

Indeed, this is all very messy :-).
I’ll ponder it a bit, but I agree there is no quick fix here.

@nomeata, just making a clarification to your description above.
It's not really the actions/checkout action at fault here. It is just using the GITHUB_SHA as provided by github, which for a pull_request is as you describe (and as specified at https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request).
I am increasingly thinking that we want to change this behaviour, however.


feat: base token integration tests

What ❔

Introduce 2 new tests for base token:

gas calculation
proper application of dynamic changes to base token ratio

Improve forced price client fluctuations implementation to limit difference between consecutive values.

Why ❔

We want better integration test coverage of custom base token

Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.

it seems base ERC20 contract checks › Base token ratio is updated on L1 is flaky (check last run logs)

Due to removal of zk based tests, this PR is blocked by https://github.com/matter-labs/zksync-era/pull/2898

@Deniallugo ^

it seems base ERC20 contract checks › Base token ratio is updated on L1 is flaky (check last run logs)

I believe I found where the price was not set properly. Seems this was not "flaky", but permanently broken. pushing a fix...


Option to limit the depth in the left sidebar

Hi there,
We've recently ported our documentation over to Sphinx in the @godotengine community, and we're currently using the rtd theme: http://godot-wiki.readthedocs.org/
One minor issue that we're having is that the left sidebar will always show the headings of the currently opened page, while we would like to keep this left sidebar only limited to :toctree: elements (i.e. it should show only page names, not give links to each page's sections).
An example would be this page for example: http://godot-wiki.readthedocs.org/en/latest/reference/richtextlabel_bbcode.html#reference
We find that two levels of depths (heading 1 (page title) and heading 2 + 3) is a bit too much and might give users the feeling that there are many subtutorials, when those are actually just (relatively short) subsections.
Is there already an option to limit this behaviour to e.g. only 1st-level heading, or if not, would that be a feature proposal that you would find pertinent? And again if not, could you give us some pointers on how to customize this ourselves on our end?
Thanks in advance!

+1

I believe the collpase_navigation option should give you the results you want:
https://github.com/snide/sphinx_rtd_theme/blob/8c880a64fa0f08785ffc560708628fc3108a2f3d/sphinx_rtd_theme/theme.conf#L10

Closing this due to inactivity, but feel free to reopen if the above fix didn't resolve your issue.

The above fix did not solve the issue, it's actually not an option of the rtd theme (if you grep for it in the repo it's used only in this file). I believe it's an option the rtd theme passed to the base theme in extends on, but I don't think it's available for rtd_theme users to modify (at least it did not change anything when I tried to use it).

Sorry, collapse_navigation doesn't set the option you are looking for, it instead sets whether the toctree for the nav should be built for all pages or only the current. A theme option should be added for toctree maximum depth, as passed into the toctree template function at the same place as collapse_navigation:
https://github.com/snide/sphinx_rtd_theme/blob/master/sphinx_rtd_theme/layout.html#L124

FYI @akien-mga et al., I wrote an extension that allows limiting the nesting depth in the nav pane.


🐛(tray) specify ingress class name to use

Purpose
To use the correct ingress controller, we have to specify in the ingress
spec wich one to use with the ingressClassName parameter set to nginx
Proposal

[x] specify ingress class name to use

We can wait a little before merging it, I'm preparing an other one on arnold and we can maybe use some improvement made in it.

Once this PR on arnold accepted and released we have to update the tray by declaring a new variable specific to joanie to configure the ingress class name.

With the new arnold version released, this PR can be merged.


chore: bump dependencies

Bumps dependencies to solve installation issues:
# Shortened version

Could not resolve dependency:
peer eslint@"^2 || ^3 || ^4 || ^5 || ^6 || ^7.2.0" from<EMAIL_ADDRESS>node_modules/eslint-plugin-import
<EMAIL_ADDRESS>from the root project

Conflicting peer dependency<EMAIL_ADDRESS>node_modules/eslint
  peer eslint@"^2 || ^3 || ^4 || ^5 || ^6 || ^7.2.0" from<EMAIL_ADDRESS>  node_modules/eslint-plugin-import
<EMAIL_ADDRESS>from the root project

Thanks @GabriMcNab, I've got not write access to merge btw 👍


[2.7] Improve method callback test

What is the purpose of the change
Improve MethodConfigCallbackTest, tests transaction of callback method #8098
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8120 (f7d10a6) into master (c8cefee) will decrease coverage by 0.79%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #8120      +/-   ##
============================================
- Coverage     61.15%   60.35%   -0.80%
+ Complexity      489      487       -2
============================================
  Files          1090     1090
  Lines         43937    43892      -45
  Branches       6418     6386      -32
============================================
- Hits          26870    26493     -377
- Misses        14089    14414     +325
- Partials       2978     2985       +7

Impacted Files
Coverage Δ

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-73.34%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
25.71% <0.00%> (-60.01%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-57.15%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-28.58%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...mmon/config/configcenter/DynamicConfiguration.java
68.18% <0.00%> (-18.19%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
55.88% <0.00%> (-17.65%)
:arrow_down:

...ient/metadata/store/RemoteMetadataServiceImpl.java
38.59% <0.00%> (-17.55%)
:arrow_down:

... and 61 more

Continue to review full report at Codecov.

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


Doing some more static typing

Travis tests have failed
Hey @silasary,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.


Quote JSON Strings

The previous approach causes problems when there are e.g. newlines in the pull request body. Also, the draft parameter was previously passed as a JSON string ("draft": "true" and "draft": "false", as opposed to "draft": true and "draft": false), which I suspect is just a bug.

While working on this, I discovered two other issues:

When the last curl fails for any reason (e.g. invalid JSON in my case), it does not report a failure (exit with non-zero)

The "check for existing pull request" should not try to match against the pull request body. However, I actually don't think the step to check for duplicates is necessary at all, as GitHub simply won't allow you to have two different pull requests with identical "head" and "base" – after all, pull requests merges the contents of a branch into another branch, not just specific commits, so two pull requests with identical "head" and "base" are by definition identical in every way.

If you try to do this (as the script would, since the check was wrong), you will get this error from GitHub:
{
  "message": "Validation Failed",
  "errors": [
    {
      "resource": "PullRequest",
      "code": "custom",
      "message": "A pull request already exists for ember-learn:super-rentals-tutorial."
    }
  ],
  "documentation_url": "https://developer.github.com/v3/pulls/#create-a-pull-request"
}

So, I think all you would have to do is to send the second request, and check the response for the "A pull request already exists" error (you can probably just grep the whole response for that substring). If it is a successful response, or the error matches that string, exit cleanly, otherwise, print the error and exit with a non-zero error code.

hey @chancancode ! The contributor that added the true/false chose to represent them as a string, which (as long as the instructions are clear) wouldn't be an issue. You are saying that your preference is for true / false, do you think this is the better / more intuitive implementation? Could you explain to me how this is different (setting to "true" or "false":
    if [[ "${5}" ==  "true" ]]; then                         # if PRs are draft
      DRAFT="true";
    else
      DRAFT="false";
    fi

than what you are describing? And could you show me an action (or screenshot) of the changes working (your tests?)

Your second suggestion would work - do you want to add to this PR and demonstrate the test? Thank you!

@vsoch it is being sent over the wire as a string, in the JSON (as far as I can tell). i.e., before this commit the curl command was sending this to GitHub:
{
  "title": "...some title...".
  ...
  "draft": "true"
}

This is because L64 in the old diff:
DATA="{\"title\":\"${TITLE}\", ..., \"draft\":\"${DRAFT}\"}"

The GitHub documentation requires the draft key to be a JSON Boolean value. I'm not sure why it wasn't causing problems (maybe it is? don't know, as I was debugging other problems I had at the time). In any case, it seems good to send the value properly so I changed it.

Oh understood, thanks for the clarification! I wonder how it worked for the previous author (that added the feature?) In any ways, this change is definitely good. I'm happy to merge here - do you want to discuss the other bugs in this PR or open another?

@vsoch The "exit code/duplicate PR" problem isn't directly related to what I was fixing here (multiple strings in the body). I'd prefer to do it separately – I also probably won't have time myself since I'm going on vacation today. Maybe we can convert it into an issue and perhaps someone else can pick it up!
As for this code in the PR, I tested it here: https://github.com/ember-learn/test-gh-actions/commit/9e6af732e0e1386cd95579e184b9f862a0e9ca41/checks

@vsoch me too! my best guess is GitHub must be using a certain weakly typed language that is happy to cast "true", "false" into true, false.. or that it was setting everyone's pull requests to draft..?
¯\_(ツ)_/¯

Thanks @chancancode - the test looks good, and I'm grateful for your fix here! The GitHub API is one of my favorites, but I'm sure it has a bug every now and then. I'll open up the issue to address the curl return code. Merging - enjoy your vacation! :tropical_drink:


CI: Update pypi workflow

Address alert https://github.com/valkey-io/valkey-glide/security/dependabot/4 and couple alerts posted there.
I also fixed indentations with prettier

@acarbonetto no need, its a voulonrability in the CI, not package


Missing 'similarity' field in knn search request

From 8.8 version, elasticsearch has field 'similarity' for search in knn field.
Right now, the api doesn't support this field and fail.
🐛 Missing type
https://www.elastic.co/guide/en/elasticsearch/reference/current/knn-search.html#_search_knn_with_expected_similarity
Unknown field 'similarity' (JSON path: similarity) (line no=2, column no=15, offset=16)
Definition
If possible provide a basic definition that we can use as base.
You can see here how to create a basic definition for any API.

I tried to push a branch with the change, but I haven't permissions.

https://github.com/elastic/elasticsearch-java/issues/411 Related.
I also tried to open a PR with the fix, but same issue.

@adrian-arapiles did you fork the project and send a PR from a branch in your fork? You cannot create a branch in the original repository. /cc @AlmostFamiliar

#2261 is merged and backported to 8.8, 8.9 and 8.10, so this should be fixed in the next patch release for each of those, or 8.11.0 when it releases.


Improvements to CTS

Improved completion status handling - requires that all synchronous tests have completed and no asynchronous events have been received and processed for at least 60s.
Removal of use of deprecated constructors.
Also (minor) restores formatting to previous code style.

Fixes issue #3741

Together with PR #3571, this fixes issue #3326


Update anima.gd

In your documentation example you seem to be using type inference:
var anima := Anima.begin(self)

anima.then({ node = $node, animation = "tada", duration = 0.7 })
anima.play()

But in your code you are not telling the compiler what type it returns which leads to the following error:

The following pull request should fix that. Otherwise you can just change your documentation to not use type inference

Due to a cycling reference, I had to remove the AnimaNode return type.
Also, at the moment the documentation is not "aligned" with the master version of the plugin, yet. I'll try to update at least that point ASAP.

Due to a cycling reference, I had to remove the AnimaNode return type. Also, at the moment the documentation is not "aligned" with the master version of the plugin, yet. I'll try to update at least that point ASAP.

Makes sense! I'm not getting any cycling reference error BTW.


Add method to re-play recent song(s)

I'd like something like either a simple "encore" command, or some way to print recent songs played, with an easy way to request one again.
Would that be a suitable addition for the bot, do you think?

is there any update on this issue

Maybe just use the logs in the terminal / cmd ?


Dynamically load partials, aka res.render

When booting up the server, the partial templates are baked into the process and do not change when the layout hbs files are updated. This is unlike res.render which appears to use the latest file contents for each render. Is there a way to achieve this same functionality for partial templates? thanks!

handlebars-layouts is not responsible for loading partials, it merely makes use of what partials has been registered. Is it possible you logged this issue in the wrong repo?

you're right, just noticed the registerPartial API in handlebars. closing


Multiple call of the inspector

Multiple call of the one inspector instance in one PLC cycle should be detected.

/cib

Branch 269-Multiple_call_of_the_inspector created!


Docker progress

Just getting things set up, serverless package was hanging every time after the following line:
Serverless: Docker Image: lambci/lambda:build-python3.6

I poked around and determined that it's just taking a quite long time (~15min) to pull the docker images. That's fine, but it was quite unclear at first. It would help to have some details on the command line, either a progress meter or at least a warning.

I'm glad I saw this. I'm manually pulling the images so I can watch the progress bar. Would be helpful if STDOUT was passed through...

Hey 👋 I'm closing this ticket as it looks like it's heavily outdated, we can of course reopen it if needed 👍


Add-SPShellAdmin grants dbo_owner role???

In the description section there is the following statement:
"However, the result of running this cmdlet is that the user specified with the UserName parameter will have the dbo_owner role access on the affected databases as described above. Therefore, you should carefully plan which users are given this access."
I don't think this statement is correct, at least for SharePoint 2013. After running the cmdlet, a user will be added to SharePoint_Shell_Access database role, which is included in SPDataAcess role, but not in "dbo_owner" (I have to point out that "dbo_owner" role is not a thing, anyway).

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

ID: c30c2c93-f238-cc97-6c24-fe4d8e0650f5
Version Independent ID: 2d458263-4211-90d8-a004-a7c440ec1bb6
Content: Add-SPShellAdmin (sharepoint-server)
Content Source: sharepoint/sharepoint-ps/sharepoint-server/Add-SPShellAdmin.md
Service: unspecified
GitHub Login: @techwriter40
Microsoft Alias: kirks

@egor-yudkin Thanks for your valuable comment. We are working to update as necessary.

We need to be clear on this one. The cmdlet has two behaviors. If the SPDataAccess role exists, it uses that. If it does not exist, it indeed does add the user to db_owner.

@tseward Trevor what needs to be changed in this sentence then?? thanks
"However, the result of running this cmdlet is that the user specified with the UserName parameter will have the dbo_owner role access on the affected databases as described above. Therefore, you should carefully plan which users are given this access."

@Techwriter40 I might phrase it like this:
"However, the result of running this cmdlet is that the user specified with the UserName parameter will have the SPDataAccess role, if it exists, or db_owner role, if the SPDataAccess role does not exist, on the affected databases as described above. Therefore, you should carefully plan which users are given this access."


Bundler errors out when trying to read mono-config.json

I'm running on Linux, so that might be related, but I have all the mono packages installed.
Stack trace:
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018: The "BundlerBuildTask" task failed unexpectedly. [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018: System.IO.DirectoryNotFoundException: Could not find a part of the path '/home/user/repos/Screeps/bin/Release/net7.0/AppBundle/mono-config.json'. [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at Interop.ThrowExceptionForIoErrno(ErrorInfo errorInfo, String path, Boolean isDirError) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String path, OpenFlags flags, Int32 mode, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, UnixFileMode openPermissions, Int64& fileLength, UnixFileMode& filePermissions, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 u.nixCreateMode) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at System.IO.StreamReader.ValidateArgsAndOpenPath(String path, Encoding encoding, Int32 bufferSize) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at System.IO.File.ReadAllText(String path, Encoding encoding) [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at ScreepsDotNet.Bundler.BundlerBuildTask.Execute() in E:\Development\VS2019 Projects\ScreepsDotNetGit\ScreepsDotNet.Bundler\BundlerBuildTask.cs:line 99 [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute() [/home/user/repos/Screeps/Screeps.csproj]
/home/user/.cache/NuGetPackages/screepsdotnet.bundler/1.1.1/build/ScreepsDotNet.Bundler.targets(4,3): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [/home/user/repos/Screeps/Screeps.csproj]

This happens when I include the ScreepsDotNet.Bundler package, the code in Program.cs has no impact. This builds successfully if I remove the package.
I could not find this mono-config.json even when it built without the package, I don't know where to get it from, or where it comes from.
Thank you for making this project, it took me a bit to find, but I was so happy when I did, since there is no documentation elsewhere on how to do this.

I think I found the problem. When I built the sample project, you made it work correctly. I think it had to do with creating a .net 8 wasmbrowser project, and retrofitting it. I could not find out how to create a .net 7 wasmbrowser project directly because the template only supports .net 8, maybe if I install an outdated version of the templates?
Possibly, the best solution to solve this would be a template specific to this project to reduce confusion, or to instruct people to start with the sample project.
It is working for me now, so I am going to close this issue. I hope my documentation of this issue helps someone in the future.
Thanks again for your amazing work, hopefully I can get it working as I want without any further problems.

How did you fixed it? I don't understand your last comment.

How did you fixed it? I don't understand your last comment.

This was a while ago, so I don't remember exactly. But basically, I just used the sample project that was in included here and the problem went away. It was due to this needing an outdated version of the wasmbrowser template, which is no longer the default,


nrf-softdevice-gen broken

Fallout from  #1
moving the .cargo/config up means nrf-softdevice-gen is currently broken unless you pass an override --target or just comment out the target in .cargo/config
Mixing std and no_std in a single workspace is kind of a pain
You probably want to move it outside the other no_std stuff or the entire repo?

True. That's the reason why .cargo/config was in examples/, but that means you can't build/run examples from the root so it isn't ideal either :(
Not sure what to do.
Relevant issue: rust-lang/cargo#8112

One thing I often do is to create a Makefile to wrap language specific tools (rake, webpack, iex, cargo...). Then you just run make build, make gen... And you ensure correct environment in the tasks. Of course this is just a suggestion.


Added tests for Solr binding

Relies on PR #574. Relates to @busbey  comment about testing the Solr binding https://github.com/brianfrankcooper/YCSB/pull/574#issuecomment-170245724. Will most likely need to be rebased after #574 is merged.

Dammit, did I comment on the wrong PR? Assume all my comments go to the other one then, if so. Sorry. :(

@busbey addressed comments about putting reference to Solr config files and configuring log4j to go to stderr.

@madrob no worries I added links to them in PR #574

@madrob @risdenk addressed in PR #574
cheers

thanks!


Make Filings | Add Submission documents type master to MDMS

[x] Add documents type master to MDMS

Submissions PRD _ PUCAR.pdf

Tested sol-dev workbench, working as expected:
Attaching screenshot for reference:

cc: @rajeshcherukumalli

Tested in UAT env, working as expected.
Attaching screenshot for reference:

cc: @rajeshcherukumalli

Hi @ansh-egov , tested in uat env, kindly provide sign-off.
cc: @Ramu-kandimalla @rajeshcherukumalli

HI @rajeshcherukumalli, @Ramu-kandimalla
This is deployed to Prod.


I want to autoPlay video using MuxPlayer react

Is there an existing issue for this?

[X] I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply
mux-player-react
Description
I want to autoPlay video using MuxPlayer react.
Expected Behavior
Currently, I'm not able to autoplay video without muting using mux player react. I know it is not recommended to autoplay without muting. but according to my business model, I need to autoplay the video without muting it like youtube playing their video.
I'm looking for a suitable solution for my use case. Please also provide any code snippets if you have any.
Thank you.

Autoplay nowadays is pretty tricky and may not be allowed to happen unless muted. Though, it depends on the browser settings for a particular user.
In Mux Player, we have extended the regular autoplay property to allow for easier configuration of autoplay. For example, with autoplay="any", it'll try to autoplay unmuted and if the player cannot autoplay, it'll mute the player and try again.
Unfortunately, there is no way to force autoplay with sound.

HI, one more issue I'm facing that is PIP is not only playing on the current video page and it stops playing on other screens. can you please guide me on how to fix that issue?


dev-c7 won't launch with userns and overlay filesystem

with --userns=keep-id
and my ~/.config/containers/storage.conf looking like this:
[storage]
  driver = "overlay"
  runroot = "/run/user/1200288"
  graphroot = "/scratch/hgv27681"
  [storage.options]
    size = ""
    remap-uids = ""
    remap-gids = ""
    remap-user = ""
    remap-group = ""
    skip_mount_home = ""
    mount_program = "/bin/fuse-overlayfs"
    mountopt = "nodev,index=off"
    [storage.options.thinpool]
      autoextend_percent = ""
...

I get the following error:
Error: error creating container storage: error creating an ID-mapped copy of layer "36b5a0bd68a2332cb83aaffcfddeb0a1b3026f7d9ff3044418eeb4573a940329": exit status 1: error during chown: storage-chown-by-maps: lchown var/lib/yum/yumdb/g/06b8dfd77204230cfc286194c8ce240ec81eca18-gstreamer-plugins-good-0.10.31-13.el7-x86_64/checksum_type: no such file or directory

I have been investigating this as part of the transition to RHEL8 workstations, I've been recording my findings on JIRA: https://jira.diamond.ac.uk/browse/CGP-291