Can the kernel support MPTCP, like Shadowsocks-rust does, and are there similar technologies like kcp MPQUIC, etc., that are supported as well?

For example, if I have a small bandwidth but multiple lines like eth0, eth1, eth2, eth3, each with 2M bandwidth, can I aggregate the bandwidth of the 4 lines? Currently, MPTCP only supports TCP multipath, but can MPQUIC achieve multipath over UDP and enable bandwidth aggregation?

I have considered this before, but decided not to implement this.  Since link aggregate is complex and there is no general solution.  Since there is no general solution, it need to be done carefully by a case-by-case fashion.  Since it need to be done by a case-by-case fashion,  in the end of day I will likely spend my time on fine-tuning for a few user's very specific use cases (it's not a good way of using time).
Here I am re-using some comments I previously sent to someone in private:

I have considered doing this before, but gave up since I couldn't find a perfect design that covers all use cases elegantly.

Suppose you have a 1000mbps 10ms link and a 10mbps 100ms link, to eliminate out-of-order, you will get a 1010mbps 100ms link.

Suppose you have a 10mbps which loses connection randomly every minute and a stable link with 1000mbps with no packet loss and no connection loss. If you bond them together, you will get a 1010mbps unstable link.

Suppose you have 2 links with bandwidth varying from 10mbps-1000mps dynamically. It's hard to correctly bond and load balance between them since bandwidth detection will have a lag. Also it's awkward to do the bandwidth/congestion detection at layer3, while tcp is also doing it at layer4.

As a result I believe the link aggregation needs to be carefully done in a case-by-case fashion, there is not a way that "simply works".

I have considered doing link aggreation in tinyfecVPN/UDPspeeder before, but decided not to implement this, since link aggregate is complex and there is no general solution.  Since there is no general solution, it need to be done carefully by a case-by-case fashion.  Since it need to be done by a case-by-case fashion,  in the end of day I will likely spend my time on fine-tuning for a few user's very specific use cases (it's not a good way of using time).
Here I am re-using some comments I previously sent to someone in private:

I have considered doing this before, but gave up since I couldn't find a perfect design that covers all use cases elegantly.

Suppose you have a 1000mbps 10ms link and a 10mbps 100ms link, to eliminate out-of-order, you will get a 1010mbps 100ms link.

Suppose you have a 10mbps which loses connection randomly every minute and a stable link with 1000mbps with no packet loss and no connection loss. If you bond them together, you will get a 1010mbps unstable link.

Suppose you have 2 links with bandwidth varying from 10mbps-1000mps dynamically. It's hard to correctly bond and load balance between them since bandwidth detection will have a lag. Also it's awkward to do the bandwidth/congestion detection at layer3, while tcp is also doing it at layer4.

As a result I believe the link aggregation needs to be carefully done in a case-by-case fashion, there is not a way that "simply works".

For you question of MPQUIC,  I am not familiar with it. But I think the idea of link aggregation doesn't work super well in general.

I have considered doing link aggreation in tinyfecVPN/UDPspeeder before, but decided not to implement this, since link aggregate is complex and there is no general solution. Since there is no general solution, it need to be done carefully by a case-by-case fashion. Since it need to be done by a case-by-case fashion, in the end of day I will likely spend my time on fine-tuning for a few user's very specific use cases (it's not a good way of using time).
Here I am re-using some comments I previously sent to someone in private:

I have considered doing this before, but gave up since I couldn't find a perfect design that covers all use cases elegantly.

Suppose you have a 1000mbps 10ms link and a 10mbps 100ms link, to eliminate out-of-order, you will get a 1010mbps 100ms link.

Suppose you have a 10mbps which loses connection randomly every minute and a stable link with 1000mbps with no packet loss and no connection loss. If you bond them together, you will get a 1010mbps unstable link.

Suppose you have 2 links with bandwidth varying from 10mbps-1000mps dynamically. It's hard to correctly bond and load balance between them since bandwidth detection will have a lag. Also it's awkward to do the bandwidth/congestion detection at layer3, while tcp is also doing it at layer4.

As a result I believe the link aggregation needs to be carefully done in a case-by-case fashion, there is not a way that "simply works".

For you question of MPQUIC, I am not familiar with it. But I think the idea of link aggregation doesn't work super well in general.

It’s actually similar to the MPTCP feature in shadowsocks-rust. Shadowsocks-rust just calls the kernel’s built-in support for MPTCP, so it doesn't directly handle the multipath implementation itself. In the configuration for shadowsocks-rust, you just add the --tcp-multipath parameter, and the kernel takes care of the MPTCP multi-path feature. I’ve tested this myself: let’s say you have two tunnels, tun1 and tun2. If tun1 is throttled, meaning it's unstable and only provides around 10Mbps, and tun2 offers 50Mbps, using layer 3 static route load balancing between tun1 and tun2 doesn’t really help. For example, if a user's request goes through tun1, the speed will always be limited to tun1's 10Mbps, even though tun2 can offer much faster speeds. And if after some time tun1 improves to, say, 60Mbps, the static load balancing won’t automatically adjust.
By using the kernel's built-in MPTCP feature together with shadowsocks-rust’s MPTCP support, you can dynamically combine the speeds. This means download speeds could vary between 30-60Mbps, rather than being stuck at tun1's 10Mbps. This happens because the MPTCP setup allows dynamic aggregation of the tunnel speeds, which solves the problem of a single TCP flow being stuck on one specific tunnel. Without MPTCP, a single TCP stream would always be limited to one tunnel's speed. That’s especially frustrating for activities like playing overseas games or streaming 4K videos. If speeds don’t add up, streaming 4K content could become choppy and laggy.
I see that shadowsocks-rust directly calls the kernel's IPPROTO_MPTCP parameter. The actual MPTCP functionality can be implemented using the kernel's built-in support for MPTCP, so the software itself doesn't need to include complex multi-path code.

This is the final test result I got by using your VPN software with the tun1 and tun2 tunnels, combined with the MPTCP support in shadowsocks-rust. The speed and stability did indeed improve. If your software natively supported MPTCP by calling the kernel's MPTCP, I wouldn’t need to deploy shadowsocks-rust anymore. Your software would be perfect.

Before acceleration

After acceleration

the user case of tinyfecvpn and shadowsocks is different.
shadowsocks typically aims for tcp traffic,  especially combined with mptcp
tinyfecvpn is:

tunnels any protocol >= ip layer,
latency-sensitive.
packet deliver order matters. (if one link has smaller latency, the other link has large latency, it will case nightmare for some udp protocols)

you might got some improvement on download speed if involve multiple path, but it will cause decrease in other aspect.
So far all your data is pointing to download speed. If you only cares your tcp download speed, doesn't care other aspects much,  in this case maybe simply go ahead and use shadowsocks-rust +mptcp.
technique-wise:
QUIC's main use case is TCP-Like stream delivery, although QUIC supports datagram by the way. Not sure how well MPQUIC works for datagram. From my analysis in previous post, I don't think MPQUIC can work magiclly well for tinyfecvpn.
implementation-wise:
Also MPQUIC is userspace programs, it's not going to be as simple as IPPROTO_MPTCP.


Docker image does not have "memgpt" command

Describe the bug
The docs ask us to run memgpt configure and other command line scripts to interact with memgpt, however if you're using the memgpt/memgpt-server:0.3.14 docker image you'll find there's no such command.
Please describe your setup

[ ] How did you install memgpt?

https://github.com/cpacker/MemGPT/blob/main/compose.yaml followed by creating a .env file with PG creds and docker-compose up -d

[ ] Describe your setup

WSL2, Docker

Additional context
There's no memgpt command:
root@memgpt-server:/# find / -name memgpt -type f
root@memgpt-server:/# find / -name memgpt.py -type f
root@memgpt-server:/#

The closest thing I can find is /memgpt/cli/cli.py, however I don't see a way to run it.

Sorry this is unclear from the documentation - you need to install the CLI with pip install pymemgpt and then run memgpt configure. This creates a ~/.memgpt/config file that can be linked in the container. It's a bit hacky right now since we are still working on improving the configuration for the container deployment.

Ahh, thank you!


Check duplicates

When I accidentally register a flight more than once, after checkin, I get a lot of emails for each time I registered. Also, flights with other names under the same email do not show up in the list of registered flights. Thanks!

Thanks for your feedback. These are shortcomings to the design that I'll try to address in the next release.


Does this work even when the app is totally closed?

Does this work even when the app is totally closed?

Does this work even when the app is totally closed?

I need the same thing dude you find any think about that?

i need it too


Remove extra slash in css url and add locale url param support

@driskull  can you verify this and merge if it looks ok?

Looks good :+1:


Regression in sources paths in declaration maps

Describe the bug
v 3.3.0 has broken the source paths in the dts files
dts({
      entryRoot: resolve(__dirname, 'src'),
      tsconfigPath: resolve(__dirname, 'tsconfig.json'),
})

This would, with 3.2.0 provide declaration maps that looked like
{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["src/index.ts"],"names":[],"mappings":"AAEA,eAAO,MAAM,KAAK,YAEjB,CAAC"}

But with 3.3.0 they look like
{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/src/index.ts"],"names":[],"mappings":"AAEA,eAAO,MAAM,KAAK,YAEjB,CAAC"}

Which doesn't work at all. There is no src/src directory.
I guess technically both are wrong? tsc alone would have the path be ../src/index.ts
Reproduction
https://github.com/ekwoka/create-kwoka-pkg
Steps to reproduce
In above repo, the focus is the nested project under kwoka-pkg directory. It's basically just a scaffold.
Just add src/index.ts in the source with whatever.
Run build.
Then update vite-plugin-dts and run build again.
The source paths are different.
But if you do pnpm exec tsc the source paths are also different...
So something is wrong here, but 3.3.0 of this plugin is definitely most wrong.
System Info
System:
    OS: macOS 13.4.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 104.64 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.3.1 - ~/Library/pnpm/node
    Yarn: 1.22.19 - ~/Library/pnpm/yarn
    npm: 9.6.7 - ~/Library/pnpm/npm
    pnpm: 8.6.7 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 114.0.5735.198
    Edge: 114.0.1823.82
    Safari: 16.5.2

Validations

[X] Read the FAQ.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Could you provide a reproduction?

Beautiful! Thank you!!


Update viessmann.

Missing changlog for v1.2.3 in io-package.json and changelog.md

The other things look good.

@misanorot the two topics mentioned by @Jey-Cee are best practices. Missing version in io-package disallow users to downgrade to these version by admin (later on when a new version is out) .
So I merge it now, but please fix for the next version!

@Jey-Cee @Apollon77  yes it´s clear. In this situation it was a little bit tricky. I fixed a little issue and I need a new version to update npm. I read the article from Jee Cee in the forum in terms of versions.
I will do it correctly the next times.


[PE-D][Tester E] Simple duplicate Detection and restriction in names

Currently, all names are space sensitive. As such "Market Analysis" project is different from "Market   Analysis" thus the following command was able to execute when there is already a project with similar name.

Perhaps,  extra spaces between words could be ignored to better model the real world situation. Optionally, names could also be considered case-insensitive if applicable.
Currently, all names can only accept alpha numeric characters. This could prevent some names from being entered. For example, the name Roy s/o Balakrishnan could not be entered.
Similarly, for project and task names, names like "Refactor Ui, Storage and Model" could not be entered. Perhaps some of the restriction on names could be relaxed. Optionally, different restrictions could be applied people names and project/task names to better model the real world.
Lastly, the restrictions on names could also have been explicitly documented in User Guide.

Labels: severity.Medium type.FunctionalityBug
original: timetraveller-123/ped#8

We have decided that this is a feature flaw of low severity. We will be adding the above name restrictions as notes to the user in our documentation. We will also include this as a future feature addition. Hence, we will be closing this issue without fixing it.


Remove experiment annotations from component models.

This fixes #99 .

I removed a little more from a couple files. Otherwise it looks fine and I'll merge.


SPOI-8304: Cassandra Input operator example

Application to showcase how to use cassandra input operator from Malhar.

@bhupeshchawda  please review.

Parent directory README needs an entry

@bhupeshchawda  addressed review comments.

@DT-Priyanka Please address the comment by Ram - Add entry in tutorials/README.md for cassandra input app.
Few unused variables in insertEventsInTable().
Added another comment for adding an assert() statement at the end.

done.


TODO: stub event bus/broadcasting package

Event system
Must haves:

Cross context comms (main thread, web worker, shared worker)
Context isolation (main thread only, or this tab only)
Decoupled from any form of persistence
Strong type safety
Strong debug tooling / observability

Nice to haves

Interop with redux dev tools for introspection

Applicable areas to start with:

Offscreen indicators
Map interaction (zoom to, tether, center on)
Baseball card management (get me out of React render cycle driving baseball cards)
Layer interaction (click / hover layer)
Layer management (hide/show layer, change layer order)

Yeah I think we need to chat with the folks that did some micro RfC's like Andrew and bring that into the larger conversation as you noted before we make a decision.


fix: correctly expose not and attr from filter-conditions

Related to issue #277
Correctly expose not and attr function from filter-conditions files to avoid import from dynamodels/dist/src/filter-conditions for consumers.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


EcoCampus: clean up and prepare electricity consumption data of EcoCampus

review provided measured data
Extract profiles for validation of demand simulation or as input data to bypass demand calculation (building level, if possible)

Method:

Remove periods of faulty reporting, i.e. zero for a long time and large number at the end (see picture)

Tip by Dr. Clayton Miller (Building Data Specialist):
https://bitbucket.org/berkeleylab/eetd-loadshape
I'm trying to get this to work at the moment.

@cmiller8 would you mind shedding some light on how to do it please? thanks!

I think we can close this for now.


Add "recent commands' to Command Palette in commandline mode

Description of the new feature/enhancement
The idea is to store the history of the recent commands dispatched from command palette in command line mode. This history will be presented below the search bar. Typing in the search bar will filter the relevant history items (similar to the behavior we already have with filtering commands in action mode).

Usually when I use command palette in the command line mode I do it to run the same commands.
This feature will save me a lot of typing and prevent typos.
Proposed technical implementation details (optional)
I am not sure what is the best place to store the history, and what are the security concerns if exist.
We can start with storing the history in memory for the current session.

I like this! VSCode will show recent commands history in its command palette and I found it useful.

dang this is a good idea
We could certainly start with work with just stashing the recently used commands in memory while a window is open, and then once #7972 is implemented, we could actually write the mru list to disk and load it from there.
I'll stick this in #5400. Thanks!

@zadjii-msft - then I would like to work on this 😊

Go for it!

Implemented in-memory, MRU list (not set) of 10 (hard-coded).
There is no auto-completion, the history entries get filtered out, however-to invoke the entry you need to select the item:


Integrate code coverage and other code quality tools

The project will be using these tools:
Codecov - For test coverage report
CircleCI - For continuous Integration
Codefactor - For tracking code quality


Which part is knowledge distillation

Hello, Author,
Your paper is very nice, thanks for sharing. I have a confusion is that I can't find which part of the codes shows the knowledge distillation.

Hi, the released code is for inference and the knowledge distillation is during training process.

Are there any plans to release the code used for training?

for

hi bro, have you found the distillation part ?  i also have the same confusion ....


Merged hac-dev test reports into the appstudio-qe report portal

Fixes

https://issues.redhat.com/browse/HAC-3020
Description

Merged hac-dev test reports into the appstudio-qe report portal
Updated the reportPortal endpoints in config file.
Type of change

[ ] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/lgtm

/approve


FLEX-2883 ~ Updates version to 4.9.0-SNAPSHOT

This pull request makes the build fail.

test this please

Closing the pull request, until the references have been updated.


Add integration test for staging env w/ Puppeteer

Pull request checklist
Make sure you:

[x] Signed the Contributor License Agreement
[x] Followed the commit message guidelines

Short description of the change(s)
Fix #810

Installs Puppeteer as a dev dependency in package.json.
Requests a collection of webhint.io staging URLs and saves the response in a
Map() object.
Iterates over the Map and checks if is there any status different from 200 or
another unknown error.
If there is an error, throws it to interrupt the test execution

🕹 How to test

[ ] Run the script node helpers/integration-tests.js ✅

[ ] If you want to see the script finding an error you can add a bad URL to the URL collection.
Example: http://pudim.com.br/xunda.jpg

Also, IMO, we should check that running a scan is working too.

Can you please clarify what you mean by "check that running a scan is working"?

@sarvaje Can you please clarify what you mean by "check that running a scan is working"?

Sorry hehehe. I mean going to the /scanner and some url e.g. https://example.com click on run scan and see if you get a status code 200.
Some times, everything looks good, but then when you scan an url, something is broken.

There are a couple methods in puppeteer that make this relatively easy:

page.type: it accepts a css selector and the text to type on the input
page.click which accepts a css selector and will "click" on it

Because we are going to navigate to a page we will probably have to use page.waitForNavigation to make sure that we navigate to a results page and get a 200 status code. The docs page has a good example on how to use waitForNavigation in combination with click and such but I think it will be something similar to:
await page.goto('/scanner/');
await page.type('selector for input', 'url to analyze');
const [result] = await Promise.all([
  page.waitForNavigation(),
  page.click('selector for button')
]);

// check result here

@molant I've rebased from upstream master and did the following:

Refactored the code to use async/await syntax sugar instead of Promise chains
Isolated the scanner test

Let me know if the changes sound good to you.
Thanks!

Sorry, @molant forgot again to remove. Now I believe everything looks fine!

@molant Looks like we two approvals now. Is there anything else you wanted or is this good to merge?

Good to merge. Haven't had the time to merge today. Can you please take care of it?

From: Tony Ross<EMAIL_ADDRESS>Sent: Monday, October 14, 2019 13:20
To: webhintio/webhint.io
Cc: Antón Molleda; Mention
Subject: Re: [webhintio/webhint.io] Add integration test for staging env w/ Puppeteer (#843)
@molanthttps://github.com/molant Looks like we two approvals now. Is there anything else you wanted or is this good to merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/webhintio/webhint.io/pull/843?email_source=notifications&email_token=AAEUDATVVJGTGFPJRANNVBTQOTIBFA5CNFSM4I5YVAX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBGLQWA#issuecomment-541898840, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEUDAVEGYOAST5C2AG376DQOTIBFANCNFSM4I5YVAXQ.


Terra chains

Hello, I wanted to check with you before submitting the PR. If I submit a PR changing the current terra identifiers to terra-classic and terra2 to terra, would it be accepted? Terra phoenix-1, the chain was never meant to be branded at Terra2, and this can misslead users. I already submitted some prs to other repos like the one for ping.pub, to start harmonizing everything everywhere. Thanks!

No I don't think we should change this. The chain_name is meant to be permanent (expect some testnets have changed) and any change could have far reaching consequences due to the dependencies that exist. The chain_name terra2 is only meant to be a Registry key, and nothing that end users should be exposed to.

Thank you for the answer @JeremyParish69, it makes a lot of sense to be a key identifier and a very important value that shouldn't be mutable.


Implement frame animation settings in encoder API

Writes the frame duration fields and frame name
Blending and cropping fields not yet implemented

PTAL: default initialization now follows the spec defaults


fix: remove hardcoded IRSA reference from README

Signed-off-by: Abhinav Sinha<EMAIL_ADDRESS>

Just noticed another PR #36 opened for the same issue 🤦♂


request on using

I'm a newbie so can you make a small / quick video tutoring how to use this !

If I make a video, I will include the url with this project.


The name movieGenre does not exist in this context

Due to differering captialisation, the above error appears while trying to follow along with this page, in particular the genre part. Either change public string MovieGenre { get; set; } to have a lowercase m, or use a capital M in the two movieGenre references in OnGetAsync() method.

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

ID: cc54dbf8-e6fe-a212-b2cd-c015a6ba5439
Version Independent ID: b96b0b44-3e3a-a6c2-6b2b-c7bab4b78f0d
Content: Add search to ASP.NET Core Razor Pages
Content Source: aspnetcore/tutorials/razor-pages/search.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Ignore this, I forgot to include the relevant part of the form in the .cshtml page.


Comprehensive code styleguide adherence

As a user, I should have the entire rimble-ui source formatted to comply with our coding styleguide standards.

Should have all files in the /src directory formatted with prettier

Resolved by #190


cmd/lncli: add --with flag to list channels

This commit adds a flag to listchannels that filters by remote pubkey.

we don't typically add tests for lncli since it's predominately a debugging tool. I have tested against my node though:

list channels --with

Why the decision to do this client side, while other filtering is taking place server side?

@joostjager didn't seem performance critical, plus rpc users can do this fairly easily. regardless, changed it to filter internally, ptal.


Automating of data collection and training of models

Setup code for automating the running of models

It is a good to have - will start after 09/02/2023 - should take max a week o finish


Fixed client builder tests that became flaky with the new version of Azure Core.

Started seeing errors such as this one when running from source tests for Azure Core. It turns out there were some performance enhancements in that library that made it so we could retry a request in a shorter amount of time and, since the HTTP date header has an accuracy of 1 sec, it made this test fail as it expects a different date altogether with each retry attempt.

API change check for com.azure:azure-data-tables
API changes are not detected in this pull request for com.azure:azure-data-tables


Error on strt up

User can not start game he gets an error
Log:
https://github.com/vicblack11/FAF/commit/490f7f6515ce44c260bd27dd75cc8fe457816ebe

This happens when u delete your client.pefs

Also selecting game folder does not help

So this bug breaks the client for new players

it has to do with the installationPath in the prefs not being set

i set teh game path again and then it worked


Config Storage environment

Config storage for development and production environment
Production environment use AWS S3.
Development environment use local storage

@fry1999 review pull cho anh


New places api

@FaridSafi Coming from this issue, this PR attempt to support Autocomplete (New) and Place Details (New) API. Also, this PR adds sessionToken support.

Awaiting your confirmation to proceed with the merge

Thank you @rojiphil, merged and deployed. Could you please double check if the published version is OK ?

Thanks @FaridSafi! I see the version published on NPM


matrix builds for redis

test redis backend

Current coverage is 100% (diff: 100%)

Merging #3 into master will not change coverage

@@           master    #3   diff @@
===================================
  Files           2     2
  Lines          57    57
  Methods         0     0
  Messages        0     0
  Branches        6     6
===================================
  Hits           57    57
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 6157469...3970558

Coverage remained the same at 100.0% when pulling 397055833ad84ae0d4e64353cf5ec6b66d293978 on test-other-backends into<PHONE_NUMBER>86911ca4dcc356142a7785ed9ba1da on master.


Instruct error

Description
ERROR:    Exception in ASGI application
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/uvicorn/protocols/http/httptools_impl.py", line 436, in run_asgi
result = await app(  # type: ignore[func-returns-value]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in call
return await self.app(scope, receive, send)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/fastapi/applications.py", line 276, in call
await super().call(scope, receive, send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/applications.py", line 122, in call
await self.middleware_stack(scope, receive, send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/middleware/errors.py", line 184, in call
raise exc
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/middleware/errors.py", line 162, in call
await self.app(scope, receive, _send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/middleware/cors.py", line 92, in call
await self.simple_response(scope, receive, send, request_headers=headers)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/middleware/cors.py", line 147, in simple_response
await self.app(scope, receive, send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 79, in call
raise exc
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 68, in call
await self.app(scope, receive, sender)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in call
raise e
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in call
await self.app(scope, receive, send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/routing.py", line 718, in call
await route.handle(scope, receive, send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/routing.py", line 276, in handle
await self.app(scope, receive, send)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/starlette/routing.py", line 66, in app
response = await func(request)
^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/fastapi/routing.py", line 237, in app
raw_response = await run_endpoint_function(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/fastapi/routing.py", line 163, in run_endpoint_function
return await dependant.call(**values)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/timur/Downloads/Agent-LLM-1.1.3-alpha/app.py", line 197, in instruct
response = agent.run(
^^^^^^^^^^
File "/Users/timur/Downloads/Agent-LLM-1.1.3-alpha/AgentLLM.py", line 99, in run
instruction_prompt = self.INSTRUCT_PROMPT
^^^^^^^^^^^^^^^^^^^^
AttributeError: 'AgentLLM' object has no attribute 'INSTRUCT_PROMPT'

Steps to Reproduce the Bug

Start back and front ends
Go to the agent page
Write instruction (mine is Write tweet about AI)
Click on Instruct agent

Expected Behavior
Instructs agent
Actual Behavior
Throws an AttributeError: 'AgentLLM' object has no attribute 'INSTRUCT_PROMPT' error
Additional Context / Screenshots
No response
Operating System

[ ] Microsoft Windows
[X] Apple MacOS
[ ] Linux
[ ] Android
[ ] iOS
[ ] Other

Python Version

[ ] Python <= 3.9
[ ] Python 3.10
[X] Python 3.11

Environment Type - Connection

[X] Local
[ ] Remote

Environment Type - Container

[ ] Using Docker
[X] Not Using Docker

Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of Agent-LLM.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Fixed in #207, please update.


Problem with example code

Trying to use your code example gives me the following error:
Traceback (most recent call last):  File "/home/pi/epm/myportainer.py", line 8, in <module>  portainer = Portainer(portainer_ip="<IP_ADDRESS>", portainer_port=9443, username="admin", password="besmalag1979") TypeError: Portainer.__init__() missing 1 required positional argument: 'session'
My example code looks like:
#!/usr/bin/python3
# -*- coding: utf-8 -*-
import asyncio

from portainer import Portainer
from portainer.exceptions import PortainerException

portainer = Portainer(portainer_ip="<IP_ADDRESS>", portainer_port=9443, username="admin", password="123456789")
try:
    asyncio.run(portainer.login())
except PortainerException:
    print("Error")

endpoints = asyncio.run(portainer.get_endpoints())
if endpoints is None:
    print("No endpoints found")
else:
    asyncio.run(endpoints[0].refresh())
    #asyncio.run(endpoints[0].docker_container["hello-world"].get_image_status())
    #asyncio.run(endpoints[0].docker_container["hello-world"].get_stats())
    #asyncio.run(endpoints[0].docker_container["hello-world"].recreate())

Hey, This is very much a work in progress, I started working on this, but didn't have had much time to work on it.
I will look at it if i find some time


SoftwareDev: A Book Library Android app using Kotlin

Pls assign this to me if possible.
Thanks

Pls assign to me


Extension method for calling health check of other services

Codecov Report

Merging #159 (b5fe3ef) into main (c842a55) will increase coverage by 7.74%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #159      +/-   ##
==========================================
+ Coverage   45.82%   53.57%   +7.74%
==========================================
  Files          38       50      +12
  Lines         491      784     +293
  Branches       65      101      +36
==========================================
+ Hits          225      420     +195
- Misses        255      344      +89
- Partials       11       20       +9

Flag
Coverage Δ

appcommon
43.94% <0.00%> (-1.88%)
:arrow_down:

logging
78.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...tics/HealthChecks/HealthChecksBuilderExtensions.cs
0.00% <0.00%> (ø)

...eLiveEndpointHealthCheck/ServiceLiveHealthCheck.cs
0.00% <0.00%> (ø)

...eware/Storage/RequestResponseLoggingBlobStorage.cs
0.00% <0.00%> (ø)

.../RequestResponseMiddleware/Models/IndexTagsKeys.cs
100.00% <0.00%> (ø)

...ource/RequestResponseMiddleware/JwtTokenParsing.cs
85.71% <0.00%> (ø)

...Middleware/Extensions/FunctionContextExtensions.cs
55.00% <0.00%> (ø)

...source/RequestResponseMiddleware/Models/LogTags.cs
90.69% <0.00%> (ø)

...RequestResponseMiddleware/Models/LogInformation.cs
100.00% <0.00%> (ø)

...urce/SampleApp/Functions/FeatureFlaggedFunction.cs
0.00% <0.00%> (ø)

.../RequestResponseMiddleware/ResponseStreamReader.cs
100.00% <0.00%> (ø)

... and 4 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Dreamview:Implement the v2x feature

实现效果

Also I am not familiar with dreamland front end, so please correct me if I am wrong: Why the perception information (v2x) is not from the /apollo/v2x module, it looks like your are still updating and subscribing from the /apollo/perception module? https://github.com/ApolloAuto/apollo/blob/f4f21841fc760cd838292caf8bd1a7a23c5d7b20/modules/dreamview/backend/simulation_world/simulation_world_service.cc#L269

Also I am not familiar with dreamland front/back end, so please correct me if I am wrong: Why the perception information (v2x) is not from the /apollo/v2x module, it looks like your are still updating and subscribing from the /apollo/perception module?

嗯嗯，刚才确认了一下邮件，需求是基于percepetionobstacle扩展，直接加在obstacle的信息里，所以这个内容加在了apollo/perception/module

Please write commit msg with the form dreamview: blablablah next time, so that we can differentiate it from commits for other modules.
下次提交时，请按照dreamview: blahblahblah的格式写提交信息，方便大家区分不同模块的提交。
Thank you.

好的收到！谢谢学长提醒
------------------ 原始邮件 ------------------
发件人: "Liu<EMAIL_ADDRESS>发送时间: 2020年8月29日(星期六) 上午8:19
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [ApolloAuto/apollo] Dreamview:Implement the v2x feature (#12262)
Please write commit msg with the form dreamview: blablablah next time, so that we can differentiate it from commits for other modules.
下次提交时，请按照dreamview: blahblahblah的格式写提交信息，方便大家区分不同模块的提交。
Thank you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


fix: rename box-shadow-control SCSS mixin to pgn-box-shadow

Description
This is a new SCSS mixin, which gives consumers the ability to add elevation (i.e., elevation) based on the pre-defined elevation levels in addition to a direction/side (e.g., top, right, centered).
Deploy Preview
n/a (SCSS change only. There probably should be an Elevation foundations page in the documentation website. Taking an action item to file this ticket).

Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New Feature: How to make sure that estimated tripping_curt_energy doesn't include VVAr and VWatt? - 34

As stated in the title.

It should not be. According to Baran, tripping cannot happen at the same time with VVAr or VWatt.


IconSmooth on grid load issue

If you load in a grid next to you the icon smoothing doesn't work properly.
For me I just use `callemergencyshuttle on the evac shuttle PR and you can see the back half doesn't smooth until you go out of range and go back in range again.

not a thing anymore


Redundant version check

Guys,
There are the following lines of code in OServerCommandPutDocument
if (currentDocument.isDirty()) {
        currentDocument.getRecordVersion().copyFrom(doc.getRecordVersion());
        currentDocument.save();
      }

As a result, to avoid exceptions request must contain @version. I think it's redundant: especially in case of partial update... Keeping of version number on client side is quite strange requirement for this API.
P.S. I'm sure that the same logic might be for another REST APIs... Please review requirements for @version specification in REST API.

In case of partialUpdateMode I agree this has not much sense. About the rest, if you provide the version it should be checked. This allows the user to manage MVCC correctly.

I think this is more appropriate:
 if (doc.getRecordVersion().isValid())
          // OVERWRITE THE VERSION
          currentDocument.getRecordVersion().copyFrom(doc.getRecordVersion());

This fix is also incorrect:( If version was not specified it's equal to 0. And 0 is a valid version number. But specifing @version: -1 every time is also redundant, isn't it?

If you don't set any "@version" field the last version is taken, because it's loaded from disk.

I meant something another. When you send PUT update by json and without '@version' - default version of a created ODocument from that JSON will be zero. And once 0 is a valid version number - OrientDB will fail to set version 0 to existing document.
It's tested and it's failing (if version was not set). I can't send a testcase: it's build on wicket - so it will be hard for to reproduce, but its logic is sime: send PUT update without version to existing document.

Fixed. Created also the test case: f5797fd9219d282d40ef73cd1ab485cde5c00f53


Attempt to fix the missing index.html in canonical URLs

Summary of the changes / Why this is an improvement
This should add the missing index.html for canonical URLs (original issue: https://github.com/crate/tech-writing/issues/417#issuecomment-2066746951). Can we do a demo to test thoroughly before going live? Tested locally, but not sure what happens when RTD is trying to add their own code, could be either ignored or this piece overwritten.
Code is borrowed from https://github.com/readthedocs/readthedocs-sphinx-ext/blob/614109c6ad11eaacbcde1040b908967c80b81eba/readthedocs_ext/_templates/readthedocs-insert.html.tmpl#L4-L19

Superseded by GH-488.


Fix edge case in color-mode.js

Description
Our team found a potential edge case while working on our Bootstrap fork.
When using the current version of color-mode.js:
  const setTheme = theme => {
    if (theme === 'auto' && window.matchMedia('(prefers-color-scheme: dark)').matches) {
      document.documentElement.setAttribute('data-bs-theme', 'dark')
    } else {
      document.documentElement.setAttribute('data-bs-theme', theme)
    }
  }

The results will be:

color-mode on device
color-mode on Bootstrap
Result in HTML

*
dark
data-bs-theme="dark"

*
light
data-bs-theme="light"

dark
auto
data-bs-theme="dark"

light
auto
data-bs-theme="auto"

At this point you may have an edge case when your device is set to 'Light mode' and you use the 'Auto color mode' on the page because you will have to add more CSS to make something like:
@media (prefers-color-scheme: light) {
  [data-bs-theme="auto"] {
    // your code...
  }
}

Proposed patch:
  const setTheme = theme => {
- if (theme === 'auto' && window.matchMedia('(prefers-color-scheme: dark)').matches) {
-   document.documentElement.setAttribute('data-bs-theme', 'dark')
+ if (theme === 'auto') {
+   document.documentElement.setAttribute('data-bs-theme', (window.matchMedia('(prefers-color-scheme: dark)').matches ? 'dark' : 'light'))
  } else {

Then you will have:

color-mode on device
color-mode on Bootstrap
Result in HTML

*
dark
data-bs-theme="dark"

*
light
data-bs-theme="light"

dark
auto
data-bs-theme="dark"

light
auto
data-bs-theme="light"

And you don't need to add extra CSS everywhere.
Motivation & Context
Fix edge case when your device is set to 'Light mode' and you use the 'Auto color mode' on the page.
Type of changes

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
(na) My change introduces changes to the documentation
(na) I have updated the documentation accordingly
(na) I have added tests to cover my changes
[x] All new and existing tests passed

Live previews

https://deploy-preview-39224--twbs-bootstrap.netlify.app/

please kindly unsubscribe me!
On Wed, Oct 4, 2023, 4:55 PM Louis-Maxime Piton @.***>
wrote:

@.**** approved this pull request.
Thanks for the PR 🎉
I'm not in the JS team but it adds some consistency for color-mode. This
looks safe to me to implement:

This doesn't interfer with any other mode than auto that we've been
adding on Bootstrap.
It adds consistency over the data-bs-theme added on the HTML.
It defaults to the two commons color modes proposed by every
browsers/OS.

—
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/39224#pullrequestreview-1657904848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A324RXXBI2E2TTDDT5HTGPTX5WBG7AVCNFSM6AAAAAA5DA47W2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNJXHEYDIOBUHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Fix race in os sandbox sharing

There is a race in os sandbox sharing code where two containers
which are sharing the os sandbox try to recreate the os sandbox
again which might result in destroying the os sandbox and recreating
it. Fixed the race by checking if the sandbox already exists and
creating it if it doesn't exist inside a lock. This ensures that
exactly one sharing container creates the os sandbox and others
simply use it.
Fixes docker/docker#17577
Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

@mrjana thanks. pls correct me if am wrong. the sharing of OSL sandbox happens only for --net=host mode. and we handle the situation very similar to any other case and hence we have less special casing (though we do make use of useDefaultSandBox as a hint for --net=host case).
So my question is, is the current fix complete without also protecting the delete side ?
Infact for --net=host case we can even avoid deleting the osl sandbox and also create it only once globally ?
BTW, I tried the patch to see if it solves docker/docker#17577, but it didnt. But when I added a small hack as follows, then I dont see the issue anymore.
 func (n *networkNamespace) Destroy() error {
+       if strings.Contains(n.path, "default") {
+               log.Debugf("Ignoring sandbox destroy")
+               return nil
+       }

So, I think we should either synchronize the creation and deletion of the OSL sandbox. Or prevent destroying the osl sandbox for default sandbox case (and also better if we create it just once globally).

@mavenugo I did test for docker/docker#17577 and it did work for me. But you brought up a good point while the sandbox is being destroyed. Let me refactor this code to just create it once and use it everywhere and never destroy it.

@mrjana thanks

@mavenugo Updated new diffs. PTAL

LGTM.


Feature request: Pass vendor information from device-profile template

Hi,
I could need the vendor (and model) information of a device. The device-profile templates contain vendor information (and sometimes model information in the name).
Would it be possible to pass the vendor information from a device-profile template to the actual device profil?
Or do I just miss any API call to get this information for a device?
Another way I can think of to get vendor/model information into a device profile would be to enter it in description (e.g. as JSON). Is there a better way?
Do you accept PRs for the lorawan-devices-repository you forked from TTN?
Best Regards

I think this should be updated as TTN as some updates that came in.

I could not find license info from their repo however found a contributor license.
Can we use codecs from respective manufacturers github repo which they made available?
From my test none of browan codec are correct. However Laird and Tektelic are still working fine.

@rsebi The related issue: #61

Forgive my ignorance i just went through the whole convo regarding this subject here and across forum posts.
As things stand might @brocaar start accepting for PR for the device profiles given that manufacturers create the codec (with licenses) so that as a community we can start somewhere.
Alternatively @brocaar could provide an insight/update on how to chirpstack plan to move forward on this matter without pulling any troubles.
Some could even reach out to manufacturers (respective use cases) and seek permission for codec use, and thus i believe over time, together we will be able to have a sizable database of devices ready for chirpstack.
Given I work on chirpstack extensively, would love to spare sometime for community as do others which I am quite sure.


OpenSSL 3.2.0 fails to build (01-test_symbol_presence.t) on 32 bit x86 architecture

Hello,
I found build (test) failure on 32 bit x86 architecture, see build log below:
01-test_symbol_presence.t ...............
# The results of this test will end up in test-runs/test_symbol_presence
# NOTE: developer test!  It's possible that it won't run on your
# platform, and that's perfectly fine.  This is mainly for developers
# on Unix to check that our shared libraries are consistent with the
# ordinals (util/*.num in the source tree), and that our static libraries
# don't share symbols, something that should be a good enough check for
# the other platforms as well.
1..3
# Number of lines in @stlib_lines before massaging: 26222
# Number of lines in @shlib_lines before massaging: 5615
# Number of lines in @def_lines before massaging: 5467
# Number of lines in @stlib_lines after massaging: 7837
# Number of lines in @shlib_lines after massaging: 5448
# Number of lines in @def_lines after massaging: 5448
# Number of lines in @stlib_lines before massaging: 5052
# Number of lines in @shlib_lines before massaging: 1147
# Number of lines in @def_lines before massaging: 581
# Number of lines in @stlib_lines after massaging: 1710
# Number of lines in @shlib_lines after massaging: 574
# Number of lines in @def_lines after massaging: 574
# Duplicates:
# OPENSSL_ia32cap_P
not ok 1 - checking no duplicate symbols in static libraries

#   Failed test 'checking no duplicate symbols in static libraries'
#   at test/recipes/01-test_symbol_presence.t line 175.
ok 2 - check that there are no missing symbols in crypto
ok 3 - check that there are no missing symbols in ssl
# Looks like you failed 1 test of 3.
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/3 subtests
01-test_test.t ..........................

This test failure is also relevant for master branch. Apparently it's caused by a duplicate symbol OPENSSL_ia32cap_P but that's all I can find out in  this case. Other architectures seems to be fine. Please let me know if more information is needed.

The test failure can safely be ignored (as the note in the test above indicates).  It appears to be happening because both crypto/cpuid.c and crypto/x86cpuid.S define the OPENSSL_ia32cap_P symbol, and both object files are included in the libcrypto archive file and dso.  The former definition defines the symbol as extern, while the latter defines it as common storage.
nm, which is used for symbol enumeration doesn't differentiate between the two, and falsely reports them as duplicate symbols.

Ah right, we've made a slight mistake.  It's perfectly valid for a C type symbol to appear multiple times, and we're not handling that properly.


Hover prefix does not take effect in webpack build

UnoCSS version
0.58.0
Describe the bug
I want to build a Vue2 UI component library using unocss based on webpack.
Does not take effect when using the hover prefix.
Perhaps my configuration is incorrect? I don't know how to solve the problem.
Reproduction
https://stackblitz.com/edit/xgaa-webpack-unocss?file=webpack.config.js,package.json,uno.config.js,src%2Findex.js,src%2Fcomponents%2FTButton.vue,lib%2Findex.html&terminal=build,start
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

// does not take effect

// does take effect

// does take effect


Export next_input from DialogBaseNode

System Information
Irrelevant
Problem

What is broken? What is not working? Make a clear and concise description of what the bug is.
There's no easy way to select an InputAction to continue to next action.

Workaround
Set it directly through code on _init

There's no next_input in https://github.com/AnidemDex/Godot-DialogPlugin/commit/fd8d4a4f2b1ad4f77530dc4afffa1358477b45ec but it's ok, I'll keep an eye on this issue


Couldn't change highlighting color of parentheses.

"Linedit" was installed through Quicklisp. (linedit-20120703-git)
In my ~/.sbclrc
--- cut--
(when (find-package 'linedit)
 (setf linedit:*highlight-color* :red))

Got error:
debugger invoked on a SIMPLE-ERROR in thread
#<THREAD "main thread" RUNNING {B3E9201}>:
  Conditional expression parser not yet written.

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [ABORT] Exit debugger, returning to top level.

((FLET #:CLEANUP-FUN-25 :IN LINEDIT:LINEDIT)) [cleanup]
0]

When I tried set color manually after linedit was loaded I've got the same error.
CL-USER(1): (setf linedit:*highlight-color* :red)

:RED

debugger invoked on a SIMPLE-ERROR in thread
#<THREAD "main thread" RUNNING {B3E9201}>:
  Conditional expression parser not yet written.

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [ABORT] Exit debugger, returning to top level.

((FLET #:CLEANUP-FUN-25 :IN LINEDIT:LINEDIT)) [cleanup]

Have tested configuration from : https://github.com/0/.../blob/master/sbclrc
Result: Error was the same.

Linedit version 0.17.5
SBCL version <IP_ADDRESS>-989a1d6

This seems a terminfo bug. (terminfo.lisp, that is)
Can you try replacing terminfo.lisp in Linedit with one from https://github.com/npatrick04/terminfo ?
Does the same error still occur?

 When I take terminfo.lisp from https://github.com/npatrick04/terminfo repository the error disappears and (setf linedit:highlight-color :red) working perfect.
Will you do "merge" terminfo files without bugs ?

Fantastic, thanks! I don't actually have a box where I could reasonably test linedit accessible right now, but if you can verify that the test branch:
https://github.com/nikodemus/linedit/tree/test

works, I'll be happy to update the master to that.

I tried to install "tree/test" branch: "terminal and colorize functionality" works fine.
You did an excellent job, THUNX sir Nikodemus!

Thanks, on master now.


toView binding not working after NativeScript v3.0 upgrade

I'm submitting a bug report
Please tell us about your environment:

Operating System:
Windows [7]

Node Version:
6.9.1

NPM Version:
3.10.8

Current behavior:
i have a valueconverter, using toView. this is then defined as an applicationresource.
before NativeScript v3.0, code for all this was like:
var view = require("ui/core/view");
var listOptionsConverter = {
toView: function (value, options) {
//custom code
}
};
appModule.resources["listOptionsConvertez"] = listOptionsConverter;
and when called in a control during binding, it produced the desired results.
with NativeScript v3.0 upgrades, both valueconverter and application resources were changed. currently, the code above does not work. when the control using listOptionsConverter is loaded, following error is generated:
JS: Cannot find function or filter: listOptionsConverter
and the control does not work properly.

Do you have the correct framework? I'm not entirely sure what you are talking about here.

all my components seems to be up to date
nativescript 3.1.2
tns-core-modules 3.1.0
tns-android 3.1.0
npm 3.10.8
node 6.9.1
let me ask a more specific question. in "NativeScript v3.0 Breaking Changes" page (https://github.com/NativeScript/NativeScript/blob/v3.0.0/Modules30Changes.md) it says:
application.resources was removed, use get/set methods:
application.getResources(): any
application.setResources(res: any)

How do i call setResources method? more specifically, how does the following line that i had previously change:
appModule.resources["listOptionsConverter"] = listOptionsConverter;
where listOptionsConverter is:
var listOptionsConverter = {
toView: function (value, options) {
//custom code
}
};

Why are you asking this here? What does it have to do with Aurelia?


Warning: possible change in GH API may cause issues with safe-settings

On latest versions:
When a change is triggered and safe-settings runs for a repository, the first change made is for repository settings. Something like
DEBUG (mark): GitHub request: PATCH https://api.github.com/repos/ORG/REPO - 422
    auto_init: true
    visibility: "internal"
    has_issues: false
    has_projects: false
    has_wiki: false
    allow_squash_merge: true
    allow_merge_commit: false
    allow_rebase_merge: false
    allow_auto_merge: true
    delete_branch_on_merge: true
    org: "REDACTED"

This request, however, has started to fail with the below message:
"data":{"message":"Visibility is already internal.","documentation_url":"https://github.com/pricing"}}
In case its relevant: this request also has the nebula header enabled "headers":{"accept":"application/vnd.github.nebula-preview+json"
I do not know where/how to track down possible changes to GH API that might cause this :/
cc @decyjphr

@decyjphr can you please confirm that this is new and broken? or is this just something I only noticed now but not a blocker?

I have never seen this before and other customers have not seen this either which makes me think this is new. I have asked GitHub engineering if anything changed in the api.

@markjm I tried to recreate this with a test repo., but couldn't. Here is the output from the logs
DEBUG (event): Updating repo with settings ["blue","green","yellow","red","uber","newone","green","blue"] {"mediaType":{"previews":["nebula-preview"]},"name":"test","org":"dec","force_create":true,"description":"description of test repository","homepage":"https://newhome.github.io/home","visibility":"internal","auto_init":true,"has_issues":true,"has_projects":true,"has_wiki":true,"has_downloads":true,"allow_squash_merge":true,"allow_merge_commit":false,"allow_rebase_merge":false,"default_branch":"develop","repo":"test","owner":"dec"}
    id: "c7198708-1789-11ed-8577-96afb59dcb6b"
DEBUG (test): GitHub request: PATCH https://api.github.com/repos/dec/test - 200
    org: "dec"
    force_create: true
    description: "description of test repository"
    homepage: "https://newhome.github.io/home"
    visibility: "internal"
    auto_init: true
    has_issues: true
    has_projects: true
    has_wiki: true
    has_downloads: true
    allow_squash_merge: true
    allow_merge_commit: false
    allow_rebase_merge: false
    default_branch: "develop"

I'll try different combinations to recreate your error. Let me know if I am missing anything. The settings are:
repository:
  name: test
  org: dec
  force_create: true
  description: description of test repository
  homepage: https://newhome.github.io/home
  topics:
  - blue
  - green
  - yellow
  - red
  visibility: internal
  auto_init: true
  has_issues: true
  has_projects: true
  has_wiki: true
  has_downloads: true
  allow_squash_merge: true
  allow_merge_commit: false
  allow_rebase_merge: false
  default_branch: develop

@decyjphr was this repo internal before you ran? can you try running on the same repo again now that is internal to see if you can "reapply" those same configs on the same repo?
I can even replicate with a simple raw htttp call

@markjm Yes, the repo was already internal.  In your API test, can you see if the error happens if there is another change in the settings -- in my case, I change the homepage URL? I am using the code in suborg-scalability branch and it requires there be a change in the repo settings and will not call the API when there are no changes in the repo settings. It could be the reason why I am not seeing this issue.

using the yadhav/safe-settings:2.0.22-rc.9 image for the safe settings app with debug enabled, I'm getting a similar error (I've anonymized the repo and org name below)
{"level":20,"time":1689054921716,"pid":24,"hostname":"cab73abe3cfb","name":"probot","name":"probot","name":"github","private":true,"visibility":"private","default_branch":"develop","allow_rebase_merge":false,"allow_auto_merge":true,"delete_branch_on_merge":true,"name":"myrepo-beta","org":"my-org","msg":"GitHub request: PATCH https://api.github.com/repos/my-org/myrepo-beta - 422"}


Update a production Sqlite database after publish.

Steps to reproduce
dotnet publish <args> // creates the following directory structure

artifacts/
    App.dll
    Database.sqlite
    The-rest-of-the-publish-output
    web.config
App/
    Migrations/
    Program.cs
    project.json
    Startup.cs

dotnet ef --assembly ..\artifacts\App.dll --startup-assembly ..\artifacts\App.dll database update

The issue
Unexpected value '..\artifacts\App.dll' for option 'assembly'

Further technical details
We have also tried other ways to run dotnet ef database update on the compiled project but have not been able to determine how to update the database.

dotnet ef needs to work on a project, and not assemblies. The --assembly argument (and other arguments) are an implementation detail of how dotnet ef called ef.exe to run the operations. dotnet ef shouldn't even allow you to specify these - and we are going to fix that.
We do have an issue tracking the ability to have commands that work on assemblies, rather than a project - https://github.com/aspnet/EntityFramework/issues/646. You could look at using ef.exe, but it's not a supported scenario as yet, so you'd be mostly on your own working it out.
We do also have a higher level issue tracking a better story for the scenario of deploying migrations https://github.com/aspnet/EntityFramework/issues/6313.
For the moment, the best option is probably to use the commands to create a deployment script during build (dotnet ef migrations script --idempotent) and then run the script during deployment.


Radzen Textbox Compoment binding Problem .

Describe the bug
After binding a value to with radzentextbox i get a  Object reference not set to an instance of an object.
Required Step to reproduce
`
<RadzenTextBox<EMAIL_ADDRESS>Change=@(args => OnChange(args, "TextBox")) Placeholder="Product ..." class="Textbox01" @onkeydown="OnTextBoxKeyDown" @ref="box">
public Product ProductItem = new Product();

private async Task OnTextBoxKeyDown(KeyboardEventArgs e)
{
	if (e.Key == "Enter" || e.Code == "NumpadEnter")
	{
		Search();
	}
}

public async void Search()
{
	condition = "1=1";

	condition = condition + " AND ProductName like '%" + ProductItem.ProductName.Replace("'", "") + "%'" + " OR (Bare_Code = '" + ProductItem.ProductName.Replace("'", "") + "')";
	//Console.WriteLine("cond is " + condition);
	Products = await ProductService.GetAllProduct(condition, "ProductName");
	StateHasChanged();

}
void OnChange(string value, string name)
{
	Console.WriteLine($"{name} value changed to {value}");
}
protected override async Task OnAfterRenderAsync(bool firstRender)
{
	if (firstRender)
	{
		await box.Element.FocusAsync();
	}
}

`
here is the video
vod
after some debugging i found out that on the first enter ProductItem.ProductName is empty after adding some Task.Delay in (OnTextBoxkeyDown) Function Somehow it manage to bind to the value
private async Task OnTextBoxKeyDown(KeyboardEventArgs e) 	{ 		await Task.Delay(500); 		if (e.Key == "Enter" || e.Code == "NumpadEnter") 		{ 			Search(); 		} 	}
vod

OS: [Windows]
Browser [chrome]
Version [4.16.4]

Hey @tn-dev-fl,
ProductItem is most probably null in your case. Please use our forum for such questions.


Only update the rbush item if the extent has changed

To be able to update a feature property while iterating. Example:
source.forEachFeature(function(feature) {
  feature.set('foo', 'bar');
});

The feature was lost during the migration to CommonJS style modules in #2867.
See also #2219

Are you sure that the extent change works while iterating? Previously we threw exceptions in this case.

If the extent is modified while iterating an exception will be thrown in remove.
The exception could be thrown in update (as before)

An exception is now thrown if the extent is modified while reading

I think this is good now. I just restarted Travis to see if she agrees. But if she does, please merge.

Thanks for the review


Projection plane tilt

Adding a projectionPlaneTilt property to Camera.ts, to tilt the projection plane about the X axis
Adding an applyVerticalCorrection method to Camera.ts to automatically tilt the projection plane so it compensates the camera pitch, and vertical lines appear vertical on the screen

I'll post on the forum a FYI detail of the maths

Playground : http://localhost:1338/Playground/index-local.html#L20FJ4#15


install fails on Python 3.7

# python3.7 --version
Python 3.7.0a1+

# python3.7 setup.py install
Unable to find pgen, not compiling formal grammar.
Traceback (most recent call last):
  File "setup.py", line 278, in <module>
    **setup_args
  File "/usr/local/lib/python3.7/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/usr/local/lib/python3.7/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/usr/local/lib/python3.7/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/install.py", line 67, in run
    self.do_egg_install()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/install.py", line 109, in do_egg_install
    self.run_command('bdist_egg')
  File "/usr/local/lib/python3.7/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/local/lib/python3.7/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/bdist_egg.py", line 152, in run
    self.run_command("egg_info")
  File "/usr/local/lib/python3.7/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/local/lib/python3.7/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 279, in run
    self.find_sources()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 306, in find_sources
    mm.run()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 533, in run
    self.add_defaults()
  File "/usr/local/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 562, in add_defaults
    sdist.add_defaults(self)
  File "/usr/local/lib/python3.7/distutils/command/sdist.py", line 228, in add_defaults
    self._add_defaults_ext()
  File "/usr/local/lib/python3.7/distutils/command/sdist.py", line 311, in _add_defaults_ext
    build_ext = self.get_finalized_command('build_ext')
  File "/usr/local/lib/python3.7/distutils/cmd.py", line 299, in get_finalized_command
    cmd_obj.ensure_finalized()
  File "/usr/local/lib/python3.7/distutils/cmd.py", line 107, in ensure_finalized
    self.finalize_options()
  File "/usr/local/src/cython/Cython/Distutils/build_ext.py", line 21, in finalize_options
    self.distribution.ext_modules, nthreads=nthreads)
  File "/usr/local/src/cython/Cython/Build/Dependencies.py", line 911, in cythonize
    ctx = c_options.create_context()
  File "/usr/local/src/cython/Cython/Compiler/Main.py", line 590, in create_context
    self.cplus, self.language_level, options=self)
  File "/usr/local/src/cython/Cython/Compiler/Main.py", line 75, in __init__
    from . import Builtin, CythonScope
  File "/usr/local/src/cython/Cython/Compiler/CythonScope.py", line 5, in <module>
    from .UtilityCode import CythonUtilityCode
  File "/usr/local/src/cython/Cython/Compiler/UtilityCode.py", line 3, in <module>
    from .TreeFragment import parse_from_strings, StringParseContext
  File "/usr/local/src/cython/Cython/Compiler/TreeFragment.py", line 17, in <module>
    from .Visitor import VisitorTransform
  File "/usr/local/src/cython/Cython/Compiler/Visitor.py", line 15, in <module>
    from . import ExprNodes
  File "/usr/local/src/cython/Cython/Compiler/ExprNodes.py", line 2876
    await = None

Thanks

Similar issue with
Package       Version
------------- -------
Cython        0.29.6
pip           18.1
pkg-resources 0.0.0
setuptools    40.6.2

 python -V
Python 3.7.3

      File "/tmp/pip-install-i10tgm43/cython/Cython/Compiler/Visitor.py", line 15, in <module>
        from . import ExprNodes
      File "/tmp/pip-install-i10tgm43/cython/Cython/Compiler/ExprNodes.py", line 2876
        await = None
              ^
    SyntaxError: invalid syntax

Check your pip output. You are not installing Cython 0.29.6 here.


Mefx like command line tool

Codecov Report

Merging #271 (8d1f94a) into main (14c11e1) will decrease coverage by 2.40%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #271      +/-   ##
==========================================
- Coverage   89.63%   87.23%   -2.41%
==========================================
  Files          87        3      -84
  Lines        6023       47    -5976
==========================================
- Hits         5399       41    -5358
+ Misses        624        6     -618

Impacted Files
Coverage Δ

...icrosoft.VisualStudio.Composition/PartDiscovery.cs

...rc/Microsoft.VisualStudio.Composition/Utilities.cs

...VisualStudio.Composition/PartDiscoveryException.cs

...mposition.AppDomainTests/ExternalExportOnMember.cs

src/Microsoft.VisualStudio.Composition/Export.cs

...isualStudio.Composition/ImportDefinitionBinding.cs

...soft.VisualStudio.Composition/ComposableCatalog.cs

...udio.Composition/ExportProvider+NonSharedExport.cs

...peReceiver/PartThatImportsLazyOfNonEmbeddedType.cs

...ualStudio.Composition/MetadataViewClassProvider.cs

... and 74 more

Continue to review full report at Codecov.

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

It looks like you somehow changed the chmod attributes of the init.ps1 file:

Can you please revert that? You can probably do it with git checkout origin/main -- :/init.ps1

nit: can you rename the doc/Images directory to doc/images? We use lowercase directory names as a convention.
On Windows, a case-only git rename can be tricky, but you should be able to do it like this:
git rm --cached doc/Images/*
git add doc/images/*

Devesh, maybe this regressed when you removed the custom IAssemblyLoader in your tool, but when I try your tool at the CLI, it fails to load the assembly:
dotnet run -- --file C:\git\MEF\bin\Tests\Debug\net6.0\CarOne.dll C:\git\MEF\bin\Tests\Debug\net6.0\CarTwo.dll

It looks like .NET Core rejects it because only the assembly name is passed to Assembly.Load, and .NET Core ignores the codebase path. Did you have this working at some point in the past? If so, can you fix it?

Note: please git pull before making more changes as I've added to your PR.

Hello, As you correctly guessed the issue was because of the fact that I removed the CustomAssemblyLoader. This custom loader would create a custom load context and then load the requested files in the custom load context. The change meant that the program defaulted to using the Resolver.DefaultInstance which in turn used the StandardAssemblyLoader.
The issue is that the standard loader calls Assembly.Load which completes ignores the path of the file. To load the file from the file path, want to use Assembly.LoadFrom as that allows us to specify a file path. Thus I created another custom loader called LoadUsingPathFirst that loads the file using Assembly.LoadFrom if a file path is specified and using Assembly.Load if not.
Doing so meant that when I run the following command on the command line dotnet run --parts --file "<path to vs-mef folder>\bin\Tests\Debug\net6.0\CarOne.dll I get the desired output of
Parts in the Catalog are
CarOne.FirstCar
CarOne.InvalidType
CarOne.MoreMetadata

Since to load any assemblies in Mefx a file path must be specified, this means that we will always have a file path for every assembly we want to load.  This means that in reality, we will always load assemblies using the Assembly.LoadFrom method.  Going through the documentation, I found that

The load-from context contains assemblies for which the user provided a path that is not included in probing. LoadFrom, CreateInstanceFrom, and ExecuteAssembly are examples of methods that load by path.

I was wondering how much of an issue is the fact that LoadFrom doesn't use the default load context is in this case and what steps would you recommend to rectify it?

I was wondering how much of an issue is the fact that LoadFrom doesn't use the default load context is in this case and what steps would you recommend to rectify it?

There are ways to rectify it (using AppDomain.AssemblyResolve event handlers) but it's very unnatural, and I don't see any problem with using the LoadFrom context in this case.
Note that some assemblies will actually load in the default context though -- assemblies that load implicitly because the ones we explicitly load reference them. Any assembly in the GAC or in the tool's own exe directory are candidates for loading automatically. That's as it should be, but just something to be aware of.
And speaking of assemblies loading implicitly, that's the next major work item to resolve in this new tool from my testing yesterday. I tried passing just one relatively simple dll from the VS installation to this tool and it failed with many assembly load errors. We need to be able to pass an .exe.config file to the tool so that it can resolve assemblies the same way a running app would. I have a (non-trivial) class that already does this for another purpose, so I'll wire it in as a switch to your tool and add to your PR.

@sarda-devesh In playing with this more, I find that all switches after --file are interpreted as file args to that one parameter instead of potentially switches of their own. In fact looking at your tests I see evidence that you may have encountered this as well, because --file is always the last switch specified in your test data files.
The --file switch is defined as a List<string>?, as is many other parameters. Does the CLI args parser library we're using even support specifying multiple parameters with more than one argument each?
I'm just information gathering at this point. We may have a few options moving forward.


feat(bt) - Enable use of RESERVED bits in COD (IDFGH-13374)

This is a very niche-case patch, one that most people should never have need to bother with.
But in my case, trying to make a replacement for an obsolete Bluetooth board, I ran into an issue with a nonstandard use of the BT COD field: the two low "reserved" bits were used!  Without access to the controlling firmware (and having to maintain compatibility with the thousands of "obsolete Bluetooth boards" in the field which will not function without said "reserved" bits being transmitted in the outgoing COD), this is the neatest solution to the problem.
I did a deep dive straight into the Bluedroid stack, and found exactly where (amongst the multiple unnecessary data type changes the provided COD went through) the two bits were getting lost.  The very simple changes here were successful in allowing the ESP32 to work as a drop-in replacement for the obsolete board.

 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.

Hi, @WebDust21 ,Thank you for your submission! We really appreciate it.  And there are two places that need to be modified:

Commit message: Suggest to modify it to fix(bt): Enable use of ......
There are three commits, suggest to merge them into one commit.

Hi, @WebDust21 ,Thank you for your submission! We really appreciate it. And there are two places that need to be modified:
1. Commit message: Suggest to modify it to `fix(bt): Enable use of ...... `

2. There are three commits, suggest to merge them into one commit.

I'm sorry, I'm not a Github whiz...I don't know how to do either of these.
from what I read, it should be easy for your side to "squash" the commits (I do not have the option).
I do not know how to change a commit message...although as this PR extends nonstandard functionality, I would consider this a "feature", and not necessarily a "fix".

from what I read, it should be easy for your side to "squash" the commits (I do not have the option).
I do not know how to change a commit message...although as this PR extends nonstandard functionality, I would consider this a "feature", and not necessarily a "fix".

I can't directly modify your commit message on GitHub. It needs to be modified on your side and changed to the correct format, otherwise it can't be merged.
According to the commit message format, this change should be considered a fix type.

You can modify it according to the following steps:
Run the command:  git rebase -i 41dd1a351b4f8630a0b9dbdd87a6bf782094dd66
After entering the following interface, change the last two pick to s

:wq Save and exit to enter the following interface

Then modify the commit information like this:

:wq Save and exit, then push it.

Hi, There are two suggestions above. I‘m sorry that if you don't modify it to the correct format, it cannot be merged. And This PR will be closed after a week.

But in my case, trying to make a replacement for an obsolete Bluetooth board, I ran into an issue with a nonstandard use of the BT COD field: the two low "reserved" bits were used! ...

Hi, @WebDust21 In the latest Bluetooth protocol, the two least significant bits are fixed to 0b00. Could you please tell me what the model of  Bluetooth board are you using?  And which version of the Bluetooth protocol are you referring to? Thank you!

from what I read, it should be easy for your side to "squash" the commits (I do not have the option).
I do not know how to change a commit message...although as this PR extends nonstandard functionality, I would consider this a "feature", and not necessarily a "fix".

1. I can't directly modify your commit message on GitHub. It needs to be modified on your side and changed to the correct format, otherwise it can't be merged.

2. According to the commit message format, this change should be considered a fix type.

You can modify it according to the following steps:
Run the command: git rebase -i 41dd1a351b4f8630a0b9dbdd87a6bf782094dd66 After entering the following interface, change the last two pick to s  :wq Save and exit to enter the following interface  Then modify the commit information like this:
:wq Save and exit, then push it.

Thank you for the instructions; they would work if I had forked a local copy of the entire ESP-IDF to my computer.  However, I just made the edits from Github.com, with no local copy--and as such, there is no place to run the commands.

But in my case, trying to make a replacement for an obsolete Bluetooth board, I ran into an issue with a nonstandard use of the BT COD field: the two low "reserved" bits were used! ...

Hi, @WebDust21 In the latest Bluetooth protocol, the two least significant bits are fixed to 0b00. Could you please tell me what the model of Bluetooth board are you using? And which version of the Bluetooth protocol are you referring to? Thank you!

Like I mentioned in the very opening comment, this is NONSTANDARD use of the two low "reserved" bits, as required for a compatibility project.  This is for SPP mode (Bluetooth Classic) on an original ESP32; it is not BLE in any way, shape or form.
For whatever reason, the designers of the existing system set the two least significant bits (RESERVED bits) to 0b01--and if the ESP32 doesn't broadcast the COD with the two LSBs set to 0b01, it will be completely ignored by the existing system.
I've tested the modifications here as fully functional in getting the ESP32 to broadcast the necessary Bluetooth COD (although the "read COD" still incorrectly returns the LSBs as 0b00 for some reason.  Need to dig into that further.)

Hi, @WebDust21 You don’t need to modify it anymore.. But in the original implementation, reserved_2 is not part of the minor class and is always set to 0. With your fix, reserved_2 may be non-zero, which changes the API behavior and affects users who don't want to overwrite reserved-2.
So after your commit, I made some changes to extend the functionality of the current API so that the previous API behavior does not change and allow users who want to set the reserved_2 to use the new esp_bt_cod_mode_t code.

Can I see the changes to the code?
It's worth noting that the Bluetooth device scan does return the "reserved_2" bits--which is how I was able to determine that the units used a nonstandard Bluetooth COD.  It just didn't allow setting a COD with those bits used, nor can I read back a hacked COD set (although it does work when set).

@WebDust21 Yes, update to the latest version of master, you can see the changes.

Thanks for contribution again, changes have been with https://github.com/espressif/esp-idf/commit/59066e3edfe22fc11466e0047909bd96d6785dc8. Thanks.


Connect login modal to the backend

Basic Information
BRANCH OFF #12 IF NOT MERGED ALREADY
Create a useAuth hook in the client/src/hooks/auth.ts. It should contain a method login() which when called

Calls the jwt endpoint with the username and password, this will return a token if it's a valid user
Store the token in cookies, which will expire after x time (up to your discretion)
Attach the token to all backend requests with an axios interceptor https://www.thedutchlab.com/en/insights/using-axios-interceptors-for-refreshing-your-api-token . This currently lives in client/src/lib/api.ts
Add an additonal condition: if the backend request fails, try refreshing the token with the corresponding endpoint. Take a look at https://github.com/codersforcauses/csf/blob/b92a97f1656e50b107151af863ee45b166431394/client/src/utils/server.ts#L22 for help (it's in vue but similar concept)

Other Information
There's no need to use tanstack query for this, (unlike the usePings example). You can just create a vanilla hook

Why don't we set the cookie by the server as an http-only to avoid all the headache.? and have the refresh token updated after:

old refresh token is sent to the server AND remaining days till expiry is < 10 days (for example)
or whenever a request is sent

We could have the server set a dummy refresh token with the same expiry date which could be accessed by the client for the sake of determining the user state i.e Login: True | False and just have the hook read the cookie and set the login state or in a context


Doc translation

docs: translate
Tween.js,
EventMixin.js,
Container.js,
Sprite.js,
Text.js,
Stage.js,
Matrix.js
comments to English

fix two translation spelling faults.
and translate Graphics.js comments to English.


Uses the deprecated add-path and set-env command

Hello,
According to GitHub, they have removed the add-path and set-env commands because there was a security issue. More information can be found here: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
add-path should be replaced with addPath and set-env should be replaced with exportVariable. Apparently, it uses @actions/core version 1.2.6. More information: https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-commands-for-github-actions#environment-files
Because it uses the deprecated add-path and set-env commands, our GitHub Actions workflows which use @microsoft/setup-msbuild fail with an error referencing it.
The affected lines in affected files are found here:
https://github.com/microsoft/setup-msbuild/blob/f25dd68be285e78e89e662c0c6deee7f5065cbb3/dist/index.js#L1200
https://github.com/microsoft/setup-msbuild/blob/f25dd68be285e78e89e662c0c6deee7f5065cbb3/dist/index.js#L1184
And these are the failed workflows in our project:
https://github.com/EoflaOE/Kernel-Simulator/actions/runs/376949373
https://github.com/EoflaOE/Kernel-Simulator/actions/runs/376949371
This is a serious issue for us as it makes workflows that don't have ACTIONS_ALLOW_UNSECURE_COMMANDS environment variable set to true unusable, especially for the project owners who seek for security like us.

Well, looks like 1.0.2 fixed it. We'll use this version from now on. Updating all projects...

Glad you found the fix! -- Yes, this was present in the core GitHub library (that pretty much every Action uses) and not directly called by this action's core code.  Either way, this is updated.


Fix typo in the starting-a-project.md (Japanese translation)

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Change (一分 → 一部)
It's a minor one, but I've found a typo in the Japanese version.
(I couldn't find any other issues, due to high-quality translation 😃).
Notes

Corresponding Google Translate (part of your project's brand)

https://translate.google.com/?hl=en&sl=auto&tl=ja&text=part of your project's brand&op=translate

Similar translation in other location

English (part of your project's brand)

https://github.com/github/opensource.guide/blob/main/_articles/starting-a-project.md?plain=1#L240

Japanese (プロジェクトのブランドの一部)

https://github.com/github/opensource.guide/blob/main/_articles/ja/starting-a-project.md?plain=1#L240

It seems the test is failing on accessing kindlegen.s3.amazonaws.com, but I'm afraid I haven't been able to find the solution...

Yeah, the build is failing for reasons unrelated to these changes. Maybe it'll be a transient thing? I'll try to rebuild in a day or so and see if it resolves itself, and if not I'll hunt for other solutions.

Hello, So I was hoping to be able to refrain from asking for help immediately by offering up a lot so brilliant question for assistance in guidance on something I'm working on that can be completely taken as disrespectful or completely unrelated to the task at hand.
With that being said.
It would be greatly appreciated if youassist me with an error code that was provided to me which states, UTXO is missing TXHEX of the input which is required by PSDT for non-Segwit input.

Rest factory


Post-processing of extracted text using heuristics

When adding a document to the system, in either the bulk load from CCLW or when adding individual documents, the system should extract the text from that document. Apply heuristic rules to handle the following layout situations:

Ignoring headers and footers
Ignoring text in tables and figures including their captions
Joining hyphenated words which are common in documents
Lists
Sentences which have been split into separate passages
Subscripts/Superscripts

We need to have reviewed the extracted dataset before tackling this task to make sure that we fully understand all the text quality issues.

Hey team! Please add your planning poker estimate with ZenHub @eurolife @kdutia @opyate

Closing as broken down into #352 #353 #354 #355 #356 #357


chore: release v0.14.2

release v0.14.2

Codecov Report

Merging #198 (2904d9b) into main (5800367) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #198      +/-   ##
==========================================
+ Coverage   82.20%   82.22%   +0.01%
==========================================
  Files          15       15
  Lines         905      906       +1
  Branches      198      198
==========================================
+ Hits          744      745       +1
  Misses         87       87
  Partials       74       74

Impacted Files
Coverage Δ

src/lib/provider-wrapper-utils.ts
88.88% <ø> (ø)

src/providers/ethereumProviderWrapper.ts
76.92% <100.00%> (ø)

src/providers/web3ProviderWrapper.ts
80.00% <100.00%> (+0.37%)
:arrow_up:

Continue to review full report at Codecov.

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


minor spell check and formatting updates

What changes does this PR introduce?
removes typos, cleans up pkg/profiles formatting
What is the current behaviour?
contains typos
What is the new behaviour
no more typos, cleaner formatting
Does this PR introduce a breaking change?
no
Other information:
seawater.F is measuring pressure in decibars not decibels as suggested by the comments. See page 43 of the reference document
Suggested addition to tag-index
minor spell checking and formatting updates

I just saw that cost_profiles.F has many modifications in PR #887, so may be I should not have change all the "K" to "k"
as this will make the merging more tricky. @IvanaEscobar I could revert my last commit and commit again something lighter after (just removing the wrong BARRIER + the minor edits in g_cost.h), just let me know.

Let's roll back the K -> k change. I wanted to make that iterator more descriptive since it's applied to different types of dimensions throughout cost_profiles.F, and it could be misleading when reading through the source code.

I'm addressing a descriptive K in PR #887 already too

LGTM,
Those changes will be covered in #887

OK, will merge this PR later today.


providers/ec2: added EC2_PUBLIC_HOSTNAME attribute

Added attribute EC2_PUBLIC_HOSTNAME for the EC2 provider

Forgot to say: I checked API compatibility too, this attribute was introduced in 2007-01-19 and we are using 2009-04-04, so :+1:.

I agree, its optional indeed.

Endpoint behaviour:

if a public ip address (floating or shared) is assigned to a vm the public hostname is available.
if the EC2 instance has no public ip address an 404 is returned
if enableDnsHostnames=false an 404 is returned.

Consumer:
I think thats a fair expectation too, it's depending on how the EC2 instance has been setup.

I amend my last commit, it seem tailor is still not happy.


Extend the API to support label uploads from users

Label uploads are only available to drone requests.
However, the organizer should use that route to upload user data.
A possible solutions is to provide a route to retrieve a drone API key for normal users (related to #413 I guess).
@nagem @gsfr I would like to know your thoughts about this.
Thanks!

Our error message erroneously reads uploads must be from an authorized drone but really it's just super_user that is required for the request (most likely to minimize permissions checking on upload). @gsfr might have a better idea on who the typical user for the organizer will be and if it is possible they don't have site admin rights.
A drone API key will give them the same rights as super_user so it wouldn't really make sense to generate drone API keys for users, and depending on implementation of #413 we'd lose user context.

If indeed we do need to support users that are not site admins using the organizer, we'll have to add permissions checks to the upload endpoints.

Anyone should be able to upload data, using tools like the Organizer or folder sniper. Permissions checks seem inevitable.

We'd probably want to check permissions before the user sends us any files, so would upload move to a two-step transaction like download? Intent to upload generates an upload ticket?

Probably a good idea.  Uploading for hours just to have every file denied would make for a very poor user experience. Not necessary right away, though. A standard permissions check might get us launched faster and won't require refactoring on the Organizer or sniper side.

For the api/upload/* endpoint, we ensure the hierarchy exists/create it during the check() phase of the placer. We could add permchecks there initially so at least you'll get a 403 earlier in the process.


test: check existence of input KIC image and use it if exists in gke tests

What this PR does / why we need it:
Enable changing KIC image for e2e tests run on GKE. This is required for running validation test against Kong 2.8.x, because we need to use KIC 2.12.x in the tests.

Which issue this PR fixes:

fixes #5228
Special notes for your reviewer:

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

LGTM 👍 Just left suggestions documenting intentions as they may be not obvious to future readers.


[Store] Issue when invoking API resource with a token obtained using password grant type

Description
A valid access token which was generated using password grant provides a valid response instead of failing with the error code 401 when invoking a resource where the security type is set to "Application"
Test Case ID: <IP_ADDRESS>

@nipunsampath how did you set the security type to 'Application' for the resource? Asking because in AM 3.0.0 this concept is not there. It has either secured or not only

@chamilaadhi I set the x-auth-type to "Application" in swagger definition.

This Application/Application User separation is removed in AM 3.0.0. What is supported now whether the resource is, secured or not.  x-auth-type: "None" is considered as not secured and anything other than that (ex: Application, Application User) will be considered as secured. (to provide backward compatibility for APIs created from previous APIM version)


revise the offset calculation for sequence labeling

The current implementations relies on the js script (https://github.com/chakki-works/doccano/blob/master/app/server/static/js/sequence_labeling.js line44-45, line51-52) to calculate the start & end offset for annotated entities. Due to the behavior of the length function in javascript, these offsets are based on the number of utf-16 units (e.g., some emoji would have a length of 2). This PR fixes this issue by revising the annotation offsets.
PS: Thanks for releasing the project :- ), love it

Hi @LiyuanLucasLiu
Thanks for your PR. I just understood the problem.
But I think it is better to convert a character in JavaScript to a unicode by using some method like normalize method. Because Python represents a character as a unicode so we can maybe solve the problem simpler.
Is there any idea?

Actually I think it would be better to directly insert the annotations to the original text (e.g., using Obama) instead of recording the offset.

Like following?
class SequenceAnnotation(Annotation):
    document = models.ForeignKey(...)
    label = models.ForeignKey(Label, on_delete=models.CASCADE)
    start_offset = models.IntegerField()
    end_offset = models.IntegerField()
    text = models.CharField()  # add

Or, remove offset?
class SequenceAnnotation(Annotation):
    document = models.ForeignKey(...)
    label = models.ForeignKey(Label, on_delete=models.CASCADE)
    text = models.CharField()

Are there any merits to insert text directly into a model?

I think offsets is kind-of ambiguous (depend on the way to calculate the length), while the inline annotations are self-explanatory :-)

Do you mean the model has an entire text that has inline annotations instead of offsets?
For example, we save the follwing text into the model:
<PER>Barack Hussein Obama II</PER> (born <DATE>August 4, 1961</DATE>) is an <OTHER>American</OTHER> attorney and politician.

Instead of:
{
    id: 16,
    label: 1,
    start_offset: 0,
    end_offset: 23,
},
{
    id: 17,
    label: 5,
    start_offset: 30,
    end_offset: 44,
}
...

Yes, exactly :-)

@LiyuanLucasLiu Your final goal is to get the XML format as export. Do I understand right? If so, you can translate the export file from JSON format to XML format, because the JSON file already contains the information you need.

@BrambleXu My final goal is to solve the disparity between utf-16 and unicode. This PR proposes a change to calculate the unicode index based on the utf-16 index.

And i don't think normalization is a good idea, because it would change the original data.

If you only implement for the utf-16, what if someone uses utf-32 encoding? This uncertainty will bring some bug in the future. Convert these to a normalized formal is a good choice. Even if the data changes, I don't think this would affect the training process due to two encoding is the same meaning(the same emoji character).

@BrambleXu Emmm... I think you mis-understood my point. The current problem is: no matter what kind of encoding is used, the annotations for sequence labeling would be recorded by a js script. Unfortunately, in javascript, the default length function is not to calculate the number of unicode, but to calculate the number of utf-16 units. So, in order to be compatible with the python script which uses the unicode encoding, i try to map the utf-16 index to the unicode index in this pr. In other words, this pr tries to fix the bug that current implementation cannot handle emoji properly.
I agree to use unicode as the text encoding (and originally i thought by normalization you mean text normalization).

@LiyuanLucasLiu Thanks for the explanation. I am clear now.

@LiyuanLucasLiu Could you provide some data samples you used? I want to what your exact use case for annotating emoji. The data samples will be used for testing.

I'm creating a twitter NER benchmark. Typically any tweets with emoji would cause this problem.


Redesign the Login page and signup page top portion

Redesign the top portion for the Login and signup page
keypoints

We already have a basic setup done for the page, focus mainly on the top portion (The black background region)
Incase you have any further suggestions let me know
Make your designs using Figma, Adobe XD or any equivalent software, and attach the prototype images here itself
For any further questions let me know.

Please assign this issue to me

Please assign this issue to me

Okay assigning you this task

Okay @diptanshumahish

can u assign me this issue

i have make the white container scrollable and the background is fix can you suggest me more changes

  i have make the white container scrollable and the background is fix can you suggest me more changes

Yeah good Idea, do one thing add this image screenshots, to the lib folder by renaming your files as (your github name)_signupPage and make a pull request, we can then  close this issue.

ok will doing this

ok will doing this

Have you changed something in the code?

yes used stack at one place and change some width of textfield and signup button

yes used stack at one place and change some width of textfield and signup button

Thanks for the contribution, merged it, good work!