block.timestamp - now is wrong

When a contract is deploying block.timestamp (alias of) now is 0. But when call some method it becomes the right value in second in Unix time. To properly test time-based code it needs to be consistent: always at 0 when deploying and after when query or always real Unix time, not a mixin.

To explain more clearly an example:
pragma solidity ^0.4.25;

contract Time {

  uint256 startTime = block.timestamp;
  uint256 constructorTime;
  uint256 nonce = 0;
  uint256 writeTime;

  constructor() public {

    constructorTime = block.timestamp;

  }

  function readOnly() public view returns (uint256, uint256, uint256, uint256, uint256){

    return (
    startTime,
    constructorTime,
    nonce,
    writeTime,
    block.timestamp
    );

  }

  function write() public {

    nonce = nonce + 1;
    writeTime = block.timestamp;

  }

}

const Time = artifacts.require('./time.sol');

contract('Time', () => {

  before(async () => {

    this.time = await Time.deployed();

  });

  it('Get wrong time', async () => {

    const read = await this.time.readOnly();
    // read = 0 0 0 0 unixTime1

    await this.time.write();

    const read1 = await this.time.readOnly();
    // read1 = 0 0 1 0 unixTime2
    // expect: read1 = 0 0 1 unixTime1 unixTime2
    // And also better expect constructorTime and startTime to be the correct unixTime not 0

  });

});

I am closing this issue because this is a (well known) issue in JavaTron.
TronBox acts as a simple interface but the smart contract is managed by JavaTron.
Please, create an issue in https://github.com/tronprotocol/java-tron/issues


Cannot save any analytics models to dashboards.

Summary
Cannot save any analytics models to dashboards.
Steps to reproduce

What is the current bug behavior?

What is the expected correct behavior?

Relevant logs and/or screenshots

Possible fixes

This is :bug: Bug Report

fixed


Digital dice roll

We want to copy the effect that FoundryVTT and DNDBeyond have, shown here: https://www.youtube.com/watch?v=ajRUv30tfyA
From reading through forum posts, the DNDBeyond implementation actually runs a physics simulation right in the browser and determines the rolls based on the final result of the physics.
Let's explore what tools we would look at to implement this.

I saw this while looking for existing D&D projects a couple days ago: https://www.npmjs.com/package/@3d-dice/dice-box
It's a cool little animation that includes a physics simulation on a flat surface, like what you were talking about.

PERFECT find. I don't know how I missed that myself. I spent a few hours looking at AMMO.js, Bullet.js, Three.js, etc. We should definitely use this.

Latest version (literally released a few hours ago) of the aforementioned package is bork. I opened an issue and hopefully it'll get resolved soon.

The developer fixed the package and I was able to successfully add it to the project. Unfortunately it lacks Typescript definitions so I had to also try to write my own TS declaration file that matches the package's API. Very fragile...


Can't control evap speed

Hi again,
after some of the latest updates (unsure which one exactly) I can no longer control the fan speed at all when set to "heat/cool" -> "evap". Instead I get this alert and the fan speed doesn't change:

I also can't change from heat/cool -> auto mode because the water pump doesn't appear after re-setting up the integration, and I get this error:

BUT once I change the mode to auto on the NC6 itself, I can set the "comfort level" temperature or set the heating temperature
Also, somewhere along the way I've lost the ability to set the "fan mode" control section and can no longer set it to fan only:

Can you try again with https://github.com/funtastix/rinnaitouch/releases/tag/0.9.3 ?
Should fix the fan speed at least.
The water pump one I cannot fully understand. Are there any steps to reproduce this?
For the fan only mode, it can only be turned on when NOT in EVAP preset. Ideally though, you would use the Circ Fan switch instead, and in Evap mode the water pump and fan switches (i.e. turn the water pump off for EVAP fan only mode). The difference is that in EVAP fan mode (water pump off) the air comes through the cooling vents, in heater fan only mode, it comes through the heating vents.

Btw, one thing I did notice is that after removing and re-adding the integration without a restart, the entity IDs get a "_2" in the name. Could this be the water pump problem root cause?

Fan speed was fixed with 0.9.3 thank you!
The way I have the climate card I'm using set up is as you've described. In heat/cool (manual) -> EVAP mode you can control the fan speed between 1 and 16 and the water pump presents as a switch that you can turn on and turn off.
To recreate:
Deleted Integration
Restarted HA
Put system into 'AUTO' mode
Re-added integration
All expected entities were exposed including water pump switch - however the entity was 'unavailable'
Put system into manual EVAP mode - water pump entity still unavailable
Turn on water pump on physical NC6 controller
Reload integration
Integration states: "Retrying setup: no specified entity for Rinnai Touch Water Pump Switch"
Turn system back to 'AUTO' mode
Reload integration
All entities restored including water pump but it remains 'unavailable'

Water pump entity should only be available in manual evap mode, and it could take up to 10 seconds to become available.
All entities register with the main system for updates, I have not actually tested reloading the integration much, as I usually start HA to get my entity names back clean.
After your last step, back in auto mode, when you then switch to Manual, either on the NC6 or HA, without reload, does the switch become available?

Found one more issue. It seems reloading doesn't affect the behaviour, it still breaks.
Try with https://github.com/funtastix/rinnaitouch/releases/tag/0.9.4

You may need a HA restart for the fix to work.

0.9.4 seems to have fixed it!
When the system is off or is in certain modes irrelevant entities are 'unavailable'. When you change to a different mode the relevant entities become available after a few seconds.

That is correct, only entities relevant to the current system state are available.


pypy{27,38,39}: 7.3.9 -> 7.3.11

Description of changes

See https://github.com/NixOS/nixpkgs/pull/201859.
This will also make pypy27 to be build with pythonInterpreters.pypy27_prebuilt, while pypy39 will be pypy27. Since pythonInterpreters.pypy27_prebuilt is only available for x86_64-linux (edit: and aarch64-linux) right now, this will limit platform support. However, it is better to have fewer platforms working than no platform working (after #201859 is merged).
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Added support for aarch64-linux in pypy27_prebuilt, that in theory should also allow pypy27 to be build for that platform.

According to the changelog there is a few fixed security issue related to the stdlib included in Pypy 3.8+, so tagging it appropriately.

CC @SuperSandro2000.

nixpkgs-review happy, aarch64-linux

In my opinion replacing "abandoned" software with a binary blob is not an improvement.

This is just for bootstrapping purposes though. It will generate a new compiled version build from our own toolchain. It is similar with what we already do with OpenJDK and a few other things (almost sure that even our stdenv is bootstrapped from a binary blob, but I am not 100% sure).

Result of nixpkgs-review pr 209100 run on x86_64-linux 1

1 package blacklisted:

tests.writers

4 packages built:

cvs2svn
pypy (pypy2 ,pypy27)
pypy3 (pypy39)
pypy38

Switched to autoPatchelfHook in pypy*_prebuilt since it reduces the code size and also it detected the missing gdbm dependency.

Added support for {aarch64,x86_64}_darwin. Now we have platform parity with master.
CC @risicle can you test aarch64_darwin?

Result of nixpkgs-review pr 209100 run on x86_64-linux 1

4 packages built:

cvs2svn
pypy
pypy3
pypy38

Result of nixpkgs-review pr 209100 run on x86_64-darwin 1

1 package failed to build:

pypy3

3 packages built:

cvs2svn
pypy
pypy38

Result of nixpkgs-review pr 209100 run on x86_64-darwin 1

4 packages built:

cvs2svn
pypy
pypy3
pypy38

CC @risicle can you test aarch64-darwin?

I can test aarch64-linux (building now) and x86_64 darwin.

This is just for bootstrapping purposes though. It will generate a new compiled version build from our own toolchain. It is similar with what we already do with OpenJDK and a few other things (almost sure that even our stdenv is bootstrapped from a binary blob, but I am not 100% sure).

Tangentially...
I initially looked into how we could extend the sourceProvenance system to be able to handle these cases, but thought I would leave it until interest gathered for it. (For the time being I decided to leave such "bootstrapping" binaries unmarked so that people wanting to run allowNonSource = false systems wouldn't be locked out because of these)
Would welcome further thinking there - I'm sure there's a clever solution lurking.

CC @risicle can you test aarch64-darwin?

I can test aarch64-linux (building now) and x86_64 darwin.

I did test x86_64-darwin already, but double-checking is always good.
Do we know anyone that could test aarch64-darwin? The build seems successfully in CI though, the errors are related to other libraries that the Python tests runs that are already probably broken anyway.

@thiagokokada Same error as in ofborg on aarch64-darwin.

@thiagokokada Same error as in ofborg on aarch64-darwin.

Did it compile at least? The errors seems to be the pypy*.passthru.tests, that I assume are broken anyway.

@thiagokokada I don't think so, since I got the errors almost instantly on nix-build -A pypy39 against master:
$ nix-build -A pypy39
these 12 derivations will be built:
  /nix/store/m8yqb7v3awpaidvm8smf2kklpd5ic1g9-setuptools-44.0.0-sdist.tar.gz.drv
  /nix/store/6cy2y2q6jfkq4ac23y0hh53k2dzs5w3c-pypy2.7-bootstrapped-pip-20.3.4.drv
  /nix/store/2r8ljgy63zdclrrkk6dfsckrjhajirz7-pypy2.7-setuptools-44.0.0.drv
  /nix/store/4jzx3mk4s0hm4fawzw17ks22hbn62kha-python-catch-conflicts-hook.drv
  /nix/store/sqy23d792ndbn9ah4h5mrg7bpckhdd39-setuptools-check-hook.drv
  /nix/store/gb3sb2fhzrmh8s5ni00vdvd2dvzynn3b-pypy2.7-wheel-0.37.1.drv
  /nix/store/w221r27kwci1c5ikw62j4xbwp0rcgsmr-setuptools-setup-hook.drv
  /nix/store/h8n4ix185w8h54aglhbq8npqw64myp2h-pypy2.7-pip-20.3.4.drv
  /nix/store/xa18xmkv753cpkb6j5776wrhs2pbqrsd-pip-install-hook.drv
  /nix/store/850qanmfm72qabafj8g6sly1ajv8h9b4-pypy2.7-pycparser-2.21.drv
  /nix/store/q3lhq96hmh0nc7w5ywmn9qa3pvg700dc-pypy-7.3.11-env.drv
  /nix/store/n5yl6ls3pvsivibcfnjl8vvr1wjh19jr-pypy3.9-7.3.11.drv
building '/nix/store/m8yqb7v3awpaidvm8smf2kklpd5ic1g9-setuptools-44.0.0-sdist.tar.gz.drv'...
unpacking sources
unpacking source archive /nix/store/vp039y542nr4qq62lxm7d3sd4piix1fi-setuptools-44.0.0-source
source root is setuptools-44.0.0-source
patching sources
applying patch /nix/store/0qzs4kq2lvl09k80nf54bl00hd2ylfam-tag-date.patch
patching file setup.cfg
updateAutotoolsGnuConfigScriptsPhase
configuring
no configure script, doing nothing
building
debug: WARNING: Library path not found, using compiled-in sys.path, with
debug: WARNING: sys.prefix = '/private/tmp/nix-build-pypy-7.3.11.drv-0/pypy2.7-v7.3.11-src'
debug: WARNING: Make sure the pypy binary is kept inside its tree of files.
debug: WARNING: It is ok to create a symlink to it from somewhere else.
'import site' failed
debug: OperationError:
debug:  operror-type: ImportError
debug:  operror-value: No module named encodings

While nix-build -A pypy39 --argstr system "x86_64-darwin" seems to build fine.

Yes.

CC @SuperSandro2000.

nixpkgs-review still happy for me, aarch64-linux

Result of nixpkgs-review pr 209100 run on aarch64-darwin 1

8 packages marked as broken and skipped:

cvs2svn
pypy
pypy2
pypy27
pypy3
pypy38
pypy39
tests.writers

Result of nixpkgs-review pr 209100 run on x86_64-darwin 1

4 packages built:

cvs2svn
pypy
pypy3
pypy38

Going to merge after the ofborg tests pass.

Result of nixpkgs-review pr 209100 run on x86_64-linux 1

4 packages built:

cvs2svn
pypy
pypy3
pypy38


cega-mock uses unreasonably heavy container.

We’re using a Python script (cega-mock.py) to set up a server that responds with the users.json file when we hit the /lega/v1/legas/users/:identifier endpoint. The script is self-contained and only needs the Python runtime that’s bundled with the cegauth image.
Expectation:
Should be a minimal container instead.
Additional info:
Probably best to refactor to use Java/Go in the repo instead of python.

Please double check the new expected CEGA endpoint format, the old python code reflects the old format of the endpoint. Updated CEGA_AUTH_URL or similar for live Proxy service connecting to live CEGA Auth service shows the new endpoint format.


Cheat sheet not working

Hi, when following the steps at https://github.com/ruby/ruby.wasm/blob/main/docs/cheat_sheet.md#nodejs, I get the following error:
import { DefaultRubyVM } from "@ruby/wasm-wasi/dist/node";
         ^^^^^^^^^^^^^
SyntaxError: Named export 'DefaultRubyVM' not found. The requested module '@ruby/wasm-wasi/dist/node' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@ruby/wasm-wasi/dist/node';
const { DefaultRubyVM } = pkg;

If I copy<EMAIL_ADDRESS>locally and import it relatively, I don't get that error. However, I then get another error:
node:wasi:51
    validateString(options.version, 'options.version');
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "options.version" property must be of type string. Received undefined
    at new WASI (node:wasi:51:5)
    at DefaultRubyVM (file:///home/blue/projects/primate-ffi/ruby/node.esm.js:1193:18)
    at file:///home/blue/projects/primate-ffi/ruby/index.js:7:22 {
  code: 'ERR_INVALID_ARG_TYPE'
}

I fixed this by changing the WASI instantiation code at line 1193 to
const wasi = new WASI({ env: options.env, version: 'preview1' });
Using Node v21.4.0.

Thank you for your report :) The first one, ESM import issue, will be fixed in the next 2.4.0 release. The sample code was recently written targeting the next release.
The second issue is related to https://github.com/ruby/ruby.wasm/issues/314, but docs should include the version field, thanks.

Thanks.

The second issue is related to #314, but we can just add the field in browser.ts now.

Alright cool, so the solution would be to vendor node.esm.js until that is fixed?

Alright cool, so the solution would be to vendor node.esm.js until that is fixed?

Yes, that would be a temporal solution until we release the next patch release

Thanks!


EAS Build failure: Failed to load app.json file

Describe the bug
Hello,
I saw this failure in my EAS android build:
$ npx bugsnag-eas-build-on-success

Error: Failed to load app.json file /home/expo/workingdir/build/app.json.
Error: Cannot find module '/home/expo/workingdir/build/app.json'
Require stack:
-<EMAIL_ADDRESS>
I believe it's because my Expo app uses app.config.ts instead of app.json.  I noticed the eas-build-on-success.js file only looks for app.json:
let appConfig, apiKey
  try {
    appConfig = require(`${PROJECT_ROOT}/app.json`)
    apiKey = appConfig?.expo?.extra?.bugsnag?.apiKey
  } catch (error) {
    console.error(`Error: Failed to load app.json file ${PROJECT_ROOT}/app.json.\n${error}`)
    exit(1)
  }

It should also look for app.config.ts or app.config.js, the same way that Expo does.
Steps to reproduce

Init a new Expo app
Change app.json to app.config.js
Submit an Android build to Expo EAS

Environment

bugsnag-source-maps version: 2.3.1
Uploading for:

[ ] Browser
[ ] Node

Node version:

[x] React Native

React Native version: 0.71.3

Command that was run:
npx bugsnag-eas-build-on-success

Hi @gburgett thanks for reporting this!
This is actually a bug in the @bugsnag/plugin-expo-eas-sourcemaps package so I've transferred it to this repo.
We have now added a fix for this that will be published in the next release - we'll let you know when that is available.

This has now been released in v48.1.0. Do let us know if you have any issues with this.


Argument Date Faking

This PR adds fake dates to argument comments.
I decided not to modify the CreateAgument action as I feel like the date will always be now except when faking

Could you take a look at the conflict?

@brendt I've resolved the conflicts

Thanks!


Fix/update minio version

Updated minio-go version to v7.0.22

Codecov Report

Merging #261 (72b3c60) into master (35145ad) will increase coverage by 0.09%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #261      +/-   ##
==========================================
+ Coverage   56.13%   56.23%   +0.09%
==========================================
  Files          37       37
  Lines        4122     4131       +9
==========================================
+ Hits         2314     2323       +9
  Misses       1685     1685
  Partials      123      123

Impacted Files
Coverage Δ

pkg/report/handler.go
46.03% <100.00%> (+1.31%)
:arrow_up:

pkg/report/presigned.go
100.00% <100.00%> (ø)

pkg/report/report.go
89.28% <100.00%> (+1.28%)
:arrow_up:

@lucasmdrs, could you please rerun the checks for this pull request? I encountered an error while running the linter, which reported: "E: Version '20211016ubuntu<IP_ADDRESS>' for 'ca-certificates' was not found." Unfortunately, I'm not entirely sure what caused this issue. Could you guide me through troubleshooting this problem so I can address it appropriately? Your assistance would be greatly appreciated.

@shashaneRanasinghe I've fixed the issue separately in #262. but for some reason this PR merge commit is not taking it in consideration :thinking:
Could you update your branch with the latest master version of Kangal (rebase/merge) or maybe push an empty commit to see if it resolves the merge commit issue?

Thank you for the contribution! :sparkle: :sparkles:


How do i use AggressiveSplittingPlugin with HtmlWebpackPlugin, can't auto inject splited chunk into html script tag?

Do you want to request a feature or report a bug?

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.

It looks like you just ignored our issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


SyntaxError in fresh project with 2.8.1

vue init webpack client
cd client
npm i
npm run dev

ERROR in ./~/css-loader?{"minimize":true,"sourceMap":true}!./~/vue-loader/lib/style-compiler?{"id":"data-v-59c95520","scoped":false,"hasInlineConfig":false}!./~/vue-loader/lib/selector.js?type=styles&index=0!./src/App.vue
Module build failed: SyntaxError: Unexpected token {

Edit:  Same with versions 2.8.0 and 2.7.0, something I'm doing wrong?

I thought node was up to date but there's a bug in the windows installer so it was still on 4.4. Uninstalling node, and installing everything again fixed it.


[WIP] feat(SlateAsInputEditor): add keyboard shortcuts - #244

Signed-off-by: lenvi<EMAIL_ADDRESS>Issue #244
Added Keyboard shortcuts to the markdown editor
Changes

Change in SlateAsInputEditor/index.js

Change in onKeyDown method.
Add HandleList & HandleBlockQuote method.

Changes done in a PR in cicero-ui transferred here with support for Mac CMD button

Flags

Link Keyboard shortcut is not implemented, I think we need to refactor the code as the methods for rendering the Link popover resides in FormattingToolbar which is a child of of the Editor where I am providing the KeyDown conditions.
-Refactoring can be moving the renderLinkSetForm() of the link popover to a new component file.

Related Issues

Issue #244

Link Keyboard shortcut is not implemented, I think we need to refactor the code as the methods for rendering the Link popover resides in FormattingToolbar which is a child of of the Editor where I am providing the KeyDown conditions.
Refactoring can be moving the renderLinkSetForm() of the link popover to a new component file.

I think these can be addressed in a future issue/PR


Fix aggregated domain metrics test fixture

The aggregated metrics for the domain test had the host incorrectly set.
Fixed the indentation to be consistent with the other fixtures.

@leklund While implementing the tests for the domain metrics, I found that the domain was marked as an origin (likely copied from the origin metrics). This caused the tests to fail:
 - 	`testspace_domain_bereq_body_bytes_total{datacenter="aggregate",domain="testhost",service_id="my-service-id",service_name="my-service-name"}`:                         16,
 + 	`testspace_domain_bereq_body_bytes_total{datacenter="aggregate",domain="testorigin",service_id="my-service-id",service_name="my-service-name"}`:                       16,

Since this is unrelated to the aggregated metrics, felt it was better to put this up as a separate pull request.


BaseURL into custom fields

Also fill other fields
"CustomFields": {
    "CatalogAssetName": "AmplifyLUTPack413",
    "BuildLabel": "Live",
    "CompatibleApps": "UE_4.13,UE_4.14,UE_4.15,UE_4.16,UE_4.17,UE_4.18,UE_4.19,UE_4.20,UE_4.21,UE_4.22,UE_4.23,UE_4.24,UE_4.25,UE_4.26",
    "Categories": "assets/fx,assets",
    "Vault.ThumbnailUrl": "https://cdn1.epicgames.com/ue/item/Store_AmplifyLUTPack_Thumb-284x284-750fe4cb4fa9691ca138c9b932426ee3.png",
    "FullAppName": "AmplifyLUTPack413: Live",
    "CatalogItemId": "e44ac2b29b774233af36eb1447f2a577",
    "Vault.TitleText": "Amplify LUT Pack",
    "InstallLocation": "C:/users/<USER>/My Documents/Unreal Projects/AmplifyLUT",
    "IsExecutable": "false",
    "TechnicalType": "",
    "BaseUrl": "<ALL BASE URLS>",
    "CatalogNamespace": "ue"
  }

Provided what can be provided into the custom fields, moving the issue up to the client to provide more details https://github.com/AchetaGames/Epic-Asset-Manager/issues/50


adding a max_fails parameter to upstream member[s]

As defined in nginx upstream documentation: http://nginx.org/en/docs/http/ngx_http_upstream_module.html#upstream
I added this optional parameter because we folks have to deal with a finicky backend that randomly timeout on a single request  when under heavy load. When this happens, the upstream gets disabled for 10 seconds (fail_timeout) and for that period of time other upstreams receive the requests which doesn't play nice with our application.
Increasing this parameter to max_fails=3 ensures that load is switched only when really necessary.
  nginx::resource::upstream { $upstream_name:
    ensure               => present,
    members              => $upstream_servers,
    upstream_cfg_prepend => {
      'ip_hash'   => '',
      'keepalive' => '128',
    },
    upstream_max_fails   => 3,
  }

becomes s:
upstream backend {

  ip_hash ;

  keepalive 65;

  server     server1.domain  fail_timeout=10s max_fails=3;
  server     server2.domain  fail_timeout=10s max_fails=3;
}

Let me know if I should change anything, thanks

Great! Thanks for the code!


Add vagrant runner

As a development team, we're adopting nocode as goto language for distributed cloud-aware micro-services. To support this, we deliver vagrant based noVMs. Please add a nocode provider so Vagrant will be a First Class Citizen.

Stop trying to make Vagrant happen.


add nhgis interface

nhgis has a new api for extracting data from their system. As far as i can tell, it's currently asynchronous like their web extracts because it takes awhile for their backend to process the request

IPUMS Metadata and Data Extract APIs, allow you to submit a JSON request to describe the data you want, monitor our progress building your data extract, and download your data when it is ready.

this likely wont be ready for some time, but as the cenpy wrapper gets up to speed, it will be easy to include this alongside the pre-harmonized LTDB and NCDB sources.
One thing to mention is that our experience with NHGIS boundary files in the past has revealed tons of topological problems that result in invalid weights matrices.

https://developer.ipums.org/docs/workflows/create_extracts/nhgis_data/

the async API they provide doesnt fit well into the workflow, so closing this. If folks want to use NHGIS data with geosnap, its better to collect the data manually ahead of time


Cannot open include file: 'opencv2/opencv.hpp': No such file or directory

I'm using Windows 10 Anniversary and OpenCV was successfully installed with .\vcpkg install opencv.
I got the above error although everything seems to work well until I run build solution on Visual Studio Community 2017.
main.cpp created as:
#include <opencv/cv.hpp>

int main()
{
	cv::Mat im = cv::Mat::zeros(cv::Size(640, 480), CV_8UC3);
	cv::imshow("test", im);
	cv::waitKey(0);
}

CMakeLists.txt is:
cmake_minimum_required(VERSION 3.0)
project(test)

find_package(opencv REQUIRED)

add_executable(main main.cpp)

target_link_libraries(main opencv)

CMakeSettings.json is:
{
  // See https://go.microsoft.com//fwlink//?linkid=834763 for more information about this file.
  "configurations": [
    {
      "name": "x86-Debug",
      "generator": "Ninja",
      "configurationType": "Debug",
      "inheritEnvironments": [ "msvc_x86" ],
      "buildRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\build\\${name}",
      "installRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\install\\${name}",
      "cmakeCommandArgs": "",
      "buildCommandArgs": "-v",
      "ctestCommandArgs": "",
      "variables": [
        {
          "name": "CMAKE_TOOLCHAIN_FILE",
          "value": "C:\\users\\ngu10t\\workspace\\vcpkg\\scripts\\buildsystems\\vcpkg.cmake"
        }
      ]
    },
    {
      "name": "x86-Release",
      "generator": "Ninja",
      "configurationType": "RelWithDebInfo",
      "inheritEnvironments": [ "msvc_x86" ],
      "buildRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\build\\${name}",
      "installRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\install\\${name}",
      "cmakeCommandArgs": "",
      "buildCommandArgs": "-v",
      "ctestCommandArgs": "",
      "variables": [
        {
          "name": "CMAKE_TOOLCHAIN_FILE",
          "value": "C:\\users\\ngu10t\\workspace\\vcpkg\\scripts\\buildsystems\\vcpkg.cmake"
        }
      ]
    },
    {
      "name": "x64-Debug",
      "generator": "Ninja",
      "configurationType": "Debug",
      "inheritEnvironments": [ "msvc_x64" ],
      "buildRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\build\\${name}",
      "installRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\install\\${name}",
      "cmakeCommandArgs": "",
      "buildCommandArgs": "-v",
      "ctestCommandArgs": "",
      "variables": [
        {
          "name": "CMAKE_TOOLCHAIN_FILE",
          "value": "C:\\users\\ngu10t\\workspace\\vcpkg\\scripts\\buildsystems\\vcpkg.cmake"
        }
      ]
    },
    {
      "name": "x64-Release",
      "generator": "Ninja",
      "configurationType": "RelWithDebInfo",
      "inheritEnvironments": [ "msvc_x64" ],
      "buildRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\build\\${name}",
      "installRoot": "${env.USERPROFILE}\\CMakeBuilds\\${workspaceHash}\\install\\${name}",
      "cmakeCommandArgs": "",
      "buildCommandArgs": "-v",
      "ctestCommandArgs": "",
      "variables": [
        {
          "name": "CMAKE_TOOLCHAIN_FILE",
          "value": "C:\\users\\ngu10t\\workspace\\vcpkg\\scripts\\buildsystems\\vcpkg.cmake"
        }
      ]
    }
  ]
}

Output when generating cache of CMakeLists.txt is:
1> Command line: C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2017\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe  -G "Ninja" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\ngu10t\CMakeBuilds\f73f3503-0fe4-d130-834f-aa25c0b54198\install\x86-Debug"  -DCMAKE_TOOLCHAIN_FILE="C:/users/ngu10t/workspace/vcpkg/scripts/buildsystems/vcpkg.cmake"  -DCMAKE_BUILD_TYPE="Debug" -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2017\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" "C:\Users\ngu10t\Workspace\Laava_tag"
1> Working directory: C:\Users\ngu10t\CMakeBuilds\f73f3503-0fe4-d130-834f-aa25c0b54198\build\x86-Debug
1> -- The C compiler identification is MSVC 19.11.25547.0
1> -- The CXX compiler identification is MSVC 19.11.25547.0
1> -- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x86/cl.exe
1> -- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x86/cl.exe -- works
1> -- Detecting C compiler ABI info
1> -- Detecting C compiler ABI info - done
1> -- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x86/cl.exe
1> -- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/Hostx86/x86/cl.exe -- works
1> -- Detecting CXX compiler ABI info
1> -- Detecting CXX compiler ABI info - done
1> -- Detecting CXX compile features
1> -- Detecting CXX compile features - done
1> -- OpenCV ARCH: x86
1> -- OpenCV RUNTIME: vc15
1> -- OpenCV STATIC: ON
1> -- Found OpenCV: C:/Users/ngu10t/Workspace/vcpkg/installed/x86-windows (found version "3.3.1")
1> -- Found OpenCV 3.3.1 in C:/Users/ngu10t/Workspace/vcpkg/installed/x86-windows/share/opencv/x86/vc15/lib
1> -- You might need to add C:\Users\ngu10t\Workspace\vcpkg\installed\x86-windows\share\opencv\x86\vc15\bin to your PATH to be able to run your applications.
1> -- Configuring done
1> -- Generating done
1> -- Build files have been written to: C:/Users/ngu10t/CMakeBuilds/f73f3503-0fe4-d130-834f-aa25c0b54198/build/x86-Debug
1> Starting CMake target info extraction ...
1> CMake server connection made.
1> Extracted includes paths.
1> Extracted CMake variables.
1> Extracted source files and headers.
1> Extracted global settings.
1> Extracted code model.
1> Collating data ...
1> Target info extraction done.

However, the above error shows up when I try to compile the solution via Build menu.

It should be
find_package(OpenCV REQUIRED)

add_executable(main main.cpp)

target_link_libraries(main ${OpenCV_LIBS})

Hi @chuong , thanks for reporting this issue.
I have tested the cmake configuration of opencv, they are correct in the latest version. Plz update vcpkg and rebuild it, then you may find notice as:

The package opencv provides CMake integration:

find_package(OpenCV REQUIRED)
target_include_directories(main PRIVATE ${OpenCV_INCLUDE_DIRS})
target_link_libraries(main PRIVATE ${OpenCV_LIBS})

Thx.


⚠ Bofrost.lu has degraded performance

In a0aad6b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1573 ms

Resolved: Bofrost.lu performance has improved in 43ed822 after 7 minutes.


Fix/xemm rate oracle

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Related to issue:
XEMM - Incorrect rate oracle conversion#5695
The strategy seemed to coerce a MakerTakerPair into a MarketTradingPair - This is just a proposal for review, since no expertise on XEMM
Tests performed by the developer:
Only live test with Kucoin/Kucoin. Screenshot shows rates after the rate oracle is connected:

Tips for QA testing:

Upon checking still report warning message can't find oracle conversion rate for AVAX-USDT, using 1.0 instead

Steps to reproduce:

clone and installed feature branch
create/start a strategy using kucoin and pangolin_avalanche_avalanche

Actual:
Bot Showed warning message can't find oracle conversion rate for AVAX-USDT, using 1.0 instead

Is this not normal? the Rate Oracle has not started yet, it shows on the next line where it finally is CONNECTED

Hey @MementoRC Any updates? :eyes:

@nikspz, @kadirc seems to have an alternate solution to mitigate waiting for the oracle to be online. He also implemented waiting for the oracle byt changing the RateOracle, which I would prefer not to do. I don't think he has made a PR yet, I think it would be good to also look at his solution, maybe @cardosofede - I am also not a user of the XEMM, so I can't say which solution is preferable for HB


[Assets] Update RoslynPad and Microsoft.CodeAnalysis

PR Details
Referencing latest Microsoft.CodeAnalysis 3.4.0 and the latest state of RoslynPad.
Description
Basically only nuget and dependency updates.
RoslynHost had to be updated to implement that latest interface changes of the RoslynPad library. For that, I've copied their reference implementation from their repository to implement the missing interface implementations.
I've tested scripting in GameStudio and everything worked as expected. Code completion and help text also was properly displayed.
Motivation and Context
Avoid version conflicts on build with other applications that use latest Microsoft.CodeAnalysis.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Thanks!


added api name to config json file and added test for URL resolver

All Submissions:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you read and signed the automated Contributor's License Agreement?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[ ] Does your submission pass tests?
[ ] Have you lint your code locally prior to submission?

Changes to Core Features:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?

Closing this PR since changes related to this was merged into master from PR #129


Slot Event changing to Sync Event and beacon node never syncs

What is a Slot Event?
What is a Sync Event?
Why is there a transition?
The following is from running beacon node only (not validator client, which is #2215.  Because of 2215, just running beacon node itself to sync the chain. But it is unable to sync and is always behind).
01:25:04.561 INFO  - Slot Event  *** Slot: 79238, Block:    ... empty, Epoch: 2476, Finalized checkpoint: 2465, Finalized root: a664f9..0b74, Peers: 17
01:25:16.521 INFO  - Slot Event  *** Slot: 79239, Block:    ... empty, Epoch: 2476, Finalized checkpoint: 2465, Finalized root: a664f9..0b74, Peers: 17
01:25:28.647 INFO  - Slot Event  *** Slot: 79240, Block:    ... empty, Epoch: 2476, Finalized checkpoint: 2465, Finalized root: a664f9..0b74, Peers: 17
01:25:40.837 INFO  - Slot Event  *** Slot: 79241, Block:    ... empty, Epoch: 2476, Finalized checkpoint: 2465, Finalized root: a664f9..0b74, Peers: 17
01:25:52.525 INFO  - Slot Event  *** Slot: 79242, Block:    ... empty, Epoch: 2476, Finalized checkpoint: 2465, Finalized root: a664f9..0b74, Peers: 17
01:26:04.073 INFO  - Sync Event  *** Current slot: 79243, Head slot: 78947, Connected peers: 18
01:26:18.918 INFO  - Sync Event  *** Current slot: 79244, Head slot: 78947, Connected peers: 18
01:26:20.611 ERROR - PLEASE FIX OR REPORT | Unexpected exception thrown for StorageUpdateChannel-0 java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: ProtoArray: Invalid delta length (See log file for full stack trace)
01:26:55.042 INFO  - Sync Event  *** Current slot: 79245, Head slot: 78947, Connected peers: 18
01:26:56.110 ERROR - PLEASE FIX OR REPORT | Unexpected exception thrown for StorageUpdateChannel-0 java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: ProtoArray: Invalid delta length (See log file for full stack trace)
01:27:33.546 INFO  - Sync Event  *** Current slot: 79246, Head slot: 78947, Connected peers: 18
The beacon node is always behind:
01:30:51.381 INFO  - Sync Event  *** Current slot: 79252, Head slot: 79053, Connected peers: 18
01:31:22.481 INFO  - Sync Event  *** Current slot: 79253, Head slot: 79065, Connected peers: 18
01:31:52.320 INFO  - Sync Event  *** Current slot: 79254, Head slot: 79078, Connected peers: 18
01:31:53.769 ERROR - PLEASE FIX OR REPORT | Unexpected exception thrown for StorageUpdateChannel-0 java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: ProtoArray: Invalid delta length (See log file for full stack trace)
01:32:44.345 INFO  - Sync Event  *** Current slot: 79255, Head slot: 79078, Connected peers: 18
01:35:19.230 INFO  - Sync Event  *** Current slot: 79256, Head slot: 79105, Connected peers: 18
01:35:26.707 ERROR - PLEASE FIX OR REPORT | Unexpected exception thrown for StorageUpdateChannel-0 java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: ProtoArray: Invalid delta length (See log file for full stack trace)
01:37:39.461 INFO  - Sync Event  *** Current slot: 79257, Head slot: 79105, Connected peers: 18

We've reverted the PRs that caused this issue so we shouldn't see this issue again.


No way to generate empty associative maps

[] is an empty array, but not an empty associative map (aka JavaScript object).

Added the [=] syntax for this, but opening an issue on deciding the overall syntax goals.


Re-Enabling Messaging Pod Linting in Travis

This was disabled in #1277.
It should be re-enabled once Core releases, or when we have support on Travis for local pod dependencies.

Closed by #1352


Add tests to FieldDiagnostics

Pull Request
Description
Adding tests for the FieldDiagnostic class in diagnostics.py
This pull request addresses #96

[x] init

[x] check_step

[x] do_diagnostic

[x] inspect_resource

[x] print_diagnose

[x] initialize

[x] csv diagnose

[x] finalize

[x] make sure you are pulling from a dev branch on your fork

[x] check for code linting

What is the field attribute supposed to do and how am I supposed to use it in the do_diagnostic method?

It says the TravisCI build failed. How do I fix that?

If you click on the "Details" link above, it will take you to TravisCI, and show you what the error was. I think one of your tests failed.

Codecov Report

Merging #108 into main will increase coverage by 6.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
+ Coverage   65.14%   71.37%   +6.23%
==========================================
  Files           6        6
  Lines         525      531       +6
==========================================
+ Hits          342      379      +37
+ Misses        183      152      -31

Impacted Files
Coverage Δ

turbopy/core.py
93.77% <0.00%> (-1.12%)
:arrow_down:

turbopy/computetools.py
21.34% <0.00%> (+1.82%)
:arrow_up:

turbopy/diagnostics.py
94.73% <0.00%> (+27.19%)
:arrow_up:

Continue to review full report at Codecov.

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

There. Sorry that took so long; I had to leave my house again. (Trust me, it didn't take me an hour to copy and paste four lines of code)

Thanks for writing these tests!


Remove stale bot to allow address issues in a timely manner

Expected Behavior
Issues opened by the community are addressed and are not automatically closed by the stale bot
Current Behavior
Due to amount of issues being opened, there's a number of bug tickets that are being not addressed because they're closed by the stale bot
A list of the bugs that have been closed by the stale bot can be found here
Possible Solution
The stale bot is disabled, and bugs are manually inspected and addressed.

CC @Golodhros

sounds good


[Presence] Prevent premature unmount

Raising for discussion.
Presence has trouble getting the latest animation value if they are updated via state inside an effect, it will retrieve a stale animationName when present changes and fall through to the UNMOUNT.
Here is an example, you can see the unmount animation doesn't occur:
https://codesandbox.io/s/exciting-rgb-j04y0
This change pushes the animation retrieval logic to next tick so that the most recent animation value can be checked.

Random sense check question, I'm sure the example is contrived, but generally our data-state should be computed from state? So I'm curious why we'd need a separate state that gets synced in an effect.

I'm curious why we'd need a separate state that gets synced in an effect

I had a use case where selectively setting the state was easier to reason about compared to deriving from state directly in the render.
I thought about it some more though and I don't think this change is a good idea, even my case is better served by an alternative.


"comment" semantic token type is overly broad vs the syntax types that it overrides when themeing

Issue Type: Bug
There are multiple TextMate scopes for comments:
Rust:
comment.line.double-slash.rust
comment.line.documentation.rust
Java:
comment.line.double-slash.java
comment.block.javadoc.java
However, the semantic tokens only allow for comment, which breaks the ability to theme doc comments (comment.block.javadoc.java, comment.line.documentation.rust) separately from standard code comments.
When semantic highlighting is enabled, all comments become comment to the theme.
  It doesn't seem to be in use for any examples, and the documentation I've found for it says "Style to use for references in documentation"), which implies that it's a modifier that would be applied to a token types such as struct or enum which are found within documentation comments, not to comment lines or blocks as a whole.
If the intention is for the latter, I'll close this and open a bug against the LSP I'm using (rust-analyzer) to map /// comments to comment.documentation semantic type+modifier.

Before adding a new type to the standard I'd suggest you to pioneer this for Rust first.
It should be done with a new token type that extends the comment token type.
{
  "contributes": {
    "semanticTokenTypes": [
      {
        "id": "docComment",
        "superType": "comment",
        "description": "A doc comment."
      }
    ]
}

Then for themes to support this, add a semanticTokenScopes contribution, with a rust specific rule (unless TextMate has a well established scope for doc comments):
{
  "contributes": {
    "semanticTokenScopes": [
      {
        "language": "rust",
        "scopes": {
          "docComment": ["comment.line.documentation.rust"]
        }
      }
    ]
  }
}

I created https://github.com/microsoft/vscode/issues/97063 to collect proposals.


The static assertion failed mistake!

Hi, when I compile the project, the static assertion mistake happened!

The gcc version in my computer is 8.3.0.
Could you please help me solve this problem?
Thanks!

Hi @sysuzyc ,
I couldn't reproduce the error with gcc-8 (Ubuntu 8.3.0-6ubuntu1~18.04.1). Can you provide more information so that I can produce the error?

Hi, I have solved this problem. This problem just happened in Ubuntu 16.04. I have updated the Ubuntu version to 18.04. This problem didn't reproduce too.
Thanks for your help!


ADD the API to the GAME

API to be added to the APP model instead of the placeholder pictures

Done!


EJS template support

I'm using Sourcery to build a Swift to Javascript binding library. Due to the potential complexity of my templates, I'd rather use EJS or Swift templates than Stencil templates so I can incorporate more complex logic.
As you have stated multiple times in the past that you do not recommend us to use Swift templates (because they are a proof of concept), I think the logical choice for me would be to use EJS templates.
What's your opinion on this? Is it safe to use EJS templates (do you recommend it), or is there a chance that they will be removed again in the future?
By the way, could you maybe elaborate on why exactly you don't recommend to use Swift templates at the moment?

Hey @Obbut
I would say definitely go for it and please do file issues if you discover some, I'll try to assist asap with EJS. Basically the only thing that can go wrong that I can think of is some properties of Sourcery types not being bridged to JavaScript, and this should be easy to fix.
Regarding Swift templates the only thing, at least for me, is that they are not battle tested so there might be issues. But again it would be cool if you start using them and file issues. PRs would be even better =)
Stencil templates proved they are the most reliable as they are used in several projects. But also they are restricted in functionality.

Thanks for the quick response!
I would be happy to help, not only by filing issues but also with PRs when I have the time. I think I'm going the EJS direction for now.


Program quits during pre-translation process

Hi,
PoEdit quits without any reason during pre-translation process. Sadly i can't reproduce the bug at any time because it ends without any message or something and not at the same location.

Please read http://www.chiark.greenend.org.uk/~sgtatham/bugs.html (as per the instructions for creating new issues) and provide some meaningful description, including — but not limited to! — such absolute basics as your platform, poedit version, your pre-pranslation settings, the file in question etc., as well as a description of what exactly you mean by "it quits".
(P.S. The name's Poedit, no "PoEdit" or any other random spelling.)


[test-only] add testing docs to ocm test

[x] add docs how to run ocm test localy
[x] add docs how to run second ocis instance use launch.json and using .env

now we have two instance localy and in the CI and one .env-federation file

https://federation-ocis-server:10200
https://ocis-server:9200

@ScharfViktor I think we need to add tests for sending invitations using email.

@ScharfViktor I think we need to add tests for sending invitations using email.

Of course, I or we will add more tests in the next Prs


List of downstream projects

We should probably have a list of downstream projects, to more thoroughly test nom's code. It's complex enough that test cases alone won't find the many places subtle bugs can hide.
When we have a PR that changes old code, we can double-check our testing by running that nom with the downstream projects, and ensuring their tests still succeed.

This is something I already do before releases, but thank you for the suggestion


Navigation URLs aren't generated correctly when using Passenger's RackBaseURI

I have the Rails 3 application containing active_admin mounted as a suburi (i.e. http://domain.com/sub-uri/).
The links in ActiveAdmin below the navigation header work properly (http://domain.com/sub-uri/admin/path).
But the links in the header navigation section don't include the suburi.  They look like http://domain.com/admin/path.

I'm also still getting this.

Tested on 0.3.3, same issue occurring.
Have an app that is deployed in a sub-ur (http://example.com/app1/admin/) via mod_passenger's RailsBaseURI.
Top-most menu with links to dashboard, models, etc generates links without proper sub-uri
E.g. dashboard point to http://example.com/admin/dashboard instead of http://example.com/app1/admin/dashboard

I meet the same issue as 0.3.4 is used. The result displayed in #header is really bad. How can we fix it?

Also having this issue

me too
as 0.3.4

+1
same issue but we're using nginx + unicorn for production.

+2 (I have two apps) with nginx + passenger

Thank you, sir!

Encountered this problem with AA 0.6.0 and Rails 3.2.
This fix:
https://github.com/activeadmin/activeadmin/issues/101#issuecomment-22273869
correctly resolved the issue fo rme.

@armahillo 's solution worked for me.  Actually, all I needed was the 1st part--didn't need to use SetEnv RAILS_RELATIVE_URL_ROOT=/foo

The only solution that worked for me with Rails 4.2 and AA from github is to manually set the menu urls like mentioned here https://github.com/activeadmin/activeadmin/issues/101#issuecomment-22273050

yeah, not the best solution but also fixed the issue for me

my solution:

config/environment.rb

Rails.application.routes.default_url_options[:script_name] = ENV['RAILS_RELATIVE_URL_ROOT']

app/controller/application_controller.rb

protected
	def after_sign_in_path_for(resource)
		ENV['RAILS_RELATIVE_URL_ROOT']
	end

	def after_sign_out_path_for(resource)
		new_user_session_path
	end

	def authenticate_user!
		if user_signed_in?
			super
		else
			redirect_to new_user_session_path
		end
	end

app mounted in sub-uri "/record"

nginx.conf

server {
        passenger_user web;
        passenger_group web;
        listen       80;
        server_name  localhost;
        client_max_body_size 20M;
        root   /home/web/ror_apps/share/static;
        passenger_enabled on;
        passenger_base_uri /record;
        rails_spawn_method smart;
        rails_env production;

        location ~ ^/(i|js|css)/ {
            expires 1d;
        }

        error_page   500 502 503 504  /50x.html;
        location = /50x.html {
            root   html;
        }

    }

@semen225 2 suggestions:
In authenticate_user!, right before the redirect, add this:
store_location_for(:user, request.fullpath) if request.get? && !request.xhr?

Also, after_sign_out_path_for could be more generic, so you don't have to change it if you ever add another devise model:
send("new_#{resource_or_scope}_session_path")

I hope this helps someone in the future because I've been struggling with this for a bit. Especially since things like AdminComments have no associated configuration in app/admin where I can specify the url for the menu item. Moreover, I really don't feel like hardcoding the app's path into the app itself, since that is no concern of the app - I should be able to move it to a different subdirectory without rewriting my code, I think.

@timoschilling Recent work and documentation seem to indicate that relative_url_rootis no longer considered deprecated.

+1


Failure to install caused by plugin default name

Plugin failed to install if "name" field in a4t.xml file was not set to appropriate value, in our specific case "MetaDataSourceTab".

Added name check at compile time so if name is not valid error message will display in "Errors" and link to a4t.xml file to correct.  .a4t plugin file will not be created until valid name is entered.


Difference FPN config between mmdet and Retinanet paper

Hi, when I see the config/base/models/retinanet_r50_fpn.py, I notice it uses 'L1 loss' in loss_bbox and its FPN is as follows:
neck=dict(
type='FPN',
in_channels=[256, 512, 1024, 2048],
out_channels=256,
start_level=1,
add_extra_convs='on_input',
num_outs=5),
But when I read the original Retinanet Paper 'Focal loss for Dense Object Detection', I notice the network uses FPN from level P3 to P7 and also it uses 'smoothL1 loss' in loss_bbox. If I want to use pyramid level P3 to P7, I wonder  i can just replace the start_level in the above FPN config to 3 and nothing else change?
Thanks.

I believe the original RetinaNet uses 5 feature maps from P2 to P6, which is consistent with our 4 in_channels plus the add_extra_convs. Please read this issue if you still have questions.


Move restore repo to internal router and invoke from command to avoid open the same db file or queues files

When execute the restore repo command and gitea is running, you may got errors to open queue files or sqlite database because the files have been locked.
This PR moves repo restore main logic to internal routers and send a http request to execute the command.
This also fixes the bug that units has been ignored from restore repository.

OK. I think we can run it in another goroute and return immediately.


Refactor/mdx content

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (36e8836) 79.40% compared to head (d0d38ca) 79.40%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #700   +/-   ##
=======================================
  Coverage   79.40%   79.40%
=======================================
  Files          31       31
  Lines         369      369
  Branches       60       60
=======================================
  Hits          293      293
  Misses         70       70
  Partials        6        6

Impacted Files
Coverage Δ

src/components/blog/layout.tsx
100.00% <ø> (ø)

src/components/project/layout.tsx
85.71% <ø> (ø)

src/components/Header.tsx
100.00% <100.00%> (ø)

src/components/Notifications.tsx
58.49% <100.00%> (ø)

src/components/blog/card.tsx
100.00% <100.00%> (ø)

src/components/blog/content.tsx
100.00% <100.00%> (ø)

src/components/project/card.tsx
100.00% <100.00%> (ø)

src/components/project/content.tsx
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Table is not working after upgrading to the new version

Describe the issue
Issue
Table has stopped working after upgrading to the new version and its generating the following error in the console.
replace monitor event exception！ RangeError: Selection points outside of document
    at checkSelection (index.js:1519:1)
    at new Transaction (index.js:2214:1)
    at Transaction.create (index.js:2221:1)
    at resolveTransaction (index.js:2385:1)
    at EditorState.update (index.js:2542:1)
    at EditorView.dispatch (index.js:7108:1)
    at Mr.replaceSelectedText (Editor.mjs:2354:1)
    at d (Editor.mjs:3052:1)
    at index.mjs:36:1
    at Array.forEach (<anonymous>)

Procedure version
4.13.5
Reproduction link
No response

Try reinstalling after uninstallation. If it still doesn't work, please provide an environment for restoring the issue

Try reinstalling after uninstallation. If it still doesn't work, please provide an environment for restoring the issue

Its working after un-install and reinstall.
As mentioned in https://imzbf.github.io/md-editor-rt/en-US/docs#📅 tableShape, the table size has been capped at [6,4]. Can we grow it dynamically based on the move move when the table is open ??

Not right now, I'll try it out

Not right now, I'll try it out

Okay. Do you want me to open a new feature request for the same or you want to continue with this issue id ?

try the beta version
npm i md-editor-rt@beta

docs: go to https://github.com/imzbf/md-editor-rt/tree/feature/4.14.0 and search for tableShape

try the beta version
npm i md-editor-rt@beta

docs: go to https://github.com/imzbf/md-editor-rt/tree/feature/4.14.0 and search for tableShape

Hi @imzbf,
I have tested and it working fine. You can GA this code.

4.14.0 Released


AttributeError: 'params' object has no attribute 'params'

I am trying the the code below
# set parameters
params = params.params()
params.rmpc = rmpc
# get SIF embedding
embedding = SIF_embedding.SIF_embedding(We, x, w, params)

an get this error
AttributeError: 'params' object has no attribute 'params'

and I verified the module params and its true, please help me how to deal with this?

Me too, can you address this problem now?


No module named 'ffmpeg'

Long time user of VSMP and recently I decided to do a clean install after having to get a new Waveshare screen (my ribbon tore on my previous screen).
After following the automated install I'm thrown this error:
Traceback (most recent call last):
File "/home/cozboogie/SlowMovie/slowmovie.py", line 20, in
import ffmpeg
ModuleNotFoundError: No module named 'ffmpeg'
did some searching and googling and attempt proposed fixes....but zero luck.
thoughts?
Thanks and cheers

Can you give a little more information on how you installed and ran the script? For ffmpeg specifically there is both an OS package and a Python package that need to be installed for it to work. The script is also configured to run under a Python virtual environment as well. Without that activated it wouldn't find the installed libraries.

Hey Rob,
Installed from the homepage and followed all the prompts.
Ran this as normal:
python3 slowmovie.py
I'll look at installing the OS and python package  to see if that remedies the error.
Thanks

If you are installing from the instructions did you activate the virtual environment? Running python3 outside of the virtual environment won't find the packages. Did pip3 freeze list the required packages?


Requirements.txt should list setuptools

I tried to install this package using PIP but failed because I lacked setuptools. It was an easy fix, but it should be listed as a dependency in requirements.txt. Thanks in advance.

setuptools is no longer required as it is now using poetry :-)


Question on null annotations in COCO

Hi,
My custom dataset has null annotations, images without bboxes are represented in the "images" tag, and all the bboxes are in the "annotations" tag.
Now, when I load using the coco parser, it is removing all of these null annotations.
This might be more of a philosophical question, but what do I do if I want to keep my "negative samples" while training. Because, in my case, the background also can change without any "objects" to detect, and I want my model to know that.
Cheers!

That is a hard question indeed hahaha
At least the models we currently support mask/faster rcnn and efficientdet cannot handle null annotations. I would actually be interested in a model that could do so (if there are any).
It's hard because there is really nothing to learn on those images, what you can do is distribute some random bboxes onto your image and mark those as background (label 0), but I'm not sure that is going to help much
@oke-aditya any thoughts?

Great Question. Let me answer it in detail as this doubt can occur to most people who try object detection with any package or repo for that matter.
Problem: -  You have no objects to detect, it is an example of having no classes that you have to detect. E.g. you are training a face detector and you pass an image of say tiger/lion. You do not want to draw any bounding box in this image.
Object-detection-theory: -
We never calculate metrics on false negatives. As there is no notion of True negatives in object detection. E.g. You draw a bounding box where there is nothing and say it as negative i.e. there is no object. Now there can such multiple boxes possible in every image. We cannot calculate such boxes. These are simple True negatives. You cannot evaluate your model on these objects or try to train on these bounding boxes as they make no sense.
Refer here and here
Problem of Null annotations : -
But you want to still try and train a face detector by showing it images of lion/tiger and saying it is NULL. This is called Negative sampling. Yes you can do that. i.e. you do not contain any positive samples but want the model to ensure that it does not confuse these type of samples. Torchvision started supporting these from release 0.6. It is fairly simple to support too.
Have a look here
Just reposting the highlight of Faster RCNN
It is now possible to feed training images to Faster / Mask / Keypoint R-CNN that do not contain any positive annotations.
This enables increasing the number of negative samples during training. For those images, the annotations expect a tensor with 0 in the number of objects dimension, as follows:

target = {"boxes": torch.zeros((0, 4), dtype=torch.float32),
          "labels": torch.zeros(0, dtype=torch.int64),
          "image_id": 4,
          "area": torch.zeros(0, dtype=torch.float32),
          "masks": torch.zeros((0, image_height, image_width), dtype=torch.uint8),
          "keypoints": torch.zeros((17, 0, 3), dtype=torch.float32),
          "iscrowd": torch.zeros((0,), dtype=torch.int64)}

How does this solve the Problem: -
What it does is simply makes Bounding box and mask to (0,0,0,0) that is nowhere in the image.
The label is also 0. Which is background class for Faster RCNN / Mask RCNN. And makes iscrowd to 0 too.
This effectively tells the detector. "Look there is a bounding box at 0,0,0,0 with the label as background".
Since we ignore the background class while you are deploying/detecting in test time. (Why would you detect background in test time?) This makes such negative samples too become background.
Solution: -
Simply pass these negative sampled images as background class (it is 0 for F-RCNN may be different for different models), and make the bounding box, areas, masks, crowds as 0,0,0,0. This would work for most models, e.g. FRCNN (any backbone or FPN) Efficient-Det, Retinanet (soon). Just take care of this preprocessing while parsing and you are done. Background class may wary from model to model so just be careful for that (we can't take care of this as it is left to the researcher to handle background model, we will simply throw an error)
Mantisshrimp support: -
For mantisshrimp since we support F-RCNN from torchvision. Yes, we support this feature too natively. Just preprocess data as mentioned above.
For efficient-det @lgvaz can you have a check from dataset code here It does make bboxes 0 from line 98-102. Please have a look and do a post here.

Thanks for the awesome answer, @oke-aditya!


use sentence-transformers in the JS client

Describe the problem
i want to embed data easily in JS
Describe the proposed solution
use ONNX inside JS to embed things
Alternatives considered
would be slow, but would create a simpler getting started experience
Importance
nice to have
Additional Information
No response

this is now possible with inclusion of Transformers.js
https://github.com/chroma-core/chroma/blob/main/clients/js/src/embeddings/TransformersEmbeddingFunction.ts#L18

@fool1280 that'd be great! the context here is that in python you can do
client.create_collection("test") and then add documents and query and it JustWorks ™ .
In JS - you have to explicitly set the embedding model
const {OpenAIEmbeddingFunction} = require('chromadb');
const embedder = new OpenAIEmbeddingFunction({openai_api_key: "your_api_key"})
const collection = await client.createCollection({name: "test", embeddingFunction: embedder})

It would be better if we had a default in JS for easier first time user experience. You can look at the python implementation to gather some inspiration for the logic
https://github.com/chroma-core/chroma/blob/main/chromadb/api/models/Collection.py#L50
Let me know if this is up your ally and I can assign it to you.
and thanks!

yep, I'd love to take it!

@fool1280 super! you are assigned. Let's make sure to add good tests for this. Let me know if you have any issues at all getting up and running.
One thing to watch out for - if you run chroma from docker-compose.yml you need to update the env to get the tests to run.
environment:
      - IS_PERSISTENT=TRUE
      - ALLOW_RESET=True

You can run tests inside clients/js with yarn test:run

thank you Jeff!

@fool1280 did you want to take a stab at this? else i can put it back in the pool

Hi Jeff, I still want to work on this. So sorry for the delay!

@fool1280 great!

this is being worked on here https://github.com/chroma-core/chroma/pull/1382

This is being released today!


Update bl3_vip_code_redeem.js

add OVERABILLION code to emails variable

Thank you! Currently, the "maintained" list of codes is being provided by the /r/borderlands3 discord server, via this link (this is just a link to the spreadsheet):
https://docs.google.com/spreadsheets/d/1QfwdILwVv19sKeveaBzZwhQLEOzmFwnOmMZsWLKwgg8/htmlview?usp=sharing&sle=true
I double checked, and this code is included within the spreadsheet.
Thanks again!


pattern &ConfChangeError(_) not covered

Bug Report

What version of TiKV are you using?

Master
What operating system and CPU are you using?

macOS
Steps to reproduce

cargo check --all
What did you expect?
run successfully
What did happened?
➜  tikv git:(master) ✗ RUST_BACKTRACE=full cargo check --all
   Compiling afl v0.6.0
   Compiling librocksdb_sys v0.1.0 (https://github.com/tikv/rust-rocksdb.git#e9b74b88)
    Checking error_code v0.0.1 (/Users/gentcys/Codes/personal/tikv/components/error_code)
    Checking keys v0.1.0 (/Users/gentcys/Codes/personal/tikv/components/keys)
    Checking external_storage v0.0.1 (/Users/gentcys/Codes/personal/tikv/components/external_storage)
error[E0004]: non-exhaustive patterns: `&ConfChangeError(_)` not covered
  --> components/error_code/src/raft.rs:23:15
   |
23 |         match self {
   |               ^^^^ pattern `&ConfChangeError(_)` not covered
   |
  ::: /Users/gentcys/.cargo/git/checkouts/raft-rs-841f8a6db665c5c0/d71daff/src/errors.rs:54:9
   |
54 |         ConfChangeError(message: String) {
   |         --------------- not covered
   |
   = help: ensure that all possible cases are being handled, possibly by adding wildcards or more match arms
   = note: the matched value is of type `&raft::Error`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0004`.
error: could not compile `error_code`

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: build failed

Figure out that raft-rs#dd4ae1e introduces ConfChangeError , i will open a PR to fix it.

I guess it was caused by the mismatched cargo version and it updated the raft-rs version.


"There was a Problem communicating..." - Installation Issues

Hello everyone,
for a university project I and a friend try to create a MagicMirror, which can be controlled via Alexa.
Everything works except the MirrorMirrorOnTheWall module.
I installed the module according to the instructions (https://joanaz.github.io/MirrorMirrorOnTheWallSkill/).
As soon as I try to give a command via the Alexa Developer Console I get the following error message: "There was a problem with the requested skill's response".
The JSON output is "null".
By the way, I get the desired answer to the wake-up word "magic Mirror".
I would be very grateful for a solution, since I have already tried everything in my power (new installation etc.).
I know that the problem description is very daring. However, I don't know how to describe my problem further.
If you need logfiles, files, code or similar, write me and I will post the things accordingly.
Many thanks for your help and a nice Tuesday!
LepMMM

I’m having the same issue. Null on response. Alexa notifies me mirror mirror is running but start magic mirror gets no response. Maybe we are getting too wrapped up in the details and missed something?


Bug: Async SequentialStream reads do not complete

Based on the binary read benchmark in https://github.com/dotnet/SqlClient/issues/593#issuecomment-642833062 I tried the code below expecting the behaviour to be fast and found that it simply blocks indefinitely.
            using var conn = new SqlConnection(ConnectionString);
            using var cmd = new SqlCommand("SELECT foo FROM data", conn);
            await conn.OpenAsync();
            using var reader = await cmd.ExecuteReaderAsync(
                System.Data.CommandBehavior.SequentialAccess
            );
            await reader.ReadAsync();
            using var stream =  reader.GetStream(0);
            using var memory = new MemoryStream(16 * 1024);

            await stream.CopyToAsync(memory);
            return (int)memory.Length;

This should work but freezes in a task wait after a single read cycle, the second read never completed. If you change it to standard mode it'll work but it does so by fetching the entire field and giving you a reader over the byte[]. So no workaround, this needs fixing if the team agree that it's a bug.
Edit: for clarity the problem is when multiple packets are needed. The current tests read from a single packet and thus do not do a network read.

Thanks @Wraith2
Ack. Yes that does seem like an issue, if you have a proposed fix, let us know if you'd like to create PR, or we will start working on it soon.

Adding note: This issue is not reproducible in NetFx driver (for .NET Framework) so it should be investigated since when this issue has started to occur for .NET Core applications.


Set bot to welcome people to irc

Inspired by Openhatch

welcomebot> Welcome simonv3!  The channel is pretty quiet right now, so I thought I'd say hello, and ping some people (like shauna, paulproteus, and marktraceur) that you're here.  If no one responds for a while, try emailing us at<EMAIL_ADDRESS>or just try coming back later.  FYI, you're now on my list of known nicknames, so I won't bother you again.

Now that we've got slackin installed, I'm not so concerned about this.


Improves github actions

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@1be44a7). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage        ?   82.68%
=======================================
  Files           ?       38
  Lines           ?     5451
  Branches        ?        0
=======================================
  Hits            ?     4507
  Misses          ?      944
  Partials        ?        0

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


spec: fix typo in raw string example

See https://github.com/google/starlark-go/pull/311 for go.starlark.net change.


MRG, DOC: Document 'ignore' on_defects value; make allowed on_missing values explicit

This is a follow-up to #9614, where @larsoner pointed out that the on_defects param should support an 'ignore' value too. I've made the required adjustments, and also search-and-replaced on_missing : str -> on_missing : 'raise' | 'warn' | 'ignore' in the entire docdict.

Failures are real, maybe just an error class change?

Arrrrrgh. Will have a look.

@larsoner I think we should be good now. Thanks for your feedback!

(Not sure why CI won't start up now?)

(oh, webhook incident. well … we'll just have to wait until it's resolved, I suppose)

Ok we're green!

Thx @hoechenberger


Update to node.js v10

What problem are you trying to solve?
Node.js v10 is now in LTS status. Currently, trying to npm install with node.js v10 installed will error out.
Describe the solution you'd like
The scope of this Issue is to fix the dependencies to make the software work in Node.js v10.

this will be accomplished once badger extension and mobile are consolidated. closing.


Descibe the naming convention in the FAQs

Done in the FAQ, the renaming of the content still hast ot be done

It is already resolved.


Initial workflow repo skeleton

Adding as a dark feature: there are no links to these pages, but you can access the workflow list at /#workflows.
This is just the first functionality of the repo: list workflows, view snapshot details, and list configs.

The questions about error reporting and setting a pattern for useEffect manipulating url query are really good; since this is just intended to be a skeleton, I'm not going to address them right now, but I'll leave comments in the code.


Label in the center of Pie Chart

It would be nice if we could add a Widget in the center of the donut.

Hi, you can wrap your PieCart in a Stack widget, then put your second widget in the middle to achieve your result.

Hi, you can wrap your PieCart in a Stack widget, then put your second widget in the middle to achieve your result.

Hi, you can wrap your PieCart in a Stack widget, then put your second widget in the middle to achieve your result.


Site Search - Default results, site-docs indexing

Is there a way to have the default search results (when no keyword is entered) be an explicitly-defined set of pages rather than the results it is giving now?  Right now it has one result whose body text only shows as “]]” (Chrono) and then near 30 different results for Any.  Even if those are the "proper" results for an empty string search, I don't think there's much use to a user to get those results.
Also, is it indexing the site docs?  I tried searching for “Explore the Content” but nothing comes up.  (https://www.stage.boost.cppalliance.org/doc/user-guide/explore-the-content.html)
Thanks!

Is there a way to have the default search results (when no keyword is entered) be an explicitly-defined set of pages rather than the results it is giving now? Right now it has one result whose body text only shows as “]]” (Chrono) and then near 30 different results for Any. Even if those are the "proper" results for an empty string search, I don't think there's much use to a user to get those results.

Yeah, I think the straightforward solution would be to avoid performing any query with an empty string.

Also, is it indexing the site docs? I tried searching for “Explore the Content” but nothing comes up. (https://www.stage.boost.cppalliance.org/doc/user-guide/explore-the-content.html)

No, it currently only searches in library documentations. I'll add other sections like docs, news, and mailing lists in the future.

The issue of showing results for empty search query has been resolved in this PR: https://github.com/cppalliance/boost-gecko/pull/142.


Right click on MacOS causes app crash

When an iOS App is run on MacOSM1 using this library. Long press or long click, more specifically does nothing. It fails to trigger long press event. Trying to right click crashes the app.

Oh very interesting, I havent tried to use it on M1 yet

Add a title to the menu - even if it's just an empty string

Ok, thank you. Setting the optional 'title' prop to an empty string fixed the problem.
title={""]


Fix upbound maintainer name

Description of your changes

The maintainer name has a typo. Should be Upbound not Upbouns.
I have:

[x] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested

NA

 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.

@donovanmuller thanks for  the fix!


Sort handler triggered twice on mobile

When tapping on the header of a sortable column on a mobile device (Android, Chrome, latest) the sort handler is triggered twice resulting in the same sorting direction. On slow devices it is visible that the sorting happens twice. This makes that the user is not able to switch the sorting direction. The issue is most probably caused by differences in touch input api's of different browsers.

I can imagine this happening if the touchstart and click events are considered seperate, causing this code not work: https://github.com/HubSpot/sortable/blob/master/coffee/sortable.coffee#L41-L44


Suite setup teardown

This adds a new optional suite_setup and suite_teardown function for etest. Also adds selftest around this new behavior.
Remove etest sharding as it's not used anymore and added complexity.

Why are the rc files checked in?

Why are the rc files checked in?

Why are the rc files checked in?

Per-project vim configuration instead of relying on end users global vim configuration.

Why are the rc files checked in?

Per-project vim configuration instead of relying on end users global vim configuration.

Why are the rc files checked in?

Per-project vim configuration instead of relying on end users global vim configuration.

https://medium.com/@dnrvs/per-project-settings-in-nvim-fc8c8877d970

Why are the rc files checked in?

Per-project vim configuration instead of relying on end users global vim configuration.

https://medium.com/@dnrvs/per-project-settings-in-nvim-fc8c8877d970


Build crashes when used with "write: false"

Hi!
When developing locally, we use esbuild watch with write: false in the esbuild config. We do this so that we have greater control over how many files we write to the file system. However, it appears that the esbuild-css-modules-plugin throws the following error when we set write: false:
✘ [ERROR] ENOENT: no such file or directory, open '<long-path>/dist/react/badge.css' [plugin esbuild-css-modules-plugin]

Would it be possible for the plugin to either create the folder before writing to the disk (and thus removing the error), or respect the write setting and return the CSS in memory like the rest of the build?
It's worth mentioning that the file is written to the disk, so the error only appears in the logs on the first build, when the folder is empty.

It's reasonable to respect the write setting, will fix this issue in next version, thanks.


Fix cross-namespace LIST and WATCH

Extracted from #4263 comments @roberthbailey
Realized after this went in that cross-namespace queries were broken.

I left a few initial comments but I'm going to be out of the office until Tuesday. Unassigning in the hopes that someone else will look at this before then, but if not feel free to reassign back to me at that point.

Please rebase. There's a merge conflict.
Otherwise, LGTM.
@lavalamp Suggested defining a struct that contains both namespace and name, which sounds good to me, but that can be a separate PR.

@nikhiljindal said he made some comments directly on a commit that were lost in a rebase. @smarterclayton did you see them in email?

I think so, but will double check when I rebase

On Feb 13, 2015, at 2:51 PM, Brian Grant<EMAIL_ADDRESS>wrote:
@nikhiljindal said he made some comments directly on a commit that were lost in a rebase. @smarterclayton did you see them in email?
—
Reply to this email directly or view it on GitHub.

This has been updated with nikhil's comment.  It resolves a fairly serious bug introduced with the first version of this pull - cross namespace LIST and WATCH (which we apparently had no test cases for) has been corrected (the scope namer passed to the cross namespace methods does not set a default namespace).  Added more test cases.

Actually... picking the bandaid a bit more, and a lot more cross namespace query arterial blood comes pouring out...  Now it seems like the client was not properly namespacing requests in a lot of cases, which means that a few api.NamespaceAll queries were only working on the default namespace.

@derekwaynecarr did we change NamespaceAll recently?  I'm seeing changes that are possibly related to my earlier api_installer changes from #4263, but also errors that look like we weren't handling namespace scoping consistently from the client (i.e. clients would not be able to properly set NamespaceAll scope).
After digging into this more:

The "query all" code in endpoints controller sends NamespaceAll == ""
client/request.go would only set namespace param if it was length > 0
server treated missing namespace param as "default namespace"
endpoints were not being set in different namespaces

My second commit here properly enforces namespace != "" on namespace scoped resources, and forces our client code to distinguish between namespace not present and namespace empty.  Given the larger scope of the change it needs deeper review.  This now properly distinguishes in client code between namespace and non-namespaced scoped resources.
The legacy behavior of handling empty namespace like "default" on POST is preserved in the first commit.

I am not aware of a change.  I will review this PR more in morning

At this point, handlers_test had cases for ensuring namespace all behaviors since that was the prior choke point. but I am guessing that the changes were at a level above that test.
Sent from my iPhone

On Feb 15, 2015, at 11:43 PM, Clayton Coleman<EMAIL_ADDRESS>wrote:
@derekwaynecarr did we change NamespaceAll recently? I'm seeing changes that are possibly related to my earlier api_installer changes from #4263, but also errors that look like we weren't handling namespace scoping consistently from the client (i.e. clients would not be able to properly set NamespaceAll scope).
After digging into this more:
The "query all" code in endpoints controller sends NamespaceAll == ""
client/request.go would only set namespace param if it was length > 0
server treated missing namespace param as "default namespace"
endpoints were not being set in different namespaces
—
Reply to this email directly or view it on GitHub.

They were - I think this implies to me that there needs to be a higher level set of tests around paths, methods, and RESTStorage objects that permutes all endpoints (like auth_test, but specifically to highlight the modes).
----- Original Message -----

At this point, handlers_test had cases for ensuring namespace all behaviors
since that was the prior choke point. but I am guessing that the changes
were at a level above that test.
Sent from my iPhone

On Feb 15, 2015, at 11:43 PM, Clayton Coleman<EMAIL_ADDRESS>wrote:
@derekwaynecarr did we change NamespaceAll recently? I'm seeing changes
that are possibly related to my earlier api_installer changes from #4263,
but also errors that look like we weren't handling namespace scoping
consistently from the client (i.e. clients would not be able to properly
set NamespaceAll scope).
After digging into this more:
The "query all" code in endpoints controller sends NamespaceAll == ""
client/request.go would only set namespace param if it was length > 0
server treated missing namespace param as "default namespace"
endpoints were not being set in different namespaces
—
Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/pull/4395#issuecomment-74465604

LGTM - the client side change will help stop subtle errors.  the server side change once I found it is right ;-)

Added a better integration test for overlapping endpoint namespaces (to explicitly validate NamespaceAll) and made SelfLink more correct for default namespaces.

LGTM. Feel free to merge.

This broke e2e on update.sh, because it gets tripped up in the check for an empty namespace.
I'm going to try to patch forward rather than roll back, but pretty please let's be careful about running e2e.

I saw passing update, but obviously this wasn't true.  Can you cc me on the fix?

On Feb 17, 2015, at 10:48 PM, Brendan Burns<EMAIL_ADDRESS>wrote:
This broke e2e on update.sh, because it gets tripped up in the check for an empty namespace.
I'm going to try to patch forward rather than roll back, but pretty please let's be careful about running e2e.
—
Reply to this email directly or view it on GitHub.


Runtime errors during param_unconstrain because of precision issues

I was re running some scripts dealing with simplexes, and ran into this.
Basically, for a valid simplex constraint, if I try to use the sampled draws from a stan program in model.param_unconstrain, I end up with
RuntimeError: param_unconstrain() failed with exception: Exception: stan::math::simplex_free:
Simplex variable is not a valid simplex. sum(Simplex variable) = 0.99999959, but should be 1

which is of course not a bridgestan issue, and other stan interfaces have also had similar problems I think, anyway this is only because of sig_figs being 6 by default. I'd wager this to be a common issue as bridgestan grows. As for a fix, I don't know if that's easy to do, or even in scope for BridgeStan, but if there'd be a way of adding tolerance for the constraint checks that would be quite nice. For example I have 400 chains with 1000 draws each for 1000 dimensions, so in this case I'd avoid more significant digits solely for the sake of convenience.
Anyway, this is not a bug, but if it's at all helpful for future scope, or if there's an easier fix other than changing sig_figs in Cmdstan itself I'd be very happy to hear about that.
I noticed this is in CmdStanR future milestone list but it sort of bleeds through all Stan Interfaces.

The tolerance is a hard coded value here: https://github.com/stan-dev/math/blob/46f3c59e63851451ed1bf9b64c8f57c89fa49e8e/stan/math/prim/err/constraint_tolerance.hpp#L12
It would be nice if this was at least a #define so we could override it easily at compile time.

okay makes sense, yeah that's unfortunate, for high dimensional simplexes this would definitely keep creating problems.

I think using more sig_figs in CmdStan (or CmdStan offering a binary format which has full double precision) are probably the best “solutions” for now, but it might be worth opening an issue in Stan-Dev/math about making that constant something configurable

From what I see, there is not a flag for disabling this. @WardBrian would you mind filing an issue in Stan Math?

I think the issue @mjhajharia opened for a configurable tolerance is better than a flag, since one could just set it to a very high tolerance and effectively disable it that way

The next version of Stan will let this be configured at compile time


Title: Correct Title Capitalization for Professionalism

Corrected title capitalization for the paper title "Attention Is All You Need" for professional presentation.

Thanks! We addressed this in #218


update findtbb

uses kitware version that properly prefers exported cmake targets + able to use onetbb

@evaleev Is there anything else you want to do with this, or can it be merged?

It's good to go
On Wed, Jan 5, 2022, 4:45 PM David Williams-Young @.***>
wrote:

@evaleev https://github.com/evaleev Is there anything else you want to
do with this, or can it be merged?
—
Reply to this email directly, view it on GitHub
https://github.com/ajaypanyala/linalg-cmake-modules/pull/2#issuecomment-1006102932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQXIZ7RFMP7L7XMC4ZSRLDUUS3XLANCNFSM5K6FZ5ZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@ajaypanyala Can you merge this?


Fix staging and release builds on Android

Fixed by rolling back Gradle and its Android plugin to the highest version compatible with React Native, and prevent them to try to use a dropped architecture toolchain.

Android Studio uses the latest NDK version (with no way to force a lower version from the build config files)
The latest NDK version dropped MIPS support
Only the latest Gradle Android plugin take that into account
React Native isn't compatible with the latest Gradle Android plugin
or even the latest Gradle version

cf android-ndk/ndk#700

Approving but


Roll Flutter from 48172589e5ac to 0dfb656e28b9 (1 revision)

https://github.com/flutter/flutter/compare/48172589e5ac...0dfb656e28b9
2022-03-10<EMAIL_ADDRESS>Roll Engine from bf2bf3a3fd2b to 6b1a24b1a108 (10 revisions) (flutter/flutter#99917)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


requireNumericLiterals: Prevent erroring when first argument is an Identifier.

Prior to this fix, the following would trigger an error:
let foo = '123';
parseInt(foo, 16);

After this change, an error is only triggered when using an actual literal (as intended and indicated by the rule name).
Addresses https://github.com/DockYard/ember-suave/pull/80#issuecomment-183779369.
/cc @hzoo

Thanks

Cool, thanks @rwjblue!


[IDEA] Collaborate / integrate with https://github.com/AlexTatiyants/pev ?

This looks nice and potentially very useful, congrats!
I've personally used the Postgres Explain Visualizer of @AlexTatiyants, whose code is hosted here https://github.com/AlexTatiyants/pev, in the past and found it very useful to grasp plans and reveal leads to improve queries or data models.
So as an idea, integrating your two solutions into one website could potentially make both even better and useful.
Cheers

Follow-up: the above PEV might have been abandoned but here's an active successor (found in the HN comments) https://github.com/dalibo/pev2

The beauty of PEV is that it's all done on the client, there is no server side component.
I have replicated some of this repo's functionality in client-side JavaScript (see pg_flame.js), that solution might be more easily embeddable.

Cool to see a JS-only implementation! Definitely useful, just depends on your workflow if you'd prefer that over a CLI.
I'm going to close this as I don't see any action items I can take at the moment. But I'm always open to any sort of collaboration.


Decrease codegen-units for Windows builds to decrease memory consumption

Might resolve #95?

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Failing OSX build is unrelated to this PR.

Well... that didn't help 🥴


Add PHP Redis extension to 8.0 & 8.1 containers

Thanks!

I just pushed new versions for both containers.


Feature/add css custom properties

Coverage increased (+0.05%) to 91.766% when pulling bc641481524017e4f340bdf1fb6950958636a35f on feature/Add_css_custom_properties into a7742bb21ce682fd904f1d287bb027cc3760a000 on develop.


How to subscribe two realsense image topic through one callback

Hi Marty,
In my laptop, I have installed the librealsense SDK so that I can use following python code for D435i camera
import cv2
import pyrealsense2
from realsense_depth import *
Initialize Camera Intel Realsense
dc = DepthCamera()
while True:
ret, depth_frame, color_frame = dc.get_frame()    ------- one function call gets two frames
------ --------------------- ---------------------------------
So I can get the color_frame and depth_frame which can be used to detect the distance.
In my real Robot, I have the D435i as well.
#following is used to subscribe color_frame
self.image = rospy.Subscriber("/realsense/color/image_raw",sensor_msgs.msg.Image,self.camera_callback)
#following is used to subscribe the depth_frame which is used to get the distance
image_topic = "/realsense/depth/image_rect_raw" #16UC1 format
self.depth_image = rospy.Subscriber(image_topic, sensor_msgs.msg.Image, self.image_callback)
Although, I can get both frames in ROS run, since frames are from two different callbacks: camera_callback and image_callback, the robot moves in on and off base. I think that two callbacks switch back and forth caused the problem.  If I only use one callbacks, then robot moves smoothly and normally.
Do you know how I can use one callback to get two frames: depth_frame and color_frame through subscribe in ROS?
Thanks so much for you help
Jimmy

Hi @qin53027   My research of your question did not find a reference for using one Python callback for two streams with ROS topics.  The closest I could find was one that apparently uses two callbacks in two separate Python scripts at https://github.com/IntelRealSense/realsense-ros/issues/1711#issuecomment-782631635
Maybe it will provide some ideas for refining your current two callback approach.

Hi Marty, thanks for your information.
At this time, I can use ApproximateTimeSynchronizer to combine two image topic into one callback. Although it works, it still is kind of slow.  So for my simple solution, I just used the depth_image only to train and detect the distance since it works
In case someone may have same question, following is just my example python code.
Thanks
Jimmy
from message_filters import TimeSynchronizer, Subscriber, ApproximateTimeSynchronizer
    image_sub = Subscriber(
        "/realsense/color/image_raw",
        sensor_msgs.msg.Image,
        queue_size=1
    )

    depth_sub = Subscriber(
        "/realsense/depth/image_rect_raw",
        sensor_msgs.msg.Image,
        queue_size=1
    )

    # Time syncronizer is implimented to make sure that all of the frames match up from all of the topics.
    #ts = TimeSynchronizer([image_sub, depth_sub], 10)
    ts = ApproximateTimeSynchronizer([image_sub, depth_sub], 1, 0.1)
    ts.registerCallback(self.mycallback)

Thanks so much @qin53027 for sharing your solution with the RealSense community!


It doesn't work

I just reinstalled Arch and found this so decided to try it. After installing arch and getting gnome to access it all, I followed every step here and then restarted. It isn't working. There is no difference and fvwm doesn't show up as a display (like gnome or plasma) in the login menu. I did switch from Gdm to slim though.
Thanks for your help.

Please try to install it without gnome :/

@Syndrizzle After messing around, I managed to get it to kind of work but because my screen is the wrong size I don't think it will work right. Thanks anyways. It looks great!

So since I can't find a better-looking rice, and I like BSPWM better than the new one, I decided to try it. My screen size is 1366x768. Could you help me with the screen sizing. Thanks.

You gotta edit almost every file :pensive:

@Syndrizzle Like every every file or just the eww ones? Do you think anyone has a list of things to change or even just files to change so I don't have to go through every file and see if there is anything to change? Or maybe just a list of options to change so at least I know what I'm looking for when I start.

I'm going to start a new issue with a different title.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/75 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit 43f64e7:
Pending

Docs Build status updates of commit 43f64e7:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[BUG] CLI server ls command is missing format option

Rancher Server Setup

Rancher version:
Installation option (Docker install/Helm Chart): N/A
Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream): N/A

User Information

What is the role of the user logged in? N/A

Describe the bug

The Rancher CLI server ls command is missing the --format option.
To Reproduce

Result
Expected Result

Screenshots

Additional context

Validated with CLI version v2.11.0-rc.3

Verify 'server ls' with format - json
Verify 'server ls' with format - yaml
Verify 'server ls' with format - custom
Verify 'server ls' help menu includes format option