Unable to create a visdom with given name

When run the following code, an error is reported by the terminal running the visdom.server
vis = Visdom(env='test')
However, if the env name is not given, it is OK. The specific code is:
vis = Visdom()
I have tried to fix this error by install simplejson, but it does not work.
ERROR:tornado.application:Uncaught exception POST /win_exists (<IP_ADDRESS>)
HTTPServerRequest(protocol='http', host='localhost:8097', method='POST', uri='/win_exists', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/tornado/web.py", line 1541, in _execute
result = method(*self.path_args, **self.path_kwargs)
File "/usr/local/lib/python3.5/dist-packages/visdom/server.py", line 481, in post
self.wrap_func(self, args)
File "/usr/local/lib/python3.5/dist-packages/visdom/server.py", line 472, in wrap_func
if args['win'] in handler.state[eid]['jsons']:
KeyError: 'test'
ERROR:root:ERROR: 500: {'exc_info': (<class 'KeyError'>, KeyError('test',), <traceback object at 0x7fca48f23608>)}

Looks like a bug, thanks for reporting!
As a work-around, you can instantiate the visdom class without environment, vis = Visdom(), and specify the environment whenever you create a plot: vis.line(Y, X, env='test').

Nice. It works. Thank you very much.


jar dependency match with json report

Problem
Coursier command:
fetch -t \
com.esotericsoftware.reflectasm:reflectasm:1.09,classifier=shaded \ # from pants jar_dep A
com.esotericsoftware.reflectasm:reflectasm:1.09 \ # from pants jar_dep B
org.ow2.asm:asm:4.0,classifier=sources \ # from pants jar_dep C
-A jar,src,doc,test-jar --json-output-file x.out

  Result:
├─ com.esotericsoftware.reflectasm:reflectasm:1.09
│  └─ org.ow2.asm:asm:4.0
├─ com.esotericsoftware.reflectasm:reflectasm:1.09
│  └─ org.ow2.asm:asm:4.0
└─ org.ow2.asm:asm:4.0

Since there are 3 roots to resolve, the tree is correct. One com.esotericsoftware.reflectasm:reflectasm:1.09 wants the default jar, and the other wants the shaded classifier jar. (UI wise it could be better to show that which is which)
Now look at the json output:
{
  "conflict_resolution": {},
  "dependencies": [
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "sources",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0-sources.jar"
        ]
      ],
      "dependencies": []
    },
    {
      "coord": "com.esotericsoftware.reflectasm:reflectasm:1.09",
      "files": [
        [
          "shaded",
          "<cache_dir>/https/repo1.maven.org/maven2/com/esotericsoftware/reflectasm/reflectasm/1.09/reflectasm-1.09-shaded.jar"
        ]
      ],
      "dependencies": [
        "org.ow2.asm:asm:4.0"
      ]
    },
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0.jar"
        ]
      ],
      "dependencies": []
    },
    {
      "coord": "com.esotericsoftware.reflectasm:reflectasm:1.09",
      "files": [
        [
          "",
          "<cache_dir>/https/repo1.maven.org/maven2/com/esotericsoftware/reflectasm/reflectasm/1.09/reflectasm-1.09.jar"
        ]
      ],
      "dependencies": [
        "org.ow2.asm:asm:4.0"
      ]
    }
  ]
}

For jar_dep A, it has to find the coordinate "com.esotericsoftware.reflectasm:reflectasm". There are two of them, but it knows to find the one with classifier 'shaded':
    {
      "coord": "com.esotericsoftware.reflectasm:reflectasm:1.09",
      "files": [
        [
          "shaded",
          "<cache_dir>/https/repo1.maven.org/maven2/com/esotericsoftware/reflectasm/reflectasm/1.09/reflectasm-1.09-shaded.jar"
        ]
      ],
      "dependencies": [
        "org.ow2.asm:asm:4.0"
      ]
    },

Great. jar_dep A grabs reflectasm-1.09-shaded.jar into its collection. Then it needs to grab its dependencies as well. Given the clue org.ow2.asm:asm:4.0, it finds:
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0.jar"
        ]
      ],
      "dependencies": []
    },

and
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "sources",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0-sources.jar"
        ]
      ],
      "dependencies": []
    },

Now, there is no way to tell which it needs, because all it knows is just org.ow2.asm:asm:4.0. Currently Pants' behavior is to grab both. However the correct jar to grab is just asm-4.0.jar and not asm-4.0-sources.jar
Proposed solution
Option 1:
Add classifier to dependencies org:name:version, i.e. org:name:version:classifier. For the same example:
{
  "conflict_resolution": {},
  "dependencies": [
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "sources",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0-sources.jar"
        ]
      ],
      "dependencies": []
    },
    {
      "coord": "com.esotericsoftware.reflectasm:reflectasm:1.09",
      "files": [
        [
          "shaded",
          "<cache_dir>/https/repo1.maven.org/maven2/com/esotericsoftware/reflectasm/reflectasm/1.09/reflectasm-1.09-shaded.jar"
        ]
      ],
      "dependencies": [
        "org.ow2.asm:asm:4.0:<empty classifier>"
      ]
    },
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0.jar"
        ]
      ],
      "dependencies": []
    },
    {
      "coord": "com.esotericsoftware.reflectasm:reflectasm:1.09",
      "files": [
        [
          "",
          "<cache_dir>/https/repo1.maven.org/maven2/com/esotericsoftware/reflectasm/reflectasm/1.09/reflectasm-1.09.jar"
        ]
      ],
      "dependencies": [
        "org.ow2.asm:asm:4.0:<empty classifier>"
      ]
    }
  ]
}

Now jar_dep A knows the dependency is org.ow2.asm:asm:4.0:<empty classifier>, thus finding the following and grabs asm-4.0.jar only.
    {
      "coord": "org.ow2.asm:asm:4.0",
      "files": [
        [
          "",
          "<cache_dir>/https/repo1.maven.org/maven2/org/ow2/asm/asm/4.0/asm-4.0.jar"
        ]
      ],
      "dependencies": []
    },

Option 2:
Make both coord and dependencies to adopt org:name:version:classifier. I am not sure what the report would look like when --sources and --javadoc are both specified, so might need more thoughts there. But option 1 should be more straight forward.

closed by d0b46864c832ec1315ff886e55f6b94dfa3dfb0d


DBZ-1980 Only top-level menu items shown for MySQL connector

This removes some attributes from the partials that interfered with generating the ToC properly.  This commit should likely also be cherry-picked back to the 1.0 branch as it exhibits the same behavior problems.

This commit should likely also be cherry-picked back to the 1.0

+1.
What about master / 1.2? Does the issue surface there, too?

Master and 1.2 branches are fine.

Gotcha. Can you do the 1.0 cherry-pick? Thanks!

Will do.

Thx!


Feature Request: Mac OS 10.13 High Sierra compatibility

Just came across this program. Tried to install the program, but it is not compatible with Mac OS 10.13 High Sierra :(
Is there a  past version of RcloneBrowser that is compatible with Mac OS 10.13 High Sierra?

Indeed current version works with Mojave and Catalina only. But now I have found the way how to keep latest OS features working (e.g. dark theme) and provide backward compatibility. The next release will work on anything down OS X Mavericks . It should be release in the next week or two.

if you want here is the latest pre-release version working on all older macOS systems:
https://www.icloud.com/iclouddrive/0TXekZ4JL1TAANEGhM3IGReNQ#rclone-browser-1.6.0-409180a-macOS

Thanks, will definitely try out the pre-release version.


Dahau VC-PTI08-A180-PV

Hello,
I have installed the addon in home assistant using HACS and have connected to a Dahau Camera. The camera has security lights and a alarm. These functions are not shown in Home Assistant.
model is VC-PTI08-A180-PV.
Firmware: 3.140.0000000.21.R,build:2024-05-14
ONVIF version: 23.12(V<IP_ADDRESS>4191)
These are the only controls shown.

Probably have to add to supports security light.
def supports_security_light(self) -> bool:
        """
        Returns true if this camera has the red/blue flashing security light feature.  For example, the
        IPC-HDW3849HP-AS-PV does https://dahuawiki.com/Template:NameConvention
        Addressed issue https://github.com/rroller/dahua/pull/405
        """
        return "-AS-PV" in self.model or self.model == "AD410" or self.model.startswith("IP8M-2796E")

Something like this:
def supports_security_light(self) -> bool:
        """
        Returns true if this camera has the red/blue flashing security light feature.  For example, the
        IPC-HDW3849HP-AS-PV does https://dahuawiki.com/Template:NameConvention
        Addressed issue https://github.com/rroller/dahua/pull/405
        """
        return "-AS-PV" in self.model or self.model == "AD410" or self.model.startswith("IP8M-2796E") or self.model.startswith("IPC-HDW3849HP")

Let me know if you need help testing and then I can help get it into code. I added IP8M-2796E to get the same features working on it. Currently that is all starting on line 561 through 567 in init.py which is in custom_components/dahua directory.
this is what mine looks like now:

To test you just have to modify the line 567 with this. If it work, then we can create a pull request and add it so it is part of the codebase.
return "-AS-PV" in self.model or self.model == "AD410" or self.model.startswith("IP8M-2796E") or self.model.startswith("IPC-HDW3849HP")


HeatMap not work

{
"id": "1",
"query": {
"sql": "SELECT ST_Point (pickup_longitude, pickup_latitude) AS point, passenger_count AS w FROM global_temp.nyc_taxi WHERE (ST_Within (ST_Point (pickup_longitude, pickup_latitude), 'POLYGON ((-73.5 40.1, -73.5 41.1, -70.5 41.1, -70.5 40.1, -73.5 40.1))'))",
"type": "heat",
"params": {
"width": 1024,
"height": 896,
"heat": {
"bounding_box": [
-73.998427,
40.730309,
-73.954348,
40.780816
],
"coordinate": "EPSG:4326",
"map_scale": 10
}
}
}
}

以上sql在上个版本是可以跑的

The data in bounding box should be consistent with the polygon parameter in ST_Within, so the json should be like so:
{
"id": "1",
"query": {
"sql": "SELECT ST_Point (pickup_longitude, pickup_latitude) AS point, passenger_count AS w FROM global_temp.nyc_taxi WHERE (ST_Within (ST_Point (pickup_longitude, pickup_latitude), 'POLYGON ((-73.5 40.1, -73.5 41.1, -70.5 41.1, -70.5 40.1, -73.5 40.1))'))",
"type": "heat",
"params": {
"width": 1024,
"height": 896,
"heat": {
"bounding_box": [-73.5, 40.1, -70.5, 41.1],
"coordinate": "EPSG:4326",
"map_scale": 10
}
}
}
}

By running this json, I got the right result.

By the way, if you still want a picture without point, that's OK. This has been fixed by #420 , and you would get a blank picture.


Drop dependency on bash-parser

We recently started using https://socket.dev/ as part of our dependencies maintenance and the tool has detected bash-parser as problematic due to being unmaintained for 6 years: https://socket.dev/npm/package/bash-parser
bash-parser itself also uses old and unmaintained dependencies, e.g. https://socket.dev/npm/package/deep-freeze which is a repo that has a non-existent npm account associated to it.

It was difficult to even find such a package, let alone a recent or actively maintained one.
On the other hand, it provides functionality that makes Knip quite powerful in finding dependencies and entry files. This test shows quite some cases it can handle, and the scripts can come from package.json#scripts, and the GitHub Action, Husky, Lefthook, lint-staged, Nx and release-it plugins. This is exactly the extra mile Knip likes to go.
I would also be happy to replace bash-parser with something more light-weight and well-maintained, but I'm not willing to consider losing this type of functionality. Perhaps it could be some combination of bash-parser becoming an optional dependency and opt-out by configuration or something.
Just for reference, I started out with a bunch of regexes, but that's far from elegant, limited and hard to maintain.
On the other hand, we're talking Bash here, syntax doesn't really change at all. It parses strings in configuration files. I wonder how can this package can actually be a threat or exploited? If it is I should perhaps add a warning to the docs.
Very open to ideas and suggestions here, I want Knip to be a good citizen in anyone's node_modules.

Thanks for the detailed explanation!
Since knip is a dev dependency, I am not worried but I simply would like to get rid of unmaintained deps as much as possible. Reporting issues to multiple repositories to move the ecosystem forward.
No need to be actionable on this right away, let's keep it open for reference and maybe a better alternative will come around for the package or someone might have a recommendation!

It's causing errors for me
$ knip
Resolving plugin configuration files...
/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/bash-parser/src/index.js:52
                throw new Error(err.stack || err.message);
                      ^

Error: Error: Parse error on line 1: Unexpected 'LESS'
    at parse (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/bash-parser/src/index.js:52:9)
    at setCommandExpansion (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/bash-parser/src/modes/posix/rules/command-expansion.js:17:21)
    at /home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/bash-parser/src/modes/posix/rules/command-expansion.js:37:12
    at Array.map (<anonymous>)
    at /home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/bash-parser/src/modes/posix/rules/command-expansion.js:35:59
    at Object.next (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/map-iterable/index.js:35:18)
    at Object.next (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/iterable-lookahead/index.js:51:24)
    at Object.next (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/map-iterable/index.js:33:30)
    at Object.next (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/iterable-lookahead/index.js:51:24)
    at Object.next (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/map-iterable/index.js:33:30)
    at parse (/home/linuxbrew/.linuxbrew/lib/node_modules/knip/node_modules/bash-parser/src/index.js:52:9)
    at getBinariesFromScript (file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/util/binaries/bash-parser.js:55:20)
    at file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/util/binaries/index.js:16:28
    at Array.flatMap (<anonymous>)
    at getReferencesFromScripts (file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/util/binaries/index.js:16:10)
    at Module.findGithubActionsDependencies (file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/plugins/github-actions/index.js:15:38)
    at async WorkspaceWorker.findDependenciesByPlugins (file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/workspace-worker.js:213:41)
    at async main (file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/index.js:187:17)
    at async run (file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/cli.js:27:46)
    at async file:///home/linuxbrew/.linuxbrew/lib/node_modules/knip/dist/cli.js:59:1

Any chance you could look up the script that contains the "less than" (<) character? It should be in some GitHub Action .yml file afaics.
The issue could be in Knip as well, and at least it should try to work around it.

For me it errored on an unclosed ' in a package.json script. Here's the value of the script entry:
NODE_ENV=test; NODE_OPTIONS=\"--max-old-space-size=3072\" node --expose-gc \"$(yarn bin jest || kill $$)\" --reporters=\"default\" --reporters=\"github-actions\"     --forceExit --logHeapUsage --maxWorkers=\"$(node -e 'process.stdout.write(os.cpus().length.toString())')\"

Removing this script entry fixed it for me (though different issue than the reported one above)

I fixed that it throws so hard on bash parser errors by swallowing them in v2.1.2. Errors still show up in --debug if debugging false positives.

On a related note, bash-parser also brings in a transitive dependency that's technically protected by copyright as it lacks a license statement of any kind - see also https://github.com/vorpaljs/bash-parser/issues/65. Even as a devDependency, this can make knip difficult to use in certain settings.
While not actively addressing the maintainance issue on which this issue is based, it might make sense to fork/vendor bash-parser and replace the problematic transitive dependency. What are your thoughts on this @webpro?

Since the latest release of vorpaljs/bash-parser is 6 years ago I wouldn't mind depending on a fork.
Still on-topic, does anybody have an idea about possible alternatives?

After seeing tree-sitter again the other day I tried using it in Knip with the Bash grammar to replace bash-parser. Turned out nicely and took little efforts.
Pros:

Works well on the existing test suite within Knip, not much logic in Knip needed (a lot less compared to bash-parser).
Seems widely used and plenty of activity in tree-sitter.

Cons:

Last publish of tree-sitter-bash is over two years ago (but recent fork: @curlconverter/tree-sitter-bash).
Handling of quotes is lacking (but seems manageable at first sight).
Not sure if pro/con, but not using tree cursor or query features, doing simple AST walking instead.

Although we may not like the age of bash-parser, it did a very decent job. So here I am asking for your opinion, security assessment, and hopefully a test run on your own project(s).
You can install v2.17.2-ts.0 with the ts tag using something like:
npm install -D knip@ts

Looking at tree-sitter, its fork and bash-parser I am coming to the conclusion:
Probably your best bet would be to ask for a new release for tree-sitter-bash as there are somewhat frequent commits on the repo recently. That dependency pulls in way more dependencies on its own though than bash-parser ~(68 vs 28)
Forks don't make sense most of the time because they are rarely ever maintained over a longer period of time.
All 3 have CVEs, so really not a great choice here.

A new release was requested in October 2022 in https://github.com/tree-sitter/tree-sitter-bash/issues/134. Tried to nudge the owner now.
At this point I think tree-sitter is the best option.

:rocket: This issue has been resolved in v2.19.6. See Release 2.19.6 for release notes.

Since<EMAIL_ADDRESS>was released the other day I've pulled the trigger here as well. Would deserve either a major bump or just a patch release. As I think it shouldn't cause friction anywhere I opted for the latter (I do expect an occasional issue around arguments w/ quotes here and there, let's see).

:rocket: This issue has been resolved in v2.19.11. See Release 2.19.11 for release notes.


nothing provides /usr/libexec/platform-python needed by osbuild-composer-tests-15-1.el8.x86_64

Installing the osbuild-composer-tests package from Brew yields:
 Problem: conflicting requests
  - nothing provides /usr/libexec/platform-python needed by osbuild-composer-tests-15-1.el8.x86_64

this will block all internal testing.

cc @teg

After Tom pinged me I realized I was trying to install a RHEL package on a Fedora system. Closing.


ENHANCEMENT: Add workflow to create and push new tags on merge to master

We should have a github workflow file that bumps the version tag for git and pushes it to trigger the release workflow. This therefore necessitates the need for a version file that should be bump accordingly on each pull request to master.

/assign

@GoGstickGo Hi 👋 , Would you like to take this?

Hi @Cian911 . If there is no time pressue on it definitely!

Hi @Cian911 . If there is no time pressue on it definitely!

No pressure at all. I'll assign it to you 👍

Hi @Cian911 I was rather occupied last week, didn't have time at all. However things are settled and I have time to dive in, will get it to during this week.

No problem @GoGstickGo, as I said, no rush 😄


Normalise LICENSE text w/ cabal's BSD2 template

Also, correct the .cabal files to advertise BSD2 instead
of the incorrect BSD3 license.

@simonmar Since you're the only explicitly named (c)-holder in the previous LICENSE text, I'm hoping you could help clarify what the licensing intent was; i.e. whether it's in fact the 2-clause BSD or something else.

This looks fine. I don't recall why the license differs from the standard one, no objection to syncing it.

Looks like CI is failing for the same reason that I recently fixed in Alex - cabal has changed where it puts the results of sdist for some reason.


Corrupted plots using the new examples.

Heya, I've got some issue going on with the new profiler examples provided in this repo.
Specifically, when you use the new plot examples you'll end up getting corrupted plot names. I suspect this is due to the CStrings being disposed of immediately before the C++ side can read them.
The new examples seem to dispose of CStrings alarmingly quickly, and so I'm a little worried about how widespread this issue is throughout the examples. Disposing of them immediately.
Here you can see the names of the plot being corrupted (there is only supposed to be one) and this causes Tracy to output more and more the longer the program continues to run. My profiling zones follow the same principles as the ones in the example profiler:

Reproduction

Lift one of the plotting functions out of the current examples.
Plot in a loop with a thread delay to test it
Note that multiple plots will appear in Tracy.

Here's a simple top-level statement that reproduces the issue cleanly (note that you may have to run it multiple times, as the issue can be sporadic)
using bottlenoselabs.C2CS.Runtime;
using static Tracy.PInvoke;

static async Task test()
{
    for (int i = 0; i < 200; i++)
    {
        Thread.Sleep(600);
        using CString newString = CString.FromString("TEST PLOT");
        TracyEmitPlotInt(newString, i);
    }
}

await test();

This issue is exacerbated if you perform more operations such as profiling a zone or emitting text, which will cause the plots to become more and more corrupted and numerous.

@Mervill

No prob! Forgive the naivety as I'm not experienced with C++ a whole lot, but would it make sense to do a bit more heavy-lifting on that side so that stuff doesn't need to be cached on the C# side? As far as I understand the tracy docs, it appears that most of the frame marking, plotting, etc. all wanna be kept on the stack as much as possible. (I haven't read the Tracy source yet, so forgive the educated guess)
There is a library I've worked with called 'soundpipe' that is written in C and takes a "Create -> Init -> Compute -> Destroy" approach on the C side. Perhaps it would be easier to manage the memory if the C++ side was the one creating and destroying the contexts? That way, you could simply receive a pointer or something that you pass around until you need to pass it back to the C++ side to be destroyed

No prob! Forgive the naivety as I'm not experienced with C++ a whole lot, but would it make sense to do a bit more heavy-lifting on that side so that stuff doesn't need to be cached on the C# side? As far as I understand the tracy docs, it appears that most of the frame marking, plotting, etc. all wanna be kept on the stack as much as possible. (I haven't read the Tracy source yet, so forgive the educated guess)
There is a library I've worked with called 'soundpipe' that is written in C and takes a "Create -> Init -> Compute -> Destroy" approach on the C side. Perhaps it would be easier to manage the memory if the C++ side was the one creating and destroying the contexts? That way, you could simply receive a pointer or something that you pass around until you need to pass it back to the C++ side to be destroyed.
If it's fast enough, you could potentially use SuppressGCTransition to reduce the overhead of P/Invoke calls too.

Okay, I've done a little bit of looking at the tracy docs and found some things.

Macros that accept a string literal and a pointer will copy the data in that literal to a temporary internal buffer. You probably do want to pool these, but you could probably increase the efficiency of the pooling by using a more conventional method than the dictionary you had before. (Tbh you could probably just wrap an ArrayPool or something so you can rent/free CString buffers all day, replacing the deallocation that Dispose() on a CString currently does)

Macros that only accept a string literal expect that literal to always be available at any point in the program's execution. The documentation suggests defining these somewhere central. Instead of trying to do this manually for the user, making it clear that this is the intended way to do this would probably be the best way here.

No prob! Forgive the naivety as I'm not experienced with C++ a whole lot, but would it make sense to do a bit more heavy-lifting on that side so that stuff doesn't need to be cached on the C# side? As far as I understand the tracy docs, it appears that most of the frame marking, plotting, etc. all wanna be kept on the stack as much as possible. (I haven't read the Tracy source yet, so forgive the educated guess)
There is a library I've worked with called 'soundpipe' that is written in C and takes a "Create -> Init -> Compute -> Destroy" approach on the C side. Perhaps it would be easier to manage the memory if the C++ side was the one creating and destroying the contexts? That way, you could simply receive a pointer or something that you pass around until you need to pass it back to the C++ side to be destroyed.
If it's fast enough, you could potentially use SuppressGCTransition to reduce the overhead of P/Invoke calls too.

Okay, I've done a little bit of looking at the tracy docs and found some things. You can ignore this previous comment.
1. Macros that accept a string literal and a pointer will copy the data in that literal to a temporary internal buffer. You probably do want to pool these, but you could probably increase the efficiency of the pooling by using a more conventional method than the dictionary you had before. (Tbh you could probably just wrap an ArrayPool<byte> or something so you can rent/free CString buffers all day, replacing the deallocation that Dispose() on a CString currently does)

2. Macros that only accept a string literal expect that literal to always be available at any point in the program's execution. The documentation suggests defining these somewhere central. Instead of trying to do this manually for the user, making it clear that this is the intended way to do this would probably be the best way here.

3. Some parts of tracy want that same pointer when doing things like marking zone starts and ends, meaning you want that CString to be around until the end of the zone.

Hmm... Instead of pooling buffers, I wonder if would be smarter to allocate the buffers on the stack for the parts of tracy that don't care about the string existing for more than the function call/zone.

No prob! Forgive the naivety as I'm not experienced with C++ a whole lot, but would it make sense to do a bit more heavy-lifting on that side so that stuff doesn't need to be cached on the C# side? As far as I understand the tracy docs, it appears that most of the frame marking, plotting, etc. all wanna be kept on the stack as much as possible. (I haven't read the Tracy source yet, so forgive the educated guess)
There is a library I've worked with called 'soundpipe' that is written in C and takes a "Create -> Init -> Compute -> Destroy" approach on the C side. Perhaps it would be easier to manage the memory if the C++ side was the one creating and destroying the contexts? That way, you could simply receive a pointer or something that you pass around until you need to pass it back to the C++ side to be destroyed.
If it's fast enough, you could potentially use SuppressGCTransition to reduce the overhead of P/Invoke calls too.

Thank you for starting the discussion! You are not wrong by suggesting to maybe do more memory management on unmanaged C/C++ side, but it will go beyond the scope of just a C# wrapper. This is always a challenge when working with managed / unmanaged languages. CString simply allows us to create a copy of a string that won't be subject to GC - ever. If memory allocation in hot path becomes a bottleneck in your app - you might want to consider linear allocator like smmalloc  https://github.com/nxrighthere/Smmalloc-CSharp ; however, for the majority of use cases heap based CString should be just fine and perform alright

No prob! Forgive the naivety as I'm not experienced with C++ a whole lot, but would it make sense to do a bit more heavy-lifting on that side so that stuff doesn't need to be cached on the C# side? As far as I understand the tracy docs, it appears that most of the frame marking, plotting, etc. all wanna be kept on the stack as much as possible. (I haven't read the Tracy source yet, so forgive the educated guess)
There is a library I've worked with called 'soundpipe' that is written in C and takes a "Create -> Init -> Compute -> Destroy" approach on the C side. Perhaps it would be easier to manage the memory if the C++ side was the one creating and destroying the contexts? That way, you could simply receive a pointer or something that you pass around until you need to pass it back to the C++ side to be destroyed.
If it's fast enough, you could potentially use SuppressGCTransition to reduce the overhead of P/Invoke calls too.

Thank you for starting the discussion! You are not wrong by suggesting to maybe do more memory management on unmanaged C/C++ side, but it will go beyond the scope of just a C# wrapper. This is always a challenge when working with managed / unmanaged languages. CString simply allows us to create a copy of a string that won't be subject to GC - ever. If memory allocation in hot path becomes a bottleneck in your app - you might want to consider linear allocator like smmalloc https://github.com/nxrighthere/Smmalloc-CSharp ; however, for the majority of use cases heap based CString should be just fine and perform alright

I can't promise anything, but would it be alright to pull-request some examples of ways to manage the CStrings like how you mentioned sometime? I could be wrong, but the current method of allocating memory via the marshal and deallocating it in rapid succession would start adding up fairly quickly, wouldn't it? Perhaps I have a skewed view of memory allocation, but it sticks out to me still.
Intuitively, my first thought is to create an unmanaged buffer pool - potentially with NativeMemory.AllignedAlloc so you can reap the benefits of cache locality - and keep "power pools" (saw this in bepu physics v2) that keep aligned buffers in powers of two and allocate them as demand requires.
Sorry if I'm being a little bit of a nut :p I could move this to a separate discussion issue since I'm diverging from the issue topic a bit

No prob! Forgive the naivety as I'm not experienced with C++ a whole lot, but would it make sense to do a bit more heavy-lifting on that side so that stuff doesn't need to be cached on the C# side? As far as I understand the tracy docs, it appears that most of the frame marking, plotting, etc. all wanna be kept on the stack as much as possible. (I haven't read the Tracy source yet, so forgive the educated guess)
There is a library I've worked with called 'soundpipe' that is written in C and takes a "Create -> Init -> Compute -> Destroy" approach on the C side. Perhaps it would be easier to manage the memory if the C++ side was the one creating and destroying the contexts? That way, you could simply receive a pointer or something that you pass around until you need to pass it back to the C++ side to be destroyed.
If it's fast enough, you could potentially use SuppressGCTransition to reduce the overhead of P/Invoke calls too.

Thank you for starting the discussion! You are not wrong by suggesting to maybe do more memory management on unmanaged C/C++ side, but it will go beyond the scope of just a C# wrapper. This is always a challenge when working with managed / unmanaged languages. CString simply allows us to create a copy of a string that won't be subject to GC - ever. If memory allocation in hot path becomes a bottleneck in your app - you might want to consider linear allocator like smmalloc https://github.com/nxrighthere/Smmalloc-CSharp ; however, for the majority of use cases heap based CString should be just fine and perform alright

I can't promise anything, but would it be alright to pull-request some examples of ways to manage the CStrings like how you mentioned sometime? I could be wrong, but the current method of allocating memory via the marshal and deallocating it in rapid succession would start adding up fairly quickly, wouldn't it? Perhaps I have a skewed view of memory allocation, but it sticks out to me still.
Intuitively, my first thought is to create an unmanaged buffer pool - potentially with NativeMemory.AllignedAlloc so you can reap the benefits of cache locality - and keep "power pools" (saw this in bepu physics v2) that keep aligned buffers in powers of two and allocate them as demand requires.
Sorry if I'm being a little bit of a nut :p I could move this to a separate discussion issue since I'm diverging from the issue topic a bit

Of course! Feel free to build on top of this https://github.com/clibequilibrium/Tracy-CSharp/pull/36 or wait till I close that up. The original Hello World example held the cached string pointers in a Dictionary

@BlueCyro
By all means tinker with the allocations. But I'd be careful to avoid premature optimizations.
The library is already extreamily performant. There is a cost to marshalling strings into unmanaged memory but it quickly disappears into the margins when the Profiler is deployed into a real world application. The C# runtime keeps memory available for unmanaged interop so I don't think quickly allocating and then freeing the CStrings is causing underlying memory thrashing (i.e. a problem a pool would alleviate). I would hesitate to muck about with byte arrays unless there's hard evidence that it would seriously improve the time to call into Tracy, which again is already extreamily fast.
The SuppressGCTransition attribute does look mildly interesting. It's likely the Tracy functions are shorter then the cost of the GC transition like the docs describe. The PInvoke class is generated though so you'd have to look at the generator options.
Optimizing the Tracy strings for cache locality seems nonsensical. We use Tracy to measure the performance of functions, we want them to use the cache so we can get an accurate measurement of their performance. Unless you are in an incredibly hot loop we wont actually be accessing these strings fast enough for cache locality to matter. Again, I'd be open to seeing evidence to the contrary.
If you do want to cache the source location strings, I wrote about that here. I'm not opposed to it, and it would totally eliminate the marshalling cost for most calls into BeginZone, but you need to make sure you're keying the data in the cache the same way that Tracy does so you don't end up mapping keys incorrectly.


Windows: AWS CLI not found because it's named aws.cmd

When running okta-awscli on Windows, it fails to find the aws cli executable to invoke after retrieving credentials.  It looks like a similar issue existed in the SAM cli and was resolved here:
https://github.com/awslabs/aws-sam-cli/pull/786

@jcaxmacher You need to install CLI from here. Then it will work.


Elegant fallback to incremental selection?

I love the ergonomics of treesitter's incremental selection by node, and textsubjects feels like a great drop-in enhancement for this. I like \ and <BS> for initiating/incrementing and decrementing, so my config is something like this (not exactly lazy.nvim):
return {
  plugins = {
    {
      'RRethy/nvim-treesitter-textsubjects',
      event = { 'BufReadPost', 'BufNewFile' },
    }
  },
  ts_opts = {
    textsubjects = {
      enable = true,
      prev_selection = '<BS>',
      keymaps = {
          ['\\'] = 'textsubjects-smart',
      },
    },
    incremental_selection = {
      enable = true,
      keymaps = {
        init_selection = 'g\\',
        node_incremental = 'g\\',
        scope_incremental = false,
        node_decremental = 'g<BS>',
      },
    },
  },
  config = function()
    vim.keymap.set('n', '\\', 'V\\', { remap = true })
  end,
}

I'm using g prefixes to retain access to incremental_selection because I work in an assortment of languages, some of which have textsubjects support and some which do not. I would love to be able to have textsubjects fallback to treesitter's increment/decrement when textsubject support is not available.
Are there any ways that this could be done more elegantly? I could see defining keymaps which conditionally call either textsubjects or incremental_selection based on whether the current treesitter node is supported by textsubjects or not, but this seems like it would be fragile with nested languages and I don't know how to query textsubjects to determine which languages support textsubjects-smart. Ideally, I think textsubjects would transparently fallback to treesitter node increments and retain the prev_selection functionality throughout.

This is not going to be implemented. Too much complexity for minimal return.


Sorry my fault!!!! This was nothing to download!

ERROR: Unable to extract media id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
dirk@Debian:~$ youtube-dl -U
Updating to version 2020.01.01 ...
Updated youtube-dl. Restart youtube-dl to use the new version.
dirk@Debian:~$ youtube-dl https://www.ardmediathek.de/ard/shows/Y3JpZDovL25kci5kZS8xNjIz/morddeutschland
[ARD:mediathek] morddeutschland: Downloading webpage
ERROR: Unable to extract media id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

enterich123 changed the title ARD mediathek Sorry my fault!!!! This was nothing to download! now


ELBv2 LBs names must be 32 char or shorter

Per the AWS docs, load balancers are also restricted to a name of <= 32 char
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-elasticloadbalancingv2-loadbalancer.html
The name of the load balancer. This name must be unique per region per account, can have a maximum of 32 characters, must contain only alphanumeric characters or hyphens, must not begin or end with a hyphen, and must not begin with "internal-".
In my specific case, I was using CF to make the both an NLB and target group and my target group was inheriting it's name from the NLB.
Down in the target group code, it failed name validation for length.
I'm suspicious there may be a secondary issue in the ELBv2 code as I would have expected a name too long error for the NLB.  Though, this may be an order of operations issue related to my edge case (where in CF, I set the TG name to the NLB name)

Coverage remained the same at 92.943% when pulling 7cc1afa25f5bdddf93da8dce2b57f9990603452e on abruehl:cf_limit_implicit_nlb_names_to_32_char into 2275c53b3e4c6d21bc3de6d2dee9a02bad7a3491 on spulec:master.

Coverage remained the same at 92.943% when pulling 7cc1afa25f5bdddf93da8dce2b57f9990603452e on abruehl:cf_limit_implicit_nlb_names_to_32_char into 2275c53b3e4c6d21bc3de6d2dee9a02bad7a3491 on spulec:master.

Coverage remained the same at 92.943% when pulling 7cc1afa25f5bdddf93da8dce2b57f9990603452e on abruehl:cf_limit_implicit_nlb_names_to_32_char into 2275c53b3e4c6d21bc3de6d2dee9a02bad7a3491 on spulec:master.

thanks!


MM-44849 - feature discovery pages show start trial button after trial completed

Summary
This PR adds the logic for opening the upgrade modal once the trial have completed showing a call to action to upgrade from the feature discovery pages in the system console
Ticket Link
https://mattermost.atlassian.net/browse/MM-44849
Related Pull Requests
n/a
Screenshots
https://user-images.githubusercontent.com/10082627/172846978-b7f6dc9e-9bee-46b6-b6eb-f4da62c94313.mp4
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/205474


(kubernetes-cli) Add new package maintainer

Description
Add a new maintainer for the kubernetes-cli package.
Motivation and Context
Add new package maintainer and update to reflect current contribution
guidelines and package details.  Specific changes made:

Refresh README to reflect current product description
Update Markdown formatting to reflect best practice
Update authors and copyright to reflect Kubernetes Authors
Include copyright handling for Nuspec updates
Powershell script formatting and style changes

chocolatey-community/chocolatey-package-requests/issues/1003
How Has this Been Tested?
The updates to the package do not reflect any functional change. Package has been tested by:

Validating the package using cnc
Resetting version numbers to force a package update followed by:
i) Add and remove the built package in the Chocolatey test environment
ii) Inspection of the package to ensure only expected changes are present in updated files

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[x] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

@dgalbraith Do you still want to merge this in? Can you address the issues raised by @AdmiringWorm ?

Closing as requested changes have not been made.


How to use a custom image for validation

Hello,
I tried using a custom image for validation since I'm training it on  a custom style i uploaded my val image on hub as the mountain.png but it always gives me error for unidentified also for mountain.png it shows validation summary on wandb but for my val image it shows nothing.
Do i need to change something somewhere also how does it compare the val images for loss do  i need to put the style image of original image somewhere

You are asking for debugging help. I would suggesting debugging things on your own here.

I'm asking have you ever used a custom val image ? or a different prompt  while training

I have. You need to provide a direct downloadable link to the validation image and it should work.


SQLite connection using WAL, which allows parallel inserting

Having some issues, will review and then re-send pull request

I was just coming here to request WAL. Excellent!


Remove Configuration and CLI of Alluxio table

What changes are proposed in this pull request?
Remove Configuration and CLI of Alluxio table

Question for myself:
remove the plugins?

client/presto/plugins/prestodb-hive-alluxio-227/
client/presto/plugins/presto-hive-alluxio-319/ ${PRESTO_HOME}/plugin/hive-alluxio/
remove from the tarball building process?

alluxio-bot, merge this please


Updated dependencies for 3.x

Updated dependencies to fix security issues for the 3.x branch - still keeps compatibility with node 4.  Based off of @compwright's #508, but without the eslint changes.
#508 will still be merged into the 4.x release, when we drop support for node 4.x.

Based on prior discussion I agree with not swapping to eslint, but as such I don't think we should merge all the styling changes.
I think #511 is more what we need to ensure we maintain deps (I closed that before I realised we'd have to drop v4 support)

@thomseddon Removed the styling changes, we'll reintroduce for 4.x when we change to eslint.  Can you review and approve this?


Unable to use AD methods using go client

Problem: Using the go client for automation doesn't provide for the methods to use AD

@leodotcloud and I discussed an alternative approach of just making this call using net/http and json libraries. Murali, if you are going to go with that approach, can you close this issue. Otherwise, let me know.

Sample code:
func Login(url, username, password string ) error {
	loginPostURL := url + "/v3-public/activeDirectoryProviders/activedirectory?action=login"
	payload := map[string]interface{}{
		"username": username,
		"password": password,
		"description" : "example description",
		"ttl": 4 * 60 * 60 * 1000,
	}

	marshaledBytes, err := json.Marshal(payload)
	if err != nil {
		log.Fatalf("%v", err)
	}

	resp, err := http.Post(loginPostURL,
		"application/json",
		bytes.NewBuffer(marshaledBytes),
	)
	if err != nil {
		return err
	}
	defer resp.Body.Close()

	var result map[string]interface{}
	json.NewDecoder(resp.Body).Decode(&result)
	if resp.StatusCode != http.StatusCreated {
		return fmt.Errorf("%v", result["message"])
	}

	return nil
}

Alternate approach worked, hence closing the issue.


Stop using org.embulk.spi.unit.ToStringMap

The Embulk core is going to deprecate org.embulk.spi.unit.ToStringMap. Mapping into Map<String, String> has almost the same effect, only except for handling null as tested in: https://github.com/embulk/embulk/pull/1267
But, in this use-case of ToStringMap in embulk-filter-calcite, we can ignore null. Then just replacing to Map<String, String> here.

Thanks!


Adds support for Mutual Information metric in LVT

This updates adds the support for mutual information metric in LVT.
Mutual Information values are calculated as:
I(X;Y)=Sum_x Sum_y PXY(x,y)log [PXY(x,y)/(PX(x)PY(y))]
Note that time series calculations are not enabled for this metric.
Also this metric only should be used to analyze one variable at a time.
Resolves #497

An example test case is available at  /discover/nobackup/svkumar/LISF_testing/497

Hi @sujayvkumar I successfully compiled and ran LVT based on your testcase. While the output of my run is identical to yours, the log contains this warning before successfully completing:
[WARN] LIS file ./LIS_OUTPUT/SURFACEMODEL/200202/LIS_HIST_200202010000.d01.nc does not exist
The warning can be resolved by altering lines 18 and 19 in lvt.config to end the run on 1/31 rather than 2/1. LVT is looking for data in ./LIS_OUTPUT/SURFACEMODEL/200202/ which does not exist, hence the error.

Hi @sujayvkumar. Jim and Eric looked over this PR with me and pointed out a few more things to bring to your attention before we can merge this. Please:

Document "Mutual information:" in lvt/configs/lvt.config.adoc

See line 231 in lvt/core/LVT_readMetricsAttributes.F90

Remove debugging print statements:

See lines 617-619, 630-635 in lvt/metrics/LVT_MutualInformationMod.F90

Jim explained that the warning I described in my previous comment is acceptable and may not need repair if you're OK with it.
Thanks,
B

Thanks, Sujay.

Configure, compile, and run results:

[x] SLES11/Intel 18 - PASS
[x] SLES11/GNU 4 - PASS

@jvgeiger @emkemp, let me know if this looks good to you and I will merge.


Spectrum class should check that freq and data are same dimension on creation

From @ehsteve on July 18, 2015 17:33
Copied from original issue: sunpy/sunpy#1501

From @sudk1896 on October 21, 2015 19:26
Hi,
I went through the Spectrum class. When you say frequency and data axis dimensions should be same, you mean you want the lengths of both arrays (frequency and data) to be same, If not  then you want to raise an error/exception. Correct me If I am wrong ?
I am assuming the init procedure is creating the data and frequency array in the class. What do you want to do, when the dimensions aren't same ?
I would like to work on this, seems like the ideal beginner task to me.

A lot.... not just in here, but in the whole library. This will be stuck for a while.

Deleted Spectrogram class and started again.


Feature request: Option to only show the tiles you actually touch when running

https://youtu.be/DX8lN3r3ALw?t=78

I came here to ask for the exact same thing, @ShaunDreclin . If @Milo-Piazza is still willing to maintain this repository, I can fork and create a pull request for this feature. If they're not, I could just fork it and look into submiting it as a new plugin.


[ iOS] Videos from assets-library are not played

I have a video uri from the file system like this: assets-library://asset/asset.mp4?id=7ED0862F-75A0-4EDE-8096-0F5F695045C2&ext=mp4 and I'm sending it to the video component like source={{ uri: this.props.source }}. This works fine on Android because the path is totally different, but on iOS I don't get the videos to play and I really need it. By checking the source code it seems that this type of path is allowed, so i don't know what's going on
let uri = source.uri || '';
    if (uri && uri.match(/^\//)) {
      uri = `file://${uri}`;
    }

    const isNetwork = !!(uri && uri.match(/^https?:/));
    const isAsset = !!(uri && uri.match(/^(assets-library|file|content|ms-appx|ms-appdata):/));

I'm experiencing the same issues on iOS. Local file system uri like file://private/var/mobile/Containers/Data/Application/76FC3720-D48A-420B-AB4D-ECC95BBC143C/tmp/7FAC69A0-7469-4B51-BE21-1347A8C18B2E

+1. ios

Did you solve it?

+1. ios too

Yup..same issue here. Started happening after upgrading my React Native version to the latest release.

FYI, downgrading to 3.0 worked for me.

Downground to 3.0 still doesn't work for me.
This one does:
https://github.com/react-native-community/react-native-video/issues/817#issuecomment-357767683
And it's actually very fast.

Copying a large file is not always workable.  Downgrading to 3.0 seems to work for me but at some point one will have to move forward.

any update on this?

Any updates on this issue?

Any update on this issue please

iirc this should work o latest version


Bring in ACK common controller files

Description of changes:
Aligns to the readme/docs used from other ACK service controller
repositories and a generator.yaml that ignores all resources in MemoryDB.
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.

/approve

/lgtm


Update Helm release rancher to v2.7.4

This PR contains the following updates:

Package
Update
Change

rancher (source)
minor
2.6.9 -> 2.7.4

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (2.7.4). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Completeness for CT Structure Semantics of STT

Show that for any CT structure and interpretation of base types and function symbols, the soundness interpretation is unique up to unique natural isomorphism.

Closing this for now. We should revisit after doing something more traditional for (bi)CCCs


Focus on tab switch

If having a conversation with someone with Android Messages, and you Tab to another window and then back, it would be very helpful if we could force a focus back to text message control, it loses its focus at the moment requiring a click.

Ah ha, I can confirm this happening. This will be a tad more involved than the other issues because this involves poking around the Webview where Messages for web itself lives, but this is a good first step into interacting with the Webview, so I'll definitely look into this.


Review and test options for multi workflow pipelines

What are the options to create several workflows in the repo, to allow

different levels of test son master, dev or test branches (e.g. more detailed tests via Test, simple via Master)
to allow some workflows to only run after previous workflows have completed
to all full CI / CD
to publish to package after successful build to master, or on schedule

https://docs.github.com/en/actions/reference/events-that-trigger-workflows#triggering-new-workflows-using-a-personal-access-token
If you would like to trigger a workflow from a workflow run, you can trigger the event using a personal access token.
You'll need to create a personal access token and store it as a secret.
To minimize your GitHub Actions usage costs, ensure that you don't create recursive or unintended workflow runs.
For more information, see "Creating and storing encrypted secrets."

Tasks

create personal access token, store as secret =

https://github.com/asarkar/build-time-tracker/blob/master/.github/workflows/publish.yml

workflows and events
You can configure your workflows to run when specific activity on GitHub happens, at a scheduled time, or when an event outside of GitHub occurs.
https://docs.github.com/en/actions/reference/events-that-trigger-workflows#workflow_dispatch
"The workflow files for that commit SHA and Git ref are inspected,
and a new workflow run is triggered for any workflows that have
on: values that match the triggering event."


Rearranging partner logos

Some logos are squarish and some are wide and there appears to be a fixed width for each logo. This means that wide logos together look a bit crowded. If we strung wide and square logos together in alternating pairs, it might look nicer.

Looks better to me, @sofer what do you think?

@sofer if you're satisfied with this work please close this issue, thanks!


Test: accessible view chat codeblock actions

Refs #199119

[ ] Windows @joyceerhl
[ ] macOS @joaomoreno
[ ] anyOS @justschen

Complexity: 3
Create Issue

Screen reader users can inspect panel chat responses in the accessible view by focusing the chat response and using alt+f2.
Now, users can apply code block actions from within the accessible view when their cursor is within a code block. These actions and keybindings are discoverable in the accessibility help dialog via alt+f1.
To verify:

Run a chat request in the panel chat that yields one or more code blocks
Use alt+f2 to open the accessible view
Use alt+f1 to see the commands that can be taken, the highlighted ones below are new (note https://github.com/microsoft/vscode/issues/207638)
Configure keybindings for these and try the default ones, if any, and verify they work as expected.

@meganrogge after alt+f2 and alt+f1 (on mac fn + alt + f2/f1), I see the two accessibility help windows, and the commands to apply code block actions. but when trying  to insert, I get these errors:
for ctrl + enter:

for ctrl + option + enter, no notification in vs code, but in Voiceover same error:

wondering if there's an issue with my setup?

@justschen a gif would help

screen recording here!
https://github.com/microsoft/vscode/assets/54879025/97f70efb-9115-4871-a935-f6cdc3c0a53a

Oh I see, thanks for the gif. You need to be in the accessible view, not the accessibility help dialog, when using those keybindings. The help dialog simply describes the actions which can be taken when in the accessible view. @justschen

@meganrogge ah makes sense. running into the same thing in the accessible view popup:
https://github.com/microsoft/vscode/assets/54879025/7e9160b2-07a9-471a-8264-f6d809f27af2

I can reproduce, thanks. related https://github.com/microsoft/vscode/issues/205401

didn't realize it was a competing keybinding, but works nicely besides that 👍


Added support for Try-Functions

There's a lot of changes here, but the major ones to focus on are:

try-now.component
app.component
app-node
arm-try.service

I merged this from my machine.


(3.5) What is the camera offline_obstacle_pipeline for? And how do you use it?

What is the camera offline_obstacle_pipeline for?
https://github.com/ApolloAuto/apollo/blob/master/modules/perception/camera/tools/offline/offline_obstacle_pipeline.cc
We have been compiling it and trying to run it, but have not yet been successful.  It appears it has a visualizer for drawing bounding boxes.   The perception/lidar/tools has a ascript (run_offline_lidar_obstacle_perception.sh)   but no script exits for camera.
What is the purpose of this tool? and is there steps on how to use it?
In Apollo 2.5/3.0 perception... there was a script: perception_offline_visualizer.sh,  which would tap run camera perception but also display a pop up window with opengl and show the camera image with bounding boxes overlayed.   This tool was useful for debugging and making sure the camera was working correctly.   Is there a tool or a way to do something like this with Apollo 3.5?
Thank you,

@snuffysasa Were you able to find more information on this? I think currently, only point cloud data is being used to feed useful information into all the downstream modules and the camera based obstacle detection doesn't happen unless the dag file is run manually.

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Move tests to public test framework

Hello Chun,
Can you please do a quick review of the tests. All the tests have already gone through one round on review by Vicky and they run clean.
There is on test that fails (Functional/flatten_operators/schema_changes/filter15.q) which is unrelated to any of my changes.
Here is the link to pre commit run - http://<IP_ADDRESS>:8080/job/Framework Regression/94/consoleFull
Thanks,
Khurram

+1

Thanks Chun, I have merged the tests.


ZenHub Export of Data Sets/Issue Tickets

Research ZenHub's current offerings for Issue Tickets visualization capabilities.
Since ZenHub uses Boards (Kanban-style) and Reports to visualize the workflow of issue tickets & PR directly within GitHub, the goal is to:

Identity and map-out the scope of data available
Format(s) the data can be exported based
What plug-ins are currently available to make this process seamless
Have the export data set available on CSV format
Moving forward, to identify & map-out key fields needed for JIRA Pilot project that will be used as target data source

Zenhub not functional to support data migration for the following reasons:

Very limited import/export functionality
Does not support RESTAPI any longer; only connection allowed is via GraphQL with literally no documentation.
Major issues with Access Permissions, Workspace Complexity and more...
Decided to pursue via GitHub & scripting and was able to migrate issue tickets with serval filters that could be customized further based on user request. As a sample, I have attached 2 csv dumps for reference that goes back to 3 years.
Only suggestion I have is for a validation via a different user to make sure issue numbers match exactly on the GUI side. @Zenhub Workspace.

CAMP_UI_GitHub Issues.csv
ECMPS_UI_GitHub Issues.csv

We need an export of:

all repos

all time

open only

ticket #

title

label

created by

description

comments

Completed

@spetros-do , just curious, we are trying to export ZenHub data to csv. "Decided to pursue via GitHub & scripting and was able to migrate issue tickets with serval filters that could be customized further based on user request." Did you do this utilizing GitHub actions or the like, or directly in GH which doesn't even look at ZenHub, but just the issues in GH?


Updated experiment code to include "null" values for users not enrolled in the experiment

Requirements:

Add a null value for experiment bucketing, in addition to "true" and "false". See example below.

For experiments that are less than 50/50 (like 10/10), can should bucket in the following way:

10% who are in control have false

10% who are in test have true

10% who are not in the experiment at all have null

That way enrollment is still balanced between control and test, and then we exclude everyone that wasn't meant to be in the test (the null group)
This will help with clean data analyses for experiments

CC: @philipp-spiess

We don't seem to be working on this so moving this back to the backlog


Putting model inside output_path folder does not work

I have been trying to run the example app and I was getting the error that the model.bin file is not found in /data/local/tmp/llm/. It's because I had put the model inside the output_path folder. Pushing just the model.bin in the device is working for me. In the Readme the instructions are to have the model.bin file inside output_path folder. So maybe we can update the Readme?
Device Used: Samsung Galaxy A52s
Platform: Android

Yep, looks like there is mistke, will fix it today, thank you for highlighting

@AliAkberAakash  pls, i'm getting the same error
could you please explain better what you've done??

I updated the manual, sorry for the delay, please take a look


Add support for ai.txt

Just like Hugo supports generating a robots.txt, it would be great Hugo could also support generating an ai.txt file.
Info: https://spawning.substack.com/p/aitxt-a-new-way-for-websites-to-set

It's not hard to generate a ai.txt to your site (there are several ways to do this, the forum at https://discourse.gohugo.io/ is a better place to ask for that). We do have some "first class" support for robots.txt, but that is backed by a simple output format definition:
https://app.slack.com/client/T015RFZ61CM/C017GG5DUR1#:~:text=50 bagger.-,CVNA,-5
We could consider adding this to Hugo's default setup if it's "popular enough".

I understand that it's easy to generate this file. Adding this to Hugo will ensure that more people will start using it.


versioned_session not working with Flask and SQL Alchemy

I tried solutions presented in #175 and some other similar but nothing seems to be working correctly. My schema has version column and [mytable]_history table created, but changes don't get saved in it.
Based on this answer, expected it would work with:
db = SQLAlchemy(app)
versioned_session(db.session)

What's the correct way of using the versioning (if it's still supported)?

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


Outdated Repository About information

The About section of the repository is outdated and needs to be updated as per the new repository and codebase restructuring to huggingface.js
Current:

Suggested:
Documentation and information hosted on the Hugging Face website.
Related to #1140

good catch! what do you suggest?

i've kept things simple for now:


[webview_flutter] Test: wait for page to load before checking the content

https://github.com/flutter/flutter/issues/31510 seems to exist. Using it to make the header loading test not flake

Let's remove the skip from the test and release as a new version if the bots are happy


Add aspect ratio, scale and additional functionality

+Set aspect ratio for bounding box
+Ability to move bounding box by dragging from inside
+Display scale for printed size
+Display feature pixel size for printed size
+Display on-the-ground distance of bounding box

Also includes /dist minified files.
To test in TileMill, update your package.json to:
`"wax": "git://github.com/tilemill-project/wax.git",

I checked out csytsma-export_scale and installed. When I run it, I get very weird behavior in the GUI with things generally not working in either Chrome or Safari. Is that the branch I should be trying out?

On Apr 19, 2019, at 5:55 PM, csytsma<EMAIL_ADDRESS>wrote:
Also includes /dist minified files.
To test in TileMill, update your package.json to:
`"wax": "git://github.com/tilemill-project/wax.git",
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/tilemill-project/wax/pull/1#issuecomment-485045926, or mute the thread https://github.com/notifications/unsubscribe-auth/ALCFZLBQJKU6LNGOTP22KT3PRJSY5ANCNFSM4HHIS5IA.

The new wax dist libraries somehow didn't pick up the new changes. I just pushed a new change to wax with the new distribution libraries.  Remove your node_modules/wax and npm install again.

On Apr 19, 2019, at 6:23 PM, Tim Potter<EMAIL_ADDRESS>wrote:
I checked out csytsma-export_scale and installed. When I run it, I get very weird behavior in the GUI with things generally not working in either Chrome or Safari. Is that the branch I should be trying out?

On Apr 19, 2019, at 5:55 PM, csytsma<EMAIL_ADDRESS>wrote:
Also includes /dist minified files.
To test in TileMill, update your package.json to:
`"wax": "git://github.com/tilemill-project/wax.git",
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/tilemill-project/wax/pull/1#issuecomment-485045926, or mute the thread https://github.com/notifications/unsubscribe-auth/ALCFZLBQJKU6LNGOTP22KT3PRJSY5ANCNFSM4HHIS5IA.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tilemill-project/wax/pull/1#issuecomment-485047823, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA7ORC5VVFDLJTUCZOCTCDPRJWAPANCNFSM4HHIS5IA.


Address linkcheck errors

NXP link in docs requires NXP account, adding link to set of links to ignore.
QA: ran clean linkcheck build, finished without error
No related tickets/issues, quick fix
PR Template and Checklist
Please complete as much as possible to speed up the reviewing process.
Readiness and adding reviewers as appropriate is required.
All PRs should be reviewed by a technical writer/documentation team and a peer.
If effecting customers—which is a majority of content changes—a member of Customer Success must also review.
Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist

[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Docs for fe161fc0cd19f56d7e668da9f37a389e182f7578 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2797/docs/artifacts/html/index.html


Add python IM event client code

Problem
Add python IM event client support
Todo: Add python API to read attribute and event at the same time.
Change overview
See above
Testing
Compilation
Add E2E python tests since IM server integration( PR#12059), and currently we cannot retrieve events.

fast track: pr created and approved by domain owners,  has been up  for a long time, touches python (i.e. not functionality) with changes that  are non-python not behavioral


Link for GitHub is incorrect in section

Describe the bug
GitHub repository link is incorrect
To Reproduce
Click the GitHub link in https://docs.aws.amazon.com/solutions/latest/audience-uploader-from-aws-clean-rooms/architecture-overview.html#aws-secrets-manager
Expected behavior
Not broken
Please complete the following information about the solution:

[ ] Version: [v1.0.0]

To get the version of the solution, you can look at the description of the created CloudFormation stack. For example, "(SOWXYZ) Solution Name v0.0.1[...]".

[ ] Region: [e.g. us-east-1] NA
[ ] Was the solution modified from the version published on this repository? NA
[ ] If the answer to the previous question was yes, are the changes available on GitHub? NA
[ ] Have you checked your service quotas for the sevices this solution uses? NA
[ ] Were there any errors in the CloudWatch Logs? NA

Screenshots
If applicable, add screenshots to help explain your problem (please DO NOT include sensitive information).
Check GitHub link in https://docs.aws.amazon.com/solutions/latest/audience-uploader-from-aws-clean-rooms/architecture-overview.html#aws-secrets-manager
Additional context
Add any other context about the problem here.

I see the link to the Github repository is incorrect.  Here is the page where I see the incorrect URL. https://docs.aws.amazon.com/solutions/latest/audience-uploader-from-aws-clean-rooms/architecture-overview.html


Create dataloader for block

Features

Unify query names to use ID
Created batch query for block object
Create batch graphql query for block object

## Note: Chain healthiness uses some of the block queries so i will probably rebase either of the PRs after one of them is merged. Done
refs #52

@chihimng updated :adore2x:


HELP: Resource constraint in JopShop like problem

Hi,
I'm trying to solve a jobShop like problem for a factory floor operations scheduling. I'm using CPSolver.
I have a task that only can run at a time if it have a resource available for it's use. So if I have two tasks of this kind, only one can run at same time. I saw there is a example using something similar in this link:
https://developers.google.com/optimization/routing/cvrptw_resources
However, I'm using a different solver. How can I achieve this kind of behaviour?

https://github.com/google/or-tools/blob/stable/ortools/sat/doc/scheduling.md
and
https://github.com/google/or-tools/blob/stable/examples/python/flexible_job_shop_sat.py
Do not use the CPSolver, use the CP-SAT one.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 26 avr. 2019 à 21:46, gustavo-barros<EMAIL_ADDRESS>a
écrit :

Hi,
I'm trying to solve a jobShop like problem for a factory floor operations
scheduling. I'm using CPSolver.
I have a task that only can run at a time if it have a resource available
for it's use. So if I have two tasks of this kind, only one can run at same
time. I saw there is a example using something similar in this link:
https://developers.google.com/optimization/routing/cvrptw_resources
However, I'm using a different solver. How can I achieve this kind of
behaviour?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1225, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3IJLGIVHBFZGHTOPATPSNLX5ANCNFSM4HIZ3U5Q
.


Modal not working on mobile.

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
I am using the modal bootstrap class. Copied the code off the bootstrap website. The code works on desktop and when i make my screen resolution to a mobile's one it still works. However, after testing on my mobile it seems the tint it adds when the modal pops up, comes ontop of the modal. Meaning you can edit/click off it only by refreshing the page. This has only been touched previously, but that was while using bootstrap 4.0.0.
I have tried editing and removing classes to help with this, but nothing worked, not sure of this is a issue allready discussesd in the community as ive not seen much on it.
In the attached images you can see on desktop the dark tint is behind the content, but on mobile is the other way around.
https://github.com/twbs/bootstrap/issues/21546
Here is a previous "work around", but that is unstable and not good option. However this issue has been traced back since bootstrap 3.0.0
Reduced test cases

What operating system(s) are you seeing the problem on?
Windows, iOS
What browser(s) are you seeing the problem on?
Chrome, Safari, Firefox, Microsoft Edge, Opera
What version of Bootstrap are you using?
5.2.2

Closing as stale.


Single characters get "cleaned" away

single_character = 'a'
result = PragmaticSegmenter::Segmenter.new(text: single_character).segment
result = []

Thanks, should be fixed now (https://github.com/diasks2/pragmatic_segmenter/commit/4ded3f98f33c0f3b63661fe72a104f594fbc9e22). Let me know if you run into any other issues.


Use standard WP function remove_all_filters()

Use standard WP function remove_all_filters()
Fix for https://github.com/szepeviktor/fix-w3tc/issues/56

This looks correct


Installation failed:  UnicodeDecodeError

Hello, I had a problem when I try to install the package. The following  is my environment.

Python 3.5.2 :: Anaconda custom (64-bit)
Windows 10
pipenv, version 11.0.1

Using pipenv install requests-html to install the package. The error is
Installing requests-html…
Collecting requests-html
  Using cached requests_html-0.6.6-py2.py3-none-any.whl
Collecting bs4 (from requests-html)
  Downloading bs4-0.0.1.tar.gz
Collecting pyppeteer (from requests-html)
  Using cached pyppeteer-0.0.12.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\secsi\AppData\Local\Temp\pip-build-e_70uy7c\pyppeteer\setup.py", line 27, in <module>
        compile_files(in_dir, out_dir, target)
      File "c:\users\secsi\.virtualenvs\lianjia-bj-ijjpmtde\lib\site-packages\py_backwards\compiler.py", line 85, in compile_files
        dependencies.update(_compile_file(paths, target))
      File "c:\users\secsi\.virtualenvs\lianjia-bj-ijjpmtde\lib\site-packages\py_backwards\compiler.py", line 57, in _compile_file
        code = f.read()
    UnicodeDecodeError: 'gbk' codec can't decode byte 0xb6 in position 2147: illegal multibyte sequence

    ----------------------------------------

Error:  An error occurred while installing requests-html!
Command "python setup.py egg_info" failed with error code 1 in C:\Users\secsi\AppData\Local\Temp\pip-build-e_70uy7c\pyppeteer\

this appears to be a problem with pyppeteer, not this library.


Redis read-only replica support

This PR introduces a Redis replica server next to an Redis server that is assumed to be in read-only mode.
If configured, Redis read commands will go there while write commands will go to the non-replica server.
All connectivity options are duplicated for the replica server.
Redis tests are not yet implemented.

Hi @madnuttah, I am not sure what the workaround is in that link but I see no relation with this feature. This feature adds support for Unbound to issue all read commands to a Redis replica server, offloading most of the of the work from the main Redis server which will only be issued write commands.
Does that answer your question?

Thanks for making it clear!

There was some time investment to bring this up to speed with recent changes so I think it is time to merge to avoid the same time investment in the future.
Since this is a PR already, I'll wait for a review before squash merging.


create DownstreamPackage controller

What this PR does:

adds a DownstreamPackage controller as discussed in https://github.com/GoogleContainerTools/kpt/issues/3488
implements creation and updating of deployment packages

What this PR does not do (will be coming in a followup PR):

Implement pruning of deployment packages that are no longer needed
Implement the proposed AdoptionPolicy field, which will allow the user to choose whether the controller adopts existing package revisions or ignore them

Regardless we need to be prepared for situations where the automatic update fails, since there might be conflicts that can't be automatically resolved

Yeah, I think we'll need to raise an error in the package revision status.


Update Quickstart: Instantiation

https://developer.tbd.website/docs/web5/quickstart/#1-instantiate-web5
Need to update the description of the title and description of Step 1: Instantiate Web5
It now instantiates Web5 and also provides a class instance representing the decentralized identifier that was created or attained connection to

https://github.com/TBD54566975/developer.tbd.website/issues/538


g++-5: internal compiler error

When compiling following program (saved as test.cc)
#include <utility>

int main() {
  std::make_index_sequence<100000> a;
  return 0;
}

with command g++-5 -std=c++14 -ftemplate-depth=200000 test.cc, I got
g++-5: internal compiler error: Segmentation fault: 11 (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <https://github.com/Homebrew/homebrew-versions/issues> for instructions.

Clang compiles this program without issues. I suspect this was an issue of gcc, but since it is directing me to here, I decided to post it here first.
Gist log is [here] (https://gist.github.com/a17d4e698391af43867f).

This is an [gcc issue] (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68642) and will be fixed in the 5.3 release.


Add a license

MIT probably

But GPL though.

Meh, MIT is more permissive. I'm not too strong either way.
On Sun, Jul 31, 2016 at 9:07 PM Mitchell Manguno<EMAIL_ADDRESS>wrote:

But GPL though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/classrank/Grouch/issues/13#issuecomment-236469074,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEz4AJE1VRdcCEUbkA5OUFsa-WhT7FpBks5qbUbegaJpZM4JY_mC
.

This seems like a significant-enough piece of software that GPL'ing it could persuade some other projects to GPL as well to get access to it.
Also, instead of MIT, why not do Apache? All the benefits of MIT permissiveness with some additional protections:

This [MIT license] is a fine license for a small program. A larger program usually ought to be copyleft; but if you are set on a lax permissive license for one, we recommend the Apache 2.0 license since it protects users from patent treachery. — FSF

I don't think we care too much about patents, but I'm less opposed to
apache than MIT. Really, I favor MIT because its simple and
straightforward, and there's very little legalspeak and no restriction, so
one doesn't need to worry about copyleft or really anything, they can just
use the software.
As an aside, with other maintainer approval, I'm interested in moving
Grouch to another organization, perhaps "GT-muppets" or somesuch and
developing some additional tools in the same vein as Grouch and
coursesat.tech.
On Sun, Jul 31, 2016 at 9:59 PM Mitchell Manguno<EMAIL_ADDRESS>wrote:

This seems like a significant-enough piece of software that GPL'ing it
could persuade some other projects to GPL as well to get access to it.
Also, instead of MIT, why not do Apache? All the benefits of MIT
permissiveness with some additional protections:
This [MIT license] is a fine license for a small program. A larger program
usually ought to be copyleft; but if you are set on a lax permissive
license for one, we recommend the Apache 2.0 license since it protects
users from patent treachery. — FSF
https://www.gnu.org/licenses/license-list.en.html#X11License
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/classrank/Grouch/issues/13#issuecomment-236473375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEz4AMFM_Qj9MebfquQSNtqtp2QZQTD0ks5qbVMXgaJpZM4JY_mC
.

That's fair, since we're not really dealing with patenting, nor are we interested in copylefting. MIT's fine then.
I'd be fine with moving it to another organization. Makes more sense for it to be in a general GT-tool organization rather than ClassRank.

Aight, and yeah I'll get to that second part closer to school, but I'm
probably going to implement a couple of tools and organize them into a
single org:

Grouch
coursesat.tech (catalog)
a course critique?
a nice alternative to degreeworks
a csp schedule creator

Not sure if there are any others that should be added to that list, but
that seems like a good start, and each would need to be its own project,
but could be hosted under one roof.
On Sun, Jul 31, 2016 at 10:23 PM Mitchell Manguno<EMAIL_ADDRESS>wrote:

That's fair, since we're not really dealing with patenting, nor are we
interested in copylefting. MIT's fine then.
I'd be fine with moving it to another organization. Makes more sense for
it to be in a general GT-tool organization rather than ClassRank.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/classrank/Grouch/issues/13#issuecomment-236475619,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEz4ABaoUB2xURDZrZG7FOAi9Oyv0374ks5qbVi5gaJpZM4JY_mC
.

Alright. I'll make a new issue for moving this out of the Classrank organization. We can close this one when the MIT license is updated.

but why

I recommend against GPL as it's too restricting.

It's already been decided to use the MIT license.

wait we had one the whole time.


Wrap deploy-accepted event stream event

This PR changes how an accepted deploy is represented in JSON on the event stream.  Previously, only the contents of the deploy were output.  This now wraps the contents in a DeployAccepted type, making it consistent with the style of other SSEs.
Closes #1743.

bors merge


NixOS remote builder with cross compilation

Issue description
Is there an established method of using a remote builder with a cross compiler?
I am attempting to run NixOS on a Raspberry Pi using the armv7 architecture, but there is very little software built for it in Hydra so much of the configuration has to be compiled. This compilation process has now been running for three days.
Is there a way to run a builder daemon on my x86_64 system to cross compile to ARM  more quickly?

nix-build -E "(import <nixpkgs> {}).pkgsCross.armv7l-hf-multiplatform.perl" (on x86_64) should compile for armv7

Yeah you can have a NixOS config that is cross compiled. The trick is to set localSystem = { system = "x86_64-linux"; }; so that rebuilds still happen on your x86_64 machine. Something like this should work:
{ pkgs, lib, config, ... }:

# Build with:
#
# nix build '((import <nixpkgs/nixos/lib/eval-config.nix> { modules = [ ./configuration.nix ]; }).config.system.build.sdImage)'

{
  imports = [
    <nixpkgs/nixos/modules/installer/cd-dvd/sd-image-raspberrypi.nix>
  ];

  boot.loader.raspberryPi.enable = true;
  boot.loader.grub.enable = false;
  boot.kernelPackages = pkgs.linuxPackages_rpi;

  # Setup cross compilation.
  nixpkgs = {
    crossSystem = lib.systems.elaborate lib.systems.examples.raspberryPi;
    localSystem = { system = "x86_64-linux"; };
  };

  nix.distributedBuilds = true;
  nix.buildMachines = [ {
    hostName = "linux64.example.org";
    sshUser = "buildfarm";
    sshKey = "/root/.ssh/id_buildfarm";
    system = "x86_64-linux";
  } ];

  # Enable tools for remote installation.
  services.openssh.enable = true;
  networking.wireless.enable = true;
  security.sudo.enable = true;
  hardware.enableRedistributableFirmware = true;

  # Setup headless mode.
  systemd.enableEmergencyMode = false;
  documentation.enable = false;
  services.nixosManual.showManual = lib.mkForce false;

}

You need to copy it to the installation afterwards and setup the ssh keys for remote buildings. This means that every time you run nixos-rebuild you will build on your x86_64 machine. Unfortunately, you won't be able to build natively because Nix doesn't recognize that x86_64 -> armv7l compilation is the same thing as armv7l -> armv7l compilation.

Maybe it's still too early for it but otherwise I think it would be good to add a section on cross-compiling NixOS to the NixOS manual.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/remote-cross-compiled-builds-for-macos-darwin-from-nixos-remote/5963/1

Still important to me.

for reference, I ended up moving to the Raspberry Pi 4 platform and using aarch64 as both the build and host architecture. I also needed to add extra-platforms = aarch64-linux arm-linux to /etc/nix/nix.conf. this meant that nix fetched prebuilt binaries from Hydra and I did not have to cross-compile anything.

this sounds useful for armv7 phones aswell


Fix direct object reference

Someone managed to register for a club with registration_method set to hidden.

Coverage increased (+0.04%) to 85.281% when pulling 17b16fb83f1f79aa72a5476fc27c599db54fcf09 on fix_direct_object_reference into b9f38902035268207d966da88da94d08a3de00fc on master.

Coverage increased (+0.08%) to 85.32% when pulling dfe968f161b5cf6f8256acb2521f6bdb46897429 on fix_direct_object_reference into b9f38902035268207d966da88da94d08a3de00fc on master.


Prevent Node Settings messages from appearing on every entrance to the screen

Connected to https://github.com/bithyve/hexa/issues/2442

Closed this as I don't think there's an issue to address here.

Closed this as I don't think there's an issue to address here.


Feature Requests: Highlight "today" and Outlook-style "week" view

Really love this card, and am currently using it to display a 2-week calendar (sunday-saturday).  Would really love an option to highlight the "Today" entry.  Yes, I know that marking the current day as "Today" is there already, but I'm talking about maybe changing the background color behind whichever day is "Today" so it's more easily visible across a room.  Also, while I love the current display showing only the calendar events, I'm curious if we could also add an Outlook-style Week view that shows the full day's worth of hours so we can easily see the empty space/free time between the events and a line across the day representing the current time.
Thanks again for all your great work!

I have worked around your first problem with card mod. You can just add:
card_mod:
style: |
.day.today {
background: rgba(30, 230, 30, .15) !important;
}
to define separate background color for today.

Not optimal solution, but maybe something for you to try.

It can be styled pretty easily, this is what i did with a bit of CSS:

You just need to style .today .date and .today .events
I can post the CSS for what i did if needed.

Thanks for the suggestions!  That was exactly what I was looking for as far
as highlighting "today".
Mike
On Tue, Oct 22, 2024 at 7:29 PM whittssg @.***> wrote:

It can be styled pretty easily, this is what i did with a bit of CSS:
chrome_TaXhbUUSSg.jpg (view on web)
https://github.com/user-attachments/assets/3b696232-61b0-4481-b92b-8488a6eb9a88
You just need to style .today .date and .today .events
I can post the CSS for what i did if needed.
—
Reply to this email directly, view it on GitHub
https://github.com/FamousWolf/week-planner-card/issues/165#issuecomment-2430500562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4D3IQFSBOGNB5WGINIAR3Z43NT7AVCNFSM6AAAAABQDVFZI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZQGUYDANJWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for the suggestions! That was exactly what I was looking for as far as highlighting "today". Mike
…
On Tue, Oct 22, 2024 at 7:29 PM whittssg @.> wrote: It can be styled pretty easily, this is what i did with a bit of CSS: chrome_TaXhbUUSSg.jpg (view on web) https://github.com/user-attachments/assets/3b696232-61b0-4481-b92b-8488a6eb9a88 You just need to style .today .date and .today .events I can post the CSS for what i did if needed. — Reply to this email directly, view it on GitHub <#165 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF4D3IQFSBOGNB5WGINIAR3Z43NT7AVCNFSM6AAAAABQDVFZI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZQGUYDANJWGI . You are receiving this because you authored the thread.Message ID: @.>

This is the code i used:
`          .today .events {
border-color: rgba(251, 140, 0, 0.5); !important;
border-style: solid;
border-bottom-left-radius: 4px;
border-bottom-right-radius: 4px;
border-width:1px;
border-top-width:4px;
padding-top: 6px;
padding-left:4px;
padding-right:4px;
padding-bottom:0px;
margin-top: 0px !important;
}
      .today .date  {
      background-color: rgba(251, 140, 0, 0.8);
      padding-left: 10px;
      border-top-left-radius: 4px;
      border-top-right-radius: 4px;
      }

`

As stated by others, highlighting today can be done using card_mod, so I'm not going to be adding an option for that.
I have no plans to add "an Outlook-style Week view that shows the full day's worth of hours" right now. It might be a nice 2.0 (which I'm not planning yet) feature, but for now it's too big a change. I will keep this issue open.

Understood, and thanks for taking the time to consider and respond.  Much
appreciated!
Mike
On Mon, Oct 28, 2024 at 11:37 AM Rudy Gnodde @.***>
wrote:

As stated by others, highlighting today can be done using card_mod, so I'm
not going to be adding an option for that.
I have no plans to add "an Outlook-style Week view that shows the full
day's worth of hours" right now. It might be a nice 2.0 (which I'm not
planning yet) feature, but for now it's too big a change. I will keep this
issue open.
—
Reply to this email directly, view it on GitHub
https://github.com/FamousWolf/week-planner-card/issues/165#issuecomment-2441934982,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4D3ITMICPGIGWJJAWWGSLZ5ZD2RAVCNFSM6AAAAABQDVFZI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBRHEZTIOJYGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


[PIP-130] Apply redelivery backoff policy for ack timeout

Master Issue: #13528
Motivation
This pull request is the specific implementation plan of PIP-130
Modifications
Apply the message redelivery policy for the ack timeout.
Alert NegativeAckBackoff  interface to  RedeliveryBackoff.
Expose AckTimeoutRedeliveryBackoff in ConsumerBulider.
Add unit test case.
Currently only the Java client is modified.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Need to update docs?


Bug: https://bridge.base.org + https://bridge.base.org/transactions = not possible to connect wallet and see transactions on bridges

Describe the bug
Doesnt allow to connect wallet on this base webpage and doesnt allow to see users txs in the /transactions
Steps
go to https://bridge.base.org it will auto redirect to https://bridge.base.org/deposit  , not possible to connect a wallet anymore.
and on https://bridge.base.org/transactions ; not possible  to see users transactions because you cant connect a wallet
Expected behavior
Before it was possible to see activities of bridge TX's on :  https://bridge.base.org/transactions

Now not even possible to connect wallet on both urls, happens also on https://bridge.base.org that auto redirects to https://bridge.base.org/deposit

Additional info
No response
Environment
No response

Contact the Support page to resolve your error.
Note: you can also initiate a chat with the live agent on the chat button to get more information about your request via Live Chat
@azeonmarket
Support.

@amin1419
Hello! Thanks for raising this issue! The official Base Bridge has been deprecated; it is only possible to utilize the community bridges. If it is redirecting you I would recommend clearing your cache for bridge.base.org. Hope that helps!


Display Only the Advanced Zones when creating a "AutoScale Vm groups".

ISSUE TYPE
Bug Report
COMPONENT NAME
component:UI
SUMMARY
Display Only the Advanced Zones when creating a "AutoScale Vm groups".
Currently, the autoscaling feature is only supported in Advanced zone as it requires LB support and only isolated network and vpc support LB.
The admin user should not be able to create an Autoscale vm Group in a basic zone.
We can add a filter to show any Advance/Core zones

Steps to reproduce the issue

Create a basic zone

Create a autoscale vm group

Compute > Autoscale vm group > New Autoscale Vm group
Screencast from 06-04-23 02:34:29 PM IST.webm
EXPECTED RESULTS
The admin user should not be able to create a Autoscale vm Group in a basic zone.
ACTUAL RESULTS
The admin user is having the option to  create a Autoscale vm Group in a basic zone.

Fixed in https://github.com/apache/cloudstack/pull/7411


Use new Travis CI container infrastructure

Should speed up the build a bit.
http://blog.travis-ci.com/2014-12-17-faster-builds-with-container-based-infrastructure/
Signed-off-by: Chris Aniszczyk<EMAIL_ADDRESS>

Coverage remained the same when pulling 05b9bfd05113297e1af8516865aff3fbdc3256c7 on new-travisci-infra into 5900a10cbafb6a9c5a13f976e9ce61f95fa064f0 on master.


"Leaving site" warning shows when user follows an "external link" which directs to the IoGT site

Steps to reproduce

create External Link in article that links to a page within the IoGT site
click the link in the article
"Leaving site" warning appears

Desired behavior

create External Link in article that links to a page within the IoGT site
click the link in the article
No "Leaving site" warning when link destination is the same ... domain subdomain? Not sure about this one

It's useful to have External Links with destination inside the site in order to send users to an anchor in another article.

You can add internal links using the "Internal Link" tab from the link. The external link tab is designed to show the intermediate screen.

Does this work?

Internal links don't support anchors, but #444 fixes anchors being dropped from external links and skips the redirect page for external links if they go back to the IoGT site, so external links can now be used for this purpose.
One caveat here is that using external links that redirect to the IoGT site will require being change if an article is transferred to a different IoGT site (e.g. a country site).

Should this be closed?


get crop coordinate

Is it possible to get the crop coordinate rather than the cropped image itself ?

Yeah that shouldn't be hard at all. Let me do a little bit of research to see if it makes more sense semantically to add a "return_type" option to the function, or if it makes more sense to return both the image and the coordinates. I'll try to push something within the next few days.

Alright, just pushed that update. There is now an argument called 'return_type' which if you set equal to 'box' will return a dictionary of the top, left, height, and width of the crop box. You should be able to get it with an upgrade to version 0.1.3.
Let me know if you have any issues.

excellent work


Fix deploy.sh for main branch

Coverage: 93.36%. Remained the same when pulling b6459328fe90fe3b28cfe1bb9cb2625107db996c on fix-deploy-for-main-branch into b576765a84f892c3c28d1b30abd68d95c62b0381 on main.

Thanks for doing this.


How to make the mat-table row selected on single or double click.

Dear Angular 4 Material Team,
I have been using Angular 5 and Angular material.
Could you please provide me the code how can I change the colour of the mat-table row on single or double click on row .
Regards,
Rupesh

@damco-rupesh You can directly apply click handlers and class/style bindings on the mat-row.
Here's an example where rows will highlight when clicked: https://stackblitz.com/edit/angular-tt9tdw?file=app/table-basic-example.html

Thanks much andrewsguin
Saved my time.


Libp2p Transport Manager

Description of the Change
Transport Manager is an entity, which knows about all supported transports and which is able to find the best transport given a Multiaddress to dial.
Benefits
New module.
Possible Drawbacks
None.
Usage Examples or Tests [optional]
test/core/libp2p/network/transport_manager_test.cpp

Codecov Report

Merging #130 into master will increase coverage by 10.43%.
The diff coverage is 96.61%.

@@            Coverage Diff             @@
##           master    #130       +/-   ##
==========================================
+ Coverage   70.86%   81.3%   +10.43%
==========================================
  Files         241     246        +5
  Lines        6735    5856      -879
  Branches     3502       0     -3502
==========================================
- Hits         4773    4761       -12
- Misses       1049    1095       +46
+ Partials      913       0      -913

Impacted Files
Coverage Δ

core/libp2p/transport/transport.hpp
100% <ø> (ø)
:arrow_up:

core/libp2p/network/transport_manager.hpp
100% <100%> (ø)

...est/core/libp2p/network/transport_manager_test.cpp
100% <100%> (ø)

...ore/libp2p/network/impl/transport_manager_impl.cpp
100% <100%> (ø)

...ore/libp2p/network/impl/transport_manager_impl.hpp
100% <100%> (ø)

test/mock/libp2p/transport/transport_mock.hpp
33.33% <33.33%> (ø)

core/common/blob.cpp
0% <0%> (-20%)
:arrow_down:

core/common/byte_stream.cpp
0% <0%> (-20%)
:arrow_down:

core/libp2p/peer/errors.cpp
0% <0%> (-14.29%)
:arrow_down:

core/runtime/impl/wasm_memory_impl.hpp
85.71% <0%> (-14.29%)
:arrow_down:

... and 109 more

Continue to review full report at Codecov.

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


XG Boost Algorithm

💥 Proposal
(A clear and concise description of what the proposal is.)
Have you read the Contributing Guidelines on Pull Requests?
(Yeah)

/assign I have read all guidelines and will follow it.

/assign