pipenv install never resolves and creates many concurrent python processes

Issue description
Command pipenv install <package name>  starts tens of concurrent python processes. The command never resolves and my machine performance decreases as the number of python processes increases.
Expected result
If I run the command below in an empty directory:
pipenv install six

I assume that a virtual environment is created with the package six. A Pipfile and a Pipfile.lock should also be created.
Actual result
Pipenv gets stuck at the step of creating the virtual environment. The activity bars ([=== ]) are still moving. Something like 6 concurrent python processes are created. At some point this number starts increasing dramatically.
Steps to replicate
Cannot tell for this is very likely caused by a specific state of my machine. I would be interested by having some clues about what to inquiry.

$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: 'c:\\program files\\python37\\lib\\site-packages\\pipenv'
Python location: 'c:\\program files\\python37\\python.exe'
Python installations found:

3.7.4: C:\Program Files\Python37\python.exe

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.4',
 'os_name': 'nt',
 'platform_machine': 'AMD64',
 'platform_python_implementation': 'CPython',
 'platform_release': '10',
 'platform_system': 'Windows',
 'platform_version': '10.0.18362',
 'python_full_version': '3.7.4',
 'python_version': '3.7',
 'sys_platform': 'win32'}

System environment variables:

ALLUSERSPROFILE
APPDATA
COMMONPROGRAMFILES
COMMONPROGRAMFILES(X86)
COMMONPROGRAMW6432
COMPUTERNAME
COMSPEC
DEFLOGDIR
DRIVERDATA
HOMEDRIVE
HOMEPATH
HOMESHARE
INTELLIJ IDEA COMMUNITY EDITION
JAVA_HOME
LOCALAPPDATA
LOGONSERVER
NUMBER_OF_PROCESSORS
ONEDRIVE
OS
PATH
PATHEXT
PROCESSOR_ARCHITECTURE
PROCESSOR_IDENTIFIER
PROCESSOR_LEVEL
PROCESSOR_REVISION
PROGRAMDATA
PROGRAMFILES
PROGRAMFILES(X86)
PROGRAMW6432
PSMODULEPATH
PUBLIC
PYCHARM COMMUNITY EDITION
SESSIONNAME
SYSTEMDRIVE
SYSTEMROOT
TEMP
TMP
USERDNSDOMAIN
USERDOMAIN
USERDOMAIN_ROAMINGPROFILE
USERNAME
USERPROFILE
VBOX_MSI_INSTALL_PATH
WINDIR
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: C:\Program Files\Python37\Scripts\;C:\Program Files\Python37\;C:\Program Files (x86)\Common Files\Oracle\Java\javapath;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\WINDOWS\System32\OpenSSH\;C:\Program Files\nodejs\;C:\Program Files\MATLAB\R2019a\bin;C:\Program Files\Java\jdk1.8.0_241\bin;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Git\cmd;C:\Program Files\Docker\Docker\resources\bin;C:\ProgramData\DockerDesktop\version-bin;C:\Users\f.laudarin\AppData\Local\Microsoft\WindowsApps;C:\Users\f.laudarin\AppData\Local\Programs\Microsoft VS Code\bin;C:\Users\f.laudarin\AppData\Roaming\npm;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files\JetBrains\PyCharm Community Edition 2019.2.1\bin;;C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2019.3.2\bin;;C:\Program Files (x86)\Launch4j;C:\apache-maven-3.6.3\bin;

This should not be an issue on pipenv==2022.8.19


Update?

Are there any plans for an update to get this supported with Rails 6? If not what are your contribution guidelines maybe I could get it working.

Good question. I don't have specific plans for adding Rails 6 support (we're not there yet with our Rails apps), but it's likely I could get to this in the next month or two. If you'd like to take a crack at it please feel free to fork it and make a pull request. No contribution guidelines for this other than new features require tests and existing tests should pass.

Hi all, take a look at https://github.com/duke-libraries/edtf-humanize/pull/26 I may be misunderstanding something, but it looks like you don't need to use Rails to test the gem, unless there are some other test cases I'm missing.

Resolved by https://github.com/duke-libraries/edtf-humanize/releases/tag/v1.0.0


Unable to set up environment variable through .env file

I am trying to send an api key in the header of api call, I trying to make the api key as environment variable in .env file.
I am using this boiler plate and have used the way posted in this article link below but i am unable to get the environment variable and getting its value undefined .
https://medium.com/@trekinbami/using-environment-variables-in-react-6b0a99d83cf5
need help to solve the issue of using environment variable and then console logging it in this boiler plate from env file.

Hi @zulfiqar313 , I had the same problem and finally found a way to get it working.

install env-cmd plugin
create a .env-cmdrc file in the root of your project with the following syntax:

{
  "development": {
    "BACKEND_API_DOMAIN": "http://domain"
  },
  "production": {
    "BACKEND_API_DOMAIN": "http://domain"
  }
}

In your package.json change your run script like this:
   "start": "cross-env NODE_ENV=development PORT=4000 env-cmd -e development node server",
in internals/webpack/webpack.base.babel.js you need to add your env variables in the enviromentPlugin:

new webpack.EnvironmentPlugin({
      NODE_ENV: 'development',
      BACKEND_API_DOMAIN: JSON.stringify(process.env.BACKEND_API_DOMAIN),
    }),

Hope this works for you!


Add all in one manifest and kustomize overlay for frr mode

The all in one deploys in native mode, and there's currently no way to deploy frr using the existing overlays (the fr-on-ci sets image-pull-policy to never)

my kustomize is pretty rusty, there isn't a clean way to have this without copying the contents from frr-on-ci?

The current layer structure (after this new PR) is:
default - kind-ci - frr-ci
default - frr
We need to have duplication at some point. It might be that we do
default - kind-ci
and
default - frr - frr-ci
That would have less duplication because it only changes the imagepullpolicy, but some is unavoidable


Add ResourcesProgressing condition to ManagedResources

How to categorize this PR?

/area robustness
/kind enhancement
What this PR does / why we need it:
Add ResourcesProgressing condition to ManagedResources (first step of https://github.com/gardener/gardener/issues/5850#issuecomment-1108251803).
Will be read by the seed care controller soon, but can also be used waiting for ManagedResources targeting shoots to be fully rolled out later on.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/5850
Special notes for your reviewer:
I improved a few things along the way:

Harmonize logging across all resource-manager controllers (ref https://github.com/gardener/gardener/issues/4251)
Move condition update on deletion to resource controller to prevent unneeded conflicts between controllers
Speed up resource controller integration test (ref https://github.com/gardener/gardener/issues/4805, https://github.com/gardener/gardener/pull/5610 etc.)
Improve conditions handling for ignored ManagedResources (previously health checks continued to be executed even if ManagedResource was marked as ignored, ref https://github.com/gardener/gardener/pull/5556)

Release note:

The status of `ManagedResources` now contains a new condition of type `ResourcesProgressing` which can be used to detect whether updates to managed resources have been fully rolled out.

/assign

@rfranzke I addressed all your suggestions except the first one. We might also do it in another PR :)

We might also do it in another PR :)

Sure, thank you! Would you like to follow up on this small addition/harmonization?

/lgtm
/approve

There seems to be new flaky behaviour in pull-gardener-integration introduced by this PR (e.g. https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/5935/pull-gardener-integration/1522480867149090816), could you kindly check? :)

Sure, thank you! Would you like to follow up on this small addition/harmonization?

Yes, will do 👍

As discussed via phone, I have now already done this change as part of #5968 (ref https://github.com/gardener/gardener/pull/5968/commits/35e3f6a6032482ea502b76fd26692f3b261564db). It would still be required to ensure that the newly introduced Progressing condition in the ManagedResource status considers still existing old Pods which are in Terminating status. Accordingly, I added this as action item to https://github.com/gardener/gardener/issues/5850#issuecomment-1108251803.


Fixed #28135 -- Refactored simplify_regex()

While using Django REST Framework's Schema generator, I found out they're using simplify_regex(); however, current version has a few shortcomings: for instance, patterns
r'^(?P<a>(?:\w+|\d+))'
r'^(?P<a>\w+(?#comment))'
r'^\b(?P<a>\w+)\b'

currently produce (respectively):
'/(P<a><var>)'
'/(P<a>\\w+<var>)'
'/\\b<a>\\b'

when they should all simply produce:
'/<a>'

This pull request features a faster simplify_regex() which also now supports patterns with:

all capturing and non-capturing groups
backreferences
comments
flags
look ahead/behind
yes-pattern
special sequences

Performance gains:
In [1]: %%timeit
    ...: simplify_regex(pattern)
    ...:
10000 loops, best of 3: 38.9 µs per loop

In [2]: %%timeit
    ...: simplify_regex_orig(pattern)
    ...:
10000 loops, best of 3: 54.4 µs per loop

https://code.djangoproject.com/ticket/28135

@Kronuz, can you update per the review comments?

Closing due to lack of follow up.


Add Powershell Syntax Highlighting

Please could Powershell syntax highlighting be added to codeblocks?
Reproduce Issue

Insert code block and after the first ``` enter either ps or powershell
No syntax highlighting provided

Should be easy to do. There is a power shell highlighter in legacy modes: https://github.com/codemirror/legacy-modes/tree/main/mode
Which needs to imported here: https://github.com/silverbulletmd/silverbullet/blob/main/common/languages.ts
I can do this after my vacation, but happy to take a PR that adds this.

On discord somebody asked for diff, may well add that one too while at it. And potentially more languages in that list.

Great!

I think this issue can be closed, since it has been solved!


VS Code crashes with error 132

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.63.2
OS Version: Ubuntu 18.4

Steps to Reproduce:

Start code with extension 'ViDE-Software.v4pvhdlforprofessionals' installed
Open a file with extension .vhd to activate the extension
Wait a couple of seconds, VS Code shows a window "The window has crashed. Reason: 'crashed' code: 132"

I could already narrow it down to some settings in the package.json of the extension (of which I am the author). There is a similar issue where I already described the behavior: #130375, but it has been closed by the bot.
Please refer to this message for details:
https://github.com/microsoft/vscode/issues/130375#issuecomment-922007487
Please find attached an crash-dump.
dump.zip

Thanks for attaching the crash dumps, renderer crashed due to OOM. Can you also confirm if the crash happens when you disable the ViDE-Software.v4pvhdlforprofessionals extension ?

It does NOT crash without the extension. The problem is somehow connected
to the description field of the semanticTokenTypes of the extension, I
added a link to my original post in another issue that has already been
closed at the time of writing.
Thanks for looking into it.
Robo @.***> schrieb am Mo., 24. Jän. 2022, 17:35:

Thanks for attaching the crash dumps, renderer crashed due to OOM. Can you
also confirm if the crash happens when you disable the
ViDE-Software.v4pvhdlforprofessionals extension ?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/141280#issuecomment-1020293402,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN4YZHNYKE4NGTD2D6SE77TUXV5T5ANCNFSM5MVQJXMA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

In https://github.com/microsoft/vscode/issues/130375#issuecomment-922007487 you have mentioned that OOM crash can be triggered with your extension as well. Is there a minimal extension example you can provide for us to repro the issue, you can use https://github.com/microsoft/vscode-extension-samples/tree/main/helloworld-sample as a starting point.

While I tried to narrow it down, I found a really simple way to reproduce it - it does not even need the binaries of my extension.
Let's just hope this is the same issue I was reporting initially :-)
Steps to reproduce, using the attached .zip file. ALL extensions except V4P are disabled:

Install the v4p_buggy.vsix extension. It is stripped down to a bare minimum.
Open code, and open the file 'test.ts' in code.
This file contains a couple of commented lines. Uncomment them, code immediately crashes.

Interestingly, this procedure does not even trigger the activation of the extension. But VS code crashes anyway. I really hope that you can reproduce the problem as well. For me it works as described every single time.
The same procedure with v4p_working.vsix does not produce the error. The only difference between the two .vsix files is the package.json, where the buggy version contains some additional "description" tags inside the semanticTokenTypes-array.
minimal.zip

Thanks for the minimal repro, can confirm the issue on my end.

May I ask if it is planned to fix this issue? While I understand that it is not critical, it is quite annoying nonetheless. Thanks for your support!

How would one remove an extension if VSCode is crashing every time it is opened?

Unable to repro the issue with latest insiders, closing as resolved.

To help anyone that finds this as I was having this issue on Pop OS, I downloaded VSCode Insiders and I just installed me extension that was broken and it finally started working. Everything can be made the same for insiders by moving over extensions and settings.json file. @deepak1556 is there a way users can track which change fixed this issue, or which build release will fix this issue?

Fix will be available in the upcoming 1.86 release that will happen next week.


Refact: Bpf log reader start

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

cc @LiZhenCheng9527

=== NAME  TestHandleKmeshManage/10._ns_without_label,_pod_ready_add_without_annotation
    manage_controller_test.go:186:
        	Error Trace:	/home/runner/work/kmesh/kmesh/pkg/controller/manage/manage_controller_test.go:186
        	            				/home/runner/work/kmesh/kmesh/pkg/controller/manage/manage_controller_test.go:425
        	Error:      	Not equal:
        	            	expected: false
        	            	actual  : true
        	Test:       	TestHandleKmeshManage/10._ns_without_label,_pod_ready_add_without_annotation
        	Messages:   	unexpected value for disabled flag

Should not influence this tests.
/retest

/lgtm
/approve

/lgtm

/lgtm
/approve


Add build scripts to create cross platform binaries

Need build scripts like the wsk CLI to create cross platform binaries.

@paulcastro , Golang supports the cross compile well, for example, if user want to build for a windows platform, only type the command like "env GOOS=windows  GOARCH=386 go build", then the windows binary will be available. Probably we could provide a .bat file for a windows user?
For a linux/mac user, the current makefile is sufficient? what's your opinion?

Yes, the Makefile is good for most things.  For Travis and Jenkins builds,  the OW CLI builds the binaries inside a Docker container using a build script similar to the Makefile -- I'd like to use that method.

We can follow the same technology that is used for wsk compiling and building. Look at tools/cli for details on docker file, script, and gradle.

Yes, totally agree with you, let's reuse the existing techs&scripts as carlos and matt comments.

@lzbj @daisy-ycguo What binaries do we expect to build? Based on the description here, I am sure that the wsk CLI is definitely one of them. Do we still expect anything else to be built, able to be described in the manifest?

@houshengbo , I think we need to use gradle to build wskdeploy as did by wsk cli.

@houshengbo we don't need to build the entire wsk CLI -- I'd focus on building the wskdeploy binary.  That means building openwhisk/openwhisk-client-go and openwhisk/openwhisk-wskdploy.  I'd build for the most common architectures now (64 bit mac, linux and windows):
amd64 darwin
amd64 windows
amd64 linux

Let me know if you have any questions as to how the WSK CLI build works.

I haven't looked through the project too much, but where will the built artifacts be placed? Is there a plan to publish them somewhere? I'm curious since this project looks like a possible candidate to become a plugin to the wsk CLI.

@dubeejw A tutorial would be great...

@dubeejw , I submitted a pr to address this issue, could you please help review it, thanks.
I tested the pr and could get a binary under the bin folder. Also the .zip package is also under bin folder. The command is ./gradlew distDocker or gradle distDocker.
@mrutkows , I could write a tutorial or we could update the README. What do you think?

@houshengbo this is the binary issue which we need as high priority as well; again use David as needed.

@dubeejw I think we want to clearly (initially) place them in a "/bin/download" dir and then link them from our README (nad perhaps create a dedicated README in that download dir.   We of course, are working a longer term plan to build them with the greater CLI (and some framework/sdk) that will take us until June time-frame.

The scripts are merged. The folder bin has an empty READEME.md. There is a section about how to download binaries in the root README.md. I think we can add this section also to bin/README.md.

Merged


vim9class: null_object - constant type 17 not supported

Steps to reproduce
(It may be that null_object is only partially implemented, and so this is not unexpected)
When there's clarification, I can try expected behavior
vim9script

# OK
var nullo = null_object
def F(): any
    return nullo
enddef

# Maybe OK: output: type(F()) 13
echo "type(F())" type(F())

# Maybe OK: output: typename(F()) object<Unknown>
echo "typename(F())" typename(F())

# OK
var o1: any
o1 = nullo

# OK
echo o1 == null_object ? "o1 == null_object" : "o1 != null_object"

# FAIL: expect o1 == null
echo o1 == null ? "o1 == null" : "o1 != null"

# FAIL compilation error
### This works as of patch 9.0.1357
def G()
    var x = null_object
enddef

class C
endclass

# FAIL
var o2: C
o2 = null_object

Expected behaviour
The expected results are indicated in the steps to reproduce.
Version of Vim
9.0.1333
Environment
ubuntu
Logs and stack traces
No response

This issue has several examples of using null_object. 3 of them failed when the issue was filed. One of the failures now works. In particular
        def G()
            var x = null_object
        enddef

Used to fail compilation. It was fixed by https://github.com/vim/vim/commit/c4e1b86cb0d88fa5ec1141d3c600e026dcc1bc21

Some additional output and comments for the last failure
class C
endclass

# FAIL
var o2: C
echo "o2:" o2
echo "o2:" typename(o2)
o2 = null_object

there's the output
    o2: object of [unknown]
    o2: object<Unknown>
    Error detected while processing command line..script /.../12043-b.vim:
    line    9:
    E1012: Type mismatch; expected object<C> but got object<Unknown>

It seems like a bug, otherwise there would be no way to null out an object, like for garbage collection. As a workaround, I guess one could craft an empty object of the correct type and use it as a null analog.

And there's
$ srcgrep null_class
./src/eval.c:	case 10: if (STRNCMP(s, "null_class", 10) == 0)
./src/testdir/test_vim9_class.vim:def Test_using_null_class()
./src/testdir/test_vim9_class.vim:      @_ = null_class.member

Please feel free to work on this.  I think PR #12867 has some changes that will be needed here. I started working on adding the test_null_object() and test_null_class() functions. These will be useful in writing tests.

started working on adding the test_null_object() and test_null_class() functions

I don't see any reference to test_null_* in PR #12867 but I do see quite a bit about class/object which I guess are steps on the way. Looks like that PR is very close, it will probably be in before I'm ready to go...

Now that PR #12890 is merged, can this issue be closed?

Closed by #12890.


Wrong path

The tutorial of Xgboost Java API tells to import classes from org.dmlc.xgboost4j.*.
However, the latest release of the jvm-package installs these classes under ml.dmlc.xgboost4j.java.*.
Maybe it should be fixed in the tutorial.

Old question, no answer. Closing


feat: start an autodiff poc.

Fixed the tests here ! Good work guys, although we need to merge #96 first


STL files as STEP for modification?

Any chance we could get the .step files for the case and cover that are currently STL downloads?
My 3D printer can't handle the build volume so I was looking to modify for laser cutting instead.
Awesome project!

@plains203 thanks for the link - do you know if their is any way of downloading this in a 3d format other than STL or OBJ?

That I don’t know.

It looks like TinkerCAD doesn't support that conversion, but there are a couple of other ways mentioned here, using either Fusion360 or FreeCAD. Hopefully that helps!
If you do end up converting them, open up a pull request and we can add them to the 3D models folder too.

Hi,  I have the same issue with the size of the cover and case, it's just a bit too big for my Creality Ender 3 3D printer.  But think I can modify the STL file through Fusion 360.  I'll have a go....
But I'm really excited about this project.  I've been working on an autonomous vineyard robot based on the Ardurover/Cube/Mission planner project and the OWL will be perfect for it.    I also wanted to get more familiar with the Raspberry Pi and computer vision and this is the perfect opportunity to do so.
Well done!
Kind Regards,
Stephen

Thanks Stephen for the feedback! It will be great to see how you use this. Make sure you share the outcome once you're done too and any suggestions for improvements along the way.
Any luck modifying the STL on Fusion 360? @williamtsalter perhaps if you linked the original Tinkercad for the base then it could be ungrouped and more easily adjusted. This length was selected because it only just fit on my Prusa Mk3S.

@MerbeinRobotics and @geezacoleman, I just had a look at the public model on TinkerCad and you should already be able to ungroup the base for easy modification. It should become a colourful mess when you ungroup. If this isn't the case for you, please let me know as I may have to change the settings for other users.

Thanks @williamtsalter - I just tried that using the link above and the model comes apart nicely as in the image. Ctrl + shift + g to ungroup or there's a button in the top right of the frame.

Thanks @geezacoleman and @williamtsalter.  After a bit of fiddling with Cura I discovered the case will fit in the Ender 3 if I print it vertically (with the end on the base).  Probably not ideal and I would prefer to print it from the base up.    I have  been able to convert the STL (mesh) file to a solid body in Fusion 360 ready for modification.  My initial thought was to rotate the camera mounts 90 degrees so that I can hopefully shorten the overall length by about 23mm.

That's good to hear @MerbeinRobotics that it fits - it is quite tall, but hopefully printing it vertically works.
I'll close this issue now as we've addressed the original question, but open up a new issue if you end up shortening it and we can incorporate at short model into the set too.


New package: DiffEqTutorials v0.1.0

Registering package: DiffEqTutorials
Repository: https://github.com/JuliaDiffEq/DiffEqTutorials.jl
Created by: @ChrisRackauckas
Version: v0.1.0
Commit: 417e202caace9ed5f3acc4d62e4689f718da5433
Reviewed by: @ChrisRackauckas
Reference: https://github.com/JuliaDiffEq/DiffEqTutorials.jl/issues/71

Needs Compat.


A question about Icons

So I am not sure if this is only me, but I couldn't find an "icon" or "icons" property for the prototypes I've been trying out. I gave the type definitions files a good search and I haven't found a definition for any "icons" property except for something related to blueprints. I also tried the BaseEntity type but it didn't have them either.
Also I apologize for my bad way of writing this, as this is the first Issue that I made.
Thank you in advance

Typings for the prototype stage previously weren't included.
However, thanks to updates from Factorio's side, prototype typings are now available in v2.0.0.
"Icons" and its various types should be available.


Used report section

Hi! Thanks for your work and publishing your code.
Which section of the report did you predict? The findings or impression section or both? Or the output of the create_section_files.py, which can be either of them depending on what's included in the report.
Thank you in advance!

Hi Chantal! In the paper, we predicted/evaluated on impression only as it was the default setting for create_section_files.py. We later retrained the model on findings (released as v2 weights in this commit tag).
Best wishes,
Benjamin

Thank you for your quick reply :)


Standardising 'next_update' field

Across the data product pages, I standardised the 'Next update' field. Now, the text No updates planned is used for all fields that contain text.
The text new version in development was previously used in this field, however, I've moved that information to a callout box at the top of the Overview tab. The reason is that we should try to keep this data in the header as short and standardised as possible, while providing explanatory notes in the content itself. This keeps the header nice and tidy. Let me know if you think this is a good idea. See an example: https://deploy-preview-145--dea-docs.netlify.app/data/product/ga-land-cover-terra-modis/?tab=overview

[x] I spell-checked my work

Also, I've documented this: https://docs.dev.dea.ga.gov.au/public_services/dea_knowledge_hub/product_metadata_fields.html#next-update

I wonder if we broaden the admonition box to just a general...
New version in development.
A new version of this product is currently under development. Subscribe (with a link to the subscription) to the DEA newsletter for information on our latest product releases.
Then we can put this box in a few more places, like WO, FC and Geomedian as well. I think keep the no updates for the products that aren't being updated, but then have this box we can put on any product currently under development.

@CEKrause I've broadened the callout box using the wording you provided.
I can open a separate branch if we want to add it to more pages. Is it FC and also FC Percentiles?
There is a small maintenance overhead in remembering to add this notification to pages and remove it when we remove old pages. Is it worth it to do this for all pages that are anticipating a new version, or only in cases where we specifically need to call it out (like in this case where there are no new updates)?

Normally, we wouldn't have so many products under development at the same time. Right now is a bit of an anomaly for our program. It should go on WO, WO stats, FC, FC percentiles, Geomedian, Waterbodies. I think that's all. We can check on Teams with everyone to make sure we haven't missed any.


KAFKA-13294: Upgrade Netty to 4.1.68 for CVE fixes

netty-codec 4.1.62.Final has the following security vulnerabilities, which in turn effects netty-transport-native-epoll Apache Kafka depends on.

CVE-2021-37136
CVE-2021-37137

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@kkonstantine Should it be merged into 3.0.0? or only for 2.8.1?

Netty is used by ZooKeeper and epoll is not configured by default. By the way, security vulnerabilities should generally not be discussed in PRs (https://www.apache.org/security/committers.html)


Custom headers

I would like to add a custom header to outgoing events but I don't see a way to add that to the emitter. Is this supported or could be supported?

What would be the purpose of this downstream in Snowplow?

We would be passing in an authentication header to ensure only authenticated clients can send events.

It's a neat idea - we are working on an "authenticated events" RFC but no specific plans around how we would implement this at this time (whether it would be headers or an update to the Snowplow Tracker Protocol).

Will it be possible any time soon to add custom headers?
I cannot add it in my client because:

The okHttpClient is private and I cannot add an interceptor to add headers to every request
I cannot subclass Emitter in order to manipulate the list returned from buildRequests to add a header to any particular request.

#269 Merging this PR will allow adding a header interceptor to the OkHttpClient which will solve this issue.

Closing as it has been closed in the PR #269

@Stonepaw , is it possible to add the custom header now. We are using react native tracker and need to send custom headers

Hi @shalinibnair , at the moment that feature is not available directly in our react-native tracker. The custom headers can be added in the native trackers (which are wrapped by React Native tracker) but there isn't any specific configuration that lets the developer to set it in the React Native tracker. We (or you, if you prefer) can open an enhancement issue in the react native tracker repository. It's not in our roadmap for this quarter but we may consider that feature for next release.

@shalinibnair, I've just created the issue in the react native repository. Feel free to continue the conversation there or add details for further improvements.
Thanks for raising this request.

Reopening this issue because it is a requested feature to add the ability to set custom headers and the approach to set a custom HTTP client is a bit awkward also because it changes the cookie jar. It would be better to have a an option in the NetworkConfiguration for it just like in the iOS tracker.


Creating a single binary file with embedded assets

I need some advice for the implementation/deployment of a revel app. The app basically serves some static files applying some logic. One of the requirements of the project is that all should be server from a single binary, this means that I should include all assets inside that binary. (no src nor media folder is allowed). This also means that the config should be embedded in the binary too.
I've been investigating for a while and I haven't found a way to do it with the legacy revel framework. I've seen some solutions like go-bindata[0] but I don't really found a way to do it without modifying the revel source code.
Have anyone worked in an environment like that?

So i need for my next project to create a single executable, images, css et all ... in one..(idea is to run on Raspberry Pi's in a lab, around 20 of them as simple info screens)
I notice that revel package my/app creates an exectable, and also the whole source directory.. and revel directories....
Would go-bindata solve this problem ??

@pedromorgan no, it's not that easy. There are some code dependencies in Revel with the source for controllers and routes. I am interested in single binary builds still though, but it may result in a major shift in Revel's design under the hood.


Clean project error in Editor

Just after importing package to my project I get this error in Editor
Assets/Branch/MiniJSON.cs(120,12): error CS1674: BranchThirdParty_MiniJSON.Json.Parser': type used in a using statement must be implicitly convertible to System.IDisposable'
I am using .NET 4.x, Unity 2018.2.2f1
Can't do anything with that. What's the reason?

@novavision , it is strange, with Unity 2018.2.5f1 I don't have any issues. Let me check several things.

oh... the only what I left in a project is my custom Utils library with IDisposable interface there.. conflict been caused because of that..
Sorry, ticket can be closed


About a Configurable Separator for Aliases

I would also agree with having one version and like @chetannanda wonder whether the / could confuse some things.
On a related "separators" topic, was this amendment ever made to allow studios to choose their preferred separator for aliases:
https://lists.aswf.io/g/ocio-user/message/565
Currently the config is full of aliases using _ as the word separator which makes them much harder to use predictably in filenames

Originally posted by @frenchie1980 in https://github.com/AcademySoftwareFoundation/OpenColorIO-Config-ACES/issues/74#issuecomment-1268294700

Good point, this is something we haven't done, rather than making it an option and thus producing a CG config incompatible with another CG config, we should maybe instead have both alias forms directly available.


Azure YAML pipeline - dependabot on Node/Express project

Hi
I am trying to run dependabot on a node/express project's package.json but I get the following error - I am assuming setting up and using dependabot is much more involved?

098f233eb6e8: Pull complete
ca3bb4b243af: Pull complete
d1d1f5dd4288: Pull complete
2527b2d06b57: Pull complete
35dae2e8b0e6: Pull complete
Digest: sha256:23b22c05e299afc1432ee09ec533a752d03003aa97b8f77cfe066661899e25be
Status: Downloaded newer image for tingle/dependabot-azure-devops:0.5
warning: parser/current is loading parser/ruby27, which recognizes
warning: 2.7.4-compliant syntax, but you are running 2.7.1.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
/home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:111:in `rescue in fetch_file_if_present': Dependabot::DependencyFileNotFound (Dependabot::DependencyFileNotFound)
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:90:in `fetch_file_if_present'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-npm_and_yarn-0.162.1/lib/dependabot/npm_and_yarn/file_fetcher.rb:55:in `package_lock'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-npm_and_yarn-0.162.1/lib/dependabot/npm_and_yarn/file_fetcher.rb:37:in `fetch_files'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:65:in `files'
	from ./update-script.rb:245:in `<main>'
/home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/clients/azure.rb:237:in `get': Dependabot::Clients::Azure::NotFound (Dependabot::Clients::Azure::NotFound)
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/clients/azure.rb:93:in `fetch_repo_contents_treeroot'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/clients/azure.rb:69:in `fetch_repo_contents'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:309:in `_azure_repo_contents'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:208:in `_fetch_repo_contents_fully_specified'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:179:in `_fetch_repo_contents'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:163:in `repo_contents'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:103:in `fetch_file_if_present'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-npm_and_yarn-0.162.1/lib/dependabot/npm_and_yarn/file_fetcher.rb:55:in `package_lock'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-npm_and_yarn-0.162.1/lib/dependabot/npm_and_yarn/file_fetcher.rb:37:in `fetch_files'
	from /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.162.1/lib/dependabot/file_fetchers/base.rb:65:in `files'
	from ./update-script.rb:245:in `<main>'
Using hostname = 'dev.azure.com', protocol = 'https', port = '443'.
Using 'https://dev.azure.com:443/' as API endpoint
Fetching npm_and_yarn dependency files for company/project/_git/repo
Targeting 'dev' branch under './' directory
Using 'bump_versions' requirements update strategy
##[error]The process '/usr/bin/docker' failed with exit code 1
Finishing: dependabot

My PAT has set permissions for:

- Code (Full)
- Pull Requests Threads (Read & Write).

The following is my YAML file:
trigger:
- dev

pool:
  vmImage: ubuntu-latest

steps:
   - task: dependabot@1
     inputs:
        packageManager: 'npm'
        directory: './'
        targetBranch: 'dev'
        versioningStrategy: 'auto'
        azureDevOpsAccessToken: '<PAT here>'
        targetRepositoryName: '<repo name here>'

Hi @samiujan , as pointed out by @MO2k4 , the issue is whether the dependency file (package.json for npm) is located. You can fix this by specifying the directory input to the correct path e.g. / for root or /frontend if you package.json file is located at frontend/package.json.
I will proceed and close the issue. Reopen if needed.


typo fix

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:bearice/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:bearice/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM
(CI failure is unrelated)

Please sign your commit first and push again.

LGTM


Update the Stylesheet for Updated Navbar

Please merge this PR also with #114 as it's the part of that only.

these small-small changes wont be accepted for the hacktoberfest.
I have already merged 3 of your PRs with these spammy changes.
Will label 'invalid' next time


update readme

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@fdfd87d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #14   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         2
  Lines             ?        96
  Branches          ?         0
==========================================
  Hits              ?        96
  Misses            ?         0
  Partials          ?         0

Impacted Files
Coverage Δ

pool.go
100.00% <ø> (ø)

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 fdfd87d...29354ca. Read the comment docs.


update Alert API page, include iOS specific functions from deprecated AlertIOS

Source:

https://github.com/facebook/react-native/blob/master/Libraries/Alert/Alert.js

This PR updates the existing Alert page and includes the following changes:

adds prompt method (iOS only, previously only available for AlertIOS)
adds Alert related types definitions
rewrites Android dismiss example as Snack
adds platform and required labels
adds small text tweaks

Preview

@mojvan Feel free to use the changeset comments.

@mojvan did you have any comments to add? I'm going to go ahead and merge. Feel free to submit a PR with any further changes you might deem worthwhile!


Node.js - Attachments, Cards and Actions

Hi,
I'm currently lookup the bot builder capabilities for the new version.
I saw that there are some examples about how to use cards, attachments and this kind of stuff through the .Net SDK.
Where can I found some example of that for Node.js?
Thanks

Some good examples here: https://github.com/Microsoft/BotBuilder/blob/master/Node/examples/demo-skype/app.js

Yes... Start with the demo bot. Better docs and more examples are coming.

Thank for your answers.
They were very usefull

When we will expect the Skype app (mac & web) will support new features like cards & carasouls.
Any date microsoft  allowance ?


Accept 2D motion data from webcam

Thanks for replying on HN. I think it'd be super fun to do this with the family: scan in the kids' drawings, turn on our webcam, and start moving around so that the drawn characters mimic our motions. Then all we have to do is record that, overlay some audio / voice, and we have ourselves a little makeshift family-friendly movie studio :)
Understandably this would be a ton of work but I just wanted to put it out there if there's interest!

i'm interested into this too, hope it will be implement soon :D

it sounds like realtime motion capture from video webcam
Vào Th 2, 17 thg 4, 2023 vào lúc 14:19 Aditya Naresh <
@.***> đã viết:

Sounds like a nice feature to work on. I think it's possible to reuse some
of the existing feature designs from other meta projects and slightly tweak
it to fit it here for the above purpose. Easier said than done :)
—
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/AnimatedDrawings/issues/98#issuecomment-1510831915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKC2O2ZZOKZ2R6QRGDVHPTXBTVIDANCNFSM6AAAAAAW66JYWQ
.
You are receiving this because you commented.Message ID:
@.***>

This seems like a good extension, but it'll be a bit of work and require some refactoring. In essence, the BVH class needs to be refactored into a 'Motion Source' class, which can be subclassed to support both 2D and 3D data. In addition, 'Motion Source' class can be subclassed to serve both full motion clips (as the BVH class current does) as well as streaming sources of motion data.

@hjessmith this feature will make AnimatedDrawings much more fun and solve the most difficult part of motion data capture,
plz consider support it.

I like this idea, but the webcam should be positioned well to achieve this. A rectangular shape can be placed in the center, indicating people should move within the rectangle in order to generate good quality animations.
This limit might reduce some development difficulties.

interested on this one as well.

I'm also looking forward to this feature.

great work,

Hello everyone, I have successfully exported a great demo that you can only provide drawings and video as input. The code can automatically generate 3D pose and corresponding BVH file from 2D data. All processes are offline and end-to-end in a virtual conda environment. The result is like this:

I have also successfully removed redundant OpenMMLab dependencies that may cause some problems in docker. That means you don't generate intermediate results from torchserver.

That's amazing! Looking forward to trying it :)

@Brian417-cup  that's exciting to hear! I'll be on the lookout for a PR from you. I'm sure folks will really enjoy it.

@hjessmith . Sure, I'm very glad to share the complete scripts. I also believe it will inspire others to find better solutions, especially for the projection method proposed in the thesis (Currently, it's static and pre-defined in the config file).

Thanks for submitting the PR. This is a big one- it's going to take me a while to sort though, and I foresee some potential licensing issues with Animated Drawings' MIT licensing if I merge the PR as is. I'll have to investigate the proper path forward here.
In the meantime, I'll drop a link to the PR here in case anyone wants to download it directly and give it a try:
https://github.com/facebookresearch/AnimatedDrawings/pull/277

Link to fork:
https://github.com/Brian417-cup/AnimatedDrawings
There is no "issues" or "discussion" section there...
Do you think you can add a web version?

@jumpjack I am afraid that I am very busy at these two month. I think if you have enough time, you can implement this by yourself. I have packed all processes as simple as possible. What you need to do is only check my main function python file with a few of paramemter. And if you have any problem in the operation, please feel free to contact with me directly on this issue.

unfortunately I have no idea about how to port a python app to web. :-(

@jumpjack I think you need to learn about the development knowledge related to front and back separation. In this project, the developer uses Flask as the backend and React framework as the front. You can write new request requests to the Flask framework and receive files from the front. Other operations can be freely arranged according to the author's previous functionality.

Hello, I tried using a virtual machine but it failed. Can you please check the reason? The configuration is 80GB of memory and 3090 Nvidia graphics card. The specific error is shown in the screenshot. I have tried setting up logs to view the reason for the error, but still cannot find the reason.

@657233220 Is the environment you installed right?  Actually, I only tested this program on windows platform, so it may occur problems on other platform.


docs(v2): add mr-pdg to resources

Added new option for PDF generation seeing as docusaurus-pdf doesn't work beyond 2.0.0.-alpha73, as establised in bug #29.

Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks, that looks nice
We need you to sign the FB CLA otherwise I technically can't merge this PR

Sorry, I missed the bit on CLA. I have signed this now.

@kohheepeace maybe we should remove the link to docusaurus-pdf at all, if there is better alternative as mr-pdf?

@lex111 Hi, mr-pdf is not for docusaurus only. So, I think only docusaurus-pdf is listed. I need to update docusaurus-pdf for v2 beta, but I can't say for sure when it will be. So, removing from the list is also ok for me.

@kohheepeace no worries, I think we only to list mr-pdf, which can work with any version of Docusaurus.

@antonygibbs thanks for update!


4.10.0-0.okd-2022-07-09-073606 fails node deployment due to corrupted nsswitch.conf file

OKD 4.10 Bootstrapping fails.  Master node kubelet reports "temporary failure in name resolution"
Discovered an error in the /etc/nsswitch.conf file.  Host entry shows:
hosts:      files myhostname resolve [!UNAVAIL=return] dns
Instead, it should be
hosts:      files dns myhostname
Fixed the file, rebooted nodes and master nodes are now coming up.


added link to preprint, fixed backward compatibility

Specified the paper with theoretical results upon which the implementation is based. Unconditional use of the backward compatibility code is replaced with running it only when needed (NumPy <= 1.12). Minor fixes: doctests in gromov-hausdorff.py, redundant import in test_distances.py,

Codecov Report

Merging #26 into master will decrease coverage by 0.26%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   80.03%   79.77%   -0.27%
==========================================
  Files          10       10
  Lines         531      534       +3
  Branches      105      105
==========================================
+ Hits          425      426       +1
- Misses         78       80       +2
  Partials       28       28

Impacted Files
Coverage Δ

persim/gromov_hausdorff.py
82.98% <50%> (-0.78%)
:arrow_down:

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 f8eb567...c172952. Read the comment docs.

I changed the code for backward compatibility in gromov-hausdorff.py from unconditional to executing only when NumPy is ≤ 1.12, see lines 518-521. As a result, these lines are now considered a miss by Codecov, which was sufficient to bring the total coverage down to below 80%. Should I still merge the pull request? Or is there a workaround I could implement?

Decided to merge PR to benefit gromov_hausdorff performance in the environments with NumPy > 1.12.


[bug]: yarn install Could not resolve "@chakra-ui/react"

OS
Linux
What version did you experience this issue on?
d94d4ef83f5be5dd10cbec3050a6b3a862ab9718
What happened?
yarn install results in numerous errors of the form
✘ [ERROR] Could not resolve "@chakra-ui/react"

    src/theme/components/button.ts:1:47:
      1 │ import { defineStyle, defineStyleConfig } from '@chakra-ui/react';
        ╵                                                ~~~~~~~~~~~~~~~~~~

  The Yarn Plug'n'Play manifest forbids importing "@chakra-ui/react" here because it's not listed as a dependency of this package:

    ../../../.pnp.cjs:38:33:
      38 │           "packageDependencies": [\
         ╵                                  ~~

  You can mark the path "@chakra-ui/react" as external to exclude it from the bundle, which will remove this error.

I've grown used to seeing some warnings about resolving chakra fly by in the frontend build, but sometime in the last few days it's gotten much louder and now ends in
✖ UNHANDLED ERROR
✖ ERROR → Error
ℹ REASON → Build failed with 49 errors:
src/theme/components/accordion.ts:1:42: ERROR: Could not resolve "@chakra-ui/anatomy"
src/theme/components/accordion.ts:5:7: ERROR: Could not resolve "@chakra-ui/styled-system"
src/theme/components/accordion.ts:6:21: ERROR: Could not resolve "@chakra-ui/theme-tools"
src/theme/components/button.ts:1:47: ERROR: Could not resolve "@chakra-ui/react"
src/theme/components/button.ts:2:21: ERROR: Could not resolve "@chakra-ui/theme-tools"
...

Full log: https://gist.github.com/keturn/ddf7f755c9909487a8eb67669c3d93a1

Have you tried deleting your node_modules and re-running? I am not seeing these errors locally

Removing node_modules was not sufficient, but somehow I had yarn.lock, package-lock.json, and package.json in my top-level git directory (not invokeai/frontend/web). Removing those cleared things up.


Issue with k8s.io/docs/concepts/configuration/assign-pod-node/

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

Hi. Is there a part of the page at https://kubernetes.io/docs/concepts/configuration/assign-pod-node/ that you think we can improve?

/triage needs-information

/close


400: User location is not supported for the API use without a billing account linked.

400: User location is not supported for the API use without a billing account linked.

解决方案在这里
https://github.com/Amery2010/TalkWithGemini?tab=readme-ov-file#%E5%B8%B8%E8%A7%81%E9%97%AE%E9%A2%98


Make Home page default + Correct Eslint errors and warnings

I propose these updates:

We make home page the default page. I.e. when people visit "/", the dummy home page is served. The getting-started page will remain in its original path, and a link to it will be added in the home page.
As of now, there are multiple eslint errors and warnings in many files. You can find a warning or error in almost every file you open. I want to correct these warnings and errors, to the best of my ability. I will be focusing mainly on the "frontend" directory.

I want to implement these updates.
Please assign this issue to me.

@Prantaneel-Pegu I have assigned this to you


group index of each group should be start on zero

Why new array of groups keep old index?

foreach ($collection as $index => $element) {
...
$groups[$groupKey][$index] = $element;

@dyorg it’s a design decision a made relatively early, that I want to preserve shape as much as possible as it’s always easy to not have that. What you can do is simply this:
use function Functional\group;
use function Functional\map;
use function Functional\partial_any;
use const Functional\…;

$groups = map(group(…, …), partial_any('array_values', …));


Setup GitHub Actions

Codecov Report

Merging #74 (39a7edc) into master (0c13852) will decrease coverage by 0.90%.
The diff coverage is n/a.

:exclamation: Current head 39a7edc differs from pull request most recent head 8aea46d. Consider uploading reports for the commit 8aea46d to get more accurate results

@@             Coverage Diff              @@
##             master      #74      +/-   ##
============================================
- Coverage     55.67%   54.77%   -0.91%
+ Complexity      568      557      -11
============================================
  Files           106      106
  Lines          2554     2554
  Branches        139      139
============================================
- Hits           1422     1399      -23
- Misses         1058     1073      +15
- Partials         74       82       +8

Impacted Files
Coverage Δ

...a/com/axibase/tsd/client/PlainStreamingSender.java
44.11% <0.00%> (-11.03%)
:arrow_down:

...om/axibase/tsd/client/DefaultStreamingManager.java
31.57% <0.00%> (-4.68%)
:arrow_down:

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 0c13852...8aea46d. Read the comment docs.


Extend dataset processing pipeline to update the dataset gene info when a new dataset is processed

Trent's prototyping / discovery work has determined that computing this at dry-run time is doable, so we will not pre-compute or persist dataset gene info anywhere


Prepare 0.2.0

same as been done for rosbag2.
Bumping 0.2.0 to master, cutting a rolling release and a galactic release

It compiles correctly over here on my side, but I haven't it in detail. Are you running into hiccups?

No actual issues - I just hadn't done this build in a while so it took me a sec to get it set up correctly. I also verify that it is building and tests pass.
How do we run CI again? With the following?

ci_packaging_linux
ci_packaging_linux-aarch64
ci_packaging_osx
ci_packaging_windows

Will you do the followups, or would you like me to?

Create and push branch galactic
Create and push tag 0.2.0
Update ros2/ros2.repos branch
Bloom to rolling
Bloom to galactic (and update source branches in rosdistro)

The tag 0.2 exists already

How do we run CI again? With the following?

We are currently not testing them in any nightlies. If we wanted to test them manually, only the linux packaging jobs make sense as we don't have a ROS1 installation on all other platforms.

The tag 0.2 exists already

Unfortunately that doesn't work right when we follow this process - if it was automatically created by catkin_prepare_release - since it refers to a commit that no longer exists, since this PR was squashed.
https://github.com/ros2/rosbag2_bag_v2/releases/tag/0.2.0 points to 36d497f but this PR squash-merged to  676475d

I've force-pushed the correct 0.2.0, and pushed the new galactic branch.

The ros2/ros2.repos don't have to be updated as these packages here are not part of it.

Of course, you're right. I was running on autopilot.

Ah, I knew this process would throw me off. Sorry for that. I guess at this point I actually don't know what to do. It looks like you're more familiar with the process of preparing a release on a separate branch. Do you mind following up here?

If we wanted to test them manually, only the linux packaging jobs make sense as we don't have a ROS1 installation on all other platforms.

Again, of course right haha. I forget that this is tied to ROS1 and so only targets Ubuntu.

Ah, I knew this process would throw me off. Sorry for that. I guess at this point I actually don't know what to do. It looks like you're more familiar with the process of preparing a release on a separate branch. Do you mind following up here?

All done. I'll run the bloom releases now too

I should probably write up a quick "ros-tooling release runbook" - the default bloom processes assume you just push commits to master, but I do like having branch protections and putting all changes through PRs with squash-merge. I'd personally prefer to avoid leaving that open just for the case of releases.

since it refers to a commit that no longer exists, since this PR was squashed.

Am I supposed to merge the commit here - opposed to "squash and merge" or what's the recommended way?

I like squash-merge and keeping the version-bump together with the changelist update. The way I do it is to say "no" to the part that asks if it should push the tag, then to manually tag after merging the PR

It's not 100% ideal but it allows us to keep the general branch configuration on, without exception. Open to alternative proposals of course

https://github.com/ros/rosdistro/pull/29567

Galactic https://github.com/ros/rosdistro/pull/29568

@emersonknapp Ima hijack this PR for a second, but can you confirm two things for me please?

the rosbag2_bag_v2_plugins compile and run (!) on an isolated build?
the test_rosbag2_play_rosbag_v2_end_to_end succeeds as-is on master?

I've just noticed that I can't get the tests to run (independently whether they pass or fail) due to linkage errors with class loader when my ros2 installation is from source with an --isolated build. It works only in a --merge-install workspace, really.
Once the tests are running, I see a reliable test failure in the play end-to-end test - which makes also sense as the Player exits with a return code other than zero when the RMW interface is not found. So I had to change line 109 to EXPECT_THAT(exit_code, Eq(EXIT_FAILURE));.

the rosbag2_bag_v2_plugins compile and run (!) on an isolated build?

I had also done colcon build --packages-up-to ros1_bridge (without --merge-install) - and was able to build against it in an overlay containing rosbag2_bag_v2_plugins - but the tests do not run successfully.
I am now trying the underlay with colcon build --packages-up-to ros1_bridge --merge-install - to see if this is more successful

Process followed:
cd /ros2_ws
colcon build --symlink-install --merge-install --packages-up-to ros1_bridge rosbag2 --packages-skip ros1_bridge
# finishes 157 packages
source /opt/ros/noetic/setup.bash
colcon build --symlink-install --merge-install --packages-select ros1_bridge --cmake-force-configure
# finishes 1 package (slow package! 10 mins)

# new shell
cd /bag_v2_ws
source /opt/ros/noetic/setup.bash
source /ros2_ws/install/setup.bash
colcon build
# 2 packages succeed
colcon test --merge-install
# one test fails

3 - test_rosbag2_play_rosbag_v2_end_to_end (Failed)
3: [ RUN      ] PlayEndToEndTestFixture.play_fails_gracefully_if_rosbag_v2_storage_id_is_not_specified
3: /bag_ws/src/rosbag2_bag_v2/rosbag2_bag_v2_plugins/test/rosbag2_bag_v2_plugins/test_rosbag2_play_rosbag_v2_end_to_end.cpp:109: Failure
3: Value of: exit_code
3: Expected: is equal to 0
3:   Actual: 1 (of type int)
3: [  FAILED  ] PlayEndToEndTestFixture.play_fails_gracefully_if_rosbag_v2_storage_id_is_not_specified (846 ms)

Ok, that matches what I see here as well - in both cases actually.
My take away from this is that the ros1 plugins can't be used correctly when build in isolation. Additionally, we should change line 109 to expect a failure for the test to pass correctly.

https://github.com/ros2/rosbag2_bag_v2/pull/45

My take away from this is that the ros1 plugins can't be used correctly when build in isolation.

Does this apply to the ros1_bridge in general? The README there does not suggest building with --merge-install


test mk_exp dann, dial

Codecov Report

Merging #326 (c2ed749) into master (b32b25b) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #326      +/-   ##
==========================================
- Coverage   95.32%   95.05%   -0.27%
==========================================
  Files         129      129
  Lines        4449     4449
==========================================
- Hits         4241     4229      -12
- Misses        208      220      +12

Flag
Coverage Δ

unittests
95.05% <ø> (-0.27%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add language support

Coverage decreased (-0.01%) to 99.669% when pulling f57e4466dda682aa0dd9e05f37acd300cb96c3e7 on language into 2f017f8a9a83bb984d5b141cf2a43669c6d11be4 on master.


Autonat v2 transport changes

This is the PR for the transport redesign. I hope I gited it correctly.
This is a follow up to #1

@thomaseizinger Can we merge that into the rust-libp2p PR?

@thomaseizinger Can we merge that into the rust-libp2p PR?

Yes!


stages: extend org.osbuild.sysconfig to create network-scripts/ifcfg-* files

Extend the org.osbuild.sysconfig stage to create ifcfg-* files under network-scripts subdirectory. It is possible to set only values currently set in RHEL AMI images, specifically:

BOOTPROTO
DEVICE
IPV6INIT
ONBOOT
PEERDNS
TYPE
USERCTL

Change all configure_* functions to raise ValueError exception, instead of returning values. As a follow up change, remove all checks of the returned value from these functions.
Update the org.osbuild.sysconfig stage test case to create ifcfg configuration files for two interfaces.
The schema string has been changed to raw string to prevent DeprecationWarning: invalid escape sequence \s warnings.

Looks good! Out of curiosity: do these network config files have any effect in the presence of cloud-init? I thought @gicmo found that for the guest images they do not (I'm all for adding the support either way)

Looks good! Out of curiosity: do these network config files have any effect in the presence of cloud-init? I thought @gicmo found that for the guest images they do not (I'm all for adding the support either way)

Yeah, this is a good point which I didn't consider. After reading cloud-init documentation and also inspecting booted RHEL image from AWS Marketplace, I can confirm that cloud-init does not obey the configuration file in the image and overwrites the ifcfg-eth0 configuration file. So in the end, creating the file is a bit useless on images with cloud-init. On the other hand, this is what is on RHEL AMI images and the question is whether not creating it would be acceptable by QE. I'm happy to bring it up though.

For the guest images we did not create this and it was the only known discrepancy, which in the end was accepted. I'm all for having the option though, so let's land this.

All review comments have been addressed.

Sorry, I had to amend the change to fix a CI fail due to too long line....


Hue light groups

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release: 0.61.1
Python release: 3.6
Component/platform: hue
Description of problem:
Every time you reboot hass, hue light groups get a new name.  For instance I have a group of lights called light.dining_room1-5.  The plain light.dining_room is the group.  When i reboot, the group gets changed to light.dining_room_2 so now i have to change all my code.  Every time i reboot it's a random change in numbers.
Expected:
Same group names remain instead of enumerating new ones.  Change hue light bulbs to light_bulb_1, light_bulb_2 seems to fix the problem.
Problem-relevant configuration.yaml entries and steps to reproduce:
none
Traceback (if applicable):
none
Additional info:

Fixed by https://github.com/home-assistant/home-assistant/pull/11979


[Enhancement Proposal] Add setting to disable wildcard topics

Hi guys, I have a proposal for a flag named disable_wildcard_topics which will default to off. The rationale for this is that many sources, including a talk at Defcon 2017 mention wildcard topics as a security issue if not implemented correctly by MQTT users. Recently we ran into an IOT privacy certification at work and pentesters found a way to exploit wildcards to write data to devices not owned by the current user. I did a fix as a plugin but thought to have this merged into the VerneMQ core as an optional setting could be a nice little security advantage over other brokers :). If you guys @dergraf @ioolkos and @larshesel agree with it i can go ahead and submit a PR.

@codeadict thanks! I'll have to research what the exploit is though. this is super interesting but needs careful thinking. I guess it's protecting users from unintentionally subscribing to topics that are too wide. Is that a case that can systematically be flagged (as dangerous, or "this is probably not what you want") or is it a completely legit MQTT subscribe scenario.

Ok, constructed an example and I think I get what you mean. It's basically an MQTT topic tree modelling issue.
*thinking what to do*

@codeadict browsing through older, issues, I think this is still a good enhancement. I guess the trick here is to find an easy way to disable wildcards. I'll eventually look into it.

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

@ioolkos I can try remembering what I did at the time. I did not benchmark it much but it was essentially ensuring the subscribe/publish packets did not have any # or +.


Typescript Declarations?

Could not find a declaration file for module 'react-native-country-codes-picker/components/CountryPicker'.

same with me,
error: Error: Unable to resolve module react-native-country-codes-picker/components/CountryPicker from

Hello @kinjal34 ! Good to see you here ! Can you describe please which one version of lib. do you use ?

Try update to version 1.3.2. and we will see maybe it will fix the issue.

@GeorgeHop thanks for reply,
but still getting same error with this version!!
version<EMAIL_ADDRESS>error which I am getting,
error: Error: Unable to resolve module react-native-country-codes-picker/components/CountryPicker from /Users/kinjal/Documents/Rn/accelevents/acceleventsapp/src/screens/DashBoardScreen/index.js: react-native-country-codes-picker/components/CountryPicker could not be found within the project or in these directories:
node_modules
../node_modules
../../../../node_modules
If you are sure the module exists, try these steps:

Clear watchman watches: watchman watch-del-all
Delete node_modules and run yarn install
Reset Metro's cache: yarn start --reset-cache
Remove the cache: rm -rf /tmp/metro-*
47 | import getEventFeedContext from "../../Component/getEventFeedContext";
48 | import WelcomeVideo from "../../Component/WelcomeVideo";

49 | import {CountryPicker} from "react-native-country-codes-picker/components/CountryPicker";
|                              ^
50 |
51 |
52 | const feedClientforAuth = getFeedContext();

I was asking for Typescript declarations like this :

Its not actually an issue but a feature I wish to see in upcoming versions

@Aryan6290 for sure I will try to add TS declaration in the next version.
If you want you can add it by yourself, I will check if everything is clear and merge it.
If not you should wait a little bit.

@kinjal34 have you tried steps from error ?
Also do you use expo or pure react-native ?

@kinjal34
I have tested it on brand new app. using this steps.

Created new app using expo init(expo version 43).
Installed package using yarn add.
And reproduce the basic sample from docs.

App.js
import React from 'react';
import {Text, View, StyleSheet, TouchableOpacity, Platform} from 'react-native';
import CountryPicker from "react-native-country-codes-picker";

export default function App() {
    const [show, setShow] = React.useState(false);
    const [countryCode, setCountryCode] = React.useState('');

    return (
        <View style={styles.container}>
            <TouchableOpacity onPress={() => setShow(true)}>
                <Text>
                    Test button
                </Text>
            </TouchableOpacity>
            <CountryPicker
                show={show}
                // when picker button press you will get the country object with dial code
                pickerButtonOnPress={(item) => {
                    setCountryCode(item.dial_code);
                    setShow(false);
                }}
            />
        </View>
    );
}

const styles = StyleSheet.create({
    container: {
        flex: 1,
        justifyContent: 'center',
        alignItems: 'center',
        paddingTop: 20,
        backgroundColor: '#ecf0f1',
        padding: 8,
    },
});

packaje.json
{
  "main": "node_modules/expo/AppEntry.js",
  "scripts": {
    "start": "expo start",
    "android": "expo start --android",
    "ios": "expo start --ios",
    "web": "expo start --web",
    "eject": "expo eject"
  },
  "dependencies": {
    "expo": "^43.0.0",
    "expo-calendar": "~10.0.3",
    "expo-status-bar": "~1.1.0",
    "react": "17.0.1",
    "react-dom": "17.0.1",
    "react-native": "https://github.com/expo/react-native/archive/sdk-43.tar.gz",
    "react-native-country-codes-picker": "^1.3.2",
    "react-native-web": "0.17.1"
  },
  "devDependencies": {
    "@babel/core": "^7.12.9"
  },
  "private": true
}

@Aryan6290 for sure I will try to add TS declaration in the next version. If you want you can add it by yourself, I will check if everything is clear and merge it. If not you should wait a little bit.

sure I will try to make it <3

@Aryan6290 it would be great man !
@kinjal34 any new info about the issue ?

@GeorgeHop
now issue in this index file
njal/Documents/Rn/accelevents/acceleventsapp/node_modules/react-native-country-codes-picker/index.js:
None of these files exist:

node_modules/react-native-country-codes-picker/components/CountryPicker(.native|.android.js|.native.js|.js|.android.json|.native.json|.json|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx)
node_modules/react-native-country-codes-picker/components/CountryPicker/index(.native|.android.js|.native.js|.js|.android.json|.native.json|.json|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx)

1 | export default from "./components/CountryPicker";

|                      ^

@kinjal34 Please provide more info.

What do you use expo or pure react-native ?
Can you provide source code ?
Can you show your package.json ?
Have you tried steps from the error ?

Without all required info I can't help you.

@GeorgeHop thanks for reply,
but still getting same error with this version!!
version<EMAIL_ADDRESS>error which I am getting,
error: Error: Unable to resolve module react-native-country-codes-picker/components/CountryPicker from /Users/kinjal/Documents/Rn/accelevents/acceleventsapp/src/screens/DashBoardScreen/index.js: react-native-country-codes-picker/components/CountryPicker could not be found within the project or in these directories: node_modules ../node_modules ../../../../node_modules
If you are sure the module exists, try these steps:

Clear watchman watches: watchman watch-del-all
Delete node_modules and run yarn install
Reset Metro's cache: yarn start --reset-cache
Remove the cache: rm -rf /tmp/metro-*
47 | import getEventFeedContext from "../../Component/getEventFeedContext";
48 | import WelcomeVideo from "../../Component/WelcomeVideo";

49 | import {CountryPicker} from "react-native-country-codes-picker/components/CountryPicker";
|                              ^
50 |
51 |
52 | const feedClientforAuth = getFeedContext();

@GeorgeHop
Its pure react native,
and I am using simple steps as mention in lid,

Same issue here

@ramisalem
I recommend this one,
https://www.npmjs.com/package/react-native-phone-number-input

Guys check your config please! Check comment in this issue it should help to you:
https://github.com/GeorgeHop/react-native-screens-swiper/issues/2#issuecomment-969183567

Still facing the same issue


SKBitmap.ByteCount should support images >2GB

The following code is problematic for large images, it throwns an overflow exception
 /// <summary>Returns the byte size of the pixels, based on the <see cref="P:SkiaSharp.SKBitmap.Height" /> and <see cref="P:SkiaSharp.SKBitmap.RowBytes" />.</summary>
    /// <value>The byte size of the pixels.</value>
    /// <remarks>Note: this truncates the result to 32-bits.</remarks>
    public int ByteCount => (int) SkiaApi.sk_bitmap_get_byte_count(this.Handle);

I suggest to convert this to a long or ulong?

We can't convert, but we can add link here:
https://github.com/mono/SkiaSharp/blob/main/binding/Binding/SKImageInfo.cs#L109
Maybe not the same code, but also to avoid the crash for now you can use bitmap.Info.ByteSize64 and see if that fixes it?


Elasticsearch.Net.UnexpectedElasticsearchClientException: expected:'Number Token', actual:'"1083729"', at offset:79

NEST/Elasticsearch.Net version: 7
Elasticsearch version: 7
Description of the problem including expected versus actual behavior: Elasticsearch.Net.UnexpectedElasticsearchClientException: expected:'Number Token', actual:'"1083729"', at offset:79
Steps to reproduce:

Method:

        await _elasticClient.SearchAsync<T>(
            s =>
                s
                .From(page * pageSize)
                .Size(pageSize));

            return resp.Documents as List<T>;

Class as T:

public class GhVacancy : BaseEntity
    {
        public string Specialization { get; set; }

        public string Name { get; set; }

        public long GhVacancyId { get; set; }

        public string Status { get; set; }
        public string AssignedToId { get; set; }
        public DateTime Timestamp { get; set; }
    }
public class BaseEntity
    {
        public string Id { get; set; } = Guid.NewGuid().ToString();
    }

JSON in ElasticSearch:

{
    "specialization": "Customer Support",
    "name": "Модератор",
    "ghVacancyId": "1083729",
    "status": "active",
    "timestamp": "2019-08-06T14:53:44.624252"
  }

The issue is that "ghVacancyId" is a string in JSON, but is a long property type on GhVacancy. The serializer is strict about the expected JSON type


md-sidenav on a component janks scrolling

Hey guys,

I don't know if this is a bug but I'm seeing lots of janky scrolling on mobile devices when I try to use a sidenav on a single component. Anyone noticed this?

Global application markup:
<!-- Main application -->
<md-sidenav-container>
   <md-sidenav #sidenav mode="over">
   Global menu
   </md-sidenav>

<router-outlet></router-outlet>
</md-sidenav-container>

This works great - pretty standard. The router-outlet is where I put all the content. But then on on of my routes I want to have another sidenav on the other side, and here it goes bad..
/mycomponent.html

<md-sidenav-container>

    <md-sidenav #clubfilter mode="over" opened="false" align="end">
        In-componet sidenav
   </md-sidenav>

     This is the main content of the component.

</md-sidenav-container>

The content in mycomponent is a long scrollable list. If I setup the sidenav it works - but scrolling becomes almost impossible on mobile devices - but it's still ok on desktop.
Any advice?
Thanks a bunch

Sorry I think the issue is related to som flex-container, closing the issue for now. Apologies.


🚸 Validate minimum uid length

https://github.com/laminlabs/lamindb/issues/2143

This is a breaking change, the uid field has been used to sync some external ids based on the definition is "the universal id of the record".

Your concern is addressed, @sunnyosun:

https://github.com/laminlabs/lamindb/pull/2167


Security issue with hoek

Hi,
I'm running the command npm audit to update all the libraries I can.
In my project, I use for example the ttn module, which has the following dependencies:
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>deduped
│ │ │ └─┬<EMAIL_ADDRESS>│ │ │   └──<EMAIL_ADDRESS>deduped
│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ ├──<EMAIL_ADDRESS>│ │ │ │ ├──<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ └──<EMAIL_ADDRESS>deduped
│ │ ├──<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ └──<EMAIL_ADDRESS>deduped

Audit found two issues with hoek package, patched for versions > 4.2.0 < 5.0.0 || >= 5.0.3. This package is used by sub-dependencies of jsonwebtoken.

Sure, I'll try to spare some time for this.

Great, please see https://help.github.com/articles/creating-a-pull-request/

Any update on this?

Looks like we never published this to npm. This repository is craving for CD...
Should be good now.


Issue on page /content/glossary/glossary.html

Correct a typo:
Metadata
annotation about the data. Different types of metadata exit depending on their function
exit -> exist

PR #8


Add a Blog Website

Is your feature request related to a problem? Please describe.
I would like to add a blog website.
Describe the solution you'd like
It will be responsive and built using HTML, CSS and JavaScript.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@Ayan-thecodeking please assign me this issue under hacktoberfest.

sure @sshiwangi make a PR with deployed link


Request 1.15.2

Please tell if there are plans to update soon

I would really like this mod to be updated, my friends and I like it so much we are creating a mod pack and would like to add it but can't because it isn't 1.15.2


Add backing image encryption and clone support on UI

What this PR does / why we need it

allow user to clone backing image with encrypt / decrypt / ignore options
show a "lock" icon besides encrypted backing image.
refactor

fix icon beside backing image name
put nodeStatusColorMap and diskStatusColorMap in utils/status.js

Issue
[UI][FEATURE] Add backing image encryption and clone support
Test Result
Encrypt backing image

Decrypt backing image

Simple clone (choose ignore) backing image

Simply clone encrpyted backing image

Additional documentation or context
Note 1. This PR needs wait for longhorn-manager and backing image manager PRs merge first.
- https://github.com/longhorn/longhorn-manager/pull/2876
- https://github.com/longhorn/backing-image-manager/pull/251
Note. longhorn-manager need below supports, I will implement this feature in next PR.

copy encrypted backing image should have copied secret and secretNamespace
restore bbi should allow bring payload

POST http://localhost:8080/v1/backupbackingimages/parrot?action=backupBackingImageRestore
payload
{
 secret: xxx,
 secret-namespace: ooo
}

cc @ChanYiLin


Release Beta8 of the framework

Summary
The title says it all
Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[x] Release
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Example scenarios has been updated / added
[ ] Methods and classes used in PR scenarios are meaningful
[ ] Commits are well encapsulated and follow the best practices

looks like gpg_private_key is missing as we do use v6 of that GH action and I guess it has been updated in minor

Github actions can only see secrets from the trunk repository. Forks PRs don't have access to secrets.
The releases PRs need to happen from this repository.

arg! thanks @mjurc I didn't check

Almost makes me think we should have a check for the origin too for when .github/project.yml is changed.

+1

https://github.com/quarkus-qe/quarkus-test-framework/issues/1179


Implement OAuth

We should have some way to get an OAuth token. Example

Do you want to implement a full OAuth authentication? Only the token should be enough, or not?

@rodkings we should provide a way so that someone could get an auth token, so we can use the parts of the API that require one. maybe a class where you can call into it with the redirect endpoint string, the user of this could implement that endpoint elsewhere, and then could capture the token.

@rodkings if you want to tackle this I am more than happy to assign this issue to you.

@rodkings if you want to tackle this I am more than happy to assign this issue to you.

I will be glad to help. Right now, I really need to make authenticated requests, so, I was thinking of implementing first authenticated requests and later, provide a way to get the token. Right now I already have a MVC app getting the token with the Owin provider. What do you think?

@rodkings thats totally fine man

I'll give this a shot in the meantime :)

@tparnell8 Great, I will probably start as soon as I fix some EF6 issues on my app.


Update Dockerfile

Added execute to launch_server.sh

I have just now seen the issue and fixed it myself. Thank you for creating the request the updated image will be available in a few minutes


How to extend timeout?

The SPARQL endpoint can execute the query, but Yasgui does not wait for the triplestore reply.
Could you please help me?
Thanks!

Right now there is no way to configure timeout because the default timeout of HTTP client library "superagent" is used. See #150 for a request to replace the library.


Error install

Registry url: https://registry.npmjs.org/
Current Time: 28.05.2015 18:35:03
Last Refreshed: 14.05.2015 19:40:52
Number of Results: 157348
====Executing command 'npm install asyncawait --save '====
npm WARN package.json<EMAIL_ADDRESS>No repository field.

<EMAIL_ADDRESS>install C:\Users\Jenok\Documents\Visual Studio 2013\Projects\NodejsWebApp3\NodejsWebApp3\node_modules\asyncawait\node_modules\fibers
node ./build.js
child_process: customFds option is deprecated, use stdio instead.
C:\Users\Jenok\Documents\Visual Studio 2013\Projects\NodejsWebApp3\NodejsWebApp3\node_modules\asyncawait\node_modules\fibers>if not defined npm_config_node_gyp (node "C:\nodejs\node_modules\npm\bin\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild --release )  else (rebuild --release )
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp ERR! stack     at failNoPython (C:\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:103:14)
gyp ERR! stack     at C:\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:64:11
gyp ERR! stack     at FSReqWrap.oncomplete (evalmachine.:95:15)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "node" "C:\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild" "--release"
gyp ERR! cwd C:\Users\Jenok\Documents\Visual Studio 2013\Projects\NodejsWebApp3\NodejsWebApp3\node_modules\asyncawait\node_modules\fibers
gyp ERR! node -v v0.12.2
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok
Build failed
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\nodejs\\node.exe" "C:\nodejs\node_modules\npm\bin\npm-cli.js" "install" "asyncawait" "--save"
npm ERR! node v0.12.2
npm ERR! npm  v2.7.4
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node ./build.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node ./build.js'.
npm ERR! This is most likely a problem with the fibers package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./build.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls fibers
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\Jenok\Documents\Visual Studio 2013\Projects\NodejsWebApp3\NodejsWebApp3\npm-debug.log

====npm command completed with exit code 1====

This is an error installing node-fibers. It is not an error with asyncawait.
asyncawait depends on node-fibers to provide its coroutines.
node-fibers is a binary module so it has a special compilation step. This step can be skipped if you use node version 0.10.x, because it bundles pre-built binaries for established node versions.
If you are using iojs or node 0.12.x, then read on.
Unfortunately, node-fibers is less actively maintained at the moment so it hasn't kept up with the latest node and iojs changes. It's still possible to install node-fibers on these newer versions, but requires doing the build step. This step is automatic, by relies on node-gyp, which requires python to be installed on your computer. The node-fibers or node-gyp READMEs should have more instructions.
For iojs even that is not enough. Many binary modules (such as node-fibers) keep breaking with the rapid iojs update cycle, because they compile against the underlying V8 version which is frequently updated in iojs, and its changing API breaks builds. node-gyp itself needs special steps to get working on iojs.
I can't give many more details than that but hopefully you get the idea:

avoid iojs for now if you depend directly or indirectly on binary modules (such as node-fibers)
if you are using node 0.12.x, try installing python first, then your problem should be fixed
if the above fails, try using node 0.10.x and it should install without problems.

ok. i install Python and Python Tools for Visual Studio, but new error:
gyp ERR! stack Error: Python executable "python" is v3.4.3, which is not supported by gyp.
It looks like a joke on me, no desire even to reinstall the old version ... ((

ok. i install Python and Python Tools for Visual Studio, but new error:

gyp ERR! stack Error: Python executable "python" is v3.4.3, which is not supported by gyp.

It looks like a joke on me, no desire even to reinstall the old version ... ((

FYI From the node-gyp README:

On Windows:
Python (v2.7.3 recommended, v3.x.x is not supported)
Make sure that you have a PYTHON environment variable, and it is set to drive:\path\to\python.exe not to a folder.

Also, no need for PTVS, only node-gyp needs python.


Broken links for frontmatter additional css scripts

In my frontmatter, I have
css:
- css/some-custom.css

and in previous versions of obsidian-advanced-slides, it loads the script in plugin's css folder just fine. But a recent update seems to have broke it:

Finally, thank you for contributing this plugin to Obsidian! It has allowed me to customize reveal.js much more straightforwardly.

Nice find, will be fixed with next version


[FormState] Pass lastSubmitErrors to render prop

What is this PR doing?
This PR adds lastSubmitErrors to FormState's FormDetails.
lastSubmitErrors describes the most recent set of validation errors that arise when the user attempts to submit an invalid form (with validateOnSubmit being set to true).
lastSubmitErrors only updates on submit attempts (i.e. formDetails.submit()); inline corrections/validations between submit attempts will not change lastSubmitErrors.
This is helpful in cases where specific UI (such as ErrorBanner) may need to be updated on submit attempts, but not on field changes. (e.g. here and here)

I think a slightly better way to handle this would be to store what we currently call errors as remoteErrors, do propagation down to fields using those, name the new state field we are adding here clientSubmitErrors, and then when it comes time to actually push errors down into the render prop combine them into the errors prop.

Makes sense 👍 I guess something we would have to watch out for is if existing consumers are doing something with errors with the expectation that they're only ever going to be remote errors, then they might have to make some adjustments.

Something that could work is to inject the validation errors into errors on submit. since we shouldn't be injecting any remote/submit errors anyways (since the validation failure should prevent the submit), it should be safe to just dump validation errors in there. That way there is a single pattern for surfacing form errors, always use errors. This should be either a no-op change for existing usage, or a nice improvement in error visualization if the consumer is using form validation.

@patsissons  the only issue I see with dumping it into errors is that those get propagated down to fields if they have a fieldPath

could we omit the field when collecting validation errors? i don't believe it's used for the error banner

Just waiting on a final ✅ from @TheMallen (or someone else on Foundations) and I'll rebase and merge 👍 🙏


Multimodal generation in one pass

Hi, thanks for sharing this awesome work.
As I was trying your system more and more, a few questions popped up in my mind:

In my experience, I am seeing instances where the LLM starts generating text but instead of finishing the sentence, it generates [IMG0] token and then because of forcibly adding the remaining IMG tokens, that text output remains incomplete. Whatever text tokens the model generates after the first batch of [IMG] tokens are mostly garbage.  What could be the possible solution?
I noticed that you had sf=1.4 in your notebook for multimodal dialogue generation. How did you choose this hyper-parameter to balance the generation between text and image? Was it the same for the 4 examples in figure 5 from your paper? Same questions go for num_words and min_word_tokens.
Did you evaluate GILL's text generation ability on any task e.g. VQA or plan to do so in future?

Thanks for your feedback. In the above code snippet, because of the conditions in line 517, this generate function will never add the remaining [IMG] tokens in a batch generation, right?

That's correct, batch generation is not really supported in this function for this reason (It might be good to add an assert here that the input has batch size = 1...), and it's tricky to get it working properly because of different sequence lengths.

Adding an assertion here will break validation as during validation, your script uses the generate function on a batch.
https://github.com/kohjingyu/gill/blob/4a0a4ef7079d7e58cc7990753f5356e14bbd4bb8/gill/validate.py#L177
For the same reasons I have noticed that the sample generated examples from here actually have only [IMG0] token.
https://github.com/kohjingyu/gill/blob/4a0a4ef7079d7e58cc7990753f5356e14bbd4bb8/gill/validate.py#L188-L189

Ah yes, you're right, I was thinking about adding it to generate_for_images_and_texts().

For the same reasons I have noticed that the sample generated examples from here actually have only [IMG0] tokens (if any).

This is true, but I think for generating captions / VQA results it does not really matter, since we only want the text, and not images. So in these cases you would always set min_word_tokens to be the same as num_words, so that IMG tokens don't appear. I think this would be the main use case for batched inference. Since you never produce images it would be much easier to handle batches > 1.


I cannot add existing test to test suite

We already have load on scroll feature! can you please confirm if that resolves your issue?

@YuliyaSavulyak can you address above comment ^

Hello, @YuliyaSavulyak, have you had a chance to try out the "load on scroll" feature? Please let us know if you're still experiencing any issues with it!

Hi @YuliyaSavulyak
We are listing all the tests with scroll behaviour, Can you please confirm that works for you as well?

Hello. Yes, it was fixed. Thank you!

Thanks @YuliyaSavulyak for confirming!


Does it have team functionality

Is it for single user or can have teams as well?

Hi Ugintl! thanks for the star. Currently the page doesn't have team functonality. To be honest, I´ve done it to practise coding skills. I still have a lot to learn, right now I'm not sure to know how to add the team funcionality. Of course any comments or suggestions are welcome!

De: ugintl @.>
Enviado: sábado, 27 de noviembre de 2021 14:28
Para: carogaltier/KanbanBoard_website @.>
Cc: Subscribed @.***>
Asunto: [carogaltier/KanbanBoard_website] Does it have team functionality (Issue #1)
Is it for single user or can have teams as well?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/carogaltier/KanbanBoard_website/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMXQGCNUHM5YOMMDBFKBTH3UOEIKZANCNFSM5I4MTUOQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Support h2c via Upgrade: header

As specified in RFC7540§3.2, allow HTTP/1.1 connections to upgrade to h2
Most of the plumbing is in place for this in the form of Bandit.DelegatingHandler. For examples of how to change protocols on the fly, see Bandit.InitialHandler's protocol sniffing implementation.

Planning issues are being moved; this is still on the books for 1.0


Mraj mapping column

Pull Request
🤨 Rationale

👩💻 Implementation

🧪 Testing

✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

Closing this PR since changes got merged via several other mapping column PRs.


Fix Communication JobRouter Samples Issue

Fixes #38445
@scottaddie , @jsquire , @mayurid and @acsdevx-msft to notification

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

/azp run net - communication - ci

@scottaddie , @jsquire , @mayurid and @acsdevx-msft Could you help review this PR? Thanks a lot.


Iteration Counter is twice the number of generated tokens in inflight-batching mode

Hi, when I enabled the logging of triton server (--log-verbose=2), I found the Iteration Counter is always two times of the number of max_tokens I set in the input request. Any explanation? If I understand the batching correctly, the number of iterations should exactly correspond to the number of generated tokens? Thanks.
For example, When I set the batch size to 1, and max_tokens to 100, the Iteration Counter will increase by 200 in each run of the single request.
I1120 02:29:01.795279 42888 libtensorrtllm.cc:1009] {"Active Request Count":1,"Iteration Counter":200,"Timestamp":"11-20-2023 02:29:01"}

Any update on this? Or it is a known issue? Thanks. @schetlur-nv
So far, my benchmark run shows the tensorrt-LLM gives good performance, but once running it with Triton, the performance degrades too much. I am curious what's happening in the in flight batching.

Hey @lly-zero-one!
You are correct - the iteration counter should be incremented once for every generated token. Can you share your test script and the triton config file? It will help me understand what is happening a bit better.

You can just run the e2e test in https://github.com/triton-inference-server/tensorrtllm_backend/blob/main/tools/inflight_batcher_llm/end_to_end_test.py
python end_to_end_test.py --max_output_len 100 --dataset alpaca_data.json
Starting the triton server beforehand:
add the --log-verbose=3 in the lauch_triton_server.py command
cmd += ' -n 1 {} --model-repository={} --disable-auto-complete-config  --log-verbose=3 --backend-config=python,shm-region-prefix-name=prefix{}_ : '.format(
python scripts/launch_triton_server.py --world_size=1  --model_repo=./YOUR_llama_7b_model_repo

@lly-zero-one I am closing this issue now. Please feel free to leave a comment here or open a new issue for more questions.


Require --password-encryption for add-user

Require user supply --password-encryption flag when adding user (which would be used for a remote endpoint)
e.g.
sqlcmd config add-user --username sa --password-encryption none
or
sqlcmd config add-user --username sa --password-encryption dpapi
sqlcmd config add-user --username sa --password-encryption keychain
etc.
Change the sqlconfig file from:
users:

name: stuartpa@mssql
authentication-type: basic
basic-auth:
username: stuartpa
password-encrypted: false
password: REDACTED

to:
users:

name: stuartpa@mssql
authentication-type: basic
basic-auth:
username: stuartpa
password-encryption: none
password: REDACTED

or
users:

name: stuartpa@mssql
authentication-type: basic
basic-auth:
username: stuartpa
password-encryption: dpapi
password: REDACTED

etc.

the content of this Pr doesn't match the description

the content of this Pr doesn't match the description

I'm working on it! :-) (It's still in "draft" state, but thanks for the feedback so far)