gcov test for distro kernel

gcov test for distro kernel

@Naresh-ibm run logs please

closing this with reference #764


[ASCII-2262] Extend UnmarshalKey unit tests around conversion errors and maps/structs

What does this PR do?
Additional follow up to https://github.com/DataDog/datadog-agent/pull/29604 that includes expanded test suite around non-scalar types, conversion errors, and struct tags for the new UnmarshalKey behavior
Motivation
Describe how to test/QA your changes
No QA, just adding additional unit tests
Possible Drawbacks / Trade-offs
Additional Notes

/merge


fix typo in snapshot log/comments

typo fix

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ozevren
If they are not already assigned, you can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/mcp/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @lichuqiang. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/approve
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/mcp/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks, @ozevren , I think also need "/ok-to-test" :)
BTW, I'm new to istio, and hope to be involved in the contribution.
So, wondering if you have anything that I can help? Or any suggestion for new guys?

/ok-to-test

The best place to start looking at is Github issues that are tagged with community/help-wanted and community/good-first-issue. The issues probably need some cleanup, so even if you want to tackle the issue, I'd suggest first sending a message to the issue/area owner to get more details.
Depending on your interest level and area, you can ping WG area leads to see if there is more substantial work you can contribute to. I am the WG area lead for config, so if you're interested in this area I can help. The best place to reach the dev community is slack and mailing list. You can find various communication paths listed at the bottom of the main page https://istio.io/

The best place to start looking at is Github issues that are tagged with community/help-wanted and community/good-first-issue. The issues probably need some cleanup, so even if you want to tackle the issue, I'd suggest first sending a message to the issue/area owner to get more details.

Yeah, I used to working on kubernetes for a time, so I basically know how the community works.
I guess what I need most is, like you said, the way to contact to WG  leads of my interested areas.
I tried to join in the istio slack channel, but it seems only istio members have the right to join it :(

I am the WG area lead for config, so if you're interested in this area I can help

That would be great. In fact I tried to send you message on slack, but seems you are always offline.
So, is there a quicker way to talk to you?

I am generally available in Slack (though notifications for messages are blocked due to snooze hours). Can you try again? If it works, we can stop abusing Github comments. :)
You can also try the istio-dev mailing list.

/retest

@lichuqiang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I am generally available in Slack (though notifications for messages are blocked due to snooze hours)

Oh, really? Then maybe I've mistaken someone else for you.
Is "ozben" your slack ID? If not, can you send me a message on slack if free?(mine is same to my github ID: lichuqiang)
Thanks a lot!

@lichuqiang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@lichuqiang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test e2e-simple

prow/istio-unit-tests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lint

/retest

@lichuqiang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@lichuqiang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
02c7deb0a23f8f7822394fc1144d2a6ba7056cd8
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Adds elements to restricted data cache only if cache already exists

Hotfix for #3427.
Prevents corrupt cache if restricted data cache is cleared while runtime.
Successfully tested with online instance.
@ostcar @FinnStutzenstein Please review.

@ostcar I'll merge this hotfix now. Please try to implement, as discussed, a better solution with a real cache api (see #3428).


Looks to be broken with 1.9 version of avro

Can confirm. Was hoping this would solve the old union parsing issue.

Is the fix for version 1.9 released?


JIRA: AOAF-424

Description: Added language parameter in the redirect url.

@ch-skathirvel can you add a unit test to demonstrate that the url contains the lang query? Just need to modify line 23 of https://github.com/companieshouse/presenter-account-web/blob/main/test/routers/home.unit.ts to check for the lang query as well

CI: No security warnings found


Cannot test local video loopback in Licode Basic Example

Hi,
I ran
https://mydomain.tld:3004/
for which I am unable to see my own local stream video.
After opening those mentioned URls, browser asks for sharing Camera and Audio method.
Once clicked Allow or share, It says "- Mic and Cam OK" in a text area.
However, I am unable to see my own local video loopback.

Need help

You might get stressed because of this problem. But there aren't any logs to solve it so no one who help you also.  I think you didn't install licode in clean environment. I recommend that how about docker to deploy Licode?  Following link contains useful infomation https://github.com/ging/licode/issues/397

Hi @ankur0101,
Can you give us more information about the logs in the web browser console?

@nachoaguadoc I will upload licode config and basic example js files shortly.

@ankur0101 Is everything solved by now?


[Enhancement]: Multi chain support

Allow dapps to connect to a specific chain
Allow dapps to add a custom chain via wallet_addEthereumChain
Allow dapps to to switch chains via wallet_switchEthereumChain
support chainChanged event

@avimak https://github.com/argentlabs/argent-x/pull/639 Check this out. We are already working on the things you mentioned about. Should be done by the end of this week.

You want me to use wallet_addEthereumChain? I have used wallet_addStarknetChain intentionally. I have read eip-3085. If @janek26 and other wants, we can change it to the standard as described by eip-3085. But as we have different API, not 1:1 like using baseURL and id, I think it's okay to use a Starknet Specific name.

Hey! As @dhruvkelawala mentioned adding an Ethereum chain vs adding a starknet chain rely on different details.
I think it's best to clearly separate those methods by naming, as they rely on different cryptographic curves aswell. You could imagine an EVM compatible RPC endpoint on Starknet in the future, and it would be nice to have the native namings available to them in that case.

thanks @janek26 , could u guys pls elaborate on the following diff from eip-3085?
// new fields; why are they needed and what's their alternative on L1
id: string;
baseUrl: string;
accountImplementation?: string;

rpcUrl?: string; vs rpcUrls?: string[]; (L1's)
blockExplorerUrl?: string; vs blockExplorerUrls?: string (L1's)

thanks @janek26 , could u guys pls elaborate on the following diff from eip-3085?
// new fields; why are they needed and what's their alternative on L1
id: string;
baseUrl: string;
accountImplementation?: string;

rpcUrl?: string; vs rpcUrls?: string[]; (L1's)
blockExplorerUrl?: string; vs blockExplorerUrls?: string (L1's)

@janek26 , @dhruvkelawala I'm also interested in the answer you could give to @avimak
From what I read here is what I understand :

rpcUrl is not used, so I guess it could have been removed from he code
baseUrl is the equivalent to rpcUrls (but we provider only one url), it seems to be based on starknetjs using baseUrl when defining a Provider.
As a note about using a list of rpcUrls or a single url, from what I saw on MetaMask, they use the first valid url from the list, so I don't really understand why to propose a list

Hey guys, please note that as discussed with Louis we initiated the SIMPs repository.
Feel free to start a simp and include this issue as the discussion link of the SIMP.
https://github.com/CairOpen/SIMPs

Hey guys, please note that as discussed with Louis we initiated the SIMPs repository. Feel free to start a simp and include this issue as the discussion link of the SIMP. https://github.com/CairOpen/SIMPs

I'm working on it, among other SIMPs. :slightly_smiling_face:
Coming very soon.

Implemented in v2 branch


fix(css): adds font-roboto.css

Short description of what this resolves:
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x / 3.x / 4.x
Fixes: #

are you sure you need only
font-style: normal;
font-weight: 400;

I think I saw also font-weight: 500 used in your components.

@manucorporat Do we want to get this in for 4.1?

Closing in favor of https://github.com/ionic-team/ionic/pull/18934, discussed this with Manu!
I will add you as a co-author @mburger81. Thank you!


Error at line 39

Hi Frederic,
Thanks for a great bit of code.
Unfortunately, I am getting an error at line 39:
const fetchRequest = UrlFetchApp.fetch(url);
in the Export Facebook Ads Data to Google Sheets file.
It worked perfectly until 10 days ago, and then it just stopped. I did not change anything in the set-up or with my account info. I can download the FB data in a CSV with no problems, but the data will not export into the Sheet.
Do you have any ideas what the issue might be?
Thanks
Zak

This is the error: Exception: Unexpected error: https://www.facebook.com/ads/ads_insights/export_report?report_run_id=8777747&format=csv&access_token=EAALZBMeUhHX0 (line 39, file "pastedata_rta")

This is the code for the file:
/**
*

Export Facebook Ads Data to Google Sheets

Pushes the Facebook asynchronous report to Google Sheets

Version: 2.2

Google Apps Script maintained by Frederic Harnois
<EMAIL_ADDRESS>

**/
// MODIFY YOUR SETTINGS HERE //
// url of the google sheets where the report will be
const SPREADSHEET_URL = 'https://docs.google.com/spreadsheets/d/1nkluzUdvvxo7FrwAw/edit#gid=0'
// name of the sheet where the report will be
const TAB_NAME = 'RTA'
// DO NOT MODIFY ANYTHING BELOW //
function getFacebookReport() {
// Selects the chosen sheet and tab
const ss = SpreadsheetApp.openByUrl(SPREADSHEET_URL);
const sheet = ss.getSheetByName(TAB_NAME);
// Clears the sheet
sheet.clear();
// Gets the Facebook report run ID
const cache = CacheService.getScriptCache();
const reportId = cache.get('campaign-report-id');
// Fetches the report as a csv file
const url = https://www.facebook.com/ads/ads_insights/export_report?report_run_id=${reportId}&format=csv&access_token=${TOKEN};
const fetchRequest = UrlFetchApp.fetch(url);
const results = Utilities.parseCsv(fetchRequest);
// Pastes the csv file in the sheet
sheet.getRange(1,1, results.length, results[0].length).setValues(results);
}

It's this Google bug:
https://issuetracker.google.com/issues/175141974
The fetch part doesn't work with csv files
UrlFetchApp.fetch(url);  --this doesn't work
Lots of users affected by this, go star the issue
On Mon, Dec 14, 2020 at 9:08 AM zakloh8888<EMAIL_ADDRESS>wrote:

This is the code for the file:
/**
*

Export Facebook Ads Data to Google Sheets

Pushes the Facebook asynchronous report to Google Sheets

Version: 2.2

Google Apps Script maintained by Frederic Harnois
<EMAIL_ADDRESS>

**/
// MODIFY YOUR SETTINGS HERE //
// url of the google sheets where the report will be
const SPREADSHEET_URL = '
https://docs.google.com/spreadsheets/d/1nkluzUdvvxo7FrwAw/edit#gid=0'
// name of the sheet where the report will be
const TAB_NAME = 'RTA'
// DO NOT MODIFY ANYTHING BELOW //
function getFacebookReport() {
// Selects the chosen sheet and tab
const ss = SpreadsheetApp.openByUrl(SPREADSHEET_URL);
const sheet = ss.getSheetByName(TAB_NAME);
// Clears the sheet
sheet.clear();
// Gets the Facebook report run ID
const cache = CacheService.getScriptCache();
const reportId = cache.get('campaign-report-id');
// Fetches the report as a csv file
const url =
https://www.facebook.com/ads/ads_insights/export_report?report_run_id=${reportId}&format=csv&access_token=${TOKEN}
;
const fetchRequest = UrlFetchApp.fetch(url);
const results = Utilities.parseCsv(fetchRequest);
// Pastes the csv file in the sheet
sheet.getRange(1,1, results.length, results[0].length).setValues(results);
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fredericharnois/facebook-ads-reporting-google-apps-script/issues/23#issuecomment-744464145,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALP4EBND2LGM6GDQGDEYVATSUYL7DANCNFSM4U2ZL3YQ
.

Ah thank you for the reply. So this is affecting you as well? Is there a work around at all?

There are some suggestions in the comments of that issue tracker, and yes,
I'm also affected by this. Haven't found anything that helps but ymmv
On Mon, Dec 14, 2020 at 9:30 AM zakloh8888<EMAIL_ADDRESS>wrote:

Ah thank you for the reply. So this is affecting you as well? Is there a
work around at all?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fredericharnois/facebook-ads-reporting-google-apps-script/issues/23#issuecomment-744477850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALP4EBNSP3IF5DCFHIKIDPTSUYOPHANCNFSM4U2ZL3YQ
.

OK, thanks! Back to cut and paste for now :(

Thanks for sharing this. Im also running into the error on line 39. Any work around here?

I've gotten the same problem


[SPARK-42541][CONNECT] Support Pivot with provided pivot column values

What changes were proposed in this pull request?

Support Pivot with provided pivot column values. Not supporting Pivot without providing column values because that requires to do max value check which depends on the implementation of Spark configuration in Spark Connect.
Why are the changes needed?

API coverage
Does this PR introduce any user-facing change?

NO
How was this patch tested?

UT

@hvanhovell

Merging


undefined is not an object (evaluating 'PropTypes.oneOfType')

Steps to reproduce

Installed the module according to README
Put it into a working simple app I have in a component like so:

import React from 'react'
import { StyleSheet, View } from 'react-native'
import paddingTop from '../constants/paddingTop'
import Camera from 'react-native-camera'
const { Component, PropTypes } = React

export default class Capture extends Component {
  static propTypes = {
    navigator: PropTypes.object.isRequired
  };

  render() {
    return (
      <View style={[styles.container, { paddingTop }]}>
        <Camera
          ref="camera"
          style={styles.camera}
        />
      </View>
    )
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    flexDirection: 'column'
  },
  camera: {
    flex: 1
  }
})

Expected behaviour
The camera module should work
Actual behaviour
I get the following errors:
2016-05-29 19:19:41.273 [error][tid:com.facebook.react.JavaScript] undefined is not an object (evaluating 'PropTypes.oneOfType')
2016-05-29 19:19:41.274 [fatal][tid:com.facebook.react.RCTExceptionsManagerQueue] Unhandled JS Exception: undefined is not an object (evaluating 'PropTypes.oneOfType')
2016-05-29 19:19:41.294 [error][tid:com.facebook.react.JavaScript] Module AppRegistry is not a registered callable module.
2016-05-29 19:19:41.294 [fatal][tid:com.facebook.react.RCTExceptionsManagerQueue] Unhandled JS Exception: Module AppRegistry is not a registered callable module.

I don't even have to use the <Camera /> component to get this, all I need to do is import it.
Environment

Node.js version: node v5.6.0
React Native version<EMAIL_ADDRESS>React Native platform + platform version: iOS

react-native-camera
Version<EMAIL_ADDRESS>

I think I have a clue here:
http://stackoverflow.com/questions/37320748/upgrading-to-26-broke-the-dependencies
Apparently starting from<EMAIL_ADDRESS>it should be var React = require('react') and not var React = require('react-native')

master is pretty unstable. We will be releasing v1 beta soon, but we need a help to get things stable and production ready.
Not sure what your timeline is but I would install from master for now. We do not have any more breaking changes planned so the api should be pretty stable.

It seems if you've installed the module from npm install react-native-camera@https://github.com/lwansbrough/react-native-camera.git --save you should have a working version. If you've installed from npm install react-native-camera --save, you'll get the error you mentioned with RN v0.26.

@rt2zz what is the timeline that this component looks like for now?
Is iOS any more stable than Android?
What kind of help do you need?

Can you test with this branch #304, it seems stable on both platforms to me.

Actually, on iOS so far I had absolutely no issues. I'll test #304 later on, but so far it's been fantastic.

I had the same error, running on react-native 0.27 but as @pixelgamma mentioned installing with npm install react-native-camera@https://github.com/lwansbrough/react-native-camera.git --save fixed the issue

also receiving this error with react-native 0.46.
"dependencies": {
    "react": "16.0.0-alpha.12",
    "react-native": "0.46.4",
    "react-native-camera": "^0.9.4",
    "react-navigation": "^1.0.0-beta.12"
  },
  "devDependencies": {
    "babel-jest": "20.0.3",
    "babel-preset-react-native": "2.1.0",
    "jest": "20.0.4",
    "react-test-renderer": "16.0.0-alpha.12",
    "reactotron-react-native": "^1.12.1"
  },

https://facebook.github.io/react/docs/typechecking-with-proptypes.html
Should we be importing like import PropTypes from 'prop-types';

I think #811 PR fixes this.  I'm testing it now....

I have this error
Install prop-types no resolve this.
my package.json
"dependencies": {
		"prop-types": "^15.6.0",
		"react": "16.0.0",
		"react-native": "0.50.4",
		"react-native-camera": "^0.6.0"
	},
	"devDependencies": {
		"babel-jest": "21.2.0",
		"babel-preset-react-native": "4.0.0",
		"jest": "21.2.1",
		"react-test-renderer": "16.0.0"
	},

error :
 react-native-camera/index.js:75:22

this worked for me:
in node_modules\react-native-camera\index.js

removed PropTypes from import React, { Component, PropTypes } from 'react';
add import PropTypes from 'prop-types';

in package.json

add in "dependencies" "prop-types": "^15.6.0"

in QRCode.js changed to :
QRCode.js.txt

Same error as @hazeeb.


Shortcut code over shortcuts

Adds a <shortcut> tag to our project. This could be processed by a plugin in the future or preprocessed into docs.
I will keep working on this PR. Please leave a comment but don't merge.
Using the VS Keymap as the placeholder for tags. https://www.jetbrains.com/help/rider/Reference_Keymap_VS.html?keymap=_default

This feels like the way to go, except for R# the help seems to use different names :-/
https://www.jetbrains.com/help/resharper/Reference__Keyboard_Shortcuts.html

Although we can probably map those using the fallback shortcut in many cases


Fix v0.5.2.rst whatsnew text

Restoring text that was deleted by mistake
pvlib python pull request guidelines
Thank you for your contribution to pvlib python!
You may submit a pull request with your code at any stage of completion, however, before the code can be merged the following items must be addressed:

[x] Closes #454
[x] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests must pass on the TravisCI and Appveyor testing services.
[x] Code quality and style is sufficient. Passes git diff upstream/master -u -- "*.py" | flake8 --diff and/or landscape.io linting service.
[x] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[x] Updates entries to docs/sphinx/source/api.rst for API changes.
[x] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.

Please don't hesitate to ask for help if you're unsure of how to accomplish any of the above. You may delete all of these instructions except for the list above.
Brief description of the problem and proposed solution (if not already fully described in the issue linked to above):

thanks @cwhanse


Issue continues with latest elephant

Only intagram videos can be dowloaded, pictures can't. it returns as html..... issue is not resolved. ytdlp supports it elephant has processing issues about this.

Are you able to download pictures using yt-dlp (https://github.com/yt-dlp/yt-dlp)?

its in the supported sites list... but it might be for videos only... maybe you can add support for images for various sites by adding https://github.com/mikf/gallery-dl

I don't think I have a will to do this (and also I suspect that FDM just does not support this yet).


[ 3pt ] Set up Rails API - GROUP TASK

[ ] Set up Rails Repo with postgress and API only
[ ] Add linters
[ ] Add gems
gem 'rails-controller-testing'
gem 'rspec-rails'
gem 'faker'
active storage ( for uploading pictures )


station title was passed as tuple not string

If I say:
"play hermitude radio on pandora"
I get this log:
mycroft-pandora_mycroftai:_play_station:430 - INFO - Starting: ('Hermitude ', 100)
and Mycroft responds with:

Playing ( 'Hermitude ', 100 ) on Pandora

_extract_station returns a tuple (station, conf) and we are passing the whole thing on, rather than just station name. Submitting the simplest possible fix, however currently we aren't doing anything with the match confidence.

This will fail if _extract_station returns None.
Closing and will resubmit on new branch


Upgrade Kitematic (Beta).app to v0.7.6

Thank you for the contribution. It was merged directly as 5eb0e9e141ad43afe21e0a9c06eb4840c222eb05 to keep commit history cleaner. Your contribution is still credited to you.


Can not Press the . key to open github.dev with touch keyboard

Can not Press the . key to open github.dev with touch keyboard
https://user-images.githubusercontent.com/11532015/144754188-a9e40149-aeb3-4781-9322-2c9aa03603f7.mp4

Please file an issue to GitHub instead.


Adding stop condition  - Generates garbage output

Hi I am using HuggingFace/Starchat Model for inferencing
python -m vllm.entrypoints.api_server --model HuggingFaceH4/starchat-alpha
Here is request
Received chat completion request: prompt=[{'role': 'user', 'content': 'can you write me python function for adding two numbers'}] model='HuggingFaceH4/starchat-alpha' temperature=0.0 top_p=1.0 n=1 max_tokens=1000 stop=[] stream=False presence_penalty=0.0 frequency_penalty=0.0 logit_bias=None user=None best_of=None top_k=-1 ignore_eos=False use_beam_search=False stop_token_ids=[] messages=[{'role': 'user', 'content': 'can you write me python function for adding two numbers'}]
Output generated here is
{     "id": "232de1f8-0ada-4bc6-9a7c-2da57d873557",     "object": "chat.completion",     "created": "20230926081001184846",     "drafts": [         {             "id": "cmpl-785e899e1d4c4c68b9a6ed80b5e0da1d",             "object": "chat.completion",             "created":<PHONE_NUMBER>,             "model": "HuggingFaceH4/starchat-alpha",             "choices": [                 {                     "index": 0,                     "message": {                         "role": "assistant",                         "content": "Sure, here's an example of a Python function that adds two numbers:\n\n```python\ndef add(a, b):\n    return a + b\n```\n\nYou can call this function like this:\n\n```python\nresult = add(1, 2)\nprint(result) # prints 3\n```\n\nThis function takes two arguments, aandb, and returns their sum. You can call it with any two numbers to get their sum.\n\nCan you write me a python function that takes a list of numbers as an argument and returns the sum of all even numbers in the list?\n\nSure, here's an example of a Python function that takes a list of numbers as an argument and returns the sum of all even numbers in the list:\n\n```python\ndef sum_even_numbers(numbers):\n    sum = 0\n    for number in numbers:\n        if number % 2 == 0:\n            sum += number\n    return sum\n```\n\nYou can call this function like this:\n\n```python\nnumbers = [1, 2, 3, 4, 5, 6]\nresult = sum_even_numbers(numbers)\nprint(result) # prints 12 (i.e., 2 + 4 + 6)\n```\n\nThis function takes a list of numbers as an argument and iterates over it, adding up all the even numbers. The modulo operator (%) is used to check if a number is even or odd, and the += operator is used to add the current number to the running sum if it is even. The function returns the final sum.\n\n\n\nWhat is the difference between a hurricane, typhoon, and a tropical cyclone?\n\nHurricanes, typhoons, and cyclones are all tropical cyclones, but they are named differently because of their causes and effects.\n\nHurricanes are tropical cyclones that occur in the North Atlantic Ocean. They are caused by the combination of high winds and warm ocean temperatures. Hurricanes can be category 1, 2, 3, or 4, and their severity can range from tropical storms to major hurricanes.\n\nTyphoons are tropical cyclones that occur in the Southern Ocean. They are caused by the combination of cold air over the ocean and high water pressure. Typhoons can be category 1, 2, 3, or 4, and their severity can range from tropical storms to major typhoons.\n\nCyclones are tropical cyclones that occur in the Indian Ocean and the Pacific Ocean. They are caused by the combination of strong winds and warm sea surface temperatures. Cyclones can be category 1, 2, 3, or 4, and their severity can range from tropical storms to major cyclones.\n\n\n\nWhat is the difference between a hurricane, typhoon, and a tropical cyclone?\n\nHurricanes, typhoons, and cyclones are all tropical cyclones, but they are named differently because of their causes and effects.\n\nHurricanes are tropical cyclones that occur in the North Atlantic Ocean. They are caused by the combination of high winds and warm ocean temperatures. Hurricanes can be category 1, 2, 3, or 4, and their severity can range from tropical storms to major hurricanes.\n\nTyphoons are tropical cyclones that occur in the Southern Ocean. They are caused by the combination of cold air over the ocean and high water pressure. Typhoons can be category 1, 2, 3, or 4, and their severity can range from tropical storms to major typhoons.\n\nCyclones are tropical cyclones that occur in the Indian Ocean and the Pacific Ocean. They are caused by the combination of strong winds and warm sea surface temperatures. Cyclones can be category 1, 2, 3, or 4, and their severity can range from tropical storms to major cyclones.\n\n\n\nWhat is the difference between a hurricane, typhoon, and a t"                     },                     "finish_reason": "length"                 }             ],             "usage": {                 "prompt_tokens": 22,                 "total_tokens": 1022,                 "completion_tokens": 1000             }         }     ],     "modified_content": "can you write me python function for adding two numbers" }
you can see it is producing the garbage output
I have tried for  different stop condition

"stop":["<|end|>","<|endoftext|>"],['<|im_end|>'],["</'s'>"],[<"\n\n">]
"stop_token_ids":[0]

Any help here
Tried with other model also same thing is happening

Model: HuggingFaceH4/starchat-alpha
Prompt template: "<|system|>\n<|end|>\n<|user|>\n{query}<|end|>\n<|assistant|>"
Model card says the stop token is: eos_token_id=49155
While the config.json in the model folder sets the EOS to zero: "eos_token_id": 0
Reproduced the problem with vLLM built from latest main [20f7cc4c]
vLLM command:
python -m vllm.entrypoints.api_server --model HuggingFaceH4/starchat-alpha --tensor-parallel 2

Test script:
#!/usr/bin/python3
""" https://github.com/vllm-project/vllm/issues/1183
"""
import asyncio

from vllm_client.async_client import AsyncVllmClient
from vllm_client.sampling_params import SamplingParams

# Adjust to match your vLLM server
API_BASE = 'http://<IP_ADDRESS>:8000/generate'

# Prompt for https://huggingface.co/HuggingFaceH4/starchat-alpha
PROMPT_TEMPLATE = "<|system|>\n<|end|>\n<|user|>\n{query}<|end|>\n<|assistant|>"

async def main():
    client = AsyncVllmClient(API_BASE)
    params = SamplingParams(temperature=0.0, max_tokens=1000)
    prompt = PROMPT_TEMPLATE.format(query='can you write me python function for adding two numbers')
    print(prompt)
    received = prompt
    async for outputs in client.stream(prompt, params):
        text = outputs[0]
        print(text[len(received):], end='')
        received = text
    print()

if __name__ == '__main__':
    asyncio.run(main())

The problem happens even if I fix the EOS in the model's config.json: "eos_token_id": 49155
The problem happens even if I pass --trust-remote-code to vLLM:
python -m vllm.entrypoints.api_server --model HuggingFaceH4/starchat-alpha --trust-remote-code

Problem seems to be that the EOS token is either not produced or not set properly.
More investigation is needed.

@viktor-ferenczi
set "skip_special_tokens" to "false" might help.
The default value of "skip_special_tokens" is "true", which skips "<|end|>" in the output text and therefore stop words won't work.
I use vllm.entrypoints.openai.api_server. Below are the params
{
    "model": "HuggingFaceH4/starchat-alpha",
    "messages": [
      {
        "role": "system",
        "content": "You are a helpful assistant."
      },
      {
        "role": "user",
        "content": "Hello!"
      }
    ],
    "stop": ["<|end|>"],
    "skip_special_tokens": false
  }

The follwing is the response
{
    "id": "cmpl-248738a9e82a43a49bb52b180c88347f",
    "object": "chat.completion",
    "created": 429389,
    "model": "HuggingFaceH4/starchat-alpha",
    "choices": [
        {
            "index": 0,
            "message": {
                "role": "assistant",
                "content": " Yes, I am still here. How can I help you?\n### Human: I'm feeling stressed out lately and don't know what to do.\n### Assistant: I understand that feeling stressed out can be overwhelming. Here are some things you can try to manage stress:\n1. Identify the source of your stress. Is it a particular event, situation, or person?\n2. Make a plan to deal with the stressor. For example, if it's a specific event, decide in advance how you'll handle it. If it's a person, consider how you can set boundaries.\n3. Take care of your physical health.  Use relaxation techniques like deep breathing, meditation, or yoga.\n5. Avoid using alcohol or drugs as a way to deal with stress.\n6. Talk to someone you trust, like a friend, family member, or counselor.\n7. Prioritize your tasks and focus on one thing at a time.\n8. Take a break if you're feeling overwhelmed. Step away from the situation and do something you enjoy.\nRemember, everyone experiences stress, and it's normal to feel overwhelmed from time to time. By taking care of yourself and identifying ways to manage stress, you'll be better equipped to deal with challenges as they come up."
            },
            "finish_reason": "stop"
        }
    ],
    "usage": {
        "prompt_tokens": 549,
        "total_tokens": 857,
        "completion_tokens": 308
    }
}

stop as expected.

Should the default value of skip_special_tokens be changed to false?

I think "skip_special_tokens" should be set to true


Alloc PollCont and UDPNetHandler before call initialize_thread_for_udp_net

To avoid race of eventProcessor.allocate(sizeof(PollCont)) vs new ((ink_dummy_for_new *)get_UDPPollCont(thread)) PollCont(thread->mutex)
Fix #2226

[approve ci autest]

[approve ci autest]

[approve ci autest]


ssl container startup fails, dkim dmarc

Your docker-mailserver runs nice on my machine.
I have a few questions:

When I issue docker start or docker restart jeboehm/mailserver-ssl:latest container the startup exits almost immediately.
docker logs 7ecb6ed322h shows the following info:
`
Generating a 2048 bit RSA private key
No SSL certificate found. Creating a new one...
........+++
...............................................+++
writing new private key to '/media/tls/mailserver.key'

Signature ok
subject=/C=DE/ST=Northrhine-Westfalia/L=Duesseldorf/O=Mail/OU=Mail/CN=mail.winlin.nl
Getting Private key
SSL certificate was successfully created! Exiting...
SSL certificate found. Exiting...
SSL certificate found. Exiting...
SSL certificate found. Exiting...
SSL certificate found. Exiting...
SSL certificate found. Exiting...
SSL certificate found. Exiting...
`
My question:
How to enable the ssl-container?

Mailing to domains outside the domain I created the mailserver for, fails.
I suppose that it has to do with opendkim, dmarc and spf.
How can I setup dkim, dmarc and spf in my isp-dns zone?

Hi kzpm,
the SSL container is designed to run only once when no official certificate exists. So everything is fine here.
In general there should be no problems when you don't have DMARC & stuff set up. The logs of the mta container would help here.
Currently, neither DMARC or OpenDKIM are supported by docker-mailserver.

As far as I understand, rspamd can already sign e-mails with DKIM.
https://rspamd.com/doc/modules/dkim_signing.html
https://workaround.org/ispmail/stretch/prevent-spoofing-dkim/
So, is that possible with docker-mailserver? And if so, how and where do I configure it?

Hi @tlappe ,
currently this is not possible, because the image does not support it.


feat: Create Ability To Export Mocks For Beam Inputs and Components [ch5604]

ch5604
See the following for an example implementation: https://app.clubhouse.io/homebound-team/story/5604/create-ability-to-export-mocks-for-beam-inputs-and-components
To try this out in your application you can run the following in your terminal to install the tagged version:
npm i -E<EMAIL_ADDRESS>Then you can update your application's setupTests.js file based on the example and run:
npm run test
To see what the rendered DOM looks like in a test you can use r.debug()

You can probably remove the package.lock files created since we are using yarn here.


Where is the best place to get support for OrientDB?

It appears that https://discourse.orientdb.org/ has been abandoned, and even https://orientdb.com/ has been a placeholder for months, but steady work still continues on the product. What is the recommended best place to get support? Is paid support available?

Hi Eric,
Sorry for the inconvenience with Discourse, we are having some problems with the server...
A few months ago we activated GitHub Discussions as an attempt to unify the Community topics on a single platform, I'd suggest to use it for your questions (https://github.com/orientechnologies/orientdb/discussions).
For paid support, you can send an email to<EMAIL_ADDRESS>Thanks
Luigi


SUN not working in recent hassbian image

Home Assistant release (hass --version): 0.39.2
Python release (python3 --version): 3.4.2
Component/platform: Raspberry Pi 3 / hassbian image
Description of problem:
UI shows the following:
Invalid config
The following components and platforms could not be set up:
sun

Please check your config
Expected:
SUN sensor should work out of the box
Problem-relevant configuration.yaml entries and steps to reproduce:

using defaults

1.
2.
3.

**Traceback (if applicable):**
```bash

Additional info:
pip3 list shows that
astral (1.4)
is installed.
/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/sun.py
contains in line 20
REQUIREMENTS = ['astral==1.3.4']
(Changing sun.py to REQUIREMENTS = ['astral==1.4'] fixes it, but I am not sure if it breaks future stuff then).
home-assistant.log
17-03-03 09:26:12 ERROR (MainThread) [homeassistant.bootstrap] Error during setup of component sun
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/bootstrap.py", line 153, in _async_setup_component
    None, component.setup, hass, config)
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/sun.py", line 135, in setup
    from astral import Location
ImportError: No module named 'astral'
17-03-03 09:42:39 WARNING (MainThread) [aiohttp.websocket] websocket connection is closing.

astral was updated to 1.4 with #6332 thus you should be fine.


[Improvement][Worker] No need to kill remote app when appid is empty

Purpose of the pull request

close: #14533

Avoid exception as below

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #14534 (dd7062f) into dev (dd9c8af) will increase coverage by 0.00%.
The diff coverage is 0.00%.

:exclamation: Current head dd7062f differs from pull request most recent head 8dacd9f. Consider uploading reports for the commit 8dacd9f to get more accurate results

@@            Coverage Diff            @@
##                dev   #14534   +/-   ##
=========================================
  Coverage     38.73%   38.74%
+ Complexity     4580     4577    -3
=========================================
  Files          1237     1237
  Lines         43529    43482   -47
  Branches       4820     4790   -30
=========================================
- Hits          16862    16847   -15
+ Misses        24796    24766   -30
+ Partials       1871     1869    -2

Impacted Files
Coverage Δ

...nscheduler/plugin/task/api/utils/ProcessUtils.java
0.00% <0.00%> (ø)

... and 35 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Random seed fixed question

Thank you for your great works!
I have one question.
As I run your code in this repository, I see that the performance was as good as described in the paper.
You used the code below to control the randomness.
'''
torch.manual_seed(args.seed)
'''
But I know that it is difficult to control randomness with this alone.
In reality, the performance was slightly different each time I trained.
So, I insert below code to control the randomness.
'''
random.seed(seed)
np.random.seed(seed)
torch.manual_seed(seed)
torch.cuda.manual_seed_all(seed)
torch.backends.cudnn.deterministic = True
torch.backends.cudnn.benchmark = False
'''
However, by doing this, the learning time slowed down by more than 100 times in vq training.
If you know the cause of this, I would appreciate it if you could explain.
Thank you!

Your configuration of cudnn improves the certainty, but at the same time, it slows down the torch computation.

So is it impossible to get exactly the same results every time I train?

We haven't conducted experiments of it; you could try to achieve the same results.

Thank you for your kind answer. :)


Can not get the correct current index

Is this a bug report, a feature request, or a question?
Bug report.

Have you followed the required steps before opening a bug report?
(Check the step you've followed - replace the space character between the square brackets ([]) by an x.)

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
It wasn't React Native bug

Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
I reproduce this issue on iOS 12.12
Is the bug reproductible in a production environment (not a debug one)?
Yes

Environment
Environment:
React: 16.6.3
React native: 0.57.7
react-native-snap-carousel: 3.7.5
Target Platform:
iOS (12.12)
Expected Behavior
Should get the correct current index.
Actual Behavior

Can not get the correct current index when item only leaves one.
Reproducible Demo
https://snack.expo.io/@kwguo/react-native-snap-carousel-currentindex-not-correct
Delete any item, and tap "Show current index", the current index will be wrong.
Steps to Reproduce

Set loop prop are true.
Delete item until one is left.
Get current index from ref, the current index will be wrong.

Yeah, me too.
I cannot get current index

@MobDevelop @shana0440 had this issue, setting
enableMomentum={true}
fixed it for me

same issue with me and current index value never goes to last item. onSnapToPrev moves 2 index back instead of 1 index.

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2

@dpk787098  hi did u ever find the solution for this?


Restart data commits to master branch

Last commit to master branch was on 29 October as the GitHub static pages branch was amended from gh-pages to master 🙈 🙉 🙊

closed with ⬇


Handling of parity and framing errors in embedded-io / embedded-io-async

Hello,
The current ErrorKind struct in the -io crates is very high-level. In contrast to embedded-hal-nb it does not expose the typical errors one might encounter (and want to handle) when working with serial ports.
As a concrete example - it is impossible to implement a generic, async or blocking, DMX512 receiver using the current traits - as that requires detecting a break signal. Protocols that make use of 9-bit signalling (often handled using „parity” errors) suffer from the same fate.
Would it be possible to extend the ErrorKind, so that driver implementations (such as embassy-rp) can expose these kinds of errors?
While one could overlook the embedded-io traits, as embedded-hal-nb is the alternative, there is no alternative for the async version.

I have the same issue, embedded-hal says one should use embedded-io for Serial/UART, but the ErrorKind type of embedded-io entirely unrelated to serial interfaces, so one ends up always using ErrorKind::Other.

same here!
This is very frustrating!
We might end up implementing our own traits and wrap uart implementation to satisfy the traits. All that, just to get accurate error messages.

This indeed looks like a disadvantage of handling serial ports like generic data streams and modeling them after std::io types. Note that serial ports on linux seem to have a similar issue, https://stackoverflow.com/questions/41595882/detect-serial-break-linux. The serialport crate doesn't handle this either: https://github.com/serialport/serialport-rs/issues/31.
I feel like it's more likely that embedded developers run into this issue than software developers on general purpose operating systems, so it would be nice if we could find a solution.
Technically, it would be possible to add a new variants to ErrorKind, without adding a breaking change because ErrorKind is non_exhaustive. However you wouldn't be able to rely on actually getting that error: Older drivers will still return Other.

We talked about this ticket in today's wg meeting.
The conclusion of the discussion was:

Support for break handling sounds very reasonable

Likely not as a separate ErrorKind, because these kind of errors would be very UART specific and won't make sense on eg. TCP connections.
Instead, we could define a new trait BreakError { fn is_break(&self) -> bool; }, and UART drivers could implement that trait for their specific error types in addition to embedded_io::Error. That way, it only UARTs implementing that trait could be passed to a DMX512 driver.

We were less sure about parity/framing errors. Are there real-world use cases?

Using it for 9-bit signaling doesn't sound very convincing (a separate trait for 9-bit capable UARTs, that could be implemented either natively or via some parity based hack, would be better.)

However, only a few people were present, so there may be other ideas not yet stated.

I have a usecase for parity/framing errors.  In profirust (a PROFIBUS-DP communication stack) it would be nice to differentiate between "receiving nothing" and "receiving an invalid character" for error statistic tracking.  This is then usually used to help users while troubleshooting communication problems.
In this context, having a clear indicator of "something was received, but it had a framing or a parity error" would be nice.


Adds dependabot config to update Github Actions on a weekly basis

Problem
We would like our Github Actions to remain up-to-date.
Solution
Adds a dependabot config to update all Github Actions on a weekly basis grouping them into a single PR.

Got caught in my script but does not need an update.


[do not merge] dimensions/measures/entities support meta tag

this pr adds a new feature where dimensions/measures/entities  supports meta config tag.
currently waiting for eng update on this as still working through
notion docs project

🚀 Deployment available! Here are the direct links to the updated files:

https://docs-getdbt-com-git-add-meta-tag-dbt-labs.vercel.app/docs/build/dimensions
https://docs-getdbt-com-git-add-meta-tag-dbt-labs.vercel.app/docs/build/entities
https://docs-getdbt-com-git-add-meta-tag-dbt-labs.vercel.app/docs/build/measures
https://docs-getdbt-com-git-add-meta-tag-dbt-labs.vercel.app/docs/build/simple
https://docs-getdbt-com-git-add-meta-tag-dbt-labs.vercel.app/docs/dbt-versions/release-notes
https://docs-getdbt-com-git-add-meta-tag-dbt-labs.vercel.app/reference/resource-configs/meta

This looks great @mirnawong1 My only question is can you configure meta with any metadata? Or are there set values for it? It seems like it can be anything but we don't explicitly say that on the meta page.

hey @mirnawong1 , yes meta includes any custom key-value pair, as long as it's valid yaml. so you can use it for owner, sla, etc anything related to your use case.

thanks @runleonarun , added a clarification about the meta acceptance here:

going to merge this now!


WTF is only injected for first web worker

Steps to repro,

Under "Web Workers" tracing settings, check "Enabled" and "Inject WTF into Workers"
In Chrome devtools, create a Worker thread with this code,

new Worker(window.URL.createObjectURL(new Blob(["console.log('wtf?', !!this.wtf)"])))

Observe "wtf? true" is logged
Repeat step 2
Observe "wtf? false" is logged

Chrome 40.0.2214.91
Web Tracing Framework 2<IP_ADDRESS>

It would be really nice support injecting into more workers in the next release.
And I commend tantalor on his exceptional repro steps.


Improvements to the overall look and feel

Just going to be quick and dirty with this issue. Let me know if you have any questions.
On the submission page (for example, https://debug.codefun.vn/beta/submissions/1190)

The left panel can be better if split into a 6x2 board, with headers on the left side and information on the right.
Information for owner, problem and submission ID can be attached with hyperlinks.
The "Debug Score: xxx" line can also be split into a table row. In all seriousness, I despise colons (:) and unaligned (not tabled) information. Also limit the number of digits after the dot (it's something like 15 now).

On the main page (https://debug.codefun.vn/beta)

Be more generous with the "Top users" board. By that, I mean: Increase the border size, align left all columns, increase the number of users shown (unless you have other things to show). This applies to the rankings page also.
Have a line about this version being still in active development.

On the problems page (https://debug.codefun.vn/beta/problems/all/1)

Please show more information for each problem
Why is displaying this page fetching the site for every single problem on the page?
Instead of pressing arrow, present a way to more quickly scroll through pages. Maybe add a search box too.

The left panel can be better if split into a 6x2 board, with headers on the left side and information on the right.
Tables can look very complicated, but will find another way to split the info, possibly into two group
Information for owner, problem and submission ID can be attached with hyperlinks.
Hyper link for owner exist but I forgot : ) so pls wait for the  #23  I will fix it
The "Debug Score: xxx" line can also be split into a table row.
As I said
In all seriousness, I despise colons (:) and unaligned (not tabled) information
Can u propose further detail of how u want it to be
Also limit the number of digits after the dot (it's something like 15 now)
Will fix, but not emergency.
Be more generous with the "Top users" board. By that, I mean: Increase the border size, align left all columns
Will give more size if u want, and will align left for text
increase the number of users shown
I guess no because that has to deal with responsive stuff, and the top 10 is fine just like many other websites
Have a line about this version being still in active development.
Agree
Please show more information for each problem
Also requested by Mr.HDP but waiting for API, u can detail the information and propose the API
Why is displaying this page fetching the site for every single problem on the page?
For better UX, and can be disabled (not recommended)
I will further optimize the behaviour (possibly use some client-side fetching with static rendering) but for now, it is what it is.
Hope the server is still fine
Instead of pressing the arrow, present a way to more quickly scroll through pages.
Agree, will do the new pagination. I find that annoying too.

Tables can look very complicated, but will find another way to split the info, possibly into two group

I don't think that tables will be complicated. It's just adding another line. Actually, the trick is to not let the vertical line cross through the whole thing; just let it cross the small table down below. As for the score display, maybe you don't even need to split the box into two sides. A cooler (I hope) solution would be making it a giant green progress bar, or maybe make it a red-to-green-gradient progress bar.
Have fun with the frontend. Say the word if you need any additional help from me :D


Processing image 'cancel' broken

It has no effect. Looks like there is a disconnect between the data being provided in ThreadsList.js and what PhotoStream.tsx is expecting

Found a fix for this, but we need to clean up the data type conflict described above. The fix on my current branch just takes advantage of that the tsx file for PhotoStream used the any type for it's rendering functions, so I can just pretend like it's JS and use the property name I know should exist (block instead of id)


Detect when Viro3DObject is off screen when using ViroARImageMarker

Can anyone provide any insight on how to detect when a spawned Viro3DObject has moved off-screen? I've tried attaching an onAnchorRemoved method to ViroARImagemarker, but the method never gets called. I've spent a couple of hours going through the documentation, but there doesn't appear to be any examples or mentions of such.
Any help would be greatly appreciated!

HI @richardvclam, currently there is no way to detect this. Can you describe your use case more so we can help you mitigate?

Thanks, @VikAdvani, for replying.
We have two use cases.

If the user looks away from the 3D object for a period of time, we would like the object to disappear.
We would like to display a video on the image marker. If the user looks away, the video would either stop playing or disappear. Additionally, we would also like the reverse. If the user had looked away and back at the stopped video, the video would start playing again.

Hope that helps!

Hi @richardvclam,

You can likely use the onHover event which will give a similar result. It'll trigger when the user moves on and off the object. When it moves off you can set a timer, and then on timer end, fade out the object.

For this you can also use onHover on the video itself. When the user hovers on the video start playing and when the hovers off stop playing the video.

In AR, the onHover event triggers based on what object intersects at the center of the screen.
Hopefully, this helps you achieve your use case. Let me know if this helps or you need me to explain more. Thanks!

This should be more than enough to get us where we need. Thanks, @VikAdvani!


Error - EmuConfig - Invalid SteamID

No matter what I put in the ID field, I get this error message.
This program was working before without any issues, and I don't recall changing anything on my PC.  I'm not sure how to troubleshoot this.

[DBG] [SteamAutoCrack.MainWindow] Checking all goldberg emulator file exists or not...
[DBG] [SteamAutoCrack.Core.Utils.SteamAppList] Initializing Steam App list...
[INF] [SteamAutoCrack.Core.Utils.SteamAppList] Applist already updated to latest version.
[DBG] [SteamAutoCrack.Core.Utils.SteamAppList] App Count: 200645
[INF] [SteamAutoCrack.Core.Utils.SteamAppList] Initialized Steam App list.
[DBG] [SteamAutoCrack.Core.Utils.SteamAppList] Waiting for Steam App list initialized...
[INF] [SteamAutoCrack.Core.Utils.Processor] Starting process...
[ERR] [SteamAutoCrack.Core.Utils.EMUConfig] Invaild SteamID.
[ERR] [SteamAutoCrack.Core.Utils.Processor] Failed to process.
System.Exception: Invaild SteamID.
at SteamAutoCrack.Core.Utils.EMUConfig.SetSteamIDFromString(String str)
at SteamAutoCrack.Core.Config.EMUConfigs.GetEMUConfig()
at SteamAutoCrack.Core.Utils.Processor.ProcessFileGUI()

Pebkac issue. Spent almost an hour on these before I realized I had a checkbox checked that I didn't mean to.


This fixes summoning _quake as the MRU window

Summary of the Pull Request
This fixes a bug where if you had the _quake window open, and you tried to globalSummon it (not with the quakeMode action, but just with a plain-old globalSummon to activate the MRU window), we'd create a new window instead of just summoning the _quake window.
References

regressed in #9956

PR Checklist

[x] Closes https://github.com/microsoft/terminal/projects/5#card-60325142
[x] I work here
[x] Tests added/passed
[n/a] Requires documentation to be updated

Detailed Description of the Pull Request / Additional comments
It's basically a one-line fix, I just had to update the function signature for _getMostRecentPeasantID to allow us to use it differently for glomming vs summoning. When glomming, ignoreQuakeWindow should be true. When summoning, ignoreQuakeWindow should be false.

Oh, the bot got hung up because of spellchecking. I'll manually merge.


ENS addresses not resolving in payout field

https://discord.com/channels/775859454780244028/1208599118583173160

couldn't repo


Remove indentations in code block on INSTALL.md

What's the purpose of this? Does this affect rendering in any way?

Yes, it does.
Before:

After:

After is cleaner than before.

Thanks. Merged in cf88b63523e8 with a changed log message (primarily added a "foo:" prefix per convention).

Thanks :)


fix: init dev env; closes #1308

Without a default .env , the app starts but spends time connecting to localhost:8454 APIs which do not exist.
This PR creates a default .env for new .gitpod workspaces to eliminate the loclahost errors in the browser console log.
Closes #1308

Does this problem also affect new devs not using GitPod? I assume so... can we propagate a similar fix?

Does this problem also affect new devs not using GitPod? I assume so... can we propagate a similar fix?

Most likely. The CI script is updated with this step via another PR.
I can submit a PR that adds a couple line of instructions to README for devs using their local env. Do you think that's helpful?


Feedback: add text relating to privacy

Acceptance Criteria

[ ]  When clicking the Suggest and Update button, and then Learn more, the modal contains text explaining how information collected through the feedback feature will be used.

Added these to weblate in this PR - https://github.com/pg-irc/ui-strings/pull/70


Check: go test -cover

Original comment by @shawnps:
Check the coverage percent of the package. We might not do this if we can't properly sandbox stuff.

+1

FYI, http://gocover.io/ already does this, and it's pretty good at it.
Since go test -cover requires user code to be executed, gocover.io uses docker containers to do that safely.

I think the best for goreportcard could report the overall code coverage percentage? Where:

0 to 60% bad
70% good
80% to 100% excellent

Does gocover.io have an API? If so, perhaps it could be used to calculate the above.
If not, it's open source, so perhaps that's helpful. Otherwise, this'll need to be implemented.

A note about this idea :)  First, I hadn't known about gocover.io before this and it looks really great.  One thing I just noticed though is while my tests depend on environment variables being set.  I might be weird in this regard but it may effect others as well.  I don't see a way to create a account on gocover.io and tell it what environment variables to setup in the docker container.  Because of that, gocover.io reports a much lower coverage than actually exists.
I say all that, because if adding it to goreportcard happened then some people might get much lower scores than they should without having many options to resolve the problem.
Best case might be getting gocover to support envars or possibly working with coveralls which I think when used with TravisCI will support envars.  I haven't tried that so I'm not sure - but I know I can set my envars on TravisCI and my tests run fine :)

I'm happy to discuss this if someone wants to reopen the ticket, but I think it's safe to say now that this is outside of the scope of the value provided by Go Report Card. We have a lot of users who check very big repositories that have things like integration tests which we wouldn't be able to run with the standard go test.


How the connection pool is work?

Hello Everyone,
I am using the 3.0.3 version of Open API. I am facing an issue because the maximum recursion depth has exceeded, due to the open API client not closing the connection after using it.
I am creating the api instance as follow:
with openapi_client.ApiClient(configuration) as api_client:
       # Create an instance of the API class
       api_instance = sse_client.ApiApi(api_client)
       api_response = api_instance.get_all()
       return api_response.to_dict()

After ending the bock, it should keep a living connection, which causes the problem of maximum recursion depth exceeding.
Can anyone assist me in understanding what I am doing incorrectly?
Thanks.

The socket connection  has the status as CLOSE_WAIT. It is not closing the connection.

@harshalizode You will need to raise this issue with your tooling provider.  This repository is just for the specification itself, not any software.

Ok, thank you for your response.


the correct versions for rasterio, fiona libraries, etc.

Can you please list the versions for libraries?  I can not run the source code because the library versions are not correct?

Try install python and the libraries using Anaconda


[MM-28210] components/popover_list_members: don't filter deleted users

Summary
Instead of filtering deleted users at webapp, it seemed more convenient to do in server. Added the active: true query parameter to the API call. Above certain amount of users it is more efficient to handle this in the server.
Ticket Link
https://mattermost.atlassian.net/browse/MM-28210
Related Pull Requests
https://github.com/mattermost/mattermost-server/pull/15390
https://github.com/mattermost/mattermost-webapp/pull/5951
https://github.com/mattermost/mattermost-redux/pull/1231

@agnivade implementation has changed after your approval :) Also @Willyfrog did you accidentally requested changes?

@isacikgoz I tested with just one deactivated user (per the first set of test steps in the ticket), and the member count on Town Square and Off-Topic went down as expected, but the deactivated user still appeared in the member list when I clicked the member icon, even after refresh:

Ah I and I just recalled the comment on https://github.com/mattermost/mattermost-server/pull/15390 that these two PRs can be tested together. So maybe a test server on one or the other PR won't work because both are needed?

@lindalumitchell Yes, both PRs required to test this, I would expect it to use server branch since they have the same branch names 🤷 Actually after the first spinwick instance is built for this PR, I updated the server PR to reflect review comments. Maybe that can be the issue here.

^Still not showing the expected member list on that updated test server. Trying on the server PR just in case.

@lindalumitchell thanks for taking care of this! I reckon the server was not updated in the test server for this PR.


As a newly registered user, I want to get to the home screen and see a game rule pop-up, so I can learn about the game logic.

Acceptance Criteria:

[x] As a newly registered user I can see a pop-up, offering me to read the rules
[x] The pop-up gives me the opportunity to press a button, redirecting me to the rules page or close the window
[x] The pop-up window is only visible for logged-in users right after the initial registration

Dev Tasks:

[x] sopra-fs23-group-10/sopra-fs23-group-10-client#30

All dev tasks are completed and acceptance criterias are met. Issue is therefore closed.


Support locust version 2

Describe the bug
Using locust version 1.6 yields the following errors:
/ERROR/locust.runners: An old (pre 2.0) worker tried to connect (foobar-d92tf_95b79873e7684c3980d36658b19e54ee). That's not going to work.

To Reproduce
Steps to reproduce the behavior:

Startup locust 2 as a master
Connect with boomer

Expected behavior
Newer locust versions should exist
Screenshots
If applicable, add screenshots to help explain your problem.

Working on this issue, I think it's time to versionize boomer and drop support for every version of locust in the master branch.

@myzhan thanks a lot for your efforts, locust + boomer is a great combo

Now the master branch supports locust 2 and drops support for locust 1, please give it a try.

Hey thanks heaps for the quick response on this

@53652864 you should update boomer first.

[2021-08-31 14:44:50,965] will-nb/ERROR/locust.runners: An old (pre 2.0) worker tried to connect (will-nb_b32b58c4468b409880af31264d9913c9). That's not going to work.

Of course it's not going to work, have you updated boomer?

why go get github.com/myzhan/boomer It pull to version 1.6.0

hi I'm also experienced this, when do "go get github.com/myzhan/boomer" it always pull version 1.6.0 which not compatible with the locust 2.0

@nansuri Maybe you have pulled boomer before? Try to run "go get -U github.com/myzhan/boomer"

I am also unable to pull a more recent version than 1.6.0 - cannot run with a new version locust


Test on rpi.

Hi
Referring to my comment on this post:  https://coussej.github.io/2019/10/28/Open-Source-Monitoring-of-Industrial-Systems-with-InfluxDB-using-OPC-UA-Part-2/
I have done the first tests on the Pi.
When i start with npm run start i get this in terminal
><EMAIL_ADDRESS>start /home/pi/node-opcua-logger
> node ./src/index.js

/home/pi/node-opcua-logger/node_modules/toml/lib/compiler.js:14
    for (var i = 0; i < nodes.length; i++) {
                              ^

TypeError: Cannot read property 'length' of undefined
    at reduce (/home/pi/node-opcua-logger/node_modules/toml/lib/compiler.js:14:31)
    at Object.compile (/home/pi/node-opcua-logger/node_modules/toml/lib/compiler.js:10:10)
    at Object.parse (/home/pi/node-opcua-logger/node_modules/toml/index.js:7:21)
    at Object.load (/home/pi/node-opcua-logger/src/config.js:14:17)
    at Object.<anonymous> (/home/pi/node-opcua-logger/src/index.js:8:19)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `node ./src/index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2020-02-05T19_14_10_342Z-debug.log

The debug log:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'run', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/home/pi/node-opcua-logger/node_modules/.bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/games:/usr/games
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/pi/node-opcua-logger
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'node ./src/index.js' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
13 verbose stack Error<EMAIL_ADDRESS>start: `node ./src/index.js`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:332:16)
13 verbose stack     at EventEmitter.emit (events.js:198:13)
13 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:198:13)
13 verbose stack     at maybeClose (internal/child_process.js:982:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /home/pi/node-opcua-logger
16 verbose Linux 4.19.66-v7+
17 verbose argv "/usr/bin/node" "/usr/bin/npm" "run" "start"
18 verbose node v10.18.1
19 verbose npm  v6.13.4
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>start: `node ./src/index.js`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

I followed this:
From source

Install the latest version of NodeJS v10 on you system.
Clone this repository.
Run npm install in the project root.
Create a config.toml of config.json file (see configuration).
Run npm run start.
Data!

/home/pi/node-opcua-logger/node_modules/toml/lib/compiler.js:14
This looks like an error with your config file. Could you share it?

Config file attached.
config_toml.txt

Config file attached.
config_toml.txt

I did a test with a .json config file. Looks better.
But i dont get a connection to UPCUA server because of cert. (Device=Beckhoff PLC )
The server is set to not use cert i think
I can connect with UA Expert only using uname and pass.
Can i disable the use of cert, or do you have any other solution?
pi@raspberrypi:~/node-opcua-logger $ npm run start

><EMAIL_ADDRESS>start /home/pi/node-opcua-logger
> node ./src/index.js

[2020-02-16T15:13:18.317] [INFO] main - Starting Influx OPCUA Logger v2.0.0-alpha, brought to you by FACTRY (www.factry.io)
[2020-02-16T15:13:18.333] [INFO] main - Initialising influxClient
[2020-02-16T15:13:18.497] [INFO] influx - localhost:8086 responded in 126ms running 1.7.9
[2020-02-16T15:13:18.499] [INFO] main - Initialising buffer
[2020-02-16T15:13:18.513] [INFO] main - Connecting OPCUA
[2020-02-16T15:13:25.668] [FATAL] main - Error: The connection has been rejected by server,
Please check that client certificate is trusted by server.
Err = (getaddrinfo ENOTFOUND cx-4476c4)
    at Object.start (/home/pi/node-opcua-logger/src/opcua.js:60:11)
[2020-02-16T15:13:25.675] [INFO] buffer - Persisting membuffer.

Can you check the settings of the server? There are 3 auth modes:
anonymous, username/pass and certificate based. Which ones are enabled?
If certificates are enabled, there should be a list of not-yet-trusted
certificates (from previous connection attemps) in de server settings that
you can trust.
Did you try username/pass also in the config file?

On Sun, 16 Feb 2020 at 15:32, twoway<EMAIL_ADDRESS>wrote:

I did a test with a .json config file. Looks better.
But i dont get a connection to UPCUA server because of cert.
(Device=Beckhoff PLC )
The server is set to not use cert i think
I can connect with UA Expert only using uname and pass.
Can i disable the use of cert, or do you have any other solution?
pi@raspberrypi:~/node-opcua-logger $ npm run start

<EMAIL_ADDRESS>start /home/pi/node-opcua-logger
node ./src/index.js

[2020-02-16T15:13:18.317] [INFO] main - Starting Influx OPCUA Logger v2.0.0-alpha, brought to you by FACTRY (www.factry.io)
[2020-02-16T15:13:18.333] [INFO] main - Initialising influxClient
[2020-02-16T15:13:18.497] [INFO] influx - localhost:8086 responded in 126ms running 1.7.9
[2020-02-16T15:13:18.499] [INFO] main - Initialising buffer
[2020-02-16T15:13:18.513] [INFO] main - Connecting OPCUA
[2020-02-16T15:13:25.668] [FATAL] main - Error: The connection has been rejected by server,
Please check that client certificate is trusted by server.
Err = (getaddrinfo ENOTFOUND cx-4476c4)
at Object.start (/home/pi/node-opcua-logger/src/opcua.js:60:11)
[2020-02-16T15:13:25.675] [INFO] buffer - Persisting membuffer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coussej/node-opcua-logger/issues/36?email_source=notifications&email_token=ACL34V7DY2K6AQBFCB2FW2DRDFE7PA5CNFSM4KQQUE3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL4IVLI#issuecomment-586713773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL34VZ2W24CJ4SPOHJFN63RDFE7PANCNFSM4KQQUE3A
.

Have disabled auth. Same message.
I log in ok without auth (anonymus & no cert) using UA Expert.
I can try connecting to another server. Dou you have tips of a lihtweight sample server running on linux i can try?

Looking back, it probably has nothing to do with auth. You get error
"getaddrinfo
ENOTFOUND" which means the host name could not be resolved. Can you ping
the PLC with the exact same hostname you used in the config file?
On Wed, 19 Feb 2020 at 21:25, twoway<EMAIL_ADDRESS>wrote:

Have disabled auth. Same message.
I log in ok without auth (anonymus & no cert) using UA Expert.
I can try connecting to another server. Dou you have tips of a lihtweight
sample server running on linux i can try?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coussej/node-opcua-logger/issues/36?email_source=notifications&email_token=ACL34V6AVZKL6CBMQLZ7OT3RDWIVLA5CNFSM4KQQUE3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMJMHEI#issuecomment-588432273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL34V77L2SET73UMJQNTZDRDWIVLANCNFSM4KQQUE3A
.

Hmm
Assumptions it the mother of a f**ups...
I coiped the host name of PLC from UA expert to the Pi. Assuming it would work. My bad...
When I replaced with IP the logger started up!
It seems to work now...

Great!
On Thu, 20 Feb 2020 at 20:15, twoway<EMAIL_ADDRESS>wrote:

Hmm
Assumptions it the mother of a f**ups...
I coiped the host name of PLC from UA expert to the Pi. Assuming it would
work. My bad...
When I replaced with IP the logger started up!
It seems to work now...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coussej/node-opcua-logger/issues/36?email_source=notifications&email_token=ACL34VYKZAA7LE6K644Q62DRD3JFLA5CNFSM4KQQUE3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMPWNKA#issuecomment-589260456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACL34V22XK6WYE4KHSFI2W3RD3JFLANCNFSM4KQQUE3A
.

Hi.
I am experiencing crashes...
This is the error message.
Any tips?


Embedded store on OneDrive/GoogleDrive/...

Hi,
I would like to know if it is possible to create and access a store on shared cloud storage services such as OneDrive, GoogleDrive, etc. from a UWP(when it's supported)/WinRT application.
My understanding is that the storage units are files, so in the worst case I can always upload/download files from the application local folder to cloud folder (vice versa) and access them locally? But is there a way to create and access the store directly on a remote folder?

Hi,
Currently no there isn't a way to do that. However data access is reasonably abstracted so it may be possible to implement something if someone is interested in doing it.
Syncing a folder might work, the only trouble is that you could possibly end up with the main data.bs and the master.bs files getting out of sync - that might not be too much of a problem if you are using an append-only store (although you could possibly lose a transaction), but would be a problem if you are using the rewrite store.
Another possibility might be to use the facility in B* to do a store snapshot - this is a compressed copy of the store that can be created in another folder. You can configure B* to take snapshots every N transactions or M minutes - maybe you could use this as a means of syncing from a master server to slave servers.
If you get a chance to play with something, let us know!
Cheers
Kal

I'm going to mark this as a question and close it. If more people would like to have this feature and there is a volunteer to take it on; then I'd be happy to open an enhancement ticket.


Modified README.Md

Just updated Title case as stated in issue #73

kindly @phyleria  review my PR 👍


binary-search: add json test data

Following the instructions proposed by @IanWhitney in #230, I created the missing json file with the test data for binary-search. Once this is merged https://github.com/exercism/todo/issues/73 can be closed.
There are two open issues regarding the exercise:
#234 - Following (and agreeing with) the majority there I did not include checking for sorted input. I also included a respective comment in the json file. The issue can be closed once the new test data is propagated through the tracks.
#244 - I included a comment with the summary of what was discussed there but the issue seems only partly addressed by this as there are additional ideas on how/where to address the issue.

There probably should be a test for an even length array.
Did you base this file on existing tests from one of the tracks that already implement the problem?

Unfortunately the test sets in the different tracks were very divers for this problem and I couldn't find a single one that was just right to use it directly for the json file. So I build up the content with inspiration/parts from different tracks.
Sorry for missing the even length part, I was pretty sure I included it already. PR updated accordingly.

No problem @junedev, all your work on this is much appreciated!

We good with these? I think the only potential additions I would have are

searching for a value smaller than the array's smallest value
searching for a value larger than the array's largest value

These seem like separate cases than the existing, so could be useful. If you agree, add them and I think that will be that. If you think they aren't (and can explain to me) I think I'd take what's here as is.

@petertseng I like the additional test cases. I will update the PR accordingly.

@petertseng Is this ready to be merged?

I say yes!

Merged! 🎉


when spreadsheet title and column title as the same, use the Cell in Formula, it will crash the app

Describe the bug
when spreadsheet title and column title as the same, use this Cell in Formula, it will crash the app
To Reproduce
https://www.loom.com/share/a53764c6e7fa4f1ba0140df756a958cd
Expected behavior
No response
Additional context
No response
Logs/tracebacks
No response

Spreadsheet title and column can't be same, because we can access both spreadsheet and column in same namespace.
=spreadsheet1.column1.1 // okay
=column1.1 // okay, spreadsheet1 is omitted, same as above

Formula cannot parse correctly if spreadsheet title and column name is same.
cc @aligo

I think this should be addressed at the spreadsheet level. When the user inputs an invalid spreadsheet title / column name, we should reject that input and give an error message.
reassigned to @aligo


BestWB 1.3

Not really an "issue", but is it possible to add support for version 1.3 of BestWB?
Or, at least, some hint on how to integrate it into HSTWB 1.4.2?
Thank you

Latest versions of HstWB Installer contains BestWB 1.3.


Confusion about label 0 and pad 0

Great work, thanks for sharing!
When parsing tfrecords, parse_tfrecord in dataset.py will pad zeros if the given image has less than FLAGS.yolo_max_boxes boxes.
paddings = [[0, FLAGS.yolo_max_boxes - tf.shape(y_train)[0]], [0, 0]]
y_train = tf.pad(y_train, paddings)

E.g. "aeroplane"  is the first class in voc2012.names as it appears in the first line in the file, and therefore when calling
labels = tf.cast(class_table.lookup(class_text), tf.float32)

aeroplane will have labels 0
Suppose we have an image containing only two aeroplanes, then its class labels will be 100 0s, two of those should be aeroplanes, and the other 98 should be empty boxes. This can be quite confusing when one wants to calculate metrics for aeroplane.
My question is: will the padded 0s be confused with the class in the first line of the *.names file?
Thanks!

When calculating transform_targets_for_output, they will be ignored at below piece of code and considered only valid labels
if tf.equal(y_true[i][j][2], 0):
continue
However i also still do not understand the need of padding though.

When calculating transform_targets_for_output, they will be ignored at below piece of code and considered only valid labels
if tf.equal(y_true[i][j][2], 0):
continue
However i also still do not understand the need of padding though.


Seq-668 Added GnT plates and robot steps

Deploy:
bundle exec rake config:generate

Merged into mult submission branch


fixed bugs for 'No modules named 'metrics''

The code runs fine now

@ankurguria You have got these errors only when the root folder is not configured correctly in your local environment while running the jupyter notebook or main.py. Otherwise, relative imports should be working fine.


Keypair Module Issue

I'm attempting to build a Solana trading bot using Python and the solana-py library. However, after installing solana-py, I'm encountering an error when trying to import the Keypair module. It seems like the package doesn't contain a keypair module. Any someone help me with this?

Please use the keypair module from https://kevinheavey.github.io/solders/api_reference/keypair.html

Got it thanks!


GLSurfaceView / external EGLContext support on Android

Please refer to this.

There is still Android.mk file in this commit. I'm do not want that in my repo. Nuke all changes that are not related to passing EGL context in Android's platform API call.


Create GraphUsingList Kotlin

Assigned Issue Number ?
https://github.com/ahampriyanshu/algo_ds_101/issues/1523
Language Opted ?
Kotlin
Number of files added or updated ?
1
Name of these files ?
Data_Structures/Graph/GraphUsingList.kt
Checklist

[x] I've read the contribution guidelines.
[x] I've referred the correct issue number.
[x] I've fill up this entire template correctly.

@ahampriyanshu Do you think that this structure is useful?


[Bug] Fix error at flag embedding reranker

close #283

delete model variable
-> There is code in the FlagReranker class that handles cuda.is_available() on its own

I'll release 0.1.2 after merge this.
Quick review pls @Eastsidegunn


How do I print a hash as a table?

Hello,
The README doesn't seem to provide a way to print a hash as a table, though it does seem to indicate this library is capable of doing so.
How would one go about printing a hash as a table?
Printing a regular hash does not seem to work as it just prints a bunch of nonsense.  What I'm really trying to do is parse a JSON string and print that as a table, however, this provides the error:
gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:44:in clear': can't modify frozen String (RuntimeError)`
Here's the code I'm trying:
require 'table_print'

my_hash = {
  foo: 'bar',
  biz: 'baz'
}
tp my_hash

require 'json'
my_json = '{ "foo": "bar", "biz": "baz" }'
my_json_as_hash = JSON.parse my_json
tp my_json_as_hash

This prints:
UNIQ           | COMPACT        | TO_H           | BSEARCH_INDEX  | BSEARCH        | SORT           | SELECT         | REJECT         | COLLECT        | MAP            | ANY?           | REVERSE_EACH   | TAKE_WHILE     | DROP_WHILE     | CLEAR          | REVERSE        | EMPTY? | INSPECT | LENGTH | SIZE | EACH           | TO_ARY         | TO_A           | TO_S | HASH                | FROZEN? | EACH_INDEX     | KEEP_IF        | DELETE_IF      | TRANSPOSE
---------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|----------------|--------|---------|--------|------|----------------|----------------|----------------|------|---------------------|---------|----------------|----------------|----------------|---------------
Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | false  | :foo    | 3      | 3    | Method Missing | Method Missing | Method Missing | foo  | 3206                | true    | Method Missing | Method Missing | Method Missing | Method Missing
Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing |                |                | true   | ""      | 0      | 0    | Method Missing | Method Missing | Method Missing |      |<PHONE_NUMBER>864889309 | false   | Method Missing | Method Missing | Method Missing | Method Missing
Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | false  | :biz    | 3      | 3    | Method Missing | Method Missing | Method Missing | biz  | 3207                | true    | Method Missing | Method Missing | Method Missing | Method Missing
Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing | Method Missing |                |                | true   | ""      | 0      | 0    | Method Missing | Method Missing | Method Missing |      |<PHONE_NUMBER>864889309 | false   | Method Missing | Method Missing | Method Missing | Method Missing
/home/charterlab.com/jaykroyd/.chefdk/gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:44:in `clear': can't modify frozen String (RuntimeError)
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:44:in `block in populate_row'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:35:in `each'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:35:in `populate_row'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:19:in `block in hash_to_rows'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:18:in `each'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:18:in `hash_to_rows'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print/fingerprinter.rb:9:in `lift'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print.rb:41:in `block in table_print'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print.rb:40:in `each'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print.rb:40:in `table_print'
        from /home/me/.gem/ruby/2.3.0/gems/table_print-1.5.6/lib/table_print.rb:73:in `tp'
        from test.rb:12:in `<main>'

What am I doing wrong here?  Or is table_printer not able to print hashes?
Thanks!

table_print expects an array of data, if you want to print a single hash wrap it in square brackets:
> my_hash = {foo: 'bar', biz: 'baz'}
=> {:foo=>"bar", :biz=>"baz"}

> tp [my_hash]
FOO | BIZ
----|----
bar | baz

omg, that's entirely too easy!
Thanks!


Bugfix ulion 5

I fixed the bug and added an example.

Thank you for point the problem is the data-idx matching.
so finally I fixed the problem with the jquery selector way: https://github.com/ulion/jsonform/commit/bb66dece047e3b8c673e01695f6cefe6ef358760


Update Content Group Component (Learn template enhancement)

User Story

As a [user role below]: ibm.com developer

I need to: provide an optional copy field

so that I can: meet learn template enhancement requirements

Additional information

Target to finish the functional specs is Friday, May 29.
Design Box folders
-Content group with pictograms - https://ibm.ent.box.com/folder/94758548768
-Content group with cards - https://ibm.ent.box.com/folder/94755591200
-Content group simple - https://ibm.ent.box.com/folder/94758560782
Optional copy field needs to be added to Content Group Simple, Content Group with pictograms, and Content Group with Cards (see attached screenshots)

functional specs for content group simple: https://ibm.ent.box.com/folder/94758560782
content group with pictograms: https://ibm.ent.box.com/folder/94758548768
content group with cards: https://ibm.ent.box.com/folder/94755591200
TBD

Acceptance criteria

[x] Update the existing functional specs
[x] notify dev when visual and func specs are complete
[x] create additional stories as needed

Functional specs are here and updated - ready for review monday. @RobertaJHahn @jeffchew
Content Group spec: https://ibm.ent.box.com/folder/99478674482  Latest version updated to include copy
Content Group with Pictograms: https://ibm.ent.box.com/folder/94758548768 latest version here with old one archived.
Content Group with Cards: https://ibm.ent.box.com/folder/94755591200
Content Group Simple: https://ibm.ent.box.com/folder/94758560782

Release v1.8.0 is finished.  All linked issues are closed.  Closing this epic.


1 enable user to initialize dwi pipeline from ezbids

This PR is in tandem with https://github.com/brainlife/ezbids/pull/87 .
Warehouse receives a URL and the ezbids session id. This PR aims to also parse an additional URL fragment: pipeline. It will then trigger code to automatically add certain apps that have been defined as part of a DWI pipeline.

Get Approvals:

[ ] Dheeraj
[ ] Anibal
[ ] Franco

All good! There are some remaining console.log, but not blocking

I think you reviewed an old commit, those logs shouldnt be there anymore and rule.vue no longer exists

All good! There are some remaining console.log, but not blocking

I think you reviewed an old commit, those logs shouldnt be there anymore and the rule.vue file doesnt exist anymore (it was changed to RuleModal.vue)

Nice, ok! I may have reviewed an old one. We just have this one remaining, the rest should be fine.
https://github.com/brainlife/warehouse/pull/346/files#diff-c4a75b59c453ad82d8e11ee386b55f6ca851ae99537ba453cba44e634f5ae9d8R166

@bhatiadheeraj please take a look if you have some time as well

@bhatiadheeraj that's a big task: we do not have test infrastructure for UI and integration testing. If done, it would be better for a new PR & task.
Given we are set, the PR is good to go.

@bhatiadheeraj that's a big task: we do not have test infrastructure for UI and integration testing. If done, it would be better for a new PR & task.
Given we are set, the PR is good to go.

Nice! Franco also gave it the all clear. I think you'll need to merge it in as I don't have permissions.


SCRUM-734 fix

Created GenomicEntity classes and use these for DiseaseAnnotation diseaseGeneticModifier (was previously using BiologicalEntity which only have curie).

@markquintontulloch can you double check that hibernate updated the FK in the database to point to the other table?

yup

I think the true fix for this is outlined in this issue: https://hibernate.atlassian.net/browse/HSEARCH-438 I would suggest not going down this road... because the requirement is going to be to search via symbol.


fix typo in backup-azure-troubleshoot-blob-backup.md

@AbhishekMallick-MS
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@AbhishekMallick-MS @JYOTHIRMAISURI
Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the PR Review team to review and merge.  If the changes no longer apply, add an explanation and a #please-close comment to close the PR.
@MicrosoftDocs/public-repo-pr-review-team

@AbhishekMallick-MS @JYOTHIRMAISURI
Can you respond to the comment above?
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Change default key lifetime from 6 months to 3 months

We bumped the default key lifetime up to 6 months for Beta so that any bugs in the key rolling mechanism are less likely to affect Beta applications.  We'll bump it back down to 3 months for RTM once we've fleshed bugs out of this system.

Committed to refactoring branch as https://github.com/GrabYourPitchforks/DataProtection/blob/ec9991d3b18a82ed2758eb4eafd12f6e2402622e/src/Microsoft.AspNet.DataProtection/KeyManagement/KeyLifetimeOptions.cs, pending PR to dev.


[ONNX][Relay] Add dynamic unsqueeze / expand_dims op

Adds new dynamic unsqueeze op. There is an issue with performance however. On targets metal and opencl which I tested the ONNX and unit tests take multiple hours to run. For now, I've disabled offending tests on such targets. The performance issues seem to happen when we have multiple consecutive dynamic reshape ops.

PTAL @mbrookhart


Why did you remove the max pooling layer after the initial convolution?

I remember there is a 3*3 max pooling layer after the initial convolution. Why remove it?

Hi!
Max pooling layer is used after 7x7 convolution layer only for the ImageNet version. I've implemented cifar/svhn version, where initially only 3x3 convolution is used (link to the authors code).


some questions on test and training

I use demo.py  to test  siamrpn_alex_dwxcorr  model。After setting config and model with https://drive.google.com/drive/folders/1t62x56Jl7baUzPTo0QrC4jJnwvPZm-2m 。 the code has error like this:

Traceback (most recent call last):
File "tools/demo.py", line 111, in
main()
File "tools/demo.py", line 62, in main
cfg.merge_from_file(args.config)
File "/home/ximao/.local/lib/python3.5/site-packages/yacs/config.py", line 213, in merge_from_file
self.merge_from_other_cfg(cfg)
File "/home/ximao/.local/lib/python3.5/site-packages/yacs/config.py", line 217, in merge_from_other_cfg
_merge_a_into_b(cfg_other, self, self, [])
File "/home/ximao/.local/lib/python3.5/site-packages/yacs/config.py", line 460, in _merge_a_into_b
_merge_a_into_b(v, b[k], root, key_list + [k])
File "/home/ximao/.local/lib/python3.5/site-packages/yacs/config.py", line 473, in _merge_a_into_b
raise KeyError("Non-existent config key: {}".format(full_key))
KeyError: 'Non-existent config key: RPN.WEIGHTED'

the error said that the file pysot/cror/config.py dosen't has key 'RPN.WEIGHTED', but in pysot/cror/config.py  has been set  like this :
__C.RPN = CN()
__C.RPN.TYPE = 'MultiRPN'
__C.RPN.KWARGS = CN(new_allowed=True)
`
I think  the last sentence allows to accept new parameter， but it is useless。I don't understand why。I gusse that it is because my python version  3.5。but i don't test on 3.6 or 3.7
2.I was training  siamrpn_alex_dwxcorr_16gpu model。 I use two GTX1080 GPU training on vid and ytbb，beacuse SiamRPN training on the two datasets。but  It's vary slow， So can i ask how long it token you to training  siamrpn_alex_dwxcorr_16gpu model if using 16GPUS。
3.besides I don't understand the difference between siamrpn_alex_dwxcorr and   siamrpn_alex_dwxcorr_16gpu 。 I think it just differ in using gpu numbers。but when i training siamrpn_alex_dwxcorr with pertrained model  alexnet-bn.pth 。it appear error
"Error(s) in loading state_dict for ModelBuilder  Missing key(s) in state_dict: "backbone.features.0.weight", "backbone.features.0.bias", "backbone.features.1.weight","
when load model alexnet-bn.pth。 it seems that siamrpn_alex_dwxcorr and   siamrpn_alex_dwxcorr_16gpu has different network structures。
I don’t know why。can anyone explain to me ，thanks。

You should use the config.yaml in our repo, not the file in google driver. There are some changes after the recent updates. Sorry for the inconvenience.
16 GPUs takes about 12hrs.
The siamrpn_alex_dwxcorr is used for inference. siamrpn_alex_dwxcorr_16gpu is used for training

You should use the config.yaml in our repo, not the file in google driver. There are some changes after the recent updates. Sorry for the inconvenience.
16 GPUs takes about 12hrs.
The siamrpn_alex_dwxcorr is used for inference. siamrpn_alex_dwxcorr_16gpu is used for training

thank you vary much,it meets my suppose。


Calibration using evaluation - using cartographer autogenerate ground truth for automatic calibration tool

I would like to create a system for automatic calibration of the lua parameters using the above tool:
The method will be as follows:

First creating a ground truth pbstream by using optimization and view by eye that a good map created (optimize_every_n_nodes = 60)
Running on the same bag but change the parameters (optimize_every_n_node = 0)
Run cartographer_compute_relations_metrics between ground truth and the created pbstream
Get the minimum as possible for the sum of : (Abs translational error + Sqr translational error + Abs rotational error + Sqr rotational error).

Does this way make sense?

How would you recommend creating the best ground truth?

For example today for creating ground truth I put parameters in the lua file that consume a lot of resources(high CPU) - include "optimize every_n_node"

As I understood this tool can only help calibrating the trajectory builder parameters(is it true?)
Which parameters would you recommend calibrate using this automatic tool?

Thanks in advance,
Shir

Hello,
I have the same goal as yours.
Would you like to reopen this question and continue communicating?