Support Renderdoc in D3D12 backend.

it seems Renderdoc works fine in D3D11, OpenGL, Vulkan backend.
but main problem is CreateGraphicsPipelineState in d3d12 backend return E_NOINTERFACE (No such interface supported) and throw Failed to create pipeline state.
It would be better to get it also working. or any suggestion use other debugger?

renderdoc use to work in D3D12 mode - I don't know why it is not working now. Could be a renderdoc issue.

I think i found problem
Here is the hook code of renderdoc
HRESULT WrappedID3D12Device::CreateGraphicsPipelineState(const D3D12_GRAPHICS_PIPELINE_STATE_DESC *pDesc,
                                                         REFIID riid, void **ppPipelineState)
{
  D3D12_GRAPHICS_PIPELINE_STATE_DESC unwrappedDesc = *pDesc;
  unwrappedDesc.pRootSignature = Unwrap(unwrappedDesc.pRootSignature);

  if(ppPipelineState == NULL)
    return m_pDevice->CreateGraphicsPipelineState(&unwrappedDesc, riid, NULL);

 ////////////////// [  The reason is renderdoc check uuid   ] //////////////////
  if(riid != __uuidof(ID3D12PipelineState))
    return E_NOINTERFACE;

  ID3D12PipelineState *real = NULL;
  HRESULT ret;
  SERIALISE_TIME_CALL(
      ret = m_pDevice->CreateGraphicsPipelineState(&unwrappedDesc, riid, (void **)&real));

  if(SUCCEEDED(ret))
  {
    // use implicit register/space
    uint32_t reg = ~0U, space = ~0U;
    if(m_VendorEXT != GPUVendor::Unknown)
      GetShaderExtUAV(reg, space);

    ProcessCreatedGraphicsPSO(real, reg, space, pDesc, riid, ppPipelineState);
  }
  else
  {
    CheckHRESULT(ret);
  }

  return ret;
}

Here is the code in PipelineStateD3D12Impl.cpp of DiligentCore
if (!m_pd3d12PSO)
{
////////// Original code
////    uuid of m_pd3d12PSO is ID3D12DeviceChild, the base class of ID3D12PipelineState, renderdoc only check ID3D12PipelineState uuid equality
    HRESULT hr = pd3d12Device->CreateGraphicsPipelineState(&d3d12PSODesc, IID_PPV_ARGS(&m_pd3d12PSO));

////////// Fixed code
////   Just appoint uuid of ID3D12PipelineState rather than ID3D12DeviceChild
    HRESULT hr = pd3d12Device->CreateGraphicsPipelineState(&d3d12PSODesc, __uuidof(ID3D12PipelineState), IID_PPV_ARGS_Helper(&m_pd3d12PSO));

    if (FAILED(hr))
        LOG_ERROR_AND_THROW("Failed to create pipeline state");

    // Add to the cache
    if (pPSOCacheD3D12 != nullptr && !WName.empty())
        pPSOCacheD3D12->StorePipeline(WName.c_str(), m_pd3d12PSO);
}

Thanks for finding this out - will fix this soon.


cannot find -lwebsockets collect2.exe: error: ld returned 1 exit status

Hello i have a problem when i trying to generate a dll (i want to change the max size of the payload)
Thank

@Princbut , do you find the solution?

I have the same problem 🤕

@Princbut , do you find the solution?

For the compilation error no but manage the max size of payload i found a solution

I guess, we need to compile the libwebsockets. I tried but I still did not succeed. :(
https://github.com/warmcat/libwebsockets

I have the same problem 🤕

At this day i still searching for a solution

Just solve the issues by:
update GCC then apply this https://github.com/krisn/lws2mql/pull/4/commits/1b7abe9402cc670909e1ec39fb4c5bf2defed6f5

@Princbut

I guess, we need to compile the libwebsockets. I tried but I still did not succeed. :(
https://github.com/warmcat/libwebsockets

I have the same problem face_with_head_bandage

At this day i still searching for a solution

How did you put this together can you tell me I don't understand?


Graph description is missing on details page

Since our results are noisy from one run to another, if I want to quote performance results for YJIT, I tend to want to link to the details page, so that I know that when people click the link, they're going to see the performance numbers I quoted. If I link people directly to speed.yjit.org, there's some risk that I could have quoted e.g. a 33% speedup, but they see 30% instead.
I'm going to be linking to this page for the upstream PR, for example:
https://speed.yjit.org/benchmarks/bench-2021-10-18-121121
However, a description for the graphs is missing which makes the page not quite self-contained. Under each graph, it should say "Speed of each Ruby implementation on benchmarks (iterations/second) relative to the CRuby interpreter. Higher is better.", as on the front page.

Sure. The Jekyll templates for this are on the "pages" branch of the yjit-metrics repo. This one in particular is a layout: https://github.com/Shopify/yjit-metrics/blob/pages/_layouts/benchmark_details.html


Conflict between the single exception for full linkage and /articles/1/relationships/comments?include=comments.author

Compound documents require "full linkage", meaning that every included resource
MUST be identified by at least one resource identifier object in the same document.
These resource identifier objects could either be primary data or represent resource
linkage contained within primary or included resources.

The only exception to the full linkage requirement is when relationship fields that would
otherwise contain linkage data are excluded via sparse fieldsets.

My interpretation of this statement is that every entity in the included{} block must have a corresponding resource identifier, unless the included entity is from a field that was excluded via  sparse fieldsets. For example, we might call
/articles?include=articles.author,articles.comments&fields[articles]=author

which includes the author and comments for all articles, but only includes the author in the sparse fieldset, resulting in the comments being listed in the included{} block, and not having any other reference:
{
  "data": [{
    "type": "articles",
    "id": "1",
    "attributes": {
      "title": "JSON API paints my bikeshed!"
    },
    "links": {
      "self": "http://example.com/articles/1"
    },
    "relationships": {
      "author": {
        "links": {
          "self": "http://example.com/articles/1/relationships/author",
          "related": "http://example.com/articles/1/author"
        },
        "data": { "type": "people", "id": "9" }
      }
    }
  }],
  "included": [{
    "type": "people",
    "id": "9",
    "attributes": {
      "first-name": "Dan",
      "last-name": "Gebhardt",
      "twitter": "dgeb"
    },
    "links": {
      "self": "http://example.com/people/9"
    }
  }, {
    "type": "comments",
    "id": "5",
    "attributes": {
      "body": "First!"
    },
    "relationships": {
      "author": {
        "data": { "type": "people", "id": "2" }
      }
    },
    "links": {
      "self": "http://example.com/comments/5"
    }
  }, {
    "type": "comments",
    "id": "12",
    "attributes": {
      "body": "I like XML better"
    },
    "relationships": {
      "author": {
        "data": { "type": "people", "id": "9" }
      }
    },
    "links": {
      "self": "http://example.com/comments/12"
    }
  }]
}

Then we have the statement
GET /articles/1/relationships/comments?include=comments.author HTTP/1.1
Accept: application/vnd.api+json

In this case, the primary data would be a collection of resource identifier objects
that represent linkage to comments for an article, while the full comments and
comment authors would be returned as included data.

This would seem to indicate the the result should be something like this, where we have a list of resource identifiers returned in the data{} block, and a list of resource objects returned in the included{} block.
{
  "data": [{
    "type": "comments",
    "id": "5"
  }],
  "included": [{
    "type": "articles",
    "id": "1",
    "attributes": {
      "title": "JSON API paints my bikeshed!"
    },
    "relationships": {
      "comments": {
        "data": { "type": "comments", "id": "5" }
      }
    },
    "links": {
      "self": "http://example.com/articles/1"
    }
  }]
}

If I am reading the JSON API spec correctly, this second result is actually a second case where full resource linkage is not required, because the entities in the included{} block have no other references in the document.
So, am I correct in thinking that there are two exclusions to the full linking requirement, or have in interpreted the spec incorrectly?

The second example request calls for the inclusion of comments and the authors of those comments. The payload you posted has neither; instead, it includes the article, which wasn't requested.
This is how I believe the response should look like:
{
  "data": [{
    "type": "comments",
    "id": "5"
  }],
  "included": [{
    "type": "comments",
    "id": "5",
    "attributes": {...},
    "relationships": {
      "author": {
        "data": { "type": "people", "id": "2" }
      }
    }
  }, {
    "type": "comments",
    "id": "12",
    "attributes": {...},
    "relationships": {
      "author": {
        "data": { "type": "people", "id": "9" }
      }
    }
  }, {
    "type": "people",
    "id": "2",
    "attributes": {...},
    "links": {
      "self": "http://example.com/people/9"
    }
  }, {
    "type": "people",
    "id": "9",
    "attributes": {...},
    "links": {
      "self": "http://example.com/people/9"
    }
  }]
}

So, there is no conflict with the full linkage requirement.
Using include at a relationship URL seems fairly pointless to me. Since the first segment of any include path must necessarily be the relationship itself, you get the same result as from the corresponding related resource URL, except that everything is now under the included array.
One scenario I can think of where it accomplishes something is when you want to add several items to a relationship, some of which already exist and some need to be created on the fly:
POST /articles/1/relationships/tags?include=tags HTTP/1.1

{
  "data": [
    { "type": "tags", "id": <existing ID> },
    { "type": "tags", "id": <new ID> }
  ],
  "included": [{
    "type": "tags",
    "id": <new ID>,
    "attributes": {
      "label": "Python"
    }
  }]
}

I agree with @bintoro that there's no conflict here, so I'm gonna close this. Feel free to reopen if I've missed something.


issue

How do you use elasticsearch server to retrieve related documents, and what is the corresponding code part?

Especially when using wiki as a corpus to retrieve single-hop questions

Thanks for your interest!
We index the Wikipedia corpus for single-hop queries by running: $ python retriever_server/build_index.py wiki, and then run each strategy using the config file in https://github.com/starsuzi/Adaptive-RAG/tree/main/base_configs.
Specifically, for the single-hop queries, we use the Wikipedia corpus with local retrieval_corpus_name = 'wiki';.


Globally configured 'consumes' attribute will be propagated down to individual requests

If the global Swagger specification defines which content-types the API
can consume, this global specification needs to be propagated to a
single requests.
Only propagate this specification down to an individual request if the
consumes field is not explicitly specified for the individual request

@egbertp thank you! :) though, we will put back the .nvmrc it's goo do have this tracked by Git


Completing conda environments and adding to gitignore

Here I add the preprocessing conda environment and add some required packages to the segmentation environment.
The additional jupyter-associated packages enable explicit toggling of conda kernel in jupyter lab (not sure what happens in vscode)
I also add some developmental files to gitignore.

woo!
This comment got me thinking, why not add pybasic to the conda environment file itself!

I love the addition of a conda environment for PyBaSiC! I am excited to try it on my end.

I tried by adding this to the end of the .yml file:
- pip:
    - git+https://github.com/peng-lab/BaSiCPy@f3fcf1987db47c4a29506d240d0f69f117c82d2b

But received the following error. We'll need to keep our current install instructions in the README and update once the PyBaSiC folks get a chance to address (we can also contribute directly, but I suspect this is a longer route).
Pip subprocess error:
  Running command git clone --filter=blob:none --quiet https://github.com/peng-lab/BaSiCPy /private/var/folders/02/q30k_4wn2dqbz5pj_vvc8xn40000gp/T/pip-req-build-m7r31ud2
  Running command git rev-parse -q --verify 'sha^f3fcf1987db47c4a29506d240d0f69f117c82d2b'
  Running command git fetch -q https://github.com/peng-lab/BaSiCPy f3fcf1987db47c4a29506d240d0f69f117c82d2b
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [14 lines of output]
      error: Multiple top-level packages discovered in a flat-layout: ['figures', 'pybasic', 'Notebooks', 'ExampleData'].

      To avoid accidental inclusion of unwanted files or directories,
      setuptools will not proceed with this build.

      If you are trying to create a single distribution with multiple packages
      on purpose, you should not rely on automatic discovery.
      Instead, consider the following options:

      1. set up custom discovery (`find` directive with `include` or `exclude`)
      2. use a `src-layout`
      3. explicitly set `py_modules` or `packages` with a list of names

      To find more information, look for "package discovery" on setuptools docs.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

failed

CondaEnvException: Pip failed

I'm going to merge!


Added New command: spo user list. closes#1672

Added New command: spo user list. closes#1672
Note - I tried to remove old commit of July 07 of spo user get from this PR but somehow unable to do so, can you please advise how can I remove this commit? or it can be ignored while merging PR? If it is too much work, please reject this PR and I can create a new branch and add new command and commit again.

Coverage remained the same at 100.0% when pulling 8d716bc36bbfa5bc97163efdecd01887965f5b84 on siddharth-vaghasia:spo-user-list into 6619b8a952e2958b29afe450ffa573f232f1d29f on pnp:master.

If it's not too much trouble, could you please create a new PR with just this one commit? It would make it easier for us to process it. 😊

Sure @waldekmastykarz ....anything which makes your process easy :)...please go ahead and reject this one...I will submit new one...

Appreciate your help @siddharth-vaghasia


Tab complete user names

Seems like a nice feature to have at some point. Something like this
function getCharBeforeInputCursor() {
    const cursorPos = chatInputElement.selectionStart;
    if (cursorPos === 0) {
        return null;
    }
    return chatInputElement.value.charAt(cursorPos - 1);
}

chatInputElement.addEventListener('keydown', (e) => {
    if (e.key === 'Tab' && getCharBeforeInputCursor() == '@') {
        e.preventDefault();
        // Logic here to show a user list to choose from
    }
});

Or if I wanted to keep it really simple whenever tab is pressed in chat and the previous character is not a space or null?
Depends on #5

Minecraft itself doesn't support this with an @ prefix. I'm on board with handling that, but we should also support naked IGN tab completion.

For some reason I thought it only supported tab completion at the start of input, but it appears it works everywhere, makes things simpler.


Spelling of HTTP 429 error is inconsistent

Hello!
I am implementing a Slack app, and I am trying to implement correct backoff handling for when Slack reports my app has exceeded rate limits.
The example in the docs says you can notice that you are being rate limited with this snippet:
try:
    response = send_slack_message(channel, message)
except SlackApiError as e:
    if e.response["error"] == "ratelimited":
        ...

The method in question is chat.postMessage. However, the documentation for the errors of that method say that the rate limiting error is spelled rate_limited (with an underscore).
I clicked around at random on a few web API methods on the API docs, and I do see a mixture of ratelimited and rate_limited spellings.
The places in code (i.e. not in the docs) all seem to be calling methods that spell the error "ratelimited" (rtm.connect and apps.connections.open), so the good news is that this mostly seems to be incorrect documentation?
Maybe this is a complaint better lodged at the API itself. For my application, I'm just going to check the HTTP status code, which is what the page on rate limits recommends.
I'd be happy to send a PR for the docs, but I'm wondering what you'd consider the best approach: should the docs check the HTTP status code instead of the error message? Should the docs suggest rate_limited when calling chat.postMessage (as in the snippet above)?
Thanks!

Hi @tjstum, thanks for taking the time to write this clear feedback!

I'd be happy to send a PR for the docs, but I'm wondering what you'd consider the best approach: should the docs check the HTTP status code instead of the error message? Should the docs suggest rate_limited when calling chat.postMessage (as in the snippet above)?

I think the best approach is, as you guessed, to check HTTP status, not the error code. If you have time, we are happy to have your contribution on the docs!


Feature Request: Keep emoji icon in the lower right when editing multiline message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Problem:
When there's multiple lines being edited, the "smiley face" emoji icon is pinned to the upper-right corner when editing an existing comment, but stays in the lower-right corner when composing a new message
Solution:
Keep the emoji icon in the lower right for both.
Context/Examples/Screenshots/Notes:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1677476774053969
View all open jobs on GitHub

Triggered auto assignment to @muttmuure (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Gonna work on this one

This is a new feature and not a bug

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.79-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/15626

If no regressions arise, payment will be issued on 2023-03-14. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

No updates because of ECX

It seems like we are still waiting on payment https://github.com/Expensify/App/issues/16257

Everything is paid out. We are good to close this!


[EAGLE-508] Fix Jenkins building failed on build #71

Pls. change to title to meaningful information which will be part of commit history.


Auto-closing feature disabled when editing a file that the host doesn't have open

Product and Version: 1.24.1
OS Version: Windows
Live Share Extension Version: 0.3.319
Target Platform or Language: HTML
Steps to Reproduce / Scenario:
Please observe this gif. As you can see, using intellisense to add an end tag during a live share session causes 2 of the same tag to appear. As I typed the < followed by the / auto closing tags were doubly added.

livesharelogs.zip

@crstamps2 Thanks for reporting this! I was just able to repro this behavior, and I also identified the problem. We'll try to get a fix out for this soon. Stay tuned!
// CC @daytonellwanger

@crstamps2 Hey! We’re chasing down the “full” solution to this, but in the meantime, we shipped an update to our extension today that disables the auto-closing feature on the guest, which will prevent this duplicate tag issue from ever occurring.
If you could update to the latest Live Share release, and let us know if that resolves things for you, that would be greatly appreciated. The downside of this current fix is that a guest wouldn’t get the auto- closing feature, when editing a file that the host didn’t also have open (they would get it when editing the same file, however). In practice, that might not be a big deal, but we’ll use this issue to track resolving that additional piece of enabling complete auto tag closing for both hosts and guests. Stay tuned!

@lostintangent Not sure if this is being caused by the same issue, but a colleague and I have been having an issue with the intellisense adding an extra period when trying to use the autocomplete. More details below:
Product and Version: 1.30.1
OS Version: Windows
Live Share Extension Version: 0.3.1071
Target Platform or Language: Javascript
Steps to Reproduce / Scenario:
When using the auto-complete feature, an extra period is always added regardless of if you use the tab or enter keys. It also occurs when you click on the suggestion. See gif below:

@stricker134 , that issue should now be resolved :).

Closing this as resolved 👍  Please re-open it if you're still running into problems. Thanks!


Kwannoel/tombstone with repro

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Test https://github.com/risingwavelabs/risingwave/pull/12708
with https://github.com/risingwavelabs/risingwave/pull/12702

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

I think the best way to solve problem with too many tombstone is compacting these files as soon as possible. :)

Not planned. Issue has been fixed, see https://github.com/risingwavelabs/risingwave/issues/12680#issuecomment-1761725781.
See also the follow up issue: https://github.com/risingwavelabs/risingwave/issues/12903


Non-functional on UnRaid

Posted on wrong account, please delete.

Hi, thanks for your report!
I’m unfortunately not familiar with the Unraid community app, I didn’t create or test it.
It does sound like something is wrong with the data mapping. There should be a single SQLite database file in your exposed directory.
Do you have access to logs that might help identify the issue?


c.f. -> see

I don't think c.f. is right; see https://en.wiktionary.org/wiki/c.f. .

You're correct. I have no idea where this came from. I've been relying too much to the copy editing of journals in the past.


Middleware config

I appreciate having pkger. Great work! I do not care about the slow speed at the moment.
This new development seems to be a chance to broaden what a packager can do.
I was looking for embedding js, css and html. So far I use a bash script to minify these assets and then embedded them into my binary:
https://github.com/PatWie/digitalmusicstand/blob/master/.drone.yml#L14-L27
Would it be possible to have a pkger config file that contains the intermediate result like ˋpkger listˋ does today? I am thinking about something the Hugo assets pipeline does: concatenate and minify files, scss parsing, ...
This further allows to also encrypt the packed data (I saw a feature request in another similar library during testing different packers) besides caching the absolute file paths.

Sorry you don’t like the speed, although without specifics I can’t help you with that, other than to say that PRs are welcome.
As for the other request, you can use the parser directly https://godoc.org/github.com/markbates/pkger/parser#Parser to return a list of all files to be packed.


Bug fix: getRollups API, rollup jobs landing page

Issue #, if available: N/A
Description of changes:

getRollups API fix: modify format of node API to take optional parameters.
Modify rollup jobs landing page to show fields correctly and have working sorting function.
Clean up code

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Output after running yarn run test:jest
Test Suites: 32 passed, 32 total
Tests:       167 passed, 167 total
Snapshots:   28 passed, 28 total
Time:        156 s, estimated 383 s
Ran all test suites.
✨  Done in 158.03s.

Output after running yarn run test:jest
Test Suites: 32 passed, 32 total
Tests:       167 passed, 167 total
Snapshots:   28 passed, 28 total
Time:        156 s, estimated 383 s
Ran all test suites.
✨  Done in 158.03s.

Unit tests and E2E cypress test passed in previous workflow run: https://github.com/opendistro-for-elasticsearch/index-management-kibana-plugin/runs/1772682858

Unit tests and E2E cypress test passed in previous workflow run: https://github.com/opendistro-for-elasticsearch/index-management-kibana-plugin/runs/1772682858


Introduce Package Identity

Introduce Package Identity

Package identity will be used to uniquely identify packages during
dependency resolution.

By default, the package identity is the basename of the package
repository URL. In future, the package references in the manifest file
will be allowed to override the name that should be used instead of the
basename. The package identity should always match (case insenstive for
now) the name of the package. This will be enforced via error checking
in future, i.e., once we have the API for overriding.

The commit will make dependency resolver to use the package identity
instead of package URL. The package graph is still constructed using the
package URL but that will be updated as well in upcoming commits.

@swift-ci please smoke test

@ddunbar @abertelrud This is mostly a refactor so I am landing it, but I'd appreciate a review. I'll address the comments post merge.

@swift-ci please smoke test


Missing website description?

All pages need a better/longer/new description meta tag. It's showing up in Google as:

And DuckDuckGo as:

I can edit descriptions for individual posts, but still can't find the website description.

A description is showing up in DDG now, but it's too short.

Descriptions for the entire site are missing on Google. This is especially odd on the blog.

DDG now lists page contents instead of the navbar, but still not using the description meta tag.


Merge development into master branch for publishing v0.2.0 release

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage decreased (-0.007%) to 43.911% when pulling 139853f2c90a0a44991dfa9371d82f5ddf574fb6 on development into 7f382155bddd5aaae8a123e3da66de20e61bc2b8 on master.


MishaShevchenko: precourse : 1 week

Learning Objectives

GitHub:

[x] 🥚 Creating Issues: You can create a new issue in a GitHub
repository using an issue template. You can correctly use labels,
milestones, and assignment to make your issue findable.
[x] 🥚 Searching Issues: You can find issues in a repository by label,
author, and milestone.
[x] 🥚 Project Boards: You can use issues on a GitHub project board to
track your class assignments and help your classmates when they need it.

HYF Workflows:

[x] 🥚 Check-Ins: You can use the check-in template to track your
module learning objectives, your weekly progress (and to post a
retrospective at the end of the module). You create one check-in issue
per module in the class repo and each Thursday you add a check-in
comment to it and the label for the corresponding week. After review, we
add the checked label. No need to remove previous labels either. So as
an example, after a 5 week module, you would have 5 comments and 5
labels added by you.
[x] 🥚 Deliverables: You can create an issue for this module's
deliverable and track your progress on the class project board. Each
week, there are different assignments / deliverables. If it is an
individual project, you create your own deliverable issue for it in the
class repo. If it is a group project, the entire group creates 1 issue
together in the class repo.
[x] 🥚 Asking for Help: You can use the help-wanted template in your
class repository's Discussion board to ask for help, and to help your
classmates when they are blocked.

Your Computer:

[x] 🥚 File System Management: You can create and organize a new folder
in your personal computer for the materials you will study at HYF.

Visual Studio Code:

[x] 🥚 Install and manage extensions
[x] 🥚 Open a folder for editing
[x] 🥚 Create new files/folders inside of the folder
[x] 🥚 Edit files and save changes

Hi @MishaShevchenko you also need to add week 1 to your issue, see this example

Hi Bernardo Martelli,
MishaShevchenko: precourse : 1 week · Issue #28 · HYF-Class22/home · GitHub
https://github.com/HYF-Class22/home/issues/28
I don't understand what is wrong. Because week 1 was added earlier.
вт, 23 янв. 2024 г. в 19:04, Bernardo Martelli @.***>:

Hi @MishaShevchenko https://github.com/MishaShevchenko you also need to
add week 1 to your issue, see this
https://github.com/HYF-Class22/home/issues/16 example
—
Reply to this email directly, view it on GitHub
https://github.com/HYF-Class22/home/issues/28#issuecomment-1906628662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZFHSV4XDSIJABPEPMEWKT3YP73UVAVCNFSM6AAAAABCHEXXLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBWGYZDQNRWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi Bernardo Martelli, MishaShevchenko: precourse : 1 week · Issue #28 · HYF-Class22/home · GitHub <#28> I don't understand what is wrong. Because week 1 was added earlier. вт, 23 янв. 2024 г. в 19:04, Bernardo Martelli @.>:
…
Hi @MishaShevchenko https://github.com/MishaShevchenko you also need to add week 1 to your issue, see this <#16> example — Reply to this email directly, view it on GitHub <#28 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZFHSV4XDSIJABPEPMEWKT3YP73UVAVCNFSM6AAAAABCHEXXLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBWGYZDQNRWGI . You are receiving this because you were mentioned.Message ID: @.>

Please have again a look at the example I have provided.


Issue after installing (attempted to read or write protected memory)

A colleague wanted to install latest haxedevelop (development build from https://github.com/fdorg/flashdevelop/commit/31cda34fea58181edc4e0e3e2831c4428f85a30b)
She gets:

I actually tried same but have no issues, any idea?

Actually she gets multiple errors in a row

x64?

Duplicate #3036 3036


[BUG] Out of the box, cannot access lexical declaration 'root' before initialization

Hello!
I was trying to use this library in my SolidJS library project and I have found a problem. I tried to create a <Motion.div> element like this:
const appliedAnimation = {
    unmount: {
      opacity: 0,
    },
    mount: {
      opacity: 1,
    },
}
let [open, setOpen] = createSignal(true);
...
return (
    <>
      {open() && (
        <Motion.div
          {...rest}
          class={classes}
          initial={appliedAnimation.unmount}
          exit={appliedAnimation.unmount}
          animate={open() ? appliedAnimation.mount : appliedAnimation.unmount}
          transition={{ duration: 0.5 }}
        >
          {icon && iconTemplate}
          {value}
          {onClose && !action && TODO}
          {action || null}
        </Motion.div>
      )}
    </>
)

When I try to visualize the component I get the following error:
Uncaught ReferenceError: can't access lexical declaration 'root' before initialization
<anonymous> index.js:5450
createAndBindMotionState index.js:5434
runComputation index.js:184
updateComputation index.js:174
createEffect index.js:64
createAndBindMotionState index.js:5432
MotionComponent index.js:5450

Digging a little bit into solid-motionone I found that the conflicting line is inside the MotionComponent:
https://github.com/solidjs-community/solid-motionone/blob/ba9005ffe20ae3b4327739c0e83ff0f1623cdb03/src/motion.tsx#L35-L45
As it can be seen the root variable is declared after it's reference into the arrow function? For me the transpiled code appears as:
var MotionComponent = (props) => {
  const [options, , attrs] = splitProps(props, OPTION_KEYS, ATTR_KEYS);
  const [state, style2] = createAndBindMotionState(() => root, () => ({
    ...options
  }), useContext(PresenceContext), useContext(ParentContext));
  let root;
  return createComponent(ParentContext.Provider, {
  ...

I'm quite new to solid-js, sorry if I'm missing something obvious here. I also tried to but the let root before the createAndBindMotionState call, and then the error is Animation state must be mounted with valid Element.
Finally here's a little bit of info about versions:

solid-js: ^1.8
solid-motionone: ^1.0.0

Thanks in advance! 😁

The code is correct
you can use variables in callbacks that are declared below
because it all depends on the order of execution
if the callback is ran after the variable has been declared, it's all good
Here, the ref needs to be assigned first before it can even be used so the placement of the variable in the component shouldn't matter
Changing the order won't fix the code, just change the error message
But what's weird is that the callback is being called immediately
As you can see the element callback is ran in an effect, which should happen after the elements have been created and ref declared and assigned with the html element.
Effects in solid are scheduled to run at the end of a internal queue, which is created by effects, roots, render calls, etc.
If the effect executes immediately, it means that there in no such queue that is can attach to, it has to run as soon as it's created.
If you don't have any other issues in you app, then my guess would be that your app and solid-motionone are using two different versions of solid at the same time, and the effects queue that solid in your app creates, is not the same that solid from solid-motion one expects. Thats just a quess but something like that is fairly common.
Can you make sure that is not the case or provide a minimal repro to try out?
You can confirm that this is an issue by doing something like this:
// somewhere in your app and in solid-motionone
import {createSignal} from "solid-js"
if (!window.createSignal) {
	window.createSignal = createSignal
} else {
	console.log(window.createSignal === createSignal ? "Good" : "Bad")
}

Figured out our root issue. The app with the failure had some weird setup logic around the initial render that was causing the:
computations created outside a `createRoot` or `render` will never be disposed

This stack overflow post helped explain it.
Once I addressed the Solid warning, the above referenced issue went away.

It might be nice to add a check in the code for root to be defined, and to throw a more helpful error if it's missing.

If you don't have any other issues in you app, then my guess would be that your app and solid-motionone are using two different versions of solid at the same time, and the effects queue that solid in your app creates, is not the same that solid from solid-motion one expects. Thats just a quess but something like that is fairly common.

Indeed, this was the problem I was programming a component library and using solid-motionone on my playground project as well as in the component library project. This caused two different versions of solidjs to be running.

I wouldn't mind having some nice error message that links to this issue during development

This would be nice though, since I spent couple of hours trying to find the root cause of the problem. I totally understand this is not a solid-motionone issue but it would be a niceity plus. Maybe we could add the same check you advised me to use:
function checkDupedSolid() {
    if (!window._smoCheck ) {
	window._smoCheck = createSignal
    } else {
	if (window._smoCheck !== createSignal) {
            console.warning('There is more than one instance of solid running. This may cause errors in the use of this library.')
        }
        window._smoCheck = undefined;
    }
}

checkDupedSolid();

Thanks!


Configuration cache should support more use cases of maven-publish plugin

The general configuration cache support of publishing to Maven was implemented in #13468, but there are some corner cases that aren't handled well. This is an umbrella issue for these corner cases.
Known issues:

[ ] #24040
[ ] #22779
[ ] #23397

Should https://github.com/gradle/gradle/issues/22779 also be on this list? It's the main thing that prevents https://github.com/vanniktech/gradle-maven-publish-plugin from fully supporting the configuration cache

Let's add the following to the list of unsupported setups:

#24765


Added CharmMod

Added the mod CharmMod to list.

so a few things:

your SHA256 seems too short for a SHA256, should have exactly 64 numbers and letters
the download should point directly to either a dll or zip file.

the dll file in the release, does it need the pictures next to it? if not, you can just link that with its SHA256

Maybe a different name? CharmMod sounds IMO too generic, maybe something like SkillCharms would fit better, since a majority of the charms improve skills/equipment.
Similar to 3. the description could also hint that the charms improve skills/equipment.

And add dependencies this mod need

what the check (and modlinks itself) want:

Link => a link that directly links to a dll or zip file, not just the release page

There i also see that the mod loads images next to the dll

While perfectly okay, there also is a technology to have the images directly embedded into the dll, though i can't find a tutorial for that rn sadly

So i would say that you zip up everything that is in the current release (so all png files, the readme and the mod with both dll and pdb files), upload that into the release as well (click Edit Release to do that) and link that zip directly in modlinks, with the SHA256 of said zip.


"Custom Dialog on Close": API bikeshedding

If the current approach is followed (i.e. ignoring the ideas in #142 which should definitely be explored), then I'd suggest specifying the various strings in a property bag rather than separate methods. i.e. instead of:
  event.dialog.setMessage("You have unsaved changes, would you like to save them?");
  event.dialog.setButtonLabel("Save");
  event.dialog.show().then( ... );

Do this:
  event.dialog.show({
    message: "You have unsaved changes, would you like to save them?",
    buttonLabel: "Save"
  }).then( ... );

This naturally extends to having enum values capturing options. This may also be identical to the state that you'd want to passively associate with a window in the #142 proposal, rather than waiting until an event fires.
EDITED: originally incorrectly referenced issue 147 instead of 142

This is a great idea, it simplifies the API surface and it's clearer and more extensible. Thanks for the suggestion!
Which ideas did you think were worth exploring in #147? Or did you mean the declarative approach from #142? I am definitely exploring the ideas presented there.

Oops, you're right - I meant #142. I'll edit my comment.

I'm a massive fan of the options object rather than method separation. I think this is one of the odd parts about the Dialog API where it has show and showModal vs show({modal: true|false}).

We have withdrawn our proposal on this problem due to the design conflict with the philosophy of keeping tab close as fast as possible. We have been exploring other alternatives including using Background Sync and Service Workers for sync'ing the app's state. However, we still see issues in sync'ing large data without loss between the document and service worker when the user closes the tab in an arbitrary moment.
We'll look at this problem space again when we have a strong signal from the community. We'll make sure to re-open this issue when that happens. Thanks!


Fix functions associated with Direct Messages

Twitter retired legacy Direct Message endpoints three days ago.

I fixed "get" part of DM
You can find pull request at: https://github.com/tweepy/tweepy/pull/1109
You can use my forked repo: https://github.com/bakayim/tweepy

I also fixed sending DM. You can find it at my forked repo
Star if it helps you please, thanks

#1081 is the open issue regarding this.


chore: update website dependencies

:tada: This PR is included in version 4.2.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


API utf-8 decode issue

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.38.3
Python release (python3 --version):
Component/platform:
api.py
Description of problem:
Using Plex WebHooks and the API is unable to handle the request payload
Expected:
API receives payload and passes to appropriate server
Problem-relevant configuration.yaml entries and steps to reproduce:

 plextest:
    alias: 'Webhook test'
    sequence:
      - service: notify.notifyall
        data_template:
          message: >
            '{{ Player.title }}'

Set up webhook in plex (https://home.domain.com/api/services/script/plextest?api_password=PasswordGoesHere)
start a video
error occurs

Traceback (if applicable):
17-02-17 15:11:23 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web_server.py", line 62, in handle_request
    resp = yield from self._handler(request)
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web.py", line 270, in _handle
    resp = yield from handler(request)
  File "/usr/local/lib/python3.5/asyncio/coroutines.py", line 213, in coro
    res = yield from res
  File "/usr/local/lib/python3.5/asyncio/coroutines.py", line 213, in coro
    res = yield from res
  File "/usr/src/app/homeassistant/components/http/__init__.py", line 427, in handle
    result = yield from result
  File "/usr/src/app/homeassistant/components/api.py", line 311, in post
    body = yield from request.text()
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web_reqrep.py", line 394, in text
    return bytes_body.decode(encoding)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 6270: invalid start byte
17-02-17 15:12:07 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web_server.py", line 62, in handle_request
    resp = yield from self._handler(request)
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web.py", line 270, in _handle
    resp = yield from handler(request)
  File "/usr/local/lib/python3.5/asyncio/coroutines.py", line 213, in coro
    res = yield from res
  File "/usr/local/lib/python3.5/asyncio/coroutines.py", line 213, in coro
    res = yield from res
  File "/usr/src/app/homeassistant/components/http/__init__.py", line 427, in handle
    result = yield from result
  File "/usr/src/app/homeassistant/components/api.py", line 312, in post
    data = json.loads(body) if body else None
  File "/usr/local/lib/python3.5/json/__init__.py", line 319, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.5/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.5/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Additional info:
Works fine when using the dev-service to call the script using the request payload here: https://support.plex.tv/hc/en-us/articles/115002267687

The payload that leads to this error would be helpful here. You can use something like fiddler to catch those. Else reproducing this behavior might prove tricky as it requires a plex pass.
~Cheers

Okay. Here's the payload that I've outputted from tcpflow in Linux and I make it human-readable. Unsure if UUID is sensitive, so I've "x'd" them out.
{
  "event":"media.play",
  "user":true,
  "owner":true,
  "Account":{
    "id":1,
    "thumb":"https://plex.tv/users/fd6f05d106f45a73/avatar?c=2017-02-19+03%3A50%3A09+UTC",
    "title":"GraysonPeddie"
  },
  "Server":{
    "title":"My SHIELD Android TV",
    "uuid":"xxxxxxxxxxxxxxxxx..."
  },
  "Player":{
    "local":true,
    "publicAddress":"<IP_ADDRESS>",
    "title":"Nexus 7 (2012)",
    "uuid":"xxxxxxxxxxxxxxxxx..."
  },
  "Metadata":{
    "librarySectionType":"show",
    "ratingKey":"12692",
    "key":"/library/metadata/12692",
    "parentRatingKey":"12691",
    "grandparentRatingKey":"12690",
    "guid":"com.plexapp.agents.thetvdb://95441/8/15?lang=en",
    "librarySectionID":4,
    "librarySectionKey":"/library/sections/4",
    "type":"episode",
    "title":"Payback",
    "grandparentKey":"/library/metadata/12690",
    "parentKey":"/library/metadata/12691",
    "grandparentTitle":"NCIS: Los Angeles",
    "parentTitle":"Season 8",
    "contentRating":"TV-14",
    "summary":"Emotions run high for the team as one agent is held in an unknown location and tortured by the mole, while another discovers he was betrayed by someone he least expected.",
    "index":15,
    "parentIndex":8,
    "year":2017,
    "thumb":"/library/metadata/12692/thumb/1487559639",
    "art":"/library/metadata/12690/art/1487559640",
    "parentThumb":"/library/metadata/12691/thumb/1487559640",
    "grandparentThumb":"/library/metadata/12690/thumb/1487559640",
    "grandparentArt":"/library/metadata/12690/art/1487559640",
    "grandparentTheme":"/library/metadata/12690/theme/1487559640",
    "duration":3600000,
    "originallyAvailableAt":"2017-02-19",
    "addedAt":1487559608,
    "updatedAt":1487559639,
    "Director":[
      {
        "id":3569,
        "filter":"director=3569",
        "tag":"Terrence OHara"
      }
    ],
    "Writer":[
      {
        "id":3568,
        "filter":"writer=3568",
        "tag":"Jordana Lewis Jaffe"
      }
    ]
  }
}

I just tried that payload and it worked fine. Any chance for getting the actual HTTP request that is causing an issue to reproduce?
~Cheers

Okay, what about this:
hass@home-server:~$ sudo tcpflow -i br0 port 8123 -c
tcpflow: listening on br0
<IP_ADDRESS>.60022-<IP_ADDRESS>.08123: POST /api/services/script/plex_webhook HTTP/1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36
Host: <IP_ADDRESS>:8123
Accept: */*
Accept-Encoding: gzip
Content-Length: 10350
Content-Type: multipart/form-data; boundary=----------------------------5887e83586f4

<IP_ADDRESS>.60022-<IP_ADDRESS>.08123: ------------------------------5887e83586f4
Content-Disposition: form-data; name="payload"
Content-Type: application/json

{"event":"media.play","user":true,"owner":true,"Account":{"id":1,"thumb":"https://plex.tv/users/fd6f05d106f45a73/avatar?c=2017-02-20+03%3A08%3A28+UTC","title":"GraysonPeddie"},"Server":{"title":"My SHIELD Android TV","uuid":"..."},"Player":{"local":true,"publicAddress":"<IP_ADDRESS>","title":"Plex Web (Chrome)","uuid":"..."},"Metadata":{"librarySectionType":"show","ratingKey":"12692","key":"/library/metadata/12692","parentRatingKey":"12691","grandparentRatingKey":"12690","guid":"com.plexapp.agents.thetvdb://95441/8/15?lang=en","librarySectionID":4,"librarySectionKey":"/library/sections/4","type":"episode","title":"Payback","grandparentKey":"/library/metadata/12690","parentKey":"/library/metadata/12691","grandparentTitle":"NCIS: Los Angeles","parentTitle":"Season 8","contentRating":"TV-14","summary":"Emotions run high for the team as one agent is held in an unknown location and tortured by the mole, while another discovers he was betrayed by someone he least expected.","index":15,"parentIndex":8,"viewOffset":3344131,"lastViewedAt":1556978583,"year":2017,"thumb":"/library/metadata/12692/thumb/1487559639","art":"/library/metadata/12690/art/1487559640","parentThumb":"/library/metadata/12691/thumb/1487559640","grandparentThumb":"/library/metadata/
<IP_ADDRESS>.60022-<IP_ADDRESS>.08123: 12690/thumb/1487559640","grandparentArt":"/library/metadata/12690/art/1487559640","grandparentTheme":"/library/metadata/12690/theme/1487559640","duration":3600000,"originallyAvailableAt":"2017-02-19","addedAt":1487559608,"updatedAt":1487559639,"Director":[{"id":3569,"filter":"director=3569","tag":"Terrence OHara"}],"Writer":[{"id":3568,"filter":"writer=3568","tag":"Jordana Lewis Jaffe"}]}}
------------------------------5887e83586f4
Content-Disposition: form-data; name="thumb"; filename="thumb.jpg"
Content-Type: image/jpeg

JFIFHHC
2!!22222222222222222222222222222222222222222222222222=!1"AQaq2#B$R3r%4CSb%!1A"Q2aq?BZIJ,t#a!NtD/JJ$XXhI8BG0UMn%q6.{,]1.gDj2  yFlR>i/+*)ck+ h]#+BG|nI([s#(VP.S,:m:(EBGb/JJ$X(Xhaal$D"i\
}-T| ?Go}e#.dgE-BjZ6FAiPoJMoVH4Fs(W{7;E1n/X"9m dRW@.4K"{!^a =+F4kn0CPD`N/gK8F#Uzz"0E
P]3 B<z<oD:Q2@airs,d4#[@a<OT,;5vDoj$gw,Nj~X7<!nOKipy(-tMfy!B2&s'qU6u^B@)juS&1ma)zz
)Fo}6ov#]8%nDJu4D-bQh@8'8*
V[X$K4]u^_cMxQeheKn`
v#@)sfu6}7$T2kKB;n@1M,oDIQi
E9s65LNka.s=,uaUb\v*2!nXU>P4,Y**\14["FN'{YC;?De]Z~U3"?2bv6-z=R66cb.|(]ST+M Fo_LaJEd.Q^l{db6['H(QT
W*[wjQ1*1)F#I_h}.6y/F]H xVW$9Q|::w:)mM:qoE~Ew;aI7
.46o>~N>-c,I]v2De>QQVUr,bZ
$E]3MkpDG$KzoR9(wnL|3s^Io !b?zr0PX<,#GA})V)MrH\D1t n=X)`*v\<)
 cSe:J<:zjRwa|lt/f9UbEQ^f3 eF"s5Ok`>U..d-nh]LzE4mV-Zb4zS]1ym9S[(qb/TVXJu#J-5^#ln^x'Ve9%fk|}ssE*UT-;7s'(~qr"Bu%<
DZ6~8jIG#hY%)Sf\c;<l>J/JU"rZi:Ft <|w&7,4];VSx?i{
H0-Q4<V|vz5>$1f{}7R,:gX@9cPmY/tKI^`%UZ446Lu8yed#S"YNaqc\F|1nSID~YL2RVr;6&pj'i4sEcj2#NGh'`6|ae,Z]5pL[q7cQEe5/X@#T+k5Q*9?zX )R*tAgx8oCJcG|&$G%ZX\eVY9H9W'nkc^fl,oQQDR-3v0y%@l;n*r1I
g9>";vnd[Ea?\H.%)aDM2{M&X{,SVttjR#5[0vs&4$>"es;9Gx#u)Mx.Y`dFMo%iL{>xMl|e%v4($&@9)Z
e~T$JJZu@I7aJ\Na|D:{c!OJTi5dIxha#\eWOVF-m$#nk6$1NIUy6Sj_""dH\.2JFb(f&n6qs6>cY,(;QL7Z+^
hpag=goO+W)GVQ/j\5VIvZdg=tzGLYNiFm05h>yRjKqRq+g}Pq=7;*cG2HuJLwp15mpGD#sSI8bqcZn,P.spw%j"3I,6l<kG%QdNkyYA$#a]bIaNRC~;J<wr8Ka8ujJ[};X3+l%SF|e`kZ7df#OqrnCAH+wg-C*4r7S)Xb9``2i$\2lyN2:R]D|E#lilz6#K^3i-
tKd2JnsX1!hy8I\Z,R[b9]<[.C<h}Eesro#",XNs$d?S\/Fc_ft}Q/-]"%N0OEZ,d:%YFXVv.Bek;*6(IHDoF0YT6<=j1O4 )Um_6:6qLYGw\tOA|
\+fm=;K(9-F"@+g'aT&d1@d[(+|1isF F2qz!CjmzfH23[mRFr:Yr\ \:#ma$Do*M35sf(
7`>QNVP{xb.A&5qMc1Q|ic<v~W*>oc}Q`s@-)GxK`5ZM2tfWbUqzr 95MHW[FX+Z(&RRhJO)m
LY0:g
x%SUOC*zf-qG,Q_$53+EP?1Jq}ZZiF!M*XZ-M:sQ.vchD#l5=7MX~&U3}ajpL{Kv|wN?h%CmuL0$n]2_$bG{R;y19yUi"!Ih2B6o+SvGH'irE18ph7?%*3I4#84+xKy@YoX^Ye+[=$i$*<OO+Gadi$4Yyk'2xl}g_zi>R[5EadeP_CUU0nG*?*
j!()PdWRO
rSi7.g.Z.0)s&Z`|+.9dP1*.dJr5]
j9JZEMY<;%'c)dc!g`miv]h\gn@n1S#wVDK3cybd->[(Ec#NT'f8}41)\]nUmsIDE{"rD
LFVHLyX=_hKiRYGm*9nWh3&,#<N`$>2%3Mgrr*v9FV]^cUZz*J|<gblmBK@:[IT4Xb5T4Av/tK;skl2[OjD2lnG\SrBP?)un`|[h\M{<9S|\9%nfF|PIafu!, id1$hb|z*oO\Qb7{@*'qB=H!;0|WmI5#]gmk2e\dfFILj9M{,U|$B[TtY|3!g
hHu=8<IP_ADDRESS>022-<IP_ADDRESS>.08123: ?C.Go=,R[12G7Y"LdapOTe
>x>9WZkPS^{=b(Sh)4%JukmR4-^AVh$?
------------------------------5887e83586f4--

<IP_ADDRESS>.08123-<IP_ADDRESS>.60022: HTTP/1.0 500 Internal Server Error
Content-Type: text/html; charset=utf-8
Content-Length: 141
Date: Tue, 21 Feb 2017 14:48:49 GMT
Server: Python/3.5 aiohttp/1.3.1

<title>500 Internal Server Error</title>
500 Internal Server Error
Server got itself in trouble
^Ctcpflow: terminating
hass@home-server:~$

Thanks for that. It did help A LOT. But unfortunatly I am just not expierenced enough to figuere out how to fix it. It comes down to using multipart from aiohttp.
If anyone a bit more into python and all that reads this in the api.py line 312 is where we need to change the normal parsing of the json into something that is able to handle multipart requests like described here http://aiohttp.readthedocs.io/en/stable/multipart.html
This is not a fix for this only but should also be implemented to prevent further errors like this.
Sorry that I am not able to help out more.
~Cheers

Do you mind if I spread the word?
https://twitter.com/GraysonPeddie/status/834086655128641540
I hope I can be of help to reach out to more Python developers.

Also to maybe speed up the proccess of get this working you might consider writing to the plex support to implement a option to send JSON only data to webhooks. Because what is making trouble here is the fact that plex does send an additional jpeg with the json.
~Cheers

I asked Plex in Twitter. Maybe Plex will implement an option to make JPEG optional and do process of elimination. I probably should asked that in PlexPass forum, but I'm going to try Twitter first.

It should be simple enough to only process the JSON part of the payload, no?

I'm not sure if it's possible for api.py or aiohttp.server to only capture JSON response and ignore JPEG image part...

I would assume it's possible, it's a bonafide HTTP server. I'm not sure how easy it is, but many HTTP stacks provide processing for multi-part POSTs.
(I'm from Plex, if it wasn't clear. Glad someone's using Webhooks!)

If the developers of Home Assistant could change to a different stack that can process multi-part messages, then at least there should be an abstraction between a stack and a core of HASS.

(We also discussed at some point adding settings to the webhooks to control such things, but I can't speak as to when/if we'd do such a thing.)

Like I linked in a previous answer it is absolutely possible to handle multipart in API.py as it uses aiohttp which supports this (as seen on my link above). I am just not able to do it myself as I have not setup a dev environment nor am I experienced enough with home assistant.
~Cheers

I'm living in the dorm at World Services for the Blind right now and I don't have access to Home Assistant and home automation devices until next week to test it.
Thanks
--Grayson Peddie
Get Outlook for Android
On Sun, Apr 30, 2017 at 1:39 AM -0500, "balloobbot"<EMAIL_ADDRESS>wrote:
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I just tested this with data provided by @GraysonPeddie and it still does not work. We need someone with an actual PlexPass to confirm this 100% tho.
~Cheers

Hi, I have PlexPass and I'm also interested on this, as I had to deploy a node.js server just to catch the webhooks.
If you need some testing, I'm on board.

I have a plex pass and can 100% confirm this bug still exists.
This what the web hook posts:
------------------------------4938c11bd7bc
Content-Disposition: form-data; name="payload"
Content-Type: application/json

{"event":"media.pause","user":false,"owner":true,"Account":{"id":5832566,"thumb":"hidden","title":"Adults"},"Server":{"title":"Docker","uuid":"**"},"Player":{"local":true,"publicAddress":"**","title":"Lounge","uuid":"**"},"Metadata":{"librarySectionType":"show","ratingKey":"274665","key":"/library/metadata/274665","parentRatingKey":"206936","grandparentRatingKey":"29776","guid":"com.plexapp.agents.thetvdb://274431/3/16?lang=en","librarySectionID":4,"librarySectionKey":"/library/sections/4","type":"episode","title":"Heroes Rise: These Delicate and Dark Obsessions","grandparentKey":"/library/metadata/29776","parentKey":"/library/metadata/206936","grandparentTitle":"Gotham","parentTitle":"Season 3","contentRating":"TV-14","summary":"The Court of Owls devises a new plan regarding the future of Gotham, as Gordon uncovers information about his father and uncle's past, connecting him back to the organization. Meanwhile, Bruce wakes up in the temple and learns of the Shaman's wish for him.","index":16,"parentIndex":3,"rating":8.7,"viewOffset":1290000,"lastViewedAt":1494789921,"year":2017,"thumb":"/library/metadata/274665/thumb/1494289723","art":"/library/metadata/29776/art/1493684389","parentThumb":"/library/metadata/206936/thumb/1493684389","grandparentThumb":"/library/metadata/29776/thumb/1493684389","grandparentArt":"/library/metadata/29776/art/1493684389","grandparentTheme":"/library/metadata/29776/theme/1493684389","originallyAvailableAt":"2017-05-01","addedAt":1493684278,"updatedAt":1494289723,"Director":[{"id":75914,"filter":"director=75914","tag":"Ben McKenzie"}],"Writer":[{"id":34345,"filter":"writer=34345","tag":"Robert Hull"}]}}
------------------------------4938c11bd7bc--

Using postman and posting this to a script for parsing give this error:
2017-05-14 20:40:21 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web_protocol.py", line 417, in start
    resp = yield from self._request_handler(request)
  File "/usr/local/lib/python3.5/site-packages/aiohttp/web.py", line 289, in _handle
    resp = yield from handler(request)
  File "/usr/local/lib/python3.5/asyncio/coroutines.py", line 213, in coro
    res = yield from res
  File "/usr/local/lib/python3.5/asyncio/coroutines.py", line 213, in coro
    res = yield from res
  File "/usr/src/app/homeassistant/components/http/ban.py", line 58, in ban_middleware_handler
    return (yield from handler(request))
  File "/usr/src/app/homeassistant/components/http/__init__.py", line 424, in handle
    result = yield from result
  File "/usr/src/app/homeassistant/components/api.py", line 313, in post
    data = json.loads(body) if body else None
  File "/usr/local/lib/python3.5/json/__init__.py", line 319, in loads
    return _default_decoder.decode(s)
  File "/usr/local/lib/python3.5/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/local/lib/python3.5/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Can #8121 fix this issue?

I doubt it...this is a problem because HA is only expecting singlepart HTTP requests but plex sends multipart. AIOHTTP already can be configured to handle this it just needs to be done.
~Cheers

Still an issue.

Obtaining some of the multipart data isn't hard, for example I was able to trigger a script like above by adding this (sorry I've never written python before)
if request.content_type == 'multipart/form-data':
  data = {}
  reader = yield from request.multipart()
  while True:
    part = yield from reader.next()
    if part is None:
      break
    name = part.name or part.filename
    if part.headers[aiohttp.hdrs.CONTENT_TYPE] == 'application/json':
      data[name] = yield from part.json()
else:
  body = yield from request.text()
  data = json.loads(body) if body else None

The problem is more about API design philosophy - multipart is awkward, it's technically recursive, and usually the recipient has some knowledge/expectations about the format (you can see my attempt above at trying to just make a dict out of it - but I would be skipping out of parts that are not JSON!) - it's harder to just 'pass through' parsing of it. This is especially true because HA uses the JSON structure of the API as essentially a tunnel for params to different functions/components.
Furthermore, I'm not sure that scripts are a good target for this. When clicking play/pause fairly quickly you get WARNING about the script already running. I think I am going to stick with running a little Node.js server that understands the Plex Webhook format (expects a part with 'payload') and then manipulates state in HA however appropriate.
I would suggest adding something that does content-type filtering to HA - you probably can't do it at the middleware level, but maybe with a base view class or something (sorry, still very new to HA), you should check that the content-type header is JSON and return a 400. At that error would be easier to understand - it's a JSON API, don't send it non-json, vs. deep byte-level parser errors as seen in this issue.

I agree on the content-type validation part! Should definitely be done. As you are using plexpass maybe you could also open a feature request with them to let you decide if you want a multipart webhook or a pure json/html without the multipart.
I also ran into the script already running issue with the webhooks. Would be great if we could get a setting to get scripts queued. With a entry in the yaml or a specific component like "queued_scripts" something like that. I use those to add information to HA and this actually results in lost data for me. For the start/play it would also be great because even if it would take a little longer to do the thing it would always stay synced to the state.
~Cheers

I mean unless you've rearchitected and overhauled the core HTTP handling logic it's going to persist.

How viable is it to add a buffer (I'm new to Home Assistant, but a script?) in between a UTF-8 webhook and Home Assistant?

Again I quote dougmoscrop on this

I mean unless you've rearchitected and overhauled the core HTTP handling logic it's going to persist.

If you want to support Plex webhook, you can implement a HTTPView for that purpose. components/rachio.py is a good example. By this way, you can also use your own token system, not depends on HA's api_password to reduce your security risk.
I also don't see any chance we will extend current HA RESTful API to support multipart-form post. Moreover, we are planning retire the api_password authentication on RESTful API.
After all, I am going to close this issue since it was a Feature Request not an issue.


Fix "pool not found" after the new rescan when oracle token in a box without R4 defined (filter oracle box properly)

Close #256

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.05%) to 60.318%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/oracle_state.rs
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/src/oracle_state.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
2049

Relevant Lines:
3397

💛  - Coveralls


Issue with k8s.io/docs/tutorials/kubernetes-basics/expose-interactive

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
There is a typo, in step 2 of katacoda module 4. get pod in "We'll use the get pod command with -l as p parameter" is not valid command.
Proposed Solution:
Need to change the get pod to get pods
Page to Update:
https://kubernetes.io/docs/tutorials/kubernetes-basics/expose-interactive/

pending resolution with upstream content PR: https://github.com/katacoda-scenarios/kubernetes-bootcamp-scenarios/pull/6

Thanks

Thank you. The PR has now been merged


Whether to operate wireguard with routing or natting should be adjustable

iptables.go line 39
	err = ipt.Append("nat", "POSTROUTING", "-s", config.Values().Wireguard.Range.String(), "-j", "MASQUERADE")
	if err != nil {
		return err
	}

Me again,
it would be great if you could set whether the traffic should be routed or natted, since routing can later be used to clearly identify which request comes from which client
example config.json
DEFAULT: false
"routing": "false"

If routing is true, just skip that code
Hope it is understandable.

Yep that makes sense and I like it. Being able to turn on and off batting makes a lot of sense.
I'll add that.

This is now on the unstable branch for you to try out

Thanks! I am currently testing

Works perfectly!


Suggestion: Remove color on underfunded categories

There are times throughout some months where I will remove some money from one budget to cover an unforeseen purchase in another budget. This is usually something like removing $25 from my entertainment budget to cover a prescription in my medical budget. When I have a goal set of $50 in entertainment, and I pull $25 out, it then highlights that budget as yellow, warning me I'm under budget.
I get that I'm under budget. And if I want to, I'll top it back up at the next paycheck. However, more often than not, I can go without that extra $25 in the entertainment budget, and would rather just not see a warning telling me I'm under budget.
Can you guys give us the ability to take that color out completely, and make it look like all other budgets? I'm a self-diagnosed OCD-person when it comes to computer-related stuff. And seeing some budgets in yellow for no reason at all (or no reason to me -- there's a technical reason) drives me absolutely crazy.

@rhealitycheck this is an interesting request and I think we have a similar request in our Trello here which is where we track feature requests though if it gets upvoted and built I really caution the use of this feature.
I would suggest lowering the goal for the particular month or removing the goal entirely and optionally setting it back up in the next month.
That said, I do think a "soft goal" feature would be really cool to add (goals have been really difficult for us to add ourselves so we'd likely need YNAB to do this). The idea of a soft goal would basically be a goal where you want tracking but meeting the goal isn't mandatory. This could cause some bad budgeting behaviors but would be an interesting feature nonetheless.


Does it has commonjs version?

some old project use commonjs, if upgrade to esm would cost a lot of time. does it has commonjs version?

See #35

Basically, I'm not strictly against a commonJS version but I won't do it myself. I consider it to be old legacy that slowly fades away and I such I will not put effort into it. Well tested PR would be accepted though.

I dont see anything in your code that might cause issues with a commonjs build.
I managed to get a commonjs build just by adding a new tsconfig with compilerOptions.module set to commonjs, setting a different outDir and adding an exports block in package.json (and removing type: module in package.json)


hotfix: duplicate select options

Applies hotfix described in #703 to develop branch

Have merged master into develop 👍


ERROR: Caller thread can't call this function in this node (/root). Use call_deferred() or call_thread_group() instead

Tested versions
Reproducible in Godot_v4.4-dev2_linux.x86_64, and Godot_v3
System information
Linux fedora 38,
Issue description
I download Godot, on the link: https://github.com/godotengine/godot-builds/releases/download/4.4-dev2/Godot_v4.4-dev2_linux.x86_64.zip, I extract the program, I do create the project, then when I edit is error is showed:
ERROR: Caller thread can't call this function in this node (/root). Use call_deferred() or call_thread_group() instead.
   at: propagate_notification (scene/main/node.cpp:2446)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.4.dev2.official (97ef3c837263099faf02d8ebafd6c77c94d2aaba)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /lib64/libc.so.6(+0x3dbb0) [0x7f624994abb0] (??:0)
[2] llvm::CmpInst::Create(llvm::Instruction::OtherOps, llvm::CmpInst::Predicate, llvm::Value*, llvm::Value*, llvm::Twine const&, llvm::Instruction*) (??:0)
[3] [0x7f622026e8c0] (??:0)
-- END OF BACKTRACE --

Steps to reproduce
The images on issue description reproduce the errors.
Minimal reproduction project (MRP)
fb_godot_2.zip

The project is created, but, when I try to edit the project, the error ocurr.

Can you give more details about your system? It sounds like you may be running in a virtual machine (which type?) which doesn't support Vulkan. It then tries to use software rendering from Mesa (llvmpipe) but the version in Fedora 38 is pretty old and is known to be broken.
If that's your situation then it's a duplicate of #82435. You can use the "Compatibility" renderer (OpenGL) instead of "Forward+" or "Mobile" (Vulkan).

The system not is a virtual machine!.
Operating System: Fedora Linux 38
KDE Plasma Version: 5.27.9
KDE Frameworks Version: 5.111.0
Qt Version: 5.15.12
Kernel Version: 6.6.8-100.fc38.x86_64 (64-bit)
Graphics Platform: X11
Processors: 8 × Intel® Core™ i7-2600 CPU @ 3.40GHz
Memory: 19.5 GiB of RAM
Graphics Processor: AMD CYPRESS Ati Radeon 5850
On the mode "Compatibility", the editor IS WORKING but, on the mode "Forward+", the editor IS NOT WORKING.


Updating and testing collections requires manual docker build

Describe the bug
I am trying to follow instruction in readme on this repo and also in public https://kabanero.io/docs/ref/general/collection-building.html.
If I use export IMAGE_REGISTRY_ORG=kabanero then it always seem to be pulling image from docker hub during appsody init so changes to the collection are not reflected even after running below command:
. ./ci/build.sh . incubator/java-microprofile/
To workaround the problem, I updated IMAGE_REGISTRY_ORG=mykabanero so now the appsody init failed as below
appsody init local/java-microprofile
Running appsody init...
Downloading java-microprofile template project from file:///Users/mtamboli/ICPA/champ/inst/collections/ci/assets/incubator.java-microprofile.v0.2.14.templates.default.tar.gz
Download complete. Extracting files from java-microprofile.tar.gz
Setting up the development environment
Running command: docker[pull mykabanero/java-microprofile:0.2]
[Warning] Docker image pull failed: exit status 1
[Warning] The stack init script failed: Failed to run the docker find command: Could not find the image either in docker hub or locally: mykabanero/java-microprofile:0.2
[Warning] Your local IDE may not build properly, but the Appsody container should still work.
[Warning] To try again, resolve the issue then run `appsody init` with no arguments.

So I had to run this command to create the image locally from java-microprofile/image:
docker build -t mykabanero/java-microprofile:0.2 -f Dockerfile-stack .
After that, appsody init pulls updates made to the stack:
appsody init local/java-microprofile
Running appsody init...
Downloading java-microprofile template project from file:///Users/mtamboli/ICPA/champ/inst/collections/ci/assets/incubator.java-microprofile.v0.2.14.templates.default.tar.gz
Download complete. Extracting files from java-microprofile.tar.gz
Setting up the development environment
Running command: docker[pull mykabanero/java-microprofile:0.2]
[Warning] Docker image pull failed: exit status 1
Using local cache for image mykabanero/java-microprofile:0.2
Running command: docker[run --rm --entrypoint /bin/bash mykabanero/java-microprofile:0.2 -c find /project -type f -name .appsody-init.sh]
Extracting project from development environment
[Warning] The stack image does not contain APPSODY_PROJECT_DIR. Using /project
Running command: docker[create --name my-project-extract -v /Users/mtamboli/.m2/repository:/mvn/repository -v /Users/mtamboli/ICPA/champ/inst/mytest/src:/project/user-app/src -v /Users/mtamboli/ICPA/champ/inst/mytest/pom.xml:/project/user-app/pom.xml mykabanero/java-microprofile:0.2]
Running command: docker[cp my-project-extract:/project /Users/mtamboli/.appsody/extract/mytest]
Running command: docker[rm my-project-extract -f]
Project extracted to /Users/mtamboli/ICPA/champ/inst/mytest/.appsody_init
Running command: ./.appsody-init.sh[]
[InitScript] [INFO] Scanning for projects...

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Need to either update document or the script.
Actual behaviour
What is the actual behaviour.
Environment Details (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

If applicable please specify:

CLI version:
Collection you are using:

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@mtamboli   Please can you take a look at this pull request https://github.com/kabanero-io/collections/pull/91 to see whether the updates answer your questions about how to test the docker images locally.
Please let me know if you have further questions so we can update the README.md accordingly.

@groeges I see that the pull request has the new instruction set which will only pull image from docker hub if not present.
export APPSODY_PULL_POLICY=IFNOTPRESENT
But which steps really does the build of the image as part running /ci/build.sh. This is needed if you have changed anything related to stack.
docker build -t kabanero/java-microprofile:0.2 -f Dockerfile-stack .

@mtamboli If you have changed anything relating to the Appsody stacks, specifically the Dockerfile-stack or Dockerfile, then the ./ci/build.sh will rebuild all of the images that have changed. There should be no need to document the exact step in the build script that generates the images, as they should get updated automatically.
So once the images have been updated, using the APPSODY_PULL_POLICY=IFNOTPRESENT environment variable will pull from the local docker registry rather than the remote registry, so the local changes will get picked up. You should not need to update IMAGE_REGISTRY_ORG to get the changes to your docker image picked up.
We are contemplating adding a new environment variable that would allow someone to include additional parameters to be passed to the docker build command. This will allow things like --no-cache to be set to tell the build to not use cache when building the image, which will force the images to be rebuilt. We may need to get this into Appsody rather than into Kabanero.

There was an issue that if you have  / at end . ./ci/build.sh . incubator/java-microprofile/, the docker build is ignored for the stack even if you had APPSODY_PULL_POLICY=IFNOTPRESENT. Opened an issue with appsody to ignore trailing /.
https://github.com/appsody/stacks/issues/379


Add State Of Polytechnic Indramayu To domains

@naneps I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge too many files, most of them do not relate to the organization which you would like to add to this repository. Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Automate the release process

The current release step followed is manual and this needs to be automated

This repository is deprecated.


ref #662: add fix for multilingual message type name lookup

Q
A

Branch
7.0.x

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#662

License
MIT

I've changed the fix to work with older bundles, where german was the default language. The method should be deprecated with implementing https://github.com/chameleon-system/chameleon-system/issues/126

How about the following idea: As there are at most 6 types: use a fixed table mapping from German names to ids (for example "Warnung" => 3)?
If there might be any case with English names (but I don't see how) we might add those to the table as well.

In some projects there are custom message types. The only IDs we know for sure are the old INT values. All other are project dependent, so i would not use a fixed list.

Now there are conflicts after the other merge.

fixed the merge conflict.


Add synthesized breaks to switch statements

The patterns proposal changes how unterminating switch cases are handled:

A long-running annoyance with switch statements is the mandatory break statements at the end of each case body. Dart does not allow fallthrough, so these break statements have no real effect. They exist so that Dart code does not appear to be doing fallthrough to users coming from languages like C that do allow it. That is a high syntactic tax for limited benefit.
...
Therefore, this proposal removes the requirement that each non-empty case body definitely exit. Instead, a non-empty case body implicitly jumps to the end of the switch after completion.

The current plan is to support this by having the front end insert a synthetic break at the end of any switch case that, according to flow analysis, is unterminating.  To do this, we'll need the shared analysis infrastructure to report to the front end when a synthetic break is needed.
I'll update the shared analysis logic accordingly, and I'll update the front end to generate the synthetic break if I can easily figure out how to do so.  Otherwise I'll reassign the task to @johnniwinther after I've completed the shared analysis work.

Shared analysis work is complete.  Turning over to @johnniwinther to implement the front end part.


[Example] Missing conf['class'] when running flask example

Hi, I'm playing with oidc-op and flask.
I'm also working on a basic Readme to help users with preliminar setup.
Running  python -m flask_op.server flask_op/conf.yaml  I get the following exception.
Am I doing something wrong?
Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "~/DEV3/oidc-op/flask_op/server.py", line 34, in <module>
    main(args.config)
  File "~/DEV3/oidc-op/flask_op/server.py", line 18, in main
    app = oidc_provider_init_app(config, 'oidc_op')
  File "~/DEV3/oidc-op/flask_op/application.py", line 56, in oidc_provider_init_app
    app.endpoint_context = init_oidc_op_endpoints(app)
  File "~/DEV3/oidc-op/flask_op/application.py", line 30, in init_oidc_op_endpoints
    cwd=folder)
  File "~/DEV3/Django-Identity.env/lib/python3.5/site-packages/oidcendpoint/endpoint_context.py", line 190, in __init__
    self.cookie_dealer = init_service(_conf)
  File "~/DEV3/Django-Identity.env/lib/python3.5/site-packages/oidcendpoint/endpoint_context.py", line 102, in init_service
    if isinstance(conf['class'], str):
KeyError: 'class'

Fixed with latests commits, thank you


Document Asynchronous Configurations in the Config File's Docs

A follow up to #3733 and #3762
Updated to use async/await per review feedback.

@devoto13 https://github.com/karma-runner/karma/pull/3762#issuecomment-1091442267
@jginsburgn https://github.com/karma-runner/karma/pull/3762#issuecomment-1092378702

Hey @npetruzzelli !
Thanks for your PR.
Async configurations are already documented as per https://github.com/karma-runner/karma/pull/3761.
I am closing this PR, but please tell me if I am missing something.

@npetruzzelli ack. Thanks for your contributions :)


File-Exporer isn't showing all files, if they have the same name

VSCode Version: 1.25.1
OS Version: Windows 10.0.17134

Steps to Reproduce:

And files to on directory with the same name, but different letter-casing
Only one file is shown in VS-File-explorer

The Main.js was created accidently, but isn't shown anywhere in vs-code

Does this issue occur when all extensions are disabled?: Yes

@Maqsyo how do you manage to have different file casing on Windows, is this a specially formatted drive?

It was just luck :D
i've created a main.js on my own, but the phaser-editor created the Main.js in the same folder. I think it's the fault of java

@Maqsyo a normal windows filesystem (NTFS) does imho not allow to create the same file with different casing on disk, that is why I was wondering. At least it is very uncommon to have this on Windows where most tools assume this does not happen.
VSCode is not supporting same-file-different-case on any OS but Linux where this is standard.

Closing this issue as there is nothing to be done from our side as per @bpasero's comment above.


In case of error, capture if there is no goals and succeed instead

This does not solve the issue described here: https://leanprover.zulipchat.com/#narrow/stream/239415-metaprogramming-.2F.20tactics/topic/delayed_abstraction.20meta-variables/near/242328447
But this is a step toward it with better error messages and a cleaner structure. Merging.


associated gene(s) on strain page

I first went to my sadA- strain and was happy to have an active associated gene link
https://dictycr.org/stockcenter/strains/DBS0236921
But any other strain, available or not, with one gene associated or more, they all don't have it
https://dictycr.org/stockcenter/strains/DBS0351344
https://dictycr.org/stockcenter/strains/DBS0350344

More examples from #28:
https://eric.dictybase.dev/stockcenter/strains/DBS0351367
https://eric.dictybase.dev/stockcenter/strains/DBS0351356
https://eric.dictybase.dev/stockcenter/strains/DBS0351312
https://eric.dictybase.dev/stockcenter/strains/DBS0351283
https://eric.dictybase.dev/stockcenter/strains/DBS0351270

this is the older issue: https://github.com/dictyBase/dicty-stock-center/issues/27

OLasmid detail pages of course should have a gene link when possible. Both should also be curatable in the future


Fix HTML used in markdown

You can check the result here (to be noted: github discard inline style).
It will very likely fix crates.io display by adding a <code> inside the <pre>:

Thank you!


Jamaica Localization is not available in AppSource

Hi Team unable to find App source apps as per the documentation.
The documentation says that Jamaica localization is available in the App source by the partner but there is no app available as such.

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

ID: 360d9409-840a-5ef9-9aa4-b43ed4733e2c
Version Independent ID: 961502fa-6d30-47fa-8afe-a15a8e521304
Content: Country/region availability and supported languages - Business Central
Content Source: dev-itpro/compliance/apptest-countries-and-translations.md
Service: dynamics365-business-central
GitHub Login: @sorenfriisalexandersen
Microsoft Alias: altotovi

Hi,
Jamaica is enabled as a country, but so far, no partner built specific localization app for AppSource. Potential reason is that W1 is enough as a localization there. But even in this situation, if only some small parts are missing, every partner worldwide is enabled to make this smaller or bigger app.


Remove ZoomControls on Android

FEATURE REQUEST / SUGGESTION
Remove the ZoomControls on Android, they are basically never needed since Android startet to support multitouch and pinch zoom.
Alternatively add property for ZoomControls
Today the only way to hide them is to turn off zooming in map but the side effect is that the map cannot be zoomed.

Hi @lindblomc ,
Is this same as #180 ?

Hi,
Ah yes, it seems to be exactly the same issue.
Best regards
/Claes
From: amay077<EMAIL_ADDRESS>Reply-To: "amay077/Xamarin.Forms.GoogleMaps"<EMAIL_ADDRESS>Date: Thursday, 2 February 2017 at 11:36
To: "amay077/Xamarin.Forms.GoogleMaps"<EMAIL_ADDRESS>Cc: Claes Lindblom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [amay077/Xamarin.Forms.GoogleMaps] Remove ZoomControls on Android (#198)
Hi @lindblomchttps://github.com/lindblomc ,
Is this same as #180https://github.com/amay077/Xamarin.Forms.GoogleMaps/issues/180 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/amay077/Xamarin.Forms.GoogleMaps/issues/198#issuecomment-276922779, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APk_ragE-uAliV9V5VD-NOtTqve-cQ4rks5rYbG4gaJpZM4L01cc.

Alright, If you have any comments, please join #180 .
I closed it.


NSFS | Improve list objects performance on top of NS FS (PR 2/3)

Explain the changes

This is Second PR for listing unordered objects.
This PR contains changes in list_object(), TellDir value is passed to next list object result and same value is used to seekdir dir location.
Global flag is added in config for controlling bucket list flow.
NB : New parameter next_pos added to API list_objects to pass the seek position. I tried to pass this along with next_marker as a object but Ceph test failed when comparing next_marker.

Issues: Fixed #xxx / Gap #xxx

https://github.com/noobaa/noobaa-core/issues/6615

Testing Instructions:

Run ListObject API after enabling global property ALLOW_NSFS_UNSORTED_OBJECTS_LIST, Result should retun with unsorted list.

[ ] Doc added/updated
[X] Tests added

Hi @naveenpaul1,
I found that when running the tests on MacOS some of them don't pass:
I tried: npx jest test_list_object.test.js
I see:

manage unsorted list objcts flow
Unsorted List objects
✓ List object unsorted with one object upload (12 ms)
✓ List object unsorted with multiple object upload (671 ms)
Telldir and Seekdir implementation
✓ telldir returns bigint (1 ms)
✓ seekdir expects bigint (13 ms)
✕ list dir files - telldir and seekdir. (2 ms)
✕ list dir files -  Dir.read() and seekdir() (1 ms)
✕ list 10000 dir files - telldir and seekdir (1074 ms)

Details:

● manage unsorted list objcts flow › Telldir and Seekdir implementation › list dir files - telldir and seekdir.
expect(received).toBe(expected) // Object.is equality

Expected: 5
Received: 8

  134 |             }
  135 |             //total number of dir and files inside list_dir_root is 5
> 136 |             expect(total_dir_entries).toBe(total_files + 1);
      |                                       ^
  137 |         });
  138 |
  139 |         it('list dir files -  Dir.read() and seekdir()', async () => {

  at Object.toBe (src/test/unit_tests/jest_tests/test_list_object.test.js:136:39)

● manage unsorted list objcts flow › Telldir and Seekdir implementation › list dir files -  Dir.read() and seekdir()
expect(received).toBe(expected) // Object.is equality

Expected: 0n
Received: 1n

  148 |                 const tell_dir = await dir_handle.telldir(DEFAULT_FS_CONFIG);
  149 |                 //verify tell_dir and dir_entry.off return same value
> 150 |                 expect(tell_dir).toBe(dir_entry.off);
      |                                  ^
  151 |                 dir_marker = {
  152 |                     dir_path: list_dir_root,
  153 |                     pos: dir_entry.off,

  at Object.toBe (src/test/unit_tests/jest_tests/test_list_object.test.js:150:34)

● manage unsorted list objcts flow › Telldir and Seekdir implementation › list 10000 dir files - telldir and seekdir
expect(received).toBe(expected) // Object.is equality

Expected: 10000
Received: 10501

  208 |             }
  209 |             //total number of dir and files inside list_dir_root is 5
> 210 |             expect(total_dir_entries).toBe(10000);
      |                                       ^
  211 |         }, 10000);
  212 |     });
  213 | });

  at Object.toBe (src/test/unit_tests/jest_tests/test_list_object.test.js:210:39)

@shirady could you please try the test after running
npm run build:native


fix: Repair the paging logic of the three pages of Home, Watch and Favorites

Home and Watch
The pagination parameter changes, the condition is the id of the last item on the previous page
Favorites
The pagination parameters are changed, the condition is the id and adding time of the last item on the previous page

And husky run failed,I don't think he is important.
Commits I made by bypassing checks.:dizzy_face:
><EMAIL_ADDRESS>type:check /Users/sowevo/git/self/e-hentai-view
> tsc --noEmit

pages/_app.tsx:76:16 - error TS2786: 'Component' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.
    Type 'Component<any, any, any>' is not assignable to type 'Element | ElementClass | null'.
      Type 'Component<any, any, any>' is not assignable to type 'ElementClass'.
        The types returned by 'render()' are incompatible between these types.
          Type 'React.ReactNode' is not assignable to type 'import("/Users/sowevo/git/self/e-hentai-view/node_modules/@types/react-transition-group/node_modules/@types/react/ts5.0/index").ReactNode'.
            Type '{}' is not assignable to type 'ReactNode'.

76               <Component {...pageProps} />
                  ~~~~~~~~~

pages/_document.tsx:67:55 - error TS2786: 'App' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<AppPropsType<any, {}>, any, any> | null' is not a valid JSX element.
    Type 'Component<AppPropsType<any, {}>, any, any>' is not assignable to type 'Element | ElementClass | null'.
      Type 'Component<AppPropsType<any, {}>, any, any>' is not assignable to type 'ElementClass'.
        The types returned by 'render()' are incompatible between these types.
          Type 'React.ReactNode' is not assignable to type 'import("/Users/sowevo/git/self/e-hentai-view/node_modules/@types/react-transition-group/node_modules/@types/react/ts5.0/index").ReactNode'.

67       enhanceApp: (App) => (props) => sheets.collect(<App {...props} />),
                                                         ~~~

pages/favorites.tsx:49:10 - error TS2786: 'SwipeableDrawer' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

49         <SwipeableDrawer
            ~~~~~~~~~~~~~~~

src/components/Layout.tsx:138:14 - error TS2786: 'Hidden' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

138             <Hidden smDown>
                 ~~~~~~

src/widgets/detail/FavIconButton.tsx:97:10 - error TS2786: 'SwipeableDrawer' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

97         <SwipeableDrawer
            ~~~~~~~~~~~~~~~

src/widgets/detail/Info.tsx:119:8 - error TS2786: 'Hidden' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

119       <Hidden mdDown>
           ~~~~~~

src/widgets/detail/Info.tsx:135:12 - error TS2786: 'Hidden' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

135           <Hidden smUp>
               ~~~~~~

src/widgets/detail/Info.tsx:142:14 - error TS2786: 'Hidden' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

142             <Hidden smUp>
                 ~~~~~~

src/widgets/detail/Info.tsx:159:12 - error TS2786: 'Hidden' cannot be used as a JSX component.
  Its element type 'ReactElement<any, any> | Component<any, any, any> | null' is not a valid JSX element.

159           <Hidden smDown>
               ~~~~~~

Found 9 errors in 6 files.

Errors  Files
     1  pages/_app.tsx:76
     1  pages/_document.tsx:67
     1  pages/favorites.tsx:49
     1  src/components/Layout.tsx:138
     1  src/widgets/detail/FavIconButton.tsx:97
     4  src/widgets/detail/Info.tsx:119
 ELIFECYCLE  Command failed with exit code 2.


feat: Enhance XMTP protocol contracts

Summary by CodeRabbit
Release Notes

New Features

Introduced upgradeable contracts for GroupMessages and IdentityUpdates, enhancing functionality and security.
Added deployment scripts for both GroupMessages and IdentityUpdates, facilitating easier contract management.
New JSON output files created for tracking deployment details of GroupMessages and IdentityUpdates.

Bug Fixes

Updated test cases to ensure proper handling of payload sizes and contract states during various operations.

Documentation

Revised README.md to reflect new features and usage instructions related to the XMTP Contracts.

Chores

Updated environment variables in .env and local.env for better configuration management.
Cleaned up .gitignore to exclude debug files.

other than a few nits here and there it seems fine. I haven't reviewed the .sol changes since its not within my wheelhouse.


WIP initial mobile client view on overview screen

@sedroche here are my changes so far

Trying it out
set up the minishift server
target the minishift server with this branch
create a mobile client
It should show up on the overview
expand the client you should see an option for browsing the catalog
delete the mobile client (should alert and remove the client)


we are showing the android placeholder icon on the snack bar after save

What

we are showing the android placeholder icon on the snack bar after save
I was expecting the Open Food Facts icon

Part of

#4166

Screenshot/Mockup/Before-After

Hello,
It seems that this issue is not app related, but a bug on Android itself: https://stackoverflow.com/questions/69630124/how-to-configure-toast-icon-in-android-12
@teolemon As explained on Stack Overflow, is-it problem still reproducible after a reboot?


Apply fix to handle int 0 as a valid key

Last 3 commits are mine, the rest came from the package after your fork was created.

Can you link the asana task in the description 🙏

Have you considered just replacing this package? I feel like it's not doing anything complicated...

Can you link the asana task in the description 🙏

Done,  Also we are not in crmv2, once this is merged I intended to PR into crmv2 with the updates to composer.json and do a bunch of browser based QA.

Have you considered just replacing this package? I feel like it's not doing anything complicated...

When you say replace, do you mean with an alternative public package or just permanently fork?
I think there a decent amount going on that I don't have a huge desire to re-write from scratch ATM.
If we were to permanently fork I would start with writing some tests (literally no tests right now), and probably tweak how validation is displayed (specifically if you are viewing english and validation fires on another locale it it is not apparent where the issue lies...)

Have you considered just replacing this package? I feel like it's not doing anything complicated...

When you say replace, do you mean with an alternative public package or just permanently fork?
I think there a decent amount going on that I don't have a huge desire to re-write from scratch ATM.
If we were to permanently fork I would start with writing some tests (literally no tests right now), and probably tweak how validation is displayed (specifically if you are viewing english and validation fires on another locale it it is not apparent where the issue lies...)

I was suggesting just throwing it out, but permanently fork would prob. be a safer option.
I'd really like good test coverage b/c reviewing this is ...tricky.

OK, if you are good to wait a bit longer I will start moving towards a permanent fork:
My plan would be:

Write tests and wire CI to run them
Tighten up types / make things more in line with our standards

@robertmarney Sounds good.
Not #1 priority, but please pursue as you have time 🙏


How to hide the id attribute when generating rules?

Dear,
First of all, I am very grateful to the online generator that provides the json schema, which is really very hard to design!
However, I have encountered a problem in use, please help me solve the problem, the problem is as follows:
When generating the schema specification, how do you not generate attributes with $id?
example:
{
"definitions": {},
"$schema": "http://json-schema.org/draft-07/schema#",
"type": "object",
"properties": {
"username": {
"type": "string"
}
}
}

Select "None" as ID type from the menu on the LHS, then generate you schema


Incremental Deployment seems to not work when factory file does not exist

Hello,
I was testing situation when there is no factory file and IncrementalDeployment and DeployGlobalParams are True.

View on the Repos:

I would expect that new file will be created based on:

But I'm receiving the following error:

Best regards
Marta

I can't reproduce the issue:

Check this example:
https://github.com/Azure-Player/azure.datafactory.tools/blob/6844dee2f81264cb1e9a345970c42e5ef890299f/adhoc/issue-195/test-195.ps1#L14C1-L27C1

Hmm but please correct me if I'm wrong, you actually have a folder called "factory", but it's empty. In this case you check in your code if there are no files, then create a new one and proceed with incremental deployment.
In my case I don't have a folder at all in the repository. I can't avoid that, because even if I create the empty folder and later build the artifact, this folder will not be present, it just disappears. I checked it once again and still the same issue:

I'm not sure what else I'm missing.

Got'ya! Yes, I had a folder but no file. The error is when you don't have a folder. ok.

Hi :)
I have one additional question regarding this bug.
Let's assume it is fixed. If a new file is created to keep information about the incremental deployment global parameter and there are already existing global parameters in the ADF instance. Will they be removed/overwritten after deployment or global parameter for incremental deployment will be just added without touching existing parameters?

All global parameters are being deployed as a one consistent object, so will be overwritten.