Feature Request: Add the "stream" parameter which can be found inside normal requests.

First of all thank you for fixing #22 I think it is not available in pypi yet. I even tried to install it manually by doing pip install git+https://github.com/yifeikong/curl_cffi but it seems to give the below error on a linux (ubuntu) platform:
$ pip3 install git+https://github.com/yifeikong/curl_cffi
Collecting git+https://github.com/yifeikong/curl_cffi
  Cloning https://github.com/yifeikong/curl_cffi to /tmp/pip-req-build-gisjnq_5
  Running command git clone -q https://github.com/yifeikong/curl_cffi /tmp/pip-req-build-gisjnq_5
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Requirement already satisfied: cffi>=1.0.0 in ./.local/lib/python3.8/site-packages (from curl-cffi==0.3.8) (1.15.1)
Requirement already satisfied: pycparser in ./.local/lib/python3.8/site-packages (from cffi>=1.0.0->curl-cffi==0.3.8) (2.21)
Building wheels for collected packages: curl-cffi
  Building wheel for curl-cffi (PEP 517) ... error
  ERROR: Command errored out with exit status 1:
   command: /usr/bin/python3 /tmp/tmpmigkk7kk build_wheel /tmp/tmp_eqeh8pr
       cwd: /tmp/pip-req-build-gisjnq_5
  Complete output (110 lines):
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build/lib.linux-x86_64-cpython-38
  creating build/lib.linux-x86_64-cpython-38/curl_cffi
  copying curl_cffi/build.py -> build/lib.linux-x86_64-cpython-38/curl_cffi
  copying curl_cffi/__init__.py -> build/lib.linux-x86_64-cpython-38/curl_cffi
  copying curl_cffi/_const.py -> build/lib.linux-x86_64-cpython-38/curl_cffi
  creating build/lib.linux-x86_64-cpython-38/curl_cffi/requests
  copying curl_cffi/requests/headers.py -> build/lib.linux-x86_64-cpython-38/curl_cffi/requests
  copying curl_cffi/requests/cookies.py -> build/lib.linux-x86_64-cpython-38/curl_cffi/requests
  copying curl_cffi/requests/session.py -> build/lib.linux-x86_64-cpython-38/curl_cffi/requests
  copying curl_cffi/requests/__init__.py -> build/lib.linux-x86_64-cpython-38/curl_cffi/requests
  copying curl_cffi/requests/errors.py -> build/lib.linux-x86_64-cpython-38/curl_cffi/requests
  running egg_info
  writing curl_cffi.egg-info/PKG-INFO
  writing dependency_links to curl_cffi.egg-info/dependency_links.txt
  writing requirements to curl_cffi.egg-info/requires.txt
  writing top-level names to curl_cffi.egg-info/top_level.txt
  reading manifest file 'curl_cffi.egg-info/SOURCES.txt'
  reading manifest template 'MANIFEST.in'
  /tmp/pip-build-env-bx5qgl50/overlay/lib/python3.8/site-packages/setuptools/config/pyprojecttoml.py:108: _BetaConfiguration: Support for `[tool.setuptools]` in `pyproject.toml` is still *beta*.
    warnings.warn(msg, _BetaConfiguration)
  warning: no files found matching 'curl_cffi/cacert.pem'
  warning: no files found matching 'curl_cffi/*.dll'
  adding license file 'LICENSE'
  writing manifest file 'curl_cffi.egg-info/SOURCES.txt'
  copying curl_cffi/cdef.c -> build/lib.linux-x86_64-cpython-38/curl_cffi
  copying curl_cffi/shim.c -> build/lib.linux-x86_64-cpython-38/curl_cffi
  creating build/lib.linux-x86_64-cpython-38/curl_cffi/include
  copying curl_cffi/include/shim.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include
  creating build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/Makefile.am -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/curl.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/curlver.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/easy.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/header.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/mprintf.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/multi.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/options.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/stdcheaders.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/system.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/typecheck-gcc.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  copying curl_cffi/include/curl/urlapi.h -> build/lib.linux-x86_64-cpython-38/curl_cffi/include/curl
  running build_ext
  generating cffi module 'build/temp.linux-x86_64-cpython-38/curl_cffi._wrapper.c'
  creating build/temp.linux-x86_64-cpython-38
  building 'curl_cffi._wrapper' extension
  creating build/temp.linux-x86_64-cpython-38/build
  creating build/temp.linux-x86_64-cpython-38/build/temp.linux-x86_64-cpython-38
  creating build/temp.linux-x86_64-cpython-38/curl_cffi
  x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -Icurl_cffi/include -I/usr/include/python3.8 -c build/temp.linux-x86_64-cpython-38/curl_cffi._wrapper.c -o build/temp.linux-x86_64-cpython-38/build/temp.linux-x86_64-cpython-38/curl_cffi._wrapper.o
  x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -Icurl_cffi/include -I/usr/include/python3.8 -c curl_cffi/shim.c -o build/temp.linux-x86_64-cpython-38/curl_cffi/shim.o
  x86_64-linux-gnu-gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-Bsymbolic-functions -Wl,-z,relro -g -fwrapv -O2 build/temp.linux-x86_64-cpython-38/build/temp.linux-x86_64-cpython-38/curl_cffi._wrapper.o build/temp.linux-x86_64-cpython-38/curl_cffi/shim.o -L/usr/local/lib -L/usr/lib -lcurl-impersonate-chrome -o build/lib.linux-x86_64-cpython-38/curl_cffi/_wrapper.abi3.so
  /usr/bin/ld: cannot find -lcurl-impersonate-chrome
  collect2: error: ld returned 1 exit status
  /tmp/pip-build-env-bx5qgl50/overlay/lib/python3.8/site-packages/setuptools/command/build_py.py:202: SetuptoolsDeprecationWarning:     Installing 'curl_cffi.include' as data is deprecated, please list it in `packages`.
      !!

      ############################
      # Package would be ignored #
      ############################
      Python recognizes 'curl_cffi.include' as an importable package,
      but it is not listed in the `packages` configuration of setuptools.

      'curl_cffi.include' has been automatically added to the distribution only
      because it may contain data files, but this behavior is likely to change
      in future versions of setuptools (and therefore is considered deprecated).

      Please make sure that 'curl_cffi.include' is included as a package by using
      the `packages` configuration field or the proper discovery methods
      (for example by using `find_namespace_packages(...)`/`find_namespace:`
      instead of `find_packages(...)`/`find:`).

      You can read more about "package discovery" and "data files" on setuptools
      documentation page.

  !!

    check.warn(importable)
  /tmp/pip-build-env-bx5qgl50/overlay/lib/python3.8/site-packages/setuptools/command/build_py.py:202: SetuptoolsDeprecationWarning:     Installing 'curl_cffi.include.curl' as data is deprecated, please list it in `packages`.
      !!

      ############################
      # Package would be ignored #
      ############################
      Python recognizes 'curl_cffi.include.curl' as an importable package,
      but it is not listed in the `packages` configuration of setuptools.

      'curl_cffi.include.curl' has been automatically added to the distribution only
      because it may contain data files, but this behavior is likely to change
      in future versions of setuptools (and therefore is considered deprecated).

      Please make sure that 'curl_cffi.include.curl' is included as a package by using
      the `packages` configuration field or the proper discovery methods
      (for example by using `find_namespace_packages(...)`/`find_namespace:`
      instead of `find_packages(...)`/`find:`).

      You can read more about "package discovery" and "data files" on setuptools
      documentation page.

  !!

    check.warn(importable)
  error: command '/usr/bin/x86_64-linux-gnu-gcc' failed with exit code 1
  ----------------------------------------
  ERROR: Failed building wheel for curl-cffi
Failed to build curl-cffi
ERROR: Could not build wheels for curl-cffi which use PEP 517 and cannot be installed directly

However I hope it will be fixed soon when the next version is pushed to pypi!
This issue is about the stream=True/False parameter which can be found inside normal requests package. If it is possible can you add it to this please so I can perform operations like below:
import requests

url="https://videos-fastly-studiodrm.jwpsrv.com/6411bcd6_0x344cee0abd6d9a757e2ea355371aba081c4aa3d3/site/Y21kWwti/media/UFCQHrhu/version/dozELTrB/manifest-34126610,34126611,34126612,34126615,34126616,34126617.ism/dash/manifest-34126610,34126611,34126612,34126615,34126616,34126617-audio_eng=112019.dash"
headers = {
    'authority': 'videos-fastly-studiodrm.jwpsrv.com',
    'accept': '*/*',
    'accept-language': 'en-GB,en-US;q=0.9,en;q=0.8',
    'cache-control': 'no-cache',
    'origin': 'https://juanflix.com.ph',
    'pragma': 'no-cache',
    'referer': 'https://juanflix.com.ph/',
    'sec-ch-ua': '"Google Chrome";v="111", "Not(A:Brand";v="8", "Chromium";v="111"',
    'sec-ch-ua-mobile': '?0',
    'sec-ch-ua-platform': '"Windows"',
    'sec-fetch-dest': 'empty',
    'sec-fetch-mode': 'cors',
    'sec-fetch-site': 'cross-site',
    'user-agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36',
}
proxies={}
raw = requests.get(url=url, headers=headers, proxies=proxies, stream=True, verify=False)
if raw.status_code == 200 or raw.status_code == 206:
	for chunk in raw.iter_content(chunk_size=4096):
		if chunk:
			print("chunk found")
			# Do something

Thanks again and I am really sorry for disturbing you again and again.

First of all, please refer to the original issue for followup questions, it's more clear to discuss one topic in the same issue. I'll still be notified even if it's closed :P
Second, pypi package has been updated. Reinstall with --upgrade.
Last but not least. stream is a cool and must-have feature. I'll need some time to investigate requests's interfaces and libcurl's internal mechanism. Stay tuned.

Thank you so much!

stream=True/False

您好，我也希望您能尽快开发stream这个参数

最近又研究了下 libcurl 的 API，发现在同步代码里加上 stream 这个参数不太现实，因为 libcurl 只支持回调，没有遍历的 API。不过，有两个 workaround：

可以使用 content_callback，传一个回调函数进去；
asyncio，理论上来说是可以支持 stream 的，不过我还没时间弄。

@yifeikong Since asyncio is supported in the library now, will this be possible to add through asyncio?

Yes, it's possible to support stream via asyncio.Queue. It should be added in the next minor version.

stream这个功能很重要，希望作者早日整合到源码中。

Still working on this, in the meanwhile, you can use the content_callback parameter for callback style streaming.
from curl_cffi import requests

def callback(data: bytes):
    print(data)

requests.get(url, content_callback=callback)

Added stream=True support in v0.5.10b1, tests yet to be added, but it seems to be working.
with requests.Session() as s:
    print("\n======================================================================")
    print("Iterating over chunks")
    print("=====================================================================\n")
    r = s.get(URL, stream=True)
    for chunk in r.iter_content():
        print("CHUNK", chunk)
    r.close()

async def async_examples():
    async with requests.AsyncSession() as s:
        print("\n====================================================================")
        print("Using asyncio")
        print("====================================================================\n")
        r = await s.get(URL, stream=True)
        async for chunk in r.aiter_content():
            print("CHUNK", chunk)
        await r.aclose()

More examples can be found here.

@yifeikong
Response status is not loaded:
curl_cffi.requests.errors.RequestsError: HTTP Error 0: OK
Example:
          response = await session.post(f"{url}/api/chat-process", json=data, headers=headers, stream=True)
          response.raise_for_status()

@hlohaus Please see if the wheels here in the artifacts section are working.

works so far:
        async with AsyncSession(impersonate="chrome110", verify=False) as session:
            async with await session.post(f"{url}/api/chat-process", stream=True) as response:
                response.raise_for_status()
                print("Empty:", response.text)
                async for line in response.aiter_lines():
                    print(line)

But response.text is empty.
Can you implement this interface:
        # new StreamSession with auto stream=True
        async with StreamSession(impersonate="chrome110", verify=False) as session:
             # session.post:
             # - has no await
             # - is not async and returns a "RequestSession" with __aenter__ and __enter__
             # - " __enter__" returns a "RequestSession" without async methods
             # - no stream=True
            async with session.post(f"{url}/api/chat-process") as response:
                response.raise_for_status()
                content_type = response.headers.get('content-type')
                if content_type == "application/octet-stream":
                    async for line in response.iter_lines():
                        print("Line:", line)
                else:
                    # text() and json() method with await:
                    text = await response.text()
                    print("Response:", text)

I found a other bug. If the response has no content:
async def run():
    async with AsyncSession(impersonate="chrome107") as session:
        response = await session.get("https://httpbin.org/status/202", stream=True)
        print(response.status_code)
asyncio.run(run())

We have a endless loop.

works so far:
        async with AsyncSession(impersonate="chrome110", verify=False) as session:
            async with await session.post(f"{url}/api/chat-process", stream=True) as response:
                response.raise_for_status()
                print("Empty:", response.text)
                async for line in response.aiter_lines():
                    print(line)

But response.text is empty.
Can you implement this interface:
        # new StreamSession with auto stream=True
        async with StreamSession(impersonate="chrome110", verify=False) as session:
             # session.post:
             # - has no await
             # - is not async and returns a "RequestSession" with __aenter__ and __enter__
             # - " __enter__" returns a "RequestSession" without async methods
             # - no stream=True
            async with session.post(f"{url}/api/chat-process") as response:
                response.raise_for_status()
                content_type = response.headers.get('content-type')
                if content_type == "application/octet-stream":
                    async for line in response.iter_lines():
                        print("Line:", line)
                else:
                    # text() and json() method with await:
                    text = await response.text()
                    print("Response:", text)

I think what you described is the aiohttp API, apart from the "no stream=True" part, it should be able to be implemented with a simple wrapper.
The underlying async with-less API will still be kept, since

I'm not a huge fan of nested with blocks
As pointed out in the httpx docs, with is not practical under some situation.

And thank you for reporting the bugs.

Here you go:
async with s.stream("GET", URL) as r:
    print(await r.atext())

async with s.stream("GET", URL) as r:
    async for chunk in r.aiter_content():
        print("Status: ", r.status_code)
        assert r.status_code == 200
        print("CHUNK", chunk)

Wheels.

    async with AsyncSession(impersonate="chrome107", headers={"Working": "Yes"}) as session:
        async with session.stream("GET", "https://httpbin.org/stream/20") as response:
            response.raise_for_status()
            async for line in response.aiter_lines():
                print(json.loads(line)["headers"]["Working"])

Yes
Yes
Yes
Yes
Yes
Yes
Yes
Yes
Yes
Yes
...

Available as 0.5.10b2.

Landed in 0.5.10


Add test for find_i

Typical Interviews is not the same as interviews in TypicalPerson
Predicate tested for candidates only currently

Codecov Report

Merging #171 (cf50f62) into master (d8af8b6) will increase coverage by 2.95%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #171      +/-   ##
============================================
+ Coverage     77.33%   80.29%   +2.95%
- Complexity      960     1005      +45
============================================
  Files           119      119
  Lines          2992     2989       -3
  Branches        417      417
============================================
+ Hits           2314     2400      +86
+ Misses          569      469     -100
- Partials        109      120      +11

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...model/interview/FindInterviewCommandPredicate.java
75.00% <ø> (+75.00%)
:arrow_up:

.../address/logic/interview/FindInterviewCommand.java
100.00% <100.00%> (+100.00%)
:arrow_up:

.../java/seedu/address/model/interview/Interview.java
85.84% <0.00%> (+2.83%)
:arrow_up:

...in/java/seedu/address/logic/parser/ParserUtil.java
92.08% <0.00%> (+11.51%)
:arrow_up:

...dress/logic/parser/FindInterviewCommandParser.java
100.00% <0.00%> (+100.00%)
: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 d8af8b6...cf50f62. Read the comment docs.


Remove all references to the old catalogd Package and BundleMetadata APIs

Description
This follows up on #343 to remove the final bits of references to the catalogd Package and BundleMetadata APIs. This is important if we want Operator Controller to be able to bump to a version of Catalogd that removes those APIs.
See https://github.com/operator-framework/catalogd/pull/149

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

Investigating apparent flakes and/or failures that occurred in the merge queue CI...

go-apidiff failure will be covered by #373
the e2e-kind error seems like a flake that was not introduced by this PR.
Re-adding to merge queue.


Expected " but got { instead. At offset: 1048 [JsonError]

JSON crashes on this JSON and I'm not sure why.
/home/user/.nimble/pkgs/jsony-1.1.3/jsony.nim(547) fromJson
/home/user/.nimble/pkgs/jsony-1.1.3/jsony.nim(429) parseHook
/home/user/.nimble/pkgs/jsony-1.1.3/jsony.nim(347) parseObjectInner
/home/user/.nimble/pkgs/jsony-1.1.3/jsony.nim(47) parseHook
Error: unhandled exception: Expected " but got { instead. At offset: 1048 [JsonError]

I'm using Nim 1.6.10 and JSONy 1.1.3 and I'm loading the JSON from a file with readFile and just passing the variable to fromJson.
I only added two simple hooks.
proc parseHook*(s: string, i: var int, v: var DateTime) =
  var str: string
  parseHook(s, i, str)
  v = parse(str, "yyyy-MM-dd'T'hh:mm:ss'.'fffzzz", utc())

proc renameHook*(v: var PreviewCard, fieldName: var string) =
  if fieldName == "type":
    fieldName = "card_type"

it seems like the you expected the date time to be a string, but its an object. Please include the JSON you are trying to parse in a runnable example.

it seems like the you expected the date time to be a string, but its an object. Please include the JSON you are trying to parse in a runnable example.

Sorry for the poor issue. Seems like your guess was correct, I was trying to turn a JSON into a string.
import jsony

type
  Status* = ref object of RootObj
    application*: string

var test_json = """{
  "application": {
    "name": "Web",
    "website": null
  }
}"""

var status_obj = test_json.fromJson(Status)


Enhance error text for Model.summary()

For following python script,
from keras.models import Sequential
from keras.layers import Dense,Conv2D, Flatten

model=Sequential()
model.add(Conv2D(10,(3,3)))
model.add(Conv2D(10,(3,3)))
model.add(Flatten())
model.add(Dense(20))
model.add(Dense(10))

model.compile(loss='categorical_crossentropy',
              optimizer='sgd',
              metrics=['accuracy'])

model.summary()

In the keras version 2.2.2, the error message is like following
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-83-30ace2d20b7d> in <module>()
     13               metrics=['accuracy'])
     14
---> 15 model.summary()

/opt/conda/lib/python3.6/site-packages/keras/engine/network.py in summary(self, line_length, positions, print_fn)
   1245         if not self.built:
   1246             raise ValueError(
-> 1247                 'This model has never been called, thus its weights '
   1248                 'have not yet been created, so no summary can be displayed. '
   1249                 'Build the model first '

ValueError: This model has never been called, thus its weights have not yet been created, so no summary can be displayed. Build the model first (e.g. by calling it on some test data).

But in the keras version 2.1.6, it is like following
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-8-30ace2d20b7d> in <module>()
      3
      4 model=Sequential()
----> 5 model.add(Conv2D(10,(3,3)))
      6 model.add(Conv2D(10,(3,3)))
      7 model.add(Flatten())

/usr/local/lib/python3.6/dist-packages/keras/models.py in add(self, layer)
    482                     # know about its input shape. Otherwise, that's an error.
    483                     if not hasattr(layer, 'batch_input_shape'):
--> 484                         raise ValueError('The first layer in a '
    485                                          'Sequential model must '
    486                                          'get an `input_shape` or '

ValueError: The first layer in a Sequential model must get an `input_shape` or `batch_input_shape` argument.

The difference here is that version 2.1.6 doesn't allow to add the first layer if input_shape is missing and version 2.2.2 allows to build the model even if input shape is missing. I am new to this repository so, I am not sure about it but I think that keras 2.2.2 will figure out input_shape when fit() is called for the first time, and that is being suggested by error msg in version 2.2.2. From looking at the code I can say that it is checking whether or not the model is built or not before calling summary utility.
But error message seems misleading, it asks to fit the model before calling summary() but many times we need to see the number of parameters and output shapes (especially CNN) before actually training. And it is very likely that such error will occur due to not specifying "input_shape" by mistake. And in such scenario debugging the error will be very difficult.
So i am suggesting following error message
ValueError: This model has not yet been built, so no summary can be displayed. Build the model first (e.g. by calling it on some test data) or specify the 'input_shape' in the first layer

Above suggested error message would help in debugging if "input_shape" is not specified due to a mistake and creating a model which computes input shape on the go is not the intention
I will be happy to send PR to fix this issue, but I want to discuss it with the community first.
Issue Guidelines List :

[X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[X] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.
Used keras 2.2.2 and 2.1.6

[X] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps
I am using tensorflow backend

[X] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).
Provided above

Thank you for the detailed report. You can see in the changelog that the structure of Sequential has changed (now a subclass of Model). But it would make sense in a UX perspective to throw the error when adding the first layer without specifying the input shape. We should get the old error message back since I would call this a regression.
PR welcome. If you need help for the PR, you can ask here.

To be clear, we should get back to the old behavior, which is, throwing an error when adding the first layer without specifying the input shape.

@gabrieldemarmiesse  okay got it!! I will work on that and send a PR.

I looked into some code and I found these lines of comments in sequential.py. It seems that input_shape is kept optional intentionally.
Also, the current code does check that whether or not batch_input_shape exists or not but it doesn't do anything if found absent. According to me rather than throwing an exception if batch_input_shape is not found, it should display a warning such as
Warning: input_shape or batch_input_shape is not provided so the model will not be built automatically. To build the model call the model.build(input_shape) or call the model.fit() on some data

@gabrieldemarmiesse What is your view on this?
Notes:

the line in sequential.py which checks for the "input_shape" or "batch_input_shape"
In base_layes.py "input_shape" and "batch_input_shape" are rendered same in this segment of code

This is something that changed recently. Thanks for bringing it to my attention.
I wasn't aware of the delayed building feature. In regard of this, I think we should go with your first idea which was to change the error message to
ValueError: This model has not yet been built, so no summary can be displayed. Build the model first (e.g. by calling it on some test data) or specify the 'input_shape' in the first layer

ok let go ahead with it....i will send a PR soon

env (4, 10, 10)
Traceback (most recent call last):
File "train.py", line 140, in
main()
File "train.py", line 123, in main
model = create_dqn_model(env, num_last_frames=4)
File "train.py", line 107, in create_dqn_model
model.summary()
File "/anaconda3/lib/python3.6/site-packages/keras/engine/network.py", line 1247, in summary
'This model has never been called, thus its weights '
ValueError: This model has never been called, thus its weights have not yet been created, so no summary can be displayed. Build the model first (e.g. by calling it on some test data).
Here is my code...
model = Sequential()
print('env' , (num_last_frames, ) + env.observation_shape)
#Printing the model params
#Output => env (4,) ObservationSHape (10, 10)
Convolutions.()
model.add(TimeDistributed(Conv2D(
16,
kernel_size=(3, 3),
strides=(1, 1),
data_format='channels_first',
input_shape= (num_last_frames, ) +  env.observation_shape
)))
model.add(TimeDistributed(Activation('relu')))
model.add(TimeDistributed(Conv2D(
32,
kernel_size=(3, 3),
strides=(1, 1),
data_format='channels_first'
)))
model.add(TimeDistributed(Activation('relu')))
Dense layers.
model.add(TimeDistributed(Flatten()))
#Timedistributed changes and the below 1 liner code change by me.
model.add(Dropout(0.25))
##Adding Lstm layer to the model MY CHANGE on 24/12/2018.
#regressor.add(LSTM(units = 50, return_sequences = True, input_shape = (X_train.shape[1], 1)))
#regressor.add(Dropout(0.2))
model.add(LSTM(units = 200 , return_sequences = True))
model.add(Dropout(0.25))
model.add(LSTM(units = 200 , return_sequences = True))
model.add(Dropout(0.25))
model.add(LSTM(units = 200))
#My code change END#####
model.add(Dense(256))
model.add(Activation('relu'))
model.add(Dense(env.num_actions))
model.summary()
model.compile(RMSprop(), 'MSE')
Please let me know how can i fix this.
Thanks in adv!!


How to train the model without distributed setting?

Hi @haotianliu001
Thank you for sharing your code.
I don't want to use the distributed setting. How can I train it with only 1 gpu?

You can simply use a command like python configs/m2mrf/fcn_hr48-M2MRF-C_40k_idrid_bdice.py
For more instructions, you can refer to the document here.

Thank you very much.
I think your work is very good.
Is this paper accepted or still under review?

Thank you. It's still under review now. 😺

Hhhh good luck! 😄

Thank you~


Identities.createIdentity returns new Identity with wrong last argument

Considering that the constructor of identity requires the provider as the last argument, shouldn't the last argument here be identityProvider instead of this?

@Ezerous 	Was this resolved natively, or was there something that you understood such that you thought this could be closed?

If I remember correctly I just had misunderstood the code.

Ah, cool. Just following up, making sure you're not lost. Sorry about the delay! :)


Fix invalid charmcraft yaml

We had an invalid charmcraft.yaml file that broke the charmcraft pack command
Note:
This is also a perfectly valid charmcraft.yaml file that we can use to enable building our charms for other architectures, which would be really good. I left it out intentionally to not slow down charmcraft pack,  if you think we need it, I can update the PR.
type: charm
bases:
  - build-on:
    - name: ubuntu
      channel: "22.04"
    run-on:
    - name: "ubuntu"
      channel: "22.04"
      architectures: [amd64, arm64, ppc64el, s390x]
    - name: "ubuntu"
      channel: "23.10"
      architectures: [amd64, arm64, ppc64el, s390x]
    - name: "ubuntu"
      channel: "23.04"
      architectures: [amd64, arm64, ppc64el, s390x]
    - name: "ubuntu"
      channel: "20.04"
      architectures: [amd64, arm64, ppc64el, s390x]

@kian99 I think the issue is that we had more than 1 build-on

I was able to build the charm from main.
livepatch-k8s-operator$ charmcraft pack
Created 'canonical-livepatch-server-k8s_ubuntu-20.04-amd64_ubuntu-22.04-amd64.charm'.
Charms packed:
    canonical-livepatch-server-k8s_ubuntu-20.04-amd64_ubuntu-22.04-amd64.charm

I suspect this action's failure might be doing to something besides the charmcraft.yaml file.


Login with Twitter using OAuth1.0a Protocol via API in PHP

Proposed title of article
Login with Twitter using OAuth1.0a Protocol via API in PHP
Proposed article introduction
Practically all Internet giants like Google, Facebook, Twitter and LinkedIn support OAuth login. They furnish API with definite documentation to assist designers with coordinating OAuth confirmation. There are numerous customer libraries accessible to carry out Twitter OAuth login. However, I will do with just plain PHP for it is really adequate, lightweight and better. Application with the OAuth login feature has many advantages:

Simplifies the login process.
Reduces friction by minimising user’s effort with a single click.
Saves developers’ effort from building a custom login.
Assures secure authentication flow.

Key takeaways
By the end of this article, the following is expected;

Twitter OAuth login flow
How to integrate Twitter OAuth login?
Generating Twitter app keys
Twitter OAuth PHP service
PHP code to handle logged-in user data
Database script
Login with Twitter PHP example output

Article quality
There are various ways to implement Twitter OAuth login in a PHP application. Generally, people use built-in client-side libraries to implement. Twitter also recommends one or more PHP libraries in its community API gallery but this article shows a simple code for “Login with Twitter” integration for it uses no external libraries to achieve this. I hope we will have a good ride all along.
References
N/A

Based on this search, this topic appears to be over-saturated on other blog sites and official documentation. Feel free to suggest another topic. Thanks


Feat/136 redirection tests

Closes #136
Changes made by this Pull Request

Add tests for correct redirections
Change tests to assert on urls rather than page ids

Codecov Report

Merging #139 (b8418a2) into dev (f0b0d10) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head b8418a2 differs from pull request most recent head 077c8db. Consider uploading reports for the commit 077c8db to get more accurate results

@@           Coverage Diff           @@
##              dev     #139   +/-   ##
=======================================
  Coverage   93.38%   93.38%
=======================================
  Files          20       20
  Lines         423      423
=======================================
  Hits          395      395
  Misses         28       28

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 f0b0d10...077c8db. Read the comment docs.


Files Successfully Uploaded with RCLONE but Unable to Download Completely (Critical Issue)

I have been having trouble uploading large files with rclone, and I am getting the following error:
failed to close object after copy: uploaded failed
Despite this error, rclone will still repeatedly upload the file multiple times, consuming a lot of time and traffic until it succeeds.
To solve this problem, I followed the suggestion I saw here and added the --multi-thread-streams=0 parameter. After that, the error above no longer appeared, and the upload was very smooth and successful. However, I found that many files seemed to disappear after being uploaded to less than half of the percentage, and then the number of successfully transferred files would increase by one. To test this, I uploaded the same files again with rclone, and the result was also correct and skipped the re-upload.
Since it took so little time and it felt strange, I tried to download the files with rclone, but I found that the download speed would drop to 0 and get stuck forever after reaching a certain percentage (each file is different, about 20-30% or even less), and there was no error (no error message in rclone and teldrive backend).
I also tried to download the files directly from the website, but I found that they could not be opened at all and displayed
ERR_EMPTY_RESPONSE.
I tested a dozen large files, and only one or two files that were uploaded successfully multiple times before adding the --multi-thread-streams=0 parameter could be downloaded normally.
It feels like the original error display and detection have been blocked, and the actual problem has not been solved. On the contrary, because there is no error, it is impossible to know that the file upload has actually failed, which may lead to file loss.
I have tried using different browsers and different internet connections，would appreciate any help in resolving this issue.
Thank you.

What a particular problem, while I have the issue of rclone doing the "fail and retry" the files play fine though I wonder why
rclone doesn't function right when using that command...
I hope someone looks into this!
For now, while I have no way to help you out I do have something that can help with detecting files that weren't properly uploaded to telegram/teldrive. You can type this in the sql editor on Neon DB or whatever program you use to handle your database.

Command to list files with id:0 (aka not uploaded to telegram/missing/can't open or play.)
select * from teldrive.files where parts::text like '%{"id": 0}%'
Command to delete missing files:
delete from teldrive.files where parts::text like '%{"id": 0}%'
Another command to delete files by letting teldrive deal with it itself.
UPDATE teldrive.files
SET status = 'pending_deletion'
WHERE parts::text like '%{"id": 0}%';

Thanks to the folks on the discord for these commands that can be used to easily detect files that aren't uploaded!
The first command will list every file that has "id:0" which means that it wasn't uploaded to telegram properly, thus if you try to play the file it'll not work at all.
Once you confirmed what files need to be re-uploaded, you can then execute the second or third command and it'll automatically delete them. Hope this helps somewhat!

@cj-neo am having this issue as well

@Linden10 Thanks for providing the testing method. However, I found that the file seems to have been uploaded to TELEGRAM and even occupies compatible space, but the content is incomplete. I'm not using this software temporarily. I'll consider using it again when the author fixes the issue.

@Linden10 Thanks for providing the testing method. However, I found that the file seems to have been uploaded to TELEGRAM and even occupies compatible space, but the content is incomplete. I'm not using this software temporarily. I'll consider using it again when the author fixes the issue.

That’s a problem alright if the files aren’t being fully uploaded to telegram properly which is strange!
Hmm…say, have you restarted teldrive and tried uploading again? If yes then IDK…I hope the author or someone can figure something out.


Auto Batch Mode?

Remove the link from a text file assigned by --batch-file (or move to a backup file) when a file completed, and automatically resume the list for incomplete download when the last line of the text is done (instead of quitting, process the batch file again--sorry for my poor english), until the list is empty or no more .part file in the current directory. It would be useful for interrupted batch download due to wifi problem or other reasons. Now i'm using sed with some Windows batch code to do this, but it's a bit troublesome. it would be nice if youtube-dl can do this. thanks

Nowadays, --download-archive ....


Fix bug on query expression writer logic when DataMember is present without a explict name

Hi, this PR https://github.com/Azure/azure-sdk-for-net/pull/35954, introduced a bug that caused the expression to be generated like this:
 ne 'someString'
Updated the logic to consider the case that the property has a DataMember annotation but has no custom name set and expression is being generated like this:
DataMemberImplictNameProperty ne 'someString'
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

Thanks @rafaelmotaalves ! I'll help you fix the test breaks

Also - please add an entry to the CHANGELOG.md for this fix.

API change check
API changes are not detected in this pull request.

Done. Thanks @christothes


Cannot Add package from GitURL: Fatal Error, not in a git directory.

Unity Editor Version: 2022.1.24f1
I just discovered this repository and i'm extremely interested in using it to simplify my addressables work, however, i am not able to clone this project into my own project using the unity package manager.
I've followed the instructions for downloading a specific target version (1.0.1), but no matter how i enter the URL in the package manager.
https://user-images.githubusercontent.com/78170922/236641111-ae943abe-ae47-4014-b85f-c0d342ed2a14.mp4
However, i stumble into these two errors on my console.
[Package Manager Window] Cannot perform upm operation: Unable to add package [https://github.com/CyberAgentGameEntertainment/SmartAddresser.git?path=/Assets/SmartAddresser#1.0.1]:
  Error when executing git command. fatal: not in a git directory
 [NotFound].
UnityEditor.EditorApplication:Internal_CallUpdateFunctions ()

and
[Package Manager Window] Error adding package: https://github.com/CyberAgentGameEntertainment/SmartAddresser.git?path=/Assets/SmartAddresser#1.0.1.
UnityEditor.EditorApplication:Internal_CallUpdateFunctions ()

I installed 2022.1.24f1 and tested it, and I was able to install successfully.
I don't think the issue is related to Smart Addresser. Have you been able to install other packages from GitHub?

Jst tried to install another package from github, didnt manage to get itt to install, which is weird since i do have Git itself installed

Update, i digged out and searcher on the webs related to the error i've been having, and i stumbled into this StackOverflow issue that matchess pretty well my issue
https://stackoverflow.com/questions/72389135/unity-cant-add-git-packages-on-external-ssd-fatal-not-in-a-git-directory
The only difference i have from this post is the fact that i have my unity projects installed on an External Hard Drive that uses exFAT, this might be the main issue i'm having and chances are not related to your package at all.
i'll find a way to download the package in some other fashion outside of the repository, since i really want to try this out.

Update, i went ahead and did as the stack overflow post suggested

Git will throw an error if you aren't the owner of a local repo folder you're trying to work with. But exfat doesn't support security permissions, so you can't even set yourself as the owner. The solution for me was to globally disable safe directories for git. I just added

[safe]
    directory = *

i've now managed to install it succesfully

Thank you for the report.
I'm grad it worked well!


Missing Linux defconfig file

There should be a Linux defconfig file in guest/linux, so that those who want to build their own kernel can do so.
For example, since PMU-related SBI call is not supported in hypocaust-2, we need to have CONFIG_PERF_EVENTS=n in Linux defconfig.

Thank you very much for your suggestion. Unfortunately, I am currently busy with other matters, and this project is not very active anymore. If you are interested, you can contribute to the project by submitting a pull request (PR).
Additionally, I have been working on another updated project called hypercraft, which is a better-designed hypervisor project. It includes empty implementations for unused SBI calls. You may also want to check out that project.


Slow detecting long press gesture on iPad

It takes about 3 seconds to detect long press gesture on iPad, how to improve it?

I changed the scale that rendering fake cell.
Probably I fixed from it by this correction.

If this is fixed, the issue should be closed.


Fix tests

All tests are running except the ones involving active-record (Affected by https://github.com/yiisoft/active-record/issues/23).
Also noticed an important change, with Yii2 the @app alias was always defined (running tests on yii2-twig), whereas here, I need to explicitly define it in config/tests.php.
Is it normal that @app is gone?

@schmunk42 I think I need some guidance here.
Since this commit in yii-jquery, requiring yiisoft/yii-jquery in twig's composer.json isn't enough to have composer download the javascript assets.
I tried adding "foxy/foxy": "^1.0.0" in my requirements (dev or not), but it doesn't seem to read and install from yiisoft/yii-jquery/package.json. Isn't that what it's supposed to do ?
Using "wikimedia/composer-merge-plugin": "dev-master" for now.

I can't make good default guess for @app alias right now. We'll see later.
Setting it in config/tests.php is good for now.

Concerning foxy - it worked for me. I think you've missed some detail.

You need npm (or yarn) for foxy, but it should throw a warning if it can't find the binary.
The error in the tests is from a SQL statement?! Why is Twig needing a DB?

I do have npm & yarn on my system. I think we're missing something in yiisoft/yii-jquery composer file (extra: { foxy: true } or a require-dev to foxy/foxy) to make foxy pick up the package.json file.
As for the errors, some tests are using ActiveRecord ¯\(ツ)/¯.

https://github.com/fxpio/foxy/blob/master/Resources/doc/usage.md#use-the-plugin-in-php-library


Use password reset mechanisms with JSON-only communication

We are developing a backend with pure JSON communication in Rails and use Devise for user authentication. Unfortunately, the password reset functions don't seem to support JSON at all. I've tried to implement the following in a custom controller.
    def create
      @user = User.send_reset_password_instructions(params[:user])
      if successfully_sent?(@user)
        head :status => 200
      else
        render :status => 422, :json => { :errors => @user.errors.full_messages }
      end
    end

Route used for this:
scope '/api/v1' do
    devise_for :users, :skip => :sessions, :controllers => { :passwords => 'api/v1/users/passwords' }
end

The successfully_sent method was copy&pasted from the DeviseController but still, instead of printing just status 200, it renders devise/mailer/reset_password_instructions.html.erb which confuses the JSON client.
Is there any way to make the password workflow support JSON?

https://github.com/plataformatec/devise/wiki/API-Mode-Compatibility-Guide


Support ‘fvar’ tables for variable fonts

This table is part of Apple’s Advanced Typography, not OpenType; so
this might be considered out of scope for the opentype.js project.
On the other hand, the FreeType project has implemented AAT variation
axes, so the utility goes beyond MacOS. If opentype.js was fine with
accepting changes for AAT tables, I would implement the ‘avar’ and
‘gvar’ tables in subsequent changes.

This is totally awesome and will be a massive massive boot for metapolator :) cc @graphicore @felipesanches

@brawer awesome! Do you have some fonts to test this with? Built-in fonts on OS X that use it?

@fdb /Library/Fonts/Skia.ttf has an fvar table.

@fdb Just sent you an e-mail with a couple test fonts. Sadly these are unfree, so I can’t post them in public. However, fonttools now has a sample code snippet for building your own multiple-master fonts (but it’s work in progress, I wouldn’t fully trust it yet).

Thanks @brawer I got them. Note that it will take until Monday-Tuesday before I have time to look at it.


Updated two name spellings and links

@todorgb : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@todorgb: I'm sorry - only the author of this article, @stevevi, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Allow closures to be passed as sanitization rules

(for Adonisjs)
This way, the code is more clear, and you don't necesary need to be creating service providers for one or two extra rules
{
  latitude: parseFloat,
  longitude: Number,
  lat_lng: (val) => { // a more complex sanitization
    try {
      return JSON.parse(val)
    } catch (err) {
      return null
    }
  }
}

There is no need to created service providers for a rule, you can simply add them inside your hooks file. I can allow closure, but then you will be writing this inline everywhere you need this sanitization rule.
Instead using start/hooks.js file and simply add it to the existing list.
const { hooks } = require('@adonisjs/ignitor')

hooks.after.providersBooted(() => {
  const { sanitizor } = use('Validator')
  sanitizor.lat_lng = (val) => {
  }
})

And then use it like any other rule.

I wasn't aware of app hooks since this file wasn't create with the api boilerplate, now I see its in the docs! Thank you very much!
Anyway, there may be the case where you need sanitization for one specific case, in the whole app, so there's no need for code resuse and a simple closure might do the job

No plans to add it now, will look into it later. Thanks!


Import Error

Just installed from main in a virtualenv with pip install .:
bell https://hooks.slack.com/services/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX                                                 Traceback (most recent call last):
  File "/home/matthew/Documents/mantis/bell/bell/bin/bell", line 5, in <module>
    from bell import cli
ImportError: cannot import name 'cli' from 'bell' (unknown location)

I do not get that error. Could you describe the steps so that we can reproduce this? I wonder whether this is related with the version of pip?

I think it was down to my virtualenv actuually. Seems to be resolved after creating a new one.


Moving deploy to composer

Lab/Demo: 13 and 14
Fixes #83
Changes/TODO

[x] Remove deployment from Bot Framework SDK lab
[ ] Add deployment to Bot Composer lab
[ ] Update Learn pre-reqs to match requirements for Bot Framework lab
[ ] Update Learn pre-reqs to match requirements for Bot Composer lab
[ ] Test both labs with Pay As You Go Subscription
[ ] Test both labs with an Azure Pass subscription

When trying to deploy my bot in the Azure Composer lab, I received this error:
Provisioning completed 3 items before encountering a problem. {"name":"CREATE_COSMOSDB_ERROR","message":"Error - The subscription is not registered to use namespace 'Microsoft.DocumentDB'. See https://aka.ms/rps-not-found for how to register subscriptions.","stack":"Error\n at new CustomizeError (C:\\Users\\Student\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:255547:25)\n at createCustomizeError2 (C:\\Users\\Student\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:255552:10)\n at AzureResourceMananger.deployCosmosDBResource (C:\\Users\\Student\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:256698:13)"}. See provision.WeatherBot.log in your bot folder

It failed to deploy, I'm trying to re-create the Publishing Profile without the optional Azure resources, since I think the error was in CosmosDB.

It failed to deploy, I'm trying to re-create the Publishing Profile without the optional Azure resources, since I think the error was in CosmosDB.

This resulted in a successful provisioning.

Error still: There was an error sending this message to your bot: HTTP status code ServiceUnavailable

Ok, I was able to identify the actual error...it's the same zip error 😭
Deploy zip error...service is unavailable

Created an issue in the Bot Composer repo: https://github.com/microsoft/BotFramework-Composer/issues/8724


it says delete indium or get a compatible version with sodium i delete it because there are non that work but it says you need indium for sodium to work please fix

it says delete indium or get a compatible version with sodium i delete it because there are non that work but it says you need indium for sodium to work please fix

I am well aware that there's new versions of Sodium and Minecraft! Please be patient, I have a full time job and limited free time, which I would rather spend updating the mod than closing all 9 (!) of these issues 🙃

ok thanks


Week 5

Closing pull request based on weekly meeting. Planning to edit and improve the Makefile, then re-open.


Support for floating point values for maxEirp

This is to track necessary changes required for requested and granted values for maxEirp.

Test harness treats maxEirp as "number", so integers and real numbers are already supported.


#Principles Naming conventions

@fedect1 In the backend you have two classes posts.js and post.js which is very close by. Through the folder it's a little bit clearer. I prefer a more declaritive approach because it's immediately clear what the class does.
posts.js -> postService.js (I'll go into more detail in the service lecture) = clear, because it provides services to anything related to posts
post.js = fine, because it is in fact a post

Another thing with constants and configuration parameters. As developers we create standards so every new dev is able to understand our code quickly.
posts.js
const expitartionTime = 3 * 15 * 60 * 1000
const commentTimeExtension = 15 * 60 * 1000
const likeTimeExtension = 5 * 60 * 1000

First I think you mean expirationTime instead of expitartionTime.
I do like that you actually spell out the calculation itself or better understand instead of simply writing the result.
From what I can see in the code it's always about the expiration time and u just use the word extension for comments and likes. Here are my ideas as to name your config values:
// base units for proper expiration date calculation
const SIXTY_SECONDS = 60 * 1000 // in case this is milliseconds, which I assume
const FIFTEEN_MINUTES = 15 * SIXTY_SECONDS

// configuration values depending on the type of element
const EXPIRATION_TIME = 3 * FIFTEEN_MINUTES
const EXPIRATION_TIME_FOR_COMMENTS = FIFTEEN_MINUTES
const EXPIRATION_TIME_FOR_LIKES = 5 * SIXTY_SECONDS


WA Delivery Failure Resubmits from 4/25/2022

We had two reported AS2 delivery failures for WA on 4/25/2022 that we need to resend to Washington State. In prior cases these were false negatives, where the files were delivered but their system reported a timeout.
We need to confirm if they received the files, and if they did not, resubmit them.
Positive Feed:
429ae247-6072-4a94-98d5-76cab42f9745
Negative Feed:
952d1dbf-f534-4e33-aa00-106ceceab42e

@jorg3lopez is working on this ticket.

Updagte: Gathered all positive feed files and parsed them. Working on the neg feed. Will send email with files once the negative feed is done.

Updagte: Gathered all positive feed files and parsed them. Working on the neg feed. Will send email with files once the negative feed is done.

@Adrian-Brewster @anshulkumar-usds both files have been resent to WA, one containing 52 test results and another containing 990 rest results.
file names:
cdcprime_testing_20220425171556_629764
cdcprime_testing_20220425171626_951856___neg

5/13: WA was notified. Awaiting confirmation from state.

5/16: WA confirmed that they received the files.


Fix module checkbox

Fix logic checkbox + add example.
And add .md in .editorconfig

That is pretty big and good change! :+1:
Can you add description to prop types? (to generate docs from them)
Can you add Checkbox definition to index.d.ts ?
PS. you can write complex examples in jsx - see https://github.com/hallister/semantic-react/tree/master/src/components/modules/modal/examples and https://raw.githubusercontent.com/hallister/semantic-react/master/src/components/modules/modal/examples/modal.examples.md

@asvetliakov,  yes, I do.

@asvetliakov,  done.

@asvetliakov, fixed.

@asvetliakov, add type


Authentication failure

Describe The Bug:
I tried to setup this plugin but it fails to authenticate with my bridge, throwing an error
To Reproduce:
I simply installed the plugin, configured it with host, user and password and set tls to false.
Expected behavior:
I've been using https://www.npmjs.com/package/homebridge-freeathome-local for some time with the same details and that one does not have authentication issues. (It has other issues, hence I'm hoping that this new plugin will make my life easier!)
Logs:
[6/16/2022, 10:08:26 PM] [free@home / local] Authentication information is missing or invalid.
[6/16/2022, 10:08:26 PM] [free@home / local] Device discovery failed Error: Authentication information is missing or invalid.
    at r.SystemAccessPoint.<anonymous> (/usr/lib/node_modules/homebridge-freeathome-local-api/node_modules/freeathome-local-api-client/dist/cjs/webpack:/freeathome-local-api-client/src/system-access-point.ts:363:15)
    at Generator.next (<anonymous>)
    at /usr/lib/node_modules/homebridge-freeathome-local-api/node_modules/freeathome-local-api-client/dist/cjs/index.js:1:155821
    at new Promise (<anonymous>)
    at n (/usr/lib/node_modules/homebridge-freeathome-local-api/node_modules/freeathome-local-api-client/dist/cjs/index.js:1:155566)
    at r.SystemAccessPoint.processRestResponse (/usr/lib/node_modules/homebridge-freeathome-local-api/node_modules/freeathome-local-api-client/dist/cjs/index.js:1:159405)
    at r.SystemAccessPoint.<anonymous> (/usr/lib/node_modules/homebridge-freeathome-local-api/node_modules/freeathome-local-api-client/dist/cjs/webpack:/freeathome-local-api-client/src/system-access-point.ts:173:17)
    at Generator.next (<anonymous>)
    at u (/usr/lib/node_modules/homebridge-freeathome-local-api/node_modules/freeathome-local-api-client/dist/cjs/index.js:1:155623)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
[6/16/2022, 10:08:26 PM] [free@home / local] Unexpected response received

Hi and thanks for bringing this to my attention!
Could it be that you are using the name of your user account instead of the user ID? The first you are using to authenticate to the web UI of the System Access Point and it is also used by the plugin you mentioned and it's predecessor, while the user ID is a guid. You can see the user ID in the free@home next app, or you can get it from the System Access Point Swagger endpoint at http://{SysAP IP or Host}/swagger/users.

Alright, tried it out, and it does work now! I'm facing one other issue but maybe we can create a separate issue:
Whenever I change the brightness of one of my dimmer lights, it says in the logs that it's changing the brightness to that value but immediately after it also changes it to 100%, hence my lights almost always go to full brightness...

Happy to hear your issue is resolved.
The other issue sounds strange, doesn't happen on my setup. But let's discuss this in another issue.

The user ID is a guid that you can obtain from the free@home next app or from the System Access Point Swagger endpoint at http://{SysAP IP or Host}/swagger/users (it's the string beneath the user account name).

Hi,
I couldn't find the user ID anywhere in the free@home next app. The link http://{SysAP IP or Host}/swagger/users just outputs an error. Do I need to be the installer of the free@home system for that?

free@home next App
You can find the local API options in the burger menu -> free@home Settings -> Locale API

Swagger Endpoint
Works for me. Can you be a little more specific as to what error you are encountering? You don't need to be authenticated to use the endpoint.

Hi sorry for the late response,

You can find the local API options in the burger menu -> free@home Settings -> Locale API

Unfortunalty, I can't find the described submenu "Lokale API", as seen here:

Works for me. Can you be a little more specific as to what error you are encountering? You don't need to be authenticated to use the endpoint.

Or when I try reaching the endpoint, I only get an error message:

Maybe I'm overlooking something simple. Thanks for your help.

So apparently I have the old Sys AP (6200 AP-101), which doesn't support the local API feature (german). Is there any other way to solve this problem? If not, I will probably have to return to Henry Spanka's version.

I'm really sorry, but there is no workaround that I am aware of. Local API is only supported on SysAP 2.0 running firmware version 2.6.0 or newer. I know that is a major pain point among the SysAP version 1 owners who feel that they are being "punished" as the early adopters. I'm keeping my fingers crossed that Busch + Jäger comes around and if they do, I'll do my best to support SysAP v1 as well.

Thanks for your effort :)


Missing execution event for FALLBACK_REJECTION

Also added unit test to help with understanding #685

NetflixOSS » Hystrix » Hystrix-pull-requests #54 SUCCESS
This pull request looks good


fix(sec): upgrade com.alibaba:fastjson to 1.2.83

What happened？
There are 1 security vulnerabilities found in com.alibaba:fastjson 1.2.47

CVE-2022-25845

What did I do？
Upgrade com.alibaba:fastjson from 1.2.47 to 1.2.83 for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
How can we automate the detection of these types of issues?
By using the GitHub Actions configurations provided by murphysec, we can conduct automatic code security checks in our CI pipeline.
The specification of the pull request
PR Specification from OSCS

 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.

Plugin dependencies only affect tests. There is no security impact.


ajaxHeaders not applied when retrieving reference strip thumbnails

Hello!
If I understand correctly the ajaxHeaders and loadTilesWithAjax configs can either be specified as a top level OpenSeadragon config, or on a per level basis as a TileSource config.
We are currently providing the viewer an array of strings as the tileSources and using the ajaxHeaders/loadtilesWithAjax configs on the top level OpenSeadragon object.
The main image loads correctly as the headers are applied as expected, however the each of the thumbnail requests for the reference strip DO NOT get the headers, and thus they do not load correctly.
I noticed the reference strip actually creates a handful of "mini viewers": https://github.com/openseadragon/openseadragon/blob/9506d6e5392e24aefa63608c9bb123f86f26424b/src/referencestrip.js#L441
Wondering if it would make sense to pass along the ajaxHeaders/loadtilesWithAjax configs from the parent viewer down to these mini viewers in order to preserve the top level setting?
Thanks!

Yes, that sounds like exactly the right fix! I believe loadTilesWithAjax was added long after the reference strip and nobody thought to pipe it down into there.

Sweet! Yes we would be honored to make that enhancement!

Awesome! Let me know if you need any pointers :)

Fixed by #1701


Try pystac

Enabler
So that we don't reinvent the wheel, we might want to use pystac to read, validate and manipulate STAC metadata.
Acceptance Criteria

[ ] pystac has been tried
[ ] check with @palmerj on previous issues / learnings
[ ] decision to continue using pystac, or not
[ ] if continuing, fully implement it

Additional context
It might be easiest to introduce this in import_metadata_file, to see whether there are any snags.
Tasks

[ ] Install pystac with the "validation" extra
[ ] Replace STAC-handling code with pystac code where applicable

Definition of Ready

[ ] This story is ready to work on, according to the team's definition

Definition of Done

[ ] This story is done, according to the team's definition

Happy to discuss. I'm on leave for 2 weeks from Monday. But will pop into office on 29yh of April so can quickly meet in person too.

Initial results documented here by @MitchellPaff. https://confluence.linz.govt.nz/display/LGDL/Pystac+Investigation


Implement declarative auth design for gRPC service

Description:
Currently, the gRPC service only has the imperative auth support. We need to provide declarative auth support as well.
Design: https://docs.google.com/document/d/1dGw5uUP6kqZNTwMfQ_Ik-k0HTMKhX70XpEA3tys9_kk/edit?usp=sharing

Duplicate of https://github.com/ballerina-platform/ballerina-standard-library/issues/1465 ?

Nope. Those are different for client and service.

Currently the authDesugar returns an error when it is Unauthenticated or PermissionDenied. Here authDesugar uses the caller to send an error to the client.
For unary instance, the client receives the error in the following manner,
helloWorld55Client hClient = check new ("http://localhost:9155");
string|Error result = hClient->hello55UnaryWithCaller(ctxString);
if result is PermissionDeniedError {
    //
} else {
    //
}

For streaming scenario, client receives the error only when trying to receive a response. Is this the expected behaviour?
@daneshk @BuddhiWathsala ?
helloWorld55Client hClient = check new ("http://localhost:9155");
Hello55BiDiWithCallerStreamingClient strClient = check hClient->hello55BiDiWithCaller();
check strClient->sendContextString({
    content: "Hello from client",
    headers: requestHeaders
});
check strClient->complete();
string|Error? s = strClient->receiveString();
if s is UnauthenticatedError {
    //
} else {
    //
}


Network connection authenticator tests

In this PR, I have added unit tests for the network connection Authenticator and made one logic update: previously, the system considered variable statuses other than "Rejected" during authentication. I have modified this to only consider "Accepted" statuses, which seems to be in line with OCPP specs:
(A01.FR.03)
precondition: The Charging Station responds to this SetVariablesRequest with a SetVariablesResponse with status Accepted.
then: The CSMS SHALL assume that the authorization key change was successful, and no longer accept the credentials previously used by the Charging Station
(A01.FR.04)
precondition: The Charging Station responds to this SetVariablesRequest with a SetVariablesResponse with status other than Accepted
then: The CSMS SHALL assume that the Charging Station has NOT changed the password. Therefore the CSMS SHALL keep accepting the old credentials.
Additionally, maybe we could consider using the "value" field of the BasicAuthPassword variable directly for password extraction, instead of relying on status updates? I believe this would achieve the same result, but I wouldd appreciate feedback on whether this approach might miss any edge cases?

Additionally, maybe we could consider using the "value" field of the BasicAuthPassword variable directly for password extraction, instead of relying on status updates? I believe this would achieve the same result, but I wouldd appreciate feedback on whether this approach might miss any edge cases?

This is a good idea! We will be moving away from the VariableStatus entity in a future release anyways.


Fixed table view not showing all redactions

Showing all TableView redactions right now
Will need another update to display each entry correctly (if accepted or rejected, then fade the entry indicating that redaction's status has been updated).

Updated tableview


Android System kills the app when user minimize the app for long

I use this library to develop on media player app, it works fine but when the user minimize the app for long the Android system silently kills the app.
I hope this bug get fixed.
Thanks

Hello @EjilolaHammedEjitomiwa.
I suspect this is a normal behavior from Android S.O. Generally, when your device RAM is small and the S.O needs more space to run Android kills apps. You can read more about it here: https://developer.android.com/guide/components/activities/process-lifecycle
I'm closing this issue for now... if you have more supplies and context for this problem, you can reopen it.

Hello @EjilolaHammedEjitomiwa.
I suspect this is a normal behavior from Android S.O. Generally, when your device RAM is small and the S.O needs more space to run Android kills apps. You can read more about it here: https://developer.android.com/guide/components/activities/process-lifecycle
I'm closing this issue for now... if you have more supplies and context for this problem, you can reopen it.


Freeze transformers package version

Latest transformers package (4.9.1) breaks the ROBERTA text encoder during pretraining, so we freeze it at version 4.5.1

Thanks for your contribution @nguyeho7! Merging this.


Build failure post Android Studio 3.0

I think it is time for an overhaul of the exercise and sunshine codebase.
Android Studio 3.0 is a significant upgrade that warrants revising the codebase while still retaining all the compat goodness. I was successful in upgrading a couple of the exercises making it work with the newest 3.0.0 gradle plugin. However, doing this for each and every exercise is totally redundant. Further, it takes away a lot of time from learning and/or implementing the concept/component from the current lesson.
So, I believe that an upgraded exercises and sunshine codebase would not only help us but posterity as well.

Seconded, some toy apps need modifications to work properly on the latest gradle plugin.

How did you upgraded the gradle plugin for the exercises to work with android studio 3.0 and above?

For AS 3.0.0, Gradle 4.1 or higher is required. Take a look at the below page.
https://developer.android.com/studio/build/gradle-plugin-3-0-0-migration


CircularReference management

This library is exactly what many people working with EF are looking for, unfortunately there's a missing feature that makes it unusable, the capability to clone graph of objects with circular references.
EF Graphs are by default CircularReference Graphs because into the Parent table there is a Collection of Cildren and into the Child table there is the Reference of the Parent.
I tried to fork and to work on your code to add this feature but I'm not enough skilled to do with an acceptable level of quality so I'm asking you if you could work on it to add this functionality.
I'm also available to pay you something for your effort.
Thanks again!
mM

I think you can achieve that by tweaking ReplicationContext. Iso of "get value" api it should get "set value" api - something like void Replicate(object source, Action<object> resultSetter). Then in case of detected circular references, it can postpone setting a reference until the end of the cloning procedure.
Though this change might have quite a significant performance impact. I'll give it a try.

I'm done with naive implementation - you can find it in commit 60bde85. Though it does not perform well enough, so I have to spend more time on tweaking it, before it can go to master.

I gave a look at the commit and it seems interesting.
In the meanwhile, since this time I'm in position to manage the t4 of the EF model, I opted for a T4 solution where I use a Queue to deal with references objects in order to avoid looping during the clone.


Write parsers for top level declarations

This forms part of #2.
Focus of this is mainly to add top level declarations to the language (requires features implemented in #3).

[ ] module declaration
[ ] import declaration
[ ] export declaration
[x] data declaration
[ ] type declaration
[ ] newtype declaration
[x] typeclasses

[x] class declaration
[x] instance declaration

[x] fixity declaration

Module, import and export declarations will be added as soon as support for multiple files is needed.
Type and newtype declarations will also be skipped until first version of type system is implemented.


[Language Request] support vietnamese

Information
Một số kí tự việt nam chưa hiển thị được.xin hổ trợ giúp ạ

Alright.  Does that mean most characters do work?  I'm guessing Vietnamese shares a lot of characters from the European sets.
Can you take a look here?  https://usefulwebtool.com/characters-vietnamese  And answer this... is it the characters in the 7000-range that do not display correctly?
I took a look at https://stackoverflow.com/questions/37579692/unicode-range-for-vietnamese and there's some good information for me here too but it makes me wonder about 8 pixels... I'm counting a minimum of 11 different accent marks above letters - ĂẰẲẴẮẶÂẦẨẪẤ - maybe more?  Even for the European & Cyrillic letters, many of accents ended up looking very similar.

I'd still like your input but it's been added anyway!

Let me know if there are any problems!


Add all proving artifacts / files to frontend

We should host as many of the necessary artifacts for proving (.wasm, .vkey, .zkey) as possible to avoid another fetch from another server.

[ ] Benchmark page load times with and without loading the artifacts
[ ] Benchmark loading the artifacts from external servers

Pending: https://github.com/project-vinyl/babyjubjub-ecdsa/issues/10


Incorrect description of join condition in INNER_JOIN.md

In INNER_JOIN.md you state:

If we want to know the name of the department each employee is in then we want our condition to filter out only those rows where the employee.department_id column is equal to the department.id column.

What I think you meant to say is:

If we want to know the name of the department each employee is in then we want our condition to select only those rows where the employee.department_id column is equal to the department.id column.

OR

If we want to know the name of the department each employee is in then we want our condition to filter out those rows where the employee.department_id column does not equal to the department.id column.

Ah yes thank you! Will fix.


Remove visible references to "concepts"

Prefer codes over IDs.
Remove "concept" from titles.

Is removing the concept_id field a request from UX or VUMC? That's been useful to me when working with the AoU CB, though I'm certainly not a representative user.

Neither, it just frees up more space for other columns. Why has it been useful? Just for debugging? Isn't it just an internal ID and the Code is the actual meaningful identifier?

Is removing the concept_id field a request from UX or VUMC? That's been useful to me when working with the AoU CB, though I'm certainly not a representative user.

Neither, it just frees up more space for other columns. Why has it been useful? Just for debugging? Isn't it just an internal ID and the Code is the actual meaningful identifier?

Yes for debugging and comparing Tanagra with the current AoU-CB when talking to VUMC folks. Maybe we can just check with them at our next checkin?


Reply to this email directly, view it on GitHub, or mute the thread.

thanks everything


Show detect rule for anomaly

Close this pull and put it in https://github.com/apache/incubator-pinot/pull/4526


empty class acts as "any" type: by design or bug?

The following typechecks fine:
class Empty {
}

// From a strongly-typed point of view, this feels strange
var e1: Empty = "Strange";
var e2: Empty = 42;

This also compiles
class Foo {
	name: string;

	constructor(name: string) {
		this.name = name;
	}

	hello() {
		return "456";
	}
}

class Bar {
	name: string;

	hello() {
		return "123";
	}
}

var foo = new Foo("Jo");

// From a strongly-typed point of view, this feels strange
var bar: Bar = foo;

Is this by design?

Yes. Note that the empty type does not behave as any - it is a valid target for all assignments, but is only a valid source for assignments to other empty types:
var e: Empty, f: Foo;
f = e; // Error

TypeScript uses a structural type system, see also:
http://stackoverflow.com/questions/22385775/typescript-type-check-failed-to-catch-wrong-assignment/22386927#22386927
http://stackoverflow.com/questions/32106253/why-does-typescript-implicitly-convert-between-unrelated-classes/32106416#32106416
http://stackoverflow.com/questions/28080106/typescript-type-confusion-variable-doesnt-mind-type/28080519#28080519

I'm using typescript in an educational situation and would find useful to have a stricter mode that doesn't allow this special case. Is there any way to disable this? I found a work-around : adding a dummy private member makes not empty (obviously).
class Empty
{
    private dummy : any;
}

let e : Empty = 2;

Type 'number' is not assignable to type 'Empty'. Property 'dummy' is missing in type 'Number'.

I use '_' instead of 'dummy' in my lib (empty File class, it's really just a handle for the user, no attributes or methods) but you get the idea.

There isn't a way to disable this behavior.

Is adding one an option or is it too tied to the type system? I haven't read the type system docs you provided above, yet.

This isn't something we'd add an option for. It's a structural type system, so empty types are equivalent by definition.


Unmock changes

Published as @meeshkanml/json-schema-ref-faker
Uses @meeshkanml/json-schema-ref-parser
Added react-native field using the same module as Node.js

Hey @ksaaskil, I'm just seeing this PR now as I'm sorting through the forks for the migration. Are you using this adjusted fork for the react-native project or rather experimenting? Sorry if this is a seemingly obvious question, but it's not clear to me 😬

Yes this is used in unmock-js, it's published as @meeshkanml/json-schema-ref-faker.

Ah ok, makes sense. But then should we merge this PR or keep it open?

Can someone add me as a collaborator on our npm packages? My username is carolstran @ksaaskil @mikesol

Done!
Get Outlook for Androidhttps://aka.ms/ghei36

From: Carolyn Stransky<EMAIL_ADDRESS>Sent: Monday, January 13, 2020 11:23:45 PM
To: unmock/json-schema-faker<EMAIL_ADDRESS>Cc: Mike Solomon<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [unmock/json-schema-faker] Unmock changes (#1)
Can someone add me as a collaborator on our npm packages? My username is carolstran @ksaaskilhttps://github.com/ksaaskil @mikesolhttps://github.com/mikesol
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/unmock/json-schema-faker/pull/1?email_source=notifications&email_token=AAEAIJX6RGPK5T2HJIHGAVTQ5VSADA5CNFSM4J3XMPH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI3SPZI#issuecomment-574040037, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEAIJVE5MEMQYW7DBR3S2LQ5VSADANCNFSM4J3XMPHQ.


doc(kic) add GWAPI redirect instructions

Description
Add GWAPI instructions to the KIC HTTPS redirect guide. Due to https://github.com/Kong/kubernetes-ingress-controller/issues/4890 this uses our vendor annotations.
Adds a TLS configuration include. This creates a certificate and injects configuration to use it into either an Ingress or Gateway. The certificate hostname is configurable. The Gateway and Ingress names are not and expect the standard echo Ingress or kong Gateway.
GWAPI does not easily allow adding HTTPS configuration without a TLS Listener, and these Listeners must have a certificate as of GWAPI v0.8. We can't use the default Kong certificates if we don't care about the specific certificates, so we need this include for any guides that touch HTTPS-specific functionality.
Testing instructions
Preview link: https://deploy-preview-6335--kongdocs.netlify.app/kubernetes-ingress-controller/latest/guides/configuring-https-redirect/
https://deploy-preview-6335--kongdocs.netlify.app/kubernetes-ingress-controller/latest/guides/configuring-https-redirect/#configure-an-https-redirect has some issues where navtabs make the CSS very angry.
https://deploy-preview-6335--kongdocs.netlify.app/kubernetes-ingress-controller/latest/guides/getting-started/#add-tls-configuration no change, but now an include.
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

Hi @mheap . I tested validated and formatted the guide. Please review and state your approval/merge so that I can port this content to 3.0

ingresshttpsredirect.txt
gatewayAPIhttpsredirect.txt


Fixed Vector.insert when index of new element is less than first element

I had an issue with Index.search() returning some results with a score of zero. The problem was that it was building a queryVector with out-of-order elements (e.g. 55/brazil, 52/brabant, 53/bracknell, 54/bratislava), which turned out to be a bug in the Vector.insert() method. The first element inserted could never be preceded by any other elements, leading to the strange order seen above.

This change is included in the latest release 0.5.8, thanks for your help!

No problem at all. Thanks for all your hard work making this library in the first place!


Add delete_chat_by_id command

Closes #22

I'll update hhh_id and group_message_id to int as part of #33.

I'll update hhh_id and group_message_id to int as part of #33.


Decide what str::Pattern impls to support in libstd.

Right now, the string pattern API can be used to search for the occurrence of a pattern in a string:
"foo".contains("foo")              // &str:                Any substring == "foo" ?
"foo".contains('o')                // char:                Any char == 'o' ?
"foo".contains(char::is_lowercase) // FnMut(char) -> bool: Any char that is lowercase?
"foo".contains(&['f', 'o'][..])    // &[char]:             Any char that is either 'f' or 'o' ?

The set of types this API is implemented for basically got grandfathered in from the old CharEq trait + &str. They are, currently, char, &str, T: FnMut(char) -> bool and &[char].
However, this is not necessary the best set of types to support in libstd, so some brainstorming should be done on whether these impls are actually fine, or whether some should be added.
My preliminary thoughts on this:
Existing impls

&str, char and F: FnMut(char) -> bool seem to be fine. They have unambiguous meaning, and good ergonomics for usage.
&[char] is problematic. It used to be the case that a [T; N] coerced to a &[T], but then the originally ergonomic "foo".split(['a', 'b']) became the un-ergonomic "foo".split(&['a', 'b'][..]). Additionally, with the switch from CharEq to Pattern, a "slice of chars" became ambiguous in meaning - one could imagine that &['a', 'b'][..] has the same meaning as "ab".
If this pattern got removed, uses of it could be replaced with the more verbose |c| ['a', 'b'].contains(c) or |c| "ab".contains(c), or with a new newtype wrapper like AnyOf("ab") or AnyOf(&['a', 'b'][..]).

Potentially missing impls

A impl for OsStr might be useful in certain situations dealing with platform interacting code.

A impl for u8 might be useful for a guaranteed fast byte search in a string.

A impl for a predicate that receives a suffix of the string might be useful for writing ad-hoc lexers:

for x in "123+456".matches(|s: &str| -> Option {
if starts_with_number(s) { ... }
else if is_operator(s.char_at(0)) { ... }
else { ... }
})  { ... }

Thanks for writing this up!
I agree that we may want to remove the impl for &[char] for now, it definitely seems like its use has gone down slightly in terms of ergonomics.

The ergonomics may have gone down, but I think the functionality is still valid. In fact, the last time I wanted to use matches() I discovered that it didn't do enough; I wanted it to support &[&str] to match any of a set of strings (which is something that cannot be done with the current F: FnMut(char) -> bool).

This is not planning to implement regexes? I've found the combo of split and join important for those inevitable string manipulations in other language.
Recently python-like behavior for split was useful (if you use a single capturing group on the whole regex, the matching separator pattern is included into the returned array, if you don't it isn't (i also had trouble because i originally included 2 separate capturing groups instead of a single for the whole pattern so the array had 'Nones')

@i30817: https://crates.io/crates/regex already implements this API, or did you mean something else?
@kballard: I agree that the behavior of both &[char] and &[&str] as a pattern is useful, I'm just concerned about it being confusing for them to implement these traits directly.
I'd have nothing against an explicit wrapper type:
// Defined for splices:
foo.contains(AnyOf(&['a', 'b'][..]));
for s in bar.split(AnyOf(&["xxx", "yyy"][..])) { ... }

// Alternatively defined for iterators:
foo.contains(AnyOf(['a', 'b'].cloned()));
for s in bar.split(AnyOf(["xxx", "yyy"].cloned())) { ... }

An explicit wrapper type is an interesting idea. I'm slightly concerned that this could be confusing if anyone wants to try making a Searcher that wraps several other Searchers and runs them all in parallel, although I don't think that's quite compatible with the current Searcher API (because you cannot instruct a Searcher to backtrack, e.g. if it tells you that it found Match(2,5), you can't tell it to try searching again from position 3).
That said, I'm still not convinced that the current setup is confusing. I agree that &[char] is not as ergonomic as it used to be, as you have to say &['a', 'b', 'c'][..] now instead of just saying ['a', 'b' ,'c'], but I don't think &[char] vs &str is confusing. The former is a list of things to search for the, the latter is a single thing to search for.

I guess the mild concern I listed before is pretty negligible. The API doesn't really allow for that, and another name could be chosen if that was desired. So while I'm still in favor of keeping &[char], I would not be unhappy with a wrapper type AnyOf. However, if we do have a wrapper type like that, I would want it to be generic over T: Pattern, even if it's only implemented at the moment for AnyOf<char> (specifically, I want to be able to extend it to support AnyOf<&str> in the future, as searching for the first match from a set of strings is useful behavior that it doesn't currently allow).

Closing in favor of https://github.com/rust-lang/rfcs/pull/2500.


Competitive Programming : Leetcode Question (Integer to English Words

Description 📜
Solving leetcode Question (Integer to English Words)
Difficulty :Hard
Link : https://leetcode.com/problems/integer-to-english-words/
Would be explaining the approach with code in C++
Domain of Contribution 📊

[x] Competitive Programming

/assign

/assign

/assign


App does not always 'connect' properly.

I will add screenshots. The app does not always connect properly and it will ask me to reconnect bluetooth or connect via USB. I don't know what triggers this behaviour.

This state appears when AndroidAutoIdrive detects that the car is connected properly, but also that BMW Connected hasn't started the tunnel to the car. This is a limitation in BMW Connected, unfortunately.
The instructions to reconnect Bluetooth are the best way to trigger the tunnel to start working, I believe. I don't know a good way to keep BMW Connected always ready to notice the first Bluetooth connection, but the second one almost always works.

What I have done is make a Bixby Routine to automatically open the BMW Connected app when the car connects via Bluetooth. It still borks out sometimes, but it generally works much better than without this. 😁

I noticed that after every update, it is better to strictly recheck the full appliance of the tips mentioned here for both AAI & Bmw\Mini Connected in order to reduce the chances to have issues with phone connectivity  https://hufman.github.io/AndroidAutoIdrive/connection.html


mocha-4 genesis file does not define an app version

Context
https://celestia-team.slack.com/archives/C02PFFCGMNW/p1725353291554399?thread_ts=1725273825.398589&cid=C02PFFCGMNW
Problem
Attempt to sync a v2.1.2 node from genesis with the genesis.json file defined at https://github.com/celestiaorg/networks/blob/master/mocha-4/genesis.json
Proposal

Identify if there are any issues with updating the existing genesis.json file.
Update the mocha-4 genesis file (I checked both celestia and arabica-11 and they have the correct app version in the genesis)

"version": {
   "app_version": "1"
}

Update https://github.com/celestiaorg/networks/blob/master/mocha-4/genesis_hash.txt
Update the checksum for the download-genesis command so it works with the modified genesis.

is it possible to modify the genesis and sync an existing network from scratch? if so, is that a bug? my understanding of the genesis is that its the first block. If that's committed to then changing it without recomputing all of the headers is not possible

As an alternative to this: can celestia-app interpret an InitGenesis invocation on a genesis file with an unpopulated app version as 1?
Did this only surface for DevOps maintained Mocha nodes because it was a regression in v2.x.x?


SkinsRestorer causes malformed reading - Incompatible

SkinsRestorer problem == Viaversion please fix this.
https://paste.ee/p/SIWLF

Consider filling out the issue template it's there for a reason.

http://hastebin.com/ejineqiheg.json (Now removed SkinsRestorer)

No

There seems to be a problem like a packet limit.

https://paste.ee/p/SIWLF

I look forward to your thoughts on the subject.

Fixed problem.

@craftrise
Are you able to provide details to others who might be experiencing the same issue?
Or was it another plugin kind of thing?


Add the jetpack compose branch

add compose branch
change UI to jetpack compose
add osmdroid mapview with compose

merge


Improve Doc - Add to endpoint which can handle interception

Is it possible to add to the document which end-points can handle SSL interception?
For security reasons, proxy servers perform SSL interception to ensure no malicious things are in the https/ssl package. Endpoints which can't handle this, and service fail because of this, need to be excluded from this kind of interception.
If this kind of info would be available, it would make life easier for the security admin who is managing the proxy settings.
It would also ensure such security measures will not break Microsoft Services crucial for the good working of the OS.

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

ID: 6b16b4c9-06ac-6c47-3a02-580d9b7da7da
Version Independent ID: 958f6888-4ddf-4fec-7fc7-084f0289d79e
Content: Connection endpoints for Windows 10 Enterprise, version 2004 - Windows Privacy
Content Source: windows/privacy/manage-windows-2004-endpoints.md
Product: w10
Technology: windows
GitHub Login: @linque1
Microsoft Alias: obezeajo

@linque1 can you help with this one please?

Let me work through the feasibility of this and get back to this in 2 weeks. I missed this while I was temporarily handing over this page. Thanks @lindspea

Just a note that we're still looking at the feasibility of this @MVDB1984

Doing repo cleanup, following up on this


copy the etcd CA and client cert/key pair from etcd operator

We no longer manage the etcd client cert/key pairs, so instead we copy them from kube-system
/assign @sanchezl @sttts

/retest

/retest

/retest

/retest


Fix ember-data 5.0 warning, use local copy of errorsHashToArray

Building an application with ember-data 5 and this addon results in this warning;
WARNING in ../rewritten-packages/active-model-adapter.621d47ca/active-model-adapter.js 172:21-38
export 'errorsHashToArray' (imported as 'errorsHashToArray') was not found in '@ember-data/adapter/error' (possible exports: AbortError, ConflictError, ForbiddenError, InvalidError, NotFoundError, ServerError, TimeoutError, UnauthorizedError, default)
 @ ../rewritten-packages/active-model-adapter.621d47ca/index.js 1:0-56 3:15-33 4:0-53
 @ ./assignment/adapter.js 7:0-54 40:66-84
 @ ./tests/co-located/assignment/adapter-test.js 3:0-75 14:16-35 25:16-35 36:16-35 47:16-35
 @ ./assets/test.js 193:13-85

As mentioned in the deprecation, errorsHashToArray is dropped in ember-data 5
https://deprecations.emberjs.com/ember-data/v4.x/#toc_ember-data-deprecate-errors-hash-to-array-helper
The recommendation is to replace it with a local copy

Users making use of these (already private) utilities can trivially copy them into their own codebase to continue using them

cc @mfeckie

@Techn1x it works for me, thank you.
@jherdman
Can you merge this?


cannot find the any data in the mongodb instance

Even after successful user account registration, booking appointments, login, in my mongodb dashboard, the collection is empty.

did u get an soln to this ..unfortunately I am also facing the same issue @justkelvin ???

hi @justkelvin  ,
was facing the same issue..the problem lies in the fetch operation in register.jsx in pages folder ..it is an asynchronous operation and hence use await before fetch(..){} ..it will resolve the issue for you !!


Make the DATA_SOURCE_URI environment variable a real URI

The DATA_SOURCE_URI environment variable is not really an URI as seen in:
https://github.com/wrouesnel/postgres_exporter/blob/6e3d130f40423ef7a55bcc9f971fce79e40e4ff2/postgres_exporter.go#L1033-L1034
It's part of the URI. What do you think in making it really an uri? Like postgresql://postgres@localhost:5432?sslmode=disable, then later parse it to set the user and password if the corresponding environment variable exists (that would also prevent uri escaping issues that exist with the current code, as it simply concatenates the user and pass without proper escaping).
BTW, why was there a need to introduce a new environment variable? Why not reuse the DATA_SOURCE_NAME? This seems like a compatible change.

I'm not sure what functional improvements this issue would bring? I'm hesitant to change configuration behavior like this at this point, since backwards compatibility does matter somewhat.

None. Its a matter of expectation, that is, when I have a variable with the URI word in its name I would assume its an URI/URL; but in this case its not really an URI. So its strange. Anyways, functionally, DATA_SOURCE_NAME works fine for me.

I came here for the same reason...DATA_SOURCE_URI isn't a URI. It's apparently just a hostname. If  don't want to change the code to accept an actual URI, and you want to keep the existing name for consistency, perhaps update the docs to make this clear?


ssss: 0.5 -> 0.5.7

Motivation for this change
Update to latest upstream release 0.5.7
Change log: https://github.com/MrJoy/ssss/blob/master/HISTORY.md#v057-december-2020
Now using a maintained fork.
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)
[ ] 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 (or backporting 21.05 Relase 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 130386 run on x86_64-linux 1

1 package built:

ssss

Result of nixpkgs-review pr 130386 at 5a13a8fe run on x86_64-linux 1

1 package built successfully:

ssss

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/tools/security/ssss/default.nix:43:5:
   |
43 |     maintainers = with maintainers; [ ];
   |     ^


Exception - OccurrenceConstraintViolationError doc string correction

https://github.com/mobilityhouse/ocpp/pull/509


Documentation: details on new features

The project is growing fast, much more packages, API calls and keys are being added. We may need to keep updating necessary tech details so that others will not get confused.

Two points on my commits:

The Celsius/Fahrenheit toggle uses package 'react-toggle'
For the header (navbar) I use the CSS grid system. The navbar is split into ten equal-width columns, the logo stays at 5th and 6th, the toggle stays at 8th. This can be found at src/assets/css/App.css

I also noted that the autocomplete feature requires package 'react-search-autocomplete' and an api key from https://developer.here.com/ . The API key is stored in env variable REACT_APP_HEREAPI.


No controls defined when running locally

I get this error when running this example locally with devcontainer and click on inverter while simulating

Hi Rafael, the inverter does not define any controls, so this is an expected behavior.

ok thanks.  Maybe not showing anything would be better,  the message looks like an error not an absence of features.


[Bug]: Cancel modification in configuration center without secondary confirmation

ODC version
odc424
OB version

What happened?
https://github.com/oceanbase/odc/assets/143492234/729d3f9e-a475-49ca-b346-92a30b0755b3
What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?

Modify Configuration Center
Click to cancel

Anything else we need to know?
No response
Cloud
No response

Fixed In Commit b02e6b6ad04d979bd9631bc30a97ac70a3010310
------
xiaokang
Thu Mar 7 15:02:34 2024 +0800

pass


Add 'Museo' Font

Get the museo font and add to the gem
https://typekit.com/fonts/museo

Closing and adding to https://github.com/on-site/gratefulgarment-ui


Update swift version (up to 5.0)

Update swift version
Fix a crash

Hi @EugeneKuropatenko ,
I have used your updated files for swift5.0
And I'm facing an issue on iPhone11ProMax, as pullToRefresh is looping and I am unable to stop the refresh.
can you please help me?
Thanks,
Saurabh Prajapati


Remove ipvs-as-sink & set ipvsfullstate as default ipvs backend

What kind of PR is this?
/kind cleanup

Why this PR is needed / What this PR do?

Which issue(s) this PR fixes?
Fixes #
Additional information about this PR

/lgtm :-)

/lgtm


Update specs2-core, specs2-scalacheck to 4.10.0

Updates

org.specs2:specs2-core
org.specs2:specs2-scalacheck

from 4.9.4 to 4.10.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update, semver-minor

Superseded by #315.

Superseded by #315.


No windows downloads available for 1.3.1

Etcher version: 1.3.1
Operating system and architecture: Windows 10 - x64

There is no download links available for 1.3.1 on Windows.

@InuyashaITB thanks for bringing this up so quickly; our build pipeline appears to have caught a bug and decided to meddle with our releases; I've fixed the v1.3.1 downloads now – will keep this issue open until the underlying issue is resolved.

Having the same problem with MacOS as well, just a heads up! @jhermsmeier

@MttArmstrong everything is back to normal as far as we can see; are you still having issues downloading it?

Closing, as the underlying cause has been tended to


[5.3][CodeCompletion] Avoid suggesting duplicated module names

Cherry-pick of #31892 into release/5.3

Explanation: Overlay modules have the same name as the shadowed modules. For global symbol completions, we should not list both names because they are identical. Maintain a set of seen module names to avoid suggesting duplicated names.
Scope: Code completion for global symbols
Risk: Very low. The change is pretty simple.
Issue: rdar://problem/63370253
Testing: Added regression test cases
Reviewers: Nathan Hawes (@nathawes)

@swift-ci Please test

@swift-ci Please nominate