ASAN failed on re2 with odr-violation

This issue wasn't revealed until https://github.com/grpc/grpc/pull/25272 add re2 to Bazel.
=================================================================
==1759701==ERROR: AddressSanitizer: odr-violation (0x7fceb4d64ac0):
  [1] size=64 'vtable for re2::Compiler' external/com_googlesource_code_re2/re2/compile.cc
  [2] size=64 'vtable for re2::Compiler' external/com_github_google_re2/re2/compile.cc
These globals were registered at these points:
  [1]:
    #0 0x4600fd in __asan_register_globals (/usr/local/google/home/lidiz/.cache/bazel/_bazel_lidiz/6db67e514c85492694524d2ec993f002/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/test/cpp/end2end/channelz_service_test+0x4600fd)
    #1 0x7fceb4bd6b0b in asan.module_ctor (/usr/local/google/home/lidiz/.cache/bazel/_bazel_lidiz/6db67e514c85492694524d2ec993f002/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/test/cpp/end2end/../../../_solib_k8/libexternal_Scom_Ugooglesource_Ucode_Ure2_Slibre2.so+0xf0b0b)

  [2]:
    #0 0x4600fd in __asan_register_globals (/usr/local/google/home/lidiz/.cache/bazel/_bazel_lidiz/6db67e514c85492694524d2ec993f002/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/test/cpp/end2end/channelz_service_test+0x4600fd)
    #1 0x7fceb20d2b0b in asan.module_ctor (/usr/local/google/home/lidiz/.cache/bazel/_bazel_lidiz/6db67e514c85492694524d2ec993f002/execroot/com_github_grpc_grpc/bazel-out/k8-fastbuild/bin/test/cpp/end2end/../../../_solib_k8/libexternal_Scom_Ugithub_Ugoogle_Ure2_Slibre2.so+0xf0b0b)

==1759701==HINT: if you don't care about these errors you may set ASAN_OPTIONS=detect_odr_violation=0
SUMMARY: AddressSanitizer: odr-violation: global 'vtable for re2::Compiler' at external/com_googlesource_code_re2/re2/compile.cc
==1759701==ABORTING

FYI (because I spent a lot of time debugging a similar issue and found this issue in the process) this is https://github.com/google/sanitizers/issues/1017, a false positive.
If it's failing on a cc_test target, setting linkstatic = True on that target will fix it. Or you can add -mllvm -asan-use-private-alias=1 to the compiler arguments.


test(types): fix type tests

where errors are reported can drift between typescript version, this combines cases where two different lines could be reported for the same issue into a single line so all typescript versions can be verified.
resolves #9

Codecov Report

Merging #10 into main will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines          121       121
=========================================
  Hits           121       121

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 86bd0f5...3686fba. Read the comment docs.

Sweet, thanks!


/bin/ld: cannot find -lstdc++

Keep getting this error when trying to run make on racon. System say's file is installed. I am using Cent 7. Can anybody please shed some insights as to why I am getting this.

Hello,
this might help: http://bytefreaks.net/programming-2/centos-7-c-static-linking-cannot-find-lstdc-lm-and-lc
Best regards,
Robert

Thank you so much Robert. That did the trick.
On Thu, Nov 16, 2017 at 1:31 AM, Robert Vaser<EMAIL_ADDRESS>wrote:

Hello,
this might help: http://bytefreaks.net/programming-2/centos-7-c-
static-linking-cannot-find-lstdc-lm-and-lc
Best regards,
Robert
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/isovic/racon/issues/41#issuecomment-344866108, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgJIp7u1KctMAqeQBiEoRV55BoJgk6r1ks5s3AEMgaJpZM4QfqkO
.

I am getting this error as well. I'm not really keen on installing static libraries. Any reason you cannot use dynamic linking here?

We prefer static libraries :)


Onboarding Request

Your Twitch username
DanThePhenom

Hi thanks for going through the onboarding steps. I checked and didn't find an email associated with your Spotify account. I need that to allowlist you to start queueing songs. My discord is saxypandabear, you can DM it to me there

Got confirmation that it worked. Closing this ticket out. Thanks for the quick response!


Fixes #1912. Adds PYTHON2 variable for use in cpplint

Submisison Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Adds a PYTHON2 make variable to control cpplint behavior.
How to Verify
Run make cpplint PYTHON2=<some other python version>
Side Effects
Removing the RUN_CPPLINT make variable.
Documentation
Updated in the top level make help.
Reviewer Suggestions
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University.
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Bob verified this. I'm merging.


Allow MQTT sensor expire_after to be a float value.

Description:
Currently when setting expire_after of a MQTT sensor to e.g. 0.4 the expire_after mechanism is not triggered. This is because the expire_after value is rounded to 0 resulting in the expire_after mechanism to be disabled.
This PR fixes that problem by interpreting the expire_after as a float value.
Related issue (if applicable): not applicable
Pull request in home-assistant.io with documentation (if applicable): not applicable
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed: not applicable
If the code communicates with devices, web services, or third-party tools: not applicable
If the code does not interact with devices: not applicable

I think Hass is working with an integer precision internal timer, are you sure this works as intended?

I've added logging https://github.com/home-assistant/home-assistant/blob/22820ccae58a570fc5671bae87b3d0217a3eda69/homeassistant/components/mqtt/sensor.py#L139 and https://github.com/home-assistant/home-assistant/blob/22820ccae58a570fc5671bae87b3d0217a3eda69/homeassistant/components/mqtt/sensor.py#L190
expire_after: 0.1:

expire_after: 1:

This seem to trigger the value_is_expired one tick earlier.

@emontnemery is right, our internal clock is based on 1 second so allowing floats to be entered would be incorrect as we can't deliver.

@balloob I agree, perhaps what we need is something like a MQTT event trigger sensor.
Background information
Zigbee2mqtt discovers stateless buttons (e.g. Xiaomi WXKG01LM) as an MQTT sensor. e.g.
sensor:
  - platform: mqtt
    value_template: '{{ value_json.click }}'
    state_topic: 'zigbee2mqtt/my_button'
    name: my_button_click

A button can have multiple click types, e.g. single, double, long, etc. When the button is pressed zigbee2mqtt produces a MQTT message like zigbee2mqtt/my_button with payload {"click": "single"}.
To respond to these button clicks, we wan't an automation in the form of:
- alias: Resond to button click
  trigger:
    platform: state
    entity_id: sensor.my_button_click
    to: 'single'
  action:
    entity_id: light.my_bulb_light
    service: light.toggle

This works for the first button click as the state changes from uknown -> single. However when the button is pressed another time, the automation isn't triggered because the state doesn't change (single -> single).
By using the expire_after: 1 the state of the sensor resets after the 1 second, but this gives the limitation that a button the automation cannot be triggered multiple times a second.
Another limitation of the expire_after: 1 is that the it doesn't expire after 1 second, but rather between 1 and 2 seconds. See https://github.com/home-assistant/home-assistant/pull/20716#issuecomment-460064186
What would be a good solution to this?

The good solution here is to follow our policy of not trying to store events in the state machine but fire events. This is how every other platform does it.
Will close this PR as this won't solve it.

Or I guess in the case of zigbee2mqtt, just have an automation with an MQTT trigger listen to the correct topic.

@balloob this is indeed how it is currently done, however it would be a better solution if the home assistant entity IDs could be used in the automations.


Exec tachikoma update<PHONE_NUMBER>3414

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 68.0% when pulling b10de5d06656dcf85e6a7ec829e9729eeeb022b6 on pazjacket:tachikoma/update-20160403163414 into 0c477cea399180e0dc0455b31d685efdbee0f3af on sanemat:master.


Power the sounder when the key is active

If the user is sending on the keyboard and then switches to the key without closing the circuit, the key sends but nothing is heard on the sounder. The program should detect that the key is active and reapply power to the sounder (unless the sounder has been disabled, see issue #119).

pykob version 1.3.4


[semantic-arc-opts] Extract out the conversion of owned -> guaranteed forwarding insts into a general utility.

I am going to use this to handle converting from guaranteed -> owned in a
different context.

This is just a pure refactor NFC using the current implementation to validate the refactor.

@swift-ci smoke test


I get Error occured, code: 139

Hi, after I run the command below
:PadawanGenerateIndex
I get error this error:
Progress [=================== ] 98%
Error occured, code: 139
Press ENTER or type command to continue
Please, can you figure out what is the problem?

@al-ani from the command line run: padawan generate and paste here last 20-30 lines of output

@mkusher, I got    bash: padawan: command not found

@al-ani how have you installed the padawan.php?


Scale Props is not working

Using The Scale Props is making the PDF Blurry

same problem with scale props

Hi @AmmarMohamadIsmaeel,
The scale prop represents the desired ratio of canvas size to PDF document size. It is set optimal by default, but can be changed it to suit your needs. The ratio should be re-evaluated if the document becomes blurry.


[flake8-async] allow async generators (ASYNC100)

Summary
Treat async generators as "await" in ASYNC100.
Fixes #13637
Test Plan
Updated snapshot

Actually, if I add this test case to main I don't see a violation (i.e., the snapshots do not change) but if I use your example from the issue they do. It seems like your new test case is not sufficient, can you add one that raises the false positive?

I have changed the test.


refactor(semantic): sort ecosystems by name

Having ecosystems sorted by their name makes it easier to review this section of code

Codecov Report
Attention: Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.

Project coverage is 68.28%. Comparing base (8af6458) to head (7d14ca8).

Files with missing lines
Patch %
Lines

internal/semantic/parse.go
88.23%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1363      +/-   ##
==========================================
- Coverage   68.41%   68.28%   -0.14%
==========================================
  Files         183      183
  Lines       17620    17620
==========================================
- Hits        12055    12031      -24
- Misses       4900     4922      +22
- Partials      665      667       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

The uncovered lines will be covered by #1364


Get an error about missing "ResolveDependenciesDesignTime" in Shared Project

Create a C# shared project:

Error	MSB4057	The target "ResolveDependenciesDesignTime" does not exist in the project.	SharedProject1	c:\users\davkean\documents\visual studio 15\Projects\SharedProject1\SharedProject1\SharedProject1.shproj	1

tag @abpiskunov

The rule has been updated to look for the new property. Next vs build will have the fix.

Actually with the updated rule, there is still an error - it says
Error	MSB4057	The target "ResolvePackageDependenciesDesignTime" does not exist in the project.

We shouldn't be putting dependencies in Shared Projects. We should turn off the capability.


Fix Drupal integration issues.

Resolves #189 #190

^ @charginghawk just waiting on the Travis build to finish before merging this in

Please hold off on merging until I can vouch for the Drupal integration. Right now it's still a little rough.

No worries - holding off for now till you give the word

Ready for this to merge on the Drupal side.

@charginghawk I've manually merged in these changes into my new product feature branch (needed a fresh clean branch to make merging back down to master a lot more straightforward) -- these changes are already there and will be part of the very next Bolt release being cut: https://github.com/bolt-design-system/bolt/blob/feature/product-page--fresh-branch/src/_patterns/02-components/bolt-device-viewer/src/device-viewer.twig#L52


Xconf - Multiple Data-services using EKS: Data synchronization issue

Issue Description:
We have deployed Xconf in the EKS cluster with applications being deployed as containers (pods) under the load balancer. The backend DB in the current setup is a single node, Cassandra.
All our API calls are working fine when we are having single container. But some of the API calls are getting failed when the number of backend containers is increased from 1 to many. Apart from that, we have Implemented  Firmware templates and firmware rule DELETE methods.
Current setup
Single Replica (1 Container/pod): Working fine
Multiple Replica (Multiple Containers/pods): Not working
Load balancer setup:  Round Robin algorithm, Stickiness is enabled

Please find the Exception from the application when an API call is fired.
com.comcast.xconf.exception.EntityConflictException: Template analyze-issue template is used by rules:
eg:
We have implemented FirmwareRule delete and FirmwareTemplate delete.
When we try to request the API FirmwareRule to delete, It is able to delete the FirmawareRule. But the immediate request to FirmwareTemplate delete triggers an exception "xconf.exception.EntityConflictException) template is used by rules" whereas the rule has already been deleted successfully by the previous operation. Seems some problem in synchronization between the instance cache memory.
Issue Log:  56381_xconf_log.txt
tried the below service.properties configuration
service.properties.txt
Observations:
We have made the above changes and tested them, but the issue is not resolved yet.

Closed, as not received any input from the dev


--testPathPattern should only match relative path

🐛 Bug Report
When running tests with --testPathPattern it should match only the relative path to the project root. Currently, it matches the full path.
It's not usefull to match the full path because all tests have a common prefix.
To Reproduce
Checkout my repo to directory like /users/<name>/my-project and run
yarn run jest --testPathPattern=users

Example:

Expected behavior
it should match only the relative path.
Instead matching
/Users/sky/work/npm/jest-test-path-pattern-bug/custom.test.js

it should match
custom.test.js
// or with subdirectories
sub-dir/another-subdir/custom.test.js

Link to repl or repo (highly encouraged)
https://github.com/BetterCallSky/jest-test-path-pattern-bug
Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: macOS High Sierra 10.13.6
    CPU: (12) x64 Intel(R) Xeon(R) CPU E5-1650 v2 @ 3.50GHz
  Binaries:
    Node: 8.11.3 - ~/.nvm/versions/node/v8.11.3/bin/node
    Yarn: 1.13.0 - ~/.nvm/versions/node/v8.11.3/bin/yarn
    npm: 6.4.1 - ~/.nvm/versions/node/v8.11.3/bin/npm
  npmPackages:
    jest: ^24.5.0 => 24.5.0

This looks like a simple fix. Let me take a look into it

A more generic approach would be to use a relative path.
const path = require('path');

isMatch: p => regex.test(path.relative(basePath, p))

basePath should be a rootPath or pwd

Not sure how that interaction would play out with the rootDir option.

I'm not sure what the desired behavior here would be. For example, right now I sometimes use checked-out-repo/test.js to run only the test.js in the repo root. The relative path would be just test.js.
I do agree though that the current behavior is confusing in many cases.

This issue is actually more complicated than I anticipated.

Passing in a value that appears in the absolute path to the project directory causes jest to run all the tests. E.g. running jest users on macos matches all tests since /Users/some/path matches the path.

The definition of what a relative path means is confusing. Echo-ing @jeysal 's example.

testPathPattern is a confusing option since doing jest --testPathPattern=hello and jest hello achieve the same thing.

This is a big problem. Common folders to match using testPathPatterns would be things like src and test. If 5 directories up from your project, there is a folder with that name, every single file will always match.
This has happened to me multiple times already under different circumstances. Using gopath for all your projects has this issue especially, because of the src folder in there.

Here is a workaround worked for me
yarn run jest --testPathPattern="$(PWD)/users"


[BUG] error results when build with --prod

The code is ok when i run "ng serve", but when I publish it  with "ng build --prod", something is wrong
It took me a long time to find where the err come from, and eventually I found it.
origin code:
PS. A function to find whether the item is in nodes and return the index.
function(item,nodes){
    let index = -1;
    for (const node of nodes) {
      index += 1;
      if (item === node) {
        return index;
      }
    }
    return -1;
  }

After the compiler's optimazation and zip the corresponding code is:
if always return 0 or -1 because the variable 'e' never plus itself by 1
function(n,l){
for(var e=-1,t=0,i=l;t<i.length;t++)
if(n===i[t])return e+1;return-1}
}

🌍  My Environment
"@angular-devkit/build-angular": # test on "0.802.2" && "0.83.18"
"@angular/animations": "^8.2.6",
"@angular/common": "8.2.2",
"@angular/compiler": "8.2.2",
"@angular/core": "8.2.2",
"@angular/forms": "8.2.2",
"@angular/platform-browser": "8.2.2",
"@angular/platform-browser-dynamic": "8.2.2",

Can you provide the output of npm ls terser?
If interested and assuming nodes is an Array, that entire function could be replaced with nodes.findIndex(....).  It is polyfill-ed automatically by the CLI as it is used by Angular itself so it is safe to use for ES5 browsers

<EMAIL_ADDRESS>D:_xxxxxx
--<EMAIL_ADDRESS> +--<EMAIL_ADDRESS> --<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>deduped

Thank you, I've learned.

I checked with both<EMAIL_ADDRESS>and 4.4.0 (latest) and the output for both was var func=function(r,n){for(var f=-1,t=0,u=n;t<u.length;t++){if(f+=1,r===u[t])return f}return-1};
My setup was this:

package.json

{
  "name": "random",
  "scripts": {
    "test": "tsc input.ts && terser --compress --mangle -- input.js"
  },
  "dependencies": {
    "terser": "4.4.0",
    "typescript": "3.7.2"
  }
}

input.ts

var func = function (item, nodes) {
  let index = -1;
  for (const node of nodes) {
    index += 1;
    if (item === node) {
      return index;
    }
  }
  return -1;
}

Not sure what's happening. Might be a weird interaction in our separate terser compress/mangle?

Thanks for your patience, I have replaced the function with old style for-loop "(let i = 0; i < xx.length; i++) ",  everything works well. Maybe this err is caused by complex context.

Tried reproducing outside of terser itself and was able to.
I added the code below to app.component.ts:
const myFn = function (item, nodes) {
  let index = -1;
  for (const node of nodes) {
    index += 1;
    if (item === node) {
      return index;
    }
  }
  return -1;
}

const nodes = [1, 2, 3];
console.log(myFn(0, nodes), myFn(1, nodes), myFn(2, nodes), myFn(3, nodes), myFn(4, nodes));

Running ng serve will show -1 0 1 2 -1 on the console, but ng serve --prod will instead show -1 0 0 0 -1.
So there definitely is something in our pipeline causing this. Will look into it.
Versions in my repro:
Angular CLI: 9.0.0-rc.1
Node: 10.16.0
OS: win32 x64
Angular: 9.0.0-rc.1
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.0-rc.1
@angular-devkit/build-angular     0.900.0-rc.1
@angular-devkit/build-optimizer   0.900.0-rc.1
@angular-devkit/build-webpack     0.900.0-rc.1
@angular-devkit/core              9.0.0-rc.1
@angular-devkit/schematics        9.0.0-rc.1
@ngtools/webpack                  9.0.0-rc.1
@schematics/angular               9.0.0-rc.1
@schematics/update                0.900.0-rc.1
rxjs                              6.5.3
typescript                        3.6.4
webpack                           4.41.2

kamik@RED-X1C6 MINGW64 /d/sandbox/next-app (master)
$ npm ls terser
<EMAIL_ADDRESS>D:\sandbox\next-app
`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>deduped
  `--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>      `--<EMAIL_ADDRESS>deduped

kamik@RED-X1C6 MINGW64 /d/sandbox/next-app (master)
$ npm ls @babel/core
<EMAIL_ADDRESS>D:\sandbox\next-app
`--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>

This can be replicated using the above code with terser only and appears to be a variation of this terser issue: https://github.com/terser/terser/issues/475.  The ++ case was fixed but += 1 appears to still fail.

Note: I changed the issue title to better reflect the problem.

Nice work!

At this point we're waiting for a Terser fix, and will pick it up automatically when available.

As per the referenced terser issue, this has been corrected in terser 4.5+.  This version will be available in 9.0.


counter

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.82%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cli/cmd/backup.go
0
1
0.0%

pkg/cli/cmd/restore.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1042

Relevant Lines:
2909

💛  - Coveralls


Setup testing

[ ] verloopcontext
[ ] vtwirp
[ ] protoc-gen-python-verloop

Hi, is this work still going on?

Hi! We keep up with twirp official spec.
From stability POV, it's used in production at least at Verloop (and probably other places)
Work on this particular issue had gone into limbo as all of us went on to do other things in life.
We're definitely accepting PRs for it and might pick this again based on community (especially contributor's) interest.


3.3.0: issue with generated setup.py in entrypoints module

Because flit generats setup.py wgichhas line from distutils.core import setup instead from setuptools import setup generated setup.py is useless not only because it is not possible use setuptools<>sphinx integration but even with distutils that file is incorrect.
+ /usr/bin/python3 setup.py build '--executable=/usr/bin/python3 -s'
/usr/lib64/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'extras_require'
  warnings.warn(msg)
/usr/lib64/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'python_requires'
  warnings.warn(msg)
running build
running build_py
creating build
creating build/lib
copying entrypoints.py -> build/lib
+ PBR_VERSION=0.3
+ SETUPTOOLS_SCM_PRETEND_VERSION=0.3
+ SPHINXOPTS=-j48
+ /usr/bin/python3 setup.py build_sphinx -b man --build-dir build/sphinx
/usr/lib64/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'extras_require'
  warnings.warn(msg)
/usr/lib64/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'python_requires'
  warnings.warn(msg)
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: invalid command 'build_sphinx'

Probably the same issue is with every other module which uses flit.

Just FTT with quick patch:
--- a/setup.py~ 1970-01-01 01:00:00.000000000 +0100
+++ b/setup.py  2021-08-24 16:31:12.249073906 +0100
@@ -1,7 +1,7 @@
 #!/usr/bin/env python
 # setup.py generated by flit for tools that don't yet use PEP 517

-from distutils.core import setup
+from setuptools import setup

 extras_require = \
 {":python_version == '2.7'": ['configparser>=3.5']}

above commands output is:
+ /usr/bin/python3 setup.py build '--executable=/usr/bin/python3 -s'
running build
running build_py
creating build
creating build/lib
copying entrypoints.py -> build/lib
+ PBR_VERSION=0.3
+ SETUPTOOLS_SCM_PRETEND_VERSION=0.3
+ SPHINXOPTS=-j48
+ /usr/bin/python3 setup.py build_sphinx -b man --build-dir build/sphinx
running build_sphinx
Running Sphinx v4.1.2
making output directory... done
WARNING: html_static_path entry '_static' does not exist
loading intersphinx inventory from https://docs.python.org/objects.inv...
intersphinx inventory has moved: https://docs.python.org/objects.inv -> https://docs.python.org/3/objects.inv
building [mo]: targets for 0 po files that are out of date
building [man]: all manpages
updating environment: [new config] 2 added, 0 changed, 0 removed
reading sources... [100%] index
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
writing... entrypoints.1 { api } done
build succeeded, 1 warning.

BTW distutils is obsoleted by https://www.python.org/dev/peps/pep-0632/

It's been deprecated, but won't be removed until python 3.12, whose release is over two years away. Legacy non-pep517 setup.py builds (using distutils or setuptools) are going to go away along with distutils and legacy setuptools usage.

setuptools is not growing anyway away as long as it provides crucial tooling infrastructure which none of new modules hooked under PEP517 are not able to replace, and as stuptools just Works™.
'distutils' offer small subset of operations which setuptools can offer, it is not pluggable and it is used now by handful of modules. That subset recently slightly increased only because few modules started using flit.
In other words flit in current form somehow is trying to rollback some already done changes.
According to observation of my rpm packages distutils before flit started generating distutils depended setup.py files was used in ~1%. Since than this doubled to ~2%.
What is the sense of sticking with distutils? I have no idea ..

The aim of using distutils there is to discourage people from relying on the generated setup.py - it was intended as a stopgap measure for older versions of pip, and it's going to go away.
https://github.com/takluyver/flit/blob/734ab12fc5bfadf7e37dc303d44fd517f7ebda1b/flit/sdist.py#L16-L18

The aim of using distutils there is to discourage people from relying on the generated setup.py

Why do you want to do that if flit cannot do the same things which setuptools do and all operations which flit provdes is possible to implemt using setuptools?

You should read PEP517 and PEP518. It might help you understand the motivations to move towards a standardized build interface that doesn't defacto depend on setuptools.

You should read PEP517 and PEP518. It might help you understand the motivations to move towards a standardized build interface that doesn't defacto depend on setuptools.

I've read both documets and I was not able to find anything about moticvations (whatever in this context it mean in tis case).
Maybe it would be better if you could point please on exaxt parts of those documents?
What can solve on this case discoraging use setuptools is generally it works?
I can understand that there is a lot of legacy in many modules which are not using recent setuptools possibilities but discarding  whole setuptoosl in case when none of other tooling is able to do what setuptools is able to do is at least a bit odd.

The problems with distutils/setuptools and legacy build systems is covered in the abstract and rationale sections of both peps.

Again .. currently setuptools is able to hadne new approach.
Just look on setup.py, setup.cfg, pyproject.toml files for example in pip modile https://github.com/pypa/pip/
Please let me know if you see anything missing in that example ..
With current state og the pip there is no any issues with using setuptools<>sphinx integration.
If you want .hwl file you can generate it without build modules. Still it is possible to handle buiilding DSOs using for example cythoon (about which .wheel doesn';t know anything). All modules which are producing DSOs it is about 1/12 of all modules. flit and poetry do not provide any way to maintaind such modules.
What is IMO missing in PEP617 is standarication of metadata description. flit uses own [] sections and the same poetry instead reusing setuptools descriptions.


Support ending for the .NET Framework 4.5.1

As annonced here, .Net Framework 4.5.1 is no longer supported.
Target framework in all project.json shouldn't be net452 instead of net451 ?

Our libraries can run down level and that's important because of things like mono. However we'll only support applications running on .NET >= 4.5.2.

I don't understand

Their librairies are backward compatible. No issue with the code.
What is ending is support. Not backward compatibility.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Language dependent values in params.toml

Hello.
Was wondering if it's possible to set different values in params.toml based on selected language.
Would be great for values such as whoami, location and even the description.

The title and description should be defined inside the _index.md file inside each language directory and theme should take values from there instead from languages.toml as it is now, what is nonsense. languages.toml should only be taken to display values in breadcrumb and nothing more.


[wip] first pass review of this library

Merged with some changes of my own. Anything else you'd add @TomHAnderson ?


Feature Request: Netlify Image CDN using Astro Adapter?

https://www.netlify.com/blog/astro-netlify-adapter/
Is this something that would be easy to implement?
I tried to do it but could not get it to work.
But for those who code it might be easy. Any thoughts?
Paul

Hey @theapplegates, thanks for your request!
I looked into it and implementation is straightforward: Documentation However, I don't see a benefit as I'm using Vercel for deployment, where this is not necessary.
But you are welcome to add this to you project if you're using it. It is a highly opinionated starter template but I encourage anyone to adjust it to their liking. 🤗

You're right. I see how they are similar. I appreciate the feedback.
Paul


CI running on push to master and PRs

I noticed that CI was running twice if I created a branch and made a PR. How do we feel about this new config?

Having CI run on push is nice if you just want to have tests running via the CI, but you are not ready for doing a PR yet. Though I don't think it's super important.


Add credits and online resume makers

HAPPY?

looks good, ty ming

i thought we were trying to be professional on our github prs


Composer

"./magento composer update" does not work, since composer is looking in the root directory of Magento for the .json file. Would it work to remove the "/web/" from the hard-coded directory in the magento wrapper script? Other ideas?
EDIT - Removing the hard-coded "/web/" from the magento script worked :)

Sorry for the late answer. I have fixed the issue using your proposed solution.
Thanks!


Reviewer 2 request 1 changes for improved instructions on installing GOMC package

Add installation instructions for GOMC in README and docs, as well as… fix tests for improved error messages and automated checking of PATH for GOMC installation

I took a quick look at this and here is my recommendations, tested in RHEL:
@Cal Craven Another thought and I actually like this better .  we may want to avoid building it in "opt" as many not experience users will not be able to get rid of it.  Why not build it in the actual conda env directly and symlink it to the conda bin directory.  This should also eliminate having to modify their bash.rc (meaning not messing it up and likely not differences between Mac and Ubuntu and RHEL).  Tested in RHEL
This will need changed in a few files with checks/errors if
Need to add to all effected python files:
	import subprocess
In the dihedral fit code,  if “gomc_binary_directory”=None (if not default, make it so)
	if  gomc_binary_directory=None:
	gomc_binary_directory = subprocess.check_output("echo ${CONDA_PREFIX}/bin", shell=True, text=True).strip()
# We may want to check if conda exists first or print an error if this command fails maybe try/execpt.

Setup the GOMC minay in the correct
conda activate mosdef_dihedral_fit
     or  (from the activated env)
cd $CONDA_PREFIX   (env needs to be activated so will only work when in that env, which is likely what we want)

git clone https://github.com/GOMC-WSU/GOMC.git --branch v2.75a
cd GOMC
chmod u+x metamake.sh
# these options turn off all but the GOMC_CPU_NVT ensemble, which is all that is really needed and used for mosdef_dihedral_fit.
./metamake.sh NVT
ln -s $CONDA_PREFIX/GOMC/bin/GOMC_CPU_NVT $CONDA_PREFIX/bin


Analyzer Feedback from IntelliJ

Analyzer Feedback from IntelliJ
Version information

IDEA AI-191.8<IP_ADDRESS>91312
2.7.0
AI-191.8<IP_ADDRESS>91312, JRE 1.8.0_202-release-1483-b03x64 JetBrains s.r.o, OS Windows 7(amd64) v6.1 Service Pack 1, screens 1366x768

Exception
Dart analysis server, SDK version 2.7.0, server version 1.27.4, FATAL error: FileSystemException: writeFrom failed, path = '' (OS Error: Insufficient system resources exist to complete the requested service.
, errno = 1450)
#0      Driver._captureExceptions.errorFunction (package:analysis_server/src/server/driver.dart:723:7)
#1      _CustomZone.handleUncaughtError (dart:async/zone.dart:1005:19)
#2      _CustomZone.runGuarded (dart:async/zone.dart:927:7)
#3      _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
#4      _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#5      _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#6      _Timer._runTimers (dart:isolate-patch/timer_impl.dart:393:30)
#7      _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:418:5)
#8      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

For additional log information, please append the contents of
file://C:\Users\Lambo\AppData\Local\Temp\report.txt.

Thanks for reporting this issue! This was previously reported at #38914. Please follow along there.

Duplicate of #38914


Successfully deployed but demo doesn't work

I've used the one click deploy and the resources have been successfully deployed in my subscription.
However when i launch the app and enter the chat, nothing happened.
I've ensured that i've filled the cosmosdb and signalr connection string though.

issue solved due to the incorrect connection string of cosmos db.
Do not put the mongodb connection string, instead please use:
"Endpoint=https://{cosmosdb-name}.service.signalr.net;AccessKey={key};".
I have also raised a pull request on the readme.md file


Recipes for ic2 compatible stuff doesn't have recipes when ic2 classic is loaded.

Advanced gens <IP_ADDRESS>
Ic2 classic <IP_ADDRESS>
It can be easily fixed by checking for the modid ic2-classic-spmod within the ic2 modid check here:
https://github.com/bdew/generators/blob/mc1.12/resources/assets/advgenerators/config/recipes.cfg#L95

I would like to second this.
Based on my very limited testing, everything in Advanced Generator's IC2 compatibility would be easily compatible with IC2 classic. It'd just be a matter of setting the default recipes, and it would likely work fine.

The mod has been recently updated to MC 1.16, older versions are no longer supported.
I'm mass closing older issues to clear up clutter in the bug tracker.
If you find this issue still exists in the current version feel free to reopen it.


Having a requirements_minimal file

That file is in the repo and it should contain the minimal amount of package for pymapdl to work. No pyvista, no reader, no matplotlib, etc.

Closed in #2443


@remix-run/firebase

Probably just grab David East's template and go.

What features should this template include?
Probably Auth & Firestore.
I would love to help.

The template is probably reference to https://github.com/davideast/remix-starter-firebase (Remix adapter for running on Firebase Functions)

Alright, so this Issue is about firebase as a deployment target, not about an example on how to use remix and firebase together.

I'd assume so yeah given the title :)
Although a Remix + Firebase example would probably be worthwhile too.

Probably not going to add an adapter for this anytime soon, though something in the examples directory would be great. Also a great candidate for someone to build a stack on top of. Closing for now since there's nothing actionable for us.


Feature Request:  compute_framingham

Feature Idea
I am happy to help program function;
#' Function to derive Framingham equation
#' #'
Relevant Input
Smoking flag, Treated for Hypertension flag, diabetes flag, age, sex, SBP, Total Chol, and HLDDIR
Relevant Output
Framingham calculated result
Reproducible Example/Pseudo Code
compute_framingham <- function(smokefl, trthypfl, diabetfl,
age, sex, sbp, tchol, hdldir) {
smokefn <-  if_else(smokefl == "Y", 1, 0, missing = 0)
diabetfn <- if_else(trthypfl == "Y", 1, 0, missing = 0)
sbpt <- if_else(trthypfl == "Y", sbp, 1, missing = 1)
spbu <- if_else(trthypfl == "N", sbp, 1, missing = 1)
logtc <- log(tchol)
loghdl <- log(hdldir)
logsbpt <- log(sbpt)
logsbpu <- log(sbpu)
aval <- case_when(
sex == "F" ~ 1 - (0.95012 ^ exp((2.32888 * logage)
+ (1.20904 * logtc)
- (0.70833 * loghdl)
+ (2.76157 * logsbpu)
+ (2.82263 * logsbpt)
+ (0.52873 * smokefn)
+ (0.69154 * diabetfn)
- 26.1931)),
sex == "M" ~ 1 - (0.88936 ^ exp((3.06117 * logage)
+ (1.12370 * logtc)
- (0.93263 * loghdl)
+ (1.93303 * logsbpu)
+ (1.99881 * logsbpt)
+ (0.65451 * smokefn)
+ (0.57367 * diabetfn)
- 23.9802)))
aval
}

Thanks @aehmann-gsk! That looks interesting. Which ADaM dataset is this applicable to? We'd also need an accompaying derive_param_framingham() function.

@thomas-neitmann We have a cardio risk factors dataset which is BDS where this is calculated at a visit level.  However, this is also used as a baseline calculation in ADSL (sometimes used as a baseline covariate in modeling or shown as a baseline characteristic).

@thomas-neitmann Here is the CDISC CT for it:

@thomas-neitmann I have time today/tomorrow.  Is it ok if I create a pull request and draft this function with an associated derive_param?

We won't have this in the latest release

@bms63 , why did you close it? It seems that it is not merged yet.

done in mistake.  i closed any with linked PRs


[Accessibility] Fixed the screen reader issues in the main page

Fixed the following issues : https://github.com/Microsoft/pxt/issues/2538, https://github.com/Microsoft/pxt/issues/2565, https://github.com/Microsoft/pxt/issues/2589, https://github.com/Microsoft/pxt/issues/2629

@veler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


9349 embed permissions

Fixes #9349

Please CR @juanignaciosl

I've made a minor suggestion, up to you 👍


Add http_options property to remote_file

This lets the remote_file resource set any of the Chef::HTTP options.
It also exposes the Net::HTTP settings like "ssl_timeout" through a "nethttp" sub-hash.
Note that "rest_timeout" can set be set at the higher level which is the Chef::HTTP/Chef::Config option that winds up setting both the read_timeout and open_timeout to the same value at the lower nethttp setting.  Setting the nethttp settings explicitly will override those completely.
The defaults on the read_timeout and open_timeout are 300 seconds (due to needing to work around crushed chef-servers caused by thundering herd starts of hundreds of chef-clients at the same time).  Since that may also happen with thundering herds of chef-clients hammering on some expensive remote_file resource at the same time that default is deliberately kept to avoid a breaking change.
remote_file "/tmp/remote_file.out" do
  source "https://www.chef.io"
  http_options({
    http_retry_delay: 0,
    http_retry_count: 0,
    keepalives: false,
    nethttp: {
      continue_timeout: 5,
      max_retries: 5,
      read_timeout: 5,
      write_timeout: 5,
      ssl_timeout: 5,
    },
  })
end

replaces #11905

Fixes https://github.com/chef/chef/issues/11870 and https://github.com/chef/customer-bugs/issues/422


Added custom color for selected title

Added custom title color feature -

Pass titleColor property to theme object to set text color of selected title.

Example -
<Chrono 	items={items} theme={{ primary: "#0f52ba",  secondary: '#ffdf00', titleColor: 'white' }} />

@nkhunters thanks for the contribution


fix datetime comparison

csrf: support aware/naive date_signed comparison
release: v1.2.8

@lnielsen for merging (I can't)


NFS: Support defining multiple attachment points

This has been on the backlog for a bit, but it came up when thinking about migrating the Scaffolder to the New Frontend System. Field Extensions are an example where these extensions can be used in multiple places and therefore potentially many PageExtensions or even just any other extensions.
There's one way around this, which is to use a Context and just have one page extension, or prop drilling but those both feel a little limiting when you might want these things outside of the React context.
We tried implementing this without touching the AppNode and just have resolveAppTree return multiple extensions, but then ran into the issue that there would be duplicate extensions, which also then lead to potentials of factory functions being called more than once, therefore this appeared like the better fix.
@Rugvip thoughts here?

Hmm, it's not my favorite solution tbh because it introduces some complexity to the app structure, although so do all potential solutions at some point tbh.
I think our best current solution to this is to refactor into using a Utility API for accessing the field extensions instead. It could be that that's enough tbh.
What I've been thinking is also kinda clean is to treat plugins as extensions, although with the way we do overrides now we couldn't use the closure approach we experimented with early on. It might for example be that the plugin itself can output data that you can via node.source.get() or such? Not sure if that has many benefits over a Utility API approach in the end though.

@Rugvip hmm, yep OK. Will keep this one around for now, but will investigate today what it looks like to move this to some scaffolderFormFieldsApiRef or something, and see how that pans out.
Maybe it would be nice to push this into API's as it's a little cleaner admittedly to just have one attachment point rather than having that list grow and become pretty brittle with more usages of field-extensions especially outside of the scaffolder plugin potentially.

Gonna close this and potentially revisit if necessary later on down the line.


can't send nearby notification in another language?

Hello
The Google beacon platform(dashboard) offers a feature where you can send a notification in local language or any other language by entering the language ISO code.
I have tried it with multiple languages but only english(en) works.
Is this an issue?Or is there any fix method to work on it
Thanks
Regards

I've forwarded this to the beacon platform team and they are looking at it

Thanks Scott
Abhishek Kanodia
Under Graduate Student
BITS Pilani
On 17 November 2016 at 23:58, Scott Jenson<EMAIL_ADDRESS>wrote:

I've forwarded this to the beacon platform team and they are looking at it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/physical-web/issues/853#issuecomment-261327975,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AINyQX2MmlllUr4J_RJ0hwkNtC6q5piYks5q_JzOgaJpZM4K08XY
.

Also, I cannot see the message "Nearby notifications is temporarily muted"
Does that mean this issue has been resolved and the new Google play
services have been released with the fix?
Abhishek Kanodia
Under Graduate Student
BITS Pilani
On 18 November 2016 at 12:10, abhishek Kanodia<EMAIL_ADDRESS>wrote:

Thanks Scott
Abhishek Kanodia
Under Graduate Student
BITS Pilani
On 17 November 2016 at 23:58, Scott Jenson<EMAIL_ADDRESS>wrote:

I've forwarded this to the beacon platform team and they are looking at it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/physical-web/issues/853#issuecomment-261327975,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AINyQX2MmlllUr4J_RJ0hwkNtC6q5piYks5q_JzOgaJpZM4K08XY
.

Hi I am from Google Nearby team. I wonder how does your beacon dashboard looks like? A screenshot would be most helpful.
Also yes the "Nearby notifications is temporarily muted" has be resolved. You should be seeing notification very soon if not already.

@xizhgoogle: Apology for the late reply.
Can you please exactly point out which specific page of the dashboard you are looking for?
Thanks

@xizhgoogle : any update on this?


Fix git fetch

Github seems to have closed down the original git protocol that results in this error:
fatal: remote error:
  The unauthenticated git protocol on port 9418 is no longer supported.
Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.

This PR fixes the error by switching to the https git protocol method in SRC_URI.

FYI, existing Yocto/bitbake releases now have a fix to patch protocol=https for github.com SRC_URI but in Yocto master it's a hard error. The patch makes this layer forward compatible with Yocto 3.5.

Thank you for the PR. It's now merged.


Avoid displaying null for unknown store address or zip code

The app currently displays 'null' when it doesn't know a location's exact address or zip code, so this renders those fields conditionally.

Thank you! Sorry I haven’t gotten a chance to review and merge this yet, but hopefully tomorrow! I really appreciate your contribution!

This is live now. Thank you!


tags-file: call cursor(x, y) doesn't work as a tag-address without a " at the end

Reading :h tags-file-format, it says that the address (the command used to move the cursor to the tag location) can contain a ;" to make it backwards compatible.
http://vimdoc.sourceforge.net/htmldoc/tagsrch.html#tags-file-format
{term}		;" The two characters semicolon and double quote.  This is
		interpreted by Vi as the start of a comment, which makes the
		following be ignored.  This is for backwards compatibility
		with Vi, it ignores the following fields.

Most tag file generating programs use /some-search/ as the command here, and that works fine. However I can't get it to work with call cursor. I made a tag generating tool that uses call cursor as the command for moving the cursor, however I can't get it to work as the doc says it should.
 This means I generate something like this:
compareTags	../stags/src/test/scala/co/pjrt/stags/Utils.scala	call cursor(15, 7)	language:scala

However, when I do that and try to call :tag compareTags I get E488: Trailing characters. This means vim does care about this? So next I tried adding :" at the end:
compareTags	../stags/src/test/scala/co/pjrt/stags/Utils.scala	call cursor(15, 7);"	language:scala

But I get the same error. Finally, I just tried:
compareTags	../stags/src/test/scala/co/pjrt/stags/Utils.scala	call cursor(15, 7)"	language:scala

I did this because I've never heard of ; being a comment in vimL, only ", and it worked! Only with that was I able to get :tag compareTags to work.
Is the documentation wrong? Or am I reading it incorrectly.

Pedro Rodriguez Tavarez wrote:

Reading :h tags-file-format, it says that the address (the command used to move the cursor to the tag location) can contain a ;" to make it backwards compatible.
http://vimdoc.sourceforge.net/htmldoc/tagsrch.html#tags-file-format
{term}		;" The two characters semicolon and double quote.  This is
		interpreted by Vi as the start of a comment, which makes the
		following be ignored.  This is for backwards compatibility
		with Vi, it ignores the following fields.

Most tag file generating programs use /some-search/ as the command here, and that works fine. However I can't get it to work with call cursor. I made a tag generating tool that uses call cursor as the command for moving the cursor, however I can't get it to work as the doc says it should.
 This means I generate something like this:
compareTags	../stags/src/test/scala/co/pjrt/stags/Utils.scala	call cursor(15, 7)	language:scala

However, when I do that and try to call :tag compareTags I get E488: Trailing characters. This means vim does care about this? So next I tried adding :" at the end:
compareTags	../stags/src/test/scala/co/pjrt/stags/Utils.scala	call cursor(15, 7);"	language:scala

But I get the same error. Finally, I just tried:
compareTags	../stags/src/test/scala/co/pjrt/stags/Utils.scala	call cursor(15, 7)"	language:scala

I did this because I've never heard of ; being a comment in vimL, only ", and it worked! Only with that was I able to get :tag compareTags to work.
Is the documentation wrong? Or am I reading it incorrectly.

The ;" works to terminate an address, such as a line number or search
string.  You are using an Ex command, there using a single " before a
comment works.  However, since the ;" is missing the extra fields won't
be found.  You can use
compareTags	Utils.scala	call cursor(15, 7)" ;"	language:scala
Thus put the ;" inside the comment
--
I got a new desk stapler. It broke on the first desk I tried.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Not sure I follow, so the " is part of the Ex command? What does call cursor(x, y)" mean? (I understand what call cursor is, just not sure what the " is doing).

Wait, are you saying that " is the comment? Then what's the purpose of ;"? And most other tags software just have /seach/;".  Is the ; part of the /search/ command?

Pedro Tavarez wrote:

Wait, are you saying that " is the comment? Then what's the purpose
of ;"? And most other tags software just have /seach/;".  Is the
; part of the /search/ command?

The double quote is the start of a comment.  The rest of the line is
ignored when it is executed.
For an address ;" is used, because the semicolon ends the address.
When parsing a tags file the code specifically looks for the ;" to
locate the extra fields.
call cursor(11, 22) positions the cursor in line 11 column 22, which
should be where the tag is defined.  This is brittle, if lines are
inserted or deleted the position will change.  That's why most tags are
defined with a search string.
--
hundred-and-one symptoms of being an internet addict:
70. ISDN lines are added to your house on a hourly basis
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

When parsing a tags file the code specifically looks for the ;" to
locate the extra fields.

If that's the case, then the doc is wrong. The doc makes it seem like it is a hack to make it compatible with vi, but you are saying it is actually required (even if you never expect the tag to be called from within vi).

This is brittle, if lines are
inserted or deleted the position will change.  That's why most tags are
defined with a search string.

Good point, didn't think about that. Thanks.

I agree the behavior is either wrong or the documentation is very misleading.  The format is listed as:
"{tagname}		{TAB} {tagfile} {TAB} {tagaddress} {term} {field}"
Where tagaddress is any ex command.  However if you actually use any ex command other than a line number or pattern, vim does not parse the terminator ;" and instead keeps it as part of the command.  So you lose kind and any extra fields (displayed in eg, tselect)

I agree the behavior is either wrong or the documentation is very misleading.  The format is listed as:
"{tagname}		{TAB} {tagfile} {TAB} {tagaddress} {term} {field}"
Where tagaddress is any ex command.  However if you actually use any
ex command other than a line number or pattern, vim does not parse the
terminator ;" and instead keeps it as part of the command.  So you
lose kind and any extra fields (displayed in eg, tselect)

Right, the command is found but any fields are ignored.
Since the command can be anything, we need a reliable way to find the
end.  I think using |;" is specific enough.  The bar is then dropped
from the command.
--
ARTHUR: The swallow may fly south with the sun, or the house martin or the
plover seek warmer hot lands in winter, yet these are not strangers to
our land.
SOLDIER: Are you suggesting coconuts migrate?
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


weave-net pod can not be created on windows server 2016 node

** Is this a BUG REPORT? **
while creating a weave-net pod on windows server 2016 node in a kubernetes cluster, i got the following error message.
I1202 10:58:09.329692    7076 eviction_manager.go:325] eviction manager: no resources are starved
I1202 10:58:09.514693    7076 volume_manager.go:366] All volumes are attached and mounted for pod "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)"
I1202 10:58:09.514693    7076 kuberuntime_manager.go:403] No ready sandbox for pod "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)" can be found. Need to start a new one
I1202 10:58:09.515691    7076 kuberuntime_manager.go:571] computePodActions got {KillPod:true CreateSandbox:true SandboxID:ac1f28b8dc1bb8f287d420abc40496753838cb9373d8142b8046ca31b08b3800 Attempt:32 NextInitContainerToStart:nil ContainersToStart:[0 1] ContainersToKill:ma
p[]} for pod "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)"
I1202 10:58:09.515691    7076 server.go:231] Event(v1.ObjectReference{Kind:"Pod", Namespace:"kube-system", Name:"weave-net-drbkl", UID:"9700a989-d74f-11e7-a4cd-0289db2a7c18", APIVersion:"v1", ResourceVersion:"559651", FieldPath:""}): type: 'Normal' reason: 'SandboxChange
d' Pod sandbox changed, it will be killed and re-created.
I1202 10:58:09.584692    7076 docker_service.go:437] Setting cgroup parent to: ""
I1202 10:58:09.585693    7076 docker_service.go:437] Setting cgroup parent to: ""
E1202 10:58:09.775691    7076 remote_runtime.go:92] RunPodSandbox from runtime service failed: rpc error: code = Unknown desc = failed to start sandbox container for pod "weave-net-drbkl": Error response from daemon: No such network: host
E1202 10:58:09.775691    7076 kuberuntime_sandbox.go:54] CreatePodSandbox for pod "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)" failed: rpc error: code = Unknown desc = failed to start sandbox container for pod "weave-net-drbkl": Error response from
daemon: No such network: host
E1202 10:58:09.776690    7076 kuberuntime_manager.go:647] createPodSandbox for pod "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)" failed: rpc error: code = Unknown desc = failed to start sandbox container for pod "weave-net-drbkl": Error response fro
m daemon: No such network: host
E1202 10:58:09.777690    7076 pod_workers.go:186] Error syncing pod 9700a989-d74f-11e7-a4cd-0289db2a7c18 ("weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)"), skipping: failed to "CreatePodSandbox" for "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd
-0289db2a7c18)" with CreatePodSandboxError: "CreatePodSandbox for pod "weave-net-drbkl_kube-system(9700a989-d74f-11e7-a4cd-0289db2a7c18)" failed: rpc error: code = Unknown desc = failed to start sandbox container for pod "weave-net-drbkl": Error response from daemon:
No such network: host"
if i create a weave network with docker:
docker create network --weave weavenet
the error message is "weave net plugin can not be located"
What you expected to happen?
What happened?

How to reproduce it?

Anything else we need to know?

Versions:

$ weave version
$ docker version 17.09
$ uname -a
$ kubectl version 1.9.3.alpha

Logs:
$ docker logs weave

or, if using Kubernetes:
$ kubectl logs -n kube-system <weave-net-pod> weave

Network:

$ ip route
$ ip -4 -o addr
$ sudo iptables-save

Weave Net is currently Linux-only.
Looks like the main error you got was that Kubernetes on Windows does not understand the "host network" concept. Given that, if didn't even run the Weave Net software.
I guess you could ask for a better error from Kubernetes.


No statistics found.

Version of the custom_component
2.0.1
Describe the bug
When an entity from this component is added to a statistic graph, all stat types are grayed out and the graph displays No statistics found.

Adding this to extra_state_attributes seems to fix the issue:
'state_class': 'measurement'

Thanks for this, I'll take a look when I get some time


Fix collection and community detection in logic condition

The InCollectionCondition and InCommunityCollection did not handle Workflow and WorkspaceItems correctly. This should be an easy fix.
Description
The DSpace Logic Framework allows to test if an item meets conditions. The conditions can be configured using Spring. This allows for configurable filters for example to decide whether an item should get a DOI assigned.
Instructions for Reviewers

Setup a Community and a collection
Configure the collection to have a WorkflowStep
Create a WorkspaceItem in the collection (create an item in the submission, do not submit it, save it for later)
Configure a filter in dspace/config/spring/api/item-filter.xml that checks if the item belongs to the collection you selected in the submission
Configure a filter in dspace/config/spring/api/item-filter.xml that checks if the item belongs to the community to which the collection belongs you selected in the submission
use /dspace/bin/dspace dsrun org.dspace.content.logic.TestLogicRunner -f <nameOfYourFilter> -i <handleOfYourItem> to check both filters. The command returns false if the item does not pass the filter and true if it passes the filter. Without the patch a Workfspace and WorkflowItem will not pass the test, with the patch it should pass the test.
reopen your submission and submit it, so that it is in the workflow
rerun the test above.

Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome).
However, reviewers may request that you complete any actions in this list if you have not done so. If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is created against the main branch of code (unless it is a backport or is fixing an issue specific to an older branch).
[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] My PR includes details on how to test it. I've provided clear instructions to reviewers on how to successfully test this fix or feature.
[x] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

The code looks good, but I'm having an issue testing this PR. I have three items in a collection with a workflow (1 archived item, 1 workflow item, and 1 workspace item). The command bin/dspace dsrun org.dspace.content.logic.TestLogicRunner -f <nameOfYourFilter> -i <handleOfYourItem> works with the archived item because it has a handle. But the workflow and workspace items don't have handles. I suspect this is true, but I'm asking to make sure. Does this PR require identifiers.submission.register = true be set in [dspace]/config/modules/identifiers.cfg?

@pnbecker Thanks for that clarification. I set that parameter to true and tested again with a few workspace and workflow items that now have handles. I'm still on the main branch, and my custom item filters for belonging to a collection or a community return true for both workspace and workflow items.
So the filters appear to be working without the PR. I may know why. While it is true that for a workspace or workflow item there are no collections returned by this line in the condition, which is what the PR is intended to fix.
https://github.com/DSpace/DSpace/blob/be9257080124a009ff193199c6e5b6ac451d8898/dspace-api/src/main/java/org/dspace/content/logic/condition/InCollectionCondition.java#L45
Later on in the condition there is also a check for the parent object of the item.
https://github.com/DSpace/DSpace/blob/be9257080124a009ff193199c6e5b6ac451d8898/dspace-api/src/main/java/org/dspace/content/logic/condition/InCollectionCondition.java#L54-L72
This second check returns true for me for workspace and workflow items. I think that is because itemService.getParentObject(context, item) checks for workspace and workflow items when returning a collection.
https://github.com/DSpace/DSpace/blob/be9257080124a009ff193199c6e5b6ac451d8898/dspace-api/src/main/java/org/dspace/content/ItemServiceImpl.java#L1560-L1578
Please let me know if I'm not understanding the situation.


Provide button to delete merged pull request

cherry-pick from gogits@44ed991, resolved #433

Linting fails.

LGTM

LGTM


Downloads 30kb XIPs

I just installed xcodes (using homebrew) and tried installing Xcode 13.4.
However it only downloads a 30kb XIB file, without reporting any error.
Installing obviously fails.
I updated aria2 to the latest version.
I tried downloading with --no-aria2.
I tried downloading a different Xcode version (13.3).
Same problem with all of those.
I am attaching the downloaded 30kb file (zipped so that Github accepts it):
Xcode-13.4.0+13F17a.xip.zip

Don't know if it is relevant, but I did not use my main Apple ID to authenticate (and I won't), but one that only has the Developer role in AppStoreConnect.

@pocketpixels shouldn't matter as long as it's an Apple Developer account
Can you try to xcodes signout first, and re-login again? How's your disk space?

I just tried signing out and back in, but it made no difference.
Free disk space is a lot more than 30kb, 130GB free currently.

It turns out the issue was with the account I was using.
I finally tried logging in with that account (which I don't typically use) and try to access https://developer.apple.com/download/, and it turns out I needed to accept the Developer Agreement terms and conditions.

After that downloading using xcodes seems to be working fine.
@MattKiazyk I'll leave it to you to close this issue or not. If there is a way to detect this condition it would be nice to present an error message and instructions to the user instead of just downloading a 30kb file and reporting success.

I thought I had this case but I guess it's not fully covering it.
No keep it open. Hopefully my account will get that scenario soon and I can debug.
Thanks for letting me know

I think you should be able to test this by adding a new "Developer" role user to your dev account, under "Users and Access" in App Store Connect.


Reading: Enable exporting reading debug data

Who is this PR for?
internal team, reading specialists or literacy coaches
What problem does this PR fix?
There's no way to get all the reading data out for other analysis.  As we migrate more data in, this keeps other uses open.
Also factors out <DownloadIcon /> svg.
Screenshot (if adding a client-side feature)

Checklists
Which features or pages does this PR touch?

[x] Reading debug

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

[x] Included specs for changes
[x] Manual testing made more sense here

selfie


Quick outline doesn't show containers anymore

re https://github.com/microsoft/vscode/issues/93643

open stopwatch.ts from our source
items like stop, _now dont have the container description anymore

1.43 stable

insider

Happy you found this, an oversight on how I  compute the container.

🦅 👁


package:pedantic/analysis_options.yaml cannot be found

seen result
$ cat pubspec.yaml
name: pmap
description: Parallel map funcion
homepage: https://github.com/gaaclarke/pmap
version: 0.3.0
environment:
  sdk: '>=2.12.0 <3.0.0'
dev_dependencies:
  pedantic: ^1.11.0
  test: ^1.16.5

$ pub get
Resolving dependencies...
Got dependencies!

$ cat analysis_options.yaml
# Defines a default set of lint rules enforced for
# projects at Google. For details and rationale,
# see https://github.com/dart-lang/pedantic#enabled-lints.
include: package:pedantic/analysis_options.yaml

# For lint rules and documentation, see http://dart-lang.github.io/linter/lints.
# Uncomment to specify additional rules.
# linter:
#   rules:
#     - camel_case_types

analyzer:
#   exclude:
#     - path/to/excluded/files/**

$ dart analyze --fatal-infos
Analyzing pmap...                      0.9s

warning • The include file package:pedantic/analysis_options.yaml in /Users/aaclarke/dev/pmap/analysis_options.yaml cannot be found at analysis_options.yaml:4:10 • (include_file_not_found)
   info • Unnecessary new keyword at lib/error.dart:4:21 • (unnecessary_new)
   info • The value of the local variable 'mapper' isn't used at lib/error.dart:6:23 • (unused_local_variable)
          Try removing the variable, or using it.
   info • Omit type annotations for local variables at lib/error.dart:10:3 • (omit_local_variable_types)
   info • Unnecessary new keyword at lib/error.dart:10:29 • (unnecessary_new)
   info • Omit type annotations for local variables at lib/error.dart:11:3 • (omit_local_variable_types)
   info • The value of the local variable 'isolate' isn't used at lib/error.dart:11:11 • (unused_local_variable)
          Try removing the variable, or using it.

7 issues found.

expected result
I expected there to be no error about finding package:pedantic/analysis_options.yaml
This tracker is for issues related to:
Analyzer
dart version
$ dart --version
Dart SDK version: 2.12.1 (stable) (Wed Mar 10 10:18:47 2021 +0100) on "macos_x64"

Hmm, the above steps should work. Is this issue consistent? When analyzing via an IDE do you see the same issue?
Does the .packages file contain an entry for pedantic (and the .dart_tool/package_config.json file as well)?

@devoncarew The error shows up in VSCode, too.  Looks like the file is there on disk.  I have a the error case setup on my machine.  I'll try to leave it like that in case you have any followup questions.
$ cd <PROJECT>
$ grep pedantic .packages
pedantic:file://<HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0/lib/
$ find <HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0/ -name "analysis_options.yaml"
<HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0//lib/analysis_options.yaml
<HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0//analysis_options.yaml

@devoncarew The error shows up in VSCode, too.  Looks like the file is there on disk.  I have a the error case setup on my machine.  I'll try to leave it like that in case you have any followup questions.
$ cd <PROJECT>
$ grep pedantic .packages
pedantic:file://<HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0/lib/
$ find <HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0/ -name "analysis_options.yaml"
<HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0//lib/analysis_options.yaml
<HOME>/.pub-cache/hosted/pub.dartlang.org/pedantic-1.11.0//analysis_options.yaml

I also just hit this recently. On Dart SDK version: 2.13.0-edge.8c64f32db50723858a511a43c6ab7a9f1c17f6c6.

The next time I see this, I'll check my .packages and .dart_tool/package_config.json as well.

See also: #42910


Revert mappingproxy hack

The hack was introduced in #829, but is unnecessary since #905.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #938   +/-   ##
=======================================
  Coverage   64.78%   64.78%
=======================================
  Files          89       89
  Lines       15881    15881
  Branches     3587     3587
=======================================
  Hits        10289    10289
  Misses       3232     3232
  Partials     2360     2360

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 80e9bd8...7d0ab63. Read the comment docs.


Add address_sorting to PYTHON-MANIFEST

Fixes https://github.com/grpc/grpc/issues/14978 for v1.11.x branch

Hi @mehrdada , I noticed that "pip install --upgrade google-cloud" is currently broken on my system due to this issue. Looks like the command resolves to "grpcio-1.11.0rc1.tar.gz", which probably doesn't have the fix.
[...]
Downloading/unpacking grpcio<2.0dev,>=1.0.2 (from google-gax<0.16dev,>=0.15.7->google-cloud-bigtable<0.29dev,>=0.28.1->google-cloud)
  Downloading grpcio-1.11.0rc1.tar.gz (14.2MB): 14.2MB downloaded
[...]
third_party/address_sorting/address_sorting.c:41:38: fatal error: address_sorting_internal.h: No such file or directory

This might be slightly off-topic, but as a pip noob, I'm wondering whether there's a way to instruct pip to incorporate this fix as a workaround until the next release is rolled out (or to use an older version without this issue).

@madari I think invoking pip install --upgrade grpcio==1.10.1' should get you to the stable build. Note that 1.11.0rc1` is a prerelease version and you should expect potential failures there.

@mehrdada thanks for the help! I just found out that this works too: pip install --upgrade grpcio google-cloud.


Problem with missing image in IE 10 & IE 11

I have posted an issue demo: http://zen33.github.io/chakra/pdf/web/, it works well in Chrome and Firefox. But the image is missing in IE 10 and IE 11.

My pdf file is: http://zen33.github.io/chakra/pdf/web/test.pdf, it should display as below:

is there any solution until now?

not yet. @TammyHe19900522

The links above are broken.

Closing as duplicate of #6957.


Support CI

Thie project needs CI based on Github Actions to ensure the quality of development. And now about the CI what I can come up with is these jobs:

python style check
rust style check: (refer to actions in ceresdb)

rustfmt
clippy

And as for unit tests, now no unit test in this project has been provided and the test framework of the python project has not been decided yet so I suggest skipping the UT part in the CI for now.

Adding style checks are easy. I was thinking more about adding CI to build Python wheels, I can do this along the way.
I do think the dependency issue I mentioned should be addressed first, otherwise CI could be terribly slow.

That all-in-one dependent delegate brings lots of unnecessary deps to other crates like this client. I've suffered from it already 😷


fix make test failure of missing sha256 package

Looks like since PR #4517 was merged, make test fails, at least on darwin/x86-64, and linux/arm64 (https://logs.openlabtesting.org/logs/26/4526/6bc6efaefef4864a2347f38c642cc24343cfe6d3/check/containerd-build-arm64/85fa8cc/).
error messages is
2020-09-04 01:03:59.591076 | ubuntu-xenial-arm64-openlab | panic: sha256 not available (make sure it is imported) [recovered]
2020-09-04 01:03:59.591113 | ubuntu-xenial-arm64-openlab | 	panic: sha256 not available (make sure it is imported)
2020-09-04 01:03:59.591127 | ubuntu-xenial-arm64-openlab |
2020-09-04 01:03:59.591149 | ubuntu-xenial-arm64-openlab | goroutine 33 [running]:
2020-09-04 01:03:59.591175 | ubuntu-xenial-arm64-openlab | testing.tRunner.func1(0x4000195800)
2020-09-04 01:03:59.591207 | ubuntu-xenial-arm64-openlab | 	/usr/local/go/src/testing/testing.go:874 +0x348
2020-09-04 01:03:59.591231 | ubuntu-xenial-arm64-openlab | panic(0x3c1460, 0x40001c69d0)
2020-09-04 01:03:59.591262 | ubuntu-xenial-arm64-openlab | 	/usr/local/go/src/runtime/panic.go:679 +0x194
2020-09-04 01:03:59.591331 | ubuntu-xenial-arm64-openlab | github.com/containerd/containerd/vendor/github.com/opencontainers/go-digest.Algorithm.Hash(0x4647e2, 0x6, 0x0, 0x40000cde18)
(snip)

Reverting c50ff694f013cac12651b813a285d2f32ce9f4cd makes this fixed, but this commit contains another workaround.
Fixes: c50ff694 ("refactor(native): separate init from implementation")
Signed-off-by: Hajime Tazaki<EMAIL_ADDRESS>

@AkihiroSuda we don't fail CI for arm64 at this time (the openlab bot just reports it in a comment), and we only lint & build binaries on darwin, so no test runs. A few people did notice (@tianon commented about it somewhere) :) the issue with arm64 but no one had dug into resolving it yet

Not a maintainer, but I love seeing the CI passing 😁 (vigorously LGTM 😅)


Add more validations and refactor and improve tests.

Add more validations to the SAMLResponse class.
Add more validations to the LogoutResponse class
Add more validations to the LogoutRequest class
Refactor and improve tests (+coverage).

@luisvm @Lordnibbler  Can you review that 1st commit of this PR?

:+1: lgtm

@luisvm, @Lordnibbler,  can you validate this PR to be merged?

:+1:


Isolation from the external networks

Is there a way to allow the communication only between the containers and the host.
This would be useful for restoring a live system, to prevent it from communicating outside to other systems.

See https://stackoverflow.com/questions/39913757/restrict-internet-access-docker-container


Soft Deletetable for EntitySchema?

As the question mention, I'm not using decorator but using entities schema, how can I use this package with that specific usecase?

I think it's not supported yet. It would be great if you could drop a PR to implement the support!


build: make init-mei-classpath depend on init target

Before this commit, the properties set in the init target were only available in this target. Consequently, they would not be available in the dist target, so the build-guidelines-pdf target was never called.

Yes, works again :)


Press event on scrollabars

I have a problem with the press event in desktop browsers. When I scroll the div with overflow:auto using scrollbar (not a mouse wheel) press event is fired despite the cursor was actually moved.
Here is the example.
Environment: Chrome 44.0.2403.130 m, Windows 8.1.

Hi, sorry for the late.
Nice catch! I won't fix this as it's really an edge case and can easily be fixed by ignoring the scrollbar in your css selector, something like this works in Chrome $('body:not(::-webkit-scrollbar)')on('...'). For other browsers you can read this.
If you want to generalize this to your whole application with something a little less verbose, you can create a custom selector.
This would allow you to do something like this: $('body:no-scrollbars)')on('...'), assuming you created a no-scrollbar custom selector.


ERR_MODULE_NOT_FOUND when yarn start

$ sst start
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/wangcode/Work/foundry-sst/node_modules/kysely-data-api/dist/esm/data-api-dialect' imported from /Users/wangcode/Work/foundry-sst/node_modules/kysely-data-api/dist/esm/index.js
    at new NodeError (node:internal/errors:371:5)
    at finalizeResolution (node:internal/modules/esm/resolve:416:11)
    at moduleResolve (node:internal/modules/esm/resolve:932:10)
    at defaultResolve (node:internal/modules/esm/resolve:1044:11)
    at ESMLoader.resolve (node:internal/modules/esm/loader:422:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:222:40)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

stupid mirror site
it cached "kysely-data-api" 0.1.3 version and never sync it.


stripped down library

I notice that the dependencies include PCL and Opencv. the chained dependencies makes the required installation pretty big. I only needs the raw images from the sensors and not the post processing part. Any recommendations on possibly getting a strip down version?
thanks

The next release of the software will include a significant modularity refactoring.  In short, we will be breaking the library apart into roughly the following components: common, config, framegrabber, and image. The last of which will carry the OpenCV and PCL dependency. The framegrabber will allow for just getting a simple byte buffer from the camera from which you can construct into whatever image data structure is most convenient for your environment. Realistically it should be available mid-to-late March.
Tagging @graugans as this will be of interest to him as well.


Masked Language Modelling with a pretrained language model

Hi,
I'm interested to continue training an already pre-trained model using the masked language task.
When I pass in the --from_pretrained flag it gives an error. I can see in create_train_parser there is no add_argument for the argument --from_pretrained. Would it be as simple as adding the functionality to read in the argument for --from_pretrained in main.py and then the pre-trained model will subsequently be loaded properly as the starting point for more training?
Thanks.
Scott

Hi Scott,
The parsers are built somewhat hierarchically from a base parser. So the construction would be:
base_parser = create_base_parser()
train_parser = create_train_parser(base_parser)

The base parser has the --from_pretrained flag. You could also add the flag manually, if you would prefer.


TODO: make the card list overlap

Make the cards in the card-list div on the game-room component overlap
Make sure that they work well on all screen sizes

Used flexbox to get cards to overlap


Keep submitted url - closes #4319

The URL is copied to the new-story page and the Link input field is disabled there.

@roelver Nice work!


Create a timeline

Make a list of all (important) life events, milestones, achievements, periods of pleasant and unpleasant experiences, etc. In this particular case, there are no long lists, but only poorly compiled ones, so we put everything in here, and you can keep silent about the "extra" events later. The list should be systematised and sorted in such a way that the reader quickly understands the "pattern" and can easily navigate the space and time of the summary.

Make a list of all (important) life events, milestones, achievements, periods of pleasant and unpleasant experiences, etc. In this particular case, there are no long lists, but only poorly compiled ones, so we put everything in here, and you can keep silent about the "extra" events later. The list should be systematised and sorted in such a way that the reader quickly understands the "pattern" and can easily navigate the space and time of the summary.


Wrong type

https://github.com/vim/vim/blob/ef0a1d5ed3566b91143d30ae9de3240f47c6e282/runtime/tutor/tutor#L557
Fix to: ** Type :s/old/new/g to substitute 'old' for 'new'. **

There is no error. «Substitute "new" for "old"» is another way to say «replace "old" by "new"». The "for" in the sentence means something like "instead of". I'm not sure how to say it in Portuguese. Sostituar 'nova' para 'velha' maybe?
Best regards,
Tony.

Thanks @tonymec by the explanation


Error when interacting with localnet

I don't get any output from the worker, it doesn't print system contract addresses.
yarn localnet

yarn run v1.22.21
$ concurrently --names "NODE,WORKER" --prefix-colors "blue.bold,green.bold" "npx hardhat node" "wait-on tcp:8545 && yarn worker"
[NODE] Started HTTP and WebSocket JSON-RPC server at http://<IP_ADDRESS>:8545/
[NODE]
[NODE] Accounts
[NODE] ========
[NODE]
[NODE] WARNING: These accounts, and their private keys, are publicly known.
[NODE] Any funds sent to them on Mainnet or any other live network WILL BE LOST.

[NODE] Any funds sent to them on Mainnet or any other live network WILL BE LOST.

When I try making a call, it throws an error:
npx hardhat zevm-call --network localhost

Error calling ReceiverEVM: ProviderError: Transaction maxFeePerGas<PHONE_NUMBER>) is too low for the next block, which has a baseFeePerGas of<PHONE_NUMBER>5
    at HttpProvider.request (/Users/fadeev/github.com/zeta-chain/protocol-contracts/node_modules/hardhat/src/internal/core/providers/http.ts:96:21)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async EthersProviderWrapper.send (/Users/fadeev/github.com/zeta-chain/protocol-contracts/node_modules/@nomiclabs/hardhat-ethers/src/internal/ethers-provider-wrapper.ts:13:20)

[NODE] eth_chainId
[NODE] eth_accounts
[NODE] eth_accounts (2)
[NODE] eth_chainId
[NODE] eth_blockNumber
[NODE] eth_chainId
[NODE] eth_estimateGas
[NODE] eth_getBlockByNumber
[NODE] eth_feeHistory
[NODE] eth_sendTransaction
[NODE]
[NODE]   Transaction maxFeePerGas<PHONE_NUMBER>) is too low for the next block, which has a baseFeePerGas of<PHONE_NUMBER>5
[NODE]

@skosito


Web based version no longer works

See screenshot...

Hiya, thanks for reporting this. There's a couple of things going on here:

Like most assemblers, the parser requires that instructions have whitespace in front of them. If you add one or more spaces or tabs before move it does work
The error handling for this looks to be broken. I'm not too sure why there's a random bit of HTML visible, but I'll sort it.

Thanks.. yeah, the random html threw me off in such a way that I forgot all about asm statements needing indentation :-)


[<IP_ADDRESS>] Client handling of points with nil value is broken

If a nil is provided as a client.Point value, the client will generate bogus values (with improper formatting). This then aborts the remainder of any batched points that are submitted.
Reproducible (for bogus values):
func TestPoint(t *testing.T) {
    pt := client.Point{
        Measurement: "pt_name",
        Tags: map[string]string{
            "tagname":   "pt_tag",
        },
        Fields: map[string]interface{}{
            "value": nil,
        },
        Time: time.Now(),
    }
    fmt.Println("PT:",pt.MarshalString())
}

This is so far in the past it's no longer relevant


BeforeSave changes are not propagated into the result of a PUT operation with REST API

Hi,
It seems that the changes into the beforeSave trigger are not propagated to the result of a PUT operation, even if they are correctly saved into the database. I'm not sure if this is the same problem as reported into #1031, because I have it only when I update an object, not when I create it.
Environment Setup
"parse": "^1.7.1",
"parse-server": "^2.1.6",
Steps to reproduce
Suppose you have a class named Book. Create a beforeSave trigger that increments a property to 10, each time a book is saved:
// main.js
Parse.Cloud.beforeSave("Book", function(request, response) {
  var test = request.object.get("test");
  request.object.set("test", test + 10);
  response.success();
});

Create a Book like this, on Parse.com and on parse-server:
curl -X POST \
  -H "X-Parse-Application-Id: appId" \
  -H "X-Parse-REST-API-Key: restApiKey" \
  -H "Content-Type: application/json" \
  -d '{"title":"The old man and the sea","test":0}' \
  http://localhost:1337/parse/classes/Book

Now, update your Book like this, on Parse.com and on parse-server:
curl -X PUT \
  -H "X-Parse-Application-Id: appId" \
  -H "X-Parse-REST-API-Key: restApiKey" \
  -H "Content-Type: application/json" \
  -d '{"title":"Don Quichotte"}' \
  http://localhost:1337/parse/classes/Book/the_object_id

Logs/Trace
The result of POST on Parse.com:
{
  "objectId":"gwMrwEKQ1K",
  "createdAt":"2016-03-15T12:14:51.583Z",
  "title":"The old man and the sea",
  "test":10
}

The result of POST on parse-server:
// OK
{
  "objectId":"uH7K8zMfxD",
  "createdAt":"2016-03-15T12:04:27.931Z",
  "updatedAt":"2016-03-15T12:04:27.931Z",
  "title":"The old man and the sea",
  "test":10
}

The result of PUT on Parse.com:
{
  "updatedAt":"2016-03-15T12:15:13.811Z",
  "title":"Don Quichotte",
  "test":20
}

The result of PUT on parse-server:
// THE UPDATED FIELD ARE MISSING
{
  "updatedAt":"2016-03-15T12:06:13.163Z"
}

In both cases, the properties are correct into the database, but the response of the PUT operation is incomplete with parse-server, whereas it contains all the updated fields with Parse.com.

Thanks for reporting! I'll check it out!

Just FYI, given the REST API documentation, the response object is not supposed to hold the updated keys. That would be an enhancement then

Hi,
Thanks for your responses.
My current application uses it to synchronize the object on the client-side with what it is into the database after a beforeSave operation that updates the fields. So, the things will not work properly on parse-server because of that. Why there is this difference on parse-server, or between POST and PUT operations?
What should I do? Re-fetch each object after each save operation to be sure that the fields are correctly synchronized with what they are into the database after a beforeSave operation? Or remove the beforeSave triggers and rewrite each client to handle it on their own side?
Thanks.

POST is to create an object, PUT to update one.
You can re-fetch the objects, but I understand that can be problematic. What client do you use in your application? Do you only do REST calls?

Yes, I use pure REST API calls.
I know that POST is to create and PUT to update. I just don't know why PUT could not return the fields with the updated values like POST does.

it's just not implemented that way, PR are welcome :)


Docs: Add Gateway Plugin Support

What type of PR is this?
/kind documentation
Which issue(s) this PR fixes:
a part of #651

I have submitted a PR #693. After the PR has been merged in, you split the content of the document and write the document in accordance with different ingress.

/retest

/lgtm

/lgtm

/lgtm

@hzxuzhonghu PTAL


[Target] Decouple ObjCLanguageRuntime from LanguageRuntime

git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@364098 91177308-0d34-0410-b5e6-96231b3b80d8
This PR also has a patch to account for needed changes to swift. The cherry-picked patch has also landed in upstream-with-swift, but the patch to adjust for swift hasn't. It should propagate from stable into upstream-with-swift.
cc @weliveindetail @dcci @JDevlieghere

@swift-ci please test


Serving: include serving throughtput in the plots

/test watsonx-serving-light quick

/test watsonx-serving-light quick

/test watsonx-serving-light

/test watsonx-serving-light

test passed, merging


treewide: remove unused/redundant arguments, format workspace module, fix whitespace, remove with lib;

Various code refactoring changes. We may want to make use of Nixpkgs' EditorConfig settings/CI for this repo as well.
One of the commits is marked as NEEDS DISCUSSION. Replacing declared, yet unused arguments with ... is frowned upon by some, but others prefer having no unused args. Let me know if that commit should be deleted.

Everything else looks good. I really liked the change from the unused args to ....

Changes were addressed, the widgets commit was renamed, and there's a new commit to modernise/normalise the URLs in the documentation/code commits.

Thanks for your contribution again :)


[SR-16123] Non-escaping closure capturing inout wrongly assessed as escaping => error

Previous ID
SR-16123

Radar
None

Original Reporter
@dabrahams

Type
Bug

Status
Resolved

Resolution
Done

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug

Assignee
None

Priority
Medium

md5: 131508c8606cf9082245704a5b9fd7a9

Issue Description:
func f(a: inout Int) {
  let change: ()->Void = {
    a += 1
  }
  _ = change()
}

Diagnostics:
x.swift:2:26: error: escaping closure captures 'inout' parameter 'a'
  let change: ()->Void = {
                         ^
x.swift:1:8: note: parameter 'a' is declared 'inout'
func f(a: inout Int) {
       ^
x.swift:3:5: note: captured here
    a += 1
    ^

Yet somehow, the compiler knows that g doesn't escape in:
func call(_ f: ()->Void) {
  let g = f
  _ = g
}

func f(a: inout Int) {
  call {
    a += 1
  }
}

@dabrahams should this be reopened? I see it was marked "resolved" on bugs.swift.org by another user within a couple hours of you filing it, and there is no comment indicating why.


Explain when client-side constants in offline-first apps get updated

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Needs team review, will remove the draft flag once we're done reviewing it!

We should not mention PWA here

Done ✔

@ConnorLand I think this is for your component, please take a look.

@ConnorLand I think this is for your component, please take a look.

@ConnorLand I think this is for your component, please take a look.

Thank you. Resolving conflict and merging now.

@ConnorLand I think this is for your component, please take a look.

Thank you. Resolving conflict and merging now.

This PR was meant for the Constants doc in RefGuide8, and successfully did so before the Mx9 migration. Therefore I've taken the text from Osama and updated the RefGuide8 doc myself.
I'll shut this PR down now, because it targets the Mx9 doc mistakenly.

This PR was meant for the Constants doc in RefGuide8, and successfully did so before the Mx9 migration. Therefore I've taken the text from Osama and updated the RefGuide8 doc myself.
I'll shut this PR down now, because it targets the Mx9 doc mistakenly.


Adding a Java ball coloring problem

Hi, I really found this repository super fun. Many great problems contributed. I too made my first contribution to this awesome repo with a simple yet fun ball coloring problem. I added the problem statement, example, explanation and time and space complexities to the code in the form of comments.
Do let me know if any improvements are to be made.
Thanks again.

What a great code 😄 !
Awesome description! I hope you had fun writing this code, it sure looks nice!
Thank you for contributing! 🚀

Thanks @CaioCordeiro. I did have fun writing the code.😄


two validation results tables in reports, ValidationResult.before_resample

[ ] Closes #xxxx .
[ ] I am familiar with the contributing guidelines.
[ ] Tests added.
[ ] Updates entries to docs/source/api.rst for API changes.
[ ] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[ ] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.
[ ] fix latex failure to show before/after correctly. hope this also fixes latex ci tests.

@alorenzo175 ci finally passing


catching the case where there are no measurements, so histFlag is emp…

…ty and no length. Setting histFlag to false so it can continue through the measurement function without erroring.
AKA AQCU-883

Coverage decreased (-0.02%) to 89.37% when pulling 31f32a1e3a0cfa1a4a7a7a42ab2da8e2299bfe0b on mhines-usgs:master into 80310dab2ad0e48702a9a5e2965435b7ce9c27f3 on USGS-R:master.


add test for disable checksumoffload on IPIP device

Description

add test for disable checksumoffload on IPIP device
Related issues/PRs

https://github.com/projectcalico/calico/pull/8031#pullrequestreview-1746597070
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

/sem-approve

@tomastigera motivation is in the description - it's from this linked PR comment: https://github.com/projectcalico/calico/pull/8031#pullrequestreview-1746597070

@cyclinder I pinged you on calico user slack, I would be happy to hear about your setup as I am trying to repro this issues so that we can fix them without disabling the offload. I would be happy to talk! Thanks

Hi @tomastigera, Received your message. Let's discuss this in slack. :)

Hi @caseydavenport @tomastigera, Is it still necessary to merge this pr？

@cyclinder I was about to ask Tomas the same thing!
@tomastigera what say you?

@cyclinder now we enabled it by default so I don't think we need that. We might eventually want to make upgrades to turn it back on and that would like to test.