rs-measure depth_frame

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Hello, thank you for answering every time. Please consider that I am a beginner in c++ and realsense.
I'm going to measure the length of the object using rs-measure and opencv. I'm going to take depth_frame, colorize it, and continue on its own. However, when I tried to use get_distance function in rs-measure, I think there was an error because it was not depth_frame. The colorize frame was not rs2::depth_frame, but rs2::frame. I used colorize because I needed a boundary line. Is there any other way to use get_distance as a frame using colorize or to detect objects to automatically measure length?

Contouring the colored frame as shown in the image.

hi @HyeYumKim  If you are using OpenCV, the distance between two points on an image can be calculated using either norm or magnitude(), as described in the C++ discussion at the link below.
https://exchangetuts.com/opencv-c-distance-between-two-points-1639904963662770
If you need to use C++ to convert a cv::mat back to rs2::frame then a RealSense team member offers advice for doing so at https://github.com/IntelRealSense/librealsense/issues/2634

Case closed due to no further comments received.


write temp sql file to stage dir

Coverage increased (+0.01%) to 81.046% when pulling 530ea7e941c0aeb00b9243b261bc3c89ec89694e on 12474-sql-files-to-stage-dir into 6a4dad8b5bfa42c5aa5bf1ef1906629d27aec3af on master.


Add note to README re: go-marathon's DSL purpose.

This is a follow-up of a discussion that took place along PR #87.

What do you think about providing a code example as well? Some people (like me) skip explanations and look only at code examples :)

To me, the note is mostly of educational nature. Thus, I'm a bit worried that embedding an entire comparative example could add some extra noise to the README file we might want to avoid.
How about just linking to @gambol99's excellent example in his PR comment instead? WDYT?

I like the idea, however the linked example might become outdated, since it's not the part of the project. Let's keep it without changes then.
LGTM


chore(package): update kronos-service to version 4.1.6

https://greenkeeper.io/

Current coverage is 74.83% (diff: 100%)

Merging #197 into master will not change coverage

@@             master       #197   diff @@
==========================================
  Files             1          1
  Lines           151        151
  Methods          22         22
  Messages          0          0
  Branches         20         20
==========================================
  Hits            113        113
  Misses           38         38
  Partials          0          0

Powered by Codecov. Last update 53e2bfb...bb95372


updating How to apply page

Trello card
https://trello.com/c/Ny7RS6Mg/4291-change-guidance-on-number-of-application-choices-from-3-to-4
Context
Apply are changing the amount of applications you can make from 3 to 4. We need to update our content to reflect that.
Changes proposed in this pull request
Guidance to review

Thanks @EllieNodder. I'll approve once we've shipped the code change on Apply! I think I saw there was a blog post that references 3 choices too, but I'm not sure what our process is for updating those if the info becomes out of date. Do we have a view @gemmadallmandfe?

Eagle eyes @MylesJarvis !
@EllieNodder these blogs reference 3 choices - probably need to update?

/blog/choosing-the-right-teacher-training-course-provider
/blog/9-tips-to-help-you-apply-for-teacher-training

Not sure if there are any others?


No attribute is_endpoint

Hello, when running this notebook, I got an error message in the cell with # turn off strict mode and see what nodes we'd remove, in red.
"AttributeError: module 'osmnx' has no attribute 'is_endpoint'".
I haven't found any updates to this in the docs; can you advise? Thanks.

How are you running it? You should use the provided Binder link as it provides a consistent environment with all the packages at the same version as those used when the course was filmed. Here's the link: https://mybinder.org/v2/gh/gboeing/planetizen-python/2017?urlpath=tree
If you run it in your own environment on your own computer, you will likely have newer versions of the packages in which little things may have changed over time.

How are you running it? You should use the provided Binder link as it provides a consistent environment with all the packages at the same version as those used when the course was filmed. Here's the link: https://mybinder.org/v2/gh/gboeing/planetizen-python/2017?urlpath=tree
If you run it in your own environment on your own computer, you will likely have newer versions of the packages in which little things may have changed over time.

Thanks for your reply. I did not know about binders so I had just downloaded the files from the Planetizen course website and was running the notebooks in an environment I had created in Anaconda Navigator. What you said makes sense, so thanks for that, I did not really know what environments were and why package versions mattered until recently. I am not sure other beginners on the course would either.
For anyone else who might be reading this and new to Jupyter and Python: the "environment.yml" file at the link above lists the versions of each package that Geoff had installed when he created the course.  (google that if you are not sure what environments are) I wanted to be able to edit and run this notebook for use in the future so I imported that environment using the "import" button in Anaconda navigator. Then I made sure to run the jupyter notebook in that environment and it worked.
I think you can also create that environment using the terminal with the instructions here: https://conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#creating-an-environment-with-commands
Thanks!

Thanks for your reply. I did not know about binders so I had just downloaded the files from the Planetizen course website and was running the notebooks in an environment I had created in Anaconda Navigator. What you said makes sense, so thanks for that, I did not really know what environments were and why package versions mattered until recently. I am not sure other beginners on the course would either.
For anyone else who might be reading this and new to Jupyter and Python: the "environment.yml" file at the link above lists the versions of each package that Geoff had installed when he created the course.  (google that if you are not sure what environments are) I wanted to be able to edit and run this notebook for use in the future so I imported that environment using the "import" button in Anaconda navigator. Then I made sure to run the jupyter notebook in that environment and it worked.
I think you can also create that environment using the terminal with the instructions here: https://conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#creating-an-environment-with-commands
Thanks!

Honestly at this point that course is 5 or 6 years old. Underlying software evolves and changes over the years as new features are released and old ones deprecated. It would really require revising it all with Planetizen at some point.
In the meantime, if you're on the "street networks" lesson, you can check this out for a regularly updated introduction to street network analysis (and more!) with Python and OSMnx: https://mybinder.org/v2/gh/gboeing/osmnx-examples/main?urlpath=lab


add ability to export to CSV

seemed more useful for further manipulation (i.e. - add birthday data to my contacts using another method such as Apple Scripts)

Hey, sorry I missed this PR, thanks for sending it in.
I'll go publish a new version with this update!


[skip ci] convert pr gate to pipeline

Put Jenkins Jobs  for "PR Gate" under source code control.
Test:
http://rackhdci.lss.emc.com/blue/organizations/jenkins/zon-core/activity
A test running of the pipeline (which failed at unit test):

A test running of the pipeline (which failed at function test):

@changev @panpan0000


Assets not in the precacheConfig are also being cached

webpack version:
1.31.2
sw-precache-webpack-plugin version:
0.9.1
Please tell us about your environment:
OSX 10.x
Browser: Chrome 57
I'm using this plugin to configure service worker and it's caching and loading the assets that are supposed to be cached with no problem. My issue is that it seems some assets that are not specified in the config are also being cached.
Specifically, my app streams mp3 files that are stored in the cloud (Amazon S3), accessed and played via standard urls. In my webpack config file, I only set the local assets to be cached, not the mp3 files. So I expect when the browser goes offline, I won't be able to play the mp3 files.
The actual behavior is that when I turn off wifi and reload the page, I can still play the MP3s! I checked my cache storage, indexDB, and local storage. None of them contain these mp3 files. So the fact that I can play them offline is rather baffling!
Here is one of the pages that contain audio files.
Would appreciate any insight about what's going on.

Webpack configuration:

var path = require('path');
var SWPrecacheWebpackPlugin = require('sw-precache-webpack-plugin');

module.exports = {
  context: __dirname,
  entry: [
    'babel-polyfill',
    './client/index.js'
  ],
  output: {
    path: path.resolve(__dirname, 'client/'),
    publicPath: '/',
    filename: 'bundle.js'
  },
  module: {
    loaders: [{
      exclude: /node_modules/,
      loader: 'babel',
      query: {
        presets: ['react', 'es2015', 'stage-1']
      }
    },
    {
      test: /\.(jpg|png)$/,
      loader: 'url-loader?mimetype=image/png'
    }]
  },
  resolve: {
    extensions: ['', '.js', '.jsx']
  },
  devServer: {
    historyApiFallback: true,
    contentBase: './client'
  },
  plugins: [
    new SWPrecacheWebpackPlugin(
      {
        cacheId: 'Soundwise',
        filename: 'service-worker.js',
        maximumFileSizeToCacheInBytes: 10485760
        minify: false,
        stripPrefix: 'client/',
        navigateFallback: 'index.html',
        staticFileGlobs: [
          'client/index.html',
          'client/css/**.css',
          'client/bundle.js',
          'client/images/soundwiselogo_white.png',
          'client/images/soundwiselogo.png',
          'client/fonts/**.woff',
          'client/fonts/**.woff2',
          'client/fonts/**.svg',
          'client/fonts/**.eot',
          'client/fonts/**.otf',
          'client/images/smiley_face.jpg',
        ],
      }
    ),
  ]
};

Generated service worker (not minified):

/**
 * Copyright 2016 Google Inc. All rights reserved.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
*/

// DO NOT EDIT THIS GENERATED OUTPUT DIRECTLY!
// This file should be overwritten as part of your build process.
// If you need to extend the behavior of the generated service worker, the best approach is to write
// additional code and include it using the importScripts option:
//   https://github.com/GoogleChrome/sw-precache#importscripts-arraystring
//
// Alternatively, it's possible to make changes to the underlying template file and then use that as the
// new base for generating output, via the templateFilePath option:
//   https://github.com/GoogleChrome/sw-precache#templatefilepath-string
//
// If you go that route, make sure that whenever you update your sw-precache dependency, you reconcile any
// changes made to this original template file with your modified copy.

// This generated service worker JavaScript will precache your site's resources.
// The code needs to be saved in a .js file at the top-level of your site, and registered
// from your pages in order to be used. See
// https://github.com/googlechrome/sw-precache/blob/master/demo/app/js/service-worker-registration.js
// for an example of how you can register this script and handle various service worker events.

/* eslint-env worker, serviceworker */
/* eslint-disable indent, no-unused-vars, no-multiple-empty-lines, max-nested-callbacks, space-before-function-paren, quotes, comma-spacing */
'use strict';

var precacheConfig = [["bundle.js","e0e2f7bc22b7468baa217868f34cdb19"],["css/animate.css","bd71aa02cf7aedd2f234c5d8540f4152"],["css/base.css","4f082492535d3bb26fa310590b369639"],["css/bootstrap.min.css","d3b96c646afa495129326de3d685c2d8"],["css/font-awesome.min.css","008e0bb5ebfa7bc298a042f95944df25"],["css/jplayer.flat.css","78b4db479b80b23edaab20b8d6f1c60c"],["css/magnific-popup.css","ff0e1af4b5443a6ffb28c6d32ec8ce44"],["css/owl.carousel.css","08b67c2fd1a1b0460ab190be8027762e"],["css/owl.transitions.css","4276863948d6d6810bac12d419bf32ba"],["css/responsive.css","c98bec8f31d93de114fe24c5275e484d"],["css/simple-line-icons.css","bc6c208c74c56cb851aabb4538beaf05"],["css/style.css","23483f2a771aacd9ba8ed65c67489e60"],["css/themify-icons.css","22e134d4d9b3aac6ba47550cd1d06565"],["fonts/FontAwesome.otf","0d2717cd5d853e5c765ca032dfd41a4d"],["fonts/Simple-Line-Icons.dev.svg","c8b47e8b3fe87328731909587fc72b85"],["fonts/Simple-Line-Icons.eot","f19a7f6c7a0b54b748277c40d7cf8882"],["fonts/Simple-Line-Icons.svg","5db2ea1bcb7a6cc078755c90458c3b65"],["fonts/Simple-Line-Icons.woff","ff94ad94c3a9d04bd2f80cb3c87dcccb"],["fonts/fontawesome-webfont.eot","674f50d287a8c48dc19ba404d20fe713"],["fonts/fontawesome-webfont.svg","912ec66d7572ff821749319396470bde"],["fonts/fontawesome-webfont.woff","fee66e712a8a08eef5805a46892932ad"],["fonts/fontawesome-webfont.woff2","af7ae505a9eed503f8b8e6982036873e"],["fonts/themify.eot","2c454669bdf3aebf32a1bd8ac1e0d2d6"],["fonts/themify.svg","9c8e96ecc7fa01e6ebcd196495ed2db5"],["fonts/themify.woff","a1ecc3b826d01251edddf29c3e4e1e97"],["images/smiley_face.jpg","1e756432ed4a415aee81508af6b9c99e"],["images/soundwiselogo.png","adc22e0335e7536067f74e36daa93297"],["images/soundwiselogo_white.png","accff5eaa99a4bc09f630f632aa62381"],["index.html","39177953d8f3c1d13358eba19437bfb1"]];
var cacheName = 'sw-precache-v3-Soundwise-' + (self.registration ? self.registration.scope : '');

var ignoreUrlParametersMatching = [/^utm_/];

var addDirectoryIndex = function (originalUrl, index) {
    var url = new URL(originalUrl);
    if (url.pathname.slice(-1) === '/') {
      url.pathname += index;
    }
    return url.toString();
  };

var cleanResponse = function (originalResponse) {
    // If this is not a redirected response, then we don't have to do anything.
    if (!originalResponse.redirected) {
      return Promise.resolve(originalResponse);
    }

    // Firefox 50 and below doesn't support the Response.body stream, so we may
    // need to read the entire body to memory as a Blob.
    var bodyPromise = 'body' in originalResponse ?
      Promise.resolve(originalResponse.body) :
      originalResponse.blob();

    return bodyPromise.then(function(body) {
      // new Response() is happy when passed either a stream or a Blob.
      return new Response(body, {
        headers: originalResponse.headers,
        status: originalResponse.status,
        statusText: originalResponse.statusText
      });
    });
  };

var createCacheKey = function (originalUrl, paramName, paramValue,
                           dontCacheBustUrlsMatching) {
    // Create a new URL object to avoid modifying originalUrl.
    var url = new URL(originalUrl);

    // If dontCacheBustUrlsMatching is not set, or if we don't have a match,
    // then add in the extra cache-busting URL parameter.
    if (!dontCacheBustUrlsMatching ||
        !(url.pathname.match(dontCacheBustUrlsMatching))) {
      url.search += (url.search ? '&' : '') +
        encodeURIComponent(paramName) + '=' + encodeURIComponent(paramValue);
    }

    return url.toString();
  };

var isPathWhitelisted = function (whitelist, absoluteUrlString) {
    // If the whitelist is empty, then consider all URLs to be whitelisted.
    if (whitelist.length === 0) {
      return true;
    }

    // Otherwise compare each path regex to the path of the URL passed in.
    var path = (new URL(absoluteUrlString)).pathname;
    return whitelist.some(function(whitelistedPathRegex) {
      return path.match(whitelistedPathRegex);
    });
  };

var stripIgnoredUrlParameters = function (originalUrl,
    ignoreUrlParametersMatching) {
    var url = new URL(originalUrl);

    url.search = url.search.slice(1) // Exclude initial '?'
      .split('&') // Split into an array of 'key=value' strings
      .map(function(kv) {
        return kv.split('='); // Split each 'key=value' string into a [key, value] array
      })
      .filter(function(kv) {
        return ignoreUrlParametersMatching.every(function(ignoredRegex) {
          return !ignoredRegex.test(kv[0]); // Return true iff the key doesn't match any of the regexes.
        });
      })
      .map(function(kv) {
        return kv.join('='); // Join each [key, value] array into a 'key=value' string
      })
      .join('&'); // Join the array of 'key=value' strings into a string with '&' in between each

    return url.toString();
  };

var hashParamName = '_sw-precache';
var urlsToCacheKeys = new Map(
  precacheConfig.map(function(item) {
    var relativeUrl = item[0];
    var hash = item[1];
    var absoluteUrl = new URL(relativeUrl, self.location);
    var cacheKey = createCacheKey(absoluteUrl, hashParamName, hash, false);
    return [absoluteUrl.toString(), cacheKey];
  })
);

function setOfCachedUrls(cache) {
  return cache.keys().then(function(requests) {
    return requests.map(function(request) {
      return request.url;
    });
  }).then(function(urls) {
    return new Set(urls);
  });
}

self.addEventListener('install', function(event) {
  event.waitUntil(
    caches.open(cacheName).then(function(cache) {
      return setOfCachedUrls(cache).then(function(cachedUrls) {
        return Promise.all(
          Array.from(urlsToCacheKeys.values()).map(function(cacheKey) {
            // If we don't have a key matching url in the cache already, add it.
            if (!cachedUrls.has(cacheKey)) {
              var request = new Request(cacheKey, {credentials: 'same-origin'});
              return fetch(request).then(function(response) {
                // Bail out of installation unless we get back a 200 OK for
                // every request.
                if (!response.ok) {
                  throw new Error('Request for ' + cacheKey + ' returned a ' +
                    'response with status ' + response.status);
                }

                return cleanResponse(response).then(function(responseToCache) {
                  return cache.put(cacheKey, responseToCache);
                });
              });
            }
          })
        );
      });
    }).then(function() {

      // Force the SW to transition from installing -> active state
      return self.skipWaiting();

    })
  );
});

self.addEventListener('activate', function(event) {
  var setOfExpectedUrls = new Set(urlsToCacheKeys.values());

  event.waitUntil(
    caches.open(cacheName).then(function(cache) {
      return cache.keys().then(function(existingRequests) {
        return Promise.all(
          existingRequests.map(function(existingRequest) {
            if (!setOfExpectedUrls.has(existingRequest.url)) {
              return cache.delete(existingRequest);
            }
          })
        );
      });
    }).then(function() {

      return self.clients.claim();

    })
  );
});

self.addEventListener('fetch', function(event) {
  if (event.request.method === 'GET') {
    // Should we call event.respondWith() inside this fetch event handler?
    // This needs to be determined synchronously, which will give other fetch
    // handlers a chance to handle the request if need be.
    var shouldRespond;

    // First, remove all the ignored parameter and see if we have that URL
    // in our cache. If so, great! shouldRespond will be true.
    var url = stripIgnoredUrlParameters(event.request.url, ignoreUrlParametersMatching);
    shouldRespond = urlsToCacheKeys.has(url);

    // If shouldRespond is false, check again, this time with 'index.html'
    // (or whatever the directoryIndex option is set to) at the end.
    var directoryIndex = 'index.html';
    if (!shouldRespond && directoryIndex) {
      url = addDirectoryIndex(url, directoryIndex);
      shouldRespond = urlsToCacheKeys.has(url);
    }

    // If shouldRespond is still false, check to see if this is a navigation
    // request, and if so, whether the URL matches navigateFallbackWhitelist.
    var navigateFallback = 'index.html';
    if (!shouldRespond &&
        navigateFallback &&
        (event.request.mode === 'navigate') &&
        isPathWhitelisted([], event.request.url)) {
      url = new URL(navigateFallback, self.location).toString();
      shouldRespond = urlsToCacheKeys.has(url);
    }

    // If shouldRespond was set to true at any point, then call
    // event.respondWith(), using the appropriate cache key.
    if (shouldRespond) {
      event.respondWith(
        caches.open(cacheName).then(function(cache) {
          return cache.match(urlsToCacheKeys.get(url)).then(function(response) {
            if (response) {
              return response;
            }
            throw Error('The cached response that was expected is missing.');
          });
        }).catch(function(e) {
          // Fall back to just fetch()ing the request if some unexpected error
          // prevented the cached response from being valid.
          console.warn('Couldn\'t serve response for "%s" from cache: %O', event.request.url, e);
          return fetch(event.request);
        })
      );
    }
  }
});

@natashache It's not your service worker, its coming from the "disk cache". Its just a browser caching thing.

Notice assets from the service worker will say they came from the service worker:

@goldhand Yes you're right. Thanks for looking into this.


ISPN-15974 Protocol buffers schema compatibility checks

https://issues.redhat.com/browse/ISPN-15974
Depends on https://github.com/infinispan/protostream/pull/260

@fax4ever Can you also take a look as this impacts Query?

@fax4ever Can you also take a look as this impacts Query?

Hi @ryanemerson. Sure, it sounds a good idea

Merged, thanks @tristantarrant


No symbol uses for enclosing types

Consider nested types:
public class A
{
  public class B
  {
    public static void M() { }
  }
}

When a nested type is used in F#, name resolution doesn't notify about its enclosing type use, so the type isn't available in features like Find Usages.
let b = A.B()
A.B.M()

Ah, it seems to be fixed in #5297. I'll check.

@auduchinok have you had a chance to check?

@cartermp Sorry, not yet. I'll check it soon when migrating our plugin to FCS from master or dev16.0.

@cartermp I can confirm it's fixed in dev16.1 branch.


Add simple halo model

In order to facilitate adding other halo models, I will create a base Theory class and a simple wrapper to pyhalomodel (https://github.com/alexander-mead/pyhalomodel) which has the advantage that it is available on PyPI so we can have it in our requirements easily.
Merging will close #64

Codecov Report

Merging #150 (61cb10b) into master (e11110d) will increase coverage by 0.74%.
The diff coverage is 94.59%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #150      +/-   ##
==========================================
+ Coverage   73.78%   74.53%   +0.74%
==========================================
  Files          31       33       +2
  Lines        1957     2022      +65
==========================================
+ Hits         1444     1507      +63
- Misses        513      515       +2

Files
Coverage Δ

soliket/__init__.py
100.00% <100.00%> (ø)

soliket/halo_model/__init__.py
100.00% <100.00%> (ø)

soliket/utils.py
92.85% <ø> (ø)

soliket/xcorr/__init__.py
100.00% <100.00%> (ø)

soliket/xcorr/xcorr.py
15.38% <ø> (-0.92%)
:arrow_down:

soliket/halo_model/halo_model.py
96.96% <96.96%> (ø)

soliket/xcorr/limber.py
8.92% <0.00%> (-1.60%)
:arrow_down:


Do not force image format if quality given

Currently, if quality is set on ImageOptions then the jpg format is forced. This any non jpg image is converted to jpg.
The official Contentful Images API doc says:

You can alter the quality of the image, expressed as a percentage value between 1 and 100. Quality value is only ignored for 8-bit PNGs.

So the quality parameter could be always passed, it will be ignored automatically by Contentful when not applicable.

This looks good. Thank you for contributing!


Improve clarity and readability of packages guides in docs

What is the purpose of this pull request?

[ ] New Icon
[ ] Bug fix
[ ] New Feature
[x] Documentation update
[ ] Other:

Description
Improve clarity and readability of packages guides in docs.
Before Submitting

[x] I've read the Contribution Guidelines.
[x] I've checked if there was an existing PR that solves the same issue.

Maybe you like to review my text written in #1521 as well. I'm not the best writer, but I try my best 😛

They look pretty good so far, but I do see a few areas that can be improved. I'll take a crack at making the necessary adjustments later today. 🙌

@jackson-jackson You're awesome tnx!


fix(forge): get contract name from identifier before passing to the debugger

Motivation
Same reasons as https://github.com/foundry-rs/foundry/pull/2102

AddressIdentity now saves the identifier [path:contractName] on AddressIdentity.contract, instead of just the name

Since the debugger expects contract names, it was not able to get the corresponding source.

Solution
Create a map from decoder.contracts with contract names as keys instead.

Does this resolve #1243?

It's unrelated.


🐛 [Bug] AttributeError: module 'torch_tensorrt' has no attribute 'set_device'

Bug Description
AttributeError: module 'torch_tensorrt' has no attribute 'set_device'
To Reproduce
>>> import torch_tensorrt
>>> torch_tensorrt.set_device(0)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'torch_tensorrt' has no attribute 'set_device'

Environment
nvcr.io/nvidian/pytorch:24.07-py3

Yes this was moved to allow us to build Torch-TensorRT without C++. I would recommend using PyTorch's cuda apis


Adding LU

@yullu First stage of verification passed, official domain is laikipia.ac.ke as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@yullu Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Fix datetime format that was messing with PM time values

Formatting with hh makes the PM times go back to AM in the datetimepicker.

Thanks @RenaudLN  for this change.


Image doesn't support multi-tenancy

Generic <IP_ADDRESS> is out there - images being generated for all builds


ability to scroll cursor to end of document

Method to scroll to end of document. Will be called from App Lab after we append code to the end of the document.

This feels like a very specific API.  I wonder if it would be better to have this method take a line number (with -1 for the last line of the document, -2 for the penultimate line, etc).

I'm somewhat wary of making a more generalized API without there being a clear need. Also moveAndScrollCursorToLine(-1) feels less clear than scrollCursorToEndOfDocument(), so we'd be sacrificing clarity for generalizability that it's not clear we need imo.

That makes sense.  LGTM


chore:  Update footer links to navigate to functional content.

Summary
Some of the footer links were pointing to old content or content that was not available anymore. This PR covers this by updating some of the links to the newer content and others to the new location.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.361%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
338

Relevant Lines:
401

💛  - Coveralls


fix

Refixes #2847
Proposed Changes
The ability to return a non default constructable object from an action was broken by ba7153c7fcc9cb63b9d8398314595a00cf6a6991 because of a missing negation of std::is_default_constructible<T>.
This wasn't caught by CI because the current test doesn't actually attempt to save or load any construction data.
Checklist

[x] I have fixed a bug and have added a regression test.

Can one of the admins verify this patch?

This looks like it was fixed on master by 23c910c08ee616394f4adeed639462deb2691a55 but opening this to fix the 1.7.X branch.

@kleinhenz We do not plan to make a point release for V1.7 at this point. How important is that fix for you for this version? We plan to release V1.8 soon, would that be sufficient for you?

ok I will close if there won't be another point release. I can wait for 1.8.

@kleinhenz thanks for submitting the patch in any case! Much appreciated.


missing emotion-insertion-point in remix example

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
In the remix example we replace insertion point with a new one with the cached styles:
const markup = html.replace(
    /<meta(\s)*name="emotion-insertion-point"(\s)*content="emotion-insertion-point"(\s)*\/>/,
    `<meta name="emotion-insertion-point" content="emotion-insertion-point"/>${stylesHTML}`,
  );

but this is not being applied because the emotion-insertion-point is not created
I solved this by injecting the styles right after the title
but this is an issue in the example.
this causes a flesh in the page when refreshing the page (easy to see on buttons)
or am I missing something
Expected behavior 🤔
consistent style when refreshing the page
Steps to reproduce 🕹
Steps:

implement the remix example
render a button to the page

Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
 System:
    OS: macOS 12.2.1
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: Not Found
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
  Browsers:
    Chrome: 103.0.5060.53
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.3
  npmPackages:
    @emotion/react: ^11.9.3 => 11.9.3
    @emotion/styled: ^11.9.3 => 11.9.3
    @mui/base:  5.0.0-alpha.86
    @mui/icons-material: ^5.8.4 => 5.8.4
    @mui/material: ^5.8.5 => 5.8.5
    @mui/private-theming:  5.8.4
    @mui/styled-engine:  5.8.0
    @mui/system:  5.8.5
    @mui/types:  7.1.4
    @mui/utils:  5.8.4
    @types/react: ^17.0.45 => 17.0.47
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    typescript: ^4.6.4 => 4.7.4

my mistake,
missed this line in root.tsx
 <meta name="emotion-insertion-point" content="emotion-insertion-point" />


Prototype - Storing the Hermes iOS tarballs on Maven.

Summary
I'm not going to merge this as it is, this is mostly for discussions with the community for the sake of
RFC #508 https://github.com/react-native-community/discussions-and-proposals/pull/508
Here I'm setting up a :ReactAndroid:external-artifacts Gradle module which is responsible of
uploading arbitrary artifacts to Maven (Central or any other repo).
In this specific example I'm uploading the Hermes iOS tarball which is 500+ Mb.
In this module I've configured the auth with Sonatype for publishing and the GPG signing of the artifacts.
Changelog
[Internal] [Changed] - Store the iOS Hermes tarball on Maven
Testing
I've tested this in a couple of ways:
Maven Local

Added the tarball in the expected location: ReactAndroid/external-artifacts/artifacts/hermes-ios.tar.gz
Publish to Maven Local with: ./gradlew :ReactAndroid:external-artifacts:publishToMavenLocal
Verify that the artifacts are there:

$ tree ~/.m2/repository/com/facebook/react/react-native-artifacts

/Users/ncor/.m2/repository/com/facebook/react/react-native-artifacts
├── 0.0.1
│   ├── react-native-artifacts-0.0.1-hermes-ios.gz
│   ├── react-native-artifacts-0.0.1-hermes-ios.gz.asc
│   ├── react-native-artifacts-0.0.1.pom
│   └── react-native-artifacts-0.0.1.pom.asc
└── maven-metadata-local.xml

Maven Central Staging Repository
I've pushed to a staging repository the tarball (so it's not released yet). It's currently available on:
https://oss.sonatype.org/service/local/repositories/comfacebook-3529/content/com/facebook/react/react-native-artifacts/0.0.1/react-native-artifacts-0.0.1-hermes-ios.tar.gz
(Link will expire in ~7 days).
If we were to release this to Maven Central, the final URL would be:
https://repo1.maven.org/maven2/com/facebook/react/react-native-artifacts/0.0.1/react-native-artifacts-0.0.1-hermes-ios.tar.gz
To publish you need to:

Make sure you have a Sonatype account which has push right on the com.facebook group
Make sure you have a GPG key + passphrase
Set the Gradle properties using environment variables as follows (or using the gradle.properties file):

export ORG_GRADLE_PROJECT_SONATYPE_USERNAME=<sonatype-username>
export ORG_GRADLE_PROJECT_SONATYPE_PASSWORD=<sonatype-password>
export ORG_GRADLE_PROJECT_SIGNING_KEY=$(cat ~/your-armored-private-key.pgp)
export ORG_GRADLE_PROJECT_SIGNING_PWD=<gpg-passphrase>

Publish with ./gradlew :ReactAndroid:external-artifacts:publishAllPublicationsToMavenCentralRepository

More on this is described here: https://github.com/cortinico/kotlin-android-template#to-maven-central
Differential Revision: D39886167

Closing as this was a prototype and it's effectively implemented


Text elements in World space aren't transparent from certain angles and distances

https://playcanvas.com/editor/scene/1130143
Zoom out at an angle and the 'background' of each glyph starts to be more visible. Related to https://forum.playcanvas.com/t/text-node-visible-through-meshes/19445/6 ?

ok, we use MIPMAP_LINEAR for texture filtering, but don't generate mipmaps for texture atlases

using FILTER_LINEAR and using discard in the fragment shader fixes the issue:

Another developer with a similar issue: https://forum.playcanvas.com/t/3d-text-blur-problem/22877


GitHub Actions: implement automation for unassigning abandoned issues

Overview
Implement a GitHub Action that automatically unassigns contributors from an issue after days of inactivity.
Starting out, we will do 3 months of inactivity, but will adjust as needed.
Action Items

[x] Draft a helpful message tagging the assignee.
[x] Design plan for the GitHub Action, document in internal docs.
[x] Write a new workflow and submit a PR.

Resources
https://docs.github.com/en/actions/using-workflows/creating-starter-workflows-for-your-organization
https://docs.github.com/en/actions/managing-issues-and-pull-requests/using-github-actions-for-project-management
https://github.com/marketplace/actions/unassign-contributor-after-days-of-inactivity
https://github.com/actions/stale
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows

Done: https://github.com/chaynHQ/bloom-frontend/commit/370401281c25db3e5cc50cc8f0d4fc83841dbc6b


Switch visualizations from Tableau to Periscope

Description
We are switching away from Tableau to use Periscope. Changes include:

Multiple dashboards, showing appropriate one based on viewport. Periscope dashboards are not responsive so we need a special mobile dashboard
Content changes throughout the site
Removing unnecessary "Researcher" section
Dynamic iframe height - previously the height was set to a static height, but the periscope iframe will update our page with the final rendered height so we can set that as the height
Update fallback message count to keep it semi-current

@clottman ready for re-review


Build fails when comiled with AOT flag

When I try to run the build with ng build --prod --aot I got errors. I have similar situation with other Angular2 package, fortunately I found AOT ready fork of it. Could you point me where I can find AOT ready version of the ng2-ace-editor?

Hi,
I don't know what is AOT.
But i think your problem has been solve here https://github.com/fxmontigny/ng2-ace-editor/pull/25

Ahead of time compilation
https://angular.io/docs/ts/latest/cookbook/aot-compiler.html

AOT means 'Ahead of Time' compilation, but I already switched to another code editor. Anyway thanks for the info.

@imjustd Would you mind sharing which editor did you choose and why did you prefer it to ACE?

I started using the Codemirror  editor, the particular reason I switched was the ACE's lack of AOT support. I'm developing production-ready platform, so I need all packages be ready for AOT compilation. Codemirror has quite a lot of plugins and options, I think it's similar to ACE editor, but if you implement AOT in ACE, maybe I'll switch again because I really like ACE and it's silly to not use it because of this, in my opinion, small issue.

jsfiddle is using Codemirror, plunker is using Ace :)
I've always had troubles trying to set-up more advanced options in codemirror - like error hints, code completion, etc.
Btw, I'm also just a user of this project.

I spent a couple of hours before I set-up advanced options in Codemirror, but once done, everything works fine.


ARM64 can make use of fused-multiply accumulate

The ARM implementation currently uses vmla_f32 /  vmlaq_f32 (Multiply-accumulate) and vmls_f32 / vmlsq_f32 (Multiply-subtract) which are supported on ARMv7.
For ARM64 which requires ARMv8, I can make use of vfma_f32 / vfmaq_f32 (Fused-multiply-accumulate) and vfms_f32 / vfmsq_f32 (Fused-multiply-subtract) for better precision

One possibility would be to use a macro to make it use the fused version in all cases where I currently use Multiply-accumulate / Multiply-subtract when building for ARM64. A more conservative option would be to just update the functions XMVectorMultiplyAdd and XMVectorNegativeMultiplySubtract to use the fused versions on ARM64.

This is in DirectXMath 3.12


Build

Hi,
Sorry, am not a java person. How do you make/compile the project. Suppose, I wish to make small change to source code, and compile in command line linux/unix- What compile tools do  you need, and what is the full command that will produce studio.jar.
thank you.

Hi,
The latest master is built with gradle. Gradle installation instruction could be taken from here:
https://gradle.org/install/
The command to build from the root folder (from the folder with build.gradle):
gradle build.
When it is done, you should have build/distributions/kdbStudio.tar with executable and libs. Extract the archive and execute kdbStudio. Alternatively you can execute from gradle:
gradle run

Thanks.

I tried to pack all classes from jars into a studio.jar, but it didn't work. I didn't look into this with details. On the another hand, I think it is strange to pack third party libraries into one big jar together.

No problem. although, I don't know why this is strange. isn't it the standard way to run studio: java -jar studio.jar? A simple, convenient interface to one jar- rather than setting classpath etc.
Anyway, no worries.


Suggestion: Remove unnecessary files when publishing to the gallery

I was packaging ImportExcel for use in a pipeline and I found that out of the total size of 26MB installed, 22.5MB are images.
I can understand the usefulness of having the README with all of the referenced images, but it seems wasteful considering how easy it is to find the GitHub repository. I would consider removing all unnecessary files when publishing the package.
It looks like it might be as easy as updating the FileList in the .psd1 here. However, I have additional files in the installed module beyond what are listed in the FileList of the .psd1. So maybe not, but either way I'd be happy to be the one to dig into this more if this was of interest.
As a workaround for my use case, I'm just manually deleting the files and re-packaging it. Works just fine for me.

Thanks @ThePoShWolf good point, early on it made sense, not it should be pruned. I'll take a look. I think it is my side when I prep it to be published to the gallery, need to strip that beforehand.

Awesome! I'm looking forward to the new version getting published. Thank you! 😄

I will publish it soon. After, let me know if additional things ought to be excluded.

Published - https://github.com/dfinke/ImportExcel/releases/tag/v7.5.3


How to change cookies secure setting?

I can use cookieDomainRewrite and cookiePathRewrite to rewrite cookies, but there are more other settings in cookies, such as secure and SameSite, as server is https://www.something.com, it set cookies with secure, but my localhost is http://localhost:8080, so if write cookies with secure switch on, Chrome will refuse to set.
Can this flag be changed by this middleware? Thanks!

Yes, an option-flag would be convenient.
But for know a little hack:
    proxy: {
      '/login': {
        onProxyRes(proxyRes, req, res) {
          if (proxyRes.headers['set-cookie']) {
            proxyRes.headers['set-cookie'][0] = proxyRes.headers['set-cookie'][0].replace('; Secure', '');
          }
        },
      },


Change ParametricMap to use new VOILUTTransformation class

Unify the API of the ParametricMap class with the presentation state SOPClasses by having the user provide a VOILUTTransformation object to the constructor. This is a backward incompatible change

What do you think about adding thevoi_lut_transformation: Optional[VOILUTTransformation] parameter and including a deprecation warning for window_width and window_center? That would make it easier to deprecate the parameters in a future version.


Implement simplex solver

Speed up RayCast, PointTest and SweepTest

i love it

Unfortunately not stable enough.

:(


Bug UI: Cycle Progress Bar is Stuck

Progress-Bar is not moving as expected, seems to be stuck.
Private Zenhub Image

Potentially related and fixed by https://github.com/encointer/encointer-wallet-flutter/pull/1533, but not sure. I will stay vigilant.

I could not observe this again, and after changing all the getters to dart, this might be resolved.


Allow for easier routing between http server and multiple grpc servers

I have two gRPC services in my server, my main server and the health check. I'm also adding a HTTP endpoint via warp. So I tried to do this and found things didn't quite work. Assuming I'm not doing anything egregiously wrong, there might be some changes to tonic-health and #272 or an equivalent may be required.
I've created an example repository here: https://github.com/xd009642/tonic-route-warp-example
In the mean time I'll start hosting two servers listening at different ports but it would be nice if this would work :smile:

Yeah, that PR is related we just need to find a way to return an easy to use service from the router.

is there anything stopping #272 being merged? I saw there was one comment adding something but otherwise (at least to my untrained) eyes it looked pretty solid

@xd009642 we just need to address https://github.com/hyperium/tonic/pull/272#issuecomment-612143535

I believe this was resolved on https://github.com/hyperium/tonic/pull/419. Please re-open if there are further questions.


PMID:20410137

irregular ascospore cell wall
thick ascospore cell wall
normal prospore membrane formation
fragmented DNA during meiotic cell cycle
decreased protein degradation during sporulation
normal RNA level during sporulation
prolonged protein localization to prospore membrane
increased prospore membrane formation

irregular ascospore wall FYPO:0004925
increased ascospore wall thickness FYPO:0004926
normal prospore membrane formation FYPO:0004927
fragmented DNA during vegetative growth FYPO:0004928
fragmented DNA during meiotic cell cycle FYPO:0004929
decreased protein degradation during sporulation FYPO:0004930
normal RNA level during sporulation FYPO:0004931
increased duration of protein localization to prospore membrane FYPO:0004932
increased prospore membrane formation FYPO:0004933

edit file: fedad3dc05dbea79bf554eff4835023ad8a5ab61
release: 154d4089fe3156b9a93a5cd829b2c555ac49371b


Function Signatures are Non-Standard

I was hoping to use Bluebird to promisify certain library functions by found an issue where in some functions the final argument doesn't conform to the node convention of lastArg = callback(err,data)

@iChip Yes, I agree with this. See #50 and #53. Basically, we need to figure out a way to (a) extend the arguments in a backwards compatible way while keeping the callback the last argument (probably using a jQuery-like hash as the first argument) and (b) migrate to that pattern.
Happy to accept pull requests and open to specific suggestions.

Duplicate of #53


(kubescape) Add Kubescape package

Description

This migrates the kubescape package to the community packages GitHub repository.
Motivation and Context
https://github.com/kubescape/kubescape
kubescape is a CNCF sandbox project, this allows kubescape package to upgrade automatically.

How Has this Been Tested?

Tested installing the package

Types of changes

[ ] 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)
[X] Migrated package (a package has been migrated from another repository)

Checklist:

[X] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[X] I have updated the package description and it is less than 4000 characters.
[X] All files are up to date with the latest Contributing Guidelines
[X] The added/modified package passed install/uninstall in the chocolatey test environment.
[X] The changes only affect a single package (not including meta package).

Original Location

Original Repository
[X] https://github.com/chocolatey-community/chocolatey-packages/issues/2190
[X] The migration guidelines have been followed

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

I'm sorry I didn't realize that I submitted the wrong branch until just now. I just corrected that and sorry for the trouble.

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

@HollowMan6 this is still coming from your master branch. Can you fix that?

@HollowMan6 this is still coming from your master branch. Can you fix that?

There's no way for me to change my branch of the PR, so I have already changed the content of it.

We can't accept the PR from your master branch. If you are unable to change it, I'll go ahead and close this so  you can raise a new PR from a branch.


ship with bundled/embedded stargz snapshotter

Is your feature request related to a problem? Please describe.
With the advent of https://github.com/containerd/stargz-snapshotter we should consider bundling it as part of k3s.
Describe the solution you'd like
Since we're building containerd we should build this as a native plugin. Otherwise k3s would need to supervise the stargz snapshotter as a separate process prior to spinning up containerd.
Describe alternatives you've considered
n/a
Additional context

https://medium.com/nttlabs/startup-containers-in-lightning-speed-with-lazy-image-distribution-on-containerd-243d94522361

cc @ktock

What's the lift on publishing stargz-compatible image layers? Is there an easy way to pull them in place of the default images, or do you need to specify an alternative image?

eStargz-converted image needs to be specified instead of the original image. But this image is backward-compatible to OCI so still pullable from lazy-pulling-agnostic nodes.
If you don't want to change the image tag config, I haven't tried but converting images and storing them to the mirrored registry with the same tags might work?

nerdctl provides Docker-like CLI for converting Docker images into eStargz images, so maybe it's also worth including?
$ nerdctl build -t example.com/foo .
$ nerdctl image convert --estargz --oci example.com/foo example.com/foo:estargz
$ nerdctl push example.com/foo:estargz

https://github.com/AkihiroSuda/nerdctl/blob/master/docs/stargz.md

The k3c project seems rebooted recently, and I assume it is going to be embedded in k3s. So probably we should port over nerdctl image convert to k3c image convert.

Opened a PR on https://github.com/k3s-io/k3s/pull/2936.

Can we move this forward?

Thank you all for the review and the discussion!

Validated using master commit eda65b19d9893a033c681ab5b0045d7a0e29d6cb

Compared the performance between stargz and the default of overlayfs and saw increased performance times when using stargz
Saw containerd configuration in the config.toml set appropriately
Ran a modified version of the test included in the PR and saw the images were the remote snapshots


feat: implement shuffling cache

Motivation

To verify attestations, we actually don't need the state but only shuffling
In case the head state of attestation is not available, lodestar may have to do a regen, or not able to validate attestation

Description

This PR started with #6008 and addressed comments from there
Implement a ShufflingCache, add new shufflings when we pass epoch transitions

Every shuffling is keyed by epoch and dependent root which is the last block root of the previous 2 epochs

Support regen state which is tracked as a promise inside ShufflingCache

this is not needed with the default chain option of maxSkippedSlot = 32, handle just in case
if there are multiple requests of the same shuffling, only do the regen once
also support 1 regen at a time, bound inside ShufflingCache

this is a prerequisite for #6008
Closes https://github.com/ChainSafe/lodestar/issues/2848

after 4 days testing, metrics are the same to unstable in term of:

memory
attestation job time + job wait time
forwarded messages in beacon_attestation message
subnet mesh peers

ready for review @dapplion

after 4 days testing, metrics are the same to unstable in term of:

memory
attestation job time + job wait time
forwarded messages in beacon_attestation message
subnet mesh peers

ready for review @dapplion

adding metrics here:

Shuffling cache size is consistently 4 or 5

No promise at all because it always hit the cache

Miss metric hasn't populated ever

No regen data which is great

also beacon_attestation job time is the same before/after I deploy the branch

:tada: This PR is included in v1.13.0 :tada:


AppImage failed on Ubuntu v24.04

I tried to execute Numara-5.5.0-x86_64.AppImage on Ubuntu v24.04, failed and got this:
[17626:1019/185307.313317:FATAL:setuid_sandbox_host.cc(163)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /tmp/.mount_Numara2qLsnz/chrome-sandbox is owned by root and has mode 4755.
Trace/breakpoint trap (core dumped)
I searched and found this:
https://github.com/electron/electron/issues/42510

Hi @yardern,
This indeed seems to be a known issue with Electron.  Not sure if there is an easy workaround at the moment.  Is using the DEB package an option for you?

I just installed Numara-5.5.1-amd64.deb, got same problem:
[4227:1021/215301.434906:FATAL:setuid_sandbox_host.cc(163)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /opt/Numara/chrome-sandbox is owned by root and has mode 4755.
Trace/breakpoint trap (core dumped)
Hope Electron will solve it. Thanks.

Joplin uses Electron and its AppImage is running ok in Ubuntu 24.04.
Unfortunatly I know nothing about the programming.

This seems to be known issue with Electron.  Afer doing some reseach I see lots of other repos having the same problem.  I am monitoring this (and https://github.com/electron/electron/issues/42510 as you posted) closely but I don't have a good solution at the moment.


OneDrive Delta API throwing 401 error: request url exceeds maxUrlLength value

I am using OneDrive's delta API to get change list. My first call succeeds. But when I try to fetch next batch results using<EMAIL_ADDRESS>I get following error message. 'The length of the URL for this request exceeds the configured maxUrlLength value.' with 401 HTTP status code.
My Request:
https://druvainternal-my.sharepoint.com/personal
/{email_id}/_api/v2.0/drives/{drive_id}/root/view.delta(token='1;%234;%231;3;78895df2-a9a4-4932-9983-6683629dc4a9;636263623679200000;96128348;%23Paged_CT=1;3;78895df2-a9a4-4932-9983-6683629dc4a9;636263587306070000;96123061')

Total length: 367 characters
Emailid: 37 character length
Drive_id: 67 character length
I have removed email_id and drive_id for security reasons.
As this URL is returned by API, I can't control its length. Because of this I am not able to fetch next batch of changes.
Would it be a good idea to make this API as POST so that token can be moved to body as done in Sharepoint APIs?

The issue still persists, but the length at which it kicks in has been increased.

We've made a change that should hopefully bypass this issue, but you'll probably need to restart the sync sessions to get a newer token.


migrate ensembl version from rep transcript field to ensembl version field

Hi @acoffman
We talked about this in the last meeting but here is an issue for tracking. Can we migrate ensembl version information, now stored as part of the representative transcript field to the new dedicated ensembl version field.
Right now we have followed the convention of storing it in the representative transcript field like this: ENST00000275493.2 (v75).  We want the 75 part to go to the Ensembl version field and drop the  (v75) from the representative transcript field.

Definitely! This shouldn't be difficult to do - just a quick regex.

This is complete and in production!


Question: Possible to inline CSS similar to Tailwind?

Hey everyone,
Something that I've been doing with Tailwind is inline-ing @tailwind directives in a <style> tag for my SPA. This allows all CSS for the application to live here, and no CSS files are generated.
<!-- In index.html -->
    <style type="text/css">
      @tailwind base;
      @tailwind components;
      @tailwind utilities;
    </style>

I was wondering if something like this was possible with Uno? I couldn't find an example - most want me to import uno in my app's entry point, but this produces a CSS file. I'd like to avoid installing an inline-ing package if possible.
Thanks!! 😊

Also related: https://github.com/vitejs/vite/pull/8400


Disable more view popup?

Version
latest

Test Environment
Ubuntu, newst TUI Version, Symfony
Current Behavior

Since I can not find any option to customize the 'more view popup' i tried to disable it in order to create my own popup and therefore my idea was a bootstrap modal when clicking the 'x more' title.
Works fine but I can not manage to disable the default popup, is there any possiblity to do so?
Expected Behavior

A option to disable the more view popup same as for the other popups.

@Bxnnet
Unfortunately, we don't have an option to disable the default more view popup at this moment.
You can fork the repo and remove this code(and related code blocks) to disable the more view popup.

@adhrinae Hey, thanks for the answer and solution idea.

Yeah, That's odd.
When the popup shows, the popup's position is affected by the browser's scroll position.
https://github.com/nhn/tui.calendar/blob/main/apps/calendar/src/components/dayGridMonth/gridCell.tsx#L124
Perhaps this calculation brings misbehavior to your application. I can only assume it from your information.

Alright the problem seems to be that I render the calendar inside a bootstrap card deck. (Left side toolbar for calendar navigation, visibility, ... and right side the calendar view). As soon as I move the calendar container to an empty space (where it renders in full width), the popup location gets calculated correctly.
Thanks for helping me finding the issue source but it should be possible, to use the calendar in my wanted way, shouldnt it?
Here is my full HTML in case you are willing to continue assisting me:
https://pastebin.com/KseNa5NL

it should be possible, to use the calendar in my wanted way, shouldnt it?

I agree. We'll note it as a future requirement but please understand that it won't be added soon since the team behind TOAST UI is taking care of various projects(including internal projects).


enable dependabot for opensearch repositories

Is your feature request related to a problem? Please describe.
i noticed that dependency updates seem to be done manually (see e.g. #661). this means that there's no guarantee that all dependencies are up-to-date and do not contain any known vulnerabilities.
some users (esp. corporate users) will run 3rd party artifact scanning on all software which they run/ship and will block it if it contains known vulnerabilities (regardless of whether there's a known way to exploit it with this software or it's just part of a library which is being pulled in but that part isn't even being used).
keeping dependencies updated with an automated process will prevent unnecessary last-minute manual updates.
Describe the solution you'd like
github has a feature called dependabot which automatically keeps track of the dependencies in a repository and creates pull request to update the dependencies when newer versions become available.
note: this doesn't just apply to this repo here but instead to all opensearch-project repos.
Describe alternatives you've considered
manual updates don't scale (things are being overlooked, etc.).
of course dependabot (and any other such feature) only works if dependencies make use of semver.
for major releases of dependencies the change must still be done manually since more adaptations will be needed (but who knows, maybe one day this can be automated away as well? 😃)
Additional context
see the dependabot documentation

This is a really good point.  Let me turn it around and ask -- is there any reason why we wouldn't turn on updating?  @hyandell?

Please turn them on :)

i think this needs to be looked at first by somebody with access to the repository settings.
based on the documentation this should be enabled by default if the settings are correct: https://docs.github.com/en/code-security/supply-chain-security/configuring-dependabot-security-updates#supported-repositories

GitHub automatically enables Dependabot security updates for every repository that meets these prerequisites.
[..]

it can still be enabled manually by providing a .github/dependabot.yml file, but it seems that this shouldn't be necessary

@CEHENKLE: are you sure that this worked? i haven't seen a single PR from dependabot being opened on this repo and can't imagine that there wouldn't be any dependency which it'd want to update (unless you're all super fast in updating to the latest versions of all dependencies at all times when they come out?)

@rursprung Hm, that's weird.  Let me take a look at that today.  Thanks for the ping!

Dependabot is enabled on OpenSearch and Dashboards repos and I could see dependabot based alerts and PRs on OpenSearch-Dashboards repository (see here), however, I don't know yet why we don't have such alerts on OpenSearch.
I was thinking it could be due to missing dependabot configuration file, as described here  - "You enable Dependabot version updates by checking a configuration file into your repository." (reference) but then I didn't see any such configuration file in Dashboards repo as well. Will try to dig more on this later


Track worker's user count.

If a worker is shared, then without tracking players it's impossible to tell when it's free, so we add a simple (not thread safe, meh) user counter.

oh don't know why I did like that : D This would fix the issue.
Don't know if IsAvailability is necessary because I have this cleanWorker based on room count : https://github.com/giongto35/cloud-game/blob/747d0d702ca19e9e82c9a8e25d5cd841f45feca3/pkg/coordinator/handlers.go#L376


Corrected typo in getting-started.html

Section "Importing Sass". Changed "custom" to "customize".

LOL.

@picturepan2 :-) I couldn't let this pass once I read it. Don't laugh ok :P


update to standard page title

Replace "Initial" with "Phase One" in the title of the page.

@duboisp please merge and squash commits.


Recommended way to negate points

I know that the underlying secp256k1 library exposes a pubkey_negate :
https://github.com/bitcoin-core/secp256k1/blob/6f6cab9989a4d3f4a28e3cdbfacc4e3e1e55c843/include/secp256k1.h#L668
but at first thought, it's not that surprising that this isn't exposed in this binding, since we have the most fundamental operations: combine (add pubkeys together) and tweak_mul, tweak_add.
Still, it leaves me a little uncertain what the best way to do -P is. In the now "legacy" compressed encoding we can flip 02/03 starting byte. That  feels like an icky way to do it (even if it wasn't the case that we're now tending to use the new Schnorr style 32 byte pubkeys); mathematical operations on keys shouldn't be executed by manipulating their encodings.
Another obvious thought is: use scalar mult (so tweak_mul) with the value "-1", but that is N-1 where N is the group order and it also feels very bad to be introducing that kind of calculation outside the library/binding.
Am I missing an obvious way to do it?

Ran into the same issue, negation, subtraction was easy enough.
<EMAIL_ADDRESS>from secp256k1 import PrivateKey, PublicKey

class PublicKeyExt(PublicKey):

    def __neg__(self):
        serialized = self.serialize()
        first_byte, remainder = serialized[:1], serialized[1:]
        # flip odd/even byte
        first_byte = {b"\x03": b"\x02", b"\x02": b"\x03"}[first_byte]
        return PublicKey(first_byte + remainder, raw=True)

 PublicKey.__neg__ = PublicKeyExt.__neg__

 class PrivateKeyExt(PrivateKey):

    def __neg__(self):
      order = 0xfffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364141 #order of secp256k1 group
      neg_num=_int_to_bytes(order-_bytes_to_int(self.private_key, 'big'), 32, 'big')
      return PrivateKey(neg_num, raw=True)

 PrivateKey.__neg__ = PrivateKeyExt.__neg__

Now I need the inverse modulo (secp256k1_scalar_inverse?)...


NPE in ExceptionCodec.

Here is the full stack trace.
java.lang.NullPointerException
org.ehcache.clustered.common.internal.messages.ExceptionCodec.decode(ExceptionCodec.java:78)
org.ehcache.clustered.common.internal.messages.ResponseCodec.decode(ResponseCodec.java:201)
org.ehcache.clustered.common.internal.messages.EhcacheCodec.decodeResponse(EhcacheCodec.java:117)
org.ehcache.clustered.common.internal.messages.EhcacheCodec.decodeResponse(EhcacheCodec.java:37)
at com.tc.object.EntityClientEndpointImpl$InvocationBuilderImpl$1.getWithTimeout(EntityClientEndpointImpl.java:174)
at com.tc.object.EntityClientEndpointImpl$InvocationBuilderImpl$1.getWithTimeout(EntityClientEndpointImpl.java:156)
at org.ehcache.clustered.client.internal.EhcacheClientEntity.waitFor(EhcacheClientEntity.java:356)
at org.ehcache.clustered.client.internal.EhcacheClientEntity.invokeInternal(EhcacheClientEntity.java:320)
at org.ehcache.clustered.client.internal.EhcacheClientEntity.configure(EhcacheClientEntity.java:246)
at org.ehcache.clustered.client.internal.EhcacheClientEntityFactory.create(EhcacheClientEntityFactory.java:116)
at org.ehcache.clustered.client.internal.service.DefaultClusteringService.autoCreateEntity(DefaultClusteringService.java:196)
at org.ehcache.clustered.client.internal.service.DefaultClusteringService.start(DefaultClusteringService.java:144)
at org.ehcache.core.internal.service.ServiceLocator.startAllServices(ServiceLocator.java:118)
at org.ehcache.core.EhcacheManager.init(EhcacheManager.java:560)
... 28 more

@esebasti Can you give the context in which you had this problem? There is effectively a missing null check in decode, but I am curious to understand what caused an exception to leave the server without a stack trace.

Well, from looking deeper at the code, that exception makes no sense. So really interested in understanding under which conditions you saw that!

I saw that during load test in one of big memory machines which seems failing due to invalid offheap resource configurations. Trying to gather more details on it.

It looks like I was running new client against old kit. Closing it.


Switchable map layers

The current problem of osm-carto is that it is a map for everything and nothing at the same time. It is supposed to serve mappers to check their mapping progress, but is also supposed to be for general public. It is just filled with too many different objects that make it too dense.
One way to go around is to have several separate tiles styles: 1. for public. 2. for mappers. 3. public transport ,...
-OR-
I was thinking that since the actual tile rendering is happening on the user side to have some sort of switches that would turn on/off displaying of some objects. So there will be one style for the tiles but the user would choose which parts of that styles would be displayed. For example there is no need to have the default of displaying of manhole covers and utility poles, but with a switch one can display them if needed.
Issues:
I don't know if this is technologically possible
Probably depends on solving of #9

There are a few distinct issues here that you're raising.

osm-carto is too information dense.  This is something I wholeheartedly agree with.  Take for example, this screenshot from the Boston area at raster zoom 15.  It does not make sense to draw buildings at this zoom level -- it adds clutter and reduces the readability of the street network.  In the upper right corner there is a park, and several athletic fields are rendered.  However, at this zoom level, the more important thing to see would be the park's name.  The individual features within the park should be suppressed until a closer zoom, and this would increase readability.  Similarly, a name on the school grounds would be far more useful to render than the building.  At this zoom, the important features are the road network, road names, and major points of interest.

Ideally I would like Americana to have a "Goldilocks" level of information density: not too dense, but not too sparse, and that will take careful planning.

Changing the map style on the fly.  This is 100% doable with vector tiles and is one of the major selling points for it.  In fact, here's an example of dynamically changing a rendered map style on the fly.  The code for how to do this is not hard -- the hard part is the design work of coming up with what options we should offer to the map user!  I am thinking that we could use the "wiki" section here in GitHub to start working out those design choices.  Design work is hard, and it's not really my skill set.

Custom layers currently we're based on OpenMapTiles, but that project may not want to have all of the things that we might want to have in our style (for example, manhole covers and utility poles).  If OpenMapTiles is amenable to adding new data elements, we can contribute them to OpenMapTiles.  For anything that's bespoke americana and not of interest to global users, we will need to create custom layers for that.  So far, OpenMapTiles has welcomed our contributions (mostly geared towards highway shields).  However, most of what we will want to display is probably already available, and we can just create new issues for features that are not currently present in the data set.

Good point. If OpenMapTiles are so open to contributions, then the custom server is a low priority.


Diagonal Wall and Ghost Block above = Lara is going through when she rolls toward the wall

https://github.com/MontyTRC89/TombEngine/assets/145125826/393983eb-6342-4e48-b3b2-ca0023592618
When performing a roll, Lara is clipping through the diagonal wall. On the room above, on the same floor coordinate there is a ghost block. It seems that it does not occur when the wall are straight.
This happens for all four directions.
Here a screenshot of the setup

a PRJ to test:
DiagWallGhostBlock.zip

Lara is also clipping and going on the upper room if she jumps and grab toward the diagonal wall

TE floordata compiler bug. Fixed.


Question

Is this repository still active? what is its relation to https://github.com/square/go-jose ?

It's stable. And there's no relation that I know of.

Thanks


Ros2 devel

Description of changes:
Migration ROS1 ObjectTracker app to ros2:

Add turtlebot3 dependencies repos to .rosinstall
Removed unnecessary CMakeLists.txt & Modified (add/remove) pkg in package.xml
Added setup.cfg & setup.py entrypoint
Added new ROS2 format python .launch.py launch files
Removed .travis.yml temporary to skip CI build

In the next PR please also include .travis.yml similar to this one if possible.

Sure, We will send .tavis with next PR.


Cookies file needs to be created programmatically

for others who may be looking on information related to this. it appears cookies.txt needs to be created in the statstat directory and chowned to the user running the script, or chmod 777 (insecure)


Allow runing github action with ok-to-test

What this PR does / why we need it
Github actions are not running for non-member contributors, unless a repo admin presses the "Approve and Run" button. The problem with that is that PRs are getting merged with no verification, causing leak of bugs into the codebase.
This PR copies a github action from the cluster-api repo, that approve running the github action by adding the ok-to-test label.
Source: https://github.com/kubernetes-sigs/cluster-api/blob/613a37e0183e818403ff086165bd619393b34178/.github/workflows/pr-gh-workflow-approve.yaml
Release notes:

None

One question (and remove hold whenever you're satisfied with the answer). A non-member can't somehow trigger the ok-to-test label, right?

yes, tested it.
/unhold

/override unit_test
wrong requirement name. See https://github.com/kubernetes/test-infra/pull/30577


Need clarification on set-AzDataLakeGen2ItemAclObject and Update-AzDataLakeGen2Item

We are trying to set the ACL for a user on existing ADLS folders. These folders already have a number of ACL defined for other users.
We use the following code to set the Default permissions for the user $id:
$newacl = set-AzDataLakeGen2ItemAclObject -AccessControlType $ACLType -EntityId $id -Permission $perm -DefaultScope
Update-AzDataLakeGen2Item -Context $ctx -FileSystem $filesystemName -path $dirname -Acl $newacl
The above deletes the Default permissions of all existing entities in the same folder! The Access permission of $id is untouched.
We use the following code to set the Access permissions for the user $id:
$newacl = set-AzDataLakeGen2ItemAclObject -AccessControlType $ACLType -EntityId $id -Permission $perm
Update-AzDataLakeGen2Item -Context $ctx -FileSystem $filesystemName -path $dirname -Acl $newacl
The above deletes the permissions (Default and Access) of all existing entities in the same folder! The Default permission of $id is also deleted.
a) This seems inconsistent.
b) If we use the following code to apply the same ACL to all the subfolders and files, it will erase existing ACL in those folders.
$items = Get-AzDataLakeGen2ChildItem -Context $ctx -FileSystem $filesystemName -Recurse -ContinuationToken $Token
$items | Update-AzDataLakeGen2Item -Acl $acl

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

ID: 562aa838-243f-bed8-c3cb-9164af5efe71
Version Independent ID: 369544f3-5260-3773-68f2-3479c6e9587e
Content: Azure Data Lake Storage Gen2 PowerShell for files & ACLs
Content Source: articles/storage/blobs/data-lake-storage-directory-file-acl-powershell.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@dckfnuance can you post a link to the doc you are following?

https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-directory-file-acl-powershell
David

From: TravisCragg-MSFT<EMAIL_ADDRESS>Sent: Wednesday, May 13, 2020 6:33:45 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Chan, David<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [MicrosoftDocs/azure-docs] Need clarification on set-AzDataLakeGen2ItemAclObject and Update-AzDataLakeGen2Item (#54804)
@dckfnuancehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dckfnuance&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=7x_MbGzqvQ0Bmapmx6CI3JqJEKHDJs-zWZOJy7ZSlvc&s=P7bWBmtVPFS-7Y6jJ0wkPqGkBnarN7S1f1Jsu1SLrPE&e= can you post a link to the doc you are following?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_MicrosoftDocs_azure-2Ddocs_issues_54804-23issuecomment-2D628280574&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=7x_MbGzqvQ0Bmapmx6CI3JqJEKHDJs-zWZOJy7ZSlvc&s=uOS0lmwcXATGN_PW6sXR_zFyvGqS8t0BvJMUeoDGOM0&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APGJNGVBICRZ7L2NIFGKDQLRRMN4TANCNFSM4NAEE2XA&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=7x_MbGzqvQ0Bmapmx6CI3JqJEKHDJs-zWZOJy7ZSlvc&s=7RTdXvRJpL__kknu_7e9AqLUvlxB5HGSbjknjFfkaZQ&e=.

@dckfnuance  There area few things going on here.
First I would like to clarify that Update-AzDataLakeGen2Item  works by overwriting/replacing the existing ACL properties.  It does not make incremental changes.
This means to make an incremental change, you need to:

Get the pre-existing properties
Add your own changes
Commit the change

The following excerpt is taken from Example 3 in the update-azdatalakegen2item doc
## Get the origin ACL
PS C:\> $acl = (Get-AzDataLakeGen2Item -FileSystem "filesystem1" -Path 'dir1/dir3/').ACL

# Update permission of a new ACL entry (if ACL entry with same AccessControlType/EntityId/DefaultScope not exist, will add a new ACL entry, else update permission of existing ACL entry)
PS C:\> $acl = Set-AzDataLakeGen2ItemAclObject -AccessControlType user -EntityId $id -Permission rw- -InputObject $acl

# set the new acl to the directory
PS C:\> update-AzDataLakeGen2Item -FileSystem "filesystem1" -Path 'dir1/dir3/' -ACL $acl

Another example found in similar MSDN thread
Please let me know if there are further queries.  I will be glad to help you.  Thank you.

What would be the fastest way to give someone access to a folder and have the permissions propagated to all existing sub-folders and files considering that the sub-folders can have different existing ACL properties? If we need to get each sub-folder's ACL and add to it individually, it will take a long time.

Any update on this?

@dckfnuance  sorry for the delay.  As to propagating the changes,
A tree-traversal algorithm would be an easy way to make the change.  The psuedo-code would look like:
acl-traversal ( current_item, new_user_perm):
    new_acl = get_acl(current_item) # Get-AzDataLakeGen2Item
    new_acl = modify_acl( new_acl, new_user_perm) # Set-AzDataLakeGen2ItemAclObject
    push_acl_change(current_item, new_acl) # update-AzDataLakeGen2Item
    if current_item.is_folder:
        for each child in get_child_items(current_item):
            acl-traversal( child, new_user_perm)

I hope this helps.  I will proceed to close the thread.  For further queries please, open a thread in Microsoft Q&A


Refactor docker pkg

What issue type does this pull request address? (keep at least one, remove the others)
/kind enhancement
What is this pull request for? Which issues does it resolve? (use resolves #<issue_number> if possible)
It refactors the docker package. The changes are very small.

Close in favor of #818


ci: codeql analysis

Add Github CodeQL security checks. Finds bugs like #21231 #21233

Hi @jfcg - can you provide an example of what this tool's output would look like? It looks like you should be able to change the on...push...branches filter to include your branch and push that commit to see it run.

Going to close this one out for now because of inactivity, but feel free to open a new PR in the future if you want to continue pursuing this!


Update convertNumber.js

Math-Based Conversions: The code now divides number by 1e12 (trillions), 1e9 (billions), etc., making it more consistent regardless of locale settings.
Precision Control with .toFixed(2): The .toFixed(2) ensures only two decimal points are shown. replace(/.?0+$/, "") removes trailing zeros for a cleaner look.
Simplified Logic: The code is simplified and should work efficiently with large numbers.

@vanshchauhan21  please once go through this pr


⚠ Bofrost.fr has degraded performance

In f881912, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 837 ms

Resolved: Bofrost.fr performance has improved in 30bae92 after 6 minutes.


variable renaming to highlight noah_owp versus noahmp; other misc. cl…

…eanup

this PR was missing consistent changes in namelist


Support for value_and_grad at the MLIR side (inside a jitting context)

Context: Lowering of value_and_grad to MLIR is now supported when @qjit is used.
Description of the Change: A new ValueAndGradOp was added and the existing BackpropOp was modified to return two results: values and gradients.
Benefits: Code reutilization.
TODO:

[ ] Add unit tests.
[ ] Test corner cases.
[ ] Update the changelog.md file

[sc-55116]

Closing this as value_and_grad is merged in by #804


Mix and match Acme challenges

As posted by @hosseinsh

Today, I learned about an important edge case that the current two plugin design doesn't support, namely, a SAN certificate request that requires mixing the validation methods. For instance, a wildcard SAN which would require DNS validation.
I am still figuring out if ACME, this case Let's Encrypt, supports this use-case (url).
My initial intuition is this should be supported since, for each domain, all types of validations are returned in the Authorization object.
It certainly gets more tricky with how to decide on the validation method per domain. I need to do more thinking about this, but one idea could be to differentiate based on the presence of wildcard domain. For instance, in a plugin design, we would branch off to http-01 validation, unless a wildcard domain is present. In this case, a hybrid approach is required, during which we first answer_challenge for all non-wild-card SANs, and then trigger the normal DNS flow for the remaining DNS authorization.

To make this possible it would be necessary to use a single AcmeIssuerPlugin which can handle all types of challenge, depending on what is offered by the Acme provider, and what is configured in Lemur.
Right now the proposed implementation in #3167 creates a new IssuerPlugin which only handles http challenges, but after the refactoring is done, it should be relatively easy to convert it into a single AcmeIssuerPlugin, which can handle both dns and http challenges.
To allow mix and matching challenges, we need a way to find the dns authority, and if none exist, the relevant destination. Therefore it might make sense, to have the destination mapping on the domain, and no longer on the authority, but this would mean, you either need to preconfigure the domains, or add the destination after creating the certificate.
It would also require some kind of deduplication on the domains, because right now a domain exists multiple times, once for each certificate.

I'll try to create an example as a basis for discussion which forces a mix and match, and try to demonstrate what I think would be a good way to handle it
The only configured dns provider is for the san.example.biz zone.
Certificate: san.example.biz

SAN: *.san.example.biz
SAN: second-sub.example.biz

Offered challenges (I'll ignore TLS since they are very uncommon)
san.example.biz

DNS
HTTP
*.san.example.biz
DNS
second-sub.example.biz
DNS
HTTP

domain/challenge mapping
san.example.biz -> can be answered using DNS -> prefer DNS
*.san.example.biz -> only DNS available -> DNS
second-sub.example.biz -> no DNS provider available -> must use HTTP

The domain/dns-provider mapping can be done easily, that is already implemented, but now we would need a way to get the destination to use for second-sub.example.biz.
It would also require to refactor the http challenge, into a celery task, or at least the order finalisation in cases where there are different challenge types used. (Which means, implementing pending_certificates for http challenge, as I avoided them until now)

While refactoring #3167 I realized that when you get the challenges the first time around, some of them could already be in the state 'valid/ready' since the ACME provider can cache the validation per account.
I think it might make sense, to check for that before starting the handle any domain validation tasks, and only do the domain validation where needed.

While refactoring #3167 I realized that when you get the challenges the first time around, some of them could already be in the state 'valid/ready' since the ACME provider can cache the validation per account.
I think it might make sense, to check for that before starting the handle any domain validation tasks, and only do the domain validation where needed.

This is now done in the HTTP-01 challenge, but would need to be adapted to the DNS challenge as well.


suite: Implement --sleep-before-teardown option

This is another try to merge the useful debugging feature which is supposed to be not breaking 'parallel' and 'sequential' tasks.
The sleep-before-teardown only works on the front line of tasks not going deep into parallel and sequential tasks and does not give a sleep within them correspondingly, allowing to finish them correctly before unwinding the main stack of tasks.
Usage hints:

option can be used with either teuthology-suite or teuthology-openstack
use teuthology-kill to cleanup the job or run to interrupt the sleep and free resources.

@susebot run deploy

retest this please

This is another try to merge the useful debugging feature which is supposed to be not breaking 'parallel' and 'sequential' tasks.
The sleep-before-teardown only works on the front line of tasks not going deep into parallel and sequential tasks and does not give a sleep within them correspondingly allowing to finish correctly, before unwinding the main task stack.

retest this please

retest this please

@susebot run deploy

:+1: to @vasukulkarni 's idea to have the feature warn the user.
In fact, it would be worth considering to raise an exception when scheduling a run with over, say, 10 jobs when this option is active.

+1 to @vasukulkarni 's idea to have the feature warn the user.
In fact, it would be worth considering to raise an exception when scheduling a run with over, say, 10 jobs when this option is active.

Wouldn't they be killed by the timeout?

Wouldn't they be killed by the timeout?

Which timeout do you have in mind here?

Wouldn't they be killed by the timeout?

Which timeout do you have in mind here?

Global job run execution time, don't remember the exact name.

Wouldn't they be killed by the timeout?

Which timeout do you have in mind here?

Global job run execution time, don't remember the exact name.

No the timeouts only apply to individual jobs; there's no total suite timeout limit.

@gregsfortytwo I think we need to be careful with this, because people will do:
teuthology-suite --suite rados --sleep-before-teardown 99999999

and lock up a large number of machines for a very long time

@gregsfortytwo I think we need to be careful with this, because people will do:
teuthology-suite --suite rados --sleep-before-teardown 99999999

and lock up a large number of machines for a very long time

Not sure that warning will help a much, since people will ignore it, because we have very noisy logs.

@kshtsk I think what @vasukulkarni (and certainly what I) had in mind was not to write a warning to the log, but rather interactively ask the user if he really wants to potentially lock up lots of machines for a long time.
The warning could translate the number of seconds given by the user into years, months, days, hours, minutes, seconds. So, for example, the user gives --sleep-before-teardown 99999999 and we ask "Do you really want to lock all machines needed for this run for Y years, M months, D days, H hours, M minutes and S seconds? (y/N)"
In addition to this, to be completely safe, once we know how many jobs are in the run, we can check if it is over a certain number (like 10) and raise an exception in that case, if "sleep before teardown" is in effect.
What do you think?

The problem is that we can't understand how much jobs will be run before they actually scheduled, this is by teuthology design. The corresponding code will be more complicated. As an option, what if to only allow use --sleep-before-teardown in combination with the --limit?

The warning could translate the number of seconds given by the user into years, months, days, hours, minutes, seconds. So, for example, the user gives --sleep-before-teardown 99999999 and we ask "Do you really want to lock all machines needed for this run for Y years, M months, D days, H hours, M minutes and S seconds? (y/N)"

Nevertheless I like questions like that :-)
I think this is not as good as it could, especially bad when you want to automate scheduling, example, from jenkins.

In addition to this, to be completely safe, once we know how many jobs are in the run, we can check if it is over a certain number (like 10) and raise an exception in that case, if "sleep before teardown" is in effect.
What do you think?

What about forcing --limit parameter to the 1, if --sleep-before-teardown is called? Or/and additionally have --limit-sleeping-jobs 1 which will override the --limit if its value greater?

I think this is not as good as it could, especially bad when you want to automate scheduling, example, from jenkins.

--non-interactive ?

How about:

if --limit not given explicitly, set it to 1
if --limit given explicitly and it's greater than 10, error

I think this is not as good as it could, especially bad when you want to automate scheduling, example, from jenkins.

--non-interactive ?

How about:
1. if `--limit` not given explicitly, set it to 1

2. if `--limit` given explicitly and it's greater than 10, error

I feel like okay, but why 10? What if I want to run 11? :-D

What about:

if --limit not given, then set it 1
if --limit is given more than 3, ask interactively for confirmation.

I think this is not as good as it could, especially bad when you want to automate scheduling, example, from jenkins.

--non-interactive ?

How about:
1. if `--limit` not given explicitly, set it to 1

2. if `--limit` given explicitly and it's greater than 10, error

I feel like okay, but why 10? What if I want to run 11? :-D

Personally I feel there is no use case to run and hold many jobs (since the purpose is to debug live) ?

I think this is not as good as it could, especially bad when you want to automate scheduling, example, from jenkins.

--non-interactive ?

We do not have such argument yet, do you mean add it withing this PR?

I think this is not as good as it could, especially bad when you want to automate scheduling, example, from jenkins.

--non-interactive ?

We do not have such argument yet, do you mean add it withing this PR?

Yes. It would enable us to implement the proposed interactive warning without causing problems for Jenkins.
This is a technique widely used by command-line tools.

@susebot run deploy

@susebot run deploy

@smithfarm @vasukulkarni I think I've addressed your requests, could you please take another look?

@gregsfortytwo @vasukulkarni after fixes applied to the patch your review is expectedly  disappeared, could you please take another look?

@susebot run deploy

@susebot run deploy

@smithfarm here you are, new push with your suggestions

@susebot run deploy

@susebot run deploy

okay, failure reason is unrelated to the change, it is because of slow osd on the openstack targets.


graphql-module accepting string typeDefs

To be able to remove the config typeDefsStyle, we should make sure that graphql-module accept string typeDefs and not gql only.

Reminder after https://github.com/jycouet/kitql/pull/272

I'll close this one


QuadTree and precision check use different X and Y semantics

In Quad Tree implementation, X and Y are coordinates of the top left corner of area. In precision intersection check, we assume that X and Y are the centre of the checking area. This difference leads to strange bugs like the one on the screenshot, where the second from the top marker doesn't prevent markers on the left from being shown.

Here is the solution: https://github.com/2gis/MarkerGeneralizationPlugin/pull/6


[x32/d3d9] 1.9 Blank Screen in Unreal Tournament 3

I get a blank screen when using the x32/d3d9.dll with Unreal Tournament 3 running on Windows 10.  The console works but backspacing text does not remove the characters leaving you with garbled text.  This is specifically with version 1.9.
I can confirm it works fine with 1.8.1, 1.7.3, 1.6.1 & 1.5.5 (I stopped here).
A diff of the log files from 1.8.1 and 1.9 shows these lines as the only significant differences (other than version number change):
1.9:
info:    VK_EXT_conservative_rasterization
info:    shaderStorageImageExtendedFormats      : 0
1.8.1:
[empty]
info:    shaderStorageImageExtendedFormats      : 1
Software information

Latest Unreal Tournament 3 version 2.1 (patch 5)

System information

GPU: Radeon RX Vega 56
Driver: 21.5.2
Wine version: N/A
DXVK version: 1.9

Log files

d3d9.log:
UT3_d3d9 (1.9).log
UT3_d3d9 (1.8.1).log

apitrace
https://drive.google.com/file/d/1cW4TwcgJ_n6VcyKvq5puqj702ZxgpyAP/view?usp=sharing

Does 1.8.1 work? This isn't really clear from the report.
shaderStorageImageExtendedFormats being 0 is expected since the feature isn't used for D3D9, and we (finally) disabled the feature for llvmpipe compatibility.

The issue specifically happens with the release 1.9 d3d9.dll though.  When I replace the d3d9.dll file with the one provided in apitrace the issue does not occur.
Does it still make sense to do the api trace in this case?

Yes.

The whole point of recording a trrace is to record what the game does so we can replay it with DXVK and (hopefully) see the breakage there.

Ok.  The instructions said to reproduce the issue which I can't so that's why I asked.
Updated the original post.

Works fine on RADV, not really sure what the proprietary driver is choking on.

Same problem as #2110, this is a driver bug.


fix bugs with DevAppServer and EE8

“IllegalStateException: Committed” after 403 Forbidden sent by DefaultServlet when directory listing is now allowed.
Investigate and fix classloading issues not being able to load jetty-servlet classes from the web.xml configuration.
Fix the redirect loop issue in the LocalResourceFileServlet.
Use the startWebApp() method introduced in 12.0.8 for EE8 AppEngineWebAppContext.
NullPointerException caused because the HtmlEscaper rejects null value for the error message from the ResponseRewriteFilter.
Fix java.lang.IllegalStateException: Response has already been committed, because the ResponseRewriterFilter.ResponseWrapper was committing the response twice.

We will need a regression test both for EE 8 and EE 10.

Also for the record, the regression was introduced in https://github.com/GoogleCloudPlatform/appengine-java-standard/pull/105/files

We will wait for the regression test, approving only for nighty build