Add package version information file with automatic update

A baked version number in a dedicated file can improve usage for end users.
Example file: https://github.com/gboeing/osmnx/blob/main/osmnx/_version.py

It is recommended by pep8, but adding it would require changing version in 2 places on bump. See if it is possible to automatically generate it e.g. from pyproject.toml

See https://packaging.python.org/en/latest/guides/single-sourcing-package-version/ for more


Fix informer factory usage

Commit 060979e introduced a problem when refactoring the workqueue
code. It solely used the NetworkService informer, which meant that
we were not properly dequeueing NetworkServerChannels or
NetworkServiceEndpoints. This commit corrects this bug.
Signed-off-by: Kyle Mestery<EMAIL_ADDRESS>

@mestery I see where the problem was, Endpoint and Channel workforver were using Network Service's Informer. I am curious since you are using a shared informer and it is supposed to be share by multiple type of watched objects, what was the reason behind to go with SharedInformer for each type of object?

@sbezverk This is a good point, at one point I was hitting an issue with trying that, but in looking at the code now, I think it can be fixed. I can do it as a separate PR, since this does fix an actual bug. Thanks!


Excellent article

This is an excellent article! I was wondering though if you could expand a bit on some of the other IOC containers such as AutoFac and Prism and how they compare to TinyIoc when creating Xamarin Forms applications with MVVM?  Also is there any plan to enhance Xamarin Forms to add some of these Dependency Injection features?

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

ID: 496aefc1-60e7-3484-12d1-ba383cfc099b
Version Independent ID: 35133eba-8bc2-a54e-b341-d873f9a3c468
Content: Dependency Injection - Xamarin
Content Source: docs/xamarin-forms/enterprise-application-patterns/dependency-injection.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @delaney31
See https://github.com/dotnet-architecture/eShopOnContainers/issues/914 for a discussion of Autofac vs TinyIoC, and a link to IoC container performance on the Xamarin platform. Hope it helps.
There are some plans around Xamarin.Forms and DI but afraid I can't say more than that right now.


Zend Server Deployment Check Task - Required Fields and Fails on ZS 8.5

There is a missing * required field on Zend Server Version paramater. Also, when doing a post deployment status check, this error occurs:
23-Sep-2015 16:57:52	Starting task 'Zend Server Deployment Check Task' of type 'org.zend.zendserver.bamboo.plugin.zendserver:ZendServerDeploymentCheckTask'
23-Sep-2015 16:57:52	Preparing test runs (in Bamboo-Deploy context).
23-Sep-2015 16:57:57	/home/bamboo/xml-data/build-dir/1703943-1769482/installApp-null.xml
23-Sep-2015 16:57:57	Finished task 'Zend Server Deployment Check Task' with result: Failed
23-Sep-2015 16:57:57	Finalising the build...
23-Sep-2015 16:57:57	Stopping timer.

Thanks for notifying.
Added the required field in #13. Please try the updated jar file.
However, I couldn't see any issues with the Deployment Check Task. Could you please provide the command which has been executed (you can find it in the logs) plus the output generated by this command?
If you're not interested in the Rollback functionality of the Check Task, you can also try the '--wait' custom param which can be added in the Deployment Task definition. The Deployment Task itself will wait until the deployment has finished.

That's the main issue it is not executing any command to use. I've been able to use the sdk to figure out most of the errors but this one has got me, this is what I've go so far from a fresh run with the latest. Same as before, the output log for the 'Zend Server Deployment Check Task' does not match the current deployment like the 'Zend Server Deployment Task'. I am getting a installApp-null.xml vs installApp-5.xml as an export to a non-existant command that is run. I've added whitespace to the log for clarity and included the deployment task logs below.
30-Sep-2015 11:22:12	Starting task 'Zend Server Deployment Task' of type 'org.zend.zendserver.bamboo.plugin.zendserver:ZendServerDeploymentTask'
30-Sep-2015 11:22:12	Deployment (in Bamboo-Deploy context) has started...
30-Sep-2015 11:22:17	Searching for ZPK file from custom definition
30-Sep-2015 11:22:17	No custom ZPK file specified.
30-Sep-2015 11:22:17	Searching for ZPK file from Artifact Download
30-Sep-2015 11:22:17	Artifact Downloader Task found
30-Sep-2015 11:22:17	ZPK directory configuration found:
30-Sep-2015 11:22:17	ZPK found: /home/bamboo/xml-data/build-dir/3768321-3866625/5.zpk
30-Sep-2015 11:22:17	Set Process Timeout to 60 sec
30-Sep-2015 11:22:31	Process command line: /usr/local/bin/zs-client installApp --zpk=/home/bamboo/xml-data/build-dir/3768321-3866625/5.zpk --baseUri=http://testing.avian-x.com --userAppName=avianx-com --zsurl=http://<IP_ADDRESS>:10081 --zskey=admin --zssecret=ca32da3d79356e24cc5ff953a415fce05a46a8acdbefd7af542df03ed7dd331d --zsversion=8.5
30-Sep-2015 11:22:31	Output written to: /home/bamboo/xml-data/build-dir/3768321-3866625/installApp-5.xml
30-Sep-2015 11:22:31	Checking Zend Server output for errors
30-Sep-2015 11:22:31	Finished task 'Zend Server Deployment Task' with result: Success
30-Sep-2015 11:22:31	Starting task 'Zend Server Deployment Check Task' of type 'org.zend.zendserver.bamboo.plugin.zendserver:ZendServerDeploymentCheckTask'
30-Sep-2015 11:22:31	Preparing test runs (in Bamboo-Deploy context).
30-Sep-2015 11:22:37	/home/bamboo/xml-data/build-dir/3768321-3866625/installApp-null.xml
30-Sep-2015 11:22:37	Finished task 'Zend Server Deployment Check Task' with result: Failed

installApp-null.xml is weird... it should get the same file name as used in the Deployment Task. In this case there should be a '5' instead of 'null', which is the build number - obviously not available for the Check Task.
Are the both jobs running in the same stage?
However, in my forked private repo https://github.com/janatzend/ZendServerBamboo/tree/master/target you can find a jar file with more verbose debug output. I'd like to see at which step the Check Task fails.
Thanks,
Jan

Agreed! Both jobs are on the same deployment env "Staging". The build stages are curretly lint->package and the zpk is the artifact of package. Not sure what's going on but I was able to get the version bump but no verbose outputs, below is todays run with 1.2.2-debug
Starting task 'Zend Server Deployment Task' of type 'org.zend.zendserver.bamboo.plugin.zendserver:ZendServerDeploymentTask'
01-Oct-2015 09:08:38	Deployment (in Bamboo-Deploy context) has started...
01-Oct-2015 09:08:43	Searching for ZPK file from custom definition
01-Oct-2015 09:08:43	No custom ZPK file specified.
01-Oct-2015 09:08:43	Searching for ZPK file from Artifact Download
01-Oct-2015 09:08:43	Artifact Downloader Task found
01-Oct-2015 09:08:43	ZPK directory configuration found:
01-Oct-2015 09:08:43	ZPK found: /home/bamboo/xml-data/build-dir/3768321-3866625/5.zpk
01-Oct-2015 09:08:43	Set Process Timeout to 60 sec
01-Oct-2015 09:08:52	Process command line: /usr/local/bin/zs-client installApp --zpk=/home/bamboo/xml-data/build-dir/3768321-3866625/5.zpk --baseUri=http://testing.avian-x.com --userAppName=avianx-com --zsurl=http://<IP_ADDRESS>:10081 --zskey= --zssecret=--zsversion=8.5
01-Oct-2015 09:08:52	Output written to: /home/bamboo/xml-data/build-dir/3768321-3866625/installApp-5.xml
01-Oct-2015 09:08:52	Checking Zend Server output for errors
01-Oct-2015 09:08:52	Finished task 'Zend Server Deployment Task' with result: Success
01-Oct-2015 09:08:52	Starting task 'Zend Server Deployment Check Task' of type 'org.zend.zendserver.bamboo.plugin.zendserver:ZendServerDeploymentCheckTask'
01-Oct-2015 09:08:52	Preparing test runs (in Bamboo-Deploy context).
01-Oct-2015 09:08:57	/home/bamboo/xml-data/build-dir/3768321-3866625/installApp-null.xml
01-Oct-2015 09:08:57	Finished task 'Zend Server Deployment Check Task' with result: Failed
01-Oct-2015 09:08:57	Finalising the build...
01-Oct-2015 09:08:57	Stopping timer.
01-Oct-2015 09:08:57	Build 3768321-3866625-5046283 completed.
01-Oct-2015 09:08:58	Finished processing deployment result Deployment of 'develop-5' on 'Staging'

ok, I think I found the issue. After I was searching in the wrong direction for quite a while, I was able to reproduce the issue.
Please check out the new zendserver.jar resp. zendserver-1.2.2-2.jar

Good to go! Thanks janatzend and Cheers!
Ubuntu 14.04 -> zendserver-<IP_ADDRESS>.jar


Angle and CobbAngle tool: set default 1 for rowPixelSpacing and columnPixelSpacing

Only added in the Angle and CobbAngle tool: rowPixelSpacing and columnPixelSpacing default to 1

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the current behavior? (You can also link to an open issue here)
When there are no pixel spacing data of the image, the values of the angle measurements are not able to be calculated, since the getPixelSpacing function returns null for the rowPixelSpacing and columnPixelSpacing values.

What is the new behavior (if this is a feature change)?
Value 1 is used in the angle value calculation if the rowPixelSpacing and/or columnPixelSpacing values are null.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.

Other information:
Default value 1 is added since in the past every tool had it, but due to some refactoring (in which the getPixelSpacing function was created), this scenario was left out.

@JamesAPetts, this doesn't change labeling from px to mm, but it does assume square pixels. That feels like a safer assumption. Thoughts on merging this?

Any idea when this will be merged? Or is it blocked by the codeclimate issue?

So, all checks have now passed. When could we expect the merge?

@dannyrb some info on this?

@JamesAPetts can we have some update on this?

Appreciate your patience on this one. I'll merge, but may revert if someone comes forward and forces the non-square pixel argument.
We may eventually want this behavior as an optional flag that can be sit when cornerstone.init() is called.

:tada: This PR is included in version 4.15.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@dannyrb Thank you very much! :)


RecyclerView fastScrollEnabled not working

Steps to reproduce
Hello
I have a fragment with a RecyclerView (androidx.recyclerview.widget.RecyclerView), with the fastScrollEnabled=true attribute.

It helps our users, by clicking on the scrollbar they can move faster in the list.
It works on an Android tablet (fortunately) and on Chrome OS (I believe by clicking on the scroll wheel), but not on WSA.
✔ Expected Behavior
Can use the component like in an Android tablet or in Chrome OS
❌ Actual Behavior
Non-clickable, nothing happens
Other Software
No response
Please specify the version of Windows Subsystem for Android
2301.40000.7.0

We'll investigate and follow up on this. Thank you for bringing this to our attention. @nieubank is this a scenario we've seen?

Tracking this under #256. This is due to our scrolling emulation for app compatibility.


aws_cloudfront_distribution.main.origin.domain_name specified twice and second is wrong

The domain_name property is provided twice:
https://github.com/riboseinc/terraform-aws-s3-cloudfront-website/blob/bfa5677b2a9eb6ff3e925dceb8e241741b345079/cloudfront.tf#L12-L19
This leads to a difficult to troubleshoot problem because everything looks right but it's subtlely wrong, or at least it didn't work for me.
It seems that either line works in my tests, but the second needs a slight modification:
domain_name = "${var.fqdn}.s3-website.${data.aws_region.main.name}.amazonaws.com"

Note the . instead of -.

On a whim, I tried uncommenting https://github.com/riboseinc/terraform-aws-s3-cloudfront-website/blob/bfa5677b2a9eb6ff3e925dceb8e241741b345079/cloudfront.tf#L18 and it worked, so that must have been added in the meantime. I'll open a PR that uses that instead.

@colindean thank you for the report, I admit the double-set is a ridiculous error. 😭
However, in my tests (our sites), the domain_name value is indeed set to something like:
example.com.s3-website-us-east-1.amazonaws.com, with the hyphen ("-") instead of the period (".").
Are you seeing something different?
P.S. The bug specified in the note is still not fixed (https://github.com/terraform-providers/terraform-provider-aws/issues/1117)

@colindean just a note regarding the uncommenting -- previously, it will work IFF you have already applied once (i.e. the S3 bucket has already been created). It did not work when it the S3 bucket needs to be created.
Did it work in the latter case?

Are you seeing something different?

Yes. example.com.s3-website.us-east-2.amazonaws.com. I don't know if that's specific to us-east-2, but I doubt it.
Using domain_name = "${aws_s3_bucket.main.bucket_domain_name}" seemed to work, too, but…

It did not work when it the S3 bucket needs to be created.

Ohhhhhhhh yeah, the bucket already existed when I tried both of those previously commented-out lines.

Yes. example.com.s3-website.us-east-2.amazonaws.com. I don't know if that's specific to us-east-2, but I doubt it.

I just created a site yesterday in us-west-2, but it gives me the hyphen...?
www.example.org.s3-website-us-west-2.amazonaws.com
The problem is the hyphen didn't work for you? I just tried both the "dot" and the "hyphen" and they both seemed to work?

http://staging.abstractions.io.s3-website.us-east-2.amazonaws.com succeeds
http://staging.abstractions.io.s3-website-us-east-2.amazonaws.com fails

Closed in #6 . Thank you @colindean !


chore: rename revision to match naming scheme

SUMMARY
When creating a new migration, I was confused because the down_revision didn't match any of the migration files that existed. Turned out the revision hash didn't match the filename for this migration, so i'm renaming the file to match. This should be a safe change since the revision number is remaining the same, only the file name is changing.
TEST PLAN
CI
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

to: @amitmiran137 @john-bodley @dpgaspar @villebro

Codecov Report

Merging #12950 (597932d) into master (9982fde) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #12950   +/-   ##
=======================================
  Coverage   69.14%   69.14%
=======================================
  Files        1025     1025
  Lines       48767    48767
  Branches     5188     5241   +53
=======================================
+ Hits        33718    33720    +2
+ Misses      14915    14913    -2
  Partials      134      134

Flag
Coverage Δ

cypress
50.83% <ø> (ø)

javascript
61.84% <ø> (ø)

python
67.62% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...11ccdd12658_add_roles_relationship_to_dashboard.py
0.00% <ø> (ø)

...set-frontend/src/dashboard/util/getDropPosition.js
92.06% <0.00%> (ø)

superset/db_engine_specs/presto.py
88.31% <0.00%> (+0.43%)
:arrow_up:

Continue to review full report at Codecov.

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


CII best practices is not recognized

If I run
howfairis https://github.com/fair-workflows/nanopub

I get:
Checking compliance with fair-software.eu...
url: https://github.com/fair-workflows/nanopub
(1/5) repository
      ✓ has_open_repository
(2/5) license
      ✓ has_license
(3/5) registry
      × has_ascl_badge
      × has_bintray_badge
      × has_conda_badge
      × has_cran_badge
      × has_crates_badge
      × has_maven_badge
      × has_npm_badge
      ✓ has_pypi_badge
      × has_rsd_badge
      × is_on_github_marketplace
(4/5) citation
      × has_citation_file
      ✓ has_citationcff_file
      × has_codemeta_file
      × has_zenodo_badge
      × has_zenodo_metadata_file
(5/5) checklist
      × has_core_infrastructures_badge
      × has_sonarcloud_badge

Calculated compliance: ● ● ● ● ○

Expected badge is equal to the actual badge. It's all good.

It does not detect the CII badge that the project has.

I worked around it using the .howfairis.yml conf file and adding:
force:
  checklist: true  # The project has a CII software checklist badge, but it is not recognized by howfairis

You could reproduce this by forking https://github.com/fair-workflows/nanopub and removing that configuration file.

Hey Sven, thanks for the nice bug report. Glad you found the workaround. Just an FYI: your workaround will need a slight change once PR https://github.com/fair-software/howfairis/pull/46 is merged. Hopefully, we'll be able to fix this bug before then (and you can git rm .howfairis.yml).

@svenvanderburg could you run howfairis --version and post the output here please?

version: 0.11.0
On Mon, 7 Dec 2020 at 14:00, Jurriaan H. Spaaks<EMAIL_ADDRESS>wrote:

@svenvanderburg https://github.com/svenvanderburg could you run howfairis
--version and post the output here please?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fair-software/howfairis/issues/54#issuecomment-739903343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL4BJ4CO6VYGLSMMHZNOQLSTTGW7ANCNFSM4UNN3EIQ
.

It seems like github mixed up my code... 🤦
Haha, I have literally no clue how this did not work at that moment apart from that brain was not functioning.
So please feel free to close this issue! Sorry for wasting your time (although I helped you find #55 ;) )

It seems like github mixed up my code... facepalm

Aha, now I remember what may cause this (better late than never): there is a few minutes delay between what you see on github.com and what howfairis gets from raw.githubusercontent.com.
No worries about wasting my time BTW this is just part and parcel of releasing software into the wild, and it can lead to useful feedback, as it did here.


Add get_num_threads function

While it is possible to change the number of threads used by FFTW plans with
set_num_threads, it is impossible to see what the current value is. This is
made difficult because FFTW3 does not, as far as I know, provide a way to do
this. However, caching the last argument passed to set_num_threads as a
constant in FFTW.jl allows later access. I have added a constant,
PLAN_NUM_THREADS to keep track of the last argument to set_num_threads,
modified set_num_threads to update this constant, and added a function
get_num_threads to access the constant.
Closes #117

This is reasonable, but I would ideally like to fix this in the FFTW C library; otherwise in principle this could return the wrong result if a C library links to FFTW and changes this.
https://github.com/FFTW/fftw3/blob/516116615362334c2051a16ec1658a888573ab63/threads/api.c#L70-L81 is where a new function int X(planner_nthreads)(void) { return X(the_planner)()->nthr; } would go.

I will attempt to add this to the FFTW C library instead of caching it in Julia (FFTW/fftw3#205).


Print versions of installed binaries

This PR updated the install.sh script to print the versions of each of the Carvel binaries that it installs. This is helpful when reviewing the output from building a docker file (for example, see https://release-integration.ci.cf-app.com/teams/main/pipelines/cf-for-k8s-dev-tooling/jobs/build-cf-for-k8s-ci-image/builds/39#L5fb9104a:91) so you know which versions were installed without having to download and run the image.

Need to sign the CLA.


Can't npm install

I get an error every time I try to run npm install.
2020-12-16T13_11_33_060Z-debug.log

Looks like you are running into an issue building / installing the sharp dependency, which is outside our control here.
The Gatsby docs for gatsby-plugin-sharp or gatsby-transformer-sharp may be of use here, since those are the ones pulling in sharp.
In case it helps, I've not had issues installing sharp on MacOS (x86_64 not ARM) in NodeJS 10 or 12.  Have not tried NodeJS 15.


No highlighting to the code

This is a bug in the latest nbdev. Use the stable version to fix this bug.


fix: update to eye v5.0.7

fix: update to eye v5.0.7

:tada: This PR is included in version 7.0.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v7.0.6

Your semantic-release bot :package::rocket:


Implement New UI Design for Customer Detail Data Overlay

Description:
The customer detail data overlay that appears after clicking the detail button (arrow button) on the applicationRequests page needs to be updated to match the new UI design provided in the Figma link above.
URLs:

Current Overlay: https://portal.dev.demo.catena-x.net/applicationRequests => next click on the detail arrow button of one of the registration to get to the overlay
New UI Design: Figma Designs

Tasks:

Review the new UI design for the customer detail data overlay in the provided Figma link.
Locate the overlay implementation within the portal codebase.
Assess whether the changes can be made/has to be made within the existing portal code or if component-level modifications are required.
Determine the impact of the proposed changes on other areas of the product to ensure compatibility and consistency across the platform.
Update the overlay to match the new UI design, ensuring responsiveness and adherence to design specifications.
Conduct thorough testing to validate the implementation and ensure no regressions in the overlay functionality.

Acceptance Criteria:

The customer detail data overlay matches the new UI design in Figma.
The overlay is responsive and functions correctly across different screen sizes and devices.
All functionalities of the overlay (e.g., buttons, links) work as expected without any regression.
Code changes are tested and verified in the local and dev environment.

Additional Notes:

Please consider the theme, colors, typography, and spacing guidelines as per the new design.

Potential Risks:

Changes in the component code could affect other implementations that use similar components.

Design Reference:

we identified that the size information is not present yet in the backend api.
@kunalgaurav-bmw  will add the "documentSize" variable that we are ready for the future as soon as the backend-api is adjusted.

Hi Team, I would like to inform that I am using the common header component "DialogHeader" which we using the same throughout the app, but the text design values in Figma are different.
Also want to add one more point that I have added the "documentSize" variable on frontend side it will be worked as soon as the backend-api is adjusted.

@kunalgaurav-bmw  good work looks perfect :)


[IOTDB-3258] implement count nodes

See details: https://issues.apache.org/jira/browse/IOTDB-3258.

Already fixed! PTAL @yifuzhou

If the conduction cost of the above suggestion is heave, the suggested simplified modification is that we should not extend the interface of child management with a parameter level, which is confusing, instead we should implement independent node management configRequest and NodeScanOperator.


Directory listing for static files

Can spark support directory listing when serving static files (via externalStaticFileLocation)?

Not something we will prioritize now.


Build fail - clang path

Hello,
I'm not even sure if I titled this correctly. Anyway - I'm trying to get the engine and examples built for android. I'm following this tutorial:
https://github.com/Nodrev/bgfx-android-activity#bgfx-android-activity---android-glue-for-bgfx
My setup: Win10 + msys64 tools
Error I get:
$ make android-arm
make -R -C .build/projects/gmake-android-arm config=debug
make[1]: Entering directory '/d/dev/bgfx/.build/projects/gmake-android-arm'
==== Building bx (debug) ====
allocator.cpp
/bin/sh: /bin/clang++: No such file or directory
make[2]: *** [bx.make:402: ../../android-arm/obj/Debug/bx/bx/src/allocator.o] Error 127
make[1]: *** [Makefile:21: bx] Error 2
make[1]: Leaving directory '/d/dev/bgfx/.build/projects/gmake-android-arm'
make: *** [makefile:64: android-arm-debug] Error 2

As far as I'm concerned, the error is about the autotool not finding the clang++ binary. But why exactly does it search for it only in /bin/ ? Why not use PATH? I know it's a common thing to have symlinks from /bin/ to other locations, but the problem is symlinks aren't working too good in msys64.
My path to clang is /mingw64/bin/clang++, is there any way to change the behavior of this thing to look for the binary elsewhere?

When you generated project, there was warning Set ANDROID_NDK_CLANG, ANDROID_NDK_ARM, and ANDROID_NDK_ROOT environment variables. that you need set before building.


Glyphicons "fonts" folder disappeared in Bootstrap v4+ NPM dist

The Glyphicons fonts folder is NPM-packaged in v3, but not v4:
https://github.com/twbs/bootstrap/tree/master/fonts
https://github.com/twbs/bootstrap/blob/master/package.json#L64
  "files": [
    "dist",
    "fonts",
    "grunt",
    "js/*.js",
    "less/**/*.less",
    "Gruntfile.js",
    "LICENSE"
  ],

https://github.com/twbs/bootstrap/blob/v4-dev/package.json#L72
  "files": [
    "dist",
    "grunt/*.js",
    "grunt/*.json",
    "js/*.js",
    "scss/**/*.scss",
    "Gruntfile.js",
    "LICENSE"
  ],

Is that an error? If not, what is the suggested alternative now?
Also see: https://github.com/twbs/bootstrap/issues/18520

There is no fonts folder whatsoever in v4 because v4 doesn't include any fonts anymore. Glyphicons has been dropped. See http://v4-alpha.getbootstrap.com/migration/#components

Thanks, that helps.

thanks !


[BatchMode] Fix a single-primary assumption in FrontendTool, add test.

This just recovers a hunk that was lost migrating the batch mode prototype forward, and adds a test that we can in fact run a single frontend with multiple primaries now, all the way through to codegen (woot!)

@swift-ci please test

(I'm going to merge this as-is since I believe it's something David already approves-of and is the one piece preventing end-to-end testing at this point; if you have refinement ideas please let me know and I'll do a followup PR.)

Glad you merged it! You are right that I already approved of it. (I think it was in the prototype, if not something very similar.) I was in a class and could not get to the review. Again: great that you didn't wait!!


Create LuaUserdata from template

hi, maybe I understand something incorrectly, but I am unable to create LuaUserdata from template class.
Here is the example:
class:
template <typename T>
class TestClass
{
	T mA;
public:
	TestClass(T a): mA(a) {}
};

code:
LuaTable testClass = mLua.CreateTable();
auto makeTestClass = lua.CreateFunction< LuaUserdata<TestClass<int>(int)> >(
[&](int prm)
{
LuaUserdata<TestClass<int>> newTestClass = mLua.CreateUserdata<TestClass<int>>(new TestClass<int>(prm));
return newTestClass;
}
);
testClass.Set("new", makeTestClass);

and error:
error: no instance of function template "CreateFunction" matches the argument list
argument types are: (lambda ->LuaUserdata<TestClass>)
object type is: Lua
Is there any other way to do that? When TestClass is not a template everything works perfect.

I assume you wanted a function that takes an int and returns a LuaUserdata< TestClass<int> >
In this case your function signature is wrong. Instead of
LuaUserdata<  TestClass<int>(int)  >
it should be
LuaUserdata<  TestClass<int>  >(int)
(note the triangular brackets' placements)

Thank you very much David, I had to be tired ;)


Ghost Pokemon

Whenever I enable the proxy (even for simple logging), Pokemon never seem to show up... instead I'll see untappable circles where Pokemon should be. I'm guessing the proxy is filtering out these assets somehow. Still need to investigate.

I wonder if #75 is related. It's possible the cert pinning workaround is not working for some requests and it's failing to fetch assets because of an untrusted CA.

Can you try restarting the app with the proxy enabled, without installing the proxy CA cert, and see if that helps?
If not, installing the cert will almost certainly take care of it.

I'm just using the Xposed module, I don't have a proxy CA cert installed.
Let me log URLs for a bit and I'll see if allowing any of them to by pass the proxy on my phone helps.

@fny @trisk How do i install on iOS ?

@trisk So definitely the proxy is blocking the assets from resolving. Right now if I bypass the proxy for these two IP addresses, everything works.

qm-in-f128.1e100.net (<IP_ADDRESS>)
qj-in-f128.1e100.net (<IP_ADDRESS>)

Not sure if both addresses are necessary.
I think you might need to open up storage.googleapis.com as well since it looks like the app requests URLs from that address for some other downloads.

False hope... that didn't work either.
Does anyone know if there's a way to proxy requests to a single address alone? It looks like if we just limit the proxy to anything to/from pgorelease.nianticlabs.com we may be cleared.

I was having the same issue, added storage.googleapis.com to the proxy bypass list and they appeared normally.

Yep that did it. Might be good note to add to the README.
Still a way to proxy only pgorelease.nianticlabs.com might be a more permanent solution. Anyone interesting in hacking together a special version of ProxyDroid?

Ah. Now it can no longer be used over Mobile Data. . Any way to fix this?

Hey hey @fny, thanks for noticing - yep, its ProxyDroid which screws around (by not sending the hostname along to the proxy in the CONNECT header). There is already a DNS resolver implemented for that, is it just storage.googleapis.com? Would adapt the logic for that, as I'm a proxydruid user as well :) Cheers!

@exploitz

Ah. Now it can no longer be used over Mobile Data. . Any way to fix this?

I using OpenVPN with TAP tunnel to bridge network to my own subnet + proxy over proxydroid, and its work nicely from anywhere (with LTE or any other connection).
And latency is somewhere around 220ms.
Maybe it will help you.

@rastapasta I think that's the only one that's been an issue. I've seen  a few other hosts pop up from time to time, but I have no idea if they're even related to Pokemon Go. Namely:

qm-in-f128.1e100.net
qj-in-f128.1e100.net

It seems that they are downloaded from:
https://storage.googleapis.com/cloud_assets_pgorelease/bundles/android/pm0141?lots_of_sensitive_data

Confirmed, images are downloaded from https://storage.googleapis.com/ - adding *.googleapis.com to "Exclude proxy for domains" does fix.

@fny @BRZ420 should be fixed now even without excluding domains - Cheers!


update day gen

Codecov Report

Merging #11 (488faae) into main (8ddd6d3) will decrease coverage by 1.46%.
The diff coverage is 16.00%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   96.67%   95.21%   -1.47%
==========================================
  Files          16       16
  Lines        1024     1044      +20
==========================================
+ Hits          990      994       +4
- Misses         34       50      +16

Impacted Files
Coverage Δ

bin/aoc2022/src/main.rs
80.00% <0.00%> (-10.67%)
:arrow_down:

lib/solution/src/day01/mod.rs
98.82% <100.00%> (+1.17%)
:arrow_up:

lib/solution/src/day02/mod.rs
100.00% <100.00%> (+2.70%)
:arrow_up:

lib/solution/src/day03/mod.rs
100.00% <100.00%> (+1.02%)
:arrow_up:

lib/solution/src/day04/mod.rs
100.00% <100.00%> (+3.12%)
:arrow_up:

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


add Python 3.7 + Django 2.2

@ryochiji was this failing because of the djay 3.x issues?

@aleontiev - Looking at the build failures, I don't think djay was the issue (or, at least, it was failing before it got to that step). Going to close anyhow since I got that other PR in.


Show keyboard/mouse actions on screen

Description of the new feature / enhancement
Add feature to show keyboard/mouse actions on screen like https://gitlab.com/screenkey/screenkey

Scenario when this would be used?
When presenting, it's clearer to show what actions the presenter has performed.
Supporting information
may duplicate #23714

/dup #981


in manifest_test, Domains and Hosts evaluation sometimes fails

I running into this issue when executing the bin/test, I get some errors like:
• Failure [0.040 seconds]
Manifests [It] removes duplicated values in 'hosts' and 'domains'
/Users/cf/Code/Go/src/github.com/cloudfoundry/cli/cf/manifest/manifest_test.go:302

  Expected
      <[]string | len:3, cap:4>: ["host-2", "my-hostname", "host-1"]
  to equal
      <[]string | len:3, cap:3>: ["my-hostname", "host-1", "host-2"]

  /Users/cf/Code/Go/src/github.com/cloudfoundry/cli/cf/manifest/manifest_test.go:301

It looks like the line of L301 expect Hosts to equal a slice of strings, but fails sometime due to the uncertain sequence of the strings after removing duplicate host. This is not found to be reproducible on Travis. Perhaps we just need to check like this:
Expect(*apps[0].Hosts).To(ConsistOf([]string{"my-hostname", "host-1", "host-2"}))

I found this issue also apply at L269 and L270.

This is due to the version of Go the test is running on. The map order in Go v1.3+ is not maintained. We have a fix in the concourse branch, it will be merged into master soon.


Code to solve ViewPropTypes issue

Due to ViewPropTypes things, react-native-keyboard-spacer no longer works after react-native v0.69.
So I rewrote this library with minimal requirements.
Hope this code helps someone.
import { css } from '@emotion/native'
import { FC, useEffect, useState } from 'react'
import {
  Keyboard,
  KeyboardEvent,
  LayoutAnimation,
  LayoutAnimationConfig,
  Platform,
  useWindowDimensions,
  View,
  ViewProps,
} from 'react-native'

const defaultAnimation: LayoutAnimationConfig = {
  duration: 500,
  create: {
    duration: 300,
    type: LayoutAnimation.Types.easeInEaseOut,
    property: LayoutAnimation.Properties.opacity,
  },
  update: {
    type: LayoutAnimation.Types.spring,
    springDamping: 200,
  },
}

export const KeyboardSpacer: FC<ViewProps> = () => {
  const { height: screenHeight } = useWindowDimensions()
  const [keyboardSpace, setKeyboardSpace] = useState(0)

  useEffect(() => {
    const updateKeyboardSpace = (event: KeyboardEvent) => {
      if (!event.endCoordinates) {
        return
      }

      let animationConfig = defaultAnimation
      if (Platform.OS === 'ios') {
        animationConfig = LayoutAnimation.create(
          event.duration,
          LayoutAnimation.Types[event.easing],
          LayoutAnimation.Properties.opacity,
        )
      }
      LayoutAnimation.configureNext(animationConfig)

      const keyboardSpace = screenHeight - event.endCoordinates.screenY
      setKeyboardSpace(keyboardSpace)
    }

    const resetKeyboardSpace = (event: KeyboardEvent) => {
      let animationConfig = defaultAnimation
      if (Platform.OS === 'ios') {
        animationConfig = LayoutAnimation.create(
          event.duration,
          LayoutAnimation.Types[event.easing],
          LayoutAnimation.Properties.opacity,
        )
      }
      LayoutAnimation.configureNext(animationConfig)

      setKeyboardSpace(0)
    }

    const updateListener =
      Platform.OS === 'android' ? 'keyboardDidShow' : 'keyboardWillShow'
    const resetListener =
      Platform.OS === 'android' ? 'keyboardDidHide' : 'keyboardWillHide'

    const listeners = [
      Keyboard.addListener(updateListener, updateKeyboardSpace),
      Keyboard.addListener(resetListener, resetKeyboardSpace),
    ]

    return () => {
      listeners.forEach(listener => listener.remove())
    }
  }, [screenHeight])

  return (
    <View
      style={css({
        left: 0,
        right: 0,
        bottom: 0,
        height: keyboardSpace,
      })}
    />
  )
}

thank you. I was using patch-package to fix it but it's way better to have the whole in a function component.

Thank you very much. This helped me a lot

Thanks really appreciate it

👍


Quesstions about MCC task?

Thanks your great work!
In step4,  I have a problem.

Hi did you modify something? I did not meet such a problem

Hi did you modify something? I did not meet such a problem

I did not modify anything.

Thanks your great work! In step4, I have a problem.

In the file decode_mcc.py:
line 80, the len_recall variable is 10.

https://github.com/ShangwenWang/Cognac/issues/4#issuecomment-955204756
I guess you are trying on your own data. In our experiment, we limit the number of tokens for a single method name (cf. the above link to another issue). Therefore, we do not meet such a problem.

I did not use my own dataset.I used the data set you provided: Java Large.

Ok. Then just make a pre-processing. Filter those bloated method names and the code can work.

Ok. Then just make a pre-processing. Filter those bloated method names and the code can work.

ok，thanks


Value of sysvar xx exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.

I have a variable that contains more than 255 characters (holidays of the year, created with the script Feiertagsliste_V3.1.txt, see https://homematic-forum.de/forum/viewtopic.php?f=31&t=4058) and I get the warning that this exceeds the allowed sysvar length. But I have no problem on the CCU (RaspberryMatic addon on HA) . Is this an "artificial" limitation?
The warning occurrs every ~ 30 seconds:
2022-06-02 10:17:37 WARNING (MainThread) [hahomematic.hub] Value of sysvar Feiertage exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.
2022-06-02 10:18:07 WARNING (MainThread) [hahomematic.hub] Value of sysvar Feiertage exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.
2022-06-02 10:18:37 WARNING (MainThread) [hahomematic.hub] Value of sysvar Feiertage exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.
2022-06-02 10:19:07 WARNING (MainThread) [hahomematic.hub] Value of sysvar Feiertage exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.
2022-06-02 10:19:37 WARNING (MainThread) [hahomematic.hub] Value of sysvar Feiertage exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.
2022-06-02 10:20:07 WARNING (MainThread) [hahomematic.hub] Value of sysvar Feiertage exceedes maximum allowed length of 255 chars. Value will be limited to 255 chars.

Home Assistant does not allow entity states with more that 255 chars.
Here we cut at 255 chars and write a warning to the log.

Understood. Is there any way to limit / throttle the warning messages?

Yes, there are concrete hints in the issue template you deleted.

Make sure you are running the latest version of hahomematic or it's custom_component (and Home Assistant) before reporting an issue.

Just update to latest version or deactivate this entity in HA.

The entity was already deactivated. Just updated to the last version and the warnings are gone. Thank you for the support!


clarify service selection logic

it seems like this function is making selections about service:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/blob/4ed7ebc5bac9d121ca4a4beb90009dd5cf51639b/R/r511.R#L72-L79
we could potentially use the start and end dates in this file for a more granular selection, assuming providers have them filled out:
https://developers.google.com/transit/gtfs/reference/calendar-file

these 4 functions together give us weird results for the MA provider:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/blob/8c261e97cf088deb0e3e6077db7039f9fda243c7/R/make_weekday_peak_bus_stops_csv.R#L50-L54
in particular, we end up with just 4 pm stops for 2 routes that have a headway below 16 minutes. need to check whether that route has more than 4 stops.

I suggest that we perhaps use the calendar and stop times to choose the max number of trips by route and use this number for the headway. Effectively filtering out different patterns schedules.

for now, using a specific date:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/blob/simplify/R/r511.R#L291
changing the title to indicate that we need to review this. may be related to:
https://github.com/MetropolitanTransportationCommission/fast-trips/issues/71

this function:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/blob/simplify/R/r511.R#L134-L141
which is based on this deduplication technique:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/blob/simplify/R/RouteBuilderStuff_KS.R#L281-L289
seems to hang on to stops for route 48 in san francisco that aren't on any high frequency route geometries. for a map of these stops, routes, see:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/issues/43

moving this to the second draft output from a refactor as its now higher priority.

confusingly, some stops, such as 16144, show in the image for SF have M/T/W/T/F service, yet the route geometries do not show the route passing by those stops. checking on all services for route 48

as discussed here:
https://github.com/MetropolitanTransportationCommission/RegionalTransitDatabase/issues/43


Update changelog for v0.17.1

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/lgtm

/approve

/cherry-pick release-0.17

/unhold


Test CCSDS header across all instrument packet definitions

Change Summary
Overview
This PR adds a new folder for storing tests that can be run on all instruments, and a new test that tests the existence and proper formatting of the CCSDS header in all packet definition XML files.
New Dependencies
None
New Files

imap_processing/tests/test_decom.py

Tests for the decommutation process, applied to all instruments

Deleted Files

None
Updated Files

None
Testing

This test is currently failing on the idex_packet_defintiion.xml file (which currently doesn't contain a properly formatted header) and a handful of others (due to inconsistency with considering SHCOARSE as part of the header or not).

Some open questions for discussion:

Is SHCOARSE to be considered part of the header or not? Should we be consistent on how this is applied across all packet definition files?
Is this a sound method for testing the header (i.e. reading in the file and comparing strings)? Or should we try to actually decom each file and check if the header is in the results?
Are there any other generalized tests we can add to this file?

Ok, I believe this is good to go and the tests should pass once SWE, Ultra, and GLOWS update their XTCE files with the fixes implemented in #185

Updated GLOWS XTCE to pass these tests in #244

pre-commit.ci autofix

I think we still need to update the packet definitions for SWE and Ultra with the changes from #185 to get these tests to pass.
@tech3371 @laspsandoval Could you update those files for SWE and Ultra (respectively)? Or if it is straightforward, I am happy to do it as part of this PR if you could share the Excel spreadsheet.

Just an FYI, but you can always xfail Ultra for now and update it later the same way you're doing with idex I think? Or do you want to update idex as well and remove all xfails altogether? Just giving you an out to get your tests passing so you can merge this in and then let others update and remove xfails later so the onus isn't on you to do that work.

It was an easy fix to get the Ultra packet definitions to pass, so I just did that. I think I will leave IDEX as xfail for now -- that one will require more work.


Apollo MASTER: run modules/perception/camera/tools/offline/offline_obstacle_pipeline FAILED: EigenMatrix Problem

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04): Ubuntu 20.04
Apollo installed from (source or binary): source
Apollo version (3.5, 5.0, 5.5, 6.0): master
Output of apollo.sh config if on master branch:

Steps to reproduce the issue:

Similar to issue #14610 , When I tried to run the offline_obstacle_pipline on the latest master branch, it outputs:

Supporting materials (screenshots, command lines, code/script snippets):

This tool is broken now, we haven't scheduled a fix yet because it's not a very high priority at the moment


[system] Improve documentation on usage of layout components: Container, Grid, Box, Stack

Part of #35158
Summary
how to use Stack, Container, Box to set up the layout for whole page? is that correct? what about gird?
grid
container
stack
box
component
box
stack
container
grid
Examples
can I only use div, and not to use any stack, container or box? do I need apply grid for every component and pages?
Motivation
where and which file to setting the grid for whole page
Search keywords: stack, box, container, grid

thanks alot!


Has TV always been active for grid trades > 1?

I upgraded to the latest version v0.0.89 (165d110) and noticed that grid trades >1 are also affected by the TradingView setting. In my case I had set it to: Allow buy trigger when recommendation is Buy .
If this affects grid trades as well, then grid trades are basically in-effective as long as the coin drops.
But from my understanding, grid trades should help counter the effect of the coin dropping.
Can someone please clarify this?

Yes, the TradingView was applying to all grid trades.
Please close the issue if your question is answered. :)

The issue is still bugging me...
Why should TV be used for any Trade >1?
Then percentage set for each grid Trade to fire would be of no use.
From my understanding, the right percentage is required in order to maintain the same percentage to break-even...
Here is an example, where we start at 50USDT at Grid Trad 1:

No
USDT
Grid %
Break Even %

0
50.0000
1.0000

1
10.0000
0.9809
0.8811

2
10.0000
0.9733
1.6664

3
25.3187
0.9701
2.0000

4
49.9498
0.9701
2.0000

5
76.1484
0.9701
1.9998

6
116.0669
0.9701
1.9997

7
176.7615
0.9701
2.0001

8
269.5341
0.9701
2.0000

9
410.7260
0.9701
2.0000

10
625.9767
0.9701
2.0000

This already requires 1820.48 USDT in order to maintain a break even of 2%. (And if I you are working with stop loss, then 96.9 will be your minimum here, right?)

If the percentage deviates from the given setup (because TV needs to approve the grid trade), then the break-even will shift, most likely to a higher value. This will break the entire calculation for the grid trades.
In my understanding only the 1st grid trade would benefit from TV.
@uhliksk @habibalkhabbaz @chrisleekr
What is your opinion here?
And I noticed that checking the box Allow buy trigger when recommendation is Buy also seems to bypass the ATH setting. So if it is checked, it will buy even though the Restriction price percentage for ATH is not met. This is also not helpful in many situations.

@Rayn0r In my opinion TV needs to be applied to all trades because you always want to buy at the lowest price possible. When the market is going down it's sometimes oscilating with big amplitude which will cause you'll buy even when it's not neccessary and then it'll be harder to compensate the loss. It's better to skip falling market and buy the dip directly.

@Rayn0r Right, so that is your strategy.
I think the solution is, simply to add a new option to apply TradingView to the first grid trade.
If anyone wants to apply to all grid trade, then uncheck the option.
When I get time (which I cannot guarantee when), I will add the option in.

It's better to skip falling market and buy the dip directly.

I never thought of it this way.
Let me do some calculations, on what happens to the break-even if you skip a few a grids on the way down and they all fire at one point(more or less).

Contrary to what I thought, it even pays to use TV for grid trades.
I did a quick test on sequence of 7 additional grid trades with a break even of 1%. If TV does not allow to buy for 3 grids, and those 3 are all fired at once it is back to Buy/Strong Buy, then your break-even is down to 0.676164104%
@chrisleekr
Thanks for the offer, but I think it is not necessary to implement TV for just Trade 1.


Make links in notes relative with respect to the last period

Please Select if your Request is Either something new or an Enhancement

[ ] Enhancement of an existing Feature.
[x] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ ] UI
[ ] Workbench
[ ] Publishing
[ ] Pods
[x] Markdown
[ ] Hierarchy
[ ] Graph
[ ] Misc (Choose this if your not sure)

Describe the solution you'd like
Links in notes to be relative w.r.t the last period.
For instance, if the user has the following notes:
ideas.blog
ideas.blog.post-1
ideas.blog.post-2

In ideas  writing [[.blog]] would be interpreted as [[ideas.blog]]
Similarly, in ideas.blog entering [[.post-1]] would be [[ideas.blog.post-1]]
This would make it easier to add links particularly in notes that might be deeply nested or have long names.
It would also help in situations where the user renames notes in bulk.

https://marketplace.visualstudio.com/items?itemName=kortina.vscode-markdown-notes has something similar but using directories instead of file names. Third screenshot on that page.

I love this. We'll definitely implement this, hopefully in April
On Tue, Mar 30, 2021 at 10:05 PM Ognjen Regoje @.***>
wrote:

https://marketplace.visualstudio.com/items?itemName=kortina.vscode-markdown-notes
has something similar but using directories instead of file names. Third
screenshot on that page.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/issues/509#issuecomment-810764053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPIPXGKU5KDYTKPEQC6XLTGKURLANCNFSM4YJXAVGQ
.

It just occurred to me that another way would be by making the autocomplete for note links prefer the notes in the current namespace.
So, from the example above, when creating a link from `ideas.blog.post-1`` you'd start by typing:
[[.|]]
(pipe just shows where the cursor is)
would result in the following in the note autocomplete:
ideas.blog.post-1
ideas.blog.post-2 # perhaps exclude current note

The period would act as if the entire path of this note has been entered. Each period could potentially move the hierarchy one level upwards.
When the note is then selected from the autocomplete, the full note name could be inserted into the link.

I like the most recent suggestion since we'd continue to be compatible with regular wikilinks.

Well, I guess procrastination has its benefits :)
Let's go with the latter suggestion then. @hikchoi, this would be a good task to take on when you've cleared a few of the initial ones from the medium sized list.

I think the original idea (of using . to refer to the current note's ancestry) would enable an outliner pattern similar to what Workflowy (and others) do:
// areas.md

[[work]]
[[life]]

// areas.work.md

[[.projects]]
[[.office]]
[[.promotion]]

// areas.work.projects.md

[[.mega-laser]]
[[.moon-blaster]]
[[.sun-gun]]

// areas.work.projects.moon-blaster.md

[[.funding]]
[[.red-tape]]
[[.bribes]]

// areas.work.projects.moon-blaster.red-tape.md

- National Astronomy community objects on the esthetics grounds
- International wolves council would like a dark-time replacement satellite to howl on

So, you can structure your notes in such a way that you don't even need the tree view since all your navigation can be done through the notes.
I don't think most will use it in this way, but this could be useful in some circumstances.
Also, there was an issue asking for a workflowy-like feature (#199) which this . link implementation might satisfy.


Parser chokes on quotation marks in --data argument

(I'm basing this report on the behavior of the online demonstration at http://curl.trillworks.com/.  My apologies if that demo is simply out of date and this problem is already fixed in the most recent code, or if this issue is an artifact of the web interface.)
It seems that I cannot pass single or double quote characters in the curl command.
For example, given:
curl -X POST http://example.com/ --data "foo=bar"

curlconverter generates the proper body content (taking the Node.js output):
var dataString = 'foo=bar';

The same is true for the single quote version:
curl -X POST http://example.com/ --data 'foo=bar'

However, if I try to add quotation marks in the body content, e.g.:
curl -X POST http://example.com/ --data 'foo="bar"'

The output body is truncated at the first quotation mark:
var dataString = 'foo=';

There are similar problems for all of the variations of this input I could think of, for example:
curl -X POST http://example.com/ --data "foo=\"bar\""
curl -X POST http://example.com/ --data 'foo=\"bar\"'
curl -X POST http://example.com/ --data "foo=\'bar\'"
curl -X POST http://example.com/ --data "foo='bar'"

etc.

Actually, it looks like this problem has been fixed in the latest version of the code, via this commit - https://github.com/NickCarneiro/curlconverter/commit/d4da3faa0b81d5448edfcaaf12f80aab9b4cef16
I guess you can read this bug as reporting that http://curl.trillworks.com/ has not been updated yet.

This bug still affects the --data param. I'll add your test cases this weekend and update the parser.


Support for plpgsql functions

When trying out this product with the plan command, the program errors, because it seems to do some live validation of the plan. If I understand correctly what is happening, this generates errors because:

some of the function are written in plpgsql, for which the program doesn't check its dependencies
those functions seem to be created in alphabetical order, so some functions are created before their dependencies

One of the following would solve this:

add flag is added that sets check_function_bodies = off before doing the validation
add support for dependency checking of plpgsql functions
don't validate the plan when the plan was created from two live databases (i.e. with both --dsn and --schema-source-dsn)

Yes, that is an unfortunate limitation of postgres. It doesn't actually build any hard dependencies with pgplsql functions, so pg-schema-diff is kind of operating in the dark. We could manually try to parse dependencies from the function but that is a lot of complexity: not only would the function itself need to be parsed, but the context of the function would need to be known (is it used in a trigger, check constraint, etc).
Plan validaiton is a confidence check. If plan validation fails, it normally implies the plan will fail against your actual database. We can definitely add an option to disable it, but I'd be curious to know if the plan validation is actually failing here, or you're okay with an invalid plan.
Thanks for the great breakdown here! I might rename the ticket to "Improve track of plpgsql functions"

Thanks for the quick reply. I'm trying to move away from migra, as it isn't developed anymore and doesn't support diffing domains. Migra would always add set check_function_bodies = off; to its diffs, and I never had any problems with it.
I'm quite sure the plan would work, but I cannot test it, as no plan is generated at all.
How does the validation work? Is it executed in the database to upgrade? Is it isolated? Is it reverted afterwards? I would like to disable it, as we always manually check the diff anyways.

I added a flag to disable plan validation in the cli via --disable-plan-validation.
The way plan validation works:

Spins up a temporary database on your postgres instance
Places the current schema in the temporary database
Runs the migration
Re-generates a diff and asserts that the diff is empty

We unfortunately do not support diffing of domains yet. If you add a ticket, we can try to get to it, since I imagine implementing that won't be too tricky.

We could manually try to parse dependencies from the function but that is a lot of complexity: not only would the function itself need to be parsed, but the context of the function would need to be known (is it used in a trigger, check constraint, etc).

@bplunkett-stripe have you considered letting the developer manually specify dependencies for the cases that they cannot be inferred (or inferred correctly)? That's what I do in pgmigrate, and it works great.

We could manually try to parse dependencies from the function but that is a lot of complexity: not only would the function itself need to be parsed, but the context of the function would need to be known (is it used in a trigger, check constraint, etc).

@bplunkett-stripe have you considered letting the developer manually specify dependencies for the cases that they cannot be inferred (or inferred correctly)? That's what I do in pgmigrate, and it works great.

Interesting. That is also a definite possibility. Not super conducive to the CLI format unfortunately, but I'm also thinking of adding support for some sort of configuration YAML.

Yeah, has to come from a configuration file, not passable on the command line, but it's pretty common for these kinds of tools to have config files (in my experience.) You could take a look at how I parse/merge cli options and config file options here in pgmigrate or just use something like Viper, which I've been meaning to try.
btw very cool project, watching with interest, keep up the good work 🫡

Awesome! pgmigrate looks pretty neat.  We use cobra already, so viper will probably slot in pretty nicely.
Hopefully can get some bigger features out soon. This project is a bit on the side-burner for me with current priorities 😢 .

This is effectively blocked by #131, which will enable us to more easily build dependencies on statements like column additions/deletions.

Migra would always add set check_function_bodies = off; to its diffs, and I never had any problems with it.

This should be an option in pg-schema-diff. It seems that SET statements are not included in the diff plan, so the plan validation fails when it doesn't really need to.

This should be an option in pg-schema-diff. It seems that SET statements are not included in the diff plan, so the plan validation fails even when I tried setting check_function_bodies = off myself.

Maybe a special filename like pre_plan.sql could let us always run certain statements on the temporary database before plan validation?
The problem you're trying to solve for here is there are certain schema objects not yet supported by pg-schema-diff but referenced by your functions?

@Navbryce Yeah exactly


[ci] Clean up verilator chip-level testing configuration.

This commit undoes the temporary actions in #12603 that were taken to
prevent blocking on the Verilator chip-level tests back when these tests
were flaky.
Closes #12603

I don't know what happened here, but it might be fixed by rebasing.


Cordova User Question

Hi Irettig,
I am interested in trying the react-native solution. I looked over the documentation and I have two questions.  I will use it in my IOS App in place of a Cordova Apple Pay plugin.

I am assuming I will not need to learn React ?
2.I am not certain what the Library Group refers to in "Drag StripeNative.xcodeproj into your Libraries group ". Are you referring to the Directory of Frameworks in Xcode?

Thanks!
Howard

Add
I am installing using your steps
I successfully installed cocoa pods with sudo gem install cocoa pods
npm i react-native-stripe --save fails with the following error messages
Error: EACCES, mkdir '/Users/howardsmith/.npm/type-is/1.6.11'
npm ERR!  { [Error: EACCES, mkdir '/Users/howardsmith/.npm/type-is/1.6.11']
npm ERR!   errno: 3,
npm ERR!   code: 'EACCES',
npm ERR!   path: '/Users/howardsmith/.npm/type-is/1.6.11',
npm ERR!   parent: 'connect' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.
npm ERR! System Darwin 15.2.0
npm ERR! command "node" "/usr/local/bin/npm" "i" "react-native-stripe" "--save"
npm ERR! cwd /Users/howardsmith
npm ERR! node -v v0.10.32
npm ERR! npm -v 1.4.28
npm ERR! path /Users/howardsmith/.npm/type-is/1.6.11
npm ERR! code EACCES
npm ERR! errno 3
npm ERR! stack Error: EACCES, mkdir '/Users/howardsmith/.npm/type-is/1.6.11'
npm ERR! not ok code 0
I ran 3 times as system/administrator and always had the same results.
Any suggestions?
Thanks!
Howard

Hi Howard
Thanks for checking out this module. First things first, it was written for react native, so I highly doubt it will work out of the box with cordova. You might want to try this cordova plugin instead:
http://plugins.telerik.com/cordova/plugin/stripe
(docs: https://github.com/Telerik-Verified-Plugins/Stripe/blob/master/doc/index.md)
For the record, you're probably getting this error because you need to run npm with sudo, i.e., sudo npm.
Lane


ADBDEV-4937-44: Add assertions to return from dynamic_cast.

Add assertions to return from dynamic_cast.
dynamic_cast may return NULL, so I added assertions.

https://github.com/arenadata/gpdb/pull/855


Improve new joiners onboarding - Cookbook - Q&A

Assign to me, please.  I have a team ready to tackle this one.

@jack-garvin - Please include @AshCos. You may want to reach out to @danielnorkin as well.

Please reopen this issue once the work will start.


Potential semantic change by Rails/FindEach cop

Rails/FindEach cop suggests to change
User.where(name: name).each { nil }

to
User.where(name: name).find_each { nil }

which might cause unexpected semantic change:

each {...} returns Array, while find_each {...} returns nil in this case.
find_each does not respect limit and order conditions.

I think using obj.each if you actually want an Array is poor coding style. It's not clear that you are doing that to get an array.
However, if the other contributors disagree, I do have code which can "fix" this.
The point about limit and order is a good one.

Just opened a PR which fixes this point regarding order and limit. I read on an Internet forum that find_each also ignores select(:field1, :field2, ...), so I included that too.
The point about the return value is (deliberately) not fixed.


Cache nonParameterCharacterSet to help minimize memory footprint in multithreaded apps using SOCKit.

TLDR:
Purpose of this pull request is to minimize memory spikes in processes using SOCKin in parallel threads. Objective is achieved by caching nonParameterCharacterSet. Footprint of single nonParameterCharacterSet is slightly more than 32KB.
More info:
Facts

iOS today extensions are limited to use 10-11MB of the heap for iPhone 5 - iPhone 6 plus
As you can see from attached image of Allocations utility, one call to nonParameterCharacterSet takes up north of 32KB. (all of these allocations in the attached image are related to single call).

Details on actual usage
One of the iOS "today extensions" I'm working with has Reskit + SOCKit stack and is issuing a bunch of networking requests + CoreData object creation in parallel. Such parallel behavior leads to significant spike of memory usage with SOCKit pattern creation consuming a lot of memory during this short spike.
Having this patch helps minimize this spike during user interacting with extension and prolong life of the extension in memory.

Could you plaase merge this? I'm totally in the same situation.


No way to upload files

forms.BaseForm's init method has a third attribute "files", MongoForm does not have such method, so uploading multipart forms with forms.ImageField and so on is impossible right now, the form just does not validate.

Moreover it is impossible to use FormView from django class based generic views as it is fails with server error on form bounding.


Describe HealthCheck as monitor

Explain the differences when registering HealthChecks as a monitor.
closes #76

Im still not sure in which situations I want a healthcheck and when I want it as a monitor. The monitor example isn't that the same as a normal gauge?  Before this monitor example healthcheck was a simple concept to grasp and understand.

Maybe the static HealthCheck part should be moved to the endpoint, as it has no effect in other situations.
Yes the monitor is basically just a Gauge with a boolean value, so it could just be deleted, and people could just use a gauge directly

Ok I'd write that explanation in the tutorial. Or really, if I may have a say, I'd prefer to delete the code and have people use the Gauge. It makes the concepts much simpler

Can I have the satisfaction of deleting it? ;-)

Sure


Deploy

Deploy master on js.waku.guide
Deploy develop on dev-js.waku.guide.

@jakubgs we migrated this repo around. It was previously deployed to docs.wakuconnect.dev
Can you please deploy to the new location as we are moving away from wakuconnect.dev?

Done:

https://github.com/status-im/infra-misc/commit/5aa52041 - configure js.waku.guide and dev-js.waku.guide sites
https://github.com/waku-org/js.waku.guide/commit/4adba002 - ci: change docs.wakuconnect.dev to js.waku.guide

Endpoints:

https://js.waku.guide/
https://dev-js.waku.guide/

@jakubgs thank you for that.
However I think there is an issue. master and develop have the same commits but the websites do not match (see the warning sign on the homepage)
https://js.waku.guide/
https://dev-js.waku.guide/
It seems that latest master did not get deployed on js.waku.guide. Indeed I can see an error in Jenkins: https://ci.infra.status.im/job/website/job/js.waku.org/

I just did a push on both master and develop (8390b61248ad34cfb08c5003fc7b670a0d3239c8) but I can't see the CI being triggered in the Jenkins UI.

You're confusing builds for js.waku.org and js.waku.guide:

https://ci.infra.status.im/job/website/job/js.waku.org/
https://ci.infra.status.im/job/website/job/js.waku.guide/

The first one is failing due to changes to package.json, the latter is not starting.

Also dude, closing quotes is a good idea:
https://github.com/waku-org/js.waku.guide/blob/8390b61248ad34cfb08c5003fc7b670a0d3239c8/config.yaml#L14
+ hugo
Error: "/home/jenkins/workspace/website/js.waku.guide/config.yaml:14:1": failed to unmarshal YAML: yaml: line 14: did not find expected key

The webhook URL was wrong, also fixed the quote: https://github.com/waku-org/js.waku.guide/commit/3bd26bc6

https://ci.infra.status.im/job/website/job/js.waku.guide/33/
https://ci.infra.status.im/job/website/job/dev-js.waku.guide/81/

I can see that gh-page branch is up to date but js.waku.guide is not so I am guessing an issue on GitHub pages now? Will push another commit see if it was a resolved outage.

There's still an issue. Jobs looks good: https://ci.infra.status.im/job/website/job/js.waku.guide/34/
I can see that gh-pages contains the latest changes but it's not being deployed on https://js.waku.guide/

The GitHub pages action says it's successful: https://github.com/waku-org/js.waku.guide/actions/runs/3433718458/jobs/5724336082

The GitHub Page action says:
{"page_url":"http://js.waku.guide/","status_url":"https://api.github.com/repos/waku-org/js.waku.guide/pages/deployment/status/0be8182e46a6129b435480cf14c74e0b57508c31","preview_url":""}
Reported success!

the page_url is http and not https.
I can see that http://js.waku.guide returns a 301 Moved Permanently to https://js.waku.guide.
▶ wget http://js.waku.guide
--2022-11-10 14:28:02--  http://js.waku.guide/
Resolving js.waku.guide (js.waku.guide)... <IP_ADDRESS>, <IP_ADDRESS>, 2606:4700:3033::6815:44f3, ...
Connecting to js.waku.guide (js.waku.guide)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://js.waku.guide/ [following]

Maybe a red herring...


A file with a trailing "-" in the name before extension does not get opened

System is Windows 10 Pro x64 1803
This file name opens just fine:
Blink of an eye - Slow Motion.mp4

This does not (there is a duplicated file extension):
Blink of an eye - Slow Motion-.mp4

Other than that DJV is awesome. Just as an additional request (this is just a personal preference). Would it be possible to make left  and right cursor arrows the default next and previous frame. I always have to go to settings and change the default mapping, because each new version writes its own registry keys. I rarely move the frame (ctrl+left and ctrl+right do this on my settings) :).
Keep up the good work!!!
Cheers

Hi,
Thanks for the bug report! That should be easy to fix, I think DJV is trying to interpret the trailing '-' as a file sequence and is getting confused.
The keyboard shortcut change sounds good, it seemed like a good idea at the time but I'm not sure I've ever actually used them either. What would you suggest for the up and down keys? Maybe go to the beginning/ending of the clip (or current in/out points)?
Sorry about the preferences resetting, that normally shouldn't happen with every version. I only do that when I make a change to the preferences and unfortunately there have been a number of those lately.

What do you think about this mapping?
Just to note, I am big fan of key combinations :)
Right cursor = Playback/Next
Shift + Right cursor = Playback/Next 10
Control + Right cursor = Playback/Next 100
Control + Shift + Right cursor = View/Right (rarely used, almost never used actually)
Left cursor = Playback/Previous
Shift + Left cursor = Playback/Previous 10
Control + Left cursor = Playback/Previous 100
Control + Shift + Left cursor = View/Left (rarely used, almost never used actually)
Here are a more cuestionable shorcuts, now we have the typical J-K-L playback shorcuts present in most video editing software, but for
me in particular I prefer to have the playback shortcuts as close to each other as possible and because this is not an editing software.
Would it be possible to add more than 1 shorcut to an action or more than 1 action to a shortcut (kind of a macro)? so we can have the best of both.
Maybe it is too much to wish, but hey, It doesn't hurt to ask :)
Up cursor = forward playback + Playback/Toggle (Houdini like)
Control + Up = Playback/Start (Houdini like) (Home key is good as well because it is next to the cursors)
Control + Shift + Up = View/Up (rarely used, almost never used actually)
Shift + Up = Go to In mark
Down cursor = Reverse playback + toggle playback (Houdini like)
Control + Down = Playback/End (End key is good as well because it is next to the cursors)
Control + Shift +  Down = View/Down (rarely used, almost never used actually)
Shift + Down = Go to Out mark
These is my prefered mapping but may change in the future of course.
Also if I may ask for another thing.

Always on top feature -
I have my own way of keeping any window on top via and Autohotkey script but It would be nice to have this feature built-in as sometimes my co-workers ask how the can do this and i have to install "things" :D

That’s just my two cents.
As always, thanks for your hard work. Cheers

Those seem like reasonable shortcuts, I'll add them for the next major version (1.3.0).
I didn't think that Qt supported multiple shortcuts per action but I just checked the documentation and it turns out they do, I'll see if I can add support for that. The multiple actions per shortcut is an interesting idea, what did you have in mind using it for?
I'll check out the "always on top" feature, I think that's easy to do with Qt.

Great! Thanks. About the multiple actions per shortcut, I just thought about that due to the fact that

Up cursor = Forward playback + Playback/Toggle (Houdini like)

for example, involves 2 actions, setting forward playback and toggling the playback too. The same apllies to

Down cursor = Reverse playback + Playback/Toggle (Houdini like)

I can't think of any other use right now but I guess it can be useful somehow. But maybe it is not worth the effort if it is hard to implement.

I vote for those too ;-)

Hi,
Sorry for the delay! I just uploaded a new version which fixes this issue (1.2.5), you can download it here:
http://djv.sourceforge.net/Download.html

Hi, sorry for the very long delay! I'm closing this issue out since the original bug is fixed. For the other issues that came up:

In the new 2.0 version the keyboard shortcuts for moving the view have been moved to the numeric keypad, and the arrow keys are now used for playback; left/right for changing the frame, and up/down for forward/reverse/stop playback.
There is a separate issue for the window stay on top feature: https://github.com/darbyjohnston/DJV/issues/55


[AutoTVM] Fix typos

Signed-off-by: Ce Gao<EMAIL_ADDRESS>I found a typo in the docs of autotvm here: https://docs.tvm.ai/tutorials/autotvm/tune_simple_template.html#step-1-define-the-search-space
While I am not sure if it works with such a comment-fix.

Thanks @gaocegege, this is merged.


App Icon

Hey! I love VS Code and am SO happy to have it on my Raspberry Pi now! However, I don't like the app icon that shows up in docks and application websites and it doesn't fit with my custom icon sets. Is there a way to replace the icon with a custom one?

@varlevi if you think about it yourself: How would you do it to change any icon oder image from any application?
I would say search for it :o) and you will find it here or a on a similar location:
/usr/share/code-oss/resources/app/resources/linux/code.png
exchange it to anything you like.

Thanks @cellcoresystems! Unfortunately, computer restrictions made me unable to add files to the /usr/share/code-oss/etc... directory, but I was able to redirect my system to a different location so the new app icon is used in docks and menus. Unfortunately, it doesn't change the icon inside the application, but I was really only annoyed because the icon didn't fit with my system design. It doesn't clash nearly as much with VS Code itself.


Go installation error

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Fedora 32
TensorFlow installed from (source or binary): binary
TensorFlow version: 2.3.0
Python version:3.8.5
Installed using virtualenv? pip? conda?: pip
CUDA/cuDNN version: 11.0
GPU model and memory: 1660 gtx super (6GB)
GO version: 1.15.2

Hello i'm trying to install the tensorflow go version to build a little application. So i installed the C library (2.3.0 version tested with classic hello.c example) and after that i run the following command:
go get github.com/tensorflow/tensorflow/tensorflow/go
but after this one i get this error:
go: found github.com/tensorflow/tensorflow/tensorflow/go in github.com/tensorflow/tensorflow v2.3.1+incompatible
go: finding module for package github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto
go: finding module for package github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto
../../../../golang/gopath/pkg/mod/github.com/tensorflow/tensorflow@v2.3.1+incompatible/tensorflow/go/saved_model.go:25:2: module github.com/tensorflow/tensorflow@latest found (v2.3.1+incompatible), but does not contain package github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto

could tou help me?
Go ENVS:
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/fedyfausto/.cache/go-build"
GOENV="/home/fedyfausto/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/fedyfausto/golang/gopath/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/fedyfausto/golang/gopath"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/fedyfausto/golang/latest/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/fedyfausto/golang/latest/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/fedyfausto/Lavoro/Project/go_predictor_2/project_name/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build398631669=/tmp/go-build -gno-record-gcc-switches"

@fedyfausto
Please refer to this issue with same error and let us know :#41808 #34580 #39307

The issue is the same but seems does not be resolved yet

i tryied to re-install all but seems some files miss (the file for_core_protos_go_proto)

The C library seems work:
[fedyfausto@desktop-fisso-fedyfausto-fedora Scrivania]$ gcc hello_tf.c -ltensorflow -o hello_tf
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 851 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 1959 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2329 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2517 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2732 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2733 (>= sh_info of 2)
[fedyfausto@desktop-fisso-fedyfausto-fedora Scrivania]$ ./hello_tf
Hello from TensorFlow C library version 2.3.0

The C library seems work:
[fedyfausto@desktop-fisso-fedyfausto-fedora Scrivania]$ gcc hello_tf.c -ltensorflow -o hello_tf
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 851 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 1959 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2329 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2517 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2732 (>= sh_info of 2)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2733 (>= sh_info of 2)
[fedyfausto@desktop-fisso-fedyfausto-fedora Scrivania]$ ./hello_tf
Hello from TensorFlow C library version 2.3.0

Do you mean the issue is resolved.

@Saduf2019 I've met same problem, too.
The C library of v2.3.0 works well.
Go version of<EMAIL_ADDRESS>or v2.3.1 does not work in go mod environment.

Same problem.
OS ubuntu/amd64
Tensorflow v2.3.1
Golang v1.15.1
This issue is STILL NOT SOLVED. All of us can't say that this trouble is fixed until this guide will be done step by step without any errors or warnings.

@advaitjain @gunan @hawkinsp @qlzh727 anyone?

@jhseu may be able to help, I do not think anyone else investigated go packages.

fuck, i accidentally delete /usr/local cause i tried to solve this issue by the hands, lol :laughing: :laughing: :laughing:. Thanks nvidia and tensorflow developers to create simple deb packages that are "just works"

Having the same issue as op

Having the same issue, after go generate github.com/tensorflow/tensorflow/tensorflow/go/op
cannot find package "github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto" in any of:
${GOPATH}/src/github.com/tensorflow/tensorflow/tensorflow/go/vendor/github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto (vendor tree)
/usr/local/Cellar/go/1.15/libexec/src/github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto (from $GOROOT)
${GOPATH}/src/github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto (from $GOPATH)
In the core dir only two dirs:

https://github.com/owulveryck/onnx-go i found this - if you want to use a machine learning model, get its onnx version and you can use the linked repo. Its only for using models, you cant train them. This helped me so hope it helps someone who reads this.

@fedyfausto @Saduf2019 @gowthamkpr  have you found any solutions?
Perhaps, we need to create custom bindings for tensorflow, cuz usually google fixing their bugs to much time.

After a lot of try and errors i finally found a little solution:
I installed the last version of TensorFlow C library (2.30.) from BINARY. After that i installed the GO VERSION of tensorflow 2.1.0 and it seems work, so there is some bug in 2.3.0 of go version (anyway the go runs 2.3.0 c libray so it is fine).
Now i'm facing to another problem, how could i compile a single solution within tensorflow library to distribuite the final solution?

@fedyfausto i could cooperate with you to fix this problem (cuz i realy need working ml framework on go), so if yo want, text me on email or telegram, links in my profile.

I will contact you! (Telegram maybe is the perfect solution @fedyfausto)
Dott. Federico Fausto Santoro
Il giorno 20 ott 2020, alle ore 15:58, Richard Cooper<EMAIL_ADDRESS>ha scritto:

@fedyfaustohttps://github.com/fedyfausto i could cooperate with you to fix this problem (cuz i realy need working ml framework on go), so if yo want, text me on email or telegram, links in my profile.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/tensorflow/issues/43847#issuecomment-712873033, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA53P4BHIB3HNKGDHDY647DSLWJSDANCNFSM4SHQXQBA.

I solved this problem by changing the TensorFlow version to 1.13.
Download the corresponding version of TensorFlow, and then go generate $GOPATH/src/......
At the same time, you need to modify the tensorflow version of require in go.mod
require github.com/tensorflow/tensorflow v1.13.2

@kep-w all participants here are already decided that tensorflow 1.13 is works well, but latest version is v2.3.1!!! Also, 2.* versions are mentioned in official guide, so this is not a solution actually.

@fedyfausto @ololosha228 @kep-w @shravanshetty1
I've placed a working Dockerfile for Go 1.15 and TensorFlow 2.3.1 at https://github.com/wamuir/golang-tf . I hope this helps.

@wamuir thanks a lot! i'll try to experiment with your image, and leave a feedback in your repo (or here, idk)

Having the same problem，

@fedyfausto
Do you still face the issue, could you please try with the latest tf version and update us.

Yes this is still a problem. tensorflow/go can only be used with bazel, or by building a localversion and putting it in vendor.
For anyone else struck here, @galeone is maintaining a fork with the propobuf files compiled: https://github.com/galeone/tensorflow (He also have a wrapper for making it even easier to use tensorflow in go: https://github.com/galeone/tfgo )

@Saduf2019 i copy that, tried to rerun "getting started" guide to install and use tensorflow in go, and still lots a lot of problems
To be honest, i prefer to recommend any user who read this comment, use galeone packages cause... Well, they work, at least...

Here's a summary for anyone who wants this summarized:  you cannot use go get to obtain a functional copy of github.com/tensorflow/tensorflow/tensorflow/go.  This is because the package relies on compiled protocol buffers, but compiled protobufs cannot be checked into this git repo (see #44655).  Previously, it was fairly straightforward to obtain a working installation by cloning the Tensorflow repo into ${GOPATH}/src and then using go generate  to compile the buffers, but recent Go module changes and the deprecation of GOPATH have all but made this approach impossible (despite what the Install Guide in this repo would have you believe!).
Things that flat out won't work:

Using go get github.com/tensorflow/tensorflow/tensorflow/go
Following the current installation guide in this repo, which needs an update

You can still use tensorflow/go.  A few solutions that may work for you:

Clone the Tensorflow repo, generate the protocol buffers locally and then use go mod's replace directive: instructions are in SIG Build's Golang Install Guide
Switch to a fork that includes the generated protocol buffers, such as https://github.com/galeone/tensorflow from @galeone
Use Docker images that have Go, the TF shared libs and tensorflow/go installed, such as https://github.com/wamuir/golang-tf
Soon, hopefully, build with Bazel; a PR is at #50934


react-scripts dev server: Support PROXY env var for overridable "proxy"

Usage example:
PROXY=https://some-server.com npm start

Or as a script in package.json:
{
  "scripts": {
    "start": "react-scripts start",
    "start:staging": "PROXY=https://some-staging-server.com react-scripts start"
  },
  "proxy": "https://localhost:8888",
  "...": "..."
}

What is the use case? What does the current proxy solution not solve?

@ianschmitz As I showed in the example you can for instance have few commands to run a frontend using either local backend server or some testing/staging server. With this environment variable you don't have to change package.json contents which is usually under git index (e.g. you can accidentally commit a change of proxy field whilst you're wanted only new dependencies). Also it's inconvenient when you have to provide instructions in readmes like "if you want to run the frontend against another backend server go to some file and change some field to this specific value" while in case of this feature you could just suggest to run npm run start:staging or npm run start:local.

In short words: it adds more flexibility to the proxy feature.

You should be able to achieve your desired behavior today (two different scripts for diff environments) using environment variables within setupProxy.js.

@ianschmitz It required more effort to do it using setupProxy.js. Also it doesn't work the same way (proxy in package.json redirects all 404 requests to the backend while in setupProxy.js you have to provide specific base URL of many routes if endpoints are starting from root). Also the docs page is outdated, I had to debug stuff to realize that API of http-proxy-middleware has changed.
But yes, it's possible to do kind of-ish the same, with more effort and more boilerplate. You're free to close this one but I think this offers a lot more simpler way of doing this whilst adding one line of code and couple of symbols to another line is not a trade-off at all.

We've updated the docs page which will be published with the next release.
I'll see if there's appetite from the other maintainers. Historically the proxy has been a tricky subject with lots of weird edge cases which is why we're generally hesitant to make changes there. Hope you understand.

I still think it's a very simple solution and setupProxy.js couldn't fully replace it, because as I said above it works a bit different. Also setupProxy.js makes it be more complicated whilst just providing environment variable is as simple as it could ever be.

Not stale bump.

Not stale bump.

Not stale bump

Not stale bump

Not stale bump

Nope

Nah

+1

bump

Nope


rainbow folders stopped working

heya again.
i just wanted to let you know the rainbow folders option for the explorer from the style settings has stopped working for some reason.
i don't even use that option, i prefer that panel plain myself.
just wanted to let you know anyway.
i was looking through all obsidians folders settings for a why to scale the font down a little bit in the explorer, and noticed nothing changes when toggling that. i do remember it was working last week.
X

Hi,
Rainbow folders in the MagicUser theme work for folders named with a numbering prefix, like 001 FolderName or 01 FolderName (up to 008 or 08).
For more details, check the "Rainbow Folders" section in the theme's documentation:
https://github.com/drbap/magicuser-theme-for-obsidian#rainbow-folders
If you have any questions about using it, let me know!
Thanks

If you still find it is not working for you, I will remove the wontfix label and check what is happening. I just tested it and it is working well for me.
All the best

gotta ya! than theyre not broken at all.


[WFLY-13633] Undeployed CMR resource JNDI names are not deregisted as needing to be treated as CMR

Issue: https://issues.redhat.com/browse/WFLY-13633
https://issues.redhat.com/browse/WFLY-13632
Thanks for submitting your Pull Request!
Please delete this text, and add a link to the Jira issue solved by this PR.
Remember to use the Jira issue ID in the PR title, and any commits.

@tomjenkinson @bstansberry @mmusgrov Could anyone please review? Thanks :)

This LGTM but let's wait a bit in case @mmusgrov has any concerns.

I don't think this fixes WFLY-13633 (but may fix WFLY-13632). The reason it does not fix WFLY-13633 is because this fix does not remove the name from the underlying collection in Narayana, just a copy of that [collection] (https://github.com/jbosstm/narayana/blob/master/ArjunaJTA/jta/classes/com/arjuna/ats/jta/common/JTAEnvironmentBean.java#L1089) by calling the remove. /cc @mmusgrov

@tomjenkinson doen't jtaBean.setCommitMarkableResourceJNDINames(connectableResourceJNDINames); update the underlying collection in narayana ?

Thanks @gaol - you might be correct there sorry, I will take a look

I think it needs more than just removing it from the list at that point in time.

The initial list is cached statically: https://github.com/jbosstm/narayana/blob/master/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/transaction/arjunacore/TransactionImple.java#L1748
Separately, that list is retrieved by the recovery manager:
https://github.com/jbosstm/narayana/blob/master/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/recovery/arjunacore/CommitMarkableResourceRecordRecoveryModule.java#L123

As you can see in that second reference, the underlying list of names (jndiNamesToContact) is added to with the current list but:
a. is never removed from
b. is not atomic with the removal https://github.com/wildfly/wildfly/pull/13525/files#diff-104e9239790ebc24b14cf99ab71fc053R94
In other words, even after setCommitMarkableResourceJNDINames then the list of names to contact would still contain the older ones.

@tomjenkinson OK, I understand now. Thank you. It is more than I expected, and it needs changes in Narayana first to keep all consistent.

Yes Tom is correct and if we cannot perform a server restart then we would also need a fix in Narayana.

It may be possible that this could resolve WFLY-13632, even if it does not resolve WFLY-13633 - I am not sure if it does and if it does it is possible that the entire patch is still applicable to that case

@gaol Hi, I created a JBTM issue to ensure that these property getters are dynamic.

@gaol [~gaol]. Note that in WildFly CMResourceAdd has a bug in how it deals with reload-required:
https://github.com/wildfly/wildfly/blob/master/transactions/src/main/java/org/jboss/as/txn/subsystem/CMResourceAdd.java#L80
That entire method should really be an if/else block -- it either the server isn't booting and it just calls context.setReloadRequired() or it is booting and it modifies the runtime state by adding the service.
Right now my impression from the discussion is that doesn't matter too much because if CMResourceAdd runs after boot, the service it adds doesn't actually update the state that Narayana is really using. So it's doing pointless work, but it's not modifying the actual runtime behavior and then falsely telling the user it didn't by setting reload-required.
That should be fixed regardless, but particularly if things change in how Narayana handles updates made to JTAEnvironmentBean. Basically, what the handlers for the 'add' and 'remove' ops on these resources do should match what happens inside Narayana. If a change doesn't take effect without a process restart, then they should set restart-required. If only a reload is needed, then reload-required. If they are able immediately to correctly change what Narayana is doing, then no reload-required or restart-required.
The synchronization stuff here is good regardless, because during boot multiple CMResourceService instances can be installed, and they need to coordinate.

@bstansberry Thanks for the review :)
Here are summary of the changes:

I updated the CMResourceAdd part to address the issue Brian mentioned.
I updated the PR summary and description to target https://issues.redhat.com/browse/WFLY-13632 only.
The https://github.com/jbosstm/narayana/pull/1670 may solve https://issues.redhat.com/browse/WFLY-13633 together with this fix.

This PR doesn't fix https://issues.redhat.com/browse/WFLY-13633 because updates to JTAEnvironmentBean by CMResourceService does not change the static references to JTAEnvironmentBean.commitMarkableResourceJNDINames(for example) in other places in narayana.
After :reload operation, the static references to JTAEnvironmentBean.commitMarkableResourceJNDINames(for example) are still the same because they are cached as the static class variables.
Together with https://github.com/jbosstm/narayana/pull/1670, which changes the static class fields to final instance fields, the :reload operation should be able to re create all needed instances with updated JTAEnvironmentBean.
But I think setting the state to restart-required should also be able to fix the problem, I am not sure if changes to JTAEnvironmentBean should lead to a hard restart or not.

This PR will protect against the data integrity issue if we mark the server as restart-required.
@gaol would you be able to update your PR to include that change?

Superseded by #13620


Will not compile on OS X 10.13

I am trying to installing mplcairo via
brew install pkg-config libffi cairo py3cairo
LIBFFI_PC_PATH=`locate libffi.pc | grep usr | head -n 1`
LIBFFI_PKGCONFIG_PATH=`dirname "$LIBFFI_PC_PATH"`
export PKG_CONFIG_PATH="$LIBFFI_PKGCONFIG_PATH"
pip install mplcairo

but I get
src/_mplcairo.cpp:42:3: error: call to unavailable function 'visit': introduced in macOS 10.14
      std::visit([&](auto const& aa) -> void {
      ^~~~~~~~~~
    /Library/Developer/CommandLineTools/usr/include/c++/v1/variant:1524:26: note: candidate function [with _Visitor = (lambda at src/_mplcairo.cpp:42:14), _Vs = <const std::__1::variant<_cairo_antialias, bool> &>] has been explicitly made unavailable
    constexpr decltype(auto) visit(_Visitor&& __visitor, _Vs&&... __vs) {

I am actually running OS X 10.13. Is it possible to install mplcairo on this version?
Just in case, I am on Python 3.7.

Are you running llvm 8?  I realized very recently that for some reason I don't understand yet, you must apparently llvm 7 on macOS10.13 (see https://github.com/anntzer/mplcairo/commit/d2ca6c134398b20c7fa961e35871a256b264b879 for that fix to the CI build).  Can you give that a try?
More insight on what changed with llvm 8 would be welcome too (as I don't actually develop on that platform, so all I can do is make sure CI passes...).

llvm-gcc -v returns
Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin17.7.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

I am not sure how to change the LLVM version in a safe way (I do not want to disrupt my otherwise working environment). Can you assist on this?

brew install <some specific llvm version> does not make that llvm the compiler by default; it prints out some messages explaining the environment variables that need to be additionally set to use that compiler ($PATH, $CPPFLAGS, $LDFLAGS -- see just below the patch linked above) so your environment will be fine (just set these environment variables before running setup.py).

Using LLVM7 works.
Full installation via:
# Install LLVM 7 and set variables for compilers to find it
brew install llvm@7
export PATH="/usr/local/opt/llvm@7/bin:$PATH"
export LDFLAGS="-L/usr/local/opt/llvm@7/lib"
export CPPFLAGS="-I/usr/local/opt/llvm@7/include"

# Install required packages
brew install pkg-config libffi cairo py3cairo

# Set variables for pkg-config to find libffi, which is needed for building
LIBFFI_PC_PATH=`locate libffi.pc | grep usr | head -n 1`
LIBFFI_PKGCONFIG_PATH=`dirname "$LIBFFI_PC_PATH"`
export PKG_CONFIG_PATH="$LIBFFI_PKGCONFIG_PATH"

pip install mplcairo

Clarified the README in https://github.com/anntzer/mplcairo/commit/152d821d7fdb8dc4a73aa6f73e6c68e1aa4cde70; feel free to suggest edits and let me know if you find any other issues.


Puff Core and Puff Blog duplicate code when parsing files in cron scripts.

Once eustasy/puff-core#1 is complete, we can use that new code.

It's complete, commence with merge.


Pipe readiness check errors to stderr

Fixes #457
This fixes the (false) errors we see when this script runs to check if the launch succeeds.
Ran it a few times locally and didn't see those errors anymore ....... so I think we're good.


Updates the version number

Not sure if this was an accident.

Sorry, shouldn't have been a PR. Thanks for closing.


Background and foreground colors do not have a sufficient contrast ratio

Describe the issue
Color for "An Inclusive Design Research Centre project" in the top bar do not have a sufficient contrast ratio.
Currently contrast ratio is 3.37
To Reproduce
N/A
Expected behavior
Contrast ratio should be at least 4.5:1
Screenshots
N/A
Applicable WCAG Success Criteria:

1.4.3 - Contrast (Minimum)

Desktop:
N/A
Mobile device:
N/A
Additional context
N/A

suggest changing to black and underline

Thanks for catching this, @jcabak — I think I'm going to close your PR and go with my colleague's design suggestion instead, which will resolve the issue you found.


findSumOfApSeries.scala

📑 Description

🐞 Related Issue

Closes #

Please add the issue id to this PR

Added sum of Ap series using scala . #3284