Add constraints to meta.yaml

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Resolves #187
Summary of Changes:

align with constraints on conda main channel
https://pandas.pydata.org/docs/dev/whatsnew/v2.2.0.html#increased-minimum-versions-for-dependencies

@conda-forge-admin, please rerender

@h-vetinari or @lithomas1 is there anything that I need to do to get this merged? I would prefer that it gets merged before another pandas version is released so that the run constraints will be added to it.

I'm not up to date with the extra-installs that pandas offers. The constraints are probably an OK fix, though it reduces the amount of legal environments that pandas can be installed in.
Better would be to create equivalent outputs here where e.g. pip install pandas[performance] is mapped to conda install pandas-performance. There are many examples of feedstocks that do this, check out for example ray or gymnasium.
But ultimately you'll have to figure this out with the maintainers of this feedstock, which doesn't include me; I'm just helping out here and there.

See https://github.com/pandas-dev/pandas/issues/52490

I agree that it will reduce the number of solves, but the ones reduced are only legal until you try to use a pandas function that relies on one of them.  My example in issue #187 is sqlalchemy 1.4 and pandas 2.2
This PR will at least align  conda-forge with the conda main channel.
While subpackages  could end up being the long term solution, there does not seem to be consensus around it yet.

I'm taking a break from pandas maintenance this summer.
I'll cycle back to this in a couple weeks, please ping if I don't.
(This looks correct at a first glance, though.)
Thanks.

Looking again, I think this just needs a bump of the build number.
(CI is red at the moment, so this wouldn't be mergeable until the next release anyways so maybe the bump is unnecessary and we just wait for the next release)

I am fine with waiting for the next release, which is why I didn't bump the build number. However, I will defer to your preference, so just let me know what you want me to do.

@conda-forge-admin, please rerender

To what extent do we want to add those constraints?
Because it can be a valid use case to use both pandas and sqlalchemy in the same project, without using the sqlalchemy integration in pandas. And in such a case, it is perfectly fine to use an older version of sqlalchemy than what pandas requires.
This might not be the majority of use cases, but AFAIU adding those constraints would make it impossible to create an environment in such a case?

It would stop them from using pandas>=2.2 with the older sqlalchemy. They could still use the older versions of pandas. That is a trade-off with this implementation.
And while they may be fine if they don't currently use the sqlalchemy integrations in pandas, as soon as they try to (eg pandas.read_sql) they will run into errors.

I think I was happy with the PR the way it is.
@jorisvandenbossche
Can you take another look at this PR?


RFC: Keyboard Shortcuts API

This is mostly based on empirical knowledge (and some inspiration from Visual Studio Code) rather than any scientific or user experience evidence, so please do chime in. I would love to hear from you. This will definitely iterate over many rounds of comments and will likely change significantly.
Being able to navigate across Backstage with keyboard shortcuts is more productive for regular users, as well as providing additional interfaces for impaired users. I have dabbled quite a bit with this in Spotify's internal use of Backstage, although it was never finished nor ever made it to production.
Concept
In this proposal, there will be two types of shortcuts: App Shortcuts and Plugin Shortcuts. Both will need to register a shortcut by sharing an unique ID, a function callback, and a preferred shortcut key. More importantly, registering a shortcut does not mean the shortcut will be enabled. Users will need to manually enable any shortcuts in Backstage for two reasons: a) for user education and b) to enable them to customize shortcuts from the very start to suit their flow (i.e. Vim, etc)
App Shortcuts
App Shortcuts are registered by the App (packages/app folder). This is usually those who deploys Backstage for their team/company. They will have the ability to register shortcuts for across Backstage, and will be for common tasks such as navigating to top-level views, creating new components, etc.
Plugin Shortcuts
Plugin Shortcuts are registered by any Plugin (plugins/* folders). They will be able to register a limited number of shortcuts (let's say 2 for now) in their plugin.ts manifest. The shortcut ID will be scoped against their plugin in someway (maybe: prepend the shortcut ID with plugin ID). The shortcuts will only be available within the context of their plugin.
Shortcuts UI
As mentioned, both App and Plugin Shortcuts will only register shortcuts. No shortcuts will be enabled by default with this API. Users can navigate to a Shortcuts UI where they can view all available keyboard shortcuts, their preferred shortcut (along with other possible metadata such as descriptions), and the ability one-click enable/customize them. Plugin Shortcuts will only work within the context of their plugin. The user's settings will be in persisted storage.
Possible Implementation
Here's some psuedocode for inspiration. Although mostly up to your imagination :)
const shortcuts = new ShortcutsBuilder();
shortcuts.registerGlobal('docs', () => history.push('/docs'),  { preferred: 'cmd+d' });
builder.add(shortcuts.build());

import { createPlugin } from '@backstage/core';
import WelcomePage from './components/WelcomePage';

export default createPlugin({
  id: 'welcome',
  register({ router, keyboardShortcuts }) {
    router.registerRoute('/', WelcomePage);

    keyboardShortcuts.register('welcome.reload-page', { preferred: 'cmd+d' });
  },
});

import React from 'react';
import { onKeyboardShortcutPress } from '@backstage/core';

export const ExampleComponent: React.FC<{}> = () => {
  onKeyboardShortcutPress(({ shortcutId, combination }) => {
    // do something here
    if (shortcutId === 'welcome.reload-page') {
      window.location.reload();
    }
  });

  // or

  onKeyboardShortcutPress('welcome.reload-page', () => {
    window.location.reload()
  });

  return
Hello world
;
});

Possible Considerations

Double key presses. How could we handle double key presses, or do we want to at all?
Limit which keys/combinations can be enabled. From my experience building out a prototype of this, there were some keys that were a no-no. Maybe it would be great if Backstage could have an "index" of key combinations that plugins can use to avoid any common/obvious keyboard shortcut pitfalls?
Limit how many shortcuts plugins can register. This would be good to keep the number of shortcuts at a minimum at first, and over time, being able to scale this out as the UI develops. The purpose is not to shortcut everything, but only a few hand-selected tasks.
App Shortcuts Defaults. Do we want to have an out-of-the-box set of keyboard shortcuts (i.e. Cmd+D) to go to the Docs view?
Ability to share configurations. Do we want to enable exporting of a user's keybindings (maybe in a JSON format?), it might be useful in an organizational-wide use case (unlikely to be useful in the greater open source community). An example might be having a Vim-like bindings configuration within Spotify's Backstage instance.

@backstage/core any comments/feedback?

@spotify/backstage-core any comments?

Thanks for the RFC!
Gonna preface this with that this is going to be pretty low prio as we don't really see a need to get this shipped early on and there are many other APIs that we'd prioritise. That said it's still good to get the discussion going :grin:
I'm interested in how onKeyboardShortcutPress would work, is it a react-hook? Should be something similar to the feature flags API.
Considerations

Double key presses.

Don't think we need to support this tbh, especially not from the start.

Limit which keys/combinations can be enabled.

Yup, and also be deliberate with when we capture input. Single-key hotkeys like s would be fine if we don't capture in input fields, but we'd need to add modifiers if we wanna capture in input fields, and ctrl+s would be a bad one :grin:.

Limit how many shortcuts plugins can register.

Don't really see why this is needed? A guy would probably group the shortcuts per plugin anyway, and they have to be explicitly enabled?

App Shortcuts Defaults.

I think we could allow the app to enable some shortcuts by default. Smth like
const app = createApp({
  enabledShortcuts: ['welcome.reload-page']
})

Ability to share configurations.

Probably rely on the app enabling things for orgs instead tbh

Gonna preface this with that this is going to be pretty low prio as we don't really see a need to get this shipped early on and there are many other APIs that we'd prioritise. That said it's still good to get the discussion going 😁

Makes sense! Is there any particular tag this should be to indicate its priority? If so, feel free to tag. The reason I decided to create an RFC is because @stefanalund asked me directly about it. I don't plan on working on this anytime soon but would leave it here if anyone else wanted to pick up / discuss 😄

I'm interested in how onKeyboardShortcutPress would work, is it a react-hook? Should be something similar to the feature flags API.

Technically yes, although under the hood it can be non-React code. I see the hook as a wrapper, so that the underlying code can be ported to any other UI framework or away from Backstage entirely.

Considerations

Double key presses.

Don't think we need to support this tbh, especially not from the start.

Good to hear. I'll remove this.

Limit which keys/combinations can be enabled.

Yup, and also be deliberate with when we capture input. Single-key hotkeys like s would be fine if we don't capture in input fields, but we'd need to add modifiers if we wanna capture in input fields, and ctrl+s would be a bad one 😁.

Yep makes sense. I think if there's any good guides or list of keys we'd want that we could (or anyone else reading) could contribute, that'd be great.

Limit how many shortcuts plugins can register.

Don't really see why this is needed? A guy would probably group the shortcuts per plugin anyway, and they have to be explicitly enabled?

In my view it was less about the technical limitation and more about the UI one. If every plugin had 10 shortcuts enabled, and there were 20 plugins say, that's 200 shortcuts. If this is something this feature wants to promote from day 0, then this point is not applicable. The fact that users have to manually opt into them might be sufficient enough.

App Shortcuts Defaults.

I think we could allow the app to enable some shortcuts by default. Smth like
const app = createApp({
  enabledShortcuts: ['welcome.reload-page']
})

That's good feedback 👍

Ability to share configurations.

Probably rely on the app enabling things for orgs instead tbh

Not sure I understand this particular point. The app enabling things for orgs?

Technically yes, although under the hood it can be non-React code. I see the hook as a wrapper, so that the underlying code can be ported to any other UI framework or away from Backstage entirely.

If it's a custom hook, it should be prefixed with use, or everyone's gonna have a bad time ⛷
I'd expect a simple hook implementation to look something like this:
function useKeyboardShortcut(shortcutId: string, callback: () => void) {
  const api = useApi(keyboardShortcutApi)

  useEffect(() => {
    const unsubscribe = api.subscribe(shortcutId, callback)
    return () => unsubscribe()
  }, [shortcutId, callback, api])
}

Not sure I understand this particular point. The app enabling things for orgs?

I meant the feature from the previous point, orgs enabling shortcuts by default in the app.

Ah yes, my bad - totally slipped that I used on instead of use :P good catch. In terms of the org enabling shortcuts, that makes sense for common/app shortcuts. I think the sharing configurations was more for the plugin shortcuts, but that might be adding premature complexity so probably not worth adding in the first iteration of this.

@bih if there are future plug-ins for embedded VS Code, Theia or other editor, these shortcuts might then cause conflicts.
Perhaps it would be better to leave shortcuts out of Backstage and instead wait for plug-ins to evolve further to better understand the options? Of course, waiting could also mean never being able to implement shortcuts, because of breaking backwards compatibility with plug-ins that have developed their own shortcuts.
So, maybe the solution is to define very simple shortcuts for frequent interactions, like menu selections, etc.

At some point we had mockups for pinned items in the sidebar, that were essentially personal numbered shortcuts into the app. Those would be a good fit for app level mapped e.g. Ctrl+Shift+number or so.

That’s awesome! Yeah definitely can be a part of this

Keyboard accessibility in web applications. So fun. A few items that may be of interest.
Keyboard shortcuts in webapps, typically, are few and far between mainly due to conflicts with the browser's short keys. There is actually an accesskey attribute specifically for shortcuts, but the general consensus is that it should not be used (because of a messy implementation).
What you'll find used quite a lot (you can see it here on GitHub even) is tab, shift-tab, arrow keys, space, enter. Also fairly common in developer tools is generic keypress with an input field not selected (e.g. pressing c while not in a text input field on GitHub leads to new issue creation).
Long story short, I would say you want to distinguish between accessibility and shortcuts. Accessibility in a webapp is highly dependent on tab, tabindex, arrows, and space/enter.
Shortcuts in a webapp are a hot mess and there's no currently working standard. So if global shortcuts are to be a thing, the very least you may want to do is reserve a namespace for them

Not prioritised.


Counting objects

Added method to CoreDataModelable extension that returns count of entities that matches optional predicate within specified NSManagedObjectContext.
extension CoreDataModelable where Self: NSManagedObject {
    static public func countInContext(context: NSManagedObjectContext, predicate: NSPredicate? = nil) throws -> Int
}

Great addition! :+1:


Fab1 and Fab2 point to same URL

....namely to http://cba.mit.edu/events/05.07.Norway/index.html
which is indeed the  meeting in Tromsø, but not in 2006: in 2005
My best guess for the Fab1 is this one (NSF site visit, first presentation of concept of FabLab by Bakthiak Mikhak) http://cba.mit.edu/events/02.09.NSF/index.html

Thank you, yes, it was my mistake. For now I pointed to the original url that was on the list that I received from Jean-Luc Pierite: http://cba.mit.edu/events/05.01.fab/index.html.  Maybe we can include the NSF Site Visit also if it's a milestone?


pluginlib: 1.10.4-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository pluginlib to 1.10.4-0:

upstream repository: https://github.com/ros/pluginlib
release repository: https://github.com/ros-gbp/pluginlib-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.22
previous version for package: 1.10.3-0

pluginlib
* Merge pull request #42 <https://github.com/ros/pluginlib/issues/42> from delftrobotics-forks/unique-ptr
  Add std::unique_ptr API
* Add unit test for unique_ptr API.
* Simplify unit tests with ASSERT_THROW.
* Add ClassLoader::createUniqueInstance.
* Wrap long comment on createInstance and friend.
* Throw exception if plugin.xml is broken (#41 <https://github.com/ros/pluginlib/issues/41>)
  * added test case for broken xml files with missing attributes of class tag
  * added checks if all needed attributes of the class tag are existing
  * removed comment and empty line
* Contributors: Maarten de Vries, Mikael Arguedas, cwecht

to be merged along https://github.com/ros/rosdistro/pull/12778

let's hold this until properly tested on kinetic before merging on earlier distros


chore: release prep for v1.41.0

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

Project coverage is 27.51%. Comparing base (ffde1fa) to head (d15e0df).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1423   +/-   ##
=======================================
  Coverage   27.51%   27.51%
=======================================
  Files          61       61
  Lines        6272     6272
=======================================
  Hits         1726     1726
  Misses       4423     4423
  Partials      123      123

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


feat(dev): run runners with terminal emulation

closes #350

wanted to tackle logger formatting of long lines #350
needed tty info, namely columns (terminal width)
was undefined
because runner is a nodejs child_process fork
and fork runs in pipe mode so that we can programatically control it
but pipe mode loses tty for child
so needed to resort to community solution for a spawn with pipe semantics but also tty
found node-pty, quality apperance, highly used, microsoft maintained now <3
but then we had lost the ipc features of nodejs child_process fork
again turned to the community, found node-ipc, looks ok, has worked well so far (I've written TCP network software years ago, this feels very easy by comparison!!)
followed this rabbit hole not only because it was the right thing to do but it fixed a variety of hacks in the existing codebase regarding tty stuff
I am happy to take more control and use node-pty + node-ipc manually given there is simply no satisfactory way to achieve with with node's child_process module

:rabbit: :hole: 😄
TODO

[x] ~docs~
[ ] ~tests~


[WIP] Player list

Description
Write your description here
Check List

[ ] No longer a work in progress, ready to be merged
[ ] All test passed

#11


Is there a demo for speech recognition using deeplearning4j?

Hello. I just started learning speech recognition. I want to program speech recognition using java. Is there a demo for speech recognition using deeplearning4j?

Closing this, please post these kinds of questions on the forums. Answer is: no but we'll work on one.


How to run “Bazel->Sync->Sync Projects with BUILD Files” using a command

I’m trying to find if there is a way to mimic
“Bazel->Sync->Sync Projects with BUILD Files” using a command from the terminal.
I noticed that doing sync from the menu bar runs quite a bunch of commands. I tried running’bazel sync’ command but seems it doesn’t solve the purpose. It would be great if we can invoke the same from command line.

@ponagraj  Could you please provide sample code to reproduce this issue?

@Pavank1992 Thank you for looking into this. This is actually not a issue. From IDE we run ‘Sync Projects With Build Files’, as shown in the image attached.

I want to know if there is a command to do the same from command line.

No, there is not a command for this. The plug-in identifies the build files then runs bazel build on them.


Only make a gene catalog

Allow users to use geneshot to only make a gene catalog and not proceed with any annotation / CAG steps thereafter.
Useful for alternative analyses (i.e. a known set of homologs to target; allelic varation of a single gene like PBP2, etc).

it would be cool to have also only the last part: calculate CAGs from a dataset of interest. Many people (me) will reach this software after having already the whole gene catalog built :)

That's a great point, @adriaaula! I'd be more than happy to review and merge in any additional functionality that you (or anyone else) would like to contribute.


Can --format (or other) be used to make VALID json objects in --output=FILE ?

What is your question?
I am trying to write a php script that uses the output from:
chat_downloader https://youtu.be/some_key --output=/path/to/my-file
PHP's decode_json command fails with almost all lines in /path/to/my-file: "Syntax error."
When I use https://jsonformatter.curiousconcept.com to verify syntax, I get a number of errors, the most recurring of which is:
Strings should be wrapped in double quotes
Additional context
When I replace all apostrophes with quotes, it's better but still not successful: "Multiple JSON root elements"
(I am completely aware that brute-force replacements of apostrophe with quote is not a good practice.  This is something that must be done on a case-by-case basis; something I would expect to be done by the application - hence this question.)
Since I know nothing of python nor of JSON, and since the (primitive) documentation claims to make json, I hope this is easily solved.
I've attached a sample from one of my livestreams.
1cCVst_Zl9ga.txt

Just taking a shot in the dark here to be sure since your sample was a txt file....  you are using --format json and json extension?
example:
chat_downloader https://www.youtube.com/user/somechannel --format json --output filename.json
I ask because I started messing with this program and the sample you uploaded looks like the default text file that is generated as a list of python dictionaries as the data instead of a json file.  See if using the format and json file extension gives the right syntax.  Not sure if you were already doing that or not.  Anyhow good luck :)

Hi! Sorry for the late response, but as @craxxon correctly suggested, you should output to a .json file. For example, the following command:
chat_downloader https://www.youtube.com/watch?v=5qap5aO4i9A --output data.json

Would output correctly formatted JSON (see file in zip, since github doesn't allow .json files to be uploaded for some reason):
data.zip

I believe this issue was caused by .txt files producing Python's "dictionary representation" as strings. This has been fixed in the latest version (v0.1.0). Sorry for the confusion!
Let me know if you have any other questions.


KAFKA-10817; Add clusterId validation to Fetch handling

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I had some offline discussion with @cmccabe and @ijuma about the clusterId type. I think it makes sense for now to keep it as a string since this is consistent with most uses in the current codebase and it is what is documented in the KIP. I do wonder if it might be more efficient to do the comparison on the string rather than converting to Uuid though.

I agree that comparing the string is going to be more efficient than converting to uuid and then comparing.

@hachikuji @ijuma Yeah, that makes sense. I have changed this.

I verified tests locally. I am going to merge to trunk and 2.8 since we seem blocked by jenkins at the moment.


[New] Edit with branch versioning

Description
This PR implements Edit with branch versioning in Edit and Manage Data category.
URL to README: README
Linked Issue(s)

swift/issues/5287

How To Test

Tap "Create", enter a version name, and selected an access to create a version.
Tap "Switch" to switch to the new version.
Tap on the feature on feature to select it.

Tap on the edit button to update feature's damage type.
Or, tap on map to move the feature.

Screenshots

I noticed that this sample can get into a state where it looks like I cannot unselect the callout no matter where I tap. Is there another way to cancel it?

I noticed that this sample can get into a state where it looks like I cannot unselect the callout no matter where I tap. Is there another way to cancel it?

Tapping "Cancel" also dismisses the callout now. Thanks for catching this!

Should the feature be unselected as well?

Thank you for reviewing this!


Error handling not working

I'm trying to handle errors, the same way it's stated in the documentation, but when I log the error, all I get is a number:

Here is my code
async authenticateUser() {
      await faceio.authenticate({
        'locale': 'es' // Default user locale
      }).then((userData: any) => {
        console.log(userData)
        console.log('Success, user identified')
        console.log('Linked facial Id: ' + userData.facialId)
        console.log('Payload: ' + JSON.stringify(userData.payload))
      }).catch((errCode: any) => {
        console.error(errCode)
        this.handleError(errCode)
        // handle authentication failure. Visit:
        // https://faceio.net/integration-guide#error-codes
        // for the list of all possible error codes
      }).finally(() => {
        faceio.restartSession()
      })
    },

Well, what you got when an enrollment  or authentication operation fails actually is an integer error code documented here. The promise is rejected with the corresponding error code. On your error handler callback, you have to make a big switch statement to isolate the returned error code just like the following gist: https://gist.github.com/symisc/34203d2811a39f2a871373abc6dd1ce9#file-faceio_integration_boilerplate-html-L95.
Please note that there is no this.handleError(errCode) method implementation, you have to implement it yourself on your code base. Please remove that statement from your code to get it working. Check the gist above for a typical implementation of the  handleError(errCode)  function.

Sorry but your documentation doesn't states anything about integer error codes and actually the link you provided with the gist, is exactly the same way I've implemented the switch statement to handle errors, but obviously, if whats returned is an integer, how is the switch statement gonna work if it's expecting something else?
Like where is defined fioErrCode?

This is not clear to me, and copying that code exactly as is, it's simply not working... please be more clear about it.

Well, what you got when an enrollment  or authentication operation fails actually is an integer error code documented here. The promise is rejected with the corresponding error code. On your error handler callback, you have to make a big switch statement to isolate the returned error code just like the following gist: https://gist.github.com/symisc/34203d2811a39f2a871373abc6dd1ce9#file-faceio_integration_boilerplate-html-L95.
Please note that there is no this.handleError(errCode) method implementation, you have to implement it yourself on your code base. Please remove that statement from your code to get it working. Check the gist above for a typical implementation of the handleError(errCode) function.

If what you mean is that we should asumme that the error list provided here is listed in order meaning that the first one would be the errorCode 1, the second one the errorCode 2, and so on, that it's absolutely not clear, and you should either provide a better, cleaner and understandable documentation or change these errorCodes to their respective string names as many other API usually do.


Server crash on restart with errors below

DO NOT REMOVE THIS
Please make sure your issue complies with these guidelines:

[ ] Be using PMMP

[ ] Make sure the issue hasn't already been reported

[ ] Make sure you are on the latest version of PMMP & PiggyAuth

[ ] Have a detailed title like "Players are being kicked randomly"

[ ] Provide any crash dumps or errors

General

Server Software:

Version of PiggyAuth:

Data Provider:

[x] MySQL
[ ] SQLite
[ ] YAML
[ ] JSON

Steps to Reproduce the Issue

Extra Information

Server crash 2-4 times continusly whenever server restarts:-
2017-06-17 [07:16:45] [Server thread/CRITICAL]: Error: "Call to a member function isAuthenticated() on null" (EXCEPTION) in "/PiggyAuth.phar/src/PiggyAuth/Main" at line 937
2017-06-17 [07:16:45] [Server thread/EMERGENCY]: An unrecoverable error has occurred and the server has crashed. Creating a crash dump

Error: Call to a member function isAuthenticated() on null
File: /PiggyAuth.phar/src/PiggyAuth/Main
Line: 937
Type: notice

THIS CRASH WAS CAUSED BY A PLUGIN
BAD PLUGIN: PiggyAuth v<IP_ADDRESS>

Code:
[928]      * @param Player $player
[929]      * @return bool
[930]      */
[931]     public function isTooManyIPOnline(Player $player)
[932]     {
[933]         $players = 0;
[934]         foreach ($this->getServer()->getOnlinePlayers() as $p) {
[935]             if ($p !== $player) {
[936]                 if ($p->getAddress() == $player->getAddress()) {
[937]                     if ($this->sessionmanager->getSession($p)->isAuthenticated()) {
[938]                         $players++;
[939]                     }
[940]                 }
[941]             }
[942]         }
[943]         return $players > ($this->getConfig()->getNested("login.ip-limit") - 1);
[944]     }
[945]
[946]     /**
[947]      * @param $password

Backtrace:
#0 /PiggyAuth.phar/src/PiggyAuth/Main(413): PiggyAuth\Main->isTooManyIPOnline(pocketmine\Player object)
#1 /PiggyAuth.phar/src/PiggyAuth/Tasks/AsyncLoginTask(130): PiggyAuth\Main->force(pocketmine\Player object, boolean 1, integer 0, NULL )
#2 /src/pocketmine/scheduler/AsyncPool(158): PiggyAuth\Tasks\AsyncLoginTask->onCompletion(pocketmine\Server object)
#3 /src/pocketmine/scheduler/ServerScheduler(339): pocketmine\scheduler\AsyncPool->collectTasks(boolean)
#4 /src/pocketmine/Server__64bit(2411): pocketmine\scheduler\ServerScheduler->mainThreadHeartbeat(integer 803)
#5 /src/pocketmine/Server__64bit(2179): pocketmine\Server->tick(boolean)
#6 /src/pocketmine/Server__64bit(2061): pocketmine\Server->tickProcessor(boolean)
#7 /src/pocketmine/Server__64bit(1646): pocketmine\Server->start(boolean)
#8 /src/pocketmine/PocketMine(501): pocketmine\Server->__construct(BaseClassLoader object, pocketmine\utils\MainLogger object, string phar:///home/gmplus/Factions/PocketMine-MP.phar/, string $
#9 (1): require(string phar:///home/gmplus/Factions/PocketMine-MP.phar/src/pocketmine/PocketMine.php)

Should be fix. Notify me if it isn't.


Continuum -> Anaconda rename

Codecov Report

Merging #2525 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2525      +/-   ##
==========================================
- Coverage   86.53%   86.52%   -0.01%
==========================================
  Files         317      317
  Lines       62631    62631
  Branches     6911     6911
==========================================
- Hits        54196    54193       -3
- Misses       7341     7343       +2
- Partials     1094     1095       +1

$ grep -ir "continuum"
buildscripts/incremental/install_miniconda.sh:  wget https://repo.continuum.io/miniconda/Miniconda3-latest-Linux-x86_64.sh -O miniconda.sh
buildscripts/incremental/install_miniconda.sh:  wget https://repo.continuum.io/miniconda/Miniconda3-latest-MacOSX-x86_64.sh -O miniconda.sh
buildscripts/condarecipe.jenkins/custom_condarc:- http://repo.continuum.io/pkgs/free
buildscripts/condarecipe.jenkins/custom_condarc:- http://repo.continuum.io/pkgs/pro
buildscripts/condarecipe.hsa/custom_condarc:- http://repo.continuum.io/pkgs/free
buildscripts/condarecipe.hsa/custom_condarc:- http://repo.continuum.io/pkgs/pro
numba/cuda/tests/cudapy/test_vectorize_decor.py:        https://github.com/ContinuumIO/numbapro/issues/90
setup.py<EMAIL_ADDRESS>examples/notebooks/numba.ipynb:    "Have questions?  Join the mailing list at<EMAIL_ADDRESS>README.rst:Join the numba mailing list<EMAIL_ADDRESS>README.rst:https://groups.google.com/a/continuum.io/d/forum/numba-users
README.rst:See if our sponsor can help you (which can help this project): http://www.continuum.io
CONTRIBUTING.md:(https://groups.google.com/a/continuum.io/forum/#!forum/numba-users).
docs/source/developer/contributing.rst:We have a public mailing-list that you can e-mail at<EMAIL_ADDRESS>docs/source/developer/contributing.rst:`Google Groups <https://groups.google.com/a/continuum.io/forum/#!forum/numba-users>`_,
docs/source/developer/contributing.rst:   Python 3.4.2 |Continuum Analytics, Inc.| (default, Oct 21 2014, 17:16:37)
docs/source/conf.py:   u'Continuum Analytics', 'manual'),
docs/source/conf.py:     ['Continuum Analytics'], 1)
docs/source/conf.py:   'Continuum Analytics', 'Numba', 'One line description of project.',
docs/source/user/faq.rst:.. _NumbaPro: http://docs.continuum.io/numbapro/
docs/source/user/installing.rst:   Python 3.4.2 |Continuum Analytics, Inc.| (default, Oct 21 2014, 17:16:37)

I think the build scripts/mailing lists need to continue to refer to continuum.io and the github URL needs to stay, but the rest can probably be changed?

repo.continuum.io hasn't moved, and I didn't want to edit output until Conda updates the Python banner.  However, the diff notes that I forgot to actually remove the NumbaPro footnote (which was the inspiration for the FAQ fix).  I think the conf.py changes are also needed.

Think the bottom of README.rst still needs changing?
See if our sponsor can help you (which can help this project): http://www.continuum.io https://github.com/seibert/numba/blame/9a82d54d1b8509954b1945e32ca7037ef5bd313a/README.rst#L131
Otherwise I think this is now all done.

Looks good. Thanks for the patch.


Incorrect state in useCustomEventListener hook

This issue started with release 1.0.9. My state variables in my custom hook are always the value they are initialized at within useCustomEventListener.
In the code below, orderId is null in useCustomEventListener even when the flow in the hook logs 123 first. 1.0.8 and earlier do not have this issue.
function myCustomHook(params) {
  const [orderId, setOrderId] = useState(null);
...
  console.log('orderid', orderId); // outputs 123
  useCustomEventListener(events.receivedOrderStatus, (data) => {
    console.log('orderid', orderId); // outputs null
  });
  useEffect(() => {
    console.log('orderid', orderId); // outputs 123
    setOrderId(params.orderId);
  }, [params.orderPlacementTimestamp]);
}

Thanks for raising the issue.
Recently I did the change to run useCustomEventListener hook once, instead of adding and removing listener on every render.
It looks like, useCustomEventListener need to support dependency array parameter (like useEffect)

Released new version to fix this, updated documentation


New theme submission - HTML/CSS

## New Theme?
We would love to support all code editors out there, but we need your help to accomplish that. Feel free to create a new theme based on our [color palette](https://github.com/dracula/dracula-theme#color-palette).

redbrain/html is a variant of Dracula for HTML, easily importable as a CSS stylesheet. Simple apps, static doc pages, and personal websites could all use a dark theme that feels as nice and looks as good as Dracula. Please let me know if you'd like any changes.

Closing since Dracula UI basically replaces this.


Feature: apply row exclusions

Description
Properly applies exclude actions taken on a project when running studio.apply_corrections() with a pandas or pyspark dataframe
How to test
Test w/ backend change in branch feat/send-actions-to-cli (changes are also in dev)

Upload a dataset from a pandas/pyspark dataframe using the Python API
Create a project using the uploaded dataset
Exclude some rows from the project using the studio UI
Run apply_corrections using the python API
Verify that the rows you excluded are filtered from the resulting dataframe

LGTM! Would like to do a refactor of this at some point, but happy with this for now

Agreed! I started working on refactoring some of this earlier this week but didn't get a chance to finish yet haha


Unable to use an object for defining types with createAction

I was wondering if this is the intended use.
export const actions = {
 login: createAction(types.LOGIN_REQUEST, (email: string, password: string) => ({
    type: types.LOGIN_REQUEST,
    payload: { email, password },
  })),
  loginSuccess: createAction('AUTH/LOGIN_SUCCESS', (authData: any) => ({
    type: 'AUTH/LOGIN_SUCCESS',
    payload: { authData },
  })),
...

With the login action above, I get this output with returnsOfActions:
const returnsOfActions = Object.values(actions).map($call);
type AppAction = typeof returnsOfActions[number];
/**** Here's what the types looks like (hovering my mouse over returnsOfActions **/
const returnsOfActions: ({
    type: string;
    payload: {
        email: string;
        password: string;
    };
} | {
    type: "AUTH/LOGIN_SUCCESS";
    payload: {
        authData: any;
    };

The type for the login_request action returns as string, and not the actual value of types.LOGIN_REQUEST.  Is there any way I can continue using my types object, or do I need to use the actual value when I use createAction?
Thanks

Can you show the code for types.LOGIN_REQUEST including it's type definition if you wrote one. You don't have to use a raw string - but what you supply has to have a literal string type - not just a string, and there are a few ways that you could end up widening your literal string type, like creating it with concatenation.

Here's my types object:
export const types = {
  AUTO_LOGIN: 'AUTH/AUTH_AUTO_LOGIN',
  SIGNUP_REQUEST: 'AUTH/SIGNUP_REQUEST',
  SIGNUP_SUCCESS: 'AUTH/SIGNUP_SUCCESS',
....

I don't have a typedef for that object.

Do i need to do this:
type MyTypes = {
  AUTO_LOGIN: 'AUTH/AUTH_AUTO_LOGIN',
  SIGNUP_REQUEST: 'AUTH/SIGNUP_REQUEST',
....
export const types: MyTypes = {
  AUTO_LOGIN: 'AUTH/AUTH_AUTO_LOGIN',
  SIGNUP_REQUEST: 'AUTH/SIGNUP_REQUEST',
...

Thank you, I switched to using an enum.  Thanks for your help.


Multi-gpu inference

How can it be done? i tried using the torch.distributed.launch module as described on training, the inference script doesnt seem to support local_rank, is it a trivial fix?

I think it requires some modification.
As testing images may have different image sizes, so we can only put one sample to one GPU.
Or another choice is to distribute different tiles into several GPUs.

Hi, I shared a script for multi-GPU testing. Maybe it can help.

Hi, I shared a script for multi-GPU testing. Maybe it can help.

Thank you 👍 , I will have a check 😃

My suggestion would be to simply allocate the next image to another gpu until it is exausted in case the input is a directory, as it is in my case

@lobomfz Thanks for the suggestion. I will look into it when I am free 😃


Option --generator not being honoured

Bug description
It seems the --generator option is not being honoured, and thus it is not possible to cope with multiple generators defined in the same prisma.schema file.
How to reproduce
I have several generators in my prisma.schema file. Passing the option to use the python generator gives:
PRISMA_PY_DEBUG=1  venv/bin/prisma generate --schema ../backend/prisma/schema.prisma --generator python_client
[DEBUG  ] prisma.cli.prisma: Running prisma command with args: ['generate', '--schema', '../backend/prisma/schema.prisma', '--generator', 'python_client']
[DEBUG  ] prisma.cli._node: Checking if nodejs-bin is installed
[DEBUG  ] prisma.cli._node: Checking for global target binary: node
[DEBUG  ] prisma.cli._node: Global target binary: node not found
[DEBUG  ] prisma.cli._node: Skipping nodeenv installation as it already exists at /root/.cache/prisma-python/nodeenv
[DEBUG  ] prisma.cli._node: Attempting to preprend /root/.cache/prisma-python/nodeenv/bin to the PATH
[DEBUG  ] prisma.cli._node: Using PATH environment variable: /root/.cache/prisma-python/nodeenv/bin:/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
[DEBUG  ] prisma.cli._node: Executing binary at /root/.cache/prisma-python/nodeenv/bin/node with args: ('/root/.cache/prisma-python/binaries/5.4.2/ac9d7041ed77bcc8a8dbd2ab6616b39013829574/node_modules/prisma/build/index.js', 'generate', '--schema', '../backend/prisma/schema.prisma', '--generator', 'python_client')
Environment variables loaded from .env
Prisma schema loaded from ../backend/prisma/schema.prisma
  prisma:GeneratorProcess  /bin/sh: 1: prisma-client-py: not found +0ms
  prisma:GeneratorProcess  child exited with code 127 on signal null +2ms
Error: Generator "prisma-client-py" failed:

/bin/sh: 1: prisma-client-py: not found

Curiously, perhaps another issue, is that not passing the --generator option creates the @prisma/client successfully, but then fails since other generators that exist on my prisma.schema file:
PRISMA_PY_DEBUG=1  venv/bin/prisma generate --schema ../backend/prisma/schema.prisma
[DEBUG  ] prisma.cli.prisma: Running prisma command with args: ['generate', '--schema', '../backend/prisma/schema.prisma']
[DEBUG  ] prisma.cli._node: Checking if nodejs-bin is installed
[DEBUG  ] prisma.cli._node: Checking for global target binary: node
[DEBUG  ] prisma.cli._node: Global target binary: node not found
[DEBUG  ] prisma.cli._node: Skipping nodeenv installation as it already exists at /root/.cache/prisma-python/nodeenv
[DEBUG  ] prisma.cli._node: Attempting to preprend /root/.cache/prisma-python/nodeenv/bin to the PATH
[DEBUG  ] prisma.cli._node: Using PATH environment variable: /root/.cache/prisma-python/nodeenv/bin:/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
[DEBUG  ] prisma.cli._node: Executing binary at /root/.cache/prisma-python/nodeenv/bin/node with args: ('/root/.cache/prisma-python/binaries/5.4.2/ac9d7041ed77bcc8a8dbd2ab6616b39013829574/node_modules/prisma/build/index.js', 'generate', '--schema', '../backend/prisma/schema.prisma')
Environment variables loaded from .env
Prisma schema loaded from ../backend/prisma/schema.prisma
  prisma:GeneratorProcess  /bin/sh: 1: typegraphql-prisma: not found +0ms
  prisma:GeneratorProcess  /bin/sh: 1: prisma-client-py: not found +3ms
  prisma:GeneratorProcess  child exited with code 127 on signal null +0ms
  prisma:GeneratorProcess  child exited with code 127 on signal null +1ms
Error: Generator "typegraphql-prisma" failed:

/bin/sh: 1: typegraphql-prisma: not found

Expected behavior
Ideally the client should honour this flag.
Prisma information

generator client {
  provider = "prisma-client-js"
  binaryTargets = ["native"]
}

generator typegraphql {
  provider            = "typegraphql-prisma"
  formatGeneratedCode = false
}

generator python_client {
  provider             = "prisma-client-py"
  interface            = "asyncio"
  recursive_type_depth = 5
}

etc...

Environment & setup

OS: Mac OS
Database: PostgreSQL
Python version: 3.12.1
Prisma version:

prisma                  : 5.4.2
prisma client python    : 0.11.0
platform                : debian-openssl-3.0.x
expected engine version : ac9d7041ed77bcc8a8dbd2ab6616b39013829574
installed extras        : []
install path            : /cask/driver/venv/lib/python3.12/site-packages/prisma
binary cache dir        : /root/.cache/prisma-python/binaries/5.4.2/ac9d7041ed77bcc8a8dbd2ab6616b39013829574

The underlying error in your first example is this:
Error: Generator "prisma-client-py" failed:

/bin/sh: 1: prisma-client-py: not found

Which I suspect is because you're executing venv/bin/prisma directly, unfortunately due to how environments work in Python you'll have to activate the virtual environment before running anything.
source .venv/bin/activate

You also maybe able to update your PATH to include venv/bin but I suspect you'll run into other issues with this.

Thanks, it seems that was the reason for the unexpected behaviour.


Remove a deprecated reference to testExamplesJaxprDoc in Understanding Jaxpr

This PR removes a deprecated reference to JaxprTest.testExamplesJaxprDoc in the Understanding Jaxpr guide.
See:

https://github.com/google/jax/pull/2994#issuecomment-625490111
https://github.com/google/jax/issues/3678

Thanks @j-towns

Thanks!


There are no build files

There are no Eclipse project files or Gradle build files.

I know.
I set this up at 4am and didn't have the patience to set up gradle :smile: . Doing it now.


Updated DotNetCoreRestore inline with Tooling v1.0 #1533

I've removed settings that are no longer valid, if you prefer for them to be marked as obsolete instead let me know.

@JonCubed,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@JonCubed, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

@JonCubed  your changes have been merged, thanks for your contribution 👍


TMS based tiles are wrong

As TMS standard force y value to be caculated in negative, this plugin is producing wrong tile number for y axis.

Good catch. This was not tested with tms: true L.TileLayers, so I missed that.
I'll see what I can do...

On getY, following code can convert to TMS based tile number:
y = Math.pow(2, z) - y - 1;
Than swap t1_y & t2_y as the values are going to be inverse.

@karigarweb  Could you please check out the issue25-tms branch and give it a try? It looks good to me, but I'm away from my work computer for a few days so can't test it thoroughly.
The patch specifically is here:  https://github.com/gregallensworth/L.TileLayer.Cordova/blob/a87c01ea160ebd34c6ecd12bf1d88ee5aa70cd5e/src/leaflet-tilelayer-cordova.js#L175-L184

@karigarweb Did you get to check out the issue25-tms version, to confirm that it's working?

Yes, everything is working like a charm! Thank you


Blackscreen after running colmap with wsl

Windows Version
Microsoft Windows [version 10.0.22621.4037]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-2
Distro Version
Ubuntu 22.04
Other Software
The default version of colmap that comes with apt. Installable via
sudo apt install colmap

Nvidia driver, cuda
Repro Steps
prepare a folder of random images. run
colmap feature_extractor --image_path ${path to folder}

Expected Behavior
colmap create a database of features extracted form the images
Actual Behavior
Blackscreen, segfault.
Diagnostic Logs
No response

D3D12: Removing Device.


Bug: Memory leak on removing app's root element

Probable Bug
Behavior: Memory leak after removing the app (by removing the app's root element)
Bug reproduction: Please check this demo http://codepen.io/djidel/pen/PGREXm
The n is how many times a primitive angular app is bootstrapped then removed.
With n=1 the whole page uses about 30Mb.
With n=500 the page uses about 70Mb.
Removing the app's root element seems to have no effect in memory use.
The expected behavior: Removing angular app's root element should "destroy" the app and clean up all the used memory.
Motivation / use case:
A web app that bootstraps angular apps on independent DOM objects while user interacts. Long use of the web app should not result in high memory use.
Angular version: 1.5.5
Browser: Google Chrome
Browser version: 53
OS: Windows

This seems to be somehow related to jQuery (and probably to its rewrapping behavior - or bad use of it).
--
I tried using jqLite and (after garbage collecting) the memory drops back to the same level as for n = 1.
With jQuery, the line that i responsible for the biggest percentage of the memory leak, is re-wrapping the $rootElement (which is necessary). By removing the unnecessary re-wrapping, the memory (after garbage collecting) drops back to ~2MB higher than the n = 1 case:
// By replacing:
var e = lastModule.get('$rootElement');
jQuery(e).remove();

// ...with:
var e = lastModule.get('$rootElement');
e.remove();

// ...the memory leak mostly goes away

Maybe @mgol knows more about it.
 It doesn't make any difference in your trivial example, but it a real application it will. I.e. in addition to removing $rootElement, also call $rootScope.$destroy().
--
Finally follow #8005 for an "official" way to destroy a bootstraped app.
Closing since every thing seems to work correctly (on the Angular side), but feel free to continue the discussion below.

Please check this: http://codepen.io/djidel/pen/vXjAqW?editors=0011
This basically bootstraps an app on a new DOM object and then destroy it many times. How many times and when to start creating them is controlled by the input text and the button respectively.
Please try to execute once and check the javascript memory usage then try 1000 times.
Does this count as memory leak?

@gkalpak also try to comment the 7th JS line (removing scope binding of the directive), this should lewer the memory use (on 1000 times).

I tried it and there is indeed a slight memory increase (~ 2.5MB in JS memory and another 2.5 in native memory for the first 1000 times). Removing the scope binding didn't make any difference for me.
TBH, I wasn't able to somehow relate this memory increase to Angular (but it may be due to my weak DevTools Timeline/Profile-foo). Both the JS Heap and Nodes seems to fall back to the initial sizes:
Before:

After:

--
Neither from the Heap Snapshot nor from the Allocation Timeline could I determine that the retained memory is due to Angular. According to my understand, most of the increase is caused by VM data related to deoptimization operations. (I could be wrong though.)


Alternative Repo Available

I'm offering an 'alternative master' repo at https://github.com/6pac/SlickGrid, while MLeibman is busy
This aims to keep the main trunk of the project up to date with bugfixes, small enhancements and recent versions of dependencies. Read more: https://github.com/6pac/SlickGrid/wiki
I have released a SlickGrid v2.2 on this repo.
This is in no way an attempt to hijack the project, I just want to ensure there is an up to date version available for new users to avoid some of the basic issues that seem to be repeatedly coming up.

I've been looking for updates/bug fixes to slickgrid for some time without going down a very customized/forked path.
I'm going to check this out.  Let's keep the project humming along.  Thanks!

A bit confused.  Are you the author of slickgrid or a contributor branching it to carry on work?
-----Original Message-----
From: "mpGitHub"<EMAIL_ADDRESS>Sent: 3/16/2015 4:55 PM
To: "mleibman/SlickGrid"<EMAIL_ADDRESS>Subject: Re: [SlickGrid] Alternative Repo Available (#1055)
I've been looking for updates/bug fixes to slickgrid for some time without going down a very customized/forked path.
I'm going to check this out. Let's keep the project humming along. Thanks!
—
Reply to this email directly or view it on GitHub.

I'm a contributor, but I don't have access to the master repo. MLeibman, the author, is the one with access to the repo, but he's been too busy to keep it updated of late.
I'm attempting to fill that maintenance role with this fork. Perhaps if I show I can do it well, one day MLeibman will give me commit access to the master, and then there won't be any need for this fork.
Read more: https://github.com/6pac/SlickGrid/wiki

:+1:

thanks.  it's good i think that others are keeping it going/active

Hi 6pac, If issues are found in your fork how do we let you know?  I don't see an issues category in your fork?

6pac, have you tested running multiple slickgrids on the same page?  The formatting of the columns in the first slickgrid is broken after loading the second slickgrid on the page.  The column widths are uneven/overlapping.

whoops, had to turn that on. done. thanks for the heads-up.

as to multiple grids, I haven't done that recently. pretty sure I've seen issues from people that have done that successfully though.
multiple grids is a requirement I have too, and I've noticed that particularly some of the events are not as supportive of that scenario as they could be (the events should pass grid-specific settings to allow event re-use on multiple grids, rather than relying on closures or global vars, which can't be easily made grid-specific if re-used).
it'll probably take me a few more weeks to explore that fully though.

Awesome, slickgrid is still the best grid around, glad to see the project gaining new life.

Nice!!! Thanks a lot for your support!

Good!

+1

@6pac you should almost rename it and not make it a fork anymore since they have deviated so much.

Hi @amcdnl, actually I have been trying really hard not to deviate it. The idea is only to fix bugs, keep it up to date, and complete features that need more work.
I have my own personal branch that has been extensively hacked for my own projects. I've completely reworked that one to add a DataProvider interface so the grid can be used with other complex datasources than DataView, and made a bunch of other changes. I have been thinking about releasing that as a separate fork again, but it still needs a little work.
But perhaps you are right. I have been hoping MLeibman might reintegrate my branch into the master at some point, but as time goes on perhaps rebranding as a separate project is a better approach.


Compile from github source error does not support std::this_thread::sleep_for

I have been following the instructions for compiling from github source, and have installed libtools, autoreconf, yaggo (and ruby to use yaggo).
autoreconf -fi ends in these errors:
configure: error: in `/mummer':
configure: error: Compiler does not support std::this_thread::sleep_for
See `config.log' for more details

config.log attached in pastebin
Are there other prerequisites missing? I am using a fresh Ubuntu 18.04 VM.

I have bypassed the need for self-compilation while maintaining automatically updated releases with this command
curl -s "https://api.github.com/repos/mummer4/mummer/releases" | grep download | grep tar.gz | head -n 1 | awk '{print $2}' | xargs curl -L -o /mummer.tar.gz
tar -zxvf mummer.tar.gz
rm mummer.tar.gz
mv mummer* mummer
cd mummer/
./configure --prefix=/mummer/
make
make install
export PATH=$Path:/mummer/
cd /


Add more detailed location information to the bevy page

Just noticed that we don't really mentioned Cineplexx as the venue on the KCD homepage. Would be cool if some can add the name address and pictures to the homepage.
On the tito ticket page there's more information.
And in the sponsor prospectus there are better images

Added the information and put link on the page


[BUILD] Enable -x on build/dist

Why are the changes needed?

Some users encounter errors on building, enable -x to help user identify the failure reason
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #3372 (7bdfb39) into master (9d615e5) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3372      +/-   ##
============================================
- Coverage     51.48%   51.44%   -0.04%
  Complexity       13       13
============================================
  Files           476      479       +3
  Lines         26537    26574      +37
  Branches       3702     3711       +9
============================================
+ Hits          13663    13672       +9
- Misses        11550    11564      +14
- Partials       1324     1338      +14

Impacted Files
Coverage Δ

...rg/apache/kyuubi/engine/ApplicationOperation.scala
74.07% <0.00%> (-20.05%)
:arrow_down:

.../plugin/spark/authz/ranger/RuleAuthorization.scala
73.33% <0.00%> (-6.67%)
:arrow_down:

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
76.13% <0.00%> (-3.41%)
:arrow_down:

...apache/kyuubi/engine/JpsApplicationOperation.scala
77.41% <0.00%> (-3.23%)
:arrow_down:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
72.26% <0.00%> (-2.74%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
78.00% <0.00%> (-2.00%)
:arrow_down:

...he/kyuubi/plugin/spark/authz/util/AuthZUtils.scala
41.02% <0.00%> (-1.84%)
:arrow_down:

...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala
89.27% <0.00%> (-0.70%)
:arrow_down:

... and 7 more

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

Thanks, merging to master

glad to see this option coming back


Need @JSSymName(..) annotation

In order to implement iterators one needs a way to tie a a method to a symbol as explained in my comment on issue 1141: Add js.Iterator according to ES6's iterator protocol.
Something like this perhaps:
@js.native
trait Iterable[+A] extends js.Object {
  @JSSymName("Symbol.iterator")
  def iterator(): Iterator[A] = js.native
}

or perhaps
@js.native
trait Iterable[+A] extends js.Object {
  @JSName(Symbol.iterator)
  def iterator(): Iterator[A] = js.native
}

when a Symbol class has been created.
Workarounds using using JSBracketAccess do not seem to work: I tried in this JSIterable class, and somehow am always getting a function back when I want a value.  The context ends up missing as discussed on gitter.

Define JSIterable as:
@js.native
trait JSIterable[+A] extends js.Object {
  @JSBracketAccess
  def method[R](symbol: js.Any)(): R = js.native
}

With the changes to JSIterable  that is called by ServiceWorkerAuth I get the following output in the console:

Ah the error was elsewhere: my logger was creating the function. This diff fixed it
https://github.com/read-write-web/rww-scala-js/commit/c8ae2c226728b6d1b4fcdbf3929c888a1974fba5

I did not find any documentation for @JSBracketCall in the online docs. How does that differ from @JSBracketAccess? I suppose this can be closed.

There is no documentation for @JSBracketCall (see issue). Briefly, for a method
@JSBracketCall
def foo(method: T)(arg1: U1, ..., argN: UN): R

obj.foo(someMethod)(arg1, ..., argN)

has the same semantics as the js:
obj[someMethod](arg1, ..., argN)

Whereas your previous formulation would translate to
(0, obj[someMethod])(arg1, ..., argN)

which indeed (intentionally) looses the this value.

Let's leave this open, though. Having to use @JSBracketCall is a bit of a last resort hack. method is known statically, so we should be able to store its info somewhere. Hopefully ...

Oh and, FTR, @JSBracketCall is the thing that powers js.Dynamic.applyDynamic.


dhcpd/telnet: don't need to dupcalite listen socket when task_create

Summary
dhcpd/telnet: don't need to dupcalite listen socket when task_create. this patch associates with https://github.com/apache/incubator-nuttx/pull/3045
Impact
normal using telnetd/dhcpd.
Testing
daily test

@Donny9
Thanks, telnetd works now.
By the way, I noticed that webserver and ftpd work without modification.
Could you tell me why only telnetd and dhcpd need to be modified?

@Donny9
Thanks, telnetd works now.
By the way, I noticed that webserver and ftpd work without modification.
Could you tell me why only telnetd and dhcpd need to be modified?

Okay，i need time to test them and modify them in other patch, thank you～

@Donny9
Thanks, telnetd works now.
By the way, I noticed that webserver and ftpd work without modification.
Could you tell me why only telnetd and dhcpd need to be modified?

@masayuki2009  I just checked ftpd daemon, it doesn't need to modify because ftpd_startworker will create a work thread for  new connection  by pthread_create API, the filelist duplicate don't occur when pthread_create. And the thttpd webserver used poll(fdwatch) to monitor the fd when a new connections occurs, the new thread or task doesn't be created, so it doesn't need  to modify.
The root cause is if parent task has socket(listened,not connect), child task will fail when it is created because the dup operate(psock_dup2) will fail.

@Donny9
Thanks, telnetd works now.
By the way, I noticed that webserver and ftpd work without modification.
Could you tell me why only telnetd and dhcpd need to be modified?

@masayuki2009 I just checked ftpd daemon, it doesn't need to modify because ftpd_startworker will create a work thread for new connection by pthread_create API, the filelist duplicate don't occur when pthread_create. And the thttpd webserver used poll(fdwatch) to monitor the fd when a new connections occurs, the new thread or task doesn't be created, so it doesn't need to modify.
The root cause is if parent task has socket(listened,not connect), child task will fail when it is created because the dup operate(psock_dup2) will fail.

https://github.com/apache/incubator-nuttx/pull/3045 check the file duplication result(ignore before this PR):
https://github.com/apache/incubator-nuttx/commit/e96c8b9283ddbdc329e96d262aefc9bb199d5578#diff-3e4edca12a292d53823387fc8d11439c7f1efe7702f80962193bdff8f3c27045L154-R83
and expose an old bug happen in tcp socket dup here:
https://github.com/apache/incubator-nuttx/blob/master/net/socket/net_dup2.c#L102
https://github.com/apache/incubator-nuttx/blob/37d5c1b0d93290212b7a7a246bd5845f8f0dd3cd/net/tcp/tcp_monitor.c#L257-L270
Since the listen socket isn't in TCP_ESTABLISHED or TCP_SYN_RCVD state, tcp_start_monitor return fail and then fail to create the new task.

Hi @masayuki2009 san,
Please help to review the below PR, which fixed the TCP socket fd dup(2) issue.
https://github.com/apache/incubator-nuttx/pull/3099

@xiaoxiang781216

So the fix has two part:
Add SOCK_CLOEXEC to server who spawn the child task, but the sock isn't really used by child
Fix the error in psock_dup2 which will provide by @anchao soon.

Thanks, I understand.


Improve some more docs

Follow up for https://github.com/fastlane/fastlane/pull/9665
cc @Lutzifer

@Lutzifer What's your Twitter username?

@wlborg

Am 05.07.2017 um 18:29 schrieb Felix Krause<EMAIL_ADDRESS>@Lutzifer What's your Twitter username?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Advanced Filter:  Timezone may throw the date off.

Description
Dates can be thrown off since we don't seem to be able to set a timezone. I think it  defaults to GMT, so for example, in filters when you select the range picker, today's date is already set to 2/13 instead of 2/12.
Also noticed setting the filter to start at 1/1/2000, will show it as 12/31/1999 in the filter chip.
Don't see an option documented for setting the corresponding timezone, so assuming there is no way to do it.
Version
Data-grid: V1.2.1
Screenshot

@ppeyro  If you check your first screenshot, it say UTC in your rangepicker. So works as planned?
And spec told to show dates in chips, not names of ranges. So that's new feature (not trivial as now we would need to store some metadata to filter rules too). Also filters will have actual dates, so if you store filter, and reuse it later, it will be same days not updated to current day. So using name is bad idea.

@alump  Yes, thanks for pointing out the UTC timezone. So the point of the issue was I don't think there is a way we can set/overwrite the timezone when specifying a date-type column, or is there?
I suspect customers would want to be able to define their timezone. For example, if the data is coming into the grid as say GMT and in the data-grid it assumes it's UTC, wouldn't it cause problems/confusion like what I mentioned above?
Also, the 2nd issue is:  there is a discrepancy between what the filter is defined as in the modal vs. what's shown in the chip. Modal defines the range as Jan/1/2000 to Jan/1/2018, but the filter chip shows it as Dec/31/1999 to Dec/31/2017. I thought maybe this had something to do with the timezone, so I dumped that in the same issue. If it's confusing we can separate them out.

@ppeyro In modal you have selector for timezone, see the UTC there. That's why it's in different timezone. Can not accept these bugs as this behavior is not specified anywhere. When specifications are done, then feature request issue should be made. Eg. what should be default timezone in editors, filters and chips. But having now issue tickets for all different parts of UI is not acceptable idea.

@alump  Please reopen this one as well.  There are issues in here we need to address before release. If you can't do the implementation, that doesn't mean the issue can go away!!  Please reopen so we can discuss who will take ownership.

Depending #679

Original issue report is not backed up by specifications, and so needs to be rewritten once wanted behavior is specified. Only case when it will be valid would be if UTC is defined as default presentation timezone, and test case have not overridden that value.

Needed for beta

Recheck after timezone fix.


Artifact study pages limit access

As the interface was before, it was giving access to the entire study once 1 artifact was public.
This limits that to only provide public access to those artifacts that are public. It also limits the interface to allow to modify the study sections only to admins, the owner and the users that the study is shared with.
Finally, if an artifact changes the visibility, all the artifacts that were used to generate that one also change visibility.

Coverage increased (+0.09%) to 82.868% when pulling dc80314310e294a920dc7144dfd7f41614550472 on josenavas:artifact-study-pages-limit-access into 2cacd696183b58d202f7582cc3e2cbb1123ea1b9 on biocore:artifact-study-pages.


Open AI Generator

The openai generator is not working

Yeah, I checked the credits for the api have finished so we need to create new account for it perhaps

Even creating a new account also OpenAi not giving free credits


🏗 Rename Quantum Serverless to Qiskit Serverless

Summary
Close #1308
Rename Quantum Serverless to Qiskit Serverless

I needed to leave old references in: aed00f1 and d2537b9
They can be updated later on.

python 3.8 is still in client/setup.py. we can fix it later.

Only thing I found was https://github.com/Qiskit-Extensions/qiskit-serverless/blob/main/client/tests/resources/test-compose.yaml#L4 , but I'm assuming we'll update that after we build new images...

Yep, basically is what I was commenting above. Some references are going to need to be updated later on.


Revert "[Access] Add start/end height to EventsResponse"

Reverts onflow/flow#1301

I agree this isn't ready to merge. There are backwards compatibility issues with the REST implementation, so I think we may need to take a different approach.


rtorrent.lock not removed when container is stopped

Support guidelines

[X] I've read the support guidelines

I've found a bug and checked that ...

[X] ... the documentation does not mention anything about my problem
[X] ... there are no open or closed issues that are related to my problem

Description
I am running into a problem:  When stopping the container (docker-compose stop rutorrent), the lock file in /session is left behind.  This prevents the container from restarting, and can be confusing to users who don't know how to check the log.
This appears to be related to the time required for rtorrent to shutdown when there are a large number of torrents (my client has > 2500 seeding).  Docker and docker-compose allow 10 seconds to elapse after the SIGTERM signal is sent before it sends SIGKILL, which doesn't give rtorrent enough time to gracefully close if there are a lot of torrents.  In cases where a lot of torrents are being seeded, rTorrent gets killed and the lockfile isn't removed.
This can be resolved by allowing a longer timeout for shutdown, either by adding a stop_grace_period to the docker-compose.yml file, or adding the -t XX parameter to the docker stop or docker-compose stop commands.
Expected behaviour
rTorrent should remove lockfile after container is stopped.
Actual behaviour
The lockfile persists after container is stopped.
Steps to reproduce

docker-compose up -d rutorrent
docker-compose stop rutorrent

Docker info
Client:
 Debug Mode: false

Server:
 Containers: 17
  Running: 17
  Paused: 0
  Stopped: 0
 Images: 37
 Server Version: 19.03.12
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: nvidia runc
 Default Runtime: nvidia
 Init Binary: docker-init
 containerd version: 7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc version: dc9208a3303feef5b3839f4323d9beb36df0a9dd
 init version: fec3683
 Security Options:
  apparmor
  seccomp
   Profile: default
 Kernel Version: 5.6.17-050617-generic
 Operating System: Ubuntu 20.04.3 LTS
 OSType: linux
 Architecture: x86_64
 CPUs: 4
 Total Memory: 15.55GiB
 Name: bunkerhill
 ID: XNA7:AVQ6:73XC:NGTK:ELFZ:TDE2:LVIV:J7D2:2AW5:XKKK:A2YY:C6MH
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false

WARNING: No swap limit support

Version

Docker compose version (type docker-compose --version):  docker-compose version 1.29.2, build 5becea4c
System info (type uname -a): Linux xxx 5.6.17-050617-generic #202006071230 SMP Sun Jun 7 12:35:03 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Docker compose
My docker-compose.yml is long and complicated, and I don't think it's relevant to the issue here.  Happy to provide it if needed.
n/a

Container logs
n/a
Additional info
This is something to consider adding to the docs and the command examples, and might reduce some issues people having with the container not starting because it finds a stale lock file.
It's also possible that a way for the startup script to check for and remove a stale lockfile would be helpful, but I think giving rTorrent sufficient time for a graceful shutdown is preferable!

@jeff47

This is something to consider adding to the docs and the command examples, and might reduce some issues people having with the container not starting because it finds a stale lock file.
It's also possible that a way for the startup script to check for and remove a stale lockfile would be helpful, but I think giving rTorrent sufficient time for a graceful shutdown is preferable!

Sgtm to add a note in the README (https://github.com/crazy-max/docker-rtorrent-rutorrent#notes). Feel free to open a PR. Thanks.


problems about generate results

Thank you for your excellent work!
I run python src/gen_samples.py --outdir=out --model={model_folder} --roots=data/roots/rootPositions_10k.txt --head_mesh=data/head.obj --seeds=0-99 --trunc=0.8 and then get the hair and noticed that the generated result has too short hair and messy hair, is this normal?
Also what is the significance of "--trunc" and "--trunc-cutoff" in the gen_samples.py ?

Hi, thanks for your interest in our work!

The flyaway strands in Fig. 1 are normal. It's indeed a phenomenon we can observe in real hairstyles, like「呆毛」
These very short strands are caused by PCA decoding, as they can not fully recover zero-length strands. You can filter them with our provided scripts, see: https://github.com/c-he/perm?tab=readme-ov-file#rendering.
We don't have an extensive ablation on trunc and trunc-cutoff, so we just pick a common value used in StyleGAN/EG3D.

Thank you for your reply! One last question, when will you open source the hair reconstruction code? I am very interested in this application! Or can you open source the code about “attach a cylinder mesh onto each strand segment”? Thank you!

Sure please stay tuned! I'm now working on other projects, but I think I will release it (at least the cylinder code) later this year.

Thank you for your reply! I'm very much looking forward to it!


feat: add drag and drop

I think I'm late for this.

https://github.com/t3dotgg/quickpic/issues/23


ThirdEye: set up separate CI pipeline

We need a separate CI pipeline to automatically unit and integration test proposed changes for ThirdEye. Currently, when submitting pull requests to the thirdeye (sub-)repo, CI/Travis only runs tests on Pinot - ThirdEye remains untested.

all working


Correct aria roles

Please change aria roles to the W3's recommendation: https://www.w3.org/WAI/ARIA/apg/patterns/carousel/
At least in Svelte implementation the role is currently listbox which is not correct and is producing warnings in Lighthouse.
Or make it possible to override these attributes.

Tx for reporting @Renkas 👍🏻 let's see what we can do 👌🏻

@Renkas please check @slidy/svelte - 3.5.3 version


show planet info on scroll instead of search

as we  can see if we want to get a info about any planet then we need to search for it what if you scroll down and get info
also that search can be moved on to the landing page you can search for satellites and other things right away

I have assigned you @yes-vikash

Hey, @mohittk actually I was not saying about autosuggestions in search instead I was saying that if we use select HTML tag instead of search then our user can select between the available planets, and by choosing any one of the planets they can get the info about that planet. In this way, our user will get info regarding planets without manually typing the name of that particular planet.

Hey, @mohittk actually I was not saying about autosuggestions in search instead I was saying that if we use select HTML tag instead of search then our user can select between the available planets, and by choosing any one of the planets they can get the info about that planet. In this way, our user will get info regarding planets without manually typing the name of that particular planet.

Still it's not clear to me. Join discord server https://discord.gg/FpEK9AqbCf
Discuss there.

Yes @mohittk, actually I didn't consider the factor API while thinking about my idea. But to let people know more information about more planets apart from the planets in our solar system I think we should mention the name of few planets which are unknown to most of the people because the average person who is gonna visit this website don't know the name of the single planet which is out of our solar system.

That's something really interesting. I think you should start working on it? should I assign you?

Yes @mohittk, actually I didn't consider the factor API while thinking about my idea. But to let people know more information about more planets apart from the planets in our solar system I think we should mention the name of few planets which are unknown to most of the people because the average person who is gonna visit this website don't know the name of the single planet which is out of our solar system.

That's something really interesting. I think you should start working on it? should I assign you?

definitely a cool idea to let people know more about exoplanets and all

Yes, but I think first of all I should create a new issue for this. What do you think?
definitely


Failed to login with SSO credentials

In my organisation we get the lynda subscription by logging in to lynda with sso login credentials, in this case I am unable to download
Steps to reproduce -
1. Lynda Course URL - https://www.lynda.com/AutoCAD-training-tutorials/160-0.html
2. Organization name - example.com ( for sample )
2.  Enter command -
python udemy-dl.py <course_url> -o <org_name>

Expected - Login and Download to start
Actual -
[*] : Username : user_name
[*] : Password : ****
[*] : Trying to login as (user_name)...
[-] : Failed to login ..

@anandchavan i might need your account to add support for SSO based accounts. you can email me

@r0oth3x49
Sharing account will be against policy.
However, I feel that the cookie support like which youhave given for udemy will help.
I may help you to test with the test branch once you start the implementation

@anandchavan sounds great, then i 'm closing this issue with enhancement and when i get some time i will implement.


Get table of content PDF file

Hi. Your lib is great. But how i can get Table of content within pdf?

What do you mean by "Table of content"?

Some PDF have "table of content " associated, like this:

In ios, how i can get it?

The bottom thumbnail bar serves this purpose by providing a visual table of contents of sorts. A textual representation of this is currently not supported.

I know that, but do you have a suggest how i can do that? I really need this feature.

You can do some digging around in the documentation, but stackoverflow would be a more appropriate environment for you to try to get this question answered. Feel free to open up a pull request if you ever find a solution!


Version 1.4.0 on Confluent Hub

It seems that the latest version was only released to Maven is there any reason it was not released on the Confluent Hub as well?
Thanks for the great library!

Hi @srolija THX for reaching out and your kind words!
It's a bit unfortunate that so far version 1.4.0 was never published to the HUB. I'm already in touch with the folks from Confluent and it should be released at the HUB rather soon. Will keep you posted!
As always I'm happy to learn about your use case for the sink connector. So maybe you can share what you are doing with it in your projects. You can reach out also via email if you want.

and v1.4.0 is published to the HUB as well. so grab it from there anytime you need it :)
https://www.confluent.io/hub/hpgrahsl/kafka-connect-mongodb

Great, thanks a lot! :)
We are trying to get it running in combination with https://github.com/blueapron/kafka-connect-protobuf-converter to get quick and dirty solution for logging of proto messages for some debugging queries.


Embed Preview not rendering with base_path set

In 0.11.0 the preview in the Embed setup fails to render and shows an "Invalid embed url" error when setting a base_path in settings.yml.  When I remove the base_path things seems to work.
Confirmed same behavior on a couple different browsers.  I'm happy to gather/share more details as needed especially if unable to reproduce.

Hi @randyoun  👋
Thanks for reporting this! We've been able to reproduce it and it has been fixed here.
🙂


iOS Overwrites

Some of the intended css is overwritten in iOS.

Hope this is the right place for this. Love your work!

I think #3 should have fixed this, thanks!


Add Azure Blob Storage backend

Hi!
I implemented support for Azure Blob Storage as a backend, inspired by the s3 and GS backends that already existed. I tried documenting and writing tests also based on what as available in the repo. Please get back to me if you have any feedback.
Thanks!

Hi @vincentsarago! Sorry for tagging you explicitly, but since you seem to be the main contributor - is this project still active?

sorry @christoe, I am the main contributor and sadly I have a lot other priority right now :-(

@christoe in order to allow the CI to run tests you must add "az" in https://github.com/developmentseed/cogeo-mosaic/blob/master/.github/workflows/ci.yml#L35 -> python -m pip install -e .["test,az"]

@vincentsarago Thanks and I totally understand. I was able to get the tests running now.


chore(scanner): misc Scanner v4 DB updates

Description
Updates include:

New GPG key

GPG keys for Postgres RPMs are now stored in a different location and have different contents. See the README.txt in https://apt.postgresql.org/pub/repos/yum/keys/

Actually import the GPG key for RPM verification

Somehow this was missed?

Creation of a new logical database, "stackrox"

This is to be used for non-ClairCore related tables Scanner v4 will require. There is probably no issue just putting them all in the same logical database, but it may be safer to just give ClairCore and ourselves separate logical databases
This is created in the init container using functions from docker-entrypoint.sh
This requires access to the postgres configuration, so that volume is now mounted
ROX_POSTGRES_DB - essentially an alias for POSTGRES_DB which is used by docker-entrypoint.sh

This is renamed to ensure the env var doesn't accidentally clash with docker-entrypoint.sh
It is expected the indexer/matcher read this env var to determine which database to use for non-ClairCore DB requirements

Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Here I tell how I validated my change
Tested via make -C scanner images e2e-deploy and then checking the init container's logs:
$ ks logs -f deploy/scanner-v4-db -c init-db
The files belonging to this database system will be owned by user "postgres".
This user must also own the server process.

The database cluster will be initialized with locale "en_US.utf8".
The default database encoding has accordingly been set to "UTF8".
The default text search configuration will be set to "english".

Data page checksums are enabled.

creating directory /var/lib/postgresql/data/pgdata ... ok
creating subdirectories ... ok
selecting dynamic shared memory implementation ... posix
selecting default max_connections ... 100
selecting default shared_buffers ... 128MB
selecting default time zone ... UTC
creating configuration files ... ok
running bootstrap script ... ok
performing post-bootstrap initialization ... ok
syncing data to disk ... ok

Success. You can now start the database server using:

    pg_ctl -D /var/lib/postgresql/data/pgdata -l logfile start

waiting for server to start....2024-01-12 05:57:36.922 UTC [21] LOG:  starting PostgreSQL 15.5 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 8.5.0 20210514 (Red Hat 8.5.0-20), 64-bit
2024-01-12 05:57:36.924 UTC [21] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2024-01-12 05:57:36.927 UTC [21] LOG:  listening on Unix socket "/tmp/.s.PGSQL.5432"
2024-01-12 05:57:36.933 UTC [24] LOG:  database system was shut down at 2024-01-12 05:57:36 UTC
2024-01-12 05:57:36.939 UTC [21] LOG:  database system is ready to accept connections
 done
server started
Creating stackrox database
CREATE DATABASE

waiting for server to shut down....2024-01-12 05:57:37.114 UTC [21] LOG:  received fast shutdown request
2024-01-12 05:57:37.118 UTC [21] LOG:  aborting any active transactions
2024-01-12 05:57:37.120 UTC [21] LOG:  background worker "logical replication launcher" (PID 27) exited with exit code 1
2024-01-12 05:57:37.122 UTC [22] LOG:  shutting down
2024-01-12 05:57:37.123 UTC [22] LOG:  checkpoint starting: shutdown immediate
2024-01-12 05:57:37.242 UTC [22] LOG:  checkpoint complete: wrote 918 buffers (2.9%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.036 s, sync=0.064 s, total=0.121 s; sync files=301, longest=0.004 s, average=0.001 s; distance=4212 kB, estimate=4212 kB
2024-01-12 05:57:37.251 UTC [21] LOG:  database system is shut down
 done
server stopped
Done initializing database

From here, you can also check the main scanner v4 db container:
postgres=# \l
                                                List of databases
   Name    |  Owner   | Encoding |  Collate   |   Ctype    | ICU Locale | Locale Provider |   Access privileges
-----------+----------+----------+------------+------------+------------+-----------------+-----------------------
 postgres  | postgres | UTF8     | en_US.utf8 | en_US.utf8 |            | libc            |
 stackrox  | postgres | UTF8     | en_US.utf8 | en_US.utf8 |            | libc            |
 template0 | postgres | UTF8     | en_US.utf8 | en_US.utf8 |            | libc            | =c/postgres          +
           |          |          |            |            |            |                 | postgres=CTc/postgres
 template1 | postgres | UTF8     | en_US.utf8 | en_US.utf8 |            | libc            | =c/postgres          +
           |          |          |            |            |            |                 | postgres=CTc/postgres
(4 rows)

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

This was created as it was requested in a review for https://github.com/stackrox/stackrox/pull/9042

Looks good to me, one question/comment regarding:

It is expected the indexer/matcher read this env var to determine which database to use for non-ClairCore DB requirements

Are the updates to read this in indexer/matcher part of another PR?

Yes, the intention is to add this to another PR

Looks good to me, one question/comment regarding:

It is expected the indexer/matcher read this env var to determine which database to use for non-ClairCore DB requirements

Are the updates to read this in indexer/matcher part of another PR?

I'm going to remove the stuff about using the separate DB in favor of taking a step back to see how others feel about how to approach it