[ERROR]  I get Python Exception while trying to download

I get this when I try to download something. Is this normal can I somehow fix this?
Exception in thread Thread-1 (download_and_convert_hls_stream):
Traceback (most recent call last):
  File "C:\Users\kemmuammer\AppData\Local\Programs\Python\Python312\Lib\threading.py", line 1073, in _bootstrap_inner
    self.run()
  File "C:\Users\kemmuammer\AppData\Local\Programs\Python\Python312\Lib\threading.py", line 1010, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\kemmuammer\Downloads\aniworld_scraper-main\aniworld_scraper-main\src\logic\downloader.py", line 60, in download_and_convert_hls_stream
    subprocess.run(ffmpeg_cmd, check=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
  File "C:\Users\kemmuammer\AppData\Local\Programs\Python\Python312\Lib\subprocess.py", line 548, in run
    with Popen(*popenargs, **kwargs) as process:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\kemmuammer\AppData\Local\Programs\Python\Python312\Lib\subprocess.py", line 1026, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Users\kemmuammer\AppData\Local\Programs\Python\Python312\Lib\subprocess.py", line 1538, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [WinError 2] Das System kann die angegebene Datei nicht finden

@KemMuammer
subprocess.run(ffmpeg_cmd, check=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
Did you download FFMPEG and either installed it or placed the portable version in the folder?

I did and added to the src folder. Should I rename it to only ffmpeg?

no you need the exe or have it installed @KemMuammer

@wolfswolke Yeah that worked thanks and also I dont know why but the ffmpeg download I have does not come with the exe variant. I dont know why. But that is a different issue.


Importing themes across projects in a monorepo

Describe the bug
howdy y'all! inside of my monorepo, i'm trying to define vanilla-extract themes one project and use them in another. i've been struggling with this for a couple of days, and was hoping someone could take a look at what i'm doing and point out what i'm doing wrong
here's a repo where the error can be easily reproduced, along w/ repro instructions https://github.com/jackmccloy/vanilla-extract-monorepo-issue
thanks! any help is appreciated
Reproduction
https://github.com/jackmccloy/vanilla-extract-monorepo-issue
System Info
System:
    OS: macOS 12.4
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 24.91 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 16.15.0 - /usr/local/opt/node@16/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.5.5 - /usr/local/opt/node@16/bin/npm
  Browsers:
    Chrome: 102.0.5005.61
    Firefox: 90.0.2
    Safari: 15.5

Used Package Manager
yarn
Logs
(in the browser console)

Uncaught Error: Styles were unable to be assigned to a file. This is generally caused by one of the following:

- You may have created styles outside of a '.css.ts' context
- You may have incorrect configuration. See https://vanilla-extract.style/documentation/setup
    at getFileScope (vanilla-extract-css-fileScope.browser.esm.js:25:11)
    at createVar (vanilla-extract-css.browser.esm.js:245:7)
    at vanilla-extract-css.browser.esm.js:293:12
    at walkObject (vanilla-extract-private.esm.js:34:20)
    at walkObject (vanilla-extract-private.esm.js:36:20)
    at walkObject (vanilla-extract-private.esm.js:36:20)
    at createThemeContract (vanilla-extract-css.browser.esm.js:292:10)
    at index.mjs:3:21

Validations

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

Hi 👋
When I load your repo and run your instructions I was having different issues but I think I can see the issue anyway. You're compiling your themes package into a single file (with tsup) without using the vanilla-extract plugin. This is likely why you're getting the above error. You'll either need to setup tsup to compile the vanilla-extract code into CSS files at the package level, or get your storybook to look at the source code of the package rather than the compiled output.

That put me on the right path! fiddling around a bit w/ tsup did the trick, and now i'm back on track. Thanks so much for the quick response (and great work in general)

@jackmccloy do you mind updating your repo with the change that made this work? also exploring using vanilla-extract in a turborepo setting and having some difficulties getting setup.

hey @danstans - I made the change in a private repo that I can't share, the repo I linked to was a minimal repro of the issue. looking back at my commits, the change mostly boiled down to packaging css at the package level by adding this to the exports key in my package's package.json
"./dist/index.css": {
  "import": "./dist/index.css",
  "require": "./dist/index.css"
}

tsup doesn't resolve the local path. You need to use the exact path you use when importing the *.css.ts file.
Example
src/index.css.ts
import { style } from '@vanilla-extract/css'

export const headerClass = style({})

src/index.ts
import { headerClass } from './index.css'

tsup.config.ts
This will work. tsup will not bundle the src/index.css.ts into a single file.
import { defineConfig } from 'tsup'

export default defineConfig({
  bundle: true,
  external: ['./index.css.ts']
})

or if you don't define all of *.css.ts in the external config one by one, you can use a RegeExp instead:
import { defineConfig } from 'tsup'

export default defineConfig({
  bundle: true,
  external: [/^\.\/[\w-_]+\.css$/]
})

This won't work. tsup still bundles the src/index.css.ts into a single file.
// INVALID CONFIGURATION

import { defineConfig } from 'tsup'

export default defineConfig({
  bundle: true,
  external: ['src/index.css.ts']
})


Fix RTL3b and RTL4b.

LGTM

One small comment, but otherwise :+1:

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

ably-ios/ARTRealtime.m, line 442 [r1] (raw file):
True... was there already, I just copypasted and didn't notice. Fixed at a60f848.

Comments from the review on Reviewable.io


Restore cat command to kshell

The kshell builtin cat command was previously removed in commit 83a747243f9c34773bf661b64e965d9c57e48945.  The command is restored in this change.
Fixes: #242

Thanks! I replaced console_write with printf because it's an abstraction on top of the console (console is one way to write to the screen but it's low level).

Thanks! I replaced console_write with printf because it's an abstraction on top of the console (console is one way to write to the screen but it's low level).


Removed bootstrap dependency

also the input width scales to the container now.

closing in favor of https://github.com/EducationLink/vue-tel-input/pull/13


added simple predict api

Pull Request Test Coverage Report for Build dd983e2782f3b9407493c4195ca295c6a205110d-PR-14

1 of 41 (2.44%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-7.7%) to 64.169%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/tflite_tensor.ex
0
3
0.0%

lib/tflite_interpreter.ex
1
38
2.63%

Totals

Change from base Build 2bfe4a9d71c0268ddd326538dd6dd91a67ad316c:
-7.7%

Covered Lines:
197

Relevant Lines:
307

💛  - Coveralls


[Feat]: Adding Piped

What feature?
Piped is an alternative privacy-friendly YouTube frontend which is efficient by design.
Add screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Go ahead and add it @prasanta10


Fix clean state initialization

Due to the unmasking of exceptions during ChainDB initialization, it is the case right now that the exit code is different from DatabaseCorruption and therefore the withCheckedDb function considers that it has to create the clean file. Because of this, a Ctrl+C during initialization would skip the validation on the next startup.
This PR ensures that the clean file is only removed after the chainDB is initialized (and therefore potentially modified) and from that point on it can be created again.

initial status
killed when
on SIGKILL
on DatabaseCorruption
on other exception (e.g. ^C)

clean
during initialization
unclean → clean
clean → unclean
clean

clean
after initialization
unclean
unclean
clean

unclean
during initialization
unclean
unclean
clean → unclean

unclean
after initialization
unclean
unclean
clean

I amended your commit with a tiny change, just renaming wrapper:
$ git show
commit ec01f675b0fe647b04e55d6c246d7bfe60bbc18b (HEAD -> jasagredo/fix-clean-state-initialization)
Author: Nicolas Frisby<EMAIL_ADDRESS>Date:   Thu Dec 2 10:13:48 2021 -0800

    TOSQUASH: rename wrapper

diff --git a/ouroboros-consensus/src/Ouroboros/Consensus/Node.hs b/ouroboros-consensus/src/Ouroboros/Consensus/Node.hs
index f530ebe37..7c655a3bb 100644
--- a/ouroboros-consensus/src/Ouroboros/Consensus/Node.hs
+++ b/ouroboros-consensus/src/Ouroboros/Consensus/Node.hs
@@ -282,7 +282,7 @@ runWith :: forall m addrNTN addrNTC versionDataNTN versionDataNTC blk p2p.
   -> m ()
 runWith RunNodeArgs{..} LowLevelRunNodeArgs{..} =

-    llrnWithCheckedDB $ \(LastShutDownWasClean lastShutDownWasClean) wrapper ->
+    llrnWithCheckedDB $ \(LastShutDownWasClean lastShutDownWasClean) continueWithCleanChainDB ->
     withRegistry $ \registry -> do
       let systemStart :: SystemStart
           systemStart = getSystemStart (configBlock cfg)
@@ -315,7 +315,7 @@ runWith RunNodeArgs{..} LowLevelRunNodeArgs{..} =
       chainDB <- openChainDB registry inFuture cfg initLedger
                 llrnChainDbArgsDefaults customiseChainDbArgs'

-      wrapper chainDB $ do
+      continueWithCleanChainDB chainDB $ do
         btime <-
           hardForkBlockchainTime $
           llrnCustomiseHardForkBlockchainTimeArgs $

I squashed that into your commit and force-pushed it back up.

bors merge


Fix root view cleanup

The tearDownReactContext method of ReactInstanceManager was not removing
root views from its private mAttachedRootViews member.
The perceived problem was that when an Android application with multiple
root views was reloaded, root JS components that existed prior to that
reload were being remounted. They did not appear on screen, but React
life cycle methods such as componentWillMount were being called.
Test Plan
I've created a MCVE project to illustrate the problem.
https://github.com/viotti/react-native-arvc-mcve
Related PRs
None.
Release Notes
[ANDROID] [BUGFIX] [ReactInstanceManager.java] - Fix root view cleanup

Rebased. Disregard the previous close & reopen, happened by accident.


Add experimental version of leiden_communities

This function is not yet defined or implemented in NetworkX, and there is high probability the API may change once added to NetworkX. For now, this is "best effort" and simply mimics PLC leiden.

It appears that theta parameter is not used, so I removed it for now.
First, it is ignored in the C->Python API here:
https://github.com/rapidsai/cugraph/blob/1f1cace91e9b97c11dc6d57026360c3a0a919676/cpp/src/c_api/leiden.cpp#L128-L141
Second, it is ignored in the refinement step here (done in https://github.com/rapidsai/cugraph/pull/3990):
https://github.com/rapidsai/cugraph/blob/1f1cace91e9b97c11dc6d57026360c3a0a919676/cpp/src/community/detail/refine_impl.cuh#L107-L124

Begin adding to networkx here: https://github.com/networkx/networkx/pull/7743

re: 24.12 vs. 25.02 - on second thought, @eriknw pointed out the potential for the API to change which would make 24.12 incompatible with future NX versions.


asterisk push configure example

hi:
where I can found a example that is the push configure with ari.  I don't  understand Asterisk() Interface...

Ah, I had not noticed the new configuration options available now.  We will need to add support for those.

Pull requests are welcome; otherwise, we'll try to get to it when we can.


Docker-based pre-commit configuration example

Description:
GItleaks can be executed via docker-based pre-commit hook.
This is a basic example how to implement it.
Checklist:

[ ] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

sick, thanks @pab1it0!


PHP Notice:  Undefined variable: order_id in /var/www/html/tlc-dev/wp-content/plugins/woocommerce/includes/class-wc-ajax.php on line 445

Steps to reproduce:

Go through checkout on the frontend to create an Order with a variable product (variation attribute is taxonomy). Use BACS to pay the order.
Go to admin and mark the order as "Processing" in the Admin -> WooCommerce -> Orders page.
The message in the subject will pop up in the error log.

You must define order_id before and the best way to do this is by using if statement isset see example :
global $woocommerce;
if (isset ($order_id)) {
$order = new WC_Order($order_id);
$items = $order->get_items();
foreach ($items as $item) {
$product_id = $item['product_id'];
if ($product_id ==  your product id) {
//do something
} else {
//do something
}
}

}

this is working with version 1.0 of this plugin and latest WooCommerce3


Add copy text above charts.

A few misc. TODOs left, but otherwise done.  Can be merged as-is to keep things moving, or I can make revisions tomorrow.

Note that I'm very open to having the text live somewhere else or otherwise improve the structure of this page (separate the charts into separate components / whatever).  But I didn't feel like making hard decisions like that right now.


fix(uiSelect): directives being created 2 times on ui-select-choices

Fixes #1000

Closing, as there are better fixes in open PRs.


Custom zip name

Hello.
could you please make the zip filename customizable.
Current pattern: BACKUP_FILE_ZIP="${BACKUP_DIR}/backup.${NOW}.${ZIP_TYPE}"
Desired pattern: BACKUP_FILE_ZIP="${BACKUP_DIR}/${ZIP_NAME}${NOW}.${ZIP_TYPE}"
Thank you!

I haven't encountered a scenario where I need to modify file names. Could you please share the specific context in which you encountered this?


Add a proper webhook-style integration for phabricator

Now that phabricator has an outgoing webhook integration, I think we could replace our https://github.com/zulip/phabricator-to-zulip integration with another incoming webhook integration, sharing all the formatting code with our other Git integrations.
See https://secure.phabricator.com/book/phabricator/article/webhooks/ for details on Phabricator's support.
Not tagging as a priority because Phabricator seems to lack the filtering features of GitHub; you get a pretty complete firehose of everything.
So we might want to carry out our vision of having a feature for limiting event types before we do this.

In my company we just decided to switch from GitLab to Phabricator. The GitLab integration with Zulip was great, but the Phabricator integration mentioned by @timabbott is absolutely (outdated)[https://secure.phabricator.com/T11856] and not working at all.
How can we get event notifications from Phabricator to Zulip?

@timabbott: I can get started on this! :)

@zulipbot claim

I got Phabricator set up on a virtual box. The main issue now is that Phabricator does not provide a list of possible events, and given the size of Phabricator I'm sure that there are tons. So which subset of events should I try generating fixtures for @timabbott and @eeshangarg?

Also, @isolin, I know that it's been a long while, but do you have a set of events that you'd like to see Zulip support?

I would focus on the core things that people care about with any version control system integration:

Code push
New pull request
New issue
Issue update
Maybe some other things from zerver/lib/webhooks/git.py

I think it's totally reasonable for us to merge an integration that just covers the above, and then get user feedback on what else to add in a later PR (there's no reason the first PR needs to cover every event).

http://webhook.site/7760feaa-0a45-4e60-94af-ab0e327ae061


Add react-appinsights

This PR adds react-appinsights to the demo project.
React-appinsights is a small wrapper of Application Insights SDK that adds features that are used in React applications, such as automated page tracking through react-router-dom history and React components usage statistics.

Waiting for the completion of this PR, I've prepared a version of this repo supporting react-appinsights, here (here  is the source). I'll move this demo project to the Azure-Samples org.

Closing in favor of https://github.com/Azure-Samples/application-insights-react-demo/pull/4/files
Thanks @pviotti


Multiple uses of the same action results in file conflict

Same issue as #131: I am trying to run ssh-deploy in two steps, one to copy the program artifacts, and one to copy a systemd config that goes in a different location. Both steps have SCRIPT_AFTER, but when the second one tries to run, the script file already exists and is not overwritten, so whatever I've put in the second step doesn't execute.

Here is a minimal repro
name: CI

on: [push, pull_request]

jobs:
  deploy:
    name: Deploy
    runs-on: ubuntu-latest
    steps:
      - uses<EMAIL_ADDRESS>        with:
          path: ./deployment/bin/myservice
          write-mode: overwrite
          contents: |
             #!/bin/sh
             echo hello world
      - uses<EMAIL_ADDRESS>        with:
          path: ./deployment/systemd/myservice.service
          write-mode: overwrite
          contents: |
            [Unit]
            Description=Application server

            [Service]
            Type=simple
            ExecStart=/home/${{ secrets.DEPLOY_USER }}/bin/myservice

            [Install]
            WantedBy=multi-user.target
      - uses: easingthemes/ssh-deploy@v4
        with:
          SSH_PRIVATE_KEY: ${{ secrets.DEPLOY_SSH_KEY }}
          REMOTE_HOST: ${{ secrets.DEPLOY_IP_ADDRESS }}
          REMOTE_USER: ${{ secrets.DEPLOY_USER }}
          SOURCE: ./deployment/bin/
          TARGET: /home/${{ secrets.DEPLOY_USER }}/bin/
          SCRIPT_AFTER: |
            chmod +x /home/${{ secrets.DEPLOY_USER }}/bin/myservice
      - uses: easingthemes/ssh-deploy@v4
        with:
          SSH_PRIVATE_KEY: ${{ secrets.DEPLOY_SSH_KEY }}
          REMOTE_HOST: ${{ secrets.DEPLOY_IP_ADDRESS }}
          REMOTE_USER: ${{ secrets.DEPLOY_USER }}
          SOURCE: ./deployment/systemd/
          TARGET: /home/${{ secrets.DEPLOY_USER }}/.config/systemd/user/
          ARGS: '--mkpath'
          SCRIPT_AFTER: |
            systemctl --user daemon-reload # Make sure systemd picks up the new Unit file
            systemctl --user enable myservice
            systemctl --user restart myservice

Fixed in the latest version v4.1.9 - https://github.com/easingthemes/ssh-deploy/pull/167/files#diff-9b163fdf32814f608c5eb67fb0841aa3dfa17545829fbbea2fe053b5991796e6R16.

@andsnpl Can you corroborate the fix and close the issue?


fix(graphql): ListCooccurrences API error

When debugging cooccurrence issue, I found bug in ListCooccurrences API.
The API is almost just for debugging purpose, though.

minumim_should_match on queries with no should query causes 0 result when filter is set
remove unused sort-vote logic

coverage: 88.507% (+0.1%) from 88.398%
when pulling 14a6b66fc8211ef12867d40373b1cf1b5ae5acb6 on fix-list-cooccurrence
into 4857e9fffb4d7ff9dedd7ff175f31fb85a4f0c7e on master.


Fix updating of layer properties

The update in commit 5d6d5e7 following issue #127 introduced a bug which made it impossible to set the visibility (and probably even other) properties of the layer once it had been created. This PR fixes it.

I added tests verifying the absence of the bug fixed in #127
I added tests that verify the correct updating of the visibility property
I slightly re-arranged how the different if-clauses were structured, hopefully making it easier to read and to avoid code duplication within that logic.

Feel free to comment on it.
(Having said this, I'd like to take the occasion to ask you @tombatossals to take a look at #120 as well. I'd like to discuss with you an upgrade scenario as I did already upgrade your v3.5 branch and use it in development. I'd like to merge it back in. thx :smiley:  )

hmm...not sure why the tests fail, they succeed on my machine

Ok, found the reason why Travis fails. It's because I added a test that verifies a bug, but I didn't commit the compiled files from the dist folder as that normally causes issues when merging with master.
@tombatossals Has that to be done?

Hi @juristr, thanks for the PR, I'm merging it. Sorry for the delay, I have been disconneted a few days.

@tombatossals The current build on master fails after merging this PR, because I didn't commit also the dist files (as mentioned in my comment).
https://travis-ci.org/tombatossals/angular-openlayers-directive
Do you expect the dist files to be compiled and submitted with the PRs as well??


Moved Telemetry Counts for new Gift Entry DI to Batch Mode

Critical Changes
Changes

Telemetry calculation of Gift Batch Data Import records have been moved to a batch context to combat the REQUEST_RUNNING_TOO_LONG error when a COUNT() query is executed in orgs with large data volumes

Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-040424

Tracking W-040424


Add feature_flags to v1 transformer for use in Aurora

What's this PR do?
🎏 Adds feature_flags to the v1 users transformer so that the field can be surfaced in Aurora. I've tested locally by making sure this field shows up when making a request through Paw. It'll be helpful for badges to be able to edit and view this field in Aurora.
How should this be reviewed?
👀
Relevant Tickets
🤠
Checklist

[ ] Documentation added for changed endpoints.
[ ] Tests added for new features/bug fixes.
[ ] Post a message in #api if this includes something that causes a rebuild!

@mendelB yup Aurora still uses the v1 API!


Step by Step API documentation

This section shall include purpose, the workflow and some examples of how it will perform in its intended use cases.
Overview of the API
Examples
Errors- A full list of error messages with corresponding error codes explaining what each error means.
Authentication- This section shall explain how authentication works in the system. (Some details about key Cloak).
API’s parameters (its headers, paths, queries, and request body parameters)
Request examples for your API
Corresponding request/response examples of API


Convert old strings files to new format

Description
Convert old strings files using <strings> or <vars> elements to the new format (#1188) that uses <variables> element.
Motivation and Context
The new format has more features and configuration in a single format is easier to maintain.
How Has This Been Tested?
Existing tests pass.
Documentation and Compatibility
Mention in release notes.

@infotexture,

Will existing overrides in a /cfg/common/vars/ folder of a custom PDF plug-in just work?

Variables under PDF2 are not overrides. They are just PDF2's variable definitions. If you list variables under org.dita.base and org.dita.pdf2, there are no duplicates. So PDF2 doesn't override anything.

If those overrides begin with a  root element as in previous versions, will that still work, or would they need to change that to the new  root element used here?

PDF2 has no overrides but some other plug-in might have overrides. The format of the override doesn't matter. You can still mix all three formats and the one specified last in the configuration will win, just like before.

If an existing custom HTML plug-in has an override file at xsl/common/strings-de-de.xml, would it still override the common German strings as in previous versions if it contains strings/str constructs, or would those need to be rewritten to the variables/variable format used here?

Plug-ins don't need to change anything. The logic how strings are resolved has not changed, just the format.

@infotexture This PR could serve as basis for deprecating the two old formats and suggesting users to move over to new (Since 2014!) format.

@infotexture Can't reproduce the problem of not seeing the modifications when strings are changed.


hub pull-request does not populate list of changes

My workflow is relatively common - one 'main' repo on the organization account, several contributors with their public forks of the repo. I am usually working on the features in separate branches that I push into my public fork, then using hub I open pull requests.
Unfortunately for some time now, hub does not pre-populate pull request message with list of commits. This was very useful functionality, without it I have to open separate terminal window with git log just to see what has been changed in this particular feature branch. It could be something with my configuration, but I am having difficulties trying to find what's wrong. Is this common issue?
My setup:

upstream remote represents main organization repository
origin remote represents my individual repository

After I push all my commits to origin/feature-branch and issue hub pull-request, my editor opens and displays following pre-populated message:
# Requesting a pull to upstream:master from skrobul:feature-branch
#
# Write a message for this pull request. The first block
# of text is the title and the rest is description.

Please note that it kind of works, I can write my own message and submit pull request without problem, it's just the list of commits that is missing from the message. Correct me if I'm wrong, but it looks like hub should insert commented list of commits.
$ hub --version
git version 2.1.3
hub version 2.2.0-preview1

$ git remote show origin
* remote origin
  Fetch URL: git@github.mycompany.com:skrobul/salt-marek.git
  Push  URL: git@github.mycompany.com:skrobul/salt-marek.git
  HEAD branch: master
  Remote branches:
    master                                     tracked
    feature-branch                             tracked
  Local branches configured for 'git pull':
    master                 merges with remote master
  Local refs configured for 'git push':
    master                       pushes to master                       (fast-forwardable)
    feature-branch                  pushes to feature-branch                  (up to date)

$ git remote show upstream
* remote upstream
  Fetch URL: git@github.mycompany.com:myorg/salt-master.git
  Push  URL: git@github.mycompany.com:myorg/salt-master.git
  HEAD branch: master
  Remote branches:
    master                                           tracked
    refs/remotes/origin/feature-branch               stale (use 'git remote prune' to remove)
  Local ref configured for 'git push':
    master pushes to master (up to date)

Any ideas how to fix/troubleshoot this?

@skrobul Did you managed to pre-populate message? May I request you to provide gist of it?

@hisavali yes, after I swapped origin with feature remote, the commit messages get prepopulated as expected. I don't have any specific example on hand, but the message was essentially the commented output of git log

@skrobul I'm glad you solved it and thanks for the update. Our branch detection isn't perfect right now. We'll work on improving it in future releases. In the meantime, I'm closing this because we're aware of this issue through other reports. #928 #877 #789 #461


Fix server terminated abnormally When the targetList of hierarchical query contains subquery

fixed #181
--  create table and build data
create table area(id int, name text, pid int);
insert into area values(1, 'china', 0);
insert into area values(101, 'zhejiang', 1);
insert into area values(102, 'jiangsu', 1);
insert into area values(103, 'heilongjiang', 1);
insert into area values(10101, 'hangzhou', 101);
insert into area values(10102, 'ningbo', 101);
insert into area values(10103, 'shaoxing', 101);
insert into area values(10104, 'wenzhou', 101);
insert into area values(1010101, 'binjiang', 10101);
insert into area values(1010102, 'shangcheng', 10101);
insert into area values(1010103, 'xihu', 10101);
insert into area values(1010101, 'gongshu', 10101);

select * from (select * from area) start with name = 'hangzhou' connect by  pid = prior id;
server closed the connection unexpectedly
        This probably means the server terminated abnormally
        before or while processing the request.
The connection to the server was lost. Attempting reset: Failed.

After testing, this PR can be merged.


Add support for deploying broker with AnsibleOperator

Describe what this PR does and why we need it:
Allows deployment of broker through AnsibleOperator

This PR has gotten out of date to the point where it was easier to manually pull over relevant changes into another PR #1078. Most of the work there is based around this one.


Simplest 2 player perfect information game?

As the title says, I'm wondering which game is the simplest (as in smallest, fastest to train on) for 2 players and perfect information. It looks like it should be Pathfinding but I am unsure how large the grid is and whether I can reduce the rows/columns. I'm not sure if multiagent Pathfinding is zero-sum or cooperative but that matters less.

Hi, pathfinding seems to be a good one. You can actually specify kExampleMultiAgentGrid to customize the grid you want. This game is general-sum, e.g., see this paper

Tic-Tac-Toe is my go-to example in this space. There are other games parameterizable by columns so you can adjust the size to very small (such breakthrough to maybe 4x4, hex to 3x3 or maybe even 2x2?).
You can also use game trees encoded in the gambit .efg format if you want to go really small (see games/efg_game), e.g. examples you'd find in a game theory text book.

Thanks for the replies. I am going to work with tic-tac-toe for now, but may need to get even smaller in the future. Is there a section on here that explains the gambit format in more detail? I also had a couple of follow up questions.

Here (https://github.com/deepmind/open_spiel/issues/486) was an issue I was getting but corrected but now I am getting NashConv values that grow (rapidly) to very large values which I don't think makes sense in tictactoe. Without getting into the details of the algorithm I am implementing, the NashConv values are as high as (t=1 nashconv=60, t=2 nashconv=190, t=3 nashconv=300, and continue to grow. Is this behavior indicative of any errors I might be making similar to in the other post?

While I am posting, another problem I have is trying to print out all of the values in colab for arrays but they print out as [0.1, 0.01, 0.1, ..., 0.0, 0.02]. I have searched all over to force all of the values to be shown but no luck. I'm wondering if anyone knows how to do this.

Hi @sullins2,
The gambit EFG representation is documented here. There are some examples in games/efg.

Yes that issue you're quoting was resolved once we switched to use State::HistoryString() as the cached value for a history. The NashConv in Tic-Tac-Toe has an upper-bound of 4, so if you're getting values above that, there's definitely something broken somewhere. It would unlikely be related to the other post, but that's an indication of a problem. As a quick test, could you measure the NashConv of the uniform policy in Tic-Tac-Toe? What value does that give? If it's in the correct range (between 0 and 4) then I suspect maybe our policy that you're sending to NashConv could be badly formed? Is it easy to submit a simple PR that reproduces the problem that we can look through? (Or maybe send me a colab by email or something?)

I'm not sure how either, I usually just write a for loop. =)

Thanks for the offers and the reply, @lanctot . I ended up getting both issues resolved. What happened was I was using the cfr example (the smaller one) and in that example the current policy is accumulated to get the average policy. This is sent directly to nash_conv() and so I assumed that it was normalizing internally. I actually didn't need the average and so I took that out and it worked fine.  Nashconv also appears to be working for tictactoe in case anyone sees this in the future. Thanks for the help (@rezunli96 too).


Log what resources are created in K8s

I ran into an issue debugging the example Tomcat operator (and a MariaDB example of my own), where the operator processes cr.yaml and creates the resources transparently (K8s Deployment and Service objects defined by the chart), but pods themselves are never created as this action was blocked by an OpenShift Security Context Constraint. It appeared as though the operator wasn't actually deploying resources when processing cr.yaml, however it actually did, it just wasn't obvious by viewing the output of oc logs <operator-pod-id>.
After giving the code a rough glance, it appears that these messages are logged via logrus. It would be ideal to have something logged in addition to processing <namespace>/my-tomcat using values: replicaCount: 2 which states the Kind and the name of resources that are created by the Helm operator.

Seems to be a few concerns here:

do we rely on logging directly from Helm or can we customize it?

showing a git style diff in the logging output is pretty helpful

do we use log levels today?

if so, how can you control it?

does the Operator write out to the status block?

To quickly answer whether the "status" field gets written to, it does get populated with info that differs somewhat from what is logged to the pods stdout (which is what I believe populates the output of oc logs <operator-pod-id>).
Looking at the status: block of the tomcat-operator pod shows the following:
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: 2018-10-13T01:10:50Z
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: 2018-10-15T18:23:07Z
    status: "True"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: 2018-10-13T01:10:49Z
    status: "True"
    type: PodScheduled
  containerStatuses:
  - containerID: docker://081324cc0935ed07b299e0b964beb175d0d796e07494985f8b167409b8734b53
    image: scan.connect.redhat.com/p7153749677c0f9fde6d2669c3d7785cffc2124c4f/tomcat-operator:0.1.5
    imageID: docker-pullable://scan.connect.redhat.com/p7153749677c0f9fde6d2669c3d7785cffc2124c4f/tomcat-operator@sha256:bf76931832fa3559029d54e8f0908b0d108da9453c96ed34a2de26b662984ac6
    lastState:
      terminated:
        containerID: docker://a6f1f4cce926976505c914d4aac8a8398e129c4a7327c6037dbdcaa956d2ac48
        exitCode: 1
        finishedAt: 2018-10-15T18:20:44Z
        reason: Error
        startedAt: 2018-10-15T18:20:40Z
    name: tomcat-operator
    ready: true
    restartCount: 3
    state:
      running:
        startedAt: 2018-10-15T18:23:06Z
  hostIP: <IP_ADDRESS>
  phase: Running
  podIP: <IP_ADDRESS>
  qosClass: BestEffort
  startTime: 2018-10-13T01:10:50Z

Whereas the (truncated) log output of the operator is:
time="2018-10-15T18:23:07Z" level=info msg="Go Version: go1.10.4"
time="2018-10-15T18:23:07Z" level=info msg="Go OS/Arch: linux/amd64"
time="2018-10-15T18:23:07Z" level=info msg="operator-sdk Version: 0.0.6+git"
time="2018-10-15T18:23:08Z" level=info msg="Registering Components."
time="2018-10-15T18:23:09Z" level=info msg="Watching apache.org/v1alpha1, Tomcat, tomcat-operator,<PHONE_NUMBER>"
time="2018-10-15T18:23:09Z" level=info msg="Starting the Cmd."
time="2018-10-15T18:23:09Z" level=info msg="processing tomcat-operator/my-tomcat"
time="2018-10-15T18:23:09Z" level=info msg="using values: replicaCount: 2\n"
time="2018-10-15T18:23:09Z" level=error msg="a release named helm-app-operator-my-tomcat already exists.\nRun: helm ls --all helm-app-operator-my-tomcat; to check the status of the release\nOr run: helm del --purge helm-app-operator-my-tomcat; to delete it"
time="2018-10-15T18:23:10Z" level=info msg="processing tomcat-operator/my-tomcat"
time="2018-10-15T18:23:10Z" level=info msg="using values: replicaCount: 2\n"
time="2018-10-15T18:23:10Z" level=info msg="adding ownerrefs to file: tomcat/templates/appsrv-svc.yaml"
time="2018-10-15T18:23:10Z" level=info msg="adding ownerrefs to file: tomcat/templates/appsrv.yaml"
time="2018-10-15T18:23:23Z" level=info msg="processing tomcat-operator/my-tomcat"
time="2018-10-15T18:23:23Z" level=info msg="skipping tomcat-operator/my-tomcat because resource version has not changed"
time="2018-10-15T18:23:28Z" level=info msg="processing tomcat-operator/my-tomcat"
time="2018-10-15T18:23:28Z" level=info msg="skipping tomcat-operator/my-tomcat because resource version has not changed"
time="2018-10-15T18:23:33Z" level=info msg="processing tomcat-operator/my-tomcat"
time="2018-10-15T18:23:33Z" level=info msg="skipping tomcat-operator/my-tomcat because resource version has not changed"
...

Great timing @jsm84. I don't think this will completely answer your need, but check out the latest PR I submitted this morning (#51), which includes a commit with a bunch of logging updates that makes it more obvious what's happening (and feel free to leave comments).
A few things not present in that PR that still need to be done are:

Configurable logging level. Right now there's no way to see debug logs.
Logging a git style diff when changes made to the CR trigger a release installation or update.

Looking at the status: block of the tomcat-operator pod...

I think the status @robszumski was referring to is the CR status, not the helm operator. It contains the full release information (e.g. release name and manifests) for the currently deployed release.

Whoa that's a hell of a status ;)

Closed by #61
We'll track log level configurability in #63.


[BUG]: Building the typescript sample throws errors

Describe the bug
It's not possible to build the typescript sample project.
To Reproduce
Steps to reproduce the behavior:

Go into samples/typescript
Run make configure && make build

Expected behavior
tsc builds the project
Experienced behavior
tsc throws errors:
src/index.ts:182:35 - error TS2345: Argument of type '{ start: any; limit: number; }' is not assignable to parameter of type 'OneParams'.
  Object literal may only specify known properties, but 'start' does not exist in type 'OneParams'. Did you mean to write 'stats'?

182     users = await User.find({}, { start, limit: 25 });
                                      ~~~~~

src/index.ts:183:19 - error TS2339: Property 'start' does not exist on type 'Paged<Entity<{ pk: { type: StringConstructor; value: string; }; sk: { type: StringConstructor; value: string; }; accountId: { type: StringConstructor; required: boolean; }; ... 7 more ...; gs1sk: { ...; }; }>[]>'.

183     start = users.start;
                      ~~~~~

src/index.ts:184:18 - error TS2339: Property 'start' does not exist on type 'Paged<Entity<{ pk: { type: StringConstructor; value: string; }; sk: { type: StringConstructor; value: string; }; accountId: { type: StringConstructor; required: boolean; }; ... 7 more ...; gs1sk: { ...; }; }>[]>'.

184   } while (users.start);
                     ~~~~~

Found 3 errors.

Environment (please complete the following information):

OS: latest macOS
Node 14

Thanks. To fix quickly, just change 'start' to 'next'

Just committed the fix.

Wow, this might be the official world record in time between filing and fixing a bug.

You got me at a good time ;-)

😄


rate should be mL/min, not mL/hr (for the numbers to work)

Not sure if a rate in mL/min is typical, but otherwise the total volumes don't add up (or the value of RATE is off)

Whoops! mL/min isn't that typical but it's good enough for the example - thanks!


Cfa (Issue #24)

Proposition for CFA taxon. Just a draft right now - let me know what you think about the general concept. Idea is to represent each factor as a vector element. Added a little example so you get the idea. Might get a little bit overwhelming with many factors - added a first plausibility check for vector lengths.

Note: Tried to change SEM.jl dependency to StenoGraphs dependency (by using rm StructuralEquationModels in package mode, after either "] activate ." or "] activate ./test". However, got some doctest errors afterwards.
Also have a weird error in the CFA testset, two expressions that should be the same are not the same. I will look at it the next days.

Since this morning I got an issue with the record function (even though there were no changes since yesterday):
┌ Warning: You really should supply an ID. May we suggest (from DOI): 8f1713c9-482b-58cb-8ed4-128c03e9dafb
└ @ Taxonomy c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\src\record.jl:19
ERROR: Can not reach DOI service. Maybe internet connection down?
Stacktrace:
[1] error(s::String)
@ Base .\error.jl:33
[2] interpret_status
@ c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\src\metadata\meta.jl:176 [inlined]
[3] interpret_status
@ c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\src\metadata\meta.jl:180 [inlined]
[4] json(location::UsualDOI{String, Missing})
@ Taxonomy c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\src\metadata\meta.jl:225
[5] MetaData(location::UsualDOI{String, Missing})
@ Taxonomy c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\src\metadata\meta.jl:251
[6] Record(; rater::String, id::Missing, location::UsualDOI{String, Missing}, meta::Missing, taxons::Vector{Factor}, spec::Bool, data::Bool)
@ Taxonomy c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\src\record.jl:30
[7] top-level scope
@ c:\Users\hafiznij\Documents\GitHub\Taxonomy.jl\docs\src\tutorials\start.md:16

Still got the ERROR: Can not reach DOI service. Maybe internet connection down?

This really means that the service is not reachable. Nothing in your code or so can account for that. Its thrown from here:
https://github.com/StructuralEquationModels/Taxonomy.jl/blob/2b713056ef6a42a8df8b2454cf8eb5f65713aeaa/src/metadata/meta.jl#L166-L178


Special Treatment for some Visuals?

I am developing a custom visual and I need allow-same-origin in my sandbox iframe in IE11 / Edge, but I don't get that attribute. But the mapbox visual does. If I imitate the mapbox guid I get the attribute. When I change to any other guid I dont?

@wysisoft, please forward your request to Power BI Custom Visuals Support<EMAIL_ADDRESS>


Where do the models originate? (question)

Hi, I was looking and trying out your code and I was wondering where your models came from, I am very curious how you got pyboy working together with gym as well.

Hi! The model in the base version is the default CNN from stablebaselines (NatureCNN). As for how pyboy and gym are working together, see this file
https://github.com/PWhiddy/PokemonRedExperiments/blob/master/baselines/red_gym_env.py

just as a tip in github instead of doing a new line and doing your link, do [yoursubject] (yourlink) of course without the space.


[Bug Report] MOTD formatting text is cleared on reload

Describe the bug
Formatting tags are removed after running minimotd reload
To Reproduce
Steps to reproduce the behavior:

Open config/MiniMOTD/main.conf
icon-enabled: false
motd-enabled: true
Create a single motd entry.
Write minimessage-style formatted text for motd lines 1 and 2
Save and close the config file.
Run /minimotd reload on the server
Re-open the config file.

Expected behavior
I expect my formatting to persist across reloads.
Server Software and Version:

Vendor: Fabric
Minecraft Version: 1.16.5

Additional context
N/A

Are there any errors in the console? Could you share the main.conf file (via something like https://paste.gg/) that doesn't save properly?

Wow, thanks for the quick reply!
The console doesn't seem to throw any errors:
[MiniMOTD] Reloading MiniMOTD...[MiniMOTD] Done reloading MiniMOTD

Here's my main.conf:
https://paste.gg/p/anonymous/28ea85b60c0942cc9aff6f21ceeb4455

Hmm, to be honest I have no idea what could cause the config to become modified like that... I tested adding the motd you provided to my config, and also tried with it as the only motd in the config
    {
        icon="random"
        line1="<white><b>Foo</white>   \u23d0   </b><red><underlined>Bar</underlined></red>"
        line2="<light_purple>Baz</light_purple>"
    }

and for me, it loaded and saved properly, resulting in this motd

For me to fix this I need to be able to reproduce it or understand what's causing it, but I don't remove formatting from text in this way anywhere in the mod, so I'm pretty confused.
Could you provide a list of other mods you are using, as well as information about your java install and OS? Theres a chance replicating your setup could help me reproduce this issue.

Sorry to have wasted your time, I was using a cloud-based server hosting interface (due to them having a lack of ssh access). It seems the website was trying to parse the formatting tags as HTML tags and just removed them every time I opened the file with their interface.


Adds the ability to inflate model with property of type [String: JSONDecodable]

There are a number of cases where it would make sense for the server to send down dictionaries of type [String: JSONDecodable] (as opposed to the currently supported [String: JSONCompatible]).
This PR adds support to decode and encode these value types.

+1
I'm currently writing some gross unwrapping logic for dictionaries instead.

Thanks for the PR!


Redis and node-cache shouldn't be package dependencies

Having them as dependencies reduces the flexibility of the package and increases bloat.
The main place this presents a problem is when using superagent-cache (which has cache-service as a dependency) from the browser - installing both redis and node-cache is not workable for the package to be used in a browser environment.

Thanks for the wise feedback! So just to be sure, you're recommending that nodeCacheModule and redisCacheModule and their associated tests be moved to separate repos and then you'll just add whichever one you want to use to your package.json?

That would probably make the most sense - it would make the cache engines more pluggable and standalone, and make cache-service more lightweight.

Are you needing this update by any specific date?

Nope - it's more a 'nice to have' to allow this to work in the browser better. This is a cool project! I may write a cache backend for the browser, using localStorage, etc.

Thanks! It's a good idea to break the cache modules out. I'll be thinking about implementation for a bit and should be able to start development in no more than couple of weeks when another project I'm working on slows down. I'd love to chat about a local storage cache module with you when I finish.

So I'm starting on this update today, although I may not be lightning fast on completion since you have no deadline. Just curious, are you using superagent-cache or cache-service?

I'm actually making a lot of fast progress right now. Question: cache-service currently provides a default configuration which means it bundles in a cache module if you instantiate it without providing one. Do you think there's any benefit in this for people who simple want to get caching without worrying about configuration? Or should the implementer be required to always provide a cache module explicitly so that there is no extra bloat in the download?

Check out the branch with your feature addition and let me know what you think: https://github.com/jpodwys/cache-service/tree/breaking_out_cache_modules

Fixed in 1.1.0. I'd love to see a cache backend for the browser like you mentioned earlier.


Any chance you could share all the .ckpt files used in the canny2img example?

These are the same you downloaded from the Draw Things app (and if you sniff the traffic, these are from https://static.libnnc.org/controlnet_scribble_2.x_f16.ckpt and sd v2.1 models (probably sd_v2.1_f16.ckpt))

sd_v2.1_f16.ckpt only has unet weights not text encoder and others.

actually they are in : open_clip_vit_h14_f16.ckpt

Yeah, later models (all accessible through https://static.libnnc.org/xxx) have text encoder / autoencoder weights separated out.


[node-agent] Refactor /pkg/node-agent/registry

How to categorize this PR?

/area usability
/kind enhancement
What this PR does / why we need it:
Binaries which are running on gardener nodes are published as container images and pulled by the nodes. The hyperkube image with kubelet and kubectl, valitail and in the future the node-agent are use cases for this.
In the current approach of node-agent those binaries are manually extracted from last layer of the docker images.
The new approach leverages containerd for pulling the images and extracting the binaries.
These are main advantages:

The binaries can be located in any layer of the image, which is less error-prone during image creation
containerd is able to pull images for the right CPU architecture out-of-the-box
We are able to leverage the registry-cache-extension for the images without any changes to the node-agent, because the images are pulled with the containerd configuration of the node.

In case of custom containerd configurations on the nodes, there is the possibility to change certain containerd parameters by well-known environment variables.
CONTAINERD_ADDRESS for the address of containerd socket.
CONTAINERD_NAMESPACE for the namespace the containerd client uses.
Which issue(s) this PR fixes:
Part of #8023
Special notes for your reviewer:
The PR uses github.com/containerd/containerd v1.6.24 and not the most recent version v1.7.7 because the later contains an incompatible version of go.opentelemetry.io. The v1.6 branch of containerd is still maintained.
Release note:

NONE

/cc @rfranzke

/assign

FYI @majst01 @vknabel @Gerrit91

@oliver-goetz thanks for taking this topic, i think this is the most straight forward path for this purpose.


Document handling of pull requests and acceptance of committers/PPMC. Link to related resources.

Looks good! I don't think you missed anything.


libxgboost.so is almost 300MB large on x86_64 platform

The size of the libxgboost.so file in the x86_64 platform is huge, around 300MB. In comparison entire xgboost .whl package in the macOS platform, is only 1.6MB in size.
This large size is causing significant problems, especially on embedded platforms.
xgboost was obtained from: https://pypi.org/project/xgboost/#files

Partly caused by restriction by pypi where we need to statically link everything, partly caused by the need to enable support for different CUDA compute capability.

You might need a different distribution method other than pypi. I'm not sure if the conda-forge build has CPU only build (@hcho3 ), you might also find https://github.com/NixOS/nixpkgs to be interesting.

Just confirmed the conda CPU build doesn't contain any GPU code. The version is a little bit behind though. Will look into keeping it up-to-date.

conda install -c conda-forge py-xgboost-cpu will do the job if you happen to be using conda.


TINKERPOP-976 Fail earlier if invalid version is supplied in validate-distribution.sh

Invalid version:
daniel@cube /projects/apache/dev/incubator-tinkerpop (TINKERPOP-976) $ bin/validate-distribution.sh 3.0.2-incubating

Validating binary distributions

* downloading Apache Gremlin Console (apache-gremlin-console-3.0.2-incubating-bin.zip)... Failed to download Apache Gremlin Console

Valid version:
daniel@cube /projects/apache/dev/incubator-tinkerpop (TINKERPOP-976) $ bin/validate-distribution.sh 3.1.0-incubating

Validating binary distributions

* downloading Apache Gremlin Console (apache-gremlin-console-3.1.0-incubating-bin.zip)... OK
* validating signatures and checksums ...
  * PGP signature ... OK
  * MD5 checksum ... OK
  * SHA1 chacksum ... OK
* unzipping Apache Gremlin Console ... OK
* validating Apache Gremlin Console's docs ... OK
* validating Apache Gremlin Console's binaries ... OK
* validating Apache Gremlin Console's legal files ...
  * LICENSE ... OK
  * NOTICE ... OK
  * DISCLAIMER ... OK
* validating Apache Gremlin Console's plugin directory ... OK
* validating Apache Gremlin Console's lib directory ... OK
* testing script evaluation ... OK

* downloading Apache Gremlin Server (apache-gremlin-server-3.1.0-incubating-bin.zip)... OK
* validating signatures and checksums ...
  * PGP signature ... OK
  * MD5 checksum ... OK
  * SHA1 chacksum ... OK
* unzipping Apache Gremlin Server ... OK
* validating Apache Gremlin Server's docs ... OK
* validating Apache Gremlin Server's binaries ... OK
* validating Apache Gremlin Server's legal files ...
  * LICENSE ... OK
  * NOTICE ... OK
  * DISCLAIMER ... OK
* validating Apache Gremlin Server's plugin directory ... OK
* validating Apache Gremlin Server's lib directory ... OK

Validating source distribution

* downloading Apache Tinkerpop 3.1.0 (apache-tinkerpop-3.1.0-incubating-src.zip)... OK
* validating signatures and checksums ...
  * PGP signature ... OK
  * MD5 checksum ... OK
  * SHA1 chacksum ... OK
* unzipping Apache Tinkerpop 3.1.0 ... OK
* building project ... OK

VOTE: +1

I ensured that curl -Lsf works on Mac. It does.
VOTE +1.

I gave this a full test - works nicely.
VOTE +1


[✨]Could add an example of Auth.js that uses username+password?

Could add an example of Auth.js that uses username+password?
Seems many people have trouble with it.
There is an example that uses GitHub auth in the docs here: https://qwik.builder.io/docs/integrations/authjs/
But it's different from username+password.

I would like to work on this

Thanks @VarPDev

@VarPDev That's great! Could use jwt instead of session?

@zwl1619 I saw there is a property you can change inside serverAuth$ named "session"
Example
export const { onRequest, useAuthSession, useAuthSignin, useAuthSignout } =
serverAuth$(({ env }) => ({
...
// with this I think you can use jwt instead session
session: { strategy: "jwt" },
providers: [
...
] as Provider[],
...
}));

auth.js says jwt is the default choice, need to figure out if it is overridden by qwik or remains default jwt as in auth.js.
This would be best to ask someone who has worked on it: @gioboa do you know anyone who knows if jwt is used by default? At first look it seems to me that the default one is used.

I don't have clue for that. if JWT is the default option, go for it.

@zwl1619
I did a PR to update the documentation.
For now here the example to use Credentials:
file<EMAIL_ADDRESS>import { serverAuth$ } from<EMAIL_ADDRESS>import type { Provider } from "@auth/core/providers";
import Credentials from "@auth/core/providers/credentials";

export const { onRequest, useAuthSession, useAuthSignin, useAuthSignout } =
  serverAuth$(({ env }) => ({
    secret: env.get("AUTH_SECRET"),
    trustHost: true,
    providers: [
      Credentials({
        async authorize(credentials, req) {
          // Add logic here to look up the user from the credentials supplied
          const user = {
            id: "1",
            name: "J Smith",
            email<EMAIL_ADDRESS>          };

          return user;
        },
      }),
    ] as Provider[],
  }));

file: sign.ts
import { component$ } from<EMAIL_ADDRESS>import { Form } from<EMAIL_ADDRESS>import { useAuthSignin } from "./plugin@auth";

export default component$(() => {
  const signIn = useAuthSignin();

  return (
    <>
      <Form action={signIn}>
        <input type="hidden" name="providerId" value="credentials" />
        <input
          type="hidden"
          name="options.callbackUrl"
          value="http://qwik-auth-example.com/dashboard"
        />
        <input name="options.email" type="email" />
        <input name="options.password" type="password" />
        <button>Sign In</button>
      </Form>
    </>
  );
});

Thanks very much!

@zwl1619 did you try it?

@gioboa Not yet! I'll try it later.


Failed Update AutoRest C# version to 3.0.0-beta.20231102.3

Update AutoRest C# version to 3.0.0-beta.20231102.3

API change check
API changes are not detected in this pull request.


make use of HTTP_PROXY/HTTPS_PROXY environment variables

Description
Describe your issue here.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

I am using the rasa stack to build a bot for slack. The built in Slack connector is not able to connect through our outgoing proxy because when i set the HTTP_PROXY or HTTPS_PROXY env vars it seems they are not been used from the http client aiohttp in the BaseClient of slack SDK.
I would like to set the proxy from e.g a Container runnning my slack app via env vars.

@iamtofr Thanks for taking the time to write in here!
We may add a new option to set trust_env in BaseClient constructor args in the future.
In the short term, could you consider using session arg instead? Here is a simple example demonstrating how to use the argument.
import asyncio
import aiohttp
import os
from slack import WebClient

async def main():
    proxy = os.environ["HTTPS_PROXY"]
    print(f"Using {proxy} ...")
    session = aiohttp.ClientSession(trust_env=True)
    client = WebClient(
        token=os.environ['SLACK_API_TOKEN'],
        run_async=True,
        session=session,
    )
    response = await client.chat_postMessage(channel="#random", text="Sent through a proxy server")
    print(response)
    await session.close()

if __name__ == '__main__':
    loop = asyncio.get_event_loop()
    loop.run_until_complete(main())

If you're using the sync-mode (the defualt: run_async=False), these env variables have been supported. Check this example: https://github.com/slackapi/python-slackclient/blob/v2.7.3/integration_tests/samples/issues/issue_714.py#L17-L20
Since v2.8.0 - a new minor version that is going to be released soon, async client users can use a new option trust_env_in_session option in slack.web.async_client.AsyncWebClient.
from slack.web.async_client import AsyncWebClient
client = AsyncWebClient(token="xoxb-***", trust_env_in_session=True)


Digital Gong missing

I cant see the digital gong as "switch". Is there a way to fix that?

This repo is currently not having any active maintainer, and it has been over a year since I used it myself. If you open a PR I am happy to review it


feat(package): add repository information to package.json

Signed-off-by: Aniruddha Shriwant<EMAIL_ADDRESS>

Closes #341

Changes

Added Package Repository Information (directory) in package.json

Coverage remained the same at 89.046% when pulling adf1a9895da372c081e4b90503a020b1f6a17e8b on Aniruddha-Shriwant:aniruddha/techdocs-341/addRepositoryInPackage.json into 12aa26679bc07541175e5d9dc7a85299cdc896e0 on accordproject:master.

Please don't merge until all related PRs are approved (tracked in https://github.com/accordproject/techdocs/issues/341)


Update contributors.json

Issue

Description

Checklist

[ ] Added contributor object in ascending order in contributors.json
[ ] Github action build passing
[ ] No merge conflicts

Please reopen after you fix conflicts


Out segmentation looks pixelated at border, beyond native resolution

When applying ADS model on an image with better resolution than the model used, the output segmentation looks "pixelated", as shown below:

This is caused by the resampling from the lower res patch to the native resolution. A way to mitigate this effect would be to apply a smoothing filter to the output segmentation, making sure it does not affect the border definition (i.e., morphometrics should not change).

Stale / we're using different models now


Python input() function does not read input (SSH Remote VM)

I'm using VS Code to SSH into a remote VM.
When running python scripts (in the terminal window) that request input .e.g. query = input("\nEnter a query: ") it does not read the provided input but waits forever.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!

Just solved my issue. It wasn't related to VS Code but to the fact that I called the python script from a nodejs app via shelljs and shelljs does not accept input. Now I call it natively via child_process.execFileSync with the stdio: 'inherit' parameter and now it works as expected. Just share it here in case someone else runs in this kind of issue.


Correct Discord Invite Regex

The current invite regex is too permissive. It captures any non-space character, while invite codes can only have letters and numbers.
https://github.com/python-discord/bot-core/blob/127d0efa219d29e15bd71339d0d2517945aed57c/botcore/utils/regex.py#L15
In https://github.com/python-discord/bot-core/commit/c9daa6f3774380794968cf28faf9ba4bde1f68e2 we made it go until the whitespace because of the way the Discord client parses invites, but we can safely account for it by adding / to the character class instead (i.e [a-zA-Z0-9/-]).

Sooo apparently there are other characters to account for here. The only thing that I can say atm is that a ? stops the invite code parsing.


[Adaptive] [LG] change LG, dialog systems to accept a expression engine instance

Current LG subsystem creates its own instance of expression engine. This gets in the way of users expanding an expression library with a set of custom functions that is also available within the LG and dialog sub-systems.
We should update LG and dialogs to be able to take and work with an expression engine instance that includes custom pre-built functions.

Template function and user-defined function wihch one should have higher priority. For example, there exists a LG file:
# t1()
- hello

# t2
- {t1()}

And customer defines a function name: t1. We know template name is a runtime concept, but customized function is a pre-build concept. So when someone use {t1()}, which one should be called?


Improving internal benchmarks

It would be nice if the emacs-lsp-booster process could optionally log some of its own benchmarks, e.g. if a --benchmark flag is passed, or with a special build flag (maybe the dev build?... not a rust person).  The obvious things to log are:

Number of complete JSON payloads received from server thus far.
Bytes of data received by the lsp server in the last (numbered) JSON payload.
Bytes of bytecode data that JSON was translated into and sent to emacs.
Time it took from writing that data onto stdout until emacs reads it (not sure how to do this, maybe time until pipe become writeable again).

Then perhaps a --passthrough flag or build option can be implemented, which does nothing to the JSON, but just passes it on as-is (obviously removing the json-read-buffer override on the Emacs side).  This would be just for equivalent logging.
Then we could generate two logs (with and without --passthrough).  These two results could be compared for a given lsp server, to see how much time it takes (per byte) on average for emacs to read in the data via JSON vs. via bytecode.

Thanks for your suggestions. I've made a few improvements since then.

Number of complete JSON payloads received from server thus far.
Bytes of data received by the lsp server in the last (numbered) JSON payload.
Bytes of bytecode data that JSON was translated into and sent to emacs.

Now there are simple loggings. https://github.com/blahgeek/emacs-lsp-booster/commit/d3b9c989c82f757be6c00b7d1e8c8e9896a7610e. Run with --verbose flag would print those debug logs.

Then perhaps a --passthrough flag or build option can be implemented, which does nothing to the JSON, but just passes it on as-is (obviously removing the json-read-buffer override on the Emacs side). This would be just for equivalent logging.

As you already seen, now there's a --disable_bytecode flag.

Time it took from writing that data onto stdout until emacs reads it (not sure how to do this, maybe time until pipe become writeable again).

This, indeed, is not trivial to do. Also I don't think this would provide accurate benchmark result for comparison.
If we really want to do that, I guess we can (for both json data in "passthrough" mode, or bytecode data in normal mode):

For each message from lsp server, mark the timestamp (t_server_generated) when this program receives it, and somehow attach the timestamp in the message when sending to emacs (e.g. as a new field in json object)
In emacs, mark the timestamp (t_emacs_parsed) after receiving and parsing the message
Calculate t_emacs_parsed - t_server_generated as the end-to-end latency

This would require modifications in both wrapper and elisp sides.
Personally I'm not that into this benchmark idea though. For example, the "emacs may be blocked while sending data" issue that this program solves cannot be easily measured. So I think the overall improvements cannot be benchmarked anyway.

Now there are simple loggings.

Thanks for these updates.

This, indeed, is not trivial to do.

Good thoughts.  Re "emacs may be blocked while sending data", you could also do the reverse: tag t_emacs_generated in elisp, then have emacs-lsp-booster log t_server_received.  Rather than trying to get the logs in the same stream, probably we could just associate each log with its request :id, and then the two logs could be decoded after the fact, making sure the times are matched correctly.
Then users would use their client with and without --disable_bytecode  for, say, a day each.  By collecting the data, interesting trends would definitely emerge in terms of send and receive latency as a function of byte size of the payload.  I'd guess for small payloads it may increase slightly, depending on the amount of "blocked I/O" which occurs.   And it will definitely vary by server.
But I agree this is not really required.  People can use the wrapper or not based on their experiences.  The only advantage to having such benchmark data would be if you ever envisioned getting builtin support inside lsp-mode and eglot; to make that case, real usage benchmarks would likely be required.  Maybe something for down the road.
Feel free to close.


Kubernetes Ingress support

In order to create inbound routing to multiple web ui based pods running within a cluster, potentially spread over different namespaces then support for Ingress resources would be required.
Using the native Kubernetes Ingress capability would be preferable over custom solutions.
http://kubernetes.io/docs/user-guide/ingress/
The features of the Rancher load balancer match reasonably closely the capabilities offered by the Kubernetes ingress resource. Having the ability and flexibility to specify domain based and context based routing will open up more potential use cases for clusters being used to host multiple apps for multiple team environments.

@prachidamle this ticket is to keep track of refactoring our current k8s lb solution to use k8s v1.2 ingress model.

Branch to keep track of project progress: https://github.com/rancher/rancher-ingress

@zekizeki
Here is the high level flow for the feature:

Rancher ingress-controller gets created as a part of k8s system stack when user enabled k8s on the environment.
User creates the ingress in k8s. Example:

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: test
spec:
  rules:
  - host: foo.bar.com
    http:
      paths:
      - path: /foo
        backend:
          serviceName: nginx-service
          servicePort: 80

Rancher-ingress controller will create Rancher LB service for every ingress, populate it with the services endpoints and host name routing rules according to the rules defined on ingress, and update ingress "Address" field with the LB public endpoint (host ip address):

> kubectl get ingress
NAME      RULE          BACKEND   ADDRESS
test      -                                            <IP_ADDRESS> // address updated by Rancher ingress-controller
          foo.bar.com
          /foo                     nginx-service:80

Rancher ingress-controller will monitor changes done to ingress and backend services of ingress controller, such as:

ingress update
backend services update (end point changes)

and update Rancher LB based on the changes.
Rancher ingress-controller will also listen to Rancher LB changes such as:

public end point change (lb container gets redeployed to a different host)

and update Ingress with the new Address if endpoint change happened.

v1 of ingress will be available in Rancher 1.1.0-dev2 release with limitations that will get addressed in 1.1 release:

TLS support
Custom public port for a LB
Horizontal scaling for LB
Stickiness policies support

Closing enhancement.  If there are bugs found, file separate issues.

Thanks @alena1108 @will-chan looking good will try this out.


handle npm aliased packages

Fixes https://github.com/google/osv-scanner/issues/588

Codecov Report

Merging #610 (e997f11) into main (c00f630) will increase coverage by 0.02%.
Report is 6 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #610      +/-   ##
==========================================
+ Coverage   80.61%   80.64%   +0.02%
==========================================
  Files          78       78
  Lines        5334     5342       +8
==========================================
+ Hits         4300     4308       +8
  Misses        866      866
  Partials      168      168

Files
Coverage Δ

pkg/lockfile/parse-npm-lock.go
100.00% <100.00%> (ø)


FOSSA scanning added to the repo

Ref - https://github.com/cncf/foundation/issues/109
Signed-off-by: Ihor Dvoretskyi<EMAIL_ADDRESS>

cc @slack @lachie83 @michelleN @rickducott


Proxy monitors its own load/health and throttles connections when required

Add self-diagnostics capability to YARP and implement throttling policies which can be applied if some thresholds have been exceeded.
Indicators to watch for:

CPU load
Memory pressure
Available TCP ports
Average/mean/p95 client requests latency

Examples of throttling policies:

Stop accepting new HTTP 1.1 requests by immediately returning 503
Stop accepting new active streams on HTTP/2 connections
Stop accepting new TCP connections

I'd advise caution for any solution that might have the explicit or implicit assumption that the current process is the only resource-intensive process running on the machine and / or that you can accurately determine being CPU limited. E.g. an instance of YARP running with a job object limiting cpu may be CPU-limited while the machine may have plenty of capacity to spare.


different implementation of _onWriteReport

I didn't have any luck getting #14 to work, hacked togeather this while trying to figure it out.
it is slightly speculative that _onWriteReport will one day receive the reporter, until then the subdirectory by browser functionality isn't going to work :-/
remapIstanbul.writeReport seems to have a few bugs in it, i.e dest doesn't appear to accept a function and writing to the console doesn't work for me.
Anyway, thought it might be useful for someone

Should be fixed in 0.2.1.


Dynamic update from function not working

Hi,
I'm currently using the "Ayu" theme that provides it's own lightline theme. I want to create a function that allows me to toggle the light/dark variants of the theme as I like to use them during day/night. I created the following function:
function! ToggleSchemeColor()
    if g:ayucolor == "light"
        let g:ayucolor="dark"
    else
        let g:ayucolor="light"
    endif
    colorscheme ayu
    call lightline#init()
    call lightline#colorscheme()
    call lightline#update()
endfunction

nnoremap <silent><C-k> :call ToggleSchemeColor()<CR>

It works as expected regarding the ayu color scheme but the lightline scheme doesn't change. It stays in the way it was initially set up in the vim config (light scheme). Due to the fact that I'm new to vim it's possible that I try to do something impossible or made some other mistake. Could you please help me correcting my function? Thank you!
Daniel

That's because once the autoload file is sourced, it will never been reloaded automatically. Instead, you can use execute 'source' globpath(&rtp, 'autoload/lightline/colorscheme/ayu.vim') just before lightline#colorscheme, to forcibly reflect the update of the colorscheme.

Great, thank you so much, it works!


WIP: Start building S390x packages again

Testing if chef-16 builds okay on s390x.
https://buildkite.com/chef/chef-chef-master-omnibus-adhoc/builds/345#0834c6ff-ac9e-487c-90c9-c210d84f1118
This reverts commit a7012bfe35dc3923537ee936417144a0328970ab.

https://buildkite.com/chef/chef-chef-master-omnibus-adhoc/builds/345 passed fine.
kitchen tests failing with Could not find 'ohai' (= 16.4.11) - did find: [ohai-16.4.12]  probably fixed elsewhere. will rebase with a DCO.


"Căpâlnița" is listed in the drop-down menu

Description:
"Căpâlnița" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Căpâlnița" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Căpâlnița, HARGHITA (Căpâlnița)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type again "Căpâlnița" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Căpâlnița, HARGHITA".


ensure consistency across the various modules

I was wading through the source w/ Dr. @markpollack  (he was helping me get a cool demo working) and i believe there's an opportunity for some conventions to help module authors:
@EnableBinding is itself transitively a @Configuration class and it ultimately factories MessageChannel instances that any component may inject. It seems odd and assymetrical to stick @EnableBinding on individual messaging endpoints (like Spring Integration service activators etc)
That annotation better belongs on a configuraton class (commonly, i'd expect to see it on the @SpringBootApplication class which is also a @Configuration class. Of course, once that happens then something needs to be done to capture messaging endpoints. @IntegrationComponentScan picks these beans up, or the beans could be returned from @Bean provider methods in a @Configuration class. If we use @IntegrationComponentScan then we'd need to add @MessagingEndpoint to all the components we currently stick @EnableBinding on
we could either as a convention turn on @IntegrationComponentScan and just trust that all the beans get turned on in a given package, implications for composability with other libraries that for some reason happen to be in the same namespace be darned.. or we could decide that all of a given modules components are to be explicitly manufactured in the @SpringBootApplication as @Beans. (Maybe we as framework authors take this approach and endorse either for consumers or other developers just trying to pull together their own solution?)
also, what's the best-of-breed approach for pollers in sources? it seems it's auto-configured for us, so why do some components define the poller explicitly in the annotation for the or provide their own PollerMetadata bean?
So, I propose:

all modules should have a FooApplication @SpringBootApplication class (already true)
all modules should have a separate FooConfiguration class with requisite @Configuration, @EnableBinding and @IntegrationComponentScan annotations.
all messaging components that are not manufactured as @Beans should have @MessagingEndpoint or similar to designate them as beans to captured in the component scan and to replace @EnableBinding
figure out how to apply pollers as consistently as possible

a lot of this stuff is already happening correctly, just.. inconsistently..

Hi @joshlong !
Thank you for the feedback and Happy New Year!
I haven't read the full comment yet, but let me comment on that what I see.
The @IntegrationComponentScan is exactly for the @MessagingGateway till now. See the source code:
@Import(IntegrationComponentScanRegistrar.class)
public @interface IntegrationComponentScan {

I guess we need one more JavaDocs and Documentation round to make that more clearer: https://jira.spring.io/browse/INT-3927

Please, take a look to the @EnableBinding a bit closer:
@Configuration
@Import({ChannelBindingServiceConfiguration.class, AggregateBuilderConfiguration.class, BindingBeansRegistrar.class,
		BinderFactoryConfiguration.class})
@EnableIntegration
public @interface EnableBinding {

Even if it is @Configuration it can be used in conjunction with @SpringBootApplication. It doesn't hurt at all to have several @Configuration on the same class. Finally, all of them is just @Component.
The main feature why that everything works for Spring Integration is @EnableIntegration on the @EnableBinding. That annotation @Import(IntegrationRegistrar.class) with the big Spring Integration stuff, like taskScheduler, messageBuilderFactory, SpEL stuff and so on. Of course, this one registers the Messaging Annotation Processors, too, like for @ServiceActivator, @InboundChannelAdapter and so on. Plus it registers IntegrationConfigurationInitializer from spring.factories, to scan something like:
org.springframework.integration.config.IntegrationConfigurationInitializer=\
org.springframework.integration.dsl.config.DslIntegrationConfigurationInitializer

Without tying to the Core module directly from source code. The same is done for Spring Boot AutoConfiguration, BTW.

we could either as a convention turn on @IntegrationComponentScan and just trust that all the beans get turned on in a given package

That does not make so much sense: the Spring Integration infrastructure will be here anyway after @EnableIntegration (@EnableBinding). If you mark your @Bean (or some service method) with the @ServiceActivator, so you expect to have an endpoint there. There is just enough to have standard @ComponentScan to distinguish the desired @Compent s with @ServiceActivator etc.
The @MessageEndpoint (not @MessagingEndpoint, BTW) is redundant, to be honest. The standard @Compent (@Service) is enough for Spring Integration to scan @ServiceActivator or others.
Maybe it's time to @Deprecated it? We don't care about the particular @Component subtype there... CC @markfisher

No objections regarding Poller!
Looks like we really just can auto-create the PollerMetadata.DEFAULT_POLLER bean and be good without @Poller in the target module code.

Yah I agree having @IntegrationComponentScan on by default isn't great.
Don't deprecate @MessageEndpoint annotation, please. It's a documentation stereotype annotation. @Repository does very little besides exception handling, but it's helpful to document a components role (it's UML stereotype).

Well, I'd say that having @IntegrationComponentScan on the @EnableBinding level by default won't hurt. Fully similar to the @ComponentScan on the @SpringBootApplication.
From here that suggests us to introduce kinda String[] scanBase* options to the @EnableBinding, like we have in the @SpringBootApplication, but here for the @IntegrationComponentScan.
Yeah! No objections for the @MessageEndpoint and its documentation purpose!

This issue was moved to spring-cloud/spring-cloud-stream-app-starters#69


bump jackson-databind:<IP_ADDRESS>

BACKEND PULL REQUEST
updated jackson-databind from 2.11.3 to <IP_ADDRESS>
Related Issue

Why is this being done? Link to issue, or a few sentences describing why this PR exists
Snyk vulnerability findings

Changes Proposed

Detailed explanation of what this PR should do

Additional Information

decisions that were made
notice of future work that needs to be done

Testing

How should reviewers verify this PR?

Checklist for Primary Reviewer

[ ] Any large-scale changes have been deployed to test, dev, or pentest and smoke tested
[ ] Any content updates (user-facing error messages, etc) have been approved by content team
[ ] Any changes that might generate questions in the support inbox have been flagged to the support team
[ ] GraphQL schema changes are backward compatible with older version of the front-end
[ ] Changes comply with the SimpleReport Style Guide
[ ] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[ ] Any dependencies introduced have been vetted and discussed

2.11.3/4 works (with regard to pipeline test)
<IP_ADDRESS> and 2.12.1 fails

2.13.3 also fails

This appears to be resolved in https://github.com/CDCgov/prime-simplereport/pull/3562


Allow resources to be in different resource groups (e.g. Event Hub Namespace and Storage Account)

Expected Behavior

It should be possible to access the resources even if they are in different resource groups. For example the EventHub Namespace is in resource group A and the Storage Account is in resource group B.
As a bonus it would be nice to allow different locations for each resource group. But in the first step it would be sufficient to have multiple resource groups in the same location.

Current Behavior

The current implementation requires all resources to be in the same resource group.

Possible Solution

Each resource has its own "configuration block" in the application.yml. A new parameter for the resource group could be introduced.

Steps to Reproduce (for bugs)

Snapshot Code for Reproduce
application.yml for a single resource group (current implementation)
spring:
  cloud:
    azure:
      credential-file-path: my.azureauth
      region: West Europe
      resource-group: example-rg
      eventhub:
        namespace: example-eh-ns
        checkpoint-storage-account: example-storage

application.yml for multiple resource groups (a possible solution for this feature) (same location for all resource groups)
spring:
  cloud:
    azure:
      credential-file-path: my.azureauth
      region: West Europe
      eventhub:
        namespace-resource-group: example-rg-for-eh-ns
        namespace: example-eh-ns
        checkpoint-storage-account-resource-group: example-rg-for-storage
        checkpoint-storage-account: example-storage

Branch

master

Your Environment

Feature does not depend on OS version etc.

Thanks for bringing this up and we'll evaluate this feature. In the meantime, PR is welcome.

Closing this issue.
Because it's not active for a long time.
If anyone have similar issue, please create issue in new repo: https://github.com/Azure/azure-sdk-for-java/issues


Speed up

Improves speed of time python explain.py MobyDick.txt from 58s to 41s to 34s for me.
It might be possible to that if (word ... statement to before the for pos_def ... loop, but this'll do for now :)

Thanks @hugovk!


The error of ‘wgs84_extent’ when process the product of Sentinel-1

Assigning CRS to a GeoDataFrame without a geometry column is not supported. Use GeoDataFrame.set_geometry to set the active geometry column.
Traceback (most recent call last):
File "d:\SAR_DATA\process\eoreader-main\eoreader\products\sar\s1_product.py", line 187, in wgs84_extent
raise InvalidProductError(
eoreader.exceptions.InvalidProductError: Cannot determine the WGS84 extent of S1A_IW_GRDH_1SDV_20200308T121318_20200308T121343_031584_03A39B_5C41
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\10992.conda\envs\DL1\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\10992.conda\envs\DL1\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "c:\Users\10992.vscode\extensions\ms-python.python-2022.5.11221002\pythonFiles\lib\python\debugpy_main.py", line 45, in
cli.main()
File "c:\Users\10992.vscode\extensions\ms-python.python-2022.5.11221002\pythonFiles\lib\python\debugpy/..\debugpy\server\cli.py", line 444, in main
run()
File "c:\Users\10992.vscode\extensions\ms-python.python-2022.5.11221002\pythonFiles\lib\python\debugpy/..\debugpy\server\cli.py", line 285, in run_file
runpy.run_path(target_as_str, run_name=compat.force_str("main"))
File "C:\Users\10992.conda\envs\DL1\lib\runpy.py", line 268, in run_path
return _run_module_code(code, init_globals, run_name,
File "C:\Users\10992.conda\envs\DL1\lib\runpy.py", line 97, in _run_module_code
_run_code(code, mod_globals, init_globals,
File "C:\Users\10992.conda\envs\DL1\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "d:\SAR_DATA\process\eoreader-main\K.py", line 15, in
s1_prod.wgs84_extent()
File "C:\Users\10992.conda\envs\DL1\lib\site-packages\methodtools.py", line 72, in call
return self.call(*args, **kwargs)
File "d:\SAR_DATA\process\eoreader-main\eoreader_init.py", line 31, in wrapper
return func(*args, **kwargs)
File "d:\SAR_DATA\process\eoreader-main\eoreader\products\sar\s1_product.py", line 196, in wgs84_extent
raise InvalidProductError(ex) from ex
eoreader.exceptions.InvalidProductError: Cannot determine the WGS84 extent of S1A_IW_GRDH_1SDV_20200308T121318_20200308T121343_031584_03A39B_5C41

I follow the tutorial and here is my code:
from eoreader.reader import Reader
from eoreader.bands import *
import sys
sys.path.append('D:/Software/SNAP/snap/bin/')
Sentinel-1 GRD path
s1_path = "E:\SAR_DATA\Sentinel1\S1A_IW_GRDH_1SDV_20200308T121318_20200308T121343_031584_03A39B_5C41.zip"
s1_path = "E:\SAR_DATA\ESA\TSX_OPER_SAR_SC_GEC_20110223T012215_N24-000_W111-445_0000_v0100.sip\TSX_OPER_SAR_SC_GEC_20110223T012215_N24-000_W111-445_0000.zip"
Create the reader object and open satellite data
reader = Reader()
The reader will recognize the constellation from its product structure
s1_prod = reader.open(s1_path)
Load some bands and index
s1_prod.get_band_paths([VV, HH])
An error occured, and I checked into the inner function and found the error is through in function of 'eoreader/products/sar/s1_product.py':
s1_prod.wgs84_extent()
When run the code 'extent_wgs84 = vectors.read(preview_overlay)', it will throw the message:
Assigning CRS to a GeoDataFrame without a geometry column is not supported. Use GeoDataFrame.set_geometry to set the active geometry column.
Finally, the code 'vect.crs = WGS84' throw the error.
How can I solve this problem? Thanks in advance!

Besides, I read the message that

Hello,
Do you have ogr2ogr accessible as pointed out in the latest FAQ ?
For some reason Sentinel-1 KML files sometimes fails to be read by geopandas (I don't know why) and I need to convert them to geojson thanks to ogr2ogr as a workaround.
To know if you have it available, just type ogr2ogr in a terminal. If the help section is displayed, it will work.
However, you are right, I should better handle this error (at least throw a proper exception).
Concerning your last comment, I don't use snappy as it is slmow and buggy. We run SNAP GPT directly.
If you have properly installed SNAP, this should work. Once again, to check if gpt is available, just type it in a terminal. If the help section is displayed, it will work.

That worked for me.  Thank you!

That worked for me. Thank you!

Do you have ogr2ogr accessible as pointed out in the latest FAQ ? For some reason Sentinel-1 KML files sometimes fails to be read by geopandas (I don't know why) and I need to convert them to geojson thanks to ogr2ogr as a workaround. To know if you have it available, just type ogr2ogr in a terminal. If the help section is displayed, it will work. However, you are right, I should better handle this error (at least throw a proper exception).

🙏


promtool: add dump-series

Copy promtool tsdb dump samples code for print series data (ignore samples)
example:
promtool tsdb dump-series | jq -r '.xxx'
...

Hello from the bug-scrub!  Sorry your PR got no attention till now.
However I think the task got simpler -the dumpSamples function now takes a formatting function.
I think you can make a formatter which just prints the labels, i.e. based on these lines:
https://github.com/prometheus/prometheus/blob/6f1fd4be96e5096a604f44b9ad6b62c68fafac3d/cmd/promtool/tsdb.go#L756-L759

Thanks for reply :) the dump-series print series string and dedup the result (add a cache map)

Yes I’m proposing you write a function which prints the string, right after the lines I quoted.
How can you get duplicate series?

Hi @bboreham the PR has new commits, and doc and testcase updated too :)


Migrate java-11 sample app to use Java17

Description
Fixes #
Migrate java-11 cloudstorage, websocket-jetty, datastore sample app to use Java 17 (and later).
Java 20+ might have warning at compile but the app runs as expected
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist

[X] I have followed Sample Format Guide
[X] pom.xml parent set to latest shared-configuration
[X] Appropriate changes to README are included in PR
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[x] Tests pass:   mvn clean verify required
[X] Lint  passes: mvn -P lint checkstyle:check required
[ ] Static Analysis:  mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[ ] Please merge this PR for me once it is approved

Do we have public doc links to these samples? Did you deploy to make sure all is good?

Yes and I have test deployment with the updated dependency as well
Cloud storage
Datastore
Websocket


Emacs mode?

I don't really think there is any demand for emacs mode. I haven't seen people asking for emacs mode even on much bigger projects. AFAIK emacs is in a slow decline and most of its users won't migrate to other editors.


Caffe module doesn't exist

Caffe doesn't exist on Pip. So I installed Caffe2, but it also couldn't find it after install. Not sure how to fix the issue as I haven't seen a module folder looking like this before.

Hi,
This codebase is very old at this point, Caffe hasn't been updated in a long time, and nowadays there exist several many cleaner implementations of neural style transfer algorithms in more recent frameworks where you'll also get quicker support - e.g. I'd recommend checking out
https://github.com/pytorch/examples/tree/master/fast_neural_style


The first build of Circle CI failed

Copy the text of Circle CI to my project in .circle/config.yml and push it to master.
Start building on Circle CI, but the result of building is failure .
The error message is as below:

3.But after that, when I push something again to master, it builds successfully! That is odd.

If not use workflow, The first build will always be successful. So I add another text of Circle CI.

It seems that based on #670 this error might be caused due to CircleCI outage. I'm closing this for now, feel free to reopen if issue persists


[fix #4265] Port Document search API for Elasticsearch 6.x

This is the first etaration of making a API for Documentation Search. Its not finalized yet and the work is not finished yet.
Need more work on highlighting as well as test.
Needed to do some refactor to keep it DRY.
@ericholscher Can you look for a slight review?

Currently the API looks something like this!

I think most of my questions are around whether we're using proper querysets, or we have things that look like querysets, but are actually search results that we're filtering.

@ericholscher We actually have a Search object that act closely similar to queryset. So we can use the Search object as queryset to easily implement the API!

@ericholscher Its ready for a final review.
r?

@ericholscher Anything left for fixing?

@ericholscher while I was working in the frontend, I realized that the link of document is needed. I have implemented that. Can you please make a review?

@ericholscher Fixed python3 compatibility and changes as you reviewed. r?


fix: only re-focus after a response stream if the window is active

Problem
In VSCode if you:
- write a message in chat
- focus somewhere else in the editor outside of chat
- wait for the response stream to finish
it will automatically focus the chat prompt
Solution
Only refocus the chat prompt if the document is currently focused

License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

To clarify the problem related with the solution a bit more in detail:
In VSCode, when user has the focus to the Q panel, we want to autofocus to the prompt input field after a streaming answer is ended if the autoFocus config is set to true.
However, it should only do it when the current focus is inside the Q panel window. Otherwise, it will grab the focus from the editor to the Q panel which might cause an issue since user may write to their editor window or focus to another panel while waiting an answer. And while they are typing in their editor window sudden switch to the Q panel may cause the text written by the user went into the Q prompt field instead of their editor file.