Chore: Added web-resources-project in opensource projects

Fixes Issue
Closes #575
Changes proposed
The project aims to provide developers with a centralized platform to access a wide range of frontend tools and resources, making their development process more efficient and productive.
Screenshots

Improved-description

Note to reviewers
Any kind of feedback would be appreciated.

@parma1111 are you a gssoc contributor?

@parma1111 are you a gssoc contributor?

NO, @rupali-codes

Well done @parma1111! You've made great progress since the last PR. Well done! 😊LGTM!

Thank you @CBID2 , I have learnt a lot of things from you.😀😀

when I push the code
1st way
I make the branch
git add .
git commit -m "message"
git push origin branch name
2nd way
I don't make branch
git add .
git commit -m "message"
git push origin old branch name
3rd way
When I get the error "git pull.... or fetch"
git pull
git merge origin/branch_name
git branch --set-upstream-to=origin/branch_name  branch_name
git push
So there is any mistake in pushing way or it takes time to merge the PR without any reason.

when I push the code
1st way
I make the branch
git add .
git commit -m "message"
git push origin branch name
2nd way
I don't make branch
git add .
git commit -m "message"
git push origin old branch name
3rd way
When I get the error "git pull.... or fetch"
git pull
git merge origin/branch_name
git branch --set-upstream-to=origin/branch_name  branch_name
git push
So there is any mistake in pushing way or it takes time to merge the PR without any reason. @rupali-codes @CBID2

Probably the latter

@parma1111 please DON'T use git add . it would you take you just 2 mins extra, but add files manually by using git add file1_pat file2_path

@parma1111 please DON'T use git add . it would you take you just 2 mins extra, but add files manually by using git add file1_pat file2_path

@rupali-codes may you explain it clearly and merge it also ("git add file1_pat file2_path"). I could not understand it.

Which commands are you using for staging your changes? is it git add .?

@parma1111 please DON'T use git add . it would you take you just 2 mins extra, but add files manually by using git add file1_pat file2_path

@rupali-codes may you explain it clearly and merge it also ("git add file1_pat file2_path"). I could not understand it.

Which commands are you using for staging your changes? is it git add .?

Yes


Add new created tables into analytics sources

circle , article_circle, circle_price, circle_subscription, circle_subscription_item, action_circle , circle_invoice

Close it due to update to matters-etl.


Make the sniffer-paths configurable

Hello,
i would suggest, you make the paths in src/Nos/Yves/TwigCodeSniffer/TwigCodeSnifferConfig.php configurable via a config or use the current KernelConstants::PROJECT_NAMESPACES

@svommer Thanks for your feature request. This tool now uses all configured project namespaces in order to generate the appropriate theme paths. These changes are already part of the newest release -> https://github.com/project-nos/spryker-twig-code-sniffer/releases/tag/v1.1.0


Requests support priority and timeout settings

In our online serving, some requests need a higher priority than others. For example, online request should have higher priority than offline request. Of course, the more priorities the better. This may be solved by either client with priority or request with priority.
It would be nice to have a request timeout parameter to achieve traffic control. For example, discard the request if its waiting time in queue exceeds the setting. This is especially important to avoid request backlog when serving large models.

Marking as enhancement request. We've had the "request timeout" feature requested before so we will look into prioritizing it.

I've just diagnosed a production incident where some combination of request deadlines (item 2 above) and queue control from https://github.com/NVIDIA/tensorrt-inference-server/issues/914 would have helped.
Specifically...
We have a separate TRTIS cluster behind an AWS load balancer servicing our "slow" models.  Each slow query can take a few seconds and pegs the GPU at 100%.  It looks like a burst of incoming calls caused requests to stack up and the TRTIS tasks ultimately hit out-of-memory.
Having a maximum queue size and a TRTIS per-request deadline would have helped mitigate the overload condition and prevented the OOMs too.

We're currently implementing a couple of "queuing policy" features and we will include priority, timeout and queue size.


Added item click listener and updated gradle version

I have added item click listener to the adapter. Included the viewholder within the adapter for this purpose. Also updated the sdk and gradle version

please, don't add .idea directory nor *.iml files to Git. They're not needed for Android projects.

Android Studio does that automatically. btw it doesn't hurt anything right?


JSON load failure on "deis apps:run 世界"

Many test runs are now failing with a JSON error trying to parse the results of docker inspect over SSH:
INFO appssample: test runs 'echo Hello, 世界'
ERROR appssample: appssample_v2.run.1 (run): Expecting object: line 40 column 28 (char 1077)
ERROR Internal Server Error: /v1/apps/appssample/run
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", line 112, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python2.7/site-packages/django/views/decorators/csrf.py", line 57, in wrapped_view
    return view_func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/rest_framework/viewsets.py", line 85, in view
    return self.dispatch(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/rest_framework/views.py", line 407, in dispatch
    response = self.handle_exception(exc)
  File "/usr/lib/python2.7/site-packages/rest_framework/views.py", line 404, in dispatch
    response = handler(request, *args, **kwargs)
  File "/app/api/views.py", line 205, in run
    output_and_rc = app.run(self.request.user, request.data['command'])
  File "/app/api/models.py", line 412, in run
    return c.run(escaped_command)
  File "/app/api/models.py", line 546, in run
    rc, output = self._scheduler.run(job_id, image, entrypoint, command)
  File "/app/scheduler/fleet.py", line 328, in run
    container = json.loads(_output)
  File "/usr/lib/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
    obj, end = self.scan_once(s, idx)
ValueError: Expecting object: line 40 column 28 (char 1077)
<IP_ADDRESS> "POST /v1/apps/appssample/run HTTP/1.0" 500 27 "python-requests/2.5.1 CPython/2.7.6 Linux/3.16.0-41-generic"

Here is one example.

Here is a log with the raw output of what was trying to be decoded:
INFO configsample: release configsample-v4 created
INFO configsample: test scaled containers cmd=1
<IP_ADDRESS> "POST /v1/hooks/build HTTP/1.1" 200 79 "deis-builder"
INFO configsample: test runs 'env'
ERROR configsample: configsample_v4.run.1 (run): [{
    "AppArmorProfile": "",
    "Args": [
        "-c",
        "env"
    ],
    "Config": {
        "AttachStderr": true,
        "AttachStdin": false,
        "AttachStdout": true,
        "Cmd": [
            "-c",
            "env"
        ],
        "CpuShares": 0,
        "Cpuset": "",
        "Domainname": "",
        "Entrypoint": [
            "/bin/bash"
        ],
        "Env": [
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
            "POWERED_BY=the Deis team",
            "GIT_SHA=ab3e88a6adfbd84324f74981a2c17f137f7efaea",
            "DEIS_APP=configsample",
            "DEIS_RELEASE=v4",
            "BUILDPACK_URL=https://github.com/heroku/heroku-buildpack-go#6eeb09f"
        ],
        "ExposedPorts": {
            "1500/tcp": {}
        },
        "Hostname": "1a7679c7389e",
        "Image": "<IP_ADDRESS>:5000/configsample:v4",
        "Labels": {},
        "MacAddress": "",
        "Memory": 0,

ERROR Internal Server Error: /v1/apps/configsample/run
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", line 112, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python2.7/site-packages/django/views/decorators/csrf.py", line 57, in wrapped_view
    return view_func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/rest_framework/viewsets.py", line 85, in view
    return self.dispatch(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/rest_framework/views.py", line 407, in dispatch
    response = self.handle_exception(exc)
  File "/usr/lib/python2.7/site-packages/rest_framework/views.py", line 404, in dispatch
    response = handler(request, *args, **kwargs)
  File "/app/api/views.py", line 205, in run
    output_and_rc = app.run(self.request.user, request.data['command'])
  File "/app/api/models.py", line 412, in run
    return c.run(escaped_command)
  File "/app/api/models.py", line 546, in run
    rc, output = self._scheduler.run(job_id, image, entrypoint, command)
  File "/app/scheduler/fleet.py", line 337, in run
    raise ValueError(_output)
ValueError: [{
    "AppArmorProfile": "",
    "Args": [
        "-c",
        "env"
    ],
    "Config": {
        "AttachStderr": true,
        "AttachStdin": false,
        "AttachStdout": true,
        "Cmd": [
            "-c",
            "env"
        ],
        "CpuShares": 0,
        "Cpuset": "",
        "Domainname": "",
        "Entrypoint": [
            "/bin/bash"
        ],
        "Env": [
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
            "POWERED_BY=the Deis team",
            "GIT_SHA=ab3e88a6adfbd84324f74981a2c17f137f7efaea",
            "DEIS_APP=configsample",
            "DEIS_RELEASE=v4",
            "BUILDPACK_URL=https://github.com/heroku/heroku-buildpack-go#6eeb09f"
        ],
        "ExposedPorts": {
            "1500/tcp": {}
        },
        "Hostname": "1a7679c7389e",
        "Image": "<IP_ADDRESS>:5000/configsample:v4",
        "Labels": {},
        "MacAddress": "",
        "Memory": 0,

<IP_ADDRESS> "POST /v1/apps/configsample/run HTTP/1.0" 500 27 "python-requests/2.5.1 CPython/2.7.6 Linux/3.16.0-41-generic"

That certainly looks like a premature exit

Also relevant is possibly #3403, although that did and does seem like it fixed things rather than broke them. :confounded:

that was the only relevant change I saw, but I agree that should've fixed things.


fix(helm): fix user value

Type of change:

Fix user value in charts/apache-shardingsphere-operator-charts/values.yaml and examples/operator/shardingsphereproxy-mysql-cluster.yaml
Make sure ComputeNode's default user is root

[x] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?

approve


Trim leading/trailing spaces or stars when parsing a JMH output

The following two examples fail to parse because of the leading spaces:
        Benchmark                   (crossCL)    (size)  Mode  Cnt      Score      Error  Units
        Misaligned.read_aligned         false      4096  avgt   25     8.755 Â±     0.086  us/op

or because it is part of a javadoc or comment:
    /*
     * Benchmark                   (crossCL)    (size)  Mode  Cnt       Score      Error  Units
     * Misaligned.read_aligned         false      4096  avgt   25      8.755 Â±     0.086  us/op
     */

Fixed in 1.0.5.


[RESTEASY-2891] Do not depend on Yasson for the Jakarta EE JSON Bindi…

…ng implementation. Also ensure the Jsonb type is closed after invocation.
https://issues.redhat.com/browse/RESTEASY-2891
One thing I'm not sure how to note is that now it would be required for a user to supply a JSON-P and JSON Binding  implementation. For containers like WildFly this is not an issue.

Replaced by https://github.com/resteasy/resteasy/pull/2978


cookies SameSite header

Hi! I published allure UI and service to heroku as separate docker containers app. Both has different URL. I specified ALLURE_DOCKER_PUBLIC_API_URL to service app. But can not login. Chrome says, that do not saves any cookies, and suggest, that Set-cookie header should have SameSite=None to allowed me have ui and service on different urls. How can i set that cookie by my self or why nobody else have this essue?

Hi @brizjin
By security that by default requires to use the same domain for api/ui. I've tried to do that configuration optional, but there is an issue with Flask. That config comes from the API, not from the UI. I haven't checked again.


add text file

@ArchanaMenon Good job

Hey @ArchanaMenon can you delete this file and make new directory for vtu-usp-lab or vtu-cg-lab ?


Miq fix

Fixes #71

How do we use this Alex?
Any exe coming soon?

Still testing this out, once its merged in and this PR is closed a new release will be issued.


Text To Bin with Fieldcaption = "NULL"

ℹ Computer information

VFP version: aktuell
FoxBin2PRG version: aktuell
Last FoxBin2PRG version w/o bug (if available):

📝 Provide detailed reproduction steps (if any)

I have been using XCASE from Resolution for years to create and maintain databases. For some time now, I have also been versioning the respective models together with my projects.
I have noticed that field contents are not restored if the field was named "NULL".

TEST:
CREATE Table tblMyTest ( MyName c(10), MyField l, Null l)
INSERT INTO tblMyTest Values( "Row 1", .T., .T. )
INSERT INTO tblMyTest Values( "Row 2", .F., .F. )
INSERT INTO tblMyTest Values( "Row 3", .T., .T. )
INSERT INTO tblMyTest Values( "Row 4", .F., .F. )

Table original

After Restore via Text to Bin

Best Regards
Ulf Neugebauer

Referencing the field name alone will of course fail,
but via table alias e.g. tblMytest.NULL actually returns the correct content.
But I have searched a little further, the error already appears when creating the *.db2 file,
If I correct the file, the restore works.
Best Regards
Ulf Neugebauer

@griessbach14943
Ooops. Somehow I missed the issue. There was a lot of noise in GoFish spamming me, so I fear I ignored the VFPX section.
I will look at the problem later this week. Would you mind to send me an example dbf?
It's always a problem to use internal designators as user defined designators. Not that VFP give a good example. Anyway, I like to discourage everybody.
The special handler needed sadly will slow down FoxBin2Prg.

TableTest.zip

@griessbach14943 Thank you. Is closing this issue intended?

@griessbach14943 I recommend that you always use the latest version of FoxBin2Prg. Best via Thor. Anyway, the problem exist in recent version 1.21.02
Analysing the problem:
Basically, the problem is on how VFP looks up fields, variables and build in literals. One must understand, that NULL is identical to .NULL. (with dots).
To determine the value of a field to store with the db2 file, FoxBin2Prg uses:
luValue = Evaluate(lcField)

this is for your table like
lcField = "NULL"
luValue = Evaluate(lcField)

and luValue is null after this.
So VFP first looks up the build in literal, then the field, then the variable.
To completely understand, try the following:
use tblMyTest.dbf alias tblMyTest
NULL = 'A'
?NULL
?m.NULL
?tblMyTest.NULL
?!NULL

SELECT null FROM tblMyTest

Again, this is a problem of using Reserved Words (see help for Reserved Words in VFP help). So, be not surprised if your field returns unexpected results.
The problem for FoxBin2Prg is simple enough to solve by just adding the alias.

@griessbach14943 Sorry, but just to be picky:
The problem is not on a fields caption (this is a fields property for a DBC bound table), its about a fields name.

@griessbach14943 New version is available here and via Thor.

Super Job, jetzt klappt es auch mit xCase, ich denke das was DIE Ausnahme von der Regel!
Danke!

Immer gerne.
NULL ist halt der eine Bezeichner, der in VFP auch als Konstante geht. Zum Glück war es weiter kein lauf-zeitlicher Aufwand, das zu flicken.


docs: Install langsmith from conda-forge

langsmith is available on conda-forge as well and also a dependency of the package so it gets installed either way by conda https://github.com/conda-forge/langchain-feedstock/blob/306ed13308c051716ced37a28962a0ac80a9aa7f/recipe/meta.yaml#L43

thanks!


TypeScript, react-native, styles

Environment

npx envinfo --system --binaries --npmPackages styled-components,babel-plugin-styled-components --markdown --clipboard
System:

OS: macOS 11.0.1
CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
Memory: 745.73 MB / 16.00 GB
Shell: 5.8 - /bin/zsh

Binaries:

Node: 12.18.4 - ~/.nvm/versions/node/v12.18.4/bin/node
Yarn: 1.10.1 - /usr/local/bin/yarn
npm: 6.14.6 - ~/.nvm/versions/node/v12.18.4/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman

npmPackages:

styled-components: ^5.1.1 => 5.2.0

styled-components "version": "5.1.4",
Reproduction
import React from 'react'
import { View, ViewStyle } from 'react-native'
import styled from 'styled-components/native'

const TestComp = styled.View({})

export const Image: React.FC<{
  style: ViewStyle
}> = ({ style }) => {
  return (
    <View style={style}> {/* OK */}
      <TestComp style={style} /> {/* TS Error */}
    </View>
  )
}

Expected Behavior
Types should match.
Actual Behavior
(JSX attribute) style?: StyleProp<ViewStyle>
No overload matches this call.

  Overload 1 of 2, '(props: Pick<Pick<ViewProps & RefAttributes<View>, "ref" | "style" | "hitSlop" | "onLayout" | "pointerEvents" | "removeClippedSubviews" | "testID" | ... 48 more ... | "key"> & Partial<...>, "ref" | ... 54 more ... | "key"> & { ...; } & { ...; } & { ...; }): ReactElement<...>', gave the following error.
    Type 'ViewStyle' is not assignable to type 'StyleProp<ViewStyle>'.
      Type 'import("./node_modules/@types/react-native/index").ViewStyle' is not assignable to type 'import("./node_modules/@types/styled-components/node_modules/@types/react-native/index").ViewStyle'.
        Types of property 'backgroundColor' are incompatible.
          Type 'string | typeof import("./node_modules/@types/react-native/index").OpaqueColorValue | undefined' is not assignable to type 'string | typeof import("./node_modules/@types/styled-components/node_modules/@types/react-native/index").OpaqueColorValue | undefined'.
            Type 'unique symbol' is not assignable to type 'string | unique symbol | undefined'.

  Overload 2 of 2, '(props: StyledComponentPropsWithAs<typeof View, DefaultTheme, {}, never>): ReactElement<StyledComponentPropsWithAs<typeof View, DefaultTheme, {}, never>, string | ... 1 more ... | (new (props: any) => Component<...>)>', gave the following error.
    Type 'ViewStyle' is not assignable to type 'StyleProp<ViewStyle>'.
      Type 'import("./node_modules/@types/react-native/index").ViewStyle' is not assignable to type 'import("./node_modules/@types/styled-components/node_modules/@types/react-native/index").ViewStyle'.ts(2769)

index.d.ts(2522, 5): The expected type comes from property 'style' which is declared here on type 'IntrinsicAttributes & Pick<Pick<ViewProps & RefAttributes<View>, "ref" | "style" | "hitSlop" | "onLayout" | "pointerEvents" | ... 50 more ... | "key"> & Partial<...>, "ref" | ... 54 more ... | "key"> & { ...; } & { ...; } & { ...; }'

index.d.ts(2522, 5): The expected type comes from property 'style' which is declared here on type 'IntrinsicAttributes & Pick<Pick<ViewProps & RefAttributes<View>, "ref" | "style" | "hitSlop" | "onLayout" | "pointerEvents" | ... 50 more ... | "key"> & Partial<...>, "ref" | ... 54 more ... | "key"> & { ...; } & { ...; } & { ...; }'

The origin of the issue is that there is separate @types/react-native module in styled-components
    "dependencies": {
        "@types/hoist-non-react-statics": "*",
        "@types/react": "*",
        "@types/react-native": "*",
        "csstype": "^3.0.2"
    },

Should @types be in peerDependencies?
PS, the issue is only with yarn.
npm does not add node_modules inside styled-components

Sorry, I'll have to close this.
We don't maintain any TypeScript typings in the main repository and are not updating the ones that live in definitely-typed, so we can't offer any help on this here. It's possible that if you browse past issues you'll find similar questions, but overall this repository's issues are not the right place to report these issues or to discuss them.

@kitten that's just stupid and a lot of people agree with me

@pt-br You're welcome to maintain them in repo.
The above actually looks like it's solved by deduplicating the @types/react-native package using yarn-deduplicate or npm dedupe on a second look over the error message.
This essentially brings me to the issue itself, we can even have a rewrite in TS and have perfect typings in a perfect world, but we'd still get issues that are essentially usage questions and we can't answer them all as is either. We simply don't have the bandwidth to do everything we'd love to do.
There are plenty of issues we get that are simply usage issues or questions and I sometimes try to answer as many of them as possible. But even with a TypeScript rewrite being on the horizon and a possibility to being the next major release, we'd still not have the bandwidth to answer questions and address all edge cases in our issues, so we have to decline some of them and set a line somewhere.
Now, on another note, I'd like to ask you to please have a read through the code of conduct and adhere to it going forward ✌ https://github.com/styled-components/styled-components/blob/main/CODE_OF_CONDUCT.md#our-standards


make test failed?

After I make successfully.
I do the "make all test". And it shows

92% tests passed, 42 tests failed out of 512

Total Test time (real) = 22.91 sec

The following tests FAILED:
63 - AlexaClientSDKInitTest.initializeNoJSONConfig (Failed)
65 - AlexaClientSDKInitTest.initializeValidJSONConfig (Failed)
67 - AlexaClientSDKInitTest.isInitialized (Failed)
217 - AVSConnectionManagerTest.createTest (Failed)
218 - AVSConnectionManagerTest.createWithNullMessageRouterAndObservers (Failed)
220 - AVSConnectionManagerTest.addConnectionStatusObserverValid (SEGFAULT)
224 - AVSConnectionManagerTest.enableAndDisableFunction (SEGFAULT)
225 - AVSConnectionManagerTest.sendMessageRequestTest (SEGFAULT)
226 - AVSConnectionManagerTest.setAVSEndpointTest (SEGFAULT)
243 - HTTP2StreamPoolTest.GetStreamSendNullPtrForMoreThanMaxStreams (Failed)
246 - HTTP2StreamPoolTest.PostStreamSendNullPtrForMoreThanMaxStreams (Failed)
247 - HTTP2StreamPoolTest.ReleaseStreamTestSendMoreThanMaxStreams (Failed)
248 - HTTP2StreamPoolTest.ReleaseStreamTestAfterNullTest (Failed)
249 - HTTP2StreamTest.testExceptionReceivedSingleWrite (Failed)
250 - HTTP2StreamTest.testExceptionReceivedMultiWrite (Failed)
251 - HTTP2StreamTest.testHeaderCallback (Failed)
252 - HTTP2StreamTest.testReadCallBack (Failed)
260 - AuthDelegateTest.createNullConfig (Failed)
261 - AuthDelegateTest.createMissingClientId (Failed)
262 - AuthDelegateTest.createMissingClientSecret (Failed)
263 - AuthDelegateTest.createMissingRefreshToken (Failed)
264 - AuthDelegateTest.create (Failed)
265 - AuthDelegateTest.addAuthObserverNull (Failed)
266 - AuthDelegateTest.removeAuthObserverNull (Failed)
267 - AuthDelegateTest.addMultipleAuthObserver (Failed)
268 - AuthDelegateTest.retry (Failed)
269 - AuthDelegateTest.expirationNotification (Failed)
270 - AuthDelegateTest.recoverAfterExpiration (Failed)
271 - AuthDelegateTest.unrecoverableErrorNotification (Failed)
480 - SettingsTest.createTest (SEGFAULT)
481 - SettingsTest.addGlobalSettingsObserverTest (SEGFAULT)
482 - SettingsTest.removeGlobalSettingsObserverTest (SEGFAULT)
483 - SettingsTest.addSingleSettingObserverWithInvalidKeyTest (SEGFAULT)
484 - SettingsTest.removeSingleSettingObserverWithInvalidKeyTest (SEGFAULT)
485 - SettingsTest.removeSingleSettingObserverWithCorrectKeyTest (SEGFAULT)
486 - SettingsTest.defaultSettingsCorrect (SEGFAULT)
487 - SettingsTest.clearDatabaseTest (SEGFAULT)
488 - SettingsTest.storeDatabaseTest (SEGFAULT)
489 - SettingsTest.modifyDatabaseTest (SEGFAULT)
490 - SettingsTest.eraseTest (SEGFAULT)
491 - SettingsTest.createDatabaseTest (SEGFAULT)
492 - SettingsTest.openAndCloseDatabaseTest (SEGFAULT)
Errors while running CTest
Makefile:127: recipe for target 'test' failed
make: *** [test] Error 8

What can I do to solve these error?
My cmake cmd is "cmake ../avs-device-sdk/  -DCMAKE_BUILD_TYPE=DEBUG"

Hi octGitHub,
Have you filled out your JSON config file AlexaClientSDKConfig.json?
https://github.com/alexa/avs-device-sdk/wiki/Linux-Quick-Start-Guide#build-for-generic-linux--macos
What platform are you running this on ? what version of the code are you using ?

Thx for reply!
    I pulled the latest code from github at 20171106.
    It was runed on linux for testing .
    My AlexaClientSDKConfig.json is out of source directory. I just filled authDelegate's fields of the JSON config file AlexaClientSDKConfig.json befroe I make test.  Should I fill all of the config fiels  if some function I never used?

{
"authDelegate":{
"clientSecret":"---myclientSecret---",
"deviceSerialNumber":"--mydeviceSerialNumber---",
"refreshToken":"",
"clientId":"---myclientId--",
"productId":"--myproductId--"
},
"alertsCapabilityAgent":{
"databaseFilePath":"",
"alarmSoundFilePath":"",
"alarmShortSoundFilePath":"",
"timerSoundFilePath":"",
"timerShortSoundFilePath":""
},
"settings":{
"databaseFilePath":"",
"defaultAVSClientSettings":{
"locale":"en-US"
}
},
"certifiedSender":{
"databaseFilePath":""
}
}
  BTW.
  If I use the lower version gcc 4.7.4 in arm linux ( qsdk ) , will it work?
  my linux compiler is gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.5).

Yes would recommend filling in the rest of the fields in the json file.

OK. thx.  I  will try.

@jjamazon
    I have tried filling  with JSON config file AlexaClientSDKConfig.json out of source.

    My json file is

{
    "authDelegate":{
     "clientSecret":"---myclientSecret---",
     "deviceSerialNumber":"--mydeviceSerialNumber---",
     "refreshToken":"",
     "clientId":"---myclientId--",
     "productId":"--myproductId--"
},

   "alertsCapabilityAgent":{
        "databaseFilePath":"/home/yummy/Desktop/2Test/res/alexa/alertsCapabilityAgent.db",
        "alarmSoundFilePath":"/home/yummy/Desktop/2Test/res/alexa/med_system_alerts_melodic_01_short._TTH_.wav",
        "alarmShortSoundFilePath":"/home/yummy/Desktop/2Test/res/alexa/med_system_alerts_melodic_01._TTH_.mp3",
        "timerSoundFilePath":"/home/yummy/Desktop/2Test/res/alexa/med_system_alerts_melodic_02._TTH_.mp3",
        "timerShortSoundFilePath":"/home/yummy/Desktop/2Test/res/alexa/med_system_alerts_melodic_02_short._TTH_.wav"
   },

   "settings":{
        "databaseFilePath":"/home/yummy/Desktop/2Test/res/alexa/settings.db",
        "defaultAVSClientSettings":{
            "locale":"en-US"
          }
    },
   "certifiedSender":{
        "databaseFilePath":"/home/yummy/Desktop/2Test/res/alexa/certifiedSender.db"
    }
 }

but it was still failed at testing

92% tests passed, 42 tests failed out of 512

Total Test time (real) =  21.80 sec

The following tests FAILED:
	 63 - AlexaClientSDKInitTest.initializeNoJSONConfig (Failed)
	 65 - AlexaClientSDKInitTest.initializeValidJSONConfig (Failed)
	 67 - AlexaClientSDKInitTest.isInitialized (Failed)
	217 - AVSConnectionManagerTest.createTest (Failed)
	218 - AVSConnectionManagerTest.createWithNullMessageRouterAndObservers (Failed)
	220 - AVSConnectionManagerTest.addConnectionStatusObserverValid (SEGFAULT)
	224 - AVSConnectionManagerTest.enableAndDisableFunction (SEGFAULT)
	225 - AVSConnectionManagerTest.sendMessageRequestTest (SEGFAULT)
	226 - AVSConnectionManagerTest.setAVSEndpointTest (SEGFAULT)
	243 - HTTP2StreamPoolTest.GetStreamSendNullPtrForMoreThanMaxStreams (Failed)
	246 - HTTP2StreamPoolTest.PostStreamSendNullPtrForMoreThanMaxStreams (Failed)
	247 - HTTP2StreamPoolTest.ReleaseStreamTestSendMoreThanMaxStreams (Failed)
	248 - HTTP2StreamPoolTest.ReleaseStreamTestAfterNullTest (Failed)
	249 - HTTP2StreamTest.testExceptionReceivedSingleWrite (Failed)
	250 - HTTP2StreamTest.testExceptionReceivedMultiWrite (Failed)
	251 - HTTP2StreamTest.testHeaderCallback (Failed)
	252 - HTTP2StreamTest.testReadCallBack (Failed)
	260 - AuthDelegateTest.createNullConfig (Failed)
	261 - AuthDelegateTest.createMissingClientId (Failed)
	262 - AuthDelegateTest.createMissingClientSecret (Failed)
	263 - AuthDelegateTest.createMissingRefreshToken (Failed)
	264 - AuthDelegateTest.create (Failed)
	265 - AuthDelegateTest.addAuthObserverNull (Failed)
	266 - AuthDelegateTest.removeAuthObserverNull (Failed)
	267 - AuthDelegateTest.addMultipleAuthObserver (Failed)
	268 - AuthDelegateTest.retry (Failed)
	269 - AuthDelegateTest.expirationNotification (Failed)
	270 - AuthDelegateTest.recoverAfterExpiration (Failed)
	271 - AuthDelegateTest.unrecoverableErrorNotification (Failed)
	480 - SettingsTest.createTest (SEGFAULT)
	481 - SettingsTest.addGlobalSettingsObserverTest (SEGFAULT)
	482 - SettingsTest.removeGlobalSettingsObserverTest (SEGFAULT)
	483 - SettingsTest.addSingleSettingObserverWithInvalidKeyTest (SEGFAULT)
	484 - SettingsTest.removeSingleSettingObserverWithInvalidKeyTest (SEGFAULT)
	485 - SettingsTest.removeSingleSettingObserverWithCorrectKeyTest (SEGFAULT)
	486 - SettingsTest.defaultSettingsCorrect (SEGFAULT)
	487 - SettingsTest.clearDatabaseTest (SEGFAULT)
	488 - SettingsTest.storeDatabaseTest (SEGFAULT)
	489 - SettingsTest.modifyDatabaseTest (SEGFAULT)
	490 - SettingsTest.eraseTest (SEGFAULT)
	491 - SettingsTest.createDatabaseTest (SEGFAULT)
	492 - SettingsTest.openAndCloseDatabaseTest (SEGFAULT)
Errors while running CTest
Makefile:127: recipe for target 'test' failed
make: *** [test] Error 8

    My cmake command is
    cmake ../avs-device-sdk/  -DCMAKE_BUILD_TYPE=DEBUG

    Is there any problem here?

The cmake command looks incomplete.
Take a look here for for Building SDK on Pi.
https://github.com/alexa/avs-device-sdk/wiki/Raspberry-Pi-Quick-Start-Guide#2-build-the-sdk

Thx for reply!
OK. thx. I will try.

I solved this make test failed now. It took me several times to try.
The key point is :
you should analyse the error report one by one.
2017-12-04 08:27:35.513 [  1] E AlexaClientSdkInit:initializeFailed:reason=curlDoesNotSupportHTTP2
I solved this by replace the curl  which not support  http2 .
Just mark here.  Hope it will help.

Hi octGitHub,
I am also got the same below error,
2017-12-04 08:27:35.513 [ 1] E AlexaClientSdkInit:initializeFailed:reason=curlDoesNotSupportHTTP2
How you replaced curl version and which version?
Note:
I'm using yocto image and there is no apt-get.

my replace curl version is  curl-7.57.0
wget  https://curl.haxx.se/download/curl-7.57.0.tar.gz
tar  -zvxf   curl-7.57.0.tar.gz   &&  cd  ./curl-7.57.0

./configure --with-nghttp2="path to your nghttp2 's library  or bin"   --with-ssl

FYI

Hi,
Thank you so much for the information.
I built the nghttp2 and called in curl configuration like below,
./configure --host=x86_64 --target=arm-linux --with-nghttp2=/home/thiru/build-avs-device-sdk-Pico_Kit_cmake/Packages/nghttp2/buildNghttp2/lib
But i' getting below error,
error: --with-nghttp2 was specified but could not find libnghttp2 pkg-config file.
Did you faced any issues like this?
if you have any idea kindly help me to solve this issue.

When you are building nghttp2 , try to use it's cmake
  cmake ../    -DCMAKE_INSTALL_PREFIX="path to install lib and bin "

After you make success, you may see its pkconfig file
    /pkgconfig/libnghttp2.pc

Hi octGitHub,
Thank you so much for the help.
Everything working fine.

with  pleasure


Typescript definitions

So, I've been using styled system with typescript and styled-components with this definition file. It was sourced originally from https://github.com/jxnblk/styled-system/issues/169 and extended to support all of the different functions.
I don't think it's perfect, but it's been working really well for me - just interested to see what everyone thinks. This is my first time contributing to something, I just love this project 👍

Codecov Report

Merging #181 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #181   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         203    203
=====================================
  Hits          203    203

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 6c6c202...58eb2aa. Read the comment docs.

Looks like styled-components will be moving definitions out of the repo to DefinitelyTyped in v4, maybe we should add these to DT too instead?

I don't think that's a bad idea at all, probably makes sense to maintain things there as well

Thanks, but as I've discussed in many, many other issues, is there any reason to not host this in DefinitelyTyped?
I don't use TS and won't be likely to keep this maintained over time

That's all good - no reason at all why this can't live in DT. I'll make a PR there 👍

@phobon I've started a PR @ https://github.com/DefinitelyTyped/DefinitelyTyped/pull/26649. Do you want to be a maintainer on it? I've got you listed as an author currently.

@zephraph Sorry about the late reply - thanks for setting that all up - looks really good! More than happy to be a maintainer 👍


GeoPolitical Update: Fix for ConformanceRemoved into master branch

Description
Conformance folder has been removed so excluding files from that folder for GeoPol now failing --> Removing excludes.
Related issues
Addresses [issue #].
Testing
Describe how this change was tested.

What is the best way to test that this change fixed the issue? Can we run GeoPol locally?

Check out aka.ms/NExTGeoPol @LTA-Thinking for how the automated GeoPol process works - No option at this stage to run locally but working on a self serve model currently.


Task 1 Using UNet to Segment the ISIC 2017/8 dataset Raul Movila S4703527

Hello,
The pull request is for task 1 of the Pattern Analysis 2023 report
The mean val_dice_coef after training resulted to 0.83335.
The pull request contains the following files:
1. `dataset.py`: # Loads and pre-processes the dataset, including the training, validation, and testing sets.
2. `model.py`: # Defines the UNet architecture, including encoder and decoder blocks, Return the model.
3. `train.py`: # Implements the training procedure for the model it trains the model then saves the best model.
4. `predict.py`: # Predict mask based on saved UNet model, performing predictions, and calculating evaluation metrics.
5. `utils.py`: # Provides utility functions used throughout the project
6. 'files' : # Stores all the data from training
7. 'results' : # Stores all the resulted masks

Refer to README for further information.
Thank you,
Raul Andrei Movila S4703527

Well organized files.
Algorithm solves the problem appropriately. Average Dice Coefficient in testing dataset: 0.83. Predicted Mask output looks good.
Try to use TensorFlow to construct the Improved U-Net model, but the model body is a U-Net model(filters start with 64 and end up with 1024) instead of the improved U-Net one(filters start with 16 and end up with 256). No data augmentation.
Sufficient docstring and comment in the code, but need some references.
Easy difficulty problem, no higher difficulty marks.

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation, but not correct model, not improved UNet -4
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Easy -10
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe acceptable, but could have more info -1
Model/technical explanation minimal -1
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
Feedback required, remove the results data images, repos should not have data in it -2
Request Description good

Feedback marks possible +2 if the requested changes are made (see above).

No feedback attempt and no feedback marks granted.

Feedback Implemented (removed data from repo).

Feedback attempt and feedback marks granted +2.

Unfortunately will not be merged because once data files are added, its in the repo forever (even if removed, its in history). Only affects merge though, grading is complete, no marks will be gained or lost. Feedback marks already gained as commented above.


⚠ gorzdrav.org has degraded performance

In ee3847e, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 1666 ms

Resolved: gorzdrav.org performance has improved in 1144c4c after 5 minutes.


Handle texture atlas resize

Handle texture atlas resize when texture size is reached.

[ ] find reasonable raster ratio for retina screens (adjusting with device pixel ratio would be ok until we find atlas size bottleneck on it)
[x] fix degenerate UVs, or use a backup texture atlas

Reviewing

Closing in favor of #356


chore(sequencer): more detailed decrease balance error statement

Summary
Added more detailed error statement to accounts::StateWriteExt::decrease_balance().
Background
Error was previously quite non-descriptive, found this change useful when testing for breaking changes to fees.
Changes

Added subtraction and balance amounts to subtraction error in accounts::StateWriteExt::decrease_balance().

Related Issues
closes #1529

This is actually not necessary. With your recent changes this information will now be part of the emitted error event.


Remove userland proxy

This builds up on @phemmer work and comments (#6810, libcontainer's #62).
A few notes on the implementation:

A new --route-localnet daemon flag was added to control the net.ipv4.route_localnet flag: it is very succinctly documented, so maybe one of you will want to come up with a nice explanation paragraph for docs/sources/articles/networking.md.
The new iptable POSTROUTING rule is not depending on any command line flag (I don't believe it has to be).
Integration tests cover both container -> container communication through the host IP and host -> container communication through localhost.

Hi guys! The userland proxy is really weird and I'd like to see an option to disable it. Are there some plans about merging this patch? Or do you have another approach?

@ikalnitsky We too, but hairpin nat seems not working on CentOS :(

@LK4D4 what about doing both the nat and the UL proxy, or was it decided this was a bad idea?

@pwaller maybe we can introduce --disable-ul-proxy

Clarification, hairpin does not work on CentOS/RHEL 6 (issue with older kernels). 7 should be fine.
@pwaller we do use both. the userland proxy is for when the iptables rules don't match.

Ah, I had missed the latter, thanks @phemmer.

Moreover, guys, perhaps it doesn't related to this thread, but I see strange behaviour on my system.
I have two containers. Both have ports bound to the host. I thought, that connectivity between these two containers through host would be performed by NAT, but actually it's performed by userland proxy. See iptables rules:
Chain DOCKER (2 references)
 pkts bytes target     prot opt in     out     source               destination
    0     0 DNAT       tcp  --  !docker0 any     anywhere             anywhere             tcp dpt:8888 to:<IP_ADDRESS>:8888
    0     0 DNAT       tcp  --  !docker0 any     anywhere             anywhere             tcp dpt:9999 to:<IP_ADDRESS>:9999

Does anybody know why we have !docker0 in the in field? Is it intended behaviour? I expected to see any here.

@ikalnitsky Yes, this is the hairpin nat issue. See #6810 and #9269


Indicator Request: Relative Moving Average

Expected Behavior
Supports: Relative Moving Average (RMA)
Actual Behavior
Indicator not yet supported.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

Hey @Nkonovalenko! it's yours to tackle 🙌
Can find examples of what is expected of an indicator PR at https://github.com/QuantConnect/Lean/pulls?q=is%3Apr+indicator+is%3Aclosed , for example Super Trend Indicator, Beta indicator, Kaufman Efficiency Ratio, etc

@Martin-Molinero Great thank you! I'm ready to push my commit to a branch, however I'm getting an error that I don't have write access in order to publish my branch.

Hey @Nkonovalenko! nice, you should create your own fork of lean and push a branch to your repository, from their you can open a PR against the main Lean repo 👍


refactor(metrics): unit tests for Metrics are written to follow similar convention as Logger/Tracer

Description of your changes
Metrics unit tests were not following the same convention as Logger/Tracer package. In Logger/Tracer tests, only public functions are unit tested. But for Metrics, unit tests are written per feature/behavior.
This PR updates the tests to match the same convention as Logger/Tracer package. Only public methods are tested, following the recommendations here: Best Practices
100% test coverage is maintained. Some additional housekeeping is implemented, constants & helpers are introduced for better code maintainability.
I've removed the previous feature tests because in my understanding leaving them will create further confusion about the testing patterns. I can add them again if maintainers feel the current sets of tests in this PR are inadequate.

How to verify this change
All the tests will run successfully along with 100% test coverage.

Related issues, RFCs

Issue number:
#163
Checklist

[x] My changes meet the tenets criteria
[x] I have performed a self-review of my own code
[x] I have commented my code where necessary, particularly in areas that should be flagged with a TODO, or hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have made corresponding changes to the examples
[x] My changes generate no new warnings
[x] The code coverage hasn't decreased
[x] I have added tests that prove my change is effective and works
[x] New and existing unit tests pass locally and in Github Actions
[x] Any dependent changes have been merged and published
[x] The PR title follows the conventional commit semantics

Breaking change checklist
Is it a breaking change?: NO

[ ] I have documented the migration process
[ ] I have added, implemented necessary warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Hi @arnabrahman thank you so much for opening this PR!
Given the size, please allow us some time to go through it and review it.
I'll get back to you here with comments as I review it.

Happy to help


Move preview to separate lambda/serverless/other

We currently do all previewing in a puppeteer instance off of the main server in https://github.com/exokitxr/exokit-backend/blob/master/routes/preview.js
This just executes https://github.com/webaverse/app/blob/master/screenshot.html to get the data out of puppeteer.
However it is prone to crashes, is not isolated from the rest of the app, and is not parallelized.
We should rewrite this to happen in a separate, possibly serverless solution. Additionally, we should trigger preview on mint and cache that result, not on request time. Note that we already have caching (in S3) on request time.

Done.

Done.


How to insert struct with an embedded array into DB?

Hello,
I am trying to insert a struct with an embedded slice into my DB but I am getting an error message like this: sql: converting Exec argument $11 type: unsupported type []string, a slice . The struct that I have is like following
type House struct {
  Name string `db:"name"`
  ...
  Pictures []string `db:"pictures"`
}

and the query that I tried is this
    // 'h' is an instance of the struct House
    statement := "INSERT INTO public.space VALUES ( :name, ..., :pictures )"
    _, err := db.Pool.NamedExec(statement, h)

but that didn't work since returns the error mentioned above. As a workaround, I am inserting the data in 2 steps, first I insert those fields with basic types and then I insert the array. However I am wondering if there is a way to do this in a single operation.
Do you know if there is a better way of doing this? Thanks in advance

Solved!
Using the pq.StringArray data type instead of []String for my Pictures field, I was able to insert all the struct data in a single operation.
Example:
// Updated data type for Pictures field
type House struct {
  Name string `db:"name"`
  ...
  Pictures pq.StringArray `db:"pictures"`
}

...

// Now this works
statement := "INSERT INTO public.space VALUES ( :name, ..., :pictures )"
_, err := db.Pool.NamedExec(statement, h)


Build failure when running make

I am trying to build as explained in README.md and get the following failure:
$ make OPTIMADE_SCHEMAS_DIR=../schemas/output
mkdir -p "output/v0.1.0/properties/structures/"
tools/optimade-property-tools/bin/process_schemas --remove-null --resolve-path ../schemas/output --basedir "src/v0.1.0" --baseid "https://example.org/schemas/v0.1/"  --schema tools/optimade-property-tools/external/json-schema/core --schema tools/optimade-property-tools/external/json-schema/format-annotation --schema tools/optimade-property-tools/external/json-schema/schema --schema tools/optimade-property-tools/external/json-schema/meta-data --schema tools/optimade-property-tools/external/json-schema/validation --schema tools/optimade-property-tools/external/json-schema/content --schema tools/optimade-property-tools/external/json-schema/applicator --schema tools/optimade-property-tools/external/json-schema/unevaluated --output "output/v0.1.0/properties/structures/_exmpl_cell_volume.json" "src/v0.1.0/properties/structures/_exmpl_cell_volume.yaml"
Processing of input failed. Error details:
- File not found when dereferencing $$inherit: defs/v1.2/units/si/general/angstrom looked in: ['src/v0.1.0', '../schemas/output'] (/home/andrius/optimade/definition-provider-template/tools/optimade-property-tools/src/process_schemas/process_schemas.py line:1453 triggered at line: 1101).

Add command line argument -d for a full traceback or one or more -v for higher verbosity.
make: *** [GNUmakefile:60: output/v0.1.0/properties/structures/_exmpl_cell_volume.json] Error 1

Please note my personal setting for OPTIMADE_SCHEMAS_DIR, as I have cloned https://github.com/Materials-Consortia/schemas (master) at ../schemas. Directory ../schemas/output does not seem to be present there, nor can I find build instructions to make it. Most likely I am doing assuming something incorrectly here.

Solved: I should have used OPTIMADE_SCHEMAS_DIR=../schemas. I suppose README.md should be adjusted.

Right, this is on me, commiting hardcoded paths that made sense in my environment without explaining the need to update them in the README.
I've landed in that the best solution probably is to have these definition-provider repo:s depend via a submodule on https://github.com/Materials-Consortia/schemas. That way they can use the latest OPTIMADE schemas without having to depend on the user keeping the main OPTIMADE repo somewhere with a fresh 'make schemas'.
That would mean OPTIMADE_SCHEMAS_DIR=dependecies/submodules/schemas.

Please pull latest version (and git submodule update --init --recursive), and check if this is fixed.

Please pull latest version (and git submodule update --init --recursive), and check if this is fixed.

Yes, I can confirm that 378f081ebd4c54b15d80b44f5b6fa3269c46e2b6 fixes the issue, thanks.


[PyTorch] bump up Android version

bump up pipeline

Codecov Report

Merging #1633 (742d854) into master (bb5073f) will decrease coverage by 1.23%.
The diff coverage is 62.14%.

:exclamation: Current head 742d854 differs from pull request most recent head c87d06d. Consider uploading reports for the commit c87d06d to get more accurate results

@@             Coverage Diff              @@
##             master    #1633      +/-   ##
============================================
- Coverage     72.08%   70.85%   -1.24%
- Complexity     5126     5432     +306
============================================
  Files           473      507      +34
  Lines         21970    23769    +1799
  Branches       2351     2589     +238
============================================
+ Hits          15838    16841    +1003
- Misses         4925     5638     +713
- Partials       1207     1290      +83

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../modality/cv/translator/YoloTranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...i/djl/modality/cv/translator/YoloV5Translator.java
5.69% <0.00%> (ø)

...odality/cv/translator/YoloV5TranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...pi/src/main/java/ai/djl/ndarray/BytesSupplier.java
54.54% <0.00%> (-12.13%)
:arrow_down:

api/src/main/java/ai/djl/ndarray/NDArray.java
78.40% <ø> (+1.13%)
:arrow_up:

...ain/java/ai/djl/ndarray/index/dim/NDIndexPick.java
100.00% <ø> (ø)

... and 242 more

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 9a18e7a...c87d06d. Read the comment docs.


Unable to start server (CUDNN_STATUS_NOT_INITIALIZED)

Just have a fresh install of Ubuntu (22.04.2) on my laptop with NVIDIA GeForce GTX 1650 with Max-Q Design. Tried to install the server following instructions in the readme (c3340cc). Using driver from NVIDIA nvidia-driver-525
When starting the sever i'm getting this message and sever will reboot over and over again (default settings)
willow-inference-server-wis-1    | [2023-07-21 15:29:23 +0000] [90] [INFO] Warming models...
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [DEBUG] WHISPER: Loading audio took 1231.369 ms
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [DEBUG] WHISPER: Feature extraction took 7.83 ms
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [DEBUG] WHISPER: Forcing language en
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [DEBUG] WHISPER: Using model tiny with beam size 1
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [ERROR] Traceback (most recent call last):
willow-inference-server-wis-1    |   File "/usr/local/lib/python3.8/dist-packages/starlette/routing.py", line 677, in lifespan
willow-inference-server-wis-1    |     async with self.lifespan_context(app) as maybe_state:
willow-inference-server-wis-1    |   File "/usr/local/lib/python3.8/dist-packages/starlette/routing.py", line 566, in __aenter__
willow-inference-server-wis-1    |     await self._router.startup()
willow-inference-server-wis-1    |   File "/usr/local/lib/python3.8/dist-packages/starlette/routing.py", line 656, in startup
willow-inference-server-wis-1    |     handler()
willow-inference-server-wis-1    |   File "/app/main.py", line 1058, in startup_event
willow-inference-server-wis-1    |     warm_models()
willow-inference-server-wis-1    |   File "/app/main.py", line 421, in warm_models
willow-inference-server-wis-1    |     do_whisper("client/3sec.flac", "tiny", beam_size, "transcribe", False, "en")
willow-inference-server-wis-1    |   File "/app/main.py", line 617, in do_whisper
willow-inference-server-wis-1    |     results = whisper_model.generate(features, [prompt]*batch_size, beam_size=beam_size, return_scores=False)
willow-inference-server-wis-1    | RuntimeError: cuDNN failed with status CUDNN_STATUS_NOT_INITIALIZED
willow-inference-server-wis-1    |
willow-inference-server-wis-1    |
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [ERROR] Application startup failed. Exiting.
willow-inference-server-wis-1    | [2023-07-21 15:29:24 +0000] [90] [INFO] Worker exiting (pid: 90)
willow-inference-server-wis-1    | [2023-07-21 15:29:25 +0000] [88] [INFO] Shutting down: Master
willow-inference-server-wis-1    | [2023-07-21 15:29:25 +0000] [88] [INFO] Reason: Worker failed to boot.

full log here
Not sure what to do how to solve this, reboot of the system doesn't help.

What's the output of nvidia-smi?

output of nvidia-smi
Fri Jul 21 17:50:52 2023
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.125.06   Driver Version: 525.125.06   CUDA Version: 12.0     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA GeForce ...  Off  | 00000000:02:00.0 Off |                  N/A |
| N/A   32C    P3     5W /  30W |      6MiB /  4096MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|    0   N/A  N/A      1073      G   /usr/lib/xorg/Xorg                  4MiB |
+-----------------------------------------------------------------------------+

I think I see the problem. 4GB of VRam is a bit too low to run everything I think.
Here's my output with WIS running and TTS enabled:

As you can see I am already at 5.27GB. Now I had thought in the latest code it will disable TTS and such if you don't have enough VRAM, but even with that I am not sure if 4GB is enough, maybe @kristiankielhofner can answer to that? :)

Im able to run it without TTS, i copied settings with
cp settings.py custom_settings.py

and set  TTS to false
support_tts: bool = False

afterwards it fits
Fri Jul 21 18:12:58 2023
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.125.06   Driver Version: 525.125.06   CUDA Version: 12.0     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA GeForce ...  Off  | 00000000:02:00.0 Off |                  N/A |
| N/A   46C    P0    11W /  35W |   3362MiB /  4096MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|    0   N/A  N/A      1073      G   /usr/lib/xorg/Xorg                  4MiB |
|    0   N/A  N/A     26685      C   gunicorn: worker [main:app]      3354MiB |
+-----------------------------------------------------------------------------+

closing ticket


Dependency version unknown

First of all, PROX-QP looks very impressive and I'm looking forward to experimenting with it.
While trying to reproduce your paper's results, I started setting up a conda environment for the benchmark suite according to the Readme. Unfortunately, the versions of the packages are missing, resulting in a really slow process that did not finish within 30 minutes on my machine.
Could you please specify the correct version or add a environment.yml to the repo?
Furthermore, the linked information on how to build qpOASES seems to be incompatible with conda.

Did you start from a clean environment (conda create -n proxqp-bench)?
You should also use mamba for a faster dependency resolution (https://github.com/mamba-org/mamba).
For QPOASES, we will try to come with a clean way of installing it on conda soon.

That was my first suspicion as well.
However, even running it from a clean docker container with python:3.9-buster image and manually installing conda didn't help either.

For Qpoases, please look at this page https://scaron.info/doc/qpsolvers/installation.html#qpoases for easy installation

Okay, so far I found out, that Python 3.8 and 3.9 do not work in my setup, but 3.7 works except for qpoases, at least without specifying any special package version during conda install.
The qpoases error persists with your installation manual. It is
python -c "import qpoases"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: /home/user/miniconda3/envs/pinocchio/lib/python3.7/site-packages/qpoases.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN7qpOASES8QProblem4initEPKdS2_S2_S2_S2_S2_S2_RlPdS2_S2_PKNS_6BoundsEPKNS_11ConstraintsES2_

Based on this issue, I assume that it is related with the version of cython. However, the proposed fix did not make any difference for me.

@jcarpent Could you please share the output of conda list on your machine with me? Hopefully, I will be able to create an environment file then.


Add HeadersDistrib rule

What does this PR do?
This PR adds another routing rule named HeadersDistrib to route requests to fixed backend (without cookie's help) based on its header (value). Meanwhile, each backend can have different weight to accept requests. For example,
router-hello:
  rule: HeadersDistrib(`X-User-Id`, `0`, `0.75`)
  service: service-hello

router-world:
  rule: HeadersDistrib(`X-User-Id`, `0.75`, `1`)
  service: service-world

Motivation
We were looking for a router for A/B testing (and we have been used traefik since v2.2 released). The backends are different machine learning model services. Each request can be marked with the ID of a front-end user (and we decided to put it in HTTP header).
The problem is, when we do A/B testing, we would like to route a certain percentage (like 75%) of requests to the old service backend and the rest (like 25%) to the new one, and make sure that all the requests with the same user ID go to the same backend. So that we can get the complete profile of a user from either old services or new services.
The solution is simple here. I just hash the specified header value (e.g. X-User-Id) and distribute the hash result to [0, 1). Then each router can be assigned a range to match the distributed result.
More

[x] Added/updated tests
[ ] Added/updated documentation

Additional Notes

I am not a native English speaker, so the name HeadersDistrib may not be accurate for its usage. Please advise me another proper rule name if this routing feature is useful.
The documentation is not updated in this PR, since I wonder this feature can be widely used and merged :)

Hello @wizawu,
Thanks for your contribution!
After a first look, we are wondering why the weighted loadBalancer combined with sticky cookies doesn't achieve the same goal?
Here is an example of configuration:
## Dynamic configuration
http:
  services:
    wrr1:
      weighted:
        sticky:
          cookie:
            name: lvl1
        services:
          - name: service-hello
            weight: 3
          - name: service-world
            weight: 1

    service-world:
      loadBalancer:
        servers:
          - url: http://<IP_ADDRESS>:8082

    service-hello:
      loadBalancer:
        servers:
          - url: http://<IP_ADDRESS>:8084

To be more clear, let's say the router (traefik)  is placed between 3 Kubernetes deployments and each has 4 pods.
deployement A -> traefik -- 75% --> deployment B
                        \-- 25% --> deployment C

There're two key points in our implementation.

Each user (identified by a unique ID, no matter we put it in header, cookie, query parameter, or hash it to traefik service name) must be routed to the same deployment (B or C).
The ratio of the amount of users routed to B and to C should be 75% : 25% (or we say 3 : 1).

Now back to the sticky cookie solution, which we do notice before. There are a few pitfalls when apply it to our scenario.

The original request is produced by a backend service. We need extra work to preserve the cookie (with no browser). Or we need to pre-generete cookie (e.g. Cookie: lvl1=service-hello) in the upstream deployment A.
As you notice, we have 4 pod in deployment A. We have to make sure that all these pods share the same cookie for a single user.
There is a slight difference between weight and distribution ratio. For example, if each pod in deployment A send a request representing user X for the first time, since none of those HTTP clients has cookies, 3 requests may go to B and 1 to C. On the other hand, when routed by HeaderDistrib, requests with the same header value will be always "distributed" to the same backend deployment.
NIT: It's quite common to have tracking ID in headers rather than cookie, such as X-Request-ID in ingress.

After all, we do admit that there are few scenarios to apply the HeaderDistrib match rule. Please feel free to close the PR. And surely we can just continue to use a forked version :)
Regards.

Hi @wizawu , if I understood your use case the only real blocker for using the example referenced by @rtribotte would be the fact that you want to determine (hash) the source criteria to apply stickiness from a header instead of a cookie right?
Other than that the weighted round robin seems to fit well for distribution if used in conjunction with multiple routers, something that would look like this:
http:
  services:
    app-svc:
      weighted:
        sticky:
          #New sticky option
          header:
            name: X-User-Id

        services:
        - name: appv1
          weight: 3
        - name: appv2
          weight: 1

    appv1:
      loadBalancer:
        servers:
          - url: "http://<IP_ADDRESS>:8080"
        healthCheck:
          scheme: http
          path: /health

    appv2:
      loadBalancer:
        servers:
          - url: "http://<IP_ADDRESS>:8180"
        healthCheck:
          scheme: http
          path: /health

    #Redirect to traefik itself for prerouting transformations
    traefik-svc:
      loadbalancer:
        servers:
          - url: http://traefik.local.address

  routers:
    dashboard:
      rule: Host(`traefik.docker.localhost`)
      service: api@internal

    app-without-header:
      rule: Host(`app.docker.localhost`)
      service: traefik-svc
      middlewares:
        # Custom plugin that would hash the source req based on custom logic
        # and then inject the Header "X-User-Id" with either appv1 or appv2
        - my-custom-plugin

    app-with-header:
      rule: Host(`app.docker.localhost`) && Header(`X-User-Id`)
      service: app-svc

I prefer this approach as it would not mix the router matching rule with the service load balancing distribution in the config.
Would something like that work for you?


DOC: update 1.3.2 release notes

update the release notes for SciPy 1.3.2
using authors.py, gh_lists.py, and a little
manual curation, after adjusting milestones
on the appropriate GitHub issues/PRs

Merged, thanks Tyler


mdns & dht discovery

This pr implements a wrapper module around mdns-sd-discovery and hyperswarm to provide peer discovery through mdns and dht with one api.

It would be good to think about the boundaries in mapeo-core between the different modules - thinking as if we would publish as a separate module. My understanding is that the "public" interface (i.e. the interface used by the rest of mapeo core) for discovery is the ./lib/discovery module, and the ./lib/localpeers is an implementation detail. If we clearly define that then we can clearly define the internal API the other pieces will use - the Discovery class - and it would be good to have that API thoroughly tested, so that we have the flexibility to change the implementation details of that module and have good separation between components of Mapeo Core. I think you've written it like this, it's just not currently that clear from how the code is organized and presented. Also good to have function methods and params documented since it really helps with future maintenance (and for code reviews).

@gmaclennan I think this is ready now unless there's anything else we should get in to this pr.
I made a few issues that seem best handled in follow-up prs:

#13
#12
#14


undefined method `validate_uniqueness_of'

Hello,
I am getting error mentioned above:
     Failure/Error: it { is_expected.to validate_uniqueness_of(:name) }
     NoMethodError:
       undefined method `validate_uniqueness_of' for #<RSpec::ExampleGroups::Word:0x007fe6c59525a0>

mongoid-5.0.0
mongoid-rspec-3.0.0

Hi @spurnaye. Could you show your spec_helper.rb and, possibly, rails_helper.rb

Hey @90yukke, I think there was some configuration missing from my side. Everything is good now.
I had to include
require 'mongoid-rspec'
  config.include Mongoid::Matchers, type: :model


Clarifies ordering of dimensions in Convs

What does this PR do?

Fixes # (issue)
Checklist

[ ] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).
[ ] This change is discussed in a Github issue/
discussion (please add a
link).
[ ] The documentation and docstrings adhere to the
documentation guidelines.
[ ] This change includes necessary high-coverage tests.
(No quality testing = no merge!)

Codecov Report

Merging #1672 (0edd518) into main (d6a2194) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1672   +/-   ##
=======================================
  Coverage   83.10%   83.10%
=======================================
  Files          69       69
  Lines        5836     5836
=======================================
  Hits         4850     4850
  Misses        986      986

Impacted Files
Coverage Δ

flax/linen/linear.py
98.38% <ø> (ø)

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 d6a2194...0edd518. Read the comment docs.


Shipwright: Add Ship of Harkinian

Ship of Harikinian is a PC Port of The Legend of Zelda: Ocarina of Time.

[x] I have read the Contributing Guide.

/verify

/verify

/verify


Images cannot float

Hello.
We can have floating image in the HTML5 backend using this syntax:
image::image.jpg[role="right"]

but it does not work in the PDF backend.
Thanks to fix this issue.

Floating images in PDF is extremely difficult, which is why we haven't done it yet. I do have a prototype of some text around images that can be found in editions (https://github.com/opendevise/editions/blob/master/lib/editions/pdf_extensions.rb#L88-L139). We're still considering how to integrate that into Asciidoctor PDF.

As an alternative, I tried using a table, but there is a lot of empty space at the bottom of the table.
Here is my code:
[cols="2*"]
|===
|Some +
text

|image:image.jpg[]
|===

Is it normal that there is a big white space at the bottom of the table?
It seems to happen only when using an image inside a table.
Thanks for your help.

It seems to happen only when using an image inside a table.

That definitely sounds like a bug. Images can throw off the cursor if not handled correctly, which could be happening in this case. I'll definitely look into it.

I think the problem is that inline images aren't properly expanding the cell height, which is the reason for #295 as well.

@mojavelinux Is there any possibility I can try your pdf extension with ascidoctor pdf, as our documentation departement would like to have floating images in pdf too.

@atomfrede Yes, by all means.

@atomfrede did you figure out how to try @mojavelinux PDF extension with Asciidoctor PDF? I'd also like to get floating images working

@cocowalla No I didn't had time. We made some changes to the documents, such that floating images are not required anymore at the moment.

In the case "image in a table cell" i have the following effect with images wider than the cell:

Images with a small height are show twice
Higher images are shown once with a lot of white space below the image

As I've said many times before, what needs to be done to get AsciiDoc table cell content to work is to write a custom table cell implementation for Prawn that recurses back into the converter to handle arbitrary AsciiDoc content. Once that is done, in theory, we get support for all block-level AsciiDoc in a table cell for "free". Until then, trying to render AsciiDoc content in a table cell by cherry picking it is just a huge hack.

So, what about a proper float implementation into PDF? Is it still in the pipe?

Any updates on this issue?

+1

I'll be posting about this new strategy on the list later this week.

Has this ever been done? A search did not reveal anything really relevant.

I've been waiting on the 1.5.0 release to push out the official announcement. I have been talking about it rather extensively in the gitter channel, though. The prototype can be found at https://github.com/mogztter/asciidoctor-pdf.js, but it's still early days.

Same issue with epub3 renderings, I have no way to position images on the side with some text
Example code:
== Test section
.Title for image
image::einstein-relatividad.jpg[,340,role="related thumb left"]
Lorem Ipsum is simply dummy text of the printing a.....

So the HTML rendering (asciidoctor) renders very good, like:
HTML:

But for PDF or for EPUB3, it always renders it in a strange and unfriendly way (I tried different options for the images, nothing found to work nicely yet):

Epub:

PDF:

Please direct Asciidoctor EPUB3 feature requests to the issue tracker for that project. https://github.com/asciidoctor/asciidoctor-epub3/issues This issue tracker is only for issues that pertain to Asciidoctor PDF.

It seems to happen only when using an image inside a table.

This issue was addressed by the fix for #1515. Here's how it looks now when using a table for layout.

It takes some fiddling to get the amount of text right per cell, but at least there is no extra bottom margin now.

I'm thrilled to report that after all the changes and improvements that were made to the code for the 2.0.0 release, it became possible to implement this feature.
The way it works is that it will wrap successive paragraphs next to the floating image (an block image with the float attribute). The float box will be cleared once the content extends past the bottom of the image or another type of block is encountered. However, the code also adds a framework for enlisting additional block types in the float wrapping, which is explained in the documentation.

@mojavelinux Is there a way to use [.float-group] with PDF? I'm traying put three image side by side, and I can do it in HTML, but not in PDF...

Please direct usage questions to the project chat at https://chat.asciidoctor.org. Thanks.


Locations not displaying on init

Hello!  Thank you for an awesome plugin! 🙌
I noticed that when reloading the field after data was stored that the data wasn't rendering.  This PR should resolve that (though I only tested with polygons, squares, etc)

hi @dwatts3624 - your contribution is very much appreciated, thank you very much 👍


[BEAM-4814] Add client configuration to aws options

I want to run a Beam job in my Spark cluster that uses the S3FileSystem. My Spark cluster is configured to require a proxy server with authentication in order to make outbound connections. A small change is required to enable ClientConfiguration to be added to the configuration to enable this.
This PR adds the ability to specify the four proxy related properties of ClientConfiguration object and passes that when creating an S3 client.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

One of the challenges I faced in writing this code was that ClientConfiguration is not serializable via Jackson. The level of effort in creating a mixin that would fully serialize the object was not, in my opinion, worth the reward. So I created a mixin that only serializes the properties I was interested in. The alternatives were a ton of mixin work, a new pojo object with only the properties I was interested in, or discrete string properties on the AwsOptions class. I did not like any of my options, but choose the one I felt was least bad. I welcome further discussion of this point.

R: @iemejia

Minor comment addressed and commits squashed. Please re-review.


Brick rendering

Learn to properly position and render the level bricks.
Depends on:

[x] #18

I think we've already did that.


add readOptions

flag consistency works for consul and etcd v2

any updates on this one?

@zyclonite I took the liberty to carry your commit in #37 to add the missing switch to etcd v3. I will merge once the CI passes.

Done :)


Support 3.14

3.13

_testcapi_datetime added, defined in Modules/_testcapi/datetime.c

3.14

Addition of annotationlib whose name is backed up by a pep.

I notice the copyright year in the generated file is out of date, do you have objections to strftime("%Y") in the generator?

RTD check is failing because this pr is from a fork.

I notice the copyright year in the generated file is out of date, do you have objections to strftime("%Y") in the generator?

Better yet just get rid of the year in the copyright statement altogether.


Unicode problem.

Godot version:
3.1 Stable (tested with latest master also)
OS/device including version:
Windows 10
Issue description:
I am receiving unicode messages through sockets that contain unicode characters, in my case Greek language characters.

StreamPeer get_utf8_string gives me Unicode error: invalid skip.

PoolByteArray get_string_from_utf8 gives me Unicode error: invalid skip.

PoolByteArray get_string_from_ascii give me the full message but all unicode characters are converted to questionmarks.

I'm a bit frustrated with thisone, searched through issues but could not find any solution.

How are strings formatted on the socket? Do they have a length prepended or are they nul-terminated? Are you sure that it's UTF-8 and not another encoding like UTF-16?
What argument (if any) are you passing to get_utf8_string? If you pass no length argument, the function expects to first read the length of the string and then read that many bytes.
What does get_string_from_utf8 return when it gives that error?

How are strings formatted on the socket? Do they have a length prepended or are they nul-terminated? Are you sure that it's UTF-8 and not another encoding like UTF-16?

nul-terminated in my case. I'm not sure if its UTF-16 since i have not written the server part. Unfortunately, i don't know what kind of encoding the server uses but, since this is the most probable case, any idea how would i proceed with decoding UTF-16 in Godot?

What argument (if any) are you passing to get_utf8_string? If you pass no length argument, the function expects to first read the length of the string and then read that many bytes.

var bytes = client.get_available_bytes()
var message = client.get_utf8_string('bytes')

What does get_string_from_utf8 return when it gives that error?

Returns the part of the message that was encoded until the error occurred. If the message was 500 chars and the error occurred at char 300 then the result is 299 chars. Everything beyond that is skipped.

I actually think UTF-16 is unlikely, I was asking just in case. It may also be a non-unicode greek encoding, but I'm assuming you know it's some kind of unicode since that's what the issue title says. If you're not sure, then I think a non-unicode encoding is more likely than UTF-16, but I'm not really that familiar.
Also you shouldn't rely on a string after get_available_bytes to be exactly one message. That function just tells you how much data it has received so far, and it's possible that it has received more or less than one full message. If you read that number of bytes you might not have a full message, or you might have a full message and the beginning of another message. Unicode characters can be multiple bytes, so the partial data might even be split in the middle of a character.
To handle null-terminated strings you would need to:

Keep a PoolByteArray to hold incomplete strings
When you read data from the client, use get_data instead of get_utf8_string, because the data can end in the middle of a unicode character, and then you would have an invalid character and lose that data.
If there's a null character in the data, then add everything before the null character to the PoolByteArray and then turn that whole thing into a string, and then clear the buffer. Put everything after the null character into the cleared PoolByteArray since that is the start of the next string.
If there is no null character, just add the whole thing to the buffer because you haven't completed a string yet.

I actually think UTF-16 is unlikely, I was asking just in case. It may also be a non-unicode greek encoding, but I'm assuming you know it's some kind of unicode since that's what the issue title says. If you're not sure, then I think a non-unicode encoding is more likely than UTF-16, but I'm not really that familiar.
Also you shouldn't rely on a string after get_available_bytes to be exactly one message. That function just tells you how much data it has received so far, and it's possible that it has received more or less than one full message. If you read that number of bytes you might not have a full message, or you might have a full message and the beginning of another message. Unicode characters can be multiple bytes, so the partial data might even be split in the middle of a character.
To handle null-terminated strings you would need to:

Keep a PoolByteArray to hold incomplete strings
When you read data from the client, use get_data instead of get_utf8_string, because the data can end in the middle of a unicode character, and then you would have an invalid character and lose that data.
If there's a null character in the data, then add everything before the null character to the PoolByteArray and then turn that whole thing into a string, and then clear the buffer. Put everything after the null character into the cleared PoolByteArray since that is the start of the next string.
If there is no null character, just add the whole thing to the buffer because you haven't completed a string yet.

Sorry, my fault, all messages end with an end tag ({end} in my case) not null terminated. Of course, everything you said applies in this case but, i'm just mentioning it because i have already build a mechanism to detect when the full message is received.

@zaksnet how do you send the string? Can you provide a minimal reproduction project? How do you detect the end tag?

@zaksnet how do you send the string? Can you provide a minimal reproduction project? How do you detect the end tag?

The string is being sent from another application (of which i do not currently have the source code). I am doing:
const ENDTAG = "{end}"
	if is_connected_to_host:
		var bytes = client.get_available_bytes()
		if bytes > 0:
			var message: String = client.get_string(bytes)
			var messages = message.split(ENDTAG)

If a split does not end with ENDTAG, i add it to the next message.

I dont think this is network related. It just happend that the data i get came from the network (i think the same thing would happend if i loaded them from a file).
I am guessing get_string decodes the message to ascii since i get the same result as get_string_from_ascii. That is, the whole string is fine (i'm pretty sure there is not partial data) except the unicde parts which are represented as questionmarks.

In conclusion, i think this is a Unicode decoding problem and not network related.

Can you test it on windows to make sure it's not platform specific? Because it seems the issue is same as #30409.

If a split does not end with ENDTAG, i add it to the next message.

Okay, there lies the problem, unicode is variably one or two bytes.
You can't decode partial data.
When reading from the stream you MUST detect the end tag, and only decode after you have the full message, as @raymoo explained you above.

If a split does not end with ENDTAG, i add it to the next message.

Okay, there lies the problem, unicode is variably one or two bytes.
You can't decode partial data.
When reading from the stream you MUST detect the end tag, and only decode after you have the full message, as @raymoo explained you above.

Cool, thing is, i never had to add data to the next message so far. Messages are small and this is being tested locally so i get the whole string all the time, some times i get 3,4 messages at the same time and all of these might contain Unicode data so between endtag 1 and endtag 2 there are question marks (this is just one stream of data) and there are 4 endtags so i don't see how the data could be malformed in my case.

@zaksnet can you give us the raw bytes of one message so we can try it?

Sure, check the attached file.  Yes, it is a π. A lot to consume here but, how would i proceed in that case? Can i somehow decode this properly?

Thank you for researching this BTW! Appreciated!

Can i somehow decode this properly?

I don't think Godot supports other encodings beside UTF-8 and ASCII.
And really, no one should use non UTF-8 encodings at this point in time.
Check if the service from which you are getting the data supports utf-8 encoding (maybe with an extra query parameter).
The only other way I can thing of, is using a translation table from ISO/IEC 8859-7 to UTF-8 and applying it to the raw bytes, then using get_string_from_utf8 on the converted bytes.

similar, but offtopic to this issue:
@Faless
Did you see this PR?
https://github.com/Faless/godot-websocket-demo/pull/3
It is for Godot 3.1+ support and changes the encoding in your example project.
Could you please either merge or reject it?


Project draco refactorings-01

Context
Dear all,
My name is Davi Freitas. Currently I am supporting a research project led by Marcos Cesar de Oliveira (a software engineering Phd student). As part of my research project I’m investigating refactoring opportunities that might improve the modularization of Java projects. We developed a tool for this purpose (https://github.com/project-draco), and the tool suggested 2 refactorings that might benefit this project, by moving methods that have frequently changed together to the same module (a Java class). I’m sending one of the refactorings in this PR.
Are you willing to integrate these changes? If that is not the case, could you explain why this PR is not useful? This might suggest that keeping things that frequently change together in the same module is not so relevant. Any feedback would be greatly appreciated and could be extremely valuable for our research.
Thank you in advance,
Davi & Marcos.
Contributor Checklist

[X] Review Contribution Guidelines
[X] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[X] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[X] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Hi @drcfts,
This refactoring doesn't quite make sense.  It's moving a static method from a utility class and putting it into a class that happens to use the utility method.
I'm going to close this PR as this isn't a change we would want to make.
Thanks!


targetOffsetUs and DefaultLoadControl

I am testing the hls start delay. Is there any correlation between setting targetOffsetUs start shard and setting DefaultLoadControl cache parameter? What effect does the cache parameter size of DefaultLoadControl have on the multicast delay
Please help me to solve it

Hi @gyc779594494,
Could you please clarify if the "cache parameter" in your question means "buffer parameter"?

It's the buffer parameter.

Thanks @gyc779594494!
I assume that you're asking about how does the method DefaultLoadControl.shouldStartPlayback consider the targetLiveOffsetUs and the bufferForPlaybackUs (duration of media that must be buffered for playback to start or resume following a user action such as a seek).
In the presence of a "set" targetLiveOffsetUs, the DefaultLoadControl basically allows the playback to start earlier if the targetLiveOffsetUs is low. On the other hand, the ExoPlayer provides the targetLiveOffsetUs by a component LivePlaybackSpeedControl. In the default implementation, this value keeps being adjusted with the current buffered duration and playback speed. The live offsets set in the LiveConfiguration are the limitations that the DefaultLivePlaybackSpeedControl tries to get closed to.
Hope the above can help!

Can I set tDefaultLoadControl (1000,3000,1000,1000), getTargetLiveOffsetUs sets two fragments, and the playback will be delayed at different timeshe buffer duration parameter very low and make the offset lower


update: Added functions for removing occlusion

Added function __get_occlusion_mask__ - which tries to make a mask of objects
that are bigger then one third of image
and are in front of a board

Added function which splits the board into sections

Added function for stitching sections of a board to final slide

cc @mrshu


CI error: ./build.sh: line 11: GOPROXY=https://goproxy.cn,direct: No such file or directory

See: https://github.com/casdoor/casdoor/runs/8149355019?check_suite_focus=true
Caused by this PR: https://github.com/casdoor/casdoor/pull/1082

@cofecatt


Row index required in onCellTap

I'm using fluttable to show a table displaying some of the properties of objects that I'm getting in a list. I've made one whole column tappable and I want to pass the object of that specific row index to another page when a cell is tapped. Can I get the row index in onCellTap to make this possible?

Good suggestion. I'll make it available

Fixed this issue in https://github.com/ZainUlMustafa/Fluttable/tree/tap-able-cells
Will be released as a main feature in v1.2.3

Right thanks!


Stepper does not generate a utility class for nonLinear

Summary
When using <Stepper nonLinear Stepper's html markup does not gain a MuiStepper-nonLinear class. When used along with alternativeLabel, making it disproportionately difficult to style StepButton differently based on the nonLinear prop.
I acknowledge this issue resembles #17367. We are trying to target the styles of StepConnector without providing a custom implementation of StepConnector.
Examples

Motivation
For our nonLinear Stepper, since the Steps do not represent linear progression, we want to style Steps in a way where they appear related to each other and where each Step is not dependent on the previous steps.
Search keywords: stepper nonLinear utilityClass utility class

Hey @alexismo, thanks for the report! I understand the issue but could you provide a minimal reproduction? This would help us understand the use case better. A live example would be perfect. This StackBlitz sandbox template may be a good starting point. Thank you!

Sure thing! @DiegoAndai, here you are: https://stackblitz.com/edit/stackblitz-starters-3a8adj?file=src%2FApp.tsx

I think we can add this class. @alexismo, feel free to create a PR; I'll mark it as an enhancement.
You can also use theme variants:
const theme = createTheme({
  components: {
    MuiStep: {
      styleOverrides: {
        root: {
          '&[class*="Mui-completed"]': {
            '& [class*="MuiStepConnector-line"]': {
              borderColor: 'green',
              borderWidth: '3px',
            },
          },
        },
      },
    },
    MuiStepConnector: {
      styleOverrides: {
        line: {
          borderWidth: '2px',
          borderColor: 'darkgrey',
        },
      },
    },
    MuiStepper: {
      variants: [
        {
          props: { nonLinear: true },
          style: {
            border: '2px solid red',
          },
        },
      ],
    },
  },
});

However, if I understood your use case correctly, if you want to target individual steps in a non-linear stepper, the above theme variants won't work.


high-contrast setting to improve accessibility

from #5010:

The dates/times are intentionally light in order to avoid having too much emphasis for most users. I think we possibly will want to put that behind a setting in order to make it possible to get high-contrast in that environment, without impacting our desire for those elements to have low visual emphasis for users who have normal vision.

Right now, Chrome Accessibility Tools is giving me ~90 elements that do not pass Text elements should have a reasonable contrast ratio.
This could possibly go under display settings as high-contrast mode.

@zulipbot claim

@timabbott @brockwhittaker do you reckon this setting should have extra features and not just a contrast improvement?

I think we can start with having it just be "high contrast for accessibility" and then add things to it over time.
Code-wise, I think probably the way we want this to work is to add a far outer CSS class if the setting is set, and then CSS for .foo can just have a .high-contrast .foo item for this setting.

Just waiting on the frontend (#5593) to be CR-ed and merged!

Still working on it.

I see this documented at https://zulip.com/help/high-contrast-mode but I don't see the high-contrast mode setting anywhere:

Did this feature actually get shipped? Or was it documented but didn't actually ship?
I would love to have a high-contrast mode. The low contrast is fairly hard for me to read comfortably in a bright room.

The feature only exists in the development environment and as a setting -- it never was implemented far enough to actually change any styling. We are in the process of a redesign that is fixing a lot of previously low-contrast elements and we may yet introduce this after that's complete.

Has there been progress on this or other accessibility improvements? I know things like increased contrast would help some people on our team, in addition to things like extra spacing between elements. If we could define our own colors for the UI, that might help, too.


Add a favorite session list, and ability to favorite a session

Another exploratory PR (not meant to be merged!)
This explores the case where a mutation updates an item that is part of a paginated list:

a filter on the Session list to filter on favorites only
a button on the Session details allows to add to / remove from favorites

Note: Favorites are mocked on the server (in-memory only and shared for all users)

@BoD nice changes...cool to see app include a mutation!  you might have seen already but look like PR build is failing.

I think it was because the snapshots were not published yet 😅  Restarted the job.

ah, went to try this out after merging and looks like it was merged in to session-list-pagination (and not main)....not sure if that was intention?

I wonder how much would be involved in doing something like approach
firebase anonymous authentication uses for now
On Tue 19 Jul 2022, 08:18 Benoit Lubek, @.***> wrote:

Yeah that's OK. Wasn't sure we wanted to merge it as the favorites
mutation is "fake" as of now: we need some sort of authentication and a way
to store user related data on the server. But maybe that's OK to merge
anyway and keep a ticket / Todo open, and implement that later.
—
Reply to this email directly, view it on GitHub
https://github.com/joreilly/Confetti/pull/55#issuecomment-1188689697,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABRHT5EWQA6TRGGI5Z5QDVUZJEDANCNFSM52W4OSUQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


[BUG] Preloader loading issue

Description
Describe the bug:
Currently the preloader is defined using CSS so it is not waiting for threejs model to load
Expected behavior:
Preloader should stop loading when threejs model is fully loaded
Actual behavior:
Preloader is working only for css loading and stops early before loading threejs model
Reproduction Steps
Steps to reproduce the behavior:

Go to 'stichhub-chi.vercel.app'
And see the loading behaviour

Hello @Naresh-chandanbatve !! I am willing to work on this issue.. Can you please assign this issue to me under gssoc?

Go for it @SINJINI-G

Hey @Naresh-chandanbatve! If someone is not working on this issue, can you assign it to me under GSSOC'23?

Go for it @Dev79844

Hey @Naresh-chandanbatve! Can you tell me from where can I get this key: VITE_GOOGLE_CLIENT_ID

@sil-samriddha
Drop the blog link here to get client id

Hey @Naresh-chandanbatve! Can you tell me from where can I get this key: VITE_GOOGLE_CLIENT_ID

hey @Dev79844
checkout this article https://www.balbooa.com/gridbox-documentation/how-to-get-google-client-id-and-client-secret for creating your client id


Feat:Add a dark mode feature

Describe the feature
Dark mode feature to reduce eye stress for some of the users
Add ScreenShots

Add a dark mode button in this navbar
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I would like to contribute please assign it

@Shivam250702 what's the update?

@Suchitra-Sahoo Please assign this issue under GSSOC'24 , i will fix this issue.

I want to work in this issue. Please assign me this task.


Fix TraceNodeIsLeader JSON parser

The log messages are formatted slightly different in each case. In particular:
{...,"data":{...,"kind":"TraceAddBlockEvent.AddedToCurrentChain","...},...}

versus
{"...,"data":{...,"val":{"kind":"TraceNodeIsLeader","slot":85}},...}

Also the parser was intended to only get the events with the kind TraceNodeIsLeader.

Converted to draft as this doesn't seem to happen on master suprisingly

bors r+

bors retry


rack: return generic error instead of leaking exception msg

Fixes #98

Why not use a middleware to do this, if that's your desire?

Coverage remained the same at 70.293% when pulling 45ab047dfccc185def05c5b82373528de3bd9b9e on davidor:generic-error-instead-of-exception into b8e9ca5aeaf748bec550a54a8bcdeb7f7c3d3fe7 on socketry:master.

Thanks for the quick response @ioquatix
I can use a middleware to do this, but I thought that it might make more sense to change this to avoid leaking sensitive information.

This kind of relates to: https://github.com/socketry/falcon/issues/112
Would love to see this PR merged :)

@ioquatix sorry, I forgot about this PR and deleted my fork some time ago. Now I can't edit this PR, so I had to open a new one. It's the same commit but rebased on master: https://github.com/socketry/falcon/pull/130


⚠ Bofrost.de has degraded performance

In ca6deea, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1698 ms

Resolved: Bofrost.de performance has improved in bf70478 after 33 minutes.


Allow volume changes with hardware keys even while screen is off

I didn't even know that this is possible, but my audiobook player app has this annoying feature that even when the phone is locked and the screen is off the volume buttons change the playback volume. But while it's annoying in this app, I think it would be a great feature (maybe with an option to toggle it) in Kore.
There are many movies with some very loud and some very quiet scenes and having to unlock the phone to change the volume is not very nice. Being able to use the volume keys immediately would be pretty useful for this.

+1
I also feel this would be a nice improvement to have.
I currently work around this by having the screen always on when using Kore, which is not good at all for the phone's battery life/health :/

Duplicate of #235? :wink:

I created a new issue (#298), requesting a more complete media control while the screen is locked (i.e.: volume and playback controls).


Login Endpoint Needed

SubTask of User Story 02
Endpoint should authenticate user, generate and return a token with username upon success.


V2 changes

Staging ground for v2 changes

All of the changes merged into here were individually reviewed previously so a review is not strictly required here but if anyone has some final comments that would be fine too.
This implements:
https://github.com/SiaFoundation/explored/issues/63
https://github.com/SiaFoundation/explored/issues/62


Add not like limbo by kendrick

Song Name
not like limbo
Artist Name
kendrick
Source
Youtube
Youtube Link
n6JWEn9InrM
Direct File Link
No response
Song ID
76743
Start Offset [ms]
6.00
Extra
Created from Auto Nong.
Created for level: 86452082

reject
mashups should have some popularity to show that people want it.
Note that you can still use the nong yourself, but it wont be public


Feature : List installable packages of a given env

Goal :

Port the following feature of Gator using Conda Store : List installable packages of a given env

Acceptance :

The feature works with mamba/conda and Conda Store

Other notes :
Show the packages with their version and channel

Blocked by https://github.com/Quansight/conda-store/issues/151.

Issue #151 has been solved now, so this issue is unblocked :)

Merged: https://github.com/mamba-org/gator/pull/155