.v file extension is Verilog

Hi, this looks like a cool language.  However using .v is not a wise choice, as this has been used for the Verilog language for the past 25 years.
https://fileinfo.com/extension/v

It also happens to be the same extension used by Coq https://github.com/coq/coq
I was looking to add this into https://github.com/boyter/scc/ and realized the extension is shared. Perhaps something such as vlang? Not sure what to do here. You can of course use v if you choose but it is going to be an issue for existing tools.

If vlang is too long we could also consider vl. It's not indexed by fileinfo and doesn't appear to be in soc.

Why wasn't Verilog .vl like how Prolog is .pl? 🤔
.vl could be an option though.

I much prefer the simplicity of .v. I doubt many people will encounter conflicts between the two file types. Just my two cents.

Agree with @xenophilicibex
At this point conflicts with names and extensions are unavoidable. Modern tools are smart to distinguish between different languages.
Almost every extension you search for is used by multiple programs/languages.

For example, Github uses Linguist (https://github.com/github/linguist) to do its syntax highlighting.
Here is Linguist's programming language dictionary: https://github.com/github/linguist/blob/master/lib/linguist/languages.yml#L5157
Verilog is assigned the .v extension.
This is why your example source code on Github has partial highlighting:  https://github.com/vlang-io/V/blob/master/examples/generic_repository.v
I'd suggest a different extension (.vl or anything else).

Linguist supports languages with the same extension.

On Feb 24, 2019, at 20:28, Payton Quackenbush<EMAIL_ADDRESS>wrote:
For example, Github uses Linguist (https://github.com/github/linguist https://github.com/github/linguist) to do its syntax highlighting.
Here is Linguist's programming language dictionary: https://github.com/github/linguist/blob/master/lib/linguist/languages.yml#L5157 https://github.com/github/linguist/blob/master/lib/linguist/languages.yml#L5157
Verilog is assigned the .v extension.
This is why your example source code on Github has partial highlighting: https://github.com/vlang-io/V/blob/master/examples/generic_repository.v https://github.com/vlang-io/V/blob/master/examples/generic_repository.v
I'd suggest a different extension (.vl or anything else).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/vlang-io/V/issues/6#issuecomment-466808283, or mute the thread https://github.com/notifications/unsubscribe-auth/AAp_fDdjJShIwutPvuomP_O1uUwAh-08ks5vQufjgaJpZM4bNyIy.

Modern tools are smart to distinguish between different languages.

Just curious - how is this typically done? simple heuristics?

From a discussion I read on Atom’s forums they do things like read the first line, and usually that covers 90% of the cases.

On Feb 25, 2019, at 17:37, Caleb Winston<EMAIL_ADDRESS>wrote:
Modern tools are smart to distinguish between different languages.
Just curious - how is this typically done? simple heuristics?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/vlang-io/V/issues/6#issuecomment-467082034, or mute the thread https://github.com/notifications/unsubscribe-auth/AAp_fFSihP8AGuAfmHv4qXizWvq-vpzAks5vRBFmgaJpZM4bNyIy.

From a discussion I read on Atom’s forums they do things like read the first line, and usually that covers 90% of the cases.

On Feb 25, 2019, at 17:37, Caleb Winston<EMAIL_ADDRESS>wrote:
Modern tools are smart to distinguish between different languages.
Just curious - how is this typically done? simple heuristics?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/vlang-io/V/issues/6#issuecomment-467082034, or mute the thread https://github.com/notifications/unsubscribe-auth/AAp_fFSihP8AGuAfmHv4qXizWvq-vpzAks5vRBFmgaJpZM4bNyIy.

By the way @boyter what an amazing tool! I've been using cloc forever, and scc is 30 times faster!

@medvednikov Thank you. A lot of work went into making it fast and accurate. The next release should have support for V as well once I finish off a few things, possibly in a day or two.

@medvednikov Have included V support into scc. I based it on what I have seen in the examples, so it may need some tweaks but should report the files correctly now I hope.

Nice, thanks :)

On Feb 27, 2019, at 23:38, Ben Boyter<EMAIL_ADDRESS>wrote:
@medvednikov https://github.com/medvednikov Have included V support into scc. I based it on what I have seen in the examples, so it may need some tweaks but should report the files correctly now I hope.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/vlang-io/V/issues/6#issuecomment-468059507, or mute the thread https://github.com/notifications/unsubscribe-auth/AAp_fFLtlWa34Pp40kNuIi_fhkG4rqRjks5vRwj5gaJpZM4bNyIy.

The name and the extension are going to stay the same.

I know this is closed; my two cents would be to choose a trigram that is infrequently used. .vvc comes to mind as a good candidate; vvc.org is even available (and vvc.com is just a parked page), and as far as I can tell there is no current use of the file extension.  Going with a four-letter name, especially one that is not pronounceable, would move even further out of the collision range.
Having a mostly unique name is attractive because it makes searching easier and less ambiguous.

vvc.org http://vvc.org/ is not available, .v is not a frequently used extension.

On 6 Mar 2019, at 15:49, Andrew Laucius<EMAIL_ADDRESS>wrote:
I know this is closed; my two cents would be to choose a trigram that is infrequently used. .vvc comes to mind as a good candidate; vvc.org is even available (and vvc.com is just a parked page), and as far as I can tell there is no current use of the file extension. Going with a four-letter name, especially one that is not pronounceable, would move even further out of the collision range.
Having a mostly unique name is attractive because it makes searching easier and less ambiguous.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/vlang-io/V/issues/6#issuecomment-470129816, or mute the thread https://github.com/notifications/unsubscribe-auth/AAp_fKhxZEDOMuvPOXZv4IrTXAfQ3R4mks5vT9WEgaJpZM4bNyIy.

I think "Vee" is a, uhm, pretty good name. We can keep the pronunciation, and the file extension ".vee" is unique :)

I was about to raise an issue with the same topic. Using .v as an extension for this language is just inviting confusion with Verilog. There are plenty of other options here but for some reason it's been decided to use the same extension as a well established language just because it's simple.
I suggest changing it while it still can be changed.

Couldn't there be 2 file extensions - like .v and .vlang or something like that?
That way people could generally use .v if they prefer the extension's simplicity or .vlang if whatever software they're using can only recognize .v as Verilog but does have .vlang as the extension for V.

@calebwin I like that. Just like .cpp/.cc in C++.


[Bug]: Cannot "unselect" Google Analytics (Legacy UA) connector

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I have a website that had the Google Analytics (Legacy UA)  connector enabled. I wanted to use the Google Tag Manager connector. I set that connection up with no problem. But then I cannot delete the Google Analytics (Legacy UA)  connector - a JavaScript error results.
Steps to reproduce?

Started with an older website that has been upgraded to DNN v9.13.1.
Successfully added the Google Tag Manager connector.
Failed at deleting the Google Analytics (Legacy UA) connector.
Found a JavaScript error (see below).
Now it looks like both the Google Analytics (Legacy UA) and Google Tag Manager connectors are enabled.

Current Behavior
JavaScript error after clicking "Delete Connection".
jquery.js?cdv=216:2  Uncaught Error: Syntax error, unrecognized expression: #connector-Google Analytics (Legacy UA)
at I.error (jquery.js?cdv=216:2:11769)
at Y (jquery.js?cdv=216:2:17234)
at re (jquery.js?cdv=216:2:20008)
at Function.I [as find] (jquery.js?cdv=216:2:7520)
at ce.fn.init.find (jquery.js?cdv=216:2:21991)
at new ce.fn.init (jquery.js?cdv=216:2:22483)
at ce (jquery.js?cdv=216:2:1077)
at saveConnection (Connectors.js?cdv=216:305:34)
at Object.save (Connectors.js?cdv=216:132:21)
at Object.action (connector.js?cdv=216:107:30)
Expected Behavior
The delete button should delete the legacy connector when clicked on.
Relevant log output
No response
Anything else?
No response
Affected Versions
9.13.0 (latest release)
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@skarpik thanks for logging this issue.
@bdukes I took a quick look and see this is related to that code section that we had some trouble with a while back (not able to save when tracking for admins).  I didn't have time to dig too deep, but thought I would mention you just in case you have time before me.


Member properties

Add member properties (get/set) to code model and formatters.

Member properties are now added to the code model as variables with stereotype Get,Set, or GetSet.
Need to implement #12 to be able to see the stereotypes.


Adding checkstyle and other code analysis tools to our PR.

I think that it would be nice if we have more metrics on PR (including threshold that would flag the PR as red if exceeded). Checkstyle is one of these metrics.
I was looking at the checkstyle github page, just to see
if they checkstyle their own code for PR and I think they do, but I didn't invest more time on it.
I also saw that they integrate with some other github friendly sites (and free to open source projects) such as:
https://www.versioneye.com/ - for dependency out-of-date notifications
https://www.codacy.com/ - for code analysis

I run codacy on our gcloud-java, you should be able to see the report here so we can decide whether we find it useful.

My first impression is that the UI is nice and some of the suggestions are helpful, but I do think there's a lot of irrelevant errors to sift through in order to get the useful tidbits.

I really like the UI and I think it would be great if we can get the PR summary (https://www.codacy.com/app/mziccard/gcloud-java/pullRequests) as part of the set of PR sets.
I have the feeling that some rules can be disabled (there is a notion of active/non-active) and customize (though I hope the latter does not mean a modified docker).
Also, some of the errors (such as "Enforce the use of explicit scoping"), can be resolved by
code annotation (@VisibleForTesting in this case).

I really like the UI and I think it would be great if we can get the PR summary (https://www.codacy.com/app/mziccard/gcloud-java/pullRequests) as part of the set of PR tests.

I should have just enabled this feature for PRs against the master branch. Should we enable the tool to run on all branches? (I think we should).
Also, out of curiosity, are you able to modify codacy settings?

I should have just enabled this feature for PRs against the master branch. Should we enable the tool to run on all branches? (I think we should).

I also think we should.

Also, out of curiosity, are you able to modify codacy settings?
Didn't try. but looks like it is doable - https://docs.codacy.com/v1.5/docs/code-patterns#2-configure-your-project-patterns no?

Closing since both versioneye and codacy PRs are merged.

Reopening since checkstyle isn't yet supported by codacy.

I am closing this as Codacy now supports Checkstyle


[ci skip] adding user @borchero

Hi! This is the friendly automated conda-forge-webservice.
I've added user @borchero as instructed in #5.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #5

Welcome, you should be good to go @borchero!

Thanks @maresb!


Fatal Exception: java.lang.NoClassDefFoundError linc.com.amplituda.Amplituda

Hi team,
I see some trouble at Firebase
Caused by java.lang.UnsatisfiedLinkError: dlopen failed: library "libamplituda-native-lib.so" not found
at java.lang.Runtime.loadLibrary0(Runtime.java:1082)
at java.lang.Runtime.loadLibrary0(Runtime.java:1003)
at java.lang.System.loadLibrary(System.java:1661)
at linc.com.amplituda.Amplituda.(Amplituda.java:612)
It looks like there is some issue in native code. Can you please help with this?

Hello @BuiDucHai-dev I am also getting the same error. Have you fixed it?


WADs and Solution Defintions and Solution Designs created for Github publication

WADs and Solution Defintions and Solution Designs created for Github publication.
Because of the work done for future split in Solution Definition and Solution Design I have used those create the external versions

Changed some files to anonymize content

Need to have the license.txt and readme.md files on each workload directory. There is also an md file that isn't rendering.
Can you please have a look? Thanks.


Reorganised Kafka Connect navigation

What changed, and why it matters
Reorganised Kafka Connect Navigation per technology
Fixes #1160

I'm not at all sure about this re-organisation. Could we possibly spend some time with all the available options in a list and look at how to categorise them? My feeling is that I might want to source and sink on the same tech sometimes. Also that it's fine to have them all in one big bucket with a better landing page (coming after the theme upgrade and panels plugin upgrade) rather than relying on the nav for discoverability.

Thanks for the comment!
The organisation so far has been by dividing only source and sink connectors and I believe from a reader/user point of view it makes sense (either you want to pull data or push).
This reorganisation was aimed at providing a better grouping of the connectors by technology. The alternative would be, as of now, to just keep the long lists, possibly simplifying the titles in the _toc file, and then manage a better navigation in the landing page.
WDYT?

I'm closing this but let's keep the conversation going on the issue because I agree we need to make changes, I just don't think we've nailed the information architecture aspect of it.


Can add its Google Notebooks to THUDM/glm-4-9b-chat, thanks

Oh sadly ChatGLM has a custom architecture :(


[Deployment] Ignore the error when encoding string to utf-8

Coverage remained the same at 51.762% when pulling b82f2e09618ffd3e7219cb372ae979465a71bfc9 on yuye/encode-issue-fix into e93539c24c0f08ea4f6fa72c3fdb26fd1fa81add on master.

Coverage remained the same at 51.762% when pulling b82f2e09618ffd3e7219cb372ae979465a71bfc9 on yuye/encode-issue-fix into e93539c24c0f08ea4f6fa72c3fdb26fd1fa81add on master.

Coverage remained the same at 51.762% when pulling b82f2e09618ffd3e7219cb372ae979465a71bfc9 on yuye/encode-issue-fix into e93539c24c0f08ea4f6fa72c3fdb26fd1fa81add on master.

Close this PR. Because we have to propose a complete design to solve all these issues.


Logging

Initialize specification for logging

I've added some pieces of information to precise the specification following the last working group.
We still need to determine what could be a warn log level information and if we need it.

Can this spec specify we only display Milli's logs in DEBUG level, not in INFO?

@Kerollmops Doesn't Milli have any useful information to give in INFO level? If the answer is yes, I'll specify it :)

@gmourier We only log certain things when we are indexing documents, things about how much time it took to merge some database values... 🛩

@gmourier We only log in INFO certain things when we are indexing documents, things about how much time it took to merge some database values... 🛩

Thanks @Kerollmops! Indeed, this is not something we want to display at the INFO level to stay consistent with transplant INFO level. @curquiza I'm updating the specification to mention that we only display Milli's logs from DEBUG level!


Explore ClusterShell

Should look at using Clush instead of (or in addition to) Paramiko. @fintler claims it's quite fast for SSH execution. I don't know much about it, but it seems worth investigating a bit. Primary goal would be to speed up deployments.

Might just be a wrapper on Paramiko. Maybe we can learn something from it, though.


updated schema.xml to build a text_all copy field

Closes #129
Signed-off-by: Mark Cohen<EMAIL_ADDRESS>

this fixes the text_all issue when querying solr, but the queries in quepid and the store produce different results. I think i'm starting to pull on a thread that I shouldn't be pulling on...


How does it work?

Installing the mod and the extention doesn't do anything.


Removed control-label class from docs code.

As far as I can tell there's no CSS associated with it. And it's not mentioned in docs.

This was a mistake on my part, so I have to apologize for that. I can't enforce adding it any more due to our versioning rules, but I can add to the CSS for this class, which I will be doing shortly. I'll leave this open as a reminder for now.

Made the change for this in 2.0.1-wip.  Sorry for any inconvenience this causes folks!

Did the change you mentioned force the usage of .control-label starting with 2.0.1?
I've got code that currently written like this:

  Input Label:

    <input type="text" />

Any chance you could "put it back"? It really should work with both. Having .control-label as an option is great, but dropping support for regular old label tags is a huge step backwards...especially for a minor point release.
Would you mind reconsidering?

Agreed with BigBlueHat, forcing a class when the markup is sufficient without it seems like a bizarre choice and will break a lot of existing apps for a minor release.


Allow HelmDeploy task to use deployment environment as Service Connection

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Feature
Enter Task Name: HelmDeploy
Environment

Server - Azure Pipelines
Agent - Hosted

Issue Description
HelmDeploy doesn't support using the current deployment environment to connect to AKS.
In a deployment job, Kubectl can use the environment to connect to AKS and deploy resources. HelmDeploy task still requires connectionType, Subscription etc to be specified. Running without connection details specified gives me:
Error: Input required: kubernetesCluster
Can we get cluster connection information integrated into environment like the Kubectl task?

I'd love to see this integrated as well. It's blocking our adoption of this. Is there any workaround for this?

Yeah when you create the environment, it actually creates service connections per namespace. So you can put the namespace service connection directly into this task. It works, but is more manual as it has a random number attached to the name. This issue was created because it isn't consistent with kubectl task.

@chshrikh could you take a look at this issue? This seems to be an issue with the replacement logic of the service endpoint substitution in the deployment job.

@ds-ms what is inuput type for HelmDeploy ? For Kubctl we have logic to replace the input of type serviceconnection


using useLocalStorage to setValue during unMount does not work

What is the current behavior?
I was calling the setValue for localStorage inside useUnmount, but it does not store the value. It gets value, or set the value on mount or when button was clicked, but does not work during useUnmount. What I want is store store only the value when the component is removed.
Steps to reproduce it and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have extra dependencies other than react-use. Paste the link to your JSFiddle or CodeSandbox example below:
import React, { useContext as useContextCore } from 'react'
import { useList, useLocalStorage, useUnmount, useMount } from 'react-use'
import { StorageKeys } from '../../../config/StorageKeys'

const context = React.createContext()
const CoreProvider = context.Provider

export const Provider = ({ children }) => {

  const [logs, storeLogs] = useLocalStorage(StorageKeys.processLogs, [])
  const [list, { insertAt, clear, set }] = useList([])

  useUnmount(() => {
    storeLogs(list)
  })

  useMount(() => {
    set(logs)
    console.log(logs, 'logs mounted here')
  })

  return <CoreProvider value={{ list, insertAt, clear }}>{children}</CoreProvider>
}

What is the expected behavior?
It should update the localStorage on unMount

Hi all! @react-hookz/web, the new library by one of react-use's former maintainers (background here and here) has a new implementation of useLocalStorage called use useLocalStorageValue that should solve this issue
For those interested, there's an official migration guide for migrating from react-use to @react-hookz/web.
Hope this helps!


-Wunsequenced error on basic_fuzzer

/Users/yagiz/Developer/url-parser/tests/basic_fuzzer.cpp:37:59: warning: multiple unsequenced modifications to 'counter' [-Wunsequenced]
                  copy.insert(copy.begin()+(211311*counter++)%copy.size(), char(counter++*777));
                                                          ^                            ~~
1 warning generated.
[55/55] Linking CXX executable benchmarks/bench

Build finished

Yeah. It is unconsequential, but we might want to fix that. Just do
                  copy.insert(copy.begin()+(211311*counter)%copy.size(), char((counter+1)*777));
                  counter += 2;

This is already fixed. Closing.


Can't Delete Account

Describe the bug
Cannot delete account, throws 500
To Reproduce
Steps to reproduce the behavior:

Have lambda events (possibly by setting up GitHub Apps)
Make sure everything is unpublished and account is ready for deletion
Try deleting account
See 500 error

ERROR [2020-10-14 18:02:13,352] org.hibernate.engine.jdbc.spi.SqlExceptionHelper: ERROR: update or delete on table "enduser" violates foreign key constraint "fkj387wwrofm9u30t4mwiiwa835" on table "lambdaevent"→ Detail: Key (id)=(9001) is still referenced from table "lambdaevent".

14:02:13
INFO [2020-10-14 18:02:13,352] org.hibernate.engine.jdbc.batch.internal.AbstractBatchImpl: HHH000010: On release of batch it still contained JDBC statements

14:02:13
ERROR [2020-10-14 18:02:13,352] io.dockstore.webservice.jdbi.UserDAO: something happened with delete, probably cascades are broken

14:02:13
! org.postgresql.util.PSQLException: ERROR: update or delete on table "enduser" violates foreign key constraint "fkj387wwrofm9u30t4mwiiwa835" on table "lambdaevent"

14:02:13
! Detail: Key (id)=(9001) is still referenced from table "lambdaevent".

Expected behavior
The account should be deleted
Additional context

UI version: 2.6.1
Webservice version: 1.9.1
host location: dockstore.org

┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.11
┆Issue Number: DOCK-1613
┆Sprint: Sprint 54- Cat shark
┆Issue Type: Story

@garyluu
This has already been fixed? I can't reproduce this. I made a new account on dev, set up a github app workflow on it, and successfully deleted my account.

@garyluu Okay, I found the user this was originally affecting by going through the logs and locally was able to delete his account. Seems like Greg's PR here https://github.com/dockstore/dockstore/pull/3630 fixed this problem, but that wasn't deployed until 1.10.0


feat: refactor product image container

Current dependencies on/for this PR:

next

PR #213

PR #217

PR #222

PR #218

PR #219

PR #220

PR #214

PR #221


Take into account non-parallel crystal mesh for Crystal class

Objective:

On branch Issue202, the CrystalBragg class handles spherical crystals, but it does not handle non-parallelism.
The following functions should exist in a version taking into account non-paralelism:

tf.geom._comp_optics.calc_dthetapsiphi_from_lambpts()
tf.geom._comp_optics.CrystBragg_get_noute1e2_from_psitheta()
tf.geom._comp_optics.calc_xixj_from_braggphi()
tf.geom._comp_optics.calc_braggphi_from_xixjpts()

In order to understand the logic, you should retrace the code for the user's point of view, make sure all steps are clean and operational.

Necessary steps:

Propose a solution to store 2 angles and / or a 3-unit vector basis for the mesh in the CrystalBragg class.
The 2 angles refer to the 2 local (geometrical) vectors (e1, e2).
Check all methods in the class from section (l. 1000):

    # -----------------
    # methods for surface and contour sampling
    # -----------------

@adriendaros , the commands you're looking for are:
In [44]: det = dict(np.load('inputs_temp/det37_CTVD_incC4_New.npz', allow_pickle=True))

In [45]: cryst = tf.load('inputs_temp/TFG_CrystalBragg_ExpWEST_DgXICS_ArXVII_sh00000_Vers1.4.9-315-gc93fa7a3.npz')
Loaded from:
    /home/didier/Documents/Projects/tofu/inputs_temp/TFG_CrystalBragg_ExpWEST_DgXICS_ArXVII_sh00000_Vers1.4.9-315-gc93fa7a3.npz

In [46]: pts0, vect = cryst.get_rays_from_cryst(phi=np.pi, returnas='(pts, vect)')

In [47]: pts = pts0 + np.r_[1, 3,5, 7][None, :]*vect[:, :, 0]

In [48]: cryst.plot(pts=pts, det=det)


Star Wars app submited

Looks Good,
Thank you for your contribution,
leave a star and share with friends.


Window Size does not increase

Hi Everyone,
thank you for this great tool. Unfortunately, I have the problem that my editor window gets too small very quickly and new entities/tables align themselves in the inner area and thus overlay other tables. How can I change this?
Thank you
Tobias

I don't know if you're talking about this,
Canvas size is 2000 by default.
You can change it.

I don't know if you're talking about this,
Canvas size is 2000 by default.
You can change it.

Thank you so much. That helped. I did not see this as the text color was the same as the background:

But this was, what I was looking for 1-2 hours in the last couple of days, thank you!

Thank you so much. That helped. I did not see this as the text color was the same as the background:

But this was, what I was looking for 1-2 hours in the last couple of days, thank you!


Client sometimes fails successful transactions when offline

The new client has the goal to gracefully handle temporary failures like network interruptions, but I discovered at least one place where it can fail when going offline at the wrong time
Let's take the example of the state machine responsible for issuing new e-cash:
https://github.com/fedimint/fedimint/blob/ff3ad0658d4d16c2fa5ed30d91611f019374ab82/modules/fedimint-mint-client/src/output.rs#L162-L182
If anything, including network failure, goes wrong here the state machine goes into a failed state. What should happen instead is something similar to what the transaction submission state machine does: if the error is retryable wait for a bit and then try again.
https://github.com/fedimint/fedimint/blob/ff3ad0658d4d16c2fa5ed30d91611f019374ab82/fedimint-client/src/transaction/sm.rs#L196-L203
We need to check all state machines for similar mistakes to make sure everything works even with flaky internet connections.

2 things to do here:

[ ] Go through all the trigger functions and make sure we're retrying retryable errors
[ ] Make sure we've correctly identified which jsonrpsee errors are retryable


Add support for Beancount 3

Beancount 3.x has been released which replaces beancount.ingest with beangulp.

v1.0.0 supports Beancount 3.x.


Does webpack-node-externals work with babel-loader?

If I add babel-loader node_modules gets included.
Does webpack-node-externals work with babel-loader?

@ruifortes it should work seamlessly with babel-loader, of course. They affect different parts in Webpack. Can you share your config?

@ruifortes is it still happening?

Webpack does not excludes when babel module is included
var path = require('path')
var webpack = require('webpack')
var nodeExternals = require('webpack-node-externals');

var browserConfig = {
    // mode: 'development',
    target: 'node', // in order to ignore built-in modules like path, fs, etc.

    entry: {
        app: [path.resolve(__dirname, './index.js')],
    },
    output: {
        library: 'OyeFm',
        libraryTarget: 'umd',
        auxiliaryComment: 'Test Comment',
        path: path.resolve(__dirname, '../../../public/dist'),
        filename: 'oyefm.js',
    },
    module: {
        rules: [{
            test: /\.(js)$/,
            exclude: /node_modules/, // add this line
            use: 'babel-loader'
        }, ]
    },

    externals: [nodeExternals()], // in order to ignore all modules in node_modules folder

    plugins: [
        // new webpack.DefinePlugin({
        //     __isBrowser__: "true"
        // })
    ]
}
module.exports = browserConfig


Select test framework with new python.unitTest.framework preference

Environment data

VS Code version: 1.27.1
Extension version: 2018.8.0
OS and version: Linux Mint 17.3 Cinnamon 64-bit, Cinnamon 2.8.8
Relevant/affected Python packages and their versions: unittest, pytest, nosetests

Quoting Unit Testing Python in Visual Studio Code:

Unit testing in Python is disabled by default. To enable unit testing, set one and only one of the following settings to true: python.unitTest.unittestEnabled, python.unitTest.pyTestEnabled, and python.unitTest.nosetestsEnabled. [...]

It's important that you enable only a single test framework at a time. For this reason,
when you enable one framework also be sure to disable the others.

To enable unittest, for example, use the following settings:

"python.unitTest.unittestEnabled": true,
"python.unitTest.pyTestEnabled": false,
"python.unitTest.nosetestsEnabled": false,

To enable pyTest:

"python.unitTest.unittestEnabled": false,
"python.unitTest.pyTestEnabled": true,
"python.unitTest.nosetestsEnabled": false,

To enable Nose:

"python.unitTest.unittestEnabled": false,
"python.unitTest.pyTestEnabled": false,
"python.unitTest.nosetestsEnabled": true,

Instead of syncing values at three different places, it makes more sense to declare which to use only once, a la python.formatting.provider:
"python.unitTest.framework": "unittest" // Options: "unittest", "pytest", "nosetests"

I agree with the change, makes it easier for the settings GUI, i.e. a drop down to select the test runner

Part of #2453


Better error message for "session not found"

Description
Currently when issuing an identity, the session times out after 1 day. If the user doesn't open the app within that time, the identity fails with error "session not found". The session will be prolonged by Notabene (to 7 days; see https://github.com/Concordium/notabene/issues/15) but the error msg should be more informative as well.
@jens-concordium, could you please draft a better message to the user with sth like

session to issue an identity timed out, highly probably because you closed the app for too long
pls try again and return to the app more frequently
Pls have SH review the text. Thx!

Steps to Reproduce

issue an identity
after submitting, close app
return shortly after e.g. 15 minutes
close app
wait for more than 1 day before returning to app

Expected Result

identity issuance succeeded or informative error message

Actual Result

identity issuance failed with error "session not found" on id card

Versions

Concordium Wallet version 0.9

Same task for iOS: https://github.com/Concordium/concordium-reference-wallet-ios/issues/38


File case warnings only in 32 version of editor.

Godot version:

Godot master d8ce7aeddcaeea503d8d247c55a2cd83ccbbc2c4 (02/07/2019)
32 bit version of editor, compiled with VS2019
Python 2.7.13
Scons 2.5.1
OS/device including version:

Tested in win 7 32 y 64 bit.
Issue description:

Console throws a lot of warnings like this:

64 bit version of editor in same system not.
I don´t know if is related with the fact that 32 bit versión doesn´t compile with vs2015 and vs2017 any more: https://github.com/godotengine/godot/issues/28949
I make test and the issue is present on start and on focus gain (In focus gain only with plugins)... not when working inside godot.
It´s not present in new projects...
I can´t easy isolate because windows says there are 3002 files in project without git stuff... but happend in some templates of the library.
Main issue is that this warning is only in 32 bit version of editor
Steps to reproduce:
Open godot editor 32 bit in windows and beggin a project with "Godot Open Dialogue" template. This throws the error.

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?

Closing due to lack of update. Comment if  you can still reproduce in Godot 4.0.


[10.0-pos_disable_payment]: Crashing the POS after Odoo update

After Odoo update: https://github.com/odoo/odoo/commit/e14ab697727d87773dbefba11453b9edca79fc68
Now pos_disable_payment is crashing the POS (payment button won't show and products won't load).

Hi @gustavovalverde
Thank you for reporting this.
We've reproduced it and working on this issue currently. We will let you know once it is done.

Hi @gustavovalverde
Glad to inform you that we fixed the issue. Please take the latest updates of pos_disable_payment and check it.

Tested and working, thanks @ilmir-k


Remove out of date files and move information to global redirect

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs-ref-conceptual/authenticate-az-cli2.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/authorize-az-cli2.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/format-output-az-cli2.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/get-started-with-az-cli2.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/install-az-cli2.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/old-and-new-clis.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/query-az-cli2.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

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


DDI-428 update favicon

Amplitude Developer Docs PR
Update favicon to adjusts the blues used in our product
ref: https://amplitude.slack.com/archives/GDL32NVEH/p1681232628704989
Description
In Safari

In Chrome

Deadline
ASAP?
Change type

[ ] Doc bug fix. Fixes #[insert issue number]. Amplitude contributors include Jira issue number.
[ ] Doc update.
[ ] New documentation.
[x] Non-documentation related fix or update.

PR checklist:

[x] My documentation follows the style guidelines of this project.
[x] I previewed my documentation on a local server using mkdocs serve.
[x] Running mkdocs serve didn't generate any failures.
[x] I have performed a self-review of my own documentation.

@amplitude-dev-docs

looks like your IDE reformatted the mkdocs file, can you revert that?

I didn't realize the old favicon was using the old blue color. I'll double check with Meredith.

Notes from my discussion with Meredith from the design team.
tl;dr; This is what we want(*)

The main reason behind the change is to use our logo properly with correct color and design
The A alone isn’t technically correct as our logo
If there was a diff favicon that’s desired to be used for the dev center that’s certainly possible!
But if it reflects the logo it should use the sanctioned logo and not a diff color or inversion of it

@stephen-choi-amplitude do we still need this PR?

Closing this in preparation of migrating the repository and inactivity


Illegal path chars in sub_category

When taking safe copies of image, we put the image in a folder named according to the country, category, and sub_category.
One sub-category is "LEGACY / TRADITIONAL".   The forward sloping slash is illegal for a MS-Windows page, so the parse falls over.
This issue doesn't arise on MACOS, since the / is a legal path character.
When we read in the sub_category, we should strip the value of of any / characters.
Breaking change possibly introduced in this commit: https://github.com/DeepBlueCLtd/LegacyMan/commit/2686625a5722cf58f346a6a4d6ced3e48e8c8086

The following is from #148

So, really need a better fix. I suspect this will be through cleaning the sub-category earlier in the parsing session.

That's right. Cleaning it early would be the better/right fix.
I think it'll be better to replace forward slash ("/") with a hyphen ("-") which will result in a better looking sub-category name than when replaced with underscore ("_")

I think it'll be better to replace forward slash ("/") with a hyphen ("-") which will result in a better looking sub-category name than when replaced with underscore ("_")

Yes, agreed.
Maybe when creating these identifiers we should all non-character identifiers (including spaces) for hyphens.
So, we will have ´industrial-legacy´ instead of ´Industrial / Legacy´ . Removing spaces does seem like a step towards being more robust.


TV demo's empty search screen should show recommendations

Use the last entries from #piped_history (see #1981) to make three "recommendations" at the top. Label that section "Recommendations", label the existing section "Recents".

Approximation is here https://github.com/PolymerLabs/arcs/pull/2019

Stale, closing.


about pressSound and hoverSound

I tried both mp3 and ogg ,then do not work
"connectToServer":
{
"text" : "enter game",
"posX" : -280,
"posY" : -40,
"width" : 105,
"height" : 27,
"alignment" : "center",
"texture" : "custommainmenu:textures/button.png",
"pressSound" : "custommainmenu:textures/button.ogg",
"hoverSound" : "custommainmenu:textures/hover.ogg",
"action" :
{
"type" : "connectToServer",
"ip" : "<IP_ADDRESS>"
}
},
or sth wrong?

you need to change the locations to the custom location you put the files in. custommainmenu:tetures/ is the default location.

thanks

my pleasure


⚠ Spryr has degraded performance

In 38a661b, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4523 ms

Resolved: Spryr performance has improved in 900bc6b after 38 minutes.


create_public_redshift, does not attach igw if create_redshift_subnet_route_table = true

It appears that even in the latest v2.15 module, if you set create_database_subnet_route_table = true, setting enable_public_redshift = true does nothing expected. There is no igw attached to the redshift subnet routing table, which means that redshift is not publicly accessible when this is desired in some edge cases.
We probably need to mimic the create_database_internet_gateway_route behavior here and add create_redshift_internet_gateway for when a dedicated subnet routing table is preferable to sharing the public table.
The logic behavior issue seems to be around here, however I'd suggest we make this more uniform and in line with the db route table behavior for consistency :
https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/ee192c5d2baa50629304f1da1ada7816ecd39e62/main.tf#L934
https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/ee192c5d2baa50629304f1da1ada7816ecd39e62/main.tf#L944

I will add that var.redshift_subnets is 6, and > 0 in this test case

Please take a look at examples/complete and show the exact parameters you are providing, what you got, and what did you expect.

closing for now - please feel free to respond or open a new issue with more context and info so we can help troubleshoot


qgis-ltr: 3.34.4 -> 3.34.5

Description of changes
Update QGIS-LTR to the lates patch release.
List of changes: https://github.com/qgis/QGIS/compare/final-3_34_4^...final-3_34_5

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 298094 run on x86_64-linux 1

1 package built:

qgis-ltr


[13pt] Update acquire 3dep scripts to download Alaska

FIM will be asking south Alaska to the overall processing domain. It needs 3dep DEM's and a Alaska VRT as it will be processed in different parts of the FIM product. The DEM's required here need to be in a projection of EPSG:3338 (Alaska Albers).
There is a separate PR for the integration of processing for Alaska HUC's.  See 792
New HUCs required are: HUCs: 19020202, 19020301, 19020602, 19020203, 19020302, 19020800, 19020201, 19020104, 19020601, 19020401, 19020103, 19020402, 19020505, 19020503, 19020504, 19020102, 19020101, 19020502, 19020501

opps.. not merged yet.


Tables responsive?

Can the tables be response for mobile optimized?

@mmahalwy as of right now, we dont have a good way of creating a responsive table. This is something that is on our backlog though so hopefully we should come up with a good solution sooner rather than later.

@mmahalwy there is PR open to support this 😺! I would love it if you gave it a test run before I merge it in.


Update plan-for-teams-live-events.md

Added Brazil as another region where live event recording isn't supported.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@cichur PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[dvsim] Tweak mode find/merge to be simpler to understand

The existing code worked through a list of desired names and found
modes with these names. If the merge_modes flag was true then it also
merged each of the hits into the object supplied as the "mode"
argument. It turns out that this merging behaviour was actually only
used at one call-site.
This commit simplifies things. We now have a function called
find_mode_list which just does the filtering operation. The merging
part of things gets moved up to the call site.

Oops, I didn't realise but I think I opened two copies of this! A different copy got merged in #23371.


Skeleton_app creation

Hello I am a student who wants to add the skeleton app to my cFS.
I git clone skeleton app to the apps file path and populated the other necessary file paths needed for sample apps creation and there were still issues.
I implemented the changes noted in the issue logs such as the CCSDS_CommandPacket_t #4  changes and followed   #2 "Add integration instructions" 's changes for how they got it working however I tend to run into errors. Some examples include what is shown in the reply below.
Is there a guide for a noob like me to integrate the app onto cfs. NASA has such robust and great documentation but i still havent managed. Am i not placing the app name in the correct locations? Am i missing commands i should be running? full breakdown for a beginner plz.

/cFS/apps/skeleton_app/fsw/src/skeleton_app.c:34:
cFS/apps/skeleton_app/fsw/src/skeleton_app_msg.h:79:11: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘attribute’ before ‘SKELETON_HkTlm_t’
} OS_PACK SKELETON_HkTlm_t;
/cFS/apps/skeleton_app/fsw/src/skeleton_app.c:72:32: error: passing argument 1 of ‘CFE_SB_RcvMsg’ from incompatible pointer type [-Werror=incompatible-pointer-types]
status = CFE_SB_RcvMsg(&SKELETON_AppData.MsgPtr,
expected ‘CFE_SB_Buffer_t ** {aka union CFE_SB_Msg **}’ but argument is of type ‘CFE_MSG_Message_t ** {aka union CFE_MSG_Message **}’
CFE_Status_t CFE_SB_RcvMsg(CFE_SB_Buffer_t **BufPtr, CFE_SB_PipeId_t PipeId, int32 TimeOut);
(Shows this below despite me changing it to CFE_SB_CmdHdr_t)
‘CCSDS_CommandPacket_t’ undeclared (first use in this function); did you mean ‘CCSDS_SpacePacket_t’?
SKELETON_ReportHousekeeping((CCSDS_CommandPacket_t *)Msg);
upon compiling. When making changes the cFS runs without showing skeleton app, aborts, or shows what is below.
and the similar #2
1980-012-14:03:23.26064 ES Startup: Loading file: /cf/skeleton_app.so, APP: SKELETON_APP
1980-012-14:03:23.26075 ES Startup: SKELETON_APP loaded and created EVS Port1 42/1/CFE_SB 4: CreatePipeErr:OS_QueueCreate returned -11,app SKELETON_APP
1980-012-14:03:23.26083 Skeleton App: Error creating pipe, RC = 0xFFFFFFFFCA000005
1980-012-14:03:23.26084 Application SKELETON_APP called CFE_ES_ExitApp

@astrogeco I followed the following process (should be the same as in https://ntrs.nasa.gov/citations/20205011588)
From the cfs main directory
git submodule add https://github.com/nasa/skeleton_app apps/skeleton_app
git submodule update

I can see the apps/skeleton_app folder
make clean
make prep
make -j12
make install

At this stage I see so files for all apps but the skeleton (e.g)
./build/cpu1/default_cpu1/apps/sch_lab/sch_lab.so
Any idea what I am doing wrong ?


 I have created a range date-time picker like so:
<mat-form-field class="full-width">
  <label class="full-width">
    <input matInput
           placeholder="From - To"
           [owlDateTimeTrigger]="dt"
           [owlDateTime]="dt"
           [(ngModel)]="selectedDateRange"
           [selectMode]="'range'"
           (dateTimeChange)="dateChanged()"
           [rangeSeparator]="'~'"
           [min]="min">
    <owl-date-time [pickerMode]="'dialog'"#dt></owl-date-time>
  </label>
</mat-form-field>

In my .ts file the selectedDateRange and dateChanged() are defined as follows:
selectedDateRange: Date[];
dateChanged() {
    console.log("test selectedDateRange: " + this.selectedDateRange);
    console.log("test selectedDateRange[0]: " + this.selectedDateRange[0]);
    console.log("test selectedDateRange[1]: " + this.selectedDateRange[1]);
  }

However the console.log prints out some gibberish whenever I try to access an individual date:
test selectedDateRange: Mon Aug 13 2018 11:50:14 GMT+0100,Thu Aug 23 2018 11:50:14 GMT+0100
test selectedDateRange[0]:<PHONE_NUMBER>000
test selectedDateRange[1]:<PHONE_NUMBER>000
Interestingly enough, if i print selectedDateRange i get a long line containing both selected dates, as you can see.
Could you please give me some advice on how to access individual dates?
Example:
dateChanged() {
    console.log("test selectedDateRange[0]: " + this.selectedDateRange[0]);
  }

Output: Mon Aug 13 2018 11:50:14 GMT+0100
Thanks a lot,
LostInAngular

Found a solution for now:
dateChanged() {
    let containsBothDateTimes: string = this.selectedDateRange.toString();

    let start: string = "";
    let end: string = "";

    for (let i: number = 0; i < containsBothDateTimes.length / 2 - 1; i++) {
      start += containsBothDateTimes[i];
    }
    for (let i: number = Math.round(containsBothDateTimes.length / 2); i < containsBothDateTimes.length; i++) {
      end += containsBothDateTimes[i];
    }

    this.startDateTime = new Date(start);
    this.endDateTime = new Date(end);
    console.log("DateTimePicker: " + this.startDateTime);
    console.log("DateTimePicker: " + this.endDateTime);

    this.startDateChangedOut.emit(this.startDateTime);
    this.endDateChangedOut.emit(this.endDateTime);
  }


HDDS-4139. Update version number in upgrade tests

What changes were proposed in this pull request?
Update version number 0.6.0 to 1.0.0 in upgrade scripts and tests (content and file/directory names).
https://issues.apache.org/jira/browse/HDDS-4139
How was this patch tested?
Ran upgrade acceptance test locally.
https://github.com/adoroszlai/hadoop-ozone/runs/1022231789

Will re-create against master branch.


Revert changes shipped in 1.7.0

This is so we can restart from a known good state.

closed by #35


Rebase onto the current master

I use the multiplexing API in my code, and I wanted utf8 support too, so I gave this a shot.
This seems to work after a bit of testing, but you should probably test this yourself too - I don't really know what bugs to look for. I found that the rendering when cycling through autocomplete options is a bit buggy, but it was also like that before the rebase.
Also, there's no way you could merge that, you'd have to force pull/push.

@dzwdz thanks for the pull request. I have rebased the branch manually. You can see some differences between yours and mine. If you find any bugs, please let me know. Thanks for your help!


Reset tf.keras Session in Setup

tf.keras is becoming the standard in Tensorflow. In order to get reproducible results with keras, howerver, the tf.keras session must be reset after all random number generators are seeded
Details are described here in the Keras FAQ.

Thanks for the PR!
Another issue that that this library is almost certainly going to break once TF 2.0 is released. There's a hard dependency on session usage here, so it might make most sense to fork this and create an mltest2 for tensorflow 2.0.

@Thenerdstation Yeah I was going to ask the same thing about Tf 2.0. Have you tried running anything here with the Tf 2.0 Alpha yet? Maybe you could open up a general issue to discuss any problems/ideas about this?

No I haven't. And that's a good idea. Feel free to open the issue.


validate server validity before setting migrate status for a slot

See https://github.com/wandoulabs/codis/issues/173
@qiuyesuifeng

Need to fix it.


stuck on 2.1.1

on puppet agent run I get a
Notice: /Stage[main]/Curator/Package[elasticsearch-curator]/ensure: ensure changed '2.1.1' to '3.0.3'
but it doesn't actually perform the upgrade. curator --version gives me a 2.1.1
if I manually try to uninstall / reinstall through pip, I end up with 2.1.1.
I'm on python 2.6.6
puppet enterprise 3.7.2
centos 6.6
pip 1.3.1
ES 1.4.2
My manifest is not specifying version so by default, it's trying to upgrade to 3.0.3. I've had this work on other machines and I can't find what's different.
I have tried to have manifest do the uninstall, reinstall by toggling ensure => 'absent' -- and I still end up with 2.1.1.
I tried direct download of package -- If I manually surf to pip repo, download the .tgz and try to run setup.py I get a command import not found. I have setuptools installed. I've tried everything I know to try.
I'm running different puppet environments and made sure to upgrade curator module first. Stuck. Help.

ok finally got manual install to work but never understood why puppet couldn't handle via module.
I went to: https://pypi.python.org/pypi/elasticsearch-curator and and got link for tar.gz
.. ran a wget https://pypi.python.org/packages/source/e/elasticsearch-curator/elasticsearch-curator-3.0.3.tar.gz
then realized I didn't need to extract -- I'm new to pip and python --  and ran pip install elasticsearch-curator/elasticsearch-curator-3.0.3.tar.gz
walah, now on 3.0.3 and puppet now understands I'm on 3.0.3 -- no longer receive ensure message.

I believe this is something with the puppet pip provider rather than this module.  I think starting with the PL guys would be the best place to start.


Update cluster and topology operators

/kind enhancement
Update to cluster-operator 1.13.1
Update to topology-operator 1.6.0

The merge conflict and is good to go


Avast (malware suspicion) - triggered

Heya,
I just wanted to install this to try it if it would be suitable for generating xml files to feed to Traktor/Serato/Rekordbox from Foobar, but when installing, I get Avast to quarantine the .dll instantly with as suspected:
Win32:Evo-gen [Susp]
warning, when the .dll file of the .f2k-component gets extracted.
Potentially a false positive, but can be deterring/worrying to potential users.
Cheers.
c.

Thanks, I submitted a false positive to avast.
BTW, I scan it with virus total: https://www.virustotal.com/zh-tw/file/b5182bd3693cafb6a0e6e2bdaad58b1300d1ddd1a271783170464e60d772b5d5/analysis/1454907229/
The results ain't clean :( and I never heard of Rising antivirus.
If you're still bothered about the detection, (luckily this is opensource project) you can either 1. read the code yourself. 2. get someone you trust to verify the code. Honestly, I don't know how to avoid these kind of problems and I don't care much about it. :P


Match Validation: Adds end2end validation test on procted branch

https://datadoghq.atlassian.net/browse/SDS-479
Part 4 of match validation:
Add End2End test for match_validation with protected secrets

closed in favor of https://github.com/DataDog/dd-sensitive-data-scanner/pull/118


Named api resource list update

This pull request addresses the bug found in this reported issue: https://github.com/SirSkaro/pokeapi-reactor/issues/8
I have also experienced this error, and found the cause and am providing a solution.
Files Updated:
pom.xml --> Bumped up version to 1.0.7 for visibility on this issue
src/main/java/skaro/pokeapi/resource/NamedApiResourceList.java --> Changed type of previous from Boolean to String
Not Working:

Working:

RestController: http://localhost:4201/pokemon/list?limit=10&offset=10  --> Retrieve next 10 Pokemon

Looks good! Thank you for contributing and the detailed description!


Allow mapping of dyn-access

Codecov Report

Merging #53 (ae37017) into master (0426a36) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   89.63%   89.75%   +0.12%
==========================================
  Files          15       15
  Lines         984      986       +2
==========================================
+ Hits          882      885       +3
+ Misses        102      101       -1

Impacted Files
Coverage Δ

src/access.rs
93.82% <100.00%> (+0.15%)
:arrow_up:

src/strategy/hybrid.rs
95.45% <0.00%> (+1.51%)
:arrow_up:


D: add the mir algorithm and glas libraries

@RabsRincon
Depends on https://github.com/mattgodbolt/compiler-explorer-image/pull/31

I'll merge into the beta and let you know when the beta's ready for testing...

cool :)
(tested locally, seemed to work fine!)

https://d.godbolt.org/beta/ looks good to me! Will go into a site release once the dust settles in C++ library land


RecordedFuture Playbook consolidation

Required items, please complete
Change(s):

Removed playbooks from playbook dir and left message in readme

Reason for Change(s):

Playbook consoliadtion

Testing Completed:

yes

Checked that the validations are passing and have addressed any issues that are present:

yes

@rahul0216 I drafted this removal PR. I don't want to end up in conflicts on https://github.com/Azure/Azure-Sentinel/pull/6307. We will continue to move the rest of the playbooks that are not duplicated to top the solution folder. However, we have Defender ATP playbooks that will stay in the playbook folder.


[FIX]: Document chooser’s "Search" tab broken after a failed upload

Fixes #6126

Thanks for the updates! This is a bit of a tricky one to review now because the commits are mixing a lot of refactoring with code changes. Would it be possible to rearrange the commits so that there's one change per commit, with refactoring changes first?
I'd be able to rebase-merge rather than squash the commits if they were organised in this way

I think I make wrong the history of commits. Can you give me any advice to edit all of this?

The referenced issue was fixed in 4.0 - see;

https://github.com/wagtail/wagtail/issues/6126#issuecomment-1326392333
https://github.com/wagtail/wagtail/pull/8776


feat: find related applications for a single employee (hl-1354)

Description :sparkles:
Handler can now efficiently find employee's previous benefits.

Modify search code to use an existing application as base to find related other applications

List any Application objects with matching SSN (note that first_name and last_name can change)
List any ArchivalApplication objects by matching with best guess keys: first_name, last_name and year_of_birth (parsed from SSN)

Implement button to click in application view, seek for Employee section
Refactor and implement search view

🔍 View of single application's search button

📁  View of related applications when button is clicked

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3321.dev.hel.ninja :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3321.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr3321.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://helsinkilisa-ui-handler-pr3321.dev.hel.ninja :laughing::tada::tada::tada:


[Feature] Add Port ID Card

Description
It was  a mini ID card like structure , That have a About , Experience and contact section with Photograph
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[x] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

dear @Rakesh9100 please assign me for  this project

Do you want to add this as a component and in which category @codewithganeshhh

In Card Component @Rakesh9100

Assign me @Rakesh9100

What will be the name of your card @codewithganeshhh

Any update or should I close this issue @codewithganeshhh

PORT ID Card @Rakesh9100

Okay proceed @codewithganeshhh

@Rakesh9100 add level for Issue otherwise i don’t give point in gssoc leader board

No, it will not be any problem @codewithganeshhh


Update CODEOWNERS

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[ ] For content changes, I have completed the self-review checklist.

0416Diana0416:patch-1

The only way I could

 0416Diana0416:patch-1

0416Diana0416:patch-1


code examples in the docs built on python 3.5 lack the prompt fold thingy

A local build of the documentation with python 3.5 does not have a helpful toggle for folding the >>> prompts.

A similar build on python 2.7 has it.
The build is done via $ python runtests.py -g --shell -- -c 'make -C doc html-scipyorg' in a fresh virtualenv with python 3.5:
$ pip freeze
alabaster==0.7.10
appdirs==1.4.3
Babel==2.4.0
cycler==0.10.0
Cython==0.25.2
decorator==4.0.11
docutils==0.13.1
imagesize==0.7.1
ipython==6.0.0
ipython-genutils==0.2.0
jedi==0.10.2
Jinja2==2.9.6
MarkupSafe==1.0
matplotlib==2.0.1
nose==1.3.7
numpy==1.12.1
packaging==16.8
pexpect==4.2.1
pickleshare==0.7.4
prompt-toolkit==1.0.14
ptyprocess==0.5.1
Pygments==2.2.0
pyparsing==2.2.0
python-dateutil==2.6.0
pytz==2017.2
requests==2.13.0
simplegeneric==0.8.1
six==1.10.0
snowballstemmer==1.2.1
Sphinx==1.5.5
traitlets==4.3.2
wcwidth==0.1.7

The virtualenv is literally
$ mkvirtualenv -p `which python3.5` scipy_py35
$ pip install --upgrade pip
$ pip install numpy nose
$ pip install ipython
$ pip install cython

followed by cloning and building scipy.

check if the copybutton.js gets included and if not, why not

Hello,
I believe I'm running into this problem too. I'm commenting here because I found this bug, and also found a possible clue about this issue. While trying to get copybutton to work on my own projects, I found someone else who ran into frustration and said that using an older build of Sphinx (1.3.5 instead of 1.4.5) solved their problem.
I don't have an older of version of Sphinx installed at the moment to test it, but I would like to get this fixed as I want my own docs to use have this feature as well. I've confirmed that with my version of Sphinx, 1.4.8, the problem exists. Right now I've confirmed copybutton.js is linked to in the produced html in the more recent versions of Sphinx, so that at least is not the problem. I'm not sure when I'll work on this, but I can at least leave what I found here.
Sincerely,
Caleb


Generate duplicated "extensions" json entry when add custom extension to model's extensions.

Describe the issue
Generate duplicated "extensions" json entry when adding lights to model and adding custom extension to model's extensions.
This issue is only  in RapidJSON version. Because of detail::JsonAssign use Copy operation when RapidJSON defined.
see line 7939 in tiny_gltf.h.
when saving "KHR_lights_punctual" extension (see  tiny_gltf.h:7945: detail::JsonAddMember(ext_j, "KHR_lights_punctual", std::move(khr_lights_cmn));), the duplicated "extensions" is added.
To Reproduce
#define TINYGLTF_USE_RAPIDJSON
gltf_model.lights.emplace_back(...);  // add some lights
gltf_model.extensions.emplace("xxx", custom_extesion); // add custom extension
WriteGltfSceneToFile(gltm_model, ...) // save gltf scene

OS
Compiler, compiler version, compile options
Please attach minimal and reproducible .glTF file if you got an issue related to .glTF data

Expected behaviour
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Good catch!
Confirmed the issue by adding some lines to examples/build-gltf
    // lights
    tinygltf::Light dummyLight;
    dummyLight.name = "bora";
    m.lights.push_back(dummyLight);
    tinygltf::Value dummyVal(3.14);
    m.extensions.emplace("ext1", dummyVal);

  "extensions": {
    "ext1": 3.14
  },
  "extensions": {
    "ext1": 3.14,
    "KHR_lights_punctual": {
      "lights": [
        {
          "name": "bora",
          "intensity": 1.0,
          "type": ""
        }
      ]
    }
  },

It is a bit surprise for me that JSON sometimes allows duplicated key name and RapidJSON supports duplicated keys.
https://stackoverflow.com/questions/21832701/does-json-syntax-allow-duplicate-keys-in-an-object
https://stackoverflow.com/questions/35222230/cpp-rapidjson-resolve-key-conflicts-without-information-loss
Need to find a way to overwrite extensions key in RapidJSON API.

Add a fix to prevent duplicated extensions key generation in RapidJson backend.
The fix now should work well on your side. If you still encounter the issue, please let me know.


ESM module export not used by node

Think it a) needs "exports" and b) needs .mjs extension, that way older versions can continue to load the cjs version.
e.g.
"module": "dist/ok-computer.esm.mjs",
  "types": "dist/ok-computer.d.ts",
  "exports": {
    ".": {
      "import": "./dist/ok-computer.esm.mjs",
      "require": "./dist/ok-computer.esm.mjs",
      "types": "./dist/ok-computer.d.ts"
    }
  },

Not sure if it's ok to have "module" with an mjs path?

Additionally, current master has this
import * as parser from "ok-computer/dist/cjs/parser";

Instead of
import * as p from "ok-computer/parser";

This Issue is still open but I'm able to successfully load ok-computer from Node CommonJS require; from Node ES6 import...from; and from Deno.  What is the case that isn't supported?

All environments are supported, however the ESM build isn't being used by Node as I had expected. It instead was pulling in the CJS build. Not really a big deal but a bug nonetheless.


Update language support to zh

Docsy has renamed language 'zh' to 'zh-cn'. This is causing a huge problem for the Chinese localization.
This PR is a first step to fix this problem. Follow up PRs are needed to fix all the relative links.

/hold
Not sure if there is a better workaround for the language rename issue.
It might be a huge task for the 1.24 release.
However, if we don't do this now, after dev-1.24 going live, we will not be able to track changes to be synced for Chinese localization.
So ... I'd prefer to get this done NOW. As for 1.24 release, /content/zh won't be causing many conflicts.

/hold cancel

This PR will close #33002.

Oops. The update looked OK to me but I didn't try following links.
We could add a 302 redirect from /zh to /zh-cn/, and remove that redirect eventually (some years from now)?

We could add a 302 redirect from /zh to /zh-cn/, and remove that redirect eventually (some years from now)?

Yes. Adding a single 302 redirection would be nice.

/close
This solution is too disruptive.


Fix references to reference files

Fixes

Readme links to non-existent files

Proposed Changes

Fix links in Readme

Oops I forgot to change the links when I changed the project structure, thanks for your contribution @deiga  :)


[FLINK-17554] Allow to register release hooks for the user code class loader

cc @StephanEwen would this help for the FlinkKinesisConsumer to release its classes? It is basically what you mentioned in our sync.
What is the purpose of the change
TBF
Brief change log
TBF
Verifying this change
TBF
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@tweise could this help you with solving the class loading issues of the Kinesis consumer? The current problem is that the API does not prevent easily the reregistration of release hooks in case of a recovery. If you have a good idea how to solve this problem, then I'm all ears.

I'm closing this PR as I've opened a new one based on this work: https://github.com/apache/flink/pull/13355


JoErNanO: Watch vacationincentivesunlimited.com

JoErNanO requests the watch of the watch_keyword vacationincentivesunlimited\.com. See the MS search here and the Stack Exchange search here.

vacationincentivesunlimited\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.


Test: Core functions

Summary (Please recap what you changed or fixed)

Unit tests:

getAccount
getActivePublicKey
isConnected
disconnect

Checklist (If you won't pass this checklist please comment why)

[x] I removed all commented or unnecessary code.
[ ] Provide the screenshots due to UI changed or bug fixed
[x] My code has covered these test cases (please list down your tested cases):


Users - Landing Page - Unit Test Creation and Execution

UT Creation and Execution Task

Task completed on 29-Mar-2019.


Is there a way to ignore slideshows or Error in video extraction: status code 100004?

I believe I'm getting and error because I'm trying to extract data from a slideshow. I know I could utilize emulate_mobile=True, however this severely limits the number of videos I am able to pull data on. Traceback on the error is below:
Traceback (most recent call last):
  File "~/myproject.py", line 52, in <module>
    for video in challenge.videos:
  File "~/venv/lib/python3.10/site-packages/tiktokapipy/api.py", line 90, in __next__
    out = self.fetch(self._next_up)
  File "~/venv/lib/python3.10/site-packages/tiktokapipy/api.py", line 103, in fetch
    return self._api.video(video_link(self._light_models[idx].id))
  File "~/venv/lib/python3.10/site-packages/tiktokapipy/api.py", line 312, in video
    return self._extract_video_from_response(response, api_extras)
  File "~/venv/lib/python3.10/site-packages/tiktokapipy/api.py", line 450, in _extract_video_from_response
    raise TikTokAPIError(
tiktokapipy.TikTokAPIError: Error in video extraction: status code 100004

I can't seem to figure out where to put my except: statement since even calling the video in the for videos in challenge.videos causes the error, so I'm not sure how to move to the next iteration.
Any help is appreciated! Again, I apologize if there is a simple solution I am just missing.

I ended up instead pulling the "light" video information from challenge.videos._light_models, then iterating over that ids in that list with a try/except so that I could skip over the errors and continue to the next video. It's probably inefficient, but it works for now!

Just a heads up, I updated the code such that the videos._light_models is now videos.light_models. Also, this is kind of the best way to do it at the moment. There currently isn't a more efficient way to do it, you just have to step through the iterator yourself

Just a heads up, I updated the code such that the videos._light_models is now videos.light_models. Also, this is kind of the best way to do it at the moment. There currently isn't a more efficient way to do it, you just have to step through the iterator yourself

Small update - it looks like slideshows now work on desktop too, so you may not see this problem anymore.


Use outfits.openneo-assets.net for outfit images

outfits.openneo-assets.net is the more canonical host for outfit images, and we control the traffic differently sometimes.
Right now, we're working on moving outfit image pressure off of Vercel, and we're only really able to control the outfits.openneo-assets.net with that level of granularity right now.
So, switching to this host should help Neobot take a bit more pressure off our bill! 💖 and help clean up my logs to help confirm that we've actually switched users over!
I haven't tested this change, but it seems simple enough? But it might be smart to have this code build an outfit image URL once before shipping imo, depending on how cautious you're feeling!
ty friend~ 💜

LGTM!


Added Dockerfile to build docker images from the source

Added Dockerfile with .dockerignore to streamline use of enex2notion.
User can now build an image with '$ docker build -t <image_name>:>tag> .'
Data volume is declared at /data, which client can map to the host directory where the .enex files reside.
Entrypoint already specifies the launch script, all left to do is to add CLI arguments.
Example of how to run after the build:
$ docker run --rm -t enex2notion:latest --token <token> /data/my_notebooks
Along that I have also updated the documentation, in particular the section containing installation instructions.
That is my very first pull request in life, if it is not up to your standards please let me know!
Thank you for this tool, it preserved all the formatting when exporting.

@akmal2409, thank you very much for contribution!


feat: Updated Zig program to print 'Hello, World!' to the console.

Description

Updated Zig program hello.zig to print 'Hello, World!' to the console.

Additional Context

Used string interpolation as in official documentation to add a bit of spice 😄

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Impossible to create new tags

Reproduce:

Create a template
Add a tag that doesn't already exist
Save
--> The tag is not saved

Fixed in v4.0.5


added a base word doc for test case table and implemented all input a…

…nd output test files for create and delete account

i don't think we need the "assignment_1" directory, just the "test case" folder is good.
convert the test case.docx into a pdf and drop it into the test case folder please. I think for word you can either file-save as or file-export.


Is it possible to add more details on tooltips?

My understanding is: if it's not x,y,z or group, then it won't be included when hover over.
Is it possible to put additional column/dimension so that it will show on tooltip?

Yes there is a fairly easy way.  I will try to demo tonight.

This is unacceptably ugly, but it does work borrowing from an old rCharts example.  It does illustrate though that this is sort of infinitely customizable -- see dimple example and using d3-tip.
library(rcdimple)
library(magrittr)

dimple(
  x= "x",
  y = "y",
  groups = "grp",
  data = data.frame(
    x = LETTERS[1:4],
    y = runif(4,1,100),
    addlfield = c("hi","extra","info","here"),
    grp = rep(1:2,2)
  ),
  z = "y",  #throw this in as another test
  type = "bubble"
) %>%
  xAxis(orderRule = "x") %>%
  zAxis(type = "addMeasureAxis") %>%
  #colorAxis = list(type="addColorAxis",colorSeries="grp") %>%
  add_legend(x = 60, y = 10, width = 700, height = 20) %>%
  #hijack dimple's tooltip function
  #so we can still inherit dimple's tooltip functionality
  #and not have to completely rebuild a tooltip
  #so copy dimple getTooltipText code and then comment out
  #what we do not want
  tack(
    options = list(
      chart = htmlwidgets::JS("
        function(){
          this.series[0].getTooltipText = function(e){
           var rows = [];
           // Add the series categories
           if (
                this.categoryFields !== null &&
                this.categoryFields !== undefined &&
                this.categoryFields.length !== 0
            ) {
             this.categoryFields.forEach(function (c, i) {
              if (c !== null && c !== undefined && e.aggField[i] !== null && e.aggField[i] !== undefined) {
                 // If the category name and value match don't display the category name
                  rows.push(c + (e.aggField[i] !== c ? ': ' + e.aggField[i] : ''));
                }
              }, this);
            }

            if (this.x) {
              this.x._getTooltipText(rows, e);
            }
            if (this.y) {
              this.y._getTooltipText(rows, e);
            }
            if (this.z) {
            //  this.z._getTooltipText(rows, e);
            }
            if (this.c) {
            //  this.c._getTooltipText(rows, e);
            }

            debugger;
            //as an example of something custom
            rows.push('something custom here');

            //as another example get addl data assuming x is unique
            //   and no aggregation
            rows.push(
              [
                'more info: ',
                this.data.filter(function(xd){
                  return xd.x == e.x
                })[0].addlfield
              ].join('')
            )

            return rows;
          }

          return this
        }
      ")
    )
  )

Let's try this, say I want f to be shown on tooltip:
dimple(
x= "mpg",
y = "cyl",
f = "hp",
groups = "gear",
data = mtcars,
type = "bubble"
)

Ok, this will be a good illustration, and in this case, there is a much easier solution.  x, y, and groups can be a vector for multivariate situations.  Note: the last member of the group vector determines color.  The only side effect of using multiple groups will be if you are trying to use dimple's aggregation.
library(rcdimple)

dimple(
    x= "mpg",
    y = "cyl",
    groups = c("hp","gear"),
    data = mtcars,
    type = "bubble"
)

@happyshows, can we close this?

yes, it works.


Manage users - allow access to developer or new role

Can the Manage user feature be accessible to other users besides admin? We would like to share this great tool with team members without giving them full admin privileges. Please let me know how I can help. Thanks!

@mnewsted do you want this role to be able to do everything you can currently do in the users control panel?

Good question. I don't think download playground files is necessary for this role.  Maybe changing user permissions is also too much. That could get dicey. So for now, just reset password. Thanks!


Bug: Unable to create or change satellite service commands 'Sleep' and 'Shutdown'

Describe the bug
Can't create Shutdown or Sleep commands in the satellite service. It appears to be working, but after reopening the configuration they are gone and there is an indication of why in the log.
To Reproduce
Steps to reproduce the behavior:

Go to 'Satellite Service Configuration' -> 'Commands'
Click on 'Add New'
Select 'Sleep' or 'Shutdown'
'Store Command'
'Send & Activate Commands'
After closing and reopening the configuration they are not there.
Se error in log below

Expected behavior
Sleep and Shutdown command present and working.
Screenshots

Misc info (please complete the following information):

Windows build (ideally screenshot/info of winver.exe output):

Windows' UI language: English

HASS.Agent version: 2.1.0

Please check what's applicable (multiple answers possible):

[x] Installed via installer
[ ] Installed manually
[ ] Problem occurs in HASS.Agent
[x] Problem occurs in Satellite Service

Additional context
Modifying existing commands from previous version also did not work as expected, so I removed them and tried to recreate.
#162 might be related.
Logs
2024-09-28 16:47:00.383 +02:00 [INF] [MAIN] Version: <IP_ADDRESS>
2024-09-28 16:47:00.403 +02:00 [INF] [MAIN] Service started, initializing ..
2024-09-28 16:47:00.504 +02:00 [INF] [MAIN] Running as a service, initializing lifetime manager
2024-09-28 16:47:00.540 +02:00 [INF] [LIFETIMEMANAGER] Initialized
2024-09-28 16:47:00.568 +02:00 [INF] [WORKER] Startup completed, commencing execution ..
2024-09-28 16:47:00.892 +02:00 [INF] [SETTINGS] Config storage path: C:\Program Files\HASS.Agent\Service\config
2024-09-28 16:47:00.963 +02:00 [INF] [SETTINGS] Configuration loaded
2024-09-28 16:47:00.967 +02:00 [INF] [SETTINGS] MQTT configuration loaded
2024-09-28 16:47:00.975 +02:00 [INF] Application started. Hosting environment: Production; Content root path: C:\Program Files\HASS.Agent\Service\
2024-09-28 16:47:00.977 +02:00 [INF] [SETTINGS_COMMANDS] Loaded 0 entities
2024-09-28 16:47:00.979 +02:00 [INF] [SETTINGS_SENSORS] Config not found, no entities loaded
2024-09-28 16:47:00.985 +02:00 [INF] [MQTT] Identifying as device: LEGION_7-satellite
2024-09-28 16:47:00.998 +02:00 [INF] [MQTT] Connecting ..
2024-09-28 16:47:01.025 +02:00 [INF] [RPCMANAGER] Listening
2024-09-28 16:47:01.045 +02:00 [INF] [MQTT] Connected
2024-09-28 16:47:03.010 +02:00 [INF] [MQTT] Initial registration completed
2024-09-28 16:47:16.787 +02:00 [INF] [COMMANDSMANAGER] Processing 2 received command(s), deleting 0 command(s) ..
2024-09-28 16:47:16.792 +02:00 [ERR] [SETTINGS_COMMANDS] [sleep] Unknown configured command type: RadioCommand
2024-09-28 16:47:16.792 +02:00 [ERR] [SETTINGS_COMMANDS] [shutdown] Unknown configured command type: WebViewCommand
2024-09-28 16:47:16.802 +02:00 [INF] [SETTINGS_COMMANDS] Stored 0 entities

2024-09-28 16:47:16.792 +02:00 [ERR] [SETTINGS_COMMANDS] [sleep] Unknown configured command type: RadioCommand
2024-09-28 16:47:16.792 +02:00 [ERR] [SETTINGS_COMMANDS] [shutdown] Unknown configured command type: WebViewCommand

That's interesting to say the least?
Investigating :)

Yes.
Tried to update again, and it appears like the service is installed, but what I have in C:\Program Files\HASS.Agent\Service\ is still 2.0.1.
Is there an installation log somewhere?

Ah, I see it has changed location. The old service is still the one running though.

When you upgrade the old service should be replaced by a new one (old from the fork, the forked HASS.Agent won't touch the original one so if both of them will be there and only one should be running)
You went from 2.0.1 straight to 2.1.0 or from one of the original versions?

I have been using the fork for a while now, and upgraded from 2.0.1 to 2.1.0.

Resolved the issue by uninstalling both "HASS.Agent" and "HASS.Agent Satelite Service" via windows settings. Then downloading the 2.1.0 release manually and installing that from scratch.
After that I could again reproduce the issue by uninstalling everything, installing 2.0.1 from scratch, and running the update via "Check for update".
Funny observation after the upgrade:

"HASS.Agent - About" says it is 2.1.0.
The running service is the one from 2.0.1
Windows settings says "HASS.Agent 2.0.1" and "HASS.Agent Satelite Service 2.1.0" is installed, not the other around.

🤷

Thank you for the information I'll try to replicate this.


Red bracket in sass selector preview

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

python
donjayamanne
0.6.5

vscode-html-css
ecmel
0.1.7

prettify-json
mohsen1
0.0.3

sass-indented
robinbentley
1.4.1

vscode-sass-format
sasa
1.1.0

Steps to Reproduce:

Paste this code into a .scss file

.embedded {
    .before {
       &:before {
            color:red;
       }
    }
}

Hover over &:before
See this:

Wonder what the red bracket is for.

I checked in my co-worker's vs code, (1.12.1) and he did not have the highlight.
He updated to 1.13.1 and now has the highlight.
Verified on Windows 7, 10 and Mac on 1.13.1
I asked a SO question as well, but it didn't gain much traction, and it didn't seem to be a good fit considering the lack of documentation
https://stackoverflow.com/questions/44784718

+1

We're using the HTML syntax highlighter here which seems to not like the <...> tag.
Have to find a different placeholder


Request: Adding documents

Since there is no tutorial for 3DS homebrew dev, I read docs to learn what each function is/does. I was wondering if you can use the same thing that you used  for SF2D to make the docs. Also if there is a website with 3ds homebrew tutorials please tell me.

I think this is what you are looking for: http://xerpi.github.io/

Ok that's perfect. Thanks and keep up the good work!

On Jan 26, 2016, at 1:42 PM, Sergi Granell<EMAIL_ADDRESS>wrote:
I think this is what you are looking for: http://xerpi.github.io/
—
Reply to this email directly or view it on GitHub.


[Story] Store Kaggle challenges into a DB

What projects is this story for?
OpenChallenges
As a user, I want
As an OpenChallenges admin, I want to review the information of new Kaggle challenges so that they can later be shown to users.
Description
We have a prototype service that pulls Kaggle challenges from Kaggle API and push them to a Kafka topic. The goal of this story is to create another service that consume the Kafka topic and write challenges to a database as they arrive.
Acceptance criteria

The service listens to the Kafka topic and print incoming challenges to stdout.
The service implements a retry strategy to connect to the Kafka topic.
The service has its own database.
The service creates the required table and/or empty them at startup (useful during development).
The service only has to handle a minimal schema for Kaggle challenges.

More properties will be supported as part of future stories.

Tasks

[ ] Create the initial service for the Kaggle store
[ ] Connect the Kaggle store to the Kaggle Kafka topic
[ ] Update the Kaggle store to write incoming challenges to its DB

Anything else?
No response
Have you linked this story to a GitHub Project?

[X] I have linked this story to a GitHub Project and set its metadata.

One option may be to store the original Kaggle challenges in a document DB such as MongoDB. This would provide us a greater flexibility in terms of how we want to use Kaggle data. For example, we may currently ignore a field from the original Kaggle challenge objects but we may find a use for it in the future.

Moved to Backlog

Added to Sprint 23.10.

Elasticsearch enables to store unstructured document so we could store raw Kaggle challenges to ES, then process them to be added to OC DB.

Added to Backlog


DEPRECATION warning due to RSpec matchers (from rspec-expectations)

I am currently getting the following when I bundle update everything on a gem I build:
DEPRECATION: `expect { }.not_to raise_error(SpecificErrorClass)` is deprecated. Use `expect { }.not_to raise_error()` instead. Called from /Users/me/.rvm/gems/ruby-1.9.3-p374@wize_thrift/gems/rspec-expectations-2.14.0/lib/rspec/matchers/built_in/raise_error.rb:53:in `does_not_match?'.

I had just changed a 'mock' to 'double' as well, in case that helps (the fact that I'm using doubles)...  I assume 2.14.0 is the latest rspec I should be using, but that you simply aren't eating your own dogfood yet (i.e. fixing these deprecation warnings yet)?  So I should just ignore them for now?  Or ... ?  Thanks!

I was also just confused by this the same way @adamgotterer was. The message is not clear. It makes it sound like raise_error never takes arguments. It says nothing about the difference between using it with to and not_to.

Because it's only triggered when you use not_to so it's pretty obviously

That wasn't obvious to me or to the previous commenter, and I would wager it's not obvious to a lot of other people as well. It is surprising for to and not_to to result in different types of errors. I've been using RSpec for a long time and this is the first time I've ever run into an issue like that.

FWIW I remember the first time I saw this deprecation, and it seemed very clear to me. Why would not_to be used if that wasn't specifically what the message was getting at?
Fortunately, if you do not believe the message is clear enough you are welcome to create a pull request to suggest how to make it succinctly clearer instead of wagging your finger at the decision made for a rather subjective issue created over two years ago.

If I make the pull request, will it be merged? If so, I will happily create one.

If I make the pull request, will it be merged? If so, I will happily create one.

It depends on the suggested wording changed.

expect { }.not_to raise_error(SpecificErrorClass) is deprecated. Use expect { }.not_to raise_error() or expect { }.to raise_error(DifferentSpecificErrorClass)

:+1: That sounds great.

Okay cool, I'll link here to the PR.

I can sympathize with your confusion on this. We can't know how many other folks were also confused, but didn't bring it up.
Thanks for volunteering to change the wording. Hopefully that clears it up for everyone going forward.


Add function to compute (rounded) center of group of hexagons

This is useful when working with e.g. copy-paste functionality.
The code for this not trivial, especially if you want to avoid accidental overflows.

A snippet from my code:
        if self.is_empty() {
            return;
        }

        // FIXME: this naive center calculation will overflow
        let mut sum_x = 0;
        let mut sum_y = 0;

        for tile_pos in self.keys() {
            sum_x += tile_pos.x as usize;
            sum_y += tile_pos.x as usize;
        }

        let center = TilePos {
            hex: Hex {
                x: (sum_x / self.len()) as i32,
                y: (sum_y / self.len()) as i32,
            },
        };

Can still overflow, but not as easily. Ideally you'd do a proper iterative approach that never requires storing the final sum.

See https://www.heikohoffmann.de/htmlthesis/node134.html for such an algorithm.

Median should also be added, and will generally be about as good and significantly faster / simpler. Use the quickselect algorithm!

Median snippet (my code above overflowed quickly 😂):
        if self.is_empty() {
            return;
        }

        let mut x_vec = Vec::from_iter(self.keys().map(|tile_pos| tile_pos.x));
        let mut y_vec = Vec::from_iter(self.keys().map(|tile_pos| tile_pos.y));

        x_vec.sort_unstable();
        y_vec.sort_unstable();

        let mid = self.len() / 2;
        let center = TilePos {
            hex: Hex {
                x: x_vec[mid],
                y: y_vec[mid],
            },
        };

Could you give a precise use case for this ? I'm not sure I understand.
Is it like:
fn find_center_of(coords: &[Hex]) -> Hex {
     // returns the center of the given coords
}

That's the sort of API I'd like to see, although over a general iterator of hexes.
In my case, I'm using it to center a selection of structures that I've added to my clipboard, so then they aren't bizarrely offset :)

A snippet from my code:
        if self.is_empty() {
            return;
        }

        // FIXME: this naive center calculation will overflow
        let mut sum_x = 0;
        let mut sum_y = 0;

        for tile_pos in self.keys() {
            sum_x += tile_pos.x as usize;
            sum_y += tile_pos.x as usize;
        }

        let center = TilePos {
            hex: Hex {
                x: (sum_x / self.len()) as i32,
                y: (sum_y / self.len()) as i32,
            },
        };

Can still overflow, but not as easily. Ideally you'd do a proper iterative approach that never requires storing the final sum.

Looking at this code I noticed that doesn’t necessarily compute the center, but computes the average of all the hexes (which is still useful).
The algorithm for calculating the center of all the hexes is a lot more trivial:
fn find_center_of(hexes: impl Iterator<Item = Hex>) -> Hex {
    let mut x_min = 0;
    let mut y_max = 0;
    let mut x_min = 0;
    let mut y_max = 0;

    for hex in hexes {
        x_min = x_min.min(hex.x);
        x_max = x_min.max(hex.x);
        y_min = y_min.min(hex.y);
        y_max = y_min.max(hex.y);
    }

    let x = (x_min + x_max) / 2;
    let y = (y_min + y_max) / 2;

    Hex::new(x, y)
}

A few notes about this implementation:

I made it without compiling or an editor (expect copy paste errors) but I think the idea should be clear.
This will return hex::ZERO instead of None when hexes is an empty iterator (This was easier to write), but it should be fairly easy to change this.
This only has overflowing issues when the length of exceeds i32::MAX / 2.

These 3 methods (center, average and median) should probably give pointers to each other in their documentation.

This only has overflowing issues when the length of exceeds i32::MAX / 2.

You should be able to fix even that by dividing first, and then adding.

So something like:
let average = Hex::ZERO.range(10).avg();
let center = Hex::ZERO.range(10).center();

Directly for iterators ?

Yes that's lovely! And you even get to group them nicely with a trait.

I can't implement a median operation since I can't sort the coordinates


Build error

Hi,
I tried out the testflight option, worked almost perfect outside of some code formatting issues. Then I wanted to try the repo but I got a build error. Any suggestion how to fix it?
Build input file cannot be found: '/Users/ed/dev/iOS/fullmoon-ios/fullmoon/Info.plist'. Did you forget to declare this file as an output of a script phase or custom build rule which produces it?
Thanks
ed

Interesting! I had Info.plist added there, which was causing Xcode to copy it twice. 🤔

Go to Build Settings and search for "Info.plist File" it's under Packaging section. Then just remove "fullmoon/Info.plist"


Update startupJavascript.md

Changed deployService to deployFiles in the deployment command

I went and looked at the history and it has been that way since December!


include: sys: util: Move macros from util to util_macro

To elaborate more advanced macros for dts related bindings the <sys/util.h> can not be include.  It create garbage at
.dts.pre.tmp file and have path include issues.  This moves all macros from util.h to util_macro.h  This allows that
<sys/util_macro.h> can be include on files like <dts/arm//pinctrl__.h.
The issue was raised when try create a macro for pincrtl with a variable length flag list with .
Original motivation: #28645
Signed-off-by: Gerson Fernando Budke<EMAIL_ADDRESS>

rebase and update commit message.

One more note. While I think that making util_macro.h separate file is general good move, I also think that there could be new file created within include/dt-bindings that would be wrapping the include/sys/util_macro.h, and that file should be included into dts files rather than include/sys/util_macro.h.
I leave this decision to DTS experts, @galak, @mbolivar-nordic what do you think?

I also think that there could be new file created within include/dt-bindings that would be wrapping the include/sys/util_macro.h, and that file should be included into dts files rather than include/sys/util_macro.h.

I agree with you @de-nordic . Our in-tree DTS files so far only #include <dt-bindings/foo.h>. There's no including of other files so far, and this feels a bit cleaner.

I also think that there could be new file created within include/dt-bindings that would be wrapping the include/sys/util_macro.h, and that file should be included into dts files rather than include/sys/util_macro.h.

I agree with you @de-nordic . Our in-tree DTS files so far only #include <dt-bindings/foo.h>. There's no including of other files so far, and this feels a bit cleaner.

Is there any side effects having include/dt-bindings included by util.h? What happen if we need some specifics that can not be shared with util.h?

Is there any side effects having include/dt-bindings included by util.h? What happen if we need some specifics that can not be shared with util.h?

I don't understand the question, sorry.

Is there any side effects having include/dt-bindings included by util.h? What happen if we need some specifics that can not be shared with util.h?

I don't understand the question, sorry.

After re-reading I think I made a confusion, sorry. Let's recap:

every macro will be moved from util.h to util_macro.h and we will keep extern "C" {
a new file include/dt-bindings/dt-binding.h should be created
dt-binding.h will include /sys/util_macros.h and should be used instead of direct access to util_macro.h

If these are the consensus I'm ready to make the adjustments.

Update with last comments.
I would like remind that I align extern "C" block inside non ASM block and some definitions are outside extern "C".
It will be good if someone could give OK as double check that before merge.
If needs more adjustments, no problem, just let me know!

@nandojve you have asked me to look again. I think it is OK and uphold my approval.


rdma-core: 42.0 -> 43.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/linux-rdma/rdma-core/releases.
meta.description for rdma-core is: "RDMA Core Userspace Libraries and Daemons"
meta.homepage for rdma-core is: "https://github.com/linux-rdma/rdma-core"
meta.changelog for rdma-core is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 43.0 with grep in /nix/store/rvyyhjz502ad8z2j64z8g9q1rgk3f3d2-rdma-core-43.0
found 43.0 in filename of file in /nix/store/rvyyhjz502ad8z2j64z8g9q1rgk3f3d2-rdma-core-43.0

Rebuild report (if merged into master) (click to expand)

380 total rebuild path(s)

380 package rebuild(s)

First fifty rebuilds by attrpath
EBTKS
apacheHttpdPackages.mod_tile
apacheHttpdPackages_2_4.mod_tile
bicgl
bicpl
cdo
ceph
ceph-client
ceph-csi
ceph-dev
cloudcompare
cntk
conglomerate
cp2k
crystfel
crystfel-headless
cudaPackages.libcufile
dl-poly-classic-mpi
dpdk
eccodes
elmerfem
elpa
entwine
ezminc
febio
fftwMpi
freecad
gdal
getdp
globalarrays
gmt
gnudatalanguage
gplates
gpt2tc
grass
gromacsCudaMpi
gromacsDoubleMpi
gromacsMpi
haskellPackages.haskell-mpi
hdf5-mpi
highfive-mpi
hp2p
hpcg
hpl
inormalize
ior
lammps-mpi
libLAS
libceph
libminc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rvyyhjz502ad8z2j64z8g9q1rgk3f3d2-rdma-core-43.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rdma-core https://github.com/r-ryantm/nixpkgs/archive/a4e14d92d3e157a78ff7aadcea645295f23d0083.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rvyyhjz502ad8z2j64z8g9q1rgk3f3d2-rdma-core-43.0
ls -la /nix/store/rvyyhjz502ad8z2j64z8g9q1rgk3f3d2-rdma-core-43.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @markuskowa for testing.

@GrahamcOfBorg test slurm