[Swift] Add class that implements the decoding protocol for the Swift Codable implementation

Describe the enhancement requested
As a Swift user I would like to be able to decode an Arrow RecordBatch to my custom type.  This is functionality is similar to the Swift JSONDecoder.
Component(s)
Swift

Issue resolved by pull request 42023
https://github.com/apache/arrow/pull/42023


update ibm-spectrum-scale-csi-operator to 1.0.1 upstream

Signed-off-by: John Dunham<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

@mew2057, as CICD is new, looks like an issue with CRD filename.

I updated the  crd name, will verify ci/cd

/retest

Need to rerun travis,  looks like the image didn't have the fix that allows this cicd  to be run.

any thoughts on this current failure?

cancellingg


AuthException returns JSON as exception message

This is more of an inconsistency bug rather than a functional bug.
Reproduction steps:

Authorize a user by getting an access token.
Unlink that user (makes the access token invalid)
Call any method (in my case it's GetCurrentAccountAsync)
An AuthException is thrown, message is in JSON format, instead of just a text description.

Although it's easy to parse the JSON and get the ".tag" value (that's what the exception returns as message), it would be nicer if there was a consistency with all other exception messages.
Suggestion:
The problem may lay in DropboxRequestHandler.RequestJsonString (line 400). The returned JSON doesn't contain the "error" key so CheckForError fails as the key is ".tag".

This has been fixed in v2.3.2. Now AuthException is no longer thrown. Instead, the app should catch ApiException which contains deserialized object of the error json.

Update: In order to retain the capability to distinguish api vs authentication related errors. We decided to keep AuthException class and add an extra ErrorResponse field of type AuthError so the client can handle it the same way as ApiException.


Feature/add crash reporting

Asana:
https://app.asana.com/0/414730916066338/471193197990046
Description:
Add crash reporting into the app, using ACRA
Steps to test this PR:

Deliberately throw an exception
Ensure a dialog is presented offering to send the crash report

At the moment, I have been unable to confirm that reports are being received and processed correctly. Awaiting action from @nilnilnil to ensure the server is up and running as expected.

@subsymbolic I realised that the existing config didn't play nicely with Instant Run, so haven't changed the way crash reporting is configued.
RR

Removed duplicated license files and added a padding. all feedback addressed.


Dark mode

Inevitable 😂

Unocss Dark mode https://unocss.dev/presets/mini#dark-mode
Vueuse dark https://vueuse.org/core/useDark/

Hi @Readpato thanks for steping in, yes I mean the actual GUI interface, I will fix the docs dark mode in another PR

Perfect then, I'll proceed with adding dark mode to the GUI then!

Hey @alvarosabu would you know why this error appears? Can seem to fix it, I have tried
GET http://localhost:5173/src/main.ts net::ERR_ABORTED 500 (Internal Server Error)

Reinstalling deps
changing import to @tresjs/leches/styles.css
Changing node version from 20.10.0 to 19.3.0
But sadly no luck.

I have also seen the answer you gave to Marcin Dekier in discord about the same issue and solving it by importing the styles globally on his Nuxt project maybe it's something related to it?

Try building the lib, if you change the source on the viteconf to src instead of dist the alias wouldn't work and you would need to import the styles from complete the .css path

Perfect, it worked after running pnpm run build

Yo buddy @Readpato I fixed some css issues and updated the deps on the latest release, make sure you merge latest main into your branch

Purrfect, will do it! Thanks <3


Kata for find the unique string

Coverage remained the same at 100.0% when pulling 525855cbdf414b6eb4ce50b06ad23e3639a066ff on kata into 592e31f98dcdddd39e1491101ac1c61aaa9ca0fd on master.

Completed find unique string kata


[netcore][51.cafe-bot] Cafe Bot is not behaving as expected after greeting

Sample information

Sample type: [\samples\]
Sample language: [dotnetcore]
Sample name: 51.cafe-bot

Describe the bug
The bot can not set the state of gathered properties after being greeted (please see Issue https://github.com/southworkscom/BotBuilder-Samples/issues/119 for a related issue).
To Reproduce
Steps to reproduce the behavior:

Go to ...\BotBuilder-Samples-SW\samples\csharp_dotnetcore\51.cafe-bot.
Open CafeBot.sln
Select and run CafeBot project
Open the bot in Bot Framework Emulator
Send Hi
The bot will reply with Hi! Is there anything else I can help you with?
Either choose or send Book a Table
The bot will reply with Sorry, it looks like something went wrong

Expected behavior
The bot should ask for the city where to book the table, as it does when Book a Table is sent without a previous greeting.
Screenshots
The following gif shows first how the Book_Table intent works without a previous greeting, then the same message is sent but after greeting the bot.

Additional context
The problem originates in line 135 of CafeBot.cs, when the _onTurnAccessor.SetAsync(context, onTurnProperties) is called after a greeting it throws the following exception:
The value was of an incorrect type for this dictionary.

[bug]

Closed due to Microsoft#850


Use more portable shebang paths in scripts

Thanks. I merged that into my fork for now, since I'm already working on another release there.


When installing Permissions table missing and permission itself

After executing this command:
php artisan create:user username email password Admin
Laravel instantly pops up a whoops error stating that the table "Permissions" does not exist. Then I manually created it on phpmyadmin. This command above will leave the field "Permissions" in the Users table empty. As a consequence of that, when you go the Dropdown menu (Users, groups, permissions) on the Dashboard and select permissions, you will find a blank page. You need to insert a json formatted permission in the blank cell in the Users table so that the Blade Interface page for Permission can be displayed when selecting Permissions from that dropdown menu. These were the issues I encountered. It would be nice to have a simple but complete example of how to code in the controllers and the models those permissions and groups, since the sentry documentation does not do it.

When I create a new user, and I select a Permissons out of the select list, this does not get assigned to the user.  This is due to that original error where the permissions table is not created by default at Sentry, so the Syntara does not seem to address that issue. The checkboxes to assign groups work, they assign user and group to the users_group table.

Merci

Hi, this command php artisan syntara:install will create the Permission table. You need to launch this command before creating the user.


test(jasmine): add failing test for custom matchers on a not-present element

Custom matchers fail if used on an element that is not present (not in the DOM), but only if used more than 2 times.
This specs show this. Will open an issue after the travis build fails.

Thanks for bringing this up. I feel like this is better handled in the issue you opened unless your PR is updated with a fix; you could still keep your fork as an example. I'll defer to an actual maintainer on that though :)

I made the PR so that the maintainers could easily see how the specs are failing on the Travis build. It's not intended to be merged.

Thanks for the actual example, this is helpful. This smells like a webdriver control flow issue - let me try it with the latest master.

With the latest webdriver, this does not give the element not found errors. Instead, it occasionally errors because it doesn't grab the correct text from the element. The problem isn't deterministic.
However, the failures only occur when doing the animation toggle. So I think that the issues are with waiting for the animation, not with the custom matcher.
Can you reproduce on master with something other than the animation toggle?

I ran my test suit on the master branch of protractor and it didn't have this issue anymore.
I'll close the issue.
Are you interested in merging in any of these specs (once a proper wait for the animation has been added)?
Myself, I feel like protractor test suit could benefit from specs on custom matchers, in file dedicated for that, since currently there is only one spec for that.

Sure, I'd be happy to take a look at a PR adding more tests. Feel free to open up a new one! I don't think it has to be related to the animation stuff at all. For now, I'm going to close this since it seems to no longer be an issue.


Intermittent test failure: managed encryption > end-to-end > should reject with a reasonable message given incorrect passphrase

I've seen this test fail once, as the only failure in the whole suite:
  1) managed encryption
       end-to-end
         should reject with a reasonable message given incorrect passphrase:
     Error: expected 400 "Bad Request", got 500 "Internal Server Error"
      at Test._assertStatus (node_modules/supertest/lib/test.js:266:12)
      at Test._assertFunction (node_modules/supertest/lib/test.js:281:11)
      at Test.assert (node_modules/supertest/lib/test.js:171:18)
      at Server.assert (node_modules/supertest/lib/test.js:131:12)
      at Server.emit (domain.js:475:12)
      at emitCloseNT (net.js:1677:8)
      at processTicksAndRejections (internal/process/task_queues.js:81:21)

Scrolling back up the logs:
::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "POST /v1/sessions HTTP/1.1" 200 228
::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "POST /v1/projects/1/key HTTP/1.1" 200 16
::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "GET /v1/projects/1/forms/simple.xml HTTP/1.1" 200 1285
::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "POST /v1/projects/1/forms/simple/submissions HTTP/1.1" 200 127
::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "POST /v1/projects/1/forms/simple/submissions/one/attachments/submission.xml.enc HTTP/1.1" 200 16
::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "GET /v1/projects/1/forms/simple/submissions/keys HTTP/1.1" 200 492
Error: error:0909006C:PEM routines:get_name:no start line
    at createPrivateKey (internal/crypto/keys.js:351:10)
    at /x/central-backend/lib/util/crypto.js:151:14
    at async Promise.all (index 0) {
  library: 'PEM routines',
  function: 'get_name',
  reason: 'no start line',
  code: 'ERR_OSSL_PEM_NO_START_LINE'
}Error: error:0909006C:PEM routines:get_name:no start line
    at createPrivateKey (internal/crypto/keys.js:351:10)
    at /x/central-backend/lib/util/crypto.js:151:14
    at async Promise.all (index 0) {
  library: 'PEM routines',
  function: 'get_name',
  reason: 'no start line',
  code: 'ERR_OSSL_PEM_NO_START_LINE'
}::ffff:<IP_ADDRESS> - - [23/Jun/2022:08:39:04 +0000] "GET /v1/projects/1/forms/simple/submissions.csv.zip?4=wrong HTTP/1.1" 500 339
      1) should reject with a reasonable message given incorrect passphrase

Thanks for filing this! That message rings a bell for me: I think I've seen this intermittent test failure as well.

This is still happening for me :slightly_frowning_face:

This is also happening for me on occasion locally.


TST: clean up test directories in shuffle

a couple of tests leaving around a temporary directory.

Merged.  Thanks @jreback !


fix broken link to the Coding Horror blog post in CONTRIBUTING.md

There's currently a busted link in CONTRIBUTING.md, but this PR fixes it.

Looks great, thank you!


Protoc compile error by make

Steps:
git submodule update --init
make clean
make

errors:
......
[CXX]     Compiling src/cpp/util/time.cc
[PROTOC]  Generating protobuf CC file from src/proto/grpc/testing/echo_messages.proto
[GRPC]    Generating gRPC's protobuf service CC file from src/proto/grpc/testing/echo_messages.proto
/Users/thinkerou/grpc/bins/opt/grpc_cpp_plugin: program not found or is not executable
--grpc_out: protoc-gen-grpc: Plugin failed with status code 1.
make: *** [/Users/thinkerou/grpc/gens/src/proto/grpc/testing/echo_messages.grpc.pb.cc] Error 1
➜  grpc git:(master) ✗

Could you run the following commands to see if grpc_cpp_plugin could be built and is executable.
make grpc_cpp_plugin
ls -l bins/opt/grpc_cpp_plugin

I could not repro.

I'm seeing this as well. bins/opt/grpc_cpp_plugin does not exist until I explicitly run "make grpc_cpp_plugin". Once I've done that, "make" succeeds.
Looking at the Makefile, PROTOC_PLUGINS and PROTOBUF_DEP are not set if HAS_SYSTEM_PROTOBUF and HAS_PKGCONFIG are both true (which they are on this system), which looks like it will result in this breakage: targets like echo_messsages.grpc.pb.cc are left without a dependency on the plugins. "make HAS_SYSTEM_PROTOBUF=false" succeeds.
Before commit cc7eb8ef88dcbd11acb3f24000e1628026b975c5, PROTOC_PLUGINS was always set. That commit probably introduced this bug.
I think the first occurence of "PROTOC_PLUGINS = $(PROTOC_PLUGINS_ALL)" needs to be moved down a line, so that it always applies if HAS_SYSTEM_PROTOBUF is true, whether or not HAS_PKG_CONFIG is true.

@y-zeng I use your command after, make is OK, but sudo make install errors, the follow info:
...
[INSTALL] Installing libgrpc++_unsecure.dylib
[INSTALL] Installing grpc protoc plugins
install: /Users/thinkerou/grpc/bins/opt/grpc_csharp_plugin: No such file or directory
make: *** [install-plugins] Error 71

Fixed in #7518


JGRP-1992 GOOGLE_PING discovery fails if port 443 isn't used

Change GOOGLE_PING to use port 443 when creating a connection and no
port is specified in the configuration.

Thanks for the fix, Alan. Did you test this with a real GCE account, or did you use some simulator?

I tested this with a real GCE account, Bela.

OK. I read that there is a Google Cloud server that can be run off of the local file system to check a client's correctness.
Might be worth taking into consideration for future unit testing...

That would be interesting. Do you have a link with more information?

https://cloud.google.com/datastore/docs/tools/devserver

Thanks a lot, Bela


Make a commands button

Along with the mic button, try to add a button which enlists all the commands BENJI can process

We are trying to make a guide for B.E.N.J.I. Hence this issue might be more directed to the 'guide button'.  But right now the guide is in process of making. This is an issue for the future.


Move piggui networking code into a module or crate

This would enable the use of the code to connect and interact with a piglet/porky to be used in other places, such as a CLI, or integration with HomeAssistant or other apps, without having to do the networking part all over again.
Problem would be the tch_helper and iroh_helper code, and some serializations/deserialization, maybe some file loading and saving methods also.

Probably would make sense to implement #505 first

Not sure why I didn't do a module in that networking folder.
If I do that it could be merged and used by both piggui and piglet

Fixed by https://github.com/andrewdavidmackenzie/pigg/pull/658


Singeltonize Measure, Reset, and Barrier

What should we add?
In https://github.com/Qiskit/qiskit/pull/10314#issuecomment-1744466964 it was pointed out after #10314 merged that we can make some of the circuit instructions singletons pretty easily just as we did for parameterless gates. The Measure, Reset instruction are super straightforward to convert to singletons because they are single qubit/clbit constructs and do not take any parameters. Barrier also conceptually can be a "singleton" with a globally shared instance for barriers  but as its variable width we'd need to store a distinct instance for each value of number of qubits that Barrier is instantiated with, which will require some custom handling as none of the singleton fixtures support this yet.

Hii @mtreinish!
If this issue still unassigned, can I take up this issue

@Matrixmang0 sure if you have the bandwidth for this feel free to give it a try. I'd recommend trying to use Measure or Reset to start (as the Barrier will be a little trickier)

I'll certainly do my best. As I'm still getting acquainted with the repository, I'm uncertain about my bandwidth to tackle this issue. However, with your guidance, support from this incredible community, I'm hopeful that I can navigate through and resolve the issue :)

Update on this: #11170 made Measure and Reset singletons, so that part of the issue is solved.  Barrier is rather more non-trivial (and arguably will provide little benefit) because it would need greater modifications to the singleton machinery, which is tricky.  We already have a prototype of this internally, made for other reasons, but it may not be worth the cost of adding it - the potential savings from barriers are not expected to be large.

Given the last comment here I'm going to close this as complete because I think Barrier isn't worth singletonizing as there typically are only a few barriers in a circuit and dealing with the variable width is more effort than the benefits it would provide.


Helpfile clarifications

In this issue we'll make some adjustments to the helpfile following suggestions from @SimonFreyaldenhoven.

I took a final pass at these help files following the changes we have made in the paper. We can revisit if necessary later.

Summary: In this issue we modified the helpfiles following changes in the paper.
Thread continues in #158


Disabled autoload reference to removed script causes IDE crash on load

Godot version:
3.1 Stable Official
OS/device including version:
Win 10 Home 10.0.17134 Build 17134
Issue description:
I created an autoload script, then disabled it. Later I removed the script entirely. After that, any time I open that specific project Godot will crash. Removing the autoload reference from the project fixes the issue. The issue also does not occur if I leave the autoload enabled even after the script has been removed.
Steps to reproduce:

Create script and add to autoload list
Disable the script in the autoload list
Remove the script file
Reopen the project, IDE crashes

Minimal reproduction project:
autoload-bug.zip

Backtrace from Ubuntu
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x3ef20) [0x7ff1bfb97f20] (??:0)
[2] Object::_predelete() (/home/rafal/Pulpit/godot/core/object.cpp:385)
[3] predelete_handler(Object*) (/home/rafal/Pulpit/godot/core/object.cpp:2011)
[4] void memdelete<Node>(Node*) (/home/rafal/Pulpit/godot/./core/os/memory.h:117)
[5] EditorAutoloadSettings::EditorAutoloadSettings() (/home/rafal/Pulpit/godot/editor/editor_autoload_settings.cpp:777)
[6] ProjectSettingsEditor::ProjectSettingsEditor(EditorData*) (/home/rafal/Pulpit/godot/editor/project_settings_editor.cpp:2018 (discriminator 7))
[7] EditorNode::EditorNode() (/home/rafal/Pulpit/godot/editor/editor_node.cpp:5493)
[8] Main::start() (/home/rafal/Pulpit/godot/main/main.cpp:1596)
[9] /usr/bin/godot(main+0xc9) [0x12f6e70] (/home/rafal/Pulpit/godot/platform/x11/godot_x11.cpp:54)
[10] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xe7) [0x7ff1bfb7ab97] (??:0)
[11] /usr/bin/godot(_start+0x2a) [0x12f6cea] (??:?)


Add package version to installation command

Installation command in the Readme is missing the @latest part, and attempt to run ends with
go: 'go install' requires a version when current directory is not in a module
	Try 'go install github.com/alesr/codetoprompt@latest' to install the latest version

Much appreciated ❤ @mmllski


Add weighted mean docs.

I like @fujiisoup's weighted mean demo in this stack overflow example:
https://stackoverflow.com/questions/48510784/xarray-rolling-mean-with-weights
I thought it'd be a useful addition to the docs on rolling.

Thanks a lot @dcherian !
And @fujiisoup for the feature and SO post!


can give an example to Subscribe  multiple exchange multiple symbols ？

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@xtuyaowu Does this work for you? https://github.com/crypto-chassis/ccapi#multiple-exchanges-andor-instruments


How can I control the duration ?

Is there anything that can allow me to change the duration of the silence pattern ?
I mean that I want the "sil" or "pau" to be 0.5 seconds or anything , how can I achieve this ?

You can do so while [preparing labels] (https://github.com/CSTR-Edinburgh/merlin/blob/master/egs/slt_arctic/s1/scripts/prepare_labels_from_txt.sh) from text.
[normalize_lab_for_merlin.py] (https://github.com/CSTR-Edinburgh/merlin/blob/master/misc/scripts/frontend/utils/normalize_lab_for_merlin.py) script does lab normalization as per either phone_align/state_align -- you can change the silence duration there.
pause duration is nothing but a phrase break, which is predicted w.r.t context and I don't recommend modifying it to a fixed duration.

So do you mean that it takes the silence pattern given to him and give it the duration from the normalize_lab_for_merlin. ?
Now I get the state labels from another source so if I want the silence pattern to be for a certain duration I should give it a fixed duration in this labels and then whenever it sees it , it will take this duration ,right ?

Yes, you are right. Silence duration is usually not predicted -- so whatever duration the initial labels has for silence, the same is reflected in synthesis.All other phones including pause are predicted from a duration model.

Yes yes , thanks for explaining (Y)


One Cycle policy in Ludwig

Hi @w4nderlust ,
is there any plan to implement Smith's One Cycle policy in the learning rate scheduler? (https://arxiv.org/pdf/1803.09820.pdf)
I see that currently the default policy is to anneal the LR (or the batch size). The One Cycle scheduler has shown very good results since it was introduced, and it seems much faster to converge than other schedulers.

Why don't you attempt a PR about it? :) All the relevant code is in the train function: https://github.com/uber/ludwig/blob/master/ludwig/models/model.py#L267

I don't have much familiarity with tensorflow unfortunately 😅
Also, in pytorch it's already implemented (https://pytorch.org/docs/stable/optim.html#torch.optim.lr_scheduler.OneCycleLR), but in TF I wouldn't know where to start, I can't find the equivalent.

@w4nderlust I am willing to work on this, if no one isn't.

@lordeddard that would be great. For starters i suggest you to look at the train function of model.py https://github.com/uber/ludwig/blob/master/ludwig/models/model.py#L293 and look at how the learning rate is treated there. Once that is clear we can discuss how to implement the one cycle policy.

Hi @w4nderlust , was a bit busy last week. Yes, I will take a look at that and also read a bit more about one cycle policy and discuss the implementation. Thanks!

Hello @w4nderlust , I have taken a look at the train function in model.py. The learning rate for every batch/step is calculated in this line. An the actual value of the learning rate will be from here. I am presuming we have add code in the same file to calculate the learning rate.

@lordeddard yes it may look similar, assuming there is a one_cycle_policy parameter in the training configuration I imagine to have in the model.train() function something that looks like
if one_cycle_policy:
  current_learning_rate = one_cycle_policy(previous_learning_rate, curr_epoch, ... other parameters)


Deprecation warning

With 2.0, we committed to maintaining backwards compatibility for some amount of API stability. This is becoming more and more important as we expand our userbase. This patch adds a deprecated decorator, which emits a DeprecationWarning on any decorated object.
It's then used on Container.reload, which was replaced prior to 2.0.1, but not officially deprecated. All calls to reload have been replaced with fetch.

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

Merging [#116][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.48%

@@             master       #116   diff @@
==========================================
  Files             8          9     +1
  Lines           333        347    +14
  Methods           0          0
  Messages          0          0
  Branches         46         47     +1
==========================================
+ Hits            279        289    +10
- Misses           43         47     +4
  Partials         11         11

Powered by Codecov. Last updated by [de3ea62...22fc058][cc-compare]
[cc-base-branch]: https://codecov.io/gh/lxc/pylxd/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/lxc/pylxd/compare/de3ea62c0b44408c051acabe857b83b342af59e8...22fc058c727b24480ed0c1f237fb50ac2521b230
[cc-pull]: https://codecov.io/gh/lxc/pylxd/pull/116?src=pr

The codecov failure is a red herring. One call to reload was not covered previously, and so is not covered in this patch. Fixing it is beyond the scope of this patch, so while overall test coverage is increased, the patch's code coverage doesn't meet the current (very high) standard for code coverage.
I knew this would happen at some point. I'm happy that it happened sooner than expected.

lgtm


Some images get rotated for some reason

I'm using @sveltejs/enhanced-img, and for some reason, my optimized images get "flipped" while the original files are the correct side up.
+page.server.js:
/** @type {import('./$types').PageServerLoad} */
export function load() {
	return {
		images: Object.values(import.meta.glob('$lib/assets/gallerij/*.{jpeg,png}', { query: { enhanced: true }, import: 'default', eager: true })),
		images2: Object.values(import.meta.glob('$lib/assets/gallerij/*.{jpeg,png}', { import: 'default', eager: true })),
	};
}

+page.svelte

{#each data.images as image, i}

        <enhanced:img src={image} class="w-100 h-auto" />

{/each}
{#each data.images2 as image, i}

{/each}

This does however not happen with all images for some reason???
Example image which gets rotated:

See below: left image is optimized, right image is original

Can you confirm this also happens with imagetools directly? The enhanced query parameter is not defined by us but by the svelte package you are using, so unless this is reproducible with imagetools directly I would assume it's a fault on their side not ours.

Sorry for my late reponse, will take a look into this in a few weeks. Life's busy atm

I'm running into the same issue and the problem is reproducible with the vite-simple demo project in this repo. The picture ends up 90 degrees rotated, even though it looks alright in the OS image viewer and likewise when you simply open it in your browser.
I've attached an archive of the jpg that causes the issue so that the file doesn't get touched by any minifiers on Github's side: example.jpg.zip
The file is a jpg right out of a camera, and while I'm not too familiar with image formats, it seems to me the image data is kept unrotated and then a metadata is used to rotate it as it should be? If all metadata is stripped away, including orientation information, that would explain the issue.

@Etheryte are you saying that the issue reproduces using one of the existing images in vite-simple? If so, which one and what OS are you on? Or are you saying that if I extract the example.jpg.zip that you've provided and put it in the vite-simple project that it will reproduce the issue?

@benmccann The latter, that the example file I provided reproduces the issue when you put it in the vite-simple project.

I've made a minimal code demo that replicates the issue for me: https://github.com/Etheryte/imagetools-demo.

This seems to be a bug in Sharp itself, at least for my image, so I've gone ahead and filed an issue there: https://github.com/lovell/sharp/issues/4047

@Etheryte's issue on Sharp reveals the way to work around the problem, but it seems like we cannot take advantage of it via the exposed API in imagetools.
Summary (hopefully correct): Some images are rotated via metadata, and some are rotated via rearranging the pixels. Sharp drops metadata during resize and so any images rotated via metadata will appear in a surprising orientation after said resize.
There is a workaround for this problem, but I think that when we apply rotation via ?enhanced&rotate=0, either nothing happens or it is applied after the metadata is already stripped.

Yeah that's a good summary, and as far as Sharp's author is concerned it's working as intended. To fix the problem for Imagetools, the mentioned workaround would have to be integrated here, using the workaround from the currently exposed API does not work as far as I tried it.

Ah, okay. There is a way for imagetools users to work around this by changing the config to the following:
imagetools({
  removeMetadata: false,
})

Seems like there is potentially a way to use a custom directive to do thing the way you want, but I am not sure if it gives you control over the order of operations, which I think is the sticking point with the rotate() workaround. It'd be nice if there was a way to whitelist certain metadata and let all the other metadata be stripped. Then we might do
imagetools({
  removeMetadata: true,
  allowMetadataFields: ["orientation"],
})

Or something like that.
Either way, this does not help in the case of enhanced:img because we have no way to pass the config for imagetools into enhanced:img. Unless there is some way of configuring Vite that I am unaware of. So now I am wondering if the workaround does indeed need to bubble up to enhanced:img after all. @benmccann, what do you think?

I've also created a PR to address this issue: https://github.com/JonasKruckenberg/imagetools/pull/724
My initial take on it was naive, and I have force-pushed a new approach as of this morning.

Nice! Let me know if you need help

I am actually playing with proposing this even further upstream to Sharp. But still playing with an implementation there, as I’ve never worked with C++ before.
The PR I have open works, but uses a buffer back into Sharp to set initial orientation in cases where Sharp won’t allow the transformation to work logically (that I could find). If you’d like to use my fork for now, feel free! If the maintainer of Sharp doesn’t like my proposal, I’ll keep tinkering on this to see if I can get it to work without the buffer step in the middle.

I've finally gotten the code to work, and have opened an issue in Sharp proposing the new feature: https://github.com/lovell/sharp/issues/4144

Btw, do you have any idea when this will be merged? Iirc, I can not (easily) use the work around in @sveltejs/enhanced-img right? Or would this be very easy to do so

Not soon, but it is still on the map. In the meantime you could potentially employ the same trick I am using in my project to get it to work: apply small patches to both this library and to Sharp.
Have a look at my patches directory here: https://github.com/postplayhouse/postplayhouse.com/tree/f524f9549ee7085e5cc6bd0b5a81e2a62347b8b9/patches
You’ll want to either use the NPM patch package library, or the built in patching tools provided by prom or yarn2.

This gives an error for me tho:


The reading mode support swiping

I hope to support swiping from top to bottom in reading 。Turning pages does not feel as good as sliding pages

There is already an option to scroll vertically

There is already an option to scroll vertically

Can the black space be removed in vertical sliding mode?


xcbuild: fix interactive applications run by xcrun

Fixes #358795. Commands like nix develop .#hello -c /usr/bin/git log now works normally.
❯ nix develop .#hello -c /usr/bin/python3 -c 'import os; import sys; print(os.isatty(sys.stdout.fileno()))'
True

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

ping @NixOS/darwin-core


nav bar cancel button is not visible in light mode

Describe the bug

In light mode it is not visible

dark mode it is visible

Assign me to fix this issue

please assign me this issue

I would like to fix this issue, can you assign it to me @MonalikaPatnaik


CppWin32Sample.exe  Crash When Click "Download update"  button

OS
Windows 10
Programming Language
C++
VPK Version
VPK 0.0.942
Library Version
cargo 1.82.0
What happened?

The response is correct when clicking "check for updates", but the program crashes when clicking "download update"。
Relevant log output
No response

We have a new cross-platform C++ sample using wxWidgets, can you try that instead?


My code stopped working that worked with v1.0.3 previously

In my kanboard git hooks checkin.php script I have this code to validate that all issue IDs mentioned in the commit message exist and belong to the current project. With v1.0.3 of JsonRPC this code would display the error messages if there were any issues with the issues (pun).
	$batch = $client->batch();
	foreach ($issue_ids as $issue_id)
		$batch->getTask($issue_id);
	$issues = $batch->send();

	# Validate all issue ids.
	foreach ($issues as $batch_id => $issue) {
		if (!$issue)
			quit('Issue #' . $client->batch[$batch_id]['params'][0] . ' not found.', 1);
		if ($issue['project_id'] != $project_id)
			quit('Issue #' . $client->batch[$batch_id]['params'][0] . ' belongs to project #' . $issue['project_id'] . '.', 1);
	}
	echo "Validated issue ids.\n";

Updating to v1.2.1 leads to this error on one of the quit(...) lines:
Fatal error: Cannot access private property JsonRPC\Client::$batch in C:\project\apps\testrepo\.git\hooks\kanboard\checkin.php on line 152
Is it a newly introduced bug in JsonRPC, or did the API change somehow and I need to make some modifications to my validation code?

You should not access to the private property $client->batch because it's internal representation of the payload that will be send. You should have everything you need in $issues.

How do I extract the request params from $issues (for formatting an error message)?


Support residual by level or bin of feature?

Would be a nice enhancement to support residual plots and analysis by feature and by level or bin of feature. This just allows for spotting of specific bugs, like in the image below.

Prototype code here: https://nbviewer.org/github/jphall663/interpretable_machine_learning_with_python/blob/master/debugging_resid_analysis_redux.ipynb, cell 18.
For categorical features, residuals can be plotted across top-k levels, for numeric features they will have to binned before plotting.

Thank you for this valuable suggestion.
Currently, the residual plot is based on full samples. We will consider adding the option to visualize only a subset of samples to be customized by users.


Add documentation for time-mask

Add examples of implementation for time-mask

Coverage remained the same at 99.034% when pulling fade9adf35234ef8838509aae39a13770ce8d38b on williansmartins:patch-1 into 8951f3bce41c57b9c93057a79d9282ca7716f653 on assisrafael:master.

Sorry, is my first contribution, the coverage is not ok, what I need todo?

Thank you for the pull request.


feat: configurable kaniko based cloud builds

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] If it is a core feature, I have added thorough tests.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.55%. Comparing base (81aa0e7) to head (9ce7aec).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
+ Coverage   74.53%   74.55%   +0.02%
==========================================
  Files          57       57
  Lines        2835     2838       +3
==========================================
+ Hits         2113     2116       +3
  Misses        722      722

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


Fix A/B Testing HTTP URI match conditions

Copy the URI match conditions specified in service.match over canaryAnalysis.match.

Codecov Report

Merging #94 into master will decrease coverage by 0.02%.
The diff coverage is 62.5%.

@@            Coverage Diff            @@
##           master     #94      +/-   ##
=========================================
- Coverage   56.32%   56.3%   -0.03%
=========================================
  Files          13      13
  Lines        1683    1689       +6
=========================================
+ Hits          948     951       +3
- Misses        587     589       +2
- Partials      148     149       +1

Impacted Files
Coverage Δ

pkg/router/istio.go
86.63% <62.5%> (-0.98%)
:arrow_down:

Continue to review full report at Codecov.

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


Bulkrax Importer does not import csv data with file column

I'm trying to import csv data via bulkrax importer it imports successfully all work entries but when i add a file column in csv file it does not import any work or collection entry.  I have placed files folder relative to csv file but it neither imports data nor files. Can you please help me figure out what i am doing wrong?

Hi @naseer-phaedra. On the importer's show page (/importers/:id), are you seeing any errors on the importer (above "Total Works") or on any of the importer's work entries (in the table at the bottom of the page)?

Hi @bkiahstroud no its not showing errors
it just shows
Frequency: PT0S
Parser klass: Bulkrax::CsvParser
Limit:
Parser fields:
rights_statement:
override_rights_statement: 0
file_style: Specify a Path on the Server
import_file_path: spec/fixtures/csv/metadata.csv
Field mapping: Toggle
contributor: {"from"=>["contributor"]}
creator: {"from"=>["creator"], "split"=>true}
keyword: {"from"=>["keyword"], "split"=>true}
date_created: {"from"=>["date"]}
description: {"from"=>["description"]}
identifier: {"from"=>["identifier"]}
language: {"from"=>["language"], "parsed"=>true, "split"=>true}
publisher: {"from"=>["publisher"]}
related_url: {"from"=>["relation"]}
rights_statement: {"from"=>["rights"]}
license: {"from"=>["license"], "split"=>"|"}
source: {"from"=>["source"]}
subject: {"from"=>["subject"], "parsed"=>true}
title: {"from"=>["title"]}
resource_type: {"from"=>["type"], "parsed"=>true}
remote_files: {"from"=>["thumbnail_url"], "parsed"=>true}

Hi @bkiahstroud no its not showing errors
it just shows
Frequency: PT0S
Parser klass: Bulkrax::CsvParser
Limit:
Parser fields:
rights_statement:
override_rights_statement: 0
file_style: Specify a Path on the Server
import_file_path: spec/fixtures/csv/metadata.csv
Field mapping: Toggle
contributor: {"from"=>["contributor"]}
creator: {"from"=>["creator"], "split"=>true}
keyword: {"from"=>["keyword"], "split"=>true}
date_created: {"from"=>["date"]}
description: {"from"=>["description"]}
identifier: {"from"=>["identifier"]}
language: {"from"=>["language"], "parsed"=>true, "split"=>true}
publisher: {"from"=>["publisher"]}
related_url: {"from"=>["relation"]}
rights_statement: {"from"=>["rights"]}
license: {"from"=>["license"], "split"=>"|"}
source: {"from"=>["source"]}
subject: {"from"=>["subject"], "parsed"=>true}
title: {"from"=>["title"]}
resource_type: {"from"=>["type"], "parsed"=>true}
remote_files: {"from"=>["thumbnail_url"], "parsed"=>true}

@bkiahstroud when i remove file column from csv file and set file path in importer it imports all works and collections without any error.

I have placed files folder relative to csv file

@naseer-phaedra Does this ^ mean you have already put the files you are intending to import on the server, similarly to how the documentation describes?

@bkiahstroud  yes. Actually i have setup hyku project with docker and this gem is being used for imports and exports via admin dashboard and i have followed the given docs for bulkrax imports.

@bkiahstroud i have placed the files and csv file like this in my project.
spec/fixtures/csv/data.csv
spec/fixtures/csv/files/sample.pdf

@naseer-phaedra Is the file column in your CSV named remote_files? If so, can you try renaming it to just files and trying the import again?
remote_files is used for importing files from the web, so that column would hold urls instead of filenames.

@bkiahstroud no the column name is file. but i have also tried with column name files which imports work entries with status failed and gives Runtime error "RSolr::Error::Http - 400 Bad Request"

@naseer-phaedra Unless you've added a custom field mapping, files should be the proper column name.
It sounds like Solr might not be running properly in your Hyku application. I recommend asking about the Solr 400 error in the #hyku channel of the official Samvera Community's Slack; the community knows far more about Solr than I do.
In the meantime:

Are you using dory?
In your docker-compose.yml file, does your solr container have a virtual host? (solr.environment.VIRTUAL_HOST)
If the answers to 1 and 2 above are yes, are you able to navigate to the virtual host in your browser? (http://<VIRTUAL_HOST>/solr)

closing this out as there's been no response in over 18 months.


Added a paginator using KnpPaginatorBundle

Please review and let me know

Please, add more fixtures for posts. Now it's only 10.

Actually the $limit argument in the queries isn't necessary, what should I do remove it or not?

Ok I got something like this:
@Route("/", name="blog_index", defaults={"page" = 1})
@Route("/page/{page}", name="page_blog_index", defaults={"page" = 1}, requirements={"page" : "\d+"})
The default for page in both routes is because I don't want to catch the page as a query string but as a  route parameter.

What about blog_index_paged name to be consistent with first route name?
Nonetheless, I don't like to use 2 routes for this action :(

@bocharsky-bw if we don't want to define two different routes, then we need to pass the page number as a query string. @ccab I agree with @bocharsky-bw about the new route name: I personally use and I've seen a lot of times the practice of naming it as *_paginated, so it could be blog_index_paginated. (The other option would be to not name it at all, because we are not going to generate this URL in any template).

What about next route definition:
/**
 *@Route("/{page_prefix}{page}", name="blog_index", defaults={"page" = 1, "page_prefix" = "page-"}, requirements={"page" : "\d+", "page_prefix" = "page-"})
 */

Does it too confused?

@bocharsky-bw yes, it looks very confusing to me.

yea @bocharsky-bw I turn around something like that but we need to think in code readability too, for newcomers it could be a little confusing. @javiereguiluz I'll remove the name for the second @Route

A clue about Travis-CI ??

Try to check this suggestion.

Tanks @bocharsky-bw, as you may see in this commit this should be solved but I am using the 2.4.2 release and I still got it, so I'm following directions from UPGRADE-3.0 and I'll enable php as templating engine

There is something weird with deps="low" env variable

I make all the restrictions in the packages versions in order to pass all test with the composer update --prefer-lowest, but I must confess I'am not happy about the end result, so what do you think if we change to Pagerfanta

@javiereguiluz this is a project with locked dependencies. Why are we running builds with different deps than the locked ones ?

@stof I'm not sure I understand you. Please tell me what should I change or add. Thanks!

@javiereguiluz why are we running composer update in some Travis jobs ? Dependencies are locked in the project, so the CI server should only run composer install

I think composer update needs for legacy PHP compatibility, doesn't it?

Actually this is the main reason for my last 4 commits, because I need to pass composer update with the latest versions :-) & composer update --prefer-lowest with the lowest versions :-(

Well, you could try to constraint lowest version knplabs/knp-paginator-bundle": "~2.4,>=2.4.2"

Yea @bocharsky-bw it could solve the problem now, but in the future we will need to add some other bundles and we could have the same problem again, always watching lowest versions for bugs, I'm with @stof on this, locked dependencies is the answer

What if we change the basic template for another one bootstrap oriented, this can be easily done changing a config key
Default:

Now with Bootstrap:

Bootstrap oriented looks better :+1:

:+1: Yeah, bootstrap oriented is totaly better. Now it seems like the Mathias Noback blog http://php-and-symfony.matthiasnoback.nl :)

@javiereguiluz what do you think, it's finished ?

@ccab sorry for the delay. Your feature is now finally merged (I made some cosmetic changes in 97e8348). Thanks a lot for working on this :)


change event in not existing element

Is this wrong or feature?
valueChanged is two times called.
The second time with value == null.
But when value == null,  the "test-element2" should not existent.
if="[[value]]"
Manfred Sattler
<dom-module id="test-element2">
    <script>
      Polymer( {
        is: "test-element2",
        properties: {
          value: {
            observer: "valueChanged",
            type: String
          }
        },
        valueChanged: function() {
          console.log( "valueChanged:", this.value );
        }
      } );
    </script>
  </dom-module>

  <dom-module id="test-element">
    <template>
      <template is="dom-if" if="[[value]]" restamp>
        <test-element2 value="[[value]]"></test-element2>
      </template>
    </template>
    <script>
      Polymer( {
        is: 'test-element',
        attached: function() {
          this.value = "Hello";
          this.async( function() { this.value = null; }, 200 );
        }
      } );

    </script>
  </dom-module>

  <test-element>

In the Polymer 0.5 world, this might have worked as you expected. However, there are two or three things at work here:

Bindings are managed by the host element, not the template or an external system. In this case, your test-element has a list of "property effects" that occur when value changes. One of these sets the if property on the template, and one sets the value property on test-element2. I'm not sure whether there is a defined order here, or whether this is racy. (@sorvell or @kevinpschaaf can perhaps clarify?)

The template stamping isn't necessarily synchronous -- the template fires a dom-change event once it's performed all of the DOM manipulations. (https://www.polymer-project.org/1.0/docs/devguide/templates.html#dom-change) So it should all be done by the time the dom-change event handler is called, right? Except ...

Shady DOM tree changes are performed asynchronously. (https://www.polymer-project.org/1.0/docs/devguide/local-dom.html#dom-api) This appears to be the actual culprit here. Check out the following jsbin:

http://jsbin.com/vohata/edit?html,console,output
As you'll note, the dom-change event gets fired (so the dom-if is getting notified first), then the value change event on the child element. Checking the tree at this time confirms that the child is still there. At this point, the dom-if has removed the child, but the remove operation is still pending.
If you call Polymer.dom.flush() at this point, a subsequent check shows the child is not there.
 However, I suspect this is an antipattern, and you'd be better off having the parent ignore the extra event.

Is this so desired or an implementation detail that is accepted, because there is no alternative?
What does restamp on the outermost dom-if template?
I thought it makes all things new.
This is, what i want in this case.

The if-condition of the outermost template set to false
and then async after a time the actual values.
in relation to my example:
test-element1 is a form and
test-element2 are maybe many fields.
In case the form changed , all fields change too.
i have this demo (http://web.simpl4.org#!accounts)  ported from 0.5(here works fine)
And with the workaround also in 1.0

Duplicate of #1716


update version

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[x] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #13931 into main will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #13931   +/-   ##
=======================================
  Coverage   59.84%   59.84%
=======================================
  Files         693      693
  Lines       38345    38345
  Branches     5514     5514
=======================================
  Hits        22949    22949
  Misses      14213    14213
  Partials     1183     1183

Continue to review full report at Codecov.

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


status

is this project dead? README.md states that a new version will be ready in "a few days"

No it isn't. I'm currently waiting on a contract for a more stable delivery of future updates and rootfs. If the sponsor I'm negotiating right now comes through, a new version will be released.
Until then, the current version should work as it always did.
I know it's taken a long time, and the holidays added even more delay, but stay tuned.

Word of note; I'd avoid saying things as you had here in the future @coderobe. Asking if a project is dead and reminding/asking for deadlines on an open source project (especially a personal one as this) is in bad taste, and will get you banned from many a community I could name. There are ways of asking about the status of a project that might be better suited in the future, be sure to look out for them

@crutchcorn Please do not speak for this project. If I would have had any issue with the way they asked for the project status, I would have said so.
In fact, your comment is even less constructive, because you just criticize them and don't actually comment on what you think would be a more suitable way to ask.
Also, this is a very common way to ask for "dead" projects. A simple search for such issues gives more than 46k results.

@crutchcorn i wasn't asking for any deadline, just wanted some info on the current status because there were no commits in a while & the readme stated there should be some by now. i'm not trying to push for a release or anything - it just seemed like development kinda stalled.
@turbo thanks for the info, let's hope for the best 👍

I'm currently waiting on a contract for a more stable delivery of future updates and rootfs. If the sponsor I'm negotiating right now comes through, a new version will be released.

I'm interested on this topic from a personal point of view, I always has problems finding sponsors for my own projects (I don't have the marketing and speech skills for that...). Congrats dude :-)

@piranna Generally goes like this:

Find companies that sponsor FOSS development
Asses your needs
Collect objective usage data on what resources you need and how much that'll grow in the future
Write in and be patient
Expect to be rejected and thank them anyway if it happens


Add Travis hook

As we are now Travis-ready, I think it would be nice to activate the setting in Github. I thought I could, but it appears even my Collaborator badge doesn't allow me to touch the settings. On to you, @yabawock!

@joallard Webhook for Travis has been enabled


Change architecture to use charting model until end of pipeline and then draw

We should change the architecture of Chartist so that it works with a model of chart elements and wait with drawing until the last step in the pipeline. This would give us several benefits:

Performance
More flexible for different rendering adapters (server side, image export etc.)
Easier to reason about
Solve some chicken / egg problems (plugins can modify the model before the chart has been drawn)

wondering what the status of this issue is; am mostly interested in using chartist in node as per #780, which is closed but perhaps not implemented yet?

Wanted to use this nice lib as well in an electron app and will have to finde another one because of this blocker. 😢


Fix update loop timeout

This PR fixes a wrong fix of the update loop introduced in #1429.

Thank you!
bors merge


Parsing user prompt

Outcome
The model should be able to take a natural language prompt from user and parse that into a dictionary where keys map to Neurobagel query model fields
Examples
prompt: How many female subjects older than 50 with a Parkinson’s diagnosis?
{sex: female, min_age: 50, diagnosis: Parkinsons}

prompt: How many male healthy control subjects?
{sex: male, healthy_control: True}

prompt: How many subjects between 20 and 80 yrs old who have at least 3 phenotypic sessions and 2 imaging sessions?
{min_age: 20, max_age: 80, num_matching_phenotypic_sessions: 3, num_matching_imaging_sessions: 2}

What should happen if a prompt includes fields we don’t support (yet) like race and socio-economic status?
Perhaps an informative message saying “Neurobagel currently doesn’t support the search for race and socioeconomic status” e.g.,
User: How many subjects of x race between the ages of 20 and 50
Model: Unfortunately, Neurobagel currently doesn’t support a search for race, would you like me to search for subjects between the ages of 20 and 50 instead?
Resources
Neurobagel query model fields
Tasks

[x] #14

@Raya679 Feel free to close this milestone since it has been addressed 🎉


Invalid layer in Procreate file

os: MacOS 10.15.2
version: 1.9.12
When opening the procreate psd file I linked in this issue, I get the following crash.
global layer mask info is broken, expected 13 bytes but found only 1
Invalid signature (b'\x00\x00\x00\x01')

Traceback (most recent call last):
  File "./scripts/export.py", line 45, in <module>
    main(base_path)
  File "./scripts/export.py", line 11, in main
    generate_textures(base_path, Path("textures"))
  File "./scripts/export.py", line 24, in generate_textures
    generate_textures(base_path, directory)
  File "./scripts/export.py", line 28, in generate_textures
    psd = PSDImage.open(str(f))
  File "/usr/local/lib/python3.7/site-packages/psd_tools/api/psd_image.py", line 104, in open
    self = cls(PSD.read(f, **kwargs))
  File "/usr/local/lib/python3.7/site-packages/psd_tools/api/psd_image.py", line 43, in __init__
    self._init()
  File "/usr/local/lib/python3.7/site-packages/psd_tools/api/psd_image.py", line 595, in _init
    current_group._layers.append(layer)
UnboundLocalError: local variable 'layer' referenced before assignment

cactus_top.psd.zip

@afrigon Thanks for reporting this, will investigate


Implement Jaeger Remote Sampler as defined by the specification

We currently don't have an implementation of the JaegerRemoteSampler as defined in the specification.
To complete this issue:

[ ] create a new experimental package @opentelemetry/sampler-jaeger-remote (this can be done in a separate PR to keep the actual implementation PR smaller)
[ ] implement the sampler based on the specification
[ ] add tests

@pichlermarc I would like to pick this up.

Thanks for picking this up. It's yours @legalimpurity :slightly_smiling_face:


Create Deploying wallet instead of FutureWallet if refund is paid

Summary
A description of the changes proposed in the pull request.
Fixes #number
Checklist

[ ] Change is small and easy to review (please split big changes into multiple PRs)
[ ] Change is consistent with architecture
[ ] Change is consistent with test architecture
[ ] Change is consistent with naming conventions
[ ] New code is covered with tests
[ ] Tests related to old code are updated
[ ] Documentation is up to date with changes

rename createFutureOrDeployingWallet to createWallet


⚠ AsianCinema has degraded performance

In 00a7728, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 6701 ms

Resolved: AsianCinema performance has improved in 787ab3a.


demInstance parameter for DEM collections.

This PR implements dem_instance parameter for DEM collections.

I'm wondering if it makes sense to add predefined enums DataCollection.DEM_COPERNICUS_30 and DataCollection.DEM_COPERNICUS_90 so that users don't have to do that?

Was wondering that myself. In eo-learn DemTask I just construct a collection based on dem_instance (from existing DataCollection.DEM), but perhaps it would be better to have them specified.
I vote yes. You?

I vote yes. You?

Yes, I'm ok with adding them.


6.0 Feedback:  Protocols > Protocol Detail overflow menu persists after no selection

Overview
When I open the overflow menu on a protocol detail it does not disappear when I click off of it.  The only way it collapses without selecting an item in the menu is to click the overflow menu kebab again.
Current Behavior
https://user-images.githubusercontent.com/502770/165359299-cf1e30aa-8570-4042-9a47-a9c9ff5c0fb5.mp4
Expected Behavior
Clicking off of the the menu or clicking the kebab would close the menu.
Steps To Reproduce
See the video.
Operating system
Mac
Robot setup or anything else?
No response

Overflow menus in the devices tab disappear when you click off of them - we should have the same behavior for the protocol detail overflow menu

verified


Do not load all vectors into memory while indexing

Avoid loading all vectors in a Map within IndexVectors for unnecessary memory consumption, especially with large vector models.

hey @tteofili I think your locale settings changed all . to , in the regression tests... can you update your PR to not do this?

hey @tteofili I think your locale settings changed all . to , in the regression tests... can you update your PR to not do this?

very weird, sure I'll fix it.

very weird, sure I'll fix it.

There doesn't seem to be any progress on this, closing for now.


Update Contractions.yml

Remove "how's" and "when's" as an acceptable contraction; neither of those is Google style

Hi. Hope this edit is ok.


Automate Housekeeping after template initialization

Description
As a developer using git I want scripts that I can run to automate removal of sample code such as the highlighted code.
This a react from running
npm create vite@latest ... --template react

Suggested solution
A simple node script that can be run using npx
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This is out of scope. We keep create-vite as simple as possible. I guess you can create a script that removes some code and use that.


Diff decorations not shown if filename contains "+" character

Type: Bug
Diff decorations not shown in files that contain + character in its filename.
 Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Putting a filter like displacementFilter on a group with many sub groups and sprites renders it white/transparent

Putting a filter like displacementFilter on a group with many sub groups and sprites renders it white/transparent
var displacementTexture = PIXI.Texture.fromImage(require("assets/images/displacementMAP.jpg"));
displacementFilter = new PIXI.DisplacementFilter(displacementTexture);
rootGroup.filters = [displacementFilter]

I've tried several filters and none works.
I've also put on phaser, but one of you can close this issue. https://github.com/GoodBoyDigital/pixi.js/issues/1356
Phaser 2.2.2; PIXI: 2.2.0; webgl

I'm going to close this issue as it's a Pixi level bug, and I'm not sure it will ever be fixed in Pixi v2. If that changes we'll integrate the updates to Phaser. Equally we may adopt Pixi v3 in the future once it's had a good amount of time to settle in.


add polars feature to instructions

Without this, python tests fail at the collection phase
If we wanted to be sure these didn't fall out of sync again, I could imagine:

pulling the cargo build in the RST out into an include.
treating that include as a shell script in CI, so docs and CI would reflect the same features.
... but probably overkill?

Change of direction: We want to use --all-features pretty much everywhere and explain the exceptions. But that is failing right now on sample_discrete_laplace_Z2k

See #1525

Closing in favor of that issue.


Add p-responsive support

I'm trying to use the p-responsive class but can't currently do so for a couple different errors:

If I try classes: "p-responsive", I get:

If I try p: :responsive, I get:

I think p: :responsive should work since it's a Primer padding class. However, it would also be cool if the classes: validation was explicit, checking only classes that are supported by other Primer attributes like p:.
I'm using version 0.0.33 of the primer_view_components gem.

As a workaround, @manuelpuyol also suggested the following to me:

to unblock you, from the docs, p-responsive == .px-3 .px-sm-6 .px-lg-3 so you should be able to use px: [3, 6, 3]


moved all triton tests into runtime triton folder

This PR is the first in a series of refactors. It isolates all tests involving triton into one sub folder in the unit tests directory.

rerun tests

rerun tests


Internal STRA

Describe the task
Document risks, propose solutions or highlight for further consideration
Acceptance Criteria

[ ] Internal TRA
[ ] Gbola to review

Additional context

TRA doc in Confluence: FAM Threat and Risk Assessment

See https://apps.nrs.gov.bc.ca/int/confluence/display/FSAST1/FAM+Threat+and+Risk+Assessment.


Please invite me to the GitHub Community Organization

Name
Md. Musfiqur Rahaman
Discord Username(if exists)
MusfiqDehan#9994
Additional Context
I am an Open Source Enthusiast and always try to contribute to different open-source projects. Recently, I completed Hacktoberfest-2022 also. I would love to be a part of this Open Source Community.

Hi.
I was an individual contributor but now I wish to become an Open Source enthusiast
I would love to be a part of this Open Source Community.


Setting Vundle key mappings to be silent

I set already existing Vundle key mappings to have the <silent> attribute so that the commands are not echoed onto the command line. This silences commands such as :silent bd! and :exec 'Install'.getline('.') (which can be confusing for users to see echoed to the screen).
This should not change actual functionality or performance (so it should be a fairly safe merge), but it should improve user experience.

@nfischer thanks, i'll take a look ASAP.


Optimize the memory used by the map-maker

Currently destriped_map (in src/mapmaker.jl) creates a full TOD containing the baselines and subtracts it from the TOD before producing a binned map. For large TODs (e.g., 1 TB), this means that the code creates a baseline TOD of the same size, thus doubling the amount of memory used by the code. It would be wise to rewrite the code so that the computation is done algorithmically, otherwise the code will not be able to run on the clusters available to LSPE.

Commit 0213409 removes applyz and simplifies the calculation in applyz_and_sum by avoiding the creation of noise_tod. When destripe is ran on a TOD of 66 MB, the memory occupation and computation time are halved:
# Before commit 0213409
BenchmarkTools.Trial:
  memory estimate:  353.93 MiB
  allocs estimate:  5895
  --------------
  minimum time:     378.260 ms (37.15% GC)
  median time:      424.896 ms (39.80% GC)
  mean time:        420.604 ms (39.58% GC)
  maximum time:     443.885 ms (41.37% GC)
  --------------
  samples:          12
  evals/sample:     1

# After commit 0213409
BenchmarkTools.Trial:
  memory estimate:  155.95 MiB
  allocs estimate:  127
  --------------
  minimum time:     157.987 ms (1.93% GC)
  median time:      179.049 ms (10.73% GC)
  mean time:        190.832 ms (16.53% GC)
  maximum time:     282.489 ms (45.54% GC)
  --------------
  samples:          27
  evals/sample:     1

Next step is the removal of baseline2tod altogether.


Create an OwnerReference based watch on the unpack job

In the controller's Reconcile function, the existing logic creates the unpack job and obtains the pods for the job and if there is an error during the parsing of unpack logs to obtain the pods, then we check if the error is due to pod not being present and if it is, then we requeue after 10s. This is essentially polling for every 10s even when no events come in, and it also leads to ignoring any events that come between 0-10s.
Ideally, we should reconcile when the state of the job changes instead of polling. Therefore, the better solution is to put an OwnerReference on the job (which we already do) and create an OwnerReference based watch on the job such that we reconcile when state of the job changes. We may also need to do a similar thing for the underlying pod itself.
Also, instead of doing a RequeueAfter for every 10s, we should just return the error as that would trigger exponential backoff.
See: https://github.com/operator-framework/catalogd/pull/34#discussion_r1164565022

FWIW, this is covered in the WIP spec.source PR. That PR inherits rukpak's unpacking mechanics which already work as described in this issue.


Hoist Operation from closure to class variable

This makes connexion.Api more extensible by inheritance.

Coverage remained the same at 100.0% when pulling be2df5bd5c50687f23c3252be7bb7d31b340942e on kislyuk:patch-1 into 78e962f5f46e0e937c704c13b8f9cf9bdac5d961 on zalando:master.

@kislyuk if this should become part of the Connexion API (and people rely on it), could you add a test for this (i.e. overwriting operation_cls)? Thanks!

@kislyuk are you still interested in getting this PR merged?

Yes, but I have no bandwidth to add the tests.

Closing since this has been refactored and exposed in #726.


build.sh does not produce a dist/tns-ios-*.tgz file

Instead, it produces a dist/TKLiveSync.framework file.
And when I try:
tns platform add ios --framework-path /Users/roblav96/Downloads/ios-runtime/dist/TKLiveSync.framework

I get this error:
Failed to retrieve data from npm. Please try again a little bit later.. Error: Error: EISDIR: illegal operation on a directory, read

makes me think the TKLiveSync.framework is not the tns-ios-*.tgz I'm looking for.
I also tried adding --symlink and I get this error:
The project file is corrupted. Additional technical information: SyntaxError: Unexpected token } in JSON at position 237

Git cloned off master branch.

The build script should also produce NativeScript.framework and tns-ios package. Can you copy the last few lines of the build.log file in the root of the repo? That would give more clues to what's happening.

Here's the terminal:

ios-runtime/cmake-build

build.log
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/roblav96/Downloads/ios-runtime/cmake-build
Build settings from command line:

What version of CMake are you using? We have some issues with newer ones.
Also try installing xcpretty and running the build again. I remember this fixing such issues on some terminals.

cmake version 3.6.1

It looks like I'm getting closer, but the directories look the same as posted above with no .tgz or .framework files.

CMake 3.6 has some incompatibilities with our JavaScriptCore build. Can you try with the one listed in the README (cmake --version should be 3.1.3).

15 minutes later, and 40% battery gone, still building...
I'm gonna say it's working. 🍰  😄
Thank you!!!


Switch to MyST-Parser

Pull Request Test Coverage Report for Build 569385771

15 of 15 (100.0%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.3%) to 97.351%

Files with Coverage Reduction
New Missed Lines
%

ocds_babel/markdown_translator.py
1
97.26%

Totals

Change from base Build 383856154:
1.3%

Covered Lines:
294

Relevant Lines:
302

💛  - Coveralls


where i can get the SI?

It's a great job!
as you said in the paper, there are some details and information in the supplementary material, but the paper i download without the material, where i can get the them?

Hi, thanks. You can find it in my website https://www.cse.wustl.edu/~muhan/papers/AAAI_2018_DGCNN_appendix.pdf

Thanks for your reply.


error when using pnp in strict mode due to undeclared dependencies

for example:
Failed to load module 'typescript-eslint-language-service' from /home/adrian/code/test-project/node_modules: Error: typescript-eslint-language-service tried to access @typescript-eslint/scope-manager, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

This affects users of yarn 3 (berry) and https://pnpm.io when using pnp.
Here's a patch which I had to put in my .yarnrc.yml to make silence the errors, which shows the deps accessed by this module in an unsound manner:
packageExtensions:
  typescript-eslint-language-service@*:
    dependencies:
      "@eslint/eslintrc": ^1.3.3
      "@typescript-eslint/scope-manager": "*"
      "@typescript-eslint/types": "*"
      "@typescript-eslint/typescript-estree": "5.41.0"
      "@typescript-eslint/visitor-keys": "*"
      globby: ^11.1.0
      is-glob: ^4.0.3

Note that I had to pin @typescript-eslint/typescript-estree to 5.41.0 since 5.42.0 introduced a breaking change which breaks typescript-eslint-language-service

@Quramy This plugin is AWESOME! do you plan to update it to support the mentioned above breaking change in dependency 5.42.0? or wait to v6 because of this comment?
https://github.com/typescript-eslint/typescript-eslint/pull/5892#discussion_r1012924718

Any plan to have this fixed? Alternatively would you accept a PR?

Alternatively would you accept a PR?

👍

@Quramy While the e2e tests are all passing, I am not able to see the reported errors in VSCode, while the linter is able to find them.
These are the last valid dependencies knwoen where errors are reported:
    "@typescript-eslint/parser": "5.41.0",
    "eslint": "^8.37.0",
    "typescript": "^4.9.5",
    "typescript-eslint-language-service": "5.0.0"

Changing typescript to v5 or typescript-eslint-language-service to other greater than 5.0.0 does not show the errors.
For example, with these newer dependencies:
    "typescript": "^5.0.3",
    "typescript-eslint-language-service": "^5.0.5"

Here a small repo: https://github.com/artola/typescript-eslint-language-service-issue.git
While running the script yarn lint reports:
[some-folder] % yarn lint

/some-folder/src/main.ts
  1:1   error  Unexpected 'debugger' statement      no-debugger
  2:12  error  There should be no space after '{'   object-curly-spacing
  2:22  error  There should be no space before '}'  object-curly-spacing
  2:26  error  Missing semicolon                    semi

✖ 4 problems (4 errors, 0 warnings)
  3 errors and 0 warnings potentially fixable with the `--fix` option.

@JoshuaKGoldberg could you please give us some hint about what could be the breaking change?

I don't use typescript-eslint-language-service so no, I don't know what's going on. 😄
If you think it's an issue with @typescript-eslint/parser then I'd encourage you to file an issue on typescript-eslint over at https://github.com/typescript-eslint/typescript-eslint/issues/new/choose.

@Quramy If I update the dependencies to the latest:
  "devDependencies": {
    "@typescript-eslint/parser": "^5.57.1",
    "eslint": "^8.37.0",
    "typescript": "^5.0.3",
    "typescript-eslint-language-service": "^5.0.5"
  },

I can see in the VSCode's tsserver.log (command: Typescript: Open TS Server log) the following log for the main.ts file:
Info 685  [19:58:43.239] [typescript-eslint-language-service] Cannot read file '/tsconfig.json'.
Info 686  [19:58:43.239] [typescript-eslint-language-service] Error: Cannot read file '/tsconfig.json'.
    at Object.diagnosticReporter [as onUnRecoverableConfigFileDiagnostic] (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/create-program/getWatchProgramsForProjects.js:99:11)
    at getParsedCommandLineOfConfigFile (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/typescript.js:35635:12)
    at parseConfigFile2 (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/typescript.js:122928:34)
    at Object.createWatchProgram (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/typescript.js:122534:7)
    at createWatchProgram (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/create-program/getWatchProgramsForProjects.js:286:22)
    at getWatchProgramsForProjects (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/create-program/getWatchProgramsForProjects.js:181:30)
    at createProjectProgram (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/create-program/createProjectProgram.js:54:95)
    at getProgramAndAST (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/parser.js:36:89)
    at parseAndGenerateServices (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/parser.js:140:11)
    at parseForESLint (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/parser/dist/parser.js:90:80)
    at ESLintAdapter.convertToESLintSourceCode (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/eslint-adapter.js:90:90)
    at ESLintAdapter.getESLintResult (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/eslint-adapter.js:117:33)
    at ESLintAdapter.getSemanticDiagnostics (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/eslint-adapter.js:138:39)
    at TemplateLanguageServiceProxy.adaptDiagnosticsCall (/Users/martin/.vscode/extensions/styled-components.vscode-styled-components-1.7.8/node_modules/typescript-template-language-service-decorator/lib/template-language-service-decorator.js:288:33)
    at Proxy.<anonymous> (/Users/martin/.vscode/extensions/styled-components.vscode-styled-components-1.7.8/node_modules/typescript-template-language-service-decorator/lib/template-language-service-decorator.js:54:25)
    at IpcIOSession.semanticCheck (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:178516:125)
    at /[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:178577:14
    at MultistepOperation.executeAction (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:177242:9)
    at /[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:177222:69
    at IpcIOSession.executeWithRequestId (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:180226:14)
    at Object.executeWithRequestId (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:178262:57)
    at Immediate._onImmediate (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:177222:26)
    at process.processImmediate (node:internal/timers:466:21)

Then it is clear that the problem is about finding /tsconfig.json. A bit of debugging and research leads me to: https://typescript-eslint.io/linting/typed-linting/
And "voilà" the solution is to tweak the .eslintrc.js in any of the following ways:

  parserOptions: {
    project: "./tsconfig.json",
    tsconfigRootDir: __dirname,
  },

  parserOptions: {
    project: true,
  },

Now errors are properly display on VSCode. I hope this help other users.
Thanks @Quramy for your support.
CC @awydler

@artola would you be able to explain how what you wrote relates to the issue I have reported? The issue is about dependencies imported by this module not being listed in it’s package.json

@adrian-gierakowski Since version 5.0.1 it is not used anymore @typescript-eslint/typescript-estree in this plugin, it is pulled as a transitive dependency by @typescript-eslint/parser but not accessed elsewhere. Then, the issues does not persist when moving forward.
[typescript-eslint-language-service-issue] % yarn why @typescript-eslint/typescript-estree
└─ @typescript-eslint/parser@npm:5.57.1 [c5205]
   └─ @typescript-eslint/typescript-estree@npm:5.57.1 [0a26c] (via npm:5.57.1 [0a26c])

There is another correlation between the dependencies and my findings. The problem is that as you mention, the semver as applied in version 5.0.0 does not guarantee matching dependencies, and such dependencies leads to some incompatibility  at 5.42.0. IMO what it is wrong in this issue is that @typescript-eslint/typescript-estree introduces a breaking change in 5.42.0 without changing the version to 6.x.
Now, if we move all the dependencies to the latests versions, everything works. At that point, pinning dependencies in the plugin is not needed, could be improved with a simple comment/warning in the README.md.

@artola thats for taking the time to explain! I’ll try it out on my project with yarn berry once I’m back to work next week and report back if it works for me.

IMO what it is wrong in this issue is that @typescript-eslint/typescript-estree introduces a breaking change in 5.42.0 without changing the version to 6.x.

breaking change

https://hynek.me/articles/semver-will-not-save-you
https://www.hyrumslaw.com
The change in typescript-eslint (https://github.com/typescript-eslint/typescript-eslint/pull/5834) was an internal refactor that didn't change the public API. There's no reason why it would have been labeled as a breaking change. It's in accurate to call it a breaking change. Please don't throw that shade at us. 😉
If typescript-eslint has a bug, then that's another story. Please file an issue on us.
But: what is the actual behavior change in @typescript-eslint/typescript-estree that you believe impacts this package? The OP links to an internal function -ensureAbsolutePath`- not exported to consumers.

I tried cloning the repro repo, bumping @typescript-eslint/parser to 5.42, and restarting ESLint & TS servers but don't see any issues. What are the explicit steps to reproduce this issue?

I tried cloning the repro repo, bumping @typescript-eslint/parser to 5.42, and restarting ESLint & TS servers but don't see any issues. What are the explicit steps to reproduce this issue?

@JoshuaKGoldberg Just set these dependencies and errors are not reported anymore (just change "@typescript-eslint/parser": "5.42.0",):
 "devDependencies": {
    "@typescript-eslint/parser": "5.42.0",
    "eslint": "^8.37.0",
    "typescript": "^4.9.5",
    "typescript-eslint-language-service": "5.0.0"
  },

Here the tsserver.log after reloading VSCode:
Info 99   [17:01:32.611] event:
    {"seq":0,"type":"event","event":"syntaxDiag","body":{"file":"/[redacted]/typescript-eslint-language-service-issue/src/main.ts","diagnostics":[]}}
Info 100  [17:01:32.611] [typescript-eslint-language-service] The "path" argument must be of type string. Received an instance of Object
Info 101  [17:01:32.611] [typescript-eslint-language-service] TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received an instance of Object
    at new NodeError (node:internal/errors:371:5)
    at validateString (node:internal/validators:120:11)
    at Object.join (node:path:1172:7)
    at ensureAbsolutePath (/[redacted]/typescript-eslint-language-service-issue/node_modules/@typescript-eslint/typescript-estree/dist/create-program/shared.js:71:26)
    at applyParserOptionsToExtra (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/ast-converter.js:156:54)
    at AstConverter.parseAndGenerateServices (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/ast-converter.js:215:21)
    at AstConverter.parseForESLint (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/ast-converter.js:280:44)
    at AstConverter.convertToESLintSourceCode (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/ast-converter.js:296:67)
    at ESLintAdapter.getESLintResult (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/eslint-adapter.js:95:43)
    at ESLintAdapter.getSemanticDiagnostics (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript-eslint-language-service/lib/eslint-adapter.js:116:39)
    at Session.semanticCheck (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:180735:52)
    at /[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:180801:31
    at MultistepOperation.executeAction (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:179660:25)
    at /[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:179639:100
    at Session.executeWithRequestId (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:182324:28)
    at Object.executeWithRequestId (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:180499:87)
    at Immediate._onImmediate (/[redacted]/typescript-eslint-language-service-issue/node_modules/typescript/lib/tsserver.js:179639:41)
    at process.processImmediate (node:internal/timers:466:21)

About "breaking change" or not, it’s not you it’s me. 😉  Something breaks in this ecosystem at that point. Anyway, moving all the dependencies forward and changing the .eslintrc.js as mentioned above everything works.
Then, the problem does not persist with the mentioned solution, and why this was affecting VSCode but not other solution as @Quramy  mentioned above.


Unhandled Exception: Converting object to an encodable object failed: Instance of 'SendMessage'

Error causing function:
sendMessage() async {
    TdApi.FormattedText messageText = TdApi.FormattedText(text: messageGetter.text, entities: []);

    TdApi.MessageSendOptions messageSendOptions = TdApi.MessageSendOptions(disableNotification: false) ;

    TdApi.InputMessageText message = TdApi.InputMessageText(text: messageText);

    final result = await context.read<TelegramService>().send(
      TdApi.SendMessage(chatId: contact_id, inputMessageContent: message, options: messageSendOptions),
    );

    print(result);
  }

The stacktrace returned:
E/flutter (13086): #0      _JsonStringifier.writeObject (dart:convert/json.dart:688:7)
E/flutter (13086): #1      _JsonStringStringifier.printOn (dart:convert/json.dart:877:17)
E/flutter (13086): #2      _JsonStringStringifier.stringify (dart:convert/json.dart:862:5)
E/flutter (13086): #3      JsonEncoder.convert (dart:convert/json.dart:262:30)
E/flutter (13086): #4      JsonCodec.encode (dart:convert/json.dart:172:45)
E/flutter (13086): #5      TdClient.clientSend (package:tdlib/src/tdclient/tdclient.dart:32:63)
E/flutter (13086): #6      TelegramService.send (package:fima/services/telegram_service.dart:195:22)
E/flutter (13086): #7      _ChatScreenState.sendMessage (package:fima/screens/main/chat.dart:210:58)
E/flutter (13086): #8      _ChatScreenState.build.<anonymous closure>.<anonymous closure> (package:fima/screens/main/chat.dart:94:50)
E/flutter (13086): #9      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:989:21)
E/flutter (13086): #10     GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182:24)
E/flutter (13086): #11     TapGestureRecognizer.handleTapUp (package:flutter/src/gestures/tap.dart:607:11)
E/flutter (13086): #12     BaseTapGestureRecognizer._checkUp (package:flutter/src/gestures/tap.dart:296:5)
E/flutter (13086): #13     BaseTapGestureRecognizer.handlePrimaryPointer (package:flutter/src/gestures/tap.dart:230:7)
E/flutter (13086): #14     PrimaryPointerGestureRecognizer.handleEvent (package:flutter/src/gestures/recognizer.dart:475:9)
E/flutter (13086): #15     PointerRouter._dispatch (package:flutter/src/gestures/pointer_router.dart:93:12)
E/flutter (13086): #16     PointerRouter._dispatchEventToRoutes.<anonymous closure> (package:flutter/src/gestures/pointer_router.dart:138:9)
E/flutter (13086): #17     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:397:8)
E/flutter (13086): #18     PointerRouter._dispatchEventToRoutes (package:flutter/src/gestures/pointer_router.dart:136:18)
E/flutter (13086): #19     PointerRouter.route (package:flutter/src/gestures/pointer_router.dart:122:7)
E/flutter (13086): #20     GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:439:19)
E/flutter (13086): #21     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:419:22)
E/flutter (13086): #22     RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:287:11)
E/flutter (13086): #23     GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:374:7)
E/flutter (13086): #24     GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:338:5)
E/flutter (13086): #25     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:296:7)
E/flutter (13086): #26     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:279:7)
E/flutter (13086): #27     _rootRunUnary (dart:async/zone.dart:1370:13)
E/flutter (13086): #28     _CustomZone.runUnary (dart:async/zone.dart:1265:19)
E/flutter (13086): #29     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1170:7)
E/flutter (13086): #30     _invoke1 (dart:ui/hooks.dart:182:10)
E/flutter (13086): #31     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:282:7)
E/flutter (13086): #32     _dispatchPointerDataPacket (dart:ui/hooks.dart:96:31)

For anyone encountering the same problem, I simply had to define MessageSendOptions TDObject in the SendMessage TDFunction


Publish Ouroborus Gem in Github Packages

depends on #50

#54 make what it was supposed to do but correctly.


Anti-Aim Monitor

https://www.unknowncheats.me/forum/counterstrike-global-offensive/302732-anti-aim-monitor.html

Why? If you see fakes angles without this.

you don't see fakeangles in osiris. :)

how change switch angle of anti-aim?

Nice English


Skip JumpList on NanoServer and IoT

PR Summary
The STA apartment state is not supported on NanoServer and Windows IoT.
Skip JumpList creation on NanoServer and IoT.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

@PoshChan Please remind me in 20 minutes

@TravisEz13, this is the reminder you requested 20 minutes ago


Change Django  Tests to pytests

Signed-off-by: sbs2001<EMAIL_ADDRESS>I have used this  pytest extension. We are able to run all the tests simply by running pytest in the root directory.
I am currently working on fixing the  following warnings(I could use some help here).
`(venv) shivam@shivam-Latitude-D630:~/play/vulnerablecode$ DJANGO_DEV=1 pytest
==================================================================== test session starts ====================================================================
platform linux -- Python 3.7.3, pytest-5.3.2, py-1.8.0, pluggy-0.13.1
Django settings: vulnerablecode.settings (from ini file)
rootdir: /home/shivam/play/vulnerablecode, inifile: pytest.ini
plugins: django-3.7.0
collected 27 items

vulnerabilities/tests/test_api.py ......                                                                                [ 22%]
vulnerabilities/tests/test_data_dump.py .......                                                                                [ 48%]
vulnerabilities/tests/test_models.py ....                                                                                [ 62%]
vulnerabilities/tests/test_api_data.py ..                                                                                [ 70%]
vulnerabilities/tests/test_import_cli.py ...                                                                                [ 81%]
vulnerabilities/tests/test_npm.py ...                                                                                [ 92%]
vulnerabilities/tests/test_scrapers.py ..                                                                                [100%]

===================================================================== warnings summary ======================================================================
vulnerabilities/tests/test_api.py::test_query_by_name
vulnerabilities/tests/test_api.py::test_query_by_invalid_package_url
vulnerabilities/tests/test_api.py::test_query_by_package_url
vulnerabilities/tests/test_api.py::test_query_by_package_url_without_namespace
vulnerabilities/tests/test_api.py::test_ubuntu_response
  /home/shivam/play/vulnerablecode/venv/lib/python3.7/site-packages/whitenoise/django.py:50: RemovedInDjango31Warning: The FILE_CHARSET setting is deprecated. Starting with Django 3.1, all files read from disk must be UTF-8 encoded.
    self.charset = settings.FILE_CHARSET

vulnerabilities/tests/test_api.py::test_query_by_name
vulnerabilities/tests/test_api.py::test_ubuntu_response
  /home/shivam/play/vulnerablecode/venv/lib/python3.7/site-packages/django_filters/rest_framework/backends.py:31: MigrationNotice: `PackageViewSet.filter_fields` attribute should be renamed `filterset_fields`. See: https://django-filter.readthedocs.io/en/master/guide/migration.html
    filterset_class = self.get_filterset_class(view, queryset)

-- Docs: https://docs.pytest.org/en/latest/warnings.html
============================================================== 27 passed, 7 warnings in 45.79s ==============================================================
`

@sbs2001 Thanks for this PR!
Since all tests are now being discovered and run with one command, can you also remove the old call to pytest from .travis.yml?
I think the warnings are unrelated to your changes. My guess is that pytest shows them by default whereas the Django test runner doesn't. Two dependencies (whitenoise and Django REST Framework) use features of Django that will be removed in a future version. It is possible that there are more recent versions of those apps that address these warnings. You can try changing their versions in requirements.txt to the latest to check.
In the chat you wrote:

I am bit shaky about whether I used the fixtures and marks effectively.

I think you use them correctly. However, the marks are only used in test_models.py which does not actually test any behaviour of vulnerablecode but just basic Django ORM operations. I don't think having these tests provides any benefit so I deleted the file in my pending PR #132.
And regarding the fixtures; Having them in a separate file increases the distance to the code using them and arguably makes it harder to read the tests. @pombredanne commented elsewhere that he prefers tests to be self-contained if possible. Personally I don't mind it and prefer to avoid duplication if that is the tradeoff to be made. Maybe @pombredanne wants to chime in on this? Otherwise, if we do leave it as is, would you mind renaming the file from conftest.py to fixtures.py?
I didn't expect you to change all the existing tests from TestCase subclasses to simple functions. But since you did I'm fine with keeping them that way. However by removing the extra namespace of the class, you created a few name clashes, effectively removing some tests. If you look at the CI log on your PR, it says 27 passed after running pytest. Before, running manage.py test, it reported 28 tests. The four tests that needed to be run separately before are now included in running pytest, so the total number of tests should be 32.

@haikoschol I have corrected the code.  I changed the names to avoid name  clashes. The warnings are not fixed though. Thanks!

The warnings are not fixed though.

We can look into that after merging.

Regarding conftest.py , after researching albeit I don't think we can change that to fixtures.py .

I didn't realize that the filename has special meaning for pytest. In that case we'll leave it as is.


Not working

vsc version: 1.45.0-insider
os: Windows 10 Corporative Insider Preview Trial version. Build 18895.rs_prerelease.190507-2033

Same here.
Version: 1.45.0-insider
Commit: fd5d9a30259bb2bebab3e62dfd8315c2e1b27744
Date: 2020-05-04T09:21:17.770Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 18.7.0

Probably a duplicate of #249

fixed

fixed

thx


Fixed ordering of includes in varusage finder.

Seemingly apple clang gets confused sometimes with this and generates invalid code
This commit does not change any documented behavior.
We only managed to reproduce it on a release package of MacOs.
It does not show up on any development version.

Jenkins:
http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/9974/

That shouldn't make a difference, or.... Thanks for finding...

Jenkins after devel update:
http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/9975/

Tests blue.

@joerg84 i was under the impression too.
I was expecting that the initializer list would just create the Object as expected.
However:
If I explicitly name the Object to be created the compiler (clang) was not able to compile this viel, because the Type was not defined.
Seemingly gcc was able to compile everything fine.
So my guess is that clang trailed down an unexpected path.