fix: UTF-8 chars "outside" ASCII will set content-type to application/octet-stream

Description
When returning a Response with a body that contains a non-ASCII character, like "ö, ä, ü, etc." in german or a 😊 for that matter, the responses content-type will be set to application/octet-stream.
That in turn will result in "save file" dialog if the route was called in a browser.
Steps To Reproduce

create new dart frog project
add a utf-8 character that isn't in ASCII as well to the Response body in index.dart
run dev server
call in Browser

Expected Behavior
Getting a text/plain response (with utf-8 encoding) when using utf-8 character in a Response body String.
Additional Context
My apologies if this is expected behavior.
This basically doesn't matter that much if one would use dart frog for an API or just set the header manually, but it took me quite some time to figure out what is wrong when doing the tutorials.

Hi @DanielBachmann 👋 Thanks for opening up this issue.
Would you be able to upload a minimal reproductive project with this setup so we can take a look at it locally? From the details that you described here the team believes that this is expected behavior. We think this is actually being driven by the default behavior of shelf in that if it sees something in the response body that doesn't fit with utf-8 encoding then it will default back to the application/octet-stream response type.

Hi @tomarra
Sure, I created a new project, added the route /with_umlaut by adding /routes/with_umlaut.dart and zipped it before the first start with dart_frog dev.
It shows the described behavior on the /with_umlaut route.
Attached of course :-)
utf8_test.zip


Viewing real errors

Hi, I really love this extension, but one thing I'm struggling with is seeing the actual errors associated with code instead of where they are. E.g

If there is an obvious option i'm missing, sorry!

Fixed, duplicate of https://github.com/DonJayamanne/typescript-notebook/issues/51


diepAPI.apis.scaling.toCanvasPos() not perfect

the function works perfectly when the player stands still.
Moving seems to shift the camera, which also shifts the calculated canvasPos
https://user-images.githubusercontent.com/30176357/194759199-cba5f476-d212-4173-a8be-94f65263ee63.mp4

After some more testing, I can reassure that the issue is not in scaling.toCanvasPos().
I suspect the root cause to be here somewhere: https://github.com/Cazka/diepAPI/blob/main/src/extensions/entity_manager.ts


Problems with constrained strings

Checks
Bug
The code provided below results in several errors:
python pydantictest.py
Traceback (most recent call last):
  File "/home/ptrxyz/scratch/pydantictest.py", line 21, in <module>
    ms = MyStuff(dept=MYLITERAL, dept2=MYLITERAL, dept3=MYLITERAL)
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 2 validation errors for MyStuff
dept
  ensure this value has at most 6 characters (type=value_error.any_str.max_length; limit_value=6)
dept2
  ensure this value has at most 6 characters (type=value_error.any_str.max_length; limit_value=6)

all three should do the same: enforce a length of 6 characters and strip whitespaces. However the results are different, since the last paramter (dept3) does not trigger a validation error.
I don't know if it's by design, but I would assume that any mutations, like strip_whitespace, would be applied before any checks, like the length checks here. If so, then the first two should not trigger a validation error, if not, then the last one should. Either way it's inconsistent and one of them is probably wrong.
Pylance validation fails with the third parameter, resulting in a GeneralTypeIssue:

Is there any workarounds/fixed for that, besides turning off pylance?

Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.9.1
            pydantic compiled: True
                 install path: /home/ptrxyz/.cache/pypoetry/virtualenvs/dfnt-account-service-Prn8tZhj-py3.10/lib/python3.10/site-packages/pydantic
               python version: 3.10.5 (main, Jun  6 2022, 18:49:26) [GCC 12.1.0]
                     platform: Linux-5.18.12-zen1-1-zen-x86_64-with-glibc2.35
     optional deps. installed: ['typing-extensions']

from typing import Annotated
from pydantic import BaseModel, Field, ConstrainedStr

MinMaxString = Annotated[str, Field(min_length=6, max_length=6, strip_whitespace=True)]

class CSTR(ConstrainedStr):
    min_length: int = 6
    max_length: int = 6
    strip_whitespace: bool = True

class MyStuff(BaseModel):
    dept: str = Field(..., min_length=6, max_length=6, strip_whitespace=True)
    dept2: MinMaxString
    dept3: CSTR

MYLITERAL = "123456 "
ms = MyStuff(dept=MYLITERAL, dept2=MYLITERAL, dept3=MYLITERAL)

This is not a bug, it's a question, we have the discussion forum for questions.
Also, if you are creating a bug, please have the courtesy to fill in the template properly.
No, there's no way to get pylance to understand dept3 AFAIK, but perhaps you should try asking the pylance team?

Oh, sorry for that, I tried to stick to the template, I added the checklist in the beginning again.
However, my bug report is about the inconsistency about how strip_whitspace and length checks are handled, the points about Pylance are indeed simply questions I thought that might be helpful.

Ye, the checks are done in a different order I'm afraid.
This will be fixed in V2, i doubt it's worth fixing before then, the plan i to get to V2 as soon as possible.

Ok, thanks! For anyone passing by in the meanwhile, I got this as a workaround until then:
(pylance, mypy are happy, as well as stripping the whitepsaces and length checks are in order and json_schema are working)
class CSTR(ConstrainedStr):
    min_length: int = 6
    max_length: int = 6
    strip_whitespace: bool = True

class MyStuff(BaseModel):
    test: CSTR | str = Field(..., min_length=6, max_length=6, strip_whitespace=True)

ms = MyStuff(test="123456")         # works, ms.test="123456"
ms = MyStuff(test=" 123456   ")     # works, ms.test="123456", whitespaces stripped before length check
ms = MyStuff(test="12345")          # fails, Exception indicates that ms.test is too short
ms = MyStuff(test="1234567")        # fails, Exception indicates that ms.test is too long
ms.schema_json()                    # works, however ms.test is defined as anyOf of two identical types


Support for making hard links in DSGUI

NOT ALL duplicate files must be deleted, they may be part of different versions.
I would like the DSGUI search results to allow replacement of duplicate files with NTFS hard or symbolic links. That would save space, too!
Thank you for developing this such a great tool!

Hey @lenny20! Thank you for the suggestion! It's been a long time since I worked on this project, but I intend to work on your suggestion. I'll let you know when the next release is ready for you to try.

Thank you very much for your dedication!


MM-Task-Admin-App -Sprint 2

Description:

Build the Screen to match the Figma design.
Integrate Screen with the Backend

@askinss I'm interested in working on this task as well. As I found out that it relates to the other 3 issues.


Update BYLAWS.md for clarification of code-change policy

The code change policy contains phrasing that sounds contradictory to me.
The sentence: "The code can be committed after the first +1" has been removed.
NOTE: This pull request should not be merged until a successful VOTE for adoption has taken place.

+1

+1

@knusbaum, would you put this to a VOTE so we can make a decision on merging it in.


Implement requirement rule CPMR0029 - Package Id Does Not End With .config

:tada: This issue has been resolved in version 0.1.0 :tada:
The release is available on:

GitHub Release
Chocolatey Package

Your GitReleaseManager bot :package: :rocket:


Using ESC key doesn't call onEnd/onSkip callbacks

Describe the bug
While the buttons call the onEnd/onSkip callbacks just fine, no callback is used when the ESC key is used.
To Reproduce
Steps to reproduce the behavior:

Specify onEnd and onSkip callbacks
Press ESC key

Expected behavior
I'd expect that one of the callbacks is fired to inform me that the tour has ended.
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome 92.0.4515.107
Version Vue-Tour 2.0.0

@mmorainville  Is this already being worked on or can i take this up?

Hi @m-mohr, @nyfer,
This is not being worked on at the moment, but I wonder if it's really a bug actually.
The end callback is actually called onStop and is already called when pressing the ESC key.
That can be seen in the VTour component:
    stop () {
      this.customCallbacks.onStop()
      document.body.classList.remove('v-tour--active')
      this.currentStep = -1
    },
    ...
    handleKeyup (e) {
      ...
      switch (e.keyCode) {
        ...
        case KEYS.ESCAPE:
          this.isKeyEnabled('escape') && this.stop()
          break
      }
    },

And pressing ESC key really stops the tour, it doesn't skip it, so I don't think it's conceptually correct to make it trigger the onSkip callback.
I added the onStop callback in the documentation, it was indeed not documented though, sorry for that.
https://github.com/pulsardev/vue-tour/wiki/Callbacks

Thanks for your thoughts. ESC ends or skips the tour (not sure), so I'd expect to either have an onEnd or onSkip event fired (or a new onEsc). Not sure which fits better, but one of them should be fired. I need to react on when the tour "disappears" and ESC right now results in a "bug" on my side.

Ah, now I see that there's a new "onStop" in the docs now, wasn't aware of that. I was confusing it with "onFinish", which I used before. Alright that works for me, thanks!


Javascript API: FileSystem operations over a FileDescriptor

Is your feature request related to a problem? Please describe.
It could be nice to have an API to work with FileDescriptors like in Node.js.
Just the principal operations:
fs.open(path, { flags, mode })
fs.read(fd, buffer, { offset, length, position })
fs.write(fd, buffer, { offset, length, position })
fs.close(fd)
With this API we can create performant ways to operate with random data inside a file.

We could use https://docs.rs/filedescriptor/0.8.0/filedescriptor/ here.

closing in favor of #2233


📖 Fix escaping in documentation

An issue has been introduce on a last PR when generating documentation, the escaping of wrong character used by docusaurus is no longer working.
Try to fix this, I hope it's work on all environments, could you please check on your environment if it's working.

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/okp4_objectarium.wasm
338.24 KB (0%)

target/wasm32-unknown-unknown/release/okp4_law_stone.wasm
640.47 KB (0%)

target/wasm32-unknown-unknown/release/okp4_cognitarium.wasm
213.09 KB (0%)

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Document how to use specific trust server

Describes how a user can use a different trust server for a repository.

@liamawhite I obviously don't have access to the IBM internal docs so I can't change that. Do you want to keep the image policy example in the readme?

I obviously don't have access to the IBM internal docs so I can't change that. Do you want to keep the image policy example in the readme?

FYI, you can actually submit edits to the IBM Cloud docs through Github - there's an "Edit in Github" link at the top of each topic. The security enforcement ones are here: https://github.com/IBM-Bluemix-Docs/Registry/blob/master/registry_security_enforce.md.
That said, we manage the rollout of the IBM built Container Image Security Enforcement tool separately from Portieris, since it's shipped as a built chart in a repo rather than installed from Github. If you do open a PR against the docs repo, please could you @ me in it so that I can coordinate with our doc writers and make sure the docs accurately reflect what's available in our chart?


[Bug Report] MessageBox.alert can be confirmed to close by the enter when mouse hasn't clicked page

Element UI version
2.8.2
OS/Browsers version
win 10 / chrome 74
Vue version
2.6.10
Reproduction Link
https://codepen.io/lvjiaxuan/pen/OYJNYe
Steps to reproduce
e:
step 1: open the MessageBox.alert and do not use the mouse to click after MessageBox opened.
step 2: click on the enter and MessageBox closes.
step 3: open MessageBox again.
step 4: click anywhere and then can not close the MessageBox by the enter.
c:
步骤 1: 打开MessageBox.alert并且不要产生鼠标点击
步骤 2: 点击回车MessageBox关闭
步骤 3: 重新打开MessageBox.alert
步骤 4: 点击页面任何地方，然后点击回车MessageBox不能关闭
What is Expected?
e:
The messagebox show same behavior whether the mouse clicks or not.
c:
希望回车行为在产生鼠标点击前后一致
What is actually happening?
e:
The messagebox's behavior is inconsistent.
c:
回车行为在产生鼠标点击前后不一致

I think this behavior works as expected. By default the focus is on the button. As soon as you click your mouse anywhere in the screen, the focus is not on the button anymore.

I think this behavior works as expected. By default the focus is on the button. As soon as you click your mouse anywhere in the screen, the focus is not on the button anymore.

Correct

messageBox弹窗出现后默认聚焦在按钮上，按钮上绑定了keydown.enter事件，所以会出现描述的情况


WebSocket.on_close never triggerred

Bug Report
In the following example https://github.com/a-alhusaini/ws_test
When I call socket.close in server.cr the on_close handler in client.cr is never called.

In client.cr, you need to call s.run. This handles reading from the websocket (and subsequently calling the on_close handler).


Change Latex to mathjax

LatexCogs is not readable in github dark mode, but mathjax is supported!

Thanks for your contribution, it also simplifies the Readme!


Testnet 49: Pasiphae

Testnet Release
Testnet name: Pasiphae
Release date: 2023-03-27
Testnet release manager: @erwanor
Testnet Release Manager Checklist
Preceding Friday (sprint planning day):

[x] Create GitHub project column, work with team to populate the milestone with tickets targeted for the release.

Tuesday (or after release of previous testnet):

[x] Construct the genesis data for the release: https://github.com/penumbra-zone/penumbra/pull/2256

[x] Create new testnet directory with initial genesis allocations for this testnet by running cd testnets && ./new-testnet.sh

This genesis data will be used for testnet-preview with a randomized version of the future testnet's chain ID.

Thursday:

[x] Check in with team again in a release meeting and update the GitHub milestone to ensure it represents what will make it into the testnet.
[ ] Draft an announcement for peer review to ensure major changes included are comprehensive.

Following Monday (release day):

[ ] Verify that testnet-preview.penumbra.zone is operational; it is redeployed on every push to main, and is an exact preview of what is about to be deployed.
[ ] Create new git tag e.g. 006-orthosie on main (tags created on any other branches will not transfer when merged in) and push to shared remote: git tag -a <tag_name> - must be annotated tag, i.e. git tag -a for Vergen build. This will create a Waiting GitHub Action for deployment.
[ ] You must manually review the Waiting deployment in the GitHub Action UI before the deployment will begin. Monitor the GitHub action to ensure it completes after it is approved.
[ ] Update the User Guide to mention the newly created git tag.
[ ] Delegate to the Penumbra Labs CI validators; use amounts of ~200k penumbra per validator.
[ ] Update Galileo deployment, following docs.
[ ] Make GitHub release object and include the announcement
[ ] Make the announcement to Discord! 🎉🎉🎉

One more: https://github.com/penumbra-zone/penumbra/pull/2268 :slightly_smiling_face:

We're live!

We followed up with 049-pasiphae.1 in order to fix a bug in balance displays: https://github.com/penumbra-zone/penumbra/pull/2278


mature release of FastAPI[Question]

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Hi,
I already used FastAPI for some time. But I still have a confusion in my mind:
Why until now (18.01.2021) there is still not a release with main version number? Is there some Issues that blocking the author to make a "real" mature release with version number higher than 1.0.0?
Thanks for all feedbacks!

Are there any missing features that you expect to see in a "mature" framework?

Are there any missing features that you expect to see in a "mature" framework?

Are there any missing features that you expect to see in a "mature" framework?

until now I think fastAPI is quite good. according to semantic versioning  (Major.Minor.Patch), "The presence of a pre-release tag (-alpha, -beta) indicates substantial risk, as does a major number of zero (0.y.z), which is used to indicate a work-in-progress that may contain any level of potentially breaking changes (highest risk)", citied from wiki

Are there any missing features that you expect to see in a "mature" framework?

until now I think fastAPI is quite good. according to semantic versioning  (Major.Minor.Patch), "The presence of a pre-release tag (-alpha, -beta) indicates substantial risk, as does a major number of zero (0.y.z), which is used to indicate a work-in-progress that may contain any level of potentially breaking changes (highest risk)", citied from wiki

I think there are two things that would be very important:

a major release, with stability warranties.
API docs. Although the tutorials are very extensive, having API docs would be very helpful for people that want build things that are not even considered in the tutorials (api-token comes to mind).

right now having no 1.0.0 feels like things might break at some point (for instance, in the 1.0.0 release) like it happens every once in a while in hugo static site generator.

I think there are two things that would be very important:

a major release, with stability warranties.
API docs. Although the tutorials are very extensive, having API docs would be very helpful for people that want build things that are not even considered in the tutorials (api-token comes to mind).

right now having no 1.0.0 feels like things might break at some point (for instance, in the 1.0.0 release) like it happens every once in a while in hugo static site generator.

Fair points, but i don't think none of the possible changes in the future will be hard to switch to.
FastAPI has a slightly simple syntax, on the other hand we are dependant to two other libraries.

pydantic
starlette

As you can see from our __init__.py file most of our functions are re-exported from Starlette
https://github.com/tiangolo/fastapi/blob/41f291524dd16575f01505f09bb4a85e98676dab/fastapi/__init__.py#L7-L24
This means some of the changes will not be dependant to FastAPI, like it happened it when Starlette deprecated UJSONResponse.
If you take a look at our Release Notes, you 'll find out we had 4 breaking change.

#463 - Updated pydantic to 0.32.2.
#1862 - Removed support for deprecated Pydantic 0.32.2.
#2434 - Added support for shared/top-level parameters-dependencies
#2548 - Upgraded Uvicorn version when installing with all option

As a slightly young framework, we had one "real" breaking change, #2434. Even in that change we deleted more code than we wrote, because it helped us to delete code repetitions.
Most of FastAPI's internal part are wrappers around Starlette, a change to Starlette will not break your code, because FastAPI wraps it up and gives you a more friendly API interface.
That being said, these are my personal thoughts and from my perspective there is nothing to worry about.

Fair points, but i don't think none of the possible changes in the future will be hard to switch to.
FastAPI has a slightly simple syntax, on the other hand we are dependant to two other libraries.

pydantic
starlette

As you can see from our __init__.py file most of our functions are re-exported from Starlette
https://github.com/tiangolo/fastapi/blob/41f291524dd16575f01505f09bb4a85e98676dab/fastapi/__init__.py#L7-L24
This means some of the changes will not be dependant to FastAPI, like it happened it when Starlette deprecated UJSONResponse.
If you take a look at our Release Notes, you 'll find out we had 4 breaking change.

#463 - Updated pydantic to 0.32.2.
#1862 - Removed support for deprecated Pydantic 0.32.2.
#2434 - Added support for shared/top-level parameters-dependencies
#2548 - Upgraded Uvicorn version when installing with all option

As a slightly young framework, we had one "real" breaking change, #2434. Even in that change we deleted more code than we wrote, because it helped us to delete code repetitions.
Most of FastAPI's internal part are wrappers around Starlette, a change to Starlette will not break your code, because FastAPI wraps it up and gives you a more friendly API interface.
That being said, these are my personal thoughts and from my perspective there is nothing to worry about.

we cannot simply say FastAPI is young(although it is comparatively younger than other frameworks). Until now FastAPI has undergone more than 2 years and made 103 Releases, but all of them are iterations on minor or patch versioning. Compared to Tornado, the first release is stable release 1.0.0 one year after appearance. There must be some thoughts under @tiangolo 's mind that he want to keep it as it is. Could you point us out on this confusion? Thanks!

we cannot simply say FastAPI is young(although it is comparatively younger than other frameworks). Until now FastAPI has undergone more than 2 years and made 103 Releases, but all of them are iterations on minor or patch versioning. Compared to Tornado, the first release is stable release 1.0.0 one year after appearance. There must be some thoughts under @tiangolo 's mind that he want to keep it as it is. Could you point us out on this confusion? Thanks!

I can't find the interview he talked about it :cry: Some interview between July and November, a podcast or YouTube video has that answers that...
I'm going to say the way I remember (correct me if I'm wrong @tiangolo). The way I recall, he said that he wants to fix some too complex functions that we currently have, add some features from the backlog and add API documentation (which should be in the backlog). But really, you should wait for his words, I'm just expressing what I think I remember, it was too long ago.
Maybe the admin interface based on OpenAPI was one of the features prerequisite for 1.0, but I'm not sure.

I can't find the interview he talked about it :cry: Some interview between July and November, a podcast or YouTube video has that answers that...
I'm going to say the way I remember (correct me if I'm wrong @tiangolo). The way I recall, he said that he wants to fix some too complex functions that we currently have, add some features from the backlog and add API documentation (which should be in the backlog). But really, you should wait for his words, I'm just expressing what I think I remember, it was too long ago.
Maybe the admin interface based on OpenAPI was one of the features prerequisite for 1.0, but I'm not sure.

I think a better title would be Stable release of FastAPI

I think a better title would be Stable release of FastAPI

I'd like to add that not having a stable 1.0.0 release of FastAPI makes it difficult to include FastAPI as a dependency to other libraries. If you are using tools like poetry to manage dependency versions you end up locking your users into very specific FastAPI versions when there isn't a major release. For example:
Util Library
-> depends on FastAPI ^0.65
Application
-> depends on Util Library
Your application is now locked into 0.65 >= FastAPI <= 0.66 which is pretty limited.


Segmentation fault (core dumped) on Manjaro

Hey everybody, simply scaffolding a new project I receive a segmentation fault error on Manjaro Segmentation fault (core dumped). I've tried also on my MacOS machine but everything looks fine (same deno version).
By the network tab I saw that it returns a 200 code for the first ALIVE request and then fails. The server doesn't return the counter client side JS but just display the DOM elements (same with or without twind).
The error occur on both dev and prod environments.
I'm not sure if the issue is about fresh or the entire deno runtime.
Following my specs.
Deno: 1.24.3
OS: Manjaro Linux x86_64
Kernel: 5.15.48-1-MANJARO

The same thing was occurred.
# deno --version
deno 1.24.3 (release, x86_64-unknown-linux-gnu)
v8 <IP_ADDRESS>
typescript 4.7.4

# cat /etc/lsb-release
DISTRIB_ID=ManjaroLinux
DISTRIB_RELEASE=21.3.7
DISTRIB_CODENAME=Ruah
DISTRIB_DESCRIPTION="Manjaro Linux"

# uname -r
5.19.1-3-MANJARO

# cat -p import_map.json
{
  "imports": {
    "$fresh/": "https://raw.githubusercontent.com/lucacasonato/fresh/main/",
    "preact"<EMAIL_ADDRESS>    "preact/"<EMAIL_ADDRESS>    "preact-render-to-string": "https://esm.sh/preact-render-to-string@5.2.1?external=preact",
    "twind"<EMAIL_ADDRESS>    "twind/"<EMAIL_ADDRESS>  }
}

This sounds like a Deno issue. Could you provide more details on when this happens?


TRSv2-588: Upgrade SQLParse

Codecov Report
Patch and project coverage have no change.

Comparison is base (5af91a0) 55.09% compared to head (8749f57) 55.09%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #378   +/-   ##
========================================
  Coverage    55.09%   55.09%
========================================
  Files          209      209
  Lines        12199    12199
  Branches      1588     1588
========================================
  Hits          6721     6721
  Misses        5478     5478

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update adapter creation guide

@coorasse I reopened this pr as there was an issue with travis.
Should pass now

There are some failures because activerecord-edge now requires Ruby >= 2.7

Any more adaptions needed?

@Liberatys can I kindly ask you to rebase once more? The file changed name, is now model_adapter.md. Thanks 🙏

Sure thing !


Update README and filename

Hi @hassox, couple of changes for your review:

Update the README to use Elixir syntax highlighting for example code
Couple of small aesthetic README changes: Title, subtitle, and GitHub capitalization
Renamed lib/ueber_github.ex to lib/ueberauth_github.ex to match the project naming.
Renamed test/ueber_github_test.exs to test/ueberauth_github_test.exs

Also cleaned up the mix.exs slightly, noticed a typo in the URL.


Unable to get hostnames

Whenever I switch to /#/logs on the dashboard, I get a toast message: Cannot read properties of null (reading 'attributes')
But everything seems to work as expected.
Maybe it's because I'm currently running in WSL?
The settings for errsole are the following:
errsole.initialize({
  appName: 'MyApp',
  storage: new ErrsoleSQLite('./logs/logs.sqlite'),
  serverName: 'localhost',
});

Errsole even logs this error:
TypeError: storageConnection.getHostnames is not a function
    at exports.getHostnames (/api/node_modules/errsole/lib/main/server/controllers/logController.js:176:44)
[...]

@ZebboKlaufix The issue is due to the latest features we added in Errsole. We have released the Errsole v2.10.x package, but the storage package, errsole-sqlite, will be released in a few hours, which will include feature compatibility. I'll update you here once the new storage package is available.

@venki91

fixed in the newest version. Thx!


Use ApiConventions to solve enum serialization on assembly scope

Description
This is a new solution, compare to https://github.com/Altinn/app-lib-dotnet/pull/773.
Here we annotate controllers automatically using apiconventions. This means we do not have to update all enums with an annotation and remember it for future enums, and do the same for enums from storage/platform.
The tradeoff is the scaffolding to make this happen.
Related Issue(s)

#{issue number}

Verification

[ ] Your code builds clean without any errors or warnings
[ ] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Could we have a test that does the inverse as well? Like have some random controller in the test project that doesn't have the same attribute?

Is this still relevant after the additional tests?


Use with Rails 5.2.2?

Is it possible to use this gem with Rails versions other than the one in the RAILS_VERSION file? It seems like only 5.2.0 and 6.0.0 were ever supported.

I see someone else has forked to change the versions, I'll be doing that too.


Cookbook should write to PowerShell ISE profile as well as PowerShell profile

There may be other issues, but the most glaring difference is that the cookbook only puts the profile in as the PowerShell profile, which is 1 of 6 profiles (and not the one used by ISE).
We need to review how profiles are selected, how they work with ISE, and make changes to support PowerShell ISE as a first-class option for installation.

http://stackoverflow.com/questions/29122839/how-to-load-profile-in-powershell-ise


Release 1.0.1 version and prepare release instructions

We have a critical bugfix (#77 #78 ) for maven plugin that will enable it. Need to release new version and update release instructions.

Closed in #96


Cannot get custom domain to apply as part of SAM template

The current SAM template creates a DynamoDB table and a Lambda function. It also creates an implicit API Gateway configuration to expose the lambda function to HTTP traffic.
But so far I've had no luck getting it to apply a custom domain configuration.
I think we need a AWS::Serverless::Api resource to explicitly define the API Gateway configuration, that includes a DomainConfiguration. But I've not been able to find a fully working example that fits with what we've already got, and all my attempts so far have failed.
We can manually apply the custom domain via the console after the template is deployed - which is what I've do so far. But this isn't ideal - it would be good to have the full configuration in the template.

I figured this out with help from Twitter.
This branch contains the changes that will configure API gateway with the custom domain - but requires you to setup the Certs yourself.
https://github.com/flowforge/usage-ping-collector/tree/add-domain-to-sam
On balance, I've decided to keep what we have. If we were to create a new API Gateway, then we'd also have to update the DNS records. That would be doable (I think) with Route53, but as we manage DNS externally, then there's not a lot of benefit automating this part.
Updated the readme via #3 with details.


Change API of CacheProviders to accept suspend functions instead of Deferred

As suggested by Roman Elizarov in https://proandroiddev.com/caching-with-kotlin-coroutines-7d819276c820 comments, I started replacing Deferred with suspend functions

@Dartlexx please resolve condlicts

@IvanSakharovskii Done


LINK-2059 | Check order and payment from Tapla API

Description
Check that order and payment exist in the payment completed page.
Re-route payment completed page to /success and payment cancelled to /cancelled
Closes
LINK-2059

e2e tests result is success for https://linkedregistrations-pr157-ui.dev.hel.ninja :laughing::tada::tada::tada:

LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr157-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is success for https://linkedregistrations-pr157-ui.dev.hel.ninja :laughing::tada::tada::tada:

LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr157-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is success for https://linkedregistrations-pr157-ui.dev.hel.ninja :laughing::tada::tada::tada:

LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr157-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is success for https://linkedregistrations-pr157-ui.dev.hel.ninja :laughing::tada::tada::tada:

LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr157-ui.dev.hel.ninja :rocket::rocket::rocket:

e2e tests result is success for https://linkedregistrations-pr157-ui.dev.hel.ninja :laughing::tada::tada::tada:


feat: add pick and omit options to <AutoTypeTable />

I love the idea of the auto type table, but sometimes the type can be a bit verbose (perhaps nested objects etc) that i'd like to document in a different way. would be nice to have it a bit more configurable

A pick option, that lets you pick what properties of the type should be documented
A omit option, that lets you omit certain properties

For example, let's say we have this type:
export interface MyInterface {
  foo: string;
  bar: number;
  __qux: boolean; // maybe this is an internal prop we don't wanna document
}

We could generate a table with only foo and bar like
<AutoTypeTable name="MyInterface" pick=["foo", "bar"] />
// or
<AutoTypeTable name="MyInterface" omit=["__qux"] />

Perhaps it'd also be nice with an "override" that let's you override what's returned from the compiler, e.g.
type MyRouter = ...;

export interface MyInterface<TEndpoint extends keyof MyRouter> {
  foo: string;
  bar: number;
  qux: TEndpoint;
}

<AutoTypeTable
  name="MyInterface"
  overrides={{
    qux: {
      type: "keyof MyRouter", // Compiler gives TEndpoint which might not be very easy to understand what it is
    }
  }}
/>

Overrides could be seen like plugging in the normal TypeTable into the AutoTypeTable

The omit and pick feature seems kinda pointless to me, you can always use the built-in utilities (Pick<T> etc) instead. I'll prefer an annotation for the override feature.

What do you mean i can use the built in? I need the type to be there on the actual type i just dont want it on the docs.
Are you saying i can do
<AutoTypeTable name="Pick<MyInterface,'foo'>" />

To be clear i dont want to omit the type in the lib, just the docs

Create a props.ts and reimport these types, I want to keep it simple.
export type MyInterface = Pick<Original.MyInterface, 'foo'>

Fair, but then we'd have multiple sources which isn't ideal imo...

Despite omit and pick, overrides will be supported in 10.0.0, with a less-hacky way using tsdoc @remarks tag.

omit will be supported with @internal tsdoc tag soon

Hmm I don't know if I understand with the approach taken here.
Putting these things in jsdoc means end users will see them when browsing the types, which might cause confusion whether or not they're supposed to use that type.
What I wanted was some way on the documentation level decide what I show, there might be some big generic etc that's just too big to document and i want it.
It's your lib and I don't mean to make up the requirements, just wanted to give you my view on the chosen approach

In case it's more recommended to take the initial approach I suggested. Hypothetically, for API types it doesn't make sense to hide any properties unless it's a private field exposed by the API (maybe for some purposes like caching). For a more advanced usage, I'd suggest using utility types instead.


Support for customizing the toplevel module list of a package.

Most likely under the form of an ocamldoc fragment installed by a package. That will be used instead of the processing done by #1.

I think @lpw25 wanted these kind of fragments to have the .mld extension (for "ml doc"). The idea being that at some point we also plan to support things like
(** Some comment blablabla
    {!inline external_doc_file.mld}
    And some more comments. *)

in odoc.

For a pkg PKG. This fragment should be installed as $DOCDIR/$PKG/odig-assets/index.mld

And @trefis suggests that if $DOCDIR/$PKG/odoc-manual/index.ml exists, that one should take over.

Maybe we can even do it without odig-assets at all. That seems simpler.

Implemented on master.
To be clear this only happens if a $(DOCDIR)/$PKG/odoc-pages/index.mld is installed and AFAIK dune based system do that automatically for their packages and index.mld files.
More details can be found in the manual.


bump @nestjs/axios from 0.1.0 to 3.0.0

Bump axios to 3.0.0
adjust tests to axios 3
supersedes: #568

The last commit fixes:
> nest build

src/datasets/dto/update-derived-dataset.dto.ts:16:12 - error TS2612: Property 'pid' will overwrite the base property in 'CreateDerivedDatasetDto'. If this is intentional, add an initializer. Otherwise, add a 'declare' modifier or remove the redundant declaration.

16   readonly pid?: string;
              ~~~
src/datasets/dto/update-raw-dataset.dto.ts:16:12 - error TS2612: Property 'pid' will overwrite the base property in 'CreateRawDatasetDto'. If this is intentional, add an initializer. Otherwise, add a 'declare' modifier or remove the redundant declaration.

16   readonly pid?: string;
              ~~~
src/origdatablocks/dto/create-origdatablock.dto.ts:66:12 - error TS2612: Property 'ownerGroup' will overwrite the base property in 'OwnableDto'. If this is intentional, add an initializer. Otherwise, add a 'declare' modifier or remove the redundant declaration.

66   readonly ownerGroup: string;

To use axios 3, I needed to set the target to es2022 in tsconfig.json and upgrade the containers to node-18.

Still need to find out why the unit and api tests now are hanging ( the e2e test runs successful)

The tests are passing now.

Waiting for a day, to see if the other reviewers wants to provide additional feedback.

@bpedersen2 should we close/delete #568?

Waiting for a day, to see if the other reviewers wants to provide additional feedback.

Yes, that would be good (that's why tried to put some of the more active users in, maybe you can add one from PSI as well?)

@bpedersen2 I'm ready to merge your PR.
Would you be able to solve the conflict on the file package-lock.json?
Thanks

@bpedersen2 I'm ready to merge your PR. Would you be able to solve the conflict on the file package-lock.json? Thanks

Yes, I'll push an update


question: input normalization and expected euclidean distance ranges

i've tried using provided frozen_model from https://github.com/sirius-ai/MobileFaceNet_TF/tree/master/arch/pretrained_model/
converting it to tfjs_graph_model and using in TensorFlow/JS - it appears to work without issues.
and i do have separate preprocessing to crop face from image and even straighten it.
based on utils/data_process.py, input should be in range -0.5..0.5 ?
img = img - 127.5
img = img * 0.0078125

but resulting feature vector contains really small values, up to a point that calculating euclidean distance results in values in range ~0.0025 for a near perfect match and up to ~0.004 for a bad match.
using non-normalized input (range 0..255) increases value of results, but precision is not that great.
but looking through old issues, i found link to https://github.com/sirius-ai/MobileFaceNet_TF/files/3551493/FaceMobileNet192_train_false.zip
which is not documented anywhere, but produces relatively good results with non-normalized inputs?
can you confirm which frozen model should be used, what is best value for input range and what is expected value of resulting euclidean distance between two embeddings for positive and negative matches?

@sirius-ai is this project still maintained?

@sirius-ai is this project still maintained?


How does work "categorical"?

How does work "categorical"?

Hello @jmbertoncelli,
Thanks for giving this a shot! The categorical data preview should be available in 0.2.0.

The feature has been introduced by https://github.com/CermakM/jupyter-datatables/commit/c138b81f5d47a599c02e5b4c9deb3df04fb49436

Closing this issue as 0.2.0 is about to be released. If your problem is not solved by the release, please, re-open the issue or open a new one. Thanks!


GH-2259 changes to PR template, guidelines, and activation of formatting plugins

GitHub issue resolved: #2259
Briefly describe the changes proposed in this PR:

xml formatting check part of automatic validation by CI
xml formatting format part of automatic formatting as part of local build
updated PR template to add links to guidelines and point out formatting conventions
made it a bit clearer which maven goals you can run to make this easier

PR Author Checklist:

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change
[ ] every commit has been signed off

Note: we merge all feature pull requests using squash and merge. See RDF4J git merge strategy for more details.

Other than my comment. Looks good.

Other than my comment. Looks good.

Sorry, which comment is that?


v1 paths under auth only - temp solution

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

However i created an issue for a long term solution using property files:
https://github.com/Activiti/Activiti/issues/1552


[CARBONDATA-2255] Rename the streaming examples

Rename the streaming examples to make it easy to understand

[x] Any interfaces changed?
no

[x] Any backward compatibility impacted?
no

[x] Document update required?
no

[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
the examples don't require
- How it is tested? Please attach test report.
compile it
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
small changes

@QiangCai please modify the appName too when create carbonsession

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3106/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4340/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3129/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/3891/

LGTM


Error: Cannot find module ".", if dynamic "name" parameter is used

I cannot explain this. Say I use html-webpack-plugin like this:
module.exports = function(templateParams) {
  return '<html>..</html>';
};

Now I want to use file-loader. Maybe like this:
module.exports = function(templateParams) {
  return `<!DOCTYPE html>
<html>
<body>
  <script src="${require('file-loader!./de_DE/foo.js')}"></script>
</body>
</html>
`;
};

This works. Let us rewrite it to this:
module.exports = function(templateParams) {
  return `<!DOCTYPE html>
<html>
<body>
  <script src="${require('file-loader?name=de_DE/foo.js!./de_DE/foo.js')}"></script>
</body>
</html>
`;
};

Works, too. Another rewrite:
module.exports = function(templateParams) {
  const locale = 'de_DE';
  return `<!DOCTYPE html>
<html>
<body>
  <script src="${require(`file-loader?name=de_DE/foo.js!./${locale}/foo.js`)}"></script>
</body>
</html>
`;
};

Still works. But now this:
module.exports = function(templateParams) {
  const locale = 'de_DE';
  return `<!DOCTYPE html>
<html>
<body>
  <script src="${require(`file-loader?name=${locale}/foo.js!./${locale}/foo.js`)}"></script>
</body>
</html>
`;
};

ERROR in Template execution failed: Error: Cannot find module "."

I have no idea why this happens. I guess this file is somehow "preprocessed" by webpack?
If I use DefinePlugin to inject locale with process.env it works:
module.exports = function(templateParams) {
  return `<!DOCTYPE html>
<html>
<body>
  <script src="${require(`file-loader?name=${process.env.locale}/foo.js!./${process.env.locale}/foo.js`)}"></script>
</body>
</html>
`;
};

Could you try removing the backticks from the require statement? I heard there were some issues with that.

Tried that, too. Same result. :(

Okay, a workaround could be to add the loader config in your webpack config instead. You could match on test: /foo\.js/.

@donaldpipowitch Is the problem still there? If yes please create minimum reproducible test repo, thanks!

I have the same issue with the new create-react-app setup.
I try to load image dynamically, based on a config file.
This used to work before, with a webpack 1.x setup.
Here's the minimal repro: https://github.com/panta82/requireissue


Please make 1.17.1

The mod is great! But please make one for 1.17.1

The mod is great! But please make one for 1.17.1

https://github.com/19MisterX98/SeedcrackerX/releases/tag/2.9

The mod is great! But please make one for 1.17.1

https://github.com/19MisterX98/SeedcrackerX/releases/tag/2.9

Yeah I found SeedCrackerX that day later. I forgot remove the above comment but Thanks for replying though :D


add rpcx in Go

It seems pull requests has some issues  so I summit this issue.
[RPCX](https://github.com/smallnest/rpcx) - A distributed RPC service framework based on net/rpc like alibaba Dubbo and weibo Motan.

Already merged.
Thank you!


Fixing some bugs so the Dataflow GA version bump doesn't break us.

What test did toString() break.  That seems like a bad test.

It's (most likely) a bad test:
GATKReadAdaptersUnitTest.testGetAndSetBases[2], I think it's getting a null SamRecord (which we don't check for).
We should fix the test and not allow null sams... but not in this PR :)

@davidadamsphd Looks good.  Just that static import.  I think it should probably change back unless it's actually a compile error.   Intelllij incorrectly marks it though.  There's a fix coming in some future version.

What test was depending on toString?  We should maybe update that test...

That test (GATKReadAdaptersUnitTest.testGetAndSetBases) is not a bad test -- it is a unit test for the getBases() and getBasesString() methods in the GATKRead interface. Its test cases include reads with no bases (which is allowed).

Ah, it's because of https://github.com/samtools/htsjdk/issues/297.  But why is the test calling toString()?

It's not calling toString() -- it only calls getBasesString() (since it is unit-testing that method).

Why is it failing when you define toString then?  It must be crashing somewhere in the test suit itself.

Possibly TestNG is calling toString() on the test method parameters when creating its report of test cases that passed / failed, and this triggers the htsjdk bug you linked to?

:+1: Merge when ready.


chore: Update to latest version of GAX

We should already be allowing breaking changes in a beta, which I believe is the only breaking change here... but I'll check.

Ah - it's Google.Cloud.Kms.V1 that's causing the failure here. We should do a release of that carefully, considering whether to do a major bump or not.


Allow retry for OSS risk audit if error is due to timeout

I have set the default number of retries to 3, but obviously you can change this.

Thank you for this PR. While retries are always a good idea I'm hesitant on this one. If you think about it if npm and pypi are timing out after 30 seconds that could mean they are struggling for some reason. If tens of thousands of scans continue to retry instead of breaking then we might cause more issues to the registries.
The idea is that oss risk audit can be performed less frequently perhaps once a day or few times a week. If it doesn't complete due to issues the users can simply retry manually.
Does this make sense?

Yes, this somewhat makes sense, but this is also why it's not a hard-coded limit. This can be set to 1 to keep the existing behavior, as it seems you prefer it that way. Then anyone who chooses to allow retries can edit this environment variable themselves. Especially for projects that aren't huge in terms of dependencies, this seems like a useful option to have.
I can also add a global limit (and by default set it to infinity): each query can retry for up to max_timeout_attempts, except if the overall number of retries exceeds that limit. That way, if too many packages require retries, it can either halt (as is the current behaviour, but where the count also includes other failures) or just cease to retry.
Still, even without this global limit, I think the behavior of retrying is correct. The idea is that if after however many (max_timeout_attempts - 1) retries it still doesn't work, the existing fail counter is incremented. If this happens to multiple packages, it will exceed the limit and halt. If it instead succeeds after some number of retries, we have done the right thing by allowing retries since we were able to audit this package and not contribute to the faliure limit.

It is true that if many queries all fail, even after retries, (for example if npm is down) and you don't have the global limit, then the audit will make more requests (max_timeout_attempts times as many) before halting than if you didn't retry, but setting the global limit equal to the failure limit stops this. In fact, if the global limit is less than the failure limit, this change will actually save network requests before halting.
The situation where you're concerned is also quite an edge case, as evident by the fact that you only had to add the timeout very recently in response to https://github.com/ShiftLeftSecurity/scan-action/issues/27.


Mega Menu

Think I'm missing something related to setting up the mega menu.  Do you have an example of that to look at?  Thanks for your work.  Nicely done :)

Hi,
I discovered a problem and fixed it – a previous modification had broken the mega menu. It is working again now. With this fix I bumped up the version number to 3.6.1. You will need to download this latest version.
Thanks for seeing this error! :-)


.Net Core3.0 HTTP/2 over TLS was not negotiated on an HTTP/2-only endpoint

What version of gRPC and what language are you using?
"Grpc.Tools" Version="1.22.0"
"Grpc.AspNetCore.Server" Version="0.1.22-pre1"
"Google.Protobuf" Version="3.8.0"
"Grpc" Version="1.22.0"
.net core 3.0 / C#
What operating system (Linux, Windows,...) and version?
Windows 10
What runtime / compiler are you using (e.g. python version or version of gcc)
Visual Studio Professional
What did you do?
Problem:
On the server side
Microsoft.AspNetCore.Server.Kestrel[0]
      HTTP/2 over TLS was not negotiated on an HTTP/2-only endpoint.

Server

Client

Attempts to solve:

With .cshtml

@page
@model IndexModel
@{
    ViewData["Title"] = "Home page";
}

Hello, gRPC!

@using Tasly.Service
@using Grpc.Core

<button class="btn btn-primary" onclick="@SayHello()">RUN</button>

@Greeting

@functions {

    private string Greeting;

    async Task SayHello()
    {
        var channel = new Channel("localhost:5001", ChannelCredentials.Insecure);
        var client = new Greeter.GreeterClient(channel);

        var reply = await client.SayHelloAsync(new HelloRequest { Name = "Blazor gRPC Client" });
        Greeting = reply.Message;

        await channel.ShutdownAsync();
    }

}

The same with .razor (but it doesn't work) or I used it incorrectly.

If you have any suggestions about how to fix, or if you need more information, please let me know.
What did you expect to see?
Expect to receive a response message. (e.g. "Hello ...")
What did you see instead?
Microsoft.AspNetCore.Server.Kestrel[0]
HTTP/2 over TLS was not negotiated on an HTTP/2-only endpoint.
).
Anything else we should know about your project / environment?
Please, let me know if you need any additional information!
Best,
Misha

Your server is setup to require HTTPS. ChannelCredentials.Insecure on the Grpc.Core client means it won't use HTTPS.
You have three options:

Turn off HTTPS on the server (note: not good for security)
Use SslCredential with Grpc.Core client
Switch to using grpc-dotnet client. It will automatically use the certificates in your cert store. See tutorial here (note that you should have a BaseAddress that is https instead of http)


Remove special characters from MQTT topics.

Email addresses can contain special characters like the plus sign (+) that cannot appear in MQTT topics. Since the SAIC user name can be an email address and we use this information as part of the MQTT topics, we need to remove special characters.
This has already been done for subscribing to the setter-topics.
However, it's still missing for publishing MQTT messages and in the Home Assistant auto-discovery messages.

I applied these changes manually, and can confirm that it resolves the issue of omitting string sanitation for mqtt topic names. Thank you!
This should be a key part to the 0.6.0 release.


texture update

dump taken from https://github.com/Nick-Riggs/tagpro-texture-packs

This will be updated in #38


Change LinkList removeX methods to return the Node removed.

I wrote these methods to return a boolean to simplify testing; I should have returned a Node object.

[x] Update remove methods to return a Node object.
[x] Update unit test code accordingly.

Build #19 completed successfully.


New command to generate AND show the APP_KEY

I just had to google how to generate AND show the generated APP_KEY. On the current version (v5.1.7) of SnipeIT, just generating the key won't show you the key to put it into the .env file. I had to add --show to the command:
  php artisan key:generate --show

oh I wasn't aware of this! Thanks @dataCore, I'll fix the docs right away


Add debug info on error

Description
Adds additional debug information that can be useful in support requests, including VIP-CLI, Node, and OS versions. It places this debug info below the error message so that someone who begins their text selection on the error message can easily extend their selection down to include it.
> vip ice-cream
Error:  `ice-cream` is not a valid subcommand. See `vip --help`
Debug:  VIP-CLI v2.0.7, Node v14.16.1, darwin 20.4.0

This also introduces more consistency to error output by always prepending "Error:" and always exiting with 1.
Steps to Test

Check out PR.
Run npm run build
Generate an error, e.g., node ./dist/bin/vip ice-cream
Inspect error output

That's a cool feature! 🚢

Thanks for fixing conflicts @dchymko!

@chriszarate np. I'm not really sure what was going on but it was just some changed file paths that were auto-resolved when I merged master in on the cli :shrug:
Thanks for spending the time on this one!


python312Packages.mkdocs-rss-plugin: 1.13.0 -> 1.13.1

Changelog: https://github.com/Guts/mkdocs-rss-plugin/blob/refs/tags/1.13.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 320456 run on x86_64-linux 1

4 packages built:

python311Packages.mkdocs-rss-plugin
python311Packages.mkdocs-rss-plugin.dist
python312Packages.mkdocs-rss-plugin
python312Packages.mkdocs-rss-plugin.dist


Record http.target on trace requests

Otherwise it looks like every request is to /
Fixes VEGA-2077 #patch

Codecov Report
Patch coverage is 30.00% of modified lines.

Files Changed
Coverage

internal/server/server.go
30.00%

:loudspeaker: Thoughts on this report? Let us know!.

Before:

After:


[libs/movex] Add specs for createDispatch() method

Where
libs/movex/src/lib/tools/dispatch.spec.ts
What
Create a new spec file to test the createDispatch() function
Why
The dispatcher is a very important piece of functionality that sits at the core of movex.
Definition of done

it tests a dispatch() fn is returned correctly
it tests the returned dispatch() can makes changes on the given $checkedState Observable
it tests a unsubscribe() fn is returned correctly
it tests that calling the unsubscriber fn stops listening for update changes from the $checkedState Observable
any other tests that are relevant

This is a pretty important and complex issue. I can discuss it further!

Hey can you assign this to me?

Yep! Thank you

Hey, I was just little baffled with using states and checked states for testcases, Can you guide me how should I use Observable and CheckedState for the same?

Sure, I can try to give you a summary.
So, the Observable is simply an object that:

holds a state
provides an API to make changes to the state – via update()
and can  notify listeners when the state changes, via onUpdated(fn: (state: CheckedState<TState>) => void)

The $checkedState is the data + the checksum of the data. You can see here what the checksums are and why they are important https://www.movex.dev/docs/how/state_synchronization#1-use-the-checksums-to-its-advantage-optimal
Let me know if this clarifies it a bit, and let me know if you still have questions!

Hey @Devanshi-Crypto, are you still working on this? Do you need any help from me?
Let me know if you have any more questions and I'd be more than happy to assist. You can also join our discord https://discord.gg/N8k447EmBh where the community can help too!

Hey sorry, so I am kind of stuck here and trying to solve errors in checkedstate. Do you have any suggestions? Also yes I'm joining your discord, let's discuss there!

Will check this tomorrow! Can you describe the issue a bit – are those compilation errors?

yep, i am trying to solve but i got confused

Hey @Devanshi-Crypto sorry for taking so long to answer – I was a bit busy with life in the past days!
So I'm a bit confused what's going on here! Could open a PR with the changes you've made so far so I can take a look at the code?!
Thank you, and looking forward to help with this! 🙌

Closing for now


Rename the two config files to config.js.default and add config.js to gitignore

This way you can have your own local copy of the config without conflicting with others.
https://github.com/jakkra/SmartMirror/blob/master/config.js
https://github.com/jakkra/SmartMirror/blob/master/client/src/config.js
Also update README

hey, can i take this up?

Sure thing! Go on :+1:


Improve support for cancellation

At the moment the connection classes provide support to cancellation with cancel member function. Although not common or even advisable, some users may want to cancel operations with timeouts like this
co_await (conn.async_run(...) || timer.async_wait(...));

This is working as long as the timeout is not too small e.g. 10ms. To handle that properly Aedis need to improve cancelation support. Below some parts of a conversation I've had with Richard

timer waits and IO reads will be cancellation-aware. But an operation that you have written yourself may not be.

these operations may want to check the cancellation context hasn't already been cancelled before initiating, and also store a cancellation handler in connected cancellation slots in order to cut short any long-standing operation.  asio coroutines do this pre-initiation check for you https://github.com/boostorg/asio/blob/0af7858e7b5603a2415a30b69e16c7ef1d47a5e9/include/boost/asio/detail/deadline_timer_service.hpp#L251

also, If you cancel after posting an operation, when the posted code is executed, it may want to check whether the cancellation_state has been cancelled while the posted step was waiting in the executor.

// some code...
asio::post(exec, some_next_step(std::move(my_handler));
// cancellation happens here...
// some_next_step will not know that the handler was cancelled unless it checks.
// ...
// ... some_next_step now executes...

void some_next_step(auto handler)
{
  auto cs = get_canecllation_state(handler);
  if (cs.cancelled() != asio::cancellation_type::none)
  {
    complete(handler, operation_aborted);
    return;
  }
  // ... not cancelled
}

Motivation for this issue was #28.

All forms of cancellation are now supported, for example, implicit with operator || and && or explicit with the cancel member function.


mavlink update POSITION_TARGET_GLOBAL_INT publish

publish continuously in position control mode when there's a valid
setpoint
optionally fill in the velocity and acceleration setpoints when
available
fixes #9841

@superware


Removed avro dependency for StatsManager

Removed the dependency on Avro for StatsManager. Avro 1.4.1 is required for another component to function and downgrading to 1.4.1 is difficult since there is no convenient way of performing Avro code generation with 1.4.1. The data/model objects (StatsSnapshot, StatsHeader, and StatsWrapper) will become generic java classes.
For convenience, StatsManager will now serialize and publish the stats to JSON via Jackson's ObjectMapper (existing dependency). If needed in the future, we can also introduce our own serialization/deserialization.
./gradlew build && ./gradlew test success
coding style applied

Updated, ./gadlew build && ./gradlew test success on both Mac and Linux.


Expand the ignore links FAQ entry to provide a code example.

Code samples make great examples :D

Thank you @jonnyscholes !


[Enhancement] (nereids)implement adminShowTabletStorageFormatCommand in nereids

adminShowTabletStorageFormat command is supported in legacy planner but not in nereids(new doris fe planner). you need implement the following adminShowTabletStorageFormat command in nereids:
unsupportedadminShowTabletStorageFormatStatement
ADMIN SHOW TABLET STORAGE FORMAT VERBOSE?

hi, may I pick this?

I think the work of this issue is almost the same as #42783, so maybe you can assign it to him  directly? I'll pick another issue :) @LiBinfeng-01

I think the work of this issue is almost the same as #42783, so maybe you can assign it to him directly? I'll pick another issue :) @LiBinfeng-01

sure


Add step size h to fun and jac for optimal preconditioning.

This seems like a bad idea for general implementations.

Enable usage of left and right (Block diagonal) preconditioners as proposend in Botasso2007.


Error message and !!MISSING: command!! messages in code lenses opening template file when params file missing

Does this occur consistently?  Yes
Repro steps:

Open a template file with params, e.g.:

// https://github.com/microsoft/vscode-azurearmtools/issues/600#issuecomment-616631029
{
    "$schema": "https://schema.management.azure.com/schemas/2018-05-01/subscriptionDeploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "rgNamePrefix": {
            "type": "string",
            "defaultValue": ""
        },
        "rgEnvList": {
            "type": "array",
            "allowedValues": [
                "DEV",
                "TEST",
                "PROD"
            ],
            "defaultValue": []
        },
        "rgLocation": {
            "type": "string",
            "defaultValue": ""
        },
        "instanceCount": {
            "type": "int",
            "defaultValue": 2
        }
    },
    "resources": []
}

Create params file and save
Close both files
Delete params file
Re-open template file
!)

first issue

second issue
Action: ResolveCodeLens
Error type: ENOENT
Error Message: ENOENT: no such file or directory, open '/Users/xxx/repos/template-examples/810.parameters.json'
Version: 0.11.0
OS: darwin
OS Release: 18.7.0
Product: Visual Studio Code - Insiders
Product Version: 1.47.0-insider
Language: en

Has been properly fixed.


Auto updater

Implement auto updater for releases which will check for new version whenever nadeko is ran and prompt you to download it

mayyyyyyyybeeeeeeeeee fit this into 0.9?!

done, https://github.com/Kwoth/NadekoUpdater/releases


Having trouble with Chrome Windows 10 Version 60.0.3112.90

Are you using the latest version of jsPDF?
I can demo the effect on http://rawgit.com/MrRio/jsPDF/master/ so I guess it is the latest version.
Have you tried using jspdf.debug.js?
no
Steps to reproduce

goto  http://rawgit.com/MrRio/jsPDF/master/

paste

var doc = new jsPDF();

doc.setFontSize(40);
doc.text(35, 25, "onesingle line");
doc.output('dataurlnewwindow');

and wait for another tab to open showing the resulting pdf.
What I saw
with Chrome Windows 10 Version 60.0.3112.90 (Offizieller Build) (64-Bit)
it opens a new tab but the pdf is not shown in the tab.
Chrome console says "Not allowed to navigate top frame to data URL: "
I googled around and found:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/GbVcuwg_QjM
Intent to Deprecate and Remove: Top-frame navigations to data URLs
It does work Chrome MacOs Version 59.0.3071.115 and firefox windows 54.0.1 (32-Bit)
I tried to wrap the pdf in an iframe as proposed here : https://bugs.chromium.org/p/chromium/issues/detail?id=594215#c64
It works on chrome but not in firefox.
// You'll need to make your image into a Data URL
// Use http://dataurl.net/#dataurlmaker

var doc = new jsPDF();

  function open_data_uri_window(url) {
    var html = '' +
      '' +
      '' +
      '' +
      '';
    a = window.open()
    a.document.write(html)
  }

doc.setFontSize(40);
doc.text(35, 25, "onesingle line");
a = doc.output('datauri');

open_data_uri_window(a);

it works in chrome but not in firefox which hangs.
What I expected
I expected another tab to open and to show the pdf.
Need more documentation
I would appreciate if documentation also would show examples, how to display the generated PDF,
In particular how it is done in http://rawgit.com/MrRio/jsPDF/master/ where the embedded pdf browser works on any platform I mentioned above.

I found a solution which also works in Firefox. See https://stackoverflow.com/questions/45585921/firefox-hangs-when-displaying-a-pdf-via-data-url/45585922#45585922
maybe jspdf could add a suggested filename to output('datauri')

I am experiencing the same/similar problem in the latest update of Chrome (60.0.3112.90) on OSX. I just see a blank screen.  Firefox is ok. It was working without a problem before.

If you follow the code in my ticket, it works fine in chrome on osx.

Thanks. @bwl21

Nice @bwl21 thank you!
I have used your code, with the only variation in
a = doc.output('datauri');

replaced with
a = doc.output('datauristring');

this to avoid the error in console.

@Estrusco indeed, my code now also uses datauristring.
As a pity, the stuff does not work on Microsoft Edge. I could not yet find out why.
Even the jspdf demo page (http://rawgit.com/MrRio/jsPDF/master/) does not show the resulting pdf.

Implemented


Trust CloudFlare IPs for reverse proxy headers.

No related issue was filed for this.
This PR trusts CloudFlare IP addresses as a reverse proxy. CF IPs have to be trusted in order to use Flexible SSL with Solder because Laravel refuses to use the X-Forwarded-Proto headers without trusting the IPs first.
Links
CloudFlare IPs
isSecure()

I feel like cloudflare-specific configuration should be left to the user. We shouldn't be trusting Cloudflare on installs that aren't behind it.

@nepeat Going to agree with @skwerlman on this. This should be left up to the user to configure on their server.

As an additional comment, if you want to expand this PR/reopen to add a configuration option and have it disabled by default. I feel that would be more acceptable, the main issue being have those IPs on always trusted.


add RM config

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

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

Codecov Report

Merging #412 (574ab00) into master (c8dccc9) will decrease coverage by 0.08%.
The diff coverage is 5.88%.

@@            Coverage Diff             @@
##           master     #412      +/-   ##
==========================================
- Coverage   40.46%   40.38%   -0.09%
==========================================
  Files         138      140       +2
  Lines        8002     8018      +16
==========================================
  Hits         3238     3238
- Misses       4517     4533      +16
  Partials      247      247

Impacted Files
Coverage Δ

pkg/rm/config.go
0.00% <0.00%> (ø)

pkg/rm/lock_config.go
0.00% <0.00%> (ø)

pkg/client/config.go
58.18% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[1.12] Item scaffold for Projector custom recipes

Projector recipes seem limited right now to placeable blocks in the world. It would be nice to be able to use items in the recipes too. Maybe a simple "scaffold" like block that can be placed in the world and can hold a single item inside so it can be used in a Projector recipe?

I've thought about this before, but it's not as easy as it might sound, because I would have to render that scaffolding block with a TESR to show the item its holding or do some trickery when rendering it during crafting and in JEI.
At the moment, with the current rendering system this probably won't happen. And other mods adding pedestal type blocks will not render properly either 😭
I have some ideas how this could be circumvented, but the best way really would be to allow TESRs to be rendered.
For now I'll leave this issue open as it's something that would neatly open up a whole new set of possible recipes.

Ah I'm guessing the TESR issue is also why when I made a recipe that had pipes in it, the pipes didn't render in the JEI panel or in the crafting animation too :)
As an intermediate stage, what if the block was a generic container (like a one-slot chest) that was opaque and didn't render the item inside it? It would allow the recipes to be made and table the rendering problem for later? Although, that would make it pretty hard to figure out where the items should go in the JEI preview ...
In any case, glad you're thinking about it, and hope you can figure it out :)


Create Matrix Multiplication.c

Its a program of matrix multiplication for unknown numbers through c
What type of PR is this? (check all applicable)

[ ] 🚀 Added Name
[ ] ✨ Feature
[ ] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Add Link of GitHub Profile

its a program of matrix multiplication through c


3D boxes of instances in LIDAR coordinates about NuScenes

Hi,
In nuSences the lidar coordinate is (
up z    y front
^   ^
|  /
| /
0  ------> x),
but the gt_bbox_3d in nuscenes_dataset not change to the LiDARInstance3DBoxes coordinate(
up z    x front
^   ^
|  /
| /
left y <------ 0)

I have the same question

@Trent-tangtao @qq1361096516 Have you solved it?


Docker Installation Issue

Q
A

Bug report?
[ ]

Feature request?
[ ]

Support request?
[x ]

Hi Cocolabs Team!
I'm really curious to test the marketplace. Unfortunally i have some issues with the docker installation.
The installation run without problems. The only warning/problem is show me is:
"PHP Warning:  Version warning: Imagick was compiled against Image Magick version 1799 but version 1800 is loaded. Imagick will run but may behave surprisingly in Unknown on line 0"
But i think this isn't the problem.
Once it's run through the installation i got the message "Cocorico is alive".
If i open http://localhost the only thing what i see is "File not Found".
If i open http://localhost:9001 it shows me that the bootstrap is exited.
If i try to start it, is a fatal error.
In the log of bootstrap is only "Cocorico is alive"..
Any ideas what could be the problem?
Running the docker on ubuntu 18.04
Thank you
Best Regards

Sorry.
Already found a topic:
https://github.com/Cocolabs-SAS/cocorico-docker/issues/25


Changed noradrenaline to norepi, swapped with NaCl

Swapped 'noradrenaline' and 'NaCl' as they had the other's units (NaCl was in mcg/kg/min).  Also changed noradrenaline to norepinephrine.

Cheers!


Publish to https://cdnjs.com/

Please host static assets on a content distribution network such as CDNJS.com, so that developers can integrate patternomaly more easily into our applications.

Also for testing purposes, codepen etc it would be really helpful

found compiled versions here: https://www.jsdelivr.com/package/npm/patternomaly
Note: latest Chartjs from jsdeliver didn't work, but it worked with https://cdnjs.cloudflare.com/ajax/libs/Chart.js/2.5.0/Chart.js


DELETE requests are not being proxied through using Cloud Provider SDK

Describe the bug
Using the openEQUELLA Cloud Provider SDK and an extension control that performs a HTTP DELETE request, the request does not appear to be proxied through to the cloud provider endpoint.
To Reproduce
Steps to reproduce the behavior:

Register a cloud provider
Add an cloud provider control to a contribution item that supports performing a HTTP DELETE request to a cloud provider endpoint
Perform the delete operation
HTTP DELETE request is made to openEQUELLA but no traffic is received from cloud provider endpoint

Expected behavior
The request is proxed through openEQUELLA and sent to cloud provider endpoint
Screenshots
N/A
Stacktrace
Not available
Platform:

OpenEquella Version: 2019.2
OS: Ubuntu 18.04
Browser: Chrome

Additional context
Viewing http traffic with Wireshark shows the request to openEQUELLA but no further traffic to the cloud provider endpoint.  Changing the cloud provider control to perform a GET or POST operation instead of a DELETE showed openEQUELLA proxying the request.

@edalex-tom did you also say UPDATE didn't work?

I've double checked the requests and the DELETE and PUT requests do not result in subsequent proxied requests to the cloud provider endpoint.


Alternatives to Relative Positional Biases?

I see in the README it says:
There's just better alternatives that don't involve explicitly biasing the attention weight matrix, and they will be more performant on top of providing similar or better accuracy levels.

What alternatives do you recommend? It seems like the NAT repo is using relative positional biases?

I moved this issue here since this issue is more related to NAT/DiNAT as opposed to NATTEN.
We'll be updating this thread soon.


Is it possible to lock the repy application files to prevent modification?

It's come to my attention that the RePy application files are mutable due to a student injecting stuff into the repy.py file as one of their attack cases.
While that is a good attack, I do think it might be good to have a method of preventing that as depending on how it was done it could corrupt the build. While this student cleaned up after themselves and provided a script to sanitize the repy.py file others might not have the foresight and it might create problems.

This is a good observation.  Usually one does not place the user directory / user code in the same directory as the repy source code. I'd recommend we make the instructions for running repy clearer so that that sort of use case is not common for students using the framework.

I think the repy.py file itself has to be in the directory at least due to the fact that the command we run is repy.py restrictions.default encasementlib.r2py <referencemonitor> <attack>

We should be encouraging them to use the --cwd flag to set the current
directory.  I think we left this out to make it easier for the students to
run the code and understand what is happening.
On Thu, Nov 19, 2020 at 8:01 AM Victoria Zhong<EMAIL_ADDRESS>wrote:

I think the repy.py file itself has to be in the directory at least due
to the fact that the command we run is repy.py restrictions.default
encasementlib.r2py
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/SeattleTestbed/docs/issues/71#issuecomment-730033537,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGROD2WXGWBEF2VU4Q35ETSQRN7BANCNFSM4T2VFQPA
.


Is it avaialble for latest version of nginx?

Hi,
Does the pagespeed ngx is available for latest version of nginx?

Hi
As far as you need to recompile nginx to add pagespeed module, yes.
I have working it with nginx 1.17.8

Hi,
Can you share instructions on the same? I am new to nginx and pagespeed both and needs clear instructions on the same.
Also, is it possible to post the docker for the latest nginx with pagespeed module for you? I am sure it will be helpful to many people using this, and results in saving a huge time for everyone.
Thanks

Hi
Maybe reading this as a start: https://www.modpagespeed.com/doc/build_ngx_pagespeed_from_source.
About docker, I don´t know nothing about it cause I don´t use it, but there is a docker here:
https://github.com/apache/incubator-pagespeed-ngx/tree/master/docker

Thanks Lofesa, this helped. I am able to get it work with instructions you shared. Thanks again.


fix typo, -XPOST had better be here.

"Fixes #issue-number" or "Add description of the problem this PR solves"
Proposed Changes

Do you think this can be improve, or leave it. Just FYI


Zombie processes

Hello
My kanshi configuration seems to be a little bit problematic. I run kanshi on sway, and I need to move my workspaces depending on the profile. Here is my configuration file:
profile soad {
    output eDP-1 enable mode 1920x1080 position 0,0

    exec swaymsg workspace "1: www", move workspace to eDP-1
    exec swaymsg workspace "2: email", move workspace to eDP-1
    exec swaymsg workspace "3: chat", move workspace to eDP-1
    exec swaymsg workspace "4: visio", move workspace to eDP-1
    exec swaymsg workspace "5: term", move workspace to eDP-1
    exec swaymsg workspace "6: code", move workspace to eDP-1
    exec swaymsg workspace "7: gaming", move workspace to eDP-1
    exec swaymsg workspace 8, move workspace to eDP-1
    exec swaymsg workspace 9, move workspace to eDP-1
    exec swaymsg workspace 10, move workspace to eDP-1
}

profile soad-home {
    output eDP-1 disable
    output DP-2 mode 1920x1080 position 0,0
    output HDMI-A-1 mode 1920x1080 position 1920,0

    exec swaymsg workspace "1: www", move workspace to DP-2
    exec swaymsg workspace "2: email", move workspace to DP-2
    exec swaymsg workspace "3: chat", move workspace to DP-2
    exec swaymsg workspace "4: visio", move workspace to DP-2
    exec swaymsg workspace "5: term", move workspace to HDMI-A-1
    exec swaymsg workspace "6: code", move workspace to HDMI-A-1
    exec swaymsg workspace "7: gaming", move workspace to HDMI-A-1
    exec swaymsg workspace 8, move workspace to HDMI-A-1
    exec swaymsg workspace 9, move workspace to HDMI-A-1
    exec swaymsg workspace 10, move workspace to HDMI-A-1
}

profile yoda {
    output DP-2 mode 1920x1080 position 0,0
    output DP-1 mode 1920x1080 position 1920,0
}

Kanshi works fine, executes the swaymsg commands, but sometimes the kanshi process dies. So I started to look for some clues and I found that it generated zombie processes:
martial    20882  0.0  0.0   2604  1792 pts/1    S+   15:28   0:00 kanshi
martial    20884  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20885  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20886  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20887  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20888  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20889  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20890  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20891  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20892  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>
martial    20893  0.0  0.0      0     0 ?        Zs   15:28   0:00 [kanshi] <defunct>

I run Kanshi 1.1.0 on Arch (community repository, not AUR). I can post the output of a strace if you need.

Hmm, it seems like our double-fork logic is incorrect. We should waitpid in the parent, not in the child. The child should execute the grandchild and then exit.
https://github.com/emersion/kanshi/blob/e55b5847a07bc3ea2dc19f3cdce8d88c8aa2f9ca/main.c#L97

Funny, I just noticed the same thing. I really wanted to give some information on the subprocess, which you can only do if you wait for it. I guess it was useful for debugging.
We could either fork a third time, or just stop trying to give this exit info. Debugging can be done in the invoked command anyway.

stop trying to give this exit info

Yeah, I'd prefer to do that.

Wish... fulfilled :-) #85


[web] Implement didGainAccessibilityFocus/didLoseAccessibilityFocus

didGainAccessibilityFocus and didLoseAccessibilityFocus are not implemented on the web. Some of our customers would like to use it.
How to implement it
This is only a hypothesis based on a very small amount of research:

Translate isFocusable to tabindex=0 (we already do this for buttons).
Translate isFocused to focus() and blur() calls on the respective HTML nodes.
Use SemanticsAction.didGainAccessibilityFocus and SemanticsAction.didLoseAccessibilityFocus on passed in updateNode as a declaration that the framework wishes to receive accessibility events, then:

Use focus and blur event listeners to get focus notifications from the browser, and forward them as respective SemanticsActions to onSemanticsAction.

Potential issues to anticipate:

It is not clear how consistently assistive technologies report focus changes when the user traverses the UI. For example, NVDA distinguishes between System Focus and Object Navigation. We may need to prepare to deal with multiple behaviors.
There may be inconsistency between native and web even when the same assistive technology is used. For example, the behavior of VoiceOver with an iOS app may be different from VoiceOver with a web app.

This was fixed https://github.com/flutter/engine/pull/41831. I forgot to close it.


Detect kernel headers even if they are splitted into source/ and build/ directories

Currently bpftrace uses only /lib/modules/uname -r/source/ if it
exists. This however leads to failures Debian where generated kernel
headers, such as include/generated/autoconf.h are put into a separate
directory, for example:
---- 8< ---- (fuse_vs_mmapsem.bt)
#include <linux/path.h>
#include <linux/dcache.h>
#include <linux/sched.h>
#include <linux/mm_types.h>

kprobe:fuse_readpage,kprobe:fuse_readpages {
        printf("%s (%d) %s\n", probe, ((task_struct *)curtask)->mm->mmap_sem.count.counter, kstack);
}
---- 8< ----

# ./bpftrace fuse_vs_mmapsem.bt
/lib/modules/4.19.0-2-amd64/source/include/linux/kconfig.h:5:10: fatal error: 'generated/autoconf.h' file not found
Unknown struct/union: 'task_struct'

Fix it by using both /lib/modules/uname -r/source/ and
/lib/modules/uname -r/build/ if both are present.
/cc @vincentbernat
P.S. I was not using C++ for ages, so please forgive me in advance if I
missed something.

Ping. @brendangregg, @mmarchini, is there any reason not to merge this patch? At present - in current state - on Debian as it is bpftrace fails to provide significant part of its functionality - kprobes are not working at all.

No reason not to merge, I was just unfamiliar with this area of code. I tested and it still builds on my system. Merging, thanks!

Even though the build worked for me, I noticed our CI testing is hitting this on llvm 5.0:
/home/travis/build/iovisor/bpftrace/src/clang_parser.cpp: In member function 'void bpftrace::ClangParser::parse(bpftrace::ast::Program*, bpftrace::StructMap&)':
/home/travis/build/iovisor/bpftrace/src/clang_parser.cpp:223:8: error: expected unqualified-id before '[' token
   auto [ksrc, kobj] = get_kernel_dirs(utsname);
        ^
/home/travis/build/iovisor/bpftrace/src/clang_parser.cpp:224:7: error: 'ksrc' was not declared in this scope
   if (ksrc != "")
       ^~~~
/home/travis/build/iovisor/bpftrace/src/clang_parser.cpp:225:55: error: 'kobj' was not declared in this scope
     kflags = get_kernel_cflags(utsname.machine, ksrc, kobj);
                                                       ^~~~

I don't know if it's a real problem for anyone yet.

@brendangregg thanks for merging and oops, I appoligize for the inconvenience.
When I was testing the patch, my compiler (g++) was issuing warning about auto [ksrc, kobj] working only in C++17 mode. However the build was working fine. I thought it should not be issue since https://github.com/iovisor/bpftrace/pull/384 is there and, if I understood correctly, was going to be merged. Maybe we should just do that now and it will hopefully fix the build.

Ok, good idea, I'll try that!

Thanks.

I think the error is still happening; from another PR: https://travis-ci.org/iovisor/bpftrace/jobs/493441040

Ok, I've tried to workaround, what is seemingly a LLVM5 limitation: https://github.com/iovisor/bpftrace/pull/423.

@vincentbernat, would you mind cherry-picking this patch (https://github.com/iovisor/bpftrace/pull/399/commits/4d76385e4f6537b5749808d7549a82197cf4eb9c) into Debian?
The reason I ask is because currently bpftrace in Debian fails to work with kprobes at all.
Debian should not need follow-up build fixups, as g++ used to build there is recent enough (I was testing the patch on Debian testing originally).
Thanks beforehand,
Kirill

I suspect we'll finish the 0.9 release in a week or two, at which point hopefully Debian will pick up all the fixes.

That would be great.

@navytux In the meantime, I have uploaded a new version with this patch.

@vincentbernat, thanks.


#351 Repl is slow when displaying lots of coloured output

#351 Repl is slow when displaying lots of coloured output
Batches edits to the repl window buffer so that the updates all occur at once.
Removes unnecessary marshalling to the UI thread.
Fixes error when opening interactive window.

Would like @DinoV to have a look at this one.


Clean-up cell output debt

Some debt has accumulated around cell output and we should start to clean things up

[x] there is transformEditsOutputs and transformSpliceOutputs which feel like they are on the wrong layer, e.g should this be part of the serialisation of output
[x] similar to transfromOutputs is IProcessedOutput which also seems wrongly placed. Maybe the internal model object specs all the data of those two and the API types are directly converted into those types
[x] the proposed API is very rough, esp the outputKind-marker is an api-alien concept

This one relates the renderer work @connor4312 is doing now. If the output renderers are all pure and lives in the webview, then we don't need IProcessedOutput or transform*Outputs at all as there is no back and forth between the UI and ExtHost, raw outputs will be the source of truth.

Yea, I can clean these up this month.

@connor4312 Can you/we tackle those debt items before going all-in on the new API and such?

@rebornix @roblourens I have created #110855 which removes the transform logic and moves that into a single place

Sure, I should have time this month to tack some of that

Hi, finally have some unallocated time to do this 🙂
Before I just in, I know you have been working on hammering out the final versions of these APIs. Does that work affect this cleanup, or can I go ahead and take care of this tomorrow?

Please hold back or join us in #115817

We are done with this


Update to 1.2.20 version of translator

Update to newest release of the CQL Translator and take advantage of the UCUM unit validation feature.

Editing this issue to update to 1.2.20 version of translator. Also update HAPI to 3.2.0.

Also adding the release configuration and matching version to publish in parallel with translator.

Resolved in release 1.2.20


Create build process

which strips out the docstrings to reduce filesize

I am going to close this. Currently the build size is not sufficient to cause issues. We can look at this in the future if it becomes a challenge.


Fix Dnsimple::Default#{domain_api_token, api_token} functions

I'm not sure if this code was correct or not

Any updates on this? I have been using this patched version for a while now. It would be better if this is fixed on the official repo too.

@oguzbilgic sorry for the big delay, and thanks for the patch. I merged #62 which was providing the same fix and an update to the comment. I'll make sure to mention both you and @rupurt in the changelog.


Run query with multi cursers selected different text.

I had a list of tables I places in a query and wanted to see the top 100 rows of each.
table1
table2
table3
As I wanted to write select top 100 * from selected them and created a curser for each.
note '|' is the curser.
|table1
|table2
|table3
Started typing.
select top 100 * from |table1
select top 100 * from |table2
select top 100 * from |table3
I arrowed over and selected all rows with each curser.
|select top 100 * from table1
|select top 100 * from table2
|select top 100 * from table3
([] is highlighted text)
[select top 100 * from table1]|
[select top 100 * from table2]|
[select top 100 * from table3]|
I used the hotkey to run the query it took a bit longer than normal and only one ran.
My guess is it ran each query individually and not as one query with multiple results. Is this the behavior I should be expecting?

That is the case right now - our query runner code doesn't handle multiple cursors it would appear. You can drag and select multiple queries using a single cursor and then if you run them it'll execute them all together though.


Query options can be used in inappropriate places.

Currently, Client::find and Client::search both accept a Query which has fields for filters, grouping, and windowing. But find and search only accept filters and windowing. I've got WIP implementations of list and findadd, the former only takes filters and grouping, the later only filters.
There are at least a couple of solutions:

the most straightforward is to simply split up Query into its component parts and only accept the parts that are valid for each command.
it'd also be possible to do something with phantom-types, but this seems more complicated for no gain.

This seems like an interesting idea. I may give it a try in a separate branch.


[1/n] Merged fine-tuning dataset: grammar + samsum

Context
What is the purpose of this PR? Is it to

[x] add a new feature
[ ] fix a bug
[x] update tests and/or documentation
[ ] other (please add here)

As discussed in the RFC in #1186, we will merged instruct and chat datasets to the following unified pipeline that can better support multimodal:

message_transform to create List[Message] from dataset with full flexibility on columns, ad-hoc modifications, etc. For multimodal, additionally images are loaded from the path
prompt_template as a optional way to add structured text around specific roles in the list of messages
model_transform that takes the list of messages and tokenizes it. For multimodal, it will additionally apply model-specific image transforms to the images associated with the sample

This PR creates the unified dataset, and refactors grammar and samsum to start off with. As a result, a few key changes were made:

New FinetuneDataset (not married to the name, best I can think of) class with the unified pipeline with associated unit test
Add ToInputOutputMessages (open to better names) which provides a generic message transform that takes input column -> user message, output column -> assistant message
Refactor grammar and samsum datasets. They now create a message transform using ToInputOutputMessages, add a default prompt template that can be changed, and use the new FinetuneDataset class
Move Message from _types.py to _messages.py. This file will now contain everything related to Message, including generic message transforms (like the ones in _converters.py which will eventually migrate here)
Make all tokenizers also double as Transforms for use in the model_transform argument
New unified PromptTemplate interface that merges functionality of instructTemplate and ChatFormat. Refactored grammar and summarize templates to use a common CustomPromptTemplate class that takes in any template and formats a list of messages accordingly. It also covers the ability to specify a custom template from configs, which we were missing. More will be discussed in an upcoming tutorial update.

Test plan

[x] update live docs and check rendering
[x] unit tests for FinetuneDataset, grammar, samsum
[x] unit tests for ToInputOutputMessages
[x] run unit tests via pytest tests
[x] run recipe tests via pytest tests -m integration_test
[ ] compare loss curves of grammar, samsum to original versions on main

Where are my deprecation warnings? :)

will be added in an upcoming PR... there's a lot stacked on this one :)

Codecov Report
Attention: Patch coverage is 87.04319% with 39 lines in your changes missing coverage. Please review.

Project coverage is 69.58%. Comparing base (bc6b7e9) to head (fed7dae).
Report is 1 commits behind head on main.

Files
Patch %
Lines

torchtune/models/llama3/_tokenizer.py
50.00%
7 Missing :warning:

torchtune/models/phi3/_tokenizer.py
50.00%
7 Missing :warning:

torchtune/models/gemma/_tokenizer.py
50.00%
5 Missing :warning:

torchtune/models/llama2/_tokenizer.py
50.00%
5 Missing :warning:

torchtune/models/mistral/_tokenizer.py
50.00%
5 Missing :warning:

torchtune/models/qwen2/_tokenizer.py
60.00%
2 Missing :warning:

recipes/full_finetune_distributed.py
0.00%
1 Missing :warning:

recipes/full_finetune_single_device.py
0.00%
1 Missing :warning:

recipes/lora_finetune_distributed.py
0.00%
1 Missing :warning:

recipes/lora_finetune_single_device.py
0.00%
1 Missing :warning:

... and 4 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1234       +/-   ##
===========================================
+ Coverage   27.41%   69.58%   +42.17%
===========================================
  Files         233      238        +5
  Lines       10591    10759      +168
===========================================
+ Hits         2903     7487     +4584
+ Misses       7688     3272     -4416

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


README.md not included with package, results in setuptools error

File ".../env/build/asyncmongo/setup.py", line 11, in
long_description=open(os.path.join(os.path.dirname(__file__),"README.md"), "r").read(),

IOError: [Errno 2] No such file or directory: '.../env/build/asyncmongo/README.md'
This error is the result of 'pip install asyncmongo'.

thanks. this was a problem with building 1.2 with python2.7 which for some reason is ignoring the manifest file. I've re-built as 1.2.1


SW-5073 Include Herb in species growth form Learn More

#2749  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sgrimm and the rest of your teammates on  Graphite


I have some questions when add TLS certification to alertmanager

hello guys, I want to add TLScertification to alermanager, i create cert and key with secret, but after i edited the manifest/alertmanager-alertmanager.yaml and alertmanager-serviceMonitor.yaml, it is still not working
i add those to alertmanager-alertmanager.yaml:

spec:
image: quay.io/prometheus/alertmanager:v0.25.0
** web:
tlsConfig:
keySecret:
key: tls.key
name: alert-tls-secret
cert:
secret:
key: tls.crt
name: alert-tls-secret**

and edit the alertmanager-serviceMonitor.yaml
spec:
endpoints:
- interval: 30s
port: https
scheme: https
tlsConfig:
insecureSkipVerify: true

interval: 30s
port: web
interval: 30s
port: reloader-web

but when check the prometheus website, the target still watch the http  endpoint of alertmanager-main,so i guess there are some format error in my edit,but i ddnt find document about this.
I would appreciate it if you could teach me edit the yaml to add TLS ,or point the document about this !
env:
kubernetes 1.25.7
kube-prometheus release:1.12


Enhancement: Start torrent after submitting upload

Related to #69 and #70. Add a listener to the upload button in the userscript and transfer the .torrent file to the client after the user submits the upload, whether by copying to a directory or transferring via an API.

Since #70 is now closed covering three major torrent clients, the goal of this enhancement is changed from adding the torrent file to the client after submitting the upload, to starting the torrent after submitting. Multiple output directories will remain supported, but (for now at least) complex behaviour involving copying to different directories at different times is not planned. Starting uploads on submission will be added for torrent client APIs, but not for watch directories.