Migrate test code to JUnit 5

Is your feature request related to a problem? Please describe.
The test code base is still on JUnit 4. Most modern frameworks use JUnit 5 nowadays.
Describe the solution you'd like
The test code should be migrated to JUnit 5.
Open Rewrite could be helpful here, it supports refactoring from JUnit 4 to 5. The migration supports partially the effort.
However, things like JUnit 4 rules can't be automatically migrated. They are used quite much in the code and it will be manual effort to rewrite these test classes.
Due to the vast amount of test code it might be necessary to do this step by step. In this case JUnit 5 and 4 code have to coexist and work together in the build. This will likely require using the vintage engine.

Manual migration is also needed for exception checking. Common pattern here:
  protected ExpectedException thrown = ExpectedException.none();

  ...
  testMethod() {
    // then
    thrown.expect(ValueMapperException.class);

    // when
    client.subscribe(EXTERNAL_TASK_TOPIC_FOO)
      .handler(handler)
      .open();

This would become:
    // when + then
    assertThatThrownBy(() ->
      client.subscribe(EXTERNAL_TASK_TOPIC_FOO)
        .handler(handler)
        .open()
    ).isInstanceOf(ValueMapperException.class);

Rewrite of WireMockRule did also not work. It needs manual effort to migrate e.g. SoapHttpConnectorSystemPropertiesTest.

Thanks for the investigation!
I talked to Christian Stein about this some years ago, and I think the inofficial suggestion of the JUnit team was to write new tests in JUnit Jupiter while keeping existing tests in JUnit 4, which can be run on the JUnit 5 engine, as the benefit of migrating legacy tests to the new API are rather small.
Having everything in a similar structure and format would be a lot nicer, however, I agree there. I also think that the Jupiter API is more expressive and has a better readability.
Maybe this is something that can be done sequentially, module after module, to keep the commits smaller? This would isolate changes like the one to Wiremock a little better

I understand that leaving and running JUnit 4 classes and incrementally migrating tests is a way to go. However, it is not as simple as it seems. The tests are relying on classes (e.g. by subclassing, rules etc.) that can be either JU4 or JU5, and having these base classes together is also cumbersome.
I'll work on this task and try to get everything running. Now is a good time to do this as not much movement is happening on the tests themselves. I might fail to get through it. And likely reviewing is more or less impossible, except for comparing the number of executed tests in the end.

That reasoning sounds good to me, thanks!

As you noticed I have started the migration to JUnit 5 submodule by submodule, starting with those with least dependencies. This is a bit try-and-error.
I will try to provide chunks by MRs that do not break the build. The dependencies are quite painful.

Problematic are also parametrized tests that have initialization in a beforeEach manner.
Example: TestModelTest and its subclasses from model-api/xml-model.
JUnit 5 does not directly support parameter resolving in methods annotated with @BeforeEach.

./distro/wildfly/subsystem can't be migrated. It depends on org.wildfly.core:wildfly-subsystem-test-framework for which no replacement exists yet for JU5.

engine-cdi should be migrated after merging engine-cdi/core with engine-cdi/jakarta.
A branch junit5-migration/engine-cdi has been prepared with first steps.


Hides application menu

Fixes https://github.com/rafgui12/Office365WebDesktop/issues/11
as the title says

@jobs-git thanks for implementing this, let's hope that this will be accepted soon!

Thanks for all the

Is there going to be a new snap released with this feature?


Windows x64, sharp v0.25.1, The specified procedure could not be found

Did you see the documentation relating to installation?
Yes
Have you ensured the platform and version of Node.js used for npm install is the same as the platform and version of Node.js used at runtime?
Yes, Node v13.10.0
Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?
Yes, v0.25.1
If you are installing as a root or sudo user, have you tried with the npm install --unsafe-perm flag?
N/A
If you are using the ignore-scripts feature of npm, have you tried with the npm install --ignore-scripts=false flag?
N/A
What is the complete output of running npm install --verbose sharp? Have you checked this output for useful error messages?
As far as I'm aware there aren't any errors.

  System:
    OS: Windows 10 10.0.18362
    CPU: (4) x64 Intel(R) Core(TM) i3-8130U CPU @ 2.20GHz
    Memory: 1.14 GB / 5.88 GB
  Binaries:
    Node: 13.10.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.19.0 - C:\Users\chris\AppData\Roaming\npm\yarn.CMD
    npm: 6.13.7 - C:\Program Files\nodejs\npm.CMD

I have been trying for hours to get it to install correctly.  The error I am getting is:
(node:10060) UnhandledPromiseRejectionWarning: Error:
Something went wrong installing the "sharp" module

The specified procedure could not be found.
\\?\F:\Documents\Tenpi\Programming\Bots\Kisaragi\node_modules\sharp\build\Release\sharp.node

- Remove the "node_modules/sharp" directory then run
  "npm install --ignore-scripts=false --verbose" and look for errors
- Consult the installation documentation at https://sharp.pixelplumbing.com/install
- Search for this error at https://github.com/lovell/sharp/issues

    at Object.<anonymous> (F:\Documents\Tenpi\Programming\Bots\Kisaragi\node_modules\sharp\lib\constructor.js:34:9)
    at Module._compile (internal/modules/cjs/loader.js:1147:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
    at Module.load (internal/modules/cjs/loader.js:996:32)
    at Module.require (internal/modules/cjs/loader.js:1036:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (F:\Documents\Tenpi\Programming\Bots\Kisaragi\node_modules\sharp\lib\index.js:3:15)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
    at Function.Module._load (internal/modules/cjs/loader.js:896:14)
    at Module.require (internal/modules/cjs/loader.js:1036:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (F:\Documents\Tenpi\Programming\Bots\Kisaragi\dist\commands\image\blur.js:16:33)
    at Module._compile (internal/modules/cjs/loader.js:1147:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
    at Module.load (internal/modules/cjs/loader.js:996:32)
    at Function.Module._load (internal/modules/cjs/loader.js:896:14)
    at Module.require (internal/modules/cjs/loader.js:1036:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at F:\Documents\Tenpi\Programming\Bots\Kisaragi\dist\index.js:39:34
(node:10060) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:10060) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

same issue here, im getting stuck at: info sharp Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.8.1/libvips-8.8.1-win32-x64.tar.gz

I have got the same problem

Error:
Something went wrong installing the "sharp" module

@miranahf Please open a new issue and answer all of the questions - https://github.com/lovell/sharp/issues/new?labels=installation&template=installation.md

@Tenpi It looks like sharp itself installed OK but there's a problem at runtime, possibly relating to DLL files.

What is the contents of the node_modules\sharp\build\Release directory?
Are there any other native module dependencies e.g. node-canvas?
Does inspecting node_modules\sharp\build\Release\sharp.node with http://www.dependencywalker.com/ provide any insight?

@Tenpi It looks like sharp itself installed OK but there's a problem at runtime, possibly relating to DLL files.

What is the contents of the node_modules\sharp\build\Release directory?
Are there any other native module dependencies e.g. node-canvas?
Does inspecting node_modules\sharp\build\Release\sharp.node with http://www.dependencywalker.com/ provide any insight?

Thanks for the fast reply. With dependency walker I found some log files in sharp/build/Release/obj/libvips-cpp/libvips-cpp.tlog that say the following:
LINK-VCTIP.DELETE.1.TLOG - No DOS or PE signature found. This file is not a valid 32-bit or 64-bit Windows module.
Error: At least one file was not a 32-bit or 64-bit Windows module.

 Based on the error it looks like the files that were downloaded are not compatible with win32/64 bit (but I am not too sure).

@miranahf Please open a new issue and answer all of the questions - https://github.com/lovell/sharp/issues/new?labels=installation&template=installation.md

Thanks.
I have resolved this problem,that caused by the slow internet connection.
I pre-downloaded a package,libvips-8.9.1-darwin-x64.tar.gz,and put it in the cache path of npm.
Then 'npm install sharp',and everything is OK.

I am using node-canvas in my project as well.

Please can you temporarily remove this dependency and see if you still have the same problem.

Unfortunately wasn't able to solve this issue, I am using a different package instead. You don't have to resolve this issue (it's probably something weird that is specific to my machine).

I am using node-canvas in my project as well.

Please can you temporarily remove this dependency and see if you still have the same problem.

I am having exactly the same issue as OP's. I do also have node-canvas in my code. The error disappears when I removed all require('canvas') lines in my code. Thus I suspect too the installation of sharp is successful, but there is a dependencies incompatibility between node-canvas and sharp.
@lovell Should I open a new issue to provide more details? Thanks for your time.

The prebuilt binaries provided by the forthcoming sharp v0.26.0 are switching from dynamically linked to statically linked, which should solve this class of problem, at least most of the time.
For now, if you're relying on a globally-installed libcairo for node-canvas, then you should also rely on a globally-installed libvips for sharp.

For now, if you're relying on a globally-installed libcairo for node-canvas, then you should also rely on a globally-installed libvips for sharp.

Hi, I have the same issue and how can I do that?
I installed the gtk+-bundle_2.22.1-20101229_win64 and then I added the C:\GTK\bin directory to the PATH... but the error still occur :(
Thanks

Can this issue be reopened until the 0.26 release? I am experiencing it also, and would like some way to be notified when the 0.26 release is available by subscribing to this thread.

@CoenraadS New releases can be subscribed to via https://github.com/lovell/sharp/tags.atom (there's usually a small delay between a tag being created and publication to npm).


[Bug]: Overlapping button on homepage

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
bottom of the homepage(footer) has overlapping button of visitors count that appears on top of social media links.
Add Screenshots / Video Clip

What browsers are you seeing the problem on?
Chrome
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[ ] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

the home page footer has a button (visitor count) which seems to  overlap on the social media icons present there i want to fix this issue.
Assign me the issue

@bhat-shubham


external link icon after image

https://dvc.org/doc/use-cases/data-registries

Looks like these external link icons are applied through CSS ::after through an a[target='_blank'] selector, so we just have to make another selector to suppress that ::after on image links.


chunk size setting in vineyard client .get method like that argument in pandas.read_csv

What is the problem:
How to get a chunk or slice from a vineyard object which is a pandas dataframe
The behaviour that you expect to work:
like "obj = vineyard_sock.get("1e24465464", chunksize=10000)" returns a generator so that we can do "for chunk in obj: ..."
the key point I'm interesed in is that if it is possible that there is no enough memory on local machine to execute vineyard_sock.get so that we need to get one chunk by one chunk?
Additional context

This feature should be part of https://github.com/v6d-io/v6d/issues/220


Adding other data to JWT Token

Hello,
I think it would be nice to be able to add other user related fields to the generated token, like user's full name or data developpers would like to send to the client.
Thanks

Sorry, my bad, I think i can use the listener.


Is hmd screen 'low persistence' setting part of OpenXR?

Hello!
I'm running the 'hello_xr' program and am getting ghosting on objects as I turn my head around and it reminds me of the old days before low persistence became a thing.
I'm doing this on a Raspberry Pi4b :)    (It was easier to get that running than PC-Ubuntu-NVidia)
And with a Oculus Rift CV1.
Is the on/off setting of the HMD screens' low persistence mode something that I should expect OpenXR to be able to take care of?
Cheers!

It should be done by the runtime or headset firmware. I assume you're using Monado? I'm not sure why you'd see ghosting, seems weird that Oculus would have low-persistence off by default, but I don't have a CV1 to test with.

Hi!  Yes correct guess I am using Monado. So you think the low-persistence setting would be already in a 'state' in the HMD firmware and unless changed it should reamin at the whatever setting?
When I'm running the hello_xr program in the Rift it now feels like I have a pretty good frame rate but it would be useful to get a confirmation of that.
The static blue square in the middle of the view that's locked to the head transform looks perfect no ghosting on that, but of course that's static relative to the view.

If your terminal isn't flooded with "frame missed" messages I think it's keeping up. In any case, let's continue this on the Monado Discord - see link at https://monado.freedesktop.org


Change Extra Pages "Add another image" button text

Since we want to allow pdfs to be uploaded as Extra Pages (#113), please change the button text,

From: Add another image
To: Add another file

👍


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Undersampler Component Initial Look

This is an initial version of the undersampling component, originally proposed here, which originated from this suggestion request. This is a custom component that performs an undersampling operation on a tf.Record dataset of tf.Examples, such as the output of an ExampleGen component, using an Apache Beam pipeline. This PR also contains an updated project proposal to reflect the move from a Python function component to a fully-custom component. The purpose of this draft PR is to get feedback on our initial work; your comments and suggestions are much appreciated.

Just a quick update: a finished initial version of the component can be found here.

@kindalime thanks for this PR. I copied this code and pasted it into my own project. My comments are based on issues I had trying to get the component running correctly. This is some great work, I just had a few changes I think you should make to make this component more compatible with the various environments.
Give my comments a read and let me know what you think. Perhaps I can create a branch and PR the changes into your branch, let me know.
I had another question about your component. So for my use case, I needed over sampling i.e. I found the class that was most represented and sampled each class until it matched the number of examples for this majority class. If we added a parameter and just called your component sampler we can have it work for both cases. What do you think? I'm happy to PR these changes into your component so we can get this included in the repo as soon as possible. What do you think?

Oversampling sounds like the natural extension to this project - this sounds like a good idea. To clarify - have you already made these changes, and want me to accept a PR?

@kindalime I've made the changes in a copy of your component that I have in another project. I'll branch off this PR and apply the changes there. Shouldn't take too long 👍

@kindalime I've finished with the changes, can you give me permission to push to your repo, please.

Would it be possible to fork + PR instead? I'd like to quickly look over the proposed changes when you PR if possible.


Write a script to create SBOMs for OCI images and rocks

Context
We need an automated way to produce SBOMs for all our OCI images and rocks
What needs to get done
Write a script that does the following:

snap install syft
calls our gather images script
for each image/rock. syft scan container@manifest -o<EMAIL_ADDRESS>(replacing "container" with the url/path to the container, e.g. grafana/grafana; and "manifest" with the manifest digest for the container)

Definition of Done
We have a script to produce SBOMs for all our images and rocks

I wrote a script that re-uses our get_all_images.py script to produce the SBOMs for kubeflow bundle images.
For mlflow bundle images, this will not apply since mlflow does not have the tools/get-images.sh. To generate the mlflow SBOMs, I ran the following commands:
syft scan docker.io/charmedkubeflow/mlflow:v2.15.1-d756ce7 -o spdx-json=docker.io/charmedkubeflow/mlflow:v2.15.1-d756ce7.spdx.json
syft scan docker.io/charmedkubeflow/mlflow-prometheus-exporter:1.0-22.04 -o spdx-json=docker.io/charmedkubeflow/mlflow-prometheus-exporter:1.0-22.04.spdx.json

And included the SBOMs for minio and mysql images generated by the script for kubeflow bundle.
The mlflow SBOMs are now uploaded in the team's Gdrive.


Maintaining under Moleculer org

Hi @tiaod,
are you still maintaining this repo? If not, could you transfer the repo into the moleculerjs organization on Github, and transfer the NPM access to me?
Thanks, in advance!

I have been writing client programs in my recent work, and I haven't written server code for a long time, so I haven't used moleculerjs for a while, and this project is currently not being maintained. I will transfer it to you.

@icebob You should first give me permission to create repositories in moleculerjs organization

I've sent an invite.

Done!

Awesome, thanks!


[bcs]fix to_addr unwrap error

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/libra/developers.libra.org, and link to your PR here.)

可以把峰哥那个测试加上

可以把峰哥那个测试加上


Change Design of Vertx HttpClient Implementation to Resolve Flaky Tests

Description
Changes the design of azure-core-http-vertx to convert requests and responses into single, potentially large, byte arrays instead of streaming smaller ByteBuffers/Buffers in an attempt to resolve reliability issues in testing. This change will disable support for request and response bodies over 2GV in azure-core-http-vertx, but may offer a path to getting a first beta released if this resolves the reliability issues. If this does resolve the reliability issues there will be follow-ups to re-enable stream request and response support.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - core - ci


Update permissions-required-for-github-apps.md

Adjusting permissions after feedback on https://github.com/github/external-identities/issues/2534.

Why:
It's not possible for GitHub Apps to access these endpoints without write permissions.  This is a known bug with querying SAML/SCIM identities with Read only permissions for members. https://github.com/github/external-identities/issues/2534. We won't have a bug fix for this out any time soon, so the engineering team responsible for it asked if I could raise a docs PR.
https://github.com/github/external-identities/issues/2534

What's being changed (if available, include any code snippets, screenshots, or gifs):

I'm updating the permissions required for GitHub Apps to access GET /scim/v2/orgs/:org/Users and GET /scim/v2/orgs/:org/Users/:external_identity_guid. Both of these endpoints require Members write permissions as highlighted on the issue.
- [`GET /scim/v2/orgs/:org/Users`](/rest/reference/scim#list-scim-provisioned-identities) (:read)
+ [`GET /scim/v2/orgs/:org/Users`](/rest/reference/scim#list-scim-provisioned-identities) (:write)

and
- [`GET /scim/v2/orgs/:org/Users/:external_identity_guid`](/rest/reference/scim#get-scim-provisioning-information-for-a-user) (:read)
+ [`GET /scim/v2/orgs/:org/Users/:external_identity_guid`](/rest/reference/scim#get-scim-provisioning-information-for-a-user) (:write)

Check off the following:

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

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

[ ] This pull request impacts the contribution experience

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

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


Q: NuGet (Xplat) Server - long term strategy

Hi
dotnet core is depending (extremely) on Nuget (Protocol, and Gallery)
Now, we have May 2017 and there is no Nuget Server (pure core/xplat) available or at least planned.
For my own/private Nuget Server i must use the full blown (Framework >=4) asp.net and SQL-Server implementation or a Java Implementation (Nexus) or something else.....
All this does not increases my confidence in this system (and i like dotnetcore)
I know that Nuget is a different project and currently not part of the dotnet foundation, but it is also MS-coupled and the Project is really critical for dotnetcore.
What is the plan, what is the strategy?
@terrajobst et.al. ?
regards
Werner

This issue was moved to NuGet/NuGetGallery#3899


Add support for reading old color palette chunks

If an indexed image is saved in Aseprite and it has a palette with exactly 256 colors with no alpha in any of them, only the old palette chunk (0x0004) is saved into the file. This makes those files fail to parse in asefile. This adds logic to parse the old palette chunks so that such files will load properly.

Thanks for the PR! Looks good.
The 6 -> 8 bit upscaling wasn't clear to me, at first. I understand it now, but it's not obvious. I'll add a comment and then put it in the next release.

Yeah, it's the same bit shifting logic used by Aseprite itself, so I just wanted to make sure it was consistent.


fix pluck typing issue when CompositeTableType is used

When a CompositeTableType is used, keyof TRecord will result in base, insert, and update, instead of the real database column names. This change fixes this issue.
Table definitions:
import { Knex } from 'knex'

declare module 'knex/types/tables' {
  interface User {
    id: string
    name: string
    createdAt: Date
    updatedAt: Date
  }

  interface UserInsert {
    id?: string
    name: string
    createdAt?: Date
    updatedAt?: Date
  }

  interface UserUpdate {
    id?: string
    name?: string
    createdAt?: Date
    updatedAt?: Date
  }

  interface Tables {
    users:  Knex.CompositeTableType<User, UserInsert, UserUpdate>
  }
}

When using:
import knex as 'knex'
import knexConfig from './knexfile'

const db = knex(knexConfig)

/*
  *  Currently shows type as `{}`,
  *  and autocomplete suggests: `base`, `insert`, `update`.
  *  This is because those are the keys of `CompositeTableType`.
  *
  *  With the fix, it shows the type as `string[]`,
  *   and autocomplete suggests `id`, `name`, `createdAt`, `updatedAt`.
  *  This is because `ResolveTableType` can pull out `User` interface.
  */
db('users').pluck('id')

could you please add tsd test for this?

@kibertoad I'll try. I've never done this before. Some hints would be nice.

@kibertoad after some research I think you are referring to adding tests under ./test-tsd directory? I have some issues running the tests though. Maybe you can help. Here is what I did:
npm install
npx tsd

The process runs a bit, with no output and just stops.  I then tried to modify a few tests and see if it fails, and ran the npx tsd command again, and same result, no output or any indications.
Am I doing something wrong?

@aq1018 I usually run it using npm run lint:types, but npx tsd should work fine as well. Unless it actually fails with an error, I think there is no output from it.

You are targetting master.

I set the state of this branch back to before I messed with it a few hours ago. I'll try to figure out how to write tests so this later on this weekend I guess.

When I added the tests, I tried removing the changes that @aq1018 made while running them and expected to see a failure. However, when I tried that, they actually still passed for some reason, even though this exact use case failed in our own application. I spent some time investigating that, but I couldn't figure out what was different between our application and the test. At the very least, this change doesn't break existing typing, though.


See if the new version of http-signatures will work without patching

Would resolve FAQ item on InvalidHeaderError

I have just tried http-signature version 1.3.5 the InvalidHeaderError: bad param format is still there.

@gregid thanks, I'll close this then - conclusion is we need to move away from http-signatures as it is also limiting our request library options


Properly set the room name in received messages

@jimmycuadra This should fix #44.
Before, the channel ID was being set as both the id and name within the message's room_object.
This PR will either lookup the correct room object, which would have a name, or it will assign the channel ID as a fallback (which will happen when it's a private conversation without a name). This relies on Lita::Source accepting either a Lita::Room or a String.

Thanks very much!


Ongoing refactoring

There will be some ongoing refactoring until support is solidified for about three platforms. Supporting more than two platforms really clarifies what code belongs in platform-specific blocks, and what can be generalized.

Tasks

[ ] confirm_preliminary() should be generalized and placed in simple_deploy.py.


ESLint - no-unused-expressions warning for template tags

I'm using styled-components' injectGlobal template tag function, like so:
injectGlobal`
  .ant-btn-clicked:after {
    border: 0 solid ${mainTheme.primaryColor.toString()} !important;
  }

  .ant-switch-checked {
    background-color: ${mainTheme.primaryColor.toString()} !important;
    border-color: ${mainTheme.primaryColor.toString()} !important;
  }
`;

And I got warnings about injectGlobal being an unused expression. So I've modified my .eslintrc to be the following:
{
  "extends": "react-app",
  "rules": {
    "no-unused-expressions": ["error", {"allowTaggedTemplates": true}]
  }
}

This seems to stop ESLint warnings in Nuclide, but I'm still getting them in the browser console and shell:
./src/themes.js
  Line 20:  Expected an assignment or function call and instead saw an expression  no-unused-expressions

Am I configuring something wrong?

Overriding our ESLint setting is not really supported. It technically "works" for IDEs so we documented it, but we still want everyone to have the same base configuration. This helps make sure it works correctly for everyone, that bugfixes are coordinated, and that everyone has mostly good experience out of the box.
In this particular case it seems like we should change our config. Our stance is that we try to create a config that's good for most users rather than allow users to make individual tweaks, thus keeping the base config less ideal.
If you send a PR for fixing this in our config I'm happy to take that in.

This issue can be closed now 👍

Thanks!

This feature has been released as of v1.0.11. :)


Fix categories consistency in rss

My previous inversion was inconsistent - first try was correct as the severity should be displayed as a TERM like showed above in the feed declaration. I opened an issue on the "feed" project to investigate why it's showing the "label" field in the json result instead of the "term" : https://github.com/jpmonette/feed/issues/178

👍


Fix and enable unit tests on Travis CI

There's a bunch of existing tests, but we're not running them with Travis CI. Unfortunately there's a lot of failing tests so we'll have to fix that before we can enable this in CI

Fixed through #56 and #61


Submit timeout is too short

Submission currently stops after 15 seconds. Students who run ok many times without backing up hit the timeout error. Increase timeout to 1 minute? (Or more? As long students see the progress, they should be fine with waiting.)

The backup protocol now does not timeout on submit.


Fix Carthage building failure

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[ ] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necesarry)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes the following issues: #790
This merge request refers to the following issues: #782
Pull Request Description
I encountered the similar error that #790 mentions.
As #782 mentions, in the first place,  mixing universal  and the specific platform target does not seem to be good.
So I changed the framework form CocoaLumberjack/CocoaLumberjackSwift(for Universal) to CocoaLumberjack-macOS/CocoaLumberjackSwift-macOS(for macOS).
(For additional info, these target are actually used only for macOS as long as I checked .travis.yml and the GettingStarted.md.)
And I fixed trivial bug(see https://github.com/CocoaLumberjack/CocoaLumberjack/commit/2bd6b6f86c15983d1695049993236bb8bc03e5a2).
This PR should fix the issue #790. At least I confirmed that Carthage build has been successful about every framework target(iOS, macOS, tvOS, and watchOS). Carthage version is 0.18.
Pls check the each commit for the detail.

Current coverage is 22.76% (diff: 100%)

Merging #819 into master will not change coverage

@@             master       #819   diff @@
==========================================
  Files            16         16
  Lines          3884       3884
  Methods         436        436
  Messages          0          0
  Branches        279        279
==========================================
  Hits            884        884
  Misses         2998       2998
  Partials          2          2

Powered by Codecov. Last update a42a14b...703d8bb


Remove version: qBittorrent.qBittorrent version 4.6.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/qbittorrent/files/qbittorrent-win32/qbittorrent-4.6.3/qbittorrent_4.6.3_x64_setup.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available.;


How to save JSON object to table string column

How do I serialize to a string to save to the DB? I was under the assumption that the JsonTypeHandler would take care of this and all I have to do is pass the object Json Data e.g.
`
      public async Task<bool> UpdateDataAsync(DataObject do)
        {
            bool response = false;
...
            string sqlUpdate = $@"
                            UPDATE [Program]
                            SET [ProgramData] = @Data
                               ,[LastUpdatedById] = @LastUpdatedById
                               ,[LastUpdatedOn] = @LastUpdatedOn
                            WHERE [Id] = @Id";

            using (var db = new SqlConnection(_connectionString))
            {
                await db.OpenAsync();
                using (var transaction = await db.BeginTransactionAsync())
                {
...
                    await db.ExecuteAsync(sqlUpdate, do, transaction, commandTimeout: 180);
                    await transaction.CommitAsync();
                }
            }
            return response;
        }

`

The following sample works as expected:
var dataObject = new DataObject
{
    Id = 3,
    Settings = new Json<UserSettings>(new UserSettings() { HideSearchBar = true })
};

var rows = await connection.ExecuteAsync(
    """
        UPDATE Users SET
            SettingsJson = @Settings
        WHERE Id = @Id
       """,
    dataObject);

class DataObject
{
    public int Id { get; set; }

    public Json<UserSettings> Settings { get; set; }
}

However, I don't like the API. I am going to improve it

Upload new version 1.0.0-preview.2. It has a better API when you want to insert something into the database. Here is example:
var id = 3;
var settings = new UserSettings
{
    Update = DateTimeOffset.Now,
    HideSearchBar = true
};

var rows = await connection.ExecuteAsync(
    """
        UPDATE Users SET
            SettingsJson = @Settings
        WHERE Id = @Id
       """,
    new { Id = id, Settings = settings.AsJson() });

var user = await connection.QueryFirstOrDefaultAsync<User>("""
                    SELECT
                        Id,
                        SettingsJson AS Settings
                    FROM Users
                    WHERE Id = @Id
                 """, new { Id = id });

Console.WriteLine(JsonSerializer.Serialize(user));
// {"Id":3,"Settings":{"Value":{"Update":"2023-02-04T17:17:15.7574945+02:00","HideSearchBar":true}}}

Will definitely try it, thanks for the update


How to perform back testing?

Sorry I'm new to backtrader.
Is there any way to use this package to test strategy on past data without placing real order?

@tonypottera24 I'm sorry for the late reply. Yes, it is possible.
Use backtrader for that, without changing the default Broker and Store.

Hi, i left a new issue before I saw this one. But if you remove broker&store, then you wouldnt use this package for anything? Not possible to download the data in "same format" using this?


header cells not aligned when column right pinned + scrollbar in Chrome

When you have a column pinned to the right and a scrollbar in the table, the right header cells aren't aligned to the right body cells.
This bug occurs in Chrome 53. Firefox 47 and Safari 9.1.2 are ok.
See: https://www.ag-grid.com/javascript-grid-pinning/index.php

don't know what you mean, that example is working fine for me.

I have same issue!

ah right, yes i see, top right, the column is slightly out. i'm aware of this. will add it to my list. medium priority.

this was fixed end of last year


fix: attempt to fix zipping in CI

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


Will not work on Windows

As far as I know, termios is a package only available at UNIX, an import of which will result in an error on any Python distribution built for Windows. Perhaps this can be replaced with something cross-platform instead?

yes, I've been trying to work on windows support. I mostly develop on a MacBook and don't know the windows terminal and overall dev experience. so it'll take some time for me to do it.


[Feature] Build WHERE filter for reading data of microbatch incremental builds from properties

Housekeeping

[X] I am a maintainer of dbt-core

Short description
In #10624 we're building a new type of incremental model strategy, microbatch. The microbatch strategy is cool for a lot of reasons, and all of those can be read about elsewhere. This feature is that the incremental filter for an incremental model should automatically get generated. What that means is that when using the microbatch strategy for an incremental model, one will NO LONGER need to put the following in the model
-- my_model.sql
...
{% if is_incremental() %}

  -- this filter will only be applied on an incremental run
  -- (uses >= to include records whose timestamp occurred since the last run of this model)
  -- (If event_time is NULL or the table is truncated, the condition will always be true and load all records)
  where event_time >= (select coalesce(max(event_time),'1900-01-01') from {{ this }} )
{% endif %}

INSTEAD the incremental where filter will automatically be generated when appropriate.
Acceptance criteria

a "where filter" is automatically generated for microbatch strategies

depends on event_time, event_time_start, event_time_end, batch_size properties being present

Suggested Tests

A "where filter" is only generated for microbatch incremental models
Generated "where filter" is correct in regards to event_time, event_time_start, event_time_end, batch_size properties

Impact to Other Teams
Adapters?
Will backports be required?
No
Context
Depends on:

#10635
#10637

can I pick it ?

Hey @tsafacjo ! What do you mean by 'pick'? If you mean pick up this PR for development, we've already got that work in flight in #10594 by the maintainer team. If you mean, "can users set this where filter?", we are building out CLI flags as well as other model-level configurations to do so here: https://github.com/dbt-labs/dbt-core/issues/10635

Actually it's was more about pick up this PR for development.

Resolved by #10594


Keyboard WKWebview Test Branch - update needed please

Hi All,
Can somebody on the ionic team please update this thread?
I left my feedback in here regarding the issue I saw.  Also, when is the test branch getting merged into main?
https://github.com/ionic-team/ionic/issues/11501
Thanks

Hi there! Let's keep the conversation in one place. I'm going to close this for now.


Database Connection

I finished setting up the SQL database and added a new route to server.js to test and make sure it is running and our server is connected. Everything ran successfully.

Accidentally make request to main instead of dev.


Feature request

Hello and thank you for that software. I'd like to request some new features.

trace route - for network diagnostics
IP subnet calculator - to calculate addresses. Calculate IPv4 and IPv6 subnet information such as network address, subnet mask, IP range.
WOL - wake on LAN function
"Server checker" - such an option, which can check remote host via ICMP, reach-ability of a TCP port or text on a specified web page. It would be great to get a graph and statistics when host was unreachable. Also I'd like to receive an alert when host goes down.

Traceroute - https://pub.dev/packages/flutter_traceroute is only available for mobile devices (Android and iOS). This will need more work to make it work on Desktop.
Wake on LAN - https://pub.dev/packages/wake_on_lan. Supports both mobile and desktop platforms. This can be implemented faster, so I am going to integrate this package soon.
I am not clear on "IP subnet calculator", Do you have any example application to try? (I think this can be in scope)
Also for "Server checker" I need some example to look to, so that I can decide whether it's in the scope of this app.
Thank you @ZMJGMADHPXWT  for your request. Please check projects under the organization to get status updates and check releases. I am developing alone, so it can take time.

[ ] Traceroute for mobile devices (https://pub.dev/packages/flutter_traceroute)
[ ] Wake on LAN (https://pub.dev/packages/wake_on_lan)
[ ] IP subnet calculator
[ ] Server Checker (ICMP, TCP, HTTP, HTTPs)


Multiple routes per segment

close #3

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow for rotation to be set with SpriteText2D with extra parameter

Hello,
Regarding this stackoverflow question: https://stackoverflow.com/questions/23508376/how-to-rotate-a-sprite-object-in-three-js
I have noticed I couldn't modify the rotation of a sprite after it has been created. Would it possible to pass an extra parameter for the rotation?

Is there any way to do this? I tried all of these but no success:
    sprite.rotation.set(new THREE.Vector3( 0, 0, Math.PI / 3));
    sprite.rotateX(Math.PI / 3);
    sprite.rotation.setFromVector3(new THREE.Vector3( Math.PI / 3, 0, 0));

@dashko
https://threejs.org/docs/#api/en/geometries/TextGeometry
https://discourse.threejs.org/t/create-3d-text-at-runtime/2045/2


Beta release

Is there any more detail on when this connector will be developed further into a production ready state? Not much point providing a connector if there is no use for it in a production environment.

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

ID: 0f04a295-e4b9-a23b-02e7-232aab5db7ce
Version Independent ID: cbcadc03-205b-c383-fd3d-7aa4d763f122
Content: Copy data from Xero using Azure Data Factory (Beta)
Content Source: articles/data-factory/connector-xero.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@xmark
Thanks for the feedback! We are currently investigating and will update you shortly.

thanks @jason-j-MSFT looking forward to hearing back on this, as it seems like a very useful connector

Is there any update on this @jason-j-MSFT

@linda33wj do you have any information as to when the Xero connector will be out of the beta state?
@xmark we will do our best but we may not have an ETA for you at the moment.

@xmark For “beta” connectors, we keep monitoring the customer usage and feedback, and evaluate if any improvement required on functionality/experience or graduate it to normal connector. For Xero specifically, we don't have ETA yet but the same principal applies. I saw you are trying it out and our engineer is helping out on your other questions/issues.
#please-close


Artist alias redirect pages have the wrong artist in the title

It should be "moved to (aliased artist)", not "moved to (alias)"!
(12 Mar 2022)

Fixed in e8d048b093021630b71b38095b1375d464a93571! (12 Mar 2022)


fix: Revert utils to builder change in adapter-static 🐛

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Fixes #3135
Here's the output of console.log(builder). You can see that the utils attribute is what we want to use in this function.
> Using @sveltejs/adapter-static
{
  utils: {
    log: [Function: log] {
      success: [Function (anonymous)],
      error: [Function (anonymous)],
      warn: [Function (anonymous)],
      minor: [Function: noop],
      info: [Function: noop]
    },
    rimraf: [Function: rimraf],
    mkdirp: [Function: mkdirp],
    copy: [Function: copy],
    copy_client_files: [Function: copy_client_files],
    copy_server_files: [Function: copy_server_files],
    copy_static_files: [Function: copy_static_files],
    prerender: [AsyncFunction: prerender]
  },
  config: {
    preprocess: {
      defaultLanguages: [Object],
      markup: [AsyncFunction: markup],
      script: [AsyncFunction: script],
      style: [AsyncFunction: style]
    },
    extensions: [ '.svelte' ],
    kit: {
      adapter: [Object],
      amp: false,
      appDir: '_app',
      files: [Object],
      floc: false,
      host: null,
      hostHeader: null,
      hydrate: true,
      package: [Object],
      paths: [Object],
      prerender: [Object],
      router: true,
      serviceWorker: [Object],
      ssr: true,
      target: '#svelte',
      trailingSlash: 'never',
      vite: [Function: input]
    }
  }
}
> builder.rimraf is not a function
TypeError: builder.rimraf is not a function
    at adapt (file:///Users/USER/gitrepos/project/node_modules/@sveltejs/adapter-static/index.js:15:12)
    at adapt (file:///Users/USER/gitrepos/project/node_modules/@sveltejs/kit/dist/chunks/index4.js:485:8)
    at file:///Users/USER/gitrepos/project/node_modules/@sveltejs/kit/dist/cli.js:910:11

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

I'm not able to run the tests though, I don't have uvu installed? I did run pnpm install but that doesn't seem to do it. I'm a little new to node, so I might be doing this wrong.
$ pnpm test

><EMAIL_ADDRESS>test /Users/USER/gitrepos/sveltekit
> pnpm -r test --workspace-concurrency=1

Scope: all 14 workspace projects

><EMAIL_ADDRESS>test /Users/USER/gitrepos/sveltekit/packages/kit
> npm run test:unit && npm run test:packaging && npm run test:integration

><EMAIL_ADDRESS>test:unit
> uvu src "(spec\.js|test[\\/]index\.js)" -i packaging

sh: uvu: command not found
/Users/USER/gitrepos/sveltekit/packages/kit:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL<EMAIL_ADDRESS>test: `npm run test:unit && npm run test:packaging && npm run test:integration`
spawn ENOENT
 ELIFECYCLE  Test failed. See above for more details.

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Thank you, but this isn't the correct fix. The latest version of Kit does provide builder rather than utils to the adapters.


Add file pasting

It should be possible to paste a file into the website (just like how imgur.com does). These files get handled just like any other.
Expected features

After the website opens you should directly be able to ctrl+V a file into the page (no click required)
These pasted files could be literal copied images (from other sources) or copied from the file explorer
These files should be handles just like any other file

This doesn't have to be too hard. A simple implementation could be:
document.addEventListener('paste', function (event) {
    if (event.clipboardData && event.clipboardData.items) {
        for (let i = 0; i < event.clipboardData.items.length; i++) {
            let item = event.clipboardData.items[i];

            // Check if the item is a file
            if (item.kind === 'file') {
                // Get the file as a File object
                let file = item.getAsFile();

                // Use the File object
                if (file) {
                    handleFile(file);
                }

                // Multiple files? Try zipping them?
            }
        }
    }
});

At the end the "Multiple files? Try zipping them?" is a reference to #11


example model can not be opened

          im also having this issue. How was it solved?

Originally posted by @yuriypodvalny in https://github.com/t-wolfeadam/AnyLogic-Pypeline/issues/40#issuecomment-2112595172

Can you share which example model youre trying to open? Share the sources for the example model if you can. Also please share the specific error message you're getting (a screenshot would be great)

i cant reproduce it. I suspect i was trying to open the model before unzipping it. So you can close this issue.


fix: correct consumers handling when custom_id is used

Right now, when a consumer has the username equal to the custom_id of another consumer, decK fails because of the way the consumers are handled in the internal in-memory DB.
_format_version: "3.0"
consumers:
- custom_id: foo
  id: ce49186d-7670-445d-a218-897631b29ada
  username: Foo
- custom_id: bar
  id: 7820f383-7b77-4fcc-af7f-14ff3e256693
  username: foo

$ deck sync
Error: inserting consumer into state: inserting consumer Foo: entity already exists

This commit fixes this defect by modifying the way consumers are looked up in the in-memory DB.

Codecov Report
Patch coverage: 40.00% and project coverage change: -0.04% :warning:

Comparison is base (24ef372) 33.71% compared to head (5743626) 33.68%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #986      +/-   ##
==========================================
- Coverage   33.71%   33.68%   -0.04%
==========================================
  Files         100      100
  Lines       12006    12018      +12
==========================================
  Hits         4048     4048
- Misses       7564     7574      +10
- Partials      394      396       +2

Files Changed
Coverage Δ

file/writer.go
14.53% <0.00%> (ø)

state/builder.go
0.00% <0.00%> (ø)

file/builder.go
48.29% <33.33%> (ø)

state/consumer.go
71.79% <45.00%> (-8.21%)
:arrow_down:

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


Notification of a new working yaml for Atom Beta I just created

Hi,
I thought you might wish to add an AppImage for the latest beta release of Atom to your bintray repositories. Here is my yaml:
app: Atom-Beta

ingredients:
  script:
    - TAR=$(wget -q https://github.com/atom/atom/releases/ -O - | grep "tar.gz" | grep beta | grep href |  head -n 1 | cut -d '"' -f 2)
    - VERSION=$(echo $TAR | sed 's|/atom/atom/releases/download/v||g' | sed 's|/atom-amd64.tar.gz||g' | sed 's/-beta/./g')
    - wget -c https://github.com/$TAR
    - echo $VERSION > VERSION
    - tar zxvf atom*tar.gz

script:
  - cp -r ../atom-*/* usr/bin/
  - find . -name atom.png -exec cp {} atom-beta.png \;
  - cat > atom-beta.desktop <<EOF
  - [Desktop Entry]
  - Type=Application
  - Name=Atom Beta
  - Icon=atom-beta
  - Exec=atom %u
  - Categories=Development;TextEditor;GTK;
  - Comment=The hackable text editor for the 21st Century - beta channel.
  - EOF

It builds and runs perfectly.
Thanks for your time,
Brenton

Did you request an official AppImage from upstream yet? :-)

Did you when you created the Atom AppImage? :p Besides in atom/atom#4980 AppImage were mentioned, but clearly as you provide an Atom AppImage unofficially they're not that interested in created one themselves, officially.

Still would be nice to have you chime in there if you'd like to see an AppImage. If many users request one, maybe it will happen one day?

I kinda doubt it, but there it is I've done it. I personally think they'll close it soon saying that their tarballs are good enough.


[FLINK-5381][web-frontend] Fix scrolling issues

This also fixes Flink-5359, Flink-5267. These should also be marked resolved.
While working on this, I also observed scrolling issues on practically every page with content that doesn't fit in one browser window height. Those are also fixed now.

I'm trying out these changes now.

Changes look good, adding them to my next batch.


Add team members to db seeds

Bug Report
When running the backend locally, the staff are not being rendered on the team page.
What is the expected behavior?
Staff / Volunteers are displayed

Since we used a rake task to populate production I think the easiest way to resolve this is to create some entries in seeds.rb.

What's the status on this? Is this something that can be done with some clear instructions? I'd love to see my name on the list, and I'd be happy to tackle this with some direction.

Is this one still open? @sethbergman I'd like to try this one did  you get any instructions?

@sabrown84 - To get you rolling, here are the two files that are being discussed:

seeds.rb
team_member.rake

Ping me if you have any questions!

Any movement on this issue? If not, I'd love to get it going.

Hi @coloradojay,
Sorry I had some stuff come up. I haven't made much head way. I am happy to pass the torch to you on this one. Enjoy.


[Android 13] forward-all-clicks not more working when screen in landscape

Environment

OS: Windows
scrcpy version: v1,25
installation method: Windows release (connection USB)
device model: Nokia XR20
Android version: 13

Hi,
I am using scrcpy for years and use --forward-all-clicks when connecting to my home computer from work, with moonlight.
It was working fine when my phone was using Android 12.
It updated itself  last week to android 13, and since --forward-all-clicks behavior have change:

when in portrait mode, it works as expected, all mouse click are passed
when in lanscape, which is forced when starting moonlight, mouse click are passed only on part of the screen.  I reproduced the problem on youtube app.

When rotating, application logs show:
INFO: New texture: 1920x864
when in lanscape, it seems that I am able to do mouse click until reaching the 864 pixel on the right, supposing that the reference is 0 is at the bottom left.
If I maintain the left mouse buttom down, I can see that the pointer is correctly moving through the whole screen, so this is only affecting mouse click, not mouse position.
If I remove the forward-all-clicks, I don't have this issue, it's working as expected.
I can also use OTG mode without issue.
I tried to force rotation without success.
I can make a test from a linux computer if you want.

Thank you for your detailed report.
See #3598.

Ok, I'll wait for a firmware update and see.
Currently i'm running:

security: 5 december 2022
Google play: 1 January 2023
Kernel: 5.4.191

I may need to wait for few month as the update was made available to my phone last week.

Hi,
got a security update today: February Security update 2023. Lanscape still bugged when using forward-all-clicks.
security: 5 february 2023
Google play: 1 January 2023
Kernel: 5.4.191
I may need to wait for a Kernel update, not just a security patch.

same error in my devices.
Environment

OS: ChromeOS
scrcpy version: v2.0
installation method: manual ninja install (connection USB)
device model: Vivo IQoo neo 6
Android version: 13

Hi,
Got a firmware update today:
security: 5 april 2023
Google play: 1 April 2023
Kernel: 5.4.210
I still have the issue even with a firmware and kernel update using lastest scrcpy version.
I tried 1.25 version and the behavior is the same as the 2.0.
I then tried 1.23 and 1.24, and with those I can click on all the screen in landscape, but the right-click don't work as corriged in #3568
This behavior has changed in 1.25:

Fix click behavior when --forward-all-clicks is set (#3568, #3579)

Is this possible that the api changed between android 12 and android 13 ?

Got it working with display-overlay, so if your device does support it, try:
adb shell settings put global overlay_display_devices 1680x1050/120
The number after / is the DPI/PPI ratio, if you set it wrong - nothing will happen, and no display will appear on your phone. If adb says you are missing WRITE_SECURE_SETTINGS permission - turn off the device permission monitoring (beware of risks)
Then you can provide the display number as scrcpy --display=2 (you will get the list of displays if you put the wrong number).
To turn off the display afterwards, run this command:
adb shell settings put global overlay_display_devices none
Scroll is working now, but for me, the problem with --forward-all-clicks is that mouse position change is not tracked by the game until i click with it, so i stick to the -M mode


Create a Contact us pg

Is your feature request related to a problem? Please describe.
Create a Contact us pg
Describe the solution you'd like
Create a Contact us pg. where if user click that page then it will redirect them to that contact us webpage
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

please assign me this issue

This issue is assigned to someone already. #115


Pylint website not available

Bug description

Github pages on CI fails
pylint website is not available: https://scicompmod.github.io/memilio/pylint/index.html

Version
Github (CI, Actions, etc.)
To reproduce
Run CI
Relevant log output
No response
Add any relevant information, e.g. used compiler, screenshots.
No response
Checklist

[X] Attached labels, especially loc:: or model:: labels.
[X] Linked to project

Pages are back. This was just a temporary error.


./routes/[id]/[path].tsx catches request for _frsh/refresh.js

Should we move this block up?
https://github.com/lucacasonato/fresh/blob/026ed86f4009fc0a35463d43218bf8c34aead98e/src/server/context.ts#L395

@kuboon Oh yeah, that would be great. We should move the INTERNAL_PREFIX handlers above everything else.


fix: e2e test

Description
After clicking start bots button, the start bots panel will open automatically if it is failed to start bots. If click it again, the panel will close. Then See Details text cannot be seen.
Task Item
#minor
Screenshots

Thanks for looking at this, but @liweitian can you describe more about what the issue is?

Coverage remained the same at 54.55% when pulling 8d7990898f69f109fef4185faf0e3ef0fe8fb956 on liweitian:fixe2etest into 57e5aaf8b2e235bd4e7a1c2b42e98a5e6ad2a3bc on microsoft:main.

Great, really appreciate it


Expand Light Stemcell Publishing

Publish CentOS Light Stemcells 3363.x & 3312.x

 Publish Alpha Light Ubuntu Stemcells

 Use canonical version of bosh-io-stemcell, not our fork

Remove ssh_tunnel config as gcp cpi does not use registry

We're merging this — hope that's okay — because the changes are scoped to our CI.


iot2050: Migrate trusted-firmwar-a to git

Snapshot of trusted-firmware-a (TFA) is no more accessible, so move to git usage.
Unable to fetch lts-v2.8.10 so, version updated to 2.9.0

@jan-kiszka, Could you please review the changes.

Thanks for the build fix, but:
A) Don't mix changes into a single commit.
B) We chose LTS for a reason, see https://github.com/siemens/meta-iot2050/commit/c2ff5a8569a43c90df60338f9b095867fdb49dee.
C) Let's just move to https://github.com/ARM-software/arm-trusted-firmware so that we can preserve tarball fetching.

Updated the changes with fetching the taball of version lts-v2.8.10 from arm-trusted-firmware .
Please review the changes.

You are still changing the version - don't do that.

You are still changing the version - don't do that.

Ok, maintained the version, thanks.

Hi @jan-kiszka, could you please merge the PR.

Merging as this is breaking our builds for too long now.


change label and add completedBy field in imaging page

Fixes #1033 .
Changes proposed in this pull request:

Change labels of Result into Result Notes and Notes into Requested Notes
Add new field completedBy in imaging models as well as on the Edit page of imaging

cc @HospitalRun/core-maintainers

I did not make the completedBy fields in Edit page of imaging a populated field, because I don't know what kind of user will complete it (imaging technician, doctor, or radiologist). However, if we can make it a populated field as requested in issue #1033 , what kind of user should I populated it with?

@baoqchau completed by should be set to the user who clicks on the complete button on the request.  If you look at how requestedBy is populated in the code you should be able to do the same for completedBy (eg completedBy is current user when complete button is clicked).

I add Completed By column in completed page of imaging, which will be easier to track down who completed that request.

I added Completed By column in Completed page of Imaging, which make it easier to track down who completed that imaging request.


Server-side rendering example

Hello and thank you for a great library.
I'm going through the documentation and I cannot find a guide to have server-side rendering set up with navi. I'd like to help add one once I gain enough knowledge about the library's internals.
I've got a working SSR with react-router and when using navi I get the following exception on the server side:
ReferenceError: window is not defined
    at Object.createBrowserNavigation (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/navi/dist/commonjs/BrowserNavigation.js:26:58)
    at new Router (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/react-navi/dist/commonjs/Router.js:42:55)
    at processChild (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/react-dom/cjs/react-dom-server.node.development.js:2846:14)
    at resolve (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/react-dom/cjs/react-dom-server.node.development.js:2812:5)
    at ReactDOMServerRenderer.render (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/react-dom/cjs/react-dom-server.node.development.js:3202:22)
    at ReactDOMServerRenderer.read (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/react-dom/cjs/react-dom-server.node.development.js:3161:29)
    at renderToString (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/react-dom/cjs/react-dom-server.node.development.js:3646:27)
    at app.get (webpack:///./src/server/middleware/ssr/index.js?:42:96)
    at Layer.handle [as handle_request] (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/express/lib/router/layer.js:95:5)
    at next (/Users/artemzakharchenko/Documents/oss/unstack/node_modules/express/lib/router/route.js:137:13)

I guess that's expected when creating browser navigation on the server :)

Expected behavior
It would be great to have a contextless example of server-side rendering using navi. I would expect such example to include:

Explanation of steps required to perform SSR.
Code examples of SSR using client React application with navi as routing, and simple express (or other) NodeJS server that would be rendering our application.

Please let me know if I missed something, or can help with anything.

I'm not 100% sure that the API is rock solid yet, so I'd like to hold off on proper documentation for a little longer. That said, the general idea is to just use createMemoryNavigation() instead of createBrowserNavigation().
Here's the docs for createMemoryNavigation().
For an example of how it all fits together, take a look at the static rendering guide. Server rendering follows a pretty similar approach, except that instead of using navi-scripts/register, you'll need to have your build system produce a bundle targeting node, ala my Create React App SSR PR.

Thanks for reply. Yeah, I've figure out some of the pieces already based on the static build. Yet it's confusing not to have a dedicated SSR section. Looking forward when the API is solid :) Let me know if I can help with anything.

I'm really hoping to get the CRA SSR PR merged before finishing off Navi's SSR story. Or at least get a fork available based on the current PR -- the reason being that the build system is at least half the story to get it working right.


Extract RequestAdapter#subcollection?

By providing an authoritative place for this knowledge, we can avoid
passing the is_subcollection boolean around through a number of methods.
The downside to this approach is that it increases coupling on global
data i.e. @req - but in a lot of places it is already depending on
this data.
@miq-bot add-label api, refactoring
@miq-bot assign @abellotti

@abellotti I've added a test to verify that the rendering of subcollection actions is preserved in this refactoring. While pushing back the boundary of what I was changing, I was still able to salvage some of this work, which will help in simplifying the primary action logic in a follow up. LMK I can do anything else!

Thanks for the update, safer change than previous iteration. LGTM!!


Check compatibility Alasql with Cordova

Goal: Check, if Alasql can work with Cordova
Questions:
a) If Alasql core functions work?
b) Which persistence databases work with Alasql and Cordova?
Results
a) Alasql works with Cordova (at least in browser and iOS)
b) Alasql can work with localStorage (at least in browser and iOS)

New Cordova test results:
a) Alasql works with iOS emulator under OSX
b) Alasql works with iPhone

Now Alasql works with Windows 8 platform

we still have an "undefined" under ionic/cordova:
   error: ReferenceError: Can't find variable: alasql

Same issue is posted and not answered here:
http://stackoverflow.com/questions/31892503/cant-find-variable-alasql-in-phonegap-app-for-ios-alasql-0-2-0
works under android, only iOS is causing problems. Any suggestions?

Thanks for getting back to us.
@agershun - do you have any idea?

I also show me the following error under phonegap 5.3.9.
Is there any hint to fix the problem.
Thanks a lot!
ReferenceError: Can't find variable: alasql``

@hkcity1111 How do you include the library?


Generate End Main Island (Suggestion)

When going to the end there is no pillars, so the ender dragon is easier to defeat, supposing you find the structure where he perches, since in some cases you spawn over the void and it's difficult to find.
It would be neat if it generated properly, it could be like a big group of hexagons where the main island is generated, however I guess it wouldn't be something easy to do.

It is very easy to do. In fact, you can do it yourself with a datapack, even just by changing one file. However, I have to get into the reason they don't spawn:

End spikes (pillars) only spawn in "The End" biome.
"The End" biome only appears in a small radius around the origin of the dimension.
In all dimensions, Hex Lands shrinks the realitive size or scale of biomes, in order to reduce the number of adjacent hexes with the same biome - as otherwise this looks very boring and not very hexagonal at all.
The end, in vanilla, has five biomes. And without this scaling effect, it barely looks hexagonal.

So, the root cause of the issue has this as an (un)intentional side effect. The end has a rather unique biome setup, and I'm not convinced I should preserve that in Hex Lands.

In the case of wanting to do it with a datapack (which I surely want to do) or changing that file, how do I make that datapack or what file I need to change?

Two ways:

Override the End dimension, exactly as described in the relavant readme section, except change the biome_scale value to 1 (or another smaller value than the default (4) which gives enough space for the end pillars to spawn). This will have the aforementioned effects on overall biome placement in the End.
Add the end spikes feature to other end biomes (not just "the end", but also any end biomes that may spawn nearby - including all the vanilla end biomes, but also any modded end biomes you've added.). This one gets tricky as you have to override the entire biome to add the feature (it's rather inconvenient like that)


Renamed "Portuguese" to "Brazilian Portuguese"

The ISO 693-1 code pt_BR used in this website corresponds to Brazilian Portuguese specifically, not just Portuguese in general. And, indeed, by observing the language used throughout the website, the variant of Portuguese used corresponds to, indeed, Brazilian Portuguese.
Thus, it would make more sense to name the language in the menu accordingly, mostly for 2 reasons:

In loads of software, when the language states just "Portuguese", it is actually referring to European Portuguese (pt_PT), this should eliminate that redundancy.
It opens the door for the support of European Portuguese (pt_PT) in the website.

@GilOliveira can you share the reason for closing this PR?

I don’t think I was the one who closed it

Awesome @tupaschoal! Thanks for the merge! I already had a fork with some pt-PT translations, will be happy to contribute them soon!


feat: Next.js@13 support

Added support for next@13 to @pankod/refine-nextjs-router
Test plan (required)
WIP
Self Check before Merge
Please check all items below before review.

[x] Corresponding issues are created/updated or not needed
[ ] Docs are updated/provided or not needed
[x] Examples are updated/provided or not needed
[x] TypeScript definitions are updated/provided or not needed
[ ] Tests are updated/provided or not needed
[x] Changesets are provided or not needed

Bumped Next.js to 13
Added support for experimental appDir option in next.config.js to allow for the latest Next.js features.

pages directory
Current support for pages directory has not changed and will continue to work as before. It will be supported as long as Next.js continues to support and prompts it as the stable way of working with Next.js.
appDir option
appDir option is a new experimental feature in Next.js that introduces a bunch of new features. It is currently in beta and is not stable. It is not recommended to use it in production. But can be used alongside the current pages directory support.
To use appDir option, you need to add it to your next.config.js file.
// next.config.js
module.exports = {
    /* ... */
    experimental: {
        appDir: true,
    },
};

Using appDir with refine
We've needed to make some changes to the @pankod/refine-nextjs-router to make it work with the current structure of the app directory feature. To make sure these do not break the current support for pages directory, we've added a new exports at the sub path @pankod/refine-nextjs-router/app that can be used with the appDir option.
Note
To make optional catch-all routes to work with the app directory, you need to define them as directories unlike the option of defining them as files with pages directory.
You need to use NextRouteComponent from @pankod/refine-nextjs-router/app instead of NextRouteComponent from @pankod/refine-nextjs-router when using appDir option.
Inside your layout file, you need to bind params to routerProvider to make sure @pankod/refine-nextjs-router can work properly with the new structure.
// app/[[...refine]]/layout.tsx
"use client";

import routerProvider from "@pankod/refine-nextjs-router/app";

const Layout = ({ children, params }) => {
    return (
        <Refine
            routerProvider={routerProvider.apply({ params })}
            /* ... */
        >
            {children}
        </Refine>
    );
};

Warning
Please note that, unlike the routerProvider from the @pankod/refine-nextjs-router, routerProvider from @pankod/refine-nextjs-router/app is a function and you need to bind params to make it work properly.
// app/[[...refine]]/page.tsx

"use client";

import { NextRouteComponent } from "@pankod/refine-nextjs-router/app";

export default NextRouteComponent;

Warning
You need to add "use client"; directive to both layout.tsx and page.tsx inside app/[[...refine]] directory.


Replace pulp with ansible-wormhole

In the README, there are a few places where pulp is written where ansible-wormhole should be.

Done.


Freeze the package versions based on the ones from nixpkgs

This prevents the plain Cabal and the Nix builds from using different versions. This is especially important when using Nix, as Cabal may start using more up to date versions of the package from Hackage if the user has run cabal update  before and a local package database exists. Cabal can also be forced to use only the global package registry in which case this problem also doesn't exist, but that requires either breaking non-Nix builds or wrapping the cabal command.
I also added the freeze files to the Nix builds so we'll get alerted if nixpkgs gets updated without also updating the freeze file.

@OpsBotPrime merge

Pull request approved for merge by @robbert-vdh, rebasing now.

Rebased as 2cd40a3da62eae63f24edb3c752c4a13c05416cd, waiting for CI …

CI job :yellow_circle: started.


If a child mesh from an AssetContainer is added to the scene without …

…its parent, call setParent on it.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14123/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14123/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14123/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14123/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14123/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14123/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14123/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14123/merge#BCU1XR#0


Feature request: Example

Hi, thanks for open-sourcing your code! Do you have a video or videogame example? I'm curious what the physics look like and play like.

This asset is based on the physics I use in my project here: Jack of all Cars. Also, there are 4 different examples included that handle very differently.

Oh, great! Thanks!


Can you mark a pipeline a sucess even if you have a failing step?

I'm curious if its possible to mark a pipeline step as not important enough that it flags the whole build pipeline as a failure it it does happen fail? I know you can mark step success as optional and keep the build going but we've got a linting checker step that we're developing which just happens a on a small section of our project and I'd like to be able to iterate on that step without marking our daily builds as complete failures as I do so.
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/stages?view=azure-devops&tabs=yaml
Thank you for your feedback!
The issues in this repo are for documentation feedback.
If you are submitting your documentation feedback issue directly through GitHub, please provide the URL of the article so we can route the feedback to the correct author.
Please report Azure DevOps or TFS software bugs, feedback, or questions in the Azure DevOps Developer Community.

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


[Feature Request] Support for TypeScript 3.x project references

Feature request
Support for TypeScript 3.0 project references.
Context
TypeScript 3.0 now supports project references, e.g. a packages/tsconfig.json that looks like:
{
    "files": [],
    "references": [
        {"path": "./subproject1"},
        {"path": "./subproject2"}
    ]
}

Then a packages/subproject1/tsconfig.json that looks like:
{
    "references": [
        {"path": "../subproject2"}
    ]
}

Effectively making monorepos far more convenient to work with, with a simple tsc --build of the root tsconfig.json capable of building the whole set of modules.
Problem
Right now, applying tslint on such a root tsconfig.json produces the following error:
error TS18002: The 'files' list in config file 'tsconfig.json' is empty.

Solution
The empty files is no longer a constraint in TypeScript 3.0, but removing the constraint in tslint would not help much anyway, since there would be no file to validate.
A nice solution would be if tslint could walk through the references and execute again each tsconfig.json it encounters.
Alternative / Workaround
For the time being, it is possible to simply have a tslint script in the package.json of each subprojectN, then lerna execute the command on all of them.

Note: I might have some time next week to work on a PR, if no such thing has been started yet.

Is there a temporary workaround for people like me who are using Typescript composite projects already? Getting error TS18002: The 'files' list in config file 'tsconfig.json' is empty.

We also just converted our monorepo to typescript 3.0 with project references and yarn workspaces and are running into the same problem. Our workaround is to add a tsconfig-lint.json in the root of our repo that reference all the appropriate src files to be linted. Of course this would only work if you have a consistent directory structure for all your modules and using the actual project references would be ideal.
{
  "include": [
    "apps/*/src/**/*.ts",
    "modules/*/src/**/*.ts"
  ],
  "compilerOptions": {
    "composite": true,
    "lib": [
        "dom",
        "es2017"
    ],
    "module": "commonjs",
    "target": "es2017",
    "strict": true,
    "esModuleInterop": true,
    "noImplicitAny": true,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": true
  }
}

By now, this is out of scope. 😿 See #4534.

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Reorganized cluster roles, added rules to watch all namespaces

This PR changes the cluster role to include all the rules from the regular role. Unfortunately, we can't compose roles, as the operator-sdk will either bind a service account for the cluster role, or to a role. It cannot bind a service account to both a role and a cluster role (!!).
Related to the finding above, this PR also fixes a problem that went unnoticed with the CSV, which listed a different service account under clusterPermissions.
Fixes #931.
Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>

@kevinearls would you be able to test this?

@jpkrohling Sure I'll test this shortly

Would you like to give it a try as well?


Issue about highly variable genes

I notice that in the the function of Preprocessor, the number of genes is limited to the n_hvg.
However, I think the scanpy calculation cannot represent biological significance. Thus, I want to learn more about the selection of this parameter and what you think of it.
Besides, if the downstream task such as cell type annotation, perturbation prediction and cell generation are also finished using the highly variable genes.

Hi, I am sure I fully understand your question. If your question is about the efficacy of the scanpy hvg computation, I think it has been widely used in single-cell community. If you have a specific alternative, I am willing to have a try.
In the meantime, the script we provided serves as an example, and I believe scGPT would support alternative preprocessing configs as well, such as alternative numbers or selection of hvgs. Just keep in mind, a higher number of hvgs would consume more GPU memories, so you might want to balance with other configs such as the batch size.
We also plan to release memory-efficient fine-tuning in the future, please stay tuned.


drupe fruit (PO:0030103)

drupe (PO:0030103)
Proposed def'n:
A fruit (PO:0009001) derived from a single carpel (PO:0009030) and is composed of one or more seeds (PO:0009010) surrounded by fleshy exocarp (PO:0009085) and mesocarp (PO:0009087), and a stony endocarp (PO:0009086).
Proposed Comment:
May develop from an inferior or superior syncarpous gynoecium (PO:0009062)(Simpson, 2010). Species whose fruit serve as examples of this fruit type include: Amborella trichopoda, Coffea canephora (coffee), Davidia spp. (Cornaceae), and Prunus spp. (peach and plum).
Synonym: stone fruit (exact)
see Fruit #76

Comments from @jaiswalp:

add comment about cashew- not a nut, but a drupe
Suggest renaming to "drupe fruit"

Revised drupe (PO:0030103) to drupe fruit
Previous def'n:
A fruit (PO:0009001) derived from a single carpel (PO:0009030) and is composed of one or more seeds (PO:0009010) surrounded by fleshy exocarp (PO:0009085) and mesocarp (PO:0009087), and a stony endocarp (PO:0009086).
Revised def'n:
A fruit (PO:0009001) which develops from a carpel (PO:0009030), and comprises a fleshy exocarp (PO:0009085), fleshy mesocarp (PO:0009087), and a stony endocarp (PO:0009086) with one or more seeds (PO:0009010).
Previous comment:
May develop from an inferior or superior syncarpous gynoecium (PO:0009062)(Simpson, 2010). Species whose fruit serve as examples of this fruit type include: Amborella trichopoda, Coffea canephora (coffee), Davidia spp. (Cornaceae), and Prunus spp. (peach and plum).
Revised comment:
May develop from an inferior or superior carpel (PO:0009030). Examples: Amborella trichopoda (a basal angiosperm), coffee (Coffea canephora), Cornaceae (including Davidia spp.), and peach and plum (Prunus spp.).

Closing issue for drupe fruit (PO:0030103):
def'n: A fruit (PO:0009001) which develops from a carpel (PO:0009030) and at maturity comprises a fleshy exocarp (PO:0009085), fleshy mesocarp (PO:0009087), and a stony endocarp (PO:0009086).
comment: May develop from an inferior or superior carpel (PO:0009030) and may contain one or more seeds (PO:0009010). Examples: Amborella trichopoda (basal angiosperm), coffee (Coffea canephora), Cornaceae (Davidia spp.), cashew (Anacardium occidentale), coconut (Cocos nucifera), and almond, peach and plum (Prunus spp.).
synonyms: drupa (exact, Spanish), drupe (exact), steinfrucht (exact, German), stone fruit (exact), indehiscent fruit (broad)


Unsupported color function "oklch"

Need to add support for oklch colors

same issue in Chrome，it's ok in Safari

Need to add support for oklch colors

use https://www.npmjs.com/package/html2canvas-pro, it supports for oklch colors


e2e: vote-on-question scenario test not reliable

https://github.com/marklogic/marklogic-samplestack/blob/10361e91ed616fad7f319825edfa278acab89169/specs/features/vote-on-question.feature#L17 is not a reliable test.
It is tagged as broken until we determine why. It probably has to do with our promises not being properly chained, and thus the test driver thinks the action is complete when in fact there is still a promise being resolved.

I'll look into this issue.

Its possible this was fixed by some promise handling improvements already. First make aurw you can reproduce I think...

Verified the e2e test is working.  Will remove broken tag and create pull request.

Current behavior is that this test fails on first run, but succeeds on successive runs.  If your server sits idle for a while, the same behavior occurs... failure on first run.  My current thinking is that this test fails due to something to do with a "warm-up" on server having to do with transactions.   I'm creating some new tests that will also be testing app operations (like Accept) that also use transactions to see if similar behavior occurs.


Remove limit of 1 entry per filter

Summary
Hello, in the build_features function, there is currently a limit of only allowing 1 entry per filter.
Code snippet below.
def build_filters(json_col: Column, filters: Dict):
    """
    PRIVATE

    Builds filters for SQL query based on provided dictionary.

    Args:
        json_col (Column): The column in the database table.
        filters (Dict): The dictionary specifying filter conditions.

    Raises:
        FilterError: If filter conditions are not correctly formatted.

    Returns:
        The filter clause for the SQL query.
    """

    if not isinstance(filters, dict):
        raise FilterError("filters must be a dict")

    if len(filters) > 1:
        raise FilterError("max 1 entry per filter")

In my example, my filters are
{'agent_id': {'$eq': 139}, 'thread_id': {'$eq': 139}}

and filtering is done through llama-index.
This should be removed.
Rationale
The limit makes it significantly harder to build useful applications that leverage vecs/supabase as the storage of choice for the RAG datastore. This limitation is not documented anywhere and if I had read that you could only filter by one data point in your json metadata, I would have gone with another option. Being able to filter by multiple metadata fields when doing a vector search is something I and many others will need to be able to do for complex RAG systems.
Design
An dense explanation in sufficient detail that someone familiar with the
project could implement the feature. Specifics and corner cases should be covered.
Examples
Illustrations and examples to clarify descriptions from previous sections.
Drawbacks
What are the negative trade-offs?
Alternatives
What other solutions have been considered?
Unresolved Questions
What parts of problem space or proposed designs are unknown or TBD?

Actually after looking at this, it looks like the llama-index SupabaseVectorStore is the one that is causing this issue.
Specifically this code. It should be creating {"$and": [{"agent_id": {"$eq": 136}}, {"thread_id": {"$eq": xx"}}] instead of
{'agent_id': {'$eq': 139}, 'thread_id': {'$eq': 139}}
Specifically this function is the culprit.
    def _to_vecs_filters(self, filters: MetadataFilters) -> Any:
        """Convert llama filters to vecs filters. $eq is the only supported operator."""
        vecs_filter = {}
        for f in filters.legacy_filters():
            vecs_filter[f.key] = {"$eq": f.value}
        return vecs_filter

I was able to fix this by creating my own VectorStore and using the following function to correctly handle metadata queries with multiple values. I'll make a PR request to llama-index. Thanks!
 def _to_vecs_filters(self, filters: MetadataFilters) -> Any:
        """Convert llama filters to vecs filters. $eq is the only supported operator."""
        vecs_filter = {}
        if len(filters.filters) == 0:
            for f in filters.legacy_filters():
                vecs_filter[f.key] = {"$eq": f.value}
        else:
            filter_cond = "${}".format(filters.condition.value)
            vecs_filter[filter_cond] = []
            for f in filters.filters:
                sub_filter = {}
                sub_filter[f.key] = {"$eq": f.value}
                vecs_filter[filter_cond].append(sub_filter)
        return vecs_filter

I'll make a PR request to llama-index

thanks! much appreciated


BrokenPipeError: [Errno 32]

Hi,
I followed this tutorial and trained a de-bar model: https://github.com/awslabs/sockeye/blob/main/docs/tutorials/wmt.md
However, I encountered a Broken Pipe error during translation. My command was:
echo "ich weiss es nicht ." |   python -m apply_bpe -c bpe.codes --vocabulary bpe.vocab.bar                                    --vocabulary-threshold 50 |   python -m sockeye.translate -m de-bar-base-model 2>/dev/null |   sed -r 's/@@( |$)//g'
And the error I received was:
Traceback (most recent call last):   File "/opt/conda/lib/python3.9/runpy.py", line 197, in _run_module_as_main     return _run_code(code, main_globals, None,   File "/opt/conda/lib/python3.9/runpy.py", line 87, in _run_code     exec(code, run_globals)   File "/home/jovyan/nmt-sockeye-de-bar/sockeye/sockeye/translate.py", line 25, in <module>     import torch as pt ModuleNotFoundError: No module named 'torch' /home/jovyan/nmt-sockeye-de-bar/sockeye/subword-nmt/apply_bpe.py:393: DeprecationWarning: this script's location has moved to /home/jovyan/nmt-sockeye-de-bar/sockeye/subword-nmt/subword_nmt. This symbolic link will be removed in a future version. Please point to the new location, or install the package and use the command 'subword-nmt'   warnings.warn( Traceback (most recent call last):   File "/opt/conda/lib/python3.9/runpy.py", line 197, in _run_module_as_main     return _run_code(code, main_globals, None,   File "/opt/conda/lib/python3.9/runpy.py", line 87, in _run_code     exec(code, run_globals)   File "/home/jovyan/nmt-sockeye-de-bar/sockeye/subword-nmt/apply_bpe.py", line 446, in <module>     args.output.write(bpe.process_line(line, args.dropout)) BrokenPipeError: [Errno 32] Broken pipe Exception ignored in: <_io.TextIOWrapper name='<stdout>' encoding='utf-8'> BrokenPipeError: [Errno 32] Broken pipe
I have reached out to subword-nmt (https://github.com/rsennrich/subword-nmt/issues/117) and we concluded that this error might originate from sockeye.
Is there any way to counter this? Thank you!

It looks like PyTorch may not be installed: ModuleNotFoundError: No module named 'torch'.
If you have a local copy of the Sockeye code (git clone https://github.com/awslabs/sockeye.git), you can run pip install -r requirements/requirements.txt to install PyTorch and Sockeye's other dependencies. If not, you can run pip install torch.

Sorry, I inserted the wrong error (there I removed the 2>/dev/null in the execution just for testing). Torch was already installed.
The error was:
Traceback (most recent call last):   File "/opt/conda/lib/python3.9/runpy.py", line 197, in _run_module_as_main     return _run_code(code, main_globals, None,   File "/opt/conda/lib/python3.9/runpy.py", line 87, in _run_code     exec(code, run_globals)   File "/home/jovyan/nmt-sockeye-de-bar/sockeye/subword-nmt/apply_bpe.py", line 446, in <module>     args.output.write(bpe.process_line(line, args.dropout)) BrokenPipeError: [Errno 32] Broken pipe Exception ignored in: <_io.TextIOWrapper name='<stdout>' encoding='utf-8'> BrokenPipeError: [Errno 32] Broken pipe

Can you run the pieces of this command individually?
echo "ich weiss es nicht ." >input.tok
python -m apply_bpe -c bpe.codes --vocabulary bpe.vocab.bar --vocabulary-threshold 50 <input.tok >input.bpe
python -m sockeye.translate -m de-bar-base-model -i input.bpe -o output.bpe
sed -r 's/@@( |$)//g' <output.bpe >output.tok

The first two lines were successful. I received error from executing the 3rd line.
Executed:
python -m sockeye.translate -m de-bar-base-model -i input.bpe -o output.bpe
Got:
Traceback (most recent call last):   File "/opt/conda/lib/python3.9/runpy.py", line 197, in _run_module_as_main     return _run_code(code, main_globals, None,   File "/opt/conda/lib/python3.9/runpy.py", line 87, in _run_code     exec(code, run_globals)   File "/home/jovyan/nmt-sockeye-de-bar/sockeye/sockeye/translate.py", line 25, in <module>     import torch as pt ModuleNotFoundError: No module named 'torch'
Even though I am certain that torch was installed. If I add "2>/dev/null", I won't see this error, but I don't get output.bpe either.

it looks like the Python environment you are running sockeye.translate with does not have torch installed. Maybe you installed it to another environment? I would expect that running python -c 'import torch' would give you the same error.

I re-installed torch and it worked. Seems like I have to install torch every time I enter the environment so that sockeye.translate is able to run.
Thank you both for the clarification!


Can't access to Config or Jobs

Steps to reproduce

Run Parse 2.2.25-1 with clientKey and javascriptKey / node 4.5 / db on mLab 3.2.1 / Mac
Run parse-dashboard 1.0.19
Go to Config or Jobs sections

Expected Results
View Config or Jobs
Actual Outcome
Get {error: "unauthorized"} - but still can view and browse other classes
Environment Setup
Server

parse-server version 2.2.25-1 beta
Operating System:     Sierra 10.12.1
Hardware:             MacBook Air 11
Localhost or remote server? localhost and AWS checked

Database
mLab
Sandbox CLOUD: AWS eu-west-1
VERSION: 3.2.9 (MMAPv1)

Found the solution.
If you add a javascript key to Parse server you should start your dashboard with --config file and pass the javascriptKey too.


Is there a Java/Android SDK?

My current Android app uses https://github.com/ibm/cloudant-java-sdk and am considering converting to HPDB/MongoDB (perhaps with a little help from https://github.com/jimishio/couchdb-to-mongodb
Does a Java SDK exist?

Following. Interested in contributing the Java SDK if there's interest.

@RogerSnook @dannysheridan As far as I know, there's no plan to provide Java SDK currently. But you can use openapi-generator to generate Java SDK with the API doc below.
hyperp-dbaas-v3-oa3.yaml.txt

Have you checked out the code quality of the openapi-generator for Java? @sanmuny @RogerSnook