Database connection

I have installed ollama and can run prompts, example: ollama run llama2 "why is the sky blue?"
Is there any way to connect to MYSQL database and start asking about database data, example:
`###### database file######
database.cnf
host="localhost"
user="admin"
passsword="admin"
database="mDatabase"
ollama run llama2 database "how many users are administrators?"
I have searched repository and I cannot see anything like this, I also went to langchain docs but did not see an example with ollama
`

Hi @FaizelK this is not built into Ollama, but it is a good example of a workflow that you could build on top of Ollama. As an aside I would recommend dumping the contents of the database to a file which you parse into structured data and feed into Ollama rather than giving the LLM direct access to query your database.

@BruceMacD i would think access directly to database is better than exporting data into a file, can you imagine a database with millions of records? also langchain and openai have this plugin (database agents) I am working on this so it can work with ollama. if you create a database user with only read access/privilege then it is no harm.

@MostlyKIGuess by using lanchain
`from langchain.agents import create_sql_agent
from langchain.agents.agent_toolkits import SQLDatabaseToolkit
from langchain.sql_database import SQLDatabase
from langchain.agents import AgentExecutor
from langchain.agents.agent_types import AgentType
from langchain.llms import Ollama
llm = Ollama(model="llama2")
db = SQLDatabase.from_uri("mysql+pymysql://username:password@localhost/databasName")
toolkit = SQLDatabaseToolkit(db=db,llm=llm)
agent_executor = create_sql_agent(
llm=llm),
toolkit=toolkit,
verbose=False,
agent_type=AgentType.OPENAI_FUNCTIONS,
)
print( agent_executor.run("how many records for today?") )
so obviously the issue here is that the agent type is openai which we dont want, we want to use something else like NLSQLTableQueryEngine (natural language SQL)
`

Hello @FaizelK,
Direct database connections are not currently supported by Ollama. However, you can create a webpage that connects to the database and extracts data from it, which can then be analyzed by Ollama using the following link:
https://ollama.ai/blog/building-llm-powered-web-apps

I'm also looking for a way to use OLLAMA with a local SQLITE database - The reason this is important is contents of a database can change, a little, or a lot, by so many external factors, and I want the query of the data in the database to be on the current data in there.
Converting the data into a file format for ingesting into a vector store means that the data is now static and instantly out of date.
I also want the system to be able to not just query the data but to also alter the data as required. Think about a stock management system, where you can ask how many of product XX is in stock, and also reduce the quantity when needed etc
All langchain examples I've seen so far require the use of OpenAI - and I'd like this to be 100% local and not via a hosted website for data querying.

@UnexpectedMaker You found the Github. Did you check out the examples folder in the repo?
There are multiple Langchain examples using Ollama. The ones I checked use a local option for generating embeddings.
Also make sure you check out SQLite-VSS support in Langchain.

@UnexpectedMaker You found the Github. Did you check out the examples folder in the repo?
There are multiple Langchain examples using Ollama. The ones I checked use a local option for generating embeddings.
Also make sure you check out SQLite-VSS support in Langchain.

I found the GitHub - I checked the examples. None deal with or show how to do ollama + local model + langchain (or llamaIndex) to actually access a standard SQL database (not as a vector source, 100% was NOT what I asked for) and query it to return results.
I dont want to ingest documents and store the quantized vector data in an SQL database - I want the model to look into a normal SQL database and have it strain itself on the structure - and then be able to pull results out of the tables.
Every example I can find online (google, blogs, YouTube) show this only working with llama.cpp + (llamaindex/langchain) and openai.
Rather than asking me to check the examples - if you think there's an actual example in there that does what I have asked about, and what the OP asked about, please point specifically to that example.
Thanks :)

I am back, back with gifts, sweets and all the good stuff.
Yes, I have found an open source solution to my original question. I will add my reference if i get 10 hearts on this solution
After 100's of hours ...... Here it goes........
`from langchain_community.chat_models import ChatOllama
#define your model
llama2_chat = ChatOllama(model="llama2:13b-chat") #only change this model if required
llama2_code = ChatOllama(model="codellama:7b-instruct")
from langchain.callbacks.manager import CallbackManager
from langchain.callbacks.streaming_stdout import StreamingStdOutCallbackHandler
from langchain.llms import Ollama
#set your model
llm = llama2_chat
#sometimes in some machine this doesnt work so you can use below to set your model
llm = Ollama(model="llama2:13b-chat", callback_manager=CallbackManager([StreamingStdOutCallbackHandler()]))
from langchain_community.utilities import SQLDatabase
db = SQLDatabase.from_uri("mysql+pymysql://database_user:password@localhost/databasName")
def get_schema(_):
return db.get_table_info()
def run_query(query):
return db.run(query)
from langchain_core.prompts import ChatPromptTemplate
template1 = """Based on the table schema below, write a SQL query that would answer the user's question:
{schema}
Question: {question}
SQL Query:"""
prompt = ChatPromptTemplate.from_messages(
[
("system", "Given an input question, convert it to a SQL query. No pre-amble."),
("human", template1),
]
)
Chain to query
from langchain_core.output_parsers import StrOutputParser
from langchain_core.runnables import RunnablePassthrough
sql_response = (
RunnablePassthrough.assign(schema=get_schema)
| prompt
| llm.bind(stop=["\nSQLResult:"])
| StrOutputParser()
)
Chain to answer
template2 = """
Based on the table schema below, question, sql query, and sql response, write a natural language response:
{schema}
Question: {question}
SQL Query: {query}
SQL Response: {response}
"""
prompt_response = ChatPromptTemplate.from_messages(
[
(
"system",
"Given an input question and SQL response, convert it to a natural language answer. No pre-amble.",
),
("human", template2),
]
)
full_chain = (
RunnablePassthrough.assign(query=sql_response)
| RunnablePassthrough.assign(
schema=get_schema,
response=lambda x: db.run(x["query"]),
)
| prompt_response
| llm
)
TheFinalResponse = full_chain.invoke({"question": ThePrompt})
print(TheFinalResponse)`

Nice.
I understand that the user describes their query in natural language. It's converted into SQL queries by an LLM. The script retrieves results from a database. Then, another LLM transforms those results into an easy-to-understand response for the user.

Yes,
The users prompt could be
User prompt
"how many users are admins?" or "how many users in total?"
first LLM
translates this user promp to sql "select count(*) from table"
second LLM
translates the sql results into english and final response is
"there are 10 users in total"

Does anyone know how to become a contributor? i would like to add this module as part of this repo

@FaizelK you can always fork the repository, copying it into your account, work on it, and in the end submit a pull request to this repository, which can then be merged into the code here.
It's not garantieed to be merged, but people can also come to your fork in that case and use that instead.

I am back, back with gifts, sweets and all the good stuff. Yes, I have found an open source solution to my original question. I will add my reference if i get 10 hearts on this solution After 100's of hours ...... Here it goes........ @BruceMacD @MostlyKIGuess
from langchain_community.chat_models import ChatOllama
from langchain.callbacks.manager import CallbackManager
from langchain.callbacks.streaming_stdout import StreamingStdOutCallbackHandler
from langchain.llms import Ollama
from langchain_community.utilities import SQLDatabase
from langchain_core.prompts import ChatPromptTemplate
from langchain_core.output_parsers import StrOutputParser
from langchain_core.runnables import RunnablePassthrough

# Define chat models
llama2_chat = ChatOllama(model="llama2:13b-chat")  # Change model if required
llama2_code = ChatOllama(model="codellama:7b-instruct")

# Set model (choose one of the following options)
llm = llama2_chat  # Option 1
# llm = Ollama(model="llama2:13b-chat", callback_manager=CallbackManager([StreamingStdOutCallbackHandler()]))  # Option 2

# Connect to database
db = SQLDatabase.from_uri("mysql+pymysql://database_user:password@localhost/databasName")

# Define functions for schema retrieval and query execution
def get_schema(_):
    return db.get_table_info()

def run_query(query):
    return db.run(query)

# Create prompt templates
template1 = """
Based on the table schema below, write a SQL query that would answer the user's question:
{schema}

Question: {question}
SQL Query:
"""
prompt = ChatPromptTemplate.from_messages(
    [
        ("system", "Given an input question, convert it to a SQL query. No pre-amble."),
        ("human", template1),
    ]
)

template2 = """
Based on the table schema below, question, sql query, and sql response, write a natural language response:
{schema}

Question: {question}
SQL Query: {query}
SQL Response: {response}
"""
prompt_response = ChatPromptTemplate.from_messages(
    [
        (
            "system",
            "Given an input question and SQL response, convert it to a natural language answer. No pre-amble.",
        ),
        ("human", template2),
    ]
)

# Construct chains for query generation and response
sql_response = (
    RunnablePassthrough.assign(schema=get_schema)
    | prompt
    | llm.bind(stop=["\nSQLResult:"])
    | StrOutputParser()
)

full_chain = (
    RunnablePassthrough.assign(query=sql_response)
    | RunnablePassthrough.assign(
        schema=get_schema,
        response=lambda x: db.run(x["query"]),
    )
    | prompt_response
    | llm
)

# Invoke the full chain and print the final response
TheFinalResponse = full_chain.invoke({"question": ThePrompt})
print(TheFinalResponse)

@FaizelK Could you please share the original reference?

@igorschlum I think here is the source: https://github.com/langchain-ai/langchain/blob/master/cookbook/LLaMA2_sql_chat.ipynb

Yep, that is the source

hi @BruceMacD have you managed to implement this feature yet?

@FaizelK this probably isn't a feature we would build into Ollama directly, but people have built tools around this use-case now.
I haven't used this yet, but Vanna looks interesting:
https://vanna.ai/docs/other-database-ollama-chromadb/

Thanks for the issue! Yes I will close this for
#834
#3880

Hi, that is super, but, how can by than in javascript
I don t can
saludos


City/Country Gangrel?

Submitter: Peter Jarrette Gordon
Is ther any way to add the option for the alternative Gangrel lines?

City/Country Gangrel are such minor alternatives that I don't think I'll be adding them as separate entities. At that point I might as well be splitting Mariner Gangrel as well. You can add that as an addendum in your own PDF download, or in your character concept text box. Generally, I'd like to cater to everyone's tastes, but this tool is meant to make it less complicated, and I personally think it would be counter-productive to add this distinction, when considering that.

Since others have expressed interest in minor splinter bloodlines, I've reconsidered, and will look into adding an additional clan/bloodline filter that includes these lesser known groups and bloodlines.

Link issue with #226.


Side Panel Web Component w/ AI enhancements

Blocked by

https://github.com/carbon-design-system/ibm-products/issues/3752

Moving this to phase 2. Also, before this happens, Carbon for IBM Products needs to publish a styles package.

@lee-chase @jeffchew @ljcarot @jeanservaas I updated this issue to the @carbon/web-components package, as that's where it will be built (with the AI enhancements) during phase 2.

We can transfer this issue to https://github.com/carbon-design-system/carbon-for-ibm-dotcom prior to starting it.

Closing per @ariellalgilmore's recommendation.


IconButton default icon style not properly overriden by iconClassName

Description
Currently IconButton default icon style is not properly overriden by iconClassName property.
If I set following properties with closeIcon class overriding icon color's
<IconButton
   disableTouchRipple
   iconClassName={classNames('ion-md-close', classes.closeIcon)}
/>

I get instead, black color which is the default one.
Images & references
See how the element appears in the console :

and the corresponding css rules:

which leads to this render:

Versions

Material-UI: 0.16.4

That's expected, the inline-style has a higher specificity. We are referring to that point in the documentation:
You have two options.

Add !important when you need too.
Override the style using the inline-style.
Wait for the next branch that is moving away from the inline-style for that specific issue.

By the way, it's cool to see that convargo is using the lib. I have a friend working on truckfly.

Since MUI uses inline JS styles, you have to include '!important' with every CSS style declaration.
i.e.
.my-class {
  margin: 0 !important;
}

If you use JS inline styles (with the props provided like 'iconStyle') you won't have to use '!important'

@oliviertassinari Ok, I agree that inline-style has higher specificity than className has. But , I was expecting to have DEFAULT inline style overriden by className passed in props overriden by style passed in props as it seems to me that even if inline style is more specific it's DEFAULT behavior and props should always take precedence over the default style written inside components. But that's a matter of taste.
Nevertheless, I missed the doc where it was explained so, thanks for the reminder ;)
Aside: Yes, to be honest, we were using the lib for our POC and we are now moving away because of new homemade design, but it was nice and helped us to build quickly ! 👍

@chris-verclytte See ROADMAP.md on more info about the future of styling in this library.


fix: build

See ziglang/zig#18160 for details

Thank you, @usdogu! It took me longer than I'd like to admit to fix this on the PDK :laughing:


fix: remove the scale effect from button

We removed this on the main repo. It has a weird effect. What do you think?

Agreed 👍


Categorize regional/national public transit trains

Connects to #471
@zerebubuth, @nvkelso could you review please?
@nvkelso should we make a more specific sort_key entry for these national/regional trains compared to the other trains (without this service tag)?
I made a small update to the docs about when train features start appearing, but I didn't list the specific tag combinations for it. Could certainly update it to be more explicit if we want to do that.

:+1: looks good to me.

Let's leave the sort_key alone at this time. (If we need to adjust it, we can do that as part of https://github.com/mapzen/vector-datasource/issues/353.)


k is not defined

in index.js line 53:
"args[k]" : "k is not defined"
it should be "args[key]"

Please also update npm registry, I am using this npm.
Thanks a lot!

It's already updated.


Activity?

I wanted to ask about this package activity.

There are no tests so it isn’t clear how close this package matches the spec. Would you consider accepting PR which uses tests from node-url so we can be closer to implementation which majority of Webpack users is using?
Node has added few new methods to core url API such as fileURLToPath. It would be nice to have these methods available in this package.
Since ESM has gained traction, it would be nice to have proper exports in package.json.

I would be willing to create PRs for this if this package is still maintained and reviewed, but from what I currently see, there isn’t any new activity.

Hi @niksy ,
I was frustrated of the inactivity of the repo but I needed this package in a TS repo, so I forked it to another repo: https://github.com/louisremi/portable-url
I've already merged the two pending PRs of this repo and I'm welcoming new PRs
Best regards


Maximum call stack exceeded

Dependencies
"protractor": "5.1.2"
node v8.4.0
Error
(node:68426) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
[12:10:01] I/launcher - Running 1 instances of WebDriver
[12:10:01] I/direct - Using ChromeDriver directly...
Started
Spec started
F
  Application environment
    ✗ should have correct API url for production
      - Failed: unknown error: Maximum call stack size exceeded
  (Session info: chrome=61.0.3163.100)
  (Driver info: chromedriver=2.33.506106 (8a06c39c4582fbfbab6966dbb1c38a9173bfb1a2),platform=Mac OS X 10.12.6 x86_64)
      - Failed: unknown error: Maximum call stack size exceeded
  (Session info: chrome=61.0.3163.100)
  (Driver info: chromedriver=2.33.506106 (8a06c39c4582fbfbab6966dbb1c38a9173bfb1a2),platform=Mac OS X 10.12.6 x86_64)

Test
const APP_BASE_URL = require('../config').APP_BASE_URL;
const TEST_EMAIL = require('../config').TEST_EMAIL;
const protractor = require('protractor');
const open_listener = require('../common/http').open_listener;
const get_requests = require('../common/http').get_requests;
const remove_listener = require('../common/http').remove_listener;

class Interceptor {
	constructor(browser) {
		this._browser = browser;
	}

	add_listener() {
		return this._browser.executeScript(open_listener);
	}

	get_requests() {
		return this._browser.executeScript(get_requests);
	}

	remove_listener() {
		return this._browser.executeScript(remove_listener);
	}
}

const intercept = new Interceptor(browser);

describe('Application environment', function() {
	afterEach(function (done) {
		browser.executeScript(remove_listener);
	});

	it('should have correct API url for production', function() {
		browser.waitForAngular();
		browser.get(`${APP_BASE_URL}/home/login`);

		intercept.add_listener();

		element(by.css('input[type="email"]')).sendKeys(TEST_EMAIL);
		element(by.css('input[type="password"]')).sendKeys(<redacted-test-password>);

		element(by.css('button[type="submit"]')).click();

		intercept.get_requests().then((b) => console.log(b));
	});
});

Looks like the issue is get_requests. Digging around has lead me to believe there's an issue with serializing the information to send over to node. Currently unusable as is, though.

I also encountered the same error. I just tried the given sample script, with minor modifications i.e. I kept browser.get(url) before the function call and changed the element's xpath. While executing the code in Node.js using protractor-cucumber driver, system displayed this error. Any idea by when this issue will be fixed?

Is someone still maintaining this repo? I am getting "JavascriptError: javascript error: error reading property" using provided code sample.
(Session info: chrome=87.0.4280.88)
(Driver info: chromedriver=86.0.4240.22 (398b0743353ff36fb1b82468f63a3a93b4e2e89e-refs/branch-heads/4240@{#378}),platform=Mac OS X 10.15.6 x86_64)
node: 12.16.1
protractor: 7.0.0

Is someone still maintaining this repo? I am getting "JavascriptError: javascript error: error reading property" using provided code sample.
(Session info: chrome=87.0.4280.88)
(Driver info: chromedriver=86.0.4240.22 (398b0743353ff36fb1b82468f63a3a93b4e2e89e-refs/branch-heads/4240@{#378}),platform=Mac OS X 10.15.6 x86_64)
node: 12.16.1
protractor: 7.0.0

I got the same error, I don't think this is being maintained. This would have been the perfect solution... looks like I'll have to do more digging

Is someone still maintaining this repo? I am getting "JavascriptError: javascript error: error reading property" using provided code sample.
(Session info: chrome=87.0.4280.88)
(Driver info: chromedriver=86.0.4240.22 (398b0743353ff36fb1b82468f63a3a93b4e2e89e-refs/branch-heads/4240@{#378}),platform=Mac OS X 10.15.6 x86_64)
node: 12.16.1
protractor: 7.0.0

I got the same error, I don't think this is being maintained. This would have been the perfect solution... looks like I'll have to do more digging


added svelte.md svelte guide

NOTE:

Please fill out the template below

Did you read the instructions in the README?

Y/N  yes

Have you read the instructions in CONTRIBUTING?

Y/N yes

Have you written this article by yourself? Have you taken inspiration from anywhere and then written it in your words?

i wrote it myself with aid of my article in medium about svelte . i also referred svelte official site to write this guide

i made few changes , let me know if there are any issues.
thank you

Merging this PR. Great job @RizkyRajitha !

thank you @the-ethan-hunt


getAccountFromAddress not working if account not deployed

Since starknet-hardhat-plugin v0.7.0, when doing
starknet.OpenZeppelinAccount.getAccountFromAddress(address, privateKey);

the plugin performs an internal validation that the provided private key matches the deployed public key.
OZ account wrapper source code:
https://github.com/Shard-Labs/starknet-hardhat-plugin/blob/9e4809b54ef10b371994a30d655112aef42aa94d/src/account.ts#L506-L514
Analogously for ArgentAccount
This is a problem if the account contract hasn't yet been deployed (so there is no public key deployed to check against). When can this happen? Assume the following use-case:
const account = await starknet.OpenZeppelinAccount.createAccount(...);
console.log("Account address:", account.address);
// you exit the script to fund the account

Then a new script is run which first tries to get the account:
const account = await starknet.OpenZeppelinAccount.getAccountFromAddress(...);
// FAILS with reporting an UNINITIALIZED_CONTRACT

The current solution is to create and deploy account in the same script. You need to wait between these two steps. This can be achieved in a few ways:

sleep
wait for keypress

const account = await starknet.OpenZeppelinAccount.createAccount(...);
yourSelectedWayOfWaiting(); // fund during this step
await account.deployAccount(...);

The question remains whether we want to remove the public key validation or somehow allow the user to specify a flag indicating if the account is deployed or not.


Updating timeout to 5 min

Summary of the Pull Request
Updating time to 5 minutes to avoid function failures.
PR Checklist

[x] Applies to work item: #124
[x] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
Receiving error

Therefore increasing the timeout.
Validation Steps Performed
Autoscaling works. No timeout failures observed. But this will not solve storage account being not available.

Integration testing in progress (currently waiting on fresh build)


Candidate for new master

This PR proves that the state of #13 is identical to what I've done here: keep all commits, except explicitly revert the parts of #1 that change the source code.
I have more faith that this will merge cleanly into master than #13 — which I made by starting from an earlier commit and cherry-picking everything except for #1.
@mauricioszabo, my reasoning here is that

we proved that these code changes were (somehow) the source of renderer process crashes in Pulsar while running package and editor tests in CI;
as I understand it, the changes in #1 don't do much on their own to achieve compatibility with newer Electron and newer V8 — we'd want the more comprehensive N-API stuff present in #10.

Once we merge this, we can point Pulsar's package.json to our copy of superstring for the first time — as has been made necessary by the fact that we must explicitly build our own copy of libiconv on macOS.

Also happy to adopt the CI stuff recently suggested by @DeeDeeG, though I'm happy to back it out again if they think it should be part of a separate PR.

Failures on Node 18 (on all platforms) are not necessarily expected, but not surprising either.
Failures on macOS on Node 14 are acceptable — two specs are failing on an obscure encoding.
Landing this!

The Node 18 failures are from reverting the code we cherry-picked from NAN, unfortunately :/.


error when following README example

the line in the example:
var yahooStockPrices = require('../yahoo-stock-prices.js');

should be the following if installed with npm:
var yahooStockPrices = require('yahoo-stock-prices');

Thank you.
Changes have been reflected in NPM version 1.0.1

Hi, I did the above import, and try to call the function as shown in readme, however, it keeps giving me Module not found: Error: Can't resolve 'fs' in '/filepath.../node_modules/request/lib'. Do you know what is wrong with it? also I tried npm i request to fix it but no luck.

Hi, I did the above import, and try to call the function as shown in readme, however, it keeps giving me Module not found: Error: Can't resolve 'fs' in '/filepath.../node_modules/request/lib'. Do you know what is wrong with it? also I tried npm i request to fix it but no luck.


FAQ section in Dark Mode Resolved

The FAQ was in light theme even under the Dark Mode. Now, The Styling has been done and the Dark Mode is perfectly fine for the FAQ section.
Closes:
#2055 FAQ section is in light theme even under Dark Mode

@VanshGarg06 resolve the conflict


Roll Engine from 9520bb15b3d2 to 302fbf41b1d3 (3 revisions)

https://github.com/flutter/engine/compare/9520bb15b3d2...302fbf41b1d3
2021-06-21<EMAIL_ADDRESS>Revert "Issues/80711 reland (#26813)" (flutter/engine#26859)
2021-06-21<EMAIL_ADDRESS>Roll Skia from 9a4824b47c03 to 64751750f474 (1 revision) (flutter/engine#26851)
2021-06-21<EMAIL_ADDRESS>Roll Dart SDK from debd5e54b544 to 77bc0eb4b39a (1 revision) (flutter/engine#26853)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Wrong ViewBuilder triggered

I am triggering the event 'A'.'B'.'C'. (context.aggregate.event) and the ViewBuilder 'A'.'B'.'C' get correctly triggered.
But also the ViewBuilder D.E.C. ('C' is the same string but context and aggregate different).
I'm fully specifying the ViewBuilders  with name/aggregate/context in the definition like you showed me here .
I tried different name for the file with the same results. I also tried to put unrelated random names just to uncouple the definition from the name of the file.
Do you have any hints? Might it be a bug?

Have you defined the context and aggregate path here?
https://github.com/adrai/node-cqrs-eventdenormalizer#define-the-event-structure

No, or better, I did but not completely. Going to try right now, thanks again for the prompt answer.

That was it. Awesome. Thanks again.


Package for dbvisualizer 9.2.5

Two small issues and it's ready. Thanks!

Thanks I'll make those changes. Actually I should double check that license
anyway...
On Apr 9, 2015 7:35 AM, "Domen Kožar"<EMAIL_ADDRESS>wrote:

Two small issues and it's ready. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/7283#issuecomment-91233036.

When you commit to a PR we don't get notified, please always add a comment when you want a further review.

Looks fine, thanks.

Thanks, will do.
-Jeff
On Apr 13, 2015 5:34 AM, "lethalman"<EMAIL_ADDRESS>wrote:

When you commit to a PR we don't get notified, please always add a comment
when you want a further review.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/7283#issuecomment-92321673.


use addDeclaration function for varswithtypes

Use the existing addDeclaration function for the varsWithTypes declaration.
This has the advantage that the actual code does not generate yet at the start, so further changes can still be done. Also code reuse.
I add 'auto' here to indicate this was automatically determined.
Use case: for my custom declare blocks, I need to remove automatic variable declaration, if a declare block is present. By using addDeclaration this becomes possible, see: https://github.com/bmcage/ardublockly/blob/blockly4arduino/blockly/generators/arduino/declare.js
where the declare blocks can do:
  //remove previous possible automatic declaration
  delete Blockly.Arduino.definitions_["auto_" + value_name];

While I see your point, I think this is something that should be done on the block side instead of the generator, otherwise we will start running on issues where a block assumes a specific type that then gets overwritten, which can be specially problematic for the block warnings. The general philosophy has also been that a type is "set on first encounter", so having something like:
variable a = 10
variable a = "string"

Would declare int a and set a block warning on the second block.
On the other hand, you have really convinced me to add all general types to Ardublockly and try to utilise only the "generic ones" (the ones currently present) for the majority of the blocks, allowing specific declarations to be declared with the "as/cast" block, and having those types already present for any future blocks.

I agree to your roadmap, but this PR is a specific coding implementation, you should not block it if the approach is sound, because you don't like how I use it at the moment somewhere else.
I believe this PR is valid, as in that it does not make a source blob at the start which no longer can be updated, managed or changed. There will be other use cases that need that, like identifying that a variable should already have a value at the start.
In your approach, you will not touch these initializations anymore, which this PR allows. Existing code generation remains exactly the same. The PR does allow to do other things though too, even if ardublockly does not do it.
As to how I use this PR, with the declare blocks, I can do intelligent things too, like see that an declaration is already present, and give an error that the use of the variable goes counter the declare block present. So the PR allows to have declare blocks as valid blocks, even if you don't expose them in the GUI.

As an extra, for the declare blocks, I should give extra typing information as opposed to NUMBER, but that will only be possible once I see how you plan to extend the types for long integers. The declare blocks come first, so they should determine the type. The difference is that they also determine a value already, and I should not end up with
int i;
int i = 5;
as that gives a compile error. So that is the reason I delete the auto genereted type again at the moment. Ideally, the type determinator can determine the type and default value from a declare block, and then the generator indeed is not needed anymore, and things can be done on the block side.
So, a way to not only infer the type, but also the default value. In C it is good practice after all to always assign a value :-)

Well, if we are going to keep variables in a "associative array"-type object then it should probably be separated from normal definitions and declared the same way as includes_, setups_, pins_, etc . Having the variable name as the "key" and C code declaration as the "value", that way there is no need to use a prefix.
I'm not a big fan of accessing the data directly from the block generators, since it locks the current implementation, but adding accessors to set or edit variable declaration code would really be promoting its use, which can lead to some of the ill effects I mentioned before.
While you are right about assigning a value on declaration (although technically all globals are zero-initialised), the Ardublockly generated code is meant to be simple even if that means compromising on code quality or efficiency. If you have a block to set "varaible item = 5" and it generates the code below, is not clear why it is first set to 0 and then to 5:
int item = 0;
item = 5;

And having the blocks determine the default value to generate int variable = 5 directly seems like a feature that might not add real value, while having quite a few corner cases to consider.
Simplicity is also why I am not a big fan of the "declare" blocks, while it follows the same logic that the C static typing has, we have quite a few cases of blocks that allow you to use variables without explicitly declaring them first with another block. I much prefer a system where you can just drop variable and start using them and then let Ardublockly define them for you.

Back to the actually changed code.
I don't find it ok that the init code has already.
Blockly.Arduino.definitions_['variables'] = variableDeclarations.join('\n');
that should be in the finish. So storing in the meantime, be it with the function I used in this PR or some other way, and in finish then building the actual code is what I advocate.
The comment of the function finish does indicate that that is it's use:
 * Prepare all generated code to be placed in the sketch specific locations.

which is why I was suggesting to do something along the lines of:
for (var varName in varsWithTypes) {
    Blockly.Arduino.variables_[varName] =
        Blockly.Arduino.getArduinoType_(varsWithTypes[varName]) + ' ' + varName + ';';
}

And then join the items at the finish state.
This is also why I felt that accessor to add or edit a variable (in the arduino generator)was better than accessing the object directly. Imagine in the future we do implement the "set value on declaration" functionality, that would probably require to change how the Blockly.Arduino.variables_ is designed and would therefore break any block that access that object directly; but then again, if a developer access or edits a "private" object, shouldn't be surprise if it breaks in the future.

Ok, all clear now. I'll let you work it out and close this PR. In case I can do something, let me know.

I'm happy to do this change today, unless you'd prefer to update this PR yourself :)

Travelling ... so go ahead

Implemented in https://github.com/carlosperate/ardublockly/commit/0bd25a4751a2c18b98fc4e2ddf60502c5b5232e3 , so now there is a provided Blockly.Arduino.addVariable() function that can add a new variable to the dictionary with the optional argument to overwrite an existing declaration.


Fix overridden method skip causing arity failures

This was causing this error in Ci:
  1) Chef::Provider::User::Useradd action :create when the user does not exist beforehand and the username contains a single quote ensures the user exists
     Failure/Error: skip(reason)
     ArgumentError:
       wrong number of arguments (1 for 0)
     # ./spec/functional/resource/user/useradd_spec.rb:172:in `block (4 levels) in <top (required)>'

@chef/client-core

lol :+1:

(works fine on my ubuntu box)


support multi-line image urls

I have been using Gravizo (http://gravizo.com/) in my markdown documents to allow me to incorporate diagrams.
This works great when I use the html / image format ..... but not the markdown format .... I suspect its because pegdown doesn't like multi-line image urls.
Is there any way that pegdown could be updated to support / render multi-line markdown image urls?
E.g. update pegdown to read the following image url such that it calls out to gravizo and displays the image correctly
 ![Alt text](http://g.gravizo.com/g?
  digraph G {
    aize ="4,4";
    main [shape=box];
    main -> parse [weight=8];
    parse -> execute;
    main -> init [style=dotted];
    main -> cleanup;
    execute -> { make_string; printf}
    init -> make_string;
    edge [color=red];
    main -> printf [style=bold,label="100 times"];
    make_string [label="make a string"];
    node [shape=box,style=filled,color=".7 .3 1.0"];
    execute -> compare;
  }
)

![Alt text](http://g.gravizo.com/g?
digraph G {
aize ="4,4";
main [shape=box];
main -> parse [weight=8];
parse -> execute;
main -> init [style=dotted];
main -> cleanup;
execute -> { make_string; printf}
init -> make_string;
edge [color=red];
main -> printf [style=bold,label="100 times"];
make_string [label="make a string"];
node [shape=box,style=filled,color=".7 .3 1.0"];
execute -> compare;
})

@richardwthompson, if you need this feature, I have it implemented in a parser I rewrote from
commonmark-java to replace pegdown in my
Markdown Navigator plugin for IntelliJ IDEs: https://github.com/vsch/idea-multimarkdown. The
parser project is https://github.com/vsch/flexmark-java has very detailed source based AST with
source offset for every part of the element.
It is CommonMark 0.27 (GitHub Comments) compliant but has parser configuration options to
emulate list indentation rules used by: markdown.pl, MultiMarkdown (like pegdown 4 space
indents) and kramdown (GitHub Docs). The only extensions that pegdown has that I did not yet
implement are: typographic quotes, smarts and definition lists. The rest of the extensions are
available, with some extra ones that pegdown does not have.
As an added bonus and what motivated me to switch the parsing is 30-50x faster than pegdown on
average documents and several thousand times faster on pegdown's pathological input like
[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[.


🎨 Buymeacoffee widget overlaps 'move to top'

Describe the bug
The Buymeacoffee widget on the lower right overlaps the 'move to top' icon in mobile view.
To Reproduce
Steps to reproduce the behavior:

Make sure youre in mobile view or on a phone browser.
Go to any page that has the move to top icon and the Buymeacoffee widget
Scroll down to have the 'move to top' icon appear, the widget will overlap the move to top.

Expected behavior
If the widget is enabled and placed on the right, it should not overlap the 'move to top'
Desktop (please complete the following information):

OS:Android
Browser: Firefox
Version: 112.2.0

Hugo & Blowfish versions
Hugo v0.111.3+extended linux/amd64 BuildDate=unknown
Blowfish v2.32.1

Thanks @rpstreef, do you have a proposed solution?

I'd propose to move the 'move to top' above the buymeacoffee widget in mobile view.
Considering the view is already fairly narrow, side positioning would leave less content room. It will either have to be above it or under it.

It's hard because they are in different "layers" - will explore your idea though.

the only other consideration is flip-flop. When coffee icon is left, move to the top is right etc.

"flip-flop" seems like a bad solution... let me see if I (or someone in the community) comes up with a better idea

In that case you already have the answer to the solution,  it's merely an implementation problem regarding the layering

another option would be to reduce the size of them when displayed together... but what do I know ;)


Error: plugin "registry" exited with error

I am trying to install  quay.io/coreos/alb-ingress-controller-helm running into Error

helm registry list quay.io | grep alb

quay.io/coreos/alb-ingress-controller-helm                     0.0.10                                          -            helm
quay.io/mgoodness/alb-ingress-controller-helm                  0.0.10                                          -            helm

helm registry install quay.io/coreos/alb-ingress-controller-helm

usage: appr [-h]
            {pull,run-server,show,inspect,list,delete-package,helm,version,logout,deploy,plugins,push,login,config,channel,jsonnet}
            ...
appr: error:
message: 'Error: no available release name found

Any help is appreciated

I don't know it seems an error reported from helm. Is it working with other app ? like
helm registry install quay.io/ant31/cookieapp

same error with cookieapp as well

there's something with your helm configuration. can you try to helm install without the registry?
are you using install or upgrade cmf?


source for 13.9 version + copy of dll for clueless ppl

https://github.com/i4TsU/R3nzSkin-13.9
fork of main branch, only change is to the sig for dx9 device that i posted earlier (just hurts my soul seeing all the dodgy dlls ppl r downloading). also uploaded a copy of the dll for those that are too lazy to build it or dont know how - just download the R3nzSkin.dll file and replace the .dll you use currently with this one

how do i use this? i downloaded r3nzskin.dll only and put it into /game but it doesnt work - do i need to download everything or?

also on main "r3nzskin" one it says "13.9" update so i guess they're updating it now regularly or?

nevermind, the main one works too so i guess ill be using that but i still dont understand - when they said they will stop with the updates bcs the guy went on military duty, when random people were posting their own .dll files, i could just put that dll in /game and it would pop up, but with main one i have to open injector, click start and then wait for it to open in game - bcs when i put the .dll file from main one it didnt work


Hi！

Here are some warning and error logs

It seems u are in China, which cannot get stable connection to steam.
Please retry or use a VPN.

I'm in China, thanks, I'll try using a vpn
------------------ 原始邮件 ------------------
发件人:                                                                                "oureveryday/Steam-auto-crack"                                                                                @.>;
发送时间: 2023年12月31日(星期天) 晚上11:39
@.>;
<EMAIL_ADDRESS>主题: Re: [oureveryday/Steam-auto-crack] Hi！ (Issue #48)
It seems u are in China, which cannot get stable connection to steam.
Please retry or use a VPN.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


capabilities:delete command does not ask for capability when none specified

The capabilities:delete command is no longer asking the user for a capability when none is specified on the command line.

Fixed by https://github.com/SmartThingsCommunity/smartthings-cli/pull/92


Update PULL_REQUEST_TEMPLATE.md

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

https://github.com/github/opensource.guide/pull/2260#issue-592433874

Sorry I am not as good at this as you


Consider to change the default behaviour of run

The method run captures the command line arguments.
This behaviour results in confusion on a regular basis (see e.g. https://github.com/gtk-rs/gtk4-rs/issues/1039).
I propose to stop capturing the command line arguments in run and require users to capture the arguments themselves and pass them to run_with_args.

Well that's how it was until a year ago: https://github.com/gtk-rs/gtk-rs-core/commit/e49d8f6768bd763acfcb9a2343b5f792a7104a7d
People were complaining about the exact opposite of what you're saying now. So what should we do? :)

PR for this was https://github.com/gtk-rs/gtk3-rs/pull/421
CC @pbor @bilelmoussaoui @GuillaumeGomez

We should provide a run_no_capture so everyone is happy. :laughing:

We should provide a run_no_capture so everyone is happy. 😆

I think the main problem is that people find it hard to figure out why they get an error message when running their apps with arguments. So that wouldn’t help much.

The amount of people that have to parse the arguments manually instead of going through glib is pretty low. So I would rather keep the situation as is.

The current situation is also closer to how the Python and GJS bindings work. What is different there that doesn't make this a problem for people?

The current situation is also closer to how the Python and GJS bindings work. What is different there that doesn't make this a problem for people?

Good question, I don't have much experience there.
Do you interact with those communities enough to know that it makes no problems there?
Maybe it could be improved by choosing better names for the methods: people might be confused that run_with_args is the one you use if you don't want argparsing be handled automatically.
And/or better doc visibility: on gtk4-rs' docs neither run nor run_with_args display any docs.
You have docs on gio but they only make little sense for our bindings.

Do you interact with those communities enough to know that it makes no problems there?

I am in both gjs/pygobject Matrix channels and haven't seen much complaints about this.

So let's close this then?

Yes, I would say so


A error was thrown when leave animated widget

Steps to Reproduce
I have this widget, which is animated all time. When i leave page, which has this animated widget - flutter throw an error. If i come to page, which have an animated widget - it's okay working next and i don't have any problem with this, but i write this issue, maybe it's a really bug.
import 'dart:math';

import 'package:flutter/material.dart';
import 'package:wave/config.dart';
import 'package:wave/wave.dart';

class Wave extends StatefulWidget {
  final Duration duration;
  Wave({this.duration});
  @override
  _WaveState createState() => _WaveState(duration: duration);
}

class _WaveState extends State<Wave> with TickerProviderStateMixin {
  Duration duration;
  _WaveState({this.duration});
  AnimationController _aController;
  List<Animation> _waveTweens = [];
  List<Color> _waveColors = [
    Colors.white,
    Colors.white,
    Colors.white,
    Colors.white,
    Colors.white,
    Colors.white,
    Colors.white,
    Colors.white,
  ];

  int getRandomHex({int max =<PHONE_NUMBER>}) {
    return Random().nextInt(max);
  }

  void initAnimationController() {
    _aController = AnimationController(vsync: this, duration: duration);
  }

  void initTweens(List<Color> colors, List<Color> oldColors) {
    int index = 0;
    _waveTweens = oldColors.map((Color oldColor) {
      var tween = ColorTween(begin: oldColor, end: colors[index]).animate(_aController);
      index++;
      return tween;
    }).toList();
    _waveColors = colors;
  }

  List<Color> getRandomColors() {
    var colors = _waveColors.map((Color color) {
      var colorValue = getRandomHex();
      var opacity = getRandomHex(max: 100) / 100;
      var color = Color(colorValue).withOpacity(opacity);
      return color;
    }).toList();
    return colors;
  }

  @override
  void initState() {
    _waveColors = getRandomColors();
    if (duration == null) {
      duration = Duration(seconds: 2);
    }
    initAnimationController();
    super.initState();
    animateWave();
  }

  @override
  void dispose() {
    _aController.value = 1;
    _aController.stop();
    _aController.dispose();
    _aController = null;
    super.dispose();
  }

  void animateWave() async {
    initTweens(getRandomColors(), _waveColors);
    if (_aController != null) {
      _aController.forward(from: 0);
    }
    await Future.delayed(duration);
    animateWave();
  }

  @override
  Widget build(BuildContext context) {
    return AnimatedBuilder(
      animation: _waveTweens[0],
      builder: (context, child) {
        return WaveWidget(
            config: CustomConfig(
              gradients: [
                [_waveTweens[0].value, _waveTweens[1].value],
                [_waveTweens[2].value, _waveTweens[3].value],
                [_waveTweens[4].value, _waveTweens[5].value],
                [_waveTweens[6].value, _waveTweens[7].value],
              ],
              durations: [48000, 24000, 12000, 6000],
              heightPercentages: [0, 0.1, 0.2, 0.6],
              blur: MaskFilter.blur(BlurStyle.solid, 15),
              gradientBegin: Alignment.bottomLeft,
              gradientEnd: Alignment.topRight,
            ),
            size: Size(double.infinity, 300));
      },
    );
  }
}

Logs
Performing hot restart...
Syncing files to device Android SDK built for x86...
Restarted application in 1 592ms.
I/flutter ( 4991): null
I/flutter ( 4991): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter ( 4991): The following assertion was thrown while finalizing the widget tree:
I/flutter ( 4991): 'package:flutter/src/widgets/framework.dart': Failed assertion: line 3485 pos 12:
I/flutter ( 4991): '_debugLifecycleState != _ElementLifecycle.defunct': is not true.
I/flutter ( 4991):
I/flutter ( 4991): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter ( 4991): more information in this error message to help you determine and fix the underlying cause.
I/flutter ( 4991): In either case, please report this assertion by filing a bug on GitHub:
I/flutter ( 4991):   https://github.com/flutter/flutter/issues/new?template=BUG.md
I/flutter ( 4991):
I/flutter ( 4991): When the exception was thrown, this was the stack:
I/flutter ( 4991): #2      Element.markNeedsBuild (package:flutter/src/widgets/framework.dart:3485:12)
I/flutter ( 4991): #3      State.setState (package:flutter/src/widgets/framework.dart:1138:14)
I/flutter ( 4991): #4      _ScannerState.dispose (package:quich/screens/scanner.dart:20:5)
I/flutter ( 4991): #5      StatefulElement.unmount (package:flutter/src/widgets/framework.dart:3916:12)
I/flutter ( 4991): #6      _InactiveElements._unmount (package:flutter/src/widgets/framework.dart:1696:13)
I/flutter ( 4991): #7      ListIterable.forEach (dart:_internal/iterable.dart:39:13)
I/flutter ( 4991): #8      _InactiveElements._unmountAll (package:flutter/src/widgets/framework.dart:1705:25)
I/flutter ( 4991): #9      BuildOwner.finalizeTree.<anonymous closure> (package:flutter/src/widgets/framework.dart:2359:27)
I/flutter ( 4991): #10     BuildOwner.lockState (package:flutter/src/widgets/framework.dart:2191:15)
I/flutter ( 4991): #11     BuildOwner.finalizeTree (package:flutter/src/widgets/framework.dart:2358:7)
I/flutter ( 4991): #12     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding&WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:702:18)
I/flutter ( 4991): #13     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:275:5)
I/flutter ( 4991): #14     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:990:15)
I/flutter ( 4991): #15     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:930:9)
I/flutter ( 4991): #16     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:842:5)
I/flutter ( 4991): #20     _invoke (dart:ui/hooks.dart:209:10)
I/flutter ( 4991): #21     _drawFrame (dart:ui/hooks.dart:168:3)
I/flutter ( 4991): (elided 5 frames from class _AssertionError and package dart:async)
I/flutter ( 4991): ════════════════════════════════════════════════════════════════════════════════════════════════════

No issues found! (ran in 2.8s)

[√] Flutter (Channel dev, v1.2.2, on Microsoft Windows [Version 10.0.17763.316], locale ru-RU)
    • Flutter version 1.2.2 at C:\flutter
    • Framework revision 007a415c2a (2 days ago), 2019-02-21 20:22:47 -0800
    • Engine revision f1f19bba8f
    • Dart version 2.2.0 (build 2.2.0-dev.2.1 c92d5ca288)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\alpha\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 33.1.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] VS Code, 64-bit edition (version 1.30.2)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 2.22.1

[√] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

I have the same problem。I wrote a animation,when I leave the page that flutter throw a error:
'_debugLifecycleState != _ElementLifecycle.defunct': is not true.

What is the solution?

i guess solutions was  to just closing issue


Query#where broken?

I can't get Neo4j::Server::CypherNode#set_label to work. I investigated and found that the private method match_start_query returns this regardless of the actual node id:
"MATCH n WHERE (ID(n) = {ID_n}) WITH n"

Then I looked into where, which is invoked in match_start_query and found that:
Neo4j::Session.query.match(:n).where(:n => {name: 'joe'}).to_cypher
# => "MATCH n WHERE (n.name = {n_name})"

This doesn't seem like it could be right. Why does 'joe' not appear in the generated query? This appears to be the root of the problem for set_label.

The Query API automatically uses Cypher parameters in queries to help with performance and to protect against injection attacks.  You can see the parameters like so:
query = Neo4j::Session.query.match(:n).where(:n => {name: 'joe'})
query.to_cypher # Gives you the Cypher
query.send(:merge_params) # Weird, I know

If you run the query MATCH n WHERE (n.name = 'joe') in your Neo4j console does it work?

Yes, that query works-- I understand about the params, but set_label still seems broken. In fact this seems related to another issue that I've run into.
The general problem is that some methods that update the state of a Neo4j::Node or Neo4j::Relationship don't seem to show their result on that object immediately. I can see the updated state of the object if I run a query to retrieve the same record from the database. I am seeing this with both Node#set_label and Relationship#update_props, but not with Node#update_props. Is it supposed to function this way? I don't see anything in the docs to indicate it, and I don't think I encountered this when using this library in the past.
To see what I'm talking about, try:
require 'neo4j-core'

neo = Neo4j::Session.open(:server_db)
x = Neo4j::Node.create({name: 'joe'}, :testlabel1)
puts x.labels
x.set_label(:testlabel2)
puts x.labels

y = neo.query.match(:n).where(:n => {name: 'joe'}).return(:n).first[:n]
puts y.labels

x.del
y.del

I see:

testlabel1
testlabel1
testlabel2

After running back through a bunch of versions, I found that Neo4j 3.0.0 produces what I would expect:

testlabel1
testlabel2
testlabel2

And throws an exception on the y.del line, also what I would expect, since x and y should be the same.

Ah, I see.  Yeah, that's probably a bug.  We can definitely look at that.
There's another thing I'm struggling with:
I'm currently building a new API which is focused on doing Cypher queries.  In the current neo4j-core gem you can do things via Cypher queries but there is also stuff on top to work with the Java APIs provide conveniences like this.
In the new API what I've been focused on is the task of being able to provide a Cypher queries string (and optionally parameters) and getting back results as objects.    If you look SQL / Mongo gems (I think) you're just getting the ability to query and to get objects back.
The ActiveNode / ActiveRel modules seem to be a place to provide this functionality (like creating/updating/deleting), but obviously it's somewhat heavier than neo4j-core.  We're working on making it lighter currently (part of that is my neo4j-core API work and part of that is some stuff that @subvertallchris is working on).
But maybe it would make sense to have another gem that doesn't have a lot of the fancy stuff that ActiveNode / ActiveRel has.  Or maybe we should just really dig it and slice into the performance problems.  The only thing that is perhaps a bit odd is that in ActiveNode you have a mapping of labels to classes, so if you do things like add_label/remove_label (which don't yet exist there) then it should potentially be turned into an object of another class.  But maybe that's still a good place to do it.
Thoughts?

Thanks for looking into it Brian.
As for the new API, I actually like the current neo4j-core API. I looked into ActiveNode/ActiveRel a while ago and found it didn't quite suit my needs (something about the way it dealt with labels), and I wanted something that functioned as you described-- allowed me to run Cypher queries and get back simple objects, without layering additional abstractions (i.e. new classes) on top of my objects. Neo4j-core fit the bill.
But I guess I'm a little unclear on what your new API is supposed to do. As you say, neo4j-core at the moment lets you work with queries, but also has some other stuff. Are you just extracting the query-centered parts of Neo4j-core, or doing something different? Also, when you say "just getting the ability to query and get objects back", are you saying that the returned objects have no capacity to write the database?

Yeah, the new API is basically just there for making Cypher queries.  It uses an adaptor pattern so when Bolt comes around in the near future it will be easy for us to throw in support for it.  The current code is pretty harry for a few reasons, hence the new one.  It should be pretty straightforward, still.  Currently it's like;
adaptor = Neo4j::Core::CypherSession::Adaptors::HTTP.new('http://localhost:7474')

session = Neo4j::Core::CypherSession.new(adaptor)

result = session.query("MATCH (n) WHERE n.prop = {prop} RETURN n LIMIT 10", prop: value)

# or to make multiple queries with one request:

results = session.queries do
  append "MATCH (n) WHERE n.prop = {prop} RETURN n LIMIT 10", prop: value
  append "MATCH (n) WHERE n.prop = {prop} RETURN n LIMIT 10", prop: value
end

And yeah, at the moment the idea is that the raw data from those queries will be wrapped into Node/Relationship/Path objects like they are currently.  It's just that those objects don't have any methods which will change the database or reflect changing state.  I'm trying to figure out if it's a good idea to have those or not.  It will even be that much harder to maintain those extra methods once we have the third adaptor for Bolt.
If you remember what you didn't like about ActiveNode  / ActiveRel I'd love to hear about it.  It may be something that we can fix

I see-- what you are implementing seems like it would be very useful as the basis for the Neo4J adaptor for ROM. In ROM read operations return immutable Relations, which are basically just sets of hashes. Write logic is completely separate. I am kind-of sort-of working on that adaptor right now, but I've been held up because:

ROM just hit 1.0 and the docs up to now have been pretty awful
I wanted the ability to return plain hashes from Cypher queries in Neo4j core, and I haven't yet figured out how from my exploration of the source
Life

I actually think that to make your new API maximally useful, you should avoid new classes altogether and simply return Hashes. Something like this:
res = session.query("MATCH (n)-[r]-() WHERE n.name = 'joe' RETURN n,r")
res.first
# => {
#      "n" => {
#        "id" => 43426,
#        "labels" => [ "label1", "label2" ]
#        "props" => {
#          "name" => 'Joe',
#          #  rest of props...
#        }
#      },
#      "r" => {
#        "id" => 23512,
#        "type" => "reltype1",
#        "props" => {
#          # props here...
#          }
#      }
#    }

At the very least, I hope there is an option to get this sort of result. I guess what I'm hoping for is a very thin client for Neo4J's REST interface.
Also, what is Bolt?

Ah, that "Life" is always a big reason ;)
I'm remembering now issue #223 .  I think you'll be glad to know that as part of my refactor I put in an option to control how objects are being wrapped.  You can see the specs for it here:
https://github.com/neo4jrb/neo4j-core/blob/new_cypher_api/spec/neo4j/core/shared_examples/adaptor.rb#L147
The wrap_level option takes a symbol.  Either :none (plain hashes), :core_entity (the objects that I've been talking about above), or :proc (which allows you to define a wrapper_callback Proc on Node/Relationship/Path and define how the objects are wrapped.  This will be used in the neo4j gem to wrap ActiveNode and ActiveRel models.
Bolt is the new protocol for communicating with Neo4j.  It will be coming out with version 3.0 of Neo4j.  It is socket-based and uses a compressed format called PackStream (similar to MessagePack).  It should help increate the performance of Neo4j connections.  You can read about it here:
http://alpha.neohq.net/docs/server-manual/bolt.html
I'm hoping that we can have full support for Bolt when Neo4j 3.0 comes out.


fix: add ts-prune package 🥷

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


Add structure for multiple Rust crates

Summary
This is a precursor to adding an entirely separate and self-contained crate to handle parsing of OpenQASM 2 (not 3 - this is kind of like a trial run for that).  It could conceivably still live within qiskit._accelerate, but having separate crates helps us enforce more sane API barriers, and has improvements in the incremental build time when working on only one of the Rust extensions.
The intent after this commit is still to have qiskit._accelerate as an easy catch-all for accelerators for Python.  Developers should not need to be concerned with defining a new crate for every new feature, and for the most part _accelerate is still logically interoperative within itself (for example, NLayout is used all over).  This is just laying out the groundwork so more complex crate additions can also be made.
Some of the niceties to do with Cargo workspaces only became stabilised in Rust 1.64.  In particular, inheritance from the workspace root for things like the package version, Rust version, and dependencies only came in then.  For now, the workspace.packages key in the root Cargo.toml is ignored with a small warning during the build, but I've put it in place mostly to keep track of what we can change once the MSRV becomes 1.64 or greater (likely not til at least Terra 0.26).

Details and comments
In particular, I'm planning to merge https://github.com/jakelishman/qiskit-qasm2 to be the primary OQ2 importer for Terra, since it's like 10x faster and lets us drop dependencies on ply (lex and yacc), which were always largely intended to be teaching projects and will not be providing fixes as installable PyPI packages moving forwards.
Since I moved all the Rust files, this may cause a break in the coverage tracking, but none of the actual Rust code has changed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 85.36%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
68043

Relevant Lines:
79713

💛  - Coveralls

John: this structure isn't specific to only Rust extension crates.  It can be any separate-binary crates.

I added a little README for qiskit._accelerate in 76e6496.  I can follow this PR with one that adds a bit more documentation to CONTRIBUTING.md about how we use Rust in general - I think that's probably a slightly separate PR.


fix: multiple of same package appears in search results

Fixes #585
Instead of showing the major version of a package (or major version of the CDK framework) to the package cards, we filter out packages appearing multiple times with different major versions from search results, always preferring to show the highest major version of the package that satisfies the search filters.
For example, if you search for @aws-cdk/cx-api, by default only one card is shown, that links to @aws-cdk/cx-api@2 that supports CDK v2. But if you apply a filter for AWS CDK v1, then @aws-cdk/cx-api@1 is shown.

Several tests needed tweaking since they tend to assume that no de-duplication goes on. Initially I thought that was OK, but now I'm realizing that for #679 I will actually need to have a way to retrieve un-de-duplicated results, so I'm going to switch "dedup" into being an optional search flag.

@Chriscbr I think it makes sense to dedup on search by default with a flag to disable it, but IMO it would be nice to have a method on the search such as findByName which could just return a list or map of packages that exact match the name

@gabewomble Good suggestion, I think I'll add a method like that in my other PR.

@eladb @gabewomble Is there anything blocking this UX fix?

Nothing on my end 👍


Potential bug in dump schema validation

There probably shouldn't be a space in the string " image":
https://github.com/mphowardlab/lammpsio/blob/d04d5dc93e14f06daa3a3e85fab80993e1c074b4/src/lammpsio/dump.py#L326

Thanks! Almost definitely not, but you have an example where this caused an issue that we can use to make a test?

It didn't cause any errors for me - I think this only has an effect with faulty dump files, in which case a different error message than intended is shown.
Here's an example I've quickly put together. Consider the following Python code:
import lammpsio
traj = lammpsio.DumpFile('dump.foo')
for snap in traj:
    pass

If run using the following dump with a missing z column:
ITEM: TIMESTEP
0
ITEM: NUMBER OF ATOMS
1
ITEM: BOX BOUNDS pp pp pp
-7.0 7.0
-7.0 7.0
-7.0 7.0
ITEM: ATOMS id type x y
1 1 0 0

The following error occurs, as intended: OSError: lammpsio requires 3-element vectors
If run using the following dump with a missing iz column, however:
ITEM: TIMESTEP
0
ITEM: NUMBER OF ATOMS
1
ITEM: BOX BOUNDS pp pp pp
-7.0 7.0
-7.0 7.0
-7.0 7.0
ITEM: ATOMS id type ix iy
1 1 0 0

This error occurs: TypeError: list indices must be integers or slices, not NoneType.
If the space in " image" is removed, the intended error message is shown.


Does druid support cumulative sum aggregation?

Does druid support cumulative sum aggregation? Like this，ElasticSearch，https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-pipeline-cumulative-sum-aggregation.html.
Especially hyperUnique.

Not in core (yet) but the moving average extension could in principle be extended to support it: https://druid.apache.org/docs/latest/development/extensions-contrib/moving-average-query.html
I'd definitely be excited to see this functionality in core at some point.


Add support to create AWS::EC2::VPCPeeringConnection through the CloudFormation interface

It was missing an entry at MODEL_MAP.

Coverage increased (+0.05%) to 95.6% when pulling ab0f95853926922ea49b422ee800e1d95bb24abf on add-cloudformation-vpc-peering-connection into a53312ee689ee071763381c31a4e8423d703f258 on master.

Coverage increased (+0.05%) to 95.6% when pulling ab0f95853926922ea49b422ee800e1d95bb24abf on add-cloudformation-vpc-peering-connection into a53312ee689ee071763381c31a4e8423d703f258 on master.

Thank you!


Update the Egress - IPVS interoperability issue in documents

The issue was fixed in Antrea v1.7.0. Update the Egress and Servic
Load-balancing documents for that.
Signed-off-by: Jianjun Shen<EMAIL_ADDRESS>

/skip-all


[nrf noup] Set BLE connection to 1 for NUS

We decided to work with a single BLE connection when NUS is active.

Are there any changes in the source code required because of that?

No, just Kconfigs.


Change Appget installation directory(not specific package but Appget itself)

What I mean is before installing Scoop I can run this
$env:SCOOP='C:\My Programs\Scoop'
[environment]::setEnvironmentVariable('SCOOP',$env:SCOOP,'User')

And for Chocolatey
$env:ChocolateyInstall='C:\My Programs\Chocolatey'
[environment]::setEnvironmentVariable('ChocolateyInstall',$env:ChocolateyInstall,'User')

This means that every package will now install in that directory. There is no option to change AppGet installation directory.

Would also be great if it would work with this.
https://gist.githubusercontent.com/kayone/16f102466a3a91438921cb79fecfc889/raw/appget.ps1

I wonder if I should just copy the entire AppGet folder to a new location.


'Between' option for Slicer settings not showing

Going to Slicer Visualization > Format > Slicer Settings > Options > Style does not have a 'Between' option. Only Vertical List, Tile, and Dropdown.

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

ID: 4bc558f3-8a74-7154-c048-2dd74ad4eb8a
Version Independent ID: f84aa1a5-4bab-6f6d-a0ba-5040c0f5defb
Content: Create a numeric or date range slicer in Power BI - Power BI
Content Source: powerbi-docs/create-reports/desktop-slicer-numeric-range.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Apologies, field [that is suppose to be a date field] seems to be a text field.


Add DatabaseMetadata Support

SQLDatabaseMetadata
java.sql.SQLFeatureNotSupportedException
:at [com.github.housepower.jdbc.wrapper.SQLConnection getMetaData SQLConnection.java

Well, I saw this issue yesterday on https://github.com/metabase/metabase/issues/3332.
We are trying to add this support.

thank you..

Hi, has any progress been made on this?
The Yandex driver has a fairly good starting point of this but I have no experience relevant to contributing (even though I would love to help)

Me too, it's a little complicated to understand all the functions in DatabaseMetaData  interfaces.
Maybe someone could help to give a better pr of this issue?

@sundy-li   https://github.com/yandex/clickhouse-jdbc/blob/master/src/main/java/ru/yandex/clickhouse/ClickHouseDatabaseMetadata.java
This source code may helps.


Search: Remove Deprecated ElasticSearch and OpenSearch1 Support

:heart: Thank you for your contribution!
Description

Remove deprecated ElasticSearch support
Remove deprecated docker version
Update deprecated docker-compose

Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings.
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

If you have added third-party code (copy/pasted or new dependencies), please reach out to an architect.

Frontend

[ ] I've followed the CSS/JS and React guidelines.
[ ] I've followed the web accessibility guidelines.
[ ] I've followed the user interface guidelines.

Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Thanks @Samk13!  Looks ready to go whenever someone with merge rights stops by. I think it should also be backported to the v12 branch.

Thanks @Samk13 ! I am merging and backporting.


Use WebAssembly in the future.

I was scrolling through /r/python when I read this interesting article. Essentially, WebKit will begin supporting binary blobs of C/C++ code. Therefore, I think PyPyJS should use WebAssembly, once it releases (still pre-release). To quote the article:

it’s much faster for the engine to decode the binary format (up to 23x faster in the current prototype) than parsing asm.js code, for example.

A 23x speedup sounds pretty nice to me, and we are already generating C++ files from the rpython bit, right?
And finally, it won't require much porting (taken from the article):

and developers will be able to use the same Emscripten tool for WebAssembly as they use for compiling asm.js code today

All in all, good news! Do you think PyPyJS will target WebAssembly?

This was already opened: #145.

Think there are some nice information. So i copy&paste it to https://github.com/pypyjs/pypyjs/issues/145#issuecomment-113192334 and will close this here...

A 23x speedup sounds pretty nice to me, and we are already generating C++ files from the rpython bit, right?

s/C++/C/


Option to add color and increase indentation in bulleted lists

Hi! I'm really enjoying the latest additions to the theme. And the "carded" layout to the theme really rocks.
One thing that could be improved in my opinion are bulleted (and numbered) lists. In Live Preview mode, the lack of enough indentation to the right (and also lack of padding room in relation to the paragraphs that immediately preceded them above) makes it hard to immediately discern bullets from the rest of the text.
These are the things which I think would help improve things:

Add an option to increase bullet indentation to the right side.
Add different decoration (different symbols) between bullet levels, making it a bit more like MS Word.
Add an option for bullets and numbered list numbers to have some color. Either inherited from the accent color (a more vibrant variation of the tone, to make them as visible as possible) or simply allowing the user to customize it, for light and dark theme alike.

The following is an example of the things we can do in MS word, which facilitate visual discernment. I believe that in Border Theme things could look even better, with better vertical lines to separate different bullet levels.

Thanks in advance, and keep up with the excellent work! Happy New Year too

Hello, some relevant options have been added to Style Settings, which you can find in the "Editor-Lists" section, please update and have a check.


Race condition: client side navigation vs SSR

Describe the bug
Quick navigation renders wrong page.
When browser navigating routes A -> B -> C, where A is a separate app, we render B, while address bar showsC.
Critical effect is this: User is served a page with broken interactive elements, if these elements are using [slug] params
Initial investigation
When quickly navigating back from a different site, the second navigation doesn't even trigger an http request, even before the scripts.
I was able to correct this effect locally by adding a location.href check into the rendered html
https://github.com/JakeBeaver/kit/blob/c1477090331570414d4104788347470fee18a2a4/packages/kit/src/runtime/server/page/render.js#L359
    blocks.push(`if (${event.url.href} !== location.href) location.refresh();`);

While this is most definitelly not the fix, it proves that browser navigation during first hydration(?) is ignored completely, but I do not know what interferes with the browser back button before client/start.js runs?
Reproduction
Project:
Create a skeleton project with the following, or clone this repo: https://github.com/JakeBeaver/sveltekit-quick-navigation-reproduction
Content:

/+page.svelte: empty
/load/+page.svelte: empty
/load/+page.server.js: load() that returns a promise
/layout.svelte: navbar, {$page.route.id}, slot
/slug/[slug]/+page.svelte: {$page.params.slug}

Steps:
Start on root route /

Use navbar to open /slug/1
Use address bar to navigate outside of the sveltekit app, like https://github.com/sveltejs/kit
Click the browser back button twice quickly

Result
/slug/[slug] gets rendered but $page.params.slug is missing

Logs
C:\repos\my-app> npm run dev

><EMAIL_ADDRESS>dev
> vite dev

  VITE v4.4.9  ready in 1311 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
  ➜  press h to show help

System Info
System:
    OS: Windows 10 10.0.22000
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 12.55 GB / 31.74 GB
  Binaries:
    Node: 18.17.0 - C:\Program Files\nodejs\node.EXE
    npm: 9.6.7 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22000.120.0), Chromium (115.0.1901.200)
    Internet Explorer: 11.0.22000.120
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.1.0
    @sveltejs/kit: ^1.20.4 => 1.22.4
    svelte: ^4.0.5 => 4.1.2
    vite: ^4.4.2 => 4.4.9

Severity
blocking an upgrade
Additional Information
No response

Found a dirty workaround, but this is not ideal:
+layout.server.ts
export async function load({ route }) {
	return { route: route.id }
}

+layout.svelte
<script>
	export let data;
	onMount(()=>{
		if (data.route != $page.route.id) {
			window.location.reload();
		}
	})
</script>


Added bg beta

The Bulgarian language was missing the (Beta) in the language menu

Hello @ctlaltdieliet,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Hello @ctlaltdieliet,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


Fix for update_platform_config.js documentation

fix(hooks): fix for update_platform_config.js documentation example
Fix for the example included in the update_platform_config after prepare hook documentation not properly closing the config-file XML tag.

DON'T MERGE THIS PR.
Sorry but this PR isn't complete. I thought it was a simple typo but now I'm facing some problems.
Setting the config.xml to the specified parent value in the PR ("/"):
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<widget id="com.xxx.yyy" version="2.0.0" xmlns="http://www.w3.org/ns/widgets">
    <platform name="android">
        <config-file target="AndroidManifest.xml" parent="/">
            <uses-permission android:name="android.permission.WRITE_CONTACTS" />
        </config-file>
    </platform>
</widget>

The ionic build --release android --info command execution gives the error: net.js:633 throw new TypeError('invalid data');
And trying to fix it with the "/*" value in the config.xml like:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<widget id="com.xxx.yyy" version="2.0.0" xmlns="http://www.w3.org/ns/widgets">
    <platform name="android">
        <config-file target="AndroidManifest.xml" parent="/*">
            <uses-permission android:name="android.permission.WRITE_CONTACTS" />
        </config-file>
    </platform>
</widget>

The ionic build --release android --info command execution gives the error: platforms/android/build/intermediates/res/release/xml/config.xml:27: error: Error parsing XML: unbound prefix
And at this xml line I have the uses-permissions statement:
    <config-file parent="/*" target="AndroidManifest.xml">
        <uses-permission android:name="android.permission.WRITE_CONTACTS" />
    </config-file>

I've been searching for the solution to this error and some people recommend to "clean the project" in the  Eclipse IDE. But I don't know how to proceed dealing with an ionic/cordova app.
I though it provably was due to the android XML namespace, so I tried with:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<widget id="com.uvinum.mobileapp" version="2.0.0"
  xmlns="http://www.w3.org/ns/widgets"
  xmlns:android="http://schemas.android.com/apk/res/android">
    <platform name="android">
        <config-file target="AndroidManifest.xml" parent="/*">
            <uses-permission android:name="android.permission.WRITE_CONTACTS" />
        </config-file>
    </platform>
</widget>

But the it gave me errors like:
platforms/android/AndroidManifest.xml:4: Error: The <supports-screens> element must be a direct child of the <manifest> root element [WrongManifestParent]
        <supports-screens android:largeScreens="false" android:smallScreens="false" android:xlargeScreens="false" />
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~platforms/android/AndroidManifest.xml:5: Error: The <uses-permission> element must be a direct child of the <manifest> root element [WrongManifestParent]
        <uses-permission android:maxSdkVersion="15" android:name="android.permission.READ_CONTACTS" />
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
platforms/android/AndroidManifest.xml:6: Error: The <uses-permission> element must be a direct child of the <manifest> root element [WrongManifestParent]
        <uses-permission android:name="android.permission.WRITE_CONTACTS" />
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

   Explanation for issues of type "WrongManifestParent":
   The <uses-library> element should be defined as a direct child of the
   <application> tag, not the <manifest> tag or an <activity> tag. Similarly,
   a <uses-sdk> tag much be declared at the root level, and so on. This check
   looks for incorrect declaration locations in the manifest, and complains if
   an element is found in the wrong place.

   http://developer.android.com/guide/topics/manifest/manifest-intro.html

So I guess I should put a value in the parent attribute of the config-file tag which make some reference to the application tag, but I don't really know which is the exact proper value.
Sorry for the confusion with the PR and thanks for your help. I hope solving this issue could help future cases who, like me, copy&paste the documentation example.

Sorry for the confusion. I've finally managed it :)
I didn't notice that after running an ionic build --release android, the script will append the configuration statements specified in the config.xml and, at the moment of removing those statements form the config.xml and executing another ionic build command, the script will NOT remove the previous statements from the AndroidManifest.xml.
So, finally, I've had only to:

Remove the previous lines from the AndroidManifest.xml
Include the Android XML namespace in the widget tag
Run again the ionic build

In conclusion: You can merge this pull request :)


Cursor in MTK Hand Onfocus never trigger

Cursor in MTK Hand Onfocus always trigger at any time
to reproduce it do the settings for leap motion
show your hand cursor always trigger onfocus even it is not hitting ray casting  anywhere.

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

ID: 69249bf8-d2cd-5efe-ec17-b72078174617
Version Independent ID: cc1ba99d-5d08-f323-55a4-f4f4ea06974c
Content: Cursors - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/design/cursors.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @thetuvix
Microsoft Alias: alexturn

I mean It is not entering to rest cursor
on press is working but on focus is always actiavted even raycast does not hit anywhere

If you're using MRTK, please file this issue in their repo to get support :) https://github.com/microsoft/MixedRealityToolkit-Unity


Allow / in asset paths

Versions
Angular CLI: 6.0.0-rc.0
Node: 9.2.0
OS: linux x64
Angular: 5.2.9
... animations, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic
... platform-server, router, service-worker

@angular/cdk: 5.2.4
@angular/cli: 6.0.0-rc.0
@angular/material: 5.2.4
@angular-devkit/architect: 0.5.0
@angular-devkit/build-angular: 0.0.10
@angular-devkit/build-optimizer: 0.4.9
@angular-devkit/core: 0.5.0
@angular-devkit/schematics: 0.5.0
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 6.0.0-beta.9
@schematics/angular: 0.5.0
@schematics/update: 0.5.0
typescript: 2.4.2
webpack: 4.1.0

Repro steps
{
                "glob": "favicon.ico",
                "input": "src",
                "output": "/"
              },

works, but
{
                "glob": "favicon.ico",
                "input": "/src",
                "output": "/"
              },

does not recognize / generate the assets
Desired behavior
Throw an error if my config is wrong / invalid, or support the /

/src and src mean different things though. /src is an absolute path, from the root of your file system.
I don't think this is a bug, I think it fails because you don't have a /src folder at the root of your file system.

I'm fine closing this. I logged this because "/src" was what the migration script created for me. I thought from a brief conversation that this was the bug, but feel free to close this one and then let's update the migration script.

Talking to @hansl about the intended semantics atm.

We decided that right now: (using Windows paths)
`/src/a/b/` => `C:\src\a\b`
`src/a/b/` => `${workspaceRoot}/a/b/`
`./src/a/b/` => `${workspaceRoot}/a/b/`

Checked the current workspace file and update logic and could not find any other instanced of paths starting with '/'. I think we are good for now.


Please

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code
// If relevant, paste all of your challenge code in here

Screenshot

Please reopen with details if this was meant to be a legitimate issue post.


LyCORIS previews missing after a1111-sd-webui-lycoris auto deactivation

Checked for extensions updates in A1111 v1.5.2, upon restart i had this message:
========================= a1111-sd-webui-lycoris =========================
Starting from stable-diffusion-webui version 1.5.0
a1111-sd-webui-lycoris extension is no longer needed

All its features have been integrated into the native LoRA extension
LyCORIS models can now be used as if there are regular LoRA models

This extension has been automatically deactivated
Please remove this extension
==========================================================================

Its removal seem to prevent LyCORIS tab to show up on both the Models Previews tab and the models viewer/selector in txt2img.
Additional note: Lycos are present in the lists, just mixed up with Loras. In the Models Previews tab, order is alphabetical, in the txt2img tab, Lycos always show up by the end of the list and insert the <lora:xxx:1> keyword (I don't know if this intended behavior ?)

Yes the new version of A1111 now has native support for LyCORIS models and treats them as standard Loras in terms of classification. So they appear in the Lora tab in the extra networks list and in the Lora tab in the model preview extension.
If you have the extra network list open and click on a model card it will load that model or add the tag to the prompt in the case of embeddings, hypernetworks and loras. This is a feature of the automatic1111 UI. Lora tags are formatted with it being LORA:{{LORA_NAME}}:{{WEIGHT}}. This should only happen when clicking the model in the extra networks section... if its happening while selecting a model in the Model Previews extension tab let me know as that is not intended and I can't yet replicate that behaviour.
In the extra networks area it will list the models in a "default sort". You can change it to sort by name in the setting beside the search field.

There is currently no way to change the sort of the list in the Model Previews tab.
It appears A1111 is intending users to mix their LyCORIS and LORA model folders. From looking at the source code it seems to still support having LyCORIS models in a separate LyCORIS folder but it is labelled as for "backwards compatibility".
If you mix the LyCORIS models in with  your LORA models the "default sort" of the extra networks tab will no longer put all the LyCORIS models at the bottom.
The one issue currently is if you want to keep your LyCORIS models in their separate folder this extension does not look in that folder for previews of Lora models so you will get "No Preview Found" when selecting a LyCORIS model in the LORA tab. For backwards compatibility support I can look at adding the LyCORIS folder as a default folder to also search in for all LORA models. Or you can just move your LyCORIS folder into your LORA folder... this will let you keep the models in separate folders but still let the extension find the preview files because it will look through all the subfolders.


Support for Xaml 2009

Please consider adding support for Xaml 2009 features:
https://msdn.microsoft.com/en-us/library/ee792007(v=vs.110).aspx
Xamarin.Forms supports this, and so does WPF, but ironically the tooling used for it does not, heh.
I would be interested in assisting!

The Tooling does not support it, cause compiled xaml only supports xaml 2006! The dynamicly parsed supports 2009

why wontfix?


HWKAPM-724 Fix issue where empty string for Elasticsearch hosts prope…

…rty causes error

@jpkrohling Could you review please?

LGTM. Ready to merge, once Travis approves it.

@jpkrohling Thanks!


compas_cloud does not allow function returning nothing, or non-iterable values

because one cannot json.dumps such values with the current encoders...
hacky solution provided below:
in server.py:
consider adding line 180

in proxy.py
consider revising final line

@brgcode
@Licini

Also... why does the try statement in server.py attempt every single possible function (as shown in the screenshot)?
If I am calling Proxy for a function, 'cache' will be in data (storing the boolean True/False), and as process() is setup now, it will end up calling the protocol's cache function anyway and storing the True/False boolean value...


fix(test): Fix edit/cancel comment test

This PR fixes the failing edit/cancel comment smoke test. @sanbornsen Please review.

@jarifibrahim fabric8/fabric8-ui:SNAPSHOT-PR-2162-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2162-fabric8-planner.badger.fabric8.io

Failing in CI due to could not connect to jenkins-slave.

@jarifibrahim fabric8/fabric8-ui:SNAPSHOT-PR-2162-3 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2162-fabric8-planner.badger.fabric8.io

@jarifibrahim fabric8/fabric8-ui:SNAPSHOT-PR-2162-4 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2162-fabric8-planner.badger.fabric8.io


koor-descheduler: fix LowNodeLoad unsupported configuration and pod sorter

Signed-off-by: Joseph<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

LowNodeLoad plugin support some missed configuration fields such as paused, dryRun and evictableNamespaces.include.
Optimize pod sorter in LowNodeLoad when Pod doesn't have Koordinator QoSClass.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

/lgtm cancel


Feature request: Add ability to point ArchiveBox to a custom favicon service instead of Google

Just found out that https://github.com/mat/besticon exists.  This would allow us to point ArchiveBox at a self-hosted replacement for the Google favicon service we currently use.
The current solution has always mildly irked me as a suboptimal way we leak archived domains to a 3rd party, which is why I document how to disable it here: https://docs.archivebox.io/en/latest/Security-Overview.html#stealth-mode, but it would be nice to provide an alternative other than disabling it entirely.
I'm imagining something like:
archivebox config --set FAVICON_PROVIDER='https://www.google.com/s2/favicons?domain={}' (the default)
that can be set to anything, e.g.:
archivebox config --set FAVICON_PROVIDER='https://someselfhostedservice.example.com/?q={}&format=png&size=64x64'

closing because PR was merged, thanks!


fgt_event not picking up "DNS query timeout" messages

Hi there,
some of our Fortinet messages are tagged as sc4s:fallback while the other system messages are correctly detected as fgt_event.
So far, they all look like this (RAWMSG from event, just "anonymized" while keeping the format) :
<187>date=2021-02-05 time=09:21:47 devname="FW-ABC-DEF-01-FG1" devid="FGT1AB1234567898" logid="1501054200" type="dns" subtype="dns-response" level="error" vd="root" eventtime=1612513307 policyid=0 sessionid=0 srcport=0 srcintf=unknown-0 srcintfrole="undefined" dstip=<IP_ADDRESS> dstport=53 dstintf=unknown-0 dstintfrole="undefined" proto=17 xid=0 qname="directregistration.fortinet.com" qtype="A" qtypeval=1 qclass="IN" msg="A DNS resolution error occurs" action="pass" error="DNS query timeout"
We are on sc4s V1.51.0

Can you post a raw event of a working fortigate event?  It appears the DNS ones do not have the requisite syslog header (which includes a date/timestamp and host), for which we test on ingest:
<111> Aug 17 00:00:00 fortigate date=2015-08-11 time=19:19:43 devname=Nosey...

Thanks for the answer! I hope this is helpful, I am just starting with SC4S:
Failed (sourcetype = sc4s:fallback):
<187>date=2021-02-08 time=07:37:02 devname="FW-ABC-DEF-01-FG1" devid="FGT1AB1234567890" logid="1501054200" type="dns" subtype="dns-response" level="error" vd="root" eventtime=1612766222 policyid=0 sessionid=0 srcport=0 srcintf=unknown-0 srcintfrole="undefined" dstip=<IP_ADDRESS> dstport=53 dstintf=unknown-0 dstintfrole="undefined" proto=17 xid=0 qname="directregistration.fortinet.com" qtype="A" qtypeval=1 qclass="IN" msg="A DNS resolution error occurs" action="pass" error="DNS query timeout"
Working (sourcetype = fgt_event):
<189>date=2021-02-08 time=07:44:53 devname="FW-ABC-DEF-01-FG1" devid="FGT1AB1234567890" logid="0100032011" type="event" subtype="system" level="notice" vd="VD-ABC-DE01" eventtime=1612766693 logdesc="Disk log rolled" action="roll-log" reason="file-size" log="tlog" msg="Disk log has rolled."
I see a difference in severity and the "type" flag is different.


Subscription dropout

We are getting subscriptions dropping intermittently using this library.  Although we log errors on both connection drop and on Error, the subscription just drops silently, not giving any hint as to why it has stopped receiving events.  This doesn't appear to be after any set period of time, sometimes it is almost straight away, sometimes after a few hours.  With DEBUG mode on, we can see the heardbeats continue on without fail after the last event before the drop.
Any idea what might be going on and how to debug this further?

At minimum, a packet capture of the traffic on port 1113 would give us some insight into what is going on.
Is the subscription (rather than the connection) onDropped handler being invoked with a reason why the subscription was cancelled?

I haven't managed to do a packet capture yet.  But I have found an issue on the server where the DNS was dropping periodically, so maybe this was the issue?  I connect by direct IP however so I wouldn't have expected that.
I was logging in all the callbacks (onDropped / onErrror) in both the connection and subscription, but nothing was logged.  I have switched to a poll based subscription for now, will revisit when i have more time.

It might be that the Event Store needs to be pinged regularly for it to consider the connection to be alive alive, and that's not something that the client does automatically, but those methods are exposed on the connection for your application to use.

Ok have updated our code to fire a ping every 10 seconds.  Will keep an eye on it and see if this prevents the dropout.

Just to confirm, firing a ping to keep the connection alive appears to have resolved this issue.  Would this not be something you'd have the client do on connect?

It would need to be done regularly so that the server knows that the client is still there. Ideally, that would be done on a timer in your application.

Yes. I have a wrapper class around the API with a connect and disconnect method.  The connect method starts a timer pinging every 10 seconds, the disconnect clears the timer.  Just thought this could maybe be behaviour the library could take over, as it's not immediately obvious for others that you'd need to do this for a long subscription.

Yeah, if there's a way we can get the library to do it that would be good.
It's been a while since I've looked at timers and threading in Node.js, so I'm not too familiar with how we can do it without potentially affecting the application itself.


Add option to provide API Key and HTTP Bearer Token via CLI if no config was provided

Add option to provide API Key and HTTP Bearer Token via CLI if no config was provided so that the user can put in this information in the cli. Might also be useful for a potential release as an exe/binary.

Added by Commit #2f6da1b


Fix typo

I noticed additional typos while reading over the tests, so I decided to just close this PR and create a separate one that included those fixes as well: https://github.com/preactjs/signals/pull/263