core[patch]: fix no current event loop for sql history in async mode

Description: When use RunnableWithMessageHistory/SQLChatMessageHistory in async mode, we'll get the following error:

Error in RootListenersTracer.on_chain_end callback: RuntimeError("There is no current event loop in thread 'asyncio_3'.")

which throwed by https://github.com/langchain-ai/langchain/blob/ddfbca38dfa22954eaeda38614c6e1ec0cdecaa9/libs/community/langchain_community/chat_message_histories/sql.py#L259. and no message history will be add to database.
In this patch, a new _aexit_history function which will'be called in async mode is added, and in turn aadd_messages will be called.
In this patch, we use afunc attribute of a Runnable to check if the end listener should be run in async mode or not.

Issue: #22021, #22022
Dependencies: N/A

@pprados Please help to review this PR.

the issue you linked to is closed. can you share the code you are running to reproduce this error?

Here is a sample langserve code to reproduce this error.
LangServe server side code
from typing import AsyncIterator, Any

from fastapi import APIRouter, FastAPI
from langchain_core.chat_history import BaseChatMessageHistory
from langchain_community.chat_message_histories.sql import SQLChatMessageHistory
from langchain_core.runnables import Runnable
from langchain_core.runnables.history import RunnableWithMessageHistory
from langchain_core.runnables.utils import ConfigurableFieldSpec
from langchain_core.tools import Tool
from langchain_openai import AzureChatOpenAI
from langchain.agents import AgentExecutor, create_tool_calling_agent
from langchain.prompts import (
    ChatPromptTemplate, MessagesPlaceholder, HumanMessagePromptTemplate,
    SystemMessagePromptTemplate
)
from langchain.schema.runnable import RunnableLambda
from langserve import add_routes
from langserve.pydantic_v1 import BaseModel, Field
from sqlalchemy.ext.asyncio import create_async_engine

# Config
sql_server_url = "mysql+asyncmy://dbuser:dbpass@localhost:3306/history"

azure_endpoint = "fixme"
azure_deployment = "fixme"
openai_api_version = "fixme"
openai_api_key = "fixme"

# Model
class LLMInput(BaseModel):
    question: str = Field(
        ...,
        description="user question",
    )
    system_prompt: str = Field(
        default="You are a helpful AI assistant",
        description="system prompt",
    )

# History Stuff
async_sql_engine = create_async_engine(sql_server_url, pool_recycle=600)

history_factory_config = [
    ConfigurableFieldSpec(
        id="user_id",
        annotation=str,
        name="User ID",
        description="Unique identifier for the user.",
        default="",
        is_shared=True,
    ),
    ConfigurableFieldSpec(
        id="conversation_id",
        annotation=str,
        name="Conversation ID",
        description="Unique identifier for the conversation.",
        default="",
        is_shared=True,
    ),
]

def get_session_history(
    user_id: str,
    conversation_id: str,
) -> BaseChatMessageHistory:
    """Get a chat history from a session ID."""
    session_id = f"{user_id}_{conversation_id}"

    return SQLChatMessageHistory(
        session_id=session_id,
        connection=async_sql_engine,
    )

# Tool Stuff
def search(query: str, **kwargs) -> str:
    return f"{query} found"

class SearchToolInput(BaseModel):
    """Input for music tool."""
    query: str = Field(description="keyword for search")

search_tool = Tool(
    name="web_search",
    description="web_search is a tool to search online information.",
    func=search,
    args_schema=SearchToolInput,
)

def build_llm_chain(llm_input: LLMInput) -> Runnable:
    llm = AzureChatOpenAI(
        azure_endpoint=azure_endpoint,
        azure_deployment=azure_deployment,
        openai_api_version=openai_api_version,
        openai_api_key=openai_api_key,
        streaming=True,
        temperature=0.01,
        max_tokens=4000,
    )
    tools = [search_tool]
    prompt = ChatPromptTemplate.from_messages([
        SystemMessagePromptTemplate.from_template(llm_input.system_prompt),
        MessagesPlaceholder(variable_name="history"),
        HumanMessagePromptTemplate.from_template("{question}"),
        MessagesPlaceholder(variable_name="agent_scratchpad"),
    ])
    agent = create_tool_calling_agent(
        llm, tools, prompt,
    )
    return AgentExecutor(
        agent=agent,
        tools=tools,
    )

def build_chain(llm_input: LLMInput) -> Runnable:
    llm_chain = build_llm_chain(llm_input)

    return RunnableWithMessageHistory(
        llm_chain,
        get_session_history,
        input_messages_key="question",
        history_messages_key="history",
        history_factory_config=history_factory_config
    ).with_types(
        input_type=LLMInput
    )

async def llm_main(_input: dict[str, Any]) -> AsyncIterator[Any]:
    _input["question"] = _input["question"].lstrip("\n").rstrip()
    _input["system_prompt"] = _input["system_prompt"] or "You are a helpful AI assistant"

    llm_input: LLMInput = LLMInput.parse_obj(_input)

    chain = build_chain(llm_input)
    config = {
        "configurable": {
            "user_id": "user1",
            "conversation_id": "conv1",
        }
    }

    async for event in chain.astream_events(_input, config, version="v1"):
        yield event

router = APIRouter()

add_routes(
    router,
    RunnableLambda(llm_main).with_types(input_type=LLMInput),
    enabled_endpoints=["invoke", "batch", "stream"]
)

app = FastAPI(
    title="Test Server",
    version="0.01",
    description="Test Server Powered by LangChain&LLM",
)
app.include_router(router)

Save it as async_agent.py, and modify configurations about sql server and azure openai in your own, then run like
uvicorn async_agent:app --host <IP_ADDRESS>

LangServe client side code
from langserve import RemoteRunnable

client = RemoteRunnable('http://localhost:8000/')
for chunk in yuan.stream({'question': "What's langchain"}):
    print(chunk)

Save it as agent_client.py, and run like
python agent_client.py

Result
On the client side, you will see many output about the event, and final answer about the question is got.
But on the server side, you will see logs like
INFO:     Will watch for changes in these directories: ['/home/mackong/Codes/playground/llm']
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [828125] using WatchFiles
INFO:     Started server process [828140]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     <IP_ADDRESS>:49590 - "POST /stream HTTP/1.1" 200 OK
Parent run 8e230381-f1bb-42ab-835b-635851ef9bba not found for run 5959e910-f062-4229-9839-1d3de2f22007. Treating as a root run.
Error in RootListenersTracer.on_chain_end callback: RuntimeError("There is no current event loop in thread 'ThreadPoolExecutor-0_0'.")

where the error is throwed by sql.py#259. and no message history will be add to database.

@hwchase17, @mackong I review this code. It's correct for me. This fixes the bug I reported here.
With this PR, it's possible to remove the hack I mentioned here in SQLChatMessageHistory.
    def add_messages(self, messages: Sequence[BaseMessage]) -> None:
        # The method RunnableWithMessageHistory._exit_history() call
        #  add_message method by mistake and not aadd_message.
        # See https://github.com/langchain-ai/langchain/issues/22021
        if self.async_mode:
            loop = asyncio.get_event_loop()
            loop.run_until_complete(self.aadd_messages(messages))
        else:
          ...

You can remove this hack.

OK, removed.

@mackong, Some checks were not successful

@pprados already passed


Fixed AD object cleanup

This PR makes the ad_site argument in __connect_to_activdirectory optional which broke AD object cleanup. The function already handles an empty argument and ad_site should be optional.

Fixes #40


Include the number of products to be sent in the sidebar "Offline Mode" item

Include the number of products to be sent in the sidebar "Offline Mode" item
https://github.com/mikepenz/Android-ActionItemBadge

I would like to work on it.

@teolemon I've checked out the GitHub link which you've mentioned that is pretty good. However, I would like to implement it without using any library :). I just wanna know that do we just need to show a number(number of items) on ActionBar or an Icon with a badge(containing the number of items).

Hi @Shubham2320, from what I can gather we would like a badge with the number of items that need to be uploaded

Thanx I would send the PR soon :).


Error with Live Grep on Windows 11

Hi,
I was trying out your plugin because it has windows support, and I got it to work for almost everything except for live grep.
To elaborate, when I run FzfxLiveGrep, it opens up the menu with a list of all the searches but, as soon as I type any character, it produces the error command failed.
Before pressing any character

After pressing any character

character*
I tried reinstalling it multiple times but nothing seems to suffice.
Please help me on this one.

hi @Sawansunar56,
would you please try this command:
sleep 0.1 && rg

and give me the output.

Hi, @Sawansunar56,
you can set live_grep.other_opts.onchange_reload_delay=nil, it may be fixed.

Hi, @Sawansunar56, you can set live_grep.other_opts.onchange_reload_delay=nil, it may be fixed.

@linrongbin16  Thank you for the help.
This command fixed my problem and now everything works. Also, just to be sure, I changed this value in the config file.
Is there a simple way, to just search current buffer. I seemed to cannot find anything related to it in the readme, it would be helpful for me.
Also, I appreciate the good work.

This command fixed my problem and now everything works. Also, just to be sure, I changed this value in the config file.

It is correct. The root cause is that your Windows didn't have the sleep command. I will improve the readme doc to add a tutorial about how to install linux commands on Windows. (because this plugin heavily need many linux commands such as mv, sleep, cp) That is actually a dependency for Windows.

Is there a simple way, to just search current buffer? I seemed to cannot find anything related to it in the readme, it would be helpful for me.

There's no way to only search on current buffer. I think we can simply use / and start searching?

I also improved the README doc for Windows dependency in https://github.com/linrongbin16/fzfx.nvim/pull/86, please see: https://github.com/linrongbin16/fzfx.nvim#requirement

I also improved the README doc for Windows dependency in #86, please see: https://github.com/linrongbin16/fzfx.nvim#requirement

Yeah, that would make things easier. Thanks for the quick reply.


New version of the Header/Footer is coming out

4.4!

See https://drupal.asu.edu/build/asu-header-footer-latest

Still not released.

Removing myself as the assignee since this will most likely not be done by the time I leave.


Deployment strategy for Kedro pipeline

Problem statement
Users like Kedro because of the ease of getting started for prototyping a data pipeline. A question that we often get from users are:
"I have built a Kedro pipeline. How do I deploy my pipeline in XYZ?"
where XYZ is a various deployment platform, such as Airflow, KuberFlow, AWS Glue, Step functions, Prefect etc.
So far we answer this question in the following ways:

On a case-by-case basis
We provide Kedro-Airflow and Kedro-Docker plugins for some deployment.
External users built custom plugins, such as Kedro-Argo.

Limitations to the above approach are:
For 1, it's not scalable.
For 2, Kedro-Airflow offers limited customisations:

It cannot easily use a different Operator other than PythonOperator.
It doesn't work with MemoryDataSet, and users have to manually convert all MemoryDataSet into persistent datasets.

For 3, the community plugins are unofficial, so we cannot guarantee the quality of these external tools.
More users are building a data pipeline with Kedro and want to deploy their pipeline in various deployment platforms, and we need a common interface for these different deployment platforms.
Possible Implementation
When thinking about deployment strategy, there are several key considerations.

We would like to identify and provide general guidance for deployment in popular platforms.
Due to the diversity and heterogeneity of existing deployment platforms, it would be difficult to cover all the we would like to able to provide general guidance to these various deployment platforms as a first iteration.
There should not be a limitation for users to be able to do customisations, such as scheduling interval, configuration, specifying which part of the pipeline to run etc.
We should provide a way to convert MemoryDataSet inputs/outputs into concrete datasets, such as PickleDataSet (related issue: https://github.com/quantumblacklabs/kedro-airflow/issues/46)

General guidance for deployment workflow
As a first iteration, we would also like to provide documentation/blog posts for general guidance for deployment workflow in major platforms.
At high-level, there are 3 different deployments: single-machine deployment, Docker-based clusters, and serverless.
In a distributed deployment environment, the proposed workflow works as follows:

Package the entire Kedro pipeline and dependencies (either wheel or dockerising it in an image),
Execute a single node (or part of the pipeline) with kedro pipeline --node=xx. This approach gives users the flexibility to run more than one node in a cluster (e.g kedro pipeline --from-node=xx --to-node=xx) if they wish.

While it might be the case that each node has different dependencies, figuring out what dependencies each node requires would add an extra burden for users to customise, especially for a large pipeline. Thus, we are packaging all the dependencies as a first iteration.
Next steps
As a bare minimum MVP, we are planning to do the followings:

Identify major deployment platforms, and prototype how to deploy our spaceflight project.
We collect our learning and necessary steps and document it, either as part of Kedro documentation or a blog post (similar to how AWS provides a number of blog posts as guidance for how to use their service).

Further extensions
Similar to how we promoted our PySpark docs  to a starter, a possible further extension to the documentation would be to provide a template project as a starter, where we provide code/files necessary for users to be able to convert a Kedro pipeline and run in a deployment platform.
The starter could include:

All code/files for building a Kedro pipeline (similar to other starters).
Configuration files for a deployment platform, such as YAML, JSON or Python scripts.
CLI command for generating an interface between Kedro pipeline and deployment pipeline. For example,

In the case of Kuberflow, this could be a JSON schema for a pipeline definition.
For Airflow, it would be a Python file for DAG definition, similar to how we generate a DAG file with [Kedro-Airflow].(https://github.com/quantumblacklabs/kedro-airflow/blob/develop/kedro_airflow/dag_template.py)
This could be scripts to help users to convert the Kedro pipeline into the targeted platform definition of a pipeline code.

Other possible interfaces for deployment are:

Plugins, similar to Kedro-Airflow.
Runner class, similar to ThreadRunner with deploy method.
New abstract class, similar to AbstractDataSet. For example, Prefect provides a Agent class and each deployment platform (AWS, Azure, GCP) override deploy_flow method

Related issue

Kuberflow: https://github.com/quantumblacklabs/kedro/issues/353
AWS Glue https://github.com/quantumblacklabs/kedro/issues/57
Kedro-Airflow https://github.com/quantumblacklabs/kedro-airflow
Kedro-Argo: https://github.com/nraw/kedro-argo
@DmitriiDeriabinQB also prototyped AWS Batch Runner, similar to ThreadRunner.

I'm going to close this issue because the incredible Kedro team completed the first iteration of this. We'll be tracking the docs to see if we continue development for AWS Batch, AWS Sagemaker, Prefect, Kubeflow, Argo and Databricks.


Release/2022 02 08 silver fox

Try out this version of the Hiro Wallet - download extension builds.

Content:

feat: add message in activity list closes https://github.com/hirosystems/stacks-wallet-web/issues/2187

fix: remove suspense, causes scroll bug, closes https://github.com/hirosystems/stacks-wallet-web/issues/2220

fix: add release rules to commit analyzer
No need to QA this

Tested all PR above once again in Silver fox regression:
feat: add message in activity list closes https://github.com/hirosystems/stacks-wallet-web/issues/2187 (Passed)
fix: remove suspense, causes scroll bug, closes https://github.com/hirosystems/stacks-wallet-web/issues/2220 (Passed)
Fix bns lookupProfile
https://github.com/hirosystems/stacks-wallet-web/pull/2089
The fix didnt work me thought @pradel indicated it worked for him
But test oassed for bns lookup only when I tested it against https://github.com/hirosystems/stacks-wallet-web/pull/2172
cc @kyranjamie @beguene @andresgalante

How can we test the BNS PR to release with confidence? What are the next steps here?

I can confirm that it's not working on my side now, will spend some time today to see what's going on and report my findings here.

After some testing, there is something really strange going on in the wallet. I managed to reproduce the issue in a consistent way.
A. User path that is working to login successfully, basically any user installing the extension for the first time.
A.1. Logout from any existing account in the Hiro wallet
A.2 Quit chrome completely - ⚠ this is really important, you need to make sure the full chrome app is closed and no background process are running.
A.3. Open chrome and login into the Hiro wallet with an account owning a .btc username.
A.4. Go to the Sigle demo app, then login
A.5. ✅  Everything is working you are logged in
B. How to reproduce the issue in a consistent way, with this path you will get the issue @Eshwari007 faced
B.1. Open chrome then login into the Hiro wallet with an account owning a .btc username.
B.2. Logout from the Hiro wallet
B.3. Now login again in the Hiro wallet without closing chrome
B.4. Go to the Sigle demo app, then login
B.4. 🛑  You should get the error @Eshwari007 had before
So basically logging in to a new account is working only if you did not log out before (fresh installation, or restarting chrome after logout). Like this, the issue seems more related to how the wallet handles things internally than in the SDK but might be wrong :).

I run the same test on https://github.com/hirosystems/stacks-wallet-web/pull/2172 and all the tests are passing there. So the bug is really likely in the stacks-wallet-web code.

After some testing, there is something really strange going on in the wallet. I managed to reproduce the issue in a consistent way.
A. User path that is working to login successfully, basically any user installing the extension for the first time. A.1. Logout from any existing account in the Hiro wallet A.2 Quit chrome completely - ⚠ this is really important, you need to make sure the full chrome app is closed and no background process are running. A.3. Open chrome and login into the Hiro wallet with an account owning a .btc username. A.4. Go to the Sigle demo app, then login A.5. ✅ Everything is working you are logged in
B. How to reproduce the issue in a consistent way, with this path you will get the issue @Eshwari007 faced B.1. Open chrome then login into the Hiro wallet with an account owning a .btc username. B.2. Logout from the Hiro wallet B.3. Now login again in the Hiro wallet without closing chrome B.4. Go to the Sigle demo app, then login B.4. 🛑 You should get the error @Eshwari007 had before
So basically logging in to a new account is working only if you did not log out before (fresh installation, or restarting chrome after logout). Like this, the issue seems more related to how the wallet handles things internally than in the SDK but might be wrong :).

I'll add this use case to testing in regression as well.
Thank you for the findings @pradel

@beguene Seems like the info message background issue resurfaced again

feat: add message in activity list closes https://github.com/hirosystems/stacks-wallet-web/issues/2187.   -Background color issue resurfaced ,attached screenshot above
fix: remove suspense, causes scroll bug, closes https://github.com/hirosystems/stacks-wallet-web/issues/2220. -Test Passed

Thanks @fbwoolf! It's not issue at all, we can indeed revert the bns commit. I will prepare a new PR for QA today.

@Eshwari007 @timstackblock I updated this PR (and excluded the BNS fix), the PR description is also updated with the content of this release.
cc: @andresgalante

The release is blocked due to https://github.com/hirosystems/stacks-wallet-web/issues/2235
cc @timstackblock @andresgalante

This issue is currently occurring on Silver Fox.
https://github.com/hirosystems/stacks-wallet-web/issues/2235

@Eshwari007 @timstackblock Please block the PR to prevent being merge when you are blocking a release

@timstackblock @Eshwari007 I just updated this release PR with the latest dev which includes the fix for #2235

@markmhx Quick question more like enhancement:
Use case 1
When a user logs into the wallet via secret key via the popup extension version, the accounts are populated accordingly but they are not on the full view window wallet
However if we sign in via the window wallet view ,the popup wallet version is synced up

Use Case 2
Locking the account
Similarly when a user locks the account in the popup wallet version, both screens were locked but when the user unlocks the popup wallet version, the full window wallet is yet locked

A few areas as above would be nice if they are synced up as the user needs to do duplicate work at times to view both screens concurrently. Thanks

A few areas as above would be nice if they are synced up as the user needs to do duplicate work at times to view both screens concurrently. Thanks

This would definitely be an enhancement to routing using messaging. Currently, we only use messaging to keep them in sync on locking (not unlocking) and sign out.

I agree with Fara, I don't think these are blockers.
@Eshwari007 would you mind creating issue for your findings please?

I agree with Fara, I don't think these are blockers.
@Eshwari007 would you mind creating issue for your findings please?

Will do the needful and. mark them as enhancements

@markmhx Can you please read #2235 and make sure that you are ok releasing 3.2 and introducing a minor bug. I don't think anyone will notice it.

@markmhx Can you please read #2235 and make sure that you are ok releasing 3.2 and introducing a minor bug. I don't think anyone will notice it.

Agree with @andresgalante on the above as the bug seemed to be a minor issue -P4

Yep I'm okay with shipping Silver Fox without a fix for https://github.com/hirosystems/stacks-wallet-web/issues/2235 if we anticipate shipping that fix by the start of next week.


Adds terraform 0.12 support

Fixes https://github.com/dependabot/dependabot-core/issues/1176
I opted for both hcl2json and terraform-config-inspect.
hcl2json for terragrunt and
terraform-config-inspect for tf 0.12
I wanted to go with terraform-config-inspect for both, but it didn't play nice with terragrunt :disappointed:
Notes:

I created my own fork of hcl2json in order to publicly host the binaries for it
opting for terraform-config-inspect adds go as a dependency to being able to support this language. Not sure if that's kosher or not..

Is there anything we can do to help push this through?

Thanks for submitting this, and sorry we've been so slow to get to it. Honestly, the thing that would help us most at the minute would be filling this role, so if you know someone suitable, please  encourage them to apply!
Until we have that role filled, we're unable to dedicate much time to languages not supported by the dependency graph. Longer term, we do have a good plan for supporting a wide range of languages, and making it much easier for the community to help with languages that aren't officially supported, so please bear with us during this slightly awkward transition period. In the meantime, I'll see if I can get this reviewed over the weekend or one evening in the next few days.
At a first glance, the main thing that stands out to me is downloading the binary dependency from the fork of hcl2json. In particular, there's currently not much visibility over the contents of that binary. Two options I see are:

Vendor the dependency, or clone it and pin to a specific commit in the build script. Then we can build it in the build script, avoiding the opaque binary dependency.
We could fork it into the dependabot org and run the build in a GitHub Action, where the build and release logs would be visible, and we'd have control over the release process.

Let me know what you think. And once again, sorry for taking so long to get to this!

@hmarr completely understand, and glad to hear that there's a plan forward with enhanced language support as well.
In regards to your concerns regarding hcl2json:
I've opened a PR upstream but it looks like @tmccombs is busy as well. My initial recommendation would be for option 2. Although, based on the dependabot team's time/resource constraints that sounds like going that route will take longer? Additionally, if the plan is to go that route with hcl2json does the same need to be done with json2hcl?

It was mentioned here that HCL 2 is no longer experimental.  I'm not sure if that affects this PR, but FYI just in case.

Honestly, the thing that would help us most at the minute would be filling this role

@hmarr Looks like that role is filled now (or at least, is showing as "no longer open") - is there any possibility on 0.12 support moving forward?

@dependabot @hmarr @feelepxyz anything we can do here to move this forward?

@lorengordon we've paused dependabot-core improvements while we focus on integrating Dependabot natively in GitHub, this is still a few months out, unfortunately.

I've given this a local test and it works - some minor remarks:

hardcoding the gopath for terraform-config-inspect has caused me a bit of pain
fetching terraform-config-inspect without specifying GO111MODULE=on was not working (see upstream issue https://github.com/hashicorp/terraform-config-inspect/issues/36 )

About terraform-config-inspect - I believe it would make sense to also pre-built the biaries here and host them, so we need to only fetch them as with other dependencies. Solves quite a bit of headache for setting this up - what do you think @userhas404d
Other than that - thanks for doing this - thinking now that I will create a temporary github action with this functionality

I've created a terraform action to provide the features from this pull request for the time being.
https://github.com/marketplace/actions/terraform-dependabot
Thanks to @userhas404d for the great initial work

thanks for the feedback @patrickjahns! I've since implemented some of the updates you suggested and a few others that I found after running this implementation against my org's terraform/terragrunt private projects. I'll open a PR against your github action here shortly.
re GOPATH/terraform_config_inspect: I'll have to defer how to handle that to the dependabot-core maintainers. Earlier on @hmarr had some concerns over how to manage the other pre-compile bins being used in this solution, and I'm not sure where that stands.  There are a few (1, 2) long-ish running issues open against the terraform_config_inspect  project requesting that automated releases be provided, but until those are addressed it looks like compiling it in the terraform helper is probably the best way forward.

I'm adding this comment here as a reminder to myself (or anyone else who want's to take a stab at fixing it): Hit a bug with this implementation supporting both legacy and current versions of terragrunt config where dependabot will bomb when trying to parse a tf012 .tfvars as a legacy terragrunt file. Thinking that the best way forward (while maintaining backwards compatibility) will be a try/catch block around legacy terragrunt config processing.

we've paused dependabot-core improvements while we focus on integrating Dependabot natively in GitHub, this is still a few months out, unfortunately.

Hi @feelepxyz just thought I'd follow up now that it's been a few months since the last update. Any word on resuming improvements like this? We've been using the github-action based on this branch for quite a while now and it's working well.

Considering now we have terraform 0.13 out, where are we with this support? :)

Any updates on this one ?

Terraform 12 (HCL 2) support is now available, thank you so much for this PR, it was instrumental in landing this.


render entities as-is (prevent server-client dom mismatch), fixes #6345

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

I guess the decode is done by intention. Note compiler is shared across ssr, client side template and vue-loader, I wonder if decode is necessary under different situation. However, I'm not familiar enough to give confident answer.
Would you mind digging deeper for this? Thanks!

Thanks for the PR, but the decode in the parser is necessary for correct client-side behavior. The bug was actually introduced by the SSR optimizing compiler in 2.4 and a proper fix is provided in 172dbf9f


Guildprofiles cant be constructed

As the title says. Heres my discord/windows build.
Canary 139299 (bf7a2ab)
BetterDiscord 1.5.3-Hotfix
Host 1.0.48 Windows 10 64-Bit (10.0.19044)

yep really dead here to on stable
Stable 139391 (896f0fd)
BetterDiscord 1.5.3-Hotfix
Host 0.0.267 OS X 10.15.7 (20.6.0)

GuildProfile.plugin.js:415 Uncaught TypeError: Cannot read property 'CONNECTION_OPEN' of undefined
    at GuildProfile.plugin.js:415
    at GuildProfile.plugin.js:1742
    at buildPlugin (GuildProfile.plugin.js:1745)
    at GuildProfile.plugin.js:1752
    at GuildProfile.plugin.js:1811
    at Object.getFileModification (<anonymous>:4:307632)
    at Object..js (<anonymous>:4:267779)
    at Function.o._load (<anonymous>:4:370331)
    at Object.loadAddon (<anonymous>:4:267936)
    at Object.loadAddon (<anonymous>:4:305881)
    at Object.loadAllAddons (<anonymous>:4:270609)
    at Object.initialize (<anonymous>:4:265058)
    at Object.initialize (<anonymous>:4:305091)
    at Object.startup (<anonymous>:4:367101)
(anonymous) @ GuildProfile.plugin.js:415
(anonymous) @ GuildProfile.plugin.js:1742
buildPlugin @ GuildProfile.plugin.js:1745
(anonymous) @ GuildProfile.plugin.js:1752
(anonymous) @ GuildProfile.plugin.js:1811
getFileModification @ VM470:4
(anonymous) @ VM470:4
o._load @ VM470:4
loadAddon @ VM470:4
loadAddon @ VM470:4
loadAllAddons @ VM470:4
initialize @ VM470:4
initialize @ VM470:4
startup @ VM470:4
async function (async)
startup @ VM470:4
(anonymous) @ VM470:4
(anonymous) @ VM470:4
(anonymous) @ VM470:4
(anonymous) @ VM470:9
e.startsWith.e.startsWith.WebFrame.<computed> @ VM75 renderer_init.js:89
(anonymous) @ VM75 renderer_init.js:117
(anonymous) @ VM75 renderer_init.js:101
emit @ VM14 events.js:315

Yep, same here. I think it's related to the new Better Discord update

Same here, without updating BetterDiscord at all

Its known. Discord broke many Plugins with the last update

Originally posted by @HypedDomi in https://github.com/Strencher/BetterDiscordStuff/issues/225#issuecomment-1201825082

I fixed the Plugin. Waiting for the merge. If you cant wait, use this  file

I fixed the Plugin. Waiting for the merge. If you cant wait, use this file

Tested and working properly

the bug can be spotted on both of BetterDiscord 1.5.3 and 1.5.3-Hotfix. Discord Stable 139460 (ea9eb84), Linux 64bit (5.18.14-Arch1-1);
file suggested by @HypedDomi works fine.

@HypedDomi This broke on PTB & Canary again with your fixed ver
Ptb 139684 (95340d3)
BetterDiscord 1.5.3-Hotfix2
Host 0.0.63 OS X 10.15.7 (20.6.0)

Console Log:
35b172dbf615e7730874.js:4267 Uncaught TypeError: Cannot read property 'register' of undefined
    at new e (35b172dbf615e7730874.js:4267)
    at new MemberCountsStore (GuildProfile.plugin.js:414)
    at GuildProfile.plugin.js:430
    at GuildProfile.plugin.js:1755
    at buildPlugin (GuildProfile.plugin.js:1758)
    at GuildProfile.plugin.js:1765
    at GuildProfile.plugin.js:1824
    at Object.getFileModification (<anonymous>:4:307640)
    at Object..js (<anonymous>:4:267787)
    at Function.o._load (<anonymous>:4:370339)
    at Object.loadAddon (<anonymous>:4:267944)
    at Object.loadAddon (<anonymous>:4:305889)
    at Object.loadAllAddons (<anonymous>:4:270617)
    at Object.initialize (<anonymous>:4:265066)
    at Object.initialize (<anonymous>:4:305099)
    at Object.startup (<anonymous>:4:367109)
e @ 35b172dbf615e7730874.js:4267
MemberCountsStore @ GuildProfile.plugin.js:414
(anonymous) @ GuildProfile.plugin.js:430
(anonymous) @ GuildProfile.plugin.js:1755
buildPlugin @ GuildProfile.plugin.js:1758
(anonymous) @ GuildProfile.plugin.js:1765
(anonymous) @ GuildProfile.plugin.js:1824
getFileModification @ VM470:4
(anonymous) @ VM470:4
o._load @ VM470:4
loadAddon @ VM470:4
loadAddon @ VM470:4
loadAllAddons @ VM470:4
initialize @ VM470:4
initialize @ VM470:4
startup @ VM470:4
async function (async)
startup @ VM470:4
(anonymous) @ VM470:4
(anonymous) @ VM470:4
(anonymous) @ VM470:4
(anonymous) @ VM470:9
e.startsWith.e.startsWith.WebFrame.<computed> @ VM75 renderer_init.js:89
(anonymous) @ VM75 renderer_init.js:117
(anonymous) @ VM75 renderer_init.js:101
emit @ VM14 events.js:315
onMessage @ VM75 renderer_init.js:93

Broken again as of today.
Canary 139699 (bdc9502)
BetterDiscord 1.5.3-Hotfix2
Host 1.0.48 Windows 10 64-Bit (10.0.19044)

Since the recent update to ZeresPluginLibrary V2.0.4, & GuildProfile V1.6.3 this works everywhere again..

I fixed the Plugin. Waiting for the merge. If you cant wait, use this file

@XxUnkn0wnxX Yes, it has been solved already.


How to try trydotnet console in my learning platform - https://ilovedotnet.org?

Dear team,
I had a vision couple of years back to give interactive experience using blazor wasm. So I started https://ilovedotnet.org 2-3 years back where I write about topic and give in browser demo using blazor wasm components. Today I came across this awesome dotnet try.
I tried using the iframe url <iframe src="https://try.dot.net/?fromGist=df44833326fcc575e8169fccb9d41fc7"></iframe> mentioned in the official page but not a successful attempt.
I quickly noticed there is something called private preview which is also mentioned in the official page.

Simple to embed
Go beyond copy and paste samples to live snippets. Our private preview customers can embed live .NET code into blogs and documentation.

But not sure where to head from there... I'm not able to find any steps / link to apply for preview. Or if there is any step by step documentation available on how to setup this I would be more happy to try this in https://ilovedotnet.org learning platform.
Please can you folks help me with your expertise on this?

@dalehhirt and @jonsequitur I saw your commits in this repo. Thank you for maintaining this repo. Now I need your help and expertise here. Please can you assist me on next steps?

Allowing you to host your own Try .NET instances is a long-term goal of the repo but is not currently being worked on. We've moved closer with the latest improvements though. If you're feeling motivated to give the latest changes a try, I'm happy to give pointers.
This Dockerfile would be a good starting point for deployment. The thing I haven't tried yet is customizing the Prebuild instances in the container, for example if you want to add support for additional NuGet packages. (Try .NET doesn't allow restoring and building in the container, for security and performance reasons.)
A good starting point for understanding how the system works might be the developer guide, here: https://github.com/dotnet/try/blob/main/Developer-guide.ipynb
And if you want to understand the whole flow all the way from the browser to the server, this diagram might be helpful (or really confusing): https://github.com/dotnet/interactive/tree/main/src/Microsoft.DotNet.Interactive.CSharpProject#try-net--microsoft-learn-flow

@jonsequitur Many thanks for your response. I'll look on the above and get back to you sooner.


fix: restrict memoization fn parameter types

#138  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @pevisscher and the rest of your teammates on  Graphite


Wrong return type on BIO_do_connect man entry

Hello there,
I'm using OpenSSL v3.0.7 on my computer.
BIO_do_connect()'s man page (located in doc/man3/BIO_s_connect.pod) indicates it returns an int. By looking at BIO_do_connect()'s definition (located in include/openssl/bio.h.in) it seems to just be an alias for BIO_do_handshake() which returns a long.
BIO_do_connect() man page should be updated to indicate it returns a long, not an int.
Regards,
Thibaud

Would you like to submit a PR with the change? It would be acceptable with CLA: trivial. https://www.openssl.org/policies/cla.html


Species Type: Full Grammar of some Ideas

Dear Maintainers,
I am currently working on a PICMI implementation, which cites the OpenPMD "Convention for Specifying Particle Species".
While reading the extension my internal "hm, maybe this might be {too permissive|not entirely clear}" went of too often for my personal comfort, and I want to share my thoughts.
I don't work with/on openPMD directly, and also tend to be a little over-pedantic: Please take my comments with a grain of salt.
(Also, my background is in computer science, so I'm not too familiar with your typical physics simulation.)
This issue summarizes a group of issues regarding the species type definition into a (well-defined) grammar:
#257 #258 #259 #260 #261
ABNF Grammar
rough sketch of a speciesType syntax definition using ABNF from the top of my head:
(... is an omitted finite list, ELEMENT a one- or two-letter chemical symbol)
A implementation of the species type extension expects an ascii string and must accept at least expressions based on the rule species-type with the following definition:
species-type = species-single / species-list
species-list = species-single *( ";" species-single )
species-single = elementary / hadron-or-jet / atom / ion / molecule / other

other = "other:" *( ALPHA / DIGIT / "+" / "-" / "_" / "." / "#" / "^" )

elementary = quark / leptop / boson
quark = quark-anti / quark-normal
qurak-anti = "anti-" quark-normal
quark-normal = "up" / "down" / "charm" / "strange" / "top" / "bottom"
lepton = lepton-anti / lepton-normal
lepton-anti = "positron" / "anti-electron-neutrino" / "anti-muon" / ...
lepton-normal = "electron" / "electron-neutrino" / "muon" / ...
boson = "photon" / "gluon" / "w-boson" / "z-boson" / "higgs"

hadron-or-jet = hadron-or-jet-normal / hadron-or-jet-anti
hadron-or-jet-anti = "anti-" hadron-or-jet-normal
hadron-or-jet-normal = "proton" / "neutron"

atom = ELEMENT / isotope
isotope = "#" 1*DIGIT ELEMENT

ion = atom charge-postfix
charge-postfix = "^" *DIGIT ("+" / "-")

molecule = 1*(atom *DIGIT) [charge-postfix]

Notes
I have no stake in this discussion (besides my general love for standards),
the implementation I'm working on only accepts a subset of species anyways.
This entire discussion might be obsolete because the title of the extension clearly states that it is (purely) a convention. (I'd still argue that even if purely convention a clear definition should be used, so others -- like PICMI -- can use it as a standard; but that's debatable.)
Once the questions here/in the related issues are discussed I'd be willing to join the results into a PR for incorporation into the existing species extension document.

@s9105947 I think your points are good ones. Could you form a PR with all this?

ABNF Grammar
A implementation of the species type extension expects an ascii string and must accept at least expressions based on the rule species-type with the following definition:

A few details: we always mean ASCII at the moment for text/string, defined by the base standard.
I like the formal definition, but I think the standard is a bit easier to read if we go by example.
If you find a compact way to add the grammar to each section as a sub-bullet ABNF grammar:, the same way as we have a sub-bullet type, scope, allowed values and examples then this can be added as a PR :)


wip: better error messages for unsupported tuple/shape types in prop types

Motivation

The current behavior is that we silently drop the prop from any kind of codegen, and people get weird errors about calls to initialize or somesuch.  We should at least issue an error message to let them know bad things are happening at the source of the problem.
The PR thus far illustrates the basic idea, but I think the Right Thing to do is have dupType take a std::optional<FailureContext> &.  We can fill in the optional with information about the actual expression that failed and its send context (if any).  If there is a send context, we can use that to generate error messages about unsupported shape types if we have the right kind of expression, etc.  If there's not a send context, but the failing expression is the top-level expression, we can generate error messages about unsupported shape types if we have the right kind of expression.  Otherwise, we'll just issue the generic message this PR currently has.
Test plan

See included automated tests.

Otherwise

Another option would be to actually parse the tuple/shape type into a tuple/shape type, and allow the behavior.
Another option (depending on how many problems there are from the current approach or ^ this approach) would be to parse the type with no error and make it be T.untyped until we migrate to fix the problems.

Otherwise

Another option would be to actually parse the tuple/shape type into a tuple/shape type, and allow the behavior.

Just for clarity's sake, are you saying that this is a separate option from making errors better (and not just a substitute for the "Otherwise" clause in the above)?
I don't really get the history here, especially when type aliases work just fine, but maybe the desire was to avoid spreading shape and tuple types around too much...?

Another option (depending on how many problems there are from the current approach or ^ this approach) would be to parse the type with no error and make it be T.untyped until we migrate to fix the problems.

That is also a reasonable path.

Another option (depending on how many problems there are from the current approach or ^ this approach) would be to parse the type with no error and make it be T.untyped until we migrate to fix the problems.

That is also a reasonable path.

I'm doubting this statement after thinking for a while -- the user is expecting static type-checking of the array/hash/whatever element and Sorbet is going to silently drop type information and let everything through.  That does not seem like the best approach.


failed to generate dataset

Hi, I'm recently building the mcnn vocoder based on your code. However, when I tried to generate training dataset via the command line python preprocess.py along with the necessary parameters, it returned  0it [00:00, ?it/s] on the terminal, without any dataset generated. I wondered there could be any solution to this problem. Thanks a lot.

Hi @maximilianchen,
What parameters did you use for preprocessing data ? path/to/ljspeech/dataset is just placeholder. You should change it to right path to ljspeech dataset.

Hi @maximilianchen,
What parameters did you use for preprocessing data ? path/to/ljspeech/dataset is just placeholder. You should change it to right path to ljspeech dataset.

Hi, @tuan3w, the following is the command line I typed in:
python preprocess.py \

--out_dir ./dataset
--data_dir ./LJSpeech-1.1/wavs
--samples_per_audio 400
--num_workers 4

It should be ./LJSpeech-1.1 .

It should be ./LJSpeech-1.1 .

Thanks a lot, @tuan3w. Now it works.


Handle the reportUnusedDisableDirectives option.

This allows eslint-parallel to forward the --report-unused-disable-directives  command-line flag to eslint.

Waiting for this as well.


feat: add new substrait methods to python docs

@pdet

@Alex-Monahan are you the right person to tag for changes to docs? Unfortunately I don't have access to request reviews in this repo

Typically yes, but I am taking some time off to help care for our newborn! I'll post in the discord when I'm back to work for DuckDB Labs!


feat: project rewrite

BREAKING CHANGE: Please see https://api.phpolar.org and https://docs.phpolar.org.
Closes #58
Closes #98
Closes #66

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 95.756%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
361

Relevant Lines:
377

💛  - Coveralls


Chapter 04: Chord element not adressed in Guidelines

Chapter 4 of the Guidelines has a section 4.2.4. Notes, Chords and Rests in CMN
Subchapters are only adressing Notes & Rests, Chords are not adressed separately. Although the handling of chords can be easily understood by experienced users using the examples in the notes section, it could be difficult for beginners to find (or understand) chords.
Related to #107 ?

Fully agree. Things that should probably be documented are:

encoding of duration on chords: do we recommend to put a duration on the chord element?
chords with notes with different durations: do we recommend to put the longest duration on the chord element?
use of dots and dots.ges when some notes are missing a dot
placement of artic (within chord or within notes)


[decompiler] json crash w/ no warning on config file

which file?

the general one like jak1_ntsc_black_label.jsonc

made title less confusing


[API][Rename] Rename T5NMTInference to T5Inference

Description
Incremental decoding for T5 has been supported in https://github.com/dmlc/gluon-nlp/pull/1498. However, the inference model is called T5NMTInference and we may consider to revise it to T5Inference or T5Seq2SeqInference.
What's your opinion on the naming strategy here? @yongyi-wu @szha @leezu @szhengac @barry-jin

It seems to be kept consistent with TransformerNMTInference. I'd suggest to keep the naming convention consistent for easier discovery, so any change should happen to both classes. In terms of naming, I think Seq2Seq seems better and more general than NMT.

It seems to be kept consistent with TransformerNMTInference. I'd suggest to keep the naming convention consistent for easier discovery, so any change should happen to both classes. In terms of naming, I think Seq2Seq seems better and more general than NMT.

May be we can keep the name for now and later on add aliases.

May be we can keep the name for now and later on add aliases.

@szha was right— I kept it consistent with TransformerNMTInference, so it would be great if we rename them altogether. Personally I prefer T5Seq2seq as it is shorter and more informative.

@szha was right— I kept it consistent with TransformerNMTInference, so it would be great if we rename them altogether. Personally I prefer T5Seq2seq as it is shorter and more informative.

I will leave the issue open and we can change the name / add alias later.

I will leave the issue open and we can change the name / add alias later.

Hi all, in #1504 TransformerNMTInference and T5NMTInference have been renamed to TransformerInference and T5Inference, respectively. After discussing with @sxjscience, we believe it is critical to highlight these two models are for inference only, so we keep the name short and straightforward.
With that, TransformerNMTInference and T5NMTInference remain in #1504, simply inheriting the renamed models. We may need to remove them in the future.

Hi all, in #1504 TransformerNMTInference and T5NMTInference have been renamed to TransformerInference and T5Inference, respectively. After discussing with @sxjscience, we believe it is critical to highlight these two models are for inference only, so we keep the name short and straightforward.
With that, TransformerNMTInference and T5NMTInference remain in #1504, simply inheriting the renamed models. We may need to remove them in the future.


[Docs] Update karma docs for react >15.5

Follows the suggestion https://github.com/airbnb/enzyme/issues/940#issuecomment-310690420 and complements https://github.com/airbnb/enzyme/pull/1030

I don't think this is relevant any more now that we're on v3.


chore: (Re)Use the public version of mobile_scanner

Hi everyone,
A blocking crash on iOS was forcing us to use a forked version of mobile_scanner.
But it's now OK to use back the version, as it's fixed (actually it's also a revert of the faulty code)
Will fix #4753 4753

Codecov Report

Merging #4765 (3b04dac) into develop (21469af) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #4765   +/-   ##
=======================================
  Coverage     9.89%   9.89%
=======================================
  Files          312     312
  Lines        15809   15809
=======================================
  Hits          1565    1565
  Misses       14244   14244

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

thanks @g123k


Make Snackbar easier to use - preconditions can be complicated

Hello!
This is just concept opinion for Snackbar.
I tried to show snackbar but in constraint.
and now lots of questions on StackOverFlow for using Snackbar.
and found it answers are "in Scaffold condition", or "use GlobalKey" for this.
In compare to Android Toast, it is quite a difficult to use.
Showing Snackbar can be easier to use?

Ok so run into same problem of "scaffold context"..
and many others
I think (am newbie) the problem is that is needs to be a context within the "body" of a scaffold...
but am stuck myself completely....
Kinda expected a widget.scaffodil.context or showSnackbar() in there "somewhere"

The easiest way is to pass a GlobalKey to Scaffold, Snackbar, ... and get the state from that GlobalKey when you need to call methods on them.
A better way is to use a kind of event bus (like React, or just streams) to communicate between different parts of the app.

I also expect to use any scaffold  but it cannot work
      //     Scaffold.of(context).showSnackBar(SnackBar(
      //       content: Text('text'),
      //     ));

Try this code sample

Code Sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Container(
          child: Text('Hello World'),
        ),
      ),
      // unccomment this for wokring solution
      // floatingActionButton: FAB(),
      // comment this, not working
      floatingActionButton: FloatingActionButton(
        child: Icon(Icons.add),
        onPressed: () {
          Scaffold.of(context).showSnackBar(SnackBar(
            content: Text('text'),
          ));
        },
      ),
    );
  }
}

class FAB extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return FloatingActionButton(
      child: Icon(Icons.add),
      onPressed: () {
        Scaffold.of(context).showSnackBar(SnackBar(
          content: Text('text'),
        ));
      },
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-7.1.pre, on Mac OS X 10.15.6 19G73, locale
    en-GB)
    • Flutter version 1.20.0-7.1.pre at /Users/taha/Code/flutter_dev
    • Framework revision 7736f3bc90 (6 days ago), 2020-07-10 16:33:05 -0700
    • Engine revision d48085141c
    • Dart version 2.9.0 (build 2.9.0-21.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google
      Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (5 available)
    • SM M305F (mobile)      • 32003c30dc19668f          • android-arm64  •
      Android 10 (API 29)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            •
      iOS 13.6
    • macOS (desktop)        • macos                     • darwin-x64     •
      Mac OS X 10.15.6 19G73
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 83.0.4103.116

• No issues found!

I think this has been resolved with the ScaffoldMessenger API. The old Scaffold SnackBar methods have been removed.
There are a number of resources on the new API as well to help folks adopt it, Here are some:

https://youtu.be/lytQi-slT5Y
https://youtu.be/sYG7HAGu_Eg?t=10271
flutter.dev/go/scaffold-messenger
flutter.dev/release/breaking-changes/scaffold-messenger

I think we can close this issue now, but if there is more work to do here, please leave a comment and we'll re-open to take another look. Thanks!


Task-based Inspection for Mingsai #3

Please review A6 in 4.4.1 Assumptions section of the SRS, it is mentioned that there is a CSV file showing the 2% growth rate in the manuscript, what would be the citation for that?

Ontario Ministry of Transportation. 2022. “AASHTOWare pavement ME design traffic map.” Accessed June 1, 2023. https://icorridor-mto-on-ca.- hub.arcgis.com/apps/50798e771bd0440dbc96fd85d8fde9a5/explore.


Release gax-java v1.48.0

This pull request was generated using releasetool.
08-20-2019 09:11 PDT
Implementation Changes

Update InstantiatingGrpcChannelProvider to be tolerant of ipv6 addresses (#768)
Logs warning message when Batcher is not closed on GC (#746)
gax-grpc , gax-httpjson : remove deprecated warnings. (#760)
gax: remove deprecated warnings. (#759)

New Features

New V2 Batching API

Adding BatchingSettings getter to BatchingCallSettings.Builder (#779)
Adding v2.BatchingCallSettings to integrate with GCJ client (#766)
Moving new Batcher API to existing Batching package (#771)
Small refactoring for v2.Batcher API (#765)
Introducing auto flush in BatcherImpl (#744)

Dependencies

Update dependency com.google.auto.value:auto-value to v1.6.6 (#778)
Update deps: protobuf, grpc, opencensus, google-http-client, google-auth-library, jackson-core (#777)
Update dependency org.apache.maven.plugins:maven-javadoc-plugin to v3.1.1 (#775)
Update annotations versions (#758)
sync api-common to latest version used by google-cloud-java (#754)

Documentation

Point javadocs links to googleapis.dev (#738)

Sorry for the last minute change, but can #779 be included in this release?

Release is at https://github.com/googleapis/gax-java/releases/tag/v1.48.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


update readme, fix typo

Codecov Report

Merging #12 (c6f6678) into master (2c874b8) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
- Coverage   98.42%   98.40%   -0.03%
==========================================
  Files          11       10       -1
  Lines         636      625      -11
==========================================
- Hits          626      615      -11
  Misses         10       10

Impacted Files
Coverage Δ

src/besselj.jl
93.69% <0.00%> (-0.12%)
:arrow_down:

src/besseli.jl
100.00% <0.00%> (ø)

src/recurrence.jl
100.00% <0.00%> (ø)

src/U_polynomials.jl
100.00% <0.00%> (ø)

src/Bessels.jl

Continue to review full report at Codecov.

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


shutdown thread pool when kmeans train done

Signed-off-by: Yaliang Wu<EMAIL_ADDRESS>Description
KMeansTrainer will create ForkJoinPool to train KMeans model in parallel, code link : https://github.com/oracle/tribuo/blob/main/Clustering/KMeans/src/main/java/org/tribuo/clustering/kmeans/KMeansTrainer.java#L284-L293
But the ForkJoinPool doesn't shutdown when training complete. As KMeansTrainer doesn't set keepAliveTime for ForkJoinPool, the default keep alive time is 60 seconds. Refer to https://docs.oracle.com/javase/9/docs/api/java/util/concurrent/ForkJoinPool.html

keepAliveTime - the elapsed time since last use before a thread is terminated (and then later replaced if needed). For the default value, use 60, TimeUnit.SECONDS.

It's risky when kmeans training requests come in high volume in short time (<60seconds).
Motivation
< Why is this change necessary? Is it a bug fix or new feature? >
Bug fix
Issue: https://github.com/oracle/tribuo/issues/223
Paper reference
N/A

@Craigacp Thanks, I'm asking approval to add me to Amazon's corporate OCA. May take some time. I published a new commit to narrow the try scope. How about your review the code first while we are waiting?

LGTM, but we'll need to wait for the OCA to come through.

Thanks I got company's internal approval. Sent an email to<EMAIL_ADDRESS>waiting for reply.

LGTM, but we'll need to wait for the OCA to come through.

Thanks I got company's internal approval. Sent an email to<EMAIL_ADDRESS>waiting for reply.

Ok. Hopefully that will be quick.

The OCA came through, so I've merged this in. Thanks for contributing.

Thanks @Craigacp , will you release a new version including this fix soon?

I'd like to get #225 merged in, and also I'm going to wait for TF-Java 0.4.1 to be released, which should be soon (we're just working out if we need another fix for a different issue before tagging the TF-Java release). Once that's done then we'll tag 4.2.1.

Cool, that's nice. thanks a lot, will keep an eye on the new version 4.2.1


⚠ Bofrost.si has degraded performance

In eb3ae29, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 2410 ms

Resolved: Bofrost.si performance has improved in 5162cce after 2 days.


Prompt is printing all environment variables

I'm submitting a ...

[X ] bug report
[ ] feature request
[ ] support request

Do you want to request a feature or report a bug?
Report a bug

What is the current behavior?
When I installed this today on a new system using Antigen, it is outputting all environment variables in the prompt. I see that the theme was updated yesterday. Possibly a bug? I even changed to directly using the git repo and it still has the bug.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

Install Oh My Zsh
Install Antigen
Include in your .zshrc file: antigen theme https://github.com/eendroroy/alien-minimal.git alien-minimal

My prompt looks like this: https://gist.github.com/sibblegp/dc6f656123e0ce1ef6e7d45b63a0bade

What is the expected behavior?
Not to print all environment variables.

What is the motivation / use case for changing the behavior?
Have it work like it usually does.

Please tell us about your environment:

Plugin Manager: antigen
ZSH Version: 5.4.2

Hmm, that is odd! I'm not seeing this issue. I'm on tag 1.4.4.

I don't see it as well.
@sibblegp please try deleting below folder and restarting the shell.
rm -rf ~/.antigen/bundles/eendroroy/alien-minimal

Line 119 in prompt.zsh:
__import_env "${2}"
is what is causing all of the environment variables to be printed. When I remove it, they no longer appear.

Tag 1.4.3 introduced the printing, and I had to go back to 1.4.0 to get no errors. But at least I have a working copy now!

I've experienced the same and described here https://github.com/eendroroy/alien-minimal/issues/22#issuecomment-527629055

I have made some changes. Please checkout to latest master. Let me know if it resolves the issue.

Although now the left prompt tag is not appearing. I have these settings in my .zshrc file:
export AM_PROMPT_END_TAG=' $'
export AM_PROMPT_END_TAG_COLOR=83

And there is no $ on my prompt.

Although now the left prompt tag is not appearing. I have these settings in my .zshrc file:
export AM_PROMPT_END_TAG=' $'
export AM_PROMPT_END_TAG_COLOR=83

And there is no $ on my prompt.

Please try below settings:
export AM_PROMPT_END_TAG='\ $'
export AM_PROMPT_END_TAG_COLOR=83

Thanks!


update my fork

This seems to be the way to get my fork up to date...

not necessary after all...


Setting up test net

We should make a test net for public access to explore Iroha

@BAStos525 @Cre-eD who could take this under their wing?


Workflows "Started" / "Finished" should be showed in detailed date and time

Summary
STARTED and FINISHED information should contain date and time. This should be done at

Workflows view
Archive view
detail view for Workflows

Use Cases
We use CronWorkflows a lot and with a schedule trigger more than 10 times a day. After one day in lists you can´t see which one runs fine and which one has failed. Every workflow has the same information like "Start time = 1d ago" (or one 1h ago and so one).
The label "Start time" point to the fact a user might expect - a concrete time.

The list is used by our developers to check out the success of our recurring processing.
Also the detail view could not be helpful at this point.

In my opinion this is relevant for all other processes done more than one time a day with the same (generated) name.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

When you hover over "2d ago", you should see a detailed timestamp.

This is an unnecessary and frustrating step when the detailed timestamp could be displayed instead of the diff.  How long ago something occurred obfuscates and inherently adds less value than when something occurred.

close since https://github.com/argoproj/argo-workflows/pull/13284 is merged?


Oversized Footprint Outline

I have a small FPGA in my design that has a properly defined footprint but the red outline in the ibom.html output is extending way past the courtyard. Part is U18 below...

After closer examination it seems the plugin is using the dimension line objects on the User.Comments layer to determine the extents for the part. I added a yellow box around the dimension lines after doing a screen grab from the Footprint Editor...

The red and yellow frames are perfectly correlated. If the courtyard is defined then that might be the best object to use to determine the red box dimensions. If there is no courtyard then use the Fab layer objects and barring that perhaps use the silk and pads. In any case I wouldn't use the User.* layer objects for this purpose.
Great plugin! 👍

Ibom doesn't look at any specific layer contents, it uses bounding box logic of kicad. If kicad thinks those drawings are within the box, ibom will draw it truthfully.
I agree that dimension objects should not be included when text is not included. But that should be fixed on KiCad side.

I see. KiCad doesn't also consider the associated text so there's some inconsistency there for sure. It's a minor thing so moving on.

Fixed in kicad with https://gitlab.com/kicad/code/kicad/-/merge_requests/1522. This should be in 7.0.2


Incompatibility with attribute_normalizer gem

Please see this issue:
https://github.com/mdeering/attribute_normalizer/issues/21
Basically, whenever I use
rails generate x
I get the following error:
retrieve_connection': ActiveRecord::ConnectionNotEstablished (ActiveRecord::ConnectionNotEstablished)

I had a quick look at this but couldn't see any obvious cause.

Thanks for helping air blade. It has been fixed on the attribute_normalizer side:
https://github.com/mdeering/attribute_normalizer/pull/19

Thanks for letting me know.


add addPrivateAccessModifier to configuration part of package.json

Counterpart to https://github.com/fsharp/FsAutoComplete/pull/1089
WHAT

🤖 Generated by Copilot at 11c5024
Add a new extension setting FSharp.codeFix.addPrivateModifier to enable or disable a codefix for adding private access modifiers to F# declarations. This is part of a pull request that improves the codefix feature and fixes some bugs.

🤖 Generated by Copilot at 11c5024

FSharp extension
Adds a new codefix option
Private in autumn

:sparkles::wrench::gear:

WHY

HOW

🤖 Generated by Copilot at 11c5024

Add a new configuration option FSharp.codeFix.addPrivateModifier to enable or disable a codefix that adds a private access modifier to F# declarations (link)

Thanks for this @dawedawe - will merge and try to get a new release out soon that includes your FSAC codefix as well.

Would you mind updating the configuration docs at https://github.com/ionide/ionide.github.io as well?


Add tests for integration test btw RHODS and Starburst Enterprise Operator

Adding coverage for ODS-2247 and ODS-2249 (i.e., install and use starburst enterprise operator).
As a side product of this PR, we got a set of keywords to Install and Uninstall ISV operator from CLI

tested against RHODS 1.28: 1873 SUCCESS


Why is Microsoft suggesting Godaddy ???  Its a very bad company.

Why is Microsoft supporting Godaddy?  I'm looking to move my business from Workspaces to O365wteams but this is very alarming.  Godaddy is a predatory company.  Don't get me wrong, they are big but many of their products are designed to take advantage of the little people or don't work at all.
For example, I've paid for Godaddy service (and #1 in line to buy) a domain if/when it expires and is not renewed nearly 10-12 years ago.  A year ago, I checked on it and it did finial expire without renewal but somehow a subsidiary of Godaddy bought it and never offered it to me per our contract - now they are trying to sell it for $16k.  This is just one example of 10 or 20 things they do that are very questionable.
Godaddy is a shady business and I'm surprised as hell that Microsoft even suggests supporting them.

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

ID: fb572a6e-0bd1-9609-4cbe-021deb497da7
Version Independent ID: 7f82b19b-d569-c955-2e2d-20ec3112b3ad
Content: Buy a domain name - Microsoft 365 admin
Content Source: microsoft-365/admin/get-help-with-domains/buy-a-domain-name.md
Service: o365-administration
GitHub Login: @efrene
Microsoft Alias: efrene

@Derek-Anderson Thank you for your feedback. You can provide the feedback on our Main feedback portal: https://feedbackportal.microsoft.com/
This channel is only for updating the documents and we do not have the exact answer to your query.
Hope this helps!
Thanks
Sri

@Derek-Anderson Thank you very much for your kind feedback. Considering your valuable feedback, please log this issue as advised. Here we are limited to documentation fixes and improvements only. Concerning team will evaluate and take the appropriate action, as per their schedule. We proceed here to close it. Thanks again for taking out some time and sharing your feedback.

godaddy is best for domain buy. i have buy more then 100$ from godaddy , if you check domain valuation. i have also buy from dynadot and namecheap very bad experience .


fixing-typos-my-first-client

This PR contains changes to the following docs:

My First Client: Typos only
Wallet Admin Dashboard: Updated instructions to include login information
My First Transaction: Closes #7368

/land

/land


make lint and it's parent dep targets are throwing errors

golint -set_exit_status cmd/... works, along with golint-set_exit_status pkg/...
Passing both does not:
$ golint -set_exit_status cmd/... pkg/...
open cmd/...: no such file or directory
open pkg/...: no such file or directory

^ Above command is what make lint runs.

Usage of golint:
        golint [flags] # runs on package in current directory
        golint [flags] package
        golint [flags] directory
        golint [flags] files... # must be a single package

I think that it needs to be two different calls.

You need to update your golint. It used to be two calls but then @djzager showed me his worked. I verified the md5sums were different.
The following version works.
c0cd41a997f2fb5c6194cd5fac39a701  /home/jesusr/dev/bin/golint

$ golint --help
Usage of golint:
	golint [flags] # runs on package in current directory
	golint [flags] [packages]
	golint [flags] [directories] # where a '/...' suffix includes all sub-directories
	golint [flags] [files] # all must belong to a single package
Flags:
  -min_confidence float
    	minimum confidence of a problem to print it (default 0.8)
  -set_exit_status
    	set exit status to 1 if any issues are found

Notice that it now supports [directories].


Add site.github.public_repositories[].contributors

Hello,
After I wrote #233, I saw a merged PR about site.github.public_repositories[].releases (#224 by @parkr )
I've never used ruby before, but with help of my friend @peniar , I made this PR. (thanks, @peniar)
Here're what I did:
First, I changed the code lib/jekyll-github-metadata/repository.rb in my installed gem, and check it worked.
And then, I wrote some spec code referring previous PR about release, assumed that return should be like spec/webmock/api_get_repo_contributors.json
Lastly, I tested with bundle exec rspec spec, and then  1 test I added failed as follows:
Failures:

  1) integration into a jekyll site calls all the stubs
     Failure/Error: expect(stub).to have_been_requested

       The request GET https://api.github.com/repos/jekyll/github-metadata/contributors?per_page=100 with headers {'Accept'=>/application\/vnd\.github\.(v3|drax-preview|mercy-preview)\+json/, 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'Authorization'=>'token 1234abc', 'Content-Type'=>'application/json', 'User-Agent'=>'Octokit Ruby Gem 4.22.0'} was expected to execute 1 time but it executed 2 times

I think the error expected to execute 1 time but it executed 2 times is because the line 16 in spec/spec_helpers/stub_helper.rb conflict with line 26 which I added. However, line 15 for releases code does not conflict with line 25. :(
I'm stuck here and don't know what to try or look for anymore.
If anyone could point out this PR, I would be very grateful.

Thanks for your review, @parkr
How about the following method? I made it work only when JEKYLL_GITHUB_METADATA_DETAILS is set to true by using an environment variable.
If you have any ideas on how to pass this flag or name convention, it would be nice if you could let me know. :)
      def owner_public_repositories
        options = {
          :type   => "public",
          :accept => "application/vnd.github.mercy-preview+json",
        }
        if ENV["JEKYLL_GITHUB_METADATA_DETAILS"] == true
          memoize_value :@owner_public_repositories, Value.new("owner_public_repositories", proc do |c|
            c.list_repos(owner, options).each do |r|
              r[:releases] = c.releases(r[:full_name])
              r[:contributors] = c.contributors(r[:full_name], options)
            end
          end)
        else
          memoize_value :@owner_public_repositories, Value.new("owner_public_repositories", proc { |c| c.list_repos(owner, options) })
        end
      end

I was able to get this to pass on my machine with the following change:
diff --git lib/jekyll-github-metadata/repository.rb lib/jekyll-github-metadata/repository.rb
index 1e4864c..ba49d95 100644
--- lib/jekyll-github-metadata/repository.rb
+++ lib/jekyll-github-metadata/repository.rb
@@ -118,9 +118,11 @@ module Jekyll
           :accept => "application/vnd.github.mercy-preview+json",
         }
         memoize_value :@owner_public_repositories, Value.new("owner_public_repositories", proc do |c|
-          c.list_repos(owner, options).each do |r|
-            r[:releases] = c.releases(r[:full_name])
-            r[:contributors] = c.contributors(r[:full_name], options)
+          c.list_repos(owner, options).map do |repo|
+            r = Sanitizer.sanitize(repo)
+            r[:releases] = Value.new("owner_public_repositories_releases", proc { c.releases(r["full_name"]) })
+            r[:contributors] = Value.new("owner_public_repositories_contributors", proc { c.contributors(r["full_name"]) })
+            r
           end
         end)
       end
diff --git lib/jekyll-github-metadata/sanitizer.rb lib/jekyll-github-metadata/sanitizer.rb
index edaf554..0a993d3 100644
--- lib/jekyll-github-metadata/sanitizer.rb
+++ lib/jekyll-github-metadata/sanitizer.rb
@@ -16,7 +16,7 @@ module Jekyll
       # resource - an Object
       #
       # Returns the sanitized resource.
-      # rubocop:disable Metrics/CyclomaticComplexity
+      # rubocop:disable Metrics/CyclomaticComplexity, Metrics/MethodLength
       def sanitize(resource)
         case resource
         when Array
@@ -31,6 +31,8 @@ module Jekyll
           nil
         when String
           resource
+        when Value
+          resource.render
         else
           if resource.respond_to?(:to_hash)
             sanitize_resource(resource)
@@ -39,7 +41,7 @@ module Jekyll
           end
         end
       end
-      # rubocop:enable Metrics/CyclomaticComplexity
+      # rubocop:enable Metrics/CyclomaticComplexity, Metrics/MethodLength

       # Sanitize the Sawyer Resource or Hash
       # Note: the object must respond to :to_hash for this to work.

I don't think we want to make this an optional change – we can just load only when requested using Value.

Thanks for the detailed guide, @parkr!
As per your guide, I wrapped the existing codes to use Value.new() and changed the Sanitizer module as well. I definetly agree that we do not want to make this an optional change, but honestly I'm still not sure what Value does. 😅 I think I need to study the whole code as well as the syntax of ruby more.
Anyway, please let me know if there is anything else I need to change or do.

(TMI: My friend @peniar received e-mail when I commented, and he also taught me how Value works and how/why to use it. Thanks, @peniar!)

I just removed spec related codes, and am waiting for CI results!

@9bow Sorry, I just meant the one line, not all your specs! I like your tests. Just remove the one line I pointed out, and combine the loop you added in spec_helper with the one right above it.

Oh, I'm sorry @parkr. I misunderstood.
I'll revive those Spec codes and re-apply your review.

I've been re-applied your review. Also verified that it passed the test locally as follows:

Sorry for the hassle and thank you for your kind and patient review!

@jekyllbot: merge +minor

Thank you very much, @parkr!
With your review, this PR was a great experience for me. :)

Could you please let me know when the next release schedule is?
I'd love to try out this new feature. 😄

v2.14.0 has been released!

Awesome! Thanks a lot @parkr!
However, when I try to apply it, a conflict with the dependency of the github-pages gem.
Could you please help on this part?

I've finally rolled out this cool new feature on our site! :tada:
I made it automatically show repositories from @PyTorchKorea, which has 2 or more contributors
Unfortunately, it's now before the official release, so I'm referring to my personal repository.
Again, thank you very much, @parkr!


Support external file system targets (CDN)

Add support for external file systems targets (CDN) like S3, CloudFlare, Cloud Storage
Can be implemented by checking if the target is an instance of FileSystemTarget. Otherwise it needs to create a local copy of the resource, optimize it and publish/replace it in the target. Possible making it blocking.

I have no plans of implementing this feature, but happily accept pull requests for it. Would also be happy to do implement it upon request if anyone wants to sponsor it.


Cannot run command when invoking from python

[Enter feedback here]
Does this command have any restrictions WRT how the command line console is spawned ?
While I can successfully use it in a cmd (or PowerShell) console I cannot run it as a shell command from another environment.
For example, from python (3.7) using
os.system(‘cmd /c “query session”’) results in “query command not found” (not an exact quote).
Simple command such as echo does work this way.
Thanks
Alison

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

ID: bdd5fe77-ea0e-0f44-2884-ee8699c6520e
Version Independent ID: 478e738b-680f-038c-636c-4956dc42598f
Content: query commands
Content Source: WindowsServerDocs/administration/windows-commands/query.md
Product: windows-server
Technology: windows-commands
GitHub Login: @eross-msft
Microsoft Alias: lizross

#assign:IngridAtMicrosoft

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


No lights drawn on WindowsDX

I seem to have a really weird issue with Penumbra where the whole screen is dimmed as usual but no lights are being rendered on the screen. I ran the HelloPenumbra test project and the result is this:

I am using the latest MonoGame version (3.7.1)
Can anyone please test this too and comment the results? please?

Same project but on WindowsGL works perfectly

If you tell me to use WindowsGL then instead of WindowsDX the answer is simple:
WindowsGL Monogame projects consume a lot of RAM, it's like 3 times as much as the same WindowsDX projects. Also loading any content file through the content pipeline takes at least 3-4 seconds.

Fixed with v1.4.1. Feel free to reopen if issue persists.

Fixed with v1.4.1. Feel free to reopen if issue persists.

fixed indeed. perfect, thanks!


FileSystemCollector stalls on Mac OS

Thanks @gfs I will wait till this is fixed.

@yitam You should be good to go.

Thanks @gfs I will check that later, but the stalling problem in ubuntu 18.04 (shown in #328 ) still exists.

@yitam Investigating.

@yitam I believe this is now fixed for good on both platforms.

I believe this is now fixed for good on both platforms.

Unfortunately, @gfs, this is not the case...  in a clean Ubuntu 18.04 I now see different errors. On Mojave, it's hung. Should I create two new issues?

Yes.
-------- Original Message --------
On Jan 6, 2020, 10:39 AM, Jenny Tam wrote:

I believe this is now fixed for good on both platforms.

Unfortunately, @gfs, this is not the case... in a clean Ubuntu 18.04 I now see different errors. On Mojave, it's hung. Should I create two new issues?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


KLayout on Windows struggles to import gdstk while running gf macros

Describe the bug
When trying to import PCell library in KLayout, it fails to import gdstk, and the entire process doesn't work.
To Reproduce
gdsfactory menu -> import_generic_PCells. I modified the error message only to make it more verbose.

Suggested fix
This is a known issue with KLayout's Python.
https://www.klayout.de/forum/discussion/1834/nazca-module-is-not-recognized

It can import flayout perfectly fine, it just can't import gdstk because it is stored as a .pyd under the same directory. KLayout's Python cannot import .pyd

Yeah this is the same issue I had with gdspy before :/

have you tried setting the PYTHONPATH to the directory with the *.pyd files?
https://stackoverflow.com/questions/50278029/how-to-import-a-pyd-file-as-a-python-module

Hey @tvt173,
According to Matthias in the link I posted, this won't work.

My understanding is that the KLayout Windows program is compiled with MSYS2 while Python C-extensions are all compiled with the MSVS compilers, hence no binary-compatibility.

My understanding is that the KLayout Windows program is compiled with MSYS2 while Python C-extensions are all compiled with the MSVS compilers, hence no binary-compatibility.

does that mean you could potentially fix the issue by compiling Klayout with the MSVS compiler?
better long term solution might be to launch a gdsfactory server on a separate process (choose which python you like) and communicate with Klayout via a client API.

My understanding is that the KLayout Windows program is compiled with MSYS2 while Python C-extensions are all compiled with the MSVS compilers, hence no binary-compatibility.

does that mean you could potentially fix the issue by compiling Klayout with the MSVS compiler?

Absolutely! I've been able to get the GUI version built on Windows using the MSVC2017 compiler and it works just fine with gdstk :)

@thomasdorch I think a conda install is a great option. nice work on getting that running! but it would also be great for the standard Klayout to have better support for these packages as well. I'd imagine this would be an issue for many compiled python packages and not just the ones we're dealing with here, right?
@klayoutmatthias, is there any particular reason for choosing MSYS2 as the compiler for Klayout Windows version, or would you be open to changing to get better python support?

thanks Matthias. sounds good


Add Address published on 05-28-24

Hey all,
I'd like to contribute to this list so I added the most recent new address from the OFAC notices.
https://ofac.treasury.gov/recent-actions/20240528_33
yunhe, WANG

@JustinDrake  could you take a look at this?


[Feature Request] Ability to pass rows type as an TS Interface/Type.

Make table configuration accepting a typescript interface to strongly typed rows data,
This might add some flexibility and confidence while loading data to the table, and also make it easier to manipulate data in the tables.

In my current project we have a huge typed abstraction for the smart table settings. But for the original settings, though.
I am too reluctant to provide this as a pull request here, because it is very specific and tends to cover only the settings we actually use in our project. Also, we implemented it as a layer on top of the (previously) untyped settings and after migrating to this fork, we just changed that layer to be compatible with this fork.
In our settings functions like valuePrepareFunction are statically typed as well as the column properties. So let's say you have an object
class Foo {
   bar: number
   baz: boolean
}

You would create smart table settings like this
settings = {
    // ...
   bar: {
       // bar column settings
   },
   baz: {
       // baz column settings
   }
}

Our implementation checks that you spell the property names correctly, and also checks that the type in the column settings is compatible. We also have helper functions makeTextColumn, makeNumberColumn, makeEnumColumn etc. that spare you the verbose and repetitive definition of column settings.
It is possible that I gradually formulate pull requests for these features some time in the future. But again, they are very specific and very custom so maybe it would be wise to just provide them as an add-on layer (as it is right now) because I do not want to force everybody to think they way we did in our project.

It would be nice of course, if we had a look at your abstraction, Maybe we can came up together with something more generic starting with your team's abstraction.
In my codebase we have tons of repetitive tables settings in various components, Hopefully I can contain this amount of code by migrating to this fork soon.

Please @uap-universe try to share as much as you can, because i think that your solution is good to eliminate an all kind of problems working with the table! Thanks

relates to #65

I am using a very old management trick and close this ticket in favor of continuing the discussion in #127. This way it looks like we are not sleeping on the job 😆😆


KAFKA-3929: Add prefix for underlying clients configs in StreamConfig

Add prefixes for consumer and producer configs to StreamsConfig, but be backward compatible.

@guozhangwang @enothereska please take a look. I added a couple of helper functions to StreamsConfig to apply the prefixes. Of course users can also do it by StreamsConfig.PRODUCER_PREFIX + "someProp"

LGTM

@guozhangwang can you please take a look?

LGTM overall, a few minor comments.

Updated @guozhangwang - thanks

Jenkins failures are un-related. Test passed locally and one more documentation comment.

Thanks @guozhangwang - java doc updated


Only getting partial data back on continuous write mode.

Hi,
We are facing an issues with NMSSH.We have reused the examples provided (PTYExample).
We send command one at a time and we got the complete data back.
We send commands one by one (after getting the result of first command we send the next command.....) it worked for some set of commands but after some time we are only getting the partial data result back.
We are getting the following logs from NMSSH
NMSSH: Return code of response -37, error -37
NMSSH: Data available on the socket!
Now if we give one more command then we will get the previous partial data result along with the new data result.
Could you please help us with this issue of not getting complete data.

Hi,
Could someone please help me with this?


Fixing Unloading Kmod on a previously failed Load Pod (#530)

This PR fixes 2 issues:

In case Load Pod is failing, its Config in status is nil.
if NMC is configured to un load the module, for whatever reason,
the code must also check if the Config is nil, and then just to
delete the status for module, in order to avoid panic
When unload Pod is failing, its status should be retained as is,
otherwise the Config is lost, and the following unload pod won't be
able to run

/lgtm


new journal entry buffer ignores .dir-locals.el

I am using org-journal-mode. I have created .dir-locals.el in my ~/Documents/journal directory and it gets correctly applied if I open a file the usual way. However, if I hit the C-xC-j to add a new entry and the entry is in a new day, the opened buffer does not have the modifications I apply through .dir-locals.el.
Can I make it recognize and process .dir-locals.el for new day journal entry buffers?
My .dir-locals.el:
((nil
  (fill-column . 79)
  (indent-tabs-mode)
  (show-trailing-whitespace . t)
  (whitespace-style face trailing lines-tail space-before-tab indentation empty)
  (whitespace-newline . t))
 (org-journal-mode
  (typo-language . "Czech")
  (typo-mode . t)
  (ispell-local-dictionary . "czech")
  (flyspell-mode . t)
  (whitespace-action warn-read-only auto-cleanup)))

(cf. http://emacs.stackexchange.com/questions/14247/org-journal-mode-new-entry-ignores-dir-locals-el)

What are the modifications you would expect to see?
The problem is probably, that org-journal-mode simply "types" a few fixed strings into the journal file. Org-mode, or any other customization really doesn't have much say in this process. In other words, your settings should be active, but they probably won't affect org-journal much.

@bastibe : Well, I want the typo-mode active with the appropriate language and also spell checking with the same language active. That works if I just C-xC-f open the file but it does not work when I just create new entry.
Sorry, if this is not the right forum for this issue. I am not sure where else to take it.

Please tell me what you expect to be seeing and what you are seeing. I still don't know what your problem is.

Please see this screencast: https://www.youtube.com/watch?v=ExGhwoQVqOg

In your screencast, the syntax of .dir-locals seems to be wrong (as per the manual).
In the screencast, you use
...
  (org-journal-mode
    (typo-mode . t)
    ...
    (flyspell-mode . t))
  ...

But according to the manual, the syntax should be
...
  (org-journal-mode .
    ((typo-mode . t)
    ...
     (flyspell-mode . t)))
  ...

Did this fix your issue?

You were right. Sorry, I forgot about this. I am closing this.


Update hid\hclient

hid\hclient has two issues that are fixed with these changes:

Extended Client Call HidP_SetUsageValueArray currently does nothing because its ListLength is hardcoded to 0

Continuous Asynchronous Reads no longer infinitely loops if a device returns a non-success code on an asynchronous read.

Didn't work as expected. Closing.


⚠ Spryr has degraded performance

In 33f68f0, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3674 ms

Resolved: Spryr performance has improved in 58be3ad after 7 minutes.


Andrei Internship Prep

[x] fill out ORNL, ANL, and INL generic applications for Summer 2018 (if they're open)
[x] update CV and either send it to @katyhuff or upload it somewhere on the web and let me know the link.
[x] look for papers related to MPACT MSR development.

After these things are done, remind katy to send email introductions.

@katyhuff I'm about close this issue.
My CV here: https://github.com/andrewryh/CV/blob/master/cv.pdf
Looks like time to send emails to people in National Labs but I know you crazy busy now....

Looks great. Quick recommendation: On line https://github.com/andrewryh/CV/blob/master/cv.tex#L76 use `` on the left and '' on the right instead of double quotes. LaTeX will handle it better.


Rename SharedTileIterator to TileIterator to reduce redundancy

TileIterator works for both global memory tiles and shared memory tiles. There is no significant difference between these two, as both are contiguous memory compared to distributed thread-local register files.
Therefore, shall we rename SharedTileIterator to TileIterator to simplify the concept?

Agreed, we can rename SharedTileIterator to TileIterator so that it can be used for both Global and Shared memory.

Let me make this change.


Fixing Broken Images for the Basic Roleplaying System

Changes / Comments
A user of the sheet notified me that some of the images on the sheet had a "Broken Link_ notification. I fixed it by finding or uploading new images to use on the sheet instead.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

It might worth uploading the images to github along with the other files to prevent the external image host from breaking sometimes.


How to terminate or exit Mage locally? (Not Docker)

I have been looking for hours for any way to disable/exit/terminate/close the local mage process in my linux terminal but every time I do that it just restards. I am unable to run my Docker Container because the local process is running the "mage start etl-demo" automatically and the port remains occupied by it.
There is not a single reference in the Docs on how to terminate the process or exit mage in general.

you can find the process id  and run kill -9 [process_id]

you can find the process id and run kill -9 [process_id]

I tried that plenty of times, it just restarts

you can find the process id and run kill -9 [process_id]

I tried that plenty of times, it just restarts

do you have any background job to restart Mage automatically? i don't know how you set it up

you can find the process id and run kill -9 [process_id]

I tried that plenty of times, it just restarts

do you have any background job to restart Mage automatically? i don't know how you set it up

None, but i found a possible workaround. when i list all processes that contain the word "mage" I find 4 of them, terminating them all together in a 1 liner fixed it

you just need to terminate the root one

Root constantly changes process_id, so catching it in a kill command was unfeasible, I noticed the rest of them had fixed processes ids and terminated them in a 1 liner, it didn't restart after doing that

Root constantly changes process_id, so catching it in a kill command was unfeasible, I noticed the rest of them had fixed processes ids and terminated them in a 1 liner, it didn't restart after doing that

I'm curious about how you run Mage. sounds very wierd.

Root constantly changes process_id, so catching it in a kill command was unfeasible, I noticed the rest of them had fixed processes ids and terminated them in a 1 liner, it didn't restart after doing that

I'm curious about how you run Mage. sounds very weird.

Usually through Docker but I might have pip installed it a while ago for some reason. So when i tried running the container and launching the UI, the example file was the only thing there is. So i suspect it to be a Linux process and when I did it turned out to be true. The hassle was that I wanted to do something like mage quit or mage terminate using the mage commands but found out there wasn't much there