[Idea] Made beacons last longer (to make them more usefull)

Hi!
The beacons appears for only 3 or 5 seconds, that is too short.
The time to almost understand where is it, and it lost.
Also, a path on ground (with jump possible?) to the nearest beacon should be show for Aliens, and Humans with helmet.
Humans bots should also put beacons, when they saw Aliens buildings for example.
When the building disappear, the beacon should too.
For beacons for defend, attack... I think 30 seconds should be enough.

This looks to be an issue for https://github.com/Unvanquished/Unvanquished/issues

Ideally we would have some lifetime management for beacons because even when "Defend" used to last 10s I often came for nothing because it was too late already

Also: https://github.com/UnvanquishedAssets/unvanquished_src.dpkdir/pull/122


Log route context as analytics parameters for Flow Router

Fixes second issue in Issue #47 where some page parameters from the previous page were mixed up with page parameters of the current page, the page / route that is being entered and logged via triggers.enter in Flow Router.
Also, added Flow Router group routes with querystring parameters to the Flow Router example, in order to test whether or not group routes would fire the global trigger at each level or not.
And, added Flow Router support language to the README.

Thanks for the PR - we'll look at this soon


Resolve CVE scan results

Hi, Our internal scan brought up these vulnerabilities. Could you please resolve them?
Thanks

Vulnerability
Severity Level
Vulnerable Library
Dependency Hierarchy
Suggested Fix

CVE-2024-4340
High
sqlparse-0.4.4-py3-none-any.whlA non-validating SQL parser.Library home page: https://files.pythonhosted.org/packages/98/5a/66d7c9305baa9f11857f247d4ba761402cea75db6058ff850ed7128957b7/sqlparse-0.4.4-py3-none-any.whlPath to dependency file: /samples/cloud_functions/requirements.txtPath to vulnerable library: /samples/cloud_functions/requirements.txt
google_pso_data_validator-3.2.0-py2.py3-none-any.whl (Root Library)google_cloud_spanner-3.1.0-py2.py3-none-any.whl❌ sqlparse-0.4.4-py3-none-any.whl (Vulnerable Library)Found in base branch: main
Upgrade versionOrigin: CVE Website Release Date: 2024-04-30Fix Resolution: sqlparse - 0.5.0

CVE-2023-47248
Critical
Python library for Apache ArrowLibrary home page: https://files.pythonhosted.org/packages/5e/1c/78a5a9264abf3f841b7e1f52a091dee116c2ae2dfe26bb974fa3154b73e3/pyarrow-6.0.1-cp37-cp37m-manylinux_2_17_x86_64.manylinux2014_x86_64.whlPath to dependency file: /samples/cloud_functions/requirements.txtPath to vulnerable library: /samples/cloud_functions/requirements.txt
google_pso_data_validator-3.2.0-py2.py3-none-any.whl (Root Library)❌ pyarrow-6.0.1-cp37-cp37m-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (Vulnerable Library)Found in base branch: main
Type: Upgrade versionOrigin: https://lists.apache.org/thread/yhy7tdfjf9hrl9vfrtzo8p2cyjq87v7nRelease Date: 2023-11-09Fix Resolution: pyarrow - 14.0.1

CVE-2023-30861
High
A simple framework for building complex web applications.Library home page: https://files.pythonhosted.org/packages/8f/b6/b4fdcb6d01ee20f9cfe81dcf9d3cd6c2f874b996f186f1c0b898c4a59c04/Flask-2.0.2-py3-none-any.whlPath to dependency file: /samples/cloud_functions/requirements.txtPath to vulnerable library: /samples/cloud_functions/requirements.txt
google_pso_data_validator-3.2.0-py2.py3-none-any.whl (Root Library)❌ Flask-2.0.2-py3-none-any.whl (Vulnerable Library)Found in base branch: main
Type: Upgrade versionOrigin: CVE Website Release Date: 2023-05-02Fix Resolution: flask - 2.2.5,2.3.2

CVE-2023-46136
High
The comprehensive WSGI web application library.Library home page: https://files.pythonhosted.org/packages/f6/f8/9da63c1617ae2a1dec2fbf6412f3a0cfe9d4ce029eccbda6e1e4258ca45f/Werkzeug-2.2.3-py3-none-any.whlPath to dependency file: /samples/cloud_functions/requirements.txtPath to vulnerable library: /samples/cloud_functions/requirements.txt
google_pso_data_validator-3.2.0-py2.py3-none-any.whl (Root Library)Flask-2.0.2-py3-none-any.whl❌ Werkzeug-2.2.3-py3-none-any.whl (Vulnerable Library)Found in base branch: main
Type: Upgrade versionOrigin: High resource usage when parsing multipart/form-data containing a large part with CR/LF character at the beginning Release Date: 2023-10-24Fix Resolution: werkzeug - 2.3.8,3.0.1

CVE-2024-36039
High
Pure Python MySQL DriverLibrary home page: https://files.pythonhosted.org/packages/4f/52/a115fe175028b058df353c5a3d5290b71514a83f67078a6482cff24d6137/PyMySQL-1.0.2-py3-none-any.whlPath to dependency file: /samples/cloud_functions/requirements.txtPath to vulnerable library: /samples/cloud_functions/requirements.txt
google_pso_data_validator-3.2.0-py2.py3-none-any.whl (Root Library)❌ PyMySQL-1.0.2-py3-none-any.whl (Vulnerable Library)Found in base branch: main
Type: Upgrade versionOrigin: CVE-2024-36039 - GitHub Advisory Database Release Date: 2024-05-21Fix Resolution: pymysql - 1.1.1

Hi @moizarafat!
It's not clear what exactly was scanned: our Github repo, our latest PyPi version, Docker image, your deployed Cloud Function for DVT or something else? Could you please clarify for us?
Meanwhile, looking just at the dependencies versions listed there and their resolutions, please check below:

CVE-2024-4340 - Resolution: sqlparse - 0.5.0 -> DVT doesn't have this library as direct dependency so it's probably used by one of our dependencies and we will need time to identify which one.
CVE-2023-47248 - Resolution: pyarrow - 14.0.1 -> DVT already has pyarrow==14.0.1 configured here
CVE-2023-30861 - Resolution: flask - 2.2.5,2.3.2 -> DVT already has Flask>=2.2 configured here
CVE-2023-46136  - Resolution: werkzeug - 2.3.8,3.0.1 -> DVT doesn't have this library as direct dependency so it's probably used by one of our dependencies and we will need time to identify which one.
CVE-2024-36039 - Resolution: pymysql - 1.1.1 -> DVT already has PyMySQL>=1.1.1 configured here

Hi @helensilva14 ,
Thanks for the prompt reply. we incorrectly scanned the version 3.2.0 instead of 6.2.0. We can close this ticket out.
-Moiz


⚠ AGEG website (prod) has degraded performance

In f51e0b7, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2020 ms

Resolved: AGEG website (prod) performance has improved in e5af1d9.


feat(earn): fetch earn positions + selector

Description
Fetch and store earn positions. Also add a selector to list them.
User and earn positions are fetched and stored together.
For each returned earn position:

if the user already holds the earn position, it is ignored (since it's already present as a user position)
if not it's added to the list

Next up: use earn positions to drive the earn screen (which was added in https://github.com/valora-inc/wallet/pull/5656)
Test plan

Updated tests

Related issues

Fixes RET-1121

Backwards compatibility
Yes
Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[x] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

@satish-ravi @finnian0826 ok to still merge this one? We can work on the latest shape for the earn data props separately.


File missing

file "dafne.modeling.nms.nms.py"   line 6 ->from poly_nms import poly_gpu_nms
where can I find this poly_nms module?
look forward to your answer

Check out the Dockerfile instructions. There you can find all dependencies that are necessary.


Switching double quotes for backticks in startup/linux.js so substitution works.

Simple fix. I was getting the ${name} in my autostart scripts. Thanks!

Thank you for the PR :)
PS: I'm on vacation with just my phone. I'll be able to publish in 2 weeks. Meanwhile you can use your fork URL in your package.json

Cool, thanks!

Published v0.2.1 :)


k3s: service fails to start, wrong go version is probably used for build

Describe the bug
Tried updating to latest nixos-unstable, but k3s service won't start due to bug described here (https://github.com/k3s-io/k3s/issues/8293#issuecomment-1714750513):
time="2023-09-04T17:58:35Z" level=info msg="Logging containerd to /var/lib/rancher/k3s/agent/containerd/containerd.log"
panic: flag alsologtostderr set at github.com/k3s-io/k3s/pkg/cli/cmds/log.go:78 before being defined

According to the issue golang 1.20 should be used for building.
Steps To Reproduce
Steps to reproduce the behavior:

update to latest nixos-unstable
have k3s service enabled
service won't start

Expected behavior
k3s is starting without errors
Additional context
I downgraded to k3s_1_25 for the time being, this version seems unaffected by the bug, 1.26 and 1.27 both seem to not work properly
Notify maintainers
@euank @mic92 @yajo

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.57, NixOS, 23.11 (Tapir), 23.11.20231024.8efd5d1`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.1`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Thanks for the clear report, and digging up the upstream issue! Appreciated!
I moved it to be built with go_1_20 in this pr: https://github.com/NixOS/nixpkgs/pull/263780
Since the tests pass there, that seems like a good sign the service starts up after that fix

@euank Thank you for the quick fix, I tried to build your branch on my kind of weak Linux server, but I think it ran out of memory, so I can't test it right now unfortunately.
Are there any tests in place for the module that check if it can start correctly? Maybe implementing something like that could save time in the future

There are such tests, yup.
On current nixpkgs master:
$ nix-build -A k3s.passthru.tests.single-node
...
machine # [    9.806518] k3s[925]: time="2023-10-27T14:01:35Z" level=info msg="Running containerd -c /var/lib/rancher/k3s/agent/etc/containerd/config.toml -a /run/k3s/containerd/containerd.sock --state /run/k3s/containerd --root /var/lib/rancher/k3s/agent/containerd"

machine # [    9.837871] k3s[925]: panic: flag alsologtostderr set at github.com/k3s-io/k3s/pkg/cli/cmds/log.go:78 before being defined
machine # [    9.839209] k3s[925]: goroutine 3303 [running]:
machine # [    9.839956] k3s[925]: flag.(*FlagSet).Var(0xc000128150, {0xa5d3a28, 0xe08daa9}, {0x9888c5a, 0xf}, {0x9a7caf8, 0x49})
machine # [    9.842124] k3s[925]:      flag/flag.go:1031 +0x33a
machine # [    9.842628] k3s[925]: k8s.io/klog/v2.InitFlags.func1(0xc0002811a0?)
machine # [    9.845121] k3s[925]:      k8s.io/klog/v2@v2.100.1/klog.go:437 +0x31
machine # [    9.845753] k3s[925]: flag.(*FlagSet).VisitAll(0xe08ace0?, 0xc00138ef30)
machine # [    9.846450] k3s[925]:      flag/flag.go:458 +0x42
machine # [    9.846938] k3s[925]: k8s.io/klog/v2.InitFlags(0xa60aa80?)
machine # [    9.850388] k3s[925]:      k8s.io/klog/v2@v2.100.1/klog.go:436 +0x45
machine # [    9.852087] k3s[925]: github.com/k3s-io/k3s/pkg/daemons/executor.(*Embedded).Bootstrap.func1()
machine # [    9.852948] k3s[925]:      github.com/k3s-io/k3s/pkg/daemons/executor/embed.go:61 +0x55
machine # [    9.854838] k3s[925]: created by github.com/k3s-io/k3s/pkg/daemons/executor.(*Embedded).Bootstrap in goroutine 1
machine # [    9.856134] k3s[925]:      github.com/k3s-io/k3s/pkg/daemons/executor/embed.go:54 +0x8f
machine # [    9.859256] systemd[1]: k3s.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
machine # [    9.860407] systemd[1]: k3s.service: Failed with result 'exit-code'.
machine # [    9.969818] systemd[1]: Failed to start k3s service.

<... test hangs forever and never passes>

In https://github.com/NixOS/nixpkgs/pull/254878, only package builds were tested, not individual packages. I think it just impacted so many packages total it wasn't practical to run all the package's tests.

Ah, that makes a lot of sense, my next question would have been why it didn't show up as failing earlier 😄
Thanks for the explanation! Did you have a chance to test it with the service? If not, I can boot up my slightly beefier NAS and see if it works there.

The k3s tests in the PR pass, which means the service starts and a pod runs!
The tests cover enough that I'm fine trusting them for this fix.


Inconsistency in Documentation and Example

I experienced  following error after installing a mattermost app on the server: (only) one of  "submit", "form", or "bindings" must be set in a binding
After looking at the code I found following switch case inside the function cleanAppBinding which specifies that one and only one the properties submit, form or bindings needs to be specified (this also applies for sub-bindings). According to the example and the quick start guide this should not be the case.
hasBindings := len(b.Bindings) > 0
hasForm := b.Form != nil
hasSubmit := b.Submit != nil
switch {
	// valid cases
	case hasBindings && !hasForm && !hasSubmit:
		var newProblems error
		b.Bindings, newProblems = cleanAppBindings(app, b.Bindings, fql, userAgent, conf)
		if newProblems != nil {
			problems = multierror.Append(problems, newProblems)
		}
		if len(b.Bindings) == 0 {
			// We do not add bindings without any valid sub-bindings
			return nil, problems
		}

	case hasForm && !hasSubmit && !hasBindings:
		clean, err := cleanForm(*b.Form, conf, app.AppID)
		if err != nil {
			problems = multierror.Append(problems, err)
		}
		b.Form = &clean

	case hasSubmit && !hasBindings && !hasForm:
		// nothing to clean for submit

	default:
		problems = multierror.Append(problems, errors.Errorf(`%s: (only) one of  "submit", "form", or "bindings" must be set in a binding`, fql))
		return nil, problems
}

FYI: The documentation around bindings is being updated in this PR: https://github.com/mattermost/mattermost-developer-documentation/pull/1111

Okay great, thanks for the reply.

Is this really solved? Because I tried to follow the quick start guide today, and had the same problem: Using the new documentation here: http://mattermost-dev-docs-preview-pulls.s3-website-us-east-1.amazonaws.com/1111/integrate/apps/quickstart/quick-start-go/
If I copy the bindings.json example file there and install my application the bindings call still fails.

Oh okay sorry, maybe I closed it to fast. I closed it because it already is worked on, but I should probably wait

Yes, you are correct @luiz-filipe . I am working on updating these docs now; A PR has been posted with the updates and is being reviewed: https://github.com/mattermost/mattermost-developer-documentation/pull/1111
I expect the PR to be merged in the coming weeks.

Closing as https://github.com/mattermost/mattermost-developer-documentation/pull/1106 got merged


Disable integration tests in parent pom

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

WordCountIT still ran in Jenkins
WordCountIT still did not run in Travis

LGTM


Prepare for 0.2.1 release

Context:
A new (patch) release of Jet is needed to address a compilation issue for CudaTensor that was introduced in #50.
Description of the Change:

The changelog now lists version 0.2.1 as the development release.
The opening line of the Jet README now mentions Python support.
The Doxygen comments in CudaScopedDevice have been formatted and expanded.

Benefits:

The GitHub and Sphinx documentation is slightly more accurate and therefore more useful.

Possible Drawbacks:
None.
Related GitHub Issues:
None.

Mikhail, what's your thoughts on adding a citation to our paper in the README like at the bottom of https://github.com/PennyLaneAI/pennylane, e.g:
Authors
Jet is the work of many contributors (link to contributors page).
If you are doing research using Jet, please cite our paper:
Trevor Vincent, Lee J. O'Riordan, Mikhail Andrenkov, Jack Brown, Nathan Killoran, Haoyu Qi, and Ish Dhand. Jet: Fast quantum circuit simulations with parallel task-based tensor-network contraction. 2021. arXiv:2107.09793

Mikhail, what's your thoughts on adding a citation to our paper in the README like at the bottom of https://github.com/PennyLaneAI/pennylane

Great idea, @trevor-vincent!  I've added the citation to the README and Sphinx documentation.


remove fixed issue that is not in release

This fix WAS released in 5801 but the delegate part is not until next week. I am not sure why it was disjointed.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


how to set webpack config with materialize?

Sorry to bother, because it could be the local problem. But it has been bothered me for days.
here is my webpack config:
var path = require('path');
var webpack = require('webpack');
var ExtractTextPlugin = require('extract-text-webpack-plugin');

module.exports = {
    entry: './src/main.js',
    output: {
        path: path.resolve(__dirname, './dist'),
        publicPath: 'dist/',
        filename: 'build.js'
    },
    resolveLoader: {
        root: path.join(__dirname, 'node_modules'),
    },
    module: {
        loaders: [
            {
                test: /\.vue$/,
                loader: 'vue'
            },
            {
                test: /\.js$/,
                loader: 'babel?{"presets":["es2015"]}',
                exclude: /node_modules/
            },
            {
                test: /\.css$/,
                loader: ExtractTextPlugin.extract(
                    "style-loader", "css-loader?sourceMap!postcss-loader")
            },
            {
                test: /\.(jpg|png|gif)$/,
                loader: "file-loader?name=images/[hash].[ext]"
            },
            {
                test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/,
                loader: "url-loader?limit=10000&minetype=application/font-woff"
            },
            {
                test: /\.(ttf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
                loader: "file-loader"
            }
        ]
    },
    vue: {
        loaders: {
            css: ExtractTextPlugin.extract("css"),
        }
    },
    plugins: [
        new ExtractTextPlugin("style.css", {
            allChunks: true,
            disable: false
        }),
        new webpack.ProvidePlugin({
            $: 'jquery',
            jQuery: 'jquery',
            'window.jQuery': 'jquery',
        })
    ],
    devtool: '#eval-source-map'
}

and my main.js:
import './libs/materialize/js/materialize.js'
import './libs/materialize/css/materialize.css'

everything is fine, but when I check out the console of chrome, it tells me this:
localhost/:13 GET http://localhost:3000/dist/dist/2751ee43015f9884c3642f103b7f70c9.woff2
localhost/:13 GET http://localhost:3000/dist/dist/ba3dcd8903e3d0af5de7792777f8ae0d.woff
localhost/:13 GET http://localhost:3000/dist/dist/df7b648ce5356ea1ebce435b3459fd60.ttf

These files are too big to be inlined.
Your woff(2) - files are handled by url-loader. You just need to increase the limit so the loader inlines them into your build.js file (output.file).
e.g.
loader: "url-loader?limit=80000&minetype=application/font-woff"
As for the .ttf fonts:
There is no limit parameter for file loader, but you can load them with url-loader as well:
{
    test: /\.(ttf|eot|svg|woff(2)?)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
    loader: "url-loader?limit=80000"
}

You might get quite big javascript files which will somewhat decrease your site's performance if they are loaded and parsed on every page.

Life savor. I looked everywhere before I found this. Thanks.

With webpack v3.1.0, I needed something like:
      {
        test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        use: [
          {
            loader: 'url-loader',
            options: {
              limit: 80000,
              mimetype: "application/font-woff"
            }
          }
        ]
      },
      {
        test: /\.(ttf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        use: [
          {
            loader: "file-loader"
          }
        ]
      }


Corrected the overflow

Changed the #body to make the width=vw(viewport width) and the width
and the #mainCanvas(which was the problem) as it was exceeding the width of the viewport, causing a scroll bar to appear.
now it is displayed within the viewport without any overflow.

Please mention your issue no.

I have updated the pr, #46


got error: model or view can not be found by params

Version
5.1.0
Steps to reproduce
 const mapJson = ... // 省/市geo.json  eg: 360000.json
 echarts.registerMap(name, mapJson);
      let option = {
        series: [
          {
            name: "map",
            type: "map",
            map: name, //自定义扩展图表类型
            label: {
              show: true,
            },
            itemStyle: {
              areaColor: "rgba(23, 27, 57,0)",
              borderColor: "#1dc199",
              borderWidth: 1,
            },
            data: [

            ],
          },
        ],
      }
      myChart.setOption(option);

load echarts geo map as above, when mousemove/mouseover the geo map, got warnnings like model or view can not be found by params,  is it realted to echarts versions?
What is expected?
no warnnings
What is actually happening?
got warnning: model or view can not be found by params

works fine on<EMAIL_ADDRESS>but got error on<EMAIL_ADDRESS>is there any demos/examples of<EMAIL_ADDRESS>about geomap useage?

i got this warning too...

i got too,so how wo solve?

Has this problem been resolved yet?


Fix parsing desktop_notification event

avatarImage field can null, and the library throws an exception:
[akka://valbo-portlet-ActorSystem/user/$a] [SlackRtmClient] Error reading event: {"type":"desktop_notification","title":"Test","subtitle":"test (bot)","msg":"1553597703.000800","ts":"1553597703.000800","content":"Some studies are unfinished. Would you like to continue?","channel":"DF0G3CBFT","launchUri":"slack:\/\/channel?id=DF0G3CBFT&message=1553597703000800&team=TBVR0AL00","avatarImage":null,"ssbFilename":"knock_brush.mp3","imageUri":null,"is_shared":false,"event_ts":"1553597704.000400"}

The commit fixes the issue.
(mccartney EDITED for formatting)

Thank you for your contribution.


68 cruise control feature

Now we can successfully set the car on cruise control, in addition you get the speed selector, where you get different speed options in which you can select any speed ranging from 0 to 150. The joystick lag has also been resolved to provide a much smoother joystick experience. This closes issues #68 and #96.
:ballot_box_with_check: Definition of Done checklist

[ ] Meaningful title and valuable description in pull request
[ ] Code change tested manually if not covered by automated tests
[ ] Pull request linked to one or more issues/stories
[ ] Acceptance criteria of the linked stories satisfied
[ ] Code change comprehended by those who approved it
[ ] State of target branch maintained or improved after the code change

Everything works as expected. A quick note about the new joystick controls, perhaps the angle is a bit under-responsive, in which case please let me know and I can provide a simple fix.

Everything works as expected. A quick note about the new joystick controls, perhaps the angle is a bit under-responsive, in which case please let me know and I can provide a simple fix.

I've manually tested the automatic parking, cruise control, camera and joystick. Well done integrating everything! Good if we fix the joystick, but I'd say it's good enough. We also need to finish the documentation and the demo video 🏁

Everything works as expected. A quick note about the new joystick controls, perhaps the angle is a bit under-responsive, in which case please let me know and I can provide a simple fix.

I've manually tested the automatic parking, cruise control, camera and joystick. Well done integrating everything! Good if we fix the joystick, but I'd say it's good enough. We also need to finish the documentation and the demo video 🏁

thank u for your review, if everything looks fine can you merge this so i can start working on demo video.


[WIP] ICMP: Option to disable checking checksum

Previously, the samples/bluetooth/ipsp example did not work.
The following appeared after enabling network debug logging.
[00:05:42.240,753]  net_ipv6.net_ipv6_input: (0x20001e48): IPv6 packet len 72 received from fe80::5ef3:70ff:fe92:5c06 to ff02::1:ffe9:6fd0
[00:05:42.240,814]  net_icmpv6.net_icmpv6_input: (0x20001e48): DROP: invalid checksum
As @tbursztyka theorized in Slack, Linux was producing a correct ICMPv6 checksum, as shown by Wireshark (this was taken after my change was applied).

After inspecting the code, indeed it seems to be the case that if net_calc_chksum_icmpv6(pkt) != 0U, then the packet is dropped.
It's possible that 6LowPan code (either in Linux or Zephyr) is not handling the IPv6 checksum properly, but I have not investigated any further.
Ideally, there would be an easy way to enable ICMPv6 checksum in Linux via /proc/sys/net/ipv6/icmp but the only node that exists there (at least on my desk) is ratelimit. If there were an easy way to enable it, then the Zephyr documentation could simply be modified.
It is a bit of a blocker to simply run a demo application. However, as @jukkar pointed out in Slack, it would be desirable to keep the checksum by default but allow it to be disabled (independently) for ICMPv6 and ICMPv4.
Therefore, I'm proposing to add a Kconfig variable to disable ICMP checksum verification and to explicitly set it to n for samples/bluetooth/ipsp.
That would also ensure that both the example and documentation remains consistent as well as the current behaviour of the ICMPv4 and ICMPv6 code.

The commit message is quite verbose, we need to probably tweak it a bit.

Previously, the samples/bluetooth/ipsp example did not work.

I don't remember the issues described when I tested it (quite some time ago though). So for me, this patch suspiciously looks like "fixing" misconfiguration of a particular Linux system by patching away Zephyr functionality.

Previously, the samples/bluetooth/ipsp example did not work.

I don't remember the issues described when I tested it (quite some time ago though). So for me, this patch suspiciously looks like "fixing" misconfiguration of a particular Linux system by patching away Zephyr functionality.

It's actually several million particular Linux systems - I'm just using Ubuntu 18.04 (LTS). However, yeah, I believe it probably worked fine in the past and could be a regression. Also, some network stacks (e.g. NetX) genuinely do not do checksumming OOTB on embedded platforms, so it could be a useful "feature" in that sense.

It's actually several million particular Linux systems - I'm just using Ubuntu 18.04 (LTS).

While @jukkar suggest to prune commit message, I'd suggest  vice-versa, to include that information to the commit message, so whoever will deal with that matter in the future, had a good understanding why it was done. For reference, I did my testing yet with ubuntu 16.04, and default kernel didn't really work for 6lowpan/BLE, so I had to upgrade it to some newer version.

so it could be a useful "feature" in that sense.

Sounds good. My concern is that you propose to enable it (disable checksum calc) by default in the reference IPSP tool, yet admit yourself that you didn't do deep enough investigation of it. (At ubuntu 18.04 ref sheds some light at the situation.)

For the commit message I propose something like this:
The samples/bluetooth/ipsp example did not work as expected and
following output appeared after enabling network debug logging.

[00:05:42.240,753] <dbg> net_ipv6.net_ipv6_input: (0x20001e48): IPv6 packet len 72 received from fe80::5ef3:70ff:fe92:5c06 to ff02::1:ffe9:6fd0
[00:05:42.240,814] <dbg> net_icmpv6.net_icmpv6_input: (0x20001e48): DROP: invalid checksum

In order to accept the ICMP packet regardless of the checksum value,
add new Kconfig options that can be used to disable ICMP checksum
checks.

It's actually several million particular Linux systems - I'm just using Ubuntu 18.04 (LTS).

While @jukkar suggest to prune commit message, I'd suggest vice-versa, to include that information to the commit message, so whoever will deal with that matter in the future, had a good understanding why it was done. For reference, I did my testing yet with ubuntu 16.04, and default kernel didn't really work for 6lowpan/BLE, so I had to upgrade it to some newer version.

I'm ok with a longer commit message as well, just to shed some light. Are you ok with that @jukkar ?

so it could be a useful "feature" in that sense.

Sounds good. My concern is that you propose to enable it (disable checksum calc) by default in the reference IPSP tool, yet admit yourself that you didn't do deep enough investigation of it. (At ubuntu 18.04 ref sheds some light at the situation.)

Well.. like I said in Slack, the likelihood that (Linux OR iputils) AND Wireshark both have simultaneously messed up their ICMPv6 checksum calculation is very slim.
I hate pointing fingers, but think that it might be a regression in Zephyr.
There is a good way to check - unit testing :smile:
It's also possible that a unit test for the ICMPv6 checksum could pass, but that an integration test for the RX chain could fail.
But I'm not familiar enough with Zephyr's test infra to say whether it is capable of integration tests for the entire RX chain going through multiple subsystems (BLE or 15.4 / 6LowPan / IP / ICMPv6).

Do I understand it correctly, there is a bug somewhere (in 6lo decompression?) and instead to fix it ICMPv6 checksum check for the IPSP sample will be turned off ?

Do I understand it correctly, there is a bug somewhere (in 6lo decompression?) and instead to fix it ICMPv6 checksum check for the IPSP sample will be turned off ?

The current theory is that yes.
I have not found the root cause though, and only suggest adding the Kconfig option to disable checksum verification (the default would still be to enable checksum verification).

Do I understand it correctly, there is a bug somewhere (in 6lo decompression?) and instead to fix it ICMPv6 checksum check for the IPSP sample will be turned off ?

The current theory is that yes.
I have not found the root cause though, and only suggest adding the Kconfig option to disable checksum verification (the default would still be to enable checksum verification).

Default is disabled for IPSP 902005dc7c3112381dc35b737e17f21baf134ec1. It is wrong. The bug should be fixed.

OK - so it looks like the issue is now gone. It was possible that it was weird temporarily unstable HEAD. I did rebase at one point and saw some changes go in by @cvinayak afterward.
Feel free to close, or if you would like to keep the change that allows checksums to be disabled, I can remove the change for the ipsp example.

so bluetooth was somehow broken then? (which is a valid possibility actually)

so bluetooth was somehow broken then? (which is a valid possibility actually)

¯_(ツ)_/¯ - maybe. It works now though. Sorry for the interruption.

My first priority is getting a new BLE platform in, so when that's done, I would be happy to help out in any way I can :-)
I'll close this PR / issue for now, if that's ok with everyone.


Plans for multi institution support

Hello are there plans to make this multi institutional or should i just do that myself?

You can modify for your needs.


checksum or similar to confirm datacube data exists and is valid

For datacube maintenance to ensure data is valid

part of the ingest process perhaps

netcdf internally supports per chunk check-summing using fletcher32. It can be enabled per measurement by setting fletcher32: True in the ingest configuration file


Don't apply dynamic enforcement to empty types

Fixes rdar://32039394.

@swift-ci Please smoke test and merge.

@atrick Seem okay?


Fixes labels on asb Endpoint in local dev template

Describe what this PR does and why we need it:
The labels appear to be incorrect, but it's not clear that they were causing any particular incorrect behavior.

Changes Unknown when pulling c1751250e251f7b1083b50f838c800e842eabab0 on mhrivnak:fix-local-dev-template into ** on openshift:master**.


individual base colors in alignment tracks

Hi,
I was wondering if there's a way that specific bases in the alignment track can be colored differently on a single read level according to a user-defined tag in the bam. This is especially helpful in visualizing single long read methylation data.
Thanks!

Hi @rrazaghi,
Can you elaborate on it a bit please? Could you also share an example of data?

No feedback, closing.


Proper error message on account not active

When we authenticate with email and password and the account is not active, the response we get is "Unable to log in with provided credentials", we do have a check later on that explicitly checks for the user "active" status, but the code never reaches there due to the user not being returned from get_auth_user
The validator for user status
@staticmethod
    def validate_auth_user_status(user):
        if not user.is_active:
            msg = _('User account is disabled.')
            raise exceptions.ValidationError(msg)

Where it fails:
def get_auth_user(self, username, email, password):
        """
        Retrieve the auth user from given POST payload by using
        either `allauth` auth scheme or bare Django auth scheme.

        Returns the authenticated user instance if credentials are correct,
        else `None` will be returned
        """
        if 'allauth' in settings.INSTALLED_APPS:

            # When `is_active` of a user is set to False, allauth tries to return template html
            # which does not exist. This is the solution for it. See issue #264.
            try:
                return self.get_auth_user_using_allauth(username, email, password)
            except url_exceptions.NoReverseMatch:
                msg = _('Unable to log in with provided credentials.')
                raise exceptions.ValidationError(msg)
        return self.get_auth_user_using_orm(username, email, password)

I guess this is because of the try block or issue #264
We should be sending the proper error message here IMO

A possible solution could be to add a pre-validation of sorts:
# Handle the non active account
    def validate_email(self, value):
        requested_user = User.objects.get(email=value)
        if not requested_user.is_active:
            raise ValidationError("The account is suspended")
        return True

@amartya-dev You can create a custom login serializer and change this behavioud of  the get_auth_user function.
class CustomLoginSerializer(LoginSerializer):
    """Custom User Login Serializer"""

    # Handle the non active account
    def get_auth_user(self, username, email, password):

        if User.objects.filter(email=email).exists():
            requested_user = User.objects.get(email=email)
            if not requested_user.is_active:
                raise exceptions.ValidationError("The account is suspended")

        return super().get_auth_user(username, email, password)

You will also need to define this serializer in your settings file.
REST_AUTH_SERIALIZERS = {
    'LOGIN_SERIALIZER': 'path.to.custom.LoginSerializer',
}

Hey @TheSinghsDen , thanks for your reply I know we can change the behavior there, it can also be validated via a validate_email on the serializer itself as I advocated in my previous comment, the point is, there should be a proper fix for that in the package I guess ... as they are trying to throw a message "Unable account is disabled" in validate_auth_user_status.

My preference is to keep it vague, but I would consider any PRs that can give some helpful error message without compromising on security.


feat: Updated Home Page UI successfully Issue 4

Pull Request Resolves[#4 ]
Title: Updated Home Page UI.
Description

Added hero Image with floating animation.

Styled the Explored button.

Added Emoji's.

Implemented nice background.

Responsive also.

Related Issues
Fixes #4
Changes Made
Improvements in Home page UI
Screenshots

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.
Thank You for this contribution.

@MeetDOD This looks great, also implement the About Us section and recent Posts section (for now simply show the first 6 posts from the posts API) as stated in the issue.

since you are also working on the pagination issue, after implementing the pagination use that API to fetch 1st page and pass page size of 6 for the recent posts case

So, sir basically also i make to design for about us section so should I raise another issue and through that issue I should work on it.
also sir what should i include in about us section like the conetent and all ?

in about section just write what the project scope is and how it works etc

Ok @VaibhavArora314 Got it :)

@VaibhavArora314 Sir, how about this Approach, if you like it then i will implement it !

That looks great, go ahead

Work continued in #78


Updated tidb dependency

What problem does this PR solve?

Issue Number: ref #9641
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/retest

Hi @mjonss I noticed some changed must be made at DM side.

For the error in dm integration tests "[Lightning:Config:ErrInvalidConfig]tikv-importer.on-duplicate cannot be used with tikv-importer.parallel-import", I think it can be resolved by "only setting IncrementalImport = true when cfg.TikvImporter.Backend == lcfg.BackendLocal"
https://github.com/pingcap/tiflow/blob/6dd155946afdaf3de7d18687f37c739d829953c0/dm/loader/lightning.go#L367

/retest

/retest

seems now the DM integration tests are failing on flaky tests

/retest

@lance6716 Hi, who would you suggest to review this? I have updated the last failing test now.

/cc @nongfushanquan


[Feature]Endpoint for calculating of Book value per share ratio , Operating margin ratio

@ighoshsubho Please review and merge the PR.

@ighoshsubho Please review and merge the PR.

@ighoshsubho  please review the pr .

Did you so all the changes I've requested? Its not reflected, that's why asking.

@ighoshsubho as you mentioned I did  all the changes accordingly . But still if  I did any mistake then tell me I will edit them.


message_tf_frame_transformer: 1.0.0-2 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository message_tf_frame_transformer to 1.0.0-2:

upstream repository: https://github.com/ika-rwth-aachen/message_tf_frame_transformer.git
release repository: https://github.com/ika-rwth-aachen/message_tf_frame_transformer-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: null

Renamed package from generic_transform to message_tf_frame_transformer as suggested in https://github.com/ros/rosdistro/pull/36653#issuecomment-1499152102.


Add Pharo10 to build

Codecov Report

Merging #45 (98bf661) into master (895e239) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   92.08%   92.08%
=======================================
  Files          39       39
  Lines        1895     1895
=======================================
  Hits         1745     1745
  Misses        150      150

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 895e239...98bf661. Read the comment docs.


util: Capture a few extra patterns of related files seen in test directories.

Description: there are a few patterns of related files previously missed by this script, such as test utility helper classes, and their headers and tests.
Risk Level: low
Testing: manually tested with simulated_time_system .cc .h _test.cc and some integration test infra
Docs Changes: n/a
Release Notes: n/a

Maybe @curiouserrandy could take a look, as he was threatening to rewrite directly in elisp to avoid shelling out to python from emacs.

Happy to, but Google envoy import is eating my brain, so I won't be able to look until Tuesday.  If that's ok with you, sure thing.

Yeah I find the direct elisp version of this I was previously using (written by another Googler) to be very difficult to read.


Update _document.tsx

fix SSR bug

[x] I have followed (at least) the PR section of the contributing guide.

fixed SSR bug as recommended by https://github.com/mui-org/material-ui/issues/16244#issuecomment-502368687

CC: @oliviertassinari


Common.Logging for .Net 4.5

Is there a reason that the NuGet package doesn't have a net45 ?

Our assumption is that there isn't an actual need for this at this time.
There isn't a specific compilation of Common.Logging for net45 (the net40 binary would be identical to a binary for net45).  When adding the package, if there isn't a perfect match for your project's framework target in the package, NuGet is supposed to fall back to the latest/closest preceding framework moniker as I always understood it.
This means that if you add the package to your .NET 4.5 project, you should get the net40 binary referenced (which would be identical to a binary labeled as net45).  Is this different from what you are experiencing?
Possible I'm (once again!) going to find myself surprised by the actual behavior of NuGet in the wild vs. what had been explained to me in the past :-/  If this is the case, we can certainly add a net45 target, but it will be the same binary as the present net40 (unless we become aware of a need to provide a different net45 compilation target).
Can you advise whether you're experiencing something different --?

Cool. Let me check / try again. And I will report back. :)

Ok, not sure what I had wrong but it works now. Thanks!

Excellent -- my kinda problem :)


load-languages command adds ~1 sec to startup time

The load-languages command is used when OpenRefine starts up, even though the information isn't shown to the user anywhere (unless they switch the Languages tab). It takes almost a full second (980 ms) to complete because the server is loading every single language file, just to get the language's name.
https://github.com/OpenRefine/OpenRefine/blob/10c046698a9586662668e31086bdd2d3c6aad801/main/src/com/google/refine/commands/lang/GetLanguagesCommand.java#L75-L82
The short term fix is to defer loading the language list until it's needed to render the Languages tab.
Longer term we can investigate caching the list on the server and/or initializing it at server startup time.

After looking at the code, I'm not sure this is actually blocking the UI, so I need to double check the performance graph.

I tried to reproduce this: on my side it takes 30ms. I am using an SSD so perhaps that makes a big difference for accessing multiple files? @elroykanye just tried with an HDD and got 50 ms. So I am not sure what makes it so slow on your side @tfmorris. If you could check if you can still reproduce this two years after, it would be awesome.

For a cold start (OpenRefine 3.5.2 standard release server running, but not used since yesterday), I see several load-language requests with latencies as follows:
4120 msec
1440 msec
944 msec
1200 msec
Overall the page took almost a minute to load:
59 requests 2.2 MB transferred 2.2 MB resources Finish: 1.0 min DOMContentLoaded: 50.78 s Load: 58.19 s
A warm refresh of the screen produced load-language latencies of 388, 39, 546, 376 msec (along with a  94 ms get-preference?name=reconciliation.standardServices call) for a total latency penalty of about 1.5 seconds.
Total home page refresh clocked in at a little over 18 seconds:
60 requests 2.2 MB transferred 2.2 MB resources Finish: 17.77 s DOMContentLoaded: 14.87 s Load: 17.39 s
This is all with Chrome Version 103.0.5060.114 (Official Build) (x86_64) running on macOS 11.6.2, 1 TB PCIe SSD disk, 2.5 GHz Core i7 processor, 16 GB DDR3 RAM (ie not the latest and greatest, but certainly a more than capable machine).
Here are the HAR files for the two loads:
OpenRefine-3.5.2-Mac-Startup.har.zip
OpenRefine-3.5.2-Mac-Home-Refresh.har.zip

There are a couple of independent issues:

All the calls are synchronous and many are serialized unnecessarily (I think there's a separate issue for this)
There are multiple load-language calls (although it's probably one per extension, which makes sense from a modularity point of view and wouldn't be an issue if they were asynchronous)
The individual calls are slow, which was the original problem report (although, again, much less of an issue if asynchronous)

Thanks a lot! I am surprised by the timings, they are indeed very long on your side although your machine is much better than mine in all respects. That's very curious!
Note that this issue is originally about the get-languages command (fired only once and only from the index page), not the load-language command (fired once per module, on the index and project pages).

Are the Java environments the same from both of you?

Note that this issue is originally about the get-languages command (fired only once and only from the index page), not the load-language command (fired once per module, on the index and project pages).

Oops! get-languages?&name=module&value=core latency was 4560 msec on cold start and 294 msec on refresh.
My JRE is whatever's bundled with the binary Mac distribution for OpenRefine.
It's possible that an anti-virus program or something else is adding latency, but making things lazy loaded and async would make that irrelevant.


Multiple versions of Barry Rowlingson

These are all me:
http://depsy.org/person/437428
http://depsy.org/person/331413
http://depsy.org/person/334287
http://depsy.org/person/440888
I also seem to collaborate with myself on several things:
http://depsy.org/package/r/geonames
"geonames by Rowlingson, Rowlingson, Ram, and 2 others "
Best of luck cleaning your database!

thanks barry!  will work to fix asap and let you know.

Fixed!  Here's your single merged profile: http://depsy.org/person/334287
Thanks for letting us know about the problem!


Suggestions for data sync tool

Describe the improvement
The following are some suggestions for the data sync tool which is used for zero downtime migrations [1].

Implement a standalone and not dependent on the WSO2 IS to startup. This is probably simple for an experienced person on WSO2 stack to do but for a DBA who supports multiple applications, it is asking for a lot.

Present benchmarks on what it can support. For example, how many updates an hour running under what configuration.

Consider rollback. If an upgrade from 5.8 to 5.10 causes reissuing of existing tokens on cutover then it will be difficult to roll back to the previous stack as you have just recreated most of your tokens and the valid ones are on the new stack.

[1] https://is.docs.wso2.com/en/5.11.0/setup/migrating-preparing-for-migration/#zero-down-time-migration

Usually any software migration do not offer rollback.
The correct strategy is that do some dry run on a copy of a live data, test everything works ok, do rehearsal for several times. Then only do the live migration.
There can be rare case where still the system has to be rolled back. Then the new tokens has to be handled case by case. Usually it will be no problem for the user to as re-login, as there will be very few users who has re-issued tokens at the time of migration.

Standalone approach is a good suggestion.
This is something on the discussion stage, where we have a separate command line option to do DB schema change. and the real data migrator.

However Blue-Green migration itself means there are two clusters accepting traffic at the same time. The green cluster accepts all new traffic and Blue cluster accepts some limited Read-only traffic like token validations and any reporting. Hence the two clusters will be normal servers.

Provide a benchmark is out of the product migrator as it is highly dependent on the current data and system resources the system has.
It needs to be reiterated that any migration has to be rehearsed with recent copy of live data. This is the time where the respective benchmark can be obtained on the system based. Any benchmark obtained on one setup is only valid for that setup. Also it is worth noting that even this is valid for the same setup only for limited timeframe.  For example, the benchmark done last years migration on the one installation is not valid a year later as the dataset would have been changed, grown , and new services are onboarded.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Export xls with image

hi,
How to export cells with image in xls , have image in buffer format .

You cant


Cluster MQTT broker based on mochi-co/mqtt

I was looking for cluster ready mqtt broker for my home environment (k8s) for some time already. But unfortunately all i checked (emqx, vernemq) have some issues (e.g not compatible with Paho, or broken in home environment).
So based on mochi-co/mqtt i build my own broker - i was heavily inspired by https://github.com/mochi-co/mqtt/pull/45
Maybe other people looking for clustering capabilities will be interested, or even better some pieces can be used inside mochi-co/mqtt!
If you think that this is not a place for this kind of communication, please close :)
https://github.com/bkupidura/broker-ha

Amazing! This is very interesting, and I like what you've done here... I will definitely look into it deeper when I get some time and have a think about how we might implement some of these features in the future :)

Cluster is implemented using gossip and raft protocol. https://github.com/wind-c/comqtt based on mochi-co/mqtt .

is there any progress on that part? I might be interested :)

@anthonycorbacho probably you are more interested in clustering capabilities directly in mochi-co/mqtt, but i just pushed new version to bkupidura/broker-ha which now is based on mochi-co/mqtt v2.

@bkupidura I've been wondering recently if it would be possible to build clustering functionality using the hooks system, but I haven't looked too much into cluster myself. I wonder if it is possible?

@mochi-co short answer would be yes, it should be possible ;) Currently broker-ha is using mochi-co/mqtt hooks to forward PUB messages to different members. But my implementation depends on hashicorp/memberlist.
Memberlist is used to exchange data between cluster members. It can be replaced by any protocol. Initially i was trying to build this on top of vanilla MQTT. But this approach can be little tricky and complicated with 2+ cluster members.
My initial idea was similar to mosquitto bridge configuration. But this requires mqtt client connected to every broker and syncing state between them over MQTT... Memberlist approach looks much simpler.
You would need to solve 3 main issues:

How to discover cluster members?  (DNS SRV discovery in broker-ha)
How to handle cluster member join/leave? (memberlist in broker-ha)
How to exchange data between cluster members? (memberlist in broker-ha)

Thanks @bkupidura, I will have a think about this and see if I can come up with any ideas

Comqtt's cluster capability was developed based on mochi-co/mqtt v2.0.
https://github.com/wind-c/comqtt

This discussion is now located at https://github.com/orgs/mochi-mqtt/discussions/280#discussioncomment-6755488


运行时报错 Error Code 1: Cuda Runtime (invalid device context)

环境：

(ocr) ➜  OCRIntegrator git:(main) ✗ pip list | grep tensor
nvidia-tensorrt           99.0.0
tensorrt                  10.0.1
tensorrt-cu12             10.0.1
tensorrt-cu12-bindings    10.0.1
tensorrt-cu12-libs        10.0.1

(ocr) ➜  OCRIntegrator git:(main) ✗ uvicorn main:app
2024-07-03 03:56:13 - INFO - /workspace/sunjinfeng/github_projet/OCRIntegrator/app/deepdoc/tensorrt_engine.py - line 13 - Start loading engine
[07/03/2024-03:56:13] [TRT] [W] Using an engine plan file across different models of devices is not recommended and is likely to affect performance or even cause errors.
2024-07-03 03:56:13 - INFO - /workspace/sunjinfeng/github_projet/OCRIntegrator/app/deepdoc/tensorrt_engine.py - line 17 - Completed loading /workspace/sunjinfeng/github_projet/OCRIntegrator/res/deepdoc/det.trt engine in 0.0303s
2024-07-03 03:56:13 - INFO - /workspace/sunjinfeng/github_projet/OCRIntegrator/app/deepdoc/tensorrt_engine.py - line 13 - Start loading engine
[07/03/2024-03:56:13] [TRT] [W] Using an engine plan file across different models of devices is not recommended and is likely to affect performance or even cause errors.
2024-07-03 03:56:13 - INFO - /workspace/sunjinfeng/github_projet/OCRIntegrator/app/deepdoc/tensorrt_engine.py - line 17 - Completed loading /workspace/sunjinfeng/github_projet/OCRIntegrator/res/deepdoc/rec.trt engine in 0.0233s
[07/03/2024-03:56:14] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::52] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::52] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::52] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaStream::43] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)

@peakhell

pycuda==2024.1

i will add this to my README

还是报上述错误。。。环境完全一样，就只有我的 cuda 版本是 12.2 ，是因为 pycuda没有初始化吗

[07/03/2024-03:56:14] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::52] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::52] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [defaultAllocator.cpp::deallocate::52] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaStream::43] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
[07/03/2024-03:56:14] [TRT] [E] 1: [scopedCudaResources.cpp::~ScopedCudaEvent::20] Error Code 1: Cuda Runtime (invalid device context)
You have multiple GPUs, which may affect the management of CUDA contexts. You need to explicitly specify the GPU device to use. I do not have an environment with multiple GPUs; search load_model_cuda function，and change the cuda_context with follow, this may help
`    import pycuda.driver as cuda
device = cuda.Device(device_id)  # replace device_id with your device id
context = device.make_context()
model_file_path = os.path.join(model_dir, nm + ".trt")

engine = load_engine(model_file_path)
return TrtModelEngine(engine, cuda_ctx=context), TrtModelEngine.get_input_tensor_names(engine)`

I tried this approach, but it seems the same issue persists; however, the context stack was not empty and could not be released now.

I tried this approach, but it seems the same issue persists; however, the context stack was not empty and could not be released now.
I wrote a script to manually clear the stack.
import pycuda.driver as cuda
cuda.init()
num_devices = cuda.Device.count()

for device_id in range(num_devices):
    device = cuda.Device(device_id)
    context = device.make_context()
    while True:
        try:
            cuda.Context.pop()
        except cuda.LogicError:
            break
    context.detach()

print(f"All contexts for {num_devices} devices have been cleaned up.")

It doesn't seem to be working.🤧

Manually close the CUDA context. I don't have a multi-GPU environment, but I don't think this is a difficult problem to solve. You just need to manually specify the GPU device and manage the CUDA context.
    try:
        model_file_path = os.path.join(model_dir, nm + ".trt")
        engine = load_engine(model_file_path)
        return TrtModelEngine(engine, cuda_ctx=context), TrtModelEngine.get_input_tensor_names(engine)
    finally:
        context.pop()

It seems unrelated to whether it's a multi-GPU environment or not, as I'm still getting the same error even when a specific GPU is designated.

@Tendo33 I have the same environment, but everything is ok. I don't know if this will work for you, maybe you can try a 12.0.1-cudnn8-devel-ubuntu20.04 NVIDIA image. I installed python 3.11 and related repo dependencies on top of this image and it works fine.

@Tendo33 I have the same environment, but everything is ok. I don't know if this will work for you, maybe you can try a 12.0.1-cudnn8-devel-ubuntu20.04 NVIDIA image and then installed python 3.11 and related repo dependencies on top of this image and it works fine. I did not modify the tensorrt_engine file, particularly with regard to configuring the CUDA context. Based on testing, it seems to use only my first GPU by default.

Please ignore their Memory usage and Volatile GPU-Util because it’s running my large model, I just used the third GPU to test it.


bump version in readme in github actions cron example

I encountered a minor issue while using your otherwise excellent tool. Here is a PR to fix the issue.

Closes #112


talloc: 2.1.11 -> 2.1.12

Semi-automatic update generated by https://github.com/ryantm/nix-update tools.
This update was made based on information from https://repology.org/metapackage/talloc/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 2.1.12 with grep in /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12
found 2.1.12 in filename of file in /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12
directory tree listing: https://gist.github.com/ab01ab39025b7d1a3a88e508b20ab4ed

cc @wkennington for review

Success on x86_64-darwin (full log)
Attempted: talloc
Partial log (click to expand)
* installing pytalloc.h as /nix/store/lk2yfgzw27s3hxbh9ni76px3pzg6dmg8-talloc-2.1.12/include/pytalloc.h
Waf: Leaving directory `/private/tmp/nix-build-talloc-2.1.12.drv-0/talloc-2.1.12/bin'
'install' finished successfully (0.354s)
ar: creating archive /nix/store/lk2yfgzw27s3hxbh9ni76px3pzg6dmg8-talloc-2.1.12/lib/libtalloc.a
post-installation fixup
gzipping man pages under /nix/store/lk2yfgzw27s3hxbh9ni76px3pzg6dmg8-talloc-2.1.12/share/man/
strip is /nix/store/0fzpxnsanc02i4jsb1yhchjp4p62b2n3-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/lk2yfgzw27s3hxbh9ni76px3pzg6dmg8-talloc-2.1.12/lib
patching script interpreter paths in /nix/store/lk2yfgzw27s3hxbh9ni76px3pzg6dmg8-talloc-2.1.12
/nix/store/lk2yfgzw27s3hxbh9ni76px3pzg6dmg8-talloc-2.1.12

Success on x86_64-linux (full log)
Attempted: talloc
Partial log (click to expand)
shrinking /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12/lib/python2.7/site-packages/talloc.so
shrinking /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12/lib/libtalloc-compat1-2.1.12.so
shrinking /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12/lib/libpytalloc-util.so.2.1.12
shrinking /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12/lib/libtalloc.so.2.1.12
gzipping man pages under /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12/lib
patching script interpreter paths in /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12
checking for references to /build in /nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12...
/nix/store/9yy48ck3ahhkisjaj5w2w3wsmqg2bgxw-talloc-2.1.12

Success on aarch64-linux (full log)
Attempted: talloc
Partial log (click to expand)
shrinking /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12/lib/python2.7/site-packages/talloc.so
shrinking /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12/lib/libpytalloc-util.so.2.1.12
shrinking /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12/lib/libtalloc.so.2.1.12
shrinking /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12/lib/libtalloc-compat1-2.1.12.so
gzipping man pages under /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12/lib
patching script interpreter paths in /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12
checking for references to /build in /nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12...
/nix/store/a3cbmgif0kg680rpli9clxm0qmf92b2d-talloc-2.1.12


fix: use flag value as default or when changed only

:bug: fixes an error seen downstream when IsOpenShift results in non-zero flag default, causing tests to fail
:broom: updates other affected commands/flags (the registry and builder flag on both deploy and build subcommands)

Updates the deploy and build commands to more selectively use the value of flags whose default is a non-zero value and whose value maps to an attribute on the function being operated upon.
In particular, the flag default will only be used when the function has no value yet defined (as a default), or when the value has been provided by the user (an override).
Note that this logic is a bit janky, and will be refined with the upcoming Global Config PR, which will provide a default value to the flag which will already take into account both the static default and the existing function state.  Then the resltant config object can always be used directly with no conditionals.
/fixes #941
/kind bug

* ✅ login: lkingland / name: Luke Kingland  ([9568ff4](https://github.com/knative-sandbox/kn-plugin-func/commit/9568ff4d1b4d9b9f43d29a254a28a4838c2a51d8))

* [❌](https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/25619798/242137258/1223/#/?version=2) - login: @serverless-qe / name: Serverless QE Robot . The commit ([05b6ef9](https://github.com/knative-sandbox/kn-plugin-func/commit/05b6ef97c6feba1fc8da23b3223e2cfa05c8a123)) is not authorized under a signed CLA. [Please click here to be authorized](https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/25619798/242137258/1223/#/?version=2). For further assistance with EasyCLA, [please submit a support request ticket](https://jira.linuxfoundation.org/servicedesk/customer/portal/4).

Offending commit removed.  I was testing against a clone of openshift/release-next to confirm the fix works as expected downstream.

/lgtm
Thanks!


Icon Request

Add all the icons from https://simplemobiletools.com/
And also,
Droid-ify/Neo Store
Feeder
KDE Connect
KeePassDX
Gboard
K-9 Mail
Material Files
PDF Viewer Plus
RethinkDNS
Riseup VPN
Tusky
Geometric Weather
Syncthing
Binary Eye
Auxio
Orbot
Organic Maps

Also please add icons for:
Privat24
Monobank
Caves(Roguelike)
(Don't want to make a duplicated request)

Hello! It would appear that this icon request doesnʼt include the package names of some or all of the apps for which icons have been requested. In addition, we are switching to a new system for icon requests. The apps for which package names have been provided (if such apps exist in this icon request) have been added to the new icon request database, meaning no action is required. For the apps for which package names havenʼt been provided, please submit an icon request via the new icon request form: https://forms.gle/Fx8vZAiWdW1Tyjo57.


Un-install should delete pGina registry key

Probably should update the Inno script so that the pGina registry key is deleted upon uninstall.  Thoughts?

ill find the pgina file and i delete system32...
but still showing PGINA WAITING FOR SERVICE

I cannot find pgina file in Programs and Features


Replace tsd with typings in TypeScript skeleton

Since tsd is deprecated, it should be replaced with typings in the TypeScript skeleton.

I will submit a PR shortly that adds in Typings support to the Skeleton.

I believe this is merged already and can close @EisenbergEffect


initialization error in adobe_api.py (adobe_tools branch)

Commit 5822bc94 in the adobe_tools branch introduced an error at lines 56-64 of adobe_api.py.  A copy of the __init__ function header, which was supposed to be indented along with the other members of the class, was left exdented and so the other members weren't defined.

This was fixed by ca86af6.


HDDS-10384. RPC client Reusing thread resources.

What changes were proposed in this pull request?
RPC clients reuse thread resources instead of creating a new thread for each Block.
Detail https://github.com/apache/ozone/pull/5791
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10384
How was this patch tested?
Existing Test

Thanks @xichen01 for the patch, @szetszwo for the review.

I think this may have caused flakiness in tests creating keys:
https://github.com/apache/ozone/actions/runs/8049147769/job/21982702731#step:6:131
https://github.com/apache/ozone/actions/runs/8045761936/job/21972281351#step:6:171
https://github.com/apache/ozone/actions/runs/8045494617/job/21971516374#step:6:113
https://github.com/apache/ozone/actions/runs/8045494617/job/21971512905#step:6:167


didSwipeCardAt not being called

When I swiped on the card, the didSwipeCardAt function is not being called.

Have you found the solution ? or did you use another package?

I just can't find solution I am sitting 1 week trying to find it. The author better to delete the package instead if he ignores main bugs

I just can't find solution I am sitting 1 week trying to find it. The author better to delete the package instead if he ignores main bugs

Have you setted the cardStack.dataSource = self ? I've just checked the latest commit in the project and it seems to work good


Improve display of "browse" and "create" buttons on cards on homepage

Removes footer on cards and makes buttons full width (and stack vertically). I do not like that these buttons do not align across cards, but any attempt to fix caused more problems.

OK, here's what I did:

wrap the text and buttons each in a div.
having those two divs then allowed me to use the flexbox justify-content: space-between to push the text to the top and the buttons to the bottom. I did this by using a Bootstrap flex utility, meaning I added an additional class (justify-content-between) because that's how Bootstrap rolls.

Flexbox is incredibly powerful and incredibly confusing. I have found the CSS Tricks very useful, and find myself going back to it often.

Thank you! It looks like exactly what I was trying to get; as you mentioned, I was getting a bit stuck with the flex utility.


Can't Add Location on Mobile

Checklist

[X] I have read the FAQ
[X] I have checked if this issue already exists (both open and closed issues)
[X] I am using the most recent version of pogom from the master branch
[X] If this is an installation issue, I am posting my OS, OS version, python version, pip version
[X] I'm posting the complete log
[X] I'm using a descriptive title for this issue

My Issue
On both iOS and Android it seems like you can't "click" the map to add a location. Am I doing it wrong? :)
...

@jcady I can confirm that it works. Are you authenticated/logged on your device?

D'oh, that was it. Much appreciated!


Integration with transformers

Amazing work! I'm sure it will open up doors for researchers to think about ways to better extrapolate during inference time.
I am interested to know if you know of any integrations that use AliBi with transformers from Hugging Face.

Thanks!
I'm not aware of any correct integrations with the transformers library from HF, but when the BigScience large language model will complete training, it will be available through huggingface and since that model uses ALiBi that will require the integration of ALiBi into the transformers library.
You can read more about it here: https://bigscience.notion.site/BigScience-176B-Model-ad073ca07cdf479398d5f95d88e218c4


Kap always shows marquee on 'system' display.

Platform:
Darwin Steves-iMac.local 16.3.0 Darwin Kernel Version 16.3.0: Thu Nov 17 20:23:58 PST 2016; root:xnu-3789.31.2~1/RELEASE_X86_64 x86_64
Kap Version:
1.0.0 (<IP_ADDRESS>)
Steps to reproduce
With multiple screens attached to a desktop Mac, open Kap from a Finder window on the non-System* display.  (* where the menu bar is dragged in the Mac's Display system preference)
Click on the red 'select recording area' tool in Kap.
Current behaviour
The marquee always appears on the Mac's system display, not the display I launched Kap on.
Expected behaviour
Should show the marquee on the screen I launched Kap on, where I want to do my recording.
Workaround
Drag marquee from the System display over to one of the other displays,.

Fixed in #494


Add documentation where uvx published programs run (working dir)

Title: Documentation needed: Clarify npx execution context and environment configuration
I'd like to request additional documentation regarding the execution context when running the server via npx. Specifically, the documentation should address the working directory context for npx execution.
This came up through a user question (see: MarkusPfundstein/mcp-obsidian#6) where they were unsure about the proper location for the .env file when running the published server. Currently, this information isn't clearly documented.
Having this information in the docs would help users properly configure and run the server without confusion.
Thanks for considering this addition to the documentation.

Does https://github.com/modelcontextprotocol/docs/pull/88 address the issue?

Yes, I think the suggestions help


Bug: Call to an undefined method CodeIgniter\HTTP\URI::getRoutePath()

PHP Version
8.2
CodeIgniter4 Version
4.4.3
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
macOS
Which server did you use?
apache
Database
MySQL 5.7
What happened?
When coding a filter i came across the following issue:
PHPStan marks the line $route = $uri->getRoutePath(); with the error in the title. That happens because getUri() return a URI instance and not a SiteURI instance.
Steps to Reproduce
public function before(RequestInterface $request, $arguments = null)
{
     $uri   = $request->getUri();
     $route = $uri->getRoutePath();
}

Expected Output
Expected no error and intellisense for $uri.
Anything else?
If i annotate $uri with /** @var \CodeIgniter\HTTP\SiteURI $uri */ it works properly.

Thank you for reporting.
Yes, PHPStan is correct. URI does not have getRoutePath().
$request->getUri() must return URI to follow PSR-7. (CI4 is not yet PSR-7 compliant, but we are working toward it.)
How could we fix this?


Bud command does not work due to error "This script must be run either from the flake's devshell"

Since commit 1df67a3f2ac5e109fef54dc0b761aca0d64c2c5c
Expected Behavior

Current Behavior
✘ piotrek@Pc1Nix  ~/Projects/nixos  ➦ 1df67a3  git checkout 1df67a3f2ac5e109fef54dc0b761aca0d64c2c5c
Found existing alias for "git checkout". You should use: "gco"
HEAD is now at 1df67a3 update digga lock
piotrek@Pc1Nix  ~/Projects/nixos  ➦ 1df67a3  bud
This script must be run either from the flake's devshell or its root path must be specified
✘ piotrek@Pc1Nix  ~/Projects/nixos  ➦ 1df67a3  git checkout f10a8fdd33e5b354e6c5cfe32029900a891e65ca
Found existing alias for "git checkout". You should use: "gco"
Previous HEAD position was 1df67a3 update digga lock
HEAD is now at f10a8fd chore: update nvfecher
direnv: loading ~/Projects/nixos/.envrc
direnv: using flake
direnv: ([/nix/store/l6flm08wv6cgy0n6myzc8g3k68wiqayf-direnv-2.28.0/bin/direnv export zsh]) is taking a while to execute. Use CTRL-C to give up.
direnv: renewed cache
🔨 Welcome to nixos
[devos]
agenix
bud                  - Your highly customizable system ctl
cachix               - Command line client for Nix binary cache hosting https://cachix.org
deploy-rs            - A Simple multi-profile Nix-flake deploy tool
nix
nvfetcher            - Generate nix sources expr for the latest version of packages
[docs]
mdbook               - Create books from MarkDown
[general commands]
menu                 - prints this menu
[linter]
editorconfig-checker - A tool to verify that your files are in harmony with your .editorconfig
nixpkgs-fmt          - Nix code formatter for nixpkgs
direnv: export +DEVSHELL_DIR +DEVSHELL_ROOT +IN_NIX_SHELL +NIXPKGS_PATH +NIX_SSL_CERT_FILE ~NIX_PROFILES ~PATH ~XDG_DATA_DIRS
piotrek@Pc1Nix  ~/Projects/nixos  ➦ f10a8fd  bud
Usage: bud COMMAND [ARGS]
Commands:
build HOST BUILD               Build a variant of your configuration from system.build
get [DEST]                     Copy the desired template to DEST
home [switch] (user@fqdn | USER HOST | USER) Home-manager config of USER from HOST or host-less portable USER for current architecture
install HOST [ARGS]            Shortcut for nixos-install
rebuild HOST (switch|boot|test) Shortcut for nixos-rebuild
repl [FLAKE]                   Enter a repl with the flake's outputs
ssh-show HOST USER | USER@HOSTNAME Show target host's SSH ed25519 key
up                             Generate /home/piotrek/Projects/nixos/hosts/Pc1Nix/default.nix
update [INPUT]                 Update and commit /home/piotrek/Projects/nixos/flake.lock file or specific input
vm HOST                        Generate & run a one-shot vm for HOST
piotrek@Pc1Nix  ~/Projects/nixos  ➦ f10a8fd 
Possible Solution

Steps to Reproduce

Context

Your Environment

As a quick workaround you can do export DEVSHELL_ROOT=/path/to/flake

Yep, I just used: ❯ DEVSHELL_ROOT=$(pwd) bud rebuild tower switch
I think CI for devos needs a Can run bud script test.

@GTrunSec Do you have any thoughts on this? It seems like you have done research on the recent devshell updates.

thanks for your reporting. @Pacman99  https://github.com/divnix/bud/pull/24 .

It would be great if someone can check if #386 fixes this problem and approve that pull request.

Yep, I just used: ❯ DEVSHELL_ROOT=$(pwd) bud rebuild tower switch
I think CI for devos needs a Can run bud script test.

I would just like to add that this workaround (perhaps predictably), doesn't work if the command requires running as root, such as the case with bud up (for btrfs subvolume show to work):
at 13:20:09 ✖1 ❯ DEVSHELL_ROOT=$(pwd) sudo bud up
This script must be run either from the flake's devshell or its root path must be specified

Yep, I just used: ❯ DEVSHELL_ROOT=$(pwd) bud rebuild tower switch
I think CI for devos needs a Can run bud script test.

I would just like to add that this workaround (perhaps predictably), doesn't work if the command requires running as root, such as the case with bud up (for btrfs subvolume show to work):
at 13:20:09 ✖1 ❯ DEVSHELL_ROOT=$(pwd) sudo bud up
This script must be run either from the flake's devshell or its root path must be specified

How about:
nix flake lock --update-input bud
sudo -E bud up

That now results in the btrfs command failing, same as not using sudo at all.
at 12:55:37 ❯ sudo -E bud up
Failed to execute: btrfs subvol show /persist

@jmc-figueira have you fixed it somehow?
i have the same issue


Prevent crash with Spotify local files

spotify_activity has no state if the user is listening to a local file, resulting in a crash.
This PR changes build_pretty_spotify to ensure the spotify activity has state, even if there's no data
This will prevent lanyard from freezing for a while whenever you play a local file.
Here's an example of the error that was occuring:
21:23:08.744 [error] GenServer :"presence:303267459824353280" terminating
** (KeyError) key :state not found in: %{id: "spotify:1", name: "Spotify", type: 2, details: "EMPTY DIARY (instrumental)", timestamps: %{start:<PHONE_NUMBER>950, end:<PHONE_NUMBER>950}, assets: %{large_text: "UNBEATABLE: DEMO TAPES"}, created_at:<PHONE_NUMBER>913, party: %{id: "spotify:303267459824353280"}}
    (lanyard 0.1.0) lib/presence/spotify.ex:5: Lanyard.Presence.Spotify.build_pretty_spotify/1
    (lanyard 0.1.0) lib/presence/presence.ex:197: Lanyard.Presence.build_pretty_presence/1
    (lanyard 0.1.0) lib/presence/presence.ex:105: Lanyard.Presence.handle_cast/2
    (stdlib 5.2.3) gen_server.erl:1121: :gen_server.try_handle_cast/3
    (stdlib 5.2.3) gen_server.erl:1183: :gen_server.handle_msg/6
    (stdlib 5.2.3) proc_lib.erl:241: :proc_lib.init_p_do_apply/3
Last message: {:"$gen_cast", {:sync, %{discord_presence: %{status: :online, user: %{id:<PHONE_NUMBER>24353280}, broadcast: nil, activities: [%{id: "spotify:1", name: "Spotify", type: 2, details: "EMPTY DIARY (instrumental)", timestamps: %{start:<PHONE_NUMBER>950, end:<PHONE_NUMBER>950}, assets: %{large_text: "UNBEATABLE: DEMO TAPES"}, created_at:<PHONE_NUMBER>913, party: %{id: "spotify:303267459824353280"}}], guild_id:<PHONE_NUMBER>83610988, client_status: %{desktop: :online}}}}}
State: %Lanyard.Presence{user_id: "303267459824353280", discord_user: %{id:<PHONE_NUMBER>24353280, username: "iamasink", avatar: "c97773d8ea60932eef12331d41effeca", discriminator: "0", bot: false, clan: nil, global_name: "lillie", avatar_decoration_data: nil, display_name: "lillie", public_flags: 128}, discord_presence: %{status: :online, user: %{id:<PHONE_NUMBER>24353280}, broadcast: nil, activities: [%{flags: 48, id: "spotify:1", name: "Spotify", type: 2, state: "水槽", session_id: "b05c49cbec04113d63ddffa38727e577", details: "p.h. - cover", timestamps: %{start:<PHONE_NUMBER>901, end:<PHONE_NUMBER>830}, assets: %{large_image: "spotify:ab67616d0000b2736f4be5c667dfbf2c2781c4c9", large_text: "p.h. (cover)"}, sync_id: "3rWyLf6o3RMbUeK4aafkHd", created_at:<PHONE_NUMBER>945, party: %{id: "spotify:303267459824353280"}}], guild_id:<PHONE_NUMBER>043872849, client_status: %{desktop: :online}}, kv: %{}, subscriber_pids: [], refmap: %{}}

I haven't any experience with Elixir so sorry if anything is wrong, but it seems to work fine ^^

Ah, that's way simpler. Thanks for explaining it!


fix: file path title in certs dialog

fixes #8231

🚀
The pointer-events=none was added to avoid an issue with react-resizable panels showing the <-> cursor on top of Svg elements if they overlapped a resize thumb.
I think we could completely remove this by upgrading to the latest version since this is now fixed https://github.com/bvaughn/react-resizable-panels/pull/427


Feedback

:wave:! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to master since the assignment started. Your teacher can see this too.

Notes for teachers

Use this PR to leave feedback. Here are some tips:

Click the Files changed tab to see all of the changes pushed to master since the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.
Click the Commits tab to see the commits pushed to master. Click a commit to see specific changes.
If you turned on autograding, then click the Checks tab to see the results.
This page is an overview. It shows commits, line comments, and general comments. You can leave a general comment below.

For more information about this pull request, read “Leaving assignment feedback in GitHub”.

Hi Kym,
your solution is already quite good. You might want to look at the concurrence problems when using the HTTP client. If you fix this and submit a PR, you'll get the A+ grade.


peer review: <Promish Kandel Team 22> For all of the requirements their is no fit critrion or it isn't apparent

artifact under review.
SRS Volere Template for TinyMorph Sections related to functional and nonfunction requirements
team number (for the team doing the review). Team 22
description of issue.
For most of the requirements (lets take 12.1 as an example) there is no fit criterion.
In 12.1 PR-SLR1 its written:  TTFT should be minimum, around 200-500ms, you give a rationale for why you have this which is good. However, you also have to write how you know that specification is successful. Similar this is also the case for the one below. I would advise going to through all of the requirements and making it clear how you know when that requirement has been achieved.

Thanks for the review. For specific requirements, we will need to devise a benchmark setup to test these specific metrics.
We will update you accordingly. But I do think this doesn't apply to all requirements that don't have concrete metrics


Info box for ILB ASE and Private Endpoint apps can't utilize the Github Actions method.

Can we add a Info box under requirements, which states that the deployment method github actions does not work with ILB ASE and Function apps which have private endpoints and restrict public access. See doc here for more info.
https://learn.microsoft.com/en-us/azure/app-service/environment/using-an-ase#publishing
Can we also add the same for the DevOps page?
Thanks!

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

ID: 9a0ad9be-7b06-1d81-0339-3776b7fedca2
Version Independent ID: 83637bff-1ba9-3421-1380-41cfaa5138f1
Content: Use GitHub Actions to make code updates in Azure Functions
Content Source: articles/azure-functions/functions-how-to-github-actions.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@VinnyBonner
Thanks for your feedback! We will investigate and update as appropriate.


Demolish Single-Point Places

We currently have a ton of "Places" that are mere single dots on the map (eg https://www.inaturalist.org/places/ravenna-seattle-wa-us), which don't serve any purpose because they can't really contain any observations and they can confuse users who think they're selecting a bounded area in Search or when creating a Project.
As discussed (and I believe agreed-upon) during the Retreat, it would be good to delete these Places so they don't continue to sow confusion. Anyone disagree?

as a non-inat employee who is just a curator and 'power user', i
emphatically enforce this proposal! The point only places seem to cause
confusion.
(if you want me to let you all be and stick to the google groups let me
know, that is fine. Haven't seen as much non-admin activity in here lately)
On Wed, Mar 14, 2018 at 1:25 PM, tiwane<EMAIL_ADDRESS>wrote:

We currently have a ton of "Places" that are mere single dots on the map
(eg https://www.inaturalist.org/places/ravenna-seattle-wa-us), which
don't serve any purpose because they can't really contain any observations
and they can confuse users who think they're selecting a bounded area in
Search or when creating a Project.
As discussed (and I believe agreed-upon) during the Retreat, it would be
good to delete these Places so they don't continue to sow confusion. Anyone
disagree?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1689, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AG1tk-CtUnniRhDOhz-wUucrUKUZV3dDks5teVKDgaJpZM4Sq2yw
.

--
Charlie Hohn
Montpelier, Vermont

I wholeheartedly agree except sometimes a project is created by creating a kml place that somehow  doesn’t stick to the project.
Before removing single point places it would be nice to know that the project has no real bounding box so that project managers can correct the problem before seeing their project place disappear.
Yours in nature,
Ali
Alison Sheehey
PO Box 1832
Kernville, CA 93238

On Mar 14, 2018, at 10:30 AM, Charlie<EMAIL_ADDRESS>wrote:
as a non-inat employee who is just a curator and 'power user', i
emphatically enforce this proposal! The point only places seem to cause
confusion.
(if you want me to let you all be and stick to the google groups let me
know, that is fine. Haven't seen as much non-admin activity in here lately)
On Wed, Mar 14, 2018 at 1:25 PM, tiwane<EMAIL_ADDRESS>wrote:

We currently have a ton of "Places" that are mere single dots on the map
(eg https://www.inaturalist.org/places/ravenna-seattle-wa-us), which
don't serve any purpose because they can't really contain any observations
and they can confuse users who think they're selecting a bounded area in
Search or when creating a Project.
As discussed (and I believe agreed-upon) during the Retreat, it would be
good to delete these Places so they don't continue to sow confusion. Anyone
disagree?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1689, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AG1tk-CtUnniRhDOhz-wUucrUKUZV3dDks5teVKDgaJpZM4Sq2yw
.

--
Charlie Hohn
Montpelier, Vermont
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey Ali, can you provide an example of a Project that matches the situation described in your first paragraph?

I assume you also want to prevent the creation of such places in the future as well, right?

For sure.

Tony,
Here is an example of a single point place that actually has 3 linked projects: http://inaturalist.ca/places/ottawa-ontario-ca
Also worth considering is that some single point places have checklists, and even more problematic, guides associated with them.

I had this problem on some of my projects before I figured out there was no bounding box and the project would search North America or even the world. I went in and recreated kml polygons and removed the bad points. This happened when I was first created the Sequoia National Forest including giant sequoia national monument project. There were several faulty polygons but unfortunately I removed them to stop making the single point polygons show when trying to add others data to the project.
Yours in nature,
Ali
Alison Sheehey
PO Box 1832
Kernville, CA 93238

On Mar 14, 2018, at 12:38 PM, tiwane<EMAIL_ADDRESS>wrote:
Hey Ali, can you provide an example of a Project that matches the situation described in your first paragraph?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes. Absolutely the frustration with creating polygons only to find them in name only gave me pause. I am so glad you guys made the site so intuitive to repair problems easily. Keep meaning to thank all of you for your amazing work. I love iNat. It is such an awesome platform.
Yours in nature,
Ali
Alison Sheehey
PO Box 1832
Kernville, CA 93238

On Mar 14, 2018, at 12:41 PM, Ken-ichi<EMAIL_ADDRESS>wrote:
I assume you also want to prevent the creation of such places in the future as well, right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Can the admins query how many single point places have data associated with
them? If so and if the number is low maybe we could just give those ones
polygons, or at least leave only those and get rid of the others.
On Thu, Mar 15, 2018 at 12:17 AM, natureali<EMAIL_ADDRESS>wrote:

I had this problem on some of my projects before I figured out there was
no bounding box and the project would search North America or even the
world. I went in and recreated kml polygons and removed the bad points.
This happened when I was first created the Sequoia National Forest
including giant sequoia national monument project. There were several
faulty polygons but unfortunately I removed them to stop making the single
point polygons show when trying to add others data to the project.
Yours in nature,
Ali
Alison Sheehey
PO Box 1832
Kernville, CA 93238

On Mar 14, 2018, at 12:38 PM, tiwane<EMAIL_ADDRESS>wrote:
Hey Ali, can you provide an example of a Project that matches the
situation described in your first paragraph?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1689#issuecomment-373255788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG1tk3rIxfBd7b3J_Mg8cmMx454P8gmvks5teetjgaJpZM4Sq2yw
.

--
Charlie Hohn
Montpelier, Vermont

@cmcheatle: as far as I can tell, none of the three projects associated with the Ottawa you linked to need to be associated with it, as in they don't have rules set to it.
Guides make up a tiny amount of use on iNat, and they're also a feature we're not supporting any more. I suppose if the place were deleted, however, they would not show up in the "Near me" search, which would reduce usefulness even more.
I don't use checklists much, but I'm not sure how useful a checklist is for a single point.
@kueda, what would be the consequence(s) for a project and a checklist that are attached to a single point if that point is deleted?
@naturalistcharlie: I think that's a good idea. Can you specify what you mean by "data"?

I'm not sure about admins, but curators cant delete a single point place if there is a guide linked to it (I did a big clean up of such places in Ontario and was blocked from deleting 1 place linked to a guide). I did not try to see if I could unlink the guide from the place.

by data i mean it might be worth looking to see how many things were mapped
to that place in a way they would show up in the checklist. but i am now
wondering if point based places even have checklists.
On Thu, Mar 22, 2018 at 3:47 PM, cmcheatle<EMAIL_ADDRESS>wrote:

I'm not sure about admins, but curators cant delete a single point place
if there is a guide linked to it (I did a big clean up of such places in
Ontario and was blocked from deleting 1 place linked to a guide). I did not
try to see if I could unlink the guide from the place.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/inaturalist/inaturalist/issues/1689#issuecomment-375434716,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG1tk6snW_-5b-ipPDtasTQ3FeLbxxhgks5tg__XgaJpZM4Sq2yw
.

--
Charlie Hohn
Montpelier, Vermont

@kueda, what would be the consequence(s) for a project and a checklist that are attached to a single point if that point is deleted?

Very little. Things like https://www.inaturalist.org/projects/herps-of-texas/stats_slideshow would stop showing the place. Associated check lists would be deleted, which might cause a problem for the few cases where a user manually created those lists. Here are those cases:
www.inaturalist.org/check_lists/130116
www.inaturalist.org/check_lists/136241
www.inaturalist.org/check_lists/369891
www.inaturalist.org/check_lists/301987
www.inaturalist.org/check_lists/515646
www.inaturalist.org/check_lists/515785
www.inaturalist.org/check_lists/515794

www.inaturalist.org/check_lists/130116
www.inaturalist.org/check_lists/136241
www.inaturalist.org/check_lists/369891
www.inaturalist.org/check_lists/301987
www.inaturalist.org/check_lists/515646
www.inaturalist.org/check_lists/515785
www.inaturalist.org/check_lists/515794

These seven point place checklists represented 4 locations.  I merged 3 with existing boundary-defined places and created a boundary for the 4th.  So don't let these checklists get in the way of deleting point places....

Revisiting this. Current check lists created by real people that are associated with point-only places:
 place_id |          display_name           | list_id
----------+---------------------------------+---------
    59101 | Penjamo, GJ, MX                 |  757482
    52087 | Arroyo Park, Bellingham, WA, US | 2696704
     9247 | Indonesia, ID                   |  742597
     9920 | Cambridge, US, MA               | 1224228
    82288 | Sardinia, IT                    | 1550820

I merged 59101 with the Municipality Pénjamo.
I gave 52087 a kml outline borrowed from OSM.  (The city webpage is very unclear about the actual park boundaries.)
I'm really not sure what the user intended for 9247, but the list is empty and the user hasn't logged on since 2017, so I think it can be deleted.
Admin level 3 is required to edit 9920 (see https://forum.inaturalist.org/t/some-non-standard-places-are-not-editable-even-by-curators/11728), but it should be safe to merge it with 130435.
I merged 82288 with Sardegna (perhaps another reason to allow multiple names for the same place: https://forum.inaturalist.org/t/add-additional-names-to-the-same-place/5480).

Cool, I merged 9920 into 130435.


Add the ability for the user to have a profile picture

If user is not connected to Discord, the user should ideally be able to upload an image and set it as their profile picture in the profile page.
Example:

Profile pictures are nice.


Handle Steamcommunity Markdown Scams

Done in https://github.com/hammy275/immersivemc-bot/commit/8d589cf38e8cc67d821a528eaf7397d90b746333


[Wasm] Document.visibilityState is not available on Wasm but is on JS

kotlinx.browser.document.visibilityState is not resolved on wasm but is on JS.
Workaround:
@JsFun("(document) => document.visibilityState")
private external fun visibilityState(document: Document): String

Compose 1.6.2
Kotlin 1.9.23

This is the Kotlin stdlib issue. Could you file it here: https://kotl.in/issue

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


restore [block] media/autoplay flag

quite late, but would it be possible to reinstate code that was featured and then removed?
"Chrome 61 is out! Autoplay policy is now available via the chrome flag." Sept 6 2017
"Ultimately this switch got removed and the autoplay blocking in Chromium/chrome is back to being half or at times not working at all." Nov 24 2019
maybe there's another way around this issue

At this point that old code could be using apis that have changed or are no longer available and I don't think anyone here wants or is capable of maintaining that old code. Your best option would be to create an extension to achieve what you want.

See https://github.com/Eloston/ungoogled-chromium/pull/1565


WIP: ClusterPool Inventory

Add a ClusterPool.Spec.Inventory allowing enumeration of the complete
set of allowable ClusterNames a pool's ClusterDeployments may have.
HIVE-1367

We're going to do this a whole different way -- see #1607.