remux prerm script fails on reenable

Just did a fresh update from <IP_ADDRESS>6 to <IP_ADDRESS>5 (Have not installed ddvk-hacks, can confirm everything is working fine)
After running toltecctl reenable, it does throw an error which it shouldn't do over a unit file not existing.
Full log:
reMarkable: ~/ toltecctl reenable
Mounting '/home/root/.entware' over '/opt'
Created symlink /etc/systemd/system/local-fs.target.wants/opt.mount → /lib/systemd/system/opt.mount.
Downloading https://bin.entware.net/armv7sf-k3.2/Packages.gz
Updated list of available packages in /opt/var/opkg-lists/entware
Downloading https://toltec-dev.org/stable/rmall/Packages.gz
Updated list of available packages in /opt/var/opkg-lists/toltec-rmall
Downloading https://toltec-dev.org/stable/rm2/Packages.gz
Updated list of available packages in /opt/var/opkg-lists/toltec-rm2
Removing package xochitl from root...
Removing package toltec-bootstrap from root...
Removing package rm2fb-client from root...
Removing package reboot-guard from root...
Removing package rm2-suspend-fix from root...
Removing package tarnish from root...
Removing package display from root...
make sure to disable / uninstall any launchers like draft, oxide or remux before
rebooting your tablet to complete the uninstallation
Removing package remux from root...
Disabling remux
Failed to disable unit: Unit file remux.service does not exist.
Removing package libvncclient from root...
Installing xochitl (0.0.0-13) to root...
Downloading https://toltec-dev.org/stable/rmall/xochitl_0.0.0-13_rmall.ipk
Installing toltec-bootstrap (0.2.2-1) to root...
Downloading https://toltec-dev.org/stable/rmall/toltec-bootstrap_0.2.2-1_rmall.ipk
Installing rm2fb-client (1:0.0.23-1) to root...
Downloading https://toltec-dev.org/stable/rm2/rm2fb-client_1_0.0.23-1_rm2.ipk
Installing rm2fb-client (1:0.0.23-1) to root...
Installing display (1:0.0.23-1) to root...
Downloading https://toltec-dev.org/stable/rm2/display_1_0.0.23-1_rm2.ipk
Installing reboot-guard (1.0.1-6) to root...
Downloading https://toltec-dev.org/stable/rmall/reboot-guard_1.0.1-6_rmall.ipk
Installing rm2-suspend-fix (0.0.0-2) to root...
Downloading https://toltec-dev.org/stable/rm2/rm2-suspend-fix_0.0.0-2_rm2.ipk
Installing tarnish (2.4-1) to root...
Downloading https://toltec-dev.org/stable/rmall/tarnish_2.4-1_rmall.ipk
Installing libsentry (0.4.17) to root...
Downloading https://toltec-dev.org/stable/rmall/libsentry_0.4.17_rmall.ipk
Installing liboxide (2.4-1) to root...
Downloading https://toltec-dev.org/stable/rmall/liboxide_2.4-1_rmall.ipk
Package display (1:0.0.23-1) installed in root is up to date.
Installing remux (0.2.3-1) to root...
Downloading https://toltec-dev.org/stable/rmall/remux_0.2.3-1_rmall.ipk
Installing libvncclient (0.9.13-2) to root...
Downloading https://toltec-dev.org/stable/rmall/libvncclient_0.9.13-2_rmall.ipk
Configuring toltec-bootstrap.
Configuring rm2-suspend-fix.
Configuring xochitl.
Configuring display.
Created symlink /etc/systemd/system/multi-user.target.wants/rm2fb.service → /lib/systemd/system/rm2fb.service.
Configuring rm2fb-client.
Configuring libvncclient.
Configuring libsentry.
Configuring liboxide.
Configuring tarnish.
Configuring remux.

Run the following command(s) to use remux as your launcher
$ systemctl enable --now remux

Configuring reboot-guard.
Collected errors:
 * pkg_run_script: package "remux" prerm script returned status 1.

We should replace the call to systemctl disable --now remux in preremove with something similar to what is done for tarnish.
https://github.com/toltec-dev/toltec/blob/ed897b71dd66a1b3fae6dfe54d8a87949483e5d2/package/oxide/package#L100-L107
This should probably be made into a command in install-lib to avoid repetition.


Executing pipeline-runner fails due to conflict with volumes

For some strange reason, everytime as of 30 minutes ago, i execute the pipeline-runner locally, i get a message that looks a bit like this:

2023-07-10 13:10:22.097 pipeline_runner.container: Creating container: project-backend-3DgOYbAU-step-build-test-laravel-execute-postman-collection
2023-07-10 13:10:22.118 pipeline_runner.runner: Error during pipeline execution: 409 Client Error for http+docker://localhost/v1.41/containers/create?name=project-backend-3DgOYbAU-step-build-test-laravel-execute-postman-collection: Conflict ("Conflict. The container name "/project-backend-3DgOYbAU-step-build-test-laravel-execute-postman-collection" is already in use by container "5df14689167e5496b58122aef3ffdc818a012448ef73e4001a1e3d2c486c8b1a". You have to remove (or rename) that container to be able to reuse that name.")

I also have docker desktop installed and I can observe that the containers has status "Created" even after the run fails.
This wasn't happening too long ago, but since it began, i have tried:

Restarting my laptop
Pruned my Docker Images
Removed all useless volumes
Used a test pipeline configuration that looks a bit like this:

pipelines:
branches:
'new':
- step:
script:
- mysql -h mysql -u root -p"$MYSQL_ROOT_PASSWORD" -e "SELECT 1;"
services:
- redis
- mysql
definitions:
services:
redis:
image: redis:latest
mysql:
image: mysql:5.7
variables:
MYSQL_DATABASE: my-db
MYSQL_ROOT_PASSWORD: password

and im still getting the error (excuse the formatting, but that is yaml)

Hi @sonneybouy
It's actually a container, not a volume, that was left over and is now causing the issue. You can remove that container with the following command:
docker rm --force 5df14689167e5496b58122aef3ffdc818a012448ef73e4001a1e3d2c486c8b1a
You may need to prune volumes and networks again after deleting the container:
docker volume prune and docker network prune.
I'm aware of this bug where containers are not cleaned properly after a pipeline execution. I'll try to look at it soon, but in the meantime, this should unblock you.

Good enough workaround for me, although it did cost me some volumes

@sonneybouy Sorry about the other volumes. Just a note that could be helpful, all volumes created through the pipeline runner are tagged with org.acidrain.pipeline_runner.project=.... If you need to prune volumes, you can filter for those with that label.


fix thread-safety regression in RSS* decoders

Introduce little ThreadLocal template helper to implement some kind of 'thread local storage' for the RSS-type readers that support 'stacked' 1D codes (i.e. 2D codes actually) and need to store some state between decoding different lines.
This is a fix for the thread-safety regression introduced in 17284ea7775d9a4da034a3f817320fbb10813de0

That is true: coming up with new threads would potentially grow that map larger and larger. Preventing that would require to hold some cleanup-token on the stack and then I'd be almost exactly where I started.
I agree that the original has the advantage of cleaning that all up during stack-unwinding and is less involved. I'll revert it but still do away with the dynamic_cast as there is only ever exactly one type at the end of the pointer.
The real issue, I think, is that those RSS-decoders are not 1D but 2D decoders and simply don't fit properly in this RowReader abstraction. Also the fact that they get fed the lines in a 'random' order makes no real sense.


docs(props): fix propTypes link and error type

Fixes https://github.com/erikras/redux-form/issues/958
Maybe the error prop for Field and FieldArray should also be any instead of string? However, I did not find those files in the repo, so this PR is just updating Props.md. Thanks.

What version of Redux Form is referenced here? I'm asking this because Field and FieldArray are available since v6 but you linked v5 codebase.

The intention is good here, it just links to master instead of v6.

Since we cannot change to target branch of a PR, a new one must be submitted... Could you do this?
By the way, thanks @fredrikolovsson for the contribution!

I did it manually. Thanks, @fredrikolovsson.

My apologies for the noob PR. I selected the v6.0.0-alpha.8 tag and then forked, but realize now I should've forked the v6 branch 😄


Problem with num_batch

In your code, the num_batch is calculated by below way:
num_batch = len(user_train) / args.batch_size
But it could cause error in
for step in tqdm(range(num_batch), total=num_batch, ncols=70, leave=False, unit='b')
Because

Traceback (most recent call last):
File "main.py", line 61, in
for step in tqdm(range(num_batch), total=num_batch, ncols=70, leave=False, unit='b'):
TypeError: 'float' object cannot be interpreted as an integer

It seems you’re using Py3, please try Py2 instead.

On Aug 1, 2019, at 8:31 PM, 張君瑞<EMAIL_ADDRESS>wrote:
In your code, the num_batch is calculated by below way:
num_batch = len(user_train) / args.batch_size
But it could cause error in
for step in tqdm(range(num_batch), total=num_batch, ncols=70, leave=False, unit='b')
Because
Traceback (most recent call last):
File "main.py", line 61, in
for step in tqdm(range(num_batch), total=num_batch, ncols=70, leave=False, unit='b'):
TypeError: 'float' object cannot be interpreted as an integer
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/kang205/SASRec/issues/6?email_source=notifications&email_token=AA5DLVBJTR4R3GANBVULGITQCOTBVA5CNFSM4II2CE7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HC635CQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AA5DLVGPFSK2WJDL22QCYNLQCOTBVANCNFSM4II2CE7A.

Thanks for your suggestion~


'Undefined' response

I followed the code template, gave it my API key and the mailgun domain address that looks like "https://api.mailgun.net/v3/sandbox###########################.mailgun.org" and followed the rest of the template. However, when I ran my code/server, I get "Undefined" as the body response. Is it because I don't have my own domain set up? Or did I miss any steps?

I also tried "https://api.mailgun.net/v3/sandbox###########################.mailgun.org/messages", also doesn't work.

Hmm not sure I understand the issue. Some sample code would be helpful. You need to supply domain to the constructor:
const mailgun = require('mailgun-js')({apiKey: api_key, domain: 'sandbox12345.mailgun.org'});

yea I had trouble figuring out the correct domain String. Got it to work! Thanks!


Refinements to date for a new event

A new event should get as a date the first day of the term it's being added to. We can probably fix the hours:minutes to 1PM - 2PM until design gives us a better default.

Coverage remained the same at 100.0% when pulling caff57a53e0ffebcf389aa61cd82204132a15927 on bp323:issue-162 into ce9fc834da749b33aedeefe33a1a40ba37dc1e25 on CUL-DigitalServices:master.

Looks good! Left one small comment. Reassigning to @bp323 for follow-up

Reassigning to @bp323 to fix the coverage issue.


ActionsSdkAssistant has the same problem with type

Same as in https://github.com/actions-on-google/actions-on-google-nodejs/issues/11, but in ActionsSdkAssistant.

This has been fixed :)


Add standalone (electron) shell

Vue-Devtools is an essential piece of the Vue ecosystem, but it is currently tied to a web browser.
It would be really nice if we had a tiny electron shell that can run the devtools without a browser, with the ability to remotely debug Vue apps.
For example, I would love to bring Vue devtools to nativescript-vue.
I'm not very familiar with the codebase, but as far as I can tell we could add a new shell, which would start an electron app, and a websocket server. The websocket server would be the bridge, and it would require a separate package to serve as the client.
(Side note: I found it a little confusing that the "client" side is called a backend, but that might be due to a different mindset.)
If someone more familiar with devtools wants to chat and get this going, I'm available!

Hi, thanks for this remote debugger.
I'm trying to use it with a cordova app using global package but I have an error with socket.io:
GET http://file/socket.io/?EIO=3&transport=polling&t=M7Rrv8y net::ERR_NAME_NOT_RESOLVED
The remote debugger works fine if I run my www folder on my computer (but not with my tablet).

Now that we released the app, let's close this issue. If anyone has an problem with it, please open a new issue.


chore(politics-tracker): add styled.d.ts, change custom theme to .ts

Step #0 - "Build Image": ./components/landing/statistics-main.js:30:13
Step #0 - "Build Image": Type error: Element implicitly has an 'any' type because expression of type 'string' can't be used to index type '{ blue: string; gray: string; orange: string; yellow: string; green: string; white: string; black5: string; highlightRed: string; greenLake: string; landingYellow: string; pink: string; brown: string; black: string; skinColor: string; disable: string; }'.
Step #0 - "Build Image":   No index signature with a parameter of type 'string' was found on type '{ blue: string; gray: string; orange: string; yellow: string; green: string; white: string; black5: string; highlightRed: string; greenLake: string; landingYellow: string; pink: string; brown: string; black: string; skinColor: string; disable: string; }'.
Step #0 - "Build Image":
Step #0 - "Build Image":   28 |   justify-content: center;
Step #0 - "Build Image":   29 |   background: ${({ color, theme }) =>
Step #0 - "Build Image": > 30 |     color ? theme.backgroundColor[color] : theme.backgroundColor.green};
Step #0 - "Build Image":      |             ^
Step #0 - "Build Image":   31 |   ${({ theme }) => theme.breakpoint.md} {
Step #0 - "Build Image":   32 |     height: 211px;
Step #0 - "Build Image":   33 |   }
Step #0 - "Build Image": error Command failed with exit code 1.

type check 時，landing page 爆掉了


Preserve focus when executing "CMake: Run Without Debugging"

This change addresses item #1138
This changes visible behavior
Resolves #1138.
Prevent terminal from taking focus when executing "CMake: Run Without Debugging".
The following changes are proposed:

Set preserveFocus argument  of Terminal.show() to true.

@bobbrow  Would you review this PR?
This is a one-liner fix and easy to review, I think.


[BUG] Raw tin,silver,lead

Description:

Minecraft version 1.17 drops when mining ore raw tin,lead,silver cannot be melted in the furnace

Technical Information:

Minecraft: 1.17
Fabric API: 0.11.6
Industrial Revolution: 1.10.0-alpha.2
Modpack: All of Fabric 4 - 1.17

Make sure you have correctly updated because those recipes exist since alpha 2. If the issue persists open another issue with screenshots.


Start button is not clickable

Describe the bug
The entire start button on home page is not clickable. Only the text written on it "Start" is clickable.
To Reproduce
Steps to reproduce the behavior:

Go to the home page
Click on start button but not on the "start" text written on it
next page will not open
it will open only after clicking on "start" text

Expected behavior
Entire button should be clickable, not just the hyperlink.
Environment (please complete the following information):

OS: Windows
Browser: Chrome

@QAZIMAAZARSHAD I would like to fix this bug as LGMSOC Participant. Plz assign this to me.

Which button.
The start button is perfectly working?

If u click this green marked area on the start button, then it will not take u to next page. Next page (login page) will open only if u click on the hypertext. So, i mean to say that entire button is not clickable, only the hypertext is clickable.

If u click this green marked area on the start button, then it will not take u to next page. Next page will open only if u click on the hypertext on the button. So, i mean to say that the entire button is not clickable, only the hypertext "Start" written on it is clickable.
@QAZIMAAZARSHAD


Fix README.md

These are my proposed changes to fix the readme file and to correctly inform people of the usage of dlin.

You're totally right. This was a typo.


Flaky Aspire.Hosting.Testing.Tests.TestingBuilderTests.SelectsFirstLaunchProfile

Build Information
Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=818717
Build error leg or test failing: Aspire.Hosting.Testing.Tests.TestingBuilderTests.SelectsFirstLaunchProfile(genericEntryPoint: True)
Pull request: N/A

Error Message
Fill the error message using step by step known issues guidance.

{
  "ErrorMessage": "System.Net.Http.HttpRequestException : Connection refused (localhost:",
  "ErrorPattern": "",
  "BuildRetry": false,
  "ExcludeConsoleLog": true
}

Known issue validation
Build: :mag_right: https://dev.azure.com/dnceng-public/public/_build/results?buildId=818717
Error message validated: [System.Net.Http.HttpRequestException : Connection refused (localhost:]
Result validation: :white_check_mark: Known issue matched with the provided build.
Validation performed at: 9/26/2024 7:49:58 PM UTC

Report

Build
Definition
Test
Pull Request

819992
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest
dotnet/aspire#5670

819961
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest
dotnet/aspire#5970

819956
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest
dotnet/aspire#5470

819952
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest

819924
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest
dotnet/aspire#5969

819869
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest
dotnet/aspire#5470

819831
dotnet/aspire
Aspire.Playground.Tests.ProjectSpecificTests.Aspire.Playground.Tests.ProjectSpecificTests.AzureFunctionsTest

818717
dotnet/aspire
Aspire.Hosting.Testing.Tests.TestingBuilderTests.SelectsFirstLaunchProfile(genericEntryPoint: True)

Summary

24-Hour Hit Count
7-Day Hit Count
1-Month Count

7
8
8

note: the error pattern here will match unrelated failures too :|

@captainsafia - can you take a look at this? At least disable it so it doesn't affect others' PRs.

@captainsafia - can you take a look at this? At least disable it so it doesn't affect others' PRs.

The PR to disable it went in last night. We have an idea for potential fix that I'll try to get a PR out for today.


Abandoned?

What is the status of this project? Are you looking for new maintainers?

it's always bad to see such great project being abandoned


Further packaging improvements.

Splitted the package into 2: the headers and the lib.

@sunqm

@sunqm


Type in Waypoint Update Update the Properties of JS Object

removed extra comma
closes #4438
closes #4434

:+1:
In the future it would be better to submit this as two separate PRs.
Review the Contribution Guidelines for naming conventions.

Thanks SaintPeter, I messed up with git when submitting them. Tried to fix it and messed that up too so just submitted them. Wasn't sure if you would accept them :-)
Happy to help

You can use git cherry-pick to pull individual commits out of a branch.
http://think-like-a-git.net/sections/rebase-from-the-ground-up/cherry-picking-explained.html
Anyway, it turned out for the best.


feat(open-payments): add create incoming payment

Changes proposed in this pull request

Method for creating an incoming payment

Context

https://github.com/interledger/rafiki/issues/663#issuecomment-1327622353

Checklist

[x] Related issues linked using fixes #number
[x] Tests added/updated
[ ] Documentation added
[x] Make sure that all checks pass

Merging as @raducristianpopa doesn't have access to do so:


Custom test framework

I opened this PR against my other PR so that changes to the code are clearer and it's hopefully easier to check if the new framework brings more clarity.
This PR currently contains the first step of such a framework: a function to build all initial records in one place. It is currently called prepareTest and returns an object intended to be later used to access created records (and in further iteration also dry-up the test definitions).
Two thing to note:

Created records can be accessed by the key used in the records definition, but will have generic auto-generated ids
Internally it still uses lab.before and lab.after, which I think is good for keeping everything in sync

At this time I'm seeking feedbacks on the type of approach and the "api definition" of the prepareTest function, basically this piece of code:
      const test = prepareTest(lab, {
        teams: {
          calledTeam: { name: 'called team', organizationId, users: ['called'] }
        },
        users: {
          caller: { name: 'caller', organizationId, policies: ['testedPolicy'] },
          called: { name: 'called', organizationId }
        },
        policies: {
          testedPolicy: Policy()
        }
       })

Please go light on the implementation has it is still temporary to demonstrate my intentions (it will probably be moved to utils)
Next steps will be:

use prepareTest in all experiments in this file, expand as needed
create framework to define auth tests, something along the lines of

      test('should authorize caller with policy for specific users')
        .withPolicy([{
          Effect: 'Allow',
          Action: ['authorization:users:read'],
          Resource: [`/authorization/user/WONKA/*/${calledId}`]
        }])
        .endpoint({
          method: 'GET',
          url: `/authorization/users/${calledId}`,
          headers: { authorization: callerId }
        })
        .shouldRespond(200)

@paolochiodi looks good ! I would be even more aggressive with the verbosity (ie get rid of all the js/json-like objects) but it's nitpicking.

@paolochiodi I think the direction is the right one. I'm not really sure about the amount of code we need to write to load the fixtures we want. Do you think we can use something like node-sql-fixtures ?
The end goal I'd like to achieve is having the fixtures loaded without necessarily depend on our own ops functions. This may also save us the trouble of changing a lot of tests if we change the interface of a function.
I'm not sure if this is the place to discuss this (we also have a ticket assigned to @nherment https://github.com/nearform/labs-authorization/issues/254 )

@p16 Yeah, we may end up using something like that.
As a first step I just wanted to extract the code we are using now.
If we expand the usage of "prepareTest" we can later rewrite its implementation to whatever fits better.
About using our own *Ops functions: for the moment it has the benefits of managing all the side effects (i.e. will delete team_users when a team is deleted).
I agree that in the long run this may not be the best solution, because the functions can get more complex and maybe have side-effects we don't want to replicate (i.e. creating an audit log, sending email etc)

Is it worth considering clean up for the whole db?
My use case scenario: I do a debug, stop debug at breakpoint, fix, npm run init db, npm run load test data, npm run again debug or tests. Generating new ids at each run avoids this type of problems, no not a big issue.
I like the test structure, looks very good.

@nherment @p16 @mihaidma I've updated the PR extending the usage of prepareTest to other tests. (I had to expand it a bit to accomodate all use cases).
I've also introduced some changes:

prepareTest is now Factory
the factory now returns the object with the records directly
other minor name changes

Let me know if you have any additional feedback.
Before the end of the day I'll close this PR and merge it into the main PR, where I'll address any further change.


TE-9.2: Added deviation static_lsp_unsupported

TE-9.2: Added deviation static_lsp_unsupported

Pull Request Functional Test Report for #3347 / a04e898937a064ebb4f644915151a0bdfafc41eb
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TE-9.2: MPLS based forwarding Static LSP

Cisco 8000E

TE-9.2: MPLS based forwarding Static LSP

Cisco XRd

TE-9.2: MPLS based forwarding Static LSP

Juniper ncPTX

TE-9.2: MPLS based forwarding Static LSP

Nokia SR Linux

TE-9.2: MPLS based forwarding Static LSP

Openconfig Lemming

TE-9.2: MPLS based forwarding Static LSP

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

TE-9.2: MPLS based forwarding Static LSP

Cisco 8808

TE-9.2: MPLS based forwarding Static LSP

Juniper PTX10008

TE-9.2: MPLS based forwarding Static LSP

Nokia 7250 IXR-10e

TE-9.2: MPLS based forwarding Static LSP

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
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 55.5%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
1983

Relevant Lines:
3573

💛  - Coveralls


add changes proposed in #889 to fix repeat function

see #889

@MiczFlor, could you check this? Reporter confirmed this works.


Documentation is not clear on how to find distributed tracing records in Application Insights

The documentation does not make it easy to understand what records will result in Application Insights as a result of making distributed tracing calls in the application. After much experimentation, what I have found is that:

The records from activities will appear in the dependencies table with a type != "HTTP", and running

dependencies | where type != "HTTP"

Is probably the best way to find them. Once you have one of those records you can search for others as part of the operation with the same operationID, using the query in the doc, but that assumes you know the operationID.

The Application map can be used to find dependencies between services. However in the right hand pane used to select items to view, there are only queries for the slowest items, or those with failures. There no list of say a sampling of operations.

Viewing one of the slowest results will bring up a gant chart view, which will show traces as entries in the chart.

What is unclear is how to get to that chart in a more efficient manner. This documentation should walk through the best mechanisms for being able to view the data.

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

ID: 95aa5722-ec96-40fb-b03a-0c9e73c7a18c
Version Independent ID: 28259d9c-5174-f138-c0aa-bc2125c3be1a
Content: Azure Application Insights telemetry correlation - Azure Monitor
Content Source: articles/azure-monitor/app/correlation.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@samsp-msft
Thanks for your feedback! We will investigate and update as appropriate.

@reyang - Tagging you as it would also be good to link to this info from the SDK.

@samsp-msft thank you for sharing this feedback.
@AaronMaxwell, could you please review this and share your thoughts? Thank you

Thanks for your dedication to our documentation. This issue has been moved to an internal work item for investigation in a future documentation sprint.
https://dev.azure.com/msft-skilling/Content/_workitems/edit/63642
Thanks in advance for your understanding as we investigate to provide the most accurate answer and documentation updates. #please-close


Arguments to compile

Feature
Trying to sandbox one of my projects in RustPython and  wasm, I ran into this difference in behaviour:
$ python3 -c 'compile("",filename = "bot-code.py", mode = "exec")'
$ wasmtime run rustpython.wasm -- -c 'compile("",filename = "bot-code.py", mode = "exec")'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: Expected at least 0 arguments (1 given)

This is rustpython.wasm 0.1.3 from https://wapm.io/package/rustpython#explore.
Python Documentation
https://docs.python.org/3/library/functions.html#compile

Also, this error message looks a bit … odd:
$ wasmtime run rustpython.wasm -- -c 'compile(source = "",filename = "bot-code.py", mode = "exec")'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: Expected at least 0 arguments (0 given)

Also, this error message looks a bit … odd:
$ wasmtime run rustpython.wasm -- -c 'compile(source = "",filename = "bot-code.py", mode = "exec")'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: Expected at least 0 arguments (0 given)

Confirmed with latest master, built with cargo build --release --target wasm32-wasi --features="freeze-stdlib"

Confirmed with latest master, built with cargo build --release --target wasm32-wasi --features="freeze-stdlib"

This is pure cargo-culting, but
diff --git a/vm/src/builtins/make_module.rs b/vm/src/builtins/make_module.rs
index 6c7f4a9a..2af6882a 100644
--- a/vm/src/builtins/make_module.rs
+++ b/vm/src/builtins/make_module.rs
@@ -103,11 +103,11 @@ mod decl {
     #[derive(FromArgs)]
     #[allow(dead_code)]
     struct CompileArgs {
-        #[pyarg(positional)]
+        #[pyarg(any)]
         source: Either<PyStrRef, PyBytesRef>,
-        #[pyarg(positional)]
+        #[pyarg(any)]
         filename: PyStrRef,
-        #[pyarg(positional)]
+        #[pyarg(any)]
         mode: PyStrRef,
         #[pyarg(any, optional)]
         flags: OptionalArg<PyIntRef>,

seems to fix it.

This is pure cargo-culting, but
diff --git a/vm/src/builtins/make_module.rs b/vm/src/builtins/make_module.rs
index 6c7f4a9a..2af6882a 100644
--- a/vm/src/builtins/make_module.rs
+++ b/vm/src/builtins/make_module.rs
@@ -103,11 +103,11 @@ mod decl {
     #[derive(FromArgs)]
     #[allow(dead_code)]
     struct CompileArgs {
-        #[pyarg(positional)]
+        #[pyarg(any)]
         source: Either<PyStrRef, PyBytesRef>,
-        #[pyarg(positional)]
+        #[pyarg(any)]
         filename: PyStrRef,
-        #[pyarg(positional)]
+        #[pyarg(any)]
         mode: PyStrRef,
         #[pyarg(any, optional)]
         flags: OptionalArg<PyIntRef>,

seems to fix it.

Yeah, I think that would fix the issue of keyword arguments! I generally use compile() with only positional arguments, so I guess I haven't run into this bug. Feel free to submit a PR/patch with that change! :)

Yeah, I think that would fix the issue of keyword arguments! I generally use compile() with only positional arguments, so I guess I haven't run into this bug. Feel free to submit a PR/patch with that change! :)


Optimizer.step() & scheduler.step() in wrong order when restoring

Hello,
first of all, thank you for your amazing work.
I am using pytorch 1.5.1 and I receive the warining that lr_scheduler.step() is called before optimizer.step() when I restore a checkpoint in order to continue a training. I don't get this warning when I start a new training by scratch.
Is this normal or should I be worried?
Thanks

The _LRScheduler class steps the scheduler when getting initialized. I removed this. I also moved the   adjust_lr call to the end of the iteration for loop.


17 1.3 js fix

This fixes adding timespans in #17 and finally enables javascript tests.
NOTE: In order for javascript tests to work, you'll need to install phantomjs
Ubuntu: apt-get install phantomjs
Mac: brew install phantomjs
Windows: https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-1.9.8-windows.zip
If you encounter problems, detailed instructions can be found here: https://github.com/teampoltergeist/poltergeist

Coverage remained the same at 100.0% when pulling a3974b418aa5174c168a71539280e9a48ed1a4b0 on 17_1.3_js-fix into 910150e6fa9ae89c7b7e500e033222f6ba3773fd on dev.

Coverage remained the same at 100.0% when pulling d2ba873cdd68224a5b24fa3ddb06ca0e5952bf3d on 17_1.3_js-fix into 910150e6fa9ae89c7b7e500e033222f6ba3773fd on dev.

Coverage remained the same at 100.0% when pulling 6568b02fac95dba56975b7e2e82940de3a929a4f on 17_1.3_js-fix into 7753cf2f7844360bc71e34c9e8c616908a603deb on dev.


New Proposal - Customer Premise Equipment Management

This API manages fixed devices provided by network operators that reside at customer premises.  The core functionality includes requesting service sites and devices associated with a customer account, managing Wi-Fi BSSs, and rebooting devices.
The API is designed to support simple use cases, such as allowing a developer to set up a guest Wi-Fi network, and more complex use cases, such as allowing management of multiple devices across several service sites.  The design also allows for extension to support future use cases such as setting up Thread networks or managing mobile hot-spots not attached to a service site.
We have chosen to use the terminology “Customer Premise Equipment” (CPE) but are open to the use of a different term. Due to some of the complexities regarding potential naming conventions this API could support (e.g., router, switch, access point, modem, fixed wireless, gateway, border router, devices that support multiple functions, etc.), we felt it best to work with CAMARA to address proper terminology.
yaml - GitHub - yaml - Swagger Editor - slides - GitHub

GitHub Repository have been created here, this issue can be closed

@TEF-RicardoSerr APIBacklog.md need to be updated as well.


[HPRO-1410] CS Fixer updates

Q
A

Target Release?
3.3.0

Bug fix?
❌

New feature?
❌

Database migration?
❌

New configuration?
❌

Composer updates?
✅

NPM updates?
❌

Jira Ticket(s)
HPRO-1410

Summary
This PR adds PHP CS Fixer to the composer dev dependencies, then updates the CS Fixer settings to:

Put imports in alphabetical order
Place class elements in order of public -> protected -> private
Remove unused imports from classes

It also adds a few other rules which are less noticeable but bring HealthPro in line with the style used on some other projects.

@thornview Could you resolve these merge conflicts?

Replaced by #1387 (to provide a cleaner update).


HTU21D_test compiler warning cleanup.

Added -Wall to compile command and added headers to HTU21D_test.c to address the resulting warnings.
Thanks for the example code!

Hi @HankB, thank you very much for your contribution to the project! Thanks, Leon


Malformed bower.json

The dependency section is set as an array ( [ ] ) but should be an object ( { } ).

Whoops, thanks for the report!


Make NVD API key optional

Description
According to DependencyCheck's upgrade notice, providing an NVD API key is not required but (only) highly recommended. That is to say, the NVD API can still be used without an API key (albeit with some restrictions). However, nvd-clojure requires providing a key. Since registering for a key incurs some organizational overhead, it would be nice to be able to use the public API in the meantime. In other words: Make the API key optional on nvd-clojure's end, as well.
Version
4.0.0
Java version
openjdk version "17.0.7" 2023-04-18
OpenJDK Runtime Environment (build 17.0.7+7-nixos)
OpenJDK 64-Bit Server VM (build 17.0.7+7-nixos, mixed mode, sharing)

Installation compliance

[X] I have read again and made sure that I'm following exactly the instructions for my tool of choice (Leiningen, Clojure CLI, Clojure CLI Tool).
[X] I understand that false positives can be skipped locally and should be reported to DependencyCheck.

Thanks for a sensible issue/suggestion.
I don't think it's in the best interest of any involved part to keep fostering a slower, issue-prone code path.
The organizational overhead is minimal as obtaining a key requires providing no PII at all. The key you will instantly receive is more akin to a user ID than to a password - therefore if it's any trouble to keep it secret for CI, I'd consider it safe to check into version control.

By organizational overhead I was mainly referring to the terms of use one has to accept for creating an account. For example, they state:

Services which utilize or access the NVD API are asked to display the following notice prominently within the application: "This product uses the NVD API but is not endorsed or certified by the NVD."

Not 100% sure what is implied by this and since I am not a lawyer, I'd either have to clear this with my legal department or have somebody higher up the chain check this off and accept the risk of not fully grokking the implications.
In any case, it's not important to me anymore since we've since figured it out and registered an account :smile: Feel free to close.

Services which utilize or access the NVD API are asked to display the following notice prominently within the application: "This product uses the NVD API but is not endorsed or certified by the NVD."

Most likely this refers to libraries/services which sole purpose is performing NVD-based checks.
Note that a production app does not package NVD in any way - it usage is typically limited to a build-time check.
Cheers - V


System monitor displays process as numeric value instead of binary on/off

Home Assistant release with the issue:

0.xx - 0.81
Last working Home Assistant release (if known):
unknown
Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

System monitor
Description of problem:
System monitor displays process as numeric value instead of binary on/off.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
  - platform: systemmonitor
    resources:
      - type: disk_free
        arg: /
      - type: memory_free
      - type: processor_use
      - type: since_last_boot
      - type: process
        arg: ffmpeg

Traceback (if applicable):

Additional information:

The systemmonitor was created when the binary_sensor platforms were not available. This is not bug but a feature request.
Please use the forum for feature requests as stated in the PR template. Thanks

Sorry, @fabaff but I don't understand your comment/approach. The sensor of process from systemmonitor shows only off or on , however, as you can see from the screenshot, it shows a graph which is always empty since the values do not match it. As for me, it is a bug in the current state and not a feature request due to some historical reason you mentioned. It is a bug since you cannot use data from this sensor as expected. Could you please reopen the issue?


[FEATURE] Subtitle Format Support?

Is there anyway you could make it so I can export the file into an subtitle format like srt or ass? I don't just want to backup the chat I want to watch the video with chat in vlc.
Thank you.

This sounds like an interesting idea. The thing is, you could make it using the python module. I spent 5 minutes implementing a basic version, which could look something like this:
from chat_downloader import ChatDownloader

def seconds_to_time(seconds):
    int_seconds = int(seconds)
    h, remainder = divmod(abs(int_seconds), 3600)
    m, s = divmod(remainder, 60)
    decimal = str(round(float(seconds) - int_seconds, 3))[2:]
    return f"{'-' if seconds < 0 else ''}{h:02}:{m:02}:{s:02},{decimal}"

url = 'https://www.youtube.com/watch?v=VlWb1RONsIw'
chat = ChatDownloader().get_chat(url, start_time=0)       # create a generator

max_duration = 5
counter = 1
last_time = 0
for message in chat:
    current_time = min(message['time_in_seconds'], last_time + max_duration)

    # Output to console
    print(counter)
    print(f'{seconds_to_time(last_time)} --> {seconds_to_time(current_time)}')
    print(f"{message['author']['name']}: {message['message']}")
    print()

    last_time = message['time_in_seconds']
    counter += 1

which outputs:
1
00:00:00,0 --> 00:00:05,0
Adam Hložek: so hikaru isnt playing?

2
00:00:10,242 --> 00:00:12,375
Swiss Reyes: future world champion the best from the WEST and SOlid in the SOuth

3
00:00:12,375 --> 00:00:17,375
Joakim Raatikainen: It's chess I guess???

4
00:00:32,123 --> 00:00:32,532
singhalarjun19: waiting for chessbrah to say ... is this theory ? :face_with_tongue:

5
00:00:32,532 --> 00:00:37,532
Adam Hložek: IS HIKARU PLAYING, YES OR NO?

6
00:01:09,42 --> 00:01:14,42
Donald Metzger: Not this tournament, although Hikaru qualifies by points. He is almost certain to have enough points for the final tournament already

7
00:01:14,936 --> 00:01:19,936
Mark Shark: Hikaru is streaming the tourney

8
00:01:31,302 --> 00:01:33,825
Adam Hložek: rip

9
00:01:33,825 --> 00:01:38,825
Rick Jena: lets go hottub Carlsen show em the power of hottub

10
00:02:07,565 --> 00:02:12,565
Marcin Beski: I wish a lot of luck to Jan Krzysztof Duda

11
00:02:45,184 --> 00:02:50,184
Alekhine Battery: Would you like to have David Howell, Tania Sachdev or Simon Williams review your chess games live? Leave a comment in this community post! - https://bit.ly/2WuKsvh

12
00:02:51,756 --> 00:02:52,111
Alekhine Battery: Would you like to have David Howell, Tania Sachdev or Simon Williams review your chess games live? Leave a comment in this community post! - https://bit.ly/2WuKsvh

13
00:02:52,111 --> 00:02:57,111
Geshvad Nasiri: so will win again

14
00:03:09,174 --> 00:03:14,174
David Trottier: Are we going to see infamous aimchess ads throughout this tour?

15
00:03:21,281 --> 00:03:24,921
Rick Jena: I would rather choose lithium battery

16
00:03:24,921 --> 00:03:29,921
Karan Anvekar: Hi

17
00:03:37,584 --> 00:03:42,584
Alekhine 2255: guys will there be dubov in next tournament

This is just a very basic example, so I'm sure you could improve it. For example, I added a "maximum duration" (a message will not stay on the screen for more than 5 seconds), but you could improve it so that messages do not disappear too quickly.

Anyway, thanks for taking the time to respond.   Unfortunately I do not know how to code but yes I would love to see srt or ass support if possible. It would be cool if the subtitle was formatted in danmaku/nico nico douga style too so multiple comments can be on the screen at the same time. Something like this https://github.com/m13253/danmaku2ass
I'm a japanese student and getting the chat into a subtitle format would help me and I'm sure people that want to archive vtubers and other japanese youtubers would be interested in this as well.  Thanks again.

@Sephy1 Here's a script to download to SRT:
import argparse
import os
from chat_downloader import ChatDownloader
from typing import List

class ChatMessage:
    TimestampSeconds: float
    Author: str
    MessageText: str
    def __init__(self, timestamp_seconds: float, author:str, message_text: str) -> None:
        self.TimestampSeconds = timestamp_seconds
        self.Author = author
        self.MessageText = message_text

class SrtLine:
    Index: int
    StartTimeSeconds: float
    EndTimeSeconds: float
    MessageText: str
    def __init__(self, index: int, start_time_seconds: float, end_time_seconds: float, message_text: str) -> None:
        self.Index = index
        self.StartTimeSeconds = start_time_seconds
        self.EndTimeSeconds = end_time_seconds
        self.MessageText = message_text
    def __seconds_to_timestamp(self, seconds: float):
        int_seconds = int(seconds)
        h, remainder = divmod(abs(int_seconds), 3600)
        m, s = divmod(remainder, 60)
        milliseconds = round(1000 * (float(seconds) - int_seconds))
        return f"{'-' if seconds < 0 else ''}{h:02}:{m:02}:{s:02},{milliseconds:03}"
    def to_string(self) -> str:
        return f'{self.Index}\n{self.__seconds_to_timestamp(self.StartTimeSeconds)} --> {self.__seconds_to_timestamp(self.EndTimeSeconds)}\n{self.MessageText}\n\n'

def even_spaced_timestamp_filter(chat_messages: List[ChatMessage], smoothing_interval_seconds: float = 5):
    """Smooths out chat message timestamps within regularly-spaced intervals, so that timestamps are more evenly-spaced. This helps readability when bursts of several messages occur at nearly the same time."""
    if len(chat_messages) == 0:
        return
    if smoothing_interval_seconds <= 0:
        raise ValueError(f'smoothingIntervalSeconds must be positive, but was {smoothing_interval_seconds}')
    minIndex = 0
    maxIndex = -1
    minTimestamp = 0
    maxTimestamp = smoothing_interval_seconds
    lastTimestamp = chat_messages[-1].TimestampSeconds
    while minTimestamp < lastTimestamp:
        while maxIndex + 1 < len(chat_messages) and chat_messages[maxIndex + 1].TimestampSeconds < maxTimestamp:
            maxIndex += 1
        commentsInInterval = maxIndex - minIndex + 1
        if commentsInInterval > 0:
            for i in range(0, commentsInInterval):
                chat_messages[minIndex + i].TimestampSeconds = minTimestamp + (2 * i + 1) * smoothing_interval_seconds / (2 * commentsInInterval)
        minIndex = maxIndex + 1
        minTimestamp += smoothing_interval_seconds
        maxTimestamp += smoothing_interval_seconds

def parse_chat_messages(chats) -> List[ChatMessage]:
    chatMessages: List[ChatMessage] = []
    for chat in chats:
        messageText: str = chat['message']
        # Replace shorthand emotes, like :partying_face:, with UTF, like 🥳.
        emotes = chat.get('emotes')
        if emotes:
            for emote in emotes:
                utfId = emote.get('id')
                shortcuts = emote.get('shortcuts')
                if utfId and shortcuts:
                    for shortcut in shortcuts:
                        messageText = messageText.replace(shortcut, utfId)
        chatMessages.append(ChatMessage(
            timestamp_seconds=chat['time_in_seconds'],
            author=chat['author']['name'],
            message_text=messageText))
    return chatMessages

def parse_srt_lines(chat_messages: List[ChatMessage], max_seconds_onscreen: float = 5) -> List[SrtLine]:
    if max_seconds_onscreen <= 0:
        raise ValueError(f'max_seconds_onscreen must be positive, but was {max_seconds_onscreen}')
    srtLines: List[SrtLine] = []
    for index, chatMessage in enumerate(chat_messages):
        nextTimestampSeconds = chat_messages[index + 1].TimestampSeconds if index + 1 < len(chat_messages) else float("inf")
        srtLines.append(SrtLine(
            index=index,
            start_time_seconds=chatMessage.TimestampSeconds,
            end_time_seconds=min(nextTimestampSeconds, chatMessage.TimestampSeconds + max_seconds_onscreen),
            message_text=f'{chatMessage.Author}: {chatMessage.MessageText}'))
    return srtLines

if __name__ == '__main__':
    parser = argparse.ArgumentParser()
    parser.add_argument('--url', required=True)
    parser.add_argument('--max_seconds_onscreen', required=False, default=5)
    parser.add_argument('--smoothing_interval_seconds', required=False, default=5)
    parser.add_argument('--title', required=False, default='subtitles')
    args = parser.parse_args()

    chatMessages = parse_chat_messages(ChatDownloader().get_chat(args.url))
    even_spaced_timestamp_filter(chatMessages, args.smoothing_interval_seconds)
    srtLines = parse_srt_lines(chatMessages, args.max_seconds_onscreen)

    srtPath = os.path.join(os.getcwd(), f'{args.title}.srt')
    with open(srtPath, 'w', encoding='utf-8') as srtFile:
        for srtLine in srtLines:
            srtFile.write(srtLine.to_string())
        print(f'Wrote SRT subtitles to {srtPath}')

It's similar to xenova's example above, with a few changes:

Fixes a bug where emojis are left in their raw input form, like :partying_face:, instead of 🥳.
Fixes a bug where messages whose milliseconds timestamp ended with 1 or 2 zeros (such as 32.500) were output with trailing zeros removed, which causes them to appear onscreen longer than they should.
Adds an evenly-spaced timestamp filter, so that when several people post comments at the same time, the spike of comments gets smoothed out over time, which makes comments easier to read.

To use it, run pip install chat-downloader if you haven't already, save the script to a file like srt_subtitle_downloader.py, and then run it:
python srt_subtitle_downloader.py --url https://www.youtube.com/watch?v=0Z75i0upejA

If you run python srt_subtitle_downloader.py --help, it'll print out a help menu. It doesn't support multiple comments onscreen, but maybe you can tweak it to do that?
@xenova It might be cool to add a --output chat.srt option to chat-downloader itself as you suggested, but the even_spaced_timestamp_filter requires having all comments upfront, which doesn't work with the continuous_write streaming abstraction, which assumes output can be done comment-by-comment. Any thoughts? Possible solutions:

Add an special case if --output ends with .srt that downloads all the chat, and then writes them to SRT in one go. This might be a little ugly, but would sidestep the problem.
Change continuous_write.py to accept the full list of chat comments. Probably impossible, since the point of continuous_write.py was to support streaming comments as they are downloaded.
Remove the even_spaced_timestamp_filter. This would allow using the existing output streaming interface, but it makes subtitles harder to read.

Oh hey, I made this! Check out https://github.com/9001/softchat

Hey ! Here is a simple edit of @xenova proposal. This version simply generates the SRT file.
from chat_downloader import ChatDownloader
import csv

def seconds_to_time(seconds):
    int_seconds = int(seconds)
    h, remainder = divmod(abs(int_seconds), 3600)
    m, s = divmod(remainder, 60)
    decimal = str(round(float(seconds) - int_seconds, 3))[2:]
    return f"{'-' if seconds < 0 else ''}{h:02}:{m:02}:{s:02},{decimal}"

url = 'https://www.youtube.com/watch?v=VlWb1RONsIw'
chat = ChatDownloader().get_chat(url, start_time=0)       # create a generator

max_duration = 5
counter = 1
last_time = 0
for message in chat:
    current_time = min(message['time_in_seconds'], last_time + max_duration)

    # Output to console
    # Open the SRT file in append mode
    with open('output.srt', 'a') as file:
        file.write(f'{counter}\n')
        file.write(f'{seconds_to_time(last_time)} --> {seconds_to_time(current_time)}\n')
        file.write(f"{message['author']['name']}: {message['message']}\n")
        file.write('\n')  # Add an empty line for spacing

    last_time = message['time_in_seconds']
    counter += 1


baseEngine.handleEmittedEvent not fully synced

When more than one goroutine is calling socket.Self for the same socket, the page might not be rendered correctly.
You can observe that in this sample: https://github.com/philippseith/gorx/blob/b1370a626b3b5018d700e55979a4cede273ace6a/ext/rxlive_test/rxlive_test.go#L18
I can see that baseEngine.handleSelf is synchronized, but not the rendering part in handleEmittedEvent.
When I synchronize the call to socket.Self (see here: https://github.com/philippseith/gorx/blob/957e88ddca118650351122095f80f681c5ea920f/ext/rxlive/viewmodel.go#L40), the page renders correctly.
IMHO all the steps executed in handleEmittedEvent for the socket should be synchronized, but just putting a mutex into this method would sync over all sockets, not only this one. Should there be a mutex in baseSocket.Self ?

Hi, how do I observe? I run the test, go to http://localhost:8086. What should I be looking for?

Ah figured it out

If correctly synced, you will see one or two rows with Id AAA or BBB, where AAA sometimes vanishes. If sync does not work, lines will double, you might get more than one BBB row often, sometimes even more than one AAA row.
Mind that you need to use the https://github.com/philippseith/gorx/tree/feature/unsynchronized_sample branch, as I saw that  my implementation of CombineLatest might cause reentrant calls to Observer[T].Next() and I fixed that in master meanwhile.

Ok, try v0.15.6

Nice catch, good luck with your project. Rxjs is an interesting beast


Can't Delete Collections

Need to add delete button to show edit and show views.

Resolved with #705


Better debug print of stdin

Even if stdout is not fully utf-8, we print all ASCII printable
characters verbatim and hex-escapes \xAB for non-printable
For inputs larger than 8KiB we only print first and last 2KiB and
omit bytes in the middle.

This helps reading error messages on larger stdin data (in my use case assert_cmd dumped ~440Mb of data on the terminal).

Finally got my hands on this. Probably fixed all the feedback.

Finally got my hands on this. Probably fixed all the feedback.

Sorry for being slow in responding and thanks for doing this!
I'm working through my crates, doing updates.  I can rush out a release of this or wait until I can include my other updates.  let me know what your timeline is for this.

Sorry for being slow in responding and thanks for doing this!
I'm working through my crates, doing updates.  I can rush out a release of this or wait until I can include my other updates.  let me know what your timeline is for this.

I'm fine using git version for now. Would appreciate if you ping me once it's released.

I'm fine using git version for now. Would appreciate if you ping me once it's released.

Just released a new version!


[Frontend] Adding uglify task to frontend when .js files are published to dist

Also making sure that minified version of the files published by bower to dist folder

#46


Error running bootstrap.pl on OSX 10.11.1 (MoarVM)

$ perl6 -c bootstrap.pl
===SORRY!===
Function 'CompUnit::Repository' needs parens to avoid gobbling block
at /Users/jbest2/code/panda/bootstrap.pl:29
------>         }⏏
Missing block (apparently claimed by 'CompUnit::Repository')
at /code/panda/bootstrap.pl:30
I added parens around the if statement on line 29 to work around the issue.

Current panda is no longer compatible with a rakudo older than a couple of days. Unfortunately due to the upcoming Perl 6.c release, we don't have enough time to go through a deprecation cycle for these changes.
Sorry for the inconvenience!


[Topology] upgrade pf react-core package

Fixes https://github.com/janus-idp/backstage-plugins/issues/286

@debsmita1 The build is failing.
plugins/topology/dev/index.tsx:31:7 - error TS2420: Class 'MockKubernetesClient' incorrectly implements interface 'KubernetesApi'.
  Property 'proxy' is missing in type 'MockKubernetesClient' but required in type 'KubernetesApi'.

Please fix the issue and update the PR.


#27

@aasthajain12 can you provide information about the code or which issue are you referring?

I am referring the issue for Queue Implementation(#27). I have written the
code for Implementation of queue using single stack in C++.
On Mon, 26 Oct 2020, 22:58 Atul Sharma<EMAIL_ADDRESS>wrote:

@aasthajain12 https://github.com/aasthajain12 can you provide
information about the code or which issue are you referring?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Iltwats/hacktoberfest/pull/179#issuecomment-716706114,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKPUUWACEB5FJK4UOB63L6LSMWWVHANCNFSM4S7VX4JA
.

Could you please label my pr with  Hacktoberfest 2020 and hacktoberfest-accepted labels? In my profile it is showing contributed to invalis repository.

PR was reverted because it was put in the wrong directory.


Can some one help me out compiling this for Moto G4 Plus?

@hugoromo see this https://forum.xda-developers.com/moto-g4-plus/development/rom-kali-nethunter-3-0-t3663523
For any queries you can post in that thread only


Improve the execution of Elyra on insecured cluster

In the workbenches with Elyra installed, while using the notebook on an insecure cluster, users have to set an
env var PIPELINES_SSL_CA_CERTS  to have the pipeline execution submission successfully sent to the pipeline server.
We would like to have a more convenient solution than asking users to set env var.
suggestions:

Use a solution make the service trust cluster certs.
https://odh-io.slack.com/archives/C035J0KA6UB/p1689970211014419?thread_ts=1689965765.550419&cid=C035J0KA6UB

With Notebook-Controller, https://github.com/opendatahub-io/kubeflow/pull/270
All certs are now mount on to workbenches in ODH,
which all these interaction to take place without hinderance.
Closing this issue.


Create repository for SantuaryWatch ioos.us website development and documentation

Repository name
sanctuarywatch-webpage
Repository purpose
In collaboration with NMS we are working on a WordPress migration of the https://sanctuarywatch.ioos.us/ website. Currently, we do not have a landing page GitHub repository for the sanctuarywatch landing page. As sanctuary staff will be developing the WordPress plug-ins and associated documentation for the website, this would be a good place to store the plug-ins and write the deployment documentation.
Following some ideas we've had to have a repository in the ioos organization for each of our ioos.us site, I am requesting we add one for this effort.
Other ioos.us websites/landing pages that have GitHub repositories:

https://github.com/ioos/atn-dac
https://github.com/ioos/gliders
https://github.com/ioos/hfradar-dac-landing
https://github.com/ioos/comt-landing
https://github.com/ioos/ioos_code_lab
https://github.com/ioos/osmc
https://github.com/ioos/ocean-acidification

cc: @superjai, @nguyandy
P.S. The repository name was a shot in the dark. If we have some common naming convention for these ioos.us sites, lets follow that.
Repository description
WordPress documentation and plug-ins for the National Marine Santuaries web-enabled Condition Reporting (WebCR) website sanctuarywatch.ioos.us.
Repository type
Public
Repository license
Creative Commons Zero v1.0 Universal
Repository options

[X] Add a README.md file
[X] Add a .gitignore file

Specify a .gitignore file type
No response

@mwengren any major issues if we create a repo for this effort? If not, do you have a preference for a naming schema?

@MathewBiddle we don't seem to really have a naming convention, but despite that, we don't really use -webpage as a suffix in any other cases.  The closest I can see to a pattern is that we just name the repo to match the DNS subdomain (gliders -> gliders.ioos.us, osmc -> osmc.ioos.us).
So I guess my preference is to just call it 'sanctuarywatch' if that's the planned domain name.
Hopefully, a few of the static 'landing page' repos can be retired in the future if we can find a path forward to consolidate the RPS-managed ioos.us sites (comt-landing, gliders, hfradar-dac-landing, osmc, ocean-acidification) under the ioos-us repo, perhaps.

sanctuarywatch is the planned domain name, so let's go with that. I'll close this issue once the repo is created.

https://github.com/ioos/sanctuarywatch


Investigate Nano Server Host Support

Now that Server 2016 is GA, it would be nice to be able to run Lability on a Nano Server host. This won't be easy, but I thought I'd put the idea out there.

Is there any need at this moment in time?
Is it too soon, too immature?
The most obvious use case that I can see is nested Hyper-V/Nano Server and integration testing - do you need an AD for integration testing anyone?!

Given that Nano is now longer supported as a Hyper-V host from RS3, I suppose this issue isn' t applicable any more. A different question might be whether support for Windows Server 2016 Core should be added or if it makes no sense spending time on it since this module is primarily intended for running on Windows 10 for lab purposes (not production systems running Server).

@janegilring Yeah - it's a shame that MS have dropped the infrastructure roles for Nano server. Maybe they'll reintroduce them in the not-too-distant future?! Until then, I'll close the issue.

@janegilring And just to reiterate, it should work on Server Core as that's where we use it for dev and in our CI pipeline..


#16 Fix - typo in codecov URL

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage     66.66%   66.66%
  Complexity        2        2
=========================================
  Files             1        1
  Lines             6        6
=========================================
  Hits              4        4
  Misses            2        2

Continue to review full report at Codecov.

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


integration-tests: add integration test for local-gadget

Add support for running integration test for local-gadget against multiple container runtimes.
Fixes #738
Idea
The idea is based on following changes:

Make command.go API public to share between different test environments.
Use minikube to spin-up different container runtime environments.
Hook the test in CI to run on each commit.
Add couple of test to ensure the setup works properly.

@blanquicet thanks for all the reviews. I have updated the PR to handle all the feedback!

@eiffel-fl Thanks for the review and the feedback. I will keep a note of this for future PRs and will split into multiple commits. Sorry for the pain :laughing: I have updated the PR to address some of the comments you had!

@mauriciovasquezbernal thanks for the review. I have adjusted the nits. Good point regrading introducing k8s directory, Currently  I see we can have docker or k8s based test but wanted to have deeper look into it. Would it be okay if handle this in follow up PR ?

@mqasimsarfraz , yes, that's fine to handle in a different PR.


fix 1503: odo project delete 2nd iteration

What is the purpose of this change? What does it change?
After the deletion of project it's not switching the active project,
this patch will modify the active project in runtime after confirming the existence of given project name.
Was the change discussed in an issue?
#1503
How to test changes?
Validate if the scenario mentioned in this issue description works properly
https://github.com/openshift/odo/issues/1503

closing this PR as we need a different approach for this fix


chore: add filter params changed tracking event

The type of this PR is: chore

This PR resolves CX-808
Description

Add tracking event for filtering

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

#squashongreen


[Bug] Unable to launch EMR with AppStream enabled

Describe the bug
Launching an EMR workspace within an environment configured with isAppStreamEnabled: true fails with the following error:
Error provisioning environment DevWorkspace01. Reason: AmazonCloudFormationException Parameter 'AccessFromCIDRBlock' must contain at least 9 characters

To Reproduce
Steps to reproduce the behavior:

From Studies
Select a study
Launch an EMR workspace
See error

Expected behavior
The workspace should launch successfully. Running the same test in an environment with isAppStreamEnabled: false produces a working workspace.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

v4.3.1

Additional context
My environment configuration:
awsRegion: eu-west-1
awsProfile: sw
solutionName: sw-d
envType: dev
isAppStreamEnabled: true

I have attempted to fix this myself with each fix leading to a different issue. Here is what I did:

Removed the MinLength requirement from AccessFromCIDRBlock. This fixed the immediate problem and started a cluster launching which later failed.
Attached the security group ${SolutionNamespace}-WorkspaceSG. This fixed the issue of being unable to access the KMS VPC endpoint to retrieve the key.
Allow S3 access to the Amazon Linux and EMR repos as detailed here https://docs.aws.amazon.com/emr/latest/ManagementGuide/private-subnet-iampolicy.html
Remove all the BootstrapActions to rule those out.

You can see these changes here: https://github.com/gazzyt/service-workbench-on-aws/commit/20992c4a6fcf17f47f60974c835440ff2b64ecc3
These left me with a cluster which launches, spends about 40 minutes trying to bootstrap and then terminates. The error is:
ElasticMapReduce Cluster with Id j-1UPL8DQFW92A4, is in state STARTING and did not stabilize due to the following reason: {}

I was able to ssh onto the master node to review the logs but nothing stood out as an obvious cause.
I am now wondering if this is even possible although this page seems to imply it is but I can't make it work: https://docs.aws.amazon.com/emr/latest/ManagementGuide/emr-clusters-in-a-vpc.html#emr-vpc-private-subnet
Everything else seems to say you need a NAT instance which would seem to defeat the purpose of a secure research environment.
A couple of other experiments I tried:

Use the console to launch an EMR cluster in the SWB private subnet. The console offered to create a NAT instance as it detected no route to the internet. I declined this and ended up with a self-terminating cluster as before.
Use EMR version 5.30 as this version seems to have a change related to private subnets but it made no difference. I did see port 9443 added to the security groups though as mentioned here https://docs.aws.amazon.com/emr/latest/ManagementGuide/emr-man-sec-groups.html.

I am out of ideas at this point but maybe this info will help others progress.

Hello @gazzyt!
EMR is not currently supported in our TRE SWB deployments. This was a design choice, and we will be updating the documentation to clearly reflect that this is not a bug. Thank you for bringing this to our attention. Would you like me to convert this issue into a Feature Request to have EMR supported in TRE/AppStream environments in the future?

Hi @maghirardelli
Thanks for the reply. I think for the moment it is enough to know that this is not supported.


Add ocis_wopi deployment backports from stable-5.0 into the next patch release

There are some smaller changes that have been created for ocis_wopi in master and backported to stable-5.0. Bigger changes are still in master only and will not get backported.
For the easy entry docs, we need the backports to be part of the next release like 5.0.5
Else, our examples lack important things we depend on.
Maybe to be added as task for the next sprint?
@micbar @ScharfViktor @tbsbdr

@kulmann do we want to have it in the new 5.0.5 patch release?

@kulmann do we want to have it in the new 5.0.5 patch release?

No, any next 5.0.x is fine for what Martin proposes (that's also what I read from Martins comment). Just go ahead with 5.0.5 with what we have in stable-5.0.

git diff for the wopi shows only
diff --git a/deployments/examples/ocis_wopi/README.md b/deployments/examples/ocis_wopi/README.md
index adfbf2a754..9a346f6505 100644
--- a/deployments/examples/ocis_wopi/README.md
+++ b/deployments/examples/ocis_wopi/README.md
@@ -2,5 +2,9 @@
 document this deployment example in: docs/ocis/deployment/ocis_wopi.md
 ---

-Please refer to [our documentation](https://owncloud.dev/ocis/deployment/ocis_wopi/)
-for instructions on how to deploy this scenario.
+# Infinite Scale WOPI Deployment Example
+
+This deployment example is documented in the [developer documentation](https://owncloud.dev/ocis/deployment/ocis_wopi/).
+See the link for more details and instructions on how to deploy this scenario.
+
+Also see the [Admin Documentation](https://doc.owncloud.com/ocis/latest/index.html) for administrative and more configuration details.
diff --git a/deployments/examples/ocis_wopi/config/ocis/csp.yaml b/deployments/examples/ocis_wopi/config/ocis/csp.yaml

PR in https://github.com/owncloud/ocis/pull/9353


update Thorin

Philosophical Changes

Created some interface types so that components that have similar properties share the same language. The biggest one is type WithColor which adds a "color" and "colorScheme" property. This is paired with utility functions for getting the right color in a theme. I took the function approach because when we move to Linaria we will need to translate property values into css module names and I saw this as a step towards that transition.
For inputs, I changed the layout so that most of the related sub components are at the same level as the input. For example, the input and prefix component are on the same level where before the input had a wrapper div around it. Inputs are always the first child of the containing component which allows the use of sibling selectors to update the styling. The primary reason for this is because we have styling for when a component is showing the placeholder. Since we are using react-hook-form, the value of the input is not reactive, so I rely on css input:placeholder-shown to control the styling when in placeholder state.
For Banner, I'm testing out using a type for icons instead of using ReactNode. Would like feedback if you think this is better.
Reverted much of the changes that were made for yalc hot reloading support. It was causing problems when building. I think part of the problem was by linking direction to the js file instead of the folder, it removed access to the package.json file.

Changes list

When publishing a branch that is not main, will release a version with tag @next.
Copy README.md to thorin package when published
Updated README.md
Added Banner component
Update Button. Simplified the component to focus on the actual use cases that we have. Added the counter component as well.
Updated Field. Updated the styling and placement. Simplified on actual use cases we have.
Added RecordItem
Update Typography: Uses the design language that Dom uses in Figma.
Updated Checkbox: Removed the toggle option
Added Currency Toggle: Is a modified checkbox
Updated Dropdown: The styling of the menu.
Update Input: Removed the number type and unit. Simplified the component to our use cases. Built in the clear button.
Updated Profile: Removed the caret icon to the right. This is not the final form that profile will have since Dom has not finalized the styling but it is getting it closer to where he wants it, which was primarily to get the size of the button to be the same as a large button so it wouldn't cause a resizing shift when loading the page.
Updated Select. Updated the styling and simplified component. Switched to using the Field ids. This component currently needs some work where it doesn't properly change the placeholder state when using react-hook-form. The problem is that you can't update the value of the input natively using the inputRef since ref in react-hook-form is not a normal ref but a functional ref so calling inputRef.current return undefined.
Updated Dialog. Moved step counter to footer. Added an alert option to Dialog heading.
For other components, it was mostly just updating the styling

imo colorScheme implies a different theme or set of colours, which is not what the prop sets. just a nit though

I'm open to a better term, looked through a list but didn't find a better one.

quite a few legibility issues on various components from 9px text, extraSmall font size should be 12px

I switched extraSmall to 12px font. Nice catch.

idk if i broke it but Hue isn't correctly exported in the built types

Hue wasn't being exported. Added it to exported types.

banner:

icon should be centred on desktop
the ETH icon shouldn't be the default for info type, is just a placeholder in designs
maybe add a subtle hover state?

Banner is a different component from Helper.
I'll ask Dom about using the Eth icon as a default as well as the hover state. I believe he was working on hover colors already.
What do you think of using a type specific for svgs for the icons?

recorditem:

should be a hover state

Asking Dom about hover states.

input:

when clicking the icon, the input isn't focused

Nice. I turned off pointer-events in icons so the click goes through to the input.

should probs keep text cursor across entire input for consistency

Done.

clear icon animation doesn't look great imo, should probs just fade in faster without any scaling

I decrease transition duration to 0.1s and increased the starting scale to 0.8. I thought it looked better but I'm willing to go with no scaling as well.

i think clearable should be false by default

After thinking about I agree.

readOnly inputs should not be clearable

Added a readOnly state where clear/action button does not show and cursor remains default.

profile:

why was the chevron removed?

Dom has an update to the profile in the Navigation  where the chevron is removed. Also the chevron svg has been resized so it works for inputs and doesn't look right in Profile. Could be done using resizing though. I didn't follow Dom's design completely because he hasn't finalized the updated profile component.

text is too far apart, i think adding margin-top: -2px to the top element looks better (this probably wasn't because of your changes, just comparing to figma)

I agree that it does look a little funky. I added a negative 0.125rem margin in between.  I'll ask Dom for clarification as well.

select:

options should have cursor: pointer across all content, instead of just in padding

Fixed. Nice catch.

add gap/padding between options

Added 2px padding in between.

I'm open to a better term, looked through a list but didn't find a better one.

maybe colorStyle? could also remove the prop entirely and instead allow the color prop to take a color instead of just a hue (e.g. color="blue", colorScheme="primary" -> color="bluePrimary")

Hue wasn't being exported. Added it to exported types.

this didn't fix the issue, created #88 to fix

Banner is a different component from Helper.

meant vertically centred, not horizontally. sorry

What do you think of using a type specific for svgs for the icons?

works well

I decrease transition duration to 0.1s and increased the starting scale to 0.8. I thought it looked better but I'm willing to go with no scaling as well.

looks a lot better

Dom has an update to the profile in the Navigation where the chevron is removed.

that's fine then

I'm open to a better term, looked through a list but didn't find a better one.

maybe colorStyle? could also remove the prop entirely and instead allow the color prop to take a color instead of just a hue (e.g. color="blue", colorScheme="primary" -> color="bluePrimary")

I broke it into two interfaces, withColor which is for things like Typography where there is just one color needed and withColorStyle for things like buttons that have a group of colors.

Banner is a different component from Helper.

meant vertically centred, not horizontally. sorry

You got a sharp eye. Fixed.


Floating point warning set during syrk call

Calling syrk on arrays larger then (32, 32), seems to cause the floating point error flag FE_INVALID to be set (as checked by fetestexcept as far as I can see).
This was tested on OpenBLAS ~master~ develop branch, unfortunately, I only have a numpy reproducer (EDIT: Do to changes in numpy, this will currently only work on master – NumPy 1.17+):
i = 33
arr = np.zeros((i, i))
res = np.matmul(arr, arr.T)  # will call syrk since it is transpose

# RuntimeWarning: invalid value encoutnered in matmul

For i < 33 the warning is not given. The actual values or transpose flag seems irrelevant.
I am not quite sure whether this is an error in OpenBLAS, but if it is not we will have to silence floating point warnings here I think.

Could you rerun your check on OpenBLAS develop branch or the recent 0.3.4 release please ? master is unfortunately stale at 0.2.20 (the last release made by xianyi 18 months ago)

@martin-frbg oops, sorry, I use develop since I was on it. Am too used to having master as the development branch from numpy.

To be more exact:
    version info: OpenBLAS 0.3.5.dev NO_AFFINITY HASWELL MAX_THREADS=4

Who or what is "c" in your example - should this be "arr" or are there lines missing in your code snippet ?

Sorry :(, that is what you get when you edit code only in one place after pasting...

Not immediately reproducible on my Haswell (i8700K), perhaps python or glibc version plays a role ?

Hmmm, possibly, have a Haswell i7-7500U. Python is 3.7.1, glibc is 2.28 unless I am confused.
But if you can't easily reproduce, that probably explains why nobody else ever complained about it. Maybe my setup is somehow weird.

Hmm, I can reproduce it on my Debian with python3:
OpenBLAS version info: NO_LAPACKE DYNAMIC_ARCH NO_AFFINITY Haswell MAX_THREADS=64

(not sure which OpenBLAS version right now, it is debian testing). With Python 3.6.7.
Man... I forgot a major information. To reproduce it with numpy, you need to use numpy master :(.

Is it possible that the relevant fetestexcept call was added only in current numpy master ? (I am not too keen on installing numpy master right now, but https://github.com/ContinuumIO/anaconda-issues/issues/9129#issuecomment-380284038 could be describing a similar situation where an originally benign signal triggers the numpy warning )

Yes, it definitely is new on master. Sorry, forgot.to mention that crucial.thing. Thr old onw did not  even call syrk in my example.
On Wednesday, 19 December 2018, Martin Kroeker wrote:

Is it possible that the relevant fetestexcept call was added only in current numpy master ? (I am not too keen on installing numpy master right now, but https://github.com/ContinuumIO/anaconda-issues/issues/9129#issuecomment-380284038 could be describing a similar situation where an originally benign signal triggers the numpy warning )
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/xianyi/OpenBLAS/issues/1924#issuecomment-44869457

Dows it magically work with 64?

@brada4 yes, indeed it does!

I see old fish in driver/*/level3_syrk_threaded.c
BLASLONG range[MAX_CPU_NUMBER + 100];
Can you check if any avoids the issue:
Using USE_SIMPLE_THREADED_LEVEL=1
Using OPENBLAS_NUM_THREADS=1

That does look like some less than creative coding by grandmaster Goto, but I do not immediately see its connection to the FE_INVALID ?

One-off INT calculation or something....

OK, so setting threads to 1 indeed fixes it. Not quite sure about the USE_SIMPLE_THREADED_LEVEL3=1, I think I screwed up the recompile the first try, so will update the comment in a bit.

it is in interface/syrk.c the strange poorly understood work dividers (but easy to take out if it is implicated)

well there is a ... / (double) nthreads that would add up as the cause of this if nthreads can reach 0 when it is an exact multiple or so.

with (double) it could reach 0+theta > 0 ....

Err, nthreads is not expected to become zero. If the explanation was that simple, OpenBLAS would be blowing up all over the place. FWIW valgrind does not find anything wrong with either the wonderful
+100 or a more common MAX_CPU_NUMBER+1, so this may have been papering over a long-gone bug.

Yeah, that made no sense. That sqrt might be a suspect, but I will tune down the guesses until they are more educated :)

Seems I cannot even get it to raise FE_INVALID when I add the feenableexcept(FE_INVALID) to OpenBLAS itself.

Sorry, I have problems myself right now with reproducing it after recompiling OpenBLAS a few time. I am wondering if it had to do with a slightly older version or not. Will post more later probably, right now I am confused :/.

Yeah, maybe I did not clean up correctly, and it disappeared in the not too far back past?

It is sort of OK if it was just some mistake building, but try again few times just in case.

@brada4 it still happens with the OpenBLAS shipped with debian testing as far as I can see (I did not mess with OpenBLAS myself on that machine). Just a bit sorry if I am wasting everyone's time because it is not in master anymore.

So I may have forgotten a git clean -xdf and/or make clean. Cannot reproduce right now, not even with open mp enabled.
Since the last time I had compiled OpenBLAS develop was when we discussed the thread safety issues (probably about ~2 months ago), it sounds like it got fixed in the mean time.
Will close this for now. Will also quickly try with version from 2 months ago and post the result. If anyone is curious (and I can repro with an older version) I can bisect just for the kicks.

Please quickly snap package version as installed on your system
dpkg --list | grep openblas
And report to debian too so they do not release broken package to masses.
Can you try options from dpkg-buildflags one by one or half by half?
Like quickest way:
make CC='cc --fsomething' FC='gfortran -fsomething'

The debian info:
ii  libopenblas-base:amd64                        0.3.3+ds-1                            amd64        Optimized BLAS (linear algebra) library (shared library)
ii  libopenblas-dev:amd64                         0.3.3+ds-1                            amd64        Optimized BLAS (linear algebra) library (development files)

I never used many flags in my own (arch) compilations, just enabling OMP/SIMPLE threaded level 3 or THREADS=4, that is about it.

OK got it, it seems at least with master it only shows up with USE_SIMPLE_THREADED_LEVEL3=1 and my ingenious patch:
diff --git a/driver/level3/syrk_thread.c b/driver/level3/syrk_thread.c
index 5f40853d..e6c54bf5 100644
--- a/driver/level3/syrk_thread.c
+++ b/driver/level3/syrk_thread.c
@@ -133,6 +133,7 @@ int CNAME(int mode, blas_arg_t *arg, BLASLONG *range_m, BLASLONG *range_n, int (
     }

   } else {
+    double sqrt_di;

     nf = (double)(arg -> n - n_from);
     nt = (double)(arg -> n - n_to);
@@ -149,6 +150,14 @@ int CNAME(int mode, blas_arg_t *arg, BLASLONG *range_m, BLASLONG *range_n, int (
       if (nthreads - num_cpu > 1) {

        di = (double)(arg -> n - i);
+       if (di * di + dnum < 0) {
+           sqrt_di = 0;
+           printf("WHAHAHAHAHA bad SQRT %f, %f!\n", di, dnum);
+       }
+       else {
+           sqrt_di = sqrt(di * di + dnum);
+       }
+       printf("SQRT DI: %f\n", sqrt_di);
        width = ((BLASLONG)((-sqrt(di * di + dnum) + di) + mask)/(mask+1)) * (mask+1);

        if ((width <= 0) || (width > n_to - i)) width = n_to - i;

Actually prints out:
In [1]: i = 33; arr = np.zeros((i, i)); res = np.matmul(arr.T, arr)
SQRT DI: 28.578838
WHAHAHAHAHA bad SQRT 1.000000, -272.250000!
SQRT DI: 0.000000
runtests.py:1: RuntimeWarning: invalid value encountered in matmul
  #!/usr/bin/env python

Which means that:
    nf = (double)(arg -> n - n_from);
    nt = (double)(arg -> n - n_to);

    dnum = (nt * nt - nf * nf) / (double)nthreads;

that seems strange, but I guess it might be just harmless, because in the end everything is already finished or so and the only left over from the whole thing is the error flag.

Thanks for tracking this down. So in the end this is the (main or only) reason for the subsequent "if width <=0" check, also inherited from the original GotoBLAS. Ugly but (mostly) harmless, I guess I will patch this along the lines of your "ingenious patch" over the weekend.

@brada4 I guess I had not upgraded in a while on that computer. The 3.4 version actually shipped with up to date Debian does not show the issue.
I do think it is a pretty mild bug:
1.It is not a regressions (and likely a bit more hidden by recent changes)
2. Numpy master test suit trips over it, but in release mode it probably only prints a warning, in a path that is rarely used in actual code.
3. Few people will use an older debian openblas with numpy master (most of those would end up using Anaconda/pip which installs a custom OpenBLAS). When debian upgrades numpy, they may also have to upgrade OpenBLAS, but that seems about it.
4. No wrong results are returned as far as I can see, only a wrong floating point error flag.

It is solved. There was a genuine mistake in 0.3.3 packaging, solved as in first line of 3.4 release notes, but broken version managed to land about everywhere.
3/ there is no problem whatsoever for older than v0.3.3
3debian/ current release with 0.2.20 is not affected (as opposed to mid-testing mistakes)
3pip/ yes, they contain non-releases of openblas, some frankensteins from openblas and atlas, that cannot be solved here, but keep crashing and filling bug trackers.

That part of the code did not change in 0.3.4 (and I notice you reproduced the problem with current develop already), perhaps the python message is generated somewhat randomly depending on which thread came in last.

@martin-frbg debian 0.3.3 did not patch out USE_TLS from Makefile.rule
If it happens with other versions without this primary suspect it is necessary to investigate deeper

@brada4 I am not convinced that USE_TLS has any bearing on this syrk oddity (and note he has been cross-checking with current develop from the start). You may be thinking of the haskell-opencv segfault bug discussed yesterday ?

My guess is that checking if dnum is negative and setting width explicitly would band-aid everything that I see away, there is a similar branch in the same file, where I think negative numbers are not possible. The only thing I wonder is if something went wrong elsewhere to produce negative numbers in the first place.
But I do not have a clear enough idea about what this code does to judge the correctness.

@martin-frbg only common thing is that it fails with 0.3.3, difference is that this does not go fixed.
The problem with +100 is that +1 might have been insufficient long ago.

Thanks @martin-frbg!


Update freecurrencyapi.net -> api.currencyapi.com

Fixes #193. Creating as a draft because I haven't actually tested this yet

The new API format was more different than I initially thought; updated to handle that. Also added error logging for when fetching rates fails. This should be good to go now.


run script with server environment

I want to run script with server environment， How do？？？

Please explain in a more detailed way what you want and submit a new issue.


[Circler Dependencies]: import cycle not allowed

We need to find a way to avoid the circle dependencies problem.
Current state: make bot
$ make bot
go run examples/ding-dong-bot.go
import cycle not allowed
package main
	imports github.com/wechaty/go-wechaty/wechaty
	imports github.com/wechaty/go-wechaty/wechaty/user
	imports github.com/wechaty/go-wechaty/wechaty
make: *** [Makefile:40: bot] Error 1

@huan I think we can close this issues.

@dingdayu great!
Could you explain a little bit about how did we solve this problem for now?

@huan
This is a mistake in the initial project structure. It is only a temporary problem and will not be encountered repeatedly.


Docker container doesn't sync data-configuration.yml on start of OpenTAXII image

Hello guys,
I pulled latest docker image from eclecticiq/opentaxii:latest (0107b7d07fd2). Started the container with default configuration file as present in git repo under examples folder. But the problem is, taxii-server isn't being configured with the settings present data-configuration.yml. One needs to manually sync the data inside the container. According to Dockerfile and entrypoint.sh script, it should be configured while booting up.
I ran container with following commands
❯ pwd
/home/vnktsh/projects/OpenTAXII/examples

❯ ls
create-fixtures.sh  data-configuration.yml  docker-compose.yml  hooks.py  Makefile  stix

❯ docker run --rm -v $PWD:/input:ro -p 9000:9000 --name "taxii" eclecticiq/opentaxii:latest
Using config:
---

domain: "localhost:9000"

persistence_api:
  class: opentaxii.persistence.sqldb.SQLDatabaseAPI
  parameters:
    db_connection: sqlite:////data/data.db
    create_tables: yes

auth_api:
  class: opentaxii.auth.sqldb.SQLDatabaseAPI
  parameters:
    db_connection: sqlite:////data/auth.db
    create_tables: yes
    secret: notVerySecret

logging:
  opentaxii: info
  root: info

hooks:
{"api": "opentaxii.persistence.sqldb.SQLDatabaseAPI", "event": "api.initialized", "logger": "opentaxii.utils", "level": "info", "timestamp": "2019-01-29T02:59:22.807415Z"}
{"api": "opentaxii.auth.sqldb.SQLDatabaseAPI", "event": "api.initialized", "logger": "opentaxii.utils", "level": "info", "timestamp": "2019-01-29T02:59:22.863277Z"}
{"event": "opentaxii.server_configured", "logger": "opentaxii.server", "level": "info", "timestamp": "2019-01-29T02:59:22.864190Z"}
{"event": "Starting gunicorn 19.9.0", "logger": "gunicorn.error", "level": "info", "timestamp": "2019-01-29T02:59:22.916027Z"}
{"event": "Listening at: http://<IP_ADDRESS>:9000 (1)", "logger": "gunicorn.error", "level": "info", "timestamp": "2019-01-29T02:59:22.917902Z"}
{"event": "Using worker: sync", "logger": "gunicorn.error", "level": "info", "timestamp": "2019-01-29T02:59:22.918415Z"}
{"event": "Booting worker with pid: 16", "logger": "gunicorn.error", "level": "info", "timestamp": "2019-01-29T02:59:22.924967Z"}
{"event": "Booting worker with pid: 17", "logger": "gunicorn.error", "level": "info", "timestamp": "2019-01-29T02:59:23.037134Z"}

Container starts without any errors.
However when I try to fetch discovery service using Cabby (taxii client) it throws 404 error
❯ taxii-discovery --host localhost --port 9000 --discovery services/discovery-a
2019-01-28 21:05:10,709 INFO: Sending Discovery_Request to http://localhost:9000/services/discovery-a
2019-01-28 21:05:10,785 ERROR: HTTP Error: status code 404

If we go into the running container I find there exists config file in input directory, and the script entrypoint.sh successfully executed opentaxii-sync-data command but the database has no entries in its tables.
❯ docker exec -it taxii "/bin/bash"
root@93fc3fccbe96:/# ls
bin   data  entrypoint.sh  home   lib	 media	opentaxii.yml  proc  run   srv	tmp  var
boot  dev   etc		   input  lib64  mnt	opt	       root  sbin  sys	usr  venv
root@93fc3fccbe96:/# ls /input/
create-fixtures.sh  data-configuration.yml  docker-compose.yml  hooks.py  Makefile  stix
root@93fc3fccbe96:/# ls /tmp/
auth.db  data.db  opentaxii.yml
root@93fc3fccbe96:/# sqlite3 /tmp/data.db
SQLite version 3.16.2 2017-01-06 16:32:41
Enter ".help" for usage hints.
sqlite> .tables
collection_to_content_block  result_sets
content_blocks               service_to_collection
data_collections             services
inbox_messages               subscriptions
sqlite> select * from services;
sqlite> select * from data_collections;

If you see above "select * from services;" return empty table.
What resolved the issue is manually syncing the data-configuration.yml inside container.
root@93fc3fccbe96:/# [ -f /input/data-configuration.yml ] && OPENTAXII_CONFIG=$OPENTAXII_CONFIG opentaxii-sync-data -f /input/data-configuration.yml
2019-01-29T03:16:12.131047Z [opentaxii.utils] info: api.initialized {api=opentaxii.persistence.sqldb.SQLDatabaseAPI}
2019-01-29T03:16:12.174448Z [opentaxii.utils] info: api.initialized {api=opentaxii.auth.sqldb.SQLDatabaseAPI}
2019-01-29T03:16:12.175265Z [opentaxii.server] info: opentaxii.server_configured
2019-01-29T03:16:12.210779Z [anyconfig] info: Loading: /input/data-configuration.yml
2019-01-29T03:16:12.262724Z [opentaxii.utils] info: sync_services.updated {id=inbox_a}
2019-01-29T03:16:12.270181Z [opentaxii.utils] info: sync_services.updated {id=inbox_b}
2019-01-29T03:16:12.275061Z [opentaxii.utils] info: sync_services.updated {id=discovery_a}
2019-01-29T03:16:12.278720Z [opentaxii.utils] info: sync_services.updated {id=collection_management_a}
2019-01-29T03:16:12.283607Z [opentaxii.utils] info: sync_services.updated {id=poll_a}
2019-01-29T03:16:12.284015Z [opentaxii.utils] info: sync_services.stats {updated=5, created=0, deleted=0}
2019-01-29T03:16:12.303758Z [opentaxii.utils] info: sync_collections.updated {name=collection-a, id=1}
2019-01-29T03:16:12.314930Z [opentaxii.utils] info: sync_collections.updated {name=collection-b, id=2}
2019-01-29T03:16:12.324109Z [opentaxii.utils] info: sync_collections.updated {name=collection-c, id=3}
2019-01-29T03:16:12.340491Z [opentaxii.utils] info: sync_collections.updated {name=col-not-available, id=4}
2019-01-29T03:16:12.340828Z [opentaxii.utils] info: sync_collections.stats {updated=4, created=0, disabled=0, deleted=0}
2019-01-29T03:16:12.349108Z [opentaxii.auth.manager] warning: update_account.unknown_collection {collection=collection-xyz}
2019-01-29T03:16:12.421779Z [opentaxii.utils] info: sync_accounts.updated {username=test}
2019-01-29T03:16:12.476292Z [opentaxii.utils] info: sync_accounts.updated {username=admin}
2019-01-29T03:16:12.476604Z [opentaxii.utils] info: sync_accounts.stats {updated=2, created=0, deleted=0}

Now the entries are successfully created in sqlite DB.
root@93fc3fccbe96:/# sqlite3 /tmp/data.db
SQLite version 3.16.2 2017-01-06 16:32:41
Enter ".help" for usage hints.
sqlite> select * from services;
2019-01-29 03:16:00.393596|inbox_a|inbox|{"address": "/services/inbox-a", "description": "Custom Inbox Service Description A", "destination_collection_required": true, "accept_all_content": true, "authentication_required": false, "protocol_bindings": ["urn:taxii.mitre.org:protocol:http:1.0"]}|2019-01-29 03:16:00.393602
2019-01-29 03:16:00.403303|inbox_b|inbox|{"address": "/services/inbox-b", "description": "Custom Inbox Service Description B", "destination_collection_required": true, "accept_all_content": false, "authentication_required": true, "supported_content": ["urn:stix.mitre.org:xml:1.1.1", "urn:custom.example.com:json:0.0.1"], "protocol_bindings": ["urn:taxii.mitre.org:protocol:http:1.0"]}|2019-01-29 03:16:00.403308
2019-01-29 03:16:00.410919|discovery_a|discovery|{"address": "/services/discovery-a", "description": "Custom Discovery Service description", "advertised_services": ["inbox_a", "inbox_b", "discovery_a", "collection_management_a", "poll_a"], "protocol_bindings": ["urn:taxii.mitre.org:protocol:http:1.0", "urn:taxii.mitre.org:protocol:https:1.0"]}|2019-01-29 03:16:00.410923
2019-01-29 03:16:00.418572|collection_management_a|collection_management|{"address": "/services/collection-management-a", "description": "Custom Collection Management Service description", "protocol_bindings": ["urn:taxii.mitre.org:protocol:http:1.0", "urn:taxii.mitre.org:protocol:https:1.0"]}|2019-01-29 03:16:00.418580
2019-01-29 03:16:00.425883|poll_a|poll|{"address": "/services/poll-a", "description": "Custom Poll Service description", "subscription_required": false, "max_result_count": 100, "max_result_size": 10, "authentication_required": true, "protocol_bindings": ["urn:taxii.mitre.org:protocol:http:1.0"]}|2019-01-29 03:16:00.425888
sqlite>
sqlite>

And I can fetch from taxii-client too after this.
But expected behaviour should be, the docker container while booting should sync data-configuration.yml file if it finds in \input folder. Instead it requires a manual sync inside the container.
Steps to reproduce:

Pull latest docker image from eclecticiq/opentaxii:latest
cd to 'examples' folder under local git repo. Mount the folder into input directory while starting the opentaxii container.
Go inside the container, check sqlite database tables. They return empty results.

Docker hub provides a URL you can POST to to trigger a new build for your project, you can simply add that as a webhook in Github's repository settings.