How to chosse the cut-off value for delete images with bad quality?

I have the MRIQC results from 1000 subjects, but i dont understand how can i choose the cut-off value for choose my best images or that ones to delete

Hi @EdithGaspar,
Could you define specifically what value are you referring to?

I have the MRIQC results from 1000 subjects, but i dont understand how can i choose the cut-off value for choose my best images or that ones to delete

There's no rule of thumb to do this. As we introduced in our MRIQC paper (https://doi.org/10.1371/journal.pone.0184661), you can train a classifier on a subset of your data (that you manually annotate) to then apply it on the remainder of the dataset. The original code for the classifier was moved into the nipreps/mriqc-learn repo.
Perhaps @jaimebarran or @t-sanchez, who have recently worked with mriqc-learn, can give you some insights or share their experience.


Update libpva Python API link

The link to the documentation of the Python API for libpva appears to have been changed from:
https://docs.graphcore.ai/projects/poplar-api/en/latest/pva-python.html
to
https://docs.graphcore.ai/projects/libpva/en/latest/api-python.html
This PR implements this change in the tutorial description.

Thanks Valerio, this was fixed in our latest release


Override default detekt.yml

More of a question.. have searched around but couldn't find the answer.
Is there a way to override certain defaults instead of generating/copying the default-detekt.yml?  Say i want to turn off/disable a rule, and/or change the MaxLineLength value?
This is good for 2 reasons:

Makes it easier to see where a project skews from the defaults
Assures that new rules are included as they are added/updated in future versions.

Thanks!  Open to giving it a shot if this doesn't exist and would take a day or so... Just get me pointed in the right direction

Maybe failFast is for you

Maybe?  Where can I learn more about failFast to see if it fits?

Here #179 and in the PRs

Thanks.  Yep that sounds about right.  Although not sure I could have found it on my own.  Seems like README needs a bit of an update?  Will try to give it a shot after work tonight...

The wiki is still something that needs to be created and hence this might not be something thats available in the docs right now.

Yeah we lack some documentation. Hopefully this will change in the future :).
I assume failFast helped you achieve what you wanted. If not feel free to open this issue again.

Yep failFast worked for me. Thanks!
On Sat, Oct 21, 2017 at 10:04 AM Artur Bosch<EMAIL_ADDRESS>wrote:

Yeah we lack some documentation. Hopefully this will change in the future
:).
I assume failFast helped you achieve what you wanted. If not feel free to
open this issue again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arturbosch/detekt/issues/500#issuecomment-338408793,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANl-eKfB6rL_EvV_QgKFV-6N6z9yCSZks5suggMgaJpZM4P-MMl
.


Multiblock Energy Storage

A multiblock energy tank capable of storing larger amounts of energy than would be feasible to store using energy cubes.
@aidancbrady, I understand you've made something of a start on this, I'll leave it for you to do when you have the time.

This is about completely done - the only other thing would be textures.


Check for changed files to see why cache is not updating

@chidozieononiwu can you please have a look at the git diff that is happening in the PR process? It seems like the lock file updates you have done recently don't match what happens when we do the install in CI.

/azp run azure-sdk - ci

@chidozieononiwu can you please have a look at the git diff that is happening in the PR process? It seems like the lock file updates you have done recently don't match what happens when we do the install in CI.

I have resolved the lock file difference. We can approve and merge this PR.

Thanks for fixing @chidozieononiwu did you setup something so that we can easily get this updated for the next time?


Training Stage

[x] Update config Yaml
[x] Research or analysis
[x] Update the entity
[x] update the configuration Manager in src config
[x] Update the components
[x] update the pipelines
[x] Test Case Added

Training Stage completed


WIP: migrate ansible sample for new layout

Description

Migrate the ansible Memcached sample for teh new layout

What is missing to do here?

Migrate the molecule
Update the Dockerfile to use an image released instead of master.

Hi @asmacdo,
I am closing this one since you are working on in the script and it was done manually.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2063

Questions ?
Please refer to the Backport tool documentation


add global quota?

There should probably be some global limit on the number of submitted samples, as well as one per user.
In particular should ensure we don't run out of disk space if a huge number of samples are submitted.

good idea - we'd probably allow 3-10 per user, but a global ceiling of 1000 or so, which would be a couple of gb


Refactoring | Centralized text styling

Overview
Right now, we're styling every text individually in the new components. This makes keeping the text styling consistent throughout the application nearly impossible. Also, this is the right time to do it:

We already have the styles defined with Figma
We don't have tons of pages to review that have been developed

Currently used styles
The styles that are being used in the design are the following. The current naming is a copy of what has been used so far, but the team should have full authority over it.
Some styles should be assumed unless specified otherwise:
font-style: normal;
line-height: 150%;
font-family: 'Silka', 'Roboto', sans-serif;
text-align: left;

h1
Used exclusively for the current user balance and page titles (e.g. send, receive).
font-weight: 600;
font-size: 40px;
line-height: 48px;

h2
font-weight: 600;
font-size: 24px;
line-height: 130%

h3
font-weight: 400;
font-size: 24px;
line-height: 130%

h4
font-weight: 400;
font-size: 20px;
line-height: 130%

h4 for mobile (< 600px width)
font-weight: 400;
font-size: 18px;
line-height: 130%

p
font-weight: 400;
font-size: 16px;
line-height: 150%

p-bold
font-weight: 600;
font-size: 16px;
line-height: 150%

label
Generally used as a label, on buttons and for USD values
font-weight: 600;
font-size: 14px;
line-height: 150%
text-transform: uppercase;

small
font-weight: 600;
font-size: 12px;
line-height: 150%

small-bold
font-weight: 600;
font-size: 12px;
line-height: 150%

@karynemayer, we also need to solve the style of the buttons which currently suffers from the same problem.
Here is your feedback about it.


Small compatibility fix for holoviews 1.8

As the title says, this was a typo which now clashes with a HoloViews method.

Merging.


Make home page more responsive

Resize SongCards (like on YouTube)

SongCards now grow when resizing window. Edge cases need some more work.


Issue when running containers through docker-compose up

We are attempting to run this project on this setup:
Software: Latest commit on main branch as of today.
OS - arm64 (2021-10-30 Bullseye)
Hardware - RaspberryPi4
When running docker-compose up, the images are successfully pulled. When running the images, however, we get some errors such as:
" ERROR:root:Unable to detect Coral boardFailed to load delegate from libedgetpu.so.1"
"cp: can't stat '/etc/letsencrypt/live/broker.mqtt.local/*.pem': No such file or directory
mqtt_1          | cat: can't open '/run/secrets/mqtt_user': No such file or directory
mqtt_1          | cat: can't open '/run/secrets/mqtt_pass': No such file or directory"
"axis-ptz_1      |     raise ValueError('Invalid host.')
axis-ptz_1      | ValueError: Invalid host.
"
These errors raised a few questions:

Do we need to have a coral board?
Is a GPS device required for this project?
If a GPS device is required, is there a way to run the project by disabling the need for the GPS?

We are a small group working on our summer internship research project and would greatly appreciate any guidance with getting this project set up. We would appreciate a point of contact and are open to receiving any communications through my email<EMAIL_ADDRESS>Thank you for your time!

Hi @aroop-biswal-vts - we're making some updates today to get this to a state where it will just come online.  @meadej will be addressing.  To answer your questions:

You don't need a coral board, we accidentally merged some dev work into the main branch.  This will be addressed today
We are deprecating GPS support - we did not find it to be accurate enough to reliably get aircraft in frame when zoomed in tight.  We generally use a surveyed position from google maps and put the lat/long into the .env file
You can delete the EGI container from the docker-compose file.

Hi @aroop-biswal-vts - thank you for bringing up these issues and for your patience.
As @mchadwick-iqt mentioned above, we have placed some current development work under the main branch. For immediate use in the field, we have set-up a stable branch which will work directly out of the box and have cut a release, SkyScan Version 2.1, which uses proven-stable containers tested in our lab.
To use this branch, reference the docker-compose.yml file included here and ensure that you have environment variables configured as shown in the example here, paying particular attention to the RELEASE_NAME variable.
Let us know if you have any further questions - they are greatly helpful in our work to build out documentation and improve stability.
Thank you!


push + pull now works with reverse proxy + basic auth on apache 2.4

@strk asked me to offer my gogs contribution to gitea
https://github.com/gogits/gogs/pull/3785
Setup:
I use gogs // gitea with reverse proxy and basic auth. Users are auto created.
http pull and push did not work, so I started to look in the code.
We use SSO (AuthBasicProvider dbd, Apache 2.4, Location /gogs - since today /gitea AuthType Basic)
Reverse Proxy is pointing to /gogs - since today /gitea.
[security]
REVERSE_PROXY_AUTHENTICATION_USER = X-WEBAUTH-USER
[service]
ENABLE_REVERSE_PROXY_AUTHENTICATION = true
ENABLE_REVERSE_PROXY_AUTO_REGISTRATION = true
Issue: SSO was not handeled by http.go
Solution: now added to the section, see contribution.

LGTM, but I hadn't tested it

It is in production mode on our server now.

Dear all,
for those to review, please focus on
               if setting.Service.EnableReverseProxyAuth {                         authUsername = ctx.Req.Header.Get(setting.ReverseProxyAuthUser)                         if len(authUsername) == 0 {                                 ctx.HandleText(401, "reverse proxy login error. authUsername empty")                                 return                         }                         authUser, err = models.GetUserByName(authUsername)                         if err != nil {                                 ctx.HandleText(401, "reverse proxy login error, got error while running GetUserByName")                                 return                         }                 }else{
Line 86 to 97. Everything else is indentation. Lint would complain causing the check to fail. Fixing the indentation makes the diff look big - which it is not when having a second glance on it.

LGTM


Hide Panel (X) tooltip should include key binding

Version: 1.87.2
Commit: 863d2581ecda6849923a2118d93a088b0745d9d6
Date: 2024-03-08T15:20:57.520Z
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.6.0
When you hover over the "X" icon to close the panel, you get a tooltip that says "Hide Panel".  I was looking for how to close it in a single keystroke and assumed that since there was no associated key binding shown in the tooltip, none existed.  Eventually I discovered it was Command-J, but had to scrub all the menus to find that; it really should be shown right in the tooltip.
I think what's really going on is that Command-J isn't actually bound to Hide Panel it's bound to Toggle Panel Visibility.  Either way, from a U/X point of view it still makes sense to show me Command-J when I hover over the "X" icon.  Maybe a better fix would be to have that icon execute Toggle Panel Visibility.  To the user, the effect would be the same, and you'd get the keybinding in the tooltip.

It does show for me, though its not defined by default:

We do not seem to have a proper way of having one keybinding for two menus with different labels. There is the toggle condition on Action2 items which could help with this, however, there seems to be some rendering glitches currently.


Feature request: Sort bullet list

I would be nice to have the ability to sort a list or multiple text lines.
From:
- a
- c
- b

into:
- a
- b
- c

and alternate into:
- c
- b
- a

As text lines.
From:
a
c
b

into:
a
b
c

and alternate into:
c
b
a

Thank you.

Maybe, you could consider merging -with the proper authorization- the development by @Vinzent03 at https://github.com/Vinzent03/obsidian-sort-and-permute-lines

Maybe, you could consider merging -with the proper authorization- the development by @Vinzent03 at https://github.com/Vinzent03/obsidian-sort-and-permute-lines

If the plugin you mentioned meet your needs, it is suggested that installing both two plugins, which is most convenient.


test: Verify build-std final crate graph against library lock file.

What does this PR try to resolve?
Adds a test to the build-std test suite which checks whether the resolved standard library unit graph accurately describes a subset of the dependencies listed in the standard library's Cargo.lock, as mentioned in an issue from wg-cargo-std-aware:

It might be a good idea to do the equivalent of --locked for the standard library to ensure that the Cargo.lock isn't modified, and that it is correct.
Source: https://github.com/rust-lang/wg-cargo-std-aware/issues/38

This PR implements this as part of the build-std test suite, as suggested by @adamgemmell in the above linked issue, however I also have a version of this patch implemented which can perform this check at runtime, if that would be a more appropriate place to perform the check.
Fixes https://github.com/rust-lang/wg-cargo-std-aware/issues/38
How should we test and review this PR?
The test can be run as follows:
CARGO_RUN_BUILD_STD_TESTS=1 cargo test validate_std_crate_graph

Additional information
N/A

Bump - maybe @ehuss is interested in this patch?

Sorry for the long delay.

which can perform this check at runtime, if that would be a more appropriate place to perform the check.

Yea, I think it would probably be the preferred route to go to verify it is a subset at runtime. Would you be able to update the PR to include that?

:umbrella: The latest upstream changes (presumably #14243) made this pull request unmergeable. Please resolve the merge conflicts.

Going to close this. See https://github.com/rust-lang/wg-cargo-std-aware/issues/38#issuecomment-2271758701.
We are still looking for closing out https://github.com/rust-lang/wg-cargo-std-aware/issues/38.


X axis not accepting value, and Y axis only accepting value

Closes #18884
Describe your proposed changes here.
These changes will fix the X axis so they will accept _value. It will fix the Y axis so it will accept values other than _values. There are end to end test testing that both the X and the Y can select all values including time, start, stop, and value.

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[ ] Well-formatted commit messages
[ ] Rebased/mergeable
[ ] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Documentation updated or issue created (provide link to issue/pr)
[ ] Signed CLA (if not already signed)

this looks solid. i would add unit tests for the reducer also.
We did try to add the unit test for the reducer, but we need to get the mocking done first, so that going to be addressed at a later date

good call @121watts. we actually discussed it and looked into it and realized we'd have to do some work that might take a little while and hold this up, so we wrote it up in a ticket https://github.com/influxdata/influxdb/issues/19046 and are going to talk about it tomorrow when we pair up :+1:


Implement the remaining ONNX in burn-import functionality

The initial implementation in https://github.com/burn-rs/burn/issues/204, which laid the foundation for generating model source code, added a few ONNX operations:

BatchNorm
Conv2d
Flatten
Gemm (Linear layer)
LogSoftmax

We need to add more operations to cover all Burn operations.

Closing since there are more specific op tickets. This issue was meant as an umbrella ticket, which is too big.


[Windows] Removes abstract mock classes Stream Encoder/Decoder

Commit Message: Removes abstract mock classes Stream Encoder/Decoder
Additional Description:
As part of our effort to port Envoy to Windows we run into an issue with GTests and GMock in particular that is causing issues when we test on Windows. More specifically chromium documentation:

NiceMock and StrictMock only work for mock methods defined using the MOCK_METHOD macro directly in the MockFoo class. If a mock method is defined in a base class of MockFoo, the “nice” or “strict” modifier may not affect it, depending on the compiler.

The way I interpret it is that when you inherit from a Mock class you can not guarantee that the parent Mock class functions are treated as interesting calls. This leads to a GMock warning and consequently a test error. Example base mock classes are MockStreamEncoder and   MockStreamDecoder
Tests that are now passing:

//test/common/http:conn_manager_impl_test
//test/common/http/http1:codec_impl_test
//test/common/http/http1:conn_pool_test
//test/common/http/http2:conn_pool_test
//test/common/http:async_client_impl_test
//test/common/router:router_test

Risk Level: Low (Test only)
Testing: Tested manually on Windows and Linux
Docs Changes: N/A
Release Notes: N/A

@nigriMSFT @sunjayBhatia @wrowe

@junr03 I will do that.
Lemme wait until tomorrow morning to see if anyone has any other comments to batch them together and fix everything in a single iteration.

Sorry for the spam folks, configuring DCO turned out to be trickier than I expected

@junr03 ready for a review when you are 😀


Forgot to actually check that the top-level object is of the list type.

Closed by ad65ef8a71a909482f4621f7bfa2e0b7434367d4


Discovery: BGMD Coverage

How many mispredicted data (single instance, single row) contain/don't contain an under-represented feature? How many contain multiple under-represented features?
The goal of this issue is to replicate the findings from the paper.

Requires packages that are not made public to us. Will continue work on this issue once they are sent to us or we have an alternate solution.

Blastoff.zone: Earn 1 ETH + $500,000 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500,000 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500,000 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 1 million tokens already claimed, seize this chance before it's too late!
bengban, @TheoKlein, @0xlax, @echoplaza, @mmacedoeu, @ravirathore, @iwasa-kosui, @P3n9W31, @adamawesomeface, @daiwanwei, @0xMatthieu, @PEMIDI, @haxdds, @DmitriyVdE, @abirjameel, @n3n, @H-Freax, @asafeeson, @bnbfounder1, @wmeira, @maxbsd, @androas35, @ChenliTemple, @lxedge, @zikovvv, @lmmsoft, @MarechJ, @Justin2997, @Charmarder, @biplob004, @jinbangyi, @tazeemc, @kfiresmith, @Lym666XD, @domimod, @joaorafaelm, @oldhat2016, @flavioarieta, @aixinchuanqi, @JSTHoffman, @fendouai, @sherxon211, @yawyd, @diedline, @thejeshpr, @Jay-Trades, @lapreamarcelo, @ATField2501, @niclausse, @orangepu


Add way to exclude files and directories to watch

This allows user a way to specify directories and/or files to ignore:

869abb1 Adds parsing of new watchOptions: excludeFiles and excludeDirectories in tsconfig as well as command line
9cd3e937162421fcbc477286e84aa48dcc8ef0dd Adds a way to create noopWatcher for files and directories that are being watched.
dddd906 Adds a way to exclude invoking callback for excluded things from sys itself. This means no invoke for excluded path if its invoked from recursive directory watching or not watching directories that are excluded on os that don't support recursive watching (like linux)
0a8d84fc8bec42bf3e47bb5b8846437d3c8c0ddf handles the server host configuration and external/inferred projects watch options with ignore.
496939d reloading projects, reloads project from scratch
cd18da8 file updates when reloading are reflected

This provides a way on systems where watching can be expensive and letting user refresh things if something changes rather than us having to poll or rely on events
Fixes #33335, #36035, #36243, #36394

This is now ready for review

letting user refresh things

Remind me how the editor does this?  Is there a protocol refresh to re-poll the disk?

os that don't support recursive watching (like linux)

Does that mean Windows and Mac do support recursive watching?

Remind me how the editor does this? Is there a protocol refresh to re-poll the disk?

Restart tsserver from vscode.. Dont know about vs command for that

Does that mean Windows and Mac do support recursive watching?

Yes: https://github.com/microsoft/TypeScript/blob/master/src/compiler/sys.ts#L1163

Remind me how the editor does this? Is there a protocol refresh to re-poll the disk?

Restart tsserver from vscode.. Dont know about vs command for that

That seems like overkill.  Wouldn't there be a shorter delay if we just repolled all the files and then did an updateGraph?  The user scenario I have in mind is hitting refresh in the file list tool window (aka the Solution Explorer in VS).

It looks like there's a reload request.  Is that relevant?

cc @mjbvz for the VS Code user experience.

@amcasey Interesting point. I think reload should definitely reload everything  and makes sense to refresh but i would need to think about how to do that when server is not restarted...

@sheetalkamat I was thinking it would be like simulating a file listener event - the server would just do whatever it does when it finds out one or more files have changed on disk (ideally, as a batch, rather than as a loop of updates).

Thanks for this PR @sheetalkamat. I really look forward for this PR to be merged as VS Code/TypeScript is unfortunately too often unresponsive on Linux when I'm working on my one of my project that has about 2000 packages in node_modules. Let me know if I can do anything to help you.

That'd be amazing to have this one resolved finally. It has been first reported almost 2 years ago and there's little progress.
   INOTIFY
   WATCHER
    COUNT     PID     CMD
----------------------------------------
  503994     4631  /usr/share/code/code /usr/share/code/resources/app/out/bootstrap-fork --type=watcherService
   20038     4631  /usr/share/code/code /usr/share/code/resources/app/out/bootstrap-fork --type=watcherService

That's quite extensive for a
(Fedora31, vsc 1.47.2)
(and I don't even have any node_modules folder, and tried it all to ignore everything and unignore only very few folders which disrupted my experience with ag, but to no avail. It's ridiculous).

@mjbvz Could you possibly confirm that the new project reload functionality behaves sensibly in VS Code?  I don't believe VS uses it.

@amcasey @mjbvz i would like to get this in for 4.2
Can you please review

Are there any docs for this feature? Also, does this automatically take the include and exclude dirs from tsconfig.json? it seems unintuitive that the watch is set up on files ignored by tsc

By default nothing is ignored. User has to opt for directories/files to ignore..
@orta i think this is not in tsconfig reference section yet.

Thanks sheetal - looks like I manually re-created a partial set of the watch flags back when I made the tsconfig reference, will move it to be automatic to pick these up!

I cant seem to find any docs on this.
https://www.typescriptlang.org/tsconfig#Watch_Options_999
This has nothing about the new fields that are supposed to be added. Im trying to add
  "watchOptions": {     "excludeDirectories" : ["node_modules"]   }
to my tsconfig but Im still getting extremely slow load times on my project when using vscode in WSL with files on the C drive

@Kierchon it also needs to be set in vscode preferences but i dont find it in vscode preferences in watchOptions
@mjbvz did we miss adding excludeDirectories and excludeFiles as watchOption in vscode preferences?

You should just be able to add them in the settings.json:
  "typescript.tsserver.watchOptions": {
    "watchFile": "useFsEvents",
    "excludeDirectories": []
  },

We pass the entire object on the TS Server
@sheetalkamat Should these settings also be documented? If so, can you please share a quick blurb about what these two settings do


update README

Some links in badges do not work yet properly. Fix this when having set up all elements (i.e. documentation etc) of the project.

updated.


Footer

Maybe adding footer with some links to github repo etc. would be nice.
If accepted im ready to work on this case

Instead add it to navbar

OK, working on that


Adding more fields in ride table:

What does this pull request do:
Adding more fields in the request table: i.e.

Vehicle Registration number
Vehicle Model
Vehicle Capacity

Pull Request Test Coverage Report for Build 52

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 97.083%

Totals

Change from base Build 48:
0.04%

Covered Lines:
233

Relevant Lines:
240

💛  - Coveralls


Handle child_block being passed as a kwarg in ListBlock.deconstruct_with_lookup

Fixes #12202

Merged in 37a5e8dabc (stable/6.2.x) and d327a0dd50 (main).


Counts tellers as voters

e.g. Universal credit roll out (I can't provide a direct link as the site doesn't change the URL when an option is picked) was 299-0, not 301-2, see https://www.theyworkforyou.com/debates/?id=2017-10-18a.953.0

Thanks for the comment, good point about tellers! I'll amend the count and potentially use a different "pot" to put their placeholder (or some other graphical feature).


Standardize settings further

Just went through the YAML and updated some small things, most importantly: list radars alphabetically within country (was not the case yet for Sweden, Finland, Poland). Countries themselves are still in the original order.
@CeciliaNilsson709 if you agree, you can merge this pull request. 👍
There is only one radar in the YAML for which I haven't received any flyway data: deemd (http://enram.github.io/data-repository/?prefix=de/emd/). That seems logical because the remark is "no data", but for the two other radars with that remark: silas and sipas we do have flyway data: http://enram.github.io/data-repository/?prefix=si/ 🤷♂ So, what is the difference between the two? Maybe we should update the remark for the si data to something regarding quality?

Looks good to me. The data from Slovenia (si) could be labeled as "No birds" instead, as the bird information is (most likely) filtered out of the data we received. Will you change it?

@CeciliaNilsson709 changed to no birds. 👌  I also removed deemb from the settings, because there are a bunch of other radars in the data repository - for which I assume we didn't receive flyway data - that don't get mentioned either:

nlhrw
plpas
sehem
sehuv
seoer
seosd
sevax
deflg
desna
deemd

If you prefer, we could also list all the above as not having any data, but I don't know if it's a complete list of radars per country...

Yeah, this is where things start getting a bit messy as to what is actually "excluded" and when, like we talked about before... But just excluding deemd  is fine, as you say it will not be a complete list anyway.


Unit Test plugin and TypeScript w/ TSX not working

Version
3.2.1
Reproduction link
https://github.com/lmiller1990/vue-cl-tsx-bug/pull/new/master
Environment info
Mac OS

Steps to reproduce
Clone above and yarn test:unit, or if you want your own:
Scaffold app
Select unit tests, typescript and babel
Change HelloWorld.vue to HelloWorld.tsx and fill with the following:
export const MyComponent = {
  name: 'MyComponent',
  render() {
     return (
asdf
)
  }
}

Update example.spec.ts to import { MyComponent } from '@/components/MyComponent and shallowMount(MyComponent)...
Run the tests with yarn test:unit. I get:
 Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /Users/lachlanmiller/javascript/vue/tsx/src/components/HelloWorld.tsx:6
            return (
asdf
);
                    ^

    SyntaxError: Unexpected token <

      1 | import { shallowMount } from '@vue/test-utils'
    > 2 | import { MyComponent } from<EMAIL_ADDRESS>        | ^
      3 |
      4 | describe('HelloWorld.vue', () => {
      5 |   it('renders props.msg when passed', () => {

      at ScriptTransformer._transformAndBuildScript (node_modules/jest-runtime/build/script_transformer.js:403:17)
      at Object.<anonymous> (tests/unit/example.spec.ts:2:1)

What is expected?
Runs the test fine
What is actually happening?
Does not appear to transpile the JSX/TSX

Maybe some work involving babel can fix this? I think all the official tools should work together. Maybe babel can do it.

Ok, so it seems if I use babel-jest instead of ts-jest in jest.config.js, and add '@babel/preset-typescript' to babe.config.js, it works. Is this intended? Perhaps installing the TS support should also check for unit tests with jest, and update the transform` appropriately.
What is the difference of using babel-jest and @babel/preset-typescript vs ts-jest?

I would consider this a bug.
In our webpack config, we use ts-loader and babel-loader after it, which compiles the JSX (Typescript can't compile JSX into Vue render functions)
In jest, we can configure ts-jest to use Babel as well, but we haven't made this a default setting so far.
This can be fixed though.
For your project, have a look at the docs for ts-jest, eh ew you find the option or set to make it use Babel (I'm on mobile right now so no code snippets)

For now it seems using babel-jest instead of ts-jest and addding the appropriate babel preset works. I'm not sure how to get ts-jest to use babel yet, I'll try a bit more tonight. Thanks for the reply so far - I'd like to contribute to this, if/when we figure out the best approach to do so.

I was getting the following error:
ts-jest[versions] (WARN) Version 4.2.3 of typescript installed has not been tested with ts-jest. If you're experiencing issues, consider using a supported version (>=2.7.0 <4.0.0). Please do not report issues in ts-jest if you are using unsupported versions.
 FAIL  tests/unit/example.spec.ts
  ● Test suite failed to run

    Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /home/maxim/WebWatcher/Frontend/node_modules/vue-tsx-support/lib/index.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){export * from "./api";
                                                                                ^^^^^^

    SyntaxError: Unexpected token 'export'

    > 1 | import * as tsx from "vue-tsx-support";
        | ^
      2 | import { PropType, VNode } from "vue";
      3 |
      4 | export default tsx.component({

      at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:537:17)
      at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:579:25)
      at Object.<anonymous> (src/components/WWHeader.tsx:1:1)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        0.857s
Ran all test suites.

My babel:
module.exports = {
  presets: ["@vue/cli-plugin-babel/preset"],
};

My jest:
module.exports = {
  preset: "@vue/cli-plugin-unit-jest/presets/typescript-and-babel",
  setupFiles: ["./tests/unit/setup.js"],
};

Adding transformIgnorePatterns: [], to jest config helped.

Best open a new issue with a repoduction repo - it's likely a different issue, since this thread is around 3 yrs old @Maxim-Mazurok.

Thanks for a motivating push, @lmiller1990 :)
Here it is: https://github.com/vuejs/vue-cli/issues/6389


Issue in importing epub on Readium Cloud Reader

We tried to add epub file on https://readium.firebaseapp.com/? but import progress stops on initial percentage.  see attached screenshots:

The "import EPUB" button should be disabled in the cloud reader. See existing issue: https://github.com/readium/readium-js-viewer/issues/82


feat: update graphql-upload

To pull in an updated busboy which avoids a vulnerable dicer, see https://github.com/jaydenseric/graphql-upload/releases/tag/v15.0.0
Both updates minimum graphql and node, so this is a breaking change

https://github.com/mercurius-js/mercurius-upload/pull/8 it seems to be working here, I basically rebased, fixed conflict and updated lockfile

#8 it seems to be working here, I basically rebased, fixed conflict and updated lockfile

feel free to do the same changes 👍

either pnpm or GH seems to have issues atm, but I rebased this now 👍

it was some rare gh actions issues, now it's working

it was some rare gh actions issues, now it's working

Yeah, https://www.githubstatus.com/incidents/6r4sms3hcj3x


Remove BuildCraft fuel integration

Fix SpongeForge 7.0.0-BETA-2568 Compatibility

Appears to be something with this SpongeForge commit
https://github.com/SpongePowered/SpongeForge/commit/5de88e706c84c549cd2415795910d3be7be8c0bd
BC is not on 1.12.1 anyway, integration could be re-added later

I'll reopen this if you've provided the requested information (preferably a complete bug report with logs)

https://github.com/SpongePowered/SpongeForge/issues/1732 contains the crashlog

Sorry, but this is something SpongeForge should fix, that code is guarded by Loader.isModLoaded() and should never run when buildcraft isn't present. It works perfectly with forge.
SpongeForge is probably loading stuff it shouldn't do.
If any of the Sponge developers need help debugging, I'm certainly willing to help finding a solution.

@tgstyle Could you confirm the latest spongeforge version fixes your issue?

Sponge 2577 still has the issue with Mekanism


close trade confusion

Please provide the follow
Description of issue:
I am getting confused on how to close the order. we see cancel order, close position, close trades, we don't know how to tell the difference so can you please clarify it?
Your environment(os, python, packages)
Expected behaviour:
Actual behaviour:
Complete self contained example of issue:

Yes, It can be a little confusing.
I believe:

cancel_order applies to pending orders that have yet to filled by OANDA
close_position close's the aggregation of current trades
close_trades will close the specific trade of a position.

This can be a little confusing and needs to be researched to provide the correct information.
Documentation will be updated in due course
Thanks for the feedback!

@jamespeterschinner awesome but it just doesn't really make sense how close trades is closing a position and how close position is closing a trade.
you can see on the oanda trading website, there is trades and positions. l made two AUD_USD trades but only shows up as one position, meaning they are both combined.
so l am guessing l should be using the close position more since close trades might close two of my AUD_USD trades if l only want to close one.

OandaClient.close_trade( specifier ) will close that trades portion of a position.
Yes you are correct trades are aggregated into positions.
It is actually the other way around. Use close position will close all trades for the specified instrument.
I hope this helps.
I'm closing the issue as I believe this information is out side the scope of async_v20.


render.ts - api key auth via process.env

from example PR provided

@psugihara have another look, i think it makes sense for render to adopt this behavior if MAILING_API_KEY is set, otherwise not.  and for the validation logic to live separately from the db api key validation (right now they exist as separate features).  wdyt?

Tests look good but I think it's kind of confusing that this api key would work for api/render but not e.g. api/sendMail. With this setup there are 2 kinds of API keys that do 2 different things. I think if I'm setting the api key, it should work as the api key in all contexts. That's simpler conceptually for me.

Makes sense. I can flip it to be one function. Do you think the REQUIRE_API_KEY flag should only affect this one type of auth then or apply to all routes and all methods of auth?
On Mon, Nov 28 2022 at 3:14 PM, Peter Sugihara < @.*** > wrote:

Tests look good but I think it's kind of confusing that this api key would
work for api/render but not e.g. api/sendMail. With this setup there are 2
kinds of API keys that do 2 different things. I think if I'm setting the
api key, it should work as the api key in all contexts. That's simpler
conceptually for me.
—
Reply to this email directly, view it on GitHub (
https://github.com/sofn-xyz/mailing/pull/377#issuecomment-1329872978 ) , or
unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAAGCISXQ6QKP7G47I7B2OTWKU4DVANCNFSM6AAAAAASNYLNWA
).
You are receiving this because you authored the thread. Message ID: <sofn-xyz/mailing/pull/377/c1329872978
@ github. com>

Cool. I think we just want REQUIRE_API_KEY to turn on the feature for /api/render (maybe there will be others in the future). Maybe there is a better name for that key.

@psugihara updated and added doc


Duplicate Rule Error for Sarif File

I've set up the ossf/scorecard-action to run for a project I'm working on. We started seeing errors from the action saying the outputted Sarif file is not valid (see below):
Error: Unable to upload "results.sarif" as it is not valid SARIF:
- instance.runs[2].tool.driver.rules contains duplicate item
Error: Unable to upload "results.sarif" as it is not valid SARIF:
- instance.runs[2].tool.driver.rules contains duplicate item
    at validateSarifFileSchema (/home/runner/work/_actions/github/codeql-action/a34ca99b4610d924e04c68db79e503e1f79f9f02/lib/upload-lib.js:189:15)
    at uploadFiles (/home/runner/work/_actions/github/codeql-action/a34ca99b4610d924e04c68db79e503e1f79f9f02/lib/upload-lib.js:238:9)
    at Object.uploadFromActions (/home/runner/work/_actions/github/codeql-action/a34ca99b4610d924e04c68db79e503e1f79f9f02/lib/upload-lib.js:132:18)
    at async run (/home/runner/work/_actions/github/codeql-action/a34ca99b4610d924e04c68db79e503e1f79f9f02/lib/upload-sarif-action.js:46:30)
    at async runWrapper (/home/runner/work/_actions/github/codeql-action/a34ca99b4610d924e04c68db79e503e1f79f9f02/lib/upload-sarif-action.js:68:9)

The workflow file we use is almost identical to the sample in the documentation: Here is the workflow file we use:
name: Scorecards supply-chain security
on:
  branch_protection_rule:
  schedule:
    # Weekly on Saturdays.
    - cron: '30 1 * * 6'
  push:
    branches: [ main ]
  workflow_dispatch:

permissions: read-all

jobs:
  analysis:
    name: Scorecards analysis
    runs-on: ubuntu-latest
    permissions:
      security-events: write
      id-token: write
      actions: read
      contents: read

    steps:
      - name: "Checkout code"
        uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # tag=3.0.2
        with:
          persist-credentials: false

      - name: "Run analysis"
        uses: ossf/scorecard-action@e38b1902ae4f44df626f11ba0734b14fb91f8f86 # tag=v2.0.4
        with:
          results_file: results.sarif
          results_format: sarif
          repo_token: ${{ secrets.SCORECARD_READ_TOKEN }}
          publish_results: true

      - name: "Upload artifact"
        uses: actions/upload-artifact@3cea5372237819ed00197afe530f5a7ea3e805c8 # tag=v3.1.0
        with:
          name: SARIF file
          path: results.sarif
          retention-days: 5

      - name: "Upload to code-scanning"
        uses: github/codeql-action/upload-sarif@515828d97454b8354517688ddc5b48402b723750 # tag=v2.1.38
        with:
          sarif_file: results.sarif

Could I get some guidance on what the issue might be? As far as I understand, the rules are default and come preset with the OSSF action?

The duplicate rule is the Code-Review
{
                     "id": "CodeReviewID",
                     "name": "Code-Review",
                     "helpUri": "https://github.com/ossf/scorecard/blob/376f465c111c39c6a5ad7408e8896cd790cb5219/docs/checks.md#code-review",
                     "shortDescription": {
                        "text": "Code-Review"
                     },
                     "fullDescription": {
                        "text": "Determines if the project requires code review before pull requests (aka merge requests) are merged."
                     },
                     "help": {
                        "text": "Determines if the project requires code review before pull requests (aka merge requests) are merged.",
                        "markdown": "**Remediation (click \"Show more\" below)**:\n\n- If the project has only one contributor, or does not have enough reviewers to practically require that all contributions be reviewed, try to recruit more maintainers to the project who will be willing to review others' work. Ideally at least some of these people will be from different organizations (see [Contributors](checks.md#contributors)). If the project has very limited utility, consider expanding its intended utility so more people will be interested in improving it, and make that larger scope clear to potential contributors.\n\n- Follow security best practices by performing strict code reviews for every new pull request / merge request.\n\n- Make \"code reviews\" mandatory in your repository configuration. ([Instructions for GitHub.](https://docs.github.com/en/github/administering-a-repository/about-protected-branches#require-pull-request-reviews-before-merging))\n\n- Enforce the rule for administrators / code owners as well. ([Instructions for GitHub.](https://docs.github.com/en/github/administering-a-repository/about-protected-branches#include-administrators))\n\n\n\n**Severity**: High\n\n\n\n**Details**:\n\nRisk: `High` (unintentional vulnerabilities or possible injection of malicious\n\ncode)\n\n\n\nThis check determines whether the project requires code review before pull\n\nrequests (merge requests) are merged.\n\n\n\nReviews detect various unintentional problems, including vulnerabilities that\n\ncan be fixed immediately before they are merged, which improves the quality of\n\nthe code. Reviews may also detect or deter an attacker trying to insert\n\nmalicious code (either as a malicious contributor or as an attacker who has\n\nsubverted a contributor's account), because a reviewer might detect the\n\nsubversion.\n\n\n\nThe check first tries to detect whether [Branch-Protection](checks.md#branch-protection) is enabled on the\n\ndefault branch with at least one required reviewer. If this fails, the check\n\ndetermines whether the most recent (~30) commits have a Github-approved review\n\nor if the merger is different from the committer (implicit review). It also\n\nperforms a similar check for reviews using\n\n[Prow](https://github.com/kubernetes/test-infra/tree/master/prow#readme) (labels\n\n\"lgtm\" or \"approved\") and [Gerrit](https://www.gerritcodereview.com/) (\"Reviewed-on\" and \"Reviewed-by\").\n\n\n\nNote: Requiring reviews for all changes is infeasible for some projects, such as\n\nthose with only one active participant. Even a project with multiple active\n\ncontributors may not have enough active participation to be able to require\n\nreview of all proposed changes. Projects with a small number of active\n\nparticipants instead sometimes aim for a review of a\n\npercentage of proposals (e.g., \"at least half of all proposed changes are\n\nreviewed\").\n\n\n\nRequiring review does not eliminate all risks. The other reviewers might fail to\n\nnotice unintentional vulnerabilities or malicious code, be colluding with a\n\nmalicious developer, or even be the same person (using a \"[sock\n\npuppet](https://en.wikipedia.org/wiki/Sock_puppet_account)\" account).\n\n"
                     },
                     "defaultConfiguration": {
                        "level": "error"
                     },
                     "properties": {
                        "precision": "high",
                        "problem.severity": "error",
                        "security-severity": "7.0",
                        "tags": [
                           "supply-chain",
                           "security",
                           "source-code",
                           "code-reviews"
                        ]
                     }
                  },
                  {
                     "id": "CodeReviewID",
                     "name": "Code-Review",
                     "helpUri": "https://github.com/ossf/scorecard/blob/376f465c111c39c6a5ad7408e8896cd790cb5219/docs/checks.md#code-review",
                     "shortDescription": {
                        "text": "Code-Review"
                     },
                     "fullDescription": {
                        "text": "Determines if the project requires code review before pull requests (aka merge requests) are merged."
                     },
                     "help": {
                        "text": "Determines if the project requires code review before pull requests (aka merge requests) are merged.",
                        "markdown": "**Remediation (click \"Show more\" below)**:\n\n- If the project has only one contributor, or does not have enough reviewers to practically require that all contributions be reviewed, try to recruit more maintainers to the project who will be willing to review others' work. Ideally at least some of these people will be from different organizations (see [Contributors](checks.md#contributors)). If the project has very limited utility, consider expanding its intended utility so more people will be interested in improving it, and make that larger scope clear to potential contributors.\n\n- Follow security best practices by performing strict code reviews for every new pull request / merge request.\n\n- Make \"code reviews\" mandatory in your repository configuration. ([Instructions for GitHub.](https://docs.github.com/en/github/administering-a-repository/about-protected-branches#require-pull-request-reviews-before-merging))\n\n- Enforce the rule for administrators / code owners as well. ([Instructions for GitHub.](https://docs.github.com/en/github/administering-a-repository/about-protected-branches#include-administrators))\n\n\n\n**Severity**: High\n\n\n\n**Details**:\n\nRisk: `High` (unintentional vulnerabilities or possible injection of malicious\n\ncode)\n\n\n\nThis check determines whether the project requires code review before pull\n\nrequests (merge requests) are merged.\n\n\n\nReviews detect various unintentional problems, including vulnerabilities that\n\ncan be fixed immediately before they are merged, which improves the quality of\n\nthe code. Reviews may also detect or deter an attacker trying to insert\n\nmalicious code (either as a malicious contributor or as an attacker who has\n\nsubverted a contributor's account), because a reviewer might detect the\n\nsubversion.\n\n\n\nThe check first tries to detect whether [Branch-Protection](checks.md#branch-protection) is enabled on the\n\ndefault branch with at least one required reviewer. If this fails, the check\n\ndetermines whether the most recent (~30) commits have a Github-approved review\n\nor if the merger is different from the committer (implicit review). It also\n\nperforms a similar check for reviews using\n\n[Prow](https://github.com/kubernetes/test-infra/tree/master/prow#readme) (labels\n\n\"lgtm\" or \"approved\") and [Gerrit](https://www.gerritcodereview.com/) (\"Reviewed-on\" and \"Reviewed-by\").\n\n\n\nNote: Requiring reviews for all changes is infeasible for some projects, such as\n\nthose with only one active participant. Even a project with multiple active\n\ncontributors may not have enough active participation to be able to require\n\nreview of all proposed changes. Projects with a small number of active\n\nparticipants instead sometimes aim for a review of a\n\npercentage of proposals (e.g., \"at least half of all proposed changes are\n\nreviewed\").\n\n\n\nRequiring review does not eliminate all risks. The other reviewers might fail to\n\nnotice unintentional vulnerabilities or malicious code, be colluding with a\n\nmalicious developer, or even be the same person (using a \"[sock\n\npuppet](https://en.wikipedia.org/wiki/Sock_puppet_account)\" account).\n\n"
                     },
                     "defaultConfiguration": {
                        "level": "error"
                     },
                     "properties": {
                        "precision": "high",
                        "problem.severity": "error",
                        "security-severity": "7.0",
                        "tags": [
                           "supply-chain",
                           "security",
                           "source-code",
                           "code-reviews"
                        ]
                     }
                  },

mhh, that's interesting, we've never seen this error before. Can you link to a run of the Action to help us see ur logs?

@laurentsimon here's a link to a recent run of ours: https://github.com/deislabs/ratify/actions/runs/4011814994/jobs/6889735657

So you're using the latest version, which is good. I was not able to reproduce on my end. From my logs, it seems that the CITestsID is absent, and CodeReviewID appears twice. Other repos that use the same version of the Action run just fine, so something else is going on.
Would you be able to add the following workflow to your repo and trigger it manually: https://github.com/laurentsimon/scorecard-action-test-2/blob/main/.github/workflows/test.yml
I'd like to see the logs for anything suspicious. If we're at least able to reproduce the SARIF problems but not find the root cause, I'l make a build that logs more. Will take a bit of fiddling around.
If you have another repo where you can reproduce the problem, I'd use that instead and I could troubleshoot in the repo instead.

Interestingly, this problem doesn't occur for forked repos with the exact same workflow files. Unfortunately, we have stricter governance standards for merging to our main branch and updating the workflow might be challenging. I want to try running it on the non default branch but the workflow doesn't seem to allow it? Is there a specific setting to enable to run on a different branch?

I think you may need to commit it to the branch you control and use "on: push" trigger. Then push to the branch. Would that work?

I was able to create a separate branch and updated the existing workflow file to run on Pull Request trigger. (from the scorecard code, it seems like it will only run if it's not default branch or its a pull request). the workflow was successful from the new branch. So it seems to be isolated to the main branch: https://github.com/deislabs/ratify/actions/runs/4019713495

I see. I need to be able to reproduce the problem to troubleshoot it. Any other ideas?
Anything special about your repo?

Thanks for taking a look. I'm wondering if there's some special setting/rule configured for our default main branch which is causing a conflict. I'll reach out to an owner to view the rules and update here with any more questions.


json feature gate

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We would like optional IO features
Describe the solution you'd like
Putting certain IO logic behind a feature gate and thereby reducing compilation size (including some dependencies)

Closed by #2601


Prettier window breaking

[x] Add a cracks overlay to windows as they take damage
[x] Change window description based on damage level
[x] Play nice sounds as windows take damage and break
[x] Add glass shards and make windows drop them when broken

Made some quick codersprites for cracked windows, I think they look alright but someone with more artistic talent can probably make prettier ones.

Decided I'm not adding silicate sprayers or welding shards into sheets yet. Maybe the latter shouldn't be a feature at all, it's ~unrealistic~ and I think it'd be cooler if the shards had to go through a recycling machine (like the one in the trash recycling room disposals goes to) to be turned back into sheets instead.

Lest I forget, the reason for changing gen_hit.ogg even though it has nothing to do with this PR was because I checked all the sound effects for stereo channels (which don't work with directional sounds) and that was the only one in addition to the glass ones to have them, so I downmixed it down to mono. Too lazy to make it into an atomic PR.

Conflicts

Tests seem to fail, it looks like a wrong prototype

yes I just fixed that, waiting on the test to succeed now


Various build fixes for MacOS.

These seem to bee needed to build on MacOS using default make arguments.

Thank you!


vym: 2.7.1 -> 2.8.41

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I'm getting following error while starting
Couldn't find tool to zip/unzip data. Please install on your platform and setpath in Settings menu:

I am investigating this. I have tried to bring zip and unzip to the PATH with no success.

I have found the bug: vym seeks hardcoded paths.
This should be reported to upstream.


Resource usage balancing

Peers track CPU and bandwidth usage, attributing it either to the peer's user, or to other peers

eg. Peer A sends request to peer B, peer B keeps track of all CPU/bw usage as a result of this request and attributes it to peer A

A peer has a user-configurable maximum CPU and upstream/downstream usage limit
Peers can request that other peers throttle requests by requiring a minimum time between requests

Another peer that violates this request will be disconnected

A peer maintains a global minimum request interval, which is determined using an isotonic regression to avoid exceeding the user-configurable CPU/bw limits

@sanity if you want leave any details in this issue about the conversation we had last week regarding automatically optimizing for max value (where we would define value as per some sort of processing rate - processing cost) so they don't get lost.

Relevant if we need fine-grained tracking of memory allocations: https://www.reddit.com/r/rust/comments/jfliht/how_to_detect_memory_allocations/g9l2ex4/


getCurrentPosition does not work on IOS

🐛 Bug Report
I tried using this library on Android and IOS. o
On Android it worked smoothly.
On IOS, the pop up for accepting the permission about location showed up. I accepted it, and then nothing happens
Expected behavior
I was expecting the same behavior as Android, get the coordinates as result of this function.
Configuration
Version: geolocator: ^6.2.1
Platform:

[14.4] :iphone: iOS

[✓] Flutter (Channel stable, 1.22.6, on Mac OS X 10.15.7 19H524 darwin-x64, locale
    en-FR)
    • Flutter version 1.22.6 at /Users/thales/Library/flutter
    • Framework revision 9b2d32b605 (5 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/thales/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/thales/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin installed
    • Dart plugin version 201.9317
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin installed
    • Dart plugin version 203.6912

[✓] VS Code (version 1.53.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.19.0

[✓] Connected device (2 available)
    • Android SDK built for x86 (mobile) • emulator-5554                        •
      android-x86 • Android 8.1.0 (API 27) (emulator)
    • iPhone 12 Pro (mobile)             • 1E8916C2-9A81-4B58-810D-CD272BB366F0 •
      ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)

• No issues found!```

@zerdoun the flutter doctor -v output indicates you are testing on the simulator. On the simulator you need to enable the simulation of location updates using the "Debug" -> "Location" options from the menu (detailed explanation can be found here: https://willowtreeapps.com/ideas/simulating-location-in-ios).
On a real device the first time you try to acquire a position it can take some time to get a fix (especially indoors).

@mvanbeusekom You're right, my bad that works perfectly well
Thanks a lot !


HTTPS tunneling ipv6

Hi
I have a vps with a public ipv4 and ipv6 address. If i use this vps as server for https tunneling it does not work. Are boringproxy ipv6 ready? Here the error message:
Get "http://localhost:36561/": dial tcp [::1]:36561: connect: connection refused
::1 ist the ipv6 localhost address.
netstat -atn:
tcp        0      0 <IP_ADDRESS>:36561         <IP_ADDRESS>:*               LISTEN
The tunnel are created but listening only on the ipv4 (<IP_ADDRESS>) localhost ip address. Boringproxy tried to connect at the ipv6 (::1) localhost address.
If i disable ipv6 on the vps completly it works...
Florian

Unfortunately it only supports ipv4 currently. ipv6 should be possible, but I'm not sure when I'll be able to get to it since it's not something I need myself. Part of the reason tools like boringproxy are useful is because ipv4 addresses are scarce. This is less of a problem with ipv6.

If you're using boringproxy and this is a blocker for you, I can try to get it done sooner than later.

Hi
at the moment it's not important for me but in the feature it will b nice if boringproxy will work with ipv6.
Florian

IPv6 should be working as of v0.9.0+


added options for LessphpFilter

I've added a method to pass an option-array to the less.php-parser (https://github.com/Less-PHP/less.php/blob/master/lessc.inc.php#L72)

My pull request oyejorge/less.php#187 was merged, so the setOptions() is now supported in the "official" repo.

Is there any more information if this pr is going to get merged? It's not letting us update our version of https://github.com/oyejorge/less.php

Thanks!


Secret

calico-kube-controllers pod   v3.17.1  版本 ERROR:
2020-12-13 08:21:12.385 [FATAL][1] main.go 101: Failed to start error=failed to build Calico client: could not initialize etcdv3 client: open /calico-secrets/etcd-cert: permission denied
calico-etcd.yaml  配置文件 calico-kube-controllers 组件  v3.17.1  版本 ，最低权限是0040 而不是 400
1.镜像：calico/kube-controllers v3.17.1
vim calico-etcd.yaml  ( v3.17.1  版本)
volumes:
# Mount in the etcd TLS secrets with mode 400.  calico-kube-controllers 组件  最低权限是0040 而不是 400
# See https://kubernetes.io/docs/concepts/configuration/secret/
- name: etcd-certs
secret:
secretName: calico-etcd-secrets
defaultMode: 0040

Please can i upvote this issue, i am facing the same problem. After changing the defaultMode to 0040 it works fine.

Hi @zaf187 & @fandongxiaokk
What summary / title would you recommend for this issue?
BTW https://youtu.be/o45vmed3Pcc?t=20 outlines how to upvote

permissions related issue around accessing etcd-secrets from the calico controller node, i think that's a reasonable summary / title.
This should actually be in the calico github ... i was just googling for this issue and came across this post here in this github.

Ah OK - it's not a documentation issue?

Ah OK - it's not a documentation issue?

no it isn't. IMO you can close this but i actually don't understand the post from the OP.

This seems to be off topic
/close

@fandongxiaokk  This topic is probably in the Calico Github, but just for your info setting the permission with defaultmode : 440  in the volume  mount will fix the issue.

@fandongxiaokk  This topic is probably in the Calico Github, but just for your info setting the permission with defaultmode : 440  in the volume  mount will fix the issue.


Man pages contains "generated by cobra" footer

Removes the "generated by cobra" footer from the generated man pages

it seems the code already removes the generate tag
https://github.com/reliablyhq/cli/blob/1c7955b155b5f7a326a640af0fe7c420b0ddd26a/cmd/reliably/root.go#L47
However, it does not work on sub commands for man pages
reliably.1 is Ok:
.nh
.TH "RELIABLY" "1" "Feb 2021" "" "Reliably Manual"

but not commands nor sub-commands, eg reliably-discover.1
.nh
.TH "RELIABLY\-DISCOVER" "1" "Feb 2021" "Auto generated by spf13/cobra" "Reliably Manual"

DisableAutoGenTag flag works fine for removing the "auto generated by" message in the body of the man pages, when top root command has the flag. Sub and sub-sub-... commands checks whether the flag is set on a parent command, to disable it to the current command.
This does well removes the message in the man page body. however, in the footer (last bottom line) of the man , the flag is only checked on the current command or when the source value is set (different from empty string)
The workaround to disable that message is to set " " (single space string) and the source will never be overridden to the auto gen tag message.
As a solution, I used the Source field to indicate the current Reliable CLI version
this produces a footer like:
Reliably v0.3.0        Feb 2021             RELIABLY-AUTH-LOGIN(1)

Seems to me valuable to have this information as well in the main pages (btw , this is similar to git man pages)


added workflow for closing old PRs

Fixes #22
@apurvjha123 please review and merge this PR
Also, please add labels to this PR

@apurvjha123 you forgot to add 'gssoc' label, please add it
Thank you!


creating a new framework target and a shared scheme, so it can be use…

…d by Carthage package manager

[Current coverage][cc-pull] is 94.44%

Merging [#45][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #45   diff @@
==========================================
  Files             1          1
  Lines           198        198
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            187        187
  Misses           11         11
  Partials          0          0

Powered by Codecov. Last updated by [12f0e61...77c24e1][cc-compare]
[cc-base-branch]: https://codecov.io/gh/inaka/EventSource/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/inaka/EventSource/compare/12f0e61edc8cac82352ba25f62f725e13f9c2a98...77c24e1d0d52d6cd98c8e6c743cc16462a8332cf
[cc-pull]: https://codecov.io/gh/inaka/EventSource/pull/45?src=pr

Done in #88


Kapacitor 1.5.5 is not available in Docker

Looking at https://hub.docker.com/_/kapacitor?tab=tags 1.5.4 is available but not 1.5.5.
Can that be corrected?
Thanks

@jjbuchan seems something blocked. https://github.com/influxdata/influxdata-docker/pull/393

@jjbuchan @hzliangbin It should be available now.


connection status - fade away after 1 second

Done
https://github.com/7flash/flutter_seeds_wallet/commit/c93f43652e3fa83bfb870ba3b7acdf0e4dc925f1


Consul - Error: Invalid parameter temp_dir(:temp_dir)

Hello,
I am getting the below error message when trying to install the consul module (KyleAnderson-consul (v2.1.0) on a node:
"Error: Invalid parameter temp_dir(:temp_dir)"
"Error: /Stage[main]/Consul::Install/Archive[/opt/consul/archives/consul-0.7.0.zip]/ensure: change from absent to present failed: Invalid parameter temp_dir(:temp_dir)".
Both master and node is running puppet 4.8.0 and on CentOS7.  The site.pp file has:
node 'consul' {
class { 'consul':
config_hash => {
'bootstrap_expect' => 1,
'data_dir'         => '/opt/consul',
#   'archive_path'     => '/opt/consul/archives/',
'log_level'        => 'INFO',
'node_name'        => 'consul',
'server'           => true,
}
}
}
Defining archive_path didn't help.  Also, I could not find the temp_dir in the source.  Is there anything else to check?

// , I'm getting the same error, in Puppet 4.8. There is no temp_dir in the code, that I can see.
The closest thing I can find is the archive_path in params.pp.
@rroque80, may we see the output of uname -a on both the CEntOS 7 machines?
It looks like we're both including at least the variables in the class declaration in the node definition recommended by https://github.com/solarkennedy/puppet-consul/blob/master/README.md#usage
Is this an error related to Consul, or to Puppet in general?

// , (and how can I tell?)

The error you're seeing is coming from the puppet/archive module (which this module uses).  Looks like the temp_dir parameter was added to puppet/archive in v1.2.0 - what version of that module are you using?  I can confirm that I'm not hitting this error, on CentOS 7, using puppet/archive v1.2.0.

// , @mmickan, is this the module you refer to:
https://forge.puppet.com/puppet/archive

// , I get the following in the output of $ sudo puppet module list:

$ sudo puppet module list
Warning: Missing dependency 'puppet-archive':
  'jsok-vault' (v1.1.3) requires 'puppet-archive' (>= 1.1.2 < 2.0.0)

I'll try installing it and see if that fixes the problem.

Output of uname -a:

puppetmaster: Linux puppet 3.10.0-229.el7.x86_64 #1 SMP Fri Mar 6 11:36:42 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
node: Linux consul 3.10.0-229.el7.x86_64 #1 SMP Fri Mar 6 11:36:42 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

The version of  puppet/archive is v1.2.0.
Thank you for pointing me towards the puppet/archive module.  I was able to get past that temp_dir issue by having to define temp_dir in the /etc/puppetlabs/code/environments/production/modules/archive/manifests/go.pp file, putting "$temp_dir     = /tmp/," in the "define archive::go" section and "temp_dir      => $temp_dir," in the "archive { $file_path:" section.

// , Is there any way around the puppet-archive dependency?
The puppet-archive module looks problematic for CEntOS 6:
https://github.com/voxpupuli/puppet-archive/commit/48df20b8c7ddf78744fbd38b20000806e9c2c195#diff-4ac32a78649ca5bdd8e0ba38b7006a1eR6
CEntOS 6 ONLY has Ruby 1.8.7:
https://serverfault.com/questions/815398/r10k-on-centos-6-7-has-anyone-achieved-this-feat#comment1037158_815398

You can avoid the puppet-archive dependency if you use install_method => package or install_method => none.
It looks like the camptocamp version of archive takes different parameters than the voxpopuli version does (e.g. it takes $url instead of $source, doesn't take $extract or $extract_path, etc.), so it's unlikely to work as a drop-in replacement.

// , Looks like we have pretty slim pickings for premade CEntOS 6 RPMs for either Consul or Vault:
https://github.com/hashicorp/consul/issues/61
https://duckduckgo.com/?q=hashicorp+consul
https://github.com/hashicorp/vault/issues/589
https://duckduckgo.com/?q=hashicorp+vault+rpm
Given my recent experience within my current work environment, I entirely agree with @sethvargo that there are better things to work on than accommodating the BROKEN packaging systems of most Linux systems.
As it is currently mandatory that most of our current internal packaging systems are manual (D:), I think I shall have to go with the install_method => none option. This means I shall have to add the installation process to a separate install_consul module that runs before the consul module, depends on the consul module, or something.

// , By the way, this little incompatibility implicitly drops support for Puppet 3.6 on CEntOS 6 servers, which puppet-archive does not support.

Sorry, we don't support ancient versions of ruby.
I recommend upgrading past puppet 3 as it is EOL
https://groups.google.com/forum/#!msg/puppet-users/IdutL5FTW7w/oLgkO9rbAAAJ
And newer versions of puppet come with their own ruby.

// , Understood. I wouldn't support it if I was you, either.
@EvanKrall, it looks like your suggestion worked, by the way.


RBAC Plugin - Conditional policies

Hi,
I have a vanilla backstage instance and i'm integrating the RBAC plugin.  Three questions:

At the moment CONDITIONAL policies can be configured only via API?
The conditional policies are supported in policy file?
Are groups supported in 'superUsers' configuration?

I'm using:
"@janus-idp/backstage-plugin-rbac": "^1.14.4",
Looking forward your feedback.
BR

Hello, @matdtr. We introduced file support for RBAC conditions with help of the file in the yaml format. See more:
https://github.com/janus-idp/backstage-plugins/blob/e6a1968fbab04326346569ad4cffdefd28c3c4e6/plugins/rbac-backend/README.md#configuring-conditional-policies-via-file

@matdtr Just wanted to check in and see if it was alright to close this issue out or if you had any other questions?

@matdtr Just wanted to check in and see if it was alright to close this issue out or if you had any other questions?

Hi @PatAKnight, sorry for late response yes it’s fine to close will give it a look on the update and test on my instance.


MSID: 96704 Version: 1 DOI: 10.1101/2<IP_ADDRESS>0128

MSID: 96704
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>0128
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=96704
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0128

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/96704v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>0128
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/96704/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

No affiliations for any of the authors


Receiving Error when sending with Defly

Tried to send Asset 249906631. In the console I get this error:

This error happens when I push any button in the app.

The app bar originally did some queries for ASA these were removed. Should not see any more.
Going to try sending with Defly on my end.

Was able to send with Defly using Pera

Defly transfer should work. It takes a little setting to make sure testnet is enabled and the right account is selected.

Deploying new build right now (1-2 minutes ready)

Stilll happening

Asset fetching does not appear in wallet page

But does still appear in send dialog

Removed cause of asset fetching, some unused component

Haha still happening when I go to the send asset dialog.

Giving it another go on my end

I had a look around but was not able to replicate the issue. I suspect that the update was not in effect at the time it was last checked.

All good!


Fastest way to evaluate a big amount of data

I have a pretrained cnn model and I want to evaluate 50000 images. I have two methods for prediction but they take about 1 day to evaluate. I'm wondering if there is a way to do it faster.
Sumary of the code:
batch_size = 144
eval_data = mx.io.ImageRecordIter(
...
    batch_size          = batch_size,
    data_shape          = (3, 224, 224),
)
ctx = mx.cpu()
prefix = 'model-cnn'
epoch = 1
eval_data.reset()
eval_data.next()
eval_batch_x = eval_data.getdata()
print(eval_batch_x.shape) # -> (144, 3, 224, 224)

First method with the executor:
symbol, arg_params, aux_params = mx.model.load_checkpoint(prefix, epoch)
arg_params, aux_params = params_update(arg_params, aux_params, ctx=ctx) #update the params and put them in the correct form
exe = symbol.bind(ctx, arg_params ,args_grad=None, grad_req="null", aux_states=aux_params)
exe.forward(data=eval_batch_x, is_train=False)
prob = exe.outputs[0].asnumpy()

Time of first method:
CPU times: user 5min 31s, sys: 15.6 s, total: 5min 47s
Wall time: 4min 29s

Second method with FeedForward:
model = mx.model.FeedForward.load(prefix, epoch, ctx=ctx, numpy_batch_size=batch_size)
pred = model.predict(X = eval_batch_x)

Time of second method:
CPU times: user 5min 24s, sys: 28.1 s, total: 5min 52s
Wall time: 4min 41s

So it takes around 4 min to evaluate 144 images, so for 50000 images it will take around a day. Is there a faster way?

use gpu?

I tried to use mx.model.FeedForward.load with ctx = [mx.gpu(0), mx.gpu(1), mx.gpu(2), mx.gpu(3)] and ctx = mx.gpu(0) but the time is even worse.
Any idea?

I added ctx=mx.gpu(0) and it was faster. 50000 images would take around 13h. I tried with 4 gpus but I think it only takes one. Thanks!


Complete parser coverage

UnitTest: cover FreeRTOS_DNS_FreeRTOS
Description
UnitTest: cover FreeRTOS_DNS_FreeRTOS.c branches at 100%
Remove dead code.
Test Steps
Unit Tests pass with code coverage at 100%
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run uncrustify

/bot run uncrustify


[iOS]Headset control not working

After upgrading to 1.1.2, headset control (remote-pause, remote-play) seems broken.
My setup code:
export const configurePlayer = (store) =>
  TrackPlayer.registerPlaybackService(() => async () => {
    TrackPlayer.addEventListener('remote-play', async () => {
      TrackPlayer.play()
    })

    TrackPlayer.addEventListener('remote-seek', async (data) => {
      TrackPlayer.seekTo(data.position)
    })

    TrackPlayer.addEventListener('remote-pause', async () => {
      TrackPlayer.pause()
    })

    TrackPlayer.addEventListener('remote-stop', async () => {
      TrackPlayer.destroy()
    })
  })

export const setupPlayer = async () => {
    await TrackPlayer.setupPlayer()
    await TrackPlayer.updateOptions({
      capabilities: [
        TrackPlayer.CAPABILITY_PAUSE,
        TrackPlayer.CAPABILITY_PLAY,
        TrackPlayer.CAPABILITY_SEEK_TO,
        TrackPlayer.CAPABILITY_STOP,
      ],
      compactCapabilities: [
        TrackPlayer.CAPABILITY_PAUSE,
        TrackPlayer.CAPABILITY_PLAY,
        TrackPlayer.CAPABILITY_SEEK_TO,
        TrackPlayer.CAPABILITY_STOP,
      ],
      notificationCapabilities: [
        TrackPlayer.CAPABILITY_PAUSE,
        TrackPlayer.CAPABILITY_PLAY,
        TrackPlayer.CAPABILITY_SEEK_TO,
        TrackPlayer.CAPABILITY_STOP,
      ],
      stopWithApp: true,
    })
}

Configuration

react-native: 0.57.8
react-native-track-player: 1.1.2

System Info

System Version: 12.1.4
Model: iPhone SE

Still exist in 1.1.3.

Seems like there are no other issues mention about this problem. Am I doing something wrong in my setup code? Or is there any clue I can debug this problem?

I Have the same issue and I dont know what that's happening. Had you any luck so far?

@appinteractive not yet, I have a backup solution using react-native-video + react-native-music-control to stream audio, so I don't go into this problem further (I'm not familiar with Swift/Obj-C).

@ouabing seems like @anderslemke is working on the issue: https://github.com/react-native-kit/react-native-track-player/pull/531

Wow thanks!


Feature Request - Support Stream

Describe the feature or improvement you're requesting
OpenAI has a great feature called stream (you can see it in action in the playground). I'd like to suggest that this feature be implemented in this library.
https://beta.openai.com/docs/api-reference/completions/create#completions/create-stream
Additional context
No response

Thx @isotropicdesign, I'll be working on it.

Done 🚀 , please check
https://github.com/orhanerday/open-ai#stream-example


Remove pkg_resources import

Apparently some Python Lambda AMIs don't include pkg_resources which is
supposed to be a part of the Python stdlib. This hotfix removes that import from the constants module.

If this is a prod hotfix, don't block on my account, do the thing and ship it.


Feature 'Format On Save Mode' in settings doesn't work properly

Type: Bug
https://code.visualstudio.com/updates/v1_49#_only-format-modified-text
According to this :arrow-up: update, choosing option 'modifications' in the 'Format On Save Mode' menu would lead to the behaviour when pretier would format only lines that you've changed. And if there would not formatted pieces of code already commited before - it wouldn't format them.
But in reality it format those pieces of code too. It works this way: if you have 'modifications' turned on, it doesnt format the file on save if you haven't made any changes to the file. But as soon as you change even one single line - it formats the whole file..
VS Code version: Code 1.74.2 (Universal) (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:26:09.430Z)
OS version: Darwin arm64 21.6.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
5, 6, 6

Memory (System)
16.00GB (0.07GB free)

Process Argv
--crash-reporter-id 1b43ade5-b992-49b8-9cdb-be7f8e7a23f7

Screen Reader
no

VM
0%

Extensions (39)

Extension
Author (truncated)
Version

codesnap
adp
1.3.4

vscode-sql-formatter
adp
1.4.4

vscode-color
ans
0.4.5

bemhelper
Box
1.4.5

htmltagwrap
bra
0.0.7

vscode-eslint
dba
2.2.6

es7-react-js-snippets
dsz
4.4.3

gitlens
eam
13.2.0

EditorConfig
Edi
0.16.4

prettier-vscode
esb
9.10.3

auto-rename-tag
for
0.1.10

code-runner
for
0.11.8

shell-format
fox
7.2.2

beautify
Hoo
1.5.0

rest-client
hum
0.25.1

elixir-ls
Jak
0.13.0

vscode-gutter-preview
kis
0.30.0

vscode-language-babel
mgm
0.0.36

vscode-language-pack-ru
MS-
1.74.12140931

data-workspace-vscode
ms-
0.3.0

mssql
ms-
1.16.0

sql-bindings-vscode
ms-
0.3.0

sql-database-projects-vscode
ms-
0.19.0

live-server
ms-
0.4.4

vsliveshare
ms-
1.0.5797

vscode-css-navigation
puc
1.13.3

vscode-thunder-client
ran
2.3.0

vscode-commons
red
0.0.6

vscode-xml
red
0.23.0

live-sass
rit
3.0.0

LiveServer
rit
5.7.9

html5-boilerplate
sid
1.1.1

autoimport
ste
1.5.4

vscode-styled-components
sty
1.7.5

vscode-stylelint
sty
1.2.3

sass-indented
syl
1.8.23

highlight-matching-tag
vin
0.10.1

JavaScriptSnippets
xab
1.8.0

ReactSnippets
xab
2.4.0

(3 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
2e4cg342:30602488
3d0df643:30613357
f6dab269:30613381
fim-prod:30623723

This issue lacks

mentioning of the language and extension providing formatting edits
copy-paste friendly source code samples

We tried it on TypeScript and XML also.

As you can see on the gif below, before editing this file I saved - and nothing have been formatted which is fine.
But after I edited one line at the end of the file and saved it - it formatted the whole document although meant to format only newly added line

https://github.com/zhkvivan/example

We don't provide a built-in XML provider. Which XML extension do you use?

Please file the issue against the extension that provides the formatter.


Make a few minor contributions

Just correcting a small typo and changing key. Fixes https://github.com/github/docs/issues/20508

Why:
Fixing a small typo in the docs and changing subkey in paste-gpg-key-id.md to match copy-gpg-key-id.md :memo:

What's being changed (if available, include any code snippets, screenshots, or gifs):
- Once your PR is merged, your contributions will be publicly visible on the [GitHubs docs](https://docs.github.com/en).
+ Once your PR is merged, your contributions will be publicly visible on the [GitHub docs](https://docs.github.com/en).

- Alternatively, when setting a subkey include the `!` suffix. In this example, the GPG subkey ID is `4BB6D45482678BE3`:
+ Alternatively, when setting a subkey include the `!` suffix. In this example, the GPG subkey ID is `42B317FD4BA89E7A`:
```shell
- $ git config --global user.signingkey 4BB6D45482678BE3!
+ $ git config --global user.signingkey 42B317FD4BA89E7A!

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Hey there folks, not too sure what the 'changes in staging' checkbox means so I've leaved that unticked.

@applecuckoo Thanks so much for opening a PR! I'll get this triaged for review :zap:

That was along Journey 🌻✨🍃

Duplicate of #


Needs to be started as network user/group in openwrt

Hi, thanks for the integration - it was included in openwrt head today.
In order for it to work I needed to alter the init script to start it as the network user - it can't access the control interface otherwise.

I've the same problem. Could you explain how can I do it?

Edit /etc/init.d/wifi-presence
Add to start_service():

procd_set_param user network
procd_set_param group network

I added the two parameters. I don't have the error anymore, but I think I have something bad configured. I can't see anything at the MQTT server. I will look again to the configuration.
Thanks for the help!
@awilliams could you add this two parameters to the OpenWrt package?

I've only seen processes run as root on OpenWrt unless explicitly set otherwise. For this reason, I've had no permissions issues. Is wifi-presence being run as root for you and you still see the permissions issues with the control sockets?
Instead of hardcoding network, I'd like to make it configurable (similar to kcptun for example). Would that solve the issue you're experiencing?

I can't say too much, I don't have too much experience with Linux or Openwrt. I only installed the package with opkg and I experienced the issue until I added the lines commented by @timdodge.

I'm not an expert on socket "files", but the permission on the control sockets are:

root@wap:~# ls -la /var/run/hostapd/wlan0
srwxrwx---    1 network  network          0 Apr 11 12:25 /var/run/hostapd/wlan0

When running as the root user, wifi-presence can't access them. I've tried catting the socket as root and just get a "No such device or address" error back. I guess that as a minimum, the wifi-presence user needs to be in the network group.

@awilliams I encountered this issue also, and was resolved using the changes suggested by @timdodge. Feel free to ping me if you want to make these changes directly in OpenWrt.

I've created a PR that I think will address this issue: PR #6.
Please try it out if you're able to. After updating the /etc/init.d/wifi-presence script, add the following to /etc/config/wifi-presence to run the process as network / network:
         option runAsUser 'network'
         option runAsGroup 'network'

@awilliams can you share a bit more detail on your OpenWrt builds? Are you running an old build/snapshot, or a target where seccomp is unavailable? Given that seccomp is enabled by default on most (all?) targets, I think that the user/group options should be uncommented in the config file.
https://github.com/awilliams/wifi-presence/pull/6/commits/ad2b6bde4e7dc99c449737d288e488a4dd2ff684#diff-5000e54b7b529e6c12b270a4c9335886f97c9b4065dda3d789740f4e7a042aa5R52-R53

@jmccrohan Thanks for letting me know that seccomp is enabled by default. I'm using a custom build of OpenWrt for my routers, so wasn't aware.
What do you think about the following idea? Modify the changes in #6 to auto-detect the proper user if runAsUser/runAsGroup aren't explicitly defined. In other words, instead of defaulting to root, do something like stat the /var/run/hostapd/wlan0 sockets and run wifi-presence as the owner of that file.

@awilliams Up to you how you implement it. A simple comment might in the config file re root/network users might suffice and is easier to maintain. Key is to get this fixed and merged into OpenWrt so that new users don't get bitten by this issue. :-)

I think the recent change in #6 will work fine.
It allows setting a user/group in /etc/config/wifi-presence. If no specific values are set, then the init script will look at the owner of the sockets in /var/run/hostapd and use that user as the user/group to start wifi-presence.
If you're able @jmccrohan, please review. Once merged in this repo, I'll make the same change to openwrt/packages.

Thanks, LGTM.
Open a PR against openwrt-packages and ping me to merge.

https://github.com/openwrt/packages/pull/18573

Fixed by openwrt/packages#18573


bst build does not handle elements with Consistency.INCONSISTENT

See original issue on GitLab
In GitLab by [Gitlab user @tristanvb] on May 19, 2017, 05:04
When typing bst build with inconsistent sources (sources that have no ref and need to be tracked), BuildStream just goes ahead and starts to try building these, but will inevitably fail because there are no refs.
We could have different approaches here:

Just bail out with a message to the user
Try building any elements which could possibly be built, but omit any inconsistent elements (by prefiltering the element list)

In GitLab by [Gitlab user @tristanvb] on Jul 27, 2017, 16:15
This is no longer true.
Now we assert consistency of the pipeline before running the build (unless bst build --track has been specified).

In GitLab by [Gitlab user @tristanvb] on Jul 27, 2017, 16:15
closed

In GitLab by [Gitlab user @tristanvb] on Nov 5, 2020, 06:58
marked this issue as related to #684

In GitLab by [Gitlab user @tristanvb] on Nov 5, 2020, 06:58
removed the relation with #684

In GitLab by [Gitlab user @tristanvb] on Nov 6, 2020, 09:26
marked this issue as related to #1375

In GitLab by [Gitlab user @tristanvb] on Nov 6, 2020, 09:26
removed the relation with #1375


[A11Y] accessibility buffer should return to the correct state after accidental user edits

Type: Bug
After mutating the accessibility buffer for the integrated terminal in the latest iteration of VS Code, the state of the buffer does not restore after tabbing away and refocusing the terminal accessibility buffer using the keyboard. This makes it so that screen reader users cannot get the correct information after inadvertently modifying the buffer. Additionally, if the buffer is in an incorrect state, newly coming output from the terminal will not be reflected in the buffer, making that particular terminal session unusable for screen reader users.
Steps to Reproduce:

Open VS Code in the screen reader optimized mode
Launch a new integrated terminal
Launch a command that outputs text to the terminal, such as echo "hello"
Shift-tab to the accessibility buffer and observe that the information is presented correctly
Modify the buffer by typing arbitrary characters (or in the extreme case, press ctrl+a and delete the entire buffer)
Tab back to the shell and shift-tab to the buffer again, observe that the buffer remains in the incorrect state
Tab back to the shell and type in the command that outputs to the terminal again
Shift-tab back to the buffer, observe that the buffer remains in the incorrect state and the newly output information does not appear in the buffer.
Actual result: The buffer do not return to its correct state after edit.
Expected result: After the buffer has been modified, refocusing the buffer via keyboard should return it to its correct state.
Note: This used to work during the early iteration of the accessibility buffer. Tested using NVDA 2022.4 on Windows 11 Pro and the latest commit on the main branch at the time of this report (commit: 339fd630e60)

VS Code version: Code - OSS Dev 1.75.0 (Commit unknown, Date unknown)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
AMD Ryzen 7 PRO 5850U with Radeon Graphics      (16 x 1896)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
27.83GB (1.18GB free)

Process Argv
.

Screen Reader
yes

VM
0%

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.74.3. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

The problem still occurs as of the latest version of VS code.


Query-and-update animals table

In this PR I :

Use database transactions.
Modify and delete data in SQL.
Prepare complex queries that answer analytical questions.

Thank you @GedeonTS


terminal display bug

VSCode Version: Code 1.11.2 (6eaebe3b9c70406d67c97779468c324a7a95db0e, 2017-04-13T08:03:11.395Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

vscode-html-css
ecmel
0.1.2

tortoise-svn
fantasytyx
0.1.1

node-module-intellisense
leizongmin
1.3.0

vetur
octref
0.6.2

vscode-icons
robertohuertasm
7.6.0

Steps to Reproduce:

Well it is actually very bug output:

I wonder when it will become sustainably suitable for work at last =)

Duplicate https://github.com/Microsoft/vscode/issues/19665
@whitecolor it's going into the next patch of Windows AFAIK, you could try enabling the legacy console to try workaround in the meantime https://github.com/Microsoft/vscode/issues/19665#issuecomment-294625978

@Tyriar yeah thanks for remind, it seems that I had turned off legacy mode some time ago, I forgot that it was a fix for vscode console. No it seem to run fine.


Fix mistyped ngbDropdownMenu in some ngbDropdown ul elements

Some dropdown menu did not show up, because ngbDropdown  directive was used in the ul element instead of ngbDropdownMenu.
This issue appeared in the notification component page, and in the traffic section in the dashboard.
It works in the live demo page, but did not work in my localhost before this correction.

Thanks!