Problem with sets in v2.9

__What TMM version are you using?__2.9
release, pre-release, nightly, or directly from GitHub/branch?
__What is the actual behaviour?__imported and rewrote all NFO, sets show up fine in TMM, cleared out data on Kodi 15.2, reimported all moves, but sets are lost (get a single set, with a name of 'name' for all movies.  Something seems to be wrong with the nfo set information from TMM or Kodi's handling of it.
What is the expected behaviour? sets should show up as before
__Steps to reproduce:__see above

first
second
...

Additional
Have you attached the Logfile from the day it happened?
Your Operating system? (win/mac/linux? version?) TMM on Windows 10, Kodi on OpenElec Pi 2

Please ignore the set issue.  I thought my OpenElec had the latest version of Kodi.  When I updated OpenElec, the sets worked fine.
You might think about an option to allow the old set nfo or the new set nfo for people that don't have the latest version of Kodi.

Yes, we're already thinking about it.
Although Kodi 16 has already been released back in February, there are still few people out there using a dated version...
Thx for reporting

The current version of OpenElec standard Kodi is 15.  You have to do a
development upgrade (or Beta) to get Kodi 16.
Thanks for your great work.  Your software is amazing.
Paul Oversmith
E-Mail<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>
On Wed, Nov 2, 2016 at 8:42 AM, Myron<EMAIL_ADDRESS>wrote:

Yes, we're already thinking about it.
Although Kodi 16 has already been released back in February, there are
still few people out there using a dated version...
Thx for reporting
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/tinyMediaManager/tinyMediaManager/issues/97#issuecomment-257853315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGZgG6nfslyxdqZ1_ExzjXc8gTMlw_1_ks5q6IUegaJpZM4Kld29
.


Difference between get_historical_data v1 and v2?

If I get 1-day historical data, I see that get_historical_data has timestamps that are at noon whereas v2 has timestamps that are at midnight.  Further, the two do not match. What is the difference? This is not described in the API documentation.
Screenshot attached.

I understand that the exact same issue has been raised many months ago and is still open. Can someone PLEASE respond?

Furthermore, I find that data from get_historical_data() matches data from NSE, yahooFinance etc but get_historical_data_v2() does NOT match. In my call with Hiten 2 days ago he told me the team recommends get_historical_data_v2. So this is puzzling me and the reason for the above has not been explained. What is the experience of others?


Added functionality to update cost dictionary in template substitution

Summary
Fixed #5232
Details and comments
Added cost_dict to __init__ constructor method of TemplateSubstitution class and updated the dictionary calls in _quantum_cost() with self instances.

This is looking good. One part that is still missing is the following: users typically interact with class TemplateOptimization(TransformationPass) which then calls TemplateSubstitution. I.e. it might be necessary to allow as argument to TemplateOptimization.__init_ an optional user provided cost dict that can be passed to TemplateSubstitution

See here for alternative implementation: https://github.com/Qiskit/qiskit-terra/pull/5521

Apologies for the delay @eggerdj. Can I complete the PR by this weekend(just after my exams)?

@born-2learn Sure, you can go and have a look at #5521 which has the needed functionality. Tests are missing in #5521. Perhaps you could start from that code and add a couple of tests?

Replaced with https://github.com/Qiskit/qiskit-terra/pull/5521

Replaced with https://github.com/Qiskit/qiskit-terra/pull/5521


Can't get more than 15 frames

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win10 64bit 1709

Kernel Version (Linux Only)
-

SDK Version
0.282.1

Running a simple loop to get a set amount of frames from the camera.
The loop constantly stops after 15 frames with a RuntimeError. The code is as follows:
for i in range(FRAMES_TO_RECORD):
	# Start measuring
	start_time = time.time()
	frame = pipeline.wait_for_frames(5000)
	frames.append(frame)
	time_now = time.time()
	print '[{}] {}'.format(time_now, i)
	time_per_frame_recieve.append(time.time() - start_time)

It fails to get another frame even if I set the timeout to 20,000ms.
However, the Realsense viewer doesn't experience that issue.

Hi @Alonmeytal
This is happening because the script is saving frames inside a collection.
At the moment, all frames are recycled and live within shared frame-pools. By default, we only allow for 16 frames to cycle in the system.
This is done to reduce allocation time (we just fetch frame from the pool) and to ensure there is no consistent resource leak (if frames are processed slower then the rate at which they arrive, you must implement some kind of throttling mechanism or the application will run out of memory very fast).
We do however acknowledge the requirement to be able to save unlimited number of frames (either for data-collection or in garbage-collected environments) and will enable it soon. For now, if you are building the library from source, you can build the development branch and replace option_range{ 0, 32, 1, 16 } with option_range{ 0, 32, 1, 0 } in source.cpp:41. After rebuilding (with -DBUILD_PYTHON_BINDINGS=true) the script should work.

Thanks for elaborating, I’ll give it a try!

Thanks for elaborating, I’ll give it a try!

Is there a workaround other than installing from source? I tried deallocating objects running gc in python, but none are helping.

I solved this problem by closing the pipelines and starting them again.

Hello !
I got the same issue.
Im doing multiprocessing so I need to stock the frames in a buffer which will be use lately. However I can not get more than 15 frames (while k in the code). I understand the problem, Dorodnic explained it very well, thanks to him. Unfortunately, I am not developper and do not know how to solve it, it seems complicated.
I am using only python and not C++.
import os
import threading
import time
import numpy as np
import pyrealsense2.pyrealsense2 as rs
import cv2
import sys
import math
current = (0,0)
class Camera():

    def __init__(self):

        self.pipeline = rs.pipeline()
        self.config = rs.config()
        self.cfg = self.pipeline.start(self.config)
        self.dev = self.cfg.get_device()
        self.colorizer = rs.colorizer()

        #self.align_to = rs.stream.depth

    def param(self):
        return self.pipeline, self.config, self.cfg, self.dev, self.colorizer

def on_mouse(event, x, y, buttons, params) :
    global current

    if event == cv2.EVENT_MOUSEMOVE:

        current = (x, y)

def acquisition_frames(framelist, lock):
    print("Acquisition des frames")
    pipeline, config, cfg, dev, colorizer = Camera().param()
    print(dev.get_info(rs.camera_info.serial_number))
    print(dev.get_info(rs.camera_info.name))
    print(dev.get_info(rs.camera_info.firmware_version))
    print(dev.get_info(rs.camera_info.product_id))
    print(dev.get_info(rs.camera_info.physical_port))

    lst = []

    k = 0
    while k < 15 :
        lock.acquire()
        try :

            frames = pipeline.wait_for_frames()
            depth_frame = frames.get_depth_frame()
            color_frame = frames.get_color_frame()
            depth_image = np.asanyarray(depth_frame.get_data())
            color_image = np.asanyarray(color_frame.get_data())
            depth_colormap = np.asanyarray(colorizer.colorize(depth_frame).get_data())
            lst.append(depth_colormap)
        finally :
            lock.release()
            k = k + 1
    framelist.put(lst)

if __name__ == '__main__':

    print("----------------------------------------------------------")
    print("Main Process : définition des process")
    time.sleep(2)
    framelist = mp.Queue()
    lock = mp.Lock()
    process_acquisition = mp.Process(target = acquisition_frames, args = (framelist,lock,))
    print("Process  : lecture des données de la caméra et stockage des frames dans le buffer")
    start = time.time()
    process_acquisition.start()
    print("Main Process : Affichage du stream")
    print("----------------------------------------------------------")
    getback_data = framelist.get()
    print("data bien récupérée")
    for i in range(15):
        cv2.imshow('RealSense depth colorized',getback_data[i])
        key = cv2.waitKey(200) # 1 frame toutes les x sec
    process_acquisition.join()
    end = time.time() - start
    print("temps realisé")
    print(end)
    print("fin ")
    print("----------------------------------------------------------")

Hello, I have the same problem : I need to surpass 15 frames because I am using multiprocessing, I read the commentary of @dorodnic which is very interesting and which explains well the problem.
However, I am not very good at programming, can you help me to solve this ?
I put my fonction below for processing :
I want to stock all the frames I want inside a process called "acquisition_frames"
After that, I use this process inside a main fonction which get all the frames to disp the video flow
    print("Acquisition des frames")
    pipeline, config, cfg, dev, colorizer = Camera().param()
    print(dev.get_info(rs.camera_info.serial_number))
    print(dev.get_info(rs.camera_info.name))
    print(dev.get_info(rs.camera_info.firmware_version))
    print(dev.get_info(rs.camera_info.product_id))
    print(dev.get_info(rs.camera_info.physical_port))

    lst = []

    k = 0
    while k < 15 :
        lock.acquire()
        try :

            frames = pipeline.wait_for_frames()
            depth_frame = frames.get_depth_frame()
            color_frame = frames.get_color_frame()
            depth_image = np.asanyarray(depth_frame.get_data())
            color_image = np.asanyarray(color_frame.get_data())
            depth_colormap = np.asanyarray(colorizer.colorize(depth_frame).get_data())
            lst.append(depth_colormap)
        finally :
            lock.release()
            k = k + 1
    framelist.put(lst)

Hello, I have the same problem : I need to surpass 15 frames because I am using multiprocessing, I read the commentary of @dorodnic which is very interesting and which explains well the problem.
However, I am not very good at programming, can you help me to solve this ?
I put my fonction below for processing :
I want to stock all the frames I want inside a process called "acquisition_frames"
After that, I use this process inside a main fonction which get all the frames to disp the video flow
def acquisition_frames(framelist, lock):
    print("Acquisition des frames")
    pipeline, config, cfg, dev, colorizer = Camera().param()
    print(dev.get_info(rs.camera_info.serial_number))
    print(dev.get_info(rs.camera_info.name))
    print(dev.get_info(rs.camera_info.firmware_version))
    print(dev.get_info(rs.camera_info.product_id))
    print(dev.get_info(rs.camera_info.physical_port))

    lst = []

    k = 0
    while k < 15 :
        lock.acquire()
        try :

            frames = pipeline.wait_for_frames()
            depth_frame = frames.get_depth_frame()
            color_frame = frames.get_color_frame()
            depth_image = np.asanyarray(depth_frame.get_data())
            color_image = np.asanyarray(color_frame.get_data())
            depth_colormap = np.asanyarray(colorizer.colorize(depth_frame).get_data())
            lst.append(depth_colormap)
        finally :
            lock.release()
            k = k + 1
    framelist.put(lst)


[bug] Nokogiri::CSS throws cryptic error messages when missing a parameter

Please describe the bug
When calling a Nokogiri::CSS selector without a parameter, this message appears:
Nokogiri::CSS::SyntaxError: unexpected '$' after ''

https://stackoverflow.com/questions/74457363/capybara-rspec-empty-html-nokogiricsssyntaxerror-unexpected-after?noredirect=1#comment131440070_74457363
Expected behavior
Tell the user a parameter is missing
Environment
❯ docker-compose exec -it app bundle exec nokogiri -v
WARN[0000] The "CONTAINER_UID" variable is not set. Defaulting to a blank string.
WARN[0000] The "CONTAINER_GID" variable is not set. Defaulting to a blank string.
WARN[0000] The "CONTAINER_UID" variable is not set. Defaulting to a blank string.
WARN[0000] The "CONTAINER_GID" variable is not set. Defaulting to a blank string.
# Nokogiri (1.12.5)
    ---
    warnings: []
    nokogiri:
      version: 1.12.5
      cppflags:
      - "-I/usr/local/lib/app-bundle/gems/nokogiri-1.12.5-x86_64-linux/ext/nokogiri"
      - "-I/usr/local/lib/app-bundle/gems/nokogiri-1.12.5-x86_64-linux/ext/nokogiri/include"
      - "-I/usr/local/lib/app-bundle/gems/nokogiri-1.12.5-x86_64-linux/ext/nokogiri/include/libxml2"
      ldflags: []
    ruby:
      version: 2.6.10
      platform: x86_64-linux
      gem_platform: x86_64-linux
      description: ruby 2.6.10p210 (2022-04-12 revision 67958) [x86_64-linux]
      engine: ruby
    libxml:
      source: packaged
      precompiled: true
      patches:
      - 0001-Remove-script-macro-support.patch
      - 0002-Update-entities-to-remove-handling-of-ssi.patch
      - 0003-libxml2.la-is-in-top_builddir.patch
      - 0004-use-glibc-strlen.patch
      - 0005-avoid-isnan-isinf.patch
      - 0006-update-automake-files-for-arm64.patch
      - 0007-Fix-XPath-recursion-limit.patch
      libxml2_path: "/usr/local/lib/app-bundle/gems/nokogiri-1.12.5-x86_64-linux/ext/nokogiri"
      memory_management: ruby
      iconv_enabled: true
      compiled: 2.9.12
      loaded: 2.9.12
    libxslt:
      source: packaged
      precompiled: true
      patches:
      - 0001-update-automake-files-for-arm64.patch
      - 0002-Fix-xml2-config-check-in-configure-script.patch
      datetime_enabled: true
      compiled: 1.1.34
      loaded: 1.1.34
    other_libraries:
      zlib: 1.2.11
      libgumbo: 1.0.0-nokogiri

Hi, I'm sorry you're having a problem. I'll try to help.
You skipped the bit in the issue template entitled "Help us reproduce what you're seeing." I spent about 15 minutes trying to reproduce this, and looking through the Capybara source code, and I can't.
As a prompt, here's the section you skipped:

If possible, please include a complete, self-contained script that reproduces the behavior you're seeing. Please try to remove external dependencies like Rails or other libraries that may be wrapping Nokogiri.
Here's an example of how you might structure such a script:
#! /usr/bin/env ruby

require 'nokogiri'
require 'minitest/autorun'

class Test < MiniTest::Spec
  describe "Node#css" do
    it "should find a div using chained classes" do
      html = <<~HEREDOC

 one

two

three

      HEREDOC

      doc = Nokogiri::HTML::Document.parse(html)

      assert_equal 1, doc.css("div.foo.bar").length
      assert_equal "three", doc.at_css("div.foo.bar").text
    end
  end
end

If you haven't included a test, please provide whatever code you can, the inputs (if any), along with the output that you're seeing. We need to reproduce what you're seeing to be able to help!

Can you also help me understand why you're reporting this here instead of with the Capybara project?

Well the code does not really matter, it is about the shell interaction.
I thought it might be trivial to reproduce in any project with Capybara. I don't know whether this is a downstream or upstream issue, parameter checking generally seems a bit awkward in ruby...
This is also a bit of a legacy project which I am slowly trying to bring up to date, so I did not want to invest too much time upfront to investigate something that has already been fixed.

Please put yourself in my spot: I don't use Capybara and don't know how to set up an isolated test case. What would you do with this issue?
Thankfully after a few more minutes I have reproduced what you're seeing:
Nokogiri::HTML5.parse("

").css("")

I think you should also report this upstream at Capybara because it doesn't appear to be checking the number of arguments passed to those methods. Although Nokogiri could be raising a better exception here, Capybara could be a bit more defensive about how it calls Nokogiri.

See https://github.com/sparklemotion/nokogiri/pull/2701 for the fix which will be in Nokogiri 1.14.0 when it's released.

thank you so much, I am rather new to the whole ruby on rails ecosystem and still struggling to grasp how everything interacts :)


StripeProvider - TypeError: Cannot read property 'getConstants' of null, js engine: hermes

Describe the bug
Although I have installed stripe-react-native as specified in documentation, The inclusion of  results in the following error on app build: "TypeError: Cannot read property 'getConstants' of null, js engine: hermes". I'm running iOS react-native with version "@stripe/stripe-react-native": "0.28.0".
Any help is greatly appreciated, thanks.
Code
import {NavigationContainer, useNavigation} from '@react-navigation/native';
import React, {useEffect, useState} from 'react';
import EntryPage from './screens/entry-page';
import HomePage from './screens/home-page';
import ProfilePage from './screens/profile-page';
import {createNativeStackNavigator} from '@react-navigation/native-stack';
import {Auth} from 'aws-amplify';
import {Button, StyleSheet, View} from 'react-native';
import {StripeProvider} from '@stripe/stripe-react-native';

const Stack = createNativeStackNavigator();

function App(): JSX.Element {
  const [user, setUser] = useState(null);
  const [isLoading, setIsLoading] = useState(true);

  useEffect(() => {
    checkAuth();
  }, []);

  // immediately check if user is logged on
  const checkAuth = async () => {
    try {
      const user = await Auth.currentAuthenticatedUser();
      setUser(user);
    } catch (error) {
      setUser(null);
    }
    setIsLoading(false);
  };

  if (isLoading) {
    const styles = StyleSheet.create({
      container: {
        flex: 1,
        backgroundColor: '#fff',
      },
    });
    return <View style={styles.container} />;
  } else {
    return (
      <StripeProvider publishableKey={'{I include my publishable key here as string'}>
        <NavigationContainer>
          <Stack.Navigator initialRouteName={user ? 'HomePage' : 'EntryPage'}>
            <Stack.Screen
              name="HomePage"
              component={HomePage}
              options={({navigation}) => ({
                headerShown: true,
                // headerStyle: {
                //   backgroundColor: 'black',
                // },
                headerTransparent: true,
                headerBlurEffect: 'systemUltraThinMaterialLight',
                headerBackVisible: false,
                headerRight: () => (
                  <Button
                    onPress={() => navigation.navigate('Profile')}
                    title="Profile"
                    color="gray"
                  />
                ),
              })}
              initialParams={{userLoggedIn: user ? true : false}}
            />
            <Stack.Screen
              name="EntryPage"
              component={EntryPage}
              options={({navigation}) => ({
                headerShown: false,
              })}
            />
            <Stack.Screen
              name="Profile"
              component={ProfilePage}
              options={({navigation}) => ({
                headerShown: false,
              })}
            />
          </Stack.Navigator>
        </NavigationContainer>
      </StripeProvider>
    );
  }
}

export default App;

Can you say a little more about your setup? This indicates a mismatch between your JS and native code

Hi Charlie, thanks so much for the reply, I've been pretty stumped by this. I'm using typescript react-native to generate my iOS app. Not touching android for now. I've tried different versions of stripe-react-native, made sure to clear node modules and cache as well as fresh npx pod install. I feel like my issue might be similar to this stripe-react-native issue, although i'm not using expo. Here is my package.json:
{
  "name": "app",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "android": "react-native run-android",
    "ios": "react-native run-ios",
    "lint": "eslint .",
    "start": "react-native start",
    "test": "jest"
  },
  "dependencies": {
    "@aws-amplify/ui-react-native": "^1.2.14",
    "@invertase/react-native-apple-authentication": "^2.2.2",
    "@react-native-async-storage/async-storage": "^1.18.1",
    "@react-native-community/masked-view": "^0.1.11",
    "@react-native-community/netinfo": "^9.3.9",
    "@react-native-picker/picker": "^2.4.10",
    "@react-navigation/native": "^6.1.3",
    "@react-navigation/native-stack": "^6.9.9",
    "@react-navigation/stack": "^6.3.12",
    "@stripe/stripe-react-native": "0.28.0",
    "amazon-cognito-identity-js": "^6.2.0",
    "aws-amplify": "^5.1.3",
    "aws-amplify-react-native": "^7.0.2",
    "core-js": "^3.30.1",
    "native-base": "^3.4.25",
    "react": "18.2.0",
    "react-native": "0.71.7",
    "react-native-gesture-handler": "^2.9.0",
    "react-native-get-random-values": "^1.8.0",
    "react-native-reanimated": "^3.1.0",
    "react-native-safe-area-context": "^4.5.1",
    "react-native-screens": "^3.20.0",
    "react-native-svg": "^13.9.0",
    "react-native-svg-transformer": "^1.0.0",
    "react-native-url-polyfill": "^1.3.0",
    "react-native-vector-icons": "^9.2.0"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@babel/preset-env": "^7.20.0",
    "@babel/runtime": "^7.20.0",
    "@react-native-community/eslint-config": "^3.2.0",
    "@tsconfig/react-native": "^2.0.2",
    "@types/jest": "^29.2.1",
    "@types/react": "^18.0.24",
    "@types/react-native-vector-icons": "^6.4.13",
    "@types/react-test-renderer": "^18.0.0",
    "babel-jest": "^29.2.1",
    "eslint": "^8.19.0",
    "jest": "^29.2.1",
    "metro-react-native-babel-preset": "0.73.9",
    "prettier": "^2.4.1",
    "react-test-renderer": "18.2.0",
    "typescript": "4.8.4"
  },
  "jest": {
    "preset": "react-native"
  }
}

that setup looks good 🤔 not sure why you'd be running into this without using Expo. For now, you could try commenting out this line and see if that unblocks you? But I'd like to understand why you're getting this error

Not sure what happened, but somewhere along the way it was resolved with "npx react-native start --reset-cache". I had run that previously, so I'm not sure what did the trick. If it comes back I'll try commenting that line you mentioned out. I appreciate the quick help!

I am facing this same issue with same set up 🤔
Was there anything additional that you did?
I tried the following steps and your last suggestion, but am getting the same result.
rm yarn.lock
rm node_modelules
yarn install
cd ios
rm Podfile.lock
rm -rf Pods
pod install
I also did a Clean Build Folder in XCode...
and
npx react-native start --reset-cache
but still am facing this error.

Still having this issue running the latest version of stripe-react-native and expo. I've cleared cache, removed all the node modules, and lock files.

运行最新版本的 stripe-react-native 和 expo 时仍然遇到此问题。我已清除缓存，删除所有节点模块并锁定文件。

m1 chip same issue.

same issue. any suggestion?

Same issue with expo

I just rebuild the app and the error was gone.
eas build --profile development --platform ios

Having this issue when trying jest.requireActual('@stripe/stripe-react-native')
TypeError: Cannot read properties of undefined (reading 'getConstants')

I'm still facing this issue for the latest stripe is 0.37.0, thanks so much if have any suggestion ?

just build the app again using : eas build --profile development --platform ios

Hello everyone
Having the same problem, before everything worked well, and I added some new components, then got this error
any solutions?
I think the problem is not directly connected with stripe

Hello I am having the same problem -  I use expo but still the same.. anyone can help ?

this error show that you might be using typescript and there is type miss-match in the tsconfig.js just add this line of code "types": ["@stripe/stripe-react-native"] in your "compilerOptions" : {
"types": ["@stripe/stripe-react-native"]} i think it will solve it

eas build --profile development --platform android
solved the problem


webaudio demo

This isn't too complex, although I'm kind of unsure whether playing a boring single tone is interesting enough. I'd have to commit some audio to the repo otherwise.

Can you use the Page Visibility API (demo) to detect whether the tab is focused?

I've updated the demo. PTAL.

:+1:


Overlays turn black with gpu plugin

Describe the bug
The overlays in the runelite plugin turn black with the new gpu pluggin
To Reproduce
While fighting the gargoyle boss, if you get caught but dusks aura move and get thrown, your overlays turn all black for the duration of the special attack
https://gyazo.com/7a5f757d9bf504a222233232dcb73fcf

Duplicate #6489


schematics-cli: running a schematic does not commit it to disk

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc
- [x] schematics

Versions

Angular CLI: 7.0.0-rc.3
Node: 10.11.0
OS: darwin x64
Angular: undefined
...

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.9.0-rc.3 (cli-only)
@angular-devkit/core         7.0.0-rc.3
@angular-devkit/schematics   7.0.0-rc.3
@schematics/angular          7.0.0-rc.3 (cli-only)
@schematics/update           0.9.0-rc.3 (cli-only)
rxjs                         6.3.3
typescript                   2.9.2

npm ls -g --depth=0
/Users/splaktar/.nvm/versions/node/v10.11.0/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>
Repro steps

Create a new schematics project using schematics schematic --name=test-schematic.
cd test-schematic
npm i
npm run build
schematics .:my-full-schematic --name=test

The log given by the failure

schematics .:my-full-schematic --name=test
    My Full Schematic: {"name":"test","index":1}
            My Other Schematic: {"option":true}
                        My Schematic: {"option":true}
CREATE /hola (5 bytes)
CREATE /allo (5 bytes)
CREATE /test2 (34 bytes)
CREATE /test1 (18 bytes)

You can see above that is at least outputs the --dryRun results, but since I didn't specify --dryRun, it should commit them to disk. I also tried with --dryRun=false but the behavior was the same.
devintent:test-schematic splaktar$ ls
README.md               node_modules            package-lock.json       package.json            src                     tsconfig.json
devintent:test-schematic splaktar$ ls /
Applications                    System                          Volumes                         dev                             installer.failurerequests       private                         usr
Library                         User Information                bin                             etc                             net                             sbin                            var
Network                         Users                           cores                           home                            opt                             tmp

You can see above that there are no files named hola, allo, etc.
Desired functionality

I should be able to run a schematic locally when authoring/testing a schematic and see the output get committed to disk if there are no errors.
Mention any other details that might be useful

I also saw this with<EMAIL_ADDRESS>

@Splaktar I have looked into this, and since the you are running the schematic locally, by default it will be running in a dry run To disable this you need to use --dry-run=false.
I noticed that at the moment only kebab case args are available, I'll do a PR to address this.


Add validation of collector_tstamp

Because collector clocks can sometimes go awry.

See also https://github.com/snowplow/snowplow/issues/336


[DS-4346] create docker compose files in code base

https://jira.duraspace.org/browse/DS-4346

https://wiki.duraspace.org/display/~terrywbrady/Docker+Compose+Refactoring

To build start DSpace from your branch using a published image for DSpace REST.
docker-compose -p d7 up --build -d

End to end testing of the rest api.
docker-compose -p d7 -f docker-compose-travis.yml up -d

The following PR's will migrate docker-compose files into the DSpace/DSpace repo to simplify the building and testing of docker images.

https://github.com/DSpace/DSpace/pull/2523
https://github.com/DSpace/DSpace/pull/2525

This PR offers a similar solution for the angular container.
@artlowel. I have a couple questions

Where should the local.cfg be stored?  Note the placeholder name that I am using.
Is an environment.dev.js needed?  If so, where should it live?

Closing in favor of https://github.com/DSpace/dspace-angular/pull/493


remove react-addons-css-transition-group

Fix https://github.com/reactstrap/reactstrap/issues/602

The DOCS are being rewritten in a separate branch. Just go off of the readme currently, which is already correct.

Oh, do you mean gatsby-docs, do you want me to switch to that branch for this PR?

Just leave for now.. Eddy knows about this and Evan can merge or close as necessary. No need to do anything ... just saying latest info will be the Readme.

Got it!

I'd merge this but it probably won't see the light of day because we're probably going going to publish the docs before v5 and v5 removes react-transition-group from peerDeps, adding it to deps so users do not need to install it separately.


Replace document limits with token limit

We currently use some arbitrary document limits like number of documents and a cap on each document length.
This is not ideal for 2 reasons:

It doesn't factor into consideration numerous small documents
It breaks up larger documents at an arbitrary character count without factoring the rest of the sources.

This was closed by #64.

closes https://github.com/bitcoinsearch/chat-btc/issues/68
Implemented an input token enforcer to limit the input tokens sent to chat to 7000 (with avg of 0.75 token/word which is about 9333 words).
Should be a separate PR but I'd like it if we get it out in this PR due to the refactoring made in

@Emmanuel-Develops, you used the "closes X" syntax, but in order for GitHub to pick it automatically you must edit the PR description and add it there.


Use my own compute_loss with RewardTrainer

Hi. I want to use my own RewardModel (not inherited from AutoModelForSequenceClassification)
But, i got message like below.
compute_loss is only implemented for RewardDataCollatorWithPadding, please implement your own compute_loss method if you are using a custom data collator

Is there any parameter which i can pass my own compute_loss to RewardTrainer ?

Hi @allzero-kwon, it looks like you passed a custom data collator which is why you get this message. @younesbelkada maybe we don't need to through an error here, no? since a user might pass a viable alternative to the default collator that does not require writing a custom loss?
In any case, if you want to write your own loss function you can do it as follows:
from trl import RewardTrainer
class MyRewardTrainer(RewardTrainer):
    def compute_loss(self, model, inputs):
        labels = inputs.pop("labels")
        outputs = model(**inputs)
        logits = outputs[0]
        return my_custom_loss(logits, labels)

Hope this helps!

Thanks !


[Bug]Copy-DbaDbTableData not copying data to a new table

Environmental information
PowerShell Version        : 5.1.14393.3471
dbatools latest installed : 1.0.115
Culture of OS             : en-GB
SQL Server:
Microsoft SQL Server 2016 (SP2-CU13) (KB4549825) - 13.0.5820.21 (X64)   May 22 2020 07:24:35   Copyright (c) Microsoft Corporation  Developer Edition (64-bit) on Windows Server 2016 Standard 10.0 <X64> (Build 14393: ) (Hypervisor)

us_english

Report
Running copy-dbadbtabledata from a table in one database with SET options to a table in a different database with different schema creates the table and errors, but does not copy the table data.
The copy completes successfully if run a second time.
Host used

[x] powershell.exe
[x] ISE
[x ] VS Code
[ ] Other (please specify)
[x] Still buggy in powershell.exe

Errors Received
WARNING: [09:25:56][Copy-DbaDbTableData] Unable to determine destination table: [dbo].[test1_archive] | Cannot find the object "dbo.test1" because it does not exist or you do not have permissions.
Steps to Reproduce
Two databases on the same SQL server DB1 and DB2, DB1 with a table named test1 with a set option e.g. lock escalation.  DB1 without a table named test1.
CREATE DATABASE DB1
GO
CREATE DATABASE DB2
go
USE DB1
GO
CREATE TABLE test1 (column1 int)
go
ALTER TABLE test1  SET (LOCK_ESCALATION = AUTO)
INSERT INTO test1 (column1) VALUES (1)
INSERT INTO test1 (column1) VALUES (2)
GO

We want to copy data from table test1 on DB1 into a new table called test1_archive on DB2:
$params = @{
    SqlInstance         = "SQL01"
    Destination         = "SQL01"
    Database            = "DB1"
    Table               = "[dbo].[test1]"
    DestinationTable    = "[dbo].[test1_archive]"
    DestinationDatabase = "DB2"
    AutoCreateTable     = $true
    Query               = "SELECT * FROM [DB1].[dbo].[test1] where column1 < 10"
    }

    Copy-DbaDbTableData @params

Expected Behavior
Expect a new table dbo.test1_archive to be created on database DB2 and populated with the 2 records that exist in dbo.test1 table on database DB1 satisfying the query statement.
Actual Behavior
The PowerShell Function fails with error  Cannot find the object "dbo.test1" because it does not exist.
The new table dbo.test1_archive is created successfully on DB2, but it is empty and the data copy has not run.
Root Cause & Suggested Resolution
This issue is caused because when the table creation script is generated by the command in copy-dbadbtabledata:
$tablescript = $sqltable | Export-DbaScript -Passthru | Out-String

it includes the SET options on the table in an ALTER statement:
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE TABLE [dbo].[test1](
        [column1] [int] NULL
) ON [PRIMARY]
GO
ALTER TABLE [dbo].[test1] SET (LOCK_ESCALATION = AUTO)
GO

The REGEX commands further down the script do a replace on the table name changing dbo.test1 to dbo.test1_archive for the CREATE statement, but NOT the ALTER statement.
In this instance the table create script that is actually run is:
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE TABLE [dbo].[test1_archive](
        [column1] [int] NULL
) ON [PRIMARY]
GO
ALTER TABLE [dbo].[test1] SET (LOCK_ESCALATION = AUTO)
GO

This causes an error creating the new table because this SQL script is running an ALTER TABLE command on database DB2 for a table that does not exist in that database (dbo.test1).
My workaround was to add an additional REGEX statements for alter commands on the table creation.  I am really not a REGEX expert but if this looks good, I am happy to submit a PR with this change.  It worked for my scenario at least.
$rX = "(ALTER TABLE \[$([regex]::Escape($sqltable.Schema))\]\.\[)$([regex]::Escape($sqltable.Name))(\])"
$tablescript = $tablescript -replace $rX, "`$1$($newTableParts.Name)`$2"

$rX = "(ALTER TABLE \[)$([regex]::Escape($sqltable.Schema))(\]\.\[$([regex]::Escape($newTableParts.Name))\])"
$tablescript = $tablescript -replace $rX, "`$1$($newTableParts.Schema)`$2"
```powershell

Apologies for the lengthy explanation.  I hope this makes sense, this is such a useful function.  :-)

I just altered the regex, that should help.


Support S3 CAS now that PUT If-none-match: * is supported

https://github.com/slatedb/slatedb/blob/main/docs/0001-manifest.md#cas-on-s3 is out of date, as S3 just recently added support for conditional PUT:
https://docs.aws.amazon.com/AmazonS3/latest/userguide/conditional-requests.html#conditional-writes

Thanks for filing this! We definitely want to adopt this feature. We'll need the corresponding support from the object_store crate first: https://github.com/apache/arrow-rs/issues/6285. Then we should be good to go.

We should update the design doc to eliminate the non-CAS solutions since S3 now supports CAS. All object stores will now work with our CAS-based design.
This is blocked on the Arrow ticket @rodesai mentions above.

I put up a PR upstream to add support for native S3 CAS to the object_store crate: https://github.com/apache/arrow-rs/pull/6682. Still waiting on reviews, but hopefully we'll have something we can use here soon.

Ha! Hopefully just a few more days/weeks that you have to make that confession. 😂

It's merged! Now we wait for https://github.com/apache/arrow-rs/issues/6596

Indeed! I've got PRs up for object_store to add support for If-Match: https://github.com/apache/arrow-rs/issues/6799#issuecomment-2499713001. If all goes well we'll get both those PRs into the next release of object_store.


SSL certificates have expired

The SSL certificate for https://getwarden.net/ and https://panel.getwarden.net/ appears to have expired on 25/09/2017 and 27/09/2017.
Perhaps setting up an automatic renewal for Lets Encrypt would be good? I believe there are programs out there for handling this on Windows.

Hi,
Apologies for the late response. The web app is currently unreachable as I had to close the VM that was being used.

Is Warden dead/abandoned then? It doesn't look like any work has been done on it since April and the SSL cert expiring is a bit concerning.

Temporarily it is possible to reach https://getwarden.net/ with this solution
However on https://panel.getwarden.net/ I see HTTP Error 502.5 - Process Failure.


Add Dockerfile for airgap-patch

Signed-off-by: hang.jiang<EMAIL_ADDRESS>
What type of PR is this?

/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm


netrw: fix typo of missing let

As kindly pointed out by @zzzyxwvut in https://github.com/vim/vim/commit/3d7e567ea7392e43a90a6ffb3cd49b71a7b59d1a#diff-04bdb3e603079bce69af82dde6018a473e9ec1da595229d3f7bfc777885a169aR39

thanks, I just fixed it.


[proposal] refactor metric-advisor in koordlet as a framework with plugins

What is your proposal:
refactor metric-advisor in koordlet as a framework.
Each collector should be a plugin.
Why is this needed:
For better extension.
Also in future, koordlet should support collector plugins calling prometheus exporters on node, parsing through prometheus api.
Is there a suggested solution, if so, please add it:

/area koordlet
/milestone v1.2

Nice!


No accès to web interface

Hi Gents,
I am working on your piece of software to look if it could fit our needs (kind of tag generator shared).
I tried to install it on a fresh Debian with Postgresql in local. (Debian 11 fresh install , no firewall, no specific installation as far as I could tell)
Connection to the database seems to the okay (no error) on both the ./r3 -run nor on the log of the database, however I can not connect to the web interface.
I track down the problem to :
failed to register listener for HTTP server, listen tcp <IP_ADDRESS>:8080: bind: address already in use
I try the port 80, 443 and 8080 (none of them is used) :
ei01:~# netstat -antu
Active Internet connections (servers and established)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:25            <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:57452         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:57806         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:57806         ESTABLISHED
tcp        0      0 <IP_ADDRESS>:53304        <IP_ADDRESS>:80      TIME_WAIT
tcp        0      0 <IP_ADDRESS>:57832         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:57474         ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:57832         ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:57452         ESTABLISHED
tcp        0      0 <IP_ADDRESS>:57474         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:57830         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:57830         ESTABLISHED
tcp6       0      0 :::8080                 :::*                    LISTEN
tcp6       0      0 :::22                   :::*                    LISTEN
tcp6       0      0 ::1:5432                :::*                    LISTEN
tcp6       0      0 ::1:25                  :::*                    LISTEN
udp6       0      0 ::1:55495               ::1:55495               ESTABLISHED

(r3 is on 8080)
While I am writing this email, I just notice the IPV6 ...
I disable fully the IPV6 stack and I got the
failed to register listener for HTTP server, listen tcp <IP_ADDRESS>:8080: bind: address already in use
I changed to 8088 (rebooted as well)
At start, the service is not working :

r3.service - Provides the REI3 platform components
Loaded: loaded (/etc/systemd/system/r3.service; enabled; vendor preset: enabled)
Active: inactive (dead) since Sat 2021-10-23 20:10:26 UTC; 8s ago
Process: 83 ExecStart=/root/r3 (code=exited, status=0/SUCCESS)
Main PID: 83 (code=exited, status=0/SUCCESS)

Oct 23 20:08:23 proxcluster-BMS-ei01 systemd[1]: Started Provides the REI3 platform components.
Oct 23 20:08:23 proxcluster-BMS-ei01 rei3[83]: Starting service
Oct 23 20:08:24 proxcluster-BMS-ei01 rei3[83]: failed to register listener for HTTP server, liste>
Oct 23 20:10:26 proxcluster-BMS-ei01 systemd[1]: Stopping Provides the REI3 platform components...
Oct 23 20:10:26 proxcluster-BMS-ei01 systemd[1]: r3.service: Succeeded.
Oct 23 20:10:26 proxcluster-BMS-ei01 systemd[1]: Stopped Provides the REI3 platform components.

A restart bring :

systemctl status r3.service

r3.service - Provides the REI3 platform components
Loaded: loaded (/etc/systemd/system/r3.service; enabled; vendor preset: enabled)
Active: active (running) since Sat 2021-10-23 20:10:51 UTC; 21s ago
Main PID: 363 (r3)
Tasks: 7 (limit: 9360)
Memory: 2.4M
CGroup: /system.slice/r3.service
`-363 /root/r3

Oct 23 20:10:51 proxcluster-BMS-ei01 systemd[1]: Started Provides the REI3 platform components.
Oct 23 20:10:51 proxcluster-BMS-ei01 rei3[363]: Starting service
Again IPV6 :

netstat -antu
Active Internet connections (servers and established)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:25            <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:59694         ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:59490         TIME_WAIT
tcp        0      0 <IP_ADDRESS>:59716         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:59694         <IP_ADDRESS>:5432          ESTABLISHED
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:59472         TIME_WAIT
tcp        0      0 <IP_ADDRESS>:5432          <IP_ADDRESS>:59716         ESTABLISHED
tcp6       0      0 :::22                   :::*                    LISTEN
tcp6       0      0 :::8088                 :::*                    LISTEN
udp        0      0 <IP_ADDRESS>:40567         <IP_ADDRESS>:40567         ESTABLISHED

// UPDATE //
I set the ip address in the config.json and it starts working !!
Sorry for the noice. I leave it in case it can be helpful for anybody.

Closed due to inactivity.


feat: add support for configuring additional page/content route sources

PR Checklist

Related to #904
What is the new behavior?

Page and content routes can be sourced from directories outside the main src/app/pages and src/content directories
Adds support for discovering page routes and content from projects in a monorepo
Adds support to the Vite plugin for Angular to include additional files in compilation
Allows the createRoutes function to be used with the Angular application builder without overrides

/// <reference types="vitest" />

import { defineConfig } from 'vite';
import analog from '@analogjs/platform';

// https://vitejs.dev/config/
export default defineConfig(({ mode }) => ({
  build: {
    target: ['es2020'],
  },
  resolve: {
    mainFields: ['module'],
  },
  plugins: [
    analog({
      // scans for `**/*.page.{ts,analog}`. Must be inside a `pages` directory
      additionalPagesDirs: ['/libs/shared/feature'],

      // scans for `**/*.{md,agx}`. Must be inside a `content` directory
      additionalContentDirs: ['/libs/shared/feature'],

      // scans for `**/*.{ts}`. Must be inside a `routes` directory
      additionalAPIDirs: ['/libs/shared/feature/src/api']

    })
  ],
  test: {
    globals: true,
    environment: 'jsdom',
    setupFiles: ['src/test-setup.ts'],
    include: ['**/*.spec.ts'],
    reporters: ['default'],
  },
  define: {
    'import.meta.vitest': mode !== 'production',
  },
}));

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
[optional] What gif best describes this PR or how it makes you feel?

we can also add a config option for API route dirs. this is currently available through nitro.scanDirs config. just an alias would be enough I think

we can also add a config option for API route dirs. this is currently available through nitro.scanDirs config. just an alias would be enough I think

Added the additionalAPIDirs option for scanning API routes


desync: 0.3.0 -> 0.4.0

Note that upstream is using Go 1.11 modules; deps.nix is now generated using vgo2nix.
Motivation for this change
Usual set of bugfixes and improvements. Progress bar available for more subcommands, better error handling, etc.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

I can confirm that the binary builds and runs on NixOS.


Update tests for live info endpoint and update docs

Pull Request Test Coverage Report for Build 9833abd7c32cb3b09f94a62786c5617dc2677a73-PR-46

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>08640d9803687648b8cf533fed49c0:
0.0%

Covered Lines:
3566

Relevant Lines:
3566

💛  - Coveralls

Pull Request Test Coverage Report for Build 9833abd7c32cb3b09f94a62786c5617dc2677a73-PR-46

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>08640d9803687648b8cf533fed49c0:
0.0%

Covered Lines:
3566

Relevant Lines:
3566

💛  - Coveralls


Update mentions of iOS to Swift

Changes are for this Asana task, as part of this parent task.
Summary: according to the parent task, I've renamed iOS to Swift in all instances for the Swift API client, Live stream lib, Uploader SDK, and Player analytics SDK.

any page where we reference iOS in the list of offered clients / SDKs / libs
navigation.yaml for API client libraries
URLs in templates\documentation\sdks\player\apivideo-swift-player.md
URLs in templates\documentation\sdks\player\apivideo-swift-player-analytics.md
URLs and titles in templates\documentation\sdks\vod\apivideo-ios-uploader.md
Filename and navigation.yaml reference of templates\documentation\sdks\vod\apivideo-ios-uploader.md
URLs in templates\documentation\sdks\vod\apivideo-react-native-uploader.md
Redirect to /sdks/vod/apivideo-swift-uploader in templates\documentation\doctave.yaml
URLs in templates\documentation\vod\upload-a-video-regular-upload.md
URLs in templates\documentation\vod\upload-from-source.md
URLs in templates\documentation\vod\upload-your-first-video.md

Have you changed the iOS thumbnail to the Swift thumbnail?


2 icons: Banxo, RATP

Description
Add 2 icons (Caisse d'epargne banxo, ratp)
Icons addition information
Added
Banxo (com.caisseepargne.android.mobilebanking)
Bonjour RATP (com.fabernovel.ratp)
Contributor's checklist

[x] I followed the Lawnicons guidelines and will make changes if someone suggests. I will also make sure that Lawnicons builds correctly.

If #2159 is accepted, you can close this PR

#2174


MethodNotAllowedHTTPException on Intended Redirect

Environment:

Laravel Version: 5.5.40
PHP Version: 7.0
Database Driver & Version: N/A

Description:
redirect()->intended() can redirect the user to a bad route if the user interacts with a form after session timeout. This leads to errors being shown to a user from reasonable use of the application and is surprisingly difficult to avoid or work around.
Steps To Reproduce:

Have an expired session or no session.
POST to a route protected by the 'auth' middleware that does not have a corresponding GET route.

Easy way to do this is to login, navigate to a form, wait a while, then submit the form.

Authenticate and be redirected to the POST route.
Receive MethodNotAllowedHTTPException.

Proposed Solution:
Alter \Illuminate\Routing\Redirector::guest() to check if the current request is suitable for redirection and if the requested route isn't suitable for redirect, either don't store it as url.intended or store the "previous" URL as the intended route to get the user back to being as close as possible to the failed request. I see a good set of conditions in \Illuminate\Session\Middleware\StartSession::storeCurrentUrl(), where the previous URL is stored in the first place.

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not for ideas, suggestions etc. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. You are able to open a ticket at https://github.com/laravel/ideas
Alternatively you are able to open a PR using the Contributions guidelines: https://laravel.com/docs/5.7/contributions#which-branch
If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.

@laurencei It depends on the semantics, but I'm fairly certain this is a bug, as reasonable use of the framework is netting me unavoidable exceptions.

@laurencei The description should be clear about how this is a framework issue. I don't think I can elaborate further. It only has the word "Proposed" in it as this is the kind of bug that doesn't have a straight-forward solution.
Also, I have no idea why you think I'm new to the community. I have the "Contributor" tag. There's no way to put this tactfully, but it makes me think you didn't fully read this issue before closing it. :disappointed:

Opened a PR for this.


Skip metric with values as lists

metrics metrics with values with type list break collectd

@InformatiQ   could you please target the upstream repo instead of this one? It would be good to avoid maintaining additional forks

I should yes, but for now we already have a diff

+1, we could also open a separated PR upstream for this patch.

This fork will be obsoleted when thosr PR are merged
https://github.com/klynch/collectd-marathon/pull/6
https://github.com/klynch/collectd-marathon/pull/3


Various date/time related fixes

Contributor checklist

[x] Changes have been tested
[x] Change-type present on at least one commit
[x] Signed-off-by is present
[x] The PR complies with the Open Embedded Commit Patch Message Guidelines

Reviewer Guidelines

When submitting a review, please pick:

'Approve' if this change would be acceptable in the codebase (even if there are minor or cosmetic tweaks that could be improved).
'Request Changes' if this change would not be acceptable in our codebase (e.g. bugs, changes that will make development harder in future, security/performance issues, etc).
'Comment' if you don't feel you have enough information to decide either way (e.g. if you have major questions, or you don't understand the context of the change sufficiently to fully review yourself, but want to make a comment)

https://github.com/resin-os/meta-resin/pull/1061/commits/f9e226ea1dcb34f3724f421112098217bd2a7371 needs to be commit after https://github.com/resin-os/meta-resin/pull/1061/commits/497f2894c6bafe36cd981d8a959f081aa422a991
Otherwise there might be a breakage between commits. causes issues while bisecting.

@ZubairLK It's a github thing. If you do a clone and checkout locally, you will see the "real" order. I know there was a reason behind this issue but I can't remember it.


Import - Check CSV for duplicate IDs

It can be a mystery when the CSV row count and the total imported documents count does not match. Sometimes this is because of a duplicate row in the CSV file.
Add a feature to sanity check the CSV file before importing. Try and check for:

Duplicate row (IDs)
Missing required field values
CSV "soundness"

Closing via 4c419cb


Issue 14 auth

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #14
Test URLs:

Before: https://main--zemax--hlxsites.hlx.page/
After: https://issue-14-auth--zemax--hlxsites.hlx.page/

@kmurugulla  When login is successful access token should be removed the url


CallMonitoringAspect invoke() not invoked

A JMX bean exposed through CallMonitoringAspect.java that does show up in Java Mission Control JMX browser.
The invoke(pjp) method in CallMonitoringAspect.java does not appear to be invoked even if the bean is enabled (which it is by default).  The count is never incremented.
Am I missing something here?

It appearst that spring-data-jpa uses classes that are not @Repository annotated. This means the CallMonitoringAspect never gets invoked.  Switch to jpa or jdbc data source and you get call counts in CallMonitoringAspect


run as service

Sorry for bringing this up again, but I have am not able to solve the problem with the script crashing after a day or so, stating "img returned null" (or similar). As a last chance for solving this I am will try to run the script in systemd, and get it to autostart when it crashes. However, it seems that I can not start a script with arguments in systemd. I have tried to hardcode my args into your code, but I only get errors. This is the args that I use:
--model ssd_mobilenet_v1_coco --rtsp --uri rtsp://username:password@<IP_ADDRESS>:554/h265/ch1/main/av_stream --height 1088 --width 1920
I would be very gratful if you can guide me how to hard code this into your code
Thanks

It should be simple to solve the systemd issue.  You could put the command to run the trt_ssd application, with all necessary arguments, in a shell script.  Then just execute the script, which does not require any arguments, from your systemd service.

Otherwise, you could replace argparse.ArgumentParser (args) with something like a SimpleNameSpace or a class, and put your hardcoded arguments in there.  For example,
from types import SimpleNamespace
from utils.camera import Camera

args = SimpleNamespace()
args.image = 'dog.jpg'
args.width = 1280
args.height = 720
args.copy_frame = False
args.do_resize = False
args.video_looping = False
cam = Camera(args)
......

Thanks, now I can start the script without the args, but I get an error when I use it in systemd
`pi@jetson:~/.config/systemd/user$ systemctl --user status python_demo_service
● python_demo_service.service - Python Demo Service
Loaded: loaded (/home/pi/.config/systemd/user/python_demo_service.service; static; vendor preset: enabled)
Active: active (running) since Sun 2020-10-18 15:53:18 CEST; 7s ago
Main PID: 10808 (python3)
CGroup<EMAIL_ADDRESS>├─10808 /usr/bin/python3 /home/pi/SSD/PersonDetectionService/trt_ssd_10.6.py
├─10833 sh -c /usr/lib/aarch64-linux-gnu/libproxy/0.4.14/pxgsettings org.gnome.system.proxy org.gnome.system.proxy.http org.gnome.system.proxy.https org.gnome.system.proxy.ftp org.gnome.system.
└─10835 /usr/lib/aarch64-linux-gnu/libproxy/0.4.14/pxgsettings org.gnome.system.proxy org.gnome.system.proxy.http org.gnome.system.proxy.https org.gnome.system.proxy.ftp org.gnome.system.proxy.
okt 18 15:53:25 jetson python3[10808]: Traceback (most recent call last):
okt 18 15:53:25 jetson python3[10808]:   File "/home/pi/SSD/PersonDetectionService/trt_ssd_10.6.py", line 378, in
okt 18 15:53:25 jetson python3[10808]:     main()
okt 18 15:53:25 jetson python3[10808]:   File "/home/pi/SSD/PersonDetectionService/trt_ssd_10.6.py", line 364, in main
okt 18 15:53:25 jetson python3[10808]:     trt_ssd = TrtSSD(args.model, INPUT_HW)
okt 18 15:53:25 jetson python3[10808]:   File "/home/pi/SSD/PersonDetectionService/utils/ssd.py", line 84, in init
okt 18 15:53:25 jetson python3[10808]:     self.engine = self._load_engine()
okt 18 15:53:25 jetson python3[10808]:   File "/home/pi/SSD/PersonDetectionService/utils/ssd.py", line 56, in _load_engine
okt 18 15:53:25 jetson python3[10808]:     with open(TRTbin, 'rb') as f, trt.Runtime(self.trt_logger) as runtime:
okt 18 15:53:25 jetson python3[10808]: FileNotFoundError: [Errno 2] No such file or directory: 'ssd/TRT_ssd_mobilenet_v1_coco.bin'
pi@jetson:~/.config/systemd/user$ systemctl --user status python_demo_service
● python_demo_service.service - Python Demo Service
Loaded: loaded (/home/pi/.config/systemd/user/python_demo_service.service; static; vendor preset: enabled)
Active: failed (Result: core-dump) since Sun 2020-10-18 15:53:26 CEST; 1s ago
Process: 10808 ExecStart=/usr/bin/python3 /home/pi/SSD/PersonDetectionService/trt_ssd_10.6.py (code=dumped, signal=ABRT)
Main PID: 10808 (code=dumped, signal=ABRT)
okt 18 15:53:26 jetson python3[10808]: PyCUDA ERROR: The context stack was not empty upon module cleanup.
okt 18 15:53:26 jetson python3[10808]: -------------------------------------------------------------------
okt 18 15:53:26 jetson python3[10808]: A context was still active when the context stack was being
okt 18 15:53:26 jetson python3[10808]: cleaned up. At this point in our execution, CUDA may already
okt 18 15:53:26 jetson python3[10808]: have been deinitialized, so there is no way we can finish
okt 18 15:53:26 jetson python3[10808]: cleanly. The program will be aborted now.
okt 18 15:53:26 jetson python3[10808]: Use Context.pop() to avoid this problem.
okt 18 15:53:26 jetson python3[10808]: -------------------------------------------------------------------
okt 18 15:53:26 jetson systemd[7191]: python_demo_service.service: Main process exited, code=dumped, status=6/ABRT
okt 18 15:53:26 jetson systemd[7191]: python_demo_service.service: Failed with result 'core-dump'.
pi@jetson:~/.config/systemd/user$ `
For the service I only use:
[Unit]
Description=Python Demo Service
[Service]
ExecStart=/usr/bin/python3 /home/pi/SSD/PersonDetectionService/trt_ssd_10.6.py
Can you understand why I can start trt_ssd_10.6.py in the terminal, but I get errors when I start is as a service?

okt 18 15:53:25 jetson python3[10808]: FileNotFoundError: [Errno 2] No such file or directory: 'ssd/TRT_ssd_mobilenet_v1_coco.bin'

This means the program could not find the TensorRT engine file.
You could consider putting the file at a fixed location and use an absolute path in the code:
https://github.com/jkjung-avt/tensorrt_demos/blob/e039e0824876d46443aa19e9ad7cf8f7723c713e/utils/ssd.py#L54

Now it works! Thank you very much!!!!!!!!!!


Write demux wrapper

api: https://developers.google.com/speed/webp/docs/container-api#demux_api
build opt : http://www.linuxfromscratch.org/blfs/view/svn/general/libwebp.html

#11 associated

.bundle

https://github.com/auriamg/macdylibbundler/
https://stackoverflow.com/questions/1596945/building-osx-app-bundle
https://developer.apple.com/library/archive/documentation/CoreFoundation/Conceptual/CFBundles/BundleTypes/BundleTypes.html#//apple_ref/doc/uid/10000123i-CH101-SW1

https://github.com/netpyoung/unity.webp/pull/17


HotKeyBox gives wrong names to multimedia buttons

What steps will reproduce this issue?
when assigning a multimedia key to the HotKeyBox it will display it with wrong name e.g.
Key.MediaPlayPause > shows like this : http://puu.sh/q3BTW/da94bd8c0d.png

Expected outcome
it should view MediaPlayPause instead of G
--
Environment

MahApps.Metro v1.3.0-ALPHA176
Windows 10
Visual Studio 2015
.NET Framework 4.5.2

I can reproduce the issue. However, I don't know how to fix it at the moment. GetKeyNameText just returns G for MediaPlayPause. I'll have to investigate further...

Why not use Key.ToString() instead ?

Unfortunately, the Windows API can't help. I asked at StackOverflow, and there's no standard way to get the names of special keys such as the media buttons. 😫

then maybe create a simple dictionary of Key,string for special keys ,and then we can handle localization later ?


[FIX] - Adding references to tutorials

This PR aims to add a reference to tutorials at the bottom of the pages that can be linked with a tutorial

Just wanted to start a quick discussion - why not keep the sections as "Where to Go Next"? A lot of other pages still use it

That came from a Slack convo with Alberto. Dropped you a link in DM.


Implement Connection Management

Currently, connection management is mostly connection limits, which is currently implemented into PeerBook using NetworkBehaviour (based on https://github.com/libp2p/rust-libp2p/pull/3386). This behaviour does support whitelisting peers (which is more of an "allow list") to bypass the limits, but this is basic (and would likely be in a separate behaviour)
Down the road, we would need a realistic connection manager that would close least useful connections upon reaching a specific limit set. This would close connections to peers that are not guarded or protected by a protocol or behaviour such gossipsub, kad, or even in cases like PeerBook, though may need to check to determine the best course on checking the peer in other behaviours outside of what we managed (eg if the peer is actively subscribed to a topic in gossip, or apart of routing in kad, used for a relay, exchanging data via bitswap, etc). A scoring system would likely need to be implemented in some way.
TODO:

[ ] Protect peers manually added to PeerBook

This is done already, but the peer should not be counted towards the connection limit, while the current implementation, it does count.

[ ] Make sure tagged peers dont count towards the connection limit
[ ] Tag peers used in specific protocols

[ ] Peers subscribed to the same topic in gossipsub, untagging when unsubscribed
[ ] Peers exchanging blocks with bitswap, untagging when exchanging of blocks are done or have timed out.
[ ] Peers used as a relay.

[ ] Automatically remove tagged peers that disconnected (though protected peers will remain in place)

Connection management itself may be pushed to be handled a bit more in libp2p, although there will still be tracking and cleanup of connections later on here if we reach a threshold.


[BUG] islandora_entity_view causes WSOD for media without a source file

What steps does it take to reproduce the issue?

Create a media without a source file (this occurred with a broken migration).
Make sure the field_gemini_uri is set to display with the media's default mode
Visit the media's page and receive a WSOD

The error occurs because $source_file = $media_source_service->getSourceFile($entity); returns FALSE and the following line, $uri = $source_file->getFileUri(); is trying to call the getFileUri function on a boolean value.
To whom does it occur (anonymous visitor, editor, administrator)?
All
What did you expect to happen?
The media's page should be displayed
Which version of Islandora are you using?
2.x
Any related open or closed issues to this bug report?
None that I found.

Throwing in the following check after the getSourceFile call resolves the issue:
        if (!$source_file) {
          \Drupal::logger('islandora')->warning("Can't get source file for " . $entity->label() . "(" . $entity->id() . ")");
          return;
        }


[Enhancement] Remove minidump

Proposed changes
Issue Number: close #xxx
Problem Summary:
Minidump is not useful in a real online environment, instead it introduces additional errors.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Does we should remove breakpad?

done, you are right

Minidump is not useful in a real online environment, instead it introduces additional errors.

@xinyiZzz What is the additional errors introduced by minidump? I think minidump is still useful in scenarios where coredump is too large to be enabled, what's the alternative to minidump in those scenarios?


sql/spanner: Add spanner driver

This is a WIP PR that starts adding spanner support.
Contributions welcome!
Current feature support:
Inspection:

[x] support primary keys
[x] support foreign keys
[x] support generated columns
[ ] support views
[ ] support checks
[ ] support table interleaving
[ ] support change streams
[ ] support TTL

Added Check inspection.

As this branch grew kind of unwieldy, we're doing a cleaner approach over on #1171


Development server crashes after upgrading to 3.14

Environment

Operating System: Darwin
Node Version:     v23.1.0
Nuxt Version:     3.14.159
CLI Version:      3.15.0
Nitro Version:    2.10.4
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      default
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-abcsb7?file=server%2Fplugins%2FexamplePlugin.ts
Describe the bug
After upgrading from Nuxt 3.13 to 3.14, the development server crashes on launch, right after ✔ Nuxt Nitro server built in 1293 ms.
I managed to isolate the crash to a server plugin using drizzle-orm on server launch, which worked fine until the upgrade.
The bug occurs only in dev mode, everything works fine when running nuxt build then node .output/server/index.mjs
Additional context
No response
Logs
ℹ Running with compatibility version 4                                                                                19:25:54
ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                                                                                nuxt:tailwindcss 19:25:54
✔ Vite client built in 64ms                                                                                19:25:55
✔ Vite server built in 1632ms                                                                                19:25:56
✔ Nuxt Nitro server built in 1293 ms                                                                                nitro 19:25:58

 ERROR  [worker reload] [worker init] .nuxt/dev/index.mjs failed

I went back to Nuxt 3.13.2 and Nitro 2.9.7 and everything seems to work again

I went back to Nuxt 3.13.2 and Nitro 2.9.7 and everything seems to work again

Same here, the only way to get all of my apps to work again is to downgrade to the same versions you mentioned.

I had the same issue recently. I fixed it after searching for cjs require imports in project, (like a require "fs", require "path" etc).
I found a few, replaced them for ESM imports, then error was gone, even at the latest Nuxt and Nitro (Nuxt 3.14.1592 with Nitro 2.10.4)
Also I noticed that the error is gone after removing the field "type: module" in package.json. But this was not an option for me.
Error occurred only in dev mode, build was fine all along.


3-generic-metrics: Allow for any metric to be retrieved instead of hardcoding cpu-utilization

Types of changes

[x] Enhancement (project structure, spelling, grammar, formatting)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

A description of the changes proposed in the Pull Request
This PR allows for any metric to be retrieved instead of hardcoding cpu-utilization. It introduces two new globalConfigs in place of cpu-utilization-metric-name, which are metric and output-metric-name.

Creating a mock of the DD api and actually validating the config looks to be something we still need to do, but I wasn’t sure if  we felt we needed me to take that on as part of this ticket. Those both may require separate tickets themselves.

Creating a mock of the DD api and actually validating the config looks to be something we still need to do, but I wasn’t sure if we felt we needed me to take that on as part of this ticket. Those both may require separate tickets themselves.

I agree that those would both be worth doing, but doesn't need to be done here. Do you want to cut those? If not I'm happy to as well.


How can I use it as a simplest CMS?

Excuse me if this question feels too much newbie to you, what I want to do is to make CPPWebFramework to work as a simple CMS. Just having a functionality to post articles and surf them. Can you please give me some hints how to do this or upload an example which may demonstrate CMS functionality please?

As I have never programmed a CMS myself so I don't know how the articles will be fetched from Database. I don't know how a link will be resolved to some data inside database. I am also confused that when the user will click on a link in the web page for example http://localhost:8080/this-is-article-heading then how I will server find the article with heading "this is article heading"? Would I need to search articles in the database using heading? If this does not make sense to you then please excuse for this nonsense. Actually I am myself confused so I don't know how to phrase the question.

This is not CMS software... as clearly stated on the web site.

Hello shujaatak, the CWF is not a CMS software.
Thank you.

Hello shujaatak, the CWF is not a CMS software.
Thank you.


The provider requires an AWS provider of the same version for the node sdk

Hello!

Issue details
This module requires the @pulumi/aws provider somehow, seemingly of the same version.
I have a Project that uses both AWS and Vault resources, using the two separate providers. My providers list locally looks like this:

When I try to run pulumi up, I see that I need the aws provider v4.5.1:

But as you can see in the image above, I want to be using aws provider 4.24.1. If I check the state file, the last deploy used 4.24.1, and in my package.json I explicitly use provider version 4.24.1:

In my statefile, the string 4.5.1 never appears, and 4.5.0 only appears in reference to this vault provider:

Running the suggested command, pulumi plugin install resource aws v4.5.1, makes everything happy, but I do not understand why this is required, or how this error is generated.
If I change the version of this package from 4.5.1 to 4.5.0, I see the same error message, but with 4.5.0 instead of 4.5.1, so the error definitely is related to this package somehow.
Steps to reproduce

Create a Project using both aws and vault resources with different provider versions
Try to create resources
The version of the @pulumi/vault plugin will be required from the @pulumi/aws provider

Expected: This provider would not change what AWS providers are required
Actual: This package requires a specific AWS provider

@dmattia Are you still seeing this issue?

Since we haven't heard back and the issue is quite old, I'll go ahead and close this issue as stale


NIFI-6523 MergeRecords properties should accept expression language variables

Description of PR

The following MergeContent attributes now accept expression language

-- Correlation Attribute Name: Accepts both from Variable Registry and Flowfile Attributes (just like MergeContent)
-- Minimum Bin Size: Accepts from Variable Registry
-- Maximum Bin Size: Accepts from Variable Registry
-- Max Bin Age: Accepts from Variable Registry
-- Maximum Number of Bins: Accepts from Variable Registry

TestMergeContent has been a little reorganized in order to have tests which validate the same component to be into the same method

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

cc @mgaido91 @ijokarumawak
Could you please review this PR?
Thanks!

@MikeThomsen Hello Mike, I opened this PR (along with the related #3634 ) some time ago, me and @mgaido91 were not sure about the test refactoring I've done here (and in the related one) since, despite being useful and related to the opened issue, it brings some entropy.
Do you think it is ok to keep this implementation, or would it be better to focus only on the minimum necessary and open another PR for the test refactor
Thanks

@adarmiento recommend you get on the nifi-dev list and ask for assistance on this one. I'm full on my schedule with other reviews and client work, plus I'm not that familiar with this processor.

@adarmiento recommend you get on the nifi-dev list and ask for assistance on this one. I'm full on my schedule with other reviews and client work, plus I'm not that familiar with this processor.

Ok, thanks for the advice!

cc @markap14 @pvillard31 since you worked on this processor, could you please take a look at this PR, and in particular could you advise me about this question?


Draft of using a FieldExtension trait

This doesn't compile; it gets stuck at StarkConfig trying to satisfy TwoAdicField-related traits. Wasn't sure the best way to fix it, but it's not important for the moment.

Closing as this idea was not pursued.


Add pipectl install controlplane command

What this PR does / why we need it:
Note: For this implementation, the pipectl does not install any external tools which are required to install controlplane (such as helm) since users' local may already have the tools installed or they have many different versions but want to use specified one from a path. So I think we should not use pipectl to install tools (like helm) but lets users specified what to use. Will rethink about it and maybe support installing required tools if does not exist later.
Comamnd which used to install PipeCD controlplane in quickstart mode
$ pipectl install controlplane --quickstart --version v0.35.0

Current help print
$ pipectl install controlplane -h
Install PipeCD Control Plane.

Usage:
  pipectl install controlplane [flags]

Flags:
      --cloud-sql-service-account-file string   The path to service account which used to access controlplane Google cloud SQL database (if using).
      --config-file string                      The path to the Control Plane configuration file.
      --encryption-key-file string              The path to the Control Plane encryption key file.
      --firestore-service-account-file string   The path to service account which used to access controlplane Firestore database (if using).
      --gcs-service-account-file string         The path to service account which used to access controlplane Google Cloud Storage service (if using).
  -h, --help                                    help for controlplane
      --internal-tls-cert-file string           The path to internal TLS certificate file (if using).
      --internal-tls-key-file string            The path to internal TLS key file (if using).
      --minio-access-key-file string            The path to access key which used to connect Minio filestore (if using).
      --minio-secret-key-file string            The path to secret key which used to connect Minio filestore (if using).
      --namespace string                        The cluster namespace where to install Control Plane. Default is 'default'. (default "default")
      --quickstart                              Whether installing the Control Plane in quickstart mode.
      --tools-dir string                        The path to directory where to install needed tools such as helm. (default "/Users/s12228/.piped/tools")
      --version string                          The Control Plane version. Default is the version of pipectl. (default "v0.34.2-15-gca1b82f-dirty")

Global Flags:
      --log-encoding string                The encoding type for logger [json|console|humanize]. (default "humanize")
      --log-level string                   The minimum enabled logging level. (default "info")
      --metrics                            Whether metrics is enabled or not. (default true)
      --profile                            If true enables uploading the profiles to Stackdriver.
      --profile-debug-logging              If true enables logging debug information of profiler.
      --profiler-credentials-file string   The path to the credentials file using while sending profiles to Stackdriver.

Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

Add ability to install PipeCD control plane via pipectl cli

Note: For this implementation, the pipectl does not install any external tools which are required to install controlplane (such as helm) since users' local may already have the tools installed or they have many different versions but want to use specified one from a path. So I think we should not use pipectl to install tools (like helm) but lets users specified what to use. Will rethink about it and maybe support installing required tools if does not exist later.

For the sake of simplicity, agree about that. We just log the error when any required dependency was not installed before.


PWA What's New update for Edge 101 and 105

This updates the PWA What's New page with new features from 101 and 105.
This PR is based on @diekus' PR #2252.
His original commit is in this PR, and I only added another one on top of it to make the docs more consistent with the rest of the repo.
Rendered page for review: https://review.learn.microsoft.com/en-us/microsoft-edge/progressive-web-apps-chromium/whats-new/pwa?branch=pr-en-us-2259

Learn Build status updates of commit 0d76e19:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/whats-new/media/custom-title-bar.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/media/pwa-notifications.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/media/sidebar-search.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/media/sidebar-search-results.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/pwa.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks @mikehoffms, I believe I've addressed all of your comments in the last commits.

Learn Build status updates of commit 9baf396:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/whats-new/media/custom-title-bar.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/media/pwa-notifications.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/media/sidebar-search.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/media/sidebar-search-results.png
:white_check_mark:Succeeded
View

microsoft-edge/progressive-web-apps-chromium/whats-new/pwa.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Cant find the SVG icons that are used in the project

HI, it is a great template, but can you share what SVG icon set have you used in this project, where can i find them?  I would like  to use the same in my project.

HI, it is a great template, but can you share what SVG icon set have you used in this project, where can i find them? I would like to use the same in my project.

Hey, i used SVG icons from here: http://svgicons.sparkk.fr
I don't think i used only one specific set, i might have taken a couple from a few different sets.

Thanks for the answer :)

Cool, I was wondering the exact same. This is really a fantastic theme. Any plans of a dark version ?

Cool, I was wondering the exact same. This is really a fantastic theme. Any plans of a dark version ?

Since people are finding this one useful, i am planning on building a totally new one which will be way more advanced & using tailwind 2.0 which will incorporate dark mode.However, i don't have a time frame for when that will happen.


refactor: NRF Server & processor

Server

[x] copy server side code
[x] implement server structure & refactor api_{SVC}.go

Processor

[x] new processor/processor.go
[x] modify accesstoken.go , nfdiscovery.go & nfmanagement.go

Hi @KunLee76
Please use go mod tidy to renew go.mod & go.sum.
Besides, please help to fix errors reported by CI workflow.
Thanks!

Remember to remove unused code.


Test explorer input uses wrong background color

open test explorer
notice that the input has the wrong background color, e.g it has two background colors

What theme are you using? I'm not seeing any backgrounds being applied to that menu / menu bar 🤔

Like, any light theme?
https://github.com/microsoft/vscode/assets/1794099/7afd478a-74e4-4691-8aca-bc6198748acb

Still confused 😕  on the latest Insiders for me:

Does it reproduce on a fresh user-data-dir for you?

I see. It doesn't show with an empty profile but I also don't have any setting that screams test explorer background color

Could you bisect your user settings or share them so I can play around? 🙂

This seems to be because I have the test explorer in the secondary sidebar. There is some settings that controls this but I have no clue which one it is

CSS is fun 🙃


Cotary Create: Step 4 styling #305

#305

How do I set the size of the spinner we're using? @mrrobot16, @cbrzn

@Flash-Git fixed the sizing:

I assumed we would have a size property for it. Thanks @dOrgJelli,


[Warning] Warning: The 'kotlin-android-extensions' Gradle plugin is deprecated.

I am using kotlinVersion with 1.4.30
When I build android with react-native-notifications
program says
Warning: The 'kotlin-android-extensions' Gradle plugin is deprecated. Please use this migration guide (https://goo.gle/kotlin-android-extensions-deprecation) to start working with View Binding (https://developer.android.com/topic/libraries/view-binding) and the 'kotlin-parcelize' plugin.

+1

@JaeWangL did you figure out a solution to this?


Added Bullet and BulletsManager

Please, add pull request description as is required by point 7 of contribution workflow guidelines

Bullet
Adds Bullet (src/object/Bullet.js) with supporting rotation to direction of movement.
Parameters:

scene [^1]
sprite
location - location of bullet spawn
scale, direction
velocity
distance - max distance of bullet life
blockedLayers - layer and objects, that collides with bullet,
falling speed - speed of sprite dissolution in end of bullet life
depth - zIndex in scene.

BulletsManager
Adds BulletsManager (src/systems/BulletsManager.js) to move bullet and filter inactive ones.
Has methods:
preload to load bullet sprites in scene.
create to init blocked layers and depth of scene for bullets.
update to move and filter bullets. Needs deltaTime from scene update method.
spawnBullet to create bullet in scene. Has the same parameters like Bullet exclude blockedLayers and depth.
Additional Info:
blockedLayers - can be layer of scene or array of objects like gameObjects in starting scene.
Demo scene
Adds demo scene  (src/scenes/bullets-demo.js) to demonstrate bullets colliding with object, that was created on creation step of scene, and object, that spawn after few time.
[^1]: Required parameter.

lgtm 🚬🗿


update codecov tooling

What did you implement:

Closes: #114
How did you verify your change:
travis run
What (if anything) would need to be called out in the CHANGELOG for the next release:

Coverage remained the same at 0.0% when pulling 7ece4f4b108744202e4603db8cb3a305a5186bc4 on tarpaulin-codecov into 92106b758e64ca1f9637272acea0eb83788df5b6 on master.


ORC-1298:[C++] Support dedicated ColumnVectorBatch of numeric types

What changes were proposed in this pull request?

This patch implements int8/int16/int32/float buffer and refactors the RLE encoder/decoder, IntegerColumnWriter/Reader, and DoubleColumnWriter/Reader.
Why are the changes needed?

For the context of ORC-1298, this change makes user no need to focus on overflow.
How was this patch tested?

Add the WriterTest.testWriterFixedWidthNumericVectorBatch UT.

Thanks @luffy-zh for the contribution!
cc @stiga-huang @coderex2522


Use Cargo workspace

To reduce deduplication between our two rust crates, use a workspace to manage shared dependencies.

There is some issue with the Windows CI build - back to Draft status until resolved (something along setting CARGO_TARGET_DIR)

LGTM; I cleaned up the old lockfiles as well.
It doesn't seem to be affecting the CI, but make dev produces a strange error for me locally:
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.05s
📦 Built wheel for abi3 Python ≥ 3.9 to /var/folders/k6/zk7vnkms1m73w0vrxjm7b2s40000gn/T/.tmprjlaQs/rfc3161_client-0.0.5-cp39-abi3-macosx_11_0_arm64.whl
✏  Setting installed package as editable
💥 maturin failed
  Caused by: failed to list package files

@DarkaMaul have you seen that before?

Never in my local tests.
It seems, it's coming from :
https://github.com/PyO3/maturin/blob/cde210bac7e4e9bef14ad8c823b3adc09d399934/src/develop.rs#L263-L286
fn fix_direct_url(
    build_context: &BuildContext,
    python: &Path,
    install_backend: &InstallBackend,
) -> Result<()> {
    println!("✏  Setting installed package as editable");
    let mut cmd = install_backend.make_command(python);
    let cmd = cmd.args(["show", "--files", &build_context.metadata23.name]);
    debug!("running {:?}", cmd);
    let output = cmd.output()?;
    ensure!(output.status.success(), "failed to list package files");
...

Which runs:
uv pip show --files rfc3161-client

Maybe the output of this command could help us in understanding what happens?

Ah, this was caused by me being on uv 0.4.x. Upgrading fixed it!


Documentation tweaks

I made a small fix to pluralize "templates" where it was previously written:

[...] to different profiles and template in machine-specific values.

Additionally, I added syntax highlighting for the code example that follows.

Thanks!


add support of luaecs

The luaecs project is:
https://github.com/cloudwu/luaecs

I think the config ecs.lua is too common.

'ok, I modified it to use ecs.world'

Thank you!


Change dashes to double dashes in command args.

@alexaltair This is incomplete, there's an etcd file in the compute node setup as well. =)

@Lukasa Got it.


Alert not supported in webamp

Hi I'm new here how to calibrate my Winamp on android and to save the playlist and totally use it as its full performance! Since I don't know nothing about programming so I will need detailed information! Thank you so much!

This project is not related to Winamp or Winamp on Android.


Append compiler name to image

We need to append the compiler name to image as we might build both Intel and GNU

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:de4cd85911bf3c01ae5cf41572e8fde682fa738f

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:b7fd561045267e5fede86affebc838d1a31c41fd

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:3afa1131fcdcb311e674315cdb129388ff4cdc93

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=1.13.1


Parameters for the orchestration and activities not working correctly

Passing parameters to the orchestration or the activities using tuples is supported by the in-process mode, but this durable isolated library is somehow not working with that.
Repro steps

Download repo https://github.com/elias-rod/Sandbox.git
Checkout to branch failing-tuples
Go to Function/MyFunctionApp/MyFunctionApp.sln
Put breakpoints to validate the value of the parameters is null
Execute
Go to Function/OldOrchestration/OldOrchestration.sln
Put breakpoints to validate the values of the parameters are correct
Execute

Orchestrator parameter failing

Activities parameter failing

@elias-rod I theorize this may be the difference in serialization abilities of Newtonsoft.Json vs System.Text.Json. In-Proc uses newtonsoft, which may have serialization support. Isolated uses STJ by default, which may not support value tuple deserialization.
You can easily workaround this by defining your own type. I recommend record structs, they are clean concise:
public record struct MyInput(string FirstProperty, string SecondProperty);

@elias-rod found the root issue. See https://github.com/dotnet/runtime/issues/70352
You can work around this by adding this to your ConfigureServices in worker startup:
services.Configure<JsonSerializerOptions>(o => o.IncludeFields = true);

However, this might have serious implications on other serialization! I actually do not recommend doing this, instead use record structs as I suggested above.

Closing as this is external to us - this is System.Text.Json behavior.


Upgrade to Airbase 45 to fix Java 8 compatibility

Previous version of Jacoco could not parse Java8 byte code and would fail the build.
Fixes #46

There are build errors. Otherwise, looks good.

@martint can you take a look at the last two commits.  They update the travis environment

Looks good


Correct name and abstract and update DOI and main GitHub page.

Kirill A. Vishnyakov -> Kirill Vishnyakov


Create a separated query endpoint for multi-stage query engine

This majorly beneficial for data Viz tooling like superset or retool which can specify a different query endpoint for multi-stage query engine without making any frontend change.
E.g. current superset connection string is: pinot://pinot-broker:8099/query/sql?controller=http://pinot-controller:9000/
Link to Add Pinot Database in Superset
We can create a new endpoint for multi-stage query at pinot://pinot-broker:8099/query/v2/sql?controller=http://pinot-controller:9000/
Changes proposed:

Add new query endpoint for multi-stage query engine in controller and broker
Follow up: Update java/python/golang clients to use the new endpoint.

I'm not a big fan of v2. If we are not going to change to v3 (hopefully not), maybe we can just use /query and eventually deprecate /query/sql

Please assign it to me if not urgent.