Mechanism such that HTTP clients can tell if a token expired or revoked

It would be really nice if this command could show a useful error message explicitly indicating if a token has been revoked or expired.

Originally posted by @simonw in https://github.com/simonw/dclient/issues/3#issuecomment-1638977178
Maybe do this with X-something HTTP response headers?

X-Auth-Token-Status: Expired I think.

Maybe X-Auth-Token-Expires-At: ... as well?

One example of something like this I can think of is GitHub's API showing rate limits:
x-ratelimit-limit: 60
x-ratelimit-remaining: 59
x-ratelimit-reset:<PHONE_NUMBER>
x-ratelimit-resource: core
x-ratelimit-used: 1

This is actually a bit tricky. The information that a token has expired or been revoked is available as part of the actor_from_request() hook here:
https://github.com/simonw/datasette-auth-tokens/blob/d5ecae97c124af840566ecf893b0a2a9df89a3b3/datasette_auth_tokens/__init__.py#L90-L111
But how to communicate that to the layer of Datasette that returns the final 403 response?
The one way I can see to do that is to attach the information to the request object here, and have some asgi_wrapper() middleware pick that up and annotate the response with it later on.

I couldn't find a way to get this to work.
Here's my experiment so far:
diff --git a/datasette_auth_tokens/__init__.py b/datasette_auth_tokens/__init__.py
index 9024223..7675c65 100644
--- a/datasette_auth_tokens/__init__.py
+++ b/datasette_auth_tokens/__init__.py
@@ -1,4 +1,5 @@
 from datasette import hookimpl, Forbidden, Permission
+from functools import wraps
 import itsdangerous
 import json
 import secrets
@@ -108,7 +109,13 @@ def actor_from_request(datasette, request):
             return None

         if config.enabled:
-            return await _actor_from_managed(datasette, incoming_token)
+            actor, invalid_reason = await _actor_from_managed(datasette, incoming_token)
+            if actor:
+                return actor
+            # Set invalid_reason so it can be returned with the 403 error
+            request.scope["datasette_auth_tokens_invalid_reason"] = invalid_reason
+            print(request.scope)
+            return None

         # First try hard-coded tokens in the list
         for token in allowed_tokens:
@@ -143,15 +150,16 @@ def actor_from_request(datasette, request):

 async def _actor_from_managed(datasette, incoming_token):
+    "Returns (actor, reason) where reason is a code explaining why the token was rejected"
     config = Config(datasette)
     db = config.db
     if not incoming_token.startswith("dsatok_"):
-        return None
+        return None, "invalid-format"
     incoming_token = incoming_token[len("dsatok_") :]
     try:
         token_id = datasette.unsign(incoming_token, "dsatok")
     except itsdangerous.BadSignature:
-        return None
+        return None, "bad-signature"

     # Potentially expire token first
     await db.execute_write_fn(make_expire_function(token_id))
@@ -162,7 +170,7 @@ async def _actor_from_managed(datasette, incoming_token):
     )
     row = results.first()
     if not row:
-        return None
+        return None, "not-found"

     actor = {
         "id": row["actor_id"],
@@ -175,11 +183,11 @@ async def _actor_from_managed(datasette, incoming_token):

     # Is token revoked?
     if row["token_status"] == "R":
-        return None
+        return None, "revoked"

     # Expired?
     if row["token_status"] == "E":
-        return None
+        return None, "expired"

     # Update last_used_timestamp if more than 60 seconds old
     if row["last_used_timestamp"] is None or (
@@ -190,7 +198,7 @@ async def _actor_from_managed(datasette, incoming_token):
             {"now": int(time.time()), "token_id": token_id},
         )

-    return actor
+    return actor, None

 def make_expire_function(token_id=None):
@@ -234,3 +242,38 @@ def render_cell(value, column, table, row):
             link="edit / revoke" if value == "L" else "view",
         )
     )
+
+
+@hookimpl
+def asgi_wrapper():
+    def wrap(app):
+        @wraps(app)
+        async def add_x_auth_token_status_header(scope, receive, send):
+            print("Scope before", scope)
+
+            async def wrapped_send(event):
+                if event["type"] == "http.response.start":
+                    headers = event.get("headers") or []
+                    print("Scope after", scope)
+                    reason = scope.get("datasette_auth_tokens_invalid_reason")
+                    if reason:
+                        headers.append([b"x-auth-token-status", reason.encode("utf-8")])
+                    event = {
+                        "type": event["type"],
+                        "status": event["status"],
+                        "headers": headers,
+                    }
+                    print(event)
+                await send(event)
+
+            await app(scope, receive, wrapped_send)
+
+        return add_x_auth_token_status_header
+
+    return wrap
+
+
+@hookimpl
+def forbidden(datasette, request, message):
+    print("datasette", datasette, "request", request, "message", message)
+    return None

The problem was passing the information from inside the actor_from_request() hook to a place where it could be returned as an additional HTTP header.
Assigning to request.scope["datasette_auth_tokens_invalid_reason"] didn't work, because the scope that was seen by asgi_wrapper() in the wrapped_send() function did not include that new key. Presumably this is because ASGI scopes are copied rather than passed by reference at various points in the stack.
I was hoping the forbidden() or handle_exception() hooks would be called, but they weren't - at least according to my debug output.
I think I would need to make changes to Datasette core - maybe adding a new, slightly lower-level plugin hook (that can still see request/response objects as opposed to asgi_wrapper() which fires at a level above those).

Marking this as "won't fix" for the moment. I'll open a Datasette issue about this too.


[PE-D][Tester A] Grammar error in DG under Launch

Should be "The window size may not be optimal"

Labels: type.DocumentationBug severity.VeryLow
original: FriedCabbageSalad/ped#24

Thanks for highlighting. We have now resolved the issue.


[Feature] 希望支持vLLM和Ollama/Hope to support vLLM and Ollama

Motivation
[Feature] 希望支持vLLM和Ollama/Hope to support vLLM and Ollama
Related resources
No response
Additional context
No response

We appreciate your interest in these features. Unfortunately, they are not yet supported. We are taking your feedback into consideration and will prioritize these features for future development. Please stay tuned for updates.

I found vllm has supported internvl2? details in:
https://github.com/vllm-project/vllm/releases/tag/v0.5.4


[fix][meta] Follow up #19817, Fix race condition between ResourceLock update and invalidation

Motivation
This PR is following up the PR #19817.  It includes three parts as follows:

Fix the previous PR's incompletable future problem.
Split revalidate into two methods to eliminate some flags.
Add strict revalidation state check to avoid revalidating expired lock.

Modifications

Fix the previous PR's incompletable future problem.
Split revalidate into two methods to eliminate some flags.
Add strict revalidation state check to avoid revalidating expired lock.

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

@merlimat  @codelipenghui @Jason918 Please take a look. Thanks!

The test has passed. And the original test updateValueWhenVersionIsOutOfSync will cover the future incomplete problem.
Could you please take a look?
/cc @merlimat @codelipenghui @hangc0276 @Jason918


[RESTEASY-1227] add testcase, tests stabilization

@kanovotn hi, is a corresponding PR against master needed?

@asoldano https://github.com/resteasy/Resteasy/pull/1194

@asoldano I have also cherry picked two commits below from jimma from master branch for ResponseStreamPrematurelyClosedTest. If it was merged intentionally only to master I will remove them.
https://github.com/resteasy/Resteasy/commit/157eeee1afda2e520a590c6699b2ccd926841c66
https://github.com/resteasy/Resteasy/commit/8365522d139867a60fa2606f12ed3873d08f4c5c

@kanovotn well, as long as tests only are backported and they pass,  I have nothing to complain...however, the strange thing is that those tests should be related to / go with https://github.com/resteasy/Resteasy/commit/f2b259976ef041933e91746bde2b28957ba093ef , which is what I didn't backport to 3.0 on purpose. The fact that the tests pass even without that commit is "interesting"...

@kanovotn it might even well be that there's something I'm missing / I don't remember about that jira

@asoldano I think the testcase ResponseStreamPrematurelyClosedTest is related to this jira RESTEASY-1138 not RESTEASY-1650. RESTEASY-1138 was fixed in 3.0.23 and I my local test run fails as expected with 3.0.22.

@kanovotn ah, right, now I remember. Jim provided additional test.
Thanks, merging then.


XPK blueprint updates to make it more compatible with the tool

Minor fixes for xpk compatibility

Update the naming convention of the node pools
Name the config map with deployment name

Should TAS be left here? @ankitkinra

Should TAS be left here? @ankitkinra

We can potentially update in a follow up PR


some .s scripts not generated from #pragma, unknown pattern

After updating to the latest version and attempting a split to my repo, some functions were not being split into .s scripts. I have noticed no discernible pattern as to which are omitted.
The repo in question: https://github.com/blackgamma7/mischief-makers
omitted functions include func_8004F514 and ActorMarina_Roll.

this often happens because the assumed name of the function doesn't match the function you have in the repo. but I can take a look. Is this happening on the master branch?

correct, master branch.

I'm getting FileNotFoundError: [Errno 2] No such file or directory: 'tools/asm-processor/prelude.inc', and indeed this file isn't there

also it doesn't appear the master branch has any issues creating ActorMarina_Roll - looks like the update isn't on master?

local clone, not been committed or pushed, yet. only notable difference is that splat and asm-processor were updated to the latest. Haven't pushed it as I wanted to bring up the issue first.

okay. yeah reproduction steps (or a branch I can switch to) is what I was looking for. I'll update asm-proc and splat and see how it goes

func_800128F0 isn't splitting for me on master. do you see this behavor? (I updated asm-proc but not splat)

func_800128F0 splits, but not func_80087568.

btw, are you in decompals / on discord anywhere? It'd be easier to sort this out there

Closing since we discovered the cause of this: incorrectly-set size property on preceeding functions in symbol_addrs


[Network] Need for Asynchronous Functionality

All network functions are blocking at the moment, async functionality is necessary for effective parallel I/O communications.

Added asynchronous network functions that can be used along side the synchronous functions. The code has been tested and is considered stable.


tqdm invalid unicode char, changed to ascii style

TQDM progress bar doesn't display unicode characters correctly on my system as per:
https://github.com/tqdm/tqdm#faq-and-known-issues

Windows consoles often only partially support unicode and thus often require explicit ascii=True (also here). This is due to either normal-width unicode characters being incorrectly displayed as "wide", or some unicode characters not rendering.

Changed to ascii to avoid this.

@Cam000 feel free to rebase and merge, or I can do this for you.

@Cam000 feel free to rebase and merge, or I can do this for you.

@PaulMcInnis Gotcha, it would be easier if you could make the change, I mangled my local repo :)

@PaulMcInnis Gotcha, it would be easier if you could make the change, I mangled my local repo :)

No worries, thanks for the contribution.

No worries, thanks for the contribution.


Add support for RedHat distros (and Nobara)

It might even be a good idea to set firefox_package_name: firefox as a default for any other distros than Debian because most use Firefox.

The error message for unsupported distros:
TASK [staticdev.firefox : Install Firefox] **************************************************************************************************************************
fatal: [localhost]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'firefox_package_name' is undefined. 'firefox_package_name' is undefined\n\nThe error appears to be in '/home/red/.ansible/roles/staticdev.firefox/tasks/main.yml': line 34, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Install Firefox\n  ^ here\n"}

The error message might confuse ansible noobs like me but I don't know if there is a good way of changing it.

@Commandcracker thanks for the contribution. In order to add support to new distros we also need to add tests, metadata and on readme that it is supported. Could you please also add them?

I have only added "RedHat" platforms that have firefox as their firefox package.
(RHEL has no official firefox package. There is only one with noscript for Enterprise Linux 7)
I'll lake a look into testing with molecule.

I wasn't able to run the workflow because act wasn't able to install podman in a docker container :D but I double checked everything.

@Commandcracker there are some last linting errors.. also, I think you will need to revert Ubuntu 22.04 because of a bug that I had commented in the issue you removed.

yamllint AnsibleLintRule's seam to be outdated
I used get-galaxy-platforms to see if they are valid and 38, 37 are in there.

yamllint AnsibleLintRule's seam to be outdated I used get-galaxy-platforms to see if they are valid and 38, 37 are in there.

I doubt the problem is on ansible-lint.. the link you sent me was updated 4 years ago. I would suggest you replace then 37 and 38 with all as in the example you sent me. So we don't need this disable lint.

You need to run get-galaxy-platforms.py, it fetches the latest platforms from ansible-galaxy and after you did that, you can see that 38, 37 are in there.

Good that I knew that ._.
"This schema rule is mandatory. You cannot use inline noqa comments to ignore it."

@Commandcracker seems like Ubuntu 22.04 requires extra effort to make it work because of default snap installation. I also removed Debian 10 and added Debian 12. I would prefer if you can remove changes related to Ubuntu and Debian from your PR and keep only RedHat/Nobara related. You will need to rebase with main branch.

@Commandcracker I ended up investing some time and incorporated your changes through:

https://github.com/staticdev/ansible-role-firefox/pull/173
https://github.com/staticdev/ansible-role-firefox/pull/174

So you don't need any changes and should work for you. I am releasing now a version 2.0.0 of this role.


Lack resources in tools section for web development

The tools section of our web development resource repository is currently in need of additional resources. Your task is to add a valuable resources that can be included in the tools section.
You can suggest as many resources as you like.
Categories that lack resources:

CSS animation
Chrome Extenstion
Code Playground

I would like to work on this issue.

I would like to work on this issue.

Assigned to you.

Please assign me with this issue I want to contribute.

You can check on this issue #81

I would like to work on this issue.

Please make sure that you at least add 1 resource in each category

I would like to work on this issue.

Please make sure that you at least add 1 resource in each category

Sure, I am adding at least 5 resources minimum in each category mentioned above.

#87


Empty Clause issue for Vanilla Installation

BTW Version: BTW CE 2.1.4
Multiplayer (yes/no): No
Issue (be as specific as possible): Loading error
Steps to Reproduce (even if obvious): Booting the game with the mod.
Addons (with versions): None outside what comes with it.
Crash Log (if there was a crash):

Image of the crash log as this was what it said. And no, I can't use other loading methods. MMC has it's own weird issue and other methods just don't even register the mod.

BTW CE is for Minecraft 1.5.2, there's a reference to 1.18.2 in your log

Follow the install guide here: https://wiki.btwce.com/index.php?title=Installation

Follow the install guide here: https://wiki.btwce.com/index.php?title=Installation

I followed the guide to a tee, the version even said I loaded up 1.5.2.

Follow the install guide here: https://wiki.btwce.com/index.php?title=Installation

I followed the guide to a tee, the version even said I loaded up 1.5.2.

Post a full log, screenshots are basically useless to us

The latest log is from 2023, and the MMC logs show that it boots, but don't
show the bizarre texture glitch that always happens when loading it,
The exit code for the vanilla launcher is always 1.
On Sun, Feb 18, 2024 at 9:43 PM Simon Meskens @.***>
wrote:

Post a full log, screenshots are basically useless to us
—
Reply to this email directly, view it on GitHub
https://github.com/BTW-Community/BTW-Public/issues/510#issuecomment-1951599209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A46I3E7EU6DMWYDUSNTH453YUK36DAVCNFSM6AAAAABDOUNCRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRGU4TSMRQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

The texture glitch is solved easily by Neodymium for BTW. Join our Discord and we can help you figure out this issue, but it doesn't sound like it's a bug on our end.


[WIP] fix flaky UT

Fixing flaky UT, and removing VA logic since SNR does not delete VA's anymore

/test 4.14-openshift-e2e

Can you separate fixing flaky UT, and removing VA logic into two commits? It would be easier for future us to know if flaky UT is related to VA logic. Do you happen to understand why this unit test is flaky?

Can you separate fixing flaky UT, and removing VA logic into two commits?

Sure done.

Do you happen to understand why this unit test is flaky?

Not sure if you refer to a specific failure regarding the VA or in general.
In any case in general most of the flakiness is due to sync issues (i.e mostly fetching a resource before it has reached the desired state).
Regarding the VAs it seems like a waste of time dealing with syncing with them when they are not needed in any case anymore.

/test 4.14-openshift-e2e

/lgtm


Add delete for label & field matchers

Requires looping through matches on client side, sending delete for each matching resource.

Fixed by e59acba18b48007cf2bd8ae201ce5983cb58c96f


[BUG] - No styling

NextUI Version
"@nextui-org/date-picker": "^2.1.8",
Describe the bug
After opening calendar, there is no styling, calendar background is transparent
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
npm install @nextui-org/date-picker
import { DateRangePicker } from "@nextui-org/date-picker"
npm run dev
Expected behavior
should look like in demo

but is like

Screenshots or Videos
No response
Operating System Version
Windows
Browser
Chrome

no tailwind conf added as required


Use cmp to make it easy to see what is broken in kfservice_controller_test

Got want is so much easier than trying to cmp to dumps of json... (this test appears to have race and fails on occasion for me... this is the first step towards figuring out what is going on)
This change does need "github.com/google/go-cmp/cmp/cmpopts", so can't be merged until that is in!
/assign @yuzisun

This change is

https://github.com/kubeflow/kfserving/issues/209

@rakelkar, looks fine to me? Can you post before and after? The issue isn't super clear

/retest

@ellis-bigelow  I've posted what it looks like right now in the issue.. after my fix it will look like this:
-before
+after

The test is failing because my PR doesnt add the required dependency (need to do that in a separate PR):
github.com/google/go-cmp/cmp/cmpopts

Gotcha. Needs rebase too. I'm happy approve as soon as those two are complete.

/retest
On Thu, Jul 11, 2019, 8:12 PM Kubernetes Prow Robot <
<EMAIL_ADDRESS>wrote:

@rakelkar https://github.com/rakelkar: The following test failed, say
/retest to rerun them all:
Test name Commit Details Rerun command
kubeflow-kfserving-presubmit 80de61e
https://github.com/kubeflow/kfserving/commit/80de61e411b27c9bba8222c460f34fae86e16e63
link
https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/kubeflow_kfserving/207/kubeflow-kfserving-presubmit/1149516667479396356/ /test
kubeflow-kfserving-presubmit
Full PR test history
https://prow.k8s.io/pr-history?org=kubeflow&repo=kfserving&pr=207. Your
PR dashboard https://gubernator.k8s.io/pr/rakelkar. Please help us cut
down on flakes by linking to
https://git.k8s.io/community/contributors/devel/sig-testing/flaky-tests.md#filing-issues-for-flaky-tests
an open issue
https://github.com/kubeflow/kfserving/issues?q=is:issue+is:open when
you hit one in your PR.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/207?email_source=notifications&email_token=AA2CTBIXGLRPYXGLN4I3723P67ZC5A5CNFSM4H4L3YQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZYRSOQ#issuecomment-510728506,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2CTBK7PXILWJGRH5P5XBTP67ZC5ANCNFSM4H4L3YQA
.

@rakelkar needs rebase to get the presubmit fix

@ellis-bigelow @yuzisun this is ready to merge!

merging !
/lgtm
/approve

Traveling this weekend. Will catch up tomorrow.
On Sun, Jul 14, 2019, 11:30 AM rakelkar<EMAIL_ADDRESS>wrote:

@ellis-bigelow https://github.com/ellis-bigelow @yuzisun
https://github.com/yuzisun this is ready to merge!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/207?email_source=notifications&email_token=AATXLCEM5BJQCVJ5AK4PTLLP7NWFHA5CNFSM4H4L3YQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ4K2EA#issuecomment-511225104,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATXLCHQMMXANPRWKRN2A43P7NWFHANCNFSM4H4L3YQA
.


React Native

This module does not work with react native, triggers Unexpected end of script. I have no idea what's causing it tho :(

var Hoverboard = (function(){ .... })();
module.exports = Hoverboard;

Seems to fix the issue. not sure what's causing it tho

@dzannotti Thanks for taking the time to report this. I haven't had the chance to use React Native yet. Are you able to put together a test case to demonstrate the error?

no need :)
npm install hoverboard

then just:
var Hoverboard = require('hoverboard');

triggers a red screen of death which goes away once the above "patch" is applied. Still not sure why

Weird, I'll try to figure it out soon, thanks for the details. Hopefully the workaround lets you work around this in the meantime, so to speak :)

Yup, no rush.
I'm 99% sure it has to do with the way react-native packager works (doesn't like a function being exported rather than an object maybe?)

Yeah I'd be interested to figure it out, if you're right then it sounds like it may actually be a bug in react-native.

@dzannotti Dumbest thing, it was the // comment at the end of the file. Adding a newline to the file fixed it. I'm guessing it had to do with JavaScript files being concatenated. Thanks again!

Awesome to know.
I didn't really think about that, because i have "auto add newline at end of file" when i save changes in my editor. But yes probably has to do with the way the files are concatenated by the packager


seems to be no longer working?

Hi,
I tried it 1h after you opened the issue, and could not detect the problem. Tried again few hours later, it works. Tried it on 3 different systems, and it works again.
Since the plugin is using Basic authentication, the only explanation I have is Atlassian has shortly changed some policies in their codebase, and it was affecting Basic Authentication on all Jira instances.
If it occurs again, I might rewrite auth part to token-based auth.

I now get this error
`popup.js:52 Uncaught (in promise) TypeError: Cannot read property 'issues' of null(…)``
Also, it logs me out of the jira page whenever i try to use the plugin.
Is username the email or the actual user name ?

Hello!  I tried using this on Chrome 85.0.4183.102 on macOS Catalina (10.15.6).  It logs in and successfully pulls down all the tasks according to the filter, but when I attempt to log a work item it clears the time in the Chrome extension UI, but nothing gets added to the actual work log in JIRA.   Any thoughts on how I can see what is breaking?

try https://github.com/fyyyyy/Toggl-to-Jira-Chrome-Extension/


Fix long topic display in nav bar

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Fee transaction data types

:clipboard: Add associated issues, tickets, docs URL here.
Fixes omisego/Feat-fee#27
Overview
Add structures needed for fee transaction
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Change 1
Change 2
...

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

Coverage remained the same at ?% when pulling 53032fd5b34533da2c21f56fe0ae340419cdbdd4 on papa/fee-27_fee_tx_data_types into 906d99717421d2ee84686556ddb46fe00aa5acdb on master.

oh, and I liked the FeeTokenClain or FeeClaim as transaction type name much better. Transaction.Fee and related names imply something totally different is meant.


mkdocs build error with python 3.7

I get the following error
+ mkdocs build

Traceback (most recent call last):

  File "/usr/local/bin/mkdocs", line 8, in <module>

    sys.exit(cli())

  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 1128, in __call__

    return self.main(*args, **kwargs)

  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 1053, in main

    rv = self.invoke(ctx)

  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 1659, in invoke

    return _process_result(sub_ctx.command.invoke(sub_ctx))

  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 1395, in invoke

    return ctx.invoke(self.callback, **ctx.params)

  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 754, in invoke

    return __callback(*args, **kwargs)

  File "/usr/local/lib/python3.7/site-packages/mkdocs/__main__.py", line 244, in build_command

    from mkdocs.commands import build

  File "/usr/local/lib/python3.7/site-packages/mkdocs/commands/build.py", line 15, in <module>

    from mkdocs.config.defaults import MkDocsConfig

  File "/usr/local/lib/python3.7/site-packages/mkdocs/config/defaults.py", line 4, in <module>

    from mkdocs.config import config_options as c

  File "/usr/local/lib/python3.7/site-packages/mkdocs/config/config_options.py", line 30, in <module>

    import markdown

  File "/usr/local/lib/python3.7/site-packages/markdown/__init__.py", line 29, in <module>

    from .core import Markdown, markdown, markdownFromFile  # noqa: E402

  File "/usr/local/lib/python3.7/site-packages/markdown/core.py", line 26, in <module>

    from . import util

  File "/usr/local/lib/python3.7/site-packages/markdown/util.py", line 87, in <module>

    INSTALLED_EXTENSIONS = metadata.entry_points().get('markdown.extensions', ())

AttributeError: 'EntryPoints' object has no attribute 'get'

Hi @ktamilvanan
By the look of the stacktrace you shared, this error doesn't occur in my plugin, but in MkDocs itself. Anyway even if it occurred because of my plugin, I am personally not going to maintain support for Python 3.7 anymore. A few days ago Python 3.11 was released, and I recommend upgrading, if possible.


US02.02.01

As a claimant, I want the list of expense claims to be sorted by starting date of travel, in order from most recent to oldest, so that ongoing or recent travel expenses are quickly accessible.

Working on this for next < 30 min


feat: add rest api tool

Some unfinished thoughts:

Should this allow atypical use cases like passing path path parameters to a POST? Probably couldn't hurt.
How should this handle the many types of schemas that could appear on the same resource for a given method? For instance GET posts, GET posts/1, GET posts/1/comments. Maybe it should take in a swagger definition? That could get tricky with token limits, but maybe ramps help here.

README examples based on this: https://jsonplaceholder.typicode.com/guide/

@collindutter, could you move README to griptape-docs please? Otherwise, it looks good.

@vasinov done! https://github.com/griptape-ai/griptape-docs/pull/23


Multi-monitor Setup cause Hitbox Misdetection

Sigma.js version : 3.0.0-beta4

Graphology version : 0.23.0

Operating system: Windows 10

Web browser: Chrome 120.0.6099.200, Edge 120.0.2210.121

Steps to reproduce

Have a dual monitor setup (e.g. left-right configuration)
Run the demo on the Secondary Display

Expected behavior

Hovering over the node should highlight the node

Actual behavior

Hovering ~1 inch to the right and bottom of the node will highlight the node

https://github.com/jacomyal/sigma.js/assets/553816/89ca108f-dd77-4f54-aed9-f2b450f419fe

Code Sandbox reproduction

<EMAIL_ADDRESS>

Hello @cardin,
Thanks a lot for your report. I can't reproduce your issue. I wonder if the cause wasn't #1399, which has been solved for version 3.0.0-beta.5. Is it still occurring for you?

Hi Jacomyal, I'm not able to reproduce the issue with 3.0.0-beta.5, it should be fixed now. Thank you!


Error when deleting slice that's already deleted

When running a command to delete a slice, if the slice is already deleted the system throws an error. Of course it does not make sense to delete the same slice twice but I don't see the harm of the user being able to run the delete slices command without getting an error, at least not such a large one. In my opinion it would be more user friendly if there was a way to make this more 'quiet' so that it would not appear as an external error message but instead a command to be ignored in the background.

I like the idea of making this (and all other) functions "quiet".  Maybe there could be two modes or a simple way to extract a simple error statement.

Fablib API slice.delete() is quiet and doesn't throw error on deleting an already deleted slice.


Add option to load already unzipped FMU

Add new function for loading an alraedy unzipped FMU: fmi4c_loadUnzippedFmu(instanceName, unzippedLocation)
Unzipping the FMU is moved to a separate function. The unzipped path is only removed in fmi4c_freeFmu() if it was generated as a temporary path.

@lochel Will this work for OMSimulator?

This looks very good. I will do a test implementation in OMSimulator and get back to you.


Project metadata update and CI additions

Codecov Report

Merging #10 (b696c2f) into main (c81d318) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   99.64%   99.64%
=======================================
  Files          12       12
  Lines         277      277
=======================================
  Hits          276      276
  Misses          1        1

Flag
Coverage Δ

unittests
99.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/nwave/core.py
97.88% <ø> (ø)

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 c81d318...b696c2f. Read the comment docs.


Are application Insights event logs applicable for custom policies as well?

The section https://learn.microsoft.com/en-us/azure/active-directory-b2c/custom-policy-developer-notes#developer-interface
has a note "Used to monitor user flows in production."
Should it be "Used to monitor user flows and custom policies  in production."?
[Enter feedback here]

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

ID: a067ceb2-fe69-6b56-eafa-ffb7ebf1968e
Version Independent ID: 6fd66d79-08fa-b5ea-daf0-d92073269b64
Content: Developer notes for user flows and custom policies - Azure AD B2C
Content Source: articles/active-directory-b2c/custom-policy-developer-notes.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@michael-freidgeim-webjet
Thanks for your feedback! We will investigate and update as appropriate.

@SaibabaBalapur-MSFT
Please also check if Preview is correct. The linked articles do not state that they are not in GA

@michael-freidgeim-webjet
Thanks for reporting this! We have created a PR for this issue once the author will review and approve the changes should go live soon.


Don't closing intent on scan

Hi,
Is it possible to NOT close the intent when a barcode is scaned?

There is a sample with a custom activity doing that on the 3.x branch.
On 13 Apr 2015 11:25 PM, "Rick de Jong"<EMAIL_ADDRESS>wrote:

Hi,
Is it possible to NOT close the intent when a barcode is scaned?
—
Reply to this email directly or view it on GitHub
https://github.com/journeyapps/zxing-android-embedded/issues/35.

Ah! Great! I'll wait till it is stable.


Edge Weighting

Provide the ability to weight the individual edges, e.g. the ability to take densities and assign those densities to the individual edges.

This functionality is available the current iteration of the project (spaghetti), and was available in the final version of network. "Officially" closing this issue with pysal/spaghetti#171.


Internal Transaction Triggers

How can an internal tx be defined as a datasource?

address of the contract being watched
function signature

What arugments can be provided to internal tx handlers?

call input
call output
from address
value sent
call type
gas used

Ordering of triggers between internal txs and events
We need a canonical order to have the index be deterministic in the decentralized network. If the event and the internal tx occur in separate txs then the tx order should be used to order triggers. If the event and the internal tx occur in the same tx there are some options:

Let the mapping author specify the order either implicitly through the index of each datasource or explicitly through some new manifest properties
Provide a canonical order for events and internal txs occurring in the same tx
Inspect the call chain within the tx to see if the event and internal tx occur in the same link. If the event and the internal tx happen in separate links of the call chain, then respect that order. If the event and the internal tx happen in the same link of the call chain, provide a canonical order

To inspect the call chain, we need to have access to the opcodes run in the VM trace. Geth labels each step of the VM with the OPCODE name, but parity-ethereum does not as far as I can tell (the labels for the opcodes might be hiding in some of the hex they provide for the trace).
To inspect the call chain we need to
State used for view functions called in a mapping triggered by an internal tx
It seems that a primary use case for internal tx triggers is when the contract being indexed does not already package the delta in state into a log event. A mapping has two options for calculating this state delta in the absence of a log event:

Use the input and output of the call combined with a priori knowledge of the logic in the contract to produce an entity
Run view functions against the contract before and after the tx has been executed to calculate a delta and produce an entity

To run view functions before and after a tx, you need the state of the contract just before the tx producing the trigger and just after.
To get this state you need to get the contract state at the end of the previous block, replay the txs up until the trigger tx, take note of the pre tx state, execute the trigger tx, and take note of the post tx state. Geth and parity-ethereum's tracing apis give you access to these pre and post tx states. With these two states, then spin up an EVM, seed a state, and run the desired view functions.
A shortcoming of the parity-ethereum tracing api is that if a single tx calls out to the contract twice and produces two internal tx triggers, we need to manually instrument the EVM to capture the incremental changes in the state before and after each internal tx. Geth gives you the ability to track the storage of a contract after each opcode.

A couple of outstanding questions:
Which ethereum-node should we use?
An archive node is needed. Geth has better tracing capabilities, but we would need to write a rust client for it.
We could also work with Parity to extend the tracing api to provide opcodes in the VM trace and track the state of the contract at incremental steps within a transaction.
What is the performance of syncing the index for a heavily used ERC20 contract leveraging internal txs?

Related issues: #298 #788
One thing that's not obvious to me is how the ethereum API will be used to listen to the internal transactions. Does that API already give us the ordering among internal transactions?
It seems that even when inspecting the call chain there are cases where we can't tell exactly what the proper order should be, I worry that those edge cases will make that approach too complicated for users to rely on. Maybe we could start with a simple canonical ordering like: In each transaction, process all internal transactions first, and then all events. With time we'll see what are the users' needs wrt the ordering.

State used for view functions called in a mapping triggered by an internal tx

Could we punt on this part and first focus on having the triggers at all?

@leodasvacas yeah, good questions.
A parity-ethereum archive node with the --tracing flag on let's you search a traces database using filters. It's similar to how we currently query logs except that you can only use a block range and the addresses involved in the call.
The returned traces include the input field for the call, so you can apply another filter for the function signature the subgraph is subscribed to. You can't apply this filter on the parity-ethereum side right now though, but we could request it or submit a patch upstream.
As far as ordering, I agree, I'd like to choose a convention initially. This way we don't have to do any heavy EVM replay to produce triggers.
We might be able to punt on intermediate state for view functions. The first use cases for internal txs are erc20 tokens without Transfer/Mint events and Gnosis Safe, so the next thing I have to do is determine if those subgraphs can work on only the IO provided to the internal tx and the post block state.


Refresh php, add protobuf types

Could you provide more information about how this PR affect the release process of PHP such as:

Is now the PHP clients self-contained after this PR?
Will the protobuf types be synced into google-cloud-php? If we publish these protobuf types as separate package, can the process be automated based on the current structure?
Will the CI tests depend on the local proto types?

Is now the PHP clients self-contained after this PR?

Yes, the PHP clients are now self contained

Will the protobuf types be synced into google-cloud-php? If we publish these protobuf types as separate package, can the process be automated based on the current structure?

Eventually yes, but currently no. Right now we are still using proto-client-php to hold the protobuf types. Once the php_namespace option is included in a protobuf release, we will be able to distribute the protobuf types in google-cloud-php. I think that automating the process based on this structure would be possible, but probably not desirable given that it will not be used after we can make the change to distributing in google-cloud-php

Will the CI tests depend on the local proto types?

I had not fully considered this, but yes that can be made to work. I have updated to PR to remove the dependency on proto-client-php, to ensure that the protobuf types used will be the ones distributed locally.

I plan to add CI for PHP staging soon (probably next week)
This can be done in a separate PR but I would recommend PHP to stage common-protos (like Java) so all the proto dependencies are locally staged for testing.

That makes sense - I will add that in a followup PR. It should only be changes in the googleapis settings, and an addition to the composer.json file


the window can't be moved if call this->winId() in the widget

If I add this line here the window can't be moved by dragging the title bar:

Window can't be moved if I drag the title bar:

Built by qt 5.15.10_msvc2019_64, this bug maybe happen in the Qt source code, who can help with this bug? Thanks.

This is a big problem, a frameless window will not receive WM_HITTEST if it has a native child window, I have been researching for a long time but now can't find a solution. If you really need to move an outside widget into the frameless window, you could wrap the widget with another frameless container.

Maybe the issue is fixed, I'll push very soon.


Remove APIv1 KEEPALIVE log

I don't really see the point of this log message and my logs always end up polluted with mostly this message, so it would be nice to just remove this instead of piping everything to /dev/null like I do now.

@ian-kent is this still maintained?


how add import java.awt.Robot in Android studio

hi i using android studio.i want added your project to android studio but java.awt.Robot library is not added please help me

Hi,  I'm using intelij Idea.
too simple to add project.
just must import the project and after that, add the using library.

@kubraaa: For the server, you need to use eclipse/intellij I think, as it is a java application not Android.

ok i will use eclips for server, well can i use android studio for client ?


Remove Picom, Pulse and add Pipewire

It only causes harm (huge perfomance issues) in desktop environments like KDE Plasma and GNOME, both have their own compositors so better removed it from the list

Would be better to install Picom per-DE if you wish, it would make sense in XFCE4 if you also made a config to disable the XFWM built-in compositor by default, the same on KDE (KWIN), running two compositors at the same time will completely obliterate performance

Also another tip would be to include a option to select the GPU brand you use, for AMD for example you need to use the vulkan-radeon package instead of AMDVLK, vulkan-radeon (and lib32-vulkan-radeon) is the driver supported by Valve for AMD GPU's, AMDVLK is the official AMD driver that is super slower and lacks important Vulkan extensions for DXVK and VKD3D-Proton.

Update: Also added Pipewire and Wireplumber to the PR

Agreed about picom had major performance issue with it a while back


Security Incident Reports Collection

Security Incident Reports Collection

[x] https://stackoverflow.blog/2021/01/25/a-deeper-dive-into-our-may-2019-security-incident

https://stackoverflow.blog/2021/01/25/a-deeper-dive-into-our-may-2019-security-incident
Key Takeaways as Fundamental Security Practices

Log all your inbound traffic.
Use 2FA.
Guard secrets better.
Educate engineers that “secrets aren’t just passwords.”
Protect SSH keys and database connection strings too.
Dont store secrets in Git repo. Use git-crypt or Blackbox if you must.
Validate customer requests. The more unusual a request from a customer, the more important it is to verify whether or not the request is legitimate.

Take security reports seriously.
“Code as if everyone is the thief.” - http://thecodelesscode.com/case/140

https://stackoverflow.blog/2021/01/25/a-deeper-dive-into-our-may-2019-security-incident
Key Takeaways as Fundamental Security Practices

Log all your inbound traffic.
Use 2FA.
Guard secrets better.
Educate engineers that “secrets aren’t just passwords.”
Protect SSH keys and database connection strings too.
Dont store secrets in Git repo. Use git-crypt or Blackbox if you must.
Validate customer requests. The more unusual a request from a customer, the more important it is to verify whether or not the request is legitimate.

Take security reports seriously.
“Code as if everyone is the thief.” - http://thecodelesscode.com/case/140


Visual manifest assembler

For simple cases it would be great to be able to compose chunks of manifest visually rather than having to tangle with the YAML.  (Though VS Code may not be the right environment for this, what with, you know, the whole TEXT-BASED TEXT EDITOR SPECIALISED IN TEXTUALLY EDITING TEXT thing.)

TEXT-BASED TEXT EDITOR SPECIALIZED IN TEXTUALLY EDITING TEXT

What you trying to say, @itowlson? 😂


Change DI For the HttpClient

Change the DI for the HttpClient so it is injected for the specific SessionizeApiClient object

Fixed


Test: Revert changes to ubuntu 24.04 for cancelled jobs

Seems like others are reporting early cancellations in here: https://github.com/actions/runner-images/issues/9848


core: fix MSVC compiler error

Somehow, on one machine these changes were needed in order to avoid MSVC complaining about:
warning C4346: 'iterator': dependent name is not a type

Fixed #719.

I have no idea why, good that you found this out.

Neither do I!


Fix/1990/confirmation on taking over evaluation

Targets #1990
Description

We are add evaluate button. When click the evaluate button then open the popup model. In this popup model have two button one is request  and another is cancel. When clicking the request button then send a mail request to existing assignee for Application Handover request link with Application Id. Upon clicking this link, the existing assignee will be redirected to the portal. Show on portal Application is assignee to requested person.

Checklist:

[ ] I have performed a self-review of my own code.

@aquibmoin786 Fix the Checks


Closed regressed bug marked as affected

A bug should not have the state of verified fixed and firefox# set to affected.
https://bugzilla.mozilla.org/show_bug.cgi?id=1799208#c14

Thank you @kbrosnan for reporting this. I agree with you; I think the nigtly version should be marked as fixed once the bug is closed as fixed. We could make the bot automate that.

In bugs that are fixed by a commit to mozilla-central, the flag is set automatically.
This is only for cases where a bug is marked as fixed "manually". Maybe in such cases it is better not to touch the flags, as there is a risk that the person who set the bug as fixed made a mistake and we might want a human to manually set the flag.
We could ask release management's opinion about this.

In the case of bug 1799208, Firefox 108 should have been marked fixed when the bug was closed, then the bot wouldn't have prompted.
Instead, it was marked disabled, which wasn't the right status, since a backout had been fully merged to central stopping the crash.


Sign in automatically authenticates email of an user that did not authenticate his email

Bug report
Describe the bug
await supabase.auth.signIn({
email: formState.email,
password: formState.password,
});
Automatically authenticates email despite the user not clicking their confirmation email.
To Reproduce
There are 2 ways to create this bug
Method 1:

Create a DB using quickstart : react
Sign up an user using email and password
Repeated attempts at signing in the unauthenticated new account will change the 'confirmed at' column of that account from 'null' to a new timestamp.
(might take a second log in attempt to trigger)
User will be logged in despite not confirming his email

Method 2:

Create a DB using quickstart : react
Sign up an user using email and password
Confirm the user's email
Delete the account from auth.users using sql command
Recreate an account using the same email and a different password
Repeated attempts at signing in the unauthenticated new account will change the 'confirmed at' column of that account from 'null' to a new timestamp.
User will be logged in despite not confirming his email

Expected behavior
User should not be able to login with email and password if they did not confirm their email

Hi @J0 ! Apologies for the very late reply, it is working after we redeployed it :)

Not at all -- good to hear that it's working! Going to close for now but feel free to reach out again if you run into any issues :)


Update contributing section on README

Updated the contributing section by adding git bash commands and pictures of various stages of a contribution.
Based on issue #16

hi @ABHIGYAN-MOHANTA well done!
Sorry I will change a bit the scope of this task, please create a file in the root of the repository called CONTRIBUTING.md and copy/paste this content that you create on that.
Can you do it?

Sure


[fix] rename it case tot ITCase

KafkaSchemaTest is a IT Test

Titile should be [flink] rename KafkaSchemaTest case to ITCase.


Configurability

Added configuration options using the built-in argparse library. New options are:

symbol_set to set one or more symbol sets to use randomly from the archive (e.g. --symbol_set THICK THIN), defaults to ASCII
max_framerate to control the sleep time between redraws, defaults to 100
color_set to allow choosing which colors to draw with. User can use 'random' to use the old behavior
min_pipes to set the fewest starting points for the algorithm, defaults to 5
max_pipes to set the most starting points for the algorithm, defaults to 25

Also modified the exit routine so that it does not trash the terminal on exit.

perf

@KonradIT couldn't you review this?  (and watch the repo)
@kedean because nature of this issue and even over 100 lines changes, I will be willing to merge this if it gets approval votes.

@KonradIT * could

Merging this, even though I'd like to make some changes request, but because this was created nearly two years ago, so I won't be asking and will be doing the changes by myself.

@kedean @neeasade I have checked out your GitHub profiles (C, Python, Bash/Shell, and/or AUR packaging), I wonder if you will be interested in joining Pipeseroni. we are looking for new devs.
As you may already know we have several pipes in different languages, currently we are focusing on pipes.sh, pipes.c, and the AUR packaging (yet to get started).  If you have the time and are interested in, please do consider.


Python Interactive Windows errors with vscode-remote

Environment data

VS Code version: insider-version 1.34.0
Extension version (available under the Extensions sidebar): tried current and insider-version but makes no difference
OS and version: Docker with ubuntu 18.04 on macOS Mojave
Python version (& distribution if applicable, e.g. Anaconda): 3.6.7
Type of virtual environment used: N/A

Expected behaviour
Python code running in the Python Interactive Window
Actual behaviour

Steps to reproduce:

Create a project directory with a sub directory with a .devcontainer/devcontainer.json file as follows:

{
	"name": "rsm-msba",
	"image": "vnijs/rsm-msba",
	"workspaceFolder": "/",
	"extensions": [
		"ms-python.python"
	]
}

Try to run the python code below in the Python interactive window. This works in the standard terminal but I get the strange output as shown in the screenshot above in the interactive window

cc-ing @chrmarti in case he has any comments/suggestions
#-------------------------------------------------------------------------------------------------------------
# Copyright (c) Microsoft Corporation. All rights reserved.
# Licensed under the MIT License. See https://go.microsoft.com/fwlink/?linkid=2090316 for license information.
#-------------------------------------------------------------------------------------------------------------

#%%
import matplotlib
import matplotlib.pyplot as plt
import numpy as np

# Data for plotting
t = np.arange(0.0, 2.0, 0.01)
s = 1 + np.sin(2 * np.pi * t)

fig, ax = plt.subplots()
ax.plot(t, s)

ax.set(xlabel='time (s)', ylabel='voltage (mV)',
       title='About as simple as it gets, folks')
ax.grid()

fig.savefig("plot.png")
plt.show()

print('Open test-project/plot.png to see the result!')

# %%

import os
print(os.getcwd())

It tries to run the script in bash. Not sure if this is an issue with the devcontainer or the Python extension. Hoping for insights from the Python extension team.

This wouldn't be us. This is jupyter running that command. We send it code to execute and it's not using python for some reason.

Can you attach your developer console output? (Help | Toggle Developer Tools, click on console, right click, save as)

See attached with some errors:
remote-interactive-window.log
I also tried restarting the "rebuild container" option and that also produced a few errors.
remote-interactive-windows-restart.log
And one more when I (think I) just switched to the the devcontainer.json file.
remote-interactive-windows-change-file.log

Just opening hello.py I just saw the follow flash by.

Yeah that looks like the extension isn't finding the python in use. Or at least one of them. It does start jupyter though.
What happens if you open a terminal and type 'python -m jupyter notebook --version'

Actually it looks like it's using 'python3', so type 'python3 -m jupyter notebook --version'

Will check tonight. Weird that it can start jupyter without finding python though right?

I think I found the issue. This container has multiple kernels, including R and Bash. When I deactivated the Bash kernel the python code does run but I still get the error shown below.
So there seems to be an issue with the extension using the correct kernel (based on the open file type). This seems directly related to the following issue:
https://github.com/microsoft/vscode-python/issues/3763
It would be great if you could specify which kernel(s) you would want to use in devcontainer.json. Even better if the user could select from a set of available kernels (https://github.com/microsoft/vscode-python/issues/3763#issuecomment-470265627)

FYI Requested output shown below:
python3 -m jupyter notebook --version
5.7.8

(The devcontainer.json can currently not include extension specific configuration.)

What's the value of python.pytjonPath in the settings.json file?
What's displayed on the bottom left of the status bar of vscode? What version of python?
Please send a screenshot of vscode

The only thing in the settings file is:

Screenshot vscode insider shown below. The selected python is python3

I just noticed that there is a .vscode/settings.json file inside the directory I'm mounting into the container. "Run file in terminal" works fine but "Python Interactive" still seems to be trying to run the python code in the "Bash kernel"
{
    "python.pythonPath": "/usr/bin/python3"
}

We use the default kernel unless the user has ipykernel installed and we can change it. Can you check what 'python3 -m ipykernel --version' returns from the terminal window?

From a terminal inside the container I get.
python3 -m ipykernel --version 7.2.0

Hmm. I wonder why it's not writing it then.
What does this list:
python3 -m jupyter kernelspec list

We use that to enumerate the installed ones and try to match the best one if ipykernel fails.

I get the following:
python3 -m jupyter kernelspec list
Available kernels:
  bash       /usr/local/share/jupyter/kernels/bash
  ir         /usr/local/share/jupyter/kernels/ir
  python3    /usr/local/share/jupyter/kernels/python3

We must not be matching for some reason. I was thinking the python one might not show up or not say python in the name, but it does, so it should match.
In order to repro this you installed the bash and ir kernels into jupyter, right?
Or did you just start with a specific container that I might be able to also try?

I created a Docker file with Jupyter and R and Bash kernels and the image is available on Docker hub as vnijs/rsm-msba. My current devcontainer.json is shown below, although I'm not able to access the specified ports after activating the container through vscode-insider. All kernels work fine and I can connect through the (local) browser just fine when not using vscode. More information and install instructions I use with my students linked below. Let me know if you need any more information. Thx!
https://github.com/radiant-rstats/docker/tree/master/install
{
	"name": "rady-test",
	"image": "vnijs/rsm-msba",
	"workspaceFolder": "rady",
	"extensions": [
		"ms-python.python",
		"christian-kohler.path-intellisense",
		"Ikuyadeu.r",
		"REditorSupport.r-lsp",
		"VisualStudioExptTeam.vscodeintellicode"
	],
	"appPort": [
		"<IP_ADDRESS>:8080:8080",
		"<IP_ADDRESS>:8787:8787",
		"<IP_ADDRESS>:8989:8989",
		"<IP_ADDRESS>:8765:8765"
	]
}

Thanks. That helps.
I'm able to repro now.

Great. Thanks for looking into this @rchiodo. Please do let me know if I can test anything. A fix for automatic selection would be great but even better would be to have an option to set which kernel to start (or running kernel to connect to) like hydrogen has would be a fantastic feature to have (https://github.com/microsoft/vscode-python/issues/3763#issuecomment-470265627) 💯 :)

We have the selection for python on the bottom, it's supposed to use that one as your 'kernel' for the Python Interactive window.
Would you want something else (besides picking the python to use)? Other language kernels?

Ah. I see. The python selection would be fine as is ... if it works :) but I would indeed really like to use R (and perhaps some other kernels) if possible in an interactive window where plots can be shown.

Using other languages is a long term goal, but unfortunately it's going to take a while :(

Would be interested to hear what the challenges are to (1) connect to running kernels and (2) connect to non-python kernels.
Any idea if the kernel issue that started this issue is addressable?

Connecting to running kernels should be relatively easy. We just specify a different kernel when start our session.
Using said kernels is much tougher. All of our internal code we run now is python based. We send a bunch of stuff to get plots and directories correct. Our code for requesting variables is all python based. So we'd have to rework these per language or figure out a language agnostic way to do the same thing. That's the part that's going to take awhile.

This bug should be addressable though. It's likely just picking the first kernel instead of the python one for some reason.

I see. Thanks for the clarification @rchiodo. It seems that the approach in hydrogen is all JS  based (https://github.com/nteract/hydrogen/tree/master/lib) so presumably easier to extend to other kernels.  Any chance you could point me to where the relevant python code is in the repo? For some reason searches like "jupyter" don't give any code results. Just curious.

The variable explorer bits are probably the biggest amount of work. Not even sure they're possible in other languages. Other stuff is just doing things like setting the current directory and modifying matplotlib (not sure if there's an equivalent in say R)
We use the python files here to query variables from jupyter:
https://github.com/microsoft/vscode-python/tree/master/pythonFiles/datascience

Thanks @rchiodo! A variable inspector for R (and other languages) would be nice but not critical. Just running (and plotting) would already be a huge plus.

FYI I upgraded jupyter and other packages in the vnijs/rsm-msba container (see link below) and am now getting the following error when trying to run a cell in vscode 1.34.20-insider

jupyter-1.0.0.log
https://github.com/radiant-rstats/docker/blob/master/rsm-msba/requirements.txt
Everything works fine when I launch without vscode and in vscode with the mounted container I see the following information so jupyter is definitely available and installed in the container. This happens with both the release and insider version of the ms-python extension.
python3 -m ipykernel --version
7.5.0

Python 3.6.7 (default, Oct 22 2018, 11:32:17)
[GCC 8.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import jupyter_core
>>> import notebook
>>> jupyter_core.version_info
(4, 4, 0)
>>> notebook.version_info
(5, 7, 8)

Starting jupyter is giving this:
workbench.main.js:238 [Extension Host] Python Extension: Error: /usr/local/lib/python3.6/dist-packages/requests/init.py:91: RequestsDependencyWarning: urllib3 (1.25.2) or chardet (3.0.4) doesn't match a supported version!
RequestsDependencyWarning)
Sounds like urllib3 or chardet are out of date.

Note to self (or whoever gets this bug), here's the directions on debugging a remote extension:
https://code.visualstudio.com/api/advanced-topics/remote-extensions

It seems this latest issue update I posted was a urllib3 version issue. I downgraded to urllib3==1.24 in the container (vnijs/rsm-msba:1.5.1) and now we are back to the "bash kernel being used instead of python3 kernel" issue we started with. Sorry for the confusion. I wrongly assumed this updated issue was related to changes in vscode-python.

@vnijs, Good news I just submitted a fix for this. It was a bug in our kernel spec matching algorithm. It was picking the bash kernel because it was using the same version of python under the covers.
Our next insider's build will have this fix in it.

It is working @rchiodo. Thanks!


Create a new flow for bulk creation of provider users

Context
As part of the Transition team's provider onboarding process, they need to be able to add users in bulk. The process is currently managed via a spreadsheet and the team inputs user details one-by-one. This is inefficient.
Changes proposed in this pull request

Add new flow for bulk adding of ProviderUsers

Guidance to review

In the support interface, click on the 'users' tab for any provider (/support/providers/PROVIDER_ID/users), then click 'add multiple users'
Enter user details in CVS format in the text field and make you way through the form
The flow makes use of a Redis store in order to temporarily store users whilst permissions are added to each ProviderUser. Once all permissions are added, the ProviderUser details (and their permissions are  saved to the database)
NOTE - this PR  allows ProviderUsers to be added (in bulk) only,  not edited. Another PR will follow with editing functionality

Link to Trello card
https://trello.com/c/LYQBq8At/3400-⛵-epic-bulk-add-provider-users
Things to check

[x] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

@simonwhatley - if a user uploads a CSV of provider users but once they come to that user's permissions form they decide they do not want to add a particular user, what is supposed to happen?
Same question re: duplicate users? Form validation will kick in to say that the duplicate user (with same email address) can not be added, but then how does the user remove that ProviderUser from consideration should they wish?

@TobyRet here are some thoughts from my review of the PR

Add multiple users button needs to use the govuk-button--secondary style
After copy-pasting content into the users' details field, submitting the form, then going back, the textarea should contain the data
Copy-pasted data can either be comma-separated or tab-separated. Currently, it only supports the former, which means copying data from a spreadsheet (a common need) will not work
On the individual user page (i.e. 'Add user (1 of 3)') the submit button should read 'Continue' not 'Add user'
The summary list should display the users in the order in which they were entered. It currently shows them in reverse order
The summary list items should include change links. The change link will take the DfE user to the individual user page, where they can change the content. When submitted, the DfE user is returned to the summary list with the content changed. (Hitting '< Back' on the individual user page will also return the DfE user to the summary list with the content unchanged)
Hitting '< Back' from the summary list should take you back through the page flow. For example, if there are three users, it will go back to the details of user 3, then 2, then 1 then the bulk upload form

The summary list items should include change links

@TobyRet here are some thoughts from my review of the PR

Add multiple users button needs to use the govuk-button--secondary style
After copy-pasting content into the users' details field, submitting the form, then going back, the textarea should contain the data
Copy-pasted data can either be comma-separated or tab-separated. Currently, it only supports the former, which means copying data from a spreadsheet (a common need) will not work
On the individual user page (i.e. 'Add user (1 of 3)') the submit button should read 'Continue' not 'Add user'
The summary list should display the users in the order in which they were entered. It currently shows them in reverse order
The summary list items should include change links. The change link will take the DfE user to the individual user page, where they can change the content. When submitted, the DfE user is returned to the summary list with the content changed. (Hitting '< Back' on the individual user page will also return the DfE user to the summary list with the content unchanged)
Hitting '< Back' from the summary list should take you back through the page flow. For example, if there are three users, it will go back to the details of user 3, then 2, then 1 then the bulk upload form

@simonwhatley I've pushed the changes apart from 'The summary list items should include change links' - this will be done in a followup PR (CC: @cwrw)

@TobyRet here are some thoughts from my review of the PR

Add multiple users button needs to use the govuk-button--secondary style
After copy-pasting content into the users' details field, submitting the form, then going back, the textarea should contain the data
Copy-pasted data can either be comma-separated or tab-separated. Currently, it only supports the former, which means copying data from a spreadsheet (a common need) will not work
On the individual user page (i.e. 'Add user (1 of 3)') the submit button should read 'Continue' not 'Add user'
The summary list should display the users in the order in which they were entered. It currently shows them in reverse order
The summary list items should include change links. The change link will take the DfE user to the individual user page, where they can change the content. When submitted, the DfE user is returned to the summary list with the content changed. (Hitting '< Back' on the individual user page will also return the DfE user to the summary list with the content unchanged)
Hitting '< Back' from the summary list should take you back through the page flow. For example, if there are three users, it will go back to the details of user 3, then 2, then 1 then the bulk upload form

@simonwhatley - I have now made the changes above
@cwrw would be grateful for a code review when you get some time 😄

Discussed in person but for posterity:

Getting error when entering less than three words in CSV textbox
Should only support tab delimited fields and not space delimited
Should clear state after exiting the wizard
A subtle bug with the position indexing and validation
An error when attempting to render user message but we think it was because of corrupt state
Attempting to create two controllers one for users and one for the whole flow to help with RESTful methods/paths
Renaming Form to wizard

@cwrw 👋
I've made the changes above.
A couple of points on the back of this ...

Redis state is cleared after users are successfully saved to the db.
I have spit up the controllers and mapped them against the 'steps' in the flow and tried (😄 ) to use more 'RESTFUL' routes
all the bugs should be addressed now

Let me know what you think.

Will squash commits once approved @cwrw


Roll Flutter Engine from 5bc3ab514546 to ef1743467be2 (1 revision)

https://github.com/flutter/engine/compare/5bc3ab514546...ef1743467be2
2023-10-03<EMAIL_ADDRESS>Restrict ImageReader backed Platform Views to Android 33+ (flutter/engine#46478)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


document the default for /servers

https://spec.openapis.org/oas/v3.1.0#fixed-fields

Hey Karen, thanks for the PR.  Can you help us understand what problem this addresses?

default is mostly for documentation purposes but some implementations may use it to fill in a value if not specified.
We use the default keyword in many other places in the schema where the specification indicates there is a default value to be assumed.


renamed all occurences of ru-ru to ru-RU

fixing build warning as mentioned in #177  by renaming "ru-ru" to the expected ISO language code "ru-RU"

fixed it


This code block is iterating the webFontStyles, so if for a specific …

…style there's an error while looking for its corresponding font in git, then it means it it "available in API but not in git".

Good catch


How to set X axis  text?

By default， x axis's  value is  0--8 ，but now i want re set X axis‘s value  like  'John' ，'Jenny','Tony'......but  i dont  now  how to do that ?  (my english is  poor) only one can help? think you

create list with x axis values you want and assign it to the axis
 List<AxisValue> axisValues = new ArrayList<AxisValue>();
 axisValues.add(new AxisValue(i, "some textt".toCharArray()));
 Axis axisX = new Axis(axisValues);

@yon86   Thank you I have solved

@yon86 how to set X axis text vertical？

@scdqs ame thing u just have pass the values to the y axis instead of the xaxis


NoRowsDeletedError not thrown on model with deletion field already set

If a model has already been previously soft-deleted, and you try to delete the model again, a NoRowsDeletedError is not thrown. It is instead registered as a successful operation and updates the deleted field accordingly.
There needs to be a check on update to make sure that the record has not been previously deleted. I'm working on a PR to demonstrate/fix the error.

@jadengore We'll need more info on how to reproduce this error. I added a test and it successfully thrown the correct error https://github.com/estate/bookshelf-paranoia/pull/25

@alanhoff It may be because I'm using PostgreSQL and they have different behavior at the knex.update spec. I'll do some more research and get back to you.


Add information to publish run page about licensing BioSimulations to distribute run

[x] Add publish run page
[x] Privacy policy/terms of service

:tada: This issue has been resolved in version 4.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


PiPy README not displaying correctly

https://packaging.python.org/tutorials/distributing-packages/#pure-python-wheels

Definitely requires .rst encoding.
Do I make two versions of the README and maintain both independently or can GitHub support .rst files?

Maintaining two versions is necessary. Use an online rst editor because the sublime support was AWFUL.


Fix ProcessDialog errors appeared after rebuilding .NET project

Fixes #96550
Add a optional paramater to EditorProgress to force use background progress when MessageQueue is flushing.

Thanks!


264 add time duration for migration task to migration reports

@bltravis @branchedelac  Anyone want to check? There are three fixed issues here, mentioned in the commit messages. Please assign yourself if you take it on.

There was a small issue with Sonarcload that has been fixed.

The Sonarcloud thing is done, but it does not get updated.


ListView passing touch events to widget under it[BUG]

I have a sidebar which is a ListView. This ListView contains multiple ExpansionTile with ListTile. In the image below, in the red portion of the side menu, all taps are being transferred to the YoutubePlayerIFrame under it. The library I used for the youtube video player is: youtube_player_iframe. The red area, I cannot expand the ExpansionTile. The green portion has no problems.

Hello,
I have the same issue with the Iframe, i used an IgnorePointer to  wrap the Iframe.The player still clickable on IOS but not on Android.
My code :
IgnorePointer(
  ignoring: true,
  child: YoutubePlayerIFrame(
    controller: _youtubeController,
    aspectRatio: 16 / 9,
  ),
),


g910 orion spectrum "|" and "" keys

What are you experiencing an issue with?

[ ] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[ ] Aurora Profile
[x] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[ ] Performance problems
[ ] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[x] Device issue
[ ] Other, (Fill in another type of issue here)

Expected behavior:
Explain here what should happen.
The 2 "" keys (also "|" if you hold shift) should work with all profiles
Actual behavior:
Explain here what happened instead.
The 2 "" keys (also "|" if you hold shift) seem completely separated from
aurora, you can't see them in profiles or in the keyboard previews;
I believe the problem is that aurora isn't optimized for the new g910.
How often does this problem occur?

This problem occurs... (every time)
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

use the newer g910 orion spectrum keyboard from logitech
set any color (even no color) for the keyboard in logitech gaming software
use any profile on aurora

What is the version of Aurora you are experiencing this issue with?

My version is... (beta v0.6.0b (but it never worked on any other version))
Does the previous version of Aurora have this issue?
The previous version, (not truly sure, I had it then switched to the new one), did have this issue.
Please include any logs, screenshots, or files that are related to this issue:

For logs, you can use: http://pastebin.com/
For screenshots, you can use: http://imgur.com/
For files, you can use: http://www.tinyupload.com/ , https://www.dropbox.com/ , or attach them to the issue.

sorry I didn't leave any logs or images, I though you should have a pretty good understanding of what the issue was

try updating to dev-3 since it works for me

uninstalled aurora, downloaded dev-3, still doesn't work. thanks for replying quickly though!

What layout do you have for your keyboard and what one do you have selected in Aurora?

If you by layout you mean "QWERTY", "QWERTZ" etc. I have a querty keyboard and i have selected United Sates in aurora (though my problem occurs with "automatic" aswell.

Sorry, I meant I had a "QWERTY" keyboard, my mistake.

Agreed, this key is misrepresented in your keyboard layout inside Aurora as well. It should be above the "Enter" key, not next to it. This "\ - |" key is not being controlled by Aurora at all. It's not affected by any effect in the app.

Well thanks for replying, I now just use the color cycle

@Draftex Could you find an image of the exact layout your keyboard has? I'm confused as to why you aren't able to find a layout in Aurora that works.


CFLastsubmissions

I'm getting the following error on using :CFLastSubmissions<CR> command:-
Error detected while processing function cfparser#CFLastSubmissions:                                            line    8: E706: Variable type mismatch for: cf_response line   10: E121: Undefined variable: status E116: Invalid arguments for function matchstr(cf_response.status, "OK")) E116: Invalid arguments for function empty(matchstr(cf_response.status, "OK")) E15: Invalid expression: empty(matchstr(cf_response.status, "OK"))

Just tested and it worked fine for me, are you logged in?
Sent from my Nextbit Robin using FastHub

Actually, looking at it more carefully, seems the codeforces API response was invalid. Maybe codeforces API was offline at the time or there was some failure in the network. Could you check if it is working now?
Sent from my Nextbit Robin using FastHub

Yes,I'm logged in.
No,it's still not working.All the other commands are working fine except :CFLastSubmissions<CR> and CFRun<CR>

What is the output with CFRun? Are you on linux or windows? Do you have curlinstalled?

I'm on linux. Yes, Curl is installed.
CFRun shows an error.
On Jul 19, 2017 8:15 PM, "Gabriel Silva Simões"<EMAIL_ADDRESS>wrote:

What is the output with CFRun? Are you on linux or windows? Do you have
curlinstalled?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gabrielsimoes/cfparser.vim/issues/3#issuecomment-316410517,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYU0mGAeNIBZTABONCON99oSLMcRAEhSks5sPhZ-gaJpZM4Ock5i
.

Which error does CFRun show? Also, whats your VIM version?

CFRun shows the following error:
"Error detected while processing function cfparser#CFRun:
line    2:
E492: Not an editor command:     RunInInteractiveShell /tmp/cfparser_exec
Press ENTER or type command to continue
"
vim version is 7.4.52
On Wed, Jul 19, 2017 at 8:55 PM, Gabriel Silva Simões <
<EMAIL_ADDRESS>wrote:

Which error does CFRun show? Also, whats your VIM version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gabrielsimoes/cfparser.vim/issues/3#issuecomment-316423024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYU0mNwcboBaQMCaIKYRJO6rdJ1X3tYYks5sPh_qgaJpZM4Ock5i
.

Just noticed CFRun depends on another VIM plugin I use. Now it should work fine. However, I have no clue what my be causing the error on CFLastSubmissions. Are you behind a proxy or something?

CFRun is working fine now.
I'm not behind any proxy.
On Wed, Jul 19, 2017 at 9:26 PM, Gabriel Silva Simões <
<EMAIL_ADDRESS>wrote:

Just noticed CFRun depends on another VIM plugin I use. Now it should
work fine. However, I have no clue what my be causing the error on
CFLastSubmissions. Are you behind a proxy or something?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gabrielsimoes/cfparser.vim/issues/3#issuecomment-316432775,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYU0mKSQ6gaLnMA6ZKJtPnELQSVTNKQTks5sPicogaJpZM4Ock5i
.

Could you run this and post the output here: curl --location --silent 'http://codeforces.com/api/user.status?handle=aspera_&from=1&count=5'
Also run this on vim :echo cfparser#CFLoggedInAs() and post the output here.

I have made some changes, could you check if it is working now?

If it still errors, please post the new output.

It's working fine.Thanks for all the help :).
On Wed, Jul 19, 2017 at 10:41 PM, Gabriel Silva Simões <
<EMAIL_ADDRESS>wrote:

If it still errors, please post the new output.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gabrielsimoes/cfparser.vim/issues/3#issuecomment-316454333,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYU0mGkDlhxCIUyHadIUFe7oQwtYdDWGks5sPji1gaJpZM4Ock5i
.


How do I run the test file?

It's not at all obvious to me how I should run the bob_test.spec.coffee file.
I have only used CoffeeScript in the browser. I assume I have to run this with node.js somehow?
Also, the file looks like:
Bob = require "./bob"
describe "Bob", ->
bob = new Bob()
it "stating something", ->
	result = bob.hey "Tom-ay-to, tom-aaaah-to."
	expect(result).toEqual "Whatever."

So I think I need to install some sort package to run the tests? AFAIK CoffeeScript doesn't include those by default?
Thanks ;-)

It's documented in the help site:
http://help.exercism.io/getting-started-with-coffeescript.html
We should probably add this to the READMEs.
If you have a moment, do you want to add a SETUP.md at the root of the https://github.com/exercism/xcoffeescript repo that explains how to run the tests?
The api checks if there is a SETUP.md file, and if it is, it sticks the contents into the README for each of the problems that get delivered via the exercism fetch command.


Integrate with new gossipsub version

Goal

Close #643
Replace #727

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tuyen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Signed CLA ^^


DEV: 'You'll lose any offline changes if you sign-out.' message appears even if you are online

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

go to Settings
click on Sign out button

Expected Result:
user should be Sign out
Actual Result:
shows 'You'll lose any offline changes if you sign-out.' message even if you are online
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: Dev
Reproducible in staging?: dev
Reproducible in production?: dev
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/cfd52b31-a124-408d-a1d8-d1f95837c19d
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690436737183979
View all open jobs on GitHub

I can't reproduce it.

This is regression from https://github.com/Expensify/App/issues/16281#issuecomment-1652952760

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@mountiny @slafortune  this is a regression from https://github.com/Expensify/App/issues/16281. Awaiting response on this.

@mananjadhav given its deploy blocker and the contributor hasnt messaged I have created the PR

@mountiny I’ll be able go check this in an hour, afk

No problem merged going to CP

@slafortune Can we pay @gadhiyamanan $250 for reporting this bug?

@gadhiyamanan offer sent here - https://www.upwork.com/nx/wm/offer/25815360

@slafortune offer accepted, thanks!

Thanks - Paid!


LFS-172: As an admin, I can import, visualize and design questionnaires

In this PR:

Create and delete questionnaires
Visualize, create, edit, and delete questions and sections
Add new field types using JSON files

TODO:

Allow conditional statements to be shown and edited
Allow drag & drop reordering of questions and sections
Show a column in questionnaires with the number of forms which links to a page displaying the forms

deleting of a question works well, however it gets deleted once you refresh
when adding a new question letters "s" and "q" do not work if used as a first character
hints do not work
min/max value does not seem to work

The following issues existed previously so I did not address them:
deleting of a question works well, however it gets deleted once you refresh
hints do not work
min/max value does not seem to work

When creating a new section, name and title get concatenated. Step to reproduce:

create new section

save>refresh>observe

go to edit mode and notice that in field name is no longer available (name and title are stored in the title field comma separated)

I believe there is a regression that was recently introduced. On my older instance of LFS-172 build I get dropdown filed for data type

On my new instance that I just built I get this:

I created a new question for the Chemotherapy Data:

Then I wanted to add a new section to the same questionnaire. The value of the question title previously entered got copied into the section name ( and vice versa - section name would get copied into the question title if I reversed the oreder of creation).

I was able to design a new questionnaire, however I was not able to get the constraints work ( e.g. with min answers is 1& max 2: I can add as many as I want) . Also, not sure how to create a radio bttn or design a field such as Tumor's malignancy filed.

This definition renders field as read only

This setting allows only 2 entries ( should be 3)

The Action column should be pinned to the right.

Two outstanding problems are reported in: LFS-540 ; LFS-542

The Action column should be pinned to the right.

@sashaandjic This should now be fixed. The other bug will be fixed in #271

@OrdiNeu did you include changes from #271 in this PR. I do not see them. I only see this one being fixed

Two more problems found and reported in separate tasks :

LFS-544 - unable to delete questions placed withing section
LFS-545 - unable to delete section

Max Answers and the actions on the right side have now been fixed. Ready for re-test.

Max Answers and the actions on the right side are fixed,
I am unable to remove Answer Options from questions once they are defined.

I deleted all of them:

Clicked on SAVE bttn, however they are all still there.

I would also like to be able to move questions within questionnaire ( probably out of scope).

How can I create a conditional section?

Min and Max value are not enforced. Note that the last character is cut of the Description field once you go to edit mode.

I am still unable to delete questions and sections.
When I specify that Min Answers is > 0, I would expect the field to be required, however I can leave such field blank.

Section deletion now works. I can delete sections from questionnaires with active forms and with data entered in them.
An error msg warsn me that I cannot delete questionnaire as I have an active form of that type. There is also message at the bottom of the screen.

Once I close the dialog, the message in red fonts at the bottom of the screen is still there.

This is good, but it makes the "warning" displayed to the user inaccurate. Do we care?
I've made the warning more generic in cf29fe :heavy_check_mark: .
Should we make this more generic and let it handle deletes for any kind of resource?
I've switched this to allow it to handle deletion of any of our resources (lfs/Resource). :heavy_check_mark:
Not sure if we want this functionality across any nt: or sling: nodes yet.

Problems found so far:

Predifined questionsrries: cannot delete sections (with or without data), cannot delete questionarries

when creating a new question with Answer Options, the Answer Options are not saved

I cliked on SAVE and opened the question in edit mode


Hugo migration - Home button wrong on top nav

This is a bug to track the "Home" navigation button displaying as "Documentation".

[ ] Feature Request
[X] Bug Report

Problem: This is a bug to track the "Home" navigation button displaying as "Documentation".
Proposed Solution: Change "Documentation" to "Home".
Page to Update:
https://k8sio-hugo-migration.netlify.com/docs/home/?path=users&persona=app-developer&level=foundational

I have written about this in another issue, but I will take it here as well. Menus, table of content etc. are now built from the content section tree. Titles and link titles are taken from the page content front matter.
This avoids hardcoding/double maintenance etc. and it enables translations ("home" isn't "home" in Norwegian).
Hugo has 2 standard titles:

title: What most themes show as the content page title/header
linkTitle: What most themes show in listings/menus etc.

You can, of course, add custom titles as params, I would think a little before doing it. Because you kind of need to make it a standard.
So: "Documentation" is the linkTitle for the documentation home page. It matches mostly with production, but not in the documentation menu.

Moving to P3. Other issues, like getting each of the left navs correct, are more important.


[Question] find all method calls of specified type

Hello everyone!
I was trying to findout is it possible to find all method calls for specified type, but I met a problem.
I have code like this:
public class Test {
`   public void run(MyClass myclass) {
        myclass.doSomething();
    }
}

And what I want is to get all calls of doSomething method of MyClass and change this calls to myclass.firstdothis().doSomething();
I am init type solver like this
 typeSolver = new CombinedTypeSolver(new ReflectionTypeSolver(), JarTypeSolver.getJarTypeSolver(pathToMyClassJarFile));
and then I trying to solve it with example that I got from book
  List<MethodCallExpr> methodCalls= Navigator.findAllNodesOfGivenClass(compilationUnit, MethodCallExpr.class);

        for (MethodCallExpr methodDeclaration:fieldDeclaration) {
        System.out.println(JavaParserFacade.get(typeSolver).solve(methodDeclaration).getCorrespondingDeclaration().getQualifiedSignature())
        }

But I`ve got such exception:
java.lang.RuntimeException: javassist.NotFoundException: org.eclipse.jetty.util.component.AbstractLifeCycle
	at com.github.javaparser.symbolsolver.javassistmodel.JavassistClassDeclaration.solveMethod(JavassistClassDeclaration.java:193)
	at com.github.javaparser.symbolsolver.resolution.MethodResolutionLogic.solveMethodInType(MethodResolutionLogic.java:614)
	at com.github.javaparser.symbolsolver.javaparsermodel.contexts.MethodCallExprContext.solveMethod(MethodCallExprContext.java:196)
	at com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade.solve(JavaParserFacade.java:258)
	at com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade.solve(JavaParserFacade.java:142)`

It would be great if somebody could help me to find out what  am I missing, thanks.

The problem seems that your library is using a dependency (jetty) and it is not listed into the TypeSolver initialization, so JSS does not know where to find it

No feedback, closing. Please reopen if you have still the issue


mark poisson_high() function for removal

What should be changed
cogent3.maths.stats.poisson_high()
Why
Not used in cogent3, available in scipy.stats?
How much notice
6 months notice. This should be represented in the user warning as a specific version. e.g. .<current month + x> where x is from the PR date.
Guidelines for deprecation
As this is a function that will be removed, see dev guidelines for deprecation of code. Indicate the limiting version (for when the old function will be removed) as the number above from when you make the change.

I would like to do this issue

I attempted to solve this issue and have submitted a PR. Please have a look, thank you!


[release-1.31] Fix crictl info for containerd

What type of PR is this?
/kind bug
What this PR does / why we need it:
We cannot just assume JSON objects because containerd will also return something like:
"golang": "go1.22.5",
"lastCNILoadStatus": "OK",
"lastCNILoadStatus.default": "OK",

For those values we just assume strings and prevent printing multiple double quotes to restore the v1.30.0 behavior of crictl.
Cherry picked from commit 793c3e08c303c5bc6ca4f9254c3e1f1ae8b25d7e
Which issue(s) this PR fixes:

Fixes https://github.com/kubernetes-sigs/cri-tools/issues/1566
Forward port from https://github.com/kubernetes-sigs/cri-tools/pull/1568
Special notes for your reviewer:
This seems to be a regression and we may have to follow-up with v1.31.1
Does this PR introduce a user-facing change?

Fixed `crictl info` output for containerd.

/approve
/lgtm


Qlik Sense certificated based authentication missing

Hi,
I would like to use this package to connect from a NodeJS application as described here:
https://github.com/qlik-oss/enigma.js/blob/master/examples/authentication/sense-using-certificates/certificates.js
Sadly, I can't find any possibility to use this package with certificated based authentication.
Am I missing something?
Best regards,
hans0

Thank you for filing this issue.
I'm anticipating that this request will be quite common, so I will look into having some way of including certificates into the qix module's websocket connection logic.


Lag in updating the problems windows in vscode

Using dotnet 2.1.500 and c# extention 1.17.1.
There is a substantial delay between the time I type code until problems detected are shown in the problems windows and same delay between the time I fix code problem and the problem being removed from problems window. Any way to fix this to make the problems window linting more real-time?

See also #4867

A lot has changed with code analysis since this issue as opened. If you are still running into problems, please open a new issue.


Specify packages available to task

I'd like to be able to specify a list of packages available to a task.
So instead of:
tasks = {
  "check:taplo-fmt" = {
    exec = "${pkgs.fd}/bin/fd --hidden --extension=toml --exec-batch ${pkgs.taplo}/bin/taplo format --check";
  };
};

You could write something like this:
tasks = {
  "check:taplo-fmt" = {
    packages = [ pkgs.fd pkgs.taplo ];
    exec = "fd --hidden --extension=toml --exec-batch taplo format --check";
  };
};

The packages option might be confusing together with the existing option tasks.<name>.package which could imo be renamed to something like tasks.<name>.entrypoint or tasks.<name>.interpreter.

@szlend would you expect those packages to be an addition to top-level packages option?

@szlend would you expect those packages to be an addition to top-level packages option?

I would expect them to be independent to minimize the closure size in CI where tasks would execute. That's how it currently works, right? Or at least make it configurable to disable inheriting from global packages.


Update serviceWorker.js

There is a small typo in !!(opts.cachePathPattern.exec(url.pathname), (a missing bracket). You could also write this using RegExp.prototype.test instead of RegExp.prototype.exec (saving the type conversion).

Thanks!