Question: Error when import getAvailableIcons from @grafana/ui in plugin-e2e

Faced a problem when using @grafana/plugin-e2e and iport in another file import { getAvailableIcons } from '@grafana/ui';
Error: require() of ES Module /home/vitali/work/volkovlabs/business-forms/node_modules/d3-interpolate/src/index.js from<EMAIL_ADDRESS>not supported.
Instead change the require of /home/vitali/work/volkovlabs/business-forms/node_modules/d3-interpolate/src/index.js in<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.

folders structure for example
├── src
│   ├── constants
│   │   ├── form-elements.ts
├── test
│   ├── panel.spec.ts

I have file form-elements.ts
Here is an example code
import { SelectableValue } from '@grafana/data';
import { getAvailableIcons } from '@grafana/ui';

export const enum FormElementType {
....
}

/**
 * Icon Options
 */
export const ICON_OPTIONS = getAvailableIcons().map((name) => ({
......
}));

When I import FormElementType into my panel.spec.ts and use it in the test, after that when I run npx playwright test I get an error:
I'm faced with a problem with imports.
If I remove the import import { getAvailableIcons } from '@grafana/ui'; from   form-elements.ts file, the test works properly.
Error: require() of ES Module /home/vitali/work/volkovlabs/business-forms/node_modules/d3-interpolate/src/index.js from<EMAIL_ADDRESS>not supported.
Instead change the require of /home/vitali/work/volkovlabs/business-forms/node_modules/d3-interpolate/src/index.js in<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.

import { test, expect } from '@grafana/plugin-e2e';
import { ModalHelper, PanelHelper } from './utils';
import { FormElementType } from '../src/constants/form-element';

 test.describe('Base', () => {
    test('DO smth', async ({ gotoDashboardPage, readProvisionedDashboard }) => {
    .... some code
/**
       * Example usage
       */
await elements.checkElementPresence('dateTime', FormElementType.DATETIME);

}
})

At this point,  I can split files into separate files that contain types and constants. And avoid using files with import import { getAvailableIcons } from '@grafana/ui in e2e tests.
I am not sure if the problem at the moment is not related to the use of imports in '@grafana/plugin-e2e'
I would be grateful if you could check this issue and confirm the problem or recommend a possible solution other than splitting the functionality into different files

Thanks for reporting this issue @vitPinchuk!
This appears to be a known issue with Playwright. You may want to try any of the workarounds suggested in the linked issue. Not tested myself, but it may be worth exploring if Node 22's improved support  for esm provides a solution to this problem.
Since this is not a bug in plugin-e2e but rather a problem with Playwright, I'll close this issue. Feel free to re-open if you feel the need to.

@sunker Thank you for your help.
I`ll check one of the workarounds


Send RST_STREAM when closing from the server without getting a client…

… half-close first
Fixes #1640

@murgatroid99 - I had to tweak Node's end-to-end test to make this all work. PTAL and let me know of concerns.

Thanks for the quick fix. I will look at it tomorrow morning.
On Tue, May 19, 2015 at 10:16 PM, Craig Tiller<EMAIL_ADDRESS>wrote:

@murgatroid99 https://github.com/murgatroid99 - I had to tweak Node's
end-to-end test to make this all work. PTAL and let me know of concerns.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/1661#issuecomment-103762135.

It was starting to pop up all over the place as a concern.

The changes to my test look fine, but I don't understand why the original one wouldn't work anymore.

So, doing a recv message in the same batch as send status is the problem.
Previously this waited for the recv. Now sending status forces the stream
closed, meaning that if all incoming bits have not been read, they will not
be ever (and instead they'll be treated as junk). The recv message will
complete with a null message, since that's legal.
On Wed, May 20, 2015, 8:49 AM Michael Lumish<EMAIL_ADDRESS>wrote:

The changes to my test look fine, but I don't understand why the original
one wouldn't work anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/1661#issuecomment-103934433.


[Testcase - Pools page] - My Pools show pools without liquidity

The following test case is not failing. However, My pools should show only pools that users have liquidity in
https://github.com/osmosis-labs/test-management/issues/51
###Steps

Visit app.osmosis.zone
Open the Pools page

As seen in the image below, the My pools section shows pools without liquidity (desktop and mobile)

Have you checked if you've added liquidity but not bonded to those pools?

Yes, I have def not added liquidity to these pools. A couple of people reported these issues on the bug report form we created. One person points out that this seems like dust liquidity that has been added to their accounts. It's a way of showing their pools on top of the list.
So for example this pool:

has a liquidity of  LP token balance: 0.000000000000000254 GAMM-577

I guess people are dropping GAMM tokens to other accounts which becomes dust and makes it visible on "My pools"

Yeah this is a known issue. It's not a bug, but someone spammed the chain with micro amounts of shares causing people's "My Pools" views to become cluttered.
We are looking into adding a toggle to "filter dust assets and pools" once we add the new nav bar and profile page.


disabling snippetSupport from client side

Is there any way to adjust the snippetSupport client capability?
The Go language server supports snippets but some users don't want them
because they don't like () included in the completion items.
I was told other editor plugins allow to control by adjusting the snippetSupport
completion feature. Currently this is hard-coded here.
https://github.com/microsoft/vscode-languageserver-node/blob/296be84f9afea51e0878fed65ca2e4847ce0ce39/client/src/common/client.ts#L1521-L1533
Is there a way to change this configuration?

No. This is a capability and not a setting. If that should be user controlled then the editor needs to provide a setting which the server can read using the getConfiguration request. Or the client provides a generic setting to filter them out. VS Code for example has settings to make snippets less prominent and sort them to the end of a completion list.
Does that make sense?

No. This is a capability and not a setting. If that should be user controlled then the editor needs to provide a setting which the server can read using the getConfiguration request. Or the client provides a generic setting to filter them out. VS Code for example has settings to make snippets less prominent and sort them to the end of a completion list.
Does that make sense?

Thanks @dbaeumer. That totally makes sense.
The question arose in the context of https://github.com/golang/go/issues/43903, where some users don't like the language server includes () or place holders in its completion items. But this is a capability, so I don't think this is the right knob to control that. Moreover, I am not sure if whether to include brackets should be controlled by snippets related settings.
I am curious how other language feature providers handle this problem (i.e. whether to include () in the function completion item). Is there a general vscode setting that the client should watch, or each language extension defines their own settings?
Thanks for the response, and I am fine with closing this issue.

Thanks @dbaeumer. That totally makes sense.
The question arose in the context of https://github.com/golang/go/issues/43903, where some users don't like the language server includes () or place holders in its completion items. But this is a capability, so I don't think this is the right knob to control that. Moreover, I am not sure if whether to include brackets should be controlled by snippets related settings.
I am curious how other language feature providers handle this problem (i.e. whether to include () in the function completion item). Is there a general vscode setting that the client should watch, or each language extension defines their own settings?
Thanks for the response, and I am fine with closing this issue.

In VS Code every language defines its own setting. TS has one which to my knowledge fills in the whole signature.

The setting for TypeScript, for reference is typescript.suggest.completeFunctionCalls.  It behaves like gopls with ui.completion.usePlaceholders turned on.  I don't think there's a typescript option to add ()s and not the full signature.

@jwalton that is correct and is what I meant with

TS has one which to my knowledge fills in the whole signature

Thank you so much @jwalton and @dbaeumer!
I wish there was a common flag used across all language extensions to control the level of info suggestions fill in. But that's beyond the scope of LSP client libraries.
I am closing this because all my questions were answered.


Highlighting <h*> elements turns them white

(This is a minor issue)
When highlighting h1, h2, etc... objects, whole words change to white as you cross over them. Then, some of the letters stay white when you un-highlight the word. (Not sure if that's clear...)
Here's what I did:

load a page with h1 / h2 text
click outside the text
drag across the text
drag back off the text
release the click
some letters are now black and others white

I'm running this in Chrome on Ubuntu 11. Not sure if other combinations have this problem.
Loren

Never seen this on OS X—is there anything else you can test on? I'm thinking this might be a browser rendering bug more than anything else.


Use https in links in speaker detail cards

Why?

because raw http is bad and vulnerable to MITM attacks

Oh for sure! Thanks for spotting that out.
I read just the PR name (which said use http…).

yeah... it's funny to have a typo in the description of fixing that same typo :laughing:

I also checked all other links hardcoded in templates and changed them to https where possible
https://github.com/pyvec/cz.pycon.org-2019/commit/3eeea2b603c6a8b91d4b86d0e5984c1b59112bf7
Is deployed now.


Examples do not work on Safari

When I attempt to drag in any of the examples in Safari, no dragging happens. In the console, I can see that the "drag start" and "drag end" messages are fired one right after the other, despite not releasing the mouse button.
I'm using Safari 15.1.

ok, give me some time to investigate - thanks for the notice!

Yes, this is Safari on macOS 12.0.1

I won't find the time to investigate today, but will do so tomorrow morning...

Ok, I can confirm that the examples do not work on Safari 15.0 under macOS 11.6 (that's my configuration) - although the problem seems to be in the REPL itself. I'll see what I can do to get everything running again

as far as I can tell right now, the problem occurs in ANY example and is NOT related to svelte-drag-and-drop-actions!

That's interesting. I assume drag-drop works okay outside of the the REPL?

Seemed to be having troubles in Firefox too.  Is that just in the REPL as well?

Hello! Firefox has the same problem, sadly...
Tried ex from below and bezier and some others (with a free movement). Dragging element just stacks to (0,0) on mouse down.
Like here
FF version 118.0.2 (64-bit)
Also tried Svelte version=3

I just tested the examples on Safari Version 16.6 (186<IP_ADDRESS>.2) - and they seem to work.
The Firefox problem still exists due to a bug in the browser itself (see issue #5) - I'm looking for a workaround, but right now it seems as if it the best solution would be to choose another browser...

the Safari problem seems to be solved by an update of the browser itself


starting front- and back-end inside a script

Hi @6br ,
I am hoping you can help with this. I am trying to automate running front-end and back-end server inside a script so I can run everything by running a single script every time.
 Problem is I got this error every time it tries to run the front-end server:
yarn run v1.22.10
$ react-app-rewired start --scripts-version react-scripts-ts
error Could not write file "/opt/MoMI-G/yarn-error.log": "EROFS: read-only file system, open '/opt/MoMI-G/yarn-error.log'"
error An unexpected error occurred: "EROFS: read-only file system, mkdir '/gpfs0'"

Inside the momig_frontend.sif, I had git cloned MoMI-G to /opt/MoMI-G and had yarn installed.
Any insights on how to fix this error?
I should add that running singularity outside of sbatch just like this singularity exec --bind $script_path:/scripts $momig_frontend_img bash /scripts/momig_frontend.sbatch inside the script works. But I needed it to run inside the sbatch so I can then start the firefox.
Thank you in advance for your help.

Singularity image is read-only, so you can't write anything. Please copy MoMI-G frontend folder into a host's writable directory and bind the host's directory to /opt/MoMI-G. Another suggestion is to use Docker.

@6br,
Thank you so much for helping me with this.
I'm confused though. Why I can run the frontend like this:
singularity exec --bind $script_path:/scripts $momig_frontend_img bash /scripts/momig_frontend.sbatch
But the following will fail:
sbatch --wrap "singularity exec --bind $script_path:/scripts $momig_frontend_img bash /scripts/momig_frontend.sbatch"
Anyway, I tried your suggestion to copy the MoMI-G frontend folder into ~/MoMI-G (host's writable directory) and bind the host's directory to /momig. Change the momig_frontend.sbatch:
#momig_frontend.sbatch
#!/bin/bash
cd /momig
yarn start

I installed yarn inside the $momig_frontend_img. So, running it like so:
sbatch --wrap "singularity exec --bind ~/MoMI-G:/momig --bind $script_path:/scripts $momig_frontend_img bash /scripts/momig_frontend.sbatch"
now generates this error:
yarn run v1.22.10
$ react-app-rewired start --scripts-version react-scripts-ts
error An unexpected error occurred: "EROFS: read-only file system, mkdir '/gpfs0'".
info If you think this is a bug, please open a bug report with the information provided in "/momig/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Again, in this case the command works if I don't use sbatch.

It's weird that react-app-rewired runs mkdir /gpfs0. Alternatively, MoMI-G frontend can serve with MoMI-G backend without using react-app-rewired.
$ yarn global add serve
$ yarn build

Then, ./build directory will be built on the current directory. Here, storing the path for the built directory in $momig_build_path.
When running MoMI-G backend, the built frontend can be served at the same time with the following command.
$ singularity run --bind $vcf_path:/vg/static --bind $momig_build_path:/vg/build --bind /tmp:/vg/tmp --pwd /vg $momig_backend_img /vg/graph-genome-browser-backend --config=/vg/static/config.yaml --interval=1500000 --http=<IP_ADDRESS>:8081 --api=/api/v2/ --tmp=/tmp/ --serve

Thanks for your suggestion. I've tried following your instruction above, and browse to <IP_ADDRESS>:8081 instead of localhost:3000
It's showing an empty page:

and these are the messages:
[2021-06-28T16:37:18Z INFO  graph_genome_browser_backend::vg] VG Version: vg version v1.25.0 "Apice"
Compiled with g++ (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0 on Linux
Linked against libstd++ 20191114
Built by root@7032b439058e

[2021-06-28T16:37:18Z INFO  graph_genome_browser_backend::vg] VG minor version: 25
[2021-06-28T16:37:18Z INFO  graph_genome_browser_backend::features] Parsing:  "static/gencode.v37lift37.basic.annotation.gff3"
[2021-06-28T16:37:59Z DEBUG graph_genome_browser_backend::features] 1844760 lines processed. end.
25
Start server on <IP_ADDRESS>:8081
[2021-06-28T16:38:51Z INFO  logger] GET http://<IP_ADDRESS>:8081/ 200 OK (1.318558 ms)
[2021-06-28T16:38:51Z INFO  logger] GET http://<IP_ADDRESS>:8081/demo2/static/css/main.0c9c7af8.css 404 Not Found (0.046607 ms)
[2021-06-28T16:38:51Z ERROR iron::iron] Error handling:
Request {
    url: Url { generic_url: "http://<IP_ADDRESS>:8081/demo2/static/css/main.0c9c7af8.css" }
    method: Get
    remote_addr: V4(<IP_ADDRESS>:56904)
    local_addr: V4(<IP_ADDRESS>:8081)
}
Error was: Os { code: 2, kind: NotFound, message: "No such file or directory" }
[2021-06-28T16:38:51Z INFO  logger] GET http://<IP_ADDRESS>:8081/demo2/static/js/main.8e6aab43.js 404 Not Found (0.016962 ms)
[2021-06-28T16:38:51Z ERROR iron::iron] Error handling:
Request {
    url: Url { generic_url: "http://<IP_ADDRESS>:8081/demo2/static/js/main.8e6aab43.js" }
    method: Get
    remote_addr: V4(<IP_ADDRESS>:56906)
    local_addr: V4(<IP_ADDRESS>:8081)
}
Error was: Os { code: 2, kind: NotFound, message: "No such file or directory" }
[2021-06-28T16:38:51Z INFO  logger] GET http://<IP_ADDRESS>:8081/demo2/static/js/main.8e6aab43.js 404 Not Found (0.026399 ms)
[2021-06-28T16:38:51Z ERROR iron::iron] Error handling:
Request {
    url: Url { generic_url: "http://<IP_ADDRESS>:8081/demo2/static/js/main.8e6aab43.js" }
    method: Get
    remote_addr: V4(<IP_ADDRESS>:56906)
    local_addr: V4(<IP_ADDRESS>:8081)
}
Error was: Os { code: 2, kind: NotFound, message: "No such file or directory" }
[2021-06-28T16:38:51Z INFO  logger] GET http://<IP_ADDRESS>:8081/favicon.ico 200 OK (0.970236 ms)

Is it possible to run yarn build on the latest master of MoMI-G?

Sorry for my lack of explanation. Because I updated MoMi-G repository to avoid the error above, MoMI-G needs to be built on the latest master of the GitHub repository. Thus, is it possible to perform git pull on the host's directory where you had git cloned MoMi-G and retry yarn build again? If there is still the same error, please let me know.

@6br, sorry I haven't had time to try this until today.
Thank you so much for your kind clarification. I'm super happy to report that this works!!
I really appreciate your help. Many Thanks!

I'm happy to hear that!


Release v230

To pick up:

#1433

$ make publish
Deploy as version: v230 [y/n]? y

Tagging commit 1d36bc6b9dbbd7becf5022e9ae04b9f4d82df09d with v230...
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
To https://github.com/heroku/heroku-buildpack-python
 * [new tag]         v230 -> v230

Publishing to the buildpack registry...
Two-factor code: ********************************************
Publishing buildpack heroku/python... done
Publishing buildpack was successful

 Version Released At              Status
 ─────── ──────────────────────── ─────────
 230     2023-04-06T09:57:54.229Z published


Update trace options

@rw2: I think you might need to rebase the PR -- looks like you're picking up unrelated commits beyond just the one in the PR title.

Closes #52


Redo without on proxied response

1. What version of Caddy are you running (caddy -version)?
9.5.0
2. What are you trying to do?
When I do a proxy, with the without feature, on the response the url are changed without the path, so the subsequent request are not correct
3. What is your entire Caddyfile?
http://www.lam.local {
  proxy /mattermost gitlab:80 {
    without /mattermost
    header_upstream Host mattermost.lam.local
    header_upstream X-Real-IP {remote}
    header_upstream X-Forwarded-For {remote}
    header_upstream X-Forwarded-Proto {scheme}
  }
  log stdout

}

4. How did you run Caddy (give the full command and describe the execution environment)?
from docker container
5. Please paste any relevant HTTP request(s) here.
curl -X GET http://www.lam.local/mattermost
subsequent request auto-generated
curl -X GET http://www.lam.local/static/info.js
6. What did you expect to see?
I would like to have the subsequent request like this:
curl -X GET http://www.lam.local/mattermost/static/info.js

I'm not sure I understand what you're asking. without is meant to remove that part from the requested URL so that you can have a virtual /path for the proxied site. It seems to me like you should just remove your without directive to get the expected result. If that doesn't fix it, can you give us more information about what you're trying to achieve?

I'm also confused, it looks like it's working as expected... feel free to clarify and, if this is a bug, we can re-open it.

Sorry for the late response, it was I'm misconfiguration on my side !!
Thanks for this project
Le dim. 30 avr. 2017 01:12, Matt Holt<EMAIL_ADDRESS>a écrit :

Closed #1619 https://github.com/mholt/caddy/issues/1619.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mholt/caddy/issues/1619#event-1063280493, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADJrq3E75OdJzUALb5j8YDUTXcOD0hXQks5r08PcgaJpZM4NHdZ6
.


Add greedy_match() and greedy_substring_match()

greedy_match() already existed within the cpp code, but greedy_substring_match() is new.  Both return multiple results.  However, greedy_match() uses prefix matching, while greedy_substring_match() uses substring matching.

@ShaneSmit great, can you provide examples of greedy/non-greedy matches in the Readme; I need to update the Readme in general as well so we have understanding of what the lib does

@ShaneSmit other than naming I'm good

@abrandoned I renamed set to push_or_update as per our conversation.


StripSuffix doc

The input is absl::string_view, not std::string.. same for the other functions in the file.
// StripSuffix()
//
// Returns a view into the input std::string 'str' with the given 'suffix' removed,
// but leaving the original std::string intact. If the suffix does not match at the
// end of the std::string, returns the original std::string instead.
ABSL_MUST_USE_RESULT inline absl::string_view StripSuffix(
    absl::string_view str, absl::string_view suffix) {
  if (absl::EndsWith(str, suffix)) str.remove_suffix(suffix.size());
  return str;
}

Thanks for pointing this out, I'll make a pass over the file.

absl/strings/escaping.h is affected too.


Shortcuts for quick executions

Hey!
I just found out about DBeaver and I'm impressed so far!
One thing that I haven't been able to find yet is some kind of shortcuts for quick executions (similar to Management Studio).
For example, I would like to bind Ctrl + 3 to this script:
EXEC dbo.SPRC_MY_UTILITY_PROCEDURE @pInput = '%s'

... where %s would be the selected text in the editor.
Is there something like this that I have missed? If not, any plans? I'd be happy to help out (I would just need to get familiar with the code, etc.)

No, there is not such feature in DBeaver. I like this.
Unfortunately I can't say when I will have time for it (it will require some extra UI).
Setting 4.1 milestone for now.

After all I think this is almost the same as templates: https://github.com/serge-rider/dbeaver/wiki/SQL-Editor-templates


making vale extensible

the use of an extern or language personality notation to declare or implement a function as the syscall equivalent to linux mmap for the purposes of making edge platforms supportable, potentially also having such denotation for mprotect. This will help minimize the work we need to do, as well as allow vale to compete with languages like rust and python in places such as embedded without the need to maintain a bloated environment, this approach encourages modularity and if done using externs fits within vales notion of platform independence. This approach can also be used in for example JVM to support creation of an array as the memory buffer used by vale, but this is optional. knowing the potential issues with this (such as safety) this function will be implemented by the vale team for major platforms including JVM, CLR, linux, mac, windows, redox, WASM, and JS. fundamentally this is no different from using externs or builtins which is already the case in Midas, and thus I don't understand the issue with safety concerning this issue. since we are implementing for major platforms safety is maintained will still allowing extensibility, and in accordance with the argument deploy in discord, supposing that a platform becomes mainstream, or grows in popularity as a build target, then the vale team will ensure the safety of the extern or syscall used to allocate memory by HGM, resilient, and arena. (and any other regions supported as part of Midas), contrary to the purposed approach of forking Midas, this approach allows rapid reimplementation, and encourages safety through the use of existing tested Midas code, rather then forcing the reimplementation of a region that could cause potentially extensive compromise to the safety and performance that people should come to expect from vale.

Just to be super clear: are you proposing we make it so the user can provide a C implementation for our regions' (HGM, arena, type-pool, etc) calls to mmap?
This would allow the user to provide an untrustworthy chunk of memory (which could be deleted or modified by someone other than the compiler), so it would degrade Vale's safety guarantees. Vale's strength is in knowing that, no matter whose code you depend on, your Vale code will never trigger memory unsafety. If someone publishes a library that defines an extern mmap, and someone accidentally indirectly depends on it, their memory safety could be at risk.
Is there a way to do this without having people publishing packages that could trigger memory unsafety in Vale code?

yeah they would only be able to do it for target triples for which the extern doesn't already exist, but also the OS could always do this, or the user could relink against an alternative version of libc with an invalid malloc, so you can never really be sure it isn't happening

I sure hope there's no way for a fountain module to re-link against a different version of libc with an invalid malloc. Is there a way we can avoid that?
Even on platforms we don't yet support, I'm nervous about someone accidentally depending on a module that breaks memory safety for their vale code.

I'm interested in this discussion from the point of view of developing for a freestanding environment, e.g. hobby kernels/OSs, embedded, etc. I currently have a hobby kernel implementation in Nim, where I just need to provide several C lib functions, namely malloc/free (and friends), memcpy/set/cmp, and strlen/strstr. I was wondering what it would take to support this in Vale.


Use previous available state

Hi @marcelveldt could you revert this change?
Previous, many matter devices in my house would periodically reboot themselves, so they will be like unavailable for 10 seconds and then become available.
However, after updating to 5.2.2 (which contains this change), such devices would become unavailable all the times after they reboot and never be available, unless I manually reboot the matter addon.

No I would not. So you have flaky devices and then we need to revert this?
Did you already update to the latest version? A lot happened since this change

No I would not. So you have flaky devices and then we need to revert this?
Did you already update to the latest version? A lot happened since this change

NVM, I don't think this is the root cause. This issue was introduced in 5.1.0, not 5.1.1
More discussion
https://github.com/home-assistant/addons/issues/3453


Run oracle server rpc tests on CI

Add scripts to actually run npm run test inside of oracle-server-ts

Looks like things are working :tada:
https://github.com/bitcoin-s/bitcoin-s-ts/runs/3767196250#step:5:24


UE 5.0.0 support

Hello! I tried to compile again for UE5 but it doesn't seem to work. Any help or thoughts? I know it's in early access still, but this is a fantastic plugin that I'd love to use.

Yes, at the moment, I cannot compile my project in UE5 because of Fetch.

Is this still an issue?
Or has there been a fix yet?

I wanted to try this utility, before I buy in marketplace.
However it does not compile, build when copied this repository plugin into a Unreal 5.1 project.  So I do not know if I will pursue, if Unreal 5.x is not supported.


raise permission_denied as default

At the moment permission_denied is only raised in certain condition. It is better to raise permission_denied as default and only let the user pass in certain conditions. So if we forget a condition, there is no security issue.
See the following test as example.
    def test_update_via_patch_no_permission(self):
        """
        Send a patch request to update  with permission to see
        users, but without permissions to manage users.
        """
        config['general_system_enable_anonymous'] = True
        self.client = APIClient()

        response = self.client.patch(
            '/rest/users/user/1/',
            {'last_name': 'New last name'})

        self.assertEqual(response.status_code, 403)

See #1589 as example for a solution.

Done, see #1592.


[BUG] Create Button @refinedev/mui  onClick still works when disabled prop is applied

Describe the bug

I added disabled to CreateButton while already specufying onClick and and although it greys out, the onClick still opens my modal.

  const createModalFormProps = useModalForm<
    ITranslationCreate,
    HttpError,
    Nullable<ITranslationCreate>
  >({
    refineCoreProps: { action: "create" },
    syncWithLocation: true,
    defaultValues: defaultTranslationCreateValues
  });

  const {
    modal: { show: showCreateModal }
  } = createModalFormProps;

<CreateButton disabled onClick={() => showCreateModal()}>
  New Translation
</CreateButton>

I believe the issue is the way the disabled and onClick conditionals work in the component, the early return doesn't seem to work.

// refine mui CreateButton
  return (
    <LinkComponent
      to={to}
      replace={false}
      onClick={(e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
        if (disabled) { // NOT WORKING
          e.preventDefault();
          return;
        }
        if (onClick) {
          e.preventDefault();
          onClick(e);
        }
      }}
      style={{ textDecoration: "none" }}
    >
      <Button
        disabled={disabled}
        startIcon={!hideText && <AddBoxOutlined {...svgIconProps} />}
        title={title}
        variant="contained"
        sx={{ minWidth: 0, ...sx }}
        data-testid={RefineButtonTestIds.CreateButton}
        className={RefineButtonClassNames.CreateButton}
        {...restProps}
      >
        {hideText ? (
          <AddBoxOutlined fontSize="small" {...svgIconProps} />
        ) : (
          children ?? label
        )}
      </Button>
    </LinkComponent>
  );

The issue persists  on the EditButton as well.

Steps To Reproduce

Import the CreateButton component from refine/mui
add the disabled prop
add the onClick prop:
the onClick function will still work

Expected behavior
the button is disabled with its onClick not functional as well.
Packages

"@refinedev/mui": "^5.14.4",

Additional Context
No response

Hello @arndom, thanks for the issue!
This bug occurs because the component doesn't use given props. The source code is here.
We try to fix this bug in the December release 🚀

hey @alicanerdurmaz i am up for this .

🚨 This bug exists in all navigation buttons.

🚨 This bug exists in all UI buttons.

Oops my bad . i didn't check other UI buttons. Thanks @alicanerdurmaz  for Fixing remaining ones.


NotFoundLookupError when using dispatch.multi in class method

master 7e9b8657
from plum import dispatch

@dispatch.multi((int, str), (int, int))
def f(x: int, y: {int, str}):
    print(f'{x}, {y}')

class C():
    @dispatch
    def g(self, x: int, y: int):
        print(f'{x}, {y}')

    @dispatch
    def g(self, x: int, y: str):
        print(f'{x}, {y}')

    @dispatch.multi((int, str), (int, int))
    def h(self, x: int, y: {int, str}):
        print(f'{x}, {y}')

f(3, 4) # OK

c = C()
c.g(3, 4) # OK
c.g(3, 'cat') # OK

c.h(3, 4) # NotFoundLookupError
c.h(3, 'cat') # NotFoundLookupError

I did not yet try this on the latest released branch.

If you'd like to use inheritance, you need to use some additional machinery:
from plum import Dispatcher, Referentiable, Self, dispatch

@dispatch.multi((int, str), (int, int))
def f(x: int, y: {int, str}):
    print(f'{x}, {y}')

class C(Referentiable):
    dispatch = Dispatcher(in_class=Self)

    @dispatch
    def g(self, x: int, y: int):
        print(f'{x}, {y}')

    @dispatch
    def g(self, x: int, y: str):
        print(f'{x}, {y}')

    @dispatch.multi((int, str), (int, int))
    def h(self, x: int, y: {int, str}):
        print(f'{x}, {y}')

f(3, 4) # OK

c = C()
c.g(3, 4)
c.g(3, 'cat')

c.h(3, 4)
c.h(3, 'cat')

3, 4
3, 4
3, cat
3, 4
3, cat

This is described here.

Thanks, this seems to work. I had read that section on inheritance, but  I don't understand why it applies here. My example does not use inheritance, right? Except insofar as C inherits from object. If they were plain functions, the two methods for g should be equivalent to the multiple-method definition of h, and in fact the code works that way. As class methods, should not the two methods for g be equivalent to those in the single definition of  h?
I'll try using Dispatcher, Referentiable, Self. The class in my application inherits from a long chain that ultimately inherits from ABC. I'll have to figure out how to combine these.

Ah, sorry, I should have said that this should also be used when classes are used, even if you don't make use of inheritance. The problem is that vanilla functions and method of classes behave slightly differently.
Plum should play nicely with ABC. Here's an example:
import abc

from plum import Dispatcher, Referentiable, Self

class A(metaclass=Referentiable(abc.ABCMeta)):
    @abc.abstractmethod
    def f(self, x):
        pass

class B(A):
    _dispatch = Dispatcher(in_class=Self)

    @_dispatch
    def f(self, x: int):
        return x

>>> A()
TypeError: Can't instantiate abstract class A with abstract methods f

>>> b = B()

>>> b.f(1)
1

>>> b.f("1")
NotFoundLookupError: For function "f" of <class '__main__.B'>, signature (builtins.str) could not be resolved.

Thanks. Your last example has solved my problem.

Perfect! Very glad to hear that.


-Force question

Is -Force really used on an Import-SPWeb command?  If so then what does it do?  I understand the use on the Export-SPWeb command but it's not really clear on the import side.

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

ID: ae171b60-a096-2eef-d096-2c560553ac54
Version Independent ID: b825e536-7724-f08a-677b-8c36d9102ade
Content: Import-SPWeb (sharepoint-server)
Content Source: sharepoint/sharepoint-ps/sharepoint-server/Import-SPWeb.md
Product: sharepoint-2016-powershell
GitHub Login: @techwriter40
Microsoft Alias: kirks

@officedocsbot assign @yogkumgit

Hello @esammann thanks for your feedback.
The article includes the description of the force parameter as Forcefully overwrites the export package if it already exists. and that is what it does.
Does that answers your question?
Thank you

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @esammann Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close

Hello @get-itips and @yogkumgit
I would have to say that doesn't answer my question.  The -Force makes sense on the Export-SPWeb because it will overwrite a previously exported package if one was previously exported to the same location.  However, Import-SPWeb doesn't export anything, therefore there is no export package to overwrite.  It doesn't seem like this parameter makes sense here.
If the -Force were to overwrite a SharePoint site then that might make sense.  However, that seems to be handled with the -UpdateVersions flag.  Therefor the -Force parameter on the Import-SPWeb is confusing.
Thanks,

Hello @Techwriter40 how are you?
The description for -Force parameter in Import-SPWeb cmdlet's article seems to have been copied from Export-SPWeb cmdlet's article, can you confirm please the use of -Force parameter in Import-SPWeb? I pulled my lab and could not find any difference using -Force parameter. Thanks

@Techwriter40  Please reply to Andres's question

I checked a TDS build of SharePoint Server 2016. The Force parameter does exist and is part of the syntax.

Exactly @Techwriter40 that has already been verified by me, the cmdlet includes the Force parameter but the Force parameter does nothing actually and the description on the article is not accurate too. Please check. Thanks

@Techwriter40  kindly confirm Force parameter is working or not and does this documentation needs  to be updated?

Gentle reminder @Techwriter40 kindly confirm Force parameter is working or not and does this documentation needs to be updated?

Hello @Techwriter40 kindly tell me if you have seen my comment about Force parameter current behavior thanks.

Hello @Techwriter40 kindly tell me if you have seen my comment about Force parameter current behavior thanks.

Based on a decompile, I don't see that -Force is referenced at all.

Many Thanks @tseward for that information, so that I think, reinforces my conclusion.

@trent-green Will you be able to please help us here.

Please just remove the -Force switch from this documentation as is must have been copied over from Export-SPWeb, but it serves no purpose on Import-SPWeb since an export package is not created, and is therefor impossible to overwrite an existing export package.

Not quite. The documentation isn't a copy -- Import-SPWeb and Export-SPWeb both have the -Force parameter. This means that this would need to be resolved in the product prior to the document. With that said, what can be done is the documentation can be updated to indicate that this parameter has no effect.

@tseward  Thank you very much for the contribution and updating the documentation with PR. @esammann Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


kube-config install runs docker with invalid flag.

When running kube-config install it fails when starting the systemctl docker service. Running systemctl status docker shows the following:
Process: 15988 ExecStart=/usr/bin/docker -d -H unix:///var/run/docker.sock -s overlay --exec-opt native.cgroupdriver=cgroupfs (code=exited, status=125)
[...]
docker[15988]: flag provided but not defined: -d

Looking in /usr/lib/systemd/system/docker.service.d/docker-overlay.conf, it tries to run /usr/bin/docker with the -d flag, which does not exist.

That's because docker 1.10 removed the -d flag in favor for daemon. I haven't switched to it earlier, because right now docker 1.7.1 is used in the project (and 1.10 was released some days ago)
I'm going to go all-in for docker 1.10 in the next release. Until that, just replace -d with daemon
Just so I know, which os/board do you run on?

Awesome, thanks. We're running Hypriot on Pi 1 Model B's.

Yeah, guessed that. The Hypriot guys adapts new docker releases fast :)

I have updated the config files now (b3d8e2224f4320bda238d7c265e0bd8ff46ff538), and will update the docker daemon later on

I also think this is fixed with v0.6.5. Hope it helps and thanks for notifying :smile:


Doesn't save

No matter whether I save it via "Save Draft" or "Publish changes" -> After reload everything is gone.
It works in some collections, in others it doesn't. I'll hopefully have more information soon...
Sometimes it works, sometimes not, in the same collection (actually global). Weird... I will close this for now since it currently works (I tried it multiple times without changing anything, at some point it suddenly worked) & will reopen it if I have more information.
As a note for others: The indicator was, that both buttons immediately turned back "on" (instead of staying grayed out) & when wanting to leave the site, payload warned me I had unsaved changes.

No matter whether I save it via "Save Draft" or "Publish changes" -> After reload everything is gone

Is this related to #1 by any chance?

Is this related to #1 by any chance?

No, I have drafts but not autosave

did you fix this? @johannesschaffer

Read the updated initial comment. Not really... It however works now (tried it in multiple collections)

Hmmm weird. Looking into the issue with payload saying it's unsaved content now. I think currently, it thinks it's updated when you just click in the richtext editor text. Which shouldn't cause any issues though, it's just a bit annoying

I already raised an issue for that a few days ago. That's a different bug. I specifically clicked on save & it immediatly unsaved (like descriped in the original comment).

Ok, just had the some bug again. Restarting the dev server solves it. I have however no idea what exactly causes it. It's probably a bug with payload (which is in genreal quite buggy:() Some cause seems to be renaming the field (changing the "name" prop).

That's really strange. I'm personally using the richtext editor a lot, and other than when autosave is enabled, I've never had this issue.
Does it only happen if you have multiple editors on one single page by any chance?


Room Background is discracting

Because of the grid (tilting doesn't help), it sometimes look like it's moving, and is unpleasant for the eye while tracking moving objects.
Proposed fix: replace it.

(This text formatting is atrocious)


chore: [DX-2781] indicate that a returned transaction hash does not imply success on sendTransaction

Hi👋, please prefix this PR's title with:

[ ] breaking-change: if you have introduced modification that necessitates immediate adjustments by this SDK's users to their applications, clients, or integrations to avert disruptions to existing features or functionalities.
[x] feat:, fix:, refactor:, docs:, or chore:.

Summary
Add a comment to sendTransaction to inform users that a returned transaction hash does not indicate a successful transaction
Anything else worth calling out?
No code changes

Closing since sendTransaction is never called directly by consumers of Passport.


EntityPopulator.fillcolumns() using method_exists calling private method

I have come across an issue where fillColumns is using 'method_exists' on line 202. This gives rise to issues with a call to a callable but private method in one of our Doctrine objects. Would it be possible to change this call to 'is_callable' instead, as the private method is not then called and causing the process to fail?

Unfortunately the contributor of the Doctrine populator is not around anymore. I'm considering removing the Doctrine populator from the Faker core.


Make the daily test run on A/A + persistent sessions like we plan this for KC26

@ahus1
Question on the Acceptance criteria,
Do we want just this ROSA Scheduled Create Active/Active cluster with Persistent Sessions or a combination of this and the ROSA Scheduled Create Active/Active cluster with External Infinispan?

As agreed in today's meeting: Keep the two scenarios:

ROSA Scheduled Create Active/Active cluster with Volatile Sessions and External Infinispan
ROSA Scheduled Create Active/Active cluster with Persistent Sessions and External Infinispan


fix: improved reliability of functional tests that activates dip24

Issue being fixed or feature implemented
Functional tests in CI and locally often fails without a good reason (pretty randomly)
What was done?
Seems as if Spork 19 (chainlocks enabled) are disabled before activate dip0024 the function tests fails less often (as tested locally)
How Has This Been Tested?
Run functional times 10 times with and without these changes.
Without these fixes at least one test related to dip-0024 failed in ~50% of runs
With these changes all failures happened only for p2p_addrv2_relay.py and mempool_unbroadcast.py
Breaking Changes
No breaking changes
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[ ] I have assigned this pull request to a milestone

> feature_llmq_hpmn.py                  | ✖ Failed  | 1802 s

seems it didn't help

No thoughts 🤷 I don't see why this changes anything. Up to Udjin

Running ./test/functional/test_runner.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py:
develop:
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 192 s
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 199 s
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 200 s
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 207 s

ALL                             | ✓ Passed  | 798 s (accumulated)
Runtime: 207 s

this PR:
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 194 s
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 199 s
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 201 s
feature_llmq_is_cl_conflicts.py | ✓ Passed  | 202 s

ALL                             | ✓ Passed  | 796 s (accumulated)
Runtime: 202 s

Basically, no difference. Also, I agree with @PastaPastaPasta , this patch shouldn't affect tests behaviour after Activated DIP0024 at height:xxx. But if it does for you then smth could be actually broken and needs to be fixed, not skipped. Or maybe you just need to lower -j - you could be triggering OOM-killer or smth.

But if it does for you then smth could be actually broken and needs to be fixed, not skipped. Or maybe you just need to lower -j - you could be triggering OOM-killer or smth.

Probably you are right about an under-lying issue, but I can't figure out what's a reason. I have 2 testing environment laptop (40Gb ram + 16 cores) and PC (32Gb RAM + 24 cores) and problem reproduced in laptop with bigger amount of RAM but smaller amount of CPU. So, I assume, that it's not OOM-killer but probably something else.

This PR:

$ git checkout c16fa61
$ test/functional/test_runner.py -j 20 feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py  feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py
<all 20 succeed>

A commit from develop:

$ git checkout 74fabcb1b9e5133b579717acace175338e729130
$ test/functional/test_runner.py -j 20 feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py  feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py feature_llmq_is_cl_conflicts.py
1st failure:
2023-04-17T11:47:06.620000Z TestFramework (INFO): Wait for dip0024 activation
2023-04-17T11:47:51.467000Z TestFramework (INFO): Activated DIP0024 at height:902
2023-04-17T11:48:02.043000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 148, in main
    self.run_test()
  File "/home/knst/projects/dash-reviews/test/functional/feature_llmq_is_cl_conflicts.py", line 78, in run_test
    self.test_chainlock_overrides_islock_overrides_nonchainlock(False)
  File "/home/knst/projects/dash-reviews/test/functional/feature_llmq_is_cl_conflicts.py", line 214, in test_chainlock_overrides_islock_overrides_nonchainlock
    islock = self.create_islock(rawtx2, deterministic)
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 1469, in create_islock
    rec_sig = self.get_recovered_sig(request_id, message_hash, node=quorum_member.node, llmq_type=llmq_type)
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 1915, in get_recovered_sig
    assert False
AssertionError
-------------
2nd failure:
2023-04-17T11:46:29.080000Z TestFramework (INFO): New quorum: height=216, quorumHash=496bbebef32e4e38fa71c48acfafaa1d5ccde556da7ddc7ed4717e42e78b64dc, quorumIndex=0, minedBlock=6d6f86b6be658de1b021812d9fa6467502bb097673fdcefd25992424d6e7e770
2023-04-17T11:46:43.153000Z TestFramework (ERROR): JSONRPC error
Traceback (most recent call last):
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 148, in main
    self.run_test()
  File "/home/knst/projects/dash-reviews/test/functional/feature_llmq_is_cl_conflicts.py", line 74, in run_test
    self.test_chainlock_overrides_islock(True, True)
  File "/home/knst/projects/dash-reviews/test/functional/feature_llmq_is_cl_conflicts.py", line 91, in test_chainlock_overrides_islock
    rawtx1 = self.create_raw_tx(self.nodes[0], self.nodes[0], 1, 1, 100)['hex']
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 1434, in create_raw_tx
    rawtx = node_from.createrawtransaction(inputs, outputs)
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/coverage.py", line 47, in __call__
    return_val = self.auth_service_proxy_instance.__call__(*args, **kwargs)
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/authproxy.py", line 144, in __call__
    raise JSONRPCException(response['error'], status)
test_framework.authproxy.JSONRPCException: Amount out of range (-3)
2023-04-17T11:46:43.654000Z TestFramework (INFO): Stopping nodes
------------
3rd failure:
2023-04-17T11:47:52.668000Z TestFramework (INFO): Activated DIP0024 at height:902
2023-04-17T11:48:03.120000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 148, in main
    self.run_test()
  File "/home/knst/projects/dash-reviews/test/functional/feature_llmq_is_cl_conflicts.py", line 78, in run_test
    self.test_chainlock_overrides_islock_overrides_nonchainlock(False)
  File "/home/knst/projects/dash-reviews/test/functional/feature_llmq_is_cl_conflicts.py", line 214, in test_chainlock_overrides_islock_overrides_nonchainlock
    islock = self.create_islock(rawtx2, deterministic)
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 1469, in create_islock
    rec_sig = self.get_recovered_sig(request_id, message_hash, node=quorum_member.node, llmq_type=llmq_type)
  File "/home/knst/projects/dash-reviews/test/functional/test_framework/test_framework.py", line 1915, in get_recovered_sig
    assert False
AssertionError
2023-04-17T11:48:03.621000Z TestFramework (INFO): Stopping nodes

<total failed 3/20>

I can't reproduce any failures even with -j 40 🤷♂
@knst pls try running it with db38a232 and ba250184

@UdjinM6 I re-wrote get_recovered_sig, please, help to review it.
With your changes fr selection/change output and the new get_recovered_sig it works more reliable at my laptop now.

pls see 896f0763d8


Qubit rotation

Further to https://github.com/qgrad/qgrad/issues/1, I have added the qubit rotation tutorial using JAX's grad on the cost function (fidelity in this case) in the gradient descent optimization routine. This hopefully gives us a better idea of how to proceed with adding autodiff capabilities in QuTiP. There are a few hiccups that we will need to tackle. I shall create separate issues for them and link them here.

Hi, it is quite difficult to review code as .ipynb notebooks so I would say that you should make a folder called examples and put the example you write here as a .py file. Then we can easily comment and change things. Please do so and then close this PR by adding a comment to the new PR with something like:
"closing this PR via #(the new PR number)"

I agree on the separate folder. This Review Notebook app makes it somewhat easier to review and visualize (since many times Github jupyter notebooks visualization fails) notebooks. Maybe we could use it.

By the way, @araza6, I like the way you listed different features in separate commits. That makes reviewing easier in general: let's stick to that.

Actually, on "diffing" and reviewing notebooks, there is this tool within the Jupyter organization that is actively developed, https://github.com/jupyter/nbdime.

Thanks for the helpful comments!

I agree with your opinion on .ipynb files in the example folder. There is a trade-off between user-experience and ease of code review. Can we do something similar to QuTiP? As in have a qgrad-notebooks repository under qgrad organization?

The motivation behind importing numpy twice (and differently each time) is because JAX's numpy, although generally faster and equipped with just-in-time compilation among other things, does not work well with QuTiP's Qobj; for that, we need the actual numpy. It should be noted, however, that JAX's numpy wrapper does not come with everything traditional numpy comes with. More here.

As for the way libraries are imported, the import convention(s) is/are largely the same. JAX's [tutorials] (https://jax.readthedocs.io/en/latest/notebooks/Common_Gotchas_in_JAX.html) use import numpy as onp, whereas we, as you point out, use import numpy as rnp. We can probably stick to JAX's convention of onp.

As for prefixes, you mean to import jax and use jax.grad in the notebook every time we use grad?

Hi, I agree with the separate notebooks repo. Notebooks are nice and there are many options to allow for beautiful documentation which we should definitely provide (including easy download buttons as the user browses through notebooks).
Since we are still tinkering around with code now, I would say we should write most of the examples as .py files and then use the nbconvert option to get to notebooks later. We could even just use sphinx gallery. This will avoid unnecessary pain later and make sure we can test the examples also.
Please note that even in .py files you can add markdown and images etc. (like this example which gets rendered as https://pennylane.ai/qml/demos/tutorial_data_reuploading_classifier.html)
I have a new example using purely Jax to implement this notebook which I will soon upload as an example here.

Also, @nathanshammah it is a bit confusing but @araza6 is right about the imports here. Since we need to work with two different Numpys. The easy solution would have been to somehow make a qutip fork which replaces all the import numpy as np with import jax.numpy as np but it will never work.

There are still outstanding changes that have not been addressed, from my comments above:

Add a title to the notebook in Markdown.

please introduce clearly the scope of the example at the beginning of the notebook

please comment the result on the Bloch sphere. Just like in paper, figures better be commented with some "captions". Also, comment what the different colored arrows represent.

That's fine with the library imports for the moment, as I need to understand better how this is done.

PR #8 is taking care of the gitignore and should be merged first, once ready. Then we can sync this one with the origin master.

@nathanshammah I am including all your suggested changes in a .py file and opening a new PR as @quantshah had suggested.

Closing this on PR https://github.com/qgrad/qgrad/pull/10


Add More Examples

I've just implemented typeorm entities + repositories in my project and would like to add them as examples

100%! That would be great! Submit a PR and we can see if it needs any tweaks and we can get the typeorm examples in! I have been meaning to create prismajs examples, maybe this would be the boost of inspiration I need

Glad to share! Do you mind if I open another issue about adding some generics to repositories and server interfaces ?

Just commented on the PR with one small ask of adding a package.json file.
Regarding the generics on repositories, I'm not as sold on that one just yet. I guess I'm not sure I understand the reason "why" just yet? Could you explain more about what benefits we are getting by making repositories generic?
My concern is that we are going to make the API even more intimidating than it already admittedly is by adding this.
If there is good reason, I am willing to listen and hear you out!

The API will remain the "same" for anyone willing not to use the generics as every generic I'm adding defaults to OAuth interfaces.
As you can see in typeorm example:
tokenRepository methods:
const tokenRepository = {
    async issueToken(client: OAuthClient, scopes: OAuthScope[], user?: OAuthUser): Promise<OAuthToken> {
      if (!user || !(user instanceof OAuthUserImp)) throw "Need valid User to issue Oauth2 Token";
      const expiration = new Date();
      expiration.setSeconds(expiration.getSeconds() + parseInt(process.env.JWT_EXPIRE as string) || 50000);
      return {
        accessToken: "fresh-token",
        accessTokenExpiresAt: expiration,
        client: client,
        scopes: scopes,
        user: user,
      };
    }

    async persist(accessToken: OAuthToken): Promise<void> {
      const tokenRepo = getRepository(OAuthTokenImp, process.env.NODE_ENV);
      const newToken = OAuthTokenFactory.from(accessToken);
      await tokenRepo.save(newToken);
      return;
    }
}

I need to create OAuthTokenImp from OAuthToken. By looking at the implementation I know the object returned in issueToken is the same passed to persist so I could return an OAuthTokenImp and save it in persist without the need of  using the factory. But the repository definition does not express this. With generics I can return an OAuthTokenImp in issueToken and be sure to get it back in persist.
const tokenRepository: OAuthTokenRepository<TToken, TClient, TScope, TUser> = {
    async issueToken(client: TClient, scopes: TScope[], user?: TUser): Promise<TToken> {
      const expiration = new Date();
      expiration.setSeconds(expiration.getSeconds() + parseInt(process.env.JWT_EXPIRE as string) || 50000);
      return new OAuthTokenImp(...params) ;
    }

    async persist(accessToken: OAuthToken): Promise<void> {
      const tokenRepo = getRepository(OAuthTokenImp, process.env.NODE_ENV);
      await tokenRepo.save(newToken);
      return;
    }
  }

Also it let you use extra properties without the need to typecheck
from the example above (this is not necesary anymore using generics)
 if (!user || !(user instanceof OAuthUserImp)) throw "Need valid User to issue Oauth2 Token";

Hey @thefat32,
Sorry, I just am still not convinced this is a nessary addition.
I updated the typeorm example and I've tweaked the repositories slightly from an object to an actual class. In the updated example, there is no need for instance checking on models, because the compiler will now know what is being returned.
By having your entities implement the @jmondi/oauth2-server entity interfaces (which your example was doing), you are able to use these implementations as the types on your repositories.
Here is an example of the same methods as above using your own typed entities:
 export class TokenRepository implements OAuthTokenRepository {
  constructor(private readonly tokenRepo: Repository<Token>) {}

  async issueToken(client: Client, scopes: Scope[], user?: User): Promise<Token> {
    const token = new Token();
    token.accessToken = generateRandomToken();
    token.accessTokenExpiresAt = new DateInterval("2h").getEndDate();
    token.client = client;
    token.clientId = client.id;
    token.user = user;
    token.userId = user?.id;
    token.scopes = []
    scopes.forEach((scope) => token.scopes.push(scope));
    return token;
  }
  async persist(accessToken: Token): Promise<void> {
    await this.tokenRepo.save(accessToken);
  }
}

There is no need to do any instance checking to verify this is the class you expect.
By refactoring your implementation into actual class based repositories, it also eliminates the need for each method requiring a repository to new a repository every time the method is called, now repositories are initialized in main.ts once, and passed into our repository implementations
Another key detail change that imo really cleans up the implementation is using the typeorm repository method findOneOrFail as seen here:
 export class TokenRepository implements OAuthTokenRepository {
  constructor(private readonly tokenRepo: Repository<Token>) {}

  async getByRefreshToken(refreshTokenToken: string): Promise<Token> {
    return this.tokenRepo.findOneOrFail(
      { refreshToken: refreshTokenToken },
      { relations: ["client", "scopes", "user"] },
    );
  }
}

This way, your application will throw an exception when attempting to find an entity that does not exist. IMO this is much better than having implementation details check to verify the object exists on every fetch.

Curious what your tsconfig.json settings are? I have mine set pretty strict and have never come across that error before. If you are able to, would you be able to set up a small example demonstrating these errors? No worries if that is too much work... It would just help me see the issues you are running into more easily. I'll reopen the ticket since it seems like there is more here :D

Here is my tsconfig
{
   "compilerOptions": {
      /* Basic Options */
      "incremental": true,
      "target": "es6",
      "module": "commonjs",
      "outDir": "dist",

      /* Strict Type-Checking Options */
      "strict": true,
      "noImplicitAny": true,
      "strictNullChecks": true,
      "strictFunctionTypes": true,
      "strictBindCallApply": true,
      "noImplicitThis": true,
      "alwaysStrict": true,

      /* Additional Checks */
      "noUnusedLocals": true,
      "noUnusedParameters": true,
      "noImplicitReturns": true,
      "noFallthroughCasesInSwitch": true,

      /* Module Resolution Options */
      "moduleResolution": "node",
      "baseUrl": ".",
      "esModuleInterop": true,
      "resolveJsonModule": true,

      /* Experimental Options */
      "experimentalDecorators": true,
      "emitDecoratorMetadata": true,

      /* Advanced Options */
      "forceConsistentCasingInFileNames": true,
      "typeRoots": [
         "@types",
         "node_modules/@types"
      ]
   }
}

Will try to setup an example project at night!

Awesome I'll see if I can recreate the errors also

I debug the problem deeply, my User entity key is number instead of string and that was the problem with OAuthToken being incompatible with OAuthTokenImp. Because OAuthTokenImp contains a relation to User. Don't waste more time on this! Sorry!
Closing the issue


The requested URL /install was not found on this server.

The requested URL /install was not found on this server.
PHP的版本有要求吗？

请参照一个标准的laravel程序去开启webserver的rewrite


Emilhotkowski/e2e tests v1

Simple e2e tests before we add auth.

It seems that only some basic tests for the disable domain endpoint are included. For that endpoint, it also does not test that disabling a domain actually prevents further signature requests.

@nategraf I believe we never talked about mechanism for disabled domain prevent signing nor did we talk about how to handle domain's that can'e be disabled. I can include more tests and add this mechanism
CC @alecps

Sorry, if I could have been more clear, but CIP-40 does state the purpose of disabling a domain and what it's effects should be.

closing in favor of https://github.com/celo-org/celo-monorepo/pull/9161


Setup environment variables

Currently we have a limited use case for this, but it may expand over time. Initially we'll need it for GitHub links. Take the Source and Docs links shown here:

These are currently hardcoded the the /dev/ branch path. However, this should be should be:

Production documentation -> /master/ branch path
Dev/Local documentation -> /dev/ branch path

Given these are not sanative information we can use the simplest method SveltKit provides for this.

I've done this many times for Angular projects, but not yet for SvelteKit, so I welcome suggestions from anyone that has set these up before. Probably worth checking the latest SvelteKit docs on the subject.
@niktek @Sarenor @luke-hagar-sp any experience with this? I'd welcome your feedback.


cp: Wrong handling of -i -u

When copying an old file over a new file, cp still needs to ask: -u is --update=older, which according to the man page only influences which files are considered for overwriting. In other words, -u does not work by automatically allowing anything, but rather by disregarding some copy actions.
Minimal example:
$ touch old; sleep 2; touch new; echo N | LC_ALL=C cp -i -u new old
cp: overwrite 'old'? [$? = 1]
$ touch old; sleep 2; touch new; echo N | cargo run cp -i -u new old
$

Found while trying to fix #4725.

I am ready to take this issue. please let me know..

@Vikrant2691 I haven't verified this myself, but I trust @BenWiederhake :) Feel free to pick this up!

@Vikrant2691 I'm glad you're helping, yes, please do! :D

Thank you @BenWiederhake and @tertsdiepraam, I will try to replicate the issue at my end and check out! :)

I was able to replicate the issue
$ echo "hello">old; touch new
$ coreutils cp -i -u new old  # no prompt
$ cp -i -u new old
cp: overwrite 'old'? N

Additionally, coreutils is not overwriting the file.

Hi @BenWiederhake and @tertsdiepraam please let me know if the PR is ok to be merged or needs more work. :)

Closing, because #6069 was factually merged.


Better animations

Sorry for bothering you again.
Current HUD transition between states without text and with some text/details (and when changing visible text/details values) does not really look good -
text/detail labels are floating over the HUD borders. Is it possible to somehow fix these transitions (to make text labels appearing from the center of HUD and not crossing the HUD borders)?
Here is the example video: http://youtu.be/IVEkKvK_wRs
Thanks.

Closed due to inactivity.


File "/usr/lib64/python2.7/ConfigParser.py", line 370, in getboolean

System

CentOS 7.1
Package python-2.7.5-58.el7.x86_64 installed
Package python-inotify-0.9.4-4.el7.noarch installed

What did I miss during installation?
2017-10-17 11:09:31.756856 Daemon started
2017-10-17 11:09:31.757472 web:nginx: /var/www/html
Traceback (most recent call last):
  File "/bin/watcher", line 384, in <module>
    daemon.start()
  File "/bin/watcher", line 124, in start
    self.run()
  File "/bin/watcher", line 271, in run
    recursive = self.config.getboolean(section,'recursive')
  File "/usr/lib64/python2.7/ConfigParser.py", line 370, in getboolean
    raise ValueError, 'Not a boolean: %s' % v
ValueError: Not a boolean:

It was a syntax error in the ini file.


Sprite.Sheet and Sprite.Draw blocks

Implements blocks Sprite.Sheet and Sprite.Draw.
Setup a spritesheet by reading an .ednfile (can be generated from a spine .atlas file using spine-rust):
(Setup
  (LoadImage "data/skeleton.png") (GFX.Texture2D) = .texture
  (read-string (slurp "data/skeleton.edn")) = .atlas
  (Sprite.Sheet :Atlas .atlas) = .sheet )

Draw the first sprite of the sprite sheet:
(Sprite.Draw :Texture .texture :Sheet .sheet :Index 0
             :Shader .shader :Blend {"Src" Blend.SrcAlpha "Dst" Blend.InvSrcAlpha "Op" BlendOp.Add})

Index can be a variable (so it can later be animated):
(->
  1 >= .index)
(Sprite.Draw :Texture .texture :Sheet .sheet :Index .index
             :Shader .shader :Blend {"Src" Blend.SrcAlpha "Dst" Blend.InvSrcAlpha "Op" BlendOp.Add})

Some considerations:

(Sprite.Draw) needs to be given a shader. We should have a default fallback.
:Blend do have a default fallback, which is the same as here (i.e. Src * alpha + Dst * (1 - alpha)).

See sprite_test.edn for a complete example.

There's a debug.edn/debug.png that's not used. maybe use that for the test or remove it.

The debug file is useful for any kind of testing. So I thought to include it. I sometimes used it in the sprite_test.edn even though at the moment it uses the other skeleton sprite.

What do we want to do with this @guusw @Kryptos-FR ?

@sinkingsugar @guusw it was based on bgfx, which we don't have anymore.
That said, the basic EDN "look" of it could be kept. It all comes down to whether we want to support that kind of sprite atlases. We should have a meeting about it.

Still it's a PR, so I'd close for now.

Closing for now. Will need some rework if we include it later.


Create List of Submissions Report (CAT)

Based on a meeting held with Chris on 1/25/24, this functionality will be included in the initial 2.0 release.

@annalbrecht and @alangmaid should this ticket #4675 be closed due to ticket #6110? We can discuss at the next standup, as needed, thx.
FYI @maheese @j-tafoya @bryanramirez1

@annalbrecht and @alangmaid should this ticket #4675 be closed due to ticket #6110? We can discuss at the next standup, as needed, thx.
FYI @maheese @j-tafoya @bryanramirez1

@ibarra-michelle @alangmaid This ticket is for Admin functionality for User Support to be able to see all Submissions made by all users. #6110 is user-facing functionality so that they can see what bulk files they have submitted.

Thanks for clarifying @annalbrecht . I withdraw my suggestion to close this ticket #4675 since #4675 is for the Admin view and #6110 is for the user's view.


GraalVM native-image support for Java, Scala, Kotlin, etc.

Can  you elaborate? Is this a request for execution support of some new languages? We don't (yet!) support any of those languages. Java support (to intermediate representation) is coming soon in #1396, but we currently have no support for Scala or Kotlin. I'm a little reluctant to add JITted languages for a variety of reasons.
What's "native-image" in this context?
Thanks!

Yes, it is about adding those languages, but using AOT compilation (not JIT).

Thanks. I'll need to check the GraalVM license allows this.


Agent UI fixes

SQL Agent UI fixes -

[x] Show Steps with full error message. Fixes https://github.com/Microsoft/sqlopsstudio/issues/1375
Screenshot -

[x] Show ellipsis in SQL Agent Job name (as per Smitha's request). Screenshot -

[x] Remove mininum width (minWidth option) from name column (Fixes https://github.com/Microsoft/sqlopsstudio/issues/1386).

[x] Changed Status indicator for "Unknown Status" job from orange to grey (as per Smitha's request).

[x] Fixed issue where resizing in the Jobs History page could crash the app.

@abist from the screenshots it looks like messages and step name are centered.  I think these fields should probably be left-aligned.


wrapdb.mesonbuild.com is not reachable

https://wrapdb.mesonbuild.com/ is not reachable. Apparently the server is offline.
$ curl -v wrapdb.mesonbuild.com
* Rebuilt URL to: wrapdb.mesonbuild.com/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* connect to <IP_ADDRESS> port 80 failed: No route to host
* Failed to connect to wrapdb.mesonbuild.com port 80: No route to host
* Closing connection 0
curl: (7) Failed to connect to wrapdb.mesonbuild.com port 80: No route to host

$ dig wrapdb.mesonbuild.com

; <<>> DiG 9.11.3-1ubuntu1.11-Ubuntu <<>> wrapdb.mesonbuild.com
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 32820
;; flags: qr rd ra; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 65494
;; QUESTION SECTION:
;wrapdb.mesonbuild.com.		IN	A

;; ANSWER SECTION:
wrapdb.mesonbuild.com.	3217	IN	A	<IP_ADDRESS>

;; Query time: 0 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Mon Jan 13 07:43:38 CET 2020
;; MSG SIZE  rcvd: 66

Pinging <IP_ADDRESS> results in Destination Host Unreachable.

/sub

Looks like the main issue is being tracked here: mesonbuild/meson#6446


add missing doc about timeout and pred_name

@pprett review please.

Coverage remained the same at 84.345% when pulling 2659485a949b9157fe9d03c14bc644662eb66960 on kyrylo/fix_some_doc into 66bd82ade9a2a6cf9356b9774626a4b6081158ec on master.

LGTM


Let functions like plotFuncs and plotFuncsDer save the figure instead of just showing it

add save_fig argument to allow plotFuncs to save figure instead of just showing plt.show()

We can do this very easily, but plotFuncs is mostly meant as a convenience function for quick interactive explorations.  I didn't put any arguments for axis labels (etc), so any "real" figure code is written out explicitly.

Not sure how to tie in this broad question with the tightly scoped issue here, but...
In our meeting this morning we discussed how there's a few different environments that seem to be informally supported:

local install
Binder
headless server

In some of the notebooks, REMARKs, etc, plotFuncs is involved in doing the work of having the code work across these different environments.

BTW, I've done this to a limited extent with the BufferStockTheory
notebook; would be happy to have what I did there improved upon.
On Thu, Dec 12, 2019 at 9:47 AM Mridul Seth<EMAIL_ADDRESS>wrote:

add save_fig argument to allow plotFuncs to save figure instead of just
showing plt.show()
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/456?email_source=notifications&email_token=AAKCK75EQARIETPJEIEOKGLQYJFHLA5CNFSM4JZ7VVUKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IACKKIA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKCK7734566EQLFDZLQOQDQYJFHLANCNFSM4JZ7VVUA
.

--

Chris Carroll

https://github.com/econ-ark/HARK/blob/master/HARK/utilities.py#L902 make_figs exists to save figures locally.


API documentation for Repository still uses changeset directly on repo

I just tried out 4.0.0.rc1 on my existing code, and my use of repo.changeset caused a NoMethodError. When trying to find out why, I noticed that the current API documentation (i.e. in the source) for Rom::Repository and Rom::Repository::Root both still contain examples where changset is called directly on a repo.
As I just stumbled over this and have little knowledge of the changes, I'm still not sure if it's the documentation or the implementation that is wrong.
(I guess I should reference the changeset on the specific relation now: repo.my_relation.changeset ?)

Yes, starting with rom 4.0 we have Relation#changeset and Repository#changeset is gone. I'll be updating docs before final is released.


Add a HealthCheck Enricher for Webapps

we should generate a readiness & liveness checks OOTB for war packaging apps.
If there is a service annotation for api.service.kubernetes.io/path we could use that if no explicit happy page URL is configured in the pom.xml; otherwise we could just default to using /

the health check enricher should work in the same spirit as the enrichers we have now for Vertx, Prometheus etc.
Alternatively, the webapp generator could also add just a Docker HEALTHCHECK we already can be reused by the DockerHealthCheckEnricher


GSSAPI Authentication (Enterprise) Doc Update

Closes #7203

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0dc2a59f2830be5621a1b4c9c1f726529f0b16d8/
Edited pages:

v19.2/gssapi_authentication.md
v20.1/gssapi_authentication.md

Slack conversation with Artem: The text repetition is intentional since it's contextual information that's relevant to both sections and also has implementation-specific information. Will revisit it when working on the GSSAPI docs project though.


IOS Web View login issue-- Error in Navigation!

Description
NOTE:'Error in Navigation!', [Error: Couldn't find a navigation context. Have you wrapped your app with 'NavigationContainer'? See https://reactnavigation.org/docs/getting-started for setup instructions.]
Steps to Reproduce
Expected Behavior
Relevant Logs / Console output
"newrelic-react-native-agent": "^1.2.1",
"react-native": "0.69.3",

ex: Output of npx react-native info
ex: Which platform is the issue happening on? (Android, iOS, tvOS, etc.)
ex: Any conflicting packages installed?

Additional context
ex: If appropriate and possible, it would be helpful to add Charles traces.

@somesingh95 can you share more details and also share your navigation container of react native app?

this is not related our agent, check stack overflow for this issue.

@somesingh95 can you share what did you add? if possible can you share example project ?

i just added these code in index.js
and webview login failure in IOS then getting error
'Error in Navigation!', [Error: Couldn't find a navigation context. Have you wrapped your app with 'NavigationContainer'? See https://reactnavigation.org/docs/getting-started for setup instructions.]
import NewRelic from 'newrelic-react-native-agent';
import Config from 'react-native-config';
let appToken=Config.NEWRELIC_TOKEN;
const agentConfiguration = {
//Android Specific
// Optional:Enable or disable collection of event data.
analyticsEventEnabled: true,
// Optional:Enable or disable crash reporting.
crashReportingEnabled: true,
// Optional:Enable or disable interaction tracing. Trace instrumentation still occurs, but no traces are harvested. This will disable default and custom interactions.
interactionTracingEnabled: true,
// Optional:Enable or disable reporting successful HTTP requests to the MobileRequest event type.
networkRequestEnabled: true,
// Optional:Enable or disable reporting network and HTTP request errors to the MobileRequestError event type.
networkErrorRequestEnabled: true,
// Optional:Enable or disable capture of HTTP response bodies for HTTP error traces, and MobileRequestError events.
httpRequestBodyCaptureEnabled: true,
// Optional:Enable or disable agent logging.
loggingEnabled: true,
// Optional:Specifies the log level. Omit this field for the default log level.
// Options include: ERROR (least verbose), WARNING, INFO, VERBOSE, AUDIT (most verbose).
logLevel: NewRelic.LogLevel.INFO,
// iOS Specific
// Optional:Enable/Disable automatic instrumentation of WebViews
// webViewInstrumentation: true,
// Optional:Set a specific collector address for sending data. Omit this field for default address.
// collectorAddress: "",
// Optional:Set a specific crash collector address for sending crashes. Omit this field for default address.
// crashCollectorAddress: "",
// Optional:Enable or disable reporting data using different endpoints for US government clients
// fedRampEnabled: false
};
NewRelic.startAgent(appToken,agentConfiguration);
NewRelic.setJSAppVersion(version);

@somesingh95 did you add this export const navigationRef = React.createRef(); ?

@ndesai-newrelic
yes I’m using this

@somesingh95 I am unable to reproduce this, can you share sample app for this?


Create the ComputationContext class containing all the Locators

The puzzle 4-628d64ce from #4 has to be resolved:
https://github.com/nergal-perm/java-decita/blob/3c0c8b7be900ea89307f94d1653b074e9050bd62/src/main/java/ru/ewc/decita/DecitaFacade.java#L52-L53
The puzzle was created by Евгений Терехов on 31-Dec-23.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd the puzzle #11 is still not solved.

@0pdd the only puzzle #11 is solved here.


Fix/prettier and add prettier check in github workflow

[x] Fix Format style using prettier
[x] Add check for prettier in GitHub workflow

This PR has changes of this PR #473 in this one, please fix it first

where ? I am not able to see the changes from #473 in this PR

Have you made changes to 17 files, these much files cannot be reviewed. Please check if there is any unnecessary changes being moved

See the number of line changes not files. Btw only prettier changes made nothing extra.

Have you made changes to 17 files, these much files cannot be reviewed. Please check if there is any unnecessary changes being moved

You are saying that it is not possible to review this PR?

Have you made changes to 17 files, these much files cannot be reviewed. Please check if there is any unnecessary changes being moved

You are saying that it is not possible to review this PR?

if only a prettier file is changed then why are you  sending extra files in PR
please see bhavika's comments

Where? Can you comment on the line which was added?


feat: port FSRS5 from py-fsrs

Port from https://github.com/open-spaced-repetition/py-fsrs/compare/1b4cbe4...a200a78
Closes #10

I think I want to add documentations alongside with this pull request. Which means #8 could be blocking this PR.

@brainwo No problem. If you require #8 on this, just put it on.

Looking forward to this! @brainwo


[BUG] Error prevents playing

Bug Description
When you try to play a song with queue.play() an error is thrown: Invalid regular expression: missing /
Reproduction Steps
Steps to reproduce the behavior:
Use queue.play with any content. This worked with the sample code copied from the Example Usage section on this page of your docs.
Expected Result
A song should play.
Attachments
node:internal/errors:466
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_UNHANDLED_ERROR]: Unhandled error. ('Invalid regular expression: missing /')
    at new NodeError (node:internal/errors:377:5)
    at Player.emit (node:events:516:17)
    at FFmpeg.<anonymous> (PATH TO MY PROJECT\node_modules\discord-music-player\dist\managers\Queue.js:242:29)
    at FFmpeg.emit (node:events:539:35)
    at PassThrough.<anonymous> (PATH TO MY PROJECT\node_modules\discord-ytdl-core\index.js:1:1100)
    at PassThrough.emit (node:events:539:35)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 'ERR_UNHANDLED_ERROR',
  context: 'Invalid regular expression: missing /'
}

Node.js v18.1.0

Versioning:

OS: Windows 10 (though the error was also present on Ubuntu 20.04 (Heroku Stack heroku-20))
Node Version v18.1.0
Module Version v9.0.2

Update: This is in fact a bug in ytdl-core. For anyone who looks at this before a fix comes out, rollback ytdl-core to 4.10.0 to solve.


Allow the ProjectContributors to appear even if access to the GitHub project is not allowed

The main goal is to be able to create allocations without needing OAuth access to the GitHub repository since the You must be a Github collaborator to access this metrics notice appears if that is the case.
That access is only needed to fetch the current repository contributors so we should be able to allow the user to find the contributor that is already in the Trinary DB and create allocations

Also might be good to improve this error notice by:

Showing the Github URL with a clickable link to go to the repository to make sure they can access it
Linking the user to the page where they can see which organization to Request or Grant the OAuth access in case they forgot to do this on the very 1st sign-in

@mecoccaro


Setting Vim's filetype

I use another plugin along with External Editor that processes the field as Markdown instead of HTML. I thought I could set the filetype by configuring the external editor plugin like this:
{
    "editor": "vim -gf -c 'set filetype=markdown'",
    "shortcut": "Ctrl+Meta+X"
}

I've tried every variation of quotes and escaped quotes or spaces that I could think of, but I can't get the editor to open with a markdown filetype the way it does on the command line.  Is there a way to do this with External Editor?

Your configuration should now work as it is, I uploaded an update for the add-on on AnkiWeb.


Misleading information in dependencies-element

The dependencies-element attributes table states that both id and version are required while example in dependency groups section below shows example without version:
<group targetFramework="net40">
    <dependency id="jQuery" />
    <dependency id="WebActivator" />
</group>

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.
ID: 06a19d7d-253d-f17f-6421-624c9867473d
Version Independent ID: cc37eb17-f715-080c-7bfc-9eae868d50ba
Content: .nuspec File Reference for NuGet
Content Source: docs/reference/nuspec.md

Updating doc


If user is not in chrome open chrome for reverse image search feature

if the user is not in a browser tab when user says "screenshot or reverse image search"
either

open browser tab
tell user this feature is only available on the the browser.

console gives errors like Error in response to tabs.query: TypeError: Cannot read property 'id' of undefined

i think i can contribute to this


Unable to get an account by provider

User model should have a method to retrieve a user account by provider.
Something like this $user->getAccountByProvider("facebook"), this is useful.

Done. Sorry for such a delay with the answer.
Release 0.9.10 with this enhancement will be ready in a few days.


Create a keybinding example

Should be tackled together with #4.

Now that #4 is done, this should be pretty straightforward to tackle. A code-only example could be done now.
A pretty, "traditional" keybinding menu is probably blocked on #21 and #24.

Done using bevy_egui!


Laravel 11 support

Hey mate. Would it be possible to bump the dependencies to support Laravel 11?

Scratch that. Was about to make a PR and realised I was looking at require-dev instead of require. All good!


Update weblinkchecker.py

Can you change this, because now can not adding blank lines? Now adding only one lines, no like before three.

Hi @zoranzoki21, first of all thank you for your pull request, but You should be aware, that only way to introduce change in official pywikibot repository is by official Wikimedia Fundation's Phabricator and Gerrit isntance.
See develop pywikibot section on https://www.mediawiki.org/wiki/Manual:Pywikibot

Okey

So if You will introduce this change through Gerrit, could You, please, close this pull request?

Yes. Thank you!
2017-06-05 10:18 GMT+02:00 Tomasz Magulski<EMAIL_ADDRESS>
So if You will introduce this change through Gerrit, could You, please,
close this pull request?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wikimedia/pywikibot-core/pull/17#issuecomment-306130661,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Abnx522OBmVMHRxWlN21IiRrYqAOxn23ks5sA7nngaJpZM4NvAJ-
.


Query binding giving incorrect results

Shouldn't these two statements be equal?
This one returns 5 records which is accurate
$buildings = DB::select('select * from buildings where square_footage < 10000');
This one returns an empty array
$buildings = DB::select('select * from buildings where square_footage < ?', [10000]);
I am rather new so forgive my ignorance in posting this. Something just seems broken.

Well DB::select accepts an array with column names to return: https://github.com/laravel/framework/blob/5.1/src/Illuminate/Database/Query/Builder.php#L225
So in your case you need to use the selectRaw function in which you can specify the parameters: https://github.com/laravel/framework/blob/5.1/src/Illuminate/Database/Query/Builder.php#L239

@bobbybouwmann That's a different select(). DB::select refers to a function in Connection

Ok well let me come at it from this angle I have a query scope setup like this
public function scopeBySize($query) {
     return $query->where('sqaure_footage', '<', 10000);
}

The above gives me an empty array... yet if I query the database I have 5 records that are < 10000
I can concede to being ignorant when it comes to laravel but please help me understand how this is supposed to give me a different result from a straight SQL query.
Thank you both for your responses earlier this did open my eyes to some things I was unaware of.
`JB

I would say post your question on a forum (Laracasts), so we can keep this issue closed. Also if you post on a forum, explain what you have tried and what is not working and your code of course ;)


Bug: Key Combination Breaks Text Editor Within List Items

Brief Description of the Problem:
A fatal error has been identified in the Lexical text editor that occurs when manipulating multi-level lists. The error is specifically triggered by a sequence of actions that includes merging external text into the list, creating new list items, and then attempting to delete them. This behavior results in a critical malfunction of the editor, rendering it completely unusable. The only way to restore the editor's functionality is to completely reload the page. This error represents a serious issue as it prevents the continuous use of the editor and jeopardizes the integrity of the user's data.
Lexical version: Latest
Steps to reproduce the error and the relevant code snippet are detailed in the next section. The problem can be reproduced in the official Lexical playground
Steps To Reproduce

Create a multi-level list in the Lexical text editor, like this:

      1. Tier 1
         A. Tier 2
            a. Tier 3
               I. Tier 4

Add text outside the list and place the cursor (Represented by |) in this position:

      1. Tier 1
         A. Tier 2
            a. Tier 3
               I. Tier 4

      *|*ExternalText

Move the cursor to the start of the external text and use the Backspace key to merge this text with the last item of the list:

1. Tier 1
   A. Tier 2
      a. Tier 3
         I. Tier 4*|*ExternalText

Press Enter to create a new item in the list:

1. Tier 1
   A. Tier 2
      a. Tier 3
         I. Tier 4
         II. *|*ExternalText

Press Enter again to add another item:

1. Tier 1
   A. Tier 2
      a. Tier 3
         I. Tier 4
         II.
         III. *|*ExternalText

6.Use the Up Arrow key to move the cursor up in the list:
1. Tier 1
   A. Tier 2
      a. Tier 3
         I. Tier 4
         II. *|*
         III. ExternalText

Press Enter, which causes a misalignment in the list structure:

1. Tier 1
   A. Tier 2
      a. Tier 3
         I. Tier 4
   b. *|*
      I. ExternalText

8.Press the Delete key, which results in the failure of the editor.
The problem can be reproduced in the official Lexical playground
Link to code example:
https://playground.lexical.dev/
The current behavior
The editor fails and crashes when deleting a list item under specific circumstances. This issue appears to be related to a problem in the code where anchorKey and focusKey do not find an existing key in the pendingNodeMap. The relevant code snippet is:
if ($isRangeSelection(pendingSelection)) {
  const pendingNodeMap = pendingEditorState._nodeMap;
  const anchorKey = pendingSelection.anchor.key;
  const focusKey = pendingSelection.focus.key;
  if (pendingNodeMap.get(anchorKey) === void 0 || pendingNodeMap.get(focusKey) === void 0) {
    throw Error(`updateEditor: selection has been lost because the previously selected nodes have been removed and selection wasn't moved to another node. Ensure selection changes after removing/replacing a selected node.`);
  }
}

The expected behavior
The editor should handle the deletion of list items without crashing, maintaining the integrity of the list structure and the editor's state.
Exported DOM from Lexical Playground:
LexicalError.json
Tree view:
 root
  └ (50) list
    ├ (49) listitem
    | └ (48) text  "Tier1"
    └ (52) listitem
      └ (53) list
        ├ (51) listitem
        | └ (54) text  "Tier2"
        └ (56) listitem
          └ (57) list
            ├ (55) listitem
            | └ (58) text  "Tier3"
            ├ (61) listitem
            | └ (62) list
            |   └ (59) listitem
            |     └ (63) text  "Tier4"
            ├ (77) listitem
            └ (79) listitem
              └ (78) list
                └ (76) listitem
                  └ (74) text  "ExternalText"

 selection: range
  ├ anchor { key: 77, offset: 0, type: element }
  └ focus { key: 77, offset: 0, type: element }

 commands:
  └ { type: UNKNOWN, payload: true }
  └ { type: UNKNOWN, payload: undefined }
  └ { type: UNKNOWN, payload: KeyboardEvent }
  └ { type: UNKNOWN, payload: undefined }
  └ { type: UNKNOWN, payload: KeyboardEvent }
  └ { type: UNKNOWN, payload: undefined }
  └ { type: UNKNOWN, payload: true }
  └ { type: UNKNOWN, payload: undefined }
  └ { type: UNKNOWN, payload: KeyboardEvent }
  └ { type: UNKNOWN, payload: false }

 editor:
  └ namespace Playground
  └ editable true

Dom view:

    Tier1

        Tier2

            Tier3

                Tier4

                ExternalText

Hi @FacuToso , we are not able to replicate the issue, could you help upload a video recording for the issue ? Thanks!


can you provide the source code

Thanks for your comments, I have already made the code open source.


Question or Feature Request: Configuration in PyCharm

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
ATM, VS Code allows any type of configuration tweaks. PyCharm doesn't (or I could not find it).
For instance, how to add a custom ollama model? How to specify the embeddings? In VS Code I have this:
{
  "models": [
    {
      "title": "Qwen2.5-Coder 1.5B",
      "provider": "ollama",
      "model": "qwen2.5-coder:1.5b",
      "apiKey": ""
    },
    {
      "title": "Qwen 2.5 Coder 7b",
      "model": "qwen2.5-coder:7b-instruct",
      "provider": "ollama"
    },
    {
      "title": "Qwen 2.5 Coder 32b",
      "model": "qwen2.5-coder:32b-instruct-q2_K",
      "provider": "ollama"
    }
  ],
  "tabAutocompleteModel": {
    "title": "Qwen2.5-Coder 1.5B",
    "provider": "ollama",
    "model": "qwen2.5-coder:1.5b",
    "apiKey": ""
  },
  "customCommands": [
    {
      "name": "test",
      "prompt": "{{{ input }}}\n\nWrite a comprehensive set of unit tests for the selected code. It should setup, run tests that check for correctness including important edge cases, and teardown. Ensure that the tests are complete and sophisticated. Give the tests just as chat output, don't edit any file.",
      "description": "Write unit tests for highlighted code"
    }
  ],
  "embeddingsProvider": {
    "provider": "ollama",
    "model": "nomic-embed-text"
  },

Where and how do I configure that in PyCharm?
Solution
If no such option exist, perhaps consider allowing customisation.

You can use "Continue" in PyCharm?

You can use "Continue" in PyCharm?

Yes, but it not everything as well as in VS Code yet due to configuration limitations described above.

Hi @barseghyanartur , both JetBrains and VS Code extensions can be configured/customized through the same file: ~/.continue/config.json
In any of the JetBrains IDEs, e.g. PyCharm, you can get to this file by clicking the gear icon in the top of the Chat panel

@Patrick-Erichsen
Amazing! Thank you!


Handle unicode correctly in writing overpass query to disk

in osm_xml.py query_txt.write(query) - this can fail on certain unicode characters

This is quite nasty to implement in Python 2; it should be handled easily by #325 since strings are natively Unicode.

Resolved by #325


How do I return a 404? or any other status message

The same way you would in a 'regular' ASP.Net project. That should still 'just work'. Can you provide some examples of what you tried and what you would have expected to happen?

Doesn't the return type of the controller action have to match the resource's underlying type?
How would I return a response that wasn't that type?

If you return HttpResponseMessage, you can do:
return Request.CreateResponse(HttpStatusCode.NotFound, new CustomException("My message"));

Which will do the right thing.
See the ASP.Net documentation.

I don't think this works, cause it appears Saule is intercepting the response handling. :-
this is what I tired:
       [HttpPost]
        [ReturnsResource(typeof (EngineResource))]
        [Route("engines/")]
        public object Create([FromBody] JObject engineData)
        {
            try
            {
                var engine = CurrentUser.Engines.Create(engineData);
                return engine;
            }
            catch (System.Data.Entity.Validation.DbEntityValidationException ex)
            {
                var errorHandler = new DbEntityValidationExceptionHandler(ex);
                var obj = errorHandler.AsErrors();
                // TODO: no 422?
                var response = Request.CreateResponse(HttpStatusCode.BadRequest, obj);
                return response;
            }
        }

Saule only supports serializing exceptions as errors; if your errorHandler.AsErrors call returns something else, it will not work. Maybe you can wrap it in custom exception?

I'm closing this. Feel free to re-open if the issue is not clear or resolved.


ability to specify placement group

+1

+1

Is this request going to be evaluated and merged anytime soon? It's been outstanding a long time. :disappointed:

+1

+1

+1