Fixing Compilation for Build Server

Trying to keep up with development I tested the build on TeamCity and I found that things did not quite build correctly in that environment. The project files for Lucene.net.Tests.Queries and Lucene.net.Tests.Classification needed a slight tweak to make msbuild look in the right place all the time.
Once we get these added I can turn on the build / tests for those libraries on the CI server.

Thanks - merged now
For future PRs, please try to work on a branch, and also squash committs to fewer when posssible...


Upgrade to Selenium 2.49.0

Selenium 2.49.1 is now available - http://docs.seleniumhq.org/download/

Now 2.50.1 is available.

Now 2.52.0 is available


Better error handling to add context

When an issue arises in a stream, it would be quite useful to catch it and add context information (stream, repo, ...).
This would make it a lot easier to replicate errors locally and fix them.
cc @laurentS

The situation is much better now with the recent updates of the SDK


go fmt on pkg

If we want to follow standard format for golang (go fmt ...), we have two places, where indenting needs minor tuning.

Thanks :) - Please rebase and i merge this one.

Formatting got fixed on more recent PR merges.


Unable to specify AAD login under Account tab with AAD-Universal with MFA Support Authentication type

Azure Data Studio Version:
I am running Azure Data Studio 1.4.5.
I am trying to connect to an Azure SQL database from on-premise tools server (window 2016).
It works with SQL authentication but not able to connect via "Azure Active Directory - Universal with MFA support".

Steps to Reproduce:

Open the Azure Data Studio (no difference whether I kick it off with "run as admin" or not)
New connection->connection Details
Unable to enter the AAD user login at all under the Account field of ADS.

Connection Type -> Microsoft SQL Server
Server -> azeBLAHBLAH.database.windows.net
AuthenticationType -> Azure Active Directory - Universal with MFA support
Account -> Add an account ...
^^^^ this drop down menu does not allow me to enter the account!!!
Version 1.4.5
Commit 887f4e8985023602e599cf86bdb44df48bb28569
Date 2019-02-12T06:20:08.633Z
VS Code 1.26.1
Shell 2.0.9
Renderer 61.0.3163.100
Node 8.9.3
V8: 6.1.534.41
Architecture x64
Please help!
Irene

me too version 1.5.2 (window does not change after selecting Add an account...)

same

this is fixed in master for me.  I downloaded the "Insider build" from the repo readme link.

Can you please send me link to the “Insider Build”?
From: Ed Sweeney<EMAIL_ADDRESS>Sent: Saturday, 13 April 2019 8:38 AM
To: Microsoft/azuredatastudio<EMAIL_ADDRESS>Cc: Irene Lam<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/azuredatastudio] Unable to specify AAD login under Account tab with AAD-Universal with MFA Support Authentication type (#4666)
this is fixed in master for me. I downloaded the "Insider build" from the repo readme link.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/azuredatastudio/issues/4666#issuecomment-482743909, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVM_y-Uh75TfYQztqwJEsmaO__adg5bIks5vgQrTgaJpZM4cCzhN.

This email, including all attachments, is confidential and for the sole use of
the intended recipient(s). If you are not the intended recipient, you are
prohibited from disclosing, copying, distributing, or in any other way using it.
If you have received this email in error, please notify me by return email,
or contact the AEMO Information and Support Hub on 1300 236 600, and then delete this email from your system.

Can you please send me link to the “Insider Build”? From: Ed Sweeney<EMAIL_ADDRESS>The links to each insider build by platform are in the README here:
https://github.com/Microsoft/azuredatastudio

Thanks.
From: Ed Sweeney<EMAIL_ADDRESS>Sent: Monday, 15 April 2019 11:55 AM
To: Microsoft/azuredatastudio<EMAIL_ADDRESS>Cc: Irene Lam<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/azuredatastudio] Unable to specify AAD login under Account tab with AAD-Universal with MFA Support Authentication type (#4666)
Can you please send me link to the “Insider Build”? From: Ed Sweeney notifications<EMAIL_ADDRESS>The links to each insider build by platform are in the README here:
https://github.com/Microsoft/azuredatastudio
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/azuredatastudio/issues/4666#issuecomment-483081883, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVM_yy4VrjIb_JliVJa4z0i6rJU1xKb9ks5vg9vegaJpZM4cCzhN.

This email, including all attachments, is confidential and for the sole use of
the intended recipient(s). If you are not the intended recipient, you are
prohibited from disclosing, copying, distributing, or in any other way using it.
If you have received this email in error, please notify me by return email,
or contact the AEMO Information and Support Hub on 1300 236 600, and then delete this email from your system.

This appears to be fixed in 1.6.0 April release.  Please let me know if the issue persists.

Thanks Karl this works for me now :-)


An elegant String solution is needed/需要一个更优雅的String实现

Currently, we are using NacosString as the alias of std::string in this project, but it's problematic.
We expect a string solution that can:

notate null string (equivalent to String s = null in java)
support encoding option
the null status can be assigned from one string to another
compare between null string and a normal string can be performed and will get false

This solution doesn't necessarily need to be a string implementation, a solution to replace the existing NacosString schema is welcome
当前系统中，NacosString其实只是std::string的一个别名，但是这会在后续开发中引发问题。
我们需要一个string解决方案，能够：

表示null字符串（相当于java中的String s = null）
支持不同的编码
将null对象从一个string传递到另外一个
null字符串和非null字符串可以进行比较而且会返回false

这并不一定是要一个实现，也可以是一个解决方案替换掉现在的NacosString方案

For the second item,i think,a string util is good enough.

"std::string*"fits other descriptions

@yzz-ihep Actually that parameter doesn't work
That is the feature needs to be added

当前没有更好的处理办法，暂且使用系统的string
As we don't have a better solution for string, we use std::string


refactor: make link between setup-env and install-env clear

One thing could make the whole flow you worked on more transparent: create an explicit link (dependency) between the setup-env and install-env targets.
When I look at the task pipeline, I can see clearly, e2e target will trigger setup-env:

But what will setup-env trigger? Now I know (since I navigated through the codebase) that it will call the install-env executor declared by the Nx plugin createNodes API.
This is all very implicit...
As soon as you search for install-e2e, TADA, things become clearer because targets have self-describing names:

Could we imagine a refactoring of the runSetupEnvironmentExecutor that depends on install-env running first?
Do you think this flow should remain opaque?

I really like the feedback!! I was already tempted to refactor it as I see it as a big downside that is is all implicit.
I did not go on with the refactor as we had no tests so far.
Now that at least unit tests are in we can more confidently change the codebase.


Customizer Not Working

I've tried several times to use the customizer and keep receiving the same error when compiling.
Ruh roh! Problem parsing or compiling Less files.
Error: error evaluating function red: a.rgb is undefined
// Customize the :focus state to imitate native WebKit styles.
.form-control-focus();
Thanks in advance for your help!

The customizer can't work if you give it invalid input :smile:
Most probably one of the variable values you set is invalid. Unfortunately, we don't have the manpower to personally help you track down which one it is. You can figure it out yourself by either looking at what variables are referenced (possibly indirectly) in the source code shown in the error message, or by only setting one more custom variable value at a time, retrying the download each time until it fails.
Or you can try asking community members on StackOverflow, IRC, or Slack: https://github.com/twbs/bootstrap/blob/master/README.md#community
Good luck!


Query return java.lang.NoSuchMethodException

Dear,
I am using Sugar 1.5, however every time I run a query, "Listall", "Find" or "first" for example, the following error appears below, I already tried other versions of suck and the error remained, I already did a clean of the project and a rebuild

12-26 11:55:36.442 2974-2974/ D/SQL Log: SQLiteQuery: SELECT * FROM SERVICE
12-26 11:55:36.443 2974-2974/ W/System.err: java.lang.NoSuchMethodException:  []
12-26 11:55:36.444 2974-2974/ W/System.err:     at java.lang.Class.getConstructor(Class.java:531)
12-26 11:55:36.444 2974-2974/ W/System.err:     at java.lang.Class.getDeclaredConstructor(Class.java:510)
12-26 11:55:36.444 2974-2974/ W/System.err:     at com.orm.SugarRecord.getEntitiesFromCursor(SugarRecord.java:212)
12-26 11:55:36.444 2974-2974/ W/System.err:     at com.orm.SugarRecord.find(SugarRecord.java:204)
12-26 11:55:36.444 2974-2974/ W/System.err:     at com.orm.SugarRecord.listAll(SugarRecord.java:127)

I just forgot to create the default sugar builder, I did not realize it was necessary to create it
// Default constructor is necessary for SugarRecord
public Service() {
}

can i see you're code?

can i see you're code?
because i have same problem with you


Use concurrent batch pipeline for ~30x speed up

Add the PVNet concurrent datapipe to the concurrent batch creation and backtest scripts. The new data is already in ocf_datapipes and is about 30x faster than the old datapipe for creating concurrent batches

Looks good!

Might be worth getting @AUdaltsova and @Sukh-P to review also as they use datapipes quite a bit

Great work adding this in and getting the speedup! Just checking has this been reran and tested locally?

Great work adding this in and getting the speedup! Just checking has this been reran and tested locally?

Yeh, it has been tested locally on a small set of examples. I'm going to leave it open for now since I am making concurrent batches to train on and Zak is doing a backtest. Just in case more bugs come out of the woodwork during those


Support for "const enum/enum"

Hi,
first of all thanks for this project. I could not find any information regarding the support of const enum. Can you elaborate if it is?

Thanks!

swc currently does not support const enums, just like babel.
It's not supported because swc works file by file.
I'm working on it, but I guess it would take some time.

@kdy1 thanks for the insanely fast answer!
I suppose this also means normal enums, e.g.
enum Foo {
  Hello = 'hello'
}

are also currently not supported?

Normal enums are supported, and I tried to make swc follow tsc's output as possible.

@kdy1 Ok I guess I am missing something then. This is my config
{
  "jsc": {
    "target": "es2019",
    "parser": {
      "syntax": "typescript",
      "tsx": true,
      "decorators": false,
      "dynamicImport": true
    }
  },
  "module": {
    "type": "commonjs",
    "strict": true,
    "strictMode": true,
    "lazy": false,
    "noInterop": false
  }
}

I am using the swc-loader, yet I get the following error:

It seems like a bug of typescript pass.
I'll fix it tomorrow (it's over 12pm in my country)

I am trying to replace the normal compiler of Webpack with SWC, but the "const enum"s are not replaced.
any configuration change I am missing which needs to be done for the TypeScript build?

I'm using the deno bundle command from Deno 1.17.1, which treats enum and const enum the same. So what the current status of this issue is?

It seems that swc supports const enum/enum, but it doesn't support 'inline' feature till now as issue#3648 mentioned.


fix test $program path expansion

Test execution command string substitution for $program should reference absolute path (or relative to calling cwd) based on python docs-
https://docs.python.org/2/library/subprocess.html#popen-constructor

If cwd is not None, the child’s current directory will be changed to cwd before it is executed. Note that this directory is not considered when searching the executable, so you can’t specify the program’s path relative to cwd.
Needed for it to work on Windows.

Thanks – could you explain what doesn't work with the current way of doing things (cwd= directory of test program, and program=name in that directory)?

No problem- there seems to be a difference in how Python executes Popen in Windows (I'm assuming the test executions work on other OS's here :-)), where it strictly follows the rule mentioned in the cwd arg in the python docs, and the $program executable is not found by the os despite the working directory being the same as the directory $program inhabits... really weird.

Another option would be to use shell=True (see example below), but I think there's other implications to that.
import subprocess, os

if not os.path.isdir('tpy'):
    os.mkdir('tpy')
with open('tpy/file.txt', 'w') as f:
    f.write('yuppers')

cmd = 'copy file.txt file2.txt'
try:
    # doesn't work on windows
    subprocess.Popen(cmd, cwd = 'tpy').wait()
except OSError as e:
    print 'what is going on here!'
    print e

# this works....
subprocess.Popen(cmd, cwd = 'tpy', shell=True).wait()

OK got it, thanks. I'll check that this doesn't have any unintended consequences and then merge.
Please could you click through the mbed contributor agreement here: https://developer.mbed.org/contributor_agreement/ and let me know which mbed account you use to do that, so we're able to merge!

Great, thank you- I think with this change native Windows support should be
there. I published a mingw target http://yotta.mbed.com/#/target/mingw/0.0.1
as a first pass.
My mbed developer account name is "Noah".
On Oct 24, 2015 7:46 AM, "James Crosby"<EMAIL_ADDRESS>wrote:

OK got it, thanks. I'll check that this doesn't have any unintended
consequences and then merge.
Please could you click through the mbed contributor agreement here:
https://developer.mbed.org/contributor_agreement/ and let me know which
mbed account you use to do that, so we're able to merge!
—
Reply to this email directly or view it on GitHub
https://github.com/ARMmbed/yotta/pull/523#issuecomment-150799287.

(sorry for the delay, I've verified that this shouldn't break any existing uses: we're in the process of changing the CLA that we use, so I can't merge this just yet)

Accepting under mbed 2.0 CLA click-through. :white_check_mark:


[BUG] Failed to Read attribute with src/app/ReadClient.cpp:343: CHIP Error 0x00000077: Missing secure session

Reproduction steps
I have used python controller to done a stress test on my RPI sample dut (all-clusters-app) on Read-Write operation for 100000 iteration. The iteration 12580 is passed without any error but the 12581 is Failed to read the node-label attribute with the below traceback error and the next iteration 12582 is also passed without any issues.

Traceback (most recent call last):
File "/Users/chaitanyajandhyala/code/python/matter-qa/src/matter_qa/scripts/reliability_scripts/TC_Read_Write.py", line 72, in verify_read_write
nodelabel_read_value = await self.read_single_attribute_check_success(cluster=self.clusters,
File "/Users/chaitanyajandhyala/code/python/matter-qa/src/matter_qa/library/helper_libs/matter_testing_support.py", line 876, in read_single_attribute_check_success
result = await dev_ctrl.ReadAttribute(node_id, [(endpoint, attribute)], fabricFiltered=fabric_filtered)
File "/Users/chaitanyajandhyala/code/python/matter-qa/.venv/lib/python3.9/site-packages/chip/ChipDeviceCtrl.py", line 1421, in ReadAttribute
res = await self.Read(nodeid,
File "/Users/chaitanyajandhyala/code/python/matter-qa/.venv/lib/python3.9/site-packages/chip/ChipDeviceCtrl.py", line 1352, in Read
ClusterAttribute.Read(future=future, eventLoop=eventLoop, device=device.deviceProxy, devCtrl=self,
File "/Users/chaitanyajandhyala/code/python/matter-qa/.venv/lib/python3.9/site-packages/chip/native/init.py", line 87, in raise_on_error
raise self.to_exception()
chip.exceptions.ChipStackError: src/app/ReadClient.cpp:343: CHIP Error 0x00000077: Missing secure session

Bug prevalence
Whenever i do this
GitHub hash of the SDK that was being used
3718e996b56f36a553c1b4571f77728396590776
Platform
python, raspi
Platform Version(s)
RPI-4, 8GB RAM
Anything else?
Example of Failed iteartion details:
The iteartion number 12580 is passed below file for the same
TH log: log_2024-05-28_22-24-05.log
The iteration number 12581 is failed with this error below the logfile
TH log : log_2024-05-28_22-24-31.log
The iteartion number 12582 is passed below file for the same
TH log: log_2024-05-28_22-24-52.log
For the detailed summary please reference this json: summary.json
Script reference: https://github.com/CHIP-Specifications/matter-qa/blob/main/src/matter_qa/scripts/reliability_scripts/TC_Read_Write.py

Please re-execute and validate whether this always happens at 12580.

Where are the DUT logs for these runs? We need the DUT logs to diagnose.

@tcarmelveilleux Currently we didn't have the dut log for this error will try to fetch the dut log from the next run

closing this issue for now and following it up in a different issue.


Add ROX_SEARCH_PAGE_UI feature flag

Description
Follow instructions in https://github.com/stackrox/stackrox/tree/master/ui/apps/platform#add-a-feature-flag-to-frontend-code
Prerequisite to hackathon project.
Intentionally not add at the end of the list to avoid potential merge conflicts.
Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

/test style-checks


Fixed and estimated column widths

It should be possible to set fixed/custom column widths.
There should be an option to auto set widths based on cell values.
Unfortunately imagettfbbox() is rather sloooooow, so thats not an option (for now), so it'll probably be "estimated" widths.
todos:

base multipliers per font, size  (+ bold & italic)
multiplier groups for ~100 ASCII characters

Initial tests with Calibri, Arial, Times are looking ok. Estimations are off by about 2-5% per 50 characters.

... so libre office wants/needs height's as well ... bummer.

ok sweet - using predefined (fixed) character-widths seems to be quite reliable. No need to take the font size into account =)


Cuckaroo29 doesn't seem to work with a RTX 2070 gpu

Here is what we get in the logs:
Dec 28 21:41:00.008 DEBG Mining: Plugin 0 - Device 1 (GeForce RTX 2070) at Cuck(at)oo29 - Status: OK : Last Graph time: 0.000124314s; Graphs per second: 8044.146 - Total Attempts: 1935

Looks like our RTX 2070 card might be faulty.
It would be great if anyone could confirm that cuckaroo29 works fine on their 2070 gpu.

I personally can't confirm @mably but looking at the gpu mining stats page it appears it has been working for others in the past at least.

Yep it has even been working on our own RTX 2070 mid-december.
We just want to be sure it's not a recently introduced regression but just our card that has been damaged.

confirmed faulty card, I believe?


Question on the outputs (yelp)

Thanks for your interesting work! May I ask whether there are some explainations for the output format? There are five outputs for each sentence and I'm wondering which models correspond to those outputs : )

The target sentiment of the five outputs are 0.1, 0.3, 0.5, 0.7, 0.9

The target sentiment of the five outputs are 0.1, 0.3, 0.5, 0.7, 0.9

Thanks a lot for your explaination! : )


added changes to use AndroidX libraries and runtime update 7

added gradle.properties
added changes to use AndroidX libraries and runtime update 7

@gunt0001 can you please review these changes. The changes are for the app to use runtime Update 7 and Androidx libraries.

@puneet-pdx looks good to me

@eri9600 can you review?

@esreli do you wanna look at the new release I added to RELEASE.md file before I go ahead and merge this ?

@puneet-pdx sure. LGTM.


Add environment variable for disabling ALPN verification

AWS ELBs proxying gRPC traffic do so in a Layer 4 TCP mode. If they are configured to terminate SSL, then they do not advertise any ALPN protocols. This causes clients dependent on the C core to fail to connect if going directly through the proxy. To work around this, one can use Envoy as it is less strict around the ALPN settings.
This change adds an env var that will globally disable ALPN verification in the SSL security connector for the duration of that process. This allows any language to by-pass the protocol verification.
I'm interested in if this is something that would be considered at all, and I believe it would be very valuable for us and possibly others. If so, I'd love feedback on how I can add tests for this or improve any aspect. It seems like there aren't any tests around the security connector right now and I'm not sure if this init pattern is quite right.
I prototyped a different approach that added a flag on channel credential construction, but that seemed very heavyweight for what I was looking for.
Here's some related issues:
https://github.com/grpc/grpc/issues/18549
https://github.com/grpc/grpc/issues/9991
The ifndef additions will go away once https://github.com/grpc/grpc/pull/18563 is merged.

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @f3nry isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

@thelinuxfoundation CLA is signed!


discussion: a development tool for app developer

To install/run/debug apps on YodaOS, it requires to use different commands deeply at user-land, just like:
$ adb push ./myapp/ /opt/apps/
$ adb shell
> /etc/init.d/vui-daemon restart

This seems inconvenient to developers especially apps, so we now have the following 2 options:

a command line that wraps the adb and other commands, to provide a convenient tool.
the VSCode plugin for App developers.

@Rokid/yodaos any ideas?

shell is a faster option.
we should also provide a way to hit the NLP to developer's app, there is an option here:

developer register their app in Rokid develop website
mock an NLP and replace the appId with developer's appId
send NLP through dbus or other IPC channel to VUI
VUI dispatch the NLP to develop's app

shell is a faster option.

Hmm, yea, but the plugin would be complete better, anyway I'm +1 on plugin firstly.

we should also provide a way to hit the NLP to developer's app, there is an option here:

These functions looks like sticky with Rokid's NLP cloud, I think @xqbumu's https://github.com/Rokid/vscode-plugin-rokidos would be an alternative for the above points?

Discussed with @legendecas, we actually cannot skip the command line tool to create the plugin, thus let's do the CLI firstly.

I proposed the below features which should be shipped:

install via npmjs, apt-get and brew.
commands

resource management

connect the device via usb/tcp.
boot a new image on connected device(s).
install the app on connected device(s).
install the shared packages on connected device(s).
uninstall the app from connected device(s).
manage the runtime services on connected device(s).

testing

run app tests.
run nlp tests.

debugging

start debugger proxy via https://github.com/jerryscript-project/jerryscript-debugger-ts.
start gdb-server which handles the given process.

Possible command line interface:
yoda-application-manager

[ ] open-url [options] url
[ ] force-stop [options] package
[ ] set-debug-app [options] package
[ ] clear-debug-app [options] package

yoda-package-manager

[ ] list packages: list installed packages
[ ] path package: get the package install path
[ ] install [options] path: install a package from given path
[ ] uninstall [options] package: uninstall a package
[ ] clear package: clear package data

Possible command line interface:

@legendecas could we have a discoverable protocol for the above set just like what we have done on mock script?

Updates, we have yoda-platform-tools.


Should we export all abstract types?

          If people are going to build there own modules that we would want to export all thee abstract types? (this is really for a new issue)

Originally posted by @seabbs in https://github.com/CDCgov/Rt-without-renewal/pull/83#discussion_r1505807188

Answer is yes!


The tag isn't updated in the opened file when viewing properties for a resource

OS: Win10
Build Version: 20220515.1
Static Web App Extension Build Version: 20220513.11
Regression: Not a Regression
Repro Steps:

Make sure the resource groups by Resource Type.
Right click a Static Web App -> Edit Tags... -> Edit the tag in the opened file as below -> Save and update.

Refresh the resource group -> The tag is updated in the opened file.
Right click a Static Web App -> View Properties.
Check whether the tag is updated in the opened file.

Expect:
The tag is updated in the opened file.

Actual:
The tag isn't updated in the opened file.

More Info:
Restart VS Code can fix this issue.

Fiex by #461


{Freeway-Bug}[Adaptive Cards- Home]: Talkback does not announce the role and position of the tabs present under home screen.

Target Platforms
Android
SDK Version
2<IP_ADDRESS>
Application Name
AdaptiveCards
Problem Description
Test Environment:
OS: Android 13
Device: S21+ 5G
Screen Reader: Talkback
AdaptiveCard Android App version: 2<IP_ADDRESS>
Repro Steps:

Turn on Talkback.
Open Adaptive Cards Visualizer Application.
Home Page will opened.
Now Navigate to "import, visual, json, conf." tabs using swipe gesture and observe the issue.

Actual Result:
Talkback does not announce the role and position of the tabs present under home screen, Talkback announce like " Import selected, Visual double tap to activate".
Expected Result:
Talkback should announce the role and position of the tabs present under home screen, Talkback should announce like " Import tab selected, Visual tab, 2 of 4 double tap to activate".
User Experience:
Screen reader users will be impacted as they will be unable to understand whether the controls are interactive or not and if they are interactive then whether they are buttons or links or tabs.
WCAG Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value
"Have feedback to share on Bugs ? Please help fill Trusted Tester Bug Feedback (office.com) “
Screenshots
https://user-images.githubusercontent.com/89897257/228434792-7dda0871-0ec6-4f6e-8a0b-322a37d41d79.mp4
Card JSON
NA

Sample Code Language
No response
Sample Code
No response

we don't fix a11y issues on sample app since we don't ship it to the public

As per the above comment closing this issue.


Remove RRE References in Documentation

There are two occurences of port 7979:
katas/000_setting_up_chorus.md
158:Bring up http://localhost:7979 and you will see a relatively unexciting empty dashboard.  Don't worry, in our first kata, we'll do a relevancy test and fill this dashboard in.

README.md
53:> open http://localhost:4000 http://localhost:8983 http://localhost:9000 http://localhost:3000 http://localhost:7979

Which refer to RRE, which was removed.

Would you like to submit a PR for this?   Thank you fro bringing it to our attention!


Set up the development environment

As a DevOps
I need Set up the development environment
So that [benefit]
Details and Assumptions
* [document what you know]

Acceptance Criteria
gherkin
Given [some context]
When [certain action is taken]
Then [the outcome of action is observed]

OK


Langchain agent with tools generates "StructuredTool is not JSON serializable"

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
The code is picked up from LangChain documentations
https://python.langchain.com/v0.2/docs/how_to/tools_chain/
from langchain import hub
from langchain.agents import AgentExecutor, create_tool_calling_agent
from langchain_core.tools import tool

model = # A mistral model

prompt = ChatPromptTemplate.from_messages(
    [
        ("system", "You are a helpful assistant"),
        ("placeholder", "{chat_history}"),
        ("human", "{input}"),
        ("placeholder", "{agent_scratchpad}"),
    ]
)

@tool
def multiply(first_int: int, second_int: int) -> int:
    """Multiply two integers together."""
    return first_int * second_int

@tool
def add(first_int: int, second_int: int) -> int:
    "Add two integers."
    return first_int + second_int

@tool
def exponentiate(base: int, exponent: int) -> int:
    "Exponentiate the base to the exponent power."
    return base**exponent

tools = [multiply, add, exponentiate]

# Construct the tool calling agent
agent = create_tool_calling_agent(model,tools, prompt)

# Create an agent executor by passing in the agent and tools
agent_executor = AgentExecutor(agent=agent, tools=tools, verbose=True)

result = agent_executor.invoke(
    {
        "input": "Take 3 to the fifth power and multiply that by the sum of twelve and three, then square the whole result"
    }
)

Error Message and Stack Trace (if applicable)
TypeError: Object of type StructuredTool is not JSON serializable
Description
I am trying to run the sample code in https://python.langchain.com/v0.2/docs/how_to/tools_chain/ to call an agent equipped with tools. I see two problems:

If I run the code as it is, it generates the error that "Object of type StructuredTool is not JSON serializable".
If I create the agent with empty tools list (i.e., tools=[]) it generates the response. However, it is not supposed to be the right way of creating agents, as far as I understand. Besides the answer with mistral7b model is very inaccurate. Even in the example provided in the link above, the answer seems to be different and wrong when checking the langSmith run.

System Info
langchain-core==0.1.52
langchain==0.1.16

The problem seems to be resolved if we use dumps function in LangChain:
from langchain.load.dump import dumps

tools = [multiply, add, exponentiate]
tools_ = [dumps(multiply), dumps(add), dumps(exponentiate)]

agent = create_tool_calling_agent(model,tools_, prompt)
agent_executor = AgentExecutor(agent=agent, tools=tools, verbose=True)

hm im not able to recreate. could you post the full stack trace and share how you're instantiating the model? could you also try bumping to langchain and langchain-core >=0.2?


Network adaptor not found: B2D 18.09 , Vmware W. Pro 15

Docker Machine Version:  0.14.0, build 89b8332
Boot2Docker: v18.09.0
VMWare: Workstation Pro 15
vmrun version 1.17.0 build-10737736
Steps to Reproduce:
docker-machine -D create --driver=vmwareworkstation vmware

Issue:
Timeout waiting for IP address
(vmware) DBG | MAC address in VMX: 00:0c:29:53:00:37
(vmware) DBG | Not there yet 58/60, error: IP not found for MAC 00:0c:29:53:00:37 in DHCP leases
(vmware) DBG | MAC address in VMX: 00:0c:29:53:00:37
(vmware) DBG | Not there yet 59/60, error: IP not found for MAC 00:0c:29:53:00:37 in DHCP leases

Root Cause:
NIC is missing:

Notes
Editing the generated VMX file and setting
ethernet0.virtualDev = "e1000"

Allows the machine to start and detect the network card.

Seems related to https://github.com/boot2docker/boot2docker/issues/1345
I can see the same problem with docker-machine vmwarefusion driver.

I can confirm this.
Broke a leg trying to figure out why docker-machine stopped working.
Booted boot2docker.iso in a newly created vm in vmware workstation pro 15, and it worked fine.
VMware itself defaults the nic for Linux-Based VMs in VMware is always to e1000.
Should we update docker-machine-vmwareworkstation to e1000?

@tekook Can you help me compile your fork? I can't work with the go environment. and I've been trying for 2 days....  my email<EMAIL_ADDRESS>thanks

@tekook Can you help me compile your fork? I can't work with the go environment. and I've been trying for 2 days.... my email<EMAIL_ADDRESS>thanks

Compile the images, but did not seem to help.
Currently trying the new beta of boot2docker, see: https://github.com/boot2docker/boot2docker/issues/1340#issuecomment-444273243

Ill update this to e1000, probably a change in updates VMWares


Change name to solarposition.rise_set_transit_spa

pvlib python pull request guidelines
Thank you for your contribution to pvlib python! You may delete all of these instructions except for the list below.
You may submit a pull request with your code at any stage of completion.
The following items must be addressed before the code can be merged. Please don't hesitate to ask for help if you're unsure of how to accomplish any of the items below:

[x] Closes issue #316
[x] I am familiar with the contributing guidelines.
[x] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
[ ] Updates entries to docs/sphinx/source/api.rst for API changes.
[ ] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
[ ] Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
[ ] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[ ] Pull request is nearly complete and ready for detailed review.

Brief description of the problem and proposed solution (if not already fully described in the issue linked to above):
Change to function name to conform with rise_set_transit_ephem and rise_set_transit_analytical. Edits to docstrings to clarify expected timezone and location.  Changed local variable time to times to avoid possible conflict with time module.

rise_set_transit_spa to parallel rise_set_transit_ephem, or sunrise_sunset_transit_spa to parallel sunrise_sunset_transit_geometric?

Or sun_rise_set_transit_spa? I don't have a preference, but we should make them all consistent before the next release. I think it only require updating the whatsnew and api files for the new functions.

We currently have rise_set_transit_ephem and 'sunrise_sunset_transit_geometric`.  I prefer the 2nd for clarity but it's a pretty long name.

Right, but you're changing get_sun_rise_set_transit to rise_set_transit_spa, dropping both get_ (which is good) and sun_ (which we might consider retaining).
I'd like to see this PR:

deprecate the existing function name.
change one or both of the recently added _ephem and _geometric functions to be consistent with the new name. no deprecation necessary because added them since the last release.

So by the end of this PR we should have one of the following sets of functions:
option A:

sunrise_sunset_transit_geometric
sunrise_sunset_transit_ephem
sunrise_sunset_transit_spa

option B:

sun_rise_set_transit_geometric
sun_rise_set_transit_ephem
sun_rise_set_transit_spa

option C:

rise_set_transit_geometric
rise_set_transit_ephem
rise_set_transit_spa

We agree. Deprecation is in the PR.
Folks, please vote for A, B or C.

B


Document how to add files to the bundle

The answer is put them in the cnab/app directory but let's document that.

Moved to the fully backed milestone since this isn't necessary to explain the project itself.


Save geolocation by using https://capacitorjs.com/docs/apis/geolocation

[ ]  Check if deep-foundation/store works with capacitor4. Update it
[ ]  Must be enablable/disablable
[ ]  Interval between getting geolocation must be able to be controlled

https://capacitorjs.com/docs/apis/geolocation

@ivansglazunov , с какими опциями надо запрашивать локацию? https://capacitorjs.com/docs/apis/geolocation#positionoptions

I have this problem in package.
https://github.com/suenot/nextjs/blob/geolocation-suenot/imports/position/save-position.ts#L24 temporary I save coordinates as String.


Similarity argoverse 1 / argoverse 2

Hey,
the argoverse 2 dataset comes with new and richer scenes. Comparing the scenes of av1 to av2 in the respective cities: How similar what you consider them? So, in short: Would you say training with argoverse 2 includes all the relevant data to perform well on argoverse 1? I would be particularly interested in the motion forecasting dataset.
Looking forward to your answer!
Thanks a lot!

Hi @odunkel
I think this depends on the particular task you are working on. For lidar-based tasks, the datasets look pretty similar. AV1 and AV2 use the same lidar sensor. For image-based tasks, we sort of expect things to generalize pretty well across cameras. The cameras are different, though. If your task is something like automatically predicting the map, then that might be more of an issue because the underlying map representation changed.
In general, AV2 scenarios contain a superset of information contained in AV1 scenarios. AV2 scenarios are also denser and more diverse. So yeah, big picture, I'd expect something trained on AV2 to perform well on AV1, assuming that comparison is even possible. E.g. you can't test your AV2 wheelchair detector on AV1, because wheelchairs are not in the AV1 taxonomy.

Hey, thanks for your answers. Yeah, I've already looked into the figures in the paper and they are definiteley helpful. My question was rather specifically directed to whether av2 "includes" all the scenario types av1 includes. For example, does av2 include the same / very similar roadways, set of surrounding agents and interactions, etc. av1 includes?
But, I guess, your comment, @James-Hays, answered that question. Thanks!

Hi @odunkel, apologies for the delayed response. The AV2 MF dataset could be viewed as a "superset" of the AV1 MF dataset when viewed from the perspective of roadway, actor, and interaction types.
We'd expect a model trained on AV2 data to perform better on AV1 data than the other way around, due to the addition of additional actor types and cities compared to the previous iteration of Argoverse.
If you have any additional questions relating to the composition of AV2, please feel free to re-open this issue! :-)


pull request for e2e test at 11/18/2016 20:56:19

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fixing redirect on /docs/

Motivation
To fix https://github.com/facebook/Docusaurus/issues/290 – that /docs/ should redirect to /docs/{language}/installation.html
Let me know if there's any changes I need to make!
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Local Docusaurus site testing – when you go to /docs/, you should be redirected to either /docs/en/installation.html  or /docs/installation.html
Related PRs
N/A

Closing due to inactivity. Feel free to reopen.


Corruption of filesystem with blackouts when porting to ESP32

Hello,
I am encountering an issue with blackouts affecting the filesystem. Whenever a blackout occurs, the filesystem becomes corrupted. Consequently, the system fails to restart because it cannot locate the necessary files to recover past transactions. I have experimented with both Spiffs and LittleFS, but encountered similar issues with both filesystems. Additionally, attempts to unregister the required partition during blackouts have not resolved the problem.
This one is the error I get with LittleFS
[OCPP] ERROR (MicroOcpp.cpp:146): multiple initialization failures detected
W (32012) esp_littlefs: ./components/esp_littlefs/src/littlefs/lfs.c:2074:warn: Superblock 0x0 has become unwritable
[OCPP] ERROR (MicroOcpp.cpp:151): clear local state folder (except WS creds): success
W (32082) esp_littlefs: ./components/esp_littlefs/src/littlefs/lfs.c:2074:warn: Superblock 0x0 has become unwritable
W (32142) esp_littlefs: ./components/esp_littlefs/src/littlefs/lfs.c:2074:warn: Superblock 0x0 has become unwritable
W (32212) esp_littlefs: ./components/esp_littlefs/src/littlefs/lfs.c:2074:warn: Superblock 0x0 has become unwritable
[OCPP] ERROR (OperationRegistry.cpp:47): DataTransfer not registered
[OCPP] info (MicroOcpp.cpp:383): Added ConnectorPluggedInput. Transaction-management is in auto mode now
[OCPP] info (MicroOcpp.cpp:383): Added ConnectorPluggedInput. Transaction-management is in auto mode now
[OCPP] info (MicroOcppMongooseClient.cpp:506): connection ws://xx.xxx.xxx.xxx:8080/steve/websocket/CentralSystemService/VCDxxxxxxxxxx -- closed
Do you have any insights on how to address this issue? Alternatively, how does MicroOCPP handle blackouts?
Thank you for your assistance.
Best regards,

Hi @imanolmc,
I don't have this issue on my devices (and actually it's the first time I see that LittleFS error message). My first guess is that maybe the partition table points to odd places on the flash, my second guess is that the flash is actually damaged. Or something goes wrong when mounting the file system? (the built-in integrations for Arduino and ESP-IDF SPIFFS provide an option to mount the partition on the host file system. When writing a custom fs integration, that's something which has to be done in the custom code)
Sorry that I have no further idea for the moment which part in the MO sources could explain this issue. If you find anything suspicious though, I would appreciate if you share it!

Hello Matth,
I am going to close the issue because the problem was a bug with the ESP processor.
Thanks for your help!
Imanol


Add mini chatbot to the website

This chatbot gives quike response to users questions . assign i like to work on it
Add labels gssoc-ext,hactoberfest accepted and level 3 @beRajeevKumar

Do it @ruchikakengal right away 🚀


[gardening] Add missing comments

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

This commit does not match LLVM style. Can you fix the namespace/c++ comment issue like in the other commits I commented on?

This LGTM.

BTW: In the future @danra, can you rebase these changes. I should have told you to do this, but I forgot.

Sure


python3Packages.pylibgen: remove, upstream unmaintained

Motivation for this change
package no longer builds, was going to fix then realized upstream has ended support for the package. https://github.com/joshuarli/pylibgen
also, @nico202 stated here that they are no longer active in nix.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

https://github.com/NixOS/nixpkgs/pull/91180
6 packages failed to build:
ape pantalaimon python37Packages.nvchecker python37Packages.sunpy python38Packages.nvchecker python38Packages.sunpy

22 packages built:
lexicon python27Packages.influxdb python27Packages.requests-file python27Packages.tldextract python37Packages.aioresponses python37Packages.graphite_api python37Packages.influxdb python37Packages.influxgraph python37Packages.requests-file python37Packages.structlog python37Packages.tldextract python37Packages.zeep python38Packages.aioresponses python38Packages.graphite_api python38Packages.influxdb python38Packages.influxgraph python38Packages.requests-file python38Packages.structlog python38Packages.tldextract python38Packages.zeep qutebrowser sewer

a few of the packages are fixed by https://github.com/NixOS/nixpkgs/pull/89694, going to merge their PR first

I guess this is a dependency of papis as well.
cc @teto

[2 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/91183
2 packages built:
papis python37Packages.papis


Display only some information's block on start

Hi,
How can i disable some information's block on start ?
Example : I want to disable The Block Standalone Containers and Images.
Thanks

If your concern is that these take up too much space, as of https://github.com/jesseduffield/lazydocker/pull/372 you can now increase the size of your selected panel, and you can also configure the panel to take up more space in an accordion-like effect with gui.expandFocusedSidePanel: true

I'm gonna close this off but let me know if the solution doesn't satisfy your needs


Fixed Hessian matrix formula for Newton-Raphson

Changed typo for Hessian matrix from dβ^2 to dβdβ'

Nice work @DianaTZhu , your help is much appreciated.
The fix will appear after we next publish, which will be within a day or two.

I'll create a new PR for this issue using the most updated version.


[windows] Error creating new project

Hello,
Thank You for creating this tool!
I am trying to create choo-app using this cli on windows. However, it errors out.
> node -v
v8.1.2
> npm -v
v5.3.0
> npx -v
v9.2.0
Globally installed
> create-choo-app --version

> create-choo-app monster

Should be all fixed! :D


Demo of appium test automation

This is a proof of concept of demoing a paperkey provisioning flow using Appium. After the provisioning completes, the device is removed and the app is quit.
To try out:

yarn global add appium
ANDROID_HOME=~/Library/Android/sdk/ JAVA_HOME=$(/usr/libexec/java_home) appium
KB_USERNAME="your_username" KB_PAPERKEY="your_paperkey" yarn test

(You'll also need to run yarn in the app-tests dir and be running an emulator for Android 7.0)

The commits 0dc76c31702a...187cdd35340a introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)


doubts with the use of the plugin

Hello! I am new to using everything that is Simplix Storage and I have a question, how can I get the data from a file? What I am doing is a constructor to create the configuration files easier, and I create them in my main plugin, but how can I access the config from another class? I would also like you to leave sample codes if possible, but mainly that is my question, since when using String aString = yaml.getString ("Key") ; // Default: "", I don't know how to use it well and also the "yaml" doesn't recognize me, maybe it's an easy question to answer but I don't know what to do, I am waiting for an answer, and thank you very much for reading!

Main Class:

Hey thefrogline,
Thank you for using SimplixStorage and asking this question here,
The way you are creating your YAML is already fine.
You can retrieve data from is using
getString("PAtH-TO-VALUE")
getInteger("PATH-TO-VALUE");

What exactly do you mean by "the config is not recognizing me"?
We would love to help you.
If you have any further questions, feel free to ask.
If you need real-time help, feel free to join our Discord: discord.simplixsoft.com
Regards,
Leonhard

Hi! Thanks for answering, because with "the config is not recognizing me" I mean this, the yml does not detect me, I am trying to make a command block to test all this of the configuration system, I also leave you an image of how it looks my project, so you can see if I'm doing something wrong. or I don't know if I'm doing something wrong, I'm new to all this library stuff and it's still a bit hard for me to understand . I am trying to do this with a configuration already with comments and configurationI am trying to do this with a configuration already with comments and configuration

Hello thefrogline,
as Intelij gives you a hint what is wrong, is says 'Cannot resolve symbol [...]', this is why the field you specified with the name 'yaml' is not in your current context. This is because there is no field named the same way. Did you create this field somewhere, if yes where?
Other than that. If you 'getString()' from some kind of Configuration you get a String object on which no method with the name 'getStringList()' is available. This is because there is a difference in a String object and some kind of Configuration object specified by SimplixStorage.
Regards yoyosource

Hello!
sorry for the delay in answering, I will explain all my code and if you do not understand something tell me, I speak Spanish and sometimes it is difficult for me to translate some things, but I will try also that I am something new in Java
1.- With my code first what I do is create a configmanager, which what it does is create all the configurations with a constructuctor called "create config":

2.- In my OnEnable () I start to create all the configurations, for now I have 3 tests, and they are all created with the same .yml format

3.- Now what I want to know is how from my main or my configmanager, get part of the configuration of some of my specific files, for example in security I want to get the blocked words, and in config.yml get the prefix of my plugin (it's just an example), perhaps because of the method by which I am doing it, it is not possible, or it would be better if they tell me how to use their api correctly, sorry for so much confusion when it comes to explaining myself and I understand very little, and perhaps misbehaving , but thank you very much for answering!

Hello thefrogline,
no problem. Your ConfigManager just creates a file and internally creates and Config object. So to access that object you need to have a field somewhere. I would consider a Map<String, Config> or a List<Config> for your use case. You should also read the Config if the config file exists, otherwise you just create a file on disk and never use it. Creating your config files in the onEnable() is fine. But your class CommandBlocker does not know the instances creates in ConfigManager or the onEnable() method. So creating a Field or saving in a Map, that is stored in a field, is your solution, I think. Maybe you could return a Config object from createConfig(String) so you can save it in the onEnable() method in some kind of field and access it in your BlockCommand class?
Regards yoyosource.

Thank you very much, what you recommended helped me and I was finally able to use the api, really good support, I haven't asked for support for a long time because I had a bad experience (in another plugin), but thank you very much for being patient and helping me! and sorry for delaying in answering :D

No problem. One problem with example plugins is, that you do not learn how use the plugin but copy what others give you. That is not good if you want to use the plugin on your own without relying on others. So yeah example help but a whole example plugin would not be good, in my opinion.
Regards yoyosource.


Please add PaWasapi_UpdateDeviceList

Please for windows make a distribution with PaWasapi_UpdateDeviceList enabled, its possible?
Thanks

Is #486 enough or do you also need a high-level Python API for that?

There is any way to use PortAudio of jitsi ?

Yes, but I don't know what's the best way to do that on Windows (which I assume that you are using).
You might be able to put a portaudio.dll file somewhere in your binary search path.
Alternatively, you can just replace the file _sounddevice_data/portaudio-binaries/libportaudio64bit.dll.
Here are some older comments about the "hotplug" fork:
https://github.com/spatialaudio/python-sounddevice/issues/125#issuecomment-514065063
https://github.com/spatialaudio/python-sounddevice/issues/382#issuecomment-986019480

or support it like an option?

It should already be possible, but this should be tested and documented.

or move the project to use that fork?

The fork doesn't seem well maintained, so no.
But you could try to help the PortAudio team to include that feature into the official PortAudio codebase.


new test qa test

Another table of tests is available for LANDSAT_47_C2_L2_SRCloudQA in the Landsat 4-7 C2L2 product guide, table 5-4.

Landsat 8 aerosol band also has one. Landsat 8 guide table 6-7


BitDropDown two-way binding issue

I am using BitDropDown in this way
    <BitDropDown                    ItemsProvider="LoadItems"                   Virtualize=true                   ShowSearchBox=true                   @bind-Value=SelectedValue>     </BitDropDown>
if I load the page by default value for SelectedValue, it will not be shown in `BitDropDown` as a selected item.

Thanks for contacting us. We're investigating this issue.
We'll let you know if it's possible to work on this issue or not.

The issue is accepted and planned. Resolving it will start ASAP.

Resolving this issue has started. It will be announced when this issue is resolved.

This issue has been resolved and it will be available in the next pre-release version. It will be announced when the pre-release becomes ready.

The pre-release has been published. Please give it a try and give feedback.


Error: Received unexpected PDU from server

Hello all!
I'm running Node-Red on Windows 10 and it is connected to OPC DA server which is installed on another machine with Windows XP. I can read data from the server with opcda-read node but sometimes I got an error message mentioned in title. I cannot recognise what can cause this error message but only server restart solve it (I mean full Windows restart on server side). I tried to restart only Node-Red on the client machine as the whole Windows as well but after the restart the issue still exist. On the picture bellow you can see the screenshot from the command promt:

When issue happens I'm not able to read out data normally with other client software installed on the same client machine (OPC Scout from Siemens) even if the server looks running normally.
Can you help me how to solve this issue? Restarting the server side Windows sadly cannot be an option.
greetings,
saalty

+1

I have the same error. @saalty303, could you solve it?

@Jaimenms it was a server side problem. It happened always, but when it depended of the amount of the data and/or cycle time. With larger amount of data or lower cycle time caused it occurs faster. If I sropped opc da service in Windows then it started again automatically and everything goes well. I made a batch file with taskkill instruction and executed it daily

Hi.
Please be sure that you configure your dcom settings first. Maybe one option is to install the matrikon simulator server to same machine with OPC server that you want to connect and try to connect to the Matrikon Simulator. Then everything is fine, there is some another problem that we search with.
https://www.matrikonopc.com/support/docs/MatrikonOPC-Windows-XP-DCOM-Configuration.pdf


[New Parser] reports error on . operator, even in valid use

Got the following error message
-- PARSE ERROR -------------------------------------------------------- Main.elm

I was not expecting this dot.

12|         |> List.map (.category >> identity)
                         ^
Dots are for record access and decimal points, so they cannot float around on
their own. Maybe there is some extra whitespace?

Detected errors in 1 module.

For the following code:
module Main exposing (..)

import Html exposing (text)

main =
    text "Hello world!"

locations =
    [ { category = "Blah" } ]
        |> List.map (.category >> identity)

Thanks for the report @Skinney, should be fixed now!
@rtfeldman, a smaller SSCCE would be:
getName = (.name)

The problem was committing too aggressively to the (+) interpretation, so it saw (. and thought that dot is not a valid symbol. I have added this case to the compiler tests in the commit that fixes the parser.

@evancz ah, nice!
This actually suggested a cool refactor to the arithmetic fuzz tests: https://github.com/elm-lang/core/pull/786/commits/c4b9fbf9c666e45ef4f0ce2d0e25a4b10ac115c7#diff-31c71c7f5985f526d8b80487f9ad3867


Update Ruby to 2.5.5 and nokogiri to 1.10.2

Ruby and nokogiri bugfixes. We're not impacted by the rubygems CVE fixes since we already bump rubygems ourselves.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

@jaymalasinha Once we merge this and ship the resulting build you should be good to go with push jobs.

Once my adhoc for this runs through jenkins I'll merge this in.


test: add tests to check content of sources.list

What type of PR is this?
This PR adds a test to post content VHD build to ensure that in case VHD is built from an official branch, any snapshot references don't exist in the /etc/apt/sources.list

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

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2566

Relevant Lines:
3613

💛  - Coveralls


Could not find class ::zulip::voyager

Trying to upgrade zulip from version 3.3 to 6.2. It seems the upgrade was successful, but when trying to restore backups, trying to run the script /home/zulip/deployments/current/scripts/setup/restore-backup /home/zulipNew/release-upgrade.backup.tar I am getting the following error,
Error: Evaluation Error: Error while evaluating a Function Call, Could not find class ::zulip::voyager for ip----.ec2.internal (line: 3, column: 1) on node ip---*-*.ec2.internal
Anything i am doing wrong or missed?

I think the issue is that you need to do some intermediate upgrades; I think we dropped some migration code because we expect you to need to upgrade the OS before upgrading from 3.3 to 6.2 anyway:
https://zulip.readthedocs.io/en/latest/production/upgrade.html#upgrading-the-operating-system may be helpful; you might need to do some intermediate upgrades.

update:
This email reception fails only when trying to generate an invite link. When user tries to register from the invite link, the email is never received. But when i tried to invite people via emails, those emails are received, they were able to signup there. So it seems the issue is only on that specific page where the system is supposed to send an email for verifying email address.

@iFieldSmart-Tech - I'd recommend reporting the email invitations problem in the #production help stream in the Zulip development community, and we’ll do our best to help you out there.
It sounds like the original issue with the restoring the backup during the upgrade process wasn't resolved, but is no longer an issue? If so, I'll close this issue, but it might be worth reporting that in a separate topic to the same #production help stream noted above.


Update README.md

The components.stoybook.stories array is used to build URLs, but the values are not encodeURI()-ed.
It would probably be a good idea to use that method.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/encodeURI or https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/encodeURIComponent

Hey @jmporchet, thanks for pointing out the issue. Instead of putting the burden on the user, I fixed it in https://github.com/zeplin/cli-connect-storybook-plugin/pull/14.


[Question] Is any list content pulled from other sources?

Just curious if these are maintained manually or you've been pulling content from any other sources.
If the latter knowing those would be good!

Hi!
Apologies for taking so long to address this. Yes, there are many domains that are taken from other sources that I find. While I haven't included the sources up until this point, I plan to now that the repo has gotten some decent traction. From this point I plan to include sources, if any, (aside from domains I find myself) in commits going forward.


result-items not clickable

I went over the code of your example with Nantes again and again but don't see what I'm missing.
The search seems to work fine until I try to click a result item. Debugging with the browser it appears to lack the class clickable.
A pull request on my own project shows what I tried:
https://github.com/lacemap/lacemap/pull/1
Live demo without fuse search: http://lacemap.github.io/

First I've clarified where and how should be used the statement : feature.layer = layer;
Next your code uses a MarkerCluster, and I'm yet to let the plugin deal with markers in cluster...

From your Nantes I example I figured out where you placed feature.layer = layer; as you can see in my pull request it does not make a difference in my situation. https://github.com/lacemap/lacemap/pull/1/files But it is good to have it in your readme for other users.
In the mean time I forked my own project so you can see a live demo of my attempt http://jo-pol.github.io/lacemap/
Does "I'm yet to let the plugin deal with markers in cluster..." mean you are in the process of changing your plugin?  That would be very kind.

I wish I had the time to work on it !  Your request is an incentive to find the time though ...

As another incentive a simple example for your readme:
http://lacemap.github.io/lacemap/searchable.html
Copy it to your own gh-pages branch, then you will have your demo on
http://naomap.github.io/leaflet-fusesearch/searchable.html
Strip the popup and you get down at 50 lines, this popup came with the quick start example and I have a use case for it.

I found another plugin that could help to fix this issue:
https://github.com/ghybs/Leaflet.MarkerCluster.Freezable
This plugin provides a disAbleClustering and enableClustering. If available these should be called when opening and closing the search popup.

My idea (and I had a prototype working) is to zoom to the level where the cluster is exploded. Seems more useful to me than disabling the cluster. Unlikely to find the time to publish that in the next couple of weeks :-(

Your solution sounds much better, I try to be patient.

jo-pol, do you still have a more simple example of the implementation of this search? I am new to coding and I can get the search to the point of pulling up a list but cannot get it to let me click on one. Having trouble understanding the
L.geoJson(data, {     onEachFeature: function (feature, layer) {         feature.layer = layer;     } }); section


Darkmode, close #85

Took a stab at this since it touches on an issue I mentioned a while back. This changes some of the colors in the existing UI because I wanted to avoid a proliferation of CSS vars.   'Figured we could iterate on it once the basic functionality was in place.
Simplest way to test this (on Mac) is to open up the web page, then go into System Preferences ->  General,  and toggle the OS settings between "Light" and "Dark" mode.  You should see changes take effect immediately (nice!).

Good to see the variables cleanup! The colors don’t match the previous ones on the regular version though

Fixed(ish).  Colors aren't an exact match because I'm just inverting the fg/bg colors for the tab bar.  This avoids creating more CSS vars just for this purpose, which I'd like to avoid if possible.  It makes the lights and darks there a bit more saturated but I don't see that as a bad thing, per se.
(Note: This also includes some some tweaks to the dependency and colorize controls so they're not quite so janky.)
Before:

After:

After (dark mode):

I'm not sure why the colors aren't an exact match in the regular version, neither version is good-looking. There are borders where there weren't before. and it's white where it wasn't.
If you want to fix the CSS variables issue using the correct colors, maybe I can look into the dark mode later.


Should source_status task fail when there are untracked changes?

Currently, I get a warning when there are untracked changes, thanks to the source_status task, but the dependent tasks continue.
@tibuch did you intend that source_status blocks on untracked changes, or was it meant merely as a warning?

Ah, ok, I must have initialized the repo before https://github.com/fmi-faim/ipa-project-template/commit/a2d1ee33c36e32764858bc9679e1e6a25d530564. Didn't see that commit before. So log_commit failed, instead of build_config. That makes sense.
Why aren't we depending on log_commit then, instead of pixi run log_commit?

Exactly. If you find a nicer solution, please let me know :)


Support for Custum Video Effects - Win10 UWP

Hey there,
as I'm not able to find a way to use SharpDX an grab an live image from a webcam for access shaders (without mapping and copying every frame from CPU side), I'm suggesting a way to support it in more general way. Therefore, creating a custom video effects using SharpDX seems to be promising, as it can be used for all kinds of video effects, too.
An example how such an effect is created can be found here:
https://docs.microsoft.com/en-us/windows/uwp/audio-video-camera/custom-video-effects
The hardware supported version provides an IDirect3DDevice and an IDirect3DSurface, which are both com based (see https://docs.microsoft.com/en-us/uwp/api/windows.graphics.directx.direct3d11.idirect3dsurface)
Unfortunately, I not really familiar with com and was not able to get any valid native points to create a shared resource for instance.
If this is already support please give me a hint ;)
Cheers,

I got it working.. I'll post some details after testing and maybe I find some time to create a small sample
thanks anyway

@kairohmer Do you plan to release example?

Unfortunately, I don't have time to write a complete example but I uploaded a file that contains the effect which allows to access the data: https://gist.github.com/kairohmer/af65c35a735879538082b04894238999
The idea is to provide a shared texture that can be consumed by you application. Note that the effect has to be part of Run-time Component Library (RTC) in order to work. A draft of the application side code can be found in the comments. Basically, you register two events. The first is fired every time a new (not yet seen) input texture arrives. Use the provided handle to open a shared Texture2D and store it in a hashmap with the handle as key. The second event is invoked for every incoming frame and provides the handle to the current input texture. Use the handle to access the dictionary and thereby the data in your shared texture. The events can occur anytime, so you probably want to store a flag when new images arrive and after processing an image to be able to use the image when it fits into your pipeline and to avoid redundant work.
I hope this will help some of you! In case of questions, feel free to ask ;)
Cheers,
Kai

@kairohmer  we need to democratize SharpDX UWP example.  Most users talk a lot about UWP XAML interrop but very few share.

@JimSEOW I'm not quite sure what you are trying to say. As far as I can see, there are no big changes concerning interrop since the "Metro" times, at least not on the windows platform. I recently ported a project from Windows 8.1 Store Apps to UWP -- the reason for the question above -- without ANY adjustments. I'm quite fine with the SwapChainPanels for SharpDX+XAML.
It's a bit different when it comes to Windows Desktop and interrop between WPF and DirectX11+, but unfortunately, the reason is the same.. nothing has changed ;)

There is no mystery to using SharpDX with UWP XAML. You simply use the ImageSource, SurfaceImageSource, VirtualSurfaceImageSource or SwapChainPanel as per the Microsoft documentation .. https://docs.microsoft.com/en-us/windows/uwp/gaming/directx-and-xaml-interop.
It's not really interop, as SharpDX already hides the interop aspect. You are just drawing to these XAML controls with SharpDX as normal. This is basic SharpDX usage, there is really no need for extra samples as there are already samples and documentation available. If you need specific help you should post to MSDN or gamedev.net

@Gavin-Williams @kairohmer
This is THE ONLY WORKING example of SharpDX XAML Interop and we are discussing  various challenging issues of making it works for different W10 and W10M insider builds
(a) W10M 15240.1000 Emulator works
(b) W10M 15240.0 950XL fails.
(c) W10: 16232.1000 fails.
(d) W10 16288.100 fails
As you see, the typical argument of porting 8.1 store app to W10 is not as trivial as you think.
Why SharpDX has problem working in different builds of W10?
We need more people testing SharpDX in UWP, especially if there is interest and need to develop the know how for a UWPCommunityToolkit template using SharpDX.

@JimSEOW ok, you gonna have to calm down otherwise I'm gonna lock your account to comment on any issues here.
Again, show a DirectX C++ example that is working on W10 Fall Creator, ported as-is to SharpDX, and is not working there.
If there is such a case, then yes, there is a bug in SharpDX. Otherwise, it is a bug in a yet unsupported OS/hardware (Same goes for 950XL and W10M)

@xoofx I tested the example https://github.com/minhcly/UWP3DTest  in desktop (Slow Ring) insider builds which will become official by Oct 15th using VS2017 (15.4 preview 2)
It would be great that this example becomes one of the "official" Store samples so there is a community effort to figure out why this XAML-interop UWP sample sometimes works and sometime not with the various (soon to be official) insider builds.

You need to take an official DirectX C++ sample, and port it to C#. That's what I did in the past for the XAML SurfaceImageSource DirectX interop sample. It is laborious but very easy to do. I won't take for granted a random code on internet that doesn't give me quite some assurance that they have almost exactly the same C++ code in C#. So if you care so much about DirectX C# on UWP, you could do this. (In the first place, I would be very surprised that the original StoreApp sample is not working anymore, unless they have changed something in the original sample since then...)

@xoofx I understand the request. I am not the right person to do that. This is why I seek help from the community. If there is no interest.  It if fine with me.

@xoofx  according to this doc.
Two ways to do UWP XAML Interop
(a) Shared Surface: SurfaceImageSource DirectX interop sample
(b) Swap chains :  UWP3DTest
I have ported the Shared Surface example before to W10. Not a problem
The challenges I face is the Swap Chians with various slow ring insider builds.

Great, so port an official DirectX sample that is using a Swapchain W10 so that we can have a look at the problem

I just tested UWP3DTest (Swap Chain)
For both VS2015 and VS2017, it works fine with Creative Update (15063) for the latest SharpDX 4.01
The problem I face is VS2017 15.4 preview 2 when targeting higher SDK (16278). still investigating, why this is the case.
The latest SharpDX is FINE. The UWP3DTest is a suitable SwapChain sample.


*.TSCN extension filter not present when loading Resource exports in editor

Operating system or device - Godot version:
Ubuntu Linux x86_64
Issue description (what happened, and what was expected):
When using export(Resource) in a GDScript file and then trying to load a tscn file into the property in the editor, the file dialog does not have a filter for *.tscn file extensions. If I select to show all files and select a tscn file it works as expected. A filter to tscn files should be added to that file dialog.
Steps to reproduce:

Open the linked example project (ResourceFileSceneBug)
Load the Main.tscn file
Click on the Node in the Scene view window
In the inspector, click the drop down for the "Set To Instance Scene" property and scroll down and click load
Notice that there is a filter for *.scn files but not one for *tscn files
If you use the filter "All Files (*)", you will see the InstanceScene.tscn file that you can select and it works

Link to minimal example project (optional but very welcome):
ResourceFileSceneBug.zip

Can someone confirm on 3.0 with import system changes?

I checked it out a few months ago, in the early 3.0 releases, when I hit the right entry by using arrow keys and randomly hitting ENTER, the problem still existed, so it might still exist.
I am unable to confirm that though, since the list is too long and doesn't scroll and I can't find the entry for loading. I'll open a bug for that issue.

In current master, following the repro steps results into the following UI message: *Error loading file: Not a resource!"
There is no resource loader that can load .tscn with a "Resource" hint.

A fix would be to edit scene_format_text.cpp and have the following:
void ResourceFormatLoaderText::get_recognized_extensions_for_type(const String &p_type, List<String> *p_extensions) const {

	if (p_type == "") {
		get_recognized_extensions(p_extensions);
		return;
	}

	if (p_type == "PackedScene" || p_type == "Resource")
		p_extensions->push_back("tscn");
	else
		p_extensions->push_back("tres");
}

However, it feels like the engine doesn't want to play around with generic "Resource" types and prefers to be explicit on the types of resources that want to be loaded.
I think the way you're supposed to do this now is the following:

Edit bug.gd and have export(PackedScene) var SetToInstanceScene. That is, type your exported variable adequately.
Now use the property inspector to load the resource. You'll be able to select your tscn file.

I believe this is working as designed.

Godot Master
Just to make things a bit simpler.
export(Resource) var SetToInstanceScene
Shows no .tscn

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about it here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

The .tscn filter is still missing in master.

I will fix these issue w/ @antoniocarreiro

Has this now been fixed? @Andre12Lazaro @antoniocarreiro
Or are there still missing functionalities?

Sooo the status is this: you can assign .tscn file to Resource, because Godot treats it as PackedScene. When you export PackedScene, you get more extensions available than just .tscn. So we should probably add all these extensions to the filters, because they are valid PackedScene.
However, I failed to pinpoint where all these extensions are defined.

This depends on the output of ResourceLoader.get_recognized_extensions_for_type("Resource") which indeed does not include .tscn:
[tres, anim, atlastex, cubemap, curvetex, font, largetex, lmbake, material, mesh, meshlib, meshtex, mp3str, multimesh, oggstr, phymat, res, sample, scn, shape, stylebox, tex, theme, translation, vs, world]

I'm not sure this is really worth fixing though, see discussion in #24212.
What's the use case for exporting a generic Resource to select a PackedScene, as opposed to exporting with PackedScene hint directly?


Fixed choice with dispatch key on array element.

DAFFODIL-2162

The idea is to insure invariants the software depends on are being upheld. This particular invariant is that any sequence, even just the one implied when a choice has an array element child, can be asked for its format annotation object, and one is positively returned, even if it's an empty one containing no annotations.
The separators of an enclosing sequence do not appear around array element occurrences for a choice branch that is an array element. To achieve that you need an explicit sequence surrounding the array element so you can express the separator.
The rationale here is that choices don't get to express separator properties, but that's where they would have to appear if the choice branch (that is an array element) was to have separators.
I suppose the alternative would be to say choices are transparent to separation by an enclosing sequence. But that's not true if the choice expresses initiator/terminator, which choices are allowed to have.  I lost this battle long ago in the DFDL Workgroup. My position was only sequences should have delimiters of any kind, period, but this did not win the day, as other existing format description systems including some based on XSD, allowed these sorts of properties to be expressed on a variety of constructs.


No metrics for failed notifications due to a non-existent receiver

What did you do?
We refactored our receivers and routes, and accidentally misconfigured some routes to point to a non-existent receiver. We had alerts trigger that used this route, and went to the non-existent receiver.
What did you expect to see?
I thought that the alertmanager_notifications_failed_total metric would increase, which we have a separate monitoring system checking. If this didn't increase, then I would expect that there was a separate counter metric that would increase.
What did you see instead? Under which circumstances?
The specified metric did NOT increase, and there were no other metrics that reported the issue. We did see many log messages that looked like this:
level=error ts=2019-02-13T03:09:24.472247912Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"

We were not checking the logs because we did not realize that anything was wrong.
Environment

System information:

/etc/alertmanager $ uname -srm
Linux 4.14.96-coreos x86_64

Alertmanager version:

/etc/alertmanager $ alertmanager --version
alertmanager, version 0.15.2 (branch: HEAD, revision: d19fae3bae451940b8470abb680cfdd59bfa7cfa)
  build user:       root@3101e5b68a55
  build date:       20180814-10:53:39
  go version:       go1.10.3

Prometheus version:

Not relevant.

Alertmanager configuration file:

It's large, so I'll skip posting all of it unless specifically asked. Here is the snippet of the routing that was executed. The receiver pagerduty-coretech was non-existent (we had a typo).
  - match_re:
      alert_owner: ^(?:.+)$
      alert_target: ^(?:^$)$
    routes:
    - match:
        alert_owner: coretech
      routes:
      - receiver: slack-coretech-automation
        match:
          severity: warning
      - receiver: slack-coretech-alerts
        match:
          severity: critical
        continue: true
      - receiver: pagerduty-coretech
        match:
          severity: critical

Prometheus configuration file:

Not relevant.

Logs:

...
level=error ts=2019-02-13T02:22:15.263925808Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:24:24.463157291Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:27:15.266087294Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:29:24.471469919Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:32:15.266334191Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:34:24.470538053Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:37:15.266720423Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-13T02:39:24.470741861Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
...

This should have been caught at config reload.

Here are the logs of the alertmanager container and the config-reload container at the time of reload:
2019/02/04 20:44:20 config map updated
2019/02/04 20:44:21 successfully triggered reload
level=info ts=2019-02-04T20:44:20.866667798Z caller=main.go:322 msg="Loading configuration file" file=/etc/alertmanager/config/alertmanager.yaml
level=error ts=2019-02-04T20:44:21.073089037Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-04T20:49:21.073262712Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"
level=error ts=2019-02-04T20:54:21.073484341Z caller=dispatch.go:280 component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="stage for receiver missing"

Note that we have an alert that's been firing for a long time, which is silenced. I believe this alert is why we see the "stage for receiver missing" error immediately after reload.
When I look at the Prometheus metrics, they don't show a change in the alertmanager_config_last_reload_success_timestamp_seconds. This is really strange to me, because it clearly shows up in the logs, but not the metrics.

In a test alertmanager cluster running v0.16.1 with active alerts, I updated the configuration to point to completely bogus receivers (as in, those receivers are not defined anywhere), and where the routing and receivers are otherwise completely valid.
I'm seeing that each alertmanager member successfully reloads its configuration, but then starts logging the error message I've posted previously.
If this is supposed to be caught at config reload, this does not appear to be the behavior.

@jutley could you try with the current master branch? I can't reproduce your issue there and the configuration management has been refactored a bit in https://github.com/prometheus/alertmanager/pull/1744.

Okay, I successfully reproduced this on master branch, and I understand why you probably couldn't. This bug only appears in at least two layers of routing. Here is what I did:
Run alertmanager:master:
docker run -p 9093:9093 --name alertmanager prom/alertmanager:master

Update the configuration with a sub-route pointing to a fake receiver:
➜  cat << EOF | docker exec -i alertmanager sh -c 'cat > /etc/alertmanager/alertmanager.yml'
global:
  resolve_timeout: 5m

route:
  group_by: ['alertname']
  group_wait: 10s
  group_interval: 10s
  repeat_interval: 1h
  receiver: 'web.hook'
  routes:
  - match:
      foo: bar
    receiver: fake
receivers:
- name: 'web.hook'
  webhook_configs:
  - url: 'http://<IP_ADDRESS>:5001/'
inhibit_rules:
  - source_match:
      severity: 'critical'
    target_match:
      severity: 'warning'
    equal: ['alertname', 'dev', 'instance']
EOF

➜  curl -X POST localhost:9093/-/reload
failed to reload config: undefined receiver "fake" used in route

As intended, this fails, as the fake receiver does not exist.
Now, repeat this, with a sub-sub-route:
➜  cat << EOF | docker exec -i alertmanager sh -c 'cat > /etc/alertmanager/alertmanager.yml'
global:
  resolve_timeout: 5m

route:
  group_by: ['alertname']
  group_wait: 10s
  group_interval: 10s
  repeat_interval: 1h
  receiver: 'web.hook'
  routes:
  - match:
      foo: bar
    routes:
    - match:
        foo: bar
      receiver: fake
receivers:
- name: 'web.hook'
  webhook_configs:
  - url: 'http://<IP_ADDRESS>:5001/'
inhibit_rules:
  - source_match:
      severity: 'critical'
    target_match:
      severity: 'warning'
    equal: ['alertname', 'dev', 'instance']
EOF

➜  curl -X POST localhost:9093/-/reload

This succeeds, as we see in the logs:
level=info ts=2019-03-01T23:16:49.0091745Z caller=coordinator.go:119 component=configuration msg="Loading configuration file" file=/etc/alertmanager/alertmanager.yml
level=info ts=2019-03-01T23:16:49.0095161Z caller=coordinator.go:131 component=configuration msg="Completed loading of configuration file" file=/etc/alertmanager/alertmanager.yml

Reproduced it. Thanks! Fixing...

https://github.com/prometheus/alertmanager/pull/1780


Test Neutron Settings change

Test changes made to Neutron settings to ensure:

Current behavior is unaffected.
Users can override existing settings at any point of the sass file.

Test what happens when user updates breakpoint settings, especially before neutron is included in the file.


chore: release v0.19.0

https://github.com/sigoden/aichat/releases/tag/v0.19.0-rc2

@sigoden I understand the issue #653 is closed and that RC tags may not be available for the Homebrew Formula.
However, could you kindly explain how RC versions work with the aichat tool? What's the typical timeline from RC to an official release? I looked at the README and wiki pages but couldn't find any information about releases. Let me know if I've missed or overlooked anything.


time: document proper usage of Timer.Stop

After calling Timer.Stop(), and then clearing the channel, the channel can later be filled anyway. Presumably the task to send to the Timer channel has been put on a run queue, and isn't remove when Stop is called. Here's a test that demonstrates it:
func TestTimerFiresAfterStop(t *testing.T) {
    fail := make(chan struct{})
    done := make(chan struct{})
    defer close(done)
    for range iter.N(1000) {
        tr := time.NewTimer(0)
        tr.Stop()
        // There may or may not be a value in the channel now. But definitely
        // one should not be added after we receive it.
        select {
        case <-tr.C:
        default:
        }
        // Now set the timer to trigger in hour. It definitely shouldn't be
        // receivable now for an hour.
        tr.Reset(time.Hour)
        go func() {
            select {
            case <-tr.C:
                // As soon as the channel receives, notify failure.
                fail <- struct{}{}
            case <-done:
            }
        }()
    }
    select {
    case <-fail:
        t.FailNow()
    case <-time.After(time.Second):
    }
}

Increase the value in iter.N if it doesn't trigger on your system initially. Note that it isn't even necessary to call Reset() to trigger this. You can remove that line. You can Stop() a timer, and then still receive a value from it, some time after you drain it.

The go routine should have a time argument, otherwise there is a race for the tr value. But even with that fixed the channel fires after some trials.

The go routine should get the timer as argument, because otherwise there will be a race. But even with that fixed the channel fires every few test runs.

Aside: you do realize the iter package was a joke, right?
I'd change your example to use for i := 0; i < 1000; i++.
@ulikunitz, I think the code looks fine. The goroutine closure captures the new tr variable each iteration.

This bit of code:
tr := time.NewTimer(0)
tr.Stop()
// There may or may not be a value in the channel now. But definitely
// one should not be added after we receive it.
select {
        case <-tr.C:
        default:
}

does not guarantee that there isn't an in-flight value on tr.C, because Stop may not have stopped the timer before it fired, yet that doesn't mean that the channel was ready to receive when the select ran. So it's a race.
See extensive previous discussion on #11513 and https://groups.google.com/forum/#!topic/golang-dev/c9UUfASVPoU.
In the latter thread, @rsc suggests doing
if !timer.Stop() {
  <-timer.C
}

I also previously filed #12721 to ask for an easier way to create a new, stopped timer.

Indeed, looks like a dup of #12721.
But let's keep this bug open as a documentation fix issue for Timer.Stop. An example might help too.

Yes, it is a race.
if !timer.Stop() {
  <-timer.C
}

Is not sufficient, as there's no way to tell if the fired event has already been received. Which in my use case it often has. This is necessary, as linked in the thread: http://play.golang.org/p/Ys9qqanqmU

@Brad: You are right regarding the closure.

The race appears to be in the function timerproc() in runtime/time.go. The mutex timers.lock is unlocked for calling the runtime timer function, which sends the time into the channel. Therefore timer.Stop can return before the time value is written to the channel.
Calling the runtime timer function with the mutex locked, ensures that timer.Stop cannot return before the channel is written too. I tested all.bash and also checked the test function from @anacrolix. I think this can be justified because there is no way for a user to set the runtime timer function. I even checked whether I can compile the time package outside of GOROOT; it didn't work.
On the other hand I can't see how
if !tr.Stop() {
    <-tr.C
}

should fail. At least on Linux AMD64 I was not able to trigger the modified test to fail, even after creating 1 million goroutines.
It appears there are two ways to address this issue:
A) Amend the documentation of time.Stop(), or
B) calling the runtime timer function with the timers.lock hold.
I prefer B because it fixes the Test above.
@bradfitz You are right, there is no race in the test code.

I still don't think there's a race here. This is a documentation issue. See
https://groups.google.com/d/msg/golang-dev/c9UUfASVPoU/tlbK2BpFEwAJ for how to use Stop correctly.
See also #11513.

I should add: There is an argument for making
tr := time.NewTimer(0)
tr.Stop()
// There may or may not be a value in the channel now. But definitely
// one should not be added after we receive it.
select {
        case <-tr.C:
        default:
}

work, but it doesn't today, and we're certainly not going to do that for Go 1.7. Maybe early in Go 1.8 if someone wants to file a different bug.

Just implemented a small fix, which can be used as a drop-in replacement.
https://github.com/desertbit/timer

@m4ng0squ4sh FWIW, ixn the revised docs (https://beta.golang.org/pkg/time/#Timer.Stop) please note the phrase "assuming the program has not received from t.C already". That wrapper package is actually impossible to use correctly if the program has received from t.C already: the receive in the wrapper's Stop method will block waiting for a value that isn't coming.

@rsc Thanks for pointing that out. That's why I included the following phrase:

This should not be done concurrent to other receives from the Timer's channel.

I'll add some more documentation so it's clear for everybody. It might be possible to even solve that problem by adding another channel and a Mutex to the wrapper. However I have to try that first. Are there any other solutions?

Note that most people would not consider:
t := time.NewTimer(1*time.Second)
<-t.C
t.Reset(2*time.Second)

to be calling t.Reset "concurrent to other receives". But the wrapper will hang in that case.
The obvious solution is to use time.Timer directly and not write a wrapper. I suppose you could kick off a new goroutine to pass time events along a new channel and keep track of whether any have been seen, but then you're creating a much more heavyweight timer, all to avoid one or two lines of code.

True story. Just have seen the updated doc (beta). Now it's clear. Thanks.
Yeah, firing up a new goroutine,... works. Just tried a small implementation but it's really too heavyweight.

Finally, here is a lightweight drop-in replacement. This makes more sense for my projects, because sometimes it's just time consuming to fix the broken Reset behavior. Especially in a concurrent situation...
@rsc Would love to get some feedback and/or some hints, if everything is correct. Thanks!

@m4ng0squ4sh your Reset() is correct.
@rsc, there is really should be separate function in runtime to serve Timer.Reset() instead of calling stoptimer()+starttimer(), cause timer.lock should be captured once. Currently concurrently called Timer.Reset() could damage timers heap (this is not my finding).

@funny-falcon Thanks for having a look at it!

@m4ng0squ4sh I don't think the Reset is correct. If it were correct I don't think this program would panic:
package main

import (
	"time"

	"github.com/desertbit/timer"
)

func main() {
	for {
		print(".")
		c := make(chan int, 1)
		t := timer.AfterFunc(1*time.Millisecond, func() { c <- 1 })
		timer.AfterFunc(1*time.Millisecond, func() { t.Reset(100 * time.Second); close(c); t.Stop() })
		<-c
		<-c
	}
}

and if you fix that, be careful not to make this one hang:
package main

import (
	"time"

	"github.com/desertbit/timer"
)

func main() {
	done := make(chan bool)
	timer.AfterFunc(1*time.Second, func() {
		println("1")
		timer.AfterFunc(1*time.Second, func() {
			println("2")
			done <- true
		})
	})
	<-done
}

I don't think you can paper over this.

@rsc first, it panics with "time" package also.
It panics not because of some mistake, but because function passed to AfterFunc runs in separate goroutine. So, both AfterFunc's handlers are triggered simultaneously after 1 millisecond timeout: goroutines are spawned and scheduled for execution, - and it happens that second one runs before first one.
It is cause of design of AfterFunc in standard package: "user function should not affect internal timerproc, so that, goroutine is fired to run function". It was told me in https://codereview.appspot.com/12876047

@rsc AfterFunc was added to the package to stay compatible to the standard lib timer implementation. As mentioned in the doc:

AfterFunc waits for the duration to elapse and then calls f in its own goroutine.

So the panic must be expected.
Here is a simple ping example showing the general problem. I do believe that many gophers are not aware of the special Reset() conditions and that this is a common mistake.
package main

import "time"

const (
	keepaliveInterval = 2 * time.Millisecond
)

var (
	resetC = make(chan struct{}, 1)
)

func main() {
	go keepaliveLoop()

	// Sample routine triggering the reset.
	// Example: this could be due to incoming peer requests and
	// a keepalive check should be reset to the max keepalive timeout.
	for i := 0; i < 1000; i++ {
		time.Sleep(time.Millisecond)
		resetKeepalive()
	}
}

func resetKeepalive() {
	// Don't block if there is already a reset request.
	select {
	case resetC <- struct{}{}:
	default:
	}
}

func keepaliveLoop() {
	t := time.NewTimer(keepaliveInterval)

	for {
		select {
		case <-resetC:
			time.Sleep(3 * time.Millisecond) // Simulate some reset work...
			t.Reset(keepaliveInterval)
		case <-t.C:
			ping()
			t.Reset(keepaliveInterval)
		}
	}
}

func ping() {
	panic("ping must not be called in this example")
}

@funny-falcon Yes, I know it panics with the Go time package. My point is that this package claims to provide a "fixed" Reset without these problems. It does not.

@rsc, it claims to fix Reset only for channel variant. There were no mention of AfterFunc before your comment.

Let's move this discussion off of a closed issue and onto golang-nuts. Thanks.


Simulation run failed for no clear reason

Run with tellurium
https://run.biosimulations.dev/simulations/615b562c2f1add0daafd6d2e

I noticed this and have been looking into it. I cannot figure out why the
status check will sometimes fail. Everything seems to run just fine on the
backend
On Mon, Oct 4, 2021 at 6:20 PM Jonathan Karr @.***>
wrote:

Assigned #3135
https://github.com/biosimulations/biosimulations/issues/3135 to
@bilalshaikh42 https://github.com/bilalshaikh42.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/biosimulations/biosimulations/issues/3135#event-5406990928,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHX4FIF3DDG567HK37VPAO3UFISEBANCNFSM5FKMFACQ
.

Here's two more (of 89) that failed in publishing the BiGG models. These are both on org. The same OMEX files succeeded in subsequent runs, so the OMEX file probably isn't the issue.

61610ddf09e39101d573cbab
61610e1ae3fd8649821315d3

I'll compiled any failed runs to the first message

This sounds like an issue I fixed. Previously, the code was only checking the status of the last step within the sbatch group reported by sacct. However, saact doesn't immediately return the full list of steps. Its possible for the last step reported by sacct to be completed, while other steps haven't even begun yet (or have failed). I think its safer way to check this is to focus on the status of the first output of sacct, which reports the status of the entire run, and to check the status of each individual step. The state of the entire job doesn't change to completed until each step completes.
In addition, the code now checks that status of each individual step, including the unzip step which wasn't previously monitored by SLURM. Simulation runs are now interpreted as failing if any step fails. Before, it was possible, for example, for the the run to be interpreted as succeeded even if the HSDS storage step failed because the code had only been checking the status of the final step (storage to S3).
Log messages are also now improved by naming each step (--job-name). This enables the code which checks their status (looking for each step to have succeeded) to list the failed steps by their names. This also removes the reliance on the job steps being at specific indices (since they're identified by their names rather than step numbers).
@bilalshaikh42 close this if it sounds like the error you saw.

Sounds great. Thanks!