syscall: in TestStdioAreInheritable, powershell on arm64/arm cannot load arm64/arm binaries because it is an intel process

ARM builds of Windows are still shipping an Intel powershell, which means we can't test arm shared libraries using powershell as a loader. This bug is to track that, if it ever changes, so we can remove the test skippage.

Is it possible to rewrite the test to not depend on powershell?

Is it possible to rewrite the test to not depend on powershell?

When I introduced TestStdioAreInheritable in CL 316269, I just adopted repro provided at https://github.com/golang/go/issues/44876#issue-825396212. After successfully built TestStdioAreInheritable, I tried to do something similar in C (compiled with GCC). But I could not do it.
I don't remember how much effort I put into conversion of TestStdioAreInheritable into C, but that was significant enough. I don't plan to work on this now. I also don't have any notes of what I tried.
Alex

I don't remember how much effort I put into conversion of TestStdioAreInheritable into C, but that was significant enough. I don't plan to work on this now. I also don't have any notes of what I tried.

Should we just remove the test? Is it particularly useful?

Should we just remove the test?

Why? What is the benefit?

Is it particularly useful?

The test verifies that #44876 stays fixed. I think it is useful.
Alex


Code completion for lua objects with :

Method completion with tab works for objects, but only if a dot is after the objectName, which is not valid for lua objects, so after performing completion the . has to be replaced with :
Since the context seems to know the class of the object already, maybe it would be easy to also make it work with :

+1


Fix duplicating items and opening from the form view

Related https://github.com/gisce/webclient/issues/663

:tada: This PR is included in version 2.0.0-alpha.32 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Generate OG summaries for shared chats

We should make a cloudflare function that injects OpenGraph title/summary into index.html for shared chats.
Title is title of shared chat and summary should be last message..or if the last message is short last n messages that fit into some string length

we might be able to just do this at share time by slightly changing format of shared chat to look like html with OG

I'm adding the ability to screenshot an element in #402, partially with this in mind.  Why don't we take a screenshot and POST that to R2 as well?

I'm adding the ability to screenshot an element in #402, partially with this in mind. Why don't we take a screenshot and POST that to R2 as well?

Lets do it! This will solve the remaining OG problem of it rendering ugly on sites that prefer image-heavy content


Delete validate-commit-msg app

When clicking through the app https://probot.github.io/apps/validate-commit-msg/ to the repo source at https://github.com/conventional-changelog-archived-repos/validate-commit-msg the repo has been archived

"This repository has been archived by the owner. It is now read-only."

The repo also contains the following notice:

"DEPRECATED. Use marionebl/commitlint instead. githook to validate commit messages are up to standard http://conventionalcommits.org"

p.s. The commitlint repo has its own probot app https://probot.github.io/apps/commitlint/

The CI failures seem unrelated from my understanding...

Thanks for the PR @ntwb and doing all that research! I'll open up a followup issue to add the new version to the site.


An empty string passes as valid data for a nullable=False, datetime=True data type.

Even though it gets munged to NaT, which is a pd.isnull object.

Note that it doesn't matter if nullable=True or not, the empty string is always junk (and not None) if datetime is True.
If you play around with it, you'll see madness ensues if we start accepting empty strings as being the equivalent to None.... they just aren't. Lets not do that.


Unable to parse

Hi,
I got this issue from alpha 2.0 build
 {"error":"unable to parse 'Oil 1 Real,equipment=Mixer\\ 1,status=Good value=12.5<PHONE_NUMBER>055000000': invalid field format
unable to parse 'Cool Mix Control Temp,equipment=Mixer\\ 1,status=Good value=464<PHONE_NUMBER>040000000': invalid field format

and this is the config for that metric
[[metrics]]
measurement = "Oil 1 Real"
dataType    = "number"
tags        = { equipment = "Mixer 1" }
nodeId      = "ns=2;s=_AdvancedTags.Oil Group.Oil 1 Real"
method      = "polled"
interval    = 300000

basically, all metrics are failed from being written to influxdb, any pointers for help?

First thought: could you try without spaces in the measurement name? If it
works like that, it's a bug in our code with spaces in the name, and we
should fix it :-)
On Thu, 2 Jul 2020 at 08:23, azwald<EMAIL_ADDRESS>wrote:

Hi,
I got this issue from alpha 2.0 build
{"error":"unable to parse 'Oil 1 Real,equipment=Mixer\ 1,status=Good value=12.5<PHONE_NUMBER>055000000': invalid field format
unable to parse 'Cool Mix Control Temp,equipment=Mixer\ 1,status=Good value=464<PHONE_NUMBER>040000000': invalid field format
and this is the config for that metric
[[metrics]]
measurement = "Oil 1 Real"
dataType    = "number"
tags        = { equipment = "Mixer 1" }
nodeId      = "ns=2;s=_AdvancedTags.Oil Group.Oil 1 Real"
method      = "polled"
interval    = 300000
basically, all metrics are failed from being written to influxdb, any
pointers for help?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/coussej/node-opcua-logger/issues/52, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL34VZIQ7X2X4DE77IKA4LRZQRUZANCNFSM4OOR5XFA
.

tried changing the space, and it works!
but I think the old data in buffer keep trying to be written to db, how to clear the buffer? thanks

Just clear the buffer files in the buffer folder that was created. :-)
On Thu, 2 Jul 2020 at 10:49, azwald<EMAIL_ADDRESS>wrote:

tried changing the space, and it works!
but I think the old data in buffer keep trying to be written to db, how to
clear the buffer? thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coussej/node-opcua-logger/issues/52#issuecomment-652877410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL34V5TNGXDFD4NOT4JC3LRZRCZVANCNFSM4OOR5XFA
.

thanks for the help! for now it seems I'd need to rename all measurements


Add package confutation

Landed in https://github.com/metarhia/metawatch/commit/0527b9bdb800e11458b8e8f612cf5fa77fdd8fc4, https://github.com/metarhia/metawatch/commit/5d6c2e437aa17c92a4f08c6b027f2bad3ec224b6, https://github.com/metarhia/metawatch/commit/59238e33b5c47d2cd66ff0c86f11d00a7af74c49, https://github.com/metarhia/metawatch/commit/a42a704deb932468dae1ec3490edd2786542005b and https://github.com/metarhia/metawatch/commit/8939936beff9b3965b091c6d42dd34fe257b3822


Move web tests to typescript: jasmine, karma

This allows us to test web with either the console or the web browser. Hypothetically, this could enable running @trinsic/web via node on the server.

Great work! As part of the next release, we should verify that this works with webpack and possibly the base react sample. Those are our manual smoke tests when it comes to integration.

This is something @MichaelEdwardBlack and I are already looking into. Good points on testing there. Karma tests here run under webpack already.


Add build stage for conda release to .travis.yml

Adds a build stage to travis to release to conda on tags, pip not yet set up due to missing credentials.

Thanks.


lancedb support

          @thinkall yes

its works but its always gives same output
from autogen.agentchat.contrib.retrieve_assistant_agent import RetrieveAssistantAgent
from autogen.agentchat.contrib.retrive_lancedb import LancedbRetrieveUserProxyAgent
from autogen.agentchat.contrib.retrive_lancedb import create_lancedb
autogen.ChatCompletion.start_logging()

# 1. create an RetrieveAssistantAgent instance named "assistant"
assistant = RetrieveAssistantAgent(
    name="assistant",
    system_message="You are a helpful assistant.",
    llm_config={
        "request_timeout": 600,
        "seed": 42,
        "config_list": config_list,
    },
)

ragproxyagent = LancedbRetrieveUserProxyAgent(
    name="ragproxyagent",
    human_input_mode="NEVER",
    max_consecutive_auto_reply=10,
    retrieve_config={
        "task": "qa",
        "docs_path": "https://raw.githubusercontent.com/microsoft/autogen/main/README.md",  # change this to your own path, such as https://raw.githubusercontent.com/microsoft/autogen/main/README.md
        "chunk_token_size": 2000,
        "client": "__",
        "embedding_model": "all-mpnet-base-v2",
    },
)

also im not user , adding 'client:'--' it is taking lancedb
? or should i need to define it again tere, also its not override any previous data thats reason i guess its always giving same output
i tried this way but working
table = db.open_table("my_table", mode="overwrite")
db = lancedb.connect('/tmp/lancedb')
table = db.create_table("pandas_docs", data=[
    {"vector": embeddings.embed_query("Hello World"), "text": "Hello World", "id": "1"}
], mode="overwrite")
docsearch = LanceDB.from_documents(documents, embeddings, connection=table)

Originally posted by @akashAD98 in https://github.com/microsoft/autogen/issues/25#issuecomment-1778514433

@thinkall okay thanks ill check it

i saw the repo, still im not sure why its not overwrite the existing data & use new data & there embeddings
every time getting same results

@thinkall can you give me some hints where its going wrong

@thinkall can you give me some hints where its going wrong

The code in the test is just for demonstrating, in your case, you need to update the code. If you're just copying the code from the test file, it won't work.

okay

Closing this issue due to inactivity. If you have further questions, please open a new issue or join the discussion in AutoGen Discord server: https://discord.com/invite/Yb5gwGVkE5


Act of Artifact Processing definition seems too restrictive for subclasses

The definition of Act of Artifact Processing is:

A Planned Act of performing a series of mechanical or chemical operations on something in order to change or preserve it.

For subclasses such as Act of Construction and Act of Manufacturing, I'm unclear what the something is that's changed or preserved.

Act of Construction uses construction of an airport as an example. The land on which the airport is to be sited is changed; is that the "something"?
Act of Manufacturing creates artifacts, possibly from raw materials. This doesn't necessarily change or preserve raw materials. The act can convert them into something else.

I think my issue may be the definition's use of singular: "something" and "it". Given the class's name, I would expect the something to be an artifact. Obviously it's not.
Here is a suggested revised definition:

A Planned Act of performing a series of mechanical or chemical operations, the output of which is new, changed, preserved, or destroyed artifacts.

("Destroyed" occurred to me because I recently worked on a project concerned in part with disassembling nuclear weapons.)

I concur, a good clarifying change.

@swartik @cameronmore  I believe the intent is that there is an identified object that is a workpiece, which is to be distinguished from byproducts and other incidentals that are changed in the course of the process. I don't think it makes sense to say that an artifact that has been intentionally destroyed has been 'processed'. cf. https://en.wikipedia.org/wiki/Industrial_processes "Industrial processes are procedures involving chemical, physical, electrical, or mechanical steps to aid in the manufacturing of an item or items, usually carried out on a very large scale. Industrial processes are the key components of heavy industry."
I think improvements can be made here, but I would not add 'destroy' as an option. Processes of destruction may be accommodated under a separate class.

Looking at the Cyber Ontology, I found Act of Artifact Destruction, defined as "An Act of Artifact Processing that consists of destroying some Artifact." If we expand our interpretation of 'change' in the definition of Artifact Processing, then this might be a properly asserted subclass, but maybe it should be a sibling. In any event, this should be raised in that forum and coordinated between the ontologies (Cyber Ontology Repo)


Persistent Connection with mysqli : define MYSQLI_NO_CHANGE_USER_ON_PCONNECT for Windows build

On page https://dev.mysql.com/doc/apis-php/en/apis-php-mysqlnd.persist.html , it introduced how to disable change user for persistent connection as follows:
shell# CFLAGS="-DMYSQLI_NO_CHANGE_USER_ON_PCONNECT" ./configure --with-mysql=/usr/local/mysql/ --with-mysqli=/usr/local/mysql/bin/mysql_config --with-pdo-mysql=/usr/local/mysql/bin/mysql_config --enable-debug && make clean && make -j6
or
shell# export CFLAGS="-DMYSQLI_NO_CHANGE_USER_ON_PCONNECT"
shell# configure --whatever-option
shell# make clean
shell# make
However this only works on Linux build, may I ask when I want to build php mysqli using PHP SAK tools for windows following steps on this git post, what should I do to define MYSQLI_NO_CHANGE_USER_ON_PCONNECT to disable change user?

Thanks for the question. On the Windows side passing CFLAGS, LDFLAGS and others is not supported. Probably it's a good idea to implement it. This is actually not related to the PHP SDK, but to the PHP core. Could you please open a feature request on bugs.php.net and assign it to me?
This is also not exported as a configure as a configure option in PHP. For now what you can do is edit the Makefile manually to add tthis define.
Thanks.

Okay, I'm closing this ticket in favor of https://bugs.php.net/77230.


App crashes on javascript errors when Hermes is enabled

I am in the processing of migrating our app from v0.59.9 to v0.63.3. While i got other things migrated successfully, i noticed that javascript (non-fatal) errors that used to be thrown in a red screen earlier are not reported the same way. For basic javascript errors like undefined is not a method, the app crashes and then i have to check adb log cat and then de-obfuscate them using metro-symbolicate to see what the error is. I am tried running the app with --variant=debug without success. Is there a better way to see these errors without this cumbersome process?
https://i.stack.imgur.com/QhTSw.png

Hermes currently doesn't fully support ES6, and it might be a cause especially with react-native-firebase. in 0.64, Hermes will support almost all ES6. Please use JSC until then. Thanks

Hermes currently doesn't fully support ES6

@dulmandakh emm I'm not sure if that is the root cause. Isn't ES6+ JavaScript source code going through Babel transformations (via Metro) anyways? cc @motiz88

in 0.64, Hermes will support almost all ES6.

Also, this is not true. Please see https://github.com/facebook/hermes/blob/master/doc/Features.md for the ES features coverage.

@kaushikwavhal are you using some of the Proxy-based libraries e.g. react-native-firebase or MobX?

@Huxpro No, i am not using these libraries. I went on fixing each errors from the log cat and now the app does not crash but i still see the errors in adb log cat instead of the red screen.
Here's one latest example. Not sure how i should trace that.

Please use JSC until then

@dulmandakh When i disable hermes and enable jsc, i am perpetually getting the React native version mismatch error. I even grep'd the whole code base to see if the older version is left somewhere but no success.

edit:
Even with Hermes disabled, all the javascript errors are logged in adb log cat and i can't see the red error screen. Also, i can't happen to open the developer menu any way even when the debug apk is installed through react-native run-android

i was able to get the developer menu working by removing this from MainApplication.java:
import com.facebook.react.BuildConfig.
Re: https://github.com/facebook/react-native/issues/26417

This worked after removing import com.facebook.react.BuildConfig and android:usesCleartextTraffic="true". Never been more happy to see an error screen.


[iOS][Custom Actions] Support custom action parsing for selectAction

Platform

iOS

Author or host
host
Version of SDK
1.2.1
Details
It doesn't currently seem possible to use custom actions for a Container's selectAction. It looks like selectAction uses ParseContext's actionParserRegistration to look for action parsers corresponding to the action type, unlike the ACRCustomActionRenderer, which will call ACRRegistration. I'm not sure how we could override the selectAction parsing unless it used ACRRegistration as some point. Could you confirm this, or let me know if there's an alternative for what I'm trying to accomplish? Thank you.

@nesalang thoughts ?

@tnev @shalinijoshi19
Hello,
we can add this feature to iOS. basically, it will allow registering custom targets for actions. action rendering involves creating button and adding the targets. Since SelectAction is adding the targets to existing UI elements such as UIImageView, current action renderer registration will not work and a new target registration is needed.
And I could share a branch that has this feature if you are blocked, but I have to check to see if this feature is supported on all other platforms, and make sure the way we support is consistent across the platforms, so complete solution may come later, but the release of iOS solution won't be depended on implementation of other platforms just designs unless our team wants to time the releases as well. because of that reason, the change I share with you may change.


Rahti allas

Bucking up to Allas from Rahti

Bucking up to Allas from Rahti

Typo

https://csc-guide-preview.rahtiapp.fi/origin/rahti-allas/

Good article overall, there are only few minor things to point out. :)


Fix p2tr spend/change bug

In version 0.6.0 when p2tr signing was introduced a bug in the change logic causes an error psbt_parser.py. It'll show something like "script.py, 165 in p2tr". This PR fixes that issue and I created a unit test for the fix.

Minor notes above, but otherwise ACK!

Thank you Nick.
I had originally setup my tests, with existing code, but using a single taproot wallet, transferring internally to it else to native segwit (to the same seed).   I still never succeeded in having provoked the error when taproot-to-taproot within the same wallet descriptor.
I was able to consistently provoke the error, with help from Manu who reported a similar problem yesterday -- after he mentioned "it only happened when transferring OUT OF the wallet to taproot".  In fact, I could never get the existing code to succeed in that case.  So anyone doing real taproot transactions, and not to themselves, must be having this problem.
Once I reinstalled your fix, the problem went away and I have not seen the same error again.
ACK tested -- with one request.  There are some tabs characters in the unit test file, would you consider converting them to 8 spaces for consistency?
We never mentioned this pr in last weeks meeting, but as soon as it's ready (I now believe it is), then I would push it to a priority for next release.

There are some "tab" characters in the unit test file, would you consider converting them to 8 spaces for consistency?

Yes


Nessie 0.30 iceberg 1.0

Yes - it tests released artifacts using the tests in this repository.

See README.md


Left over error message when successfully adding an API

['Character Creation onKiah Strawberryfailed', 'Character Creation onBrown Cyanfailed', 'Character Creation onKailea Tradeberryfailed']
It still works though.

fixed in development pushing later


How to implement the Summer tileset?

Hello, awesome job on this port!
I've used the examples you provided and they work perfectly fine, so I've tried to implement the "Summer" tileset from the original repo, but it doesn't seem to work.
Here the gist of the definition that I've made using your example and the original repo xml: https://gist.github.com/MatveyK/8e04c76120a0e10e0cffa9dac611b8f2
Except that when I try to run the Simple Tiled Model with it and the associated tile files in the /summer folder, I get this message:
(node:11499) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: ENOENT: no such file or directory, open './data/summer/cliffcorner.png'
Thus my question is: how do you organize the definition file in this case? Because translating the original repo xml to json object doesn't seem to work in this case.
I've seen that in your animated simple tile example you run the summer tileset, I was wondering if you could upload the definition file for it? Or if you could explain the structure of the definition file?

Hi, thank you ! It seems that the issue is that I haven't implemented support for the 'unique' attribute in the node.js example so it tries to load the wrong file (ie cliffcorner.png instead of cliffcorner 0.png, cliffcorner 1.png, cliffcorner 2.png and cliffcorner 3.png) whereas it is correctly handled in the browser demo. I'll try to get that done later today.
For reference, all the translated SimpleTiledModel definitions for the browser demo can be seen here : http://www.kchapelier.com/wfc-example/simple-tiled-model-data.js The only difference with your summer definition is that it includes 'weight' attributes on some tiles and that it doesn't include the 'path' on each tile because of the different image loading process.

Thanks for the quick answer and for your help!
Another quick question: could you enlighten me as to what the "symmetry" values are meant to represent? I understood that the neighbors list defines the rules for right-left neighbours (as in which tile can be followed by which tile), but the meaning of the symmetry values eludes me.

https://github.com/kchapelier/wavefunctioncollapse/commit/0e394c5842252dcd330c33ca829fc5f5302b6663
I fixed the node simple-tiled-model example so that it supports unique tilesets. It can be tested by just replacing the require('./data/castle.definition.js'); with require('./data/summer.definition.js');.
Regarding the symmetry value, you should check this discussion on the original repository.

Thanks a million for the quick reply and help!


Updated chat URL to 'help'

Found this additional update!
General note: should it be a bullet? The formatting looks a bit weird:
(Current screenshot)

https://help.aragon.org/article/25-failed-organization-creation-transaction

Good catch! Thanks @stellarmagnet. Have fixed the formatting too, I think the idea was to give a list of options in case there was more than one way to resolve the issue but we haven't yet developed any other way to resolve the issue other than doing it manually as mentioned in the docs.


Issue with escape characters

After the previous fix related to the string, containing special characters, seems that escape character is still messing up tree structure.
Here is the string containing those: nonAlphaChars = "!@#$%^&*()-_=+[]{}\|;:'~",<.>/?~~"
If the characters ~ are in the string then the code from some previous double quoted string up until this one is considered as a double_quoted_string and it brings up errors. When I removed all ~ from the string, then issue is not there anymore.
Example:
  method public static log isAlpha(oneChar as char):
    if oneChar >= "a" and oneChar <= "z" then
      return true.

    return false.
  end.

  method public static char replaceNonAlpha():
    def var nonAlphaChars as char no-undo.

    nonAlphaChars = "!@#$%^&*()-_=+[]{}\|;:'~",<.>/?~~".

    return nonAlphaChars.
  end method.

Tree shows that double_quoted_string starts from "a" and ends with ending of special char string.

Okay... I think we'll try to just use external scanner for all string, now double_quoted_string and single_quoted string will be just string_literal... Deploying in 0.0.35

This is quite drastic change, let me know if it stays the same or you care about diffrentiating between single_quoted and double_quoted (I don't think progress does..., does it?)

It is working correctly now. And at least at the moment I don't see a reason, why we would need to differentiate between those. All good! Thank you!


Keyboard shortcuts do not work in on-file-version abapgit_full

The reason seems to be that keyboard shortcuts are collected from all classes implementing the interface zif_abapgit_gui_page_hotkey. Global classes. This is also then needed for settings to redefine shortcuts on system basis. However in abapgit_full there are no global classes.
P.S. I'd suggest implementing shortcuts so that a page register own shortcuts on rendering (also do the same with on_event handlers, was discussed somewhere here ...). As for shortcuts redefinition - keep the existing approach but disable shortcut change in one-file version probably.

@sbcgua should work now, can you try?

Yeap, I will on the weekend. Thanks!

Works perfectly, thanks ! closing


Help, error zlib

hi everyone, i tried install heron in my noteook   (ubuntu 19.10) with the steps from docs folder and y have this error:
ERROR: /home/nico/Escritorio/tarea4cripto/apacheheron/incubator-heron/heron/executor/src/python/BUILD:13:11: PexPython heron/executor/src/python/heron-executor.pex failed (Exit 1)
Traceback (most recent call last):
File "/usr/local/lib/python2.7/runpy.py", line 165, in _run_module_as_main
mod_name, loader, code, fname = _get_main_module_details(_Error)
File "/usr/local/lib/python2.7/runpy.py", line 133, in _get_main_module_details
return _get_module_details(main_name)
File "/usr/local/lib/python2.7/runpy.py", line 121, in _get_module_details
raise error(format(e))
ImportError: can't decompress data; zlib not available
INFO: Elapsed time: 3.583s, Critical Path: 2.36s
INFO: 3 processes: 3 local.
FAILED: Build did NOT complete successfully
i have install zlib1g but python don't use the module. help i need make a video with heron

I tried ubuntu20.04+python3, looks good to me. Do you still see this issue?

Closing this issue. Please re-open a new issue if the problem persists.


bufio: make zero value of Reader useful

Consider the following code pattern:
var br bufio.Reader
for _, r := range readers {
    br.Reset(r)
    ... // make use of br
}

Currently, this pattern results in an infinite loop since the internal buffer of bufio.Reader is empty (resulting in no progress being made). The bufio.Reader.Reset method should initialize the internal buffer with defaultBufSize if it is empty.

this pattern results in an infinite loop

How so? The program below doesn't hang.
package main

import (
	"bufio"
	"io"
	"os"
	"strings"
)

func main() {
	var br bufio.Reader
	readers := []io.Reader{strings.NewReader("data\n")}
	for _, r := range readers {
		br.Reset(r)
		io.Copy(os.Stdout, &br)
	}
}

Interesting, this program does hang:
package main

import (
	"bufio"
	"io"
	"strings"
)

func main() {
	var br bufio.Reader
	readers := []io.Reader{strings.NewReader("data\n")}
	for _, r := range readers {
		br.Reset(r)
		br.ReadBytes('\n')
	}
}

this pattern results in an infinite loop

How so? The program below doesn't hang.

It does not hang, when calling io.Copy. Because io.Copy calls src.Read, bufio.Reader.Read here, and in turn, bufio.Reader.Read calls the internal reader's Read function when buffer is empty.

Isn't it OK just doing this:
func (b *Reader) Reset(r io.Reader) {
        // add this if statement
	if len(b.buf) < minReadBufferSize {
		b.buf = make([]byte, minReadBufferSize)
	}
	b.reset(b.buf, r)
}

minReadBufferSize

This will not save memory

Isn't it OK just doing this:
func (b *Reader) Reset(r io.Reader) {
        // add this if statement
	if len(b.buf) < minReadBufferSize {
		b.buf = make([]byte, minReadBufferSize)
	}
	b.reset(b.buf, r)
}

This will not save memory

Isn't it OK just doing this:

Yea, that's more or less what's proposed with saying "bufio.Reader.Reset method should initialize the internal buffer with defaultBufSize if it is empty."

This will not save memory

I'm not sure what you're getting at. This issue isn't about saving memory, but making the zero value of bufio.Reader useful.


draft feat(app): Persist the activity logger in the storage

Signed-off-by: Talwinder kaur<EMAIL_ADDRESS>

Codecov Report
Base: 92.09% // Head: 92.09% // No change to project coverage :thumbsup:

Coverage data is based on head (24b87bd) compared to base (6d123c5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #246   +/-   ##
=======================================
  Coverage   92.09%   92.09%
=======================================
  Files          49       49
  Lines        1873     1873
=======================================
  Hits         1725     1725
  Misses        104      104
  Partials       44       44

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Need to revisit this logic later


Upgrade to ES 2.1.0

Please let my changes will be applied. :+1:

Thank you. But the patch is not correct. Fixed with https://github.com/jprante/elasticsearch-langdetect/commit/e505506b3c14cc84e6b064bbd2b27e85dc1385ec


Grid component warning with non truthy zeroMinWidth prop

Grid component issues a warning when handling false or null values for zeroMinWidth prop

[] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
No warning
Current Behavior 😯
Warning
Steps to Reproduce 🕹
Toggle the testFalse and/or testNull variables to true and look in the developer tools console. The warning will not appear in the Repl.it console.
Example
Context 🔦
I'm just trying to use the component according to it's docs
Your Environment 🌎

Tech
Version

Material-UI
3.9.2

React
16.8.2

Browser
chrome

TypeScript

etc.

@pmacom Thank you for raising the issue. We can fix it like this:
--- a/packages/material-ui/src/Grid/Grid.js
+++ b/packages/material-ui/src/Grid/Grid.js
@@ -385,7 +385,7 @@ if (process.env.NODE_ENV !== 'production') {
     spacing: requireProp('container'),
     wrap: requireProp('container'),
     xs: requireProp('item'),
-    zeroMinWidth: requireProp('zeroMinWidth'),
+    zeroMinWidth: requireProp('item'),
   };
 }

Do you want to submit a pull request? (on the next branch ideally)


fix(addon-editor): class constructor NodeView cannot be invoked without new

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Build or CI related changes
[ ] Documentation content changes

What is the new behavior?
Closes #3815

@nsbarsukov fixed, but I need retrigger nightly build for deploy new token from cdk package


DBAAS-364: update Instance Phase field to use the constants from DBaaS API

Update Phase field in the status of DBaaSInstance object with the values defined in the DBaaS API.
Follow-up to https://github.com/CrunchyData/crunchy-bridge-operator/pull/54

@redhatHameed @jeremyary  please review.


When do I need this?

As a new user, I am missing a little documentation when I should use this.

Which minimum Thumbor version is required to use this engine?
Up to wich version was this engine included in Thumbor?
Which Thumbor features require this engine to be installed?

Seems this does not work with Thumbor 6: https://github.com/thumbor/thumbor/commit/c19e2b0e3ad11334f7d94aa9054a6a84246a758d#commitcomment-14364435


How to add a table entry field in multi_index without deleting node data

I want to save the additional data information of the user. I know I will put it into the database, for example, resource_limits_index. This will generate content of memory does not match data expected by executable.How can you avoid this without deleting the original node data?

You must create the new version of your multi_index alongside the old and transfer data from the old to the new.  You may do so as part of your normal access pattern, first checking the new table to see if the entry you seek is present and if not, check the original table, and if it's present, migrate it while adding the data for the new field, then remove it from the original table to save RAM costs.  You must retain both versions of your multi_index until you have completed this migration, at which point you may update your contract to remove the original version of your multi_index.
If you prefer less code complexity and can accept downtime for your application, you may upload a version of your contract solely for migration purposes, and run migration transactions on every row of your table until complete, then transition to a new contract using only the new version of the table.
Either way, you must write code supporting both versions of your multi_index and migrate the data yourself.  There are no native provisions for adding fields to an existing multi_index.


x/website: github icon in nav is not visible in dark-mode

In "Get Connected" section

how do you activate the dark mode? I can't see it on the go.dev?

There should be an option in your browser's web dev tools


Expand module API documentation in line with recent submission to wiki.

Sync ChakraCore.h comments with recent submission to wiki:
https://github.com/Microsoft/ChakraCore-wiki/commit/e4aba5f23ebb2fbac99cc144aead3653d171847a
@dilijev @liminzhu

This is pretty exhaustive, thanks a lot for your contribution here. I will look it over in detail but looks good from a glance.

Whatever text changes arise from this PR will need to be sync'd back to the wiki. @liminzhu do you want to work on a script to automate generating the wiki markdown from the source code?

I'd like to give @liminzhu a chance to take a look but baring any issues there, seems we have a consensus that this can be landed.

@rhuanjl :shipit: thanks for another awesome contribution!
@dilijev yeah that'll make a fun weekend project for me

Thanks for your contribution. 👍


chore(examples): remove examples cargo lock

This commit removes the Cargo.lock for http-blobstore that was likely accidentally checked in, and adds a .cargo/config.toml so that the project can be built with cargo build.
Feature or Problem

Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

@rvolosatovs we don't check in Cargo.lock for any other example, why do you want it to persist for this one?

@rvolosatovs The main downside for examples being a part of the workspace is when we generate them as a part of creating a new project template, then the examples don't stand by themselves. Outside of the ones that we generate as templates I'm fine with the workspace, though

Still probably best to make all components part of a single workspace

Agree with that! Also @brooksmtownsend 's poitn is good, I think the Cargo.lock would get overriden instantly in a new template project, but not having it is nice as well.
AFAIK we shouldn't make this change for tests/components because that really is another workspace sort of hidden inside the top level one, but this one seemed reasonable. Thanks for looking at it closely!

@rvolosatovs The main downside for examples being a part of the workspace is when we generate them as a part of creating a new project template, then the examples don't stand by themselves. Outside of the ones that we generate as templates I'm fine with the workspace, though

sounds good, let's discuss and address in a separate PR/issue


OTGR	Object type group

serialization of OTGR Object type group
use class CL_CLS_OBJECT_TYPE_GROUP, similar to https://github.com/larshp/abapGit/pull/1513
transaction SOTG

this one should be straightforward, see hints above


Cleanup welcome message on creating new account

should match voice / content of new account email

@jonhorton did we ever get copy for a welcome to your newspring account? We need around 2-3 sentences


Bump test projects up to .NET 4.5.2

aspnet/Testing#248
xUnit no longer supports .NET 4.5.1
build tests for desktop .NET only on Windows

Hi @dougbu, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Another check-less repo. Eyes appreciated.

4df172a


"Update fields" feature

For example, to be able to update other fields of the domain record (like "port", "priority" etc).

@toin0u just to clarify - is this repo still maintained? or should I work in a separate fork?

Yeh, it's still maintained. :)

Is this totally BC?

@GrahamCampbell

Is this totally BC?
It's totally BC! update and updateData use the same code to make a PUT request.
I just moved that code to a single updateFields method, where you can set any fields to be updated.

@GrahamCampbell sorry, not quite familiar with the PhpSpec methodology, but I've tried to add the updateFields test as you asked.

Thanks. :)

@GrahamCampbell could you please update the composer/packagist data as well?


Repeat request after 429 error

If a 429 error is returned by Instagram, the request will simply be skipped, instead of be repeated. This commit changes that behaviour.

Hi, thanks for your commit.
Don't you think that this may cause an infinite loop in case of an account being blocked?

Good afternoon, thank you for your feedback!
Take in mind that all of this is based in my little experince with your bot (follow, unfollowing and disliking actions mostly), and I haven't tested all of the methods that it provides, so I have little experience in which errors codes are returned by Instagram in certain tasks.
AFAIK, the 429 (too many requests) error code must only be returned when a client has sent the maximum allowed number of requests per an established amount of time to a server. I suppose that when you are trying to follow an user that has blocked you, Instagram should return a 403 (forbidden) error. If that's not the case, then, they are doing stuff in a non-standard way.
If the request we are trying to send is malformed or the action is not allowed by Instagram, they will return us other error codes that should be handled differently by the bot. However, 429 error is only (again, based in what the 429 error means) raised when the server requests the client to wait before sending another request.
This commit only repeats the sent request if the 429 error code is returned, so only requests that are blocked by Instagram because we need to wait before sending another one will be repeated. Other kind of errors are untouched and will be handled as they are handled right now.

Updated to master

Updated my branch to current master and fixed styling issues to comply with Travis tests

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1106   +/-   ##
=========================================
  Coverage          ?   43.65%
=========================================
  Files             ?       32
  Lines             ?     3569
  Branches          ?      579
=========================================
  Hits              ?     1558
  Misses            ?     1869
  Partials          ?      142

Impacted Files
Coverage Δ

instabot/api/api.py
50.38% <0%> (ø)

Continue to review full report at Codecov.

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

@ferferga Thank you for your help!

Great PR by the way


doc: add section ids

The ids are used to create readable links when linking to (sub-)sections directly. This commit also use enforces consistent use of the prefixes "sec" for section and "ssec" for subsection.

Thanks!

It seems this is breaking the build http://hydra.nixos.org/build/22724020.

Should build now again (9c983ade4d56f9585fd2a806c8ee1650e180b3eb). Somehow I managed to not stage and commit the search & replace changes.


Developer Color : Chrome Extension Added

🛠 Fixes Issue
Fixes #286
👨💻 Changes proposed
This Chrome Extension would highlight certain words for Developers, so that they can easily use it for their customized use case.
🔧 Check List (Check the type of issue it fixes)

[ ] Bug Fix
[x] New Extension
[ ] Enhancement
[ ] Documentation

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📷 Screenshots

@harshita214 Kindly Review this PR.


Auto pause and resume sql synapse analytics database

Hi,
As i mean want to pause database on Saturday, Sunday and Monday morning want to resume automatically using any script or any option is to do ? or how?
Thanks!

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

ID: d0b3077e-af25-be25-e8c4-8414870fdc00
Version Independent ID: 37fda171-c093-9913-9464-35479bd062fb
Content: Pause and resume compute in Synapse SQL pool via the Azure portal - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/pause-and-resume-compute-portal.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @kevinvngo
Microsoft Alias: kevin

@bhumid123 - Thanks for posting the query. This can be achieved using Azure Automation. You can find the sample PowerShell scripts in the automation gallery as shown below and this documentation helps you in understanding scheduling this runbook for specific days as per requirement.
If you are using Synapse SQL pool, the PowerShell commands can be found from this link else SQL DW commands in gallery would work.
Also, since your issue is not related to documentation, I would recommend you to post such issues on forums. The audience on forums is better suited to answer such queries.
We will go ahead and close this issue now. Feel free to open a thread on Q&A or Stack Overflow if your query was not resolved.


bug: pubsub connection attempts in endless loop

Describe the bug
While attempting to run all of the go tests in this project from my personal machine, which I understand probably doesn't have the proper test environment, I noticed the pubsub tests attempting to make a connection to the pubsub in an endless loop.
{"level":"ERROR","time":"2024-10-18T15:18:45.425787-05:00","message":"error in subscription for topic products: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425809-05:00","message":"cannot subscribe as consumer_id is not provided in configs","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425798-05:00","message":"error in subscription for topic order-logs: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.42582-05:00","message":"cannot subscribe as consumer_id is not provided in configs","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425826-05:00","message":"error while reading from topic order-logs, err: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.42583-05:00","message":"error in subscription for topic order-logs: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425815-05:00","message":"error while reading from topic products, err: consumer group id not provided","gofrVersion":"dev"}

I think we should consider implementing some kind of exponential retry/backoff strategy here and consider returning an error.
To Reproduce
Steps to reproduce the behavior, if applicable:

The code is

go test -v ./...

2. The error is

{"level":"ERROR","time":"2024-10-18T15:18:45.425787-05:00","message":"error in subscription for topic products: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425809-05:00","message":"cannot subscribe as consumer_id is not provided in configs","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425798-05:00","message":"error in subscription for topic order-logs: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.42582-05:00","message":"cannot subscribe as consumer_id is not provided in configs","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425826-05:00","message":"error while reading from topic order-logs, err: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.42583-05:00","message":"error in subscription for topic order-logs: consumer group id not provided","gofrVersion":"dev"}
{"level":"ERROR","time":"2024-10-18T15:18:45.425815-05:00","message":"error while reading from topic products, err: consumer group id not provided","gofrVersion":"dev"}

**Expected behavior**
Should return an error eventually.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Environments (please complete the following information):**
- OS: [e.g. Linux]
- gofr version [e.g. v1.5.0]
- go version [e.g. 1.21]

**More description**
Add any other context about the problem here.

@mfreeman451 This is not a retry connection, but a subscription issue with Kafka where when it is trying to subscribe to the topic, the consumer group ID is not provided, and it throws the error that you mentioned.
I think we should look for a solution where the subscription cannot start until the pubsub is connected and ready.

@mfreeman451 , can you help me understand , if this is a issue only during test or while actual implementation also ?


add networklogging_v1alpha1.json

This PR adds CRD for NetworkLogging

@JackOneil can you add a link to the CRD origin on github? (example)

@JackOneil can you add a link to the CRD origin on github? (example)

Hi, unfortunately i was not able to find the original CRD anywhere on the github nor public pages as its an internal GKE resource.
In case the CRD is needed, i could export it from the running cluster.


Syntax error in MITT ACPI code for device at end of TP3

The device TP3 has an extra right curly bracket, so the sample I2C MITT ACPI code does not compile.

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

ID: 09394a3d-9e07-9591-2588-f2d9ca0934da
Version Independent ID: 716cd7b5-3888-8afa-f2fc-01843afe91dd
Content: I2C controller tests in MITT - Windows drivers
Content Source: windows-driver-docs-pr/spb/run-mitt-tests-for-an-i2c-controller-.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Hi @jcbottorff-ampere,
Thank you for reporting this. The topic has been updated.
~ Mark


how to set ScaleDenominator when using WebMapTileServiceImageryProvider?

Our project use wmts provided by other company. but there only 10 TileMatrix in the TileMatrixSet, and every ScaleDenominator is unusual. where can i set the scale?

The ScaleDenominator isn't configurable in CesiumJS. Can you describe your use case a bit more? Are the map tiles you're loading not at the right scale and need to be scaled up or down?

The ScaleDenominator isn't configurable in CesiumJS. Can you describe your use case a bit more? Are the map tiles you're loading not at the right scale and need to be scaled up or down?

Yes, it‘s load the wrong tiles, the TileMatrix TileRow TileCol to request are all not right. I think the reason is wmts ScaleDenominator is unusual.

This is a part of the Capabilities.


Maybe a problem，about windows path

i wrote a custom filter wiht jinja2：
the content of demo.py:
import jinja2
from jinja2 import  Environment

scriptLoader = jinja2.FileSystemLoader(searchpath='.')#
env = Environment(loader=scriptLoader)
def extract(_factory_args, *_factory_kwargs):
    def _extract(path, _request_args, *_request_kwargs):
        print request_args[0]
        return 0
    return _extract

env.filters['extract'] = extract()
tpl = env.get_template("demo.tpl")
print tpl.render()

and the demo.tpl is:
{{"error"|extract("E:\project_test\Edu_Root3.0\OnlineService\data\320068142153001.txt")}}
after rendered,  the jinja2 render '\320' to be \xd, and result like this,
E:\project_test\Edu_Root3.0\OnlineService\dataÐ068142153001.txt
this is wrong result. i want it to be E:\project_test\Edu_Root3.0\OnlineService\data\320068142153001.txt
i know one solution is  change the path in tpl, but is there any other method that deal it well in jinja2 filter's funtion?

Simply escape the backslashes..

Because  user can copy from windows‘s explorer, and  not want modify it self.
Now i use re library to replace them , but i still confused about  how jinja2  deal with it, and what does it do in dealing  wih the  ‘\’ character

That's just not how strings work in Python. Escape characters are going to escape things.


doc: Update K8s installation doc

@tong101
Can you please add a Signed-off-by to the commit? You can do that with "git commit --amend -s". I think we should get rid of the quick start guide anyway and simply point to the vagrant directory. But we can do that in a different commit.


[Feature Request] Add a way to remove groups

Since groups stay in memory even after everyone in the group disconnects (or at least this is the behavior I've been experiencing), you could end up with millions of groups sitting around in memory with nobody connected. Adding something like the below would be helpful.
nowjs.removeGroup(groupName);

yes, it sounds like a good improvement. but then only the server side should be allowed to call it. otherwise, a client may easily disconnect other clients in the same group.

Yes, good point.

already refined it on the branch removegroup. it may come out later

Hi goldfire,
This functionality is now in the github master. It'll be released with the next version on npm.
If you're on the github master, you can use nowjs.removeGroup(groupName);
Thanks to @steveWang @Peterfly  for helping with this.


How to close market and claim back SOL

Hi,
I want to know if there is any way we can close the Market
or edit its parameters (Minimum order size, Minimum price ticket size).

I'm also interested in wether this is possible?


Feature request: min_group_size for other category encoders, e.g. one hot

CountEncoder() has a min_group_size parameter that sets a minimum number of obs in a group required in order not for the group to be lumped together with other small groups. It would be nice to have this for the other encoder classes.

I did this for OrdinalEncoder, since there was also #81 issue, which was basically asking for the same thing. I can create pr if this is still needed

Hi @glevv
I'm sorry, I haven't seen you've been working on this. Why did you close the Pull Request after only one day? I think it would be a useful feature and I'd be happy to include your changes.

@PaulWestenthanner I think I started doing this, made a beta version, but after testing it realized that I will have to go deeper and change a lot of things and I did not have spare time for that then.

Ok nice! Do you want to continue working on it? Otherwise I might look into it in March/April

I would like to work on it.

Hi Julia,
great! Looking forward to your contribution :)

Hi @PaulWestenthanner ! I refactored the code count.py so the functionality can be extracted and pulled into the base class, so all classes can inherit this feature.
Some comments that I'd like to make:
There's an attribute called self.combine_min_nan_groups. It has three values and defines how to handle nan values in the categories. The behavior is accordingly:

If self.combine_min_nan_groups==True: Treat nan just like any other group. If it is a small group, treat it accordingly and combine it with the other small groups.
If self.combine_min_nan_groups=='force': Add the whole nan group to the combined category group, no matter its size
If self.combine_min_nan_groups=='False': The nan group is not added to the combined category group, no matter ist size.

I would recommend to leave out this parameter and always go for the default for the following reasons:

Some force parameters is not compatible with several other settings of the countEncoder class, e.g. handle_missing_values. This has to be caught in the code and makes it harder to extract the behavior to other classes and there might be additional problems.

The force behavior is unintuitive and not what a user might expect. The docstring is hard to understand. And, in my opinion, the category encoder's responsibility is not to deal with nan values. Or again, we do a feature on a BaseEncoder level, so it is consistent for all modules. If we don't handle this here, the user can handle NaN themselves in their preprocessing pipeline.

If we do this refactor, the code shrinks by about a third, and becomes easier understandable and maintainable.
What do you think?

@PaulWestenthanner Another simplification I'd suggest. There is right now another attribute called min_group_name. The logic is as follows:
if the min_group_name is set for a column, the minimum category gets this custom name. Else, all the minimum category name strings are combined with _ to a long string.
I think this parameter should be ommitted because it seems to lead to inconsistent behavior and changing names,             Note: The default name can be long and may keep changing, for example,              in cross-validation.
This is unexpected behavior. Moreover, the user should never see this feature, because the data will go right into the category encoder. Moreover, in the docs it says the custom option is 'in case the combined string' gets too long. I wonder when that is, and if the user should be aware of this.  I'd rather hide that behavior.
I suggest having a kind of default name wrangling without custom option. Then we get rid of this parameter and the code gets easier to understand. I would suggest for name wrangling a name that's like __other__ + some random hash, so we make sure the name does not appear in the original dataset by chance.
if we do the above two suggestions, the min_group feature can be controlled by just one parameter, called min_group_size. This would be much cleaner.

My suggestion for the integration to BaseEncocer: We add the min_group_size parameter to BaseEncoder attributes, so every subclass has it. The min_group preprocessing happens before the actual fitting and the actual transforming of the encoder, so we could add this behavior to the beginning of the BaseEncoder __super__().fit() and __super__().transform() method.
Then all the subclasses have it without us having to change their code.

Hi Julia,
thanks for all the work you've put into this. I really like your suggestions:

The CategoryEncoders library aims to handle missing values. Especially for target-encoding-based encoders this is very helpful since a good default value for filling the nulls can only be calculated during fit time.
I'm not sure if I'd just drop the force option. Null values might have a meaning and hence there should be the option to treat them separately, so we need the option to not include them in the other category, but also the option include them in the other category as other data sets might require otherwise. If the force option is necessary I'm not sure. I agree that most of the time you probably won't want it. On the other hand, this would make the change not backwards compatible, so I'm a little hesitant there. Also if you have the option to either include the NaNs in other or not you'll necessarily have conflicts with handle_missing='return_nan' for example. So it won't be easy to get rid of all these checks
Couldn't agree more with the naming thing. Good catch! We should definitely change this.
Regarding BaseEncoder: This sounds perfect!


fix NPE in getWholeImageRect(). resolve #339

we have check in getCropRect(), but haven't in getWholeImageRect().
it cause many NPE in the prod

@ArthurHub hi. could u check please, seems urgent

may be better return Rect() in both getCropRect and getWholeImageRect() in case bitmap equal null?

thanks!


the go.exe returns none under windows

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
I installed a stable release of go1.6.3 on my windows. However, i execute 'go version' under the cmd.exe as administrator.

What operating system and processor architecture are you using (go env)?
my development environment is  the windows7 enterprise.

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

actually, i had ever built projects written using golang. But i cannot use it after my pc joined into an domain of my company. I tried 'go version' 'go help'... etc, but it cannot returns any words.
I am sure that GOROOT and GOPATH is set.
Therefore, I can only launch the remote Linux(CentOS 7.1) machine and upload my project code to build my project.

$ ls
go.exe*  godoc.exe*  gofmt.exe*
milky@PC-Milky MINGW64 /c/Go/bin
$ go version
milky@PC-Milky MINGW64 /c/Go/bin
$ go build
=====================================
that all my console returns
I just hide the username and hostname of windows

Please try and use Windows command prompt (cmd.exe) to run 'go' command. Do not use mingw shell.
Thank you.
Alex

C:\Go\bin>dir
C:\Go\bin 的目录
2016-07-13  10:20              .
2016-07-13  10:20              ..
2016-04-19  23:51        12,507,648 go.exe
2016-04-19  23:52        18,429,440 godoc.exe
2016-04-19  23:51         4,080,640 gofmt.exe
C:\Go\bin>go
C:\Go\bin>go version
C:\Go\bin>go help
C:\Go\bin>
C:\Go\bin>

the output is those above
and i tried to install go1.6.3 again, but it doesn't work

@horizon3d could you please try this?
C:\Go\bin>go version > version.log
C:\Go\bin>type version.log

C:\Go\bin>
C:\Go\bin>go version > ver.log
C:\Go\bin>type ver.log
C:\Go\bin>
that is the result above. and the output file is empty although the ver.log is producted

I am puzzled. Maybe you could use gdb to run go.exe. See where it takes you.
Alex

Any chance your antivirus software thinks go.exe is malware? I suppose I could see that turning into a silent execution failure.

i run the go.exe using the windows7 compatibility mode, and now it works well.

@horizon3d you are already in windows7 as you said.

my development environment is the windows7 enterprise.

yes, that why i am puzzled. the os on my pc is windows7 enterprise X64. it worked well before i joined the domain. and then didn't work after i joined the domain util i run those executable files(.exe) under the windows7 compatibility mode.
that case would be test, i think

What does go version and go env output (now that you can see their output) ?
Thank you.
Alex

C:\Windows\system32>go version
go version go1.6.3 windows/amd64
C:\Windows\system32>go env
set GOARCH=amd64
set GOBIN=C:\Go\bin
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=D:\Coding
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GO15VENDOREXPERIMENT=1
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1
the output of go version and go env
that's the output expected

the output of go version and go env

Thank you. All looks OK to me.
I really don't know what your problem is.
Alex

@horizon3d what is different between https://github.com/golang/go/issues/16614#issuecomment-237767684 and https://github.com/golang/go/issues/16614#issuecomment-238453420 ?

one is returns nothing, and the other one return the expected output.
i don't understand what is your mean by different between the two output.
the only difference is that i set the executable file(s) to run under the comatibility mode, then it can output the result i expected.
@mattn


add connect-repl

How connect.kak can leverage the use of a REPL?

When working on a script that uses connect.kak it is useful to be able to test parts of it by sending it to a connected REPL. It would be just convenient if I can simply use the repl-send-text command for that. Furthermore when having a connected terminal one can edit a long command in a scratch buffer and send it to the terminal.

But when you use the REPL, you use the connect.kak’s features?

Yes. For example when writing a plugin that uses connect.kak commands I can run connect-repl bash in Kakoune and interactively test parts of it by sending a selection to the connected REPL.

I hesitate to merge it.  We have the > (connect-terminal) and $ (connect-shell) commands.  I would like to unify the implementations and make a connected terminal or program a REPL at any time.

@listentolist Can you test the latest commit 5ad59e5201d21ac1221be7c373bd25623efe8337 and implementation?

Thank you! connect repl-new works as expected.

Very happy with that.
We can now use the explicit:
connect tmux-terminal-horizontal

and all other flavors.

Yes. For example when writing a plugin that uses connect.kak commands I can run connect-repl bash in Kakoune and interactively test parts of it by sending a selection to the connected REPL.

I was also using it a lot. As a repl for bash scripts for example. :fifo is just great. Also as a connected repl for compilation: $(:get %opt{makecmd}) `:it`

I’ve added the @ (think kitty’s REPL) command, alongside the + popup command.
Example – Open a connected Janet REPL:
@ janet

Example – Open a popup with fzf to open files:
+ :fzf

I’ve added the @ (think kitty’s REPL) command, alongside the + popup command.
Example – Open a connected Janet REPL:
@ janet

Example – Open files with fzf in a popup window:
+ :fzf

Default popup handling is very nice. I had this in my kakrc:
    define-command fzf-files -override -params .. -file-completion -docstring 'Open files with fzf' %{
        terminal-set global kitty kitty-terminal-tab kitty-focus
        > :fzf-files %arg{@}
        terminal-set global x11 x11-terminal x11-focus
    }
    define-command fzf-buffers -override -params ..1 -buffer-completion -docstring 'Open buffers with fzf' %{
        terminal-set global kitty kitty-terminal-tab kitty-focus
        > :fzf-buffers %arg{@}
        terminal-set global x11 x11-terminal x11-focus
    }

Connected repls are also useful.


Update JDK 11 Dockerfile Checksums

Updating checksums for version 11.0.20 JDK downloads as per oracle website.
https://www.oracle.com/a/tech/docs/11-0-20-checksum.html

Thanks for the pull request @harrisd19.  It was correct, but was superseded by https://github.com/oracle/docker-images/pull/2671/ which include this and other updates before this one was reviewed.


getting game mode from map status

determine what mode compare is going to be in by looking at mapinfo.status in compare.js

Original Message by @yoru#9267

typo* should be mapinfo.mode


Secret provided can not contain more than 120 characters. Please provide a valid credential.

[Enter feedback here]
The secret in your example is 142, and when I generate a shared secret it is 149.
Synapse current version installed about 3 weeks back

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

ID: f873d0f6-d930-d354-5da1-8fa698f77366
Version Independent ID: 3feff33e-55c1-90a3-ba2f-c27a71a1316e
Content: Create and use external tables in serverless SQL pool - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql/create-use-external-tables.md
Service: synapse-analytics
Sub-service: sql
GitHub Login: @vvasic-msft
Microsoft Alias: vvasic

@JerryMWeeks  Since this issue is not related directly to the doc, I would recommend you to create a thread on the forums - Microsoft Q&A or Stack Overflow
Once you post your issue on forums, it will have visibility across the community which is a better suited audience for such types of issues.

@JerryMWeeks what type of SQL pool were you using? This article is for serverless SQL pool only. Dedicated SQL pool doesn't support SHARED ACCESS SIGNATURE yet.

#assign:julie MSFT

julie is not a valid GitHub ID, or is not a collaborator on this repo.

@JerryMWeeks Hi, I will go ahead with closing this issue since we have not heard back from you. The @notification mechanism continues to work even after an item is closed, in case further communication is needed. We hope to hear from you again.

#please-close


Add ability to set default world gen

Basically to give modpack creators the ability take the choice away from players regarding world gen (especially the seed) like Default World Generator does, which unfortunately doesn't seem to work with OTG.
It appears that OTG supersedes DWG maybe that could be corrected? With OTG installed DWG's options simply don't appear but OTG's do and it should be the opposite.
I guess the biggest issue is players spawning in deep water so often, especially since I'm removing their ability to swim. I'm sure that's something you all are working on but I expect it's not going to be fixed soon.

Hey, v5 has the option to set a world seed and spawn point in the worldconfig, cheers :).


Fix Select with Filter Highlight

Codecov Report

Merging #1051 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1051      +/-   ##
==========================================
- Coverage   99.82%   99.82%   -0.01%
==========================================
  Files         316      316
  Lines        7802     7801       -1
  Branches     1892     1891       -1
==========================================
- Hits         7788     7787       -1
  Misses         14       14

Impacted Files
Coverage Δ

src/components/listBox/ListBox.tsx
100% <ø> (ø)
:arrow_up:

src/components/select/SelectWithFilter.tsx
100% <100%> (ø)
:arrow_up:

.../components/autocomplete/AutocompleteConnected.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

All SingleSelects are broken in the demo

Fixed by #1058


skytemple: add webkitgtk to buildInputs

Motivation for this change
WebKit is used for rendering interactive statistics graph, see https://github.com/SkyTemple/skytemple/blob/6413bd9028b658afe20d31a248cf9ea4cb8562e3/skytemple/module/monster/controller/level_up.py#L455. This feature is rather minor, and optional, but may as well put into dependencies if that dependency is actually used. This increases closure size quite a bit (140MB or so), however the impact is pretty much none when using GNOME as webkitgtk is used by GNOME, and the package is admittedly quite niche, so 99.99% of NixOS/Nixpkgs users wouldn't be affected.
I tested that this change actually works and enables interactivity in the graph.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126653 at c78150c6 run on x86_64-linux 1

1 package failed to build:

skytemple

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


bugfix: update clusterrole and certificate for yurt-tunnel-server and fix kubeconfig path error for yurtctl

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

/kind bug
What this PR does / why we need it:

update clusterrole of yurt-tunnel-server in order to support dns mode
add x-tunnel-server-internal-svc info into yurt-tunnel-server certificate
fix kubeconfig path error for yurtctl covert edgenode cmd

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/assign @Fei-Guo
/assign @Peeknut
Does this PR introduce a user-facing change?

other Note

LGTM

/approve
/lgtm


Drop __get__ and __set__ from unnecessary-dunder-call

These are for descriptors which affects the behavior of the object as a property; I do not think they should be called directly but there is no alternative when working with the object directly.
Closes https://github.com/astral-sh/ruff/issues/9789

Unclear if we should drop the violation entirely. I think we should though — will require a bit more work.

Yeah I say we drop it.

The suggestion for __delete__ is similarly weird — calling .__delete__() on a descriptor instance is very different to using the del statement

Ah good catch yeah that one is for descriptors too...

Can't approve but lgtm, thanks!


create a filter in shiny (via a slider) to select the minimum number of rides that a park has (or had when closed)

not worth it right now


Implement GenericDataType with void* [14323]

Implement a Data Type that could be used with any kind of information, given it as a void* and a size.

Codecov Report

Merging #29 (165cd61) into feature/amlip_cpp/main (2331ab2) will increase coverage by 7.67%.
The diff coverage is 54.43%.

@@                    Coverage Diff                     @@
##           feature/amlip_cpp/main      #29      +/-   ##
==========================================================
+ Coverage                   33.48%   41.15%   +7.67%
==========================================================
  Files                           5        6       +1
  Lines                         227      294      +67
  Branches                       42       46       +4
==========================================================
+ Hits                           76      121      +45
- Misses                        133      153      +20
- Partials                       18       20       +2

Impacted Files
Coverage Δ

amlip_cpp/src/cpp/types/InterfaceDataType.cpp
0.00% <0.00%> (ø)

...p/src/cpp/types/impl/AmlipGenericTopicDataType.ipp
31.88% <50.00%> (+7.24%)
:arrow_up:

amlip_cpp/src/cpp/types/GenericDataType.cpp
55.88% <55.88%> (ø)

amlip_cpp/src/cpp/types/AmlipIdDataType.cpp
41.96% <80.00%> (+2.14%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support for delete by value

Add support for delete by value. This changeset borrows the changes from https://github.com/evanphx/json-patch/pull/103/files and also adds support for deletion by partial value in cases when the value in the array is an object.

I suggest merging to a branch other than master. It's easier to keep master exactly the same as upstream: when there is a change upstream, you can simply do a git pull without any risk of having merge conflicts.

I suggest merging to a branch other than master. It's easier to keep master exactly the same as upstream: when there is a change upstream, you can simply do a git pull without any risk of having merge conflicts.

Also, If the changes look ok, is this something that we'd want to roll this under CiscoM31 ?

Can you fork under CiscoM31?

Can you fork under CiscoM31?


HVAC mode not working

Hi Guys,
I am having an issue with controlling the mode of my thermostat from homekit.
If i change the temp from homekit indigo changes the temp but when i try to turn on or off indigo does not change anything.
After looking on the wiki i started playing with a few things, on line 1566 of index.js i changed qs = { hvacOperationModeIsOff: "true" }; to qs = { hvacCurrentMode: "all off"}; and i can now turn the hvac system off from home kit.
I also changed the next few lines to "cool on", "heat on" and "auto on" and those modes now work, is this ok to change these lines? should it be working without any mods at the moment?
I am pretty new to all this and just want to make sure i don't break anything.

I had the same issue!

Clearly, I didn't test this well enough.  I will fix it shortly.

I have published a new version to npm that includes this fix.  Thanks for reporting this!


Added history for reddit

Description
Implemented history store with reddit to match 4chan's functionality.
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style of this project
[x] I have performed a self-review of my own code
[x] My code is self documenting
[x] I have made corresponding changes to the documentation
[x] My changes generate no new major crashes/bugs
[x] Any new features/functionality have corresponding test cases added

Looks good Sam, thank you for bringing this up to date with the fchan features!

Thank you for your check and approval! I appreciate your continued support of the project 🚀


feat: Supply PR comment from file

Also updated docs

/retest

The previous comment was a test of the functionality of this PR and made using the command
./build/jx-gitops pr comment -s delete-and-create -f README.md --source-url https://github.com/jenkins-x-plugins/jx-gitops.git --pr 969


How do you compile?

I havn't done any android stuff since like 4x, so this is all new to me.
I tried to compile using the mingc/android-build-box docker image and running ./gradlew build
docker_run_pwd mingc/android-build-box ./gradlew build
Downloading https://services.gradle.org/distributions/gradle-6.5-rc-1-all.zip
...........................................................................................................................................
Unzipping /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/wrapper/dists/gradle-6.5-rc-1-all/3wuei038oeawusap0fm9vh98s/gradle-6.5-rc-1-all.zip to /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/wrapper/dists/gradle-6.5-rc-1-all/3wuei038oeawusap0fm9vh98s
Set executable permissions for: /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/wrapper/dists/gradle-6.5-rc-1-all/3wuei038oeawusap0fm9vh98s/gradle-6.5-rc-1/bin/gradle

Welcome to Gradle 6.5-rc-1!

Here are the highlights of this release:
 - Experimental file-system watching
 - Improved version ordering
 - New samples

For more details see https://docs.gradle.org/6.5-rc-1/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)

> Configure project :app
Unable to initialize metrics, ensure /opt/android-sdk is writable, details: /opt/android-sdk/analytics.settings (Permission denied)
AnalyticsSettings call failed
java.io.FileNotFoundException: /opt/android-sdk/analytics.settings (Permission denied)
[snip stacktrace]
> Task :app:preBuild UP-TO-DATE
> Task :app:preDebugBuild UP-TO-DATE
> Task :app:compileDebugAidl NO-SOURCE
> Task :app:generateDebugBuildConfig
> Task :app:compileDebugRenderscript NO-SOURCE
> Task :app:prepareDebugKotlinCompileTask
> Task :app:checkDebugAarMetadata
> Task :app:generateDebugResValues
> Task :app:generateDebugResources
> Task :app:createDebugCompatibleScreenManifests
> Task :app:extractDeepLinksDebug
> Task :app:mergeDebugResources
> Task :app:processDebugMainManifest
> Task :app:processDebugManifest
> Task :app:mergeDebugNativeDebugMetadata NO-SOURCE
> Task :app:javaPreCompileDebug
> Task :app:processDebugManifestForPackage
> Task :app:processDebugResources

> Task :app:kaptGenerateStubsDebugKotlin
w: ATTENTION!
This build uses unsafe internal compiler arguments:

-XXLanguage:+NonParenthesizedAnnotationsOnFunctionalTypes

This mode is not recommended for production use,
as no stability/compatibility guarantees are given on
compiler or generated code. Use it at your own risk!

w: Runtime JAR files in the classpath have the version 1.3, which is older than the API version 1.4. Consider using the runtime of version 1.4, or pass '-api-version 1.3' explicitly to restrict the available APIs to the runtime of version 1.3. You can also pass '-language-version 1.3' instead, which will restrict not only the APIs to the specified version, but also the language features
w: /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/caches/transforms-2/files-2.1/0f3d1718673e7d3a26e5a36876c5bf7d/jetified-kotlin-stdlib-common-1.3.72.jar: Runtime JAR file has version 1.3 which is older than required for API version 1.4
w: /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/caches/transforms-2/files-2.1/799a98d39dd19e9f191c3f4cf5148ce7/jetified-kotlin-stdlib-1.3.72.jar: Runtime JAR file has version 1.3 which is older than required for API version 1.4

> Task :app:mergeDebugShaders
> Task :app:compileDebugShaders NO-SOURCE
> Task :app:generateDebugAssets UP-TO-DATE
> Task :app:mergeDebugAssets
> Task :app:compressDebugAssets
> Task :app:processDebugJavaRes NO-SOURCE
> Task :app:kaptDebugKotlin

> Task :app:compileDebugKotlin FAILED
w: ATTENTION!
This build uses unsafe internal compiler arguments:

-XXLanguage:+NonParenthesizedAnnotationsOnFunctionalTypes

This mode is not recommended for production use,
as no stability/compatibility guarantees are given on
compiler or generated code. Use it at your own risk!

w: Runtime JAR files in the classpath have the version 1.3, which is older than the API version 1.4. Consider using the runtime of version 1.4, or pass '-api-version 1.3' explicitly to restrict the available APIs to the runtime of version 1.3. You can also pass '-language-version 1.3' instead, which will restrict not only the APIs to the specified version, but also the language features
w: /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/caches/transforms-2/files-2.1/0f3d1718673e7d3a26e5a36876c5bf7d/jetified-kotlin-stdlib-common-1.3.72.jar: Runtime JAR file has version 1.3 which is older than required for API version 1.4
w: /home/halkeye/git/halkeye/hass-android-controls/?/.gradle/caches/transforms-2/files-2.1/799a98d39dd19e9f191c3f4cf5148ce7/jetified-kotlin-stdlib-1.3.72.jar: Runtime JAR file has version 1.3 which is older than required for API version 1.4
e: /home/halkeye/git/halkeye/hass-android-controls/app/src/main/java/com/example/hasscontrolsprovider/ui/MainActivity.kt: (18, 20): Unresolved reference: livedata
e: /home/halkeye/git/halkeye/hass-android-controls/app/src/main/java/com/example/hasscontrolsprovider/ui/MainActivity.kt: (56, 36): Unresolved reference: observeAsState

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileDebugKotlin'.
> Compilation error. See log for more details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1m 27s
18 actionable tasks: 18 executed

So, based on the 1.3 error, I tried upgrading to 1.4
diff --git a/build.gradle b/build.gradle
index f9a0b71..dbf910a 100644
--- a/build.gradle
+++ b/build.gradle
@@ -1,6 +1,6 @@
 // Top-level build file where you can add configuration options common to all sub-projects/modules.
 buildscript {
-    ext.kotlin_version = "1.3.72"
+    ext.kotlin_version = "1.4.10"
     repositories {
         google()
         jcenter()
@@ -23,4 +23,4 @@ allprojects {

 task clean(type: Delete) {
     delete rootProject.buildDir
-}
\ No newline at end of file
+}

--- a/gradle/wrapper/gradle-wrapper.properties
+++ b/gradle/wrapper/gradle-wrapper.properties
@@ -1,6 +1,5 @@
-#Sun Jul 05 11:07:28 CEST 2020
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
+distributionUrl=https\://services.gradle.org/distributions/gradle-6.6.1-all.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-6.5-rc-1-all.zip

Which seemed to mostly work, then errored out with a system stacktrace:
e: java.lang.NoSuchMethodError: org.jetbrains.kotlin.cli.common.messages.MessageCollector$DefaultImpls.report$default(Lorg/jetbrains/kotlin/cli/common/messages/MessageCollector;Lorg/jetbrains/kotlin/cli/common/messages/CompilerMessageSeverity;Ljava/lang/String;Lorg/jetbrains/kotlin/cli/common/messages/CompilerMessageSourceLocation;ILjava/lang/Object;)V

I started adding some experimental Compose UI elements to the project. Compose is still a very unstable alpha library, it's using a a special Kotlin compiler version and the project can only built with the latest Canary release of Android Studio. You can either remove the Compose-related code and Gradle config, or try opening the project with Android Studio Canary.


NonFinalizedState: Finish implementation and testing according to RFC0005

Motivation
This PR is tracking the follow up changes necessary to fully implement the
non-finalized state according to the state RFC and to test the implementation.

[ ] Correctly handle duplicate blocks in CommitBlock
[ ] test the best chain is selected correctly
[ ] test the best chain height is correctly returned
[ ] check that the right queued blocks are pruned

Related Issues

Prior PR - https://github.com/ZcashFoundation/zebra/pull/1101
Relevant comments - https://github.com/ZcashFoundation/zebra/pull/1101#discussion_r500716408 and https://github.com/ZcashFoundation/zebra/pull/1101#discussion_r501365663
PR updating RFC - https://github.com/ZcashFoundation/zebra/pull/1136

Now that #1101 has merged, can we rebase this PR (or open another PR), so it's easy to see the new commits?

I think the TODO list in this PR should actually become a tracking issue.

Or maybe a section in the RFC5 tracking issue - whatever works best for the developers and reviewers.

This is good work, and it's an improvement, so I'm going to merge it now.
I moved the remaining TODOs into the state tracking issue #1049.


README.md demo results in GrpcConnectionError(MaxDiscoveryAttemptReached(3))

Hello,
Using Docker Desktop for Windows, I've set up a docker based cluster of 3 secured nodes based on the docker-compose.yml and vars.env files in the official docs. I then set up a fresh rust project with:
cargo new --bin esdbtest
cd esdbtest
cargo add tokio --features full
cargo add serde serde-json eventstore

I pasted the contents of the readme code snippet into the main.rs file and made one small edit. I changed the connection string from
esdb://admin:changeit@localhost:2113 to esdb://admin:changeit@localhost:2113?tls=true&tlsVerifyCert=false as the docs state because the docs say the secure connection is required and the compose file sets up a self-signed CA, so I need to ignore the certificate trustworthiness.
I am still getting the following output on running the example: Error: GrpcConnectionError(MaxDiscoveryAttemptReached(3)).
How can I get this running properly so I can experiment with the database and learn about it?

Using the single-node insecure docker-compose.yml file along with the ?tls=false parameter in the connection string, it worked perfectly. What is the fix for the secure cluster version?


Adapt regex from poetry project

This PR solves #360  and  #384
We adept the Regex in use in the poetry project  https://github.com/python-poetry/poetry-core/blob/main/src/poetry/core/packages/package.py in order to be just as permissive as the Poetry project, for which we are validating.

This PR is our first contribution. We are very open to suggestions and improvements.
This PR was a group effort by Ordina Pythoneers, they would like to thank Ordina for the time to do this.
@SanderBeekhuis
@RoelAdriaans
@Coen-Smulders

I think it has to be fixed in SchemaStore since that's the primary source for schemas used
https://github.com/SchemaStore/schemastore/blob/94ae74640898dcdb162a68a963b6fb6a3a29ef9e/src/schemas/json/pyproject.json#L39

@panekj (or somebody else).
It has been fixed upstream in https://github.com/SchemaStore/schemastore/pull/3209  what will  be the process of landing this fix in this extension?

I don't think there's anything to do. The catalog and (if I'm not mistaken) the schemas are downloaded at runtime. So as soon as they publish the new version (which I think is quite fast and seems to already be the case now because the issues are fixed for me), then you should see the fix (possibly restarting your editor). I don't know if schemas are cached though. Please ping here if you it's not fixed.

@ia0 You are right. It also works on my machine.


Date parsing fails when "time/large-dates" feature is enabled

When the "time/large-dates" feature is enabled, the format_description strings used to parse ASN.1 GeneralizedTime and ASN1.UTCTime becomes ambiguous. The parser will greedily consume the month digits for the leading year.
Arguably a deficiency in the upstream time crate, not seemingly allowing anyway to specify how many digits are expected in a year.

Upstream issue: https://github.com/time-rs/time/issues/683


47 - Fix inaccurate money limit

Codecov Report

Merging #54 (a9688aa) into master (f3a393b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #54   +/-   ##
=========================================
  Coverage     71.11%   71.11%
  Complexity      423      423
=========================================
  Files            72       72
  Lines          1364     1364
  Branches        150      150
=========================================
  Hits            970      970
  Misses          348      348
  Partials         46       46

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/model/person/Money.java
77.77% <ø> (ø)

Continue to review full report at Codecov.

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


Bug: Example of Session Auth Backend

Either it's a bug or the documentation is just not correct resp. not clear to me.
Describe the bug
Trying to test and implement a session-based authentication as described in Session Auth Backend (copied exactly into an empty main.py) yields to an internal server error ('user' is not defined in scope, install an AuthMiddleware to set it) if appending the root path specifier at the end of the exclude of the SessionAuth constructor as in exclude=["/login", "/signup", "/schema", "/"],. Without the trailing "/" it works quite fine, i.e. the access to request.user is possible.
To Reproduce
Using the generated Swagger UI:

Signup a test user using arbitrary test data (route /signup)
Login with the given credentials of the test data (route /login)
Accessing the logged-in user (route /user) yields the quoted error

I realize that I should rather use include instead of exclude when serving html files to an api. Sorry for the noise.


Sample repo or a a guide for getting this working in aws?

Do you have a little setup guide for this or an example repo? I swapped this in, but haven't been able to get it working yet - probably all the aws setup i'm not doing right...

Thank you! Yes, I'll prepare a guide and a little example in the next week.
What does not working as expected from your perspective?

I was wondering how you are getting it deployed - seems to be different ways to do that with aws - fargate, serverless, s3, etc. I was hoping to run a build or deploy command and it would magically publish it to AWS :)

Also, i get this error:

Using adapter-awscdk
✘ [ERROR] Could not resolve "@sveltejs/kit/install-fetch"

build/server/shims.js:1:50:
  1 │ export { fetch, Response, Request, Headers } from '@sveltejs/kit/install-fetch';
    ╵                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The path "./install-fetch" is not exported by package "@sveltejs/kit":


Prepare Economic data for analysis

Wrangle the economic data  for hromadas.

I'm closing this issue, because this data source does not seem relevant any more (we'll be relying on Open Budget data)