Update server without delay if nobody is connected

Context

The Updater should not wait to update the server, if there are no Players connected.

Choices

To check how many players are connected we are using RCON, since there is currently no other method
If players connects after the script has sent the message, the player will not know that there will be an update (if no discord webhook is used). This reduces the chance of this happening.

Test instructions

set the following environment variables:

AUTO_UPDATE_ENABLED=true
AUTO_UPDATE_WARN_MINUTES=1
AUTO_UPDATE_CRON_EXPRESSION=* * * * *

Connect to the Server with a client
change the buildid in palworld\steamapps\appmanifest_2394010.acf to a different value
verify that the script waits for the time set in AUTO_UPDATE_WARN_MINUTES
wait for the server to restart
change the buildid again without connecting
verify that the server is being stopped immediately after the cronjob was ran.

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I've added documentation about this change to the README.
[x] I've not introduced breaking changes.

We probably wanna combine this with my PR: https://github.com/thijsvanloef/palworld-server-docker/pull/347
Both need to get the playerlist

We probably wanna combine this with my PR: #347 Both need to get the playerlist

I was actually thinking we make a function in helper_functions called get_player_count.

We probably wanna combine this with my PR: #347 Both need to get the playerlist

I was actually thinking we make a function in helper_functions called get_player_count.

Should I create a separate PR for this or does someone else wanna do it?

We probably wanna combine this with my PR: #347 Both need to get the playerlist

I was actually thinking we make a function in helper_functions called get_player_count.

Should I create a separate PR for this or does someone else wanna do it?

You could maybe just create a new PR with both our features implemented.
Or you could just copy my feature this pr and extract the players count to the helpers function.
Doesn't really matter to me :)

I have added the new helper function get_player_count. Now the auto_reboot.sh and update.sh script both use this function to check how many players are online.


Add qos_metrics, scheduling, notifications to GET connection request

Add qos_metrics, scheduling, notifications to GET connection request.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.3%

Covered Lines:
1082

Relevant Lines:
1922

💛  - Coveralls

Let's close #337 together w/ this PR.


Fix wrong option name in README Changelog

It looks --mysql-use-alter is removed. but README Changelog says --mysql-alter-index.
ref: a5a94893f72bcd4e4479f7ef7f082251803726a4 and https://github.com/winebarrel/ridgepole/pull/330
Am I missing something?

Thank you!


Create an automated functional test recipe

Create one automated functional test to server as a recipe for creating automated test cases.

@ashakunt Can you assign this to Alapan? To be completed by 4 March

@Alapan is already working on this now.

A couple of test cases were written with Selenium (using Eclipse, as the available documentation is better), and the corresponding tests with Robot will be finished tomorrow.

The tests have been written with Robot as well. More tests were written with Robot than Selenium.

@Alapan Will you open a PR on this task? This enables us to review your work / code.

@bajiat A pull request was created.

To install Robot framework, I executed the following commands:

sudo apt-get update
sudo apt-get install python-pip python-dev build-essential
pip install robotframework
sudo pip install robotframework-selenium2library
Verify installation went OK:
pybot –version
python
in interpreter, type “import Selenium2Library”
(if OK, no errors will be shown, press ctrl+D to exit)

To run the tests, in the folder having tests.txt, the command to be executed is :
pybot tests.txt

I created a new pull request with branch 'test-recipes' and deleted the 'tests' branch. This was because the earlier branch had conflicts since it was mistakenly created on top of the 'master' branch, and not 'develop'. The commits from 'tests' were cherry-picked to 'test-recipes', and the earlier pull request referencing this issue was closed.

I removed the workspace folder with Selenium tests. The plan is to stick to JavaScript as much as possible and instead use the text-based Robot scripts for UI automation.


Table data__WOquantityNormalized__Value__Value has 2 long text fields

ATTENTION: Table data__WOquantityNormalized__Value__Value has 2 long text fields, and therefore no UNIQUE index!

Do you have an example input triple?

Just to say, this is not an error per se, the data will be generated correctly, it just might allow duplicate rows in the table. It is unclear if that's an actual issue yet.


spanner: can not create database containing - in the database id

Client
spanner
Describe Your Environment

Go: go1.11.1 darwin/amd64
google-cloud-go: v0.25.0

This is a test code to reproduce the bug.
package main

import (
	"context"
	"fmt"

	adminapi "cloud.google.com/go/spanner/admin/database/apiv1"
	adminpb "google.golang.org/genproto/googleapis/spanner/admin/database/v1"
)

func main() {
	ctx := context.Background()
	client, _ := adminapi.NewDatabaseAdminClient(ctx)

	projectId := "<PROJECT_ID>"
	instanceId := "<INSTANCE_ID>"
	_, err := client.CreateDatabase(ctx, &adminpb.CreateDatabaseRequest{
		Parent:          fmt.Sprintf("projects/%s/instances/%s", projectId, instanceId),
		CreateStatement: "CREATE DATABASE test-1",
	})
	if err != nil {
		fmt.Println(err)
	}
}

Expected Behavior
The table is successfully created.
(Syntax reference: https://cloud.google.com/spanner/docs/data-definition-language#create_database)
Actual Behavior
Following error happened.
rpc error: code = InvalidArgument desc = Error parsing Spanner DDL statement: CREATE DATABASE test-1 : Syntax error on line 1, column 21: Expecting 'EOF' but found '-1'

Additional
From web console, I was able to create a database containing - in the database id.

This is a Spanner issue, not a Go client issue. @snehashah16.

Hey Jonathan, afair this should be allowed .  I have created databases in the past with hyphens and underscores . I could be wrong and I can reproduce this. Could u give me the error response

As Jonathan mentioned, this is the Spanner API returning an error, not the client library. So, I'm going to close this issue and I'll create an internal bug for the API authors. Thanks for mentioning this to us @snehashah16.


feat: start both http and grpc server in udf

Signed-off-by: jyu6<EMAIL_ADDRESS>start both http and grpc server in udf

I am thinking we should remove the support for HTTP. In golang it is easy to do a go routine and serve both HTTP and gRPC requests. In python, it won't be that easy. Maybe we should enforce namespaced installation starting with the introduction of gRPC.
WDYT?


KubeMemoryOvercommit always return empty sets at prometheus /graph

What happened?
is the memory overcommited (KubeMemoryOvercommit) a bug?
This always return a empty result
(sum(namespace:kube_pod_container_resource_requests_memory_bytes:sum)
        /
      sum(kube_node_status_allocatable_memory_bytes)) >  ((count(kube_node_status_allocatable_memory_bytes)-1)
        /
      count(kube_node_status_allocatable_memory_bytes))

This will return a %
(sum(namespace:kube_pod_container_resource_requests_memory_bytes:sum)
        /
      sum(kube_node_status_allocatable_memory_bytes))

This will also return a %
 ((count(kube_node_status_allocatable_memory_bytes)-1)
        /
      count(kube_node_status_allocatable_memory_bytes))

Did you expect to see some different?
Should return true/false?
How to reproduce it (as minimally and precisely as possible):
just helm install it
Environment
EKS (helm install kube-prometheus-stack-13.5.1)
Anything else we need to know?:

This repository is not shipping helm charts, in the future please provide kube-prometheus branch that was used to deploy the stack.

The current version of the alert is using different expressions to match metric names change in kube-state-metrics v2. Plus it is using suggestions from this bug report. Here is a new expression: https://github.com/prometheus-operator/kube-prometheus/blob/main/manifests/kubernetes-prometheusRule.yaml#L263-L278
I am closing this as the alert works correctly when a stack is deployed from code in the repository.


[ip6] use mesh local prefix length as ALOC prefix length

This change is made so that address callback can use prefix length to determine if the changed address is multicast or not.

@LuDuda Yes, I can check the first byte. Thanks.


Upgrade postgresql92 to rh-postgresql94.

Change postgresql92 SCL references to rh-postgresql94.
https://trello.com/c/aLn9eAj1
Related PRs:
ManageIQ/manageiq#3676
ManageIQ/manageiq-appliance#9
ManageIQ#19 [this branch]

Ready for review/merge.

This is finally working and ready to be merged.  :tada:

LGTM !!  :cake:


Added controller diagrams

Issue #90 : Controller Diagrams have been added. Can be found in the README file & the controllerimages folder.

I believe there are functions on the Driver controller scheme for changing the drive orientation between field relative and robot relative, and also for resetting field-forward. Have those been removed or do they need to be added to this controller map?

I did not see it in the code, but I will test it now.


update installation

https://github.com/instructlab/instructlab/ has a newer getting started

Maybe we should just remove this file completely and replace all references to the instructlab/instructlab README ?

Maybe we should just remove this file completely and replace all references to the instructlab/instructlab README ?

i am ok with that - at least , as a new user, I can test that my ilab generate and ilab train work ?
Should we then

explain how to create a PR
explain how remove that PR (so not all new user submit the same PR) ?

@celek do you intend to revisit this PR?


Switch to Guava's Java 8 compatible variant

We should probably give another go to switching to the Java 8-compatible variant, as there were changes around ImmutableSet.Builder:

28.1 – collect: Fixed a bug in ImmutableSet.Builder that could lead to infinite loops when building multiple sets from the same builder. (https://github.com/google/guava/commit/0007cb257b593400097f37cf5fa9c8d5fb598c62)

cc: @gradle/build-cache

dupe #15744

This is not so much about upgrading to Guava 30, but about switching from -android (aka Java 7 variant) to -jre (aka Java 8 variant).

There is a performance regression that is introduced by the switch (at least when we last tried it around 27.1-jre). It might be gone, or we might accept the regression.

Some work has been done on this branch exposing some performance problems

This issue is no longer relevant.
Fixed in

https://github.com/gradle/gradle/pull/17480


Idiomatic go 4

Another lot of many changes to make the packages and use more go idiomatic

Coverage decreased (-0.7%) to 73.671% when pulling d4145607f16b23c36cd0c3247667da9cd2b05883 on idiomatic-go-4 into 465d7409b10b9402b7dc850b610c60bfcde805c5 on master.


Add statusformatl directives

Proposing adding a statusformatl directive that shows the number of lines in the file, perhaps lines, and perhaps one that show the number of bytes in the file and one that shows the number of columns in the current line. Also, the col directive currently counts characters rather than columns - tabs are counted as 1. Being able to see columns would be helpful.

The statusformat directives support writing your own functions in Lua, so I think most of these are currently possible to implement, although you make a good point that the number of pre-implemented directives should probably be higher. I have added status.lines, status.vcol, status.bytes, and status.size. See the referenced commit for details.


chore: release 4.1.2

:robot: I have created a release *beep* *boop*
4.1.2 (2019-07-23)
Bug Fixes

deps: drop unused dependency lodash.merge, related to sec vulnerability (fixes #301) (#302) (018efd4)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-translate/releases/tag/v4.1.2

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

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


[BUG] Problem

Hi, I have this problem:

Hi, I have this problem:

Make sure you have manually logged into your bing rewards account and clicked join microsoft rewards to show you the "Microsoft Bing Dashboard".
If that doesnt work, comment out the following lines of code around line 31 and 32 to disable headless mode and watch the script work.
if headless_mode : #comment out to disable headless mode (makes window visable)
options.add_argument("--headless")

Fixed with v3.0 #188


sql chat convert natural language to sql then sql to natural language

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.

Example Code
from langchain_community.chat_models import ChatOllama
from langchain.callbacks.manager import CallbackManager
from langchain.callbacks.streaming_stdout import StreamingStdOutCallbackHandler
from langchain.llms import Ollama
from langchain_community.utilities import SQLDatabase
from langchain_core.prompts import ChatPromptTemplate
from langchain_core.output_parsers import StrOutputParser
from langchain_core.runnables import RunnablePassthrough

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

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

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

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

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

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

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

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

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

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

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

# Invoke the full chain and print the final response
full_chain.invoke({"question": "how many total records?"})

Error Message and Stack Trace (if applicable)
No response
Description
I am expecting this to only return
"There are a total of 15 records"
instead I am getting the response "Select count(*) from table, this will get the total number of record in the table, The total is 15 record in the table "
How can i modify the templates or anything so that the actual queries are not returned or explained and only the final answer in natural language is returned?
System Info

System Information
------------------
> OS:  Linux
> OS Version:  #102~20.04.1-Ubuntu SMP Mon Jan 15 13:09:14 UTC 2024
> Python Version:  3.8.10 (default, Nov 22 2023, 10:22:35)
[GCC 9.4.0]

Package Information
-------------------
> langchain_core: 0.1.24
> langchain: 0.1.8
> langchain_community: 0.0.21
> langsmith: 0.1.3

Packages not installed (Not Necessarily a Problem)
--------------------------------------------------
The following packages were not found:

> langgraph
> langserve

Closing as this is not a bug. Please use github discussions Q&A for usage questions


Retain custom alignment colors from gff3

It would be great if igv-reports were to retain custom color info from gff3 format files. Currently, it appears that it strips out “color=#hexid” fields but retains everything else.

Ah, good catch.  I'll look into it.

This is fixed with release 1.10.0


Add Approval to rails config/initializers/load_extensions.rb

Fixes #181
Didn't add approval to the initializer.

cc @mkanoor @syncrou


Discussion: Single SignUp/SingIn

What about adding signup/signin via GitHub, Google, Bitbucket and Facebook?

I personally always choose to sign up via another service I'm on if I get the chance, so I think it's a good idea. I'd say add LinkedIn in there too, for obvious reasons.

I second the motion. If I may sign in with GitHub on a given service, I choose to do so in 90% cases.

Should we collaborate on the function?


Keeps crashing right after opening

Describe the bug
Idk if this is a bug or not but I'm on a mac and whenever I open it through finder, it opens for a sec but crashes. Then I tried opening it through the command prompt like the documentation says but it still opens for sometime but crashes.

To Reproduce
Steps to reproduce the behavior:

Download the EOCV Sim
Go to finder and hold control and click on the file, then click open
It'll open for a sec but immediately close for me
If that doesn't work try using the terminal
cd downloads
java -jar "EOCV-Sim-3.4.1-all.jar"   Then click Enter
For me it opens for a short time but then crashes

Expected behavior
I expected it to just open and stay open not abruptly crash
Screenshots

Desktop (please complete the following information):

OS: [Mac OS Monterey]
Processor architecture: [M1]
EOCV-Sim Version [3.4.1]
Attach logs if possible [e.g autogenerated crash report files in the same directory as the jar file or the console output]

Additional context
idk if this is just a problem with mac or a problem with the M1 chip that I have

Hi, sorry this is happening to you. Could you navigate through the folders /Users//.eocvsim/default_workspace and delete the apriltag detection pipeline.java file in there?
You might need to turn on the display of hidden folders to see .eocvsim

Fixed on v3.4.2 as AprilTags are not supported


Fix create_database_if_needed output

We should only notify that we're creating a database when we actually
are. Sqlite has always been missing this, and it seems to have been
introduced to Postgres unintentionally.


Temporal credentials

Hi,
I'm looking for a python implementation for accessing cognito and specifically cognito identity so I can get temporal credentials to access other stuff in AWS.
To login with warrant works like a charm, then I think I need to do
client = boto3.client('cognito-identity', 'eu-central-1')
credentials = client.get_credentials_for_identity()

The problem is that I do not know what AWS want for parameters. I think IdentiyId is IdentiyPoolId (if not client.getId() will get it) For the Logins part I'm lost, well the first part should be 'cognito-idp.amazonaws.com/'+userPool I think but for the second part I haven't figure it out. Nor find any examples ether.
I know that this are currently out of scope for warrant, but I just hopping that one of you have done this before and can give me a hand.

I finally figure it out,
In the config file I have user_pool_id, client_id and identity_pool_id in as provided from AWS. Hope this helps others that gets stuck in the same mess. :)
def get_temporary_credentials(username, password):
  try:
    aws = AWSSRP(username = username,
         password = password,
         pool_id = config.get('cognito', 'user_pool_id'),
         client_id = config.get('cognito', 'client_id'))
    tokens = aws.authenticate_user()

    client = boto3.client('cognito-identity', config.get('cognito', 'region'))

    login_provider = 'cognito-idp.' + config.get('cognito', 'region') + '.amazonaws.com/' + config.get('cognito', 'user_pool_id')

    identity_pool = client.get_id(IdentityPoolId = config.get('cognito', 'identity_pool_id'),
      Logins={login_provider: tokens['AuthenticationResult']['IdToken']})

    credentials = client.get_credentials_for_identity(IdentityId = identity_pool['IdentityId'],
      Logins={login_provider: tokens['AuthenticationResult']['IdToken']})
    return credentials

  except ClientError as e:
    print "*** FAILED ***"
    print e
    sys.exit(1)

Awesome


Update listener on props changes

If we skip updating listener on prop changes the behavior seems off. This makes it to where we always update the listeners when we get new props. Especially in places where we use something like react-router to change the last portion of a URL that affects our props but doesn't create an entirely new view this is especially problematic.

@fraserxu ping? Any concerns over this change?

It seems to be a valid case:
When updating chart we need to re-apply event handlers as for stateless components next props won't be updated in handler closure:
      const TestComponent = (props) => {
        return (
          <ChartistGraph listener={{ draw: () => console.log(props) }} />
        );
      }

In example above on component re-render console will receive initial props while it's expected to see new ones. It's not an issue with class-based component if we're retrieving props through its instance:
      class TestComponent extends Component {
        render() {
          return (
            <ChartistGraph listener={{ draw: () => console.log(this.props) }} />
          );
        }
      }


Ignore whitespace text nodes when parsing projects

This just ignores purely whitespace text nodes which seem to be part of the parsing in .NET Core and newer versions of System.Xml
Closes #270
Fixes test failures in #1004 when this change is ported to master.

@rainersigwald figured out the problem.  The XML parser only looks at the first 4k of text to see if it's whitespace or not.  Since our whitespace is 70k, it gives up and assumes it's text.
https://github.com/dotnet/corefx/blob/bffef76f6af208e2042a2f27bc081ee908bb390b/src/System.Private.Xml/src/System/Xml/Core/XmlTextReaderImpl.cs#L5571


App Demo with Next.js

Development is on the sveltekit branch

Switching to Next.js

Technically done; making a private project for the rest of the app.


Is Marker missing the data attribute in the typescript definition?

Is the marker class missing data prop as a type?
I've seen a few other people using it around, and it can be accessed, but doesn't have the associated type in the typescript definition.
Thanks

Typescript definitions are not kept in this repo.

Hi, and thanks for taking the time to open a bug report in Leaflet.
However, in this repository we only handle bugs in "vanilla" Leaflet. This means that we do not handle bugs which are specific to frameworks such as:

React
Ractive
AngularJS
Bootstrap
Leaflet for R
Joomla
Wordpress
Ionic
Cordova
Vue
Electron
Polymer
TypeScript
D3
Folium
NativeScript
...etc

Please understand that we only have the time and energy to test Leaflet in plain web browsers.
Please try to either reproduce the bug without using any frameworks, or submit a bug to the appropriate repo.


Listbox - Fix scoll when mouseover focus is triggered

hello,
I'm using the listbox and here is an issue I found:
When there is a scroll in the listbox options and the user mouseovers it, there should be no scroll (as in the video below).
https://github.com/user-attachments/assets/91630e2e-c118-4d5e-8be5-55baa74a1fc6
p.s. There is a simple fix for this, can I create a PR?
Awesome Library though!

You can create a PR. Although based on the video, I think this is a browser behavior.

When you do the el.focus() on mouseenter, you can provide one options, el.focus({ preventScroll: true }), this will fix the issue.

Here is the PR for this issue: https://github.com/lxsmnsyc/terracotta/pull/38

When you do the el.focus() on mouseenter, you can provide one option, el.focus({ preventScroll: true }), this will fix the issue.

ah that's an interesting solution, somehow I missed that. Thanks!

Thanks for this library, it's awesome!


GPU version for OSX also available

as per: #664

Can one of the admins verify this patch?

Can you sign the CLA? Thanks.

sorry, signed the CLA but forgot to reply to the email, just did that.

I signed it!
On Thu, Aug 18, 2016 at 1:39 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/3901#issuecomment-240698399,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACeGjdRAiLaNAwJ4VKY-P8fQ7PRzIN4iks5qhER9gaJpZM4JnZqT
.

Jenkins, test this please.

Thanks, I'll merge it as soon as the test results come back.

There was a test failure, but given that this pull request only touches markdown I'm concluding the test failure was a flake. Merging, thanks for the fix!


more complicated example?

A more complicated example, for instance, we want a robot that can solve some basic functions like what's the weather today? Tell me a joke, do u know how is the tallest man in the world, how to combines all these intent into a full example?

Hello @jinfagang and thank you for your interest.
I have created a new example, the Misc Bot, which does some of the tasks that you requested.
The finite state machine is not very complex, since the actions work mostly as commands, but the interesting aspect is the integration with multiple APIs through the chatto extensions. The extensions in the example are written in Go but you can replace them with anything you want.
Take a look at what this example can do on Telegram by clicking here.
Let me know what you think.

Hello @jinfagang and thank you for your interest.
I have created a new example, the Misc Bot, which does some of the tasks that you requested.
The finite state machine is not very complex, since the actions work mostly as commands, but the interesting aspect is the integration with multiple APIs through the chatto extensions. The extensions in the example are written in Go but you can replace them with anything you want.
Take a look at what this example can do on Telegram by clicking here.
Let me know what you think.

@jaimeteb That's awesome!
You are so brilliant and quickly showing a dedicated example of it. I have another suggestion: can we make another project which using cahtto as a lib (or directly make a complicated example inside chatto), which can support many languages? On my side I want send you some PR that I want implement in Mandrian.
what I want is to maintian a rich robot that have many basic functions.

@jaimeteb By the way, do u have a telegram group to talk about this lib? I really hope to learn more about it

I am not very familiar with Mandarin but I believe it should behave well with the Naive Bayes classifier, since it treats words as separate features.
I'll do some tests in Mandarin and let you know how it goes!

@jaimeteb thank u! 其实中文也很简单, 简单易懂易学,哈哈哈哈哈

Looks like bot is down

Looks like bot is down

Oh yes, sorry, I took it down. I have another working demo on Telegram, please feel free to check it out:
@chatto_demo_trivia_bot


fix to populate engine

fixed: use clone of populate[path] because it is changed in execution context and then became unusable --> so it drives to query returns different objects from what i've expected.
explanation:
in _populate method there is condition
options.conditions || (options.conditions = {});
options.conditions._id || (options.conditions._id = { $in: oid });
and when query returns more that one document that must be populated, then the condition on query itself became changed. so next query will never populate anything for the same path.

thanks!

do you have test for this? I'm having trouble reproducing the error before applying this patch locally.


Spike: decide whether the AVM needs new opcodes&context for fee-prep/distr

Communicate with @just-mitch and @PhilWindle to determine what is needed here.
We can get away with avoiding new context/opcodes by using calldata and having the kernel enforce that the first few words of calldata are tied to the kernel's TxContext.... But that is arguable pretty ugly.
The other option is to have dedicated opcodes for things like "totalFee" that revert if called when not in the setup/teardown phases.

Pinging @fcarreiro and @Maddiaa0 to make aware of this ticket


Incorrect MD5 for a sequence submit

Hi, usually I need to calculate the MD5 value of large files which could up to 20 GB or more, so I submit the jobs one by one with 10 MB chunks sequentially for one file. And I noticed that for the same context the new job submit is not allowed before the previous one is completed, so I use this
while (hash_ctx_processing(ctx)) {
            md5_ctx_mgr_flush(mgr);
        }

before each submits, and also before the fetch of the digest. But if the submit size is larger than 63 Bytes the result is incorrect. However, if I put the flush method after the submit method instead of before, there is no such issue. I just wonder what' the difference between before and after, no matter which way I have used before the fetch of digest I have the status HASH_CTX_STS_COMPLETE.
The reason I wanna the flush before the submit because I have a context pool so that I have tried to avoid flush after every submit.
code just like this:
the one has the issue:
while (processed < size) {
    while (hash_ctx_processing(ctx)) {
            md5_ctx_mgr_flush(mgr);
        }
    ctx = md5_ctx_mgr_submit(mgr, ctx, chunk, step, flag);
}
while (hash_ctx_processing(ctx) {
        md5_ctx_mgr_flush(mgr);
 }
Get digest

The one has no such issue:
while (processed < size) {
    ctx = md5_ctx_mgr_submit(mgr, ctx, chunk, step, flag);
    while (hash_ctx_processing(ctx)) {
            md5_ctx_mgr_flush(mgr);
        }
}
Get digest

Thanks for your patience, the size problem is just a test I found, I will set the size a couple of MBs, so it's OK. But I still do not understand the difference to flush before submit and after submit. I have created another test, where I only use one context of the manager:
while (processed < size) {
        int step = chunk_size;
        if (processed == 0) {
            flag = HASH_FIRST;
        } else if (size - processed <= chunk_size) {
            flag = HASH_LAST;
            step = size - processed;
        } else {
            flag = HASH_UPDATE;
        }
        char sub[step];
        memcpy(sub, &hello[processed], step);
        ctx = md5_ctx_mgr_submit(mgr, &ctxpool[0], sub, step, flag);
        processed += step;
        while (md5_ctx_mgr_flush(mgr));
    }

This works well, however, if I put flush before the submit, the result is incorrect, and I didn't find any error by checking ctx->error.  I thought the order should have no matter as there is only one job submitted in the same time, only one context is in use. Is there any difference in the two orders?
while (processed < size) {
        int step = chunk_size;
        if (processed == 0) {
            flag = HASH_FIRST;
        } else if (size - processed <= chunk_size) {
            flag = HASH_LAST;
            step = size - processed;
        } else {
            flag = HASH_UPDATE;
        }
        char sub[step];
        memcpy(sub, &hello[processed], step);
        while (md5_ctx_mgr_flush(mgr));
        ctx = md5_ctx_mgr_submit(mgr, &ctxpool[0], sub, step, flag);
        processed += step;
    }

The reason why I tried this test because I have tried to use a queue to cache the request and submit the jobs if the related context has the return and is idle, however, the result is incorrect for the same reason as I mentioned above.
I understand that the ctx return order is the same as the submit order, but in the application the coming data order is unpredicted and the chunks' number of the data is not the same.  in md5_mb_over_4GB_test.c, the order is restricted to keep same and after the job submit the flush will be called immediately except for the first submit of each contexts. Do we have any other solutions to avoid the submit-flush pair?

Hi @HaoQinC, still not sure.  The last example is missing a final flush or check, possibly leaving the last update in the queue waiting for more submits to join.
Also to be clear, the ctx return can be in a different order than submit.  If a small block is submitted after long one for example, the small block will likely finish and return next.

Thanks, the order is a key point. Sorry I didn't make it clear, before I print the md5 value I also try the code while (md5_ctx_mgr_flush(mgr)); to guarantee the job is finished. So this is confused me why the flush order is a matter as I only use one context for one job with a couple of sequent submit.
The scenario is just like I have multiple producers that provide different size data, some may need to submit only once with HASH_ENTIRE, and most of them need to submit multiple times, and the coming order is random. With multiple solutions, all of them need to check if the previous job is completed before the next submit for the same context, everything looks good in the process and there is no any error reported, but the final result incorrect comparing with OpenSSL.


content-api: Infrastructure

[x] An ECS service https://github.com/wellcomecollection/content-api/pull/3
[x] Behind a load balancer https://github.com/wellcomecollection/content-api/pull/3
[x] Behind APIGW https://github.com/wellcomecollection/content-api/pull/5
[x] Behind CloudFront https://github.com/wellcomecollection/platform-infrastructure/pull/372
[x] CI publishing the image and deploying the service https://github.com/wellcomecollection/content-api/pull/7

@jamieparkinson Did you say this one can be closed? I don't have the power to close it.

@jamieparkinson Ignore me! Didn't refresh my board before asking.


Add no-op bgtest target for ampi datatypes test

shouldn't we put exit into NOBGDIRS in tests/ampi/Makefile ?

I'll have a patch that implements exit() correctly for BigSim soon, so this PR might not be necessary.

I'll have a patch that implements exit() correctly for BigSim soon, so this PR might not be necessary.

Nvm, just saw that this is about datatypes, not exit


Fixes issue #2313 - Convert WSL2 file paths to Windows file paths

Description
When using VSCode on Windows where projects are hosted in a linux WSL2 environment the Open Files Context Provider fails. The reason is that VSCode does not know how to interpret the filepaths.
This PR adds a conversion step to all file paths if the given path is coming from a WSL2 environment.
Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Testing
Added a unit test for the VsCodeIde.ts class covering these specific changes.

@sestinj is there something that is blocking this from being merged, anything I can address? It'd be nice to get that sweet open file functionality in my instance of VSCode without running my local copy.

Just a heads up, I think i've discovered a much simpler approach to solve this issue without having to run the wslpath process. I'll be pushing the update here shortly.

@sestinj I've push my change to remove the process call to wslpath, i've tested this locally and it works. I suspect this could also fix the remote-file issue I read about earlier however I have not had a chance to test that.

Potentially fixes #2451.

@steverhoades this is excellent! Thanks for finding a cleaner way, and sorry for delay. If you could do me a quick favor and remove the package-lock/package.json changes (they look unnecessary now) I'll go ahead and merge!

done, there was one small version change that was picked up, let me know if you want that reverted as well - happy to do it.

Beautiful, thank you!


Add drop_last_batch to IterableDataset.map

Addresses this comment: https://github.com/huggingface/datasets/pull/3801#pullrequestreview-901736921

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/18 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit b44dd52:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Unresponsive redis connection and hanging requests

I really need your help. Perhaps someone has faced a similar problem.
In our project, the general process of data processing is as follows:
An HTTP request arrives.
We retrieve a large amount of data from Redis.
We process the data and send a response.
Under normal circumstances, our service takes around 200-300 milliseconds to complete. However, after some time, all requests to Redis become unresponsive. After 10-30 minutes, an error message appears in the log indicating a connection issue with Redis: "read ECONNRESET." Subsequently, the application recreates the connection within 100 milliseconds. At this point, all the "hanging" requests are instantly released and can be processed further.
At the time of the error, we output the Redis object as a string representation using JSON.stringify(redis) (I added this output in the attached redis_connect.txt file)
A large number of requests of the same type (hget platform:audience:cache:boqos42202 810) occurs with every new HTTP-request to our application. However, subsequent calls to Redis do not happen as expected because the initial request gets stuck, causing a bottleneck in the process.
At the same time, while the requests to Redis are freezing, the application itself continues to function properly. Other requests that do not require data from Redis are processed quickly and without any delay.
I'm not an expert in the ioredis library, so I could really use a hint. Does the library have any impact on the hanging requests? Could it be that the Redis connection is limiting the available bandwidth?
We have invested a significant amount of time in refactoring our application and are still actively working on it. If anyone has encountered a similar situation, please share any tips or potential solutions.
redis_connect.txt

We have noticed the same issue.  When we do a lot of lookups using HGETALL (and then we use a Function to transform to a JSON Object), the HGETALL seem to get stuck.  Sometimes we see a delay between the await redis.hgetall(KEY) and a response of like a minute or more.  And then any other requests which need redis just get stuck.
We have seen also then a memory leak (memory usage climbs).  Probably because of the command Queue.
I even now enabled Auto Pipelining, and also a command Timeout of 5000ms.


Touch input.

The gesture design seems close, but there’s a slight issue with touch, of course. With touch, we must track either one or two pointers. (We could track more than two, but since we don’t support 3D transformations, and we don’t even support rotation, there’s no benefit to observing more than two touches during a single gesture.) After a touchmove, we’ll want to update the active mouse or wheel pointers, if any, but not the touches.


For spack-stack-1.5.0: update ufs-weather-model-static template

Is your feature request related to a problem? Please describe.
The template ufs-weather-model-static has fallen behind and will continue to do so, since we use unified-dev and skylab-dev on all systems except Acorn.
Describe the solution you'd like
Someone needs to go and update the versions in the template before the 1.5.0 release to match what is built in the unified environment.
Additional context
n/a

i can take this one.

whoops-- i take it back (misinterpreted the issue initially). i forgot alex is the one who maintains ufs-wm-static for acorn.

@AlexanderRichert-NOAA Do you have time to work on this during this week before we tag the release?

Yep will do.

Closed via #772


[Bug] onDidAttemptReadOnlyEdit is not triggered by commands that edit

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
// The Monaco Editor can be easily created, given an
// empty container and an options literal.
// Two members of the literal are "value" and "language".
// The editor takes the full size of its container.

const editor = monaco.editor.create(document.getElementById('container'), {
	value: "function hello() {\n\talert('Hello world!');\n}",
	language: 'javascript',
    readOnly: true
});

editor.onDidAttemptReadOnlyEdit(() => alert("onDidAttemptReadOnlyEdit"))

Actual Behavior
Pressing Tab key or cmd+/ to comment the current line doesn't trigger the browser alert.
Expected Behavior
Pressing Tab key or cmd+/ to comment the current line should trigger the browser alert.
Additional Context
N/A

At least the command does not show in the command palette anymore.
Probably that's also the reason why the keybinding doesn't do anything, because it is conditionally.
Then it is not possible to show a warning.
But I think this issue also applies to VS Code.

@hediet thanks for the explanation. I understand your rationale for the cmd+/ command, but pressing tab to add indentation to the code is not a command, is it? That's typing the \t character.
As for your recommendation to open an issue in VS Code, how do I provide a reproduction for this if I first need to set up monaco's onDidAttemptReadOnlyEdit ?

I think pressing tab invokes a command actually.

As for your recommendation to open an issue in VS Code, how do I provide a reproduction for this if I first need to set up monaco's onDidAttemptReadOnlyEdit ?

VS Code shows a read only message when trying to write to read only documents (e.g. default keyboard shortcuts).


Sticky Desktop notifications for unmentioned messages

Sometimes I get desktop message notifications that don't go away when I am not mentioned
Branch: develop
Browser: Chrome (Windows)

Aha! I thought I was the only one with this problem!

@emintham ya I knew you weren't crazy all along ;)

#overly-attached-bug-reporter

I see it too.

I'm having the issue when I get mentioned. Going to give it a check.

Or that's kind of the idea?

Yes when you're mentioned the notifications are meant to be sticky

Got it!
El dom., 5 de abril de 2015 22:45, geoffwhittington <
<EMAIL_ADDRESS>escribió:

Yes when you're mentioned the notifications are meant to be sticky
—
Reply to this email directly or view it on GitHub
https://github.com/sdelements/lets-chat/issues/387#issuecomment-89886819
.

I cannot replicate, is there a specific workflow for this?

Seems to happen randomly as far as I can tell...


Change dev setup to use real RHACM grc-ui-api instead of local instance

Signed-off-by: Zack Layne<EMAIL_ADDRESS>Hey @dhaiducek, I created this PR so the grc-ui will use a setup-env script to configure grc-ui to run against a real RHACM grc-ui-api service rather than needing a local grc-ui-api running in parallel.
I have also removed the headerUrl env var as I didn't see it used, and noticed the searchApiUrl var isnt used either. Would both of these be safe to remove from the codebase or am I missing something?
Please let me know if these changes makes sense, thanks!

/hold - I have a few minor changes to request.

@dhaiducek Thanks for the review. Before I merge, is the searchapiUrl used at all to your knowledge? if not I will remove this as well.

That may need further investigation. I see it referenced in app.js, but I don't know the history on it or its function.
https://github.com/open-cluster-management/grc-ui/blob/745c6cdd65ca864a16805fcba9ad46f5bb67cfd0/app.js#L115-L118

Okay ill leave out for now and merge. Thanks


Configure OBS + screenkey for linux only

This will be my first Linux-only config. It's probably worth separating out into a separate nix file. Neither OBS nor screenkey is supported on darwin.

This should be a system check for platform-specific packages.


Created: Java: ReverseOfTheString

Hey I created a new program (https://github.com/1dharode/Hacktoberfest2022/pull/1176) kindly review it and merge it into the main branch

Hey Can you please assign it to me?

plz asiign it to me

Hello, Can you please assign this issue to me?
Thanks in advance :)

Kindly assign this issue to me

please assign this to me


Improve player scheduling

Switching tab in browser will slow down the setTimeout scheduling, causing odd effects on the player.
New way of scheduling is needed. MIDI.js may be a good reference.

Ref:
http://www.html5rocks.com/en/tutorials/audio/scheduling/


attribute name parsed as variable when square brackets are included

For this issue, a little while ago, we decided that the @attribute_name should be accessible from an array. Say we have:
a = ["xyz", "test"];
xyz_vals = pos@a[0] would be acceptable and thus equivalent to xyz_vals = pos@xyz. So in this case, it requires that the parameter before the square brackets is an existing variable name
I'm not sure how we should resolve this


Automated cherry pick of #9427

Cherry pick of #9427 on release-6.3.
/cc  @JulienTant
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: b3295c6dfd736371e181a48bb55edff3cc645f32


feature: Supporting core+package templates stored on NuGet

Description
Core of this PR:

Adds a new type of template to Composer called Conversational Core

Conversational Core is stored as a public nuget package and is made up of a core bot with dependent packages
Conversational Core is a dotnet template that starts off with an ejected runtime that is packaged with the template

Adds logic to Composer to support this new type of template

If the users selects this template in the creation flow and selects 'create' then Composer

installs that dotnet template stored on NuGet to the users machine if it does not already exist (dotnet new -u )
Creates an instance of that template and stores the resulting bot in memory.
Runs the dialog merge tool to grab any dependent packages that the template declares and merge them with the bot in memory (for conversational core there are 4 lg packages that are merged at this step)
User is outputted to editing view where they see the core bot and the 4 lg assets ready to be edited or tested

Side things that the PR does:

Consolidates redundant types BotProject and BotTemplate
Allows for relative paths in settings.runtime.path which is required to support a template that comes ejected on initialization
Adds a public feature flag that toggles all externally stored templates from being visible in the creation flow
Updates feature flag defaults
Simplifies feature flag init logic

Task Item
fixes #4371
fixes #4567

Placeholder while CI builds are down:
Test:Coverage ran on this PR through the CI earlier and showed these two tests breaking, the fixes for these tests are in but the CI build is down and wont run test: coverage. I have fixed these tests and tested the fixes locally (output below).
Fixed test output
 PASS   server  ../../services/__tests__/project.test.ts (12.771 s)
 PASS   server  ./project.test.ts (16.73 s)
 PASS   server  ../../models/bot/__tests__/botProject.test.ts (21.783 s)

Test Suites: 3 passed, 3 total
Tests:       50 passed, 50 total
Snapshots:   0 total
Time:        23.326 s, estimated 28 s
Ran all test suites matching /project.test.ts/i.

 PASS   client  ../DialogWrapper/index.test.tsx (8.085 s)
 PASS   client  DefineConversation/index.test.tsx (15.57 s)
 PASS   client  CreateOptions/index.test.tsx (15.702 s)
 PASS   client  ./index.test.tsx (17.988 s)

Test Suites: 4 passed, 4 total
Tests:       7 passed, 7 total
Snapshots:   0 total
Time:        20.113 s, estimated 41 s
Ran all test suites matching /index.test.tsx/i.

test:coverage output
Summary of all failing tests
FAIL packages/server/src/controllers/__tests__/project.test.ts
  ● create a Empty Bot project › should create a new project

    expect(jest.fn()).toHaveBeenCalledWith(...expected)

    Expected: 200
    Received: 404

    Number of calls: 1

      177 |     } as Request;
      178 |     await ProjectController.createProject(mockReq, mockRes);
    > 179 |     expect(mockRes.status).toHaveBeenCalledWith(200);
          |                            ^
      180 |   });
      181 | });
      182 | //current opened bot is the newBot

      at Object.toHaveBeenCalledWith (src/controllers/__tests__/project.test.ts:179:28)

FAIL packages/client/__tests__/components/CreationFlow/index.test.tsx
  ● <CreationFlow/> › should render the component

    TypeError: Cannot read property 'enabled' of undefined

      20 |       }
      21 |     }
    > 22 |     if (!featureFlags.REMOTE_TEMPLATE_CREATION_EXPERIENCE.enabled) {
         |                                                           ^
      23 |       filteredTemplates = filteredTemplates.filter((template: BotTemplate) => {
      24 |         if (template.path) {
      25 |           return template;

      at enabled (src/recoilModel/selectors/projectTemplates.ts:22:59)
      at evaluateSelectorFunction (../../node_modules/recoil/cjs/recoil.js:2981:22)
      at getFromCacheOrEvaluate (../../node_modules/recoil/cjs/recoil.js:2947:44)
      at ../../node_modules/recoil/cjs/recoil.js:3073:47
      at detectCircularDependencies (../../node_modules/recoil/cjs/recoil.js:3039:14)
      at Object.myGet [as get] (../../node_modules/recoil/cjs/recoil.js:3073:14)
      at getNodeLoadable (../../node_modules/recoil/cjs/recoil.js:306:25)
      at getRecoilValueAsLoadable (../../node_modules/recoil/cjs/recoil.js:714:37)
      at useRecoilValueLoadable_LEGACY (../../node_modules/recoil/cjs/recoil.js:2056:10)
      at useRecoilValueLoadable (../../node_modules/recoil/cjs/recoil.js:2073:12)
      at useRecoilValue (../../node_modules/recoil/cjs/recoil.js:2090:20)
      at useRecoilValue (src/components/CreationFlow/CreationFlow.tsx:51:28)
      at renderWithHooks (../../node_modules/react-dom/cjs/react-dom.development.js:14803:18)
      at mountIndeterminateComponent (../../node_modules/react-dom/cjs/react-dom.development.js:17482:13)
      at beginWork (../../node_modules/react-dom/cjs/react-dom.development.js:18596:16)
      at beginWork$1 (../../node_modules/react-dom/cjs/react-dom.development.js:23179:14)
      at performUnitOfWork (../../node_modules/react-dom/cjs/react-dom.development.js:22157:12)
      at workLoopSync (../../node_modules/react-dom/cjs/react-dom.development.js:22130:22)
      at performSyncWorkOnRoot (../../node_modules/react-dom/cjs/react-dom.development.js:21756:9)
      at scheduleUpdateOnFiber (../../node_modules/react-dom/cjs/react-dom.development.js:21188:7)
      at updateContainer (../../node_modules/react-dom/cjs/react-dom.development.js:24373:3)
      at ../../node_modules/react-dom/cjs/react-dom.development.js:24758:7
      at unbatchedUpdates (../../node_modules/react-dom/cjs/react-dom.development.js:21903:12)
      at legacyRenderSubtreeIntoContainer (../../node_modules/react-dom/cjs/react-dom.development.js:24757:5)
      at Object.render (../../node_modules/react-dom/cjs/react-dom.development.js:24840:10)
      at ../../node_modules/@testing-library/react/dist/pure.js:86:25
      at batchedUpdates$1 (../../node_modules/react-dom/cjs/react-dom.development.js:21856:12)
      at act (../../node_modules/react-dom/cjs/react-dom-test-utils.development.js:929:14)
      at render (../../node_modules/@testing-library/react/dist/pure.js:82:26)
      at renderWithRouter (__tests__/components/CreationFlow/index.test.tsx:48:10)
      at renderWithRouter (__tests__/components/CreationFlow/index.test.tsx:61:9)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:62:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:288:22)
      at Generator.prototype.<computed> [as next] (node_modules/regenerator-runtime/runtime.js:114:21)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
      at _next (node_modules/@babel/runtime/helpers/asyncToGenerator.js:25:9)
      at node_modules/@babel/runtime/helpers/asyncToGenerator.js:32:7
      at Object.<anonymous> (node_modules/@babel/runtime/helpers/asyncToGenerator.js:21:12)

Test Suites: 2 failed, 224 passed, 226 total
Tests:       2 failed, 1 todo, 901 passed, 904 total
Snapshots:   5 passed, 5 total
Time:        439.651 s
Ran all test suites in 13 projects.
Force exiting Jest: Have you considered using `--detectOpenHandles` to detect async operations that kept running after all tests finished?
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Coverage decreased (-0.05%) to 55.043% when pulling f6d71db5014ad7d256eee2e5f5cd47278e136a52 on pavolum/hostedTemplateSupport into 44f841a02fc59bc4219372becd5f53d7b90ffba7 on main.

@hatpick @cwhitten @lzc850612 Unit test build passed, but others did not due to myget being down. This code has been reviewed and does not break tests.
@hatpick approve when ready and then @cwhitten can squash and merge with his mighty admin privilege's.


Find & install necessary library/module to implement full Markdown support

Need to find an open source library/module that can fully parse Markdown files into HTML.

Found a npm library called Markdown-It that supports full markdown parsing into HTML.
Also supports additional plugins that can parse even more Markdown features which I will attempt to implement as well.


Issue with event align results

Hi,
Thank you so much for always answering my questions.
I have noticed an issue with the results from eventalign for direct RNA sequencing.
This is the command that I had used with nanopolish 0.8.4:
nanopolish eventalign  --reads all.fastq --bam all.s.bam  --genome reference.fa --t 8 --scale-events > scale.event
I am able to run the command fine, with no error.
However, I had noticed for some of the reads, the reference_kmer and model_kmer are not the same.
The event index seems to be flipped too, it is running in the opposite direction.
I am unsure if the the scaling of the event level mean to the model mean are affected.
For most of the strands there is no issue, it only occurs for a small number of strands.
I have attached the one of the reads that have issue.
issue.txt
Thank you so much.

Hi,
In this case the read matches the reverse complement strand of the reference genome. The model_kmer will be the reverse complement of the ref_kmer and from the perspective of the reference genome the events are in reverse.
Jared

Thank you so much for the clarification!


'(' and ',' do nothing

According to the readme:
Open parenthesis and comma keys display Haxe type hints in the status bar and inserts smart snippets
I get nothing in the status bar and can't print either parentheses or commas.
Arch Linux 64 bit.

I have this same issue when installing through Sublime Package Control. I had to clone master into my Packages folder in order to get '(' and ',' to work.


Add include_optimizer argument to save_model

save_model saves the model with optimizer's state.
If output model is used only for predicting, optimizer's state is useless.
This PR make it possible to save model without optimizer's state.
(It also helps avoiding #6046.)

This is a good feature to have. include_optimizer=True would make more sense though, API-wise.

Renamed it.
Thank you for your advice.


cmd/doc: args are treated case-insensitive in some cases

What version of Go are you using (go version)?

$ go version
go version go1.17 darwin/amd64

$ gotip version
go version devel go1.18-d34051b Thu Dec 2 07:04:05 2021 +0000 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/kanataninaoki/Library/Caches/go-build"
GOENV="/Users/kanataninaoki/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/kanataninaoki/dev/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/kanataninaoki/dev"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.17/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.17/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.17"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/kanataninaoki/dev/src/github.com/kanata2/go-doc-case-insensitive-issue/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/02/8930c3s52tlfpw2_8ml_xm5w0000gn/T/go-build3338848241=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
The following is one of the examples to reproduce.
$ go doc Net/HttP/httpTEST.DefaultRemoteAddr

What did you expect to see?
According to https://go.dev/blog/package-names, lowercase is recommended in Go,
so I think that the output should also follow it.
Thus I was expecting that the go doc will return an error(like not_found_error)
or the output which converted to lowercase.
What did you see instead?
The following is the actual output.
The import path import "Net/HttP/httpTEST" is not a recommended style.
package httptest // import "Net/HttP/httpTEST"

const DefaultRemoteAddr = "<IP_ADDRESS>"
    DefaultRemoteAddr is the default remote address to return in RemoteAddr if
    an explicit DefaultRemoteAddr isn't set on ResponseRecorder.

Investigation
This problem seems to occur in cases where the passed arguments are passed directly to build.Import function.
I don't know if this is the intended behavior, but it is because the build.Import function treats path as case-insensitive.
package main

import (
	"go/build"
	"log"
	"os"
)

func main() {
	wd, _ := os.Getwd()
	pkg, err := build.Import("Net/HttP/httpTEST", wd, build.ImportComment)
	log.Printf("pkg: %v\nerr: %v", pkg, err) // err is nil
}

I'm thinking of submitting a CL based on the discussion here.

CC @mvdan via https://dev.golang.org/owners
Please CC anyone else you think this is relevant to.

I agree that this seems like an unintended feature; we should add a test and fix the code.
To begin with, we should probably be using<EMAIL_ADDRESS>on import paths, to reject clearly invalid ones outright.
As for uppercase, I do believe module/package paths can contain uppercase letters, but they are also case-sensitive. See<EMAIL_ADDRESS><EMAIL_ADDRESS>implements the escaping, for instance, though I don't think build.Import will know what to do with escaped import paths.
The proper fix here might be to fix build.Import so it doesn't assume that NET/HTTP is not equivalent to net/http.
cc @bcmills @matloob in case I got anything wrong in regards to x/mod and go/build.

@mvdan
Thanks for the confirmation!

As for uppercase, I do believe module/package paths can contain uppercase letters, but they are also case-sensitive. See<EMAIL_ADDRESS>
I looked at the GoDoc for golang.org/x/mod which you posted and found a few things I hadn't considered.
Also, as you may already know, let me summarize the possible cases of this problem and expected behavior again as follows.

This problem is only possible with case-insensitive file systems such as macOS. For example, the following returns the correct result.
$ docker run --rm -it golang:1.17 go doc NET/HTTP
doc: package NET/HTTP is not in GOROOT (/usr/local/go/src/NET/HTTP)
exit status 1

On top of that, this issue should only occur for the following two cases:

search for standard packages (i.e., args are like NET/HTTP or STRINGS.Replace)
search for packages in relative path (i.e., args are like ./AUTH or ./GOLANG/MOCK/GOMOCK)

Based on the above, we hope that the behavior of go doc will be able to handle correct import paths by validating arguments even if the filesystem is handled case-insensitively.
For case 1, the standard library is always in lowercase, so I think forcing the conversion to lowercase is not bad, but for case 2, I can't think of an appropriate way to judge...
And in this case, it should not be possible to use module#EscapePath to make a decision since it is not in GOMODCACHE and is not escaped.


API should invoke error handler or throw an error if not yet connected, but other apis such as publish are invoked.

See #22


feature request: reply

Ideally if I quote reply the body of an incoming email, the bot will send an email reply:

Good point, although I don't work on this bridge anymore. I'll keep this open since its a handy feature and I still merge PRs.

Email replies are really simple, they just need the Re: subject line and you just quote the text. So this is something that can already be done if you format your subject right and you quote the original email.
Not to say it can't be better, but this works too.


Why does ntlk.tokenize.wordpunct_tokenize not split on _

I am using nltk.tokenize.wordpunct_tokenize, but do not see the string containg _ being split up.
Test Code
from nltk.tokenize import wordpunct_tokenize
wordpunct_tokenize("Hello_World")

Current Output: ['Hello_world']
Expected_output: ['Hello', '_', 'World']
Am I missing something?

Which version of NLTK are you using?
There's no problem on v3.2.5:
>>> import nltk
>>> nltk.__version__
'3.2.5'

>>> from nltk.tokenize import wordpunct_tokenize
>>> wordpunct_tokenize("Hello_World")
['Hello_World']

If we trace back the code of wordpunct_tokenize
From https://github.com/nltk/nltk/blob/develop/nltk/tokenize/init.py#L69, it points to   https://github.com/nltk/nltk/blob/develop/nltk/tokenize/regexp.py#L199 which points to the WordPunctTokenizer object at https://github.com/nltk/nltk/blob/develop/nltk/tokenize/regexp.py#L171
If we look at the #L183 regex carefully:
 RegexpTokenizer.__init__(self, r'\w+|[^\w\s]+')

The regex there checks out to be okay: https://regex101.com/r/KJQenr/1

@alvations I am using version 3.2.2

The expected behavior is to treat underscore as valid \w+ character. Please see the extended explanation above.

Thanks.


Initial Mesh (GAMMA) conformance tests

What type of PR is this?
/area conformance
What this PR does / why we need it:
This PR introduces the basic scaffolding for Mesh testing. Mesh testing is fundamentally different from ingress testing, as we need to send requests from pod-to-pod instead of "go test" to "reachable IP address".
This PR introduces the basic scaffolding to run this test. With this PR, you can run a test against GAMMA tests.
This will:

Deploy a new namespace and a few applications
Run tests against them. Currently, there is only a trivial test that ensures applications can talk to each other; this PR is just intended to get the base infra, not add useful tests (yet)

Architecturally, we are deploying gcr.io/istio-testing/app:latest to each app. This is a client+server pair. We then on-demand exec into each pod to run client http://some-request, and parse the result and validate against expectations.
Likely desired improvements:

Move image into k8s org, do not use latest
Move to use a long-lived port-forward to each application; exec has a high overhead. The echo app has a grpc server we can control it from (instead of exec, we send it a ForwardMessage request). However, this has a downside that it requires us to pull in gRPC.
Make code a bit more robust, etc

Opening this PR to get early feedback before moving further.
I think we could also merge this as-is, as it doesn't impact non-Mesh cases and will allow further iteration.
Which issue(s) this PR fixes:

Fixes #1340
Does this PR introduce a user-facing change?:

NONE

Alright, enough people have seen this now so we can
/unhold

Thanks all. Need 1 lgtm to get this in

Looks good from here, FTR.

Thanks @howardjohn!
/lgtm


Release 1.0.8?

Hey @owainlewis , is there a chance of getting a new version out with the fix 😁

@rymndhng done.


allow not, or etc. in query subscriptions

this should be pretty easy now that we just us an in-memory crux node to evaluate subscriptions. We just need to make sure the sub queries don't have any un-whitelisted functions.

Biff no longer has query subscriptions


OS::_exit_code is never updated

Godot version
master branch at 87de2e7c4a38b1258d9a554c7f50cdbd8ac825fa
System information
Arch Linux
Issue description
Recently I've been rebasing https://github.com/godotengine/godot/pull/52523 against the master branch several times a day, and at one commit, the following String documentation was deleted even though no recent changes to String were made to to warrant dropping the following section:
		<method name="operator []" qualifiers="operator">
			<return type="String" />
			<argument index="0" name="index" type="int" />
			<description>
			</description>
		</method>

For some reason, a couple of rebases after, class reference kept the section above.
For what is worth, exit status code of godot --doctool is 1 which probably means an error happened while generating the class reference.
On 3.x, generating the documents with --doctool will have an exit status of 0, meaning no error happened.
Steps to reproduce
On a Linux terminal:
./bin/godot.linuxbsd.opt.tools.64 --doctool
echo $? # returns 1 on master, 0 on 3.x

Minimal reproduction project
N/A. The --doctool flag is provided by the engine binary itself.

Welp, turns it it's just that OS::_exit_code is never updated to EXIT_SUCCESS.
Looks like this happened after some refactoring of main.cpp.


Check for code coverage when running tests

Gonna merge this for now so the tool is available; getting coverage up to expectations and a GitHub actions runner will come later.


Add --dev-traffic flag

Motivation
Adds a --dev-traffic flag which determines if node 0 generates transactions if the --dev flag is enabled.
NOTE: traffic is also generated in production, so having a --generate-traffic which applies both in dev and production mode would be cleaner.
Test Plan
Manually ran the devnet.sh script with and without the --dev-traffic flag to observe correct behaviour.

There's also https://github.com/AleoHQ/snarkOS/pull/2822, which I think is more useful (but is outdated).

@vicsn I've updated the linked PR.


[Bug]: Unable to upload binary files to S3

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Although the base64 option works for S3, the binary option results in corrupted data in S3. The upload does succeed with the correct size and type, this might be a case of unexpected encoding formats.
Steps To Reproduce

Drop a filepicker widget and edit its properties to store binary data
Add a new S3 query to create a new file
Choose Text / Binary option for this query
Try to upload a PDF file to S3

Public Sample App
No response
Issue video log
https://user-images.githubusercontent.com/5298848/216295545-993d62c2-61cf-497b-9686-5bf04d54c91d.mov
Version
Cloud

I had the same problem when uploading via post request.

This problem also affects the file upload to other APIs like Drive. This user reported it: https://app.frontapp.com/open/cnv_11do02bw?key=nOrL1kbM8-zURYl1tdYkhdU0cqbswQM0

i am facing the same issue while uploading a zip file to S3 bucket using binary option in File Picker and S3 Query.

this works now, we recently shipped a fix but didn't get around to closing this issue.
https://github.com/appsmithorg/appsmith/assets/94514895/1a44c823-daf8-4afd-91bc-9215106c853c


Model selection after review

@ChristianSch I've documented internal and helper functions.
@ljvmiranda921 I've adhered to your review, there's less bloat in class properties, private functions are made private, helpers are moved out of the class, lot's of docs added.
Please take a look at it, and approve the PR if it's ok, or comment if not :) - your remarks are still in #86, but it got messed up after I did a rebase instead of merge.

@ChristianSch i've merged it, because i need it for another thing, but i'm still happy to adhere to your remarks!
@ljvmiranda921 noted, this will be good to tackle with the code reorganization phase after the release


[v-test][PHP] Debug add to watch doesn't show value of array item

From @vsccarl on March 11, 2016 19:8
OS: Windows 10
Build#: 0.10.12-alpha
Node: 4.1.1
When adding a single array element to the watch list, it doesn't correctly show the value of the element.
Adding the entire array to the watch list works as expected.
Example:

Copied from original issue: Microsoft/vscode#4029

From @felixfbecker on March 12, 2016 10:42
Afaik VS Code doesn't include a PHP debugger, so in case you are using my extension, you might want to move the issue to https://github.com/felixfbecker/vscode-php-debug

That's because $array.foo is not valid syntax for array access:

The correct syntax is $array['foo']. When you execute $array.foo, the dot acts as a string concatenation operator (the same as $array . foo). PHP will convert $array to a string, which results in Array. It will then concatenate that with foo, and since foo is not a defined constant, it will implicitly assume you meant "foo" and convert it to a string, resulting in Arrayfoo.


Implement Race Mixins

Extend the racing_data.Race class with the following properties:

active_runners
win_value
exacta_value
trifecta_value
first_four_value
total_value

Also add the 'get_winning_combinations' method to the racing_data.Race class.
(closes #28)

Coverage increased (+3.8%) to 86.111% when pulling 32f98f9b9fe9222c2ac152f259fbdcc3d787b1ac on races into fad7ed1e9c908f4af33681339804233a524b8f82 on master.


Banking Calculation in DiagonalBanking is Wrong

Probably already known, but the app DiagonalBanking is broken because it seems to be having trouble figuring out the banks for the read ports.  The node is x603 = SRAMBankedRead(x298,Vector(List(x598), List(x598), List(x598)),Vector(x600, x601, x602),List(Set(b585, b586), Set(b585, b587), Set(b585, b588)),Vec[Fix[TRUE,_32,_0]]) which is a red flag because all three elements in the vector point to the same bank at different addresses

I think this is the same bug but here is another app that I'm using right now to work on this
@test class MultilevelPar extends SpatialTest {
  override def runtimeArgs: Args = NoArgs

  def main(args: Array[String]): Unit = {
    val d = DRAM[Int](16,16)
    Accel {
      val s = SRAM[Int](16,16)
      Foreach(16 by 1 par 2, 16 by 1 par 4){(i,j) => s(i,j) = i+j}
      d store s
    }

    val result = getMatrix(d)
    val gold = (0::16, 0::16){(i,j) => i+j}
    printMatrix(result, "Result:")
    printMatrix(gold, "Wanted:")
    println(r"Pass? ${result == gold}")
    assert(result == gold)
  }

}

This appears to be fixed now thanks to your fixes to offset calc


nixos/tests/chromium: Re-enable the chrome://gpu test for M107

I need to fix copying the chrome://gpu content to the clipboard (Ctrl+a doesn't work anymore so we have to click the button) but we can at least test the font rendering for now.
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg test chromium


NFT purchase ?

Hi !
I would like to write a rust contract for a NFT purchase action, using this example as a base.
My questions :

For the NFT's royalties, do we need to code it manually or is it implicit in an instruction ?
If i understand well, instead of using spl_token::instruction::transfer for transfering Tokens, there is another instruction to be used to transfer SOL ? (or should i use SOL as an SPL Token ?)
Thanks !

Hi, thanks for your reply, once again.
I figured out for royalties that we have to compute them in the rust program, so that's my answer.
For the SOL transfer, i really don't manage to find any example... If you have an idea, it would be awesome, i explain what i did :

in the ts script, added this line :
systemProgram: anchor.web3.SystemProgram.programId,
(in the accounts array in the program.rpc.exchange() , at the end)

in the rust, added this line :
pub system_program: AccountInfo<'info>,
in the
#[derive(Accounts)]
pub struct Exchange<'info>
section, at the end also.

And i get an error "program failed to complete". Sounds weird, i just added the system program in the accounts...
I also tried to use :
// tentative d'envoi de sols
let ix = anchor_lang::solana_program::system_instruction::transfer(
&ctx.accounts.taker.to_account_info().key,
&ctx.accounts.initializer.to_account_info().key,
999999999,
);
	SolanaProgram::invoke(
		&ix,
		&[
		ctx.accounts.taker.to_account_info().clone(),
		ctx.accounts.initializer.to_account_info().clone(),
		// ctx.accounts.system_program.to_account_info(),
		],
	)?;

i'm really not sure about the .clone() and .key, but i don't know what expect this transfer instruction, so very hard to guess.
If i remove the line pub system_program: AccountInfo<'info>, the program runs, but i get the error with escalation error (i think i miss a signature, but i don't understand which one since i have the taker signature, and the receiver signature should not be necessary to receive sol...)
Thanks for your reply !

i wonder something :
some accounts are declared like this :
Account<'info, TokenAccount>,
others are declared like this :
AccountInfo<'info>
may be there is a special way to declare system program account ? (like for token accounts)

Hi, please refer to this document for more details
https://hackmd.io/@ironaddicteddog/solana-anchor-escrow


Update to AnkiOCR to work for newer anki version

would you consider updating the AnkiOCR add on for the newer anki versions?

This is now being worked on, fix is on master, will release this week :)  Closing this as it is better tracked in #45

This is now being worked on, fix is on master, will release this week :)  Closing this as it is better tracked in #45


[Night Light] ...

Extension
https://www.raycast.com/devmoath/night-light
Description
It'd be nice if the extension closed/dismissed Raycast after using one of the commands. It's very strange that Raycast stays open, forcing me to close it manually.
Who will benefit from this feature?
No response
Anything else?
No response

There is a setting to do this:


Fix some ways to create inconsistent Viewport size states

In the editor, it was possible to set the size of a SubViewport even in cases where a parent SubViewportContainer had stretch enabled (which automatically adjusts the size).
This PR disables editing a SubViewport.size while the parent disallows it and it makes necessary adjustments during NOTIFICATION_ENTER_TREE.
fix #73160 by disallowing to create the situation where the described problem can happen
continuation of #66906
supersedes #72556

Thanks!


Remove message length from messages

This is a fix for the issue identified in https://jira.hyperledger.org/browse/STL-1707
When a transaction processor uses the C++ Sawtooth SDK, the Validator will return a python exception like the one shown below, because the validator isn't expecting the length to be in the message:

[2020-05-15 01:44:06.927 ERROR interconnect] Received a message on address tcp://<IP_ADDRESS>:4004 that caused an error: too many values to unpack (expected 2)
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/sawtooth_validator/networking/interconnect.py", line 368, in _receive_message
yield from self._socket.recv_multipart()
ValueError: too many values to unpack (expected 2)

Thanks. I'm confused how this SDK worked without that change.
I don't see any related interfaces that would have required the message length in the commit history in sawtooth-core. Odd.
Anyway, thanks for the fix. Could you please sign off your commit. You can do this at commit time with the -s flag, or now after the fact by amending the commit messages in similar fashion.

Thanks. I'm confused how this SDK worked without that change.
I don't see any related interfaces that would have required the message length in the commit history in sawtooth-core. Odd.
Anyway, thanks for the fix. Could you please sign off your commits. You can do this at commit time with the -s flag, or now after the fact by amending the commit messages in similar fashion.

I've completed sign off on the commits

@agunde406 can this be merged in too please?


docs: fix typo

"it's" means "it is", and is not a possessive.

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


Workspace - WS chat for member displays "error loading more messages" after submitting

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.84-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4635959&group_by=cases:section_id&group_id=309128&group_order=asc
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition: Have a Collect workspace with workflows enabled, and 3 expensifail accounts as the member, owner and approver. The approver was selected under Workflows - Add approvals.

Member: Navigate to https://staging.new.expensify.com/
in Chrome incognito
Member: Submit two expenses with different amounts to the workspace chat
Approver: Navigate to https://staging.new.expensify.com/
in Chrome without incognito
Approver: Hold one of the submitted expenses
Approver: Navigate back to the combined IOU thread
Approver: Approve just the pending IOU
Member: Wait for the held IOU to appear as a separate one in the workspace chat / Refresh the browser
Member: Submit another IOU to the workspace chat

Expected Result:
There shouldn't be error messages, or long loading on the workspace chat.
Actual Result:
WS chat for member displays "There was an error loading more messages." after submitting. Sometimes it's paired with an endless, flickering skeleton loader.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/f3d0e714-9890-4fe8-a46a-c04d2347a104

View all open jobs on GitHub

We think this issue might be related to the #collect project.

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

I'm not able to reproduce this following the above steps - I'm on v9.0.1-9 staging so I am guessing this may have already been fixed but will wait for the reproduction test to confirm

waiting on the repro test


Feature request - onLetter,bool for imperfect diagonal,hint coloring

Hey, I had a few ideas :

onLetter function that records the current letter painted so we can make ourselves a widget similiar to the upper text between "names" and the grid :
https://github.com/Amonc/crossword/assets/74504869/7a1dd531-33d4-4b4e-b3b8-3b2b366c7fe2

A boolean for turning off the imperfect diagonal line feature as described in (saw your reaction on it) :  https://github.com/Amonc/crossword/issues/6

a function that draws on one of the letters based on a given (row,col) to give a hint like :
https://github.com/Amonc/crossword/assets/74504869/d7060f35-f78b-433f-9918-db952ce1b575

Hope you will consider those ideas, I think it will make the package much more complete.
Enjoying the package a lot ! Thank you guys.

@pazelisha

Use to render words realtime while line is being drawn

onLineUpdate: (String word) {
                    setState(() {
                      this.word = word;
                    });
                  },

I couldn't find any quick solution to disable the imperfect line, I added a faded color as default. Colors can be customized:

  invalidLineColors: [Colors.red.withOpacity(0.1)]

Added a slight animate and decoration to control the speed and animate factors

check example
Thanks a lot for sharing ideas. Happy Coding!!!


Model access: enforce_access for packages

Note: This is still in design/implementation
Copying from https://github.com/dbt-labs/dbt-core/issues/7713#issuecomment-1584795533:
IMO differentiation between public & private access is an important part of the dbt-core language construct for model access. It should be possible to enforce this, even if you're just adopting a "mesh" paradigm via packages. It should be opt-in (for now) to avoid breaking status quo, but we might want to make this "on" by default in a future version of dbt (perhaps v2).
Let's call it enforce_access. When set to True, we should raise an error during ref resolution if a model with access: protected is being referenced across projects.
# packages.yml
packages:
  - git: https://my-org/upstream-project
    enforce_access: True  # default is False

Closing in favor of https://github.com/dbt-labs/docs.getdbt.com/issues/3632


docker-build: 404 brew command not found

As per the build logs, the brew command went missing. I may need to look at the logs for starter-pack if any issues is found.

This bug is being fixed in https://github.com/code-server-boilerplates/starter-pack/commit/a3923fa1fd18e5759c1b21dc96567856347eadfb, I may need to retrigger the CI stuff.

This problem was now resolved in https://github.com/code-server-boilerplates/starter-pack-as-base-image/commit/a99034c896fff71000e61de666789829d732fb83, now closing.


added support for the ValueMetatype Instrution

please resolve the conflicts that exist now that I have merged the previous PRs


Can’t update UDT using gocqlx

Hi so I have a usecase in which a user can create a secondary index on any table by calling an API. And I’m recording these indexes in a common table using map<text, frozen> where indexes is a User Defined Type:
CREATE TYPE indexes (
    local BOOLEAN,
    index_name TEXT,
    table_name TEXT,
    columns SET<TEXT>
);

When the API is called the secondary index is created. This part is working fine. But updating the row in the common tables is not. All the data in the UDT is stored as null:
 {'table_table1_20e24b85_d425_11e_int13_idx': {local: null, name: null, table_name: null, columns: null}}

Here’s my gocqlx code:
selectedTable.Indexes[indexName] = models.IndexModel{
	Local:     reqBody.Local,
	Name:      indexName,
	TableName: selectedTable.InternalName,
	Columns:   reqBody.Columns,
}

stmt, names = qb.Update("tables").Set("indexes").
	Where(qb.Eq("internal_name"), qb.Eq("name"), qb.Eq("description")).ToCql()
if err := config.GetScylla().Query(stmt, names).BindStruct(&selectedTable).ExecRelease(); err != nil {
	utils.HandleErrorResponse(c, err)
	return
}

I hope the code is self-explanatory enough but all I’m doing is creating a map of string->indexes(UDT) to put in the common table and updating the common table with it.
Side note: why is the documentation on gocqlx so bad?

Answered also on Scylla University forum https://forum.scylladb.com/t/cant-update-udt-using-gocqlx/1336/2?u=sylwiaszunejko


Dashboard shows things that shouldn't be available to Role

Laravel Version: 5.5.21
Voyager Version: 1.0
PHP Version: 7.1.10
Database Driver & Version: MySQL 14.14

I have a role that only should allow access to Posts. But I have to also allow the role to be able to "Browse Admin". So when that user logs in to the dashboard they see this:

They shouldn't see the Users or Pages section at all. It doesn't seem like the dashboard is respecting the role privileges. If either of those is clicked, they get this:

Those widgets don't currently respect permissions.  You're welcome to submit a PR to change that.

Okay. Is it possible to hide certain widgets?

@Jakobud try the config/voyager.php.

Fixed in https://github.com/the-control-group/voyager/pull/3058


Feature/selective fill

Small tweak that lets datasets restrict the domain over which line charts fill.

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.


The app shouldn't crash when the connection to a node fails

Apparently the app crashes, when gaia can't connect to a certain node. The expected behavior is, that the app chooses a new node and tries to connect to that node.

in addition... i think we should consider refreshing the view for the user when we reconnect. otherwise, the view just displays the loading graphic and you have to refresh manually.
electron  gaia server ready
          reconnected to <IP_ADDRESS>:46657

electron  requesting to reconnect

electron  Picked seed: <IP_ADDRESS>:46656 of [ '<IP_ADDRESS>:46656',
            '<IP_ADDRESS>:46656',
            '<IP_ADDRESS>:46656',
            '<IP_ADDRESS>:46656',
            '<IP_ADDRESS>:46656',
            '<IP_ADDRESS>:46656' ]

electron  1:20:16 PM <IP_ADDRESS>:46657 is alive

electron  quitting running baseserver
          starting gaia server with nodeIP <IP_ADDRESS>:46657
          startBaseserver /Users/jolesbi/.Cosmos-dev/baseserver
          spawning /Users/jolesbi/goApps/bin/gaia with args ""rest-server" "--port" 9070 "--home" "/Users/jolesbi/.Cosmos-dev/baseserver" "--node" "<IP_ADDRESS>:46657""

electron  gaia exited with code null

electron  gaia: 2018/02/05 13:20:16 Serving on ":9070"

electron  gaia server ready

electron  reconnected to <IP_ADDRESS>:46657


Go through API review notes regarding SOP-support for rpgp

SOP is an evolving standard way to run interop pgp tests, driven by the nice https://sequoia-pgp.org/ folks.
Here are some notes from @teythoon while he tried to write a sop backend for rpgp.
https://gitlab.com/sequoia-pgp/rpgp-sop/-/blob/hack/notes.org

Closing this, as a lot of feedback was processed, including the integration into rpgpie which implements the full sop interface now


Long compile times

Hey Envoy,
I'm trying to write a filter for Envoy to render HTML pages. I downloaded the echo2 filter example but the compile times take so long on my machine! After around 6 hours I had only completed 6/150 integration tests and things were still compiling. I don't need to build all of Envoy. I can build the library as a .so and load it into the prebuilt Docker.

Am I doing something wrong?
Is there a way to build filters without building all on Envoy?

Hey @ripread ,
Envoy builds in extensions to the resulting binary through static linking. There's no way to dynamically load a .so. Though you probably don't need to run all of envoy integration tests for your filter. Ideally you'd only run the integration tests for your specific filter. However, you'd still need to run a build of envoy which will probably still take awhile if your progress on integration tests is a good indicator.
That being said I'd recommend using one of the other extension mechanisms for just rendering HTML.

You could use LUA: https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/lua_filter (which seems like the most ideal right now based off of the limited information).

Or if you wanted to be more on the cutting edge you can look at WASM: https://github.com/envoyproxy/envoy-wasm/blob/master/WASM.md .


Old Dependencies from Python 2.7 (future, pymavlink)

DroneKit appears to have some older dependencies that need updating.
dronekit 2.9.1 has requirement future==0.15.2, but you'll have future 0.17.1 which is incompatible.
dronekit 2.9.1 has requirement pymavlink==2.0.6, but you'll have pymavlink 2.2.21 which is incompatible.
Unfortunately other libraries we're using require newer versions.

Appears master is up to date, so may be that pip and anaconda need to incorporate changes.

Uhm... there are still pinned dependencies in requirements.txt and setup.py which need fixing.
The problem also appears in CI. I think it's because the installation procedure installs dronekit-sitl, which in turn installs the old PyPI version of dronekit. This will be resolved when the next release is made.

I think this can be closed, those errors will go away upon a new DroneKit release on PyPI.

@peterbarker perhaps its best we cut a new release was this in your plans? do you have pypi.org credentials for dronekit?


[AKS] Switch AKS to use 2020-02-01 API

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/marwanad/azure-cli-extensions.git@vendor-aks-feb-api#subdirectory=src/$EXT&egg=$EXT"


Add Tags

http://docs.aws.amazon.com/cli/latest/reference/ec2/create-tags.html

Might be able to use the tag for the index.... hmmmm.

We're now adding tags.  Trying to think about how to bake the index in.  Some of the bash isn't right currently...

Karthik gave me an example script to add the index.
https://s3.amazonaws.com/scratch_bucket/Couchbaseinit.sh

A bit on the fence about whether or not to do this.  Only the rally node is really special, so the numbers don't add a lot here.

Decided not to do this.  We're not indexing with 0 anymore, but calling the rally ServerRally.


Add Fiat (FCA) Support for 500e

Currently there is no open Implentation of the APIs used for the Fiat 500e.
There is an App for Android and iOS whichs uses APIs to query SOC and additional vehicle information.
Goal
FCA vehicles respective Fiat 500e are configurable within evcc.yaml
FCA Vehicle SOC information can be fetched from FCA APIs and is usable within evcc

I have working credentials and will try to reverse engineer some of the API calls and can volunteer on implementation testing with my vehicle.

I have working credentials and will try to reverse engineer some of the API calls and can volunteer on implementation testing with my vehicle.

Sounds good! Please contact me on Slack if you have something to get startted.

Sounds good! Please contact me on Slack if you have something to get startted.

@mfueser any success on reverse engineering? Did you see https://github.com/andig/evcc/discussions/1090?

Hi, not yet unfortunately. Have to check that using Android oder the Webapp mentioned in #1090.

Seems there's also https://myuconnect.fiat.com which has the relevant data and might be easier to debug.


PDF preview doesn't work if the file extension is written in capital letters (.PDF)

PDF preview doesn't work if the file extension is written in in capital letters (.PDF) and the preview icon gets bigger. On the screenshot below the PDF files are the same, except their file names.

I can reproduce with Redmine 3.1.1.stable.14618
Seems to be a Redmine bug as they generate different CSS classes: pdf and PDF - but CSS is case sensitive!
Will see whether I can fix it with a workaround...

Fixed for redmine 2.6.x

Fixed for redmine 3.x
New version out soon...


[Bug Report] change server port to other udp than 8211

Have you read the Important information text above

[X] Yes i did, sorry if not

Current behavior
Changing PUBLIC_PORT to anything in env file and correct it in compose file.
The game sees the server offline.
Desired behavior
You need to set the following:
servermanager.sh:
Line: START_OPTIONS="-port=$PUBLIC_PORT"
Then it worked
Links to screenshots
No response
To Reproduce
Steps to reproduce the behavior:

Change PUBLIC_PORT in env file
Change port in docker-compose.yml
Start the server will lead the game to say that the server is offline / connection timeout.
Correct servermanager.sh may fix it.

Software setup

OS: Debian12
Docker: not relevant

Hardware setup

vCPU: 4
RAM: 32
Disk: 50GB

Additional context
Port Cmd-Line found here:
https://www.reddit.com/r/Palworld/comments/19b5byz/dedicated_server_not_letting_me_change_the_port/

Hey @chackl1990 - The PUBLIC_PORT is for external usage (same as PUBLIC_IP) and telling the Master-Sever on how to connect to your server. You dont need to change the Port=8211, thats inside Docker, thats why you can change it in Docker-Compose. There is no bug here. Just setup your Docker-Compose and default.env the right way.

ok got it - i may have missed that sorry.


Client Allocation Request for: Protocol Labs

Client Allocation Questions
Core Information

Name: Protocol Labs
Website / Social Media: protocol.ai
DataCap Requested: 1GiB
Addresses to be Notarized: f1m7rt4hasnidwe2dqf7475fizb3mn6npur2l4egq

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzacedcbxij6a3tyn6pw55xzqrfrtqdvm5vxdftaodpjkgjhkjeye2mbc

Address

f1m7rt4hasnidwe2dqf7475fizb3mn6npur2l4egq

Datacap Allocated

1GiB


MSA limit + API updates

The amount of Microsoft accounts per IP was dropped from 10 to 5 in the latest API update (3.58.0_build205), this sniper still has a max of 10, the rate limit was also changed from 6 post requests to 2, I have pull requested the rate limit update but will leave it up to you to change the account limit

Gotcha


[keepalived-operator] maintenance

Type of repository request
maintenance
Requested name for the repository
keepalived-operator
Additional details
Please add the following github secrets
QUAY_USERNAME
QUAY_PASSWORD
with push grants to the following quay repo (create them if necessary):
quay.io/redhat-cop/keepalived-operator
quay.io/redhat-cop/keepalived-operator-bundle
these repos should be public
Finally disconnect travis and any permission that it holds against this repo.
Review request:
cc: @redhat-cop/container-cop-core

Resolved by #311


Add example/blog preview on PR

Currently, opening a PR only deploy a preview deployment on Vercel for the example/docs example. We should have the same for example/blog (and add more stuff in it, but that's another story).
@shuding is it something you can do on your end ?
Thanks

Will get that added when I have time!


[Dev] Fix Sentry versioning and make dSYMs available for download from CircleCI.

CI failure is because of Fastlane not being able to parse the iTC response, which is probably a temporary situation.