Feature Request: Support for Multiple Simultaneous LLM AI API Endpoints for Self-Hosting and Model Selection

Description:
We would like to propose the addition of a new feature to  AutoGen that enables users to configure and utilize multiple Language Model (LLM) AI API endpoints for self-hosting and experimentation with different models. This feature would enhance the flexibility and versatility of AutoGen for developers and researchers working with LLMs.
Feature Details:

Endpoint Configuration:

Allow users to store API keys and configure multiple endpoints.
Implement support for an environment (env) file to securely store sensitive information locally, facilitating scripted pass-through of values. Add to .gitignore

Custom Endpoint Names:

Provide the ability to assign user-friendly names to each configured endpoint. This helps users easily identify and differentiate between endpoints. It would also allow multiple models to be leveraged on the same endpoint by having different configurations for each. A check could occur to validate that the endpoint has the model expected, and if not, do a quick unload/load of the desired model.

Chat Parameters:

Integrate settings for chat parameters, such as temperature and other relevant options, that can be adjusted per endpoint. This allows users to fine-tune model behavior.

Model Selection (if applicable):

If applicable to the specific LLM, enable users to preset a model for each endpoint. This feature can be especially useful when working with multiple LLMs simultaneously.

API Key Management (if applicable):

For LLM services like OpenAI that require API keys, provide a dedicated parameter in each endpoint for users to input and manage their API keys for each endpoint.

Endpoint Address:

Allow users to specify the endpoint address (URL) to which API requests should be sent. This flexibility is crucial for self-hosted instances or when working with different LLM providers.

Optional - Endpoint Tagging:

Allowing us to add tags like #code #logic, or #budget could let us give key indicators of where a model's strengths are in, and select from a pool of models with a particular benefit, allowing more diverse outcomes. It could also allow for side-by-side comparisons. This could allow future result tracking/scoring to better identify which models are best at particular features, by having multiple #code models, and testing each's results you can identify and retrain or replace under-performing models to build an optimum workflow.

Expected Benefits:
This feature will benefit developers, researchers, and users who work with LLMs by offering a centralized and user-friendly interface for managing multiple AI API endpoints. It enhances the ability to experiment with various models, configurations, and providers while maintaining security and simplicity. This could allow different characters to leverage specific fine-tuned models rather than the same model for each. This could also allow self-hosted users to experiment with expand the number of repeated looped calls without drastically increasing the bill.
Additional Notes:
Consider implementing an intuitive user interface for configuring and managing these endpoints within the GitHub platform, making it accessible to both novice and experienced users.
References:
Include any relevant resources or references that support the need for this feature, such as the growing popularity of LLMs in various fields and the demand for flexible API management solutions.
Related Issues/Pull Requests:
Assignees:
If you permit this ticket to remain open, I will assemble some links and resources, as well as opening another ticket to handle TextGenWebUI with relevant links there to implementing it. I can try implementing and doing a PR if someone else doesn't get to it first.
Thank you for considering this feature request. I believe that this enhancement will greatly benefit the  AutoGen community and its users working with Language Model AI API endpoints.
edit: 9.28
Looking through the repo, it looks like there's a standardized json config, going to look into this next as a method for expanding and holding the features listed above. page found while reading documentation, note near top how it loads the json, then references it further down as : https://github.com/microsoft/autogen/blob/main/notebook/agentchat_groupchat_research.ipynb
Found it https://github.com/microsoft/autogen/blob/main/OAI_CONFIG_LIST_sample
Going to look into how it gets loaded.

If using textgenwebui locally, it'd be great to be able to switch between models without having to host multiple models simultaneously.

Wow, I am deeply appreciative of your work. And I am looking for a way to implement some similar target, such as to connect AutoGen to some LLM model's endpoint like 'QWen-turbo''s online service. May I join your team or at least do something for you guys?

Same! Happy to help.

@taoyiran tao So am I! It's important for me to support the qwen online server. Feel free to contact me anytime if you need assistance

@ImagineL Glad to see your message! Now I just study on this project and try to connect AutoGen to Qwen's online service. I will update my status and, if possible, my code here. Thx everyone!

@taoyiran I'm looking forward to your code sharing! I had to analyze the source code, and it seems hard to resolve if we don't modify the source code.   good luck !

In looking for how to do this, I found this thread. I also found the answer. You can just set multiple configurations. It seems as though this feature is already implemented if I understand what the feature request is.
from autogen import AssistantAgent, UserProxyAgent, GroupChat, GroupChatManager

config_list = [
    {
        "api_type": "open_ai",
        "api_base": "http://localhost:1234/v1",
        "api_key": "NULL"
    }
]
config_list2 = [
    {
        "api_type": "open_ai",
        "api_base": "http://localhost:1235/v1",
        "api_key": "NULL"
    }
]
config_list3 = [
    {
        "api_type": "open_ai",
        "api_base": "http://localhost:1236/v1",
        "api_key": "NULL"
    }
]

llm_config = {
    "config_list": config_list,
    "seed": 47,
    "temperature": 0.5,
    "max_tokens": -1,
    "request_timeout": 6000
}
llm_config2 = {
    "config_list": config_list2,
    "seed": 47,
    "temperature": 0.5,
    "max_tokens": -1,
    "request_timeout": 6000
}
llm_config3 = {
    "config_list": config_list3,
    "seed": 47,
    "temperature": 0.5,
    "max_tokens": -1,
    "request_timeout": 6000
}

user_proxy = UserProxyAgent(
    name="user_proxy",
    system_message="A human admin.",
    max_consecutive_auto_reply=10,
    llm_config=llm_config,
    human_input_mode="ALWAYS"
)

person_1 = AssistantAgent(
    name="person_1",
    system_message="sys_message",
    llm_config=llm_config2,
)

person_2 = AssistantAgent(
    name="person_2",
    system_message="sys_message",
    llm_config=llm_config3,
)

person_3 = AssistantAgent(
    name="person_3",
    system_message="sys_message",
    llm_config=llm_config,
)

groupchat = GroupChat(
    agents=[user_proxy, person_1, person_2, person_3], messages=[]
)
manager = GroupChatManager(groupchat=groupchat, llm_config=llm_config)

user_proxy.initiate_chat(manager, message="""message""")

@2good4hisowngood @taoyiran @Pakmandesign @ImagineL @weldonla  you can do this using LiteLLM Proxy Server
It can process (500+ requests/second)
Here's the quick start:
Doc: https://docs.litellm.ai/docs/simple_proxy#load-balancing---multiple-instances-of-1-model
Step 1 Create a Config.yaml
model_list:
  - model_name: gpt-4
    litellm_params:
      model: azure/chatgpt-v-2
      api_base: https://openai-gpt-4-test-v-1.openai.azure.com/
      api_version: "2023-05-15"
      api_key:
  - model_name: gpt-4
    litellm_params:
      model: azure/gpt-4
      api_key:
      api_base: https://openai-gpt-4-test-v-2.openai.azure.com/
  - model_name: gpt-4
    litellm_params:
      model: azure/gpt-4
      api_key:
      api_base: https://openai-gpt-4-test-v-2.openai.azure.com/

Step 2: Start the litellm proxy:
litellm --config /path/to/config.yaml

Step3 Make Request to LiteLLM proxy:
curl --location 'http://<IP_ADDRESS>:8000/chat/completions' \
--header 'Content-Type: application/json' \
--data ' {
      "model": "gpt-4",
      "messages": [
        {
          "role": "user",
          "content": "what llm are you"
        }
      ],
    }
'

This looks very reliable, thank you! I'm going to try it!

We are closing this issue due to inactivity; please reopen if the problem persists.


Error: spawn sass ENOENT

After downloading https://api.github.com/repos/google/web-starter-kit/zipball/v0.5.2 and running npm install in the resulting directory, gulp serve:dist produces:
$ gulp serve:dist
[20:39:22] Using gulpfile ~/tmp/google-web-starter-kit-489a6ab/gulpfile.js
[20:39:22] Starting 'clean'...
[20:39:22] Finished 'clean' after 8.35 ms
[20:39:22] Starting 'default'...
[20:39:22] Starting 'styles'...
{ [Error: spawn sass ENOENT]
  code: 'ENOENT',
  errno: 'ENOENT',
  syscall: 'spawn sass',
  path: 'sass',
  spawnargs:
   [ '--style',
     'expanded',
     '--precision',
     '10',
     '--update',
     '/tmp/gulp-ruby-sass:/tmp/gulp-ruby-sass/_14139e58-9ebe-4c0f-beca-73a65bb01ce9',
     '--load-path',
     'app/styles',
     '--load-path',
     'app/styles/components' ],
  name: 'Error',
  message: 'spawn sass ENOENT',
  stack: 'Error: spawn sass ENOENT\n    at exports._errnoException (util.js:734:11)\n    at Process.ChildProcess._handle.onexit (child_process.js:1022:32)\n    at child_process.js:1114:20\n    at process._tickCallback (node.js:349:13)',
  showStack: false,
  showProperties: true,
  plugin: 'gulp-ruby-sass' }
[20:39:23] 'styles' all files 0 B
[20:39:23] Finished 'styles' after 964 ms
[20:39:23] Starting 'jshint'...
...

Is there a step missing in https://developers.google.com/web/fundamentals/getting-started/web-starter-kit/setting-up?

The docs site currently needs an update to reflect we no longer require Ruby to use the kit. Can you try downloading https://github.com/google/web-starter-kit/archive/v0.5.3.zip, running npm install and let us know if you still run into the above issues?

@PaulKinlan You mentioned that https://developers.google.com/web/starter-kit/ is now auto-updating to use the latest version (but I may be wrong on that). It currently still points to the older 0.5.2 version rather than 0.5.3. Anything you can do to help with that?

Its in the build step.... Apparently. Mat can look at this .
On Sat, 21 Mar 2015 12:50 Addy Osmani<EMAIL_ADDRESS>wrote:

@PaulKinlan https://github.com/PaulKinlan You mentioned that
https://developers.google.com/web/starter-kit/ is now auto-updating to
use the latest version (but I may be wrong on that). It currently still
points to the older 0.5.2 version rather than 0.5.3. Anything you can do to
help with that?
—
Reply to this email directly or view it on GitHub
https://github.com/google/web-starter-kit/issues/659#issuecomment-84331849
.

cc @wibblymat :)

@addyosmani Web Fundamentals checks the version of WSK using https://github.com/jbnicolai/latest-release. This looks at releases, and checking WSK I see that 0.5.3 is tagged but the latest actual release is 0.5.2.

@wibblymat ah, that makes sense. I've just published a new release for 0.5.3 (based on my understanding of how latest-release works). Would https://github.com/google/web-starter-kit/releases/tag/v0.5.3 pull in the correct version now?

After switching to 0.5.3 (which still isn't the advertised version on https://developers.google.com/web/starter-kit/), I get:
$ gulp serve:dist
[09:33:44] Using gulpfile ~/tmp/web-starter-kit-0.5.3/gulpfile.js
[09:33:44] Starting 'clean'...
[09:33:44] Finished 'clean' after 9.66 ms
[09:33:44] Starting 'default'...
[09:33:44] Starting 'styles'...
[09:33:44] 'styles' errored after 103 ms
[09:33:44] Error: `libsass` bindings not found. Try reinstalling `node-sass`?
    at getBinding (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass/lib/index.js:22:11)
    at Object.<anonymous> (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass/lib/index.js:188:23)
    at Module._compile (module.js:444:26)
    at Object.Module._extensions..js (module.js:462:10)
    at Module.load (module.js:339:32)
    at Function.Module._load (module.js:294:12)
    at Module.require (module.js:349:17)
    at require (module.js:368:17)
    at Object.<anonymous> (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/index.js:3:17)
    at Module._compile (module.js:444:26)
[09:33:44] 'default' errored after 105 ms
[09:33:44] Error: [object Object]
    at formatError (/home/jyasskin/opt/iojs/lib/node_modules/gulp/bin/gulp.js:169:10)
    at Gulp.<anonymous> (/home/jyasskin/opt/iojs/lib/node_modules/gulp/bin/gulp.js:195:15)
    at emitOne (events.js:82:20)
    at Gulp.emit (events.js:166:7)
    at Gulp.Orchestrator._emitTaskDone (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/index.js:264:8)
    at /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/index.js:275:23
    at finish (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:21:8)
    at cb (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:29:3)
    at finish (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/run-sequence/index.js:48:5)
    at Gulp.onError (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/run-sequence/index.js:55:4)

npm install node-sass prints:
$ npm install node-sass
-
><EMAIL_ADDRESS>install /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass
> node scripts/install.js

Binary downloaded and installed at /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass/vendor/linux-x64-43/binding.node

><EMAIL_ADDRESS>postinstall /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass
> node scripts/build.js

` /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass/vendor/linux-x64-43/binding.node ` exists.
 testing binary.
Binary is fine; exiting.
<EMAIL_ADDRESS>node_modules/node-sass
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Which doesn't look obviously broken. I haven't run any steps in my OS (ubuntu) to install libsass, since I don't see any instructions to do so.

We need a push to devsite which we can do on monday.
On Sat, 21 Mar 2015 16:40 Jeffrey Yasskin<EMAIL_ADDRESS>wrote:

After switching to 0.5.3 (which still isn't the advertised version on
https://developers.google.com/web/starter-kit/), I get:
$ gulp serve:dist
[09:33:44] Using gulpfile ~/tmp/web-starter-kit-0.5.3/gulpfile.js
[09:33:44] Starting 'clean'...
[09:33:44] Finished 'clean' after 9.66 ms
[09:33:44] Starting 'default'...
[09:33:44] Starting 'styles'...
[09:33:44] 'styles' errored after 103 ms
[09:33:44] Error: libsass bindings not found. Try reinstalling node-sass?
at getBinding (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass/lib/index.js:22:11)
at Object. (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass/lib/index.js:188:23)
at Module._compile (module.js:444:26)
at Object.Module._extensions..js (module.js:462:10)
at Module.load (module.js:339:32)
at Function.Module._load (module.js:294:12)
at Module.require (module.js:349:17)
at require (module.js:368:17)
at Object. (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/index.js:3:17)
at Module._compile (module.js:444:26)
[09:33:44] 'default' errored after 105 ms
[09:33:44] Error: [object Object]
at formatError (/home/jyasskin/opt/iojs/lib/node_modules/gulp/bin/gulp.js:169:10)
at Gulp. (/home/jyasskin/opt/iojs/lib/node_modules/gulp/bin/gulp.js:195:15)
at emitOne (events.js:82:20)
at Gulp.emit (events.js:166:7)
at Gulp.Orchestrator._emitTaskDone (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/index.js:264:8)
at /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/index.js:275:23
at finish (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:21:8)
at cb (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:29:3)
at finish (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/run-sequence/index.js:48:5)
at Gulp.onError (/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/run-sequence/index.js:55:4)
npm install node-sass prints:
$ npm install node-sass

<EMAIL_ADDRESS>install /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass
node scripts/install.js

Binary downloaded and installed at /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass/vendor/linux-x64-43/binding.node

<EMAIL_ADDRESS>postinstall /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass
node scripts/build.js

/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/node-sass/vendor/linux-x64-43/binding.node exists.
testing binary.
Binary is fine; exiting.
<EMAIL_ADDRESS>node_modules/node-sass
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Which doesn't look obviously broken. I haven't run any steps in my OS
(ubuntu) to install libsass, since I don't see any instructions to do so.
—
Reply to this email directly or view it on GitHub
https://github.com/google/web-starter-kit/issues/659#issuecomment-84388716
.

@jyasskin Sorry for the dependency issues you're been running into. Can you try this?
npm install sass/node-sass#v3.0.0-alpha.1 --save-dev

and try again? It unfortunately looks like there were some build issues around Ubuntu in the last version of node-sass and they're trying to get out a release that addresses it.

I had been using IO.js. Switching back from IO.js 1.5.1 to node.js 0.12.0 fixed the problem with sass (but left me with the problem in #649).
I also tried starting from scratch with IO.js. When it installs node-sass under gulp-sass for the first time, it prints the error:
><EMAIL_ADDRESS>install /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass
> node scripts/install.js

Can not download file from https://raw.githubusercontent.com/sass/node-sass-binaries/v2.0.1/linux-x64-iojs-1.5/binding.node

><EMAIL_ADDRESS>postinstall /home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass
> node scripts/build.js

module.js:322
    throw err;
          ^
Error: Cannot find module '/home/jyasskin/tmp/web-starter-kit-0.5.3/node_modules/gulp-sass/node_modules/node-sass/node_modules/pangyp/bin/node-gyp'
    at Function.Module._resolveFilename (module.js:320:15)
    at Function.Module._load (module.js:262:25)
    at Function.Module.runMain (module.js:485:10)
    at startup (node.js:112:16)
    at node.js:865:3
Build failed

but otherwise appears to return successfully.
Then I cd'ed into node_modules/gulp-sass and ran npm install sass/node-sass#v3.0.0-alpha.1 --save-dev, which didn't print any errors. However, then gulp serve:dist from the root web-starter-kit directory says:
$ gulp serve:dist
[15:25:23] Using gulpfile ~/tmp/web-starter-kit-0.5.3/gulpfile.js
[15:25:23] Starting 'clean'...
[15:25:23] Finished 'clean' after 7.83 ms
[15:25:23] Starting 'default'...
[15:25:23] Starting 'styles'...

and quits, with no server but a successful error code.

I still get error #649 when using the latest webkit-0.5.3. I installed dependencies without issue... so no errors with node-sass for me... gulp serve - works fine... but gulp alone still throws:
~/dev.server/wsk-0.5.3 $ gulp
[14:38:34] Using gulpfile ~/dev.server/wsk-0.5.3/gulpfile.js
[14:38:34] Starting 'clean'...
[14:38:34] Finished 'clean' after 11 ms
[14:38:34] Starting 'default'...
[14:38:34] Starting 'styles'...
[14:38:39] 'styles' all files 93.17 kB
[14:38:39] Finished 'styles' after 5.28 s
[14:38:39] Starting 'jshint'...
[14:38:42] Starting 'html'...
[14:38:45] Starting 'images'...
[14:38:47] Starting 'fonts'...
[14:38:47] Starting 'copy'...
[14:38:47] Finished 'jshint' after 8.3 s
events.js:85
throw er; // Unhandled 'error' event
^
RangeError: Maximum call stack size exceeded
I get the same error when using- gulp serve:dist

gulp-sass error still there on 5.4 as well (at least on Windows), which is marked as latest. The problem goes away after downloading 6.0-r1, so I think it might be fixed. The 'latest release' tag just needs to be moved.

The very latest RC addresses this issue and we're planning on tagging a final version either this week or next. Closing as fixed (since I can't reproduce these issues on Windows or Mac).


Built-in template tags relying on RequestContext broken

Praise
First off, I want to thank you for this awesome library!
I wrote something superficially similar, but it's just as a wrapper for django-components.
Your solution really takes away all the boilerplate and makes component based design really fun in Django. :tada:
Problem
I noticed a minor issue, when using the new querystring template tag, which usees request.context internally.
This breaks, when used inside a django-cotton component, because of flattening the context into a plain dict during rendering: Link
I'm not sure, what a good fix would be. Maybe re-creating a RequestContext object after flattening? Or maybe not flattening at all? I'm not entirely sure why it is needed.
In my particular case, this is easily solvable, but I guess a good compatibility with standard Django would be nice.

@fbinz - thanks for words of support! I also see the benefits and enjoy the ease I can create my frontends now! I spotted your library recently and appreciated the clean approach in the loader.
So, this package was kind of rushed together so I could stay productive elsewhere, and among others, context handling is an area that needs ironing out.. I'm reactively addressing one by one.  Was it literally when you tried to use the new querystring tag?

Fixed in latest release v0.9.39


notifications: show notifications for all active dates

This seems like an obvious/easy fix, but maybe (probably) there was a reason it wasn't done like this to begin with?
It works for me so far.
Closes https://github.com/nvim-orgmode/orgmode/issues/374

I have made the same change in my fork, and the reasoning in the linked issue is sound and I agree with it, but from a software product standpoint it would make sense to make it a configurable option for backwards compatibility and extensibility.

Hey, sorry for not getting back to you @jgollenz, and thanks for your interest @dtvillafana. I'm still using this patch as well.
Making it configurable is of course always an option but to be honest I see absolutely no reason not to include active dates in the agenda, I mean otherwise what's the difference to passive dates?

Sorry, forget what I said. This is not about the agenda but about notifications. Yes I suppose one could have a config option whether to show notifications for active timestamps. I still wonder what the reasoning is to not show notifications for timestamped events by default, but to show them for SCHEDULED and DEADLINE.

I took a look at the docs and actually the behavior you expect should already be happening, so strictly speaking this is a bug fix :wink: It says agenda tasks notifications there, so you are absolutely right. We also already have options for this, albeit the other way round:
deadline_reminder = true,
scheduled_reminder = true,

I tested your change and it looks good to me. When you mark the PR as ready I can merge it :+1:

it would make sense to make it a configurable option for backwards compatibility and extensibility

In this case behavior was actually not in line with what the docs say, so I don't consider it necessary to add an option. If need for it exists, we can think of introducing it but for now this is a bug-fix.

Ok, ready for review!

Did some more checks and it seems that
deadline_reminder = false,
scheduled_reminder = false,

have no effect any more. I get notifications for both DEADLINE and SCHEDULED tasks even when they should be turned off. Can you confirm this?

Sorry, I was too eager to just fix my own use-case and didn't think about these settings. I'll have to make this PR much more through. I'll switch it back to draft.


feat: update rsbuild version

feat: update rsbuild version

https://github.com/web-infra-dev/modern.js/pull/5343


Enhancement: Split Screen (?)

I know this is really far fetched, and would require some really big changes to how a lot of things work, but it would be a cool addition if this ever managed to make it into the modpack. I totally understand if this is way out of scope though; there's a reason that nobody has ever made a split screen mod for java, ever.

iirc l4j is compatible with nucleus co-op, allowing for splitscreen

iirc l4j is compatible with nucleus co-op, allowing for splitscreen
https://github.com/Wilyicaro/Legacy-Minecraft/issues/257#issuecomment-2095051937 yep

#257


feat(tikv/migration): add more test case tikvcdc

add more integration test cases
replay link:
https://do.pingcap.net/jenkins/blue/organizations/jenkins/tikv%2Fmigration%2Fpull_integration_test/detail/pull_integration_test/10/pipeline/314

/cc @zeminzhou

LGTM~


I can't get the same sexuality dataset as mentioned in the paper or github.

The sexuality dataset is colletced from Jan 2018 to Oct 2018 by PushShif as the author mentioned in the paper.
I downloaded the data as above, decompressed the data, and then run on the preprocessor_gab.py. I got 20,360,815 posts , far exceeding 2,894,869 mentioned in the paper.
How you fixed this ?

Hello LoveGaara, thank you for your interest in our work. Can you send me an email<EMAIL_ADDRESS>and I will send you the dataset?

Thanks for your kind corperation. I sent you an email<EMAIL_ADDRESS>just now.I am looking forward to your early reply. All the bests.


Configuration#4:Step#1: simulating alpha_S=0.2, matter: q_0=1 | matterlbt: q_0=2: matterlbt06

run the simulation with the above config by jetscape,
transfer the final state hadron to the related directory named as config-XX-final-state-hadrons
after doing step1 on all 12 simulations go to step 2 for the configurationX

port to Excel for overleaf


Switch to rabbitmq/amqp091-go

Hi,
Following #27, I'm testing if this library working with rabbitmq/amqp091-go instead of streadway/amqp.
On my firsts tests on my project (based with RMQ microservices), all features works with no changes.

Thanks for contributing!


Tasks after Meeting 12May15

[ ] Add instructions in 'How to contribute' for contributors to vignettes to manually add their names to the bottom of the vignettes  @coughls
[ ] Test citations using Bibtech @hlapp (logged as #40)
[x] Create an .md file linking existing sites with supporting information @mmlopezu
[ ] Please add microsatellite vignette @mgiresi
[ ] Create .Rmd files for developers @zkamvar

@zkamvar tried making a pull request of the file with the modified extension (.md instead of .Rmd) but I do not see it in github.

@zkamvar problem fixed. The .md file is in my branch (Sequence_vignette_update) in Github. Feel free to add the developer stuff in there.


Revert #15983 - gRPC channels blocking indefinitely and not respecting deadlines on network disconnect

Revert #15983 - gRPC channels blocking indefinitely and not respecting deadlines on network disconnect

****************************************************************

libgrpc.so

     VM SIZE                                                                   FILE SIZE
 ++++++++++++++ GROWING                                                     ++++++++++++++

 -------------- SHRINKING                                                   --------------
  -0.1%    -648 [None]                                                      -2.45Ki  -0.0%
  -0.6%    -176 src/core/ext/transport/chttp2/transport/chttp2_transport.cc    -176  -0.6%
     -21.1%     -95 set_write_state                                                 -95 -21.1%
      -6.3%     -39 grpc_chttp2_complete_closure_step                               -39  -6.3%
      -1.2%     -29 grpc_chttp2_cancel_stream                                       -29  -1.2%
      -1.4%     -10 [Unmapped]                                                      -10  -1.4%
      -2.7%      -8 grpc_chttp2_add_ping_strike                                      -8  -2.7%
  -6.4%    -128 src/core/ext/transport/chttp2/transport/stream_lists.cc        -128  -6.4%
      [DEL]     -40 grpc_chttp2_list_add_waiting_for_write_stream                   -40  [DEL]
      [DEL]     -40 grpc_chttp2_list_remove_waiting_for_write_stream                -40  [DEL]
     -14.4%     -22 [Unmapped]                                                      -22 -14.4%
      -7.8%     -16 stream_list_id_string                                           -16  -7.8%
      [DEL]     -10 grpc_chttp2_list_pop_waiting_for_write_stream                   -10  [DEL]

  -0.1%    -952 TOTAL                                                       -2.75Ki  -0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] Performance differences noted:
Benchmark                                                                                atm_add_per_iteration    cpu_time    real_time
----------------------------------------------------------------------------------------  -----------------------  ----------  -----------
BM_PumpStreamClientToServer<InProcess>/2097152                                                                     -8%         -8%
BM_PumpStreamClientToServer<InProcess>/262144                                                                      -7%         -7%
BM_PumpStreamServerToClient<InProcess>/2097152                                                                     -7%         -7%
BM_StreamCreateSendInitialMetadataDestroy<RepresentativeClientInitialMetadata>            -4%
BM_StreamingPingPong<InProcess, NoOpMutator, NoOpMutator>/262144/1                                                 -4%         -4%
BM_StreamingPingPong<MinInProcess, NoOpMutator, NoOpMutator>/262144/2                                              -4%         -4%
BM_StreamingPingPongWithCoalescingApi<InProcess, NoOpMutator, NoOpMutator>/262144/1/1                              -8%         -8%
BM_StreamingPingPongWithCoalescingApi<InProcess, NoOpMutator, NoOpMutator>/262144/2/1                              -9%         -9%
BM_StreamingPingPongWithCoalescingApi<MinInProcess, NoOpMutator, NoOpMutator>/262144/1/1                           -4%         -4%
BM_TransportStreamSend/0                                                                  -12%
BM_TransportStreamSend/1                                                                  -11%
BM_TransportStreamSend/134217728                                                          -8%
BM_TransportStreamSend/16777216                                                           -9%
BM_TransportStreamSend/2097152                                                            -9%
BM_TransportStreamSend/262144                                                             -11%
BM_TransportStreamSend/32768                                                              -11%
BM_TransportStreamSend/4096                                                               -11%
BM_TransportStreamSend/512                                                                -11%
BM_TransportStreamSend/64                                                                 -11%
BM_TransportStreamSend/8                                                                  -11%
BM_UnaryPingPong<InProcess, NoOpMutator, NoOpMutator>/0/2097152                                                    -6%         -6%
BM_UnaryPingPong<InProcess, NoOpMutator, NoOpMutator>/0/262144                                                     -6%         -6%
BM_UnaryPingPong<MinInProcess, NoOpMutator, NoOpMutator>/0/2097152                                                 -5%         -5%
BM_UnaryPingPong<MinInProcess, NoOpMutator, NoOpMutator>/262144/262144                                             -6%         -6%

****************************************************************

libgrpc.so

     VM SIZE                                                                   FILE SIZE
 ++++++++++++++ GROWING                                                     ++++++++++++++

 -------------- SHRINKING                                                   --------------
  -0.1%    -648 [None]                                                      -2.45Ki  -0.0%
  -0.6%    -176 src/core/ext/transport/chttp2/transport/chttp2_transport.cc    -176  -0.6%
     -21.1%     -95 set_write_state                                                 -95 -21.1%
      -6.3%     -39 grpc_chttp2_complete_closure_step                               -39  -6.3%
      -1.2%     -29 grpc_chttp2_cancel_stream                                       -29  -1.2%
      -1.4%     -10 [Unmapped]                                                      -10  -1.4%
      -2.7%      -8 grpc_chttp2_add_ping_strike                                      -8  -2.7%
  -6.4%    -128 src/core/ext/transport/chttp2/transport/stream_lists.cc        -128  -6.4%
      [DEL]     -40 grpc_chttp2_list_add_waiting_for_write_stream                   -40  [DEL]
      [DEL]     -40 grpc_chttp2_list_remove_waiting_for_write_stream                -40  [DEL]
     -14.4%     -22 [Unmapped]                                                      -22 -14.4%
      -7.8%     -16 stream_list_id_string                                           -16  -7.8%
      [DEL]     -10 grpc_chttp2_list_pop_waiting_for_write_stream                   -10  [DEL]

  -0.1%    -952 TOTAL                                                       -2.75Ki  -0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Known "out of space" error in MacOS test.
Bazel tests failing due to timeout.

[microbenchmarks] Performance differences noted:
Benchmark                                                                       atm_add_per_iteration
------------------------------------------------------------------------------  -----------------------
BM_StreamCreateSendInitialMetadataDestroy<RepresentativeClientInitialMetadata>  -4%
BM_TransportStreamSend/0                                                        -12%
BM_TransportStreamSend/1                                                        -11%
BM_TransportStreamSend/134217728                                                -8%
BM_TransportStreamSend/16777216                                                 -9%
BM_TransportStreamSend/2097152                                                  -9%
BM_TransportStreamSend/262144                                                   -11%
BM_TransportStreamSend/32768                                                    -11%
BM_TransportStreamSend/4096                                                     -11%
BM_TransportStreamSend/512                                                      -11%
BM_TransportStreamSend/64                                                       -11%
BM_TransportStreamSend/8                                                        -11%


Receiving "Failed to set admission audit annotation... annotation key has invalid format, the right format is a DNS subdomain prefix and '/' and key name."

Greetings,
I've deployed this project to AWS EKS v1.21, and noticed that the webhook pod is issuing valid AdmissionReviews, it seems that have field allowed: true, but the API server doesn't accept them. So I've looked in Cloudwatch Logs and found the following error messages:
W1207 11:32:02.918031      11 dispatcher.go:268] Failed to set admission audit annotation namespace-node-affinity.namespace-node-affinity.svc/namespace-node-affinity.idgenchev.github.com/applied-patch to [
{
    "op": "add",
    "path": "/spec/affinity",
    "value": {
        "nodeAffinity": {
            "requiredDuringSchedulingIgnoredDuringExecution": {
                "nodeSelectorTerms": [
                    {
                        "matchExpressions": [
                            {
                                "key": "dedicated",
                                "operator": "Equal"
                            }
                        ]
                    }
                ]
            }
        }
    }
}
,
{
    "op": "add",
    "path": "/spec/tolerations/-",
    "value": {
        "key": "dedicated",
        "operator": "Equal",
        "value": "critical",
        "effect": "NoSchedule"
    }
}
] for mutating webhook namespace-node-affinity.namespace-node-affinity.svc: annotation key has invalid format, the right format is a DNS subdomain prefix and '/' and key name. (e.g. 'podsecuritypolicy.admission.k8s.io/admit-policy')

EDIT: and the new pods do not appear in cluster, so with this hook deployed, if it tries to apply configuration, new pod definition will be rejected and not created in cluster, so this is a blocking bug, which stops us from using this useful piece of software.

Seems to be a PEBKAC, sorry. Turns out you can't use operator: Equal in matchExpressions for nodeSelectorTerms - after i switched to using operator: In, and values: [...] it worked.

Okay, I'm sorry - it turns out the problem still exists. It's just that it doesn't happen on every pod/namespace:
W1207 11:54:47.015951      11 dispatcher.go:268] Failed to set admission audit annotation namespace-node-affinity.namespace-node-affinity.svc/namespace-node-affinity.idgenchev.github.com/applied-patch to [
{
    "op": "add",
    "path": "/spec/affinity/nodeAffinity/requiredDuringSchedulingIgnoredDuringExecution/nodeSelectorTerms/-",
    "value": [
        {
            "matchExpressions": [
                {
                    "key": "dedicated",
                    "operator": "In",
                    "values": [
                        "critical"
                    ]
                }
            ]
        }
    ]
}
,
{
    "op": "add",
    "path": "/spec/tolerations/-",
    "value": {
        "key": "dedicated",
        "operator": "Equal",
        "value": "critical",
        "effect": "NoSchedule"
    }
}
] for mutating webhook namespace-node-affinity.namespace-node-affinity.svc: annotation key has invalid format, the right format is a DNS subdomain prefix and '/' and key name. (e.g. 'podsecuritypolicy.admission.k8s.io/admit-policy')

Hi @erplsf thanks for raising this!
Looks like the annotation key that the webhook is trying to set here has the wrong format. I'll look into that, meanwhile it'll be super useful if you can share a bit more info around your setup so I can try and reproduce this. For example is the webhook deployed on the same cluster, is it in the same namespace as the one it's enabled for, etc.

Sure, absolutely!
webhook is deployed on the same cluster, in it's own namespace, configmap for it also lives in the same namespace. I use the default deployment with init container that creates CA bundle for me, and I pin the images to latest release - v2.1.1. I have configured two separate, different namespaces with same config - the idea that I have is to reserve my nodes for all pods in some namespace. In one namespace there's Istio system pods, in the other openfaas related ones.

and what's more interesting, pods in one namespace (openfaas) are allowed and created, and in (istio) they don't - I think this is related to the fact that the original pods also have preconfigured nodeAffinity and webhook somehow conflicts with it.

Hey @erplsf looks like you have hit a bug with the tool that @ca-scribner is working on to fix as part of #31. I'll be pushing a new release as soon as that's merged.
Having said that, from the error message it looks like the annotationKey has an incorrect format, so I'll spend some time to see if I can reproduce this anyway :)


GTB returns zeros

As discussed on Microsoft  Discord about a custom MIDi 2.0 device that showed zeros for the GTB in Workbench.  There was some doubt about the device. Due to lack of any peer device, tried a Linux USB MIDI 2.0 gadget.
A. How to reproduce:

Ubuntu 23.10 updated with 6.7.5 kernel and USB gadget support
Gadget created for MIDI 2.0
Workbench report as follows:

B. A tool  to read the GTB descriptors with libusb returns a correct GTB for the gadget:

C. Conclusion : would think there is an issue in Workbench.

Can you please send through the commands you used to create the Linux Gadget?
This will help in replicating the issue.
Thanks

I just tried to replicate your issue using a MIDI 2 Gadget and I get:

It seems I'll need your script to create the Linux Gadget to replicate this issue.

Tried the gadget from the USB Linux page. Script attached.
midi2_gadget.sh.txt
it curious why the config descriptors isn't parsed correctly.

Add the lsusb log for the gadget:
gadget_log.txt

Any further ideas?

Ok I finally found some of the issue around this.
When I connect:

Linux Gadget -> OSX = Working as expected
Linux Gadget -> Linux = (After small fix) Mostly Working - Getting GTB's seems to be odd. I think I might have a race condition
A88-MK2 -> OSX = Working as expected
A88-MK2 -> Linux = Same Issue as Linux Gadget

I will keep investigating - however it may be an unsolvable issue for Linux host.

Ok I had disabled the getting GTB infomation over LibUSB where the MIDI 2.0 Kernel is used as it always causes an error. See https://github.com/node-usb/node-usb/issues/749


How to use provider_configs with the API?

The docs seems to be light on that one or I just don't understand it (that's probably the what's happening here...).
How can I use the provider_configs argument with the download_best_subtitles function? I understand that this argument require a dictionary but how to format it?
Thanks!

Ok, finally found it by myself:
providers_list = ['addic7ed','legendastv','opensubtitles']
providers_auth = {'addic7ed': {'username': 'test', 'password': 'test'},'legendastv': {'username': 'test', 'password': 'test'},'opensubtitles': {'username': 'test', 'password': 'test'}}

download_best_subtitles([video], {Language('eng')}, providers=providers_list, provider_configs=providers_auth)

Hope it help other in the future!

it does not work for me? version 2.0.5
download_best_subtitle needs class_pool argument from ProviderPool class, which is not callable from the subliminal core.


Dump github event information in CI

cc @jihuayu

LGTM. Do you mean we can parser this JSON in sonar action and get more information?

Yeah. You can use some cli tools like jq, which is already installed.


Exception with unclear message

Hello,
The validator throws "Cannot read properties of null (reading '0')"
Probably expecting reading an array?
The exception occurs while validating the combination of the files in the attachment:
test shacl.zip
It runs ok when the propertyshape "gwsw_cfk:AfmetingLeiding_card" is removed.
Also it runs ok without checking on closed properties.
Hope this helps, with best regards.

I was able to reproduce the problem. See the PR for more details. A test was added and the bug was fixed.
Fixed in #17 released in v0.1.5

Ok Thomas, thanks again!
Best regards, Marinus
Van: Thomas Bergwinkl @.>
Verzonden: woensdag 6 maart 2024 22:31
Aan: rdf-ext/shacl-engine @.>
CC: Marinus @.>; Author @.>
Onderwerp: Re: [rdf-ext/shacl-engine] Exception with unclear message (Issue #16)
I was able to reproduce the problem. See the PR for more details. A test was added and the bug was fixed.
Fixed in #17https://github.com/rdf-ext/shacl-engine/pull/17 released in v0.1.5
—
Reply to this email directly, view it on GitHubhttps://github.com/rdf-ext/shacl-engine/issues/16#issuecomment-1981821226, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AETWJ6GIDDV5WC3YBBJLGB3YW6DIDAVCNFSM6AAAAABD52LEQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBRHAZDCMRSGY.
You are receiving this because you authored the thread.Message ID: @.***>


Align Event error handling with the error handling for reading of Attributes

The same errors should be generated for reading events as those that would be generated for reading attributes:

generate UNSUPPORTED_NODE
generate UNSUPPORTED_ENDPOINT
generate UNSUPPORTED_CLUSTER
generate UNSUPPORTED_EVENT

Note that right now none of this error generation is implemented at all.  And implementing it may require data that the SDK does not really want to carry around...


OMG Network Upgrade to deploy v0.2

We need to perform a "hard" network upgrade to deploy a new version of plasma-contracts and the features of elixir-omg that are dependent of the new version.
This hard upgrade will require all token holders on Ari to perform an exit and re-deposit into the new chain.

There are multiple related steps/tasks/parts in the upgrade story. Currently:

Tests for Non-Consolidated Exits  is happening this week
Tests for Consolidated Exits (TODO)
Actual Coordination of Exit and re-deposit process (TODO)

I will make 2. and 3. as individual GH issues and we will have the discussions there

There are some current blockers to upgrade path:

We need a quick solution to handling process exits/ giving visibility now before the upgrade happens. Otherwise the Exit queue will be clogged with transactions and plasma-dog/ ODP users will not have any oversight on whether they received their funds back on the Rootchain.

V0.2 Compatibility needs to happen on a) omg-js, js-starter-kit before we can spin up a V0.2 cluster. Otherwise people will not be able to interact with the network

RE: Temporary Solution to dealing with clearing out the clogged exit queue, we do have a few options, given the time frame we have CC @pdobacz :
1/ User calling processExit themselves. This is the default way provided by upgrade scripts.
pros:

user dealing with their own funds
cons:
user still does not know how many exits there are, or how many is in the queue, or where their exits are.

2/ We (OmiseGO) constantly poll ExitQueue.currentSize() for number of exits and calling processExit for users.
pros:

1 less upgrade step for user
cons:
require building a service ourselves (albeit a temporary one)
require a lot of gas

IMO, point 2 is an option we can take

:+1: 2/ sounds good, also there is a twist we can have - we only monitor ExitQueue.currentSize() and postpone work on auto-process exiting for the users until we see it bloats. Just to cover the scenario when few users decide to exit.
In case only few users decide to exit and the queue doesn't get clogged up, it's better to have the pro "user dealing with their own funds" for exploration of the UX and a more real-life experience there.
An unrelated question - how are you figuring out which ExitQueues to poll? there's one per each token addTokened. Are we leveraging the fee_specs from the child chain deployment?
Oh, one more: did we addToken all the tokens people will consider exiting?

In case only few users decide to exit and the queue doesn't get clogged up, it's better to have the pro "user dealing with their own funds" for exploration of the UX and a more real-life experience there.>

Yup, great idea !

An unrelated question - how are you figuring out which ExitQueues to poll? there's one per each token addTokened. Are we leveraging the fee_specs from the child chain deployment?>

Yup, from fee_specs and Plasma Dog tokens. Another alternative is to get the logs of AddToken() events.

Oh, one more: did we addToken all the tokens people will consider exiting?>

I might have missed this completely in ERC20 tokens 😨 I don't think even Hoard is aware that this steps needs to be taken first, but I can check with Radek. I will make this crystal clear in the Exit + Upgrade guides.
Luckily enough for us, most people have been transacting with ETH on Ari. So aside from edge-cases and Plasma Dog, people will be exiting ETH mostly

Luckily enough for us, most people have been transacting with ETH on Ari. So aside from edge-cases and Plasma Dog, people will be exiting ETH mostly

Not sure what CP did and whether their tokens are in the fee_specs (unless they're planning to exit, possibly not since this is not within the scope of their usecase).
 You're right -  (possibly) facilitating the "managed" exits done by Plasma Dog a bit + explanation in guides should do.


[WIP] Move missing hparams creation logic down to _load_model_state

This PR fixes the hparams override logic in load_from_checkpoint.
What does this PR do?
The crux of the matter is that load_from_checkpoint first checks if hparams is missing and then adds an empty dict. However in _load_model_state the newly updated, but formerly non-existent hparams gets added to model's args.

Is there an issue that relates to this PR? Would be good to know how to reproduce the bug so we can write a test for it.

I apologize, for not going through the proper process of making an issue and then PR-ing. I can remove the PR and create an issue instead.
The issue arose when i tried to do something like this:
import torch
import pytorch_lightning as pl

class MyModel(pl.LightningModule):
    def __init__(self, n, m, *, bias=False):
        super().__init__()
        self.n, self.m = n, m
        self.model = torch.nn.Linear(n, m, bias=bias)

    def forward(self, x):
        return self.model(x)

    def training_step(self, batch, batch_nb):
        return {'loss' : torch.abs(1 - self(batch).sum())}

    def train_dataloader(self):
        if not hasattr(self, 'data_'):
            self.data_ = torch.randn(4, self.n)
        return self.data_

    def configure_optimizers(self):
        return torch.optim.Adam(self.parameters())

# fit the model
trainer = pl.Trainer(gpus=0, max_epochs=20)

trainer.fit(MyModel(4, 1))

trainer.save_checkpoint("example.ckpt")

# restore the model
MyModel.load_from_checkpoint("example.ckpt", 4, 1)

raises TypeError: __init__() takes 3 positional arguments but 4 were given. Upon inspection i noticed that cls_args contains tuple ({}, 4, 1), and not 4,1 a s expected. The following patch seemed to solve this:
from pytorch_lightning.core.saving import pl_load

def load_from_checkpoint(cls, checkpoint_path, *args, map_location=None, **kwargs):
    if map_location is not None:
        checkpoint = pl_load(checkpoint_path, map_location=map_location)
    else:
        checkpoint = pl_load(checkpoint_path, map_location=lambda storage, loc: storage)

    return cls._load_model_state(checkpoint, *args, **kwargs)

load_from_checkpoint(MyModel, "example.ckpt", 1, 4)

Here I essentially skip the cls.CHECKPOINT_HYPER_PARAMS_KEY checking logic entirely.


Sub aggregates

This is an alternative solution to issues 383.  There's a way to do sub-aggregates, which can show averages,min/max,etc... as shown in this file.
If I had a pivot table like this, I can get these values to generate error plots.  Is sub-aggregates a consideration?  Thanks.

It is not a consideration, unfortunately, see the related FAQ question


MINOR: Add missing @Test annotation to MetadataTest#testMetadataMerge

We missed adding the @Test annotation to said method which meant it wouldn't run on unit test execution.

cc @dengziming @bdbyrne @rajinisivaram

LGTM, thanks!

ok to test


Question: How to use subquery with GT?

Hi maintainers, thank you for the nice library!
I want to use GT with subquery.
I tried the following code. But it seems to be something wrong..
todos, err = r.client.Todo.Query().Where(func(s *sql.Selector) {
	t := sql.Table(todo.Table)
	s.Where(
		sql.GT(
			s.C(todo.FieldCreatedAt),
			sql.Select(
				t.C(todo.FieldCreatedAt)).
				From(t).
				Where(sql.EQ(t.C(todo.FieldID), "some id")).
				Limit(1),
		),
	)
}).Order(ent.Desc("created_at")).All(ctx)

expected
SELECT DISTINCT `todos`.`id`, `todos`.`title`, `todos`.`description`, `todos`.`done`, `todos`.`created_at` FROM `todos`
WHERE `todos`.`created_at` > (SELECT `todos`.`created_at` FROM `todos` WHERE `todos`.`id` = ? LIMIT 1) ORDER BY `todos`.`created_at` DESC

got
SELECT DISTINCT `todos`.`id`, `todos`.`title`, `todos`.`description`, `todos`.`done`, `todos`.`created_at` FROM `todos`
WHERE `todos`.`created_at` > SELECT `todos`.`created_at` FROM `todos` WHERE `todos`.`id` = ? LIMIT 1 ORDER BY `todos`.`created_at` DESC

Thank you!

Hey @kitagry and thanks for the kind words ❤
I added support for this in #2237. Please, use go get entgo.io/ent@master to upgrade your version to master. Thanks 🙏

Thank you for the prompt response! It works well! Thank you!


Fix the bug in plasticity_3d.py

It seems like we need to add the optimizer.step() in line 228 of plasticity_3d.py.
Without this line of code, the model cannot be optimized during training.

Thanks for the catch! It was probably mistakenly removed when we cleaned and uploaded the code.

Thanks for your prompt reply.


add merge-stream to clean the code

we can use merge-stream to clean the counter

:+1:  Nice! Will test locally then merge. Just came across merge-stream a couple weeks ago. Perfect use case. Thanks.


mwc-textfield label not updated after value change

If the field is empty and the label is inside the field, setting value programmatically doesn't trigger the label repositioning.
Here's the reproduction : https://mwc-textfield-update-value.glitch.me/ (workaround included)

👍


Remove Android WebView from view system before calling destroy()

Bug description:
React-native needs to remove WebView from the view hierarchy before calling .destroy(). From this comment it seems react-native is not doing so:

...
W/cr_AwContents: WebView.destroy() called while WebView is still attached to window.
...

This probably isn't causing a crash (WebView tries to handle this case gracefully), but older versions of WebView may not handle this as gracefully (I've observed crashes on really old Android versions).
To Reproduce:
Not sure. This log message probably repros every time you trigger the cleanup code path.
Expected behavior:
The documentation states:

This method should be called after this WebView has been removed from the view system. No other methods may be called on this WebView after destroy.

https://developer.android.com/reference/android/webkit/WebView.html#destroy()
Screenshots/Videos:
N/A
Environment:

OS: Android
OS version: probably all
react-native version:
react-native-webview version:

Hi Nate is there any fix for the above issue coming soon . We are facing exactly the same issue. on android.

I don't work on react-native-webview, but the fix would need to be in this project.

I resolved with using conditional rendering, like this. Before pop action, I set show state to false
const AccountDetailScreen = ({route, navigation}) => {
  const {title, url} = route.params;
  const [show, setShow] = useState(true);

  useFocusEffect(
    useCallback(() => {
      const onBackPress = () => {
        setShow(false);
        return false;
      };
      BackHandler.addEventListener('hardwareBackPress', onBackPress);
      return () =>
        BackHandler.removeEventListener('hardwareBackPress', onBackPress);
    }, []),
  );

  const goBack = () => {
    setShow(false);
    navigation.pop();
  };

  return (
    <SafeAreaView style={[styles.container]}>
      <Header title={title} onPress={goBack} />
      {show && (
        <WebView
          style={styles.view}
          renderLoading={() => <Loader backgroundColor="white" />}
          startInLoadingState
          source={{uri: url}}
        />
      )}
    </SafeAreaView>
  );
};
```_

when using react navigation, we can hide WebView in event listener

  React.useEffect(() => navigation.addListener('blur', () => setShow(false)), [ navigation ]);

when using react navigation, we can hide WebView in event listener
  React.useEffect(() => navigation.addListener('blur', () => setShow(false)), [ navigation ]);

work here!!!! thank you <3


HDDS-9261. SCM Decomissioning Robot tests improvement

What changes were proposed in this pull request?
Improving the SCM decommisioning robot tests. Refactoring the code and adding pre and post decommissioning data integrity check.
What is the link to the Apache JIRA
HDDS-9261
How was this patch tested?
Modifed the robot test.

@swamirishi would you like to review when this is out of draft?

@swamirishi would you like to review when this is out of draft?

Hi @errose28 @swamirishi ,Marked it ready for review.

@LZD-PratyushBhatt , could you check the failed acceptance test. It's caused by the code change in patch.

@LZD-PratyushBhatt , could you check the failed acceptance test? It's caused by the code change in patch.

Hi @ChenSammi , Added a fix, please check. Thanks!
Ran CI on my fork, looks good now: https://github.com/LZD-PratyushBhatt/ozone/pull/2/checks?check_run_id=16864862995

@LZD-PratyushBhatt , could you check the failed acceptance test? It's caused by the code change in patch.

Hi Sammi, fixed the test. Failed splits are passing in my fork. https://github.com/LZD-PratyushBhatt/ozone/actions/runs/6224020946/job/16892248612


delete a race ?

how to design it ? hard to place on small screens
maybe the race need to be focusable and deletable with suppr key

Solution : swipe race to show buttons under it 🤩

Not that easy to implement and it's not a priority, I will see later 😐


call window.URL.revokeObjectURL(file.preview) to avoid memory leak

fixes https://github.com/okonet/react-dropzone/issues/398

What kind of change does this PR introduce?

[ ] bugfix
[ ] feature
[ ] refactoring / style
[ ] build / chore
[x] documentation

Did you add tests for your changes?

[ ] Yes, my code is well tested
[x] Not relevant

If relevant, did you update the documentation?

[x] Yes, I've updated the documentation
[ ] Not relevant

Summary

Does this PR introduce a breaking change?

Other information

Fixed. Thanks :)

Awesome! Thanks for this


straight-prune-build-cache: print message when pruning

See #727

Thanks @progfolio!
I have learned more about straight and saw that another function is straight-prune-build-directory which is even more destructive than straight-prune-build-cache (especially if the wrong directory is being deleted for some reason). Do you think a similar message should be added there as well?

Do you think a similar message should be added there as well?

Added in #735


Report the skipped files when exit code 60450

Fixes #7467 (well, there is nothing to be fixed, just print the files skipped)
Report in wmstats the exact files that were skipped.

@ticoann Seangchan, I didn't manage to reproduce this issue, so I don't know if this patch is working as expected :)
Can you please review it? I'm not sure whether we add it to the new agent or not...

Patch looks good to me. I think we should add in the new agent.

@ticoann feel free to merge this one. I've also picked it for the wmagent branch already.


Still maintained?

Hello, is this repo still maintained...? Last commit was 18 Nov 2017 and very few comments on issues or PR's since then... 😒
If this is not maintained is there a fork that someone is maintaining? If not is there another handlebar helpers package that can replace this one that is maintained.
The reason is that I would like to use a package that it maintained.

Bump. I am wondering the same thing.

Please read this: https://github.com/jonschlinkert/idiomatic-contributing#code-of-conduct-and-beyond
Closing since there will never be a time when I answer this question, or view the question as anything but trolling.


Could not locate MSBuild instance to register with OmniSharp

Hello,
I have installed omnisharp-vim with plug-vim.
And success installed omnisharp-roslyn.
But when the server start, it shows the message as title.
Logs like this:
[info]: OmniSharp.Stdio.Host
8     ¦   Starting OmniSharp on kali 2022.2 (x64)
9 [info]: OmniSharp.Services.DotNetCliService
10     ¦   Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK
11 [info]: OmniSharp.Services.DotNetCliService
12     ¦   Using the 'dotnet' on the PATH.
13 [info]: OmniSharp.Services.DotNetCliService
14     ¦   DotNetPath set to dotnet
15 [info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
16     ¦   Located 0 MSBuild instance(s)
I have installed dotnet and mono.
dotnet msbuild -h
Microsoft (R) Build Engine version 17.2.0+41abc5629 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.
mono --version
Mono JIT compiler version <IP_ADDRESS> (Debian <IP_ADDRESS>+dfsg-3.2 Tue Jun 29 21:01:01 UTC 2021)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
What should i do to solve this problem?

If your project is a .net core project, try using the net6 version of the server. Installation steps described here: https://github.com/OmniSharp/omnisharp-vim/issues/791#issuecomment-1170502526

If your project is a .net core project, try using the net6 version of the server. Installation steps described here: #791 (comment)

Thanks.
The problem solved  after setting "g:OmniSharp_server_use_net6 = 1" and reinstall the omnisharp server.


measuring 3d coorfdination for each part of face.

regarding to this code
P3D_RIGHT_SIDE = numpy.float32([-100.0, -77.5, -5.0]) #0
P3D_GONION_RIGHT = numpy.float32([-110.0, -77.5, -85.0]) #4
P3D_MENTON = numpy.float32([0.0, 0.0, -122.7]) #8
P3D_GONION_LEFT = numpy.float32([-110.0, 77.5, -85.0]) #12
P3D_LEFT_SIDE = numpy.float32([-100.0, 77.5, -5.0]) #16
P3D_FRONTAL_BREADTH_RIGHT = numpy.float32([-20.0, -56.1, 10.0]) #17
P3D_FRONTAL_BREADTH_LEFT = numpy.float32([-20.0, 56.1, 10.0]) #26
P3D_SELLION = numpy.float32([0.0, 0.0, 0.0]) #27
P3D_NOSE = numpy.float32([21.1, 0.0, -48.0]) #30
P3D_SUB_NOSE = numpy.float32([5.0, 0.0, -52.0]) #33
P3D_RIGHT_EYE = numpy.float32([-20.0, -65.5,-5.0]) #36
P3D_RIGHT_TEAR = numpy.float32([-10.0, -40.5,-5.0]) #39
P3D_LEFT_TEAR = numpy.float32([-10.0, 40.5,-5.0]) #42
P3D_LEFT_EYE = numpy.float32([-20.0, 65.5,-5.0]) #45
#P3D_LIP_RIGHT = numpy.float32([-20.0, 65.5,-5.0]) #48
#P3D_LIP_LEFT = numpy.float32([-20.0, 65.5,-5.0]) #54
P3D_STOMION = numpy.float32([10.0, 0.0, -75.0]) #62
#The points to track
#These points are the ones used by PnP
to estimate the 3D pose of the face
TRACKED_POINTS = (0, 4, 8, 12, 16, 17, 26, 27, 30, 33, 36, 39, 42, 45, 62)
ALL_POINTS = list(range(0,68)) #Used for debug only
that code explain that, there are 68 points of the face. every point is represent by 3d coord.
the thing that i confused is how can i determine that ? (let say how can i measure that stomion is 10, 0, -75
i need to measure the gap between eyelid to determine whether the aye is closed or not by measuring both axis, and i have no idea to find 3d coord, could you please help me?

Hi @galihjozu
The coordinates are based on a simple convention.  As reference point (0,0,0) there is the sellion (the bridge of the nose):

The sellion is the deepest point of the nasofrontal angle at the intersection of forehead slope and the proximal nasal bridge.

All the other points are estimated based on the distance from the sellion. The three coordinates represent the distance in millimeters from that point and are expressed as X-Y-Z coordinates, with X pointing forward, Y on the left and Z up. All these measures are average distances based on an average human head. The results can be improved if the distances are calibrated on a specific user's head.
Be careful, if you want to find out if the eyes are closed this method is not the proper one, because the point used here must be stable points, whereas the eyelid distance can change. The best thing you can do is to dynamically find the point on the eylids and estimate their distance instead of using an external point as reference. If you can calibrate the system on a single user then everything is easier and an external reference point may be used.

ok then ,, one more question, i red some articles and they only used 5 or 7 point of 3D coord, i tried to modified your code and used only 8 3D coord, around nose, around forehead, and around lips.
but the result kinda weird, the 3d lines suddenly move to center of lip, while i dont change this code
sellion_xy = (landmarks_2D[7][0], landmarks_2D[7][1])
cv2.line(frame, sellion_xy, tuple(imgpts[1].ravel()), (0,255,0), 3) #GREEN
cv2.line(frame, sellion_xy, tuple(imgpts[2].ravel()), (255,0,0), 3) #BLUE
cv2.line(frame, sellion_xy, tuple(imgpts[0].ravel()), (0,0,255), 3) #RED
do you have any idea what is going on?

Yes you can use just a few 3D points but you have to choose points located on planes that are as much as possible non-parallel in order to facilitate the PNP algorithm. For instance, you can use points on the tip of the nose, eyes and close to the ears.

i am trying to use this only,
P3D_MENTON = numpy.float32([0.0, 0.0, -122.7]) #8
P3D_NOSE = numpy.float32([21.1, 0.0, -48.0]) #30
P3D_RIGHT_EYE = numpy.float32([-20.0, -65.5,-5.0]) #36
P3D_LEFT_EYE = numpy.float32([-20.0, 65.5,-5.0]) #45
P3D_LIP_RIGHT = numpy.float32([-20.0, 65.5,-5.0]) #48
P3D_LIP_LEFT = numpy.float32([-20.0, 65.5,-5.0]) #54
and i dont change this
sellion_xy = (landmarks_2D[7][0], landmarks_2D[7][1])
cv2.line(frame, sellion_xy, tuple(imgpts[1].ravel()), (0,255,0), 3) #GREEN ke pinggir
cv2.line(frame, sellion_xy, tuple(imgpts[2].ravel()), (255,0,0), 3) #BLUE yg ke atas
cv2.line(frame, sellion_xy, tuple(imgpts[0].ravel()), (0,0,255), 3) #RED yang lurus
either this..
cv2.putText(frame, "x core =" + str(landmarks_2D[7][0]), ( 200, 20 ) ,font,0.5,(167,119,35),2)
cv2.putText(frame, "y core =" + str(landmarks_2D[7][1]), ( 200, 40 ) ,font,0.5,(167,119,35),2)
but i have an error
sellion_xy = (landmarks_2D[7][0], landmarks_2D[7][1])
IndexError: index 7 is out of bounds for axis 0 with size 6
my questions are

what is it for ((landmarks_2D[7][0], landmarks_2D[7][1]))
can i change the coordinate? which coordinate do i supposed to use?
i change it into [0][0], [0][1],

and i get this error
OverflowError: Python int too large to convert to C long
may i know why?
help me please.. i am trying to understand every line of your code

Hi @galihjozu
You have to be more specific in your report. Which code in the Deepgaze library are you using? Which kind of modifications to the original code did you implement?
The error that you get is connected with the shape of the object called  landmarks_2D[7][0]`. In particular this object does not have an element in the position that you are trying to access.


Folder selection is not working?

I can see the available folders, but when I click on the TS folder, it is not selected and does not appear on the 'selected file' line.
Using Chrome Version 90.0.4430.93 (Official Build) (x86_64)

refreshed and now it's working


Fix development shortcuts

Split repo into two parts:

sensorkit library
sensorkit-monitor application

This repo will have the library portion. Create new repo for application.

yacobucci/sensorkit is split between sensorkit and yacobucci/sensorkit-monitor.
sensorkit is a library being posted to pypi. sensorkit-monitor is the application portion.


Update dependencies

Hey, i would love if we could update databases to newest version as we are a couple bugfixes behind, let me know how can we get this merged.

Codecov Report

Merging #1197 (8607f4b) into master (242114e) will not change coverage.
Report is 31 commits behind head on master.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@            Coverage Diff            @@
##            master     #1197   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          201       201
  Lines        16647     16647
=========================================
  Hits         16647     16647


Reporting broken link

For encrypting the disks (OS drive and other managed disks) attached to your nodes, we leverage the Azure Disk Encryption. Azure Disk Encryption is a new capability that helps you encrypt your Windows virtual machine disks.

In the above, the https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-enable-azure-disk-encryption-windows behind encrypt your Windows virtual machine disks is broken, it points to https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-cluster-creation-via-portal

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

ID: 6202fb05-c2b0-b87f-71c9-5683e54f5e0c
Version Independent ID: 39505638-9926-fbc6-a795-3d7699859cde
Content: Create an Azure Service Fabric cluster from a template
Content Source: articles/service-fabric/service-fabric-cluster-creation-via-arm.md
Service: service-fabric
GitHub Login: @aljo-microsoft
Microsoft Alias: aljo

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

hmm odd. It seems that even typing in https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-enable-azure-disk-encryption-windows
points you to the same link..... Will figure it out and get it fixed .

@aljo-microsoft looks like this file referenced is no longer in the repository so it is just redirecting
https://github.com/MicrosoftDocs/azure-docs/tree/master/articles/service-fabric
Cc @masnider @dkkapur @srrengar

@aljo-microsoft can you fix this link as the author?

@srrengar I can fix it however I am not finding the correct link to actually change it to.
If you put in https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-enable-azure-disk-encryption-windows it just redirects to the basic creating a cluster doc.
I searched the SF github repo but no longer see any doc referring to enabling disk encryption. I am wondering if it might have been accidentally deleted
The same is true for the link that talks about encrypting linux machines in SF
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-enable-azure-disk-encryption-linux

not sure what happened to those articles. @aljo-microsoft can help provide proper links or suggest other changes

got it. Thanks @srrengar!
@aljo-microsoft feel free to ping me offline and we can get it sorted it out.

The encryption articles were removed.  I've removed the text and links from this article.  #please-close


Integration stopped working

Today the integration stopped working and wanted a re-setup.
But my credentials won't be accepted anymore. Authentication within Minimed app and on carelink.minimed.com is still working.
Maybe Medtronic changed something with anthentication ?

Carelink had maintenance today and after they send out an email to notify their clients the service is down. Right now nobody can login. It's best to wait for Carelink to come up again I guess.

Haha, thanks for the info. I got that email today but just flew over it.
Sorry for bothering.

Got an email, that Medtronic fixed the software issue. But I am still getting the invalid credentials error.

Yeap, same here:
[code]
Logger: custom_components.carelink
Source: custom_components/carelink/api.py:435
Integration: carelink
First occurred: 00:12:56 (433 occurrences)
Last logged: 09:43:59
Unexpected error fetching carelink data: 'auth_tmp_token'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 283, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/carelink/init.py", line 149, in _async_update_data
recent_data = await self.client.get_recent_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/carelink/api.py", line 473, in get_recent_data
if await self.__get_authorization_token() is not None:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/carelink/api.py", line 435, in __get_authorization_token
auth_token = self.async_client.cookies[CARELINK_AUTH_TOKEN_COOKIE_NAME]
~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/httpx/_models.py", line 1148, in getitem
raise KeyError(name)
KeyError: 'auth_tmp_token'[/]

My carelink account (which i also use in the regular app) got completely reset. I have to re-allow the carelink user to request access from the patient to monitor. During the outage something weird happened, so for now my account does not have access.
I also had to reconfirm terms and conditions when using the app.
So all these things might affect access using the Integration. If it works in the app, the integration should also work.

The app is connected with my carelink account and synchronizes well. It's only the integration which can not connect.

Updated to 2023.6.5. --> Working.
Thank you for the quick fix.

For me with 2023.6.5 and the develop branch i am not able to login, using an app working carelink account.
I de-installed the customer component completely and reconfigured from scratch. in all versions i am getting wrong username / password. :-(

@hhuitema Yeap, me too. App on phone works fine, integration in HA only complains for wrong username/pw.


Regression of #306 in 1.1.5-51  - bracketed paste characters

Description of the problem or steps to reproduce
Regression of #306 - bracketed paste characters output to terminal at exit.

Launch micro
Press Ctrl-Q

Specifications
Version: 1.1.5-51
Commit hash: ed6951a
Compiled on March 11, 2017
OS: Linux
Terminal: Gnome-terminal

I'm not experiencing this issue. I have tested on MacOS and Ubuntu with Gnome terminal. I think it probably has to do with the update to tcell for terminal titles which was reverted but not completely (also a new PR is on the way). I have now completely reverted the SetTitle PR (zyedidia/tcell#6) so it should be fixed.
Let me know if this fixes it. If you are compiling from source, make sure to update tcell.

Please reopen if it is not fixed.

Sorry, wilco.


Allow protected branch deletion

This PR adds a feature to allow the processing of protected branches via the new optional parameter include-protected-branches. This is useful if you have protected branches, which allow deletions. With this feature, also these branches can be maintained via this workflow. Currently, any protected branches are just ignored.
Excerpt from the updated readme:
If this is enabled, the action will include protected branches in the process.Note: When you use this, the token passed to repo-token must include "Administration" repository permissions (read). For details, see https://docs.github.com/en/rest/branches/branch-protection?apiVersion=2022-11-28
This can be achieved by creating a fine-grained access token with these permissions:

The additional permission "Administration" repository (read) is not available in the default GITHUB_TOKEN. It is just needed to call API github.rest.repos.getBranchProtection in order to identify and exclude branches, that don´t allow deletions from the process. See src/functions/check-branch-protection.ts
It would be nice if you accept this change and I´m open for a discussion.
Regards,
Michael

closing this PR as it was created from the wrong base branch


Power Types section of Wiki is empty

The page for Power Types is empty and cannot be clicked on. This makes it difficult for those making Data packs with the addon. Without proper documentation, there could be potential errors or powers that will continue to be unknown unless someone looks into the code itself.
If the Wiki is updated to include all power types, that would be good for data pack developers.

The wiki is being worked on now, and the new URL for it is: here

Conditions is also broken

Wiki is fixed


Tracking multiple keywords with Steam causes 401

Stream.filter(track=['hello']) # 1 word works!
Stream.filter(track=['hello', 'world']) # 2 words AND'd works!
Stream.filter(track=['hello world']) # 2 words OR'd 401 error
Stream.body is 'track=hello+world'.
Is the '+' breaking oAuth? I honestly can't think of any other reason for a space to cause a 401 error.

I ran into this too - the commit that @simonyun linked to on 25th Jan works for me.


build-rpm  error: License field must be present in package: (main package)

hi,
When compiling netopeer2-1.1.34, I used the following steps to compile the RPM package.
mkdir build; cd build
cmake ..
make build-rpm

and the error infomation as follow:
[root@localhost build]# make build-rpm
./
./CMakeLists.txt
./CMakeModules/
./CMakeModules/FindCMocka.cmake
./CMakeModules/FindLibNETCONF2.cmake
./CMakeModules/FindLibSSH.cmake
./CMakeModules/FindLibYANG.cmake
./CMakeModules/FindSysrepo.cmake
./CMakeModules/UseCompat.cmake
./CMakeModules/uninstall.cmake
./CODINGSTYLE.md
./LICENSE
./README.md
./cli/
./cli/CMakeLists.txt
./cli/README.md
./cli/certs/
./cli/certs/ca.pem
./cli/certs/client.crt
./cli/certs/client.key
./cli/cli_version.h.in
./cli/commands.c
./cli/commands.h
./cli/completion.c
./cli/completion.h
./cli/configuration.c
./cli/configuration.h
./cli/doc/
./cli/doc/netopeer2-cli.1
./cli/linenoise/
./cli/linenoise/LICENSE
./cli/linenoise/linenoise.c
./cli/linenoise/linenoise.h
./cli/main.c
./cli/sample_script.sh
./compat/
./compat/compat.c
./compat/compat.h.in
./example_configuration/
./example_configuration/ssh_callhome.xml
./example_configuration/tls_callhome.xml
./example_configuration/tls_certs/
./example_configuration/tls_certs/ca.key
./example_configuration/tls_certs/ca.pem
./example_configuration/tls_certs/client.crt
./example_configuration/tls_certs/client.key
./example_configuration/tls_certs/server.crt
./example_configuration/tls_certs/server.key
./example_configuration/tls_keystore.xml
./example_configuration/tls_listen.xml
./example_configuration/tls_truststore.xml
./modules/
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./packages/
./packages/CMakeLists.txt
./packages/create-package.sh
./packages/debian.compat
./packages/debian.control.in
./packages/debian.copyright
./packages/debian.netopeer2.install
./packages/debian.postinst.in
./packages/debian.rules.in
./packages/local-deb.sh.in
./packages/local-rpm.sh.in
./packages/netopeer2.dsc.in
./packages/netopeer2.spec.in
./packages/upload-package.sh
./scripts/
./scripts/merge_config.sh
./scripts/merge_hostkey.sh
./scripts/setup.sh
./src/
./src/common.c
./src/common.h
./src/config.h.in
./src/log.c
./src/log.h
./src/main.c
./src/netconf.c
./src/netconf.h
./src/netconf_acm.c
./src/netconf_acm.h
./src/netconf_monitoring.c
./src/netconf_monitoring.h
./src/netconf_nmda.c
./src/netconf_nmda.h
./src/netconf_server.c
./src/netconf_server.h
./src/netconf_server_ssh.c
./src/netconf_server_ssh.h
./src/netconf_server_tls.c
./src/netconf_server_tls.h
error: License field must be present in package: (main package)
Built target build-rpm

Hi,
unfortunately, we are building the packages differently and this script is not used. However, we are planning some overhaul so it should be improved in future. Or you can provide a patch, if you can fix it.
Regards,
Michal

Hi,
unfortunately, we are building the packages differently and this script is not used. However, we are planning some overhaul so it should be improved in future. Or you can provide a patch, if you can fix it.
Regards,
Michal

I also get the license error. I assume the overhaul has not happened yet. Could you copy and paste the actual build script?

There is no actual build script available. And there will be no overhaul, at least none is planned anymore. This is just some minor problem that anyone can solve. So I would appreciate if someone or you would take a look and fix it, I do not have much experience with packaging so I would have to spend some time learning the basics first. I will keep this issue open but I simply have more important work to do.


Help component

Integrate help-component as initial hints with 'First steps' to help understanding the app

What do you think about renaming the whole component to first steps -> , FirstSteps.js, FIRST_STEPS_CONTENT_URL, etc... ?

Sure

Thx for your hints!


code usage

Looks like you used some code from libvlcsharp. A mention/credits would be nice.

@mfkl Thanks for letting me know. Yes, I did adapt the logging from a dotnet github issue. Sorry about that. I will add a comment at the top of that file saying who wrote it/where it is from, is that okay?

Appreciate it, thanks!


select dom had duplicate "down arrow"

When I use unfold, I find that there are duplicate first arrows in the drop-down selection box, like the snapshot.
I add a same className to fix by myself.
<style>
  select:not([class*=bg-none]):not([multiple]) {
      -webkit-appearance: none;
      -moz-appearance: none;
      appearance: none;
  }
</style>

You admin class must inherit from unfold.admin.ModelAdmin


screen cannot be powered by immersive engineering wires

https://github.com/BluSunrize/ImmersiveEngineering/issues/173

Ummm... any resolution?

Yes, you need to power the 'screen controller'. Not the individual screens.


can delete text in readonly TextEdit

If you select the text (shift + home/end/arrow keys) you can delete the selected text from a read only TextEdit control.  It doesn't appear to be accepting any other keystrokes.  Backspace and delete do nothing (as expected) unless there is selected text.

should be fixed, please test

Assuming it has been fixed as per above comment.


Old maven compiler

https://github.com/Appdynamics/tomcat-monitoring-extension/blob/master/pom.xml#L69
Source option 5 is no longer supported. Use 7 or later

Use https://github.com/Appdynamics/jmx-monitoring-extension.


Cron jobs should use the app_configs for Solr URL

Currently, the Solr URL is specified on the command line in the cron config for regular incremental indexing.  But that means that it has to be updated separately when the ansible config is changed for the webapp.  We should use the webapp config (/home/deploy/app_configs/marc_libration) instead of hard-coding this in the cron config.

Either read from env variable, or add cron jobs to Ansible

currently these are
irb(main):001:0> ENV['SOLR_URL']
=> "http://lib-solr.princeton.edu:8983/solr/catalog"
irb(main):002:0> ENV['SOLR_REINDEX_URL']
=> "http://lib-solr.princeton.edu:8983/solr/catalog-reindex"

but they should be
irb(main):001:0> ENV['SOLR_URL']
=> "http://lib-solr.princeton.edu:8983/solr/catalog-production"
irb(main):002:0> ENV['SOLR_REINDEX_URL']
=> "http://lib-solr.princeton.edu:8983/solr/catalog-rebuild"

once crons are on whenever this should be removed from ansible:
https://github.com/pulibrary/princeton_ansible/blob/d7b3b44a1d712a4c14289d16e775e8f03d941224/roles/pulibrary.bibdata/tasks/main.yml#L18

That ark cache seeder job doesn't appear to have been run on staging or production since march 19th and march 17th, respectively. I think because it's not setting RAILS_ENV=production. That will be the first job I move to whenever, then we can deploy and check on it before proceeding with more jobs.


systrace doesn't include JS or native modules thread info

I'm using systrace via the process detailed in the documentation. In my trace results there is no useful JS thread or native modules thread info. This makes it more or less useless for profiling my app.
React native version:
System:
    OS: macOS 10.14.6
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
    Memory: 791.28 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 8.11.3 - ~/.nvm/versions/node/v8.11.3/bin/node
    Yarn: 1.17.3 - ~/.yarn/bin/yarn
    npm: 6.10.1 - ~/.nvm/versions/node/v8.11.3/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 12.4, macOS 10.14, tvOS 12.4, watchOS 5.3
  IDEs:
    Android Studio: 3.4 AI-183.54<IP_ADDRESS>52501
    Xcode: 10.3/10G8 - /usr/bin/xcodebuild
  npmPackages:
    @react-native-community/cli: ^2.9.0 => 2.9.0
    react: 16.9.0 => 16.9.0
    react-native: ^0.60.5 => 0.60.5
  npmGlobalPackages:
    create-react-native-app: 2.0.2
    react-native-macos-cli: 2.0.1

Steps To Reproduce

Take a systrace profile per the documentation
Open profile and note that the JS and native modules threads don't contain useful information like they do in the documentation example

Describe what you expected to happen:
JS and native modules threads appear in the systrace as per the documentation. See the following screenshots of my trace -- the JS and native modules information is missing. There is mqt_js and mqt_native_mod in the trace but they don't contain any meaningful data like the documentation example. This trace definitely contains some heavy JS work but it appears like it isn't recorded.

Snack, code example, screenshot, or link to a repository:
Example trace: https://www.dropbox.com/s/n0ig26bctwq359k/trace.html?dl=0

I updated to latest react native. Problem still persists, there is no useful information about the js or native modules threads like is shown in the documentation. A trace file can be found here: https://www.dropbox.com/s/n0ig26bctwq359k/trace.html?dl=0

The issue does conform to the template...?

Anything I can do here? The bot seems to have erroneously confined my issue to the dustbin.

This issue never got examined. I think it got ignored because the bot flagged it as needs more information even though I provided the info. Needs discussion.

I'm getting the exact same issue. I think this is related to the fact that FB has its own closed-source implementation of systrace that doesn't have these issues so it's not a priority (see https://github.com/react-native-community/discussions-and-proposals/issues/69#issuecomment-446115060).

Any updates here? This is a big issue on our end.

This is still an issue on latest RN 0.61.5. Symptoms are exactly the same.
System:
    OS: macOS 10.15.3
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
    Memory: 133.43 MB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 12.10.0 - ~/.nvm/versions/node/v12.10.0/bin/node
    Yarn: 1.19.2 - ~/.yarn/bin/yarn
    npm: 6.13.7 - ~/dev/TaskHero/node_modules/.bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
    Android SDK:
      Android NDK: 21.0.6113669
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
    Xcode: 11.3.1/11C504 - /usr/bin/xcodebuild
  npmPackages:
    @react-native-community/cli: ^4.1.0 => 4.1.0
    react: 16.9.0 => 16.9.0
    react-native: github:AndrewMorsillo/react-native#0.61-stable-th-custom => 0.61.5

For the JS information, are you using JSC or Hermes ? JSC profiling had broken a while ago, and now that we are supporting Hermes, we have that information in the Hermes sampling profiler, which is not yet integrated into Systrace.
The native modules have as much information as you have added trace markers in your code.
To solve this issue, i would recommend adding trace markers to JS and native modules to see how long the communication takes.

I am using JSC. Hermes is unfortunately not an option (at least for now) due to lack of Proxy support. My app relies heavily on mobx-state-tree which requires Proxy.
I don't quite understand what you mean by the native modules having as much information as I have added trace markers. None of the native code in my app was written by me, it's all react-native internal or npm modules. If you compare the screenshot in the issue above to the screenshot of the expected output from the docs you can see that the expected tracing information is missing from mqt_native_modules.
Can you provide a bit more detail about how I would go about adding the trace markers you're referring to? I apologize if that may seem like an obvious thing but I have little to no experience with native android code. My app has no custom native code, all native code is built in react-native or from npm modules.

can confirm this still happens with Hermes on react 0.62

This is still an issue 1 year later. As far as I can see there are no working profiling tools for react-native Android at the moment and haven't been for a couple years. This is really painful for my team, we have customers reporting performance issues on Android but have no way to diagnose and solve them other than trial and error. Only chrome profiler appears to work but as we all know that doesn't offer a picture of what's really happening, only a somewhat relative measurement of perf against itself using v8 with debug mode caveats. @axe-fb can you offer any guidance about the state of performance profiling for RN Android?

can confirm this still happens on 0.63.3 – with or without Hermes.

Issue also happens on V8.

JS thread still not recorded:

Would it be possible to realase the facebook's internal systrace tool, which is capable of showing the data?

unstale

This is still an issue.


OSM ARM 64 support

Please describe the Improvement and/or Feature Request

Ensure that OSM CLI and control plane components work on ARM64 environments. Also tests, demos, and release pipelines need to be configured so that they can run both on AMD and ARM.
OSM install & envoy: https://github.com/openservicemesh/osm/issues/4702
PR to parameterize nodeselector, affinity, tolerations: https://github.com/openservicemesh/osm/pull/4706
Instructions on setting up OSM to run on ARM64: https://github.com/flomesh-io/osm/wiki/How-to-setup-osm-to-run-on-arm64-platform
Scope (please mark with X where applicable)

New Functionality          [ ]
Install                    [X]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [ ]
Egress                     [ ]
Envoy Control Plane        [ ]
CLI Tool                   [ ]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [X]
CI System                  [X]
Demo                       [X]
Project Release            [ ]

Possible use cases

Team, I would like to see the enablement of testing on Arm64. If you need any help I am happy to contribute.

Putting this issue on hold until Contour releases support v1.23 of envoy-distroless. Older versions of envoy-distroless do not have an ARM64 image, so sidecar injection will fail on ARM machines. Once contour releases support, we can upgrade the envoy version both for OSM and Contour.

Update, contour has released support for v1.23 of envoy. https://github.com/openservicemesh/osm/pull/4954. However, the ARM64 image for envoy-distroless v1.23.0 seems to have the amd binary (https://github.com/envoyproxy/envoy/issues/22261), so sidecar injection is still failing when testing with the envoy-distroless v1.23.0 image, will likely need to wait for v1.23.1.

Looks like we also need to update some of the test images that we use for client and server pods: for instance https://github.com/openservicemesh/osm/blob/main/tests/e2e/e2e_permissive_smi_switching_test.go#L65

Also need to update manifest yamls for manual demo: https://raw.githubusercontent.com/openservicemesh/osm-docs/release-v1.2/manifests/apps/bookbuyer.yaml

Looks like we also need to update some of the test images that we use for client and server pods: for instance https://github.com/openservicemesh/osm/blob/main/tests/e2e/e2e_permissive_smi_switching_test.go#L65

As per your comment here the images which need to be updated are kennethreitz/httpbin and songrgg/alpine-debug. I have looked into it and found that the last update of both the images in dockerhub and GitHub is around 4 and 6 years respectively.
Can we replace them with some other images as they may not respond to release arm64 images?
If you want to continue with those images, I can help in adding the required changes for arm64 and raise the PR in the respective repositories.

@nshankar13 Can you please give your suggestion on the above comment.


Add HallowedSack Tables

Add the loot tables for the Hallowed Sack so it can be bought to osb and bso

[] I have tested all my changes thoroughly.

I don't know how to resolve this problem so going to close the PR

Reopened following the PR for the Hiscores to be updated

Reopened following the PR for the Hiscores to be updated


src/test/java is not found?

I import the oryx project into the eclipse.But it has some wrong.It is src/test/java is missing.I do not know why you delete it?The maven may be requst the src/test/java.How can I do?

I don't know what you mean? no directories have been deleted. There are src/test/java dirs where they should be, and the Travis builds pass: https://travis-ci.org/OryxProject/oryx

It is the oryx/app/example/src/main/.This dir has not /test dir.The eclipse flag as a error.

There are not supposed to be tests in the example module. This is entirely fine. It's an Eclipse problem.

Oh,It's an Eclipse problem.Please tell me what IDE do you use?


Added Brett Bergquist (bbergquist) as a committer for the project

Description

Added myself "Brett Bergquist" (LDAP ID "bbergquist") as a maintainer of the IvyTrigger  plugin.
This plugin is up for adoption and I contacted the Jenkins developer list.   This plugin was being maintained by @gboiss.   I have also attempted to contact Gregory Boissinot directly via email.
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Context: https://groups.google.com/d/msg/jenkinsci-dev/A0_zqJo2oAI/AbpnhlINCwAJ

FYI @gboissinot

@bbergquist0930 Please let me know if you still need commit access to the GitHub repo.


Auto-Reload for SecretsManager & ParameterStore property sources

Auto-reloading SecretsManager & ParameterStore property source inspired by Spring Cloud Kubernetes implementation.
Turned off by default.
TODO: tests for autoconfigurations, update samples & reference docs.

@MatejNedic docs are pending but other than that ready for review. Depending on how goes with https://github.com/spring-cloud/spring-cloud-commons/issues/1156 maybe there will be a space for improvements in the future.

@MatejNedic reference docs added.

Fixes https://github.com/awspring/spring-cloud-aws/issues/377

@MatejNedic thanks for a review. Changes applied.