Default error summary heading to match GDS guidelines

GDS guidelines on https://design-system.service.gov.uk/components/error-summary/ states that the heading must be ‘There is a problem’:

We should therefore default the heading to this value.
I haven't changed the docs for the component, since we do still allow the override.  Was thinking about something like:

Optional message to alert the user to a problem goes here, defaulting to 'There is a problem'

[ ] Documentation
[x] Tests
[x] Ready to be merged

Deploy preview for govuk-react processing.
Building with commit 66154da4e461e39a4e1bd32cb4c5ae4f09dd9bd3
https://app.netlify.com/sites/govuk-react/deploys/5c94c1eb0aad600007f868db

Codecov Report

Merging #636 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #636      +/-   ##
==========================================
+ Coverage   99.66%   99.66%   +<.01%
==========================================
  Files         104      104
  Lines         603      604       +1
  Branches       69       69
==========================================
+ Hits          601      602       +1
  Misses          2        2

Impacted Files
Coverage Δ

components/error-summary/src/index.js
87.5% <100%> (+1.78%)
:arrow_up:

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 02427f2...66154da. Read the comment docs.

Cool stuff.
The docs will get updated to show the default value when they're next re-generated.


arch/sim: Add support for ucontext API

Summary of Changes
The ucontext API allows us to jump from the signal handler without altering the
handler's execution flow. A new host API is introduced:
getcontext()/setcontext()/swapcontext()/makecontext().
This feature can be enabled by setting CONFIG_SIM_UCONTEXT_PREEMPTION=y.
This is a continuation of the work done here: https://github.com/apache/incubator-nuttx/pull/1663
Impact
When the option CONFIG_SIM_UCONTEXT_PREEMPTION is not enabled the patch is using the previous mechanism :
setjmp/longjmp and is not altering the original functionality.
Signed-off-by: Sebastian Ene<EMAIL_ADDRESS>

@xiaoxiang781216  I will close the other pull request. Please review this one, it adds the ucontext API without removing the original setjmp/longjmp mechanism.

Let's close this old PR since the same functionality is implemented by https://github.com/apache/incubator-nuttx/pull/1859.


Styling issue when using the background picker in mobile mode

This gif recorded from the codesandbox demo demonstrates the issue. On the website, the problem is prevented by the language picker.

https://github.com/excalidraw/excalidraw/pull/3399 changes the positioning to absolute which kinda fixes the problem, though it's not ideal either. (https://excalidraw-71irf132f-excalidraw.vercel.app/)


Rental model for contexts?

We have discussed possibly doing an approach where you borrow a context for a while and then return it.

We've not yet found any reasons to need to do this. At one point we thought either SQL or Gremlin would need it, but it turns out they don't. But we've not exhaustively considered every known storage type, so this might come back at some point. But it's definitely not a v1 thing...


Allow Both Models and ActiveRecord::Relations as Input

Issue #18 was opened and commented about how Sprig-Reap appears to handle ActiveRecord::Relation-type input.  In this PR, I made a pretty minimal number of adjustments in order to accommodate AR-relations.  I also took the opportunity to refactor and clean up some of the code.
Fixes #18.

nice abstraction on those Inputs :+1:

:+1: Nice work!

Awesome -- thanks for reviewing!


refactor: usePlaybackState without initial undefined

The idea is to store the data in a singleton and consume it as an initial value for state.
The problem with undefined as an initial value comes in the scenario, when music is playing and hook gets mounted (you open a player screen) -> in this case first state value will be undefined and for play/pause/spinner button most likely it'll be shown as "loading" and only after that will be shown as "playing" (such blink is annoying and this PR attempts to resolve it).
Let me know if you have any concerns with such approach 👀

@dcvz may I kindly ask you to review this PR? 👀

I am not sure automatically installing a listener like is a good idea, since you are unnecessarily using resources if it goes unused - i.e. when you don't use the hook. If you want to cache this value across unmounts and remounts, another approach would be to lift your usage of the hook up to the root of you app and pass the value down as props / context.. In a any case you would always need to start out with undefined instead of State.NONE

Ah, okay @dcvz and @puckey
I understand your point of view and it makes sense to me 👍 Feel free to close this PR then 😊


ScriptHandler::clearCache on composer install throws exception "Unable to parse file"

Hey,
I have recently tried to update to Symfony3 and as a result had to update to "sensio/distribution-bundle": "5.*".  I get an error when I run composer install:

composer install

Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache

  [Symfony\Component\DependencyInjection\Exception\InvalidArgumentException]
  Unable to parse file "/project/folder/vendor/symfony/framework-bundle/DependencyInjection/../Resources/config/web.xml".

  [InvalidArgumentException]
  [ERROR 3067] Element '{http://www.w3.org/2001/XMLSchema}import': Failed to parse the XML resource 'file:///project/folder/vendor/symfony/dependency-injection/Loader/schema/dic/services/services-1.0.xsd'. (in in_memory_buffer - line 8, column 0)

Script Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache handling the post-update-cmd event terminated with an exception

  [RuntimeException]
  An error occurred when executing the "'cache:clear --no-warmup'" command:

    [Symfony\Component\DependencyInjection\Exception\InvalidArgumentException]

    Unable to parse file "/project/folder/vendor/symfony/framework-bundle/Depende
  ncyInjection/../Resources/config/web.xml".

    [InvalidArgumentException]

    [ERROR 3067] Element '{http://www.w3.org/2001/XMLSchema}import': Failed to parse the XML resource 'file:////Users/romain
  /project/folder/vendor/symfony/dependency-injection/Loader/schema/dic/services/services-1.0.xsd'. (i
  n in_memory_buffer - line 8, column 0)

BUT running manually bin/console cache:clear --no-warmup:
 // Clearing the cache for the dev environment with debug true
 [OK] Cache for the "dev" environment (debug=true) was successfully cleared.

I never had this issue before and could be a non-issue but if anyone has an idea it would be very helpful.
Thanks.

OS X 10.11.2 / PHP 5.5.30

Hi @my2ter can you still reproduce this issue?

This issue seems to come from the FrameworkBundle. I'm closing, if you encounter this issue again, please open an issue on the symfony repository.
Thanks.


switching to tehwalris/go-freeipa instead of ubccr/goipa

hi,
would you mind switching to the other ipa library, it is created based on the IPA api definition, that is available via there API endpoint.
this means that the "go-freeipa" client more or less supports all available JSON RPC methods the API offers, where goipa only supports methods that that have been implemented manually.
with this, it would be possible to create more resources for this provider.
i would do it myself, but i am not familiar with go.

Hi @schlitzered we are not actively working on this at the moment. I'll see if I can take a look at it in the near future but can't offer any promises.


Is cfscrape dead?

Is this module dead? Owner not supporting anymore?

Yup
Tonnes and tonnes of issues about it not working, and no update from the owner in many months. Pretty sure it's dead.
Some folks moved over to https://github.com/VeNoMouS/cloudscraper/ but I believe that's moved over to a subscription model where you have to contact the owner over discord? and the owner has disabled issues, and shuts down any comments on PRs about functionality.
I've not seen a lot of other options around.

Some folks have said to use scrapy-splash: https://github.com/scrapy-plugins/scrapy-splash/issues/92#issuecomment-680050082 but I'm not sure how that goes.
There's some JS options around, again, not sure whether they're working:

https://www.npmjs.com/package/hooman (last update 5 months ago, repo issues say it's broken, dev says he'll work on it, but is very busy https://github.com/sayem314/hooman/issues/34#issuecomment-703052304)
https://www.npmjs.com/package/cloudflare-scraper (last update 6 months ago, repo issues say it's broken)
https://www.npmjs.com/package/@ctrl/cloudflare (last update 8 months ago, repo archived, suggests )
https://www.npmjs.com/package/cloudscraper (last update a year ago)

Browser based solutions:

https://github.com/FlareSolverr/FlareSolverr

That last one sounds like the best bet to me. It's a docker image that spins up browsers to do the request, so it's going to be slow and quite memory-heavy. But development seems active, latest commit to master was 5 days ago.
But I've not tried it myself.

Yup
Tonnes and tonnes of issues about it not working, and no update from the owner in many months. Pretty sure it's dead.
Some folks moved over to https://github.com/VeNoMouS/cloudscraper/ but I believe that's moved over to a subscription model where you have to contact the owner over discord? and the owner has disabled issues, and shuts down any comments on PRs about functionality.
I've not seen a lot of other options around.

Some folks have said to use scrapy-splash: https://github.com/scrapy-plugins/scrapy-splash/issues/92#issuecomment-680050082 but I'm not sure how that goes.
There's some JS options around, again, not sure whether they're working:

https://www.npmjs.com/package/hooman (last update 5 months ago, repo issues say it's broken, dev says he'll work on it, but is very busy https://github.com/sayem314/hooman/issues/34#issuecomment-703052304)
https://www.npmjs.com/package/cloudflare-scraper (last update 6 months ago, repo issues say it's broken)
https://www.npmjs.com/package/@ctrl/cloudflare (last update 8 months ago, repo archived, suggests )
https://www.npmjs.com/package/cloudscraper (last update a year ago)

Browser based solutions:

https://github.com/FlareSolverr/FlareSolverr

That last one sounds like the best bet to me. It's a docker image that spins up browsers to do the request, so it's going to be slow and quite memory-heavy. But development seems active, latest commit to master was 5 days ago.
But I've not tried it myself.

image

I didn't mean to sound like I was being negative towards you in any way, you can see here how many duplicate issues there are, and I can see why you might shut them down

image

I didn't mean to sound like I was being negative towards you in any way, you can see here how many duplicate issues there are, and I can see why you might shut them down


URL is missing : when clicking through from Page Builder

When clicking the link at the top of the content editor page, the URL is malformed, missing a single ':' after http.
E.g. clicking this:

Leads to this:

Hi @jkhamler!
I'm really surprised by this one considering where the url is coming from in the codebase.
The base url can also come from a urlWithoutSlug attribute, maybe you implemented an accessor for this?

I can confirm this is happening for me too

This happens to me as well:

@jkhamler - I think it happens becuase in your .env file you have the http:// as part of the APP_URL - I looked at the documentation and they obviously don't have it, removed it and it fixed the issue


Make flake URLs point to Gubernator instead of storage.googleapis.com.

The previous URL was not useful (see kubernetes/kubernetes#26509).
individualFlakeSource/brokenJobSource.ID() now returns things like:
"/kubernetes-jenkins/logs/e2e-gce/123/\n", which allows some flexibility
to make the Body() return different URLs.
There are tests for compatibility with old ID() results.

Added a comment describing gubernator, fixed the contrib/test-utils vending problem by using a symlink.

LGTM, thanks! I will push this soon.


Different map values look the same in ZJSON

Repro is with Zed commit 3a40788.
From what I can see, the ZSON spec is not super explicit about the syntax for an "empty map", but based on the closing remark in this section I think this is the one.
$ zq -version
Version: v1.10.0-31-g3a40788b

$ echo '|{}|' | zq -z 'yield this,typeof(this)' -
|{}|
<|{null:null}|>

If we put a value like this alongside a null value in the same array, ZSON shows them differently as one might expect.
$ echo '[null, |{}|]' | zq -Z -
[
    null (|{null:null}|),
    |{
    }| (|{null:null}|)
]

Along the lines of the variations @jameskerr was addressing in https://github.com/brimdata/zui/pull/2876, he noted that the ZJSON representation that comes from the backend for this leaves no way to tell the empty map value from the null value.
$ echo '[null, |{}|]' | zq -f zjson - | jq .
{
  "type": {
    "kind": "array",
    "id": 31,
    "type": {
      "kind": "map",
      "id": 30,
      "key_type": {
        "kind": "primitive",
        "name": "null"
      },
      "val_type": {
        "kind": "primitive",
        "name": "null"
      }
    }
  },
  "value": [
    null,
    null
  ]
}

Verified in Zed commit 40a7db8.
Now we see two different values in the ZJSON output.
$ zq -version
Version: v1.10.0-34-g40a7db87

$ echo '[null, |{}|]' | zq -f zjson - | jq .
{
  "type": {
    "kind": "array",
    "id": 31,
    "type": {
      "kind": "map",
      "id": 30,
      "key_type": {
        "kind": "primitive",
        "name": "null"
      },
      "val_type": {
        "kind": "primitive",
        "name": "null"
      }
    }
  },
  "value": [
    null,
    []
  ]
}

I also Zui's pointed Zed dependency at this commit and the rendering of this same array of minimal map values shows each element differently more as we'd expect.

Strictly speaking, there's a small divergence between this and the ZSON representation from the Zed CLI tools since at the CLI there's an explicit type decorator shown there.
$ echo '[null, |{}|]' | zq -Z -
[
    null (|{null:null}|),
    |{
    }| (|{null:null}|)
]

However, since the explicit decorators were absent from the input, it seems Zui's rendering is also defensibly correct. @nwt I take it you agree with that? (cc: @jameskerr)


Setting Fat Feeder range to 100,000 lags out dedicated server

This was discovered by one of my playtesters, so I may be missing a few details. The player set the range of a fat feeder block to 100,000, and the server crashed due to performance issues after about one minute. Setting the block back to a more reasonable number fixed the issue.
The fat feeder was inside this player's personal dimension. Running /cofh tps shows that that dimension was running at 0.8 TPS, while the other dimensions were running smoothly.
Crash log: https://pastebin.com/8knUCdue

Let's take this one step at a time:

Player on a server uses a block that normally has a range of like 10
Player, I assume, has what they think is a genius idea to make the range cover everything
Player manages to bump the range of to 100,000, which not only stretches over multiple chunks, it stretches over multiple region files. It, in fact, stretches farther than most single player worlds are ever generated to
Player is surprised that when giving a block that normally has quite a small range a range that covers several times the length of the world and therefore queries every single entity in the entire dimension every few ticks causes lag

This is like when people build a mountain out of TNT, explode it all at once, and report it to Mojang as a bug for lagging.
I suppose I could have put a maximum range in as a sanity check?


evidence level display class

Our data import has extended the values for evidence level.  It works seamlessly on the back end.
However, the client displays the text as white with no background.
I wonder if there could be a neutral background (grey?) to provide better contrast
Thanks

    FDA-approved

The same issue occurs in the evidence summary section:

              FDA-approved - response

We can and probably should have a readable default style for that cell. But are you sure you want to denote FDA approval as an evidence level? We intended those to be a qualitative assessment of clinical evidence, and FDA approval didn't seem to quite fit as an attribute of a study - it's more an attribute of a drug, unless I've misunderstood something.

I guess I can kind of see this.  Right now a mutation-treatment relationship from the literature that happens to be FDA approved should be considered level A, but we don't strictly require that level A evidence be FDA approved.  There probably are diagnostic and prognostic tests in addition to predictive evidence types that are FDA approved.
For display purposes, perhaps a short hand could be used for this new evidence level?  For example, maybe AA, where it is understood that FDA approval is required to achieve an AA status?

An AA rating sounds like a good idea to me and would be easy to add to the rest.

Yeah. I expect there are going to be very few cases where a specific mutation-treatment-cancer (as documented by an evidence statement) is FDA approved. But, when there is I guess we want to document that. It would be great to get some specific examples from @bwalsh of when they would use such a status. Is it more like Josh asked that the drug is just FDA approved for use in some context?

Yeah, agreed, if FDA approval refers to the drug in any context, then this should be metadata on the drug as suggested by @jmcmichael

At our last meeting, we decided to table the inclusion FDA approval metadata in CIViC until we have the time to explore the utility of such a feature and strategies for including it in our schema. Closing.


show_progress non positional argument is defined but not used

Hi Alex,
The keyword argument show_progress in functions raostriling_interdisciplinarity and field_citation_distance from file measures/raostirling.py is never actually used within those functions. Should we remove it or add an if statement to show the progress when show_progress is equal to True?
Best,
Rodrigo

I think for consistency with the other metric functions it would be good to keep the show_progress argument.  Would you want to add a few checkpoints to print progress when show_progress=True?

Sure, I can add that. I'll open a pull request when I have it.


Change dict-object recipe key to dict_object.

Both the meta.yaml and bakeries.yaml files are loaded into types for validation and type checking by pangeo-forge-prefect.  Dataclass properties cannot contain dashes so it may be best to modify this for consistency with the yaml and types.  refs https://github.com/pangeo-forge/pangeo-forge-prefect/pull/7

Good catch!

@cisaacstern That is my understanding of the intent.  You can see an example in the unit tests in pangeo-forge-prefect and the corresponding example fixture.


Fix compilation error associated with release of OpenSSL 1.1

Adds a wrapper function for use in case an older version of OpenSSL is used in order to partially resolve https://github.com/grpc/grpc/issues/10589.
The other errors can be handled by using functions also available in OpenSSL 1.02.
Note that we do not use a shared wrapper layer because this is the only place requiring a wrapper.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

This is OK to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

No significant performance differences

No significant performance differences

@matt-kwong or @jtattermusch could you help me with checking if the reported errors are okay to ignore and merge? Thanks!

All the failing tests seem to be known issues.

Hi I just wanted to ask when this pull request will be merged into master, or if it should not be merged what has to be done for it to be merged to master.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

[trickle] No significant performance differences

@nicolasnoble Can you please approve the change as well? Can @agc-sec merge it then or will someone from the grpc core team do it?

Jenkins, test this please

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Test failures:
Linux - #10894
MacOS - #11223 #11522
Asan C - #11530

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Is it worth merging this one without #11496 fixed first however ?


Update README.md

Just found out a very handy plugin that provides two traits:

JsonControllerTrait turns any Controller into a JSON-friendly controller
JsonResponseTrait exposes a number of response functions

This is already in the list.


refactor(reactflow,svelteflow): use context to persist dragged node type

🚀 What's changed?

Use React/Svelte context to persist the dragged node type
Use context value on drop of node
Replace dataTransfer usages

🌻 Resolves

#465

Awesome :) Thanks @bcakmakoglu!


Can't use keys on custom components.

For now if you try to specify a key for JSX component it won't be actually used by hyperapp's diff/patch algorithm:
const state = {
  posts: [
    'Hello',
    'world'
  ]
}

const actions = {}

const Post = (props, children) => <h1>{children}</h1>

const view = (state, actions) => (
  <main>
    {state.posts.map(post =>
      <Post key={post}>{post}</Post> // <= the `key` prop is not used by diff algorithm
    )}
  </main>
)

app(state, actions, view, document.body)

console.log(view(state, actions)) // =>
// { name: 'main', props: {}, children: [
//   { name: 'h1', props: { no key here }, children: ['Hello'] },
//   { name: 'h1', props: { no key here }, children: ['World'] }
// ] }

Demo: https://codepen.io/frenzzy/pen/dJmEOm?editors=0010
Let's discuss, should we automatically pass the key down into actual virtual DOM node or user must do it manually? How not to forget to do it?

@frenzzy Could you suggest how you think this could be implemented in core? How would you do it?

A possible implementation will add extra bytes and logic with more computations to the core (see examples above), which is not desirable IMO. Now I think it's better to detect misusage by development build #417 and just show warnings to a developer.

@frenzzy I was just curious how would you apply the key to the component.
Would this be easier to implement now because we are storing key in the vnode? https://github.com/hyperapp/hyperapp/commit/9222f99f3de604397cfe9a3580fb8f30c1331947

https://github.com/hyperapp/hyperapp/blob/1.1.1/src/index.js#L19-L26
export function h(nodeName, attributes = {} /*, ...rest*/) {
  var key = attributes.key

  // ...

  if (typeof nodeName === "function") {
    node = nodeName(attributes, children)
    if (key) {
      if (node.pop /* Array? */) {
        for (length = node.length; length--; ) {
          node[length].key = key + (node[length].key || '')
        }
      } else {
        node.key = key + (node.key || '')
      }
    }
    return node
  }

  return {
    nodeName,
    attributes,
    children,
    key
  }
}

Thanks, @frenzzy! I am going to close this as wontfix. If you need this feature can you use a HOA?

Yes, this is solvable via higher order function. Thanks!


fix: add no-sandbox to the Linux builds

Closes #909
Issue related to not fixed issue with Electron and opened tickets around it, some of them u can find here
Shortly, they recommend to use this flag only for Ubuntu 20.04+ versions right now.

I'm a bit confused by these additional screenshots. What do you want to demonstrate? In the rest, 👍

I wanted to show , that , for Linux/Windows/MacOS the app is running.


Pythonise docs: "pxd files" (pxd_files.rst)

Continuing to Pythonize Сython documentation!
This PR Pythonizes this documentation page.
Lots of process details and standardization are found here: #4187

vs

and

vs

We maybe want to correct warning text. Not like it's wrong, but feels little bit like it

I'll fix fail tests soon, it's because code that doesn't ment to work is being tested, for now I'm awaiting reviews

@scoder can u give any comments about this and this?
If we do not change anything, then this is PR is ready to merge.

@scoder can you give any comments about this and this?
If we do not change anything, then this is PR is ready to merge.

I'm still waiting :)

@scoder can you give any comments about this and this?
If we do not change anything, then this is PR is ready to merge.

@scoder, I'm still waiting :)

Are there any other comments?

Thanks


Major UX cleanup+refactor for CLI.

Is it easy to change "Show application version." -> "Show CLI version."?
Flags:
  --help                   Show context-sensitive help (also try --help-long and --help-man).
  --version                Show application version.
  --access-key=ACCESS-KEY  Access Key used for authentication.
  --project=PROJECT        Project name.
  --config="~/.deviceplane/config"
                           Config file to use.

We should marshal here
x:deviceplane josh$ ./deviceplane application inspect demo -c
|
  demo:
    image: deviceplane/demo:2ce33972e04a041e79a2887b52c4758eb6b8d2ed
    ports:
    - 3000:3000

FWIW I don't know how important it is that this flag respects the -o flag. It uses it differently than other -o uses which is a little confusing. Maybe we can just always print the raw config?


Use correct language hints for fenced code blocks.

LGTM!
I'll merge this in, @maxmeyers

Thanks @nettofarah :)


模型加载问题

按照项目流程进行部署，但是出现了下面模型加载问题，请问这是什么原因呀
Loads checkpoint by local backend from path: weights/dwpose/dw-ll_ucoco_384.pth
cuda start
Load from local weight: /home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_detection/detection/sfd/s3fd.pth
Traceback (most recent call last):
File "/home/pod/shared-nvme/VideoChat/video_chat/app.py", line 45, in
from src.pipeline_llm import llm_pipeline
File "/home/pod/shared-nvme/VideoChat/video_chat/src/pipeline_llm.py", line 25, in
from src.thg import Muse_Talk
File "/home/pod/shared-nvme/VideoChat/video_chat/src/thg.py", line 25, in
from src.musetalk.utils.blending import get_image,get_image_prepare_material,get_image_blending
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/blending.py", line 6, in
fp = FaceParsing()
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_parsing/init.py", line 12, in init
self.net = self.model_init()
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_parsing/init.py", line 18, in model_init
net = BiSeNet(resnet_path)
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_parsing/model.py", line 233, in init
self.cp = ContextPath(resnet_path)
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_parsing/model.py", line 95, in init
self.resnet = Resnet18(resnet_path)
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_parsing/resnet.py", line 69, in init
self.init_weight(model_path)
File "/home/pod/shared-nvme/VideoChat/video_chat/src/musetalk/utils/face_parsing/resnet.py", line 83, in init_weight
state_dict = torch.load(model_path) #modelzoo.load_url(resnet18_url)
File "/home/pod/miniconda3/envs/metahuman/lib/python3.10/site-packages/torch/serialization.py", line 1028, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "/home/pod/miniconda3/envs/metahuman/lib/python3.10/site-packages/torch/serialization.py", line 1246, in _legacy_load
magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, 'v'.

weights文件夹里的权重下载了吗？

重新试了一下，原因是模型没有下载完全，下载的模型不完整。


Windows cannot install packages: This operation must be run in a work tree

Output of bower -v && npm -v && node -v:
1.7.9
3.10.8
v6.9.1

Additional environment details (proxy, private registry, etc.):
Windows 7 Pro
Git 2.11.1.windows.1
Steps to reproduce the issue:

Run bower install bootstrap#v4.0.0-alpha.6

Describe the results you received:
bower bootstrap#4.0.0-alpha.6                                     ECMDERR Failed to execute "git clone https://github.com/twbs/bootstrap.git -b v4.0.0-alpha.6 --progress . --depth 1", exit code of #12
Resolving deltas: 100% (41/41), done. Note: checking out '65ffb1c3b6e94440812a2e228cb8e8be512652ec'.  You are in 'detached HEAD' state. You can look around, make experimental changes and commit them,
and you can discard any commits you make in this state without impacting any branches by performing another checkout.  If you want to create a new branch to retain commits you create, you may do so (n
ow or later) by using -b with the checkout command again. Example:    git checkout -b <new-branch-name>  fatal: This operation must be run in a work tree warning: Clone succeeded, but checkout failed.
 You can inspect what was checked out with 'git status' and retry the checkout with 'git checkout -f HEAD'

Additional error details:
Cloning into '.'...
remote: Counting objects: 450, done.
remote: Compressing objects: 100% (437/437), done.
remote: Total 450 (delta 41), reused 118 (delta 6), pack-reused 0
Receiving objects: 100% (450/450), 3.05 MiB | 1.07 MiB/s, done.
Resolving deltas: 100% (41/41), done.
Note: checking out '65ffb1c3b6e94440812a2e228cb8e8be512652ec'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

fatal: This operation must be run in a work tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'

Describe the results you expected:
Package should install.
Additional information:
I know this is very similar to #2321, but running set GIT_DIR= (from cmd) or unset GIT_DIR from Git Bash did nothing to help the issue.
I get the same error when trying to install a particular commit instead of a release. I noticed in the GitRemoteResolver that there are two modes (fast and slow) and I thought slow mode might work better. It did not.
This is not specific for the Bootstrap package, this was happening with all packages.

I feel so dumb. I thought that I was running the latest version of bower (thanks npm update bower!) but it turns out I wasn't.
Updated to 1.8.0 and all is working.


Bumps google-cloud-java API versions to 0.28

Why so much breaking changes?
We can't afford such a luxury in Spring with our backward compatibility policy from day first.
Yes, that's just a rhetorical question, but I'm afraid a maintenance pain is waiting for us...

I think there are some breaking changes because it's alpha/beta.

Beta: Libraries defined at a Beta quality level are expected to be mostly stable and we're working towards their release candidate. We will address issues and requests with a higher priority.

Alpha: Libraries defined at an Alpha quality level are still a work-in-progress and are more likely to get backwards-incompatible updates.

Thanks, @Mistic92 , for explanation!

In any case, I'm not super worried about this because the set of dependencies that we cut releases with should remain stable.
This issue only appeared because @Mistic92 overrode google-cloud-pubsub's version to 0.28, which uses higher gax versions than 1.9, which is what M1 uses in its BOM.
FWIW we need to specify the gax's version on our side because we're using classes from it (ExecutorProvider, ChannelProvider, etc.)


Improve keyboard support

The code of keyboard support is really messy, it should be refractored :zap:

Can I take this one?

@flurmbo Yes :smile:

#8  :fire:


Fixing flaky test_crossref_retraction_status

Adds a VCR for the flaky test_crossref_retraction_status
Adds assertions to resolve union-attr errors

Realized the test test_crossref_retraction_status doesn't check for the presence of retractions.csv on local file system, so the cassette is incorrect. Going to let @geemi725 handle this one


fix: send analytics on successful commands

🎉 Thanks for submitting a pull request! 🎉
Summary
Fixes https://github.com/netlify/cli/issues/4196
Will think on how to test this and add a test Done

For us to review and ship your PR efficiently, please perform the following steps:

[x] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[x] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[x] Update or add tests (if any source code was changed or added) 🧪
[x] Update or add documentation (if features were changed or added) 📝
[x] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

This PR shows a package size of 508MB for some reason. Going to open an issue to investigate

Size change unrelated to this change. See PR to fix in https://github.com/netlify/cli/pull/4247.
Looks like https://github.com/netlify/cli/pull/4233 did it since netlify build had a pinned version of zisi (didn't dive into it too much)


Verify ETH1 chain ID

Description
When the PoWService first connects to the ETH1 node and when sending deposits, check that the chain ID reported by the ETH1 node matches the one in the network config.

I'd like to tackle this, but I have a question. What do you mean by checking the chain ID?
Should it just do an ERROR log in that case, or somehow fail or shutdown?

Good question.  The ideal case is that the ETH1 node is up at startup, and Teku fails to start with a clear error message if the chain ID doesn't match but the ETH1 chain may not be available at startup and we don't want to delay Teku waiting for it to become available.  So I think we'll just have to have something that tries to query the chain ID - started by PowchainService.doStart but that returns immediately and then in the background continues retrying until the ETH1 node is available and we can get it's chain ID.  Then if it doesn't match log a message via the StatusLogger at error level.
The sending deposits part of this description doesn't apply anymore as we'll be removing the subcommands that send deposits (we recommend users use other tools like the launchpad).

Thank you for your input. I still do have a couple of questions :smiley:

Against which chain id value do I have to validate the ETH1 chain id? I couldn't find any mention of a chain id in the existing code.

Regarding the StatusLogger... you're not referring to fatalError with a following System.exit, but to adding a new logging method to StatusLogger, right?

The expected chain ID is in the network definition (so for example the MainNet config: https://github.com/PegaSysEng/teku/blob/aa936b9c36607f434a2a412ad63ad693263ef71c/util/src/main/resources/tech/pegasys/teku/util/config/mainnet.yaml#L55) and those definitions will be loaded into the Constants class at startup (https://github.com/PegaSysEng/teku/blob/ffc0579ab701df30ac43feb26f500b18215d5e17/util/src/main/java/tech/pegasys/teku/util/config/Constants.java#L124).  So you'd request the chain ID from the ETH1 node and check that it matches Constants.DEPOSIT_CHAIN_ID.

Requesting the chain ID from the ETH1 node would be a new method to add to Eth1Provider which will wind up calling web3j.ethChainId() to do the actual work.

Yes, it would be a nw method on StatusLogger.  I'd suggest starting the message with PLEASE CHECK YOUR ETH1 NODE |  to match what eth1DepositEventsFailure outputs.

Just wanted to say that I'm still working on this. I do have a day-job therefore I cannot allocate my time freely to work on teku, that's why it takes a bit longer :)

@systemfreund Thanks for the update, no problem if it takes time.  Feel free to ask more questions either here or on our Discord (https://discord.gg/7hPv2T6) if there's anything we can do to help.


check for lastBuffer for redraw

this fixes reusing a texture like a tilemap atlas and changing offsets

don't merge

LGTM. Thank you for your awesome work :smile:
Will merge when travis is OK with it.


Dashboard should gracefully handle the resource server going offline

Today the gRPC calls retry for few seconds and then this error happens:

We should fail more gracefully, tell the user the resource server is unavailable and avoid unhandled exceptions.

@JamesNK This is fixed right?

@drewnoakes  please confirm if this is fixed. If not please move over to preview 6

This is fixed. We retry.


[TRI-1865] eventTrigger source isn't getting set

Use a custom source with eventTrigger. It doesn't get set on the EventDispatcher, so events sent with the same source don't trigger runs.
TRI-1865

This is fixed in main with https://github.com/triggerdotdev/trigger.dev/commit/129f023d1154d339455381595ad175b283ec2b47
It will be released with the next package release (v2.3.12)


OCPBUGS-655: Update blueocean-autofavorite to 1.2.5

Update to newer version which does not use the groovy-sandbox library

/assign @akram
for lgtm

/jira refresh

/jira refresh

/lgtm

/refresh

/cherry-pick release-4.11


Add support for bars/circles like in eww

Improvement suggestion!
Add support for displaying configurable bars/circles to display things like memory or CPU usage.
Is something like this doable?

That would be cool!


Fix proxyImage annotation in injector

Previously, if you had an image like istio/proxy:1.1.7 it would add the
image and tag again, causing this to fail.
Fixes https://github.com/istio/istio/issues/14365

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn,sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Vision] Add pdf text detection sample

I cancelled Circle since it's not passing right now (focusing on migrating to Kokoro). The tests passed on Kokoro, so you're good on that front. :+1:

@frankyn I'm giving this sample my 👍 I'm explicitly biased towards getting this functionality in users' hands to play with. But, wow, man oh man is this different! Yuck yuck yuck. The code works & is clean (for what it is) & provides access to a powerful, requested feature. But. Yeah. It is what it is.
I've added you to get an explicit LGTM if you're cool with this sample going in – to counterbalance my bias 😇

Update: pending google-cloud-ruby PR2198 & gem update

Unblocked! Thanks to @TheRoyalTnetennba & @dazuma a new version of the library was published with this functionality (google-cloud-vision 0.30.0)
@alixhami you can update the gem and now safely make the change from Google::Cloud::Vision::V1.new to Google::Cloud::Vision.new


detected by Zillya

so i tried to test my file on hybrid analysis, the most of the grabber works, other than detections, in which this case is detected by zillya.
heres the link to the analysis: https://www.hybrid-analysis.com/sample/3bd142722a9bd89b0992ea15f95fba2ae4b5b92811e7de11d7f5afaca4fda8c6

Some antivirus can detect pyinstaller. In the readme file, I have written, its undetected by Yara (used by virustotal).

It can still be detected by modifying yara easily but virustotal use that one and only yara rule for pyinstaller

Solved, I guess?


General Questions: Slack Plugin

Hi Team,
I was wondering if we get more details on what the following fields are used for in the slack plugin? I understand the fields
SLACK_PROFILE_DEPARTMENT_FIELD_ID=""
SLACK_PROFILE_TEAM_FIELD_ID=""
SLACK_PROFILE_WEBLINK_FIELD_ID=""

Also, may I know what goes into the value this config variable?
SLACK_USER_ID_OVERRIDE=""

i.e. Would this be an email ID or just username?
Finally, what's the significance of this variable?
SLACK_TIMELINE_EVENT_REACTION=stopwatch

What are the other options for the values here?

I believe all of those options are documented here: https://hawkins.gitbook.io/dispatch/configuration/plugins/configuring-slack#dispatch-configuration

SLACK_TIMELINE_EVENT_REACTION=stopwatch

This variable denotes which emoji you want dispatch to key off of in order to add slack messages the incident timeline. The value can be any slack emoji.
Let us know if you need any more details than what the documentation provides.

I believe all of those options are documented here: https://hawkins.gitbook.io/dispatch/configuration/plugins/configuring-slack#dispatch-configuration

SLACK_TIMELINE_EVENT_REACTION=stopwatch

This variable denotes which emoji you want dispatch to key off of in order to add slack messages the incident timeline. The value can be any slack emoji.
Let us know if you need any more details than what the documentation provides.

@kevgliss: I gathered all the other slack config data from the gitbook link provided above. I'm still not clear on the purpose of some of the fields:
For example, In the below config variable what are the allowed values for department field ID values? What is this variable? Is it specific to slack or something else.
SLACK_PROFILE_DEPARTMENT_FIELD_ID [Optional. Default: ""]
Specifies the profile field ID where Department is mapped.

Same thing for the below variable. This description in the docs still lacks context of what are the allowed values and where the variable will be used.
SLACK_PROFILE_TEAM_FIELD_ID [Optional. Default: ""]
Specifies the profile field ID where Team is mapped.
Same thing for the below variable.
SLACK_PROFILE_WEBLINK_FIELD_ID [Optional. Default: ""]
Specifies the profile field ID where the weblink is mapped.
I believe a little more context on the use of these above config variables might help immensely.
I found the following code use the config variables, but I was still unable to establish what the purpose of this SlackContactPlugin is:
@apply(counter, exclude=["__init__"])
@apply(timer, exclude=["__init__"])
class SlackContactPlugin(ContactPlugin):
    title = "Slack Plugin - Contact Information Resolver"
    slug = "slack-contact"
    description = "Uses Slack to resolve contact information details."
    version = slack_plugin.__version__

    author = "Netflix"
    author_url = "https://github.com/netflix/dispatch.git"

    def __init__(self):
        self.client = slack.WebClient(token=str(SLACK_API_BOT_TOKEN))

    def get(self, email: str, **kwargs):
        """Fetch user info by email."""
        team = department = weblink = "Unknown"

        profile = get_user_profile_by_email(self.client, email)
        profile_fields = profile.get("fields")
        if profile_fields:
            team = profile_fields.get(SLACK_PROFILE_TEAM_FIELD_ID, {}).get("value", "Unknown")
            department = profile_fields.get(SLACK_PROFILE_DEPARTMENT_FIELD_ID, {}).get(
                "value", "Unknown"
            )
            weblink = profile_fields.get(SLACK_PROFILE_WEBLINK_FIELD_ID, {}).get("value", "Unknown")

        return {
            "fullname": profile["real_name"],
            "email": profile["email"],
            "title": profile["title"],
            "team": team,
            "department": department,
            "location": profile["tz"],
            "weblink": weblink,
            "thumbnail": profile["image_512"],
        }

@kevgliss: I gathered all the other slack config data from the gitbook link provided above. I'm still not clear on the purpose of some of the fields:
For example, In the below config variable what are the allowed values for department field ID values? What is this variable? Is it specific to slack or something else.
SLACK_PROFILE_DEPARTMENT_FIELD_ID [Optional. Default: ""]
Specifies the profile field ID where Department is mapped.

Same thing for the below variable. This description in the docs still lacks context of what are the allowed values and where the variable will be used.
SLACK_PROFILE_TEAM_FIELD_ID [Optional. Default: ""]
Specifies the profile field ID where Team is mapped.
Same thing for the below variable.
SLACK_PROFILE_WEBLINK_FIELD_ID [Optional. Default: ""]
Specifies the profile field ID where the weblink is mapped.
I believe a little more context on the use of these above config variables might help immensely.
I found the following code use the config variables, but I was still unable to establish what the purpose of this SlackContactPlugin is:
@apply(counter, exclude=["__init__"])
@apply(timer, exclude=["__init__"])
class SlackContactPlugin(ContactPlugin):
    title = "Slack Plugin - Contact Information Resolver"
    slug = "slack-contact"
    description = "Uses Slack to resolve contact information details."
    version = slack_plugin.__version__

    author = "Netflix"
    author_url = "https://github.com/netflix/dispatch.git"

    def __init__(self):
        self.client = slack.WebClient(token=str(SLACK_API_BOT_TOKEN))

    def get(self, email: str, **kwargs):
        """Fetch user info by email."""
        team = department = weblink = "Unknown"

        profile = get_user_profile_by_email(self.client, email)
        profile_fields = profile.get("fields")
        if profile_fields:
            team = profile_fields.get(SLACK_PROFILE_TEAM_FIELD_ID, {}).get("value", "Unknown")
            department = profile_fields.get(SLACK_PROFILE_DEPARTMENT_FIELD_ID, {}).get(
                "value", "Unknown"
            )
            weblink = profile_fields.get(SLACK_PROFILE_WEBLINK_FIELD_ID, {}).get("value", "Unknown")

        return {
            "fullname": profile["real_name"],
            "email": profile["email"],
            "title": profile["title"],
            "team": team,
            "department": department,
            "location": profile["tz"],
            "weblink": weblink,
            "thumbnail": profile["image_512"],
        }

Slack allows additional profile data to be set. These fields tell Dispatch where to find that data (as each organization may name them differently). Typically, these are set by some other internal source (e.g. workday) and sync'd to slack by some other means. The values are just strings. If the organization decides not to set that profile information in Slack tit should be a no-op and Dispatch will simply set "unknown" for those values.
If that helps, I can add that context to the docs.

Slack allows additional profile data to be set. These fields tell Dispatch where to find that data (as each organization may name them differently). Typically, these are set by some other internal source (e.g. workday) and sync'd to slack by some other means. The values are just strings. If the organization decides not to set that profile information in Slack tit should be a no-op and Dispatch will simply set "unknown" for those values.
If that helps, I can add that context to the docs.

Gotcha. I think adding to the docs might help.

Gotcha. I think adding to the docs might help.


Preparing for pre-release of v0.16.34

Because people asked for it for testing :)

Ok, usual list:

Change numbers.
Change descriptions.
.jar
Test
Release with list.

And done!


have the option to replace GoogleMaps with OpenstreetMap or any other map service

Hi, first great demo and showcase of redhat tech!
Would it be possible to replace the google maps with a free opensource solution such as openstreetmap  ?

Thank you!
We will definitely have a look into this, as it was also our first intention to use OSM.
Main reason for the demo to use GMaps was StreetView. Not sure if this works with OSM.
OpenStreetView moved to KartaView.... so we will check this.

agree that Kartaview will not be there as Google ... I am thinking about dishing out the streetmapview and perhaps use OSM for BobbyPlane I know OSM is used for planetracking elsewhere


Updated MIA srl logo

Signed-off-by: ugho16<EMAIL_ADDRESS>Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you including a repo_url? You need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project open source?
[x] Is this project hosted by the Linux Foundation?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

LGTM

Hi @ugho16 - thanks for the updated logo. I've updated this in our internal systems - in the future you can maintain this at https://myorg.lfx.dev. Should be fixed overnight.


absolute_url corrupts url escaping it

I have an URL like this: https://www.example.com/path?query_param=N%2CU. absolute_url method:
https://github.com/vifreefly/kimuraframework/blob/512217924cab76498ee20594be3e15d26cabe426/lib/kimurai/base_helper.rb#L5-L8
escapes it so it becomes https://www.example.com/path?query_param=N%252CU, corrupting the URL and breaking the spider link following. What about adding an argument to absolute_url in order to skip escaping?

I also come across this issue in various scraping frameworks, and firewall NAT! It means you then have to spend time investigating why your URL is getting mangled.
You probably already know this, but you can consider decoding that %2C to , before passing it to absolute_url.
I would love if there was an easier way to get the developer on the right track more quickly. And I agree that there should be a way to signal that you want to send the URL as it is.

As workaround at the moment I included the following method in the spider:
  def absolute_url(url, base:, skip_url_escape: false)
    super(url, base: base) unless skip_url_escape

    return unless url

    URI.join(base, url).to_s
  end

After reviewing the class a bit more thoroughly: https://github.com/vifreefly/kimuraframework/blob/master/lib/kimurai/base_helper.rb
I've come to the conclusion that the correct course of action would be to not include escaping in the absolute_url method. There are already other helpers for escaping (escape_url), and combining escape + absolute url (normalize_url).
@vifreefly Do you agree with this? Are there other parts of the code or expectations that absolute_url is also handling escaping? This could potentially be a breaking change.

I agree with you about that

@ttilberg I think you're probably right, absolute_url should making the url is absolute, not escaping it.
I think it the next version of the framework we can make absolute_url not escaping url by default, and provide an option to escape it as well, like: absolute_url(some_url, base: url, escape: true).
As a workaround for now it's easy to redefine absolute url method in ApplicationHelper the way as you need, (like @mdesantis already did).


Add widthSelectedItems prop

This PR is to Fixes #81
Add a new property named widthSelectedItems, it's a string and can set a percent o value to width selected items, if this prop don't set, the default value is item[displayKey].length * 8 + 60

Any update on merging this?

I can approve it once the conflicts are solved. @cs-manughian let me know if you wanna do it.

any updates on this??


Add more test mesh files for Abaqus

Extended set of Abaqus mesh files is needed. They should be as small as possible, ideally only a few elements. This set should include 2D and 3D meshes, different element topology and order,  combinations of different element types, etc.

It turns out that Abaqus online documentation includes loads of (simple) .inp files.  They are found under sections: "Example problems manual", "Benchmarks manual" and "Verification manual".


Fix import paths after project rename.

Huh... interesting that I did not spot this issue when renaming. Apparently sometimes you need to cold start the project. I'll avoid such blunders in the future. Thanks for the fix.

This commit was overwrited but the feature preserved.


Upgrade nom to 7.0.0

Upgraded due to problem with crate compatibility in dependency graphs: https://github.com/Geal/nom/issues/1330

@mbrubeck ping :) in case you didn't see this.
nom 6.0 causes some problems in the ecosystem. See original link.

Thanks!

Published: https://crates.io/crates/tree_magic_mini/3.0.1


feat(esl-scrollbar): add RTL support

update for rtl esl-utils: add normalizeScrollLeft method
refactor horziontal esl-scrollbar to support rtl

:tada: This PR is included in version 3.0.0-beta.14 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve piping heat loss in molten salt power tower model

The Molten Salt Power Tower Model now uses a piping heat loss coefficient in units of W/m2-K to calculate piping thermal losses which replaces units of W/m in the previous versions. This new convention allows the performance model to calculate new piping losses as the receiver thermal power and design temperatures vary. We found that the old W/m default value, 10200, in the default case corresponded to a W/m2-K value of around 13, which we think is likely too high for insulated piping, so we set the new W/m default value to 2. This change results in 0.76% increase in annual energy for the default case. The change will be smaller as temperatures increase relative to the default case, and the change will be larger as the receiver thermal power decreases relative to the default case.
In the Version Upgrade, we do not attempt to convert the W/m value from the old file to the new W/m2-K input, and instead set it to the default value and instruct the user to review the input on the Tower and Receiver page and modify it as necessary for their design.

Looks good and works well. I revised the message a little:
We have changed the units for the piping heat loss coefficient for the Molten Salt Power Tower Model from W/m to W/m2-K.This new convention allows the performance model to calculate piping thermal losses as the receiver thermal power and design temperatures vary. We found that the old default value of 10,200 W/m corresponds to about 13 W/m2-K, which we think is too high for insulated piping, so we chose a new default value of 2 W/m2-K. We are changing the piping heat loss coefficient value in your file of 10200 W/m to the default value of 2 W/m2-K. Please review this input on the Tower and Receiver page and modify it as necessary for your design.

In the SSC repository, I created a new tcsmolten_salt cmod input for the W/m2-K input, and I attempted to deprecate the old W/m input. Both inputs are optional in the var table, and then checked at the beginning of exec(). If the old is provided but not the new, then the code throws an exception explaining the change. If both are provided, then the code prints a notice that the old input is not used. You can see this behavior by using Shift+F5 to generate a script of the default MSPT case in SAM 2020 and then running that script in the SDKtool.
https://github.com/NREL/ssc/commit/41fdaf10c22fe049cd1a7dc7e40d13544b47b332

FYI, this issue will affect your old MSPT scripts, old MSPT SAM files, and MSPT in pysam. Let me know if you have any thoughts on the piping loss change or recommendations for deprecating cmod variables.
@jannamartinek @Matthew-Boyd @qualand @mjwagner2 @dguittet


test/k8s: update host policies for firewall tests.

Allow ICMP/ICMPv6 traffic on all nodes.
Fixes: #25344 #25343 #25342

/test

/test


Display as obsidian table

Could you update the plugin so that it displays as a table instead of the custom setup you have.
I'm using the Faded theme which can be found in community themes. This theme has various custom things including custom bullet point styles. However, whatever html you're using to create the current list has three issues.

It just takes up a ton of space. This can be seen on the default obsidian theme as well as the Faded theme.
The information is just harder to read when it's not placed in a table
Something with your html causes the custom bullet points in the Faded theme to show in the rendered element

This image shows what it looks like when using the Faded theme (Many themes using custom bullets will have the same issues - ITS theme for example)

This second image shows what it could look like if you used a table instead

The Deck and Sideboard titles could use the same styling as what you have now but my example image above is just using regular text so it looks smaller than what I would Ideally want.
the code for those tables is pretty simple here is the markdown for the sideboard
| Count | Name                | Price |
| ----- | ------------------- | ----- |
| 2     | Disdainful Stroke   | $0.16 |
| 4     | Negate              | $0.44 |
| 4     | Out of the Way      | $0.12 |
| 1     | Reckoner Bankbuster | $3.02 |
| 4     | Ertai's Scorn       | $0.28 |
|       |                     |       |
| 15    | cards               | $4.02 |

and here is the raw html for it

Count
Name
Price

2
Disdainful Stroke
$0.16

4
Negate
$0.44

4
Out of the Way
$0.12

1
Reckoner Bankbuster
$3.02

4
Ertai's Scorn
$0.28

15
cards
$4.02

Even excluding my particular gripes with how it renders via the Faded theme, I really think this change should be made because it would look better even when using the default theme. Plus any theme that adds basic table styling will automatically work with your plugin without css devs taking any extra time to support it.

I might make a fork with this change if I have time.

@Serif-7 this is awesome!  Definitely something I can roll in on the next version.


react-native 0.70.* - iOS build failed : use of undeclared identifier 'RNNotifications'

Hi,
I've make all instructions from Getting Started and when I launch $npx react-native run-ios I get this error :
error: use of undeclared identifier 'RNNotifications'
  [RNNotifications didReceiveBackgroundNotification:userInfo withCompletionHandler:completionHandler];

NTE : cd ios/ && pod install is ok
This error occur in blank react-native project with this configuration :
System:
    OS: macOS 12.6
    CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
    Memory: 16.49 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 18.6.0 - ~/.nvm/versions/node/v18.6.0/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.13.2 - ~/.nvm/versions/node/v18.6.0/bin/npm
    Watchman: Not Found
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 16.0, macOS 12.3, tvOS 16.0, watchOS 9.0
    Android SDK: Not Found
  IDEs:
    Android Studio: 2021.3 AI-213.7172.25.2113.9014738
    Xcode: 14.0.1/14A400 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.16 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.1.0 => 18.1.0
    react-native: 0.70.2 => 0.70.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Sorry, it was a wrong import order in my AppDelegate.mm.

😅 same issue
same solution.
Move import outside of
#import "RNNotifications.h"
#if RCT_NEW_ARCH_ENABLED`
…
#endif


Github login not working with SSL enabled

When using SSL, the following error occurs when trying to login using Github:
Request Method: | GET
-- | --
http://<IP_ADDRESS>/oauth/complete/github/?error=redirect_uri_mismatch&error_description=The+redirect_uri+MUST+match+the+registered+callback+URL+for+this+application.&error_uri=https%3A%2F%2Fdocs.github.com%2Fapps%2Fmanaging-oauth-apps%2Ftroubleshooting-authorization-request-errors%2F%23redirect-uri-mismatch&state=nnRwRg6fsZBXQkLSjTmk5maz6yePnA0V
1.11.29
AuthFailed
Authentication failed: The redirect_uri MUST match the registered callback URL for this application.
/usr/local/lib/python2.7/site-packages/social_core/backends/oauth.py in process_error, line 380
/usr/local/bin/python

...

The Github Oauth configuration will need to be updated to use https:// and the domain tools.spdx.org once we go live.  I'll make that change just before going into production with the new code.

Resolved in PR #258 - however there is now a new issue #260


Handle server setup container

Build and run a container which creates the Handle Server sitebndl.zip file.

[x] Install Handle Server
[x] Create public and private keys
[x] Create siteinfo file
[x] Convert siteinfo to bin
[x] Zip up sitebndl.zip file

[ ] item 1

Questions:

[ ] what is meaning of site/version serial number

this is done. additional tweaks will result in new issues.


Add county field to Address page layout

Critical Changes
Changes
Issues Closed

@scbushell this looks like it touched a ton of files. Just verifying that you only added the field to the page layout for Address correct? If so, we should probably remove this PR, create a different branch and only commit the page layout change.

Oh shoot.  I was trying out Metecho.  Why were so many files touched?  I
did three things:

Added county to Address_cc
Checked the box so all profiles could view it
Verified that it was added (automatically) to the page layout

Sorry if I messed anything up!  I thought I was in a safe space away from
where the real work happens...
On Tue, May 5, 2020 at 4:02 PM lifewithryan<EMAIL_ADDRESS>wrote:

@scbushell https://github.com/scbushell this looks like it touched a
ton of files. Just verifying that you only added the field to the page
layout for Address correct? If so, we should probably remove this PR,
create a different branch and only commit the page layout change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SalesforceFoundation/EDA/pull/1177#issuecomment-624304532,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANEM7ENAJP2HARL5NW6SJ2TRQB5HZANCNFSM4MZ4JMPA
.

--
SARA BUSHELL
Sr. UX Designer | Salesforce
Mobile:<PHONE_NUMBER>
https://smart.salesforce.com/sig/sbushell//us_mb/default/link.html

Oh shoot. I was trying out Metecho. Why were so many files touched? I did three things: 1. Added county to Address_cc 2. Checked the box so all profiles could view it 3. Verified that it was added (automatically) to the page layout Sorry if I messed anything up! I thought I was in a safe space away from where the real work happens...
…
On Tue, May 5, 2020 at 4:02 PM lifewithryan @.***> wrote: @scbushell https://github.com/scbushell this looks like it touched a ton of files. Just verifying that you only added the field to the page layout for Address correct? If so, we should probably remove this PR, create a different branch and only commit the page layout change. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1177 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANEM7ENAJP2HARL5NW6SJ2TRQB5HZANCNFSM4MZ4JMPA .
-- SARA BUSHELL Sr. UX Designer | Salesforce Mobile:<PHONE_NUMBER> https://smart.salesforce.com/sig/sbushell//us_mb/default/link.html

It looks like metéchō automatically spins up a PR, which is interesting. I ended up having to pull down the branch and open it the way we usually do as I wasn't able to spin up a scratch org in metéchō to 'test'.

It also appears that CCI did some automagically reordering and removing of some line breaks over the files that we have. I don't necessarily think that's a bad thing, given that it seems to be alphabetical order and such. It might be worthwhile to run our codebase through it for that reordering every once in awhile.


[PP-1956] Ensure the context transaction manager is being used for cr…

…eating identifiers in SweepMonitor runs.
Description
This PR attempts to fix a problem we are seeing in production where the database session associated with the identifier object (obtained via the Session.object_session() method )  is sometimes not the same as the one associated with the context manager when the sweep monitor is running.   I'm not sure that this will fix it, but hopefully is will get us closer to a fix for the sweep monitors.
Motivation and Context
https://ebce-lyrasis.atlassian.net/browse/PP-1956

How Has This Been Tested?
Unit tests still pass.

Checklist

[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

@dbernstein why do think that we are getting the wrong session here? I'm trying to understand how that could happen.

This one has been going for a LONG time at this point, and it seems like we are adding fixes on top of other fixes to try to get this sorted out, but it hasn't been working, I'm not sure I'm comfortable adding another fix here without understanding why this is happening.
This issue is related to at least these two other PRs: https://github.com/ThePalaceProject/circulation/pull/1948 (PP-1491) & https://github.com/ThePalaceProject/circulation/pull/2091 (PP-1755). Perhaps we need to go back to the original problem, and see if there is a better approach that we can take there so we get this actually sorted.

@jonathangreen :  My thinking here is that at least we're ensuring  that the session that is being used by the identifier class to add a link  is the same same that is being used by the context manager.  Unless I have fundamentally misunderstood something here.


[KAFKA-12644] Add Missing Class-Level Javadoc to Exception Classes

…ache.kafka.common.errors.ApiException
Added missing class-level javadocs to Exception classes
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@showuon I appreciate all your comments, I will work on this again and resubmit with your requested changes. It's always great to have another perspective.

@izzyacademy Thanks for the PR!
Do you plan to update the PR and address the feedback you received? If not, no worries I'll just close this PR.


Multiple custom designs not loaded

Describe the bug
I have a Angular project and want to display two different types of custom sockets but it always uses the design of the first requested socket.
editor.ts:
render.addPreset(
       Presets.classic.setup({
         customize: {
           socket(data) {
             if (Math.random() > 0.5) {
               console.log("bool")
               return BoolSocketComponent;
             } else {
               console.log("real")
               return RealSocketComponent;
             }
           }
         }
       })
     );

50/50 chance between the design, for testing later will be dependent on socket type
components:
socket 1:
css:
:host{
    display: inline-block;
    cursor: pointer;
    border: 3px solid white;
    border-radius: 1em;
    width: 20px;
    height: 20px;
    vertical-align: middle;
    background: #ffffff66;
    margin: 0.1em 0.7em;
    z-index: 2;
    box-sizing: border-box;
}

:host:hover{
    border-width: 4px;
}

ts:
import {
    Component,
    Input,
    HostBinding,
    ChangeDetectorRef,
    OnChanges
  } from "@angular/core";

  @Component({
    template: ,
    styleUrls: ["./bool-socket.component.css"]
  })
  export class BoolSocketComponent implements OnChanges {
    @Input() data!: any;
    @Input() rendered!: any;

    @HostBinding("title") get title() {
      return this.data.name;
    }

    constructor(private cdr: ChangeDetectorRef) {
      this.cdr.detach();
    }

    ngOnChanges(): void {
      this.cdr.detectChanges();
      requestAnimationFrame(() => this.rendered());
    }
  }

The second socket-component is implemented in the same way just a different name and style.
I logged which type was returned (see top)
And what i found was that all sockets always have the same design as the first requested one...

I cant find what the issue is...
Any ideas?
Looks like a bug to me
I have the same issue with the connections!
Example to reproduce
No response
How to reproduce

Use Angular
create two components for different styled sockets
let the render function switch between them

Expected behavior
I expect each socket to have the color of the returned custom socket
Dependencies
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>Also tried with angular 15, also did not work
Platform
Chrome and Angular 17/ 15
Relevant log output
No response
Code of Conduct

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

Problem was that i had the same selector for all three elements, which caused the issue


Add is_certificate and export_cdi flags to DeriveContext

As well as other boiler plate work to make sure the invariants laid out in the spec are enforced.

Will follow up with clean up / refactors once most of the feature work is out of the way.


Improve numerical stability of _torch_impl.project_cov3d_ewa

Fixes the a minimal set of numerical issues in _torch_impl so that it can be used in actual training and not just unit tests. This makes testing and optimizing various things a lot easier (example here). Similar to one of the goals in https://github.com/nerfstudio-project/gsplat/pull/172. However, #172 seems to be still a work-in-progress, so this PR still makes sense too and should not conflict with #172.
The numerical issues were mostly related to:

Computing things for Gaussians that are behind the current camera
Trying to differentiate $\sqrt x$ too close to $x = 0$

Looks good to me. Have you tested the test_project_gaussians.py ?

Yes. Still passing. (Just double checked after realizing the passing CI build does not apparently test CUDA things)


Transform misalignment on mesh

~There seems to be some joint angles offest in the urdf-viz implementation (or maybe in the upstream urdf-rs).~
~The offset seems to be by a value of $$\pi$$ (180 degrees)~
With some preliminary testing, it seems it's the world transform that has some misalignment.
The example panda arm urdf can be downloaded here: panda.tar.gz
Visualisation from pybullet
import time
import pybullet as p

physicsClient = p.connect(p.GUI)
planeId = p.loadURDF("panda/urdf/panda_relative.urdf")
time.sleep(10000)

urdf-viz panda/urdf/panda_relative.urdf

The bug is due to the upstream library mesh-loader.
Addressing in https://github.com/openrr/mesh-loader/pull/55


how do i turn off notesplash hue/RGB?

Describe your problem here.
i want to turn off the notesplash hue/RGB
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

figured it out i found the disable RGB thing in the chart data


Date return different values

Hi,
I got an issue after I converted the date back to the original format using datetime.timestamp().
The dates are different from before. I tried your data, and it still got the same issue. Could you please help check?
For example, my data is a time series with 15 min resolution. But the result gives me one in 19 min. Thanks a lot.
Date | Value
2016-10-09 00:00:00 | 10
2016-10-09 00:19:32 | 9
Yue

datetime.strptime("2016-10-09 00:00:00", "%Y-%m-%d %H:%M:%S").timestamp()
1475953200.0
datetime.strptime("2016-10-09 00:19:32", "%Y-%m-%d %H:%M:%S").timestamp()
1475954372.0


Helper class methods are added into  Azure Service activity class if the user cancels  the Add Options

 Environment  : Intellij14
 Repo

Create Azure Service Activity by selecting  the option Azure Mobile service and Notification Hub.
Click on Finish
It loads Add Mobile Service Wizard
Select  any  of the available Mobile Service
Click on Next and Set the Valid Values on SenderID,Connection String,Hub name.
Click on  Cancel

 Actual
Windows Azure Mobile Service Client libraries are not added into the project .But it adds the Client instantiate code on Azure Service  Activity Class
MobileServiceClient mobileServiceClient = MobileService.getInstance(this);
 Expected  If the User  clicks on Cancel Button on Add Microsoft Service Wizard,it should not add helper class code on AzureService Activity Class

This issue is still not fixed , it is still adding client instantiation code
MobileServiceClient mobileServiceClient = getMobileServiceClient(this);
to the azure service activity class .

Not able to test as plug in throwing exception while configuring the Notification Hub.
Will test it once issue #341.

This issue is reproducible in the build 0.1.235.and 0.1.36 build.

obsolete


DEV: using pydata theme, revert sphinx>6 change

[CHANGED] sphinx_panels conflicts with sphinx> 5

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
10 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.001%) to 91.623%

Files with Coverage Reduction
New Missed Lines
%

.nox/test-3-10/lib/python3.10/site-packages/cogent3/evolve/likelihood_function.py
1
96.3%

.nox/test-3-11/Lib/site-packages/cogent3/evolve/likelihood_function.py
1
96.3%

.nox/test-3-9/lib/python3.9/site-packages/cogent3/maths/scipy_optimize.py
2
83.88%

.nox/test-3-11/Lib/site-packages/cogent3/maths/scipy_optimize.py
3
84.92%

.nox/test-3-8/lib/python3.8/site-packages/cogent3/maths/scipy_optimize.py
3
83.88%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
235907

Relevant Lines:
257476

💛  - Coveralls


dl: embed directive not working with gotip

What version of Go are you using (go version)?

$ gotip version
go version devel +c800222 Sun Oct 25 20:56:22 2020 +0000 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ gotip env
GO111MODULE="on"
GOARCH="amd64"
GOBIN="/Users/adhoc/bin"
GOCACHE="/Users/adhoc/Library/Caches/go-build"
GOENV="/Users/adhoc/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/adhoc/src/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/adhoc/src/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/adhoc/sdk/gotip"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/adhoc/sdk/gotip/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/var/folders/p7/jc4qc9n94r3f6ylg0ssh1rq00000gs/T/tmp.eGCNfh4R/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/p7/jc4qc9n94r3f6ylg0ssh1rq00000gs/T/go-build525352014=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Run this:
example.txt: "abc123"
go.mod:
module example

go 1.16

example.go:
package main

import (
	"fmt"
	_ "embed"
)

func main() {
	//go:embed example.txt
	var x string
	fmt.Println(x)
}

What did you expect to see?
abc123
What did you see instead?
$ gotip run .
# example
./example.go:9:4: invalid go:embed: build system did not supply embed configuration

Is this just a temporary error as development proceeds, or does gotip need to be modified to work with embeds?

What makes you think that the work on embed is done?
This CL hasn't been merged yet, so I don't think you can expect embed to work on tip right now.

Okay. Could just be too early then. I saw that the embed package is there and wanted to test it out. Will close until more CLs are closed.


Sharing event loop group on client side

On jetty 4.1.11 I try to share same eventLoopGroup between some clients using following code
`public class SimpleClientFactory implements AutoCloseable {
private final EventLoopGroup workerGroup;
private final HandlerFactory clientHandlerFactory;

public SimpleClientFactory(HandlerFactory clientHandlerFactory) {
    this(clientHandlerFactory, 1);
}

public SimpleClientFactory(HandlerFactory clientHandlerFactory, int workers) {
    this.workerGroup = new NioEventLoopGroup(workers);
    this.clientHandlerFactory = clientHandlerFactory;
}

@Override
public void close() throws Exception {
    MLU.safeAutocloseble(workerGroup).close();
}

public void session(String addr, int port) {
    session(new InetSocketAddress(addr, port));
}

public void session(SocketAddress connectTo) {
    ChannelPipeline pipeline;

    try {
        ChannelFuture f;

            Bootstrap bootstrap = new Bootstrap();
            bootstrap.group(workerGroup);
            bootstrap.channel(NioSocketChannel.class);
            bootstrap.option(ChannelOption.SO_KEEPALIVE, true);
            bootstrap.handler(new ChannelInitializer<SocketChannel>() {
                @Override
                public void initChannel(SocketChannel ch) throws Exception {
                    ch.pipeline().addLast(new BinChannelDecoder(32 * 1024)); // some my handlers
                    ch.pipeline().addLast(new BinChannelStateHandler()); // some my handlers
                    ch.pipeline().addLast(clientHandlerFactory.createNewHandler()); // some my handlers
                }
            });

            f = bootstrap.connect(connectTo).sync();

        pipeline = f.channel().pipeline();
        f.channel().closeFuture().sync();
    } catch (Exception e) {
        throw new IllegalStateException("Connection error", e);
    }

    if(pipeline!=null){
        AbstractHandler handler = pipeline.get(AbstractHandler.class); //my handler
        if(handler!=null) {
            handler.validateError();
        }
    }

}

}
If I create one connectiontion everething is good. But if run 3 client (call session method) in 3 different threads I see traffic from server mixed between clients.
`

I'm sorry. I found bug in my server logic implementation. So code in my message is valid example of sharing EventLoopGroup between clients.


Fix to get method of SPInstallPrereqs

Fixes #329 - quick fix to remove a parameter that was hanging around from the changeset described in the issue there

[X] Change details added to Unreleased section of readme.md?
[n/a] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[n/a] Examples updated for both the single server and small farm templates in the examples folder?
[X] New/changed code adheres to [Style Guidelines]?(https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md)?
[n/a] Unit and Integration tests created/updated where possible?

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


HubConnectionState in TS client should have public getter

Currently users have to manually keep track of connection state as mentioned in this bug report https://github.com/aspnet/AspNetCore/issues/7465 As shown below, the method to get the state of the HubConnection in the TS client is private.  https://github.com/aspnet/AspNetCore/blob/abd2a61bdbebfcbc55021bd4212727f0fa95ce52/src/SignalR/clients/ts/signalr/src/HubConnection.ts#L94-L97
This though is a public  API in the C# client https://github.com/aspnet/AspNetCore/blob/abd2a61bdbebfcbc55021bd4212727f0fa95ce52/src/SignalR/clients/csharp/Client.Core/src/HubConnection.cs#L92-L108 and Java client https://github.com/aspnet/AspNetCore/blob/abd2a61bdbebfcbc55021bd4212727f0fa95ce52/src/SignalR/clients/java/signalr/src/main/java/com/microsoft/signalr/HubConnection.java#L262-L269

I think the getter is public by default.
https://github.com/Microsoft/TypeScript/blob/master/doc/spec.md#16-classes

Public is the default accessibility for class members, but a programmer can also specify private or protected accessibility for a class member.

closing as this is the default cc @mikaelm12


[qt5-webengine] Build error

Host Environment

OS: [e.g. Windows]
Compiler: vc2022

To Reproduce
./vcpkg install qt5-webengine:x64-windows
vcpkg-tool version: 2022-05-05-67e17c1782801cf481be9ac0b3765dff3e4bdeb8
vcpkg-scripts version: f78f4440d 2022-05-31 (9 hours ago
package-build-x64-windows-dbg-out.log
package-build-x64-windows-dbg-err.log
)

Dup to https://github.com/microsoft/vcpkg/issues/25013


should I use angular http client interceptor or apollo link to add the authorization header?

As Apollo angular can use HttpClientModule under the hood, can I use http client interceptor to implement all the token issues (e.g. refresh token, authorization header)?

I depends. With ApolloLink it will use that header only on requests to the graphql endpoint but HttpInterceptor will attach it to all requests if won't catch the one you need. There is no one good practice, it depends what you want to achieve.


fix: add CNAME to siteConfig.js

Motivation
Since we have use `sumochallenge.org', we should add CNAME.

Hi @dhuber23b
Can you try to merge this out and then publish ?
Don't add the blog.html fix yet. I think this will work. Sorry for the trouble


Consumer logging tweaks

Add some context to the log message Consumer generates when the processor function returns. It logged the offset, but not the partition, which isn't meaningful.

Ooops, looks like I pushed to master by mistake. I backed that out and enabled branch protection. This PR has been replaced by #105.


Where's IComponent

This content fails to differentiate between IComponent and ComponentBase.  Some of the content describes ComponentBase functionality, not component functionality.
The UI render process will manage and "render" any object that implements IComponent.  There's no requirement to inherit ComponentBase.  ComponentBase is an implementation of IComponent provided with Blazor that provides functionality most programmers need in a component.  They can either use ComponentBase or write their own implementations.
By default Razor files inherit from ComponentBase.

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

ID: 60c195c0-99f6-64f5-66a3-4e69109dc8ee
Version Independent ID: 24368808-4b90-f75a-e887-eb72a1c85f5b
Content: Build reusable UI components with Blazor
Content Source: docs/architecture/blazor-for-web-forms-developers/components.md
Product: dotnet-architecture
Technology: blazor
GitHub Login: @danroth27
Microsoft Alias: daroth

Hi @danroth27, what are your thoughts?

It's certainly true that this chapter doesn't cover IComponent. I think that's intentional given that the most common way to implement components is using .razor files.
@ShaunCurtis Are there any particular passages of text that you see as inaccurate and should be generalized? Or are you just pointing out that this is a missing concept in the eBook?

While there are passages where ComponentBase and Component are treated as synonymous, I think that this is a missing concept in the whole community.  This comment section at the top of the ComponentBase code is enlightening.  Few have probably ever read it, and even fewer taken the author up.
// Most of the developer-facing component lifecycle concepts are encapsulated in this
// base class. The core components rendering system doesn't know about them (it only knows
// about IComponent). This gives us flexibility to change the lifecycle concepts easily,
// or for developers to design their own lifecycles as different base classes.

I'm probably in a minority of one in not using ComponentBase as my core component.

yes, and thanks.


Edges broken when scaling flow renderer

Describe the Bug
When applying transform: scale() animation to the flow renderer element the edges are positioned with offset:

Related discussion: https://github.com/wbkd/react-flow/discussions/3229
Your Example Website or App
https://codesandbox.io/s/reactflow-scale-repro-gxspht?file=/src/Flow.js
Steps to Reproduce the Bug or Issue
Open the repro sandbox and notice the edges are rendered with offset due to:
  useEffect(() => {
    document
      .querySelector(".react-flow__renderer")
      .animate([{ transform: "scale(10)" }, { transform: "scale(1)" }], {
        duration: 666,
        easing: "ease"
      });
  }, []);

Expected behavior
Edges should be rendered correctly, independent of the animation.
Screenshots or Videos
No response
Platform

OS: any
Browser: any
Version: latest

Additional context
No response

I would be happy to help with the fix, but not really sure where to dig. I've initially guessed that edges are rendered in an async manner after setEdges is invoked, but after examining the sources it doesn't seem the case.

Thank you for the insight! It doesn't completely solve the issue though, as the edges are still positioned incorrectly while the animation is running. Is there a way to force the positioning before the animation starts (given the scale is 1 at that moment)? I've tried:
 useEffect(() => {
    const nodeIds = store
    .getState()
    .getNodes()
    .map((node) => node.id);
    updateNodeInternals(nodeIds);

    document
      .querySelector(".react-flow__renderer")
      .animate([{ transform: "scale(10)" }, { transform: "scale(1)" }], {
        duration: 1000,
        easing: "ease"
      });
  }, [updateNodeInternals, store]);

— but it didn't work.

What are you trying to achieve here? I think it would be better to work with the viewport instead of scaling the whole renderer. Is this about a zoom effect?

I need to animate the graph so that "camera" dives in to a node when it's expanded (transitioning to the graph stored inside the node) and dives out of the same node when returning back. I'm using 2 graph instances to achieve that. Here is a demo:
https://github.com/wbkd/react-flow/assets/2056864/c8a523dc-4c7a-4dc6-aaa8-f4c765810183
Basically, I just need a way to force react-flow calculate the positions / initialize at the moment when no transformations are applied and everything is valid, then run the animations.

That's a super nice effect 👌 Did you try to animate the viewport instead of using transform scale?

That's what I've initially tried with ViewportHelperFunctions.zoomIn/Out, but for this effect to work correctly I actually need two animations running with unique timing functions and durations: one for translate and another for zoom/scale. Since css won't allow that on a single element, I have to apply them on 2 parent elements of the viewport (pane and renderer).

did you try to use setCenter for this https://codesandbox.io/s/gifted-glade-wkc993?file=/App.js:1069-1182 (button is on the top right) ?

Yep, I've played with all the available funcs, but it didn't work just right for me. Eg, setCenter starts zooming with a delay, while I need translation and scale happen in parallel, but with different easings and durations.

Just to clarify, I'm really just want react-flow to position the edges/handles in a predictable/consistent manner. Currently, it seems the evaluation happens in an undefined order. For example, if we add a delay before running the animation, the edges are placed correctly:
  useEffect(() => {
    setTimeout(
      () =>
        document
          .querySelector(".react-flow__renderer")
          .animate([{ transform: "scale(10)" }, { transform: "scale(1)" }], {
            duration: 666,
            easing: "ease"
          }),
      100
    );
  }, []);

Either changing it to work in sync or adding a hook for when edges/handles calculation is complete (so that we can then run our custom transformations) will resolve the issue here.

The useNodesInitialized hook returns true when all measurements are done. Could you try to use that one?
const nodesInitialized = useNodesInitialized();

 useEffect(() => {
    if (nodesInitialized) {
      document
        .querySelector(".react-flow__renderer")
        .animate([{ transform: "scale(10)" }, { transform: "scale(1)" }], {
          duration: 666,
          easing: "ease"
        });
    }
  }, [nodesInitialized]);

That was it! useNodesInitialized was exactly the hook I was looking for :) Had to fight with react shenanigans a bit (handling the state when graph props are changed, but nodeInitialized is in stale true state), but now it's working perfectly w/o any hacks. That you very much for the help!
https://github.com/wbkd/react-flow/assets/2056864/9f11e0ea-9e87-40c2-87d3-6f8e702e7a81

I am glad we could solve it :) Looks super dope 🔥🔥🔥


Include kube-logging namespace

For logging system.

this will be handled by helm when charts are installe.d


Expected loss

Implemented the Expected Loss training objective from http://aclweb.org/anthology/P17-1138.
This involves sampling from the decoder while keeping track of the sampled outputs, computing rewards from references, and keeping track of previous rewards in the decoder (variables, but not stored/updated).

all checks passed 🥇  👍  :-)


Simple extension filter example (for File type validation plugin)

Firstly Thank you so much for this great library.
There is no bug
Only i need to explain how to simply validate files just with extension. because i spent couple of hours for this.
I readed a hundred times the documents and other issues but it's not written anywhere how we return with promise ?
what is the promise man?  we are the copy-and-paste programmers :)
we will return via  " resolve('.'+source.name.split('.').pop());  "  complete example following
<head>
<script src="/plugins/filepond/filepond-plugin-file-validate-type.js"></script> //this plugin must be added

</head>

<script>
FilePond.registerPlugin(FilePondPluginFileValidateType); //!! this line must be here

var muploader = FilePond.create(document.querySelector('.input'), {
    acceptedFileTypes: ['.stl'],
    fileValidateTypeDetectType: (source, type) =>
        new Promise((resolve, reject) => {
           resolve('.'+source.name.split('.').pop());  //!!!WRITE HERE it's will just check your extension!!!
        }),
});
</script>

There is no bug it's just a simple example

acceptedFileTypes should be an array of mime types not extensions, so image/jpeg instead of .jpg
Now your fileValidateTypeDetectType should resolve with a mime type.
(source, type) => new Promise(resolve) => {
  // type is the mime type the browser has detected
  // source is the source file

  // if source is a jpeg image file, resolve with JPEG mime type
  resolve('image/jpeg')
}


netcdf-c-4.5.0-rc2 build on cygwin broken

I noticed that on current release candidate the NOUNDEFINED that is set in configure
for cygwin and other platform was removed from the Makefile.am.
This break the build on cygwin, and it was there on <IP_ADDRESS>-1.
Reinstating it, the build is fine and pass all tests.
--- origsrc/netcdf-c-4.5.0-rc2/liblib/Makefile.am       2017-08-07 19:25:26.000000000 +0200
+++ src/netcdf-c-4.5.0-rc2/liblib/Makefile.am   2017-09-16 22:44:50.747636500 +0200
@@ -18,7 +18,7 @@ lib_LTLIBRARIES = libnetcdf.la
 # for information regarding incrementing `-version-info`.
 ##

-libnetcdf_la_LDFLAGS = -version-info 12:0:1
+libnetcdf_la_LDFLAGS = -version-info 12:0:1 ${NOUNDEFINED}

 libnetcdf_la_CPPFLAGS = ${AM_CPPFLAGS}
 libnetcdf_la_LIBADD =

This is very old. Is this issue still relevant or should it be closed?

forgot about it. Same of https://github.com/Unidata/netcdf-c/issues/2196 that has been solved