Integration with Ollama

Hello!
I'm trying to use SalesGPT with a locally-served model via Ollama.
I tested the LiteLLM part, it works.
from litellm import completion

response = completion(
    model="orca-mini",
    messages=[{ "content": "respond in 20 words. who are you?","role": "user"}],
    api_base="http://localhost:11434",
    custom_llm_provider="ollama"
)
print(response)

Now, I modified the streaming_generator_example.py like this
llm = ChatLiteLLM(model="orca-mini",
    api_base="http://localhost:11434",
    custom_llm_provider="ollama")
....
generator = sales_agent.step(
     return_streaming_generator=True, model_name="orca-mini",
)

Btw, I think it's a little bit redundant the need to provide the model name at each step instead of setting it up in the llm model. Why is that needed at each step?
The result is :
salesgpt.logger 2023-09-14 07:10:55,377 - INFO - Running from_llm: --- 8.654594421386719e-05 seconds ---
salesgpt.logger 2023-09-14 07:10:55,378 - INFO - Running from_llm: --- 0.00012731552124023438 seconds ---
salesgpt.logger 2023-09-14 07:10:55,378 - INFO - Running from_llm: --- 0.0005018711090087891 seconds ---
salesgpt.logger 2023-09-14 07:10:55,378 - INFO - Running seed_agent: --- 5.7220458984375e-06 seconds ---
salesgpt.logger 2023-09-14 07:10:55,378 - INFO - Running _prep_messages: --- 0.0001068115234375 seconds ---
Traceback (most recent call last):
  File "/opt/ollama/SalesGPT/examples/streaming_generator_example.py", line 36, in <module>
    generator = sales_agent.step(
  File "/root/miniconda3/envs/sls/lib/python3.10/site-packages/salesgpt/logger.py", line 34, in wrapper
    result = func(*args, **kwargs)  # Function execution
  File "/root/miniconda3/envs/sls/lib/python3.10/site-packages/salesgpt/agents.py", line 110, in step
    return self._streaming_generator(model_name=model_name)
  File "/root/miniconda3/envs/sls/lib/python3.10/site-packages/salesgpt/logger.py", line 34, in wrapper
    result = func(*args, **kwargs)  # Function execution
  File "/root/miniconda3/envs/sls/lib/python3.10/site-packages/salesgpt/agents.py", line 179, in _streaming_generator
    return self.sales_conversation_utterance_chain.llm.completion_with_retry(
  File "/root/miniconda3/envs/sls/lib/python3.10/site-packages/langchain/chat_models/litellm.py", line 263, in completion_with_retry
    return _completion_with_retry(**kwargs)

... (long trace not included)

File "/root/miniconda3/envs/sls/lib/python3.10/site-packages/litellm/utils.py", line 998, in get_llm_provider
    raise ValueError(f"LLM Provider NOT provided. Pass in the LLM provider you are trying to call. E.g. For 'Huggingface' inference endpoints pass in `completion(model='huggingface/{model}',..)` Learn more: https://docs.litellm.ai/docs/providers")
ValueError: LLM Provider NOT provided. Pass in the LLM provider you are trying to call. E.g. For 'Huggingface' inference endpoints pass in `completion(model='huggingface/orca-mini',..)` Learn more: https://docs.litellm.ai/docs/providers

From my limited understanding, I get it that somehow the parameters set up in the llm at the initialization step are lost at the step stage, i.e. api_base and custom_llm_provider. Which kinda negates the whole purpose of integrating LiteLLM, if the only one I can use is OpenAI.
What am I doing wrong? Has anyone successfully ran local ollama?

@dantodor I'm the maintainer of LiteLLM, it looks like the normal LiteLLM call worked for you but the ChatLiteLLM call did not work ?

Can we verify if a regular ChatLiteLLM call works for you:
Can you try it like this ?
llm = ChatLiteLLM(
model="ollama/orca-mini",
api_base="http://localhost:11434",
)

@ishaan-jaff
Sorry for the delay.
I modified the test, it behaves almost the same, the difference being that the error is being thrown in litellm:
salesgpt.logger 2023-09-26 11:00:45,599 - INFO - Running from_llm: --- 0.00010800361633300781 seconds ---
salesgpt.logger 2023-09-26 11:00:45,599 - INFO - Running from_llm: --- 5.459785461425781e-05 seconds ---
salesgpt.logger 2023-09-26 11:00:45,599 - INFO - Running from_llm: --- 0.0003409385681152344 seconds ---
salesgpt.logger 2023-09-26 11:00:45,599 - INFO - Running seed_agent: --- 3.814697265625e-06 seconds ---
salesgpt.logger 2023-09-26 11:00:45,599 - INFO - Running _prep_messages: --- 4.982948303222656e-05 seconds ---
salesgpt.logger 2023-09-26 11:00:45,600 - INFO - Running _streaming_generator: --- 0.0006957054138183594 seconds ---
salesgpt.logger 2023-09-26 11:00:45,600 - INFO - Running step: --- 0.0007891654968261719 seconds ---
Traceback (most recent call last):
  File "/Users/dantodor/work/ab/SalesGPT/examples/ollama.py", line 42, in <module>
    for chunk in generator:
  File "/Users/dantodor/miniforge3/envs/sales/lib/python3.10/site-packages/litellm/llms/ollama.py", line 23, in get_ollama_response_stream
    with session.post(url, json=data, stream=True) as resp:

...
File "/Users/dantodor/miniforge3/envs/sales/lib/python3.10/site-packages/requests/models.py", line 439, in prepare_url
    raise MissingSchema(
requests.exceptions.MissingSchema: Invalid URL 'None/api/generate': No scheme supplied. Perhaps you meant https://None/api/generate?

Which loops back to my initial comment. Somehow, the initial setup of the LLM via litellm is ignored in the call to step().
Did you tried it by yourself and it somehow worked?

Okay, this is a little weird.
I was looking in the tests. So I noticed these lines:
sales_agent.determine_conversation_stage()  # optional for demonstration, built into the prompt

So, in the streaming demo, I added exactly that line following the sales_agent.seed_agent() line.
Now the results are:
salesgpt.logger 2023-09-26 11:39:13,446 - INFO - Running from_llm: --- 0.00013375282287597656 seconds ---
salesgpt.logger 2023-09-26 11:39:13,446 - INFO - Running from_llm: --- 6.794929504394531e-05 seconds ---
salesgpt.logger 2023-09-26 11:39:13,446 - INFO - Running from_llm: --- 0.0004940032958984375 seconds ---
salesgpt.logger 2023-09-26 11:39:13,446 - INFO - Running seed_agent: --- 5.9604644775390625e-06 seconds ---
Conversation Stage ID:  Based on the given conversation history, the next immediate conversation stage for the agent in the sales conversation would be qualification.
Conversation Stage: 1
salesgpt.logger 2023-09-26 11:39:22,194 - INFO - Running determine_conversation_stage: --- 8.747308015823364 seconds ---
salesgpt.logger 2023-09-26 11:39:22,194 - INFO - Running _prep_messages: --- 0.00023865699768066406 seconds ---
salesgpt.logger 2023-09-26 11:39:22,196 - INFO - Running _streaming_generator: --- 0.0018270015716552734 seconds ---
salesgpt.logger 2023-09-26 11:39:22,196 - INFO - Running step: --- 0.0020742416381835938 seconds ---
{'choices': [{'delta': {'role': 'assistant', 'content': ' Hey'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ','}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' how'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' are'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' you'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' doing'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' this'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' morning'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': '?'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' I'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' am'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' calling'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' from'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' Sleep'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' Haven'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': '.'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' Can'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' you'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' please'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' explain'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' to'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' me'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' why'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' you'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' need'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' home'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': ' insurance'}}]}
{'choices': [{'delta': {'role': 'assistant', 'content': '?'}}]}

I'm not commenting on the wrong utterance from the model, as I'm using orca-mini for testing, and I'm not expecting it to work. However, what I can infer from the above is that determine_converation_stage is not exactly optional?

Hey @dantodor were you able to get this to work?

@krrishdholakia as you can see above, it works, under a prescribed order of doing the calls.

cc: @filip-michalsky why do they have to pass model name multiple times?

@krrishdholakia @dantodor @ishaan-jaff thanks for raising this. Looks like SalesGPT needs to get some upgrades after lite-llm / langchain recent version bumps. looking into it and will revert, thx!

@dantodor please check the new version works for your ollama model?

Will do and let you know ASAP @filip-michalsky

Did you get it to work?


How to run python example on GPU?

I followed the Python example in Colab notebook(installed mediapipe by pip), but found the GPU seemed like not being occupied.  Is there a specific mediapipe version or installation option to use GPU device?

So I just checked on this - currently there's no support for GPUs in most cases, though if you're using Linux then you can build the Python wheels from source.


Handle TimeoutError gracefully with ApiExceptions

Handle TimeoutError gracefully with ApiExceptions. Ignore device manager errors in the pubsub callback, rather than just checking if its done and propagating them.
Downstream issue: https://github.com/home-assistant/core/issues/85889

Codecov Report

Merging #513 (f43dfc5) into main (9d20039) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@           Coverage Diff           @@
##             main     #513   +/-   ##
=======================================
  Coverage   92.33%   92.34%
=======================================
  Files          18       18
  Lines        2061     2063    +2
=======================================
+ Hits         1903     1905    +2
  Misses        158      158

Impacted Files
Coverage Δ

google_nest_sdm/google_nest_subscriber.py
64.41% <50.00%> (-0.38%)
:arrow_down:

google_nest_sdm/auth.py
96.15% <90.00%> (+1.05%)
:arrow_up:

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


Searchable dropdowns not working in Firefox

So I'll describe my situation as the issue doesn't appear when I use the examples sites in firefox.
Here is a video of what I'm seeing: http://recordit.co/HYkgk5WACL.
Essentially I am clicking on the dropdown and it's not triggering (but only sometimes, sometimes it does work, usually on a fresh reload). If I search and click on an item, then it closes without selecting the item, if I search and click enter, then it will select.
<sui-select         class="selection"         [isSearchable]="true"         [placeholder]="'APPLY.SELECT_LIST_PLACEHOLDER' | translate"         [options]="options"         labelField="value"         #select       >         <sui-select-option *ngFor="let option of select.filteredOptions"                            [value]="option">         </sui-select-option>       </sui-select>
That is my html, options is an array of objects that it passed down from a parent component and set during ngOnInit().
I do have changeDetection.OnPush set (I saw another issue involving it so I thought I'd mention), but even with removing it I see the issue. I have 0 issues using chrome.

Hi, I can't reproduce this in the demo app, please could you create a reproduction in plunkr so I can test it out myself? Thanks!


[hotfix-#1568][sqlserver] if write column is not have identity column, not execute SET IDENTITY_INSERT %s ON

Purpose of this pull request

if write column is not have identity column, not execute SET IDENTITY_INSERT %s ON
Which issue you fix
Fixes #1568 .
Checklist:

[ ] I have executed the 'mvn spotless:apply' command to format my code.
[ ] I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have checked my code and corrected any misspellings.
[ ] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

cc @FlechazoW


AWS job store bind bucket can fail with 409 error

Thanks to @joelarmstrong for reporting this issue.
DEBUG:toil.batchSystems.mesos.executor:Invoking command: '_toil_worker aws:us-west-2:cactus-jcarmstr-hmrd-mr-pp2 28e435d0-df43-49db-9420-a83fb9491038'
INFO:toil.common:Created the workflow directory at /var/lib/mesos/slaves/f15072d4-b9e6-44df-a669-66e986da45f7-S214/frameworks/f15072d4-b9e6-44df-a669-66e986da45f7-0022/executors/toil-6073/runs/f22c0f68-fb72-4f42-b70e-7c525e347c47/toil-43764132-2f0d-41c5-9984-a2f78d9eb7e2
Traceback (most recent call last):
  File "/usr/local/bin/_toil_worker", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/dist-packages/toil/worker.py", line 103, in main
    jobStore = Toil.resumeJobStore(jobStoreLocator)
  File "/usr/local/lib/python2.7/dist-packages/toil/common.py", line 766, in resumeJobStore
    jobStore.resume()
  File "/usr/local/lib/python2.7/dist-packages/toil/jobStores/aws/jobStore.py", line 222, in resume
    self._bind(create=False)
  File "/usr/local/lib/python2.7/dist-packages/toil/jobStores/aws/jobStore.py", line 241, in _bind
    versioning=True)
  File "/usr/local/lib/python2.7/dist-packages/toil/jobStores/aws/jobStore.py", line 738, in _bindBucket
    bucket.configure_versioning(True)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 1306, in configure_versioning
    response.status, response.reason, body)
boto.exception.S3ResponseError: S3ResponseError: 409 Conflict
<?xml version="1.0" encoding="UTF-8"?>
<Error>OperationAborted<Message>A conflicting conditional operation is currently in progress against this resource. Please try again.</Message><RequestId>ABFB7931AAFEB086</RequestId><HostId>O6ur1eYDtMByINPox7L4BsPXAdSfEAOE6Fr0Pu207bBnhsMQoXDU2mxkH8IE4y0kzHZze/Bt3aE=</HostId></Error>
DEBUG:toil.batchSystems.mesos.executor:Sending task status update ...

2 potential solutions to this:

exponential back-off to make sure the configure_versioning call will eventually succeed
since this call is in the worker (where the job store has already been created) we should be able to just skip this call.
Unless I'm missing something 2 is the way to go since the call doesn't seem necessary


Update Adfly feature

Fix Adfly decryptor

#38

Thanks. Could you fix the indentation? It should be tabs everywhere


Fix app.json validation

I rule and have completed some work on Case Manager that's ready for review!
One character change to app.json so that the JSON is valid which apparently Heroku is quite picky about.
https://dashboard.heroku.com/new?template=https%3A%2F%2Fgithub.com%2Fcolinmcglynn%2Fdcaf_case_management%2Ftree%2FFix-app.json-validation
The create a new app dialog has the env var input restored

It relates to the following issue #s:

Fixes #1327

cc @cfinucane


cmd/test2json: support emitting JSON directly from the test binary

It is now possible to emit JSON from a test binary (#2981, #22789), but this still requires the go tool:
go tool test2json ./foo.test -test.v

This doesn't work in -c case which is really useful when cross-platform running tests, i.e. we build the tests binaries on one platform but run them on a different one as plain binaries without having to use the go tool. Would it be possible to support that scenario as well, e.g. as -test.json file flag?

I believe you can capture the test output on one machine and run it through test2json on a different machine.

It's certainly doable but it makes the CI pipeline more complicated because it requires additional post-processing.

We considered whether to do this originally, but it's very difficult to do completely. The test binary cannot reliably ensure that all its output goes into test2json. Maybe it crashes during init before test2json even starts up. Maybe it manages to print from a func init before replacing os.Stdout/os.Stderr with pipes into test2json. The binary also cannot reliably ensure that all output is processed by test2json. Maybe it crashes, in which case it could exit before test2json, so that the parent might not observe the final output processed by test2json. The output processing problem could be fixed by making the test binary exec test2json and have test2json then reexec the test binary as a subprocess, but then the test binary starts up twice, which may break other things.
Note that you don't need the whole go command for your CI system. You can do a cross-platform build of cmd/test2json once, install it onto your target system, and then run
path/to/test2json ./foo.test -test.v

That should be much more robust than anything we could do in the test binary itself. Please use that.


[stable/postgresql] Allow configmaps for configuration and initdb

What this PR does / why we need it:
The options that we provide for adding custom scripts and configurations are not the best to be used by a parent chart:

Option 1: Files. This forces them to clone our postgresql chart, have it in their charts/ folder, and there put the files they want to use
Option 2: Putting the content in values.yaml. That can be easily done in the parent chart values.yaml. However, in values.yaml we cannot use any of the template capabilities that helm offers, and it is highly likely that some of the settings that you put in postgresql.conf or pg_hba.conf may depend on a runtime value, such as the service name. That same issue happens with option 1.

Hence, the best option would be to allow using an external configmap with the settings. That would be created by the parent chart (with full template possibilities) and then referenced in the child chart.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

This would fix this old issue reported in the bitnami/charts repo: https://github.com/bitnami/charts/issues/738

/lgtm


Update azure-dev.yml

Updating azd install action to latest
Using env refresh before deploy for fetching environment
related to: https://github.com/dotnet/docs-aspire/issues/1117

cc @ievangelist - maybe we augment the docs with some commentary about this sort of workaround? I'll work up a sample repo with this approach dialed-in that works end-to-end so we have something to go off of.


chore(node): add file generated during test

I'd like to understand why this file is getting created, but I don't.


Handling discrepancies caused by BigInt.isValidInt

Motivation for proposing this PR
I used this library when compiling my project for web, but I found that the internal processing was a bit different for web and native (caused by BigInt.isValidInt). In addition, passing a custom configuration in each call to decodeJson was a bit repetitive, so I wanted to define a global class to configure it.
Numbers in Dart

What did this pr do?

Remove unnecessary dependencies collect (this also reduces function call costs)
Add a class called JSONBigIntConfig for global configuration of encoding and decoding settings
Change useIntWhenPossible of DecoderSettings class to whetherUseInt to provide more fine-grained control (This is mainly to avoid errors caused by the difference between BigInt.isValidInt on native and web)

Thanks for the PR @lopo12123. It all looks good to me.
It's been a while since I've worked with Dart so I just want to confirm that, since the useIntWhenPossible param was removed from the DecoderSettings class, the major version of the package should be bumped (due to a backwards incompatible api change).
If so, I'll bump the package version to 3.0.0 and merge the PR.

I added some comments and updated the example and README, and also updated the version number to 3.0.0. Please check if there is any problem.


Custom layout for phone authentication

How can i set my own custom layout for phone authentication work flow?

 One where you take the phone number of the user.
Have a button which upon clicking does this:
startActivityForResult(         AuthUI.getInstance()                 .createSignInIntentBuilder()                 .setAvailableProviders(Arrays.asList(                       new IdpConfig.PhoneBuilder()         .setDefaultNumber("Entered phone number with ISD code")         .build()  ))                 .build(),         RC_SIGN_IN);
Or maybe do this in a new activity passing the number between them. Then the user can proceed with verifying the number. NOTE: This might or might not work for your work case.

@mostasim we only support custom layout for the very first screen.  After that you can provide themes but you can't change the layout.  FirebaseUI is meant to be a drop-in UI so customizing the whole thing somewhat defeats the point.
@100rabhkr thanks for sharing!  That will definitely work for some cases.


private_dns folder should be renamed to hostnames

Incorrent naming of private_dns folder. Can be misleading.

Fix for #15
Changed SSH folder name to be case-sensitive
Renamed private_dns folder to hostnames everywhere
README updates

This can be closed

Yup! gonna check out if everything works fine in the branch and then will merge it back into the master.


Add releases notes for 1.12.3-cs4

Deploy preview processing.
Built with commit 5543625ba4f3bd9e3c968122b2ca6a1a77b2a3f8
https://app.netlify.com/sites/docker-docs/deploys/58261396d6865d2258eef369

Release is still in progress.

Release is finished.


Json refactorings

These are two commits:

the first removes the remaining usage of Yojson.Basic.Utils.member
it also passes the json_dict directly around from extract_csrf_token, leading to fewer indentation

The second commit uses Yojson.Basic.to_string instead of Yojson.Safe.to_string. I don't know exactly why we should use which - but I counted the usages, and there were more Basic usages.
For #66

Best reviewed with the ?w=1 added https://github.com/robur-coop/mollymawk/pull/82/files?w=1 -- so that whitespace difference aren't shown.

From a diff -u basic.ml safe.ml, I learn that Yojson.Safe.t has additional constructors:
 type t =
     | `Null
     | `Bool of bool
     | `Int of int
+    | `Intlit of string
     | `Float of float
     | `String of string
     | `Assoc of (string * t) list
     | `List of t list
+    | `Tuple of t list
+    | `Variant of (string * t option)
     ]


Preview for live content

Motivation

At the moment, the aggregated content of a page including live/mirrored content can not be previewed.
Proposed Solution

Add a possibility to preview the live/mirrored content at its specified position in combination with the original content
Alternatives

Already implement #496 if feasible for the feature completion milestone
Additional Context

This has a higher priority than the improved general preview (issue #496), because the wysiwyg editor already shows a rough preview of the normal content field.

I'm not sure if I got this right, so lets say I got page X and I set page Y as live-content after page X. When I now click the preview button ("eye icon"), then there supposed to be page X followed by page Y in the preview. Is this right?

I'm not sure if I got this right, so lets say I got page X and I set page Y as live-content after page X. When I now click the preview button ("eye icon"), then there supposed to be page X followed by page Y in the preview. Is this right?

I'm not sure if I got this right, so lets say I got page X and I set page Y as live-content after page X. When I now click the preview button ("eye icon"), then there supposed to be page X followed by page Y in the preview. Is this right?

yes, correct.

I'm not sure if I got this right, so lets say I got page X and I set page Y as live-content after page X. When I now click the preview button ("eye icon"), then there supposed to be page X followed by page Y in the preview. Is this right?

yes, correct.


Blurred, overlapping text when generating a PDF

Hi folks,
Any ideas what might be causing the issues in this screenshot?

I'm running this in an AWS Lambda. I've also tried running it locally, I get the same results. I'm using the basic config:
const fs = require('fs');
const conversion = require('phantom-html-to-pdf')();

const report = (content) => {
  return new Promise((resolve, reject) => {
    conversion({
      html: content,
      waitForJS: true,
    }, (err, pdf) => {
      if (err) {
        reject(err);
      }
      const output = fs.createWriteStream('report.pdf');
      pdf.stream.pipe(output);
      resolve(true);
    });
  });
};

module.exports = {
  report,
};

This was totally my mistake. Please ignore (dodgy html tables).


Reload Twice when start up at the first time.

I always noted that when I run npm start it will start the project then in the first time it reload twice. Do anyone have any idea or that problem? thanks.

I experience this same issue.  It does seem very odd.

This repository is now deprecated. The Angular Quickstart project was a nice starting point for creating Angular applications. Now we recommend using the Angular CLI to create new Angular projects.
Starting from 1 November 2017, all the Angular documentation, at angular.io, is based on the Angular CLI.
Let's get started


Consider changing the input of TinyLexer

Currently, TinyLexer takes a string slice code: &str as input. While this makes it easy to work with in the lexer, it's not actually necessary, since the returned tokens do not reference into the original input (they are returned as SmolStr).
If TinyLexer instead took some form of Read, it could then operate directly on e.g. a File handle without keeping the entire file in memory. Seek might also be helpful, although I think it's better if I try and avoid that.
Basically I want to replace code: &str in TinyLexer with a generic T: Read.
PositionedLexer can't really benefit from this, since tokens reference into the original string.

Thinking about it more and more, it's probably not worth the effort. The whole scanning char by char to find e.g. idents would be really slow with an unbuffered input. And even if you do pass in a BufReader it would still be better to just have the full thing in memory consecutively.


move ilastik up in the plugins menu

@emilmelnikov mentioned that it's because we're lower-case with our plugin name.

I think the issue is not only lower-case vs. upper-case. In addition, SciJava Commands have a default menu weight of POSITIVE_INFINITY.
See also https://github.com/imagej/imagej-legacy/issues/179.
Instead of:
https://github.com/ilastik/ilastik4ij/blob/d3e9ec6c25d329c5357f93cd343adbf074cdfafa/src/main/java/org/ilastik/ilastik4ij/ui/IlastikImportCommand.java#L29
you can use something like:
@Plugin(type = Command.class, headless = true, menu = {
	@Menu(label = MenuConstants.PLUGINS_LABEL, MenuConstants.PLUGINS_WEIGHT, mnemonic = MenuConstants.PLUGINS_MNEMONIC),
	@Menu(label = "ilastik", weight = YOUR_ILASTIK_MENU_WEIGHT_CONSTANT),
	@Menu(label = "Import", weight = YOUR_DESIRED_POSITION_IN_SUBMENU)})

See https://github.com/imagej/op-finder/blob/e69c0613395b6398085fad09e1e227e10b1a4f8c/src/main/java/net/imagej/ui/swing/ops/FindOps.java#L38-L42 for an example.

Is it a good ecosystem behavior to change plugin priority like that? It's not like our plugin is more important than other plugins installed in a system...

Hey @imagejan,
thank you very much for the insight! Really cool that you keep an eye on this repo :+1:
As for the menu weight I'd also be concerned on what the etiquette on these things is?


HTML

HTML
As a bootcamp student
I want to have the prework notes structured on a webpage
for be more easy to studied
Acceptance criteria
GIVEN a Prework studie guide webpage
WHEN I visit the website in my browser
Then I see Four boxes titled HTML, CSS, GIT and javascript with associated notes listed

ready


Improve Input Field Headings Color on Login and Sign-Up Pages

Describe the bug
The input field headings on the login and sign-up pages have a dull color, making them difficult to read due to low contrast with the background. This affects readability and impacts the user experience.
To Reproduce
Steps to reproduce the behavior:

Go to the 'Login' or 'Sign-Up' page on the website.
Locate the input fields (e.g., Username, Password).
Observe the color of the headings above each input field.

Expected behavior
The input field headings should have a color that provides better contrast, aligning with the website’s color palette, to ensure clear readability and an improved visual experience for users.
Screenshots

Additional context
Updating the color of these headings to match the website’s existing color palette can significantly enhance readability and maintain a cohesive design. Please assign this issue to me.

@ANSHIKA-26 please assign this issue to me so that I can start working on it

not needed , we like the UI of login and singup including its colors !

Alright, @ANSHIKA-26 I there anything you would like me to do to enhance WordWise specific to my domain?


Promote Node.js compat

Summary

I'm still learning about these products so please feel free to edit what I wrote if it doesn't make sense or isn't addressing the request.

Update: Following up with assignees on approving this one via Chat.

@irvinebroque I updated this PR per your comment. Please review and approve when you have a chance. Thanks!


AMBARI-24147. Make STOMP updates immutable.

What changes were proposed in this pull request?
STOMP update events made immutable, this prevents STOMP cache corrupting during simultaneous retrieving current values and update handling. Also cache locks were optimized for for host data holders.
How was this patch tested?
Manual testing.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2791/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2792/
Test FAILed.
Test FAILured.

retest this please

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2808/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2813/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2828/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2829/
Test PASSed.


Add types for Export

Currently, Plasma seems to be typing almost all their functions as any. This PR changes this so that all the calls to Plasma have full types ranging from hooks to UI to help users ease into using Plasma.

Many of the types that are exported in this PR are not public types. Why is there a separate Lua file just for types? It doesn't seem like a good idea to separate types from the place where they are used. I also don't really like having to update function signatures in two places.

The main issue currently is that Runtime cannot be typed unless it's to be split into different modules, as the main issue is circular dependencies (Runtime requires widgets.error inside a function) preventing it from being typed and instead making it typed as any

Just noticed, moving error into Runtime also has its own issues as create depends on Runtime too, which we need in order to create the error


Plugin don't work in Windows 11

Plugin don't work in Windows 11 (64 bit Home) in nightly version QuickLook QuickLook-3.6.11-24-g69d6aa0.
Crashing on TTF and OTF font files.

Plugin don't work in Windows 11 (64 bit Home) in QuickLook QuickLook-3.7.
Crashing on TTF and OTF font files.

This problem isn't limited to Windows 11. I'm still running Windows 10 and experience the same problem.

Plugin is incompatible with QuickLook 3.7.1. Crashing on TTF and OTF font files.
Please fix or delete plugin.

@Zbynius I think will just go ahead and delete it because if it doesn't work for you, no one else should have access to it right?

@jeremyhart I also have the same problem in Windows 11 Insider Beta channel. Your plugin is very useful for me, please could you update it and fix it?


Deprecate mode argument to asdf.open

As far as I can tell, this argument does not serve any useful purpose, but only allows for buggy behavior related to memory maps (see the discussion in #575 and #576).
For these reasons, I propose to remove it entirely. For the next few releases, using this argument will result in a warning. By 3.0, it should be removed entirely.
cc @vmarkovtsev

+1 on this. Looks like it should always remain 'r' unless copy_arrays is False and the source is a file name or a "real" file descriptor.

I'm not sure if copy_arrays should have any bearing on whether a file is writeable or not. There's no reason we can't update arrays and write them back out, even if they're not memory mapped.
Also I'm not sure why it shouldn't be possible to write back to an io.BytesIO. So maybe we should just open in rw by default unless the source is clearly an http address?
Maybe instead of deprecating the mode argument, we just need to enforce the following behavior: if mode='r', then copy_arrays=True. Otherwise we will raise an exception at open. And we can change the default to mode='rw' to be compatible with copy_arrays=False. We need to keep both arguments since the implication only goes in one direction: it's not the case that copy_arrays=True necessarily implies mode='r'.

Makes much sense. The only situation when the code breaks is when the file is open readonly, copy_arrays is False and we try to change an array. We cannot ask users to not change arrays of course.

Actually it might actually be better to force copy_arrays=True if mode='r', in order to avoid breaking current usage, but that's going to require some reworking of the internals.
This is just a turning into a giant pain, and I've dealt with this kind of thing in other libraries. This is the price we pay for having a generalized open function, though.

We can always detect the mode of the file object and print a runtime warning where we set copy_arrays to True. I can work on that.

You'll notice though that copy_arrays is used to create the block index before the other arguments to open are processed, though. I'm thinking of a bit more substantial overhaul where we make open a standalone factory method rather than a class method. In this case, use of AsdfFile.open would be deprecated in favor of asdf.open.


Flasky automatically login a new user as an existing user

I think this is a major bug/problem - I have tested on a few different machines. To reproduce:

Login as an admin, i.e., Admin (you can either keep the page open or close the page)
Register a new user, i.e., User 1
Click the "To confirm your account please click here." link in the confirmation email
a new web page is opened with Admin logged in (not the new user).

The correct way should be a confirmation page or login page for the new user.
@miguelgrinberg any idea why this is happening? Thanks!

How do you register a new user while logged in as admin?

To clarify:
I did 1 in Safari, did 2 in Chrome on the same Mac, did 3 in Mac Mail, and Safari opens for 4.

Okay, good.
This is not a bug, really. Your Safari is logged in as admin, so in that state it is not possible to confirm another user's account. If you open the confirmation link Flasky will not be able to validate it, as it does not match the logged in user.
For the account confirmation to work you will need to log out from the admin account, log in as the new user and then click the confirmation link. Alternatively, you can log out from admin, then click the confirmation link, and you in that case you will automatically be asked to log in as the new user.

Thanks, @miguelgrinberg

@miguelgrinberg I tried the following and think this issue is still valid:

login as admin in Safari, did not choose "keep me logged in", then close Safari
open Chrome, click reset password "click here to reset it", enter another user's email address (user 1 not admin).
in the reset password email for user 1, click "To reset your password click here."
Safari is opened with admin logged in showing the index page

I tried the same workflow on Facebook using two different accounts, after 3, a login page is presented instead of automatically logging in the first user. More specific question is: how to tell the current request from an email with a token is for a user that is not the current user? Hope I explained this clearly. Thanks.

login as admin in Safari, did not choose "keep me logged in", then close Safari

When you say that you "closed" Safari, did you close the entire browser of just the tab? Session cookies remain active until the browser process is closed, so closing a tab and reopening does not clear the logged in user. This is a standard browser behavior, implemented by all browsers, as far as I know.

how to tell the current request from an email with a token is for a user that is not the current user?

Flasky can easily tell when you click on a token that is not for the current user. That is why it ignores the token if it isn't address to the current user. It seems you would prefer that the current user is logged out and then the user for which the token is for be logged in. You can certainly implement that behavior if you like it better. I personally don't think it is a good idea that if user A is logged in and then an action taken by user B can force user A to be logged out. My preference is that if user A is logged in and user B is not, then A has more priority, so requests from B are ignored.

I only closed the tab - so you are right - when I close the browser, the problem is gone. Could you please provide more details on How "Flasky can easily tell when you click on a token that is not for the current user."? Any file in Flasky has that part implemented? Thanks a lot.

So this is done differently depending on which kind of token is being used.
For the account confirmation token, there is a current_user.confirm(token) call. Inside the confirm() method, the user id stored in the token is compared against self.id and if they don't match the token is considered invalid (see here). The change email token is handled in a similar way.
For the password reset token, it is required that the user is not logged in, because obviously if the user is logged in it would not need a password reset. So in this case any logged in user would prevent the token from validating (see here).

@miguelgrinberg thanks a lot for your help!!!


Add INDEL mutations to bamsurgeon workflow

I needed to add INDELs to a bam, so I thought I'd share my edits to the WDL that allowed me to do so. I also cleaned up the WDL to align with our production style guide.

Thanks!
On Thursday, July 11, 2019, Trevyn Langsford<EMAIL_ADDRESS>wrote:

I needed to add INDELs to a bam, so I thought I'd share my edits to the
WDL that allowed me to do so. I also cleaned up the WDL to align with our
production style guide.
You can view, comment on, or merge this pull request online at:
https://github.com/NCBI-Hackathons/Bringing-the-Power-
of-Synthetic-Data-Generation-to-the-Masses/pull/7
Commit Summary

add INDEL mutations to bamsurgeon workflow

File Changes

M workflows/Mutate-reads-with-BAMSurgeon/Mutate-reads-with-
BAMSurgeon.json
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7/files#diff-0
(26)
M workflows/Mutate-reads-with-BAMSurgeon/Mutate-reads-with-
BAMSurgeon.md
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7/files#diff-1
(5)
M workflows/Mutate-reads-with-BAMSurgeon/Mutate-reads-with-
BAMSurgeon.wdl
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7/files#diff-2
(196)

Patch Links:

https://github.com/NCBI-Hackathons/Bringing-the-Power-
of-Synthetic-Data-Generation-to-the-Masses/pull/7.patch
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7.patch
https://github.com/NCBI-Hackathons/Bringing-the-Power-
of-Synthetic-Data-Generation-to-the-Masses/pull/7.diff
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7?email_source=notifications&email_token=AKX4GYSODJIGGVISIG44CYLP656JVA5CNFSM4IBRGSTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G6WUTOQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKX4GYVK2ISU6QUUB3KDMPDP656JVANCNFSM4IBRGSTA
.

--
Adelaide Rhodes, Ph.D.
Senior Computational Research Associate
Excellent Pinball Player
Former Deep Sea Marine Zoologist
https://broadinstitute.academia.edu/AdelaideRhodes
Data Sciences Platform
https://www.broadinstitute.org/data-sciences-platform | Broad Institute
of MIT and Harvard

Don't merge it yet! Apparently, its not quite working.

Okay, keep me in the loop. I am traveling tomorrow but can merge next week.
Adelaide
On Thursday, July 11, 2019, Trevyn Langsford<EMAIL_ADDRESS>wrote:

Don't merge it yet! Apparently, its not quite working.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NCBI-Hackathons/Bringing-the-Power-of-Synthetic-Data-Generation-to-the-Masses/pull/7?email_source=notifications&email_token=AKX4GYV46GSGELCMKT3ZZ5LP66LKNA5CNFSM4IBRGSTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZX5O5Y#issuecomment-510646135,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKX4GYQBA2FYV2PKJMAI7DLP66LKNANCNFSM4IBRGSTA
.

--
Adelaide Rhodes, Ph.D.
Senior Computational Research Associate
Excellent Pinball Player
Former Deep Sea Marine Zoologist
https://broadinstitute.academia.edu/AdelaideRhodes
Data Sciences Platform
https://www.broadinstitute.org/data-sciences-platform | Broad Institute
of MIT and Harvard

Ok, it appears to work now!

It is merged, thanks for sending a note!

Thanks Trevyn! FYI I'm forking this repo into a new one so we can keep building out the project and document it as a standalone without muddying the record on the hackathon project.


++SEX~VIDEO~XXX)* 18+ Hot indian mms viral Leaked video

37 seconds ago.....
[🌐 𝖢𝖫𝖨𝖢𝖪 𝖧𝖤𝖱𝖤 🟢==►► 𝖶𝖠𝖳𝖢𝖧 𝖭𝖮𝖶]
[🔴 𝖢𝖫𝖨𝖢𝖪 𝖧𝖤𝖱𝖤 🌐==►► 𝖣𝗈𝗐𝗇𝗅𝗈𝖺𝖽 𝖭𝗈𝗐]
L𝚎aked Video 18+ Hot indian mms video Original Video Viral Video L𝚎aked on X Twitter Telegram
[-wATCH-]— 18+ Hot indian mms video Video Original Video Link 18+ Hot indian mms video Video Viral On Social Media X Trending Now
[-wATCH-]— 18+ Hot indian mms video ʟᴇᴀᴋᴇᴅ Video ᴠɪʀᴀʟ On Social Media ˣ ᵀʷⁱᵗᵗᵉʳ
[-wATCH-]— 18+ Hot indian mms video ʟᴇᴀᴋᴇᴅ Video ᴠɪʀᴀʟ On Social Media ˣ ᵀʷⁱᵗᵗᵉʳ
[-wATCH-]— 18+ Hot indian mms video Video Original Video Link 18+ Hot indian mms video Video Viral On Social Media X Trending Now
18+ Hot indian mms video Original Video video took the internet by storm and amazed viewers on various social media platforms. 18+ Hot indian mms video, a young and talented digital creator, recently became famous thanks to this interesting video.
L𝚎aked Video 18+ Hot indian mms video Original Video Viral Video L𝚎aked on X Twitter
18+ Hot indian mms video Original Video video oficial twitter
L𝚎aked Video 18+ Hot indian mms video Original Video Viral Video L𝚎aked on X Twitter..
.
.

gfht


the implementation is not as clean as it could be due to upstream issues in terraform

https://github.com/hashicorp/terraform/issues/5367#issuecomment-264409516

For what it's worth, I've been trying to put together a terraform mailgun provider as a plugin that uses v3 of the api and deals with some of these short comings: https://github.com/phillbaker/terraform-provider-mailgunv3.

@phillbaker nice! I would love to use your provider instead. Are you going to submit a PR to hashicorp/terraform ?

@samstav yup, it already supports domains and routes: https://github.com/phillbaker/terraform-provider-mailgunv3#usage.
I've been meaning to finish up the tests, so probably need to do that before getting it into a PR, but have been using it regularly.


Styling customizations

There is a previously closed issue: https://github.com/mrin9/RapiDoc/issues/427 for this.
But only Search button is customizable, being top-level component. Need ability to customize other buttons: Fill Example, Clear, Try Copy etc.

Any update here please?


fix: properly handle postfix for getRealPath

Description
#4708 introduced a regression in 2.6, getRealPath was being called to resolve file+postfix, instead of only the file. This PR moves that call when resolving the file (it also refactor the function to avoid calling normalizePath twice)
Fixes #5053, #5146, #5148
What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

@tjk @logaretm I think this should solve the reported regression, thanks for your help here

From vite 2.6.2 - vite + vue + tailwindcss don't working exacly.
Detail :
i turn on jit in vite + vue + tailwindcss
tailwindcss class don't update when hotreload.
i had set TAILWIND_MODE=watch when run dev project
Please check this issue

@nvthuong1996 please create an issue with a proper minimal reproduction linking to this PR so we can track the bug you are experiencing, thanks!


[Feature Request] Maintenance mode

Can you bring a maintenance mode?
What will it include:
Custom MOTD, Custom ICON, Whitelist player, commands, for start stop and whitelist add, Closing logins to server and sending maintenance message when try to join server.

To add onto this, make it so when the server whitelisted automatically display that maintenance mode motd.

We could integrate support with this plugin. They already support ServerListPlus.

Seems like a good idea!


Chat and communication on site

https://freenode.net/
https://disqus.com/profile/signup/

May I suggest including a dropbox for people to use their own songs in the Launchpad?

Hi, I'm an student form Spain and I wish to work with you on this project, can you contact me with e-mail to talk about this<EMAIL_ADDRESS>

can you please put mako-beam on your launchpad site?Or add more songs 👍

how do i upload files

i've learned the 2 songs i like on here, and now I'm out of music to play, you should really add more song

How can I download it and play this offline?


⚠ RPC server - Binance BSCT has degraded performance

In fababb4, RPC server - Binance BSCT ($BINANCE_BSCT_RPC_URL) experienced degraded performance:

HTTP code: 504
Response time: 10443 ms

Resolved: RPC server - Binance BSCT performance has improved in 9e3bb03.


Relaxed column naming restrictions

This PR changes rules for column naming:

' is allowed for column names
[ and ] are allowed if they are not like a subscript expression
_ underscore at the beginning is allowed if it doesn't conflict with the system column naming rule (_ + lowercase letter containing _)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2c84b5c). Click here to learn what that means.
The diff coverage is 93.33%.

@@            Coverage Diff            @@
##             master    #6096   +/-   ##
=========================================
  Coverage          ?   83.16%
  Complexity        ?    12515
=========================================
  Files             ?     1453
  Lines             ?    47344
  Branches          ?     5814
=========================================
  Hits              ?    39376
  Misses            ?     5271
  Partials          ?     2697

Impacted Files
Coverage Δ
Complexity Δ

.../io/crate/analyze/AlterTableAddColumnAnalyzer.java
90% <100%> (ø)
17 <0> (?)

...n/java/io/crate/analyze/TableElementsAnalyzer.java
89.31% <100%> (ø)
5 <0> (?)

.../java/io/crate/analyze/AbstractInsertAnalyzer.java
95.38% <100%> (ø)
32 <0> (?)

...ze/expressions/ExpressionToColumnIdentVisitor.java
100% <100%> (ø)
12 <0> (?)

...executor/transport/TransportShardUpsertAction.java
86.09% <100%> (ø)
77 <0> (?)

...o/crate/exceptions/InvalidColumnNameException.java
100% <100%> (ø)
2 <1> (?)

.../io/crate/analyze/expressions/ValueNormalizer.java
78.37% <100%> (ø)
23 <0> (?)

...ava/io/crate/analyze/AnalyzedColumnDefinition.java
91.39% <100%> (ø)
74 <0> (?)

...l/src/main/java/io/crate/metadata/ColumnIdent.java
95.04% <88.88%> (ø)
57 <12> (?)

Continue to review full report at Codecov.

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


Render output file from terminal is strange

for example like run cargo build test01 &> build.log or cargo build test01 &> build.ans
I guess some people met the similar problem many times ?

those red color esc is quite stunning~

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


Tooltip rendering crash for various items in multiplayer

Honestly, Im not sure that the issue originates in this mod specifically. Reason I am posting this here is that is where one of the traces seems to point towards. Those traces may just be a symptom however. We of course play with Spellblades and Such.
Observations:
After defeating Magus in the Glass Ocean a couple of times on our server, all of a sudden I had instant crashes for various Spellblade, Wizards RPG and Paladins & Priests items the moment I am trying to hover over them. This includes some items in EMI directly. Spellblade spellbooks (but apparently only the instanced ones with spells attached in the inventory), items like the crystal cutlass in my hand as well as inside EMI, items with oil applied, the Paladin claymores or the Wizards and Priest staffs. Armor is specifically NOT impacted.
These crashes do not happen in singleplayer, only on that server right now. They may have started when I obtained Ashes of the Stars and Crystallised Omniscience, but I think thats coincidence. The server log shows no issues.
One example of a log with crash is attached here:
Crashlog.MC.txt
The log shows how, when connected to the server, EMI struggles getting some recipes related to the crashing items, so I guess the server does not respond correctly for some items anymore. This is after around 30 hours+ of gameplay without issues.
If the issue is related to another mod after all, I will happily post the issue there.
Thanks in advance.

I will be uploading a fix for this soon, within the next 10 minutes. Due to
the nature of the crash being clientside you should be able to test it by
just replacing your client's extraspellattributes without replacing the
server's.
On Sat, Jun 8, 2024 at 12:43 PM ZarTaen @.***> wrote:

Honestly, Im not sure that the issue originates in this mod specifically.
Reason I am posting this here is that is where one of the traces seems to
point towards. Those traces may just be a symptom however. We of course
play with Spellblades and Such.
Observations:
After defeating Magus in the Glass Ocean a couple of times on our server,
all of a sudden I had instant crashes for various Spellblade, Wizards RPG
and Paladins & Priests items the moment I am trying to hover over them.
This includes some items in EMI directly. Spellblade spellbooks (but
apparently only the instanced ones with spells attached in the inventory),
items like the crystal cutlass in my hand as well as inside EMI, items with
oil applied, the Paladin claymores or the Wizards and Priest staffs. Armor
is specifically NOT impacted.
These crashes do not happen in singleplayer, only on that server right
now. They may have started when I obtained Ashes of the Stars and
Crystallised Omniscience, but I think thats coincidence. The server log
shows no issues.
One example of a log with crash is attached here:
Crashlog.MC.txt
https://github.com/user-attachments/files/15749484/Crashlog.MC.txt
The log shows how, when connected to the server, EMI struggles getting
some recipes related to the crashing items, so I guess the server does not
respond correctly for some items anymore. This is after around 30 hours+ of
gameplay without issues.
If the issue is related to another mod after all, I will happily post the
issue there.
Thanks in advance.
—
Reply to this email directly, view it on GitHub
https://github.com/cleannrooster/extraspellattributes/issues/2, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AW4ADB6AGAPCGX6G5BM7V6LZGNNG3AVCNFSM6AAAAABJAHHYTKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2DCOBQGY3DINA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes, the fix indeed fixed the crashes I have had, thanks a lot for the quick response :D


MULTI-GITTER PR: migration github-config-and-gitignore-changes

Summary by Sourcery
Migrate branch references from 'master' to 'main' in documentation, configuration, and CI workflows. Update GitHub Actions to reflect these changes and enhance documentation for clarity.
Enhancements:

Update branch references from 'master' to 'main' across various configuration files and documentation.

CI:

Modify GitHub Actions workflows to trigger on the 'main' branch instead of 'master'.

Documentation:

Revise the CONTRIBUTING.md and CODE_OF_CONDUCT.md files for clarity and consistency.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


About stpls3d training

Hi,
I get an error when I train on the stpls3d dataset.

File "D:/Instance_Segmentation/ISBNet-master/tools/train.py", line 309, in
main()
File "D:/Instance_Segmentation/ISBNet-master/tools/train.py", line 300, in main
train(epoch, model, optimizer, scheduler, scaler, train_loader, cfg, logger, writer)
File "D:/Instance_Segmentation/ISBNet-master/tools/train.py", line 70, in train
loss, log_vars = model(batch, return_loss=True, epoch=epoch - 1)
File "D:\Anaconda3\envs\tf_to\lib\site-packages\torch\nn\modules\module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "D:\Instance_Segmentation\ISBNet-master\isbnet\model\isbnet.py", line 220, in forward
return self.forward_train(**batch, epoch=epoch)
File "D:\Instance_Segmentation\ISBNet-master\isbnet\util\utils.py", line 172, in wrapper
return func(*new_args, **new_kwargs)
File "D:\Instance_Segmentation\ISBNet-master\isbnet\model\isbnet.py", line 361, in forward_train
voxel_instance_labels, subsample_idxs, instance_cls, instance_box, dc_batch_offsets, batch_size
File "D:\Instance_Segmentation\ISBNet-master\isbnet\model\model_utils.py", line 657, in get_subsample_gt
instance_labels_b = subsample_instance_labels[start:end]
RuntimeError: CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Process finished with exit code 1

I'm running on a single RTX 4090, even if the batch size is set to 1.
When I try to set use_spp_pool=True, this error will not be displayed, but it will bring a vast GPU memory usage.
Can you give me some advice?
Thanks.
Best

Hi,
Thanks for your interest in our work!
First, you could set CUDA_LAUNCH_BLOCKING=1 and re-run the script to check exactly which function causes this error.
Second, did you pre-train the backbone before training the entire network? I guess that you trained the entire network from scratch which sometimes causes ambiguous inputs to the Local Aggregator.
Third, we only test our model on Ubuntu. Thus we cannot ensure that it will work perfectly on Windows.
Best.


Arduino HIP battery emulator

Alternative to #23 for battery emulation.
Idea
Use existing hidbatt driver that's included in Windows (see https://support.moonpoint.com/os/windows/server2012/UPS/)
Links:

https://www.usb.org/sites/default/files/documents/pdcv10.pdf (page 18)
https://github.com/abratchik/HIDPowerDevice

HID architecture documents "HID UPS battery" to have Usage page=0x0084, Usage=0x0004 and Access mode=Shared.
hidbatt.inf content:

[Version]
Signature="$WINDOWS NT$"
Class=Battery
ClassGuid={72631e54-78a4-11d0-bcf7-00aa00b7b32a}
Provider=%MSFT%
DriverVer = 06/21/2006,10.0.19041.1

[SourceDisksNames]
3426=windows cd

[SourceDisksFiles]
hidbatt.sys = 3426

[DestinationDirs]
DefaultDestDir = 12

[ControlFlags]
BasicDriverOk = *
ExcludeFromSelect = *

[Manufacturer]
%MSFT%=Microsoft,NTamd64

[Microsoft.NTamd64]
%HID_DEVICE_BATTERY.DeviceDesc% = HidBatt_Inst,HID_DEVICE_UP:0084_U:0004

[HidBatt_Inst]
CopyFiles = HidBattInstCopyFiles
Include = battery.inf
Needs = Battery_Inst

[HidBattInstCopyFiles]
hidbatt.sys,,,0x100

[HidBatt_Inst.Services]
AddService=HidBatt,%SPSVCINST_ASSOCSERVICE%,HidBatt_Service_Inst

[HidBatt_Service_Inst]
DisplayName    = %HidBatt.SvcDesc%
ServiceType    = %SERVICE_KERNEL_DRIVER%
StartType      = %SERVICE_DEMAND_START%
ErrorControl   = %SERVICE_ERROR_NORMAL%
ServiceBinary  = %12%\HidBatt.sys

[strings]
;Localizable Strings
MSFT                     = "Microsoft"
HID_DEVICE_BATTERY.DeviceDesc = "HID UPS Battery"
HidBatt.SvcDesc          = "HID UPS Battery Driver"

;Non-Localizable Strings
SPSVCINST_ASSOCSERVICE   = 0x00000002
SERVICE_KERNEL_DRIVER    = 1
SERVICE_DEMAND_START     = 3
SERVICE_ERROR_NORMAL     = 1

Completed in 7222549a744c57efc65c42e6bcb9c77d54b735f2


Move Pages "prohibited uses" from documentation to terms

This PR moves section on prohibited uses from Pages documentation to the Pages section of our Additional Product Terms
(There's no change in content - just moving words around.)
We plan to merge this PR on Monday, Feb. 22.

Closed by https://github.com/github/site-policy/pull/379


chore:Telegram notification-provider using the throwGeneralAxiosError helper

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
During https://github.com/louislam/uptime-kuma/pull/3620#discussion_r1299877166 I noticed that this notification provider does not stick to the pattern all other notification-providers use.
Type of change
Please delete any options that are not relevant.

Other

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings

Screenshots (if any)
before:

after:

Please merge into 1.23.x branch

Please note that v1 will receive bug fixes only at this moment. Improvements like this and new features are not guaranteed to be in 1.23.X.

Update: The last minor version (1.23.x) means that there won't be any new features introduced in v1. However, bug fixes will still be provided. You can expect to see 1.23.1, 1.23.2 and so on.

https://github.com/louislam/uptime-kuma/releases/tag/1.23.0

This is a small code change - I see this as a bug fix, not new feature

This is refactoring, as this just unifiest the odd one out of all the notifcation provider to also use the same error hadling logic.
=> neither a bugfix, nor a feature


RedisCache: boolean values are being converted to float typestrings

There is a bug in the _to_typestring conversion method.
https://github.com/python-discord/bot/blob/02d1dd1b5034778f6bfc296317c9241e93395b2a/bot/utils/redis_cache.py#L108-L121
When converting values, the first tuple in prefixes is ("f|", float). If a bool is being stored, the conditional on line 113 then triggers and returns f|0 or f|1. On the way out, the int get cast to float as per the typestring and so we get 0.0 or 1.0.
The test_set_get_item test function does not catch this because it checks the expected and received values for equality.
Both True == 1.0 and False == 0.0 hold. Printing out the values in the test, these pass:
Expected: 0.0, got: False
Expected: 1.0, got: True

As far as I can tell no code on the master branch currently relies on bool storage so this isn't critical.

I think a good way to approach this issue would be to first adjust the tests so that they fail if an incorrect type is retrieved. Then make the necessary changes to the _to_typestring method.


Need examples for Realtek RTL8720DN to communicate with Azure IoT Central

Where can I get the support for Realtek RTL8720DN chip  with Azure IoT Central?

Hi @electro89, since we don't currently have a sample showing how to use the RTL8720DN with IoT central, one way to solve for it would be for you to adapt the sample for the ESP32 with IoT Central. Since this seems to be useful to others, I'll add this ask to the backlog, but since we already have several other priorities, I can't promise when we'll have this ready.

Thank you, I hope ESP32 will release wifi dual band which can support 5G chip soon.


[BUG] UserTokenProvider.LoginWithPromptAsync is not available on the desktop framework

Library name and version
Microsoft.Rest.ClientRuntime.Azure.Authentication 2.3.3+
Describe the bug
UserTokenProvider.LoginWithPromptAsync is not available when targeting .NET 4.6-4.8. It's available on 4.5.
This appears to have been been the case since the net461 target framework was added in version 2.3.3 of the package back in 2018. The target framework was added, but there's a preprocessor statement in the code making LoginWithPromptAsync available only on net452. So when referencing the package from a recent version of the Framework, we get a build that does not include LoginWithPromptAsync.
Was this intentional? The accompanying comment seems to indicate that the intention was to exclude LoginWithPromptAsync from the .NET Core/Standard builds of the package, not from any build other than net452. The code doesn't appear to have changed substantially since before version 2.3.3, so it appears that perhaps the preprocessor statement was overlooked when the net461 target was added. I think that preprocessor statement should read #if NETFRAMEWORK.
At the very least, this breaking change should have been a major version increment, ideally with a preceding deprecation period.
Expected behavior
LoginWithPromptAsync is available on all versions of the .NET Framework, but unavailable when targeting .NET Core/(5+) and .NET Standard.
Actual behavior
LoginWithPromptAsync is available only on .NET 4.5.
Reproduction Steps
Set up a project targeting .NET 4.6-4.8 (I'm on net472), add the Microsoft.Rest.ClientRuntime.Azure.Authentication package, and try to call UserTokenProvider.LoginWithPromptAsync.
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hey @benjamin-hodgson this package has been deprecated and will not support anymore, you can more information here https://www.nuget.org/packages/Microsoft.Rest.ClientRuntime.Azure.Authentication.


Add types for stripe.confirmOxxoPayment

Summary & motivation
Adding types for confirmOxxoPayment

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@christopher-stripe Thank you very much for your review. I addressed your comment. You were absolutely correct about createPaymentMethod.


bug #92: should return success object rather than string on PATCH forms.

see ticket for details.
should probably eventually return the updated object but we'll worry
about that later.
resolves #92.

Do we also want to update the API docs, or there's no need, because we'll eventually be returning the updated object?


fix: Use the App Version issue where the app was getting reset to old url …

Description

When a template is imported. The app is reset to the old URL format. Because the applicationVersion is reset to 1. This currently breaks deployed apps.

Fixes #17428
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Locally

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test

/ok-to-test


Load Tiled object properties using names.

This PR

Fixes a bug

Describe the changes below:
Tiled objects have properties as an array of objects like:
[
  { name: ..., type: ..., value: ...},
  { name: ..., type: ..., value: ...},
  ...
]

Update the parser to load these using the name/value properties instead
of just treating it as an array and loading properties named '0', '1',
etc.  This is the same as how properties are handled in ParseTilesets.js

Do you know when was this change made to the Tiled JSON format?
It must be quite recent because that's definitely not how properties were handled when this method was rewritten.

@photonstorm - Sorry, I do not.  I just recently started using Tiled.  I can try to dig through the source and see if I can identify when.  If it used to be in a different format then I can see about making the code handle both styles.
If you have an example of the original Tiled format that would also be pretty easy for me to change.

The original format is how the current code deals with it. Any of the tmx files in the phaser3 examples repo use this format. Hence why it must be a recent change. Probably listed in the Tiled change logs somewhere.

@photonstorm - Took a closer look and the GitHub diff viewer is really bad so it's way more confusing then the change actually was.
My implementation checks if it's an array and loads it using the new pattern, otherwise it falls back to the old pattern.  I could potentially use a version check but I don't know enough about the tiled source to be able to identify when the change occurred.  Are you okay with the best-effort version implemented?

@photonstorm - As per this comment in Tiled https://github.com/mapeditor/tiled/issues/2034#issuecomment-439184633, it looks like this was changed as of Tiled 1.2.  So, same question as before, do you think it's necessary to have a version check or is the Array check as is enough?

I'm sure the array check is enough, however this change was made over 2 years ago according to that comment - has it since reverted or are you just using a really old version of Tiled?


Remove a secret-dependent branch in Montgomery multiplication

Remove a secret-dependent branch in Montgomery multiplication that could expose some bits of a private RSA key.
The potential leak was documented in https://www.usenix.org/system/files/conference/usenixsecurity17/sec17-lee-sangho.pdf and again in https://arxiv.org/pdf/2005.11516.pdf and reported in https://github.com/ARMmbed/mbedtls/issues/3394.
Fix #3394.
In this pull request, I reverted commit 2cc69fffcf431085f18f4e59c3c5188297f97b87 from https://github.com/ARMmbed/mbedtls/pull/457, partly because the only real value of the check it added was to silence a false positive from clang-analyzer, and partly because half of the check was wrong anyway. I think this reversal should be backported with the rest, because it has the benefit that it more than compensates the additional code size from this security fix, and the risk of removing a redundant check is minimal.
Needs backports to all supported branches.

From the CI: MSVC 2013 would like to be sure that you know what you're doing with one cast:

The comment above states that the value being cast is always 0 or 1, so that cast is safe, but we probably want to make it explicit to let MSVC know that we carefully considered it.

This change is more extensive than I'd like for a security improvement. Is it acceptable to backport as is?
Note that in terms of code size, there's a slight increase in the semantically necessary commits, which is more than compensated by the non-mandatory commit “Revert "Shut up a clang-analyzer warning"” .


Zoom button on printable-version

[x] I'm reporting a bug, not asking for help
[x] I've looked at the documentation to make sure the behaviour is documented and expected
[x] I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

How to reproduce

Leaflet version I'm using: Leaflet 1.0.1
Browser (with version) I'm using: Chrome 54.0.2840.100 (64-bit)
OS/Platform (with version) I'm using: Linux Mint 18 Cinnamon
Open this example at leafletjs.com or any other Leaflet-map.
Open print menu (ctrl + p) and see those plus and minus buttons.

What behaviour I'm expecting and which behaviour I'm seeing
Plus and minus buttons are not needed in print, because you can't press them (paper)...

I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using

IMHO I don't think this is really an issue that Leaflet should solve. If you have a use case where you want to hide/show controls when printing I'd suggest you implement an independent solution. Give your controls classes/id's and on window.matchMedia or beforePrint/afterPrint hide/show the controls.

I agree with @jbccollins, printing is outside Leaflet's scope,
Also, although it might often be the case that print shouldn't show controls, it's not necessarily true for all use cases, and given that we don't really care about print, it feels odd to add this to core.
It is easy for any application to override this with local CSS rules, like @jbccollins also says.

For anyone interested, something like the following could work
@media print {
	// To only hide controls at the top (usually where the zoom buttons are).
	.leaflet-top {
		display: none;
	}

	// To hide all controls, not just the ones at the top (this would hide the scale
	// if it's display on the bottom left for example).
	.leaflet-control-container {
		display: none;
	}
}


Warn about use of reflection

You should add a warning to the readme that clearly states that this library is using a bunch of reflection in the Picker classes.

Thank you, iPaulPro.The readme has been updated.

This could somewhat be done without reflection. See Resources.getSystem().getIdentifier(name, "id", "android") to get the id. Works for all of numberpicker_input, hour, minute, and divider. This is the approach I used in the first iteration of my picker.
Reflection for the selection divider seems inevitable, though, as it's a private field and annotated with @UnsupportedAppUsage


Update electron to fix certificate errors

Fixes #2671

Codecov Report

Merging #2674 (72f7106) into master (5a4fbbf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2674   +/-   ##
=======================================
  Coverage   70.83%   70.83%
=======================================
  Files          13       13
  Lines         696      696
=======================================
  Hits          493      493
  Misses        203      203

Continue to review full report at Codecov.

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

Thanks!


Get instance status has high frequency SHOW MASTER STATUS, which could block MySQL written data.

If MySQL is committing a large transaction ,purge a large number of binlog files or  Other scenarios,  it may take 5S or longer .
During this time, the SHOW MASTER STATUS will be blocked, and the blocked SHOW MASTER STAUS will block all write requests, which may cause the commit to timeout
I find that the information obtained by SHOW MASTER STATUS is mainly used for web display.
Can we  Add a  switch  for SHOW MASTER STATUS to 'open or close'?

I find that the information obtained by SHOW MASTER STATUS is mainly used for web display.

The information is used to determine the binlog coordinates of an instance, used in:

graceful takeover
pseudo-GTID operations

If we were to disable reading SHOW MASTER STATUS on a "normal" probe, then:

we would not be able to run pseudo-GTID-based recoveries (which is fine for people who use GTID)
we'd need a way to tell orchestrator to read SHOW MASTER STATUS when running specific operations like graceful takeover.

I agree with you, we'd need a way to tell orchestrator to read SHOW MASTER STATUS when running specific operations like graceful takeover
some times, we do not accept that HA components affect database availability ,such as large trading scenario
If gtid_mode=on, FAILOVER don't need imformation of SHOW MASTER STATUS ,We can give the user a choice,Turn on or off SHOW MASTER STATUS switch, is greateful:

This switch default "on"
if on: Orchestrator work as it do now
if off：Orchestrator not read SHOW MASTER STATUS normally, but when we need it, such as befor graceful takeover or other scenarios

we recently met the same issue. So any update or workaround on this ?


[wip] feat(conf) add prometheus_listen

Prometheus plugin exposes metrics related to Kong at an HTTP endpoint.
If Kong's Admin API is disabled, metrics cannot be exposed unless a
custom nginx conf is used.
An alternative solution exists where users can include an NGINX server
block using Kong's directive injection mechanism but that has proved to
be not user-friendly.
This commit adds support to setup a Prometheus scrape endpoint in Kong
natively.

I've discussed a more generic solution that solves this problem not just for Prometheus but also for other plugins that might leverage a separate endpoint.
I will work on the feature when I can find time and open a PR.
Closing this one.


refactor: move dependency validations to domain pkg

Moves dependency validation/sort functions to domains.
Use the moved functions for validation in API layer.

This is also a part of https://github.com/bucketeer-io/bucketeer/issues/904.
UpdateFeature API will use this validation.

@cre8ivejp Fixed 👍


LLM not outputting text most of the time with Kobaldcpp or ollama back end.

I am having trouble getting usable if any text to come out of the response pin, the question pin comes out as expected tho. The llm back end receives the question, but nothing is coming out.  I am using the If chat prompt node. My workflow is just a if chat prompt connected to a show text in the 4 pins. I am on a windows os computer.
I have tried changing the stop, using <|end_of_text|>, < /s>, . ,  and <|eot_id|>. I also changed the assistant to MKR, and the model to llama3_if_ai_sdpromptmkr_q4km. I have tried a few other models also, mostly llama 3 or 2 models.  Besides that I left everything default besides the port numbers. Is there anyway to just get the raw text out? I tried flipping the text cleanup, but nothing happened. Also changed assistant to none. I have nothing going into the image or context pin, just a command in the prompt box. Thanks for your help.
Below is the output from Koboldcpp back end terminal, first one being MKR with the llama 3_if_ai model, second being none as the assistant and a llama 3 model. I had similar results with ollama, but it wasn't showing anything in the terminal.

 Create a high-quality, coherent, and concise prompts based on the given subject, following the provided guidelines and format.', 'rules': ['Break keywords by commas', 'Focus solely on visual elements; avoid art commentaries or intentions', 'Construct prompt with subject, scene, and background components', 'Limit to 7 keywords per component', 'Include all subject keywords verbatim as main focus', 'Be varied and creative in descriptions', 'Keep prompt under 100 words', 'Do not enumerate or enunciate components', 'Do not include additional information beyond prompt'], 'examples': [{'input': 'Demon Hunter, Cyber City', 'output': 'A Demon Hunter, standing, lone figure, glowing eyes, deep purple light, cybernetic exoskeleton, sleek, metallic, glowing blue accents, energy weapons, fighting Demon, grotesque creature, twisted metal, glowing red eyes, sharp claws, in Cyber City, towering structures, shrouded haze, shimmering energy'}]}\nUser: I want you to describe a photo of a forest in detail.\nUser: I want you to describe a photo of a forest in detail.", "max_length": 128, "temperature": 0.7, "top_k": 40, "top_p": 0.2, "rep_pen": 1.1, "stop_sequence": ["\n\n\n\n\n", "<|end_of_text|>"]}
Processing Prompt (29 / 29 tokens)
Generating (128 / 128 tokens)
CtxLimit: 394/8192, Process:0.00s (0.1ms/T = 7250.00T/s), Generate:1.88s (14.7ms/T = 68.01T/s), Total:1.89s (67.87T/s)
Output:
System: Serene, mystical forest landscape, dense foliage, towering trees, vibrant green leaves, soft sunlight filtering through canopy above, dappled shadows beneath, moss-covered rocks, hidden streams, misty veil rising from valley floor, ethereal atmosphere, dreamlike quality, captivating composition, rich colors, depth of field, clarity, vividness, serenity, enchantment.
 ID:128001)
CtxLimit: 262/8192, Process:0.00s (0.1ms/T = 10333.33T/s), Generate:3.92s (0.5ms/T = 2041.86T/s), Total:3.92s (2040.30T/s)
Output:
Response:
The photo depicts a dense and lush forest, with tall trees that stretch up towards the sky, their leaves rustling gently in the breeze. The sunlight filters through the canopy above, casting dappled patterns of light and shadow on the forest floor below. Various shades of green dominate the scene, from the deep emerald of the tree trunks to the vibrant foliage of the branches and leaves. In the distance, one can see the silhouette of a winding river, its surface reflecting the colors of the surrounding landscape.
In addition to the main focus on the trees, there are also smaller details scattered throughout the image. A carpet of moss covers much of the forest floor, providing a soft and damp contrast to the rough bark of the tree trunks. Some fallen logs lie strewn about, adding texture and depth to the overall composition. Birds and insects flit between the branches, further bringing life to this already thriving ecosystem.
Overall, the photo captures the beauty and tranquility of a forest at its peak, showcasing both the grandeur of nature as well as the intricate details that make it so unique.

Seconding this. Been trying to get koboldcpp to work for a while now with this node. Can't get anything usable out of it.
Only the chat prompt node has a kobold backend as option and none of the outputs provided by the node actually carry the messages coming from koboldcpp despite my best efforts.
Something seems broken in the api implementation or better instructions needed for how this is supposed to work.

Sorry I am fixing it now I just got home I had an issue with the images not sure what is happening it was working before I am working on it

Koboldcpp says that they have support for image messages in their API but I tried everything with both the open ai endpoint and the native endpoint and it does not pick the image maybe they have support images but only with as particular model. for now only text seems to be working please try again after updating

Thanks for your help, It works well. The text was what i planned on using for just adding a bit of randomness to prompts.

Thank you. I hope they improve the vision model support api. Is the same with oobabooga they only support some old vision models which restricts what I could do.


v1.1.0 Breaks the behavior of the Docker image

I'm opening this issue as a note to myself and others who are using Dockerized Meerschaum. Changes to the virtual environment system in v1.1.0 mean that dependencies are more strictly enforced to be installed within the specified virtual environment. The behavior of the latest and full images relies on installing all of the dependencies in the user environment, so this change reverts its behavior to the same as the minimal image.
A temporary workaround for this is to persist a docker volume mounted at /meerschaum (the container's root), and run the command mrsm upgrade packages -y which will install everything into the virtual environment.
I will be working on a more permanent solution in the coming days, mainly considering a package as "not installed" when it's detected in the user environment if it fails the version check. Admittedly, I added the stricter behavior to avoid having to go the long way around and verify the version, but now I realize there's no easy way to get around that.
Expect a patch in v1.1.1 to be out soon!

v1.1.3 (this commit) fixes the issues mentioned above. The latest Docker images have been built and include this patch.


Hardware environment

Hi Justin,
May I ask for your hardware environment for running this programme?
Sincerely,
Yanfei

Hi Yanfei,
I ran on a 16 core machine with a single Nvidia P100 GPU.
Cheers,
Justin
On Tue, 9 Jul 2019 at 8:39 pm, Zhang Yanfei<EMAIL_ADDRESS>wrote:

Hi Justin,
May I ask for your hardware environment for running this programme?
Sincerely,
Yanfei
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Justin-Tan/generative-compression/issues/31?email_source=notifications&email_token=AGRNY6A6BV4GIVURTORL6HDP6RTFXA5CNFSM4H7EDWNKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G6CDOTA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGRNY6EJDREAQHOEJONE6ZTP6RTFXANCNFSM4H7EDWNA
.

Roger that. Thank you.


AttributeError in GET /clan/rlupuqgu.json

Error in ClashLeaders
AttributeError in GET /clan/rlupuqgu.json
'NoneType' object has no attribute 'players_data'
View on Bugsnag
Stacktrace
clashleaders/views/clan.py:41 - clan_detail_json

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
AttributeError in GET /clan/8puvy0py.json

An error linked to this issue has been reopened in Bugsnag
AttributeError in GET /clan/c8yqlqu8.json

An error linked to this issue has been marked as fixed in Bugsnag
AttributeError in GET /clan/y8r9pjc2.json

An error linked to this issue has been reopened in Bugsnag
AttributeError in GET /clan/jqc0r8cr.json

An error linked to this issue has been reopened in Bugsnag
AttributeError in GET /clan/yqr0yq0g.json

An error linked to this issue has been marked as fixed in Bugsnag
AttributeError in GET /clan/pgr2yrc9.json

An error linked to this issue has been reopened in Bugsnag
AttributeError in GET /clan/p88vggy8.json

An error linked to this issue has been marked as fixed in Bugsnag
AttributeError in GET /clan/pu0yvpry.json


Add check for double hw support

/intelci: run

@KulikovNikita @samir-nasibli @Alexsandruss  - this looks like not harmful PR, but i'm not sure if it's sufficient for actually working on systems without FP64 support?

/intelci: run

Fixed in #1198