Error 200 - No modification done

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
without done nothing, and using this inventree from some month without any problem, today when i try to modify from web (not from admin view where works) any information i have this error:
Unhandled Error Code
Error code: 200
URL: /api/order/po/263/
Steps to Reproduce

Go to for example to Purchase order
Open an order
Click on Modify Order
Change some information
Press "Save"

Expected behaviour
Saving new information modified
Deployment Method

[ ] Docker
[X] Package
[ ] Bare metal
[ ] Other - added info in Steps to Reproduce

Version Information
Latest
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
No response

Hi there @AntoWkill ! The provided information is not sufficient to debug this issue. Please paste the version information from the version information dialogue in the web interface or inventree run cli. Latest is not reproducible without knowing OS and installer details.
On a general note, not really specific to this case due to above-stated reason:

Most update issues are solve by following the steps in the FAQ.
The InvenTree package installation updates - as is usual for Debian-family distros - when apt update && apt upgrade is run (if not configured otherwise on an OS/apt level)

Hi there @AntoWkill ! The provided information is not sufficient to debug this issue. Please paste the version information from the version information dialogue in the web interface or inventree run cli. Latest is not reproducible without knowing OS and installer details.
On a general note, not really specific to this case due to above-stated reason:

Most update issues are solve by following the steps in the FAQ.
The InvenTree package installation updates - as is usual for Debian-family distros - when apt update && apt upgrade is run (if not configured otherwise on an OS/apt level)

InvenTree - inventree.org
The Open-Source Inventory Management System
Installation paths:
Base        /opt/inventree
Config      /etc/inventree/config.yaml
Media       /opt/inventree/data/media
Static      /opt/inventree/data/static
Versions:
Python      3.10.12
Django      4.2.14
InvenTree   0.15.6
API         196
Node        N/A
Yarn        N/A
Commit hash:71db557
Commit date:2023-02-26

@AntoWkill Is it working now? What was the solution?

@AntoWkill Is it working now? What was the solution?
As happen without doing nothing, disappear without doing nothing. I think some cron run and solve the error 200.


speedup for doublemod and mulmod

In lieu of Nim not currently having a fast BigInt std library I've been looking at your various repos.  I'm translating my Miller-Rabin primality test from Ruby to Nim and want it to be deterministic over (for now) at least 64-bits. Your doublemod can be made a one-liner (and you can eliminate it),  by inserting the one-lner directly into mulmod. This significantly sped up my code. I believe doublemod is a private to this file, so removing it shouldn't affect anything else. My benchmarks establish a significant speedup inserting the one-liner directly into mulmod.  I thought it should be inlined, as if you were writing it like that, but there is a definite speed penalty on my system (using Nim 0.19.4, gcc 4.9.2, Linux 64-bit laptop, I7 cpu). I don't know if I need to compile with certain directives on. I'm just doing: nim c filename.nim or  nim c -d:release filename.nim.  Get the same file size and performance.

Note: I have a miller rabin implemented here: https://github.com/mratsim/nim-project-euler/blob/09c6bf59d974b530e0a4a2b64aa1ce263214d0f9/src/lib/primes.nim#L32-L105.
For bigInt I've implemented an efficient one for fixed power of two like uint256 here: https://github.com/status-im/nim-stint

Hey Mamy,
Thanks for getting back to me.
I've only been learning|using Nim seriously since about 2017.
I've been studying your codes repos (arraymancer, euler project, number
theory, etc) to learn how to code modular numerical functions in Nim, in
lieu of it (currently) not having a standard library implementation of
BigInts these numerical functions. Your code has been a great teacher for
me!
My immediate need was for coding a 64-bit deterministic M-R primality test
function. I've attached the file mrtest.nim of my implementation  using
some of your functions (with some modifications). Just compile and run the
file to see its test result and mini benchmark.
I've also attached a pdf of an email I just sent yesterday (Wed Feb 27,
2019) to the maintainer of the site -- https://miller-rabin.appspot.com/ --
about valid M-R implementation using those optimum witness bases.  I think
you implementation needs amending to use them like my original code did.
I'm also attaching files of 2 Nim parallel MR implementations I just
finished yesterday too. They work great, if run separately, but run inside
a long loop it hangs, and I haven't debugged that yet. Any ideas would be
helpful.
Finally, I've attached a recent paper I've released on generating Twin
Primes, with some other code, for your interest.
Take Care
Jabari Zakiya
On Thu, Feb 28, 2019 at 11:10 AM Mamy Ratsimbazafy<EMAIL_ADDRESS>wrote:

Note: I have a miller rabin implemented here:
https://github.com/mratsim/nim-project-euler/blob/09c6bf59d974b530e0a4a2b64aa1ce263214d0f9/src/lib/primes.nim#L32-L105
.
For bigInt I've implemented an efficient one for fixed power of two like
uint256 here: https://github.com/status-im/nim-stint
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mratsim/number-theory/pull/1#issuecomment-468332043,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEQ4F1WDY-xxB3AH48m-Y--lhhAlUnyks5vR_9ygaJpZM4bRAjH
.

oops seems like this completely escaped me


Implement one shot SHA and reduce allocs

This PR contains two performance improvements which landed into dev.boringbranch during this development cycle, both committed in https://go-review.googlesource.com/c/go/+/395876/11.

a99ec67c99c0e0d2bc69e495dbb0794969ec3314: Implement one shot SHA calls. This is helpful for callers which don't need to keep the hash object returned by NewSha1 and friends, as these functions return a pointer which normally escapes in the caller side.
9fafecd26c83a05018bd5a64541c488c2cb247db and 31857f33f60aef96df3b40722c163ebdf13b6fc5: Makes SHA calls allocation-free by hiding openssl pointers from the escape analysis.

The benchmarks demonstrates that the new code does not allocate:
name          old time/op    new time/op    delta
Hash8Bytes-4    4.57µs ± 4%    4.46µs ± 4%    -2.55%  (p=0.009 n=10+10)

name          old speed      new speed      delta
Hash8Bytes-4   224MB/s ± 4%   230MB/s ± 4%    +2.62%  (p=0.009 n=10+10)

name          old bytes/op   new bytes/op   delta
Hash8Bytes-4     24.0B ± 0%      0.0B       -100.00%  (p=0.000 n=10+10)

name          old allocs/op  new allocs/op  delta
Hash8Bytes-4      1.00 ± 0%      0.00       -100.00%  (p=0.000 n=10+10)

Removing the SHA allocations is important for when openssl is integrated into go1.19, else we would have to disable a new test that checks boring SHAs don't allocate.

I've found something good: noescapeCtx is not necessary because we are passing OpenSSL C pointers as uintptr instead of unsafe.Pointer, and the former are already safely hidden from the escape analysis.


python312Packages.etcd3: fix build

add env PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION to make protobuf compatible with old versions
See https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages marked as broken and skipped:
python311Packages.flask-appbuilder python311Packages.flask-appbuilder.dist
8 packages failed to build:
python311Packages.flask-limiter python311Packages.flask-limiter.dist python311Packages.libretranslate python311Packages.libretranslate.dist python311Packages.limits python311Packages.limits.dist python311Packages.slowapi python311Packages.slowapi.dist
4 packages built:
python311Packages.etcd3 python311Packages.etcd3.dist python312Packages.etcd3 python312Packages.etcd3.dist

Most importantly, python3.11-aetcd-1.0.0a4.drv
Checking runtime dependencies for aetcd-1.0.0a4-py3-none-any.whl
  - protobuf<5,>4 not satisfied by version 5.28.2

I have tried directly replacing protobuf>4,<5 in aetcd then the build succeeded, but I am not sure if this will affect the function

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:fast_forward: 4 packages marked as broken and skipped:

python311Packages.flask-appbuilder
python311Packages.flask-appbuilder.dist
python312Packages.flask-appbuilder
python312Packages.flask-appbuilder.dist

:white_check_mark: 35 packages built:

calibre-web
calibre-web.dist
fittrackee
fittrackee.dist
frigate
irrd
irrd.dist
libretranslate (python312Packages.libretranslate)
libretranslate.dist (python312Packages.libretranslate.dist)
octoprint
octoprint.dist
python311Packages.aetcd
python311Packages.aetcd.dist
python311Packages.etcd3
python311Packages.etcd3.dist
python311Packages.flask-limiter
python311Packages.flask-limiter.dist
python311Packages.libretranslate
python311Packages.libretranslate.dist
python311Packages.limits
python311Packages.limits.dist
python311Packages.slowapi
python311Packages.slowapi.dist
python312Packages.aetcd
python312Packages.aetcd.dist
python312Packages.etcd3
python312Packages.etcd3.dist
python312Packages.flask-limiter
python312Packages.flask-limiter.dist
python312Packages.limits
python312Packages.limits.dist
python312Packages.slowapi
python312Packages.slowapi.dist
theharvester
theharvester.dist

I have tried directly replacing protobuf>4,<5 in aetcd then the build succeeded, but I am not sure if this will affect the function

All packages that depend on aetcd successfully built and passed the test, I think there should be no problem


PI: Winlogs - Do not reset the check/uncheck state of checkbox, if the user has changed it.

Summary of the pull request
Bug: If the user checked/unchecked one of the logs checkbox in Winlogs page -> Then go to a different page-> go back to winlogs page, the checkboxes are reset and we do not maintain the state.
Fix:

Store the checkbox status in the viewmodel to make sure the same logs are enabled/disabled when the user goes back to Winlogs page.
Also, enable NavigationCacheMode on all main pages to avoid creating new instances of Page everytime a user navigates in and out of it.

Validation steps performed

Attach PI to an app
Go to winlogs page
Unselect EventViewer and select debugoutput
go to any other page
Go back to winlogs page and made sure EventViewer is not selected and DebugOutput was selected.

PR checklist

[ ] Closes #3049

                    RunAsAdminVisibility = Visibility.Visible;

should we set the enablement of the etw logs toggle to false here?

Refers to: tools/PI/DevHome.PI/ViewModels/WinLogsPageViewModel.cs:104 in 75c4f67. [](commit_id = 75c4f678b61a457c9e917be119d3321405c5070f, deletion_comment = False)

                    RunAsAdminVisibility = Visibility.Visible;

We stop Winlogs before we reach here and also collapse the GridVisibility, so etw logs toggle value will not matter

In reply to:<PHONE_NUMBER>

Refers to: tools/PI/DevHome.PI/ViewModels/WinLogsPageViewModel.cs:104 in 75c4f67. [](commit_id = 75c4f678b61a457c9e917be119d3321405c5070f, deletion_comment = False)


Address view summary at top

Transactions:
Total Received:
Current Balance:
Unconfirmed:

dup https://github.com/dcrdata/dcrdata/issues/121


Fix merging with empty values

Adds isEmpty option. Then the following code
const newObject = {
  foo: {
    one: null,
    two: "New text",
    three: ""
  }
}

const defaultObject = {
  foo: {
    one: "Some default text",
    two: "Some other default text",
    three: "Even more default text"
  }
}

const mergedObject = deepmerge(defaultObject, newObject, {
  isEmpty: a => a === null || a === '',
});
console.log(mergedObject);

gives you
{
  foo: {
    one: 'Some default text',
     two: 'New text',
     three: 'Even more default text'
  }
}

This is way too specialized/specific.  It's fine as a fork to use in a specific project, but isn't a good feature for a broadly-used utility.
I am open to changing the customMerge function signature to make it easy to solve for your use case without having to modify the code, though.

@TehShrike I need this too! So I basically want to merge only when the new object has a value (avoiding nulls overriding). Something like here:
https://www.rubydoc.info/gems/deep_merge/1.2.1
:merge_nil_values       DEFAULT: false
  Set to true to merge nil hash values, overwriting a possibly non-nil value

This does not seem to be possible to do using customMerge function? However, it seems to be quite common case, so I would really welcome a new merge option
mergeNilValues DEFAUL: true
This would not break existing behavior but would allow to turn it off.

I have the same use case to ignore empty strings when merging. I think it would make sense to extend the signature of customMerge to provide values, so this and similar use cases could easily be solved. This would make this package a lot more flexible, in my opinion.

I would love to have this as well.

+1  To the modification to customMerge signature. Allows users to implement multiple edge cases.

+1 here need some way of handling this

My previous comment stands – I'm open to PRs to make the necessary changes to customMerge to enable this.


Add lock to collection insert / delete / update methods

These methods are not currently thread-safe e.g. because two documents that are inserted simultaneously may both satisfy the uniqueness criteria in collection.find() before actually getting inserted into the mongodb collection.

Maybe a lock is not the best solution as this would require a server / client in addition to the existing mongodb. There maybe alternatives, e.g. using mongodb unique indices:
https://docs.mongodb.com/manual/core/index-unique/#std-label-index-type-unique


pdf.contentStream(), pdf.textStream() and pdf.propertyStream() have no output

#Hi!
I'm trying this lib and although I can get this to work:
pdf
  .pdfStream()
  .pipe(fs.createWriteStream('out.pdf'))
  .on('finish', function() {
    console.log("We're done!")
  })
  .on('error', function(err) {
    throw err
  })

The following shows nothing (same goes for contentStream and propertyStream):
pdf
  .textStream()
  .on('data', function(data) {
    console.log(data)
  })

The following works, although it exits with an error "Error: write EPIPE"
pdf.pngStream(300).pipe(fs.createWriteStream('out-page1.png'));

I'm not to familiar with streams so I'm wondering if I'm doing something wrong?

I have the same issue

Sorry all for not responding. We're unable to actively work on the project at the moment (and to work on the bugs you report), but happy to merge PRs if you find a solution.


⚠ DDoS-Relay US-W-sfo3-03 has degraded performance

In ec83b23, DDoS-Relay US-W-sfo3-03 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 141 ms

Resolved: DDoS-Relay US-W-sfo3-03 performance has improved in a5a3d8e after 17 minutes.


Allow access every time I connect the usb

Problem
Is there any way to allow access  to a specific USB just once?I have to allow access to usb every time I connect this USB(btw Im using always the same USB)
Expected behavior
Actual behavior
Stacktrace of Excpetion
(if exists)

Code where problem occurs

Does this help https://stackoverflow.com/questions/40182096/usb-permissions-without-prompt ?


[client] pillage history not shown after pillage

not shown after pillage and also need to refresh page to see it appear in entity details

This was fixed by events not being synced correctly


Achieve IE9 compatibility

Catalog is also useful for enterprisey projects that still need to support older browsers. We should try to support IE9 at least. It currently complains about missing requestAnimationFrame support.

We're only using https://github.com/chrisdickinson/raf/ which should fall back to setTimeout. Are you sure that it's an issue of Catalog?

IE 11 should work now, I don't think we can support IE9/10 because we use of Flexbox. I'm gonna close this. If you feel like giving this a shot anyway, please open a PR 😁


Bug regarding "back to article"-link

/test/cases/basic.html
Link of footnote 1:  basic.html#fnref-%1
Link of footnote 2:  basic.html#fnref-%1
should be:
Link of footnote 1:  basic.html#fnref-1
Link of footnote 2:  basic.html#fnref-2

Added PullRequest to fix bug.
https://github.com/tivie/showdown-footnotes/pull/3


⚠ Bofrost.lu has degraded performance

In 1a85626, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1242 ms

Resolved: Bofrost.lu performance has improved in f57f195 after 19 minutes.


Datetimepicker background does not cover whole control

I'm submitting a...

[x] Bug report
[ ] Feature request
[ ] Documentation issue or request

Current behavior

Expected behavior
The datetimepicker's background color should cover the whole control. Also the picker should have a max width and not stretch indefinetly.

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

@danielleroux @nuke-ellington This issue should be closed, background issue is already fixed and the max-width issue, i asked Silvio and he said it should not have max-width and the width should adapt to the content needs.

@goncalosard 👍


question about receive and close

Hi
I attached the python server and Js client code.
I have added my questions as inline comments in the code but here they are

ws.receive() seems to be blocking until it receives message from websocket client from the JS code
when I close the socket from JavaScript client, I don't think the connection is getting closed. I might be doing something dumb but cannot figure it out yet.

Tried this from chrome (Version 49.0.2623.112 m) and  IE (version: 11.0.9600.18163).
On IE, after I tried closing the socket from JS Client, I got an error:
SCRIPT12002: WebSocket Error: Network Error 12002, The operation timed out
Thanks a lot for your time.
example.zip

It isn't clear from your server.py code how you are actually running the flask server. Are you using the non-blocking gevent-based WSGIServer? I just copied the code from the main flask_sockets example and got non-blocking sockets.
The ws.receive() line isn't going to block the rest of your app, but it will prevent further execution in the start_timing function, as the code executes serially. You need to have a different function that handles the sending of new messages to the client.

I'm butting up against this too although it seems like a gevent-websocket issue. The problem I have is, if Idon't ever call receive in the while True loop (i.e. only call send), client disconnects won't notify the socket and the loop will run indefinitely. Seems like receive is needed with a timeout just t0 check the client is still connected or not.
FYI: That's running from gunicorn with flask_sockets.worker

FYI digging into this more, I found fervent Timeouts, which might be the answer
http://www.gevent.org/gevent.html#timeouts


Viewbinding

Removed Kotlin Android Extensions, Added ViewBidning. Updated gradle plugin version to 4.0.0 to include buildFeatures, removed deprecated android.enableUnitTestBinaryResources from gradle-wrapper.properties

thank you for the Pull Request,  like to consolidated this one to. https://github.com/android/camera-samples/pull/379.
The variable name is good(or we could drop the "binding" in it). closing this one and please feel free to update in https://github.com/android/camera-samples/pull/379.


python38Packages.bayespy: 0.5.21 -> 0.5.22

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.bayespy is: "Variational Bayesian inference tools for Python"
meta.homepage for python38Packages.bayespy is: "http://www.bayespy.org"
meta.changelog for python38Packages.bayespy is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.5.22 with grep in /nix/store/hb8ly9k0x7pk0mrabsaf4519fmd65spn-python3.8-bayespy-0.5.22

directory tree listing: https://gist.github.com/9def2ab660d391996f0d7074c866fa06

du listing: https://gist.github.com/ced78180e748887527fe906dbda24381

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

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.bayespy
python39Packages.bayespy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hb8ly9k0x7pk0mrabsaf4519fmd65spn-python3.8-bayespy-0.5.22 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hb8ly9k0x7pk0mrabsaf4519fmd65spn-python3.8-bayespy-0.5.22
ls -la /nix/store/hb8ly9k0x7pk0mrabsaf4519fmd65spn-python3.8-bayespy-0.5.22/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python38Packages.bayespy
python39Packages.bayespy

Maintainer pings
cc @jluttine for testing.

Result of nixpkgs-review pr 117091 at c440fdda run on aarch64-linux 1

2 packages built successfully:

python38Packages.bayespy
python39Packages.bayespy


GetScaleRecommendation query issue

Hi @cgillum
I am currently using the mssql scaler with KEDA and my durable function application by following the process described here:
https://keda.sh/docs/2.0/migration/
https://microsoft.github.io/durabletask-mssql/#/
Everything works as expected except the dt.GetScaleRecommendation scaler function. When there are no events/requests it returns a value of 0. This causes the number of pods in the cluster to scale down to 0 which in turn makes the cluster unavailable when hitting the external IP as there are no instances of the durable function running to process the request.
To remedy this I added a line before the return statement in the script:
IF (@recommendedWorkersForOrchestrations + @recommendedWorkersForActivities) < 1 SET @recommendedWorkersForOrchestrations = 1
Which simply ensures that a minimum of 1 is returned by the scaler function. I'm not sure if this is best practice  but it immediately fixed the issue and the cluster is scaling up and down (to a minimum of 1 pod)  perfectly.
I've opened this ticket in response to your comment in:
https://github.com/kedacore/keda-external-scaler-azure-durable-functions/issues/18
Please let me know if I should provide any more information and thanks for the great scaler!
Cheers,
Sam

Thanks @samfisher07 for the feedback! Scaling to zero is by-design since theoretically you shouldn't need a pod unless there is work to do. However, I recognize that if you have triggers other than durable triggers running in the same container that this can be problematic. However, when I tested this using the Functions Core tools, it would actually create two function apps: one for HTTP triggers with a minimum count of 1 and one for everything else, with a minimum count of 0, so I didn't encounter this type of issue.
Can you tell me more about the composition of your app and the availability issues you ran into? For example, do you have both HTTP triggers and Durable triggers running in the same app? Also, is configuring the KEDA scaler with a minimum of 1 replica (instead of 0) an alternative workaround that would work for you?
One option I could consider for dt.GetScaleRecommendation is to add another parameter for a minimum instance count, but I want to get a bit more information to understand how useful this would be (or not).

Sure @cgillum .
My durable function application using a HTTP trigger via the 'Httpstart' function (in similar fashion to the durable function tutorials) that initiates a durable function orchestration. I'm actually unfamiliar with a durable trigger.
The option of adding a minimum replicas parameter is an excellent idea since it resolves this 0 instances issue but also allows anyone to configure the minimum availability of their cluster. For instance I would actually prefer to have minimum replicas be 3 as availability is of more importance to me than minimum cost. I'm sure adding this would be useful to many.

@samfisher07 are you able to work around this by configuring KEDA with a minimum replica count? For example, if you're using the Azure Functions Core Tools, you can use the ---min-replicas command-line parameter to specify a non-zero minimum. Something like this:
func kubernetes deploy (...args...) --min-replicas 3 --max-replicas 10

@cgillum I'm a bit of a novice with the Kubernetes deployments but if I'm not mistaken these min-replicas and max-replicas values are set in the horizontal pod autoscaler?
This is set on my side to a non-zero minimum which is actually how I discovered the problem. I got a warning from the horizontal pod scaler description (kubectl describe hpa) that said the scaled object requested a replica count below the minimum set. Thereafter, the scaler set the instance count to 0 and didn't scale up again.
I do still think that adding a minimum replicas parameter to the scaler function would be best solution, although a manual edit of the scaler function worked perfectly well for me.

The minimum count is actually supposed to be configured via the minReplicaCount property of the ScaledObject spec. According to the KEDA docs, if minReplicaCount is not configured, the default value is 0. I haven't tried to see what happens when the ScaledObject and the HPA disagree on minimum instance counts, but based on your experience it sounds like the ScaledObject configuration wins.
My main hesitation to making a change to GetScaleRecommendation is that it's likely going to require a breaking change. I also hesitate because this is just a "recommendation" and I feel it's the responsibility of the scaling infrastructure to decide whether or how to apply the recommendation.


Add CodeQL Analysis

:tada: This PR is included in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Restoring menu highlights

SUMMARY

Following up on https://github.com/apache/incubator-superset/pull/11915
This PR restores the little animated underlines, and the nav link hover backgrounds, which got lost somewhere along the line in a refactor.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

After:

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12024 (7082b22) into master (90ed326) will decrease coverage by 2.92%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12024      +/-   ##
==========================================
- Coverage   66.58%   63.65%   -2.93%
==========================================
  Files         951      471     -480
  Lines       46663    29013   -17650
  Branches     4571        0    -4571
==========================================
- Hits        31070    18468   -12602
+ Misses      15456    10545    -4911
+ Partials      137        0     -137

Flag
Coverage Δ

cypress
?

javascript
?

python
63.65% <ø> (-0.95%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.62%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
69.95% <0.00%> (-12.45%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

... and 500 more

Continue to review full report at Codecov.

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


echo v4

can support echo v4 ?

Supported in #6

Using go get github.com/sevenNt/echo-pprof downloads the old 0.1.0 version.
Pulling from master worked for me: go get github.com/sevenNt/echo-pprof@master


Error: SQLSTATE=22504 SQLCODE=-191 Mixed data or UTF-8 data not properly formed.

Still getting this issue  with idb-connector 1.2.16 and NodeJS 16.

Duplicate to #147


Remove warning from README

Removes warning about a limited ReplaySubject now that #3 has been fixed.

Great catch, thanks!


[accordion item] new story for outlined

StoryBook:
https://tmp.dev.vivid.vonage.com/1183/index.html


Evaluation Context / Bookmarks / Oct. 2018 Release

I have noticed that I have lost the ability to hold an evaluation context when using bookmarks to toggle between visualizations on the same page. For example if I was to select a bar on a bar chart and switch between visuals, the selected context goes back to the original state. “Data” is unchecked under data properties and so I am not sure why I’m seeing this response. I used this technique very frequently prior to the oct release. Your assistance is very appreciated

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

ID: 6e579557-27e4-8a24-69b7-55189b0f4966
Version Independent ID: 0c97fa47-9695-aaaf-efcd-63b35362310e
Content: Using bookmarks in Power BI - Power BI
Content Source: powerbi-docs/desktop-bookmarks.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi @joerobert06 -- I see this is a duplicate of your other comment, right? You've done the right thing by posting in the community. I'm going to close this. Thanks and I hope they can figure out what's not working!


Python version confusion

Hi all,
You have used python version ["3.8", "3.9", "3.10"] in your pylint.yml file. It showed error while running the git hub action. Any reason why you used all the versions?

I think one version is enough? Or may be this is for the reason to make sure if one version is not available the other will work?. I would also like to hear the answer. thank you,


Added note for Service Loader.

I'm sorry for this small PR but I wanted to document the result of our discussion (also for myself) and I cant push directly to master : )

@sruehl thanks for the hint, good catch.
I have problems mit mvn site:run and filed jira PLC4X-52.

Sorry for that and thanks for the catch. I will ask chris on the list to look into the maven problems (if he doesnt already) to be able to run "tests"

@sruehl I noticed that I wasnt able to reference the PlcDriverManager as it is placed in core. So I suggest to extract an interface PlcDriverManager and to rename the implementation to DefaultPlcDriverManager.
The Interface is especially necessary for us to implement our proxy, as the PlcDriverManager can be used as an equivalent of the javax.sql Datasource.

@sruehl I noticed that I wasnt able to reference the PlcDriverManager as it is placed in core. So I suggest to extract an interface PlcDriverManager and to rename the implementation to DefaultPlcDriverManager.
The Interface is especially necessary for us to implement our proxy, as the PlcDriverManager can be used as an equivalent of the javax.sql Datasource.

fair enough im just thinking if there would be a better way to retrieve the PlcDriverManager as to use the DefaultPlcDriverManager. Might confuse a Api User but im not sure what would be best though...

I agree, and I also did not like that.
I shortly thought about addind a static factory method to the interface but this is something that I dont like that much.
I'm not sure what the best way is as it feels right to have the interface in the api module but I agree that it is confusing for users.

What we could do I add a Interface on top of the existing driver manager like IPlcDriverManager or something and write in the jdoc that it needed for proxies. This way the usage stays the same and the purpose is clear. But dunno :/

I'm also unsure about that.
We can also leave it as is but perhaps we should move the PlcDriverManager to the api package?
Another approach would be to use a PlcDriverManagerFactory witha  static create method (kind of like the entity manager).

I would have left the DriverManager (But I do like the name PlcDriverManager more) in the API module, but the problem was that at that time I couldn't write any tests for that as that would have required a cyclic dependency in the maven project graph. I definitely agree that it would be cooler to just depend on the API module. Even if the DriverManager isn't really an API class ... but I guess users would probably forgive us that.

the factory PlcDriverManagerFactory approach might be the best at the end... but keep in mind that this factory can't the reference the default one directly as we would get an circular dependency

Yes... this is kind of tricky.
A kingdom for some injection frameworks.. : )
I suggest to revert my changes and remove the link in the javadoc to make this merge-able.
Then we discuss the topic seperate, what do you think?

good, idea. Maybe you could squash/soft-reset your changes into one commit.

@chrisdutz So you suggest to move the PlcDriverManager to the api module to resolve the problem?

@chrisdutz So you suggest to move the PlcDriverManager to the api module to resolve the problem?

as far as I Understand we wouldn't have the cyclic dependency problem then and you could reference the default one from a factory and still use the interface for your proxy...

I think things changed with the latest refactoring of the API as we no longer have real classes, just interfaces ... I guess we could try moving that class there and get rid of the "core" module. Would definitely be worth it. I just knew that I wanted it in the API but I had to introduce the core in order to resolve the cyclic dependencies (At that time ... might be obsolete now)


[SPARK-13231] Make count failed values a user facing API.

I have left-out the renaming for now, not sure about it.

Test build #50920 has started for PR 11115 at commit f84fc4d.

Test build #50920 has finished for PR 11115 at commit f84fc4d.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50920/
Test FAILed.

Test build #50923 has started for PR 11115 at commit 5e5c58f.

Test build #50923 has finished for PR 11115 at commit 5e5c58f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50923/
Test PASSed.

/cc @andrewor14, we should look at this PR since I know that we discussed making the metric-style accumulators a public API while working on that patch.
@ScrapCodes, if we do move forward with this I think we'll want to update the programming guide in order to help users discover this feature.

@JoshRosen Thanks for taking a look. I will update it, did you also consider renaming - or may be just updating the doc suffices ?

@andrewor14 Can you take a look ?

I'm hesitant about this because we would need to maintain backward compatibility if we make it fully public. All the exposed constructors will have to be there forever. If we do decide to do this we need to make sure we like the name because that can't change.

I just talked with @rxin and we decided that this flag isn't quite ready to be exposed yet. The problem is the default value (false) has weird semantics on stage retries; if you're counting the number of rows in your data and your stage was run twice, then you might get 2X the actual number of rows. This is what #11105 is looking at, and we might proceed from there.
Either way I think we should try to fix the default behavior before exposing a flag that we won't be able to change in the future. Let's keep the issue open but close this PR for now.

@andrewor14 As far as I knew, it worked fine. There is a check in Task.collectAccumulatorUpdates, you mean that check is not sufficient ? Ahh looking at the jira, I understood there are a lot more ways than just task failure that a stage is recomputed e.g. cache eviction. Correct me if I am wrong.
Alright closing this.


Pressed "v", but nothing appeared.

As indicated in the title, I initiated the window manager using startx and successfully entered it. However, when I pressed v, nothing happened.
Log in logs/matrix.log:
[2024-05-08 13:26:30.971] [wm] [debug] WindowManager()
[2024-05-08 13:26:31.014] [World] [debug] Hello World!
[2024-05-08 13:26:31.032] [wm] [info] enter createAndRegisterApps()
[2024-05-08 13:26:31.032] [wm] [info] exit createAndRegisterApps()
[2024-05-08 13:26:49.949] [wm] [info] DestroyNotify event
[2024-05-08 13:27:34.427] [wm] [debug] WindowManager()
[2024-05-08 13:27:34.471] [World] [debug] Hello World!
[2024-05-08 13:27:34.474] [wm] [info] enter createAndRegisterApps()
[2024-05-08 13:27:34.474] [wm] [info] exit createAndRegisterApps()
[2024-05-08 13:28:09.028] [wm] [info] DestroyNotify event
[2024-05-08 13:29:46.445] [wm] [debug] WindowManager()
[2024-05-08 13:29:46.482] [World] [debug] Hello World!
[2024-05-08 13:29:46.485] [wm] [info] enter createAndRegisterApps()
[2024-05-08 13:29:46.485] [wm] [info] exit createAndRegisterApps()
[2024-05-08 13:30:23.338] [wm] [info] DestroyNotify event

My bad, dmenu was not installed on my system.


Update Mat-table row without refreshing  page or component

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/fork/components-issue
Steps to reproduce:

when  timers value 00:00:00 want to change status from open to closed.  without refreshing page.

Expected Behavior
What behavior were you expecting to see?
Actual Behavior
What behavior did you actually see?
Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

Please fill out the template so that we can help you resolve the bug you are reporting


Create a test for the subtraction operation

Task
Create a test for the subtraction operation. this can be seen at tests/test_operations.py with the function test_subtraction.
Acceptance Criteria
the test should fail if subtraction is mathematically wrong
the test should succeed if subtraction is mathematically correct
Answer
Note: It is not expected that you know how to write tests, your goal is to understand what the below code does and perform the change on your branch that you have created and create a pull request
def test_subtraction():
 """
  GIVEN: Two numbers
  WHEN:  passed in to the function subtraction
  THEN:  the resulting is the subtraction of the two numbers
 """
    assert subtraction(7, 3) == 4
    assert subtraction(0, 0) == 0
    assert subtraction(2, 5) == -3

#25 awaiting issue to be merged first


Need to do cache

Is your feature request related to a problem? Please describe.
If the user refreshes the browser each time the posts are fetching each time. If we are caching this process to a certain time period user could render our elements easily.
Describe the solution you'd like
Cache the process for a certain time and add a parameter called  cache_seconds. the user could change the caching time manually.
Describe alternatives you've considered
FYI, you could see this project and know how they achieve the caching

@Aynn Is there any way to reduce the time of loading?


Export TableData to CSV quotes numbers and not strings

See below example of exported data to CSV with quotes enabled:
2016-01-12|40|1370|||||"41860"|"43-6014.00"|CA|SAPX||SAN MATEO|USA|ST|"0"|

DBeaver quotes values in CSV only if:

Value contains tab, line feed character or cell delimiter
Column has string type but value starts from a digit (quote says that this values is a string and avoids reformatting in Excel/Calc)

Is there a way how to avoid quotes for number in string column? Or maybe add such feature? I ahve tried some settings in Export wizard but without success.

DBeaver quotes values in CSV only if:
Value contains tab, line feed character or cell delimiter
Column has string type but value starts from a digit (quote says that this values is a string and avoids reformatting in Excel/Calc)

hmm.. Is there any way to export string column as quoted and numeric column as not quoted?
Some columns of my database are changed value quote dynamically, I have to check almost of all large data has over 80 columns. (Diff after remove quote is not good in my case.)
Any Idea?


Always null out the connectPromise once the connect is complete

Motivation:
We did miss to null out the connectPromise in some cases which could
lead to incorrectly handling of POLLOUT.
Modifications:
Refactor code in connectComplete(...) to always null out the
connectPromise and cancel the timeout when needed.
Result:
Correctly handle POLLOUT in all cases

This did cause an test error from time to time here.

let me just merge this so I can make progress... its a tiny fix anyway


python312Packages.example-robot-data: 4.1.0 -> 4.2.0

https://github.com/Gepetto/example-robot-data/releases/tag/v4.2.0
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 363802

x86_64-linux

:white_check_mark: 45 packages built:

aligator
aligator.doc
crocoddyl
crocoddyl.doc
example-robot-data
example-robot-data.doc
mim-solvers
ndcurves
ndcurves.doc
pinocchio
pinocchio.doc
proxsuite-nlp
proxsuite-nlp.doc
python311Packages.aligator
python311Packages.aligator.doc
python311Packages.crocoddyl
python311Packages.crocoddyl.doc
python311Packages.example-robot-data
python311Packages.example-robot-data.doc
python311Packages.mim-solvers
python311Packages.ndcurves
python311Packages.ndcurves.doc
python311Packages.pinocchio
python311Packages.pinocchio.doc
python311Packages.proxsuite-nlp
python311Packages.proxsuite-nlp.doc
python311Packages.tsid
python311Packages.tsid.doc
python312Packages.aligator
python312Packages.aligator.doc
python312Packages.crocoddyl
python312Packages.crocoddyl.doc
python312Packages.example-robot-data
python312Packages.example-robot-data.doc
python312Packages.mim-solvers
python312Packages.ndcurves
python312Packages.ndcurves.doc
python312Packages.pinocchio
python312Packages.pinocchio.doc
python312Packages.proxsuite-nlp
python312Packages.proxsuite-nlp.doc
python312Packages.tsid
python312Packages.tsid.doc
tsid
tsid.doc


Confusion in TiDB Cluster System Report

Feature Request
Is your feature request related to a problem? Please describe:

The Diagnosis Result part in the TiDB Cluster System Report contains the diagnosis information both from the selected time period and the current system, which will easily cause confusion of understanding and comprehension.
Describe the feature you'd like:

It would be better that the Diagnosis Result could be devided into at least two part:

The Diagnosis Result for the selected time period, ordered by log timestamp, if applicatable.
The Diagnosis Result for the current system

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

Thanks for the report! Since we are now deprecating this feature, which will be substituted by the clinic service, I'm going to close this issue. If you have questions, feel free to leave comments.


Keyboard shortcuts

We should provide default shortcuts + documentation on configuring shortcuts for Run Deephaven File and Run Selection commands.

I think we should allow it to be set but not ship a default. I actually kinda assumed it would take the shortcut of that button group if one existed, and deephaven was the last command run though.


Module integration

integrating modules according to user story

User story:

User add melodic with megnic to white board, and robot plat that. (lilypond)
voice command to robot, commands include [read, start, stop, repeat, concatenate, slower, faster, louder, softer]
Test to speech, for hurman robot interaction, not for debug.
Excution of the command

We should add a ros package called marimbabot_behavior for the state machine / behavior tree / ... (what ever we want to use)

After the user story, I think we have some new component of the system(the following topic name just as example):

Speech recognition
Recognize the speech signal from user and publish it to a ros topic  /user_speech .
Speech synthesis.
Listening the message in the ros topic /speech_synthesis, and speak it out.
High level control. (marimbabot_behavior).
Listening the message in the ros topic /user_speech and response for that.

see https://github.com/UHHRobotics22-23/marimbabot/issues/53 for topic (& communication) description

I assume it just means "shifting" the notes by #,b or full notes. https://en.wikipedia.org/wiki/Transposition_(music)

Transposition is one example, but not the only one meant here. We require that you can adapt different parameters that show that the system has a model of music theory (you get a lot of that for free with lilypond). Otherwise why would you have bothered with musical notation in the first place? Here are a number of suggestions, but there are lots more and please come up with whatever interests you:

With Transposition you can create melodies that use the marimbaphone bars that you cannot detect in vision right now (I believe the whole upper octave)
Mode Changes are different from transposition because the distance between individual notes in the scale changes (the position of the half-tone steps in the diatonic scale) and not all notes together. The classic difference is major to minor scale ("happy" to "sad" mood), but everything else is just as easy when you are there already.
Time signature changes. This is very open-ended and there are any number of ways of creating melodies in one time signature that were originally in a different one, e.g., moving between 4/4,  3/4, 2/4, 6/8, (5/4???)
Style-changes: Many musical styles are based around specific rhythms, e.g., there is the classic tango rhythm, so any melody you convert to such a rhythm  (yes, there are many ways to do that) will sound closer to Tango. I wouldn't be surprised if there are multiple projects around that can move melodies/pieces between styles.
Swing is another obvious idea here. <biased opinion>Everything sounds better when you swing it</biased_opinion> :-)
With the two-mallet setup you could decide the notes to play with the second mallet based on harmonization, e.g., improvise a second voice
change loudness in different ways.

overall loudness of the piece
Syncopation (yes, "stressing" might imply more than loudness changes)
add crescendi/decrecendi to a phrase.

In the end these commands can also help you to bring everything together in the integration even when not everything is supported throughout the pipeline (e.g., vision does not support addressing the upper octave or users don't have to specify 2-note-chords in the input to get both mallets to play something interesting).
@iibrahimli @berkgungor @cvhex for reference.

I assume it just means "shifting" the notes by #,b or full notes. https://en.wikipedia.org/wiki/Transposition_(music) @yunlong-wang-cn

Based on my understanding, the possible text-form speech examples for "shifting" could be: "C four transpose" or "C sharp", is that right? @Juphex

@yunlong-wang-cn  sure, that would be possible

For the record here:
Last day,  we try to input the command from speech recognition, and then we have the following findings:

The noise before the speech could influence command recognition.

Therefore, consider to:

#225 Use keyword spotting to activate the whisper, only the keyword is detected, then the whisper with recognize the speech.
#223  Consider to using some denoising technic(optional, depend on testing result.)

The current status is that everything is integrated well.

Maybe let's close this ticket?


two topics in one conf

Hi I am using this output plugin to send different logs to different pub topics. following are my td-agent.conf
<match tag1.xxx>
  @type copy
  <store>
    .....
  </store>
  <store>
    type gcloud_pubsub
    project my.project
    topic my.topic1
    key my.key
    flush_interval 1
  </store>
</match>

<match tag2.xxx>
  @type copy
  <store>
    .....
  </store>
  <store>
    type gcloud_pubsub
    project my.project
    topic my.topic2
    key my.key
    flush_interval 1
  </store>
</match>

Then fluentd will keep restarting, looks like the configuration is conflict. Any ideas?

Hi @randyh0329 we had the similar problem.
What we have found that problem is in gcloud gem (used by the plugin).
What version of gcloud do you use?
For instance the latest version of gcloud 0.9.0  depends on 'google-protobuf' which was not the case in v0.4.1. So the conflict happens when you use fluentd via td-agent. Because in the startup script td-agent assigns LD_PRELOAD to  jemalloc replacing the default malloc.
So if you use just fluentd you should not have any problems with your configuration.
If use fluentd from td-agent then you could have this problem
You could try to comment out the line in you init.d td-agent config and restart it and test it, this solved our problem.


Doesn't want to open java code

OS - windows 7 home premium
Brackets version - <IP_ADDRESS>
When I right click a .jar file and then select "Open with brackets" it gives the error message "Brackets currently only supports UTF-8 encoded text files.
I know that brackets is just glorified notepad, but it would be awesome if I could open and edit java code inside brackets (since I mainly use it as my "IDE for everything")

Hi @AlexStefanov2,
the error message isn't due to Java-files exactly, but with the character encoding of the files. The files are most likely encoded in UTF-8 but with BOM or as some other ASCII format, which Brackets didn't handle up until https://github.com/adobe/brackets/pull/13412 and https://github.com/adobe/brackets/pull/13477.
While those should land in 1.10, meanwhile you could open the files in another editor (for example Notepad++ on Windows), convert the files to plain UTF-8 and open them again in Brackets.

Closing as fixed, please do reopen if issue persists with Brackets 1.10-1.11


Error Message Text Not Displaying

-- Error message notification works, but when attempting to "show details" nothing happens and notification window closes.

Moved: https://github.com/BitShares/web_wallet/issues/554


Better Quote parsing ergonomics

Instead of parsing all the bytes, could we have some known-good lengths in a union, like in this example?

Closing in lieu of #1212


Share Linter and IntelliSense information across multiple Notebooks

Context & Problem
I work with datasets that are quite large, and generally take a long time to load into memory.
When I am conducting feature engineering with notebooks, I have to work across the spectrum of:

creating several transformations
testing them on different ML models
comparing variations of (1) and (2)
temporarily scraping work/results that aren't useful (maybe now, but could be useful in the future)

After a day or two of these, my notebooks become large, complex, and incoherent.
I find myself looking for something I did several rounds of analysis ago, and because I can't find it, I attempt at rewriting it, adding to even more code in my notebooks.
Possible Solution - but still not perfect
I split my work into several notebooks, and have all the notebooks connect to the same Jupyter kernel:

Imports Notebook
Load Data Notebook
Multiple Feature Notebooks
Multiple ML Notebooks

Now I can more easily track my work, but what makes this solution less than perfect is that none of the linting and intellisense features will work across notebooks - i.e. I will not have autocomplete work and I will get warnings that variables/modules have not been declared/imported.
Feature Request - a more perfect solution
I would like linting and intellisense to work across all notebook files contained in a folder.
For example, all notebooks will parse through the imports notebook and be able to understand that I have numpy imported as np and then autocomplete np... even though I did not do this import in a previous cell.
I imagine creating some hierarchy file in the folder that specifies which notebooks can access which other notebooks to understand declarations.
Reasons Not to Bother with Adding this Feature - and why they are invalid

Why not split each combination of analysis into separate notebooks, with all full imports, data loading, preprocessing, etc. Besides being repetitive in code, it is an inefficient use of memory/compute resources and my time. For each variation, I need to wait for my machine to process all necessary cumulative prior steps. This locks up resources.
Why not save data files at each step. This is a variation of the not so perfect possible solution. It is imperfect, because when dealing with large datasets you still need to load the data and also have some place to store the data. It also requires having a very good system of data versioning, and this is simply not practical for the exploratory stage.

Why is this Feature a good Foundation for Future Features
I can see this leading to the ability for analysts/scientists to more easily develop books with chapters - and the hierarchy file serving as a table of contents.

@Zenome84 great feature request and thank you for providing all the details and reasoning. It makes good sense to me. Long/large notebooks can quickly become hard to manage and splitting the work into multiple notebooks is a good idea.
We have similar ideas on how to improve this, i.e., support sharing kernel between notebooks or even implement scratchpad/repl for notebooks, thus users can use the scratchpad or secondary notebook for experimenting but use the main notebook to capture the validated ideas.
Regarding linting/intellisense, currently language extensions don't have this concept. They can handle single notebook (technically multiple cell text document in the same notebook), but it would be an architecture change to understand multiple notebook can share the same state. With that said, it's worth brainstorming and see how we can improve this.

@rebornix

We have similar ideas on how to improve this, i.e., support sharing kernel between notebooks or even implement scratchpad/repl for notebooks, thus users can use the scratchpad or secondary notebook for experimenting but use the main notebook to capture the validated ideas.

So sharing kernel between notebooks is already supported through vscode, a feature that I stumbled upon by mistake. I have found it indispensable.

Regarding linting/intellisense, currently language extensions don't have this concept. They can handle single notebook (technically multiple cell text document in the same notebook), but it would be an architecture change to understand multiple notebook can share the same state. With that said, it's worth brainstorming and see how we can improve this.

This is in fact the feature that would complement sharing a kernel between notebooks amazing. I am not versed in the architecture nor have I every built extensions, but as you say, linting/intellisense does work across cells in the same notebook (albeit sometimes buggy) and could be some place to work from
Also, when importing from other py files, linting/intellisense also works in the standard way expected in python. This could be another direction perhaps, like importing the namespace of another notebook. However, I have never heard of such a concept, yet.

Thank you for filing this issue, given the fact that we haven't had sufficient upvotes on this issue I'm closing this as not planned.


feat: Pydantic model for the serialization schema

This PR ports the pydantic definition from guppy (including https://github.com/CQCL/guppylang/pull/172), and updates it to sync up with the current schema.
Changes to the schema:

Renamed the InputExtensions class to ExtensionSet
Added a named definition for Const
Dropped DummyOp. It was referenced in a couple field types, but the model was never defined.

The main difference with the model in guppy is the use of RootModel-based classes instead of variables assigned to a TypeAliasType for defining aliases on type unions.
They both work the same, but mypy doesn't like using variables in types.
Additionally, I updated some READMEs and added a CI check to ensure that the json in specification/schema/ is always up to date.

Sorry for the noisy metadata changes, the CI still needed some fixes to work correctly...

We could publish a 0.0.0.a1 alpha version and use it from guppy.
I would wait for the builder API before releasing 0.1.0.


Adding the rest of the basic moderation commands

Pull-request

[x] I have read the code of conduct.
[x] I have created a relating issue.

Changes

[x] Existing code
[x] New feature

Closes : #8
Description
I have added all the commands list in issue #8, this also means that there will be new additions to the database

Codecov Report

Merging #31 (29bc9ae) into main (6418409) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##            main     #31    +/-   ##
======================================
  Coverage   0.00%   0.00%
======================================
  Files         46      52     +6
  Lines       1130    1258   +128
  Branches      67      71     +4
======================================
- Misses      1130    1258   +128

Impacted Files
Coverage Δ

...ing/backend/builder/slash/SlashCommandBuilder.java
0.00% <0.00%> (ø)

...hub/org/programming/bot/RegisterSlashCommands.java
0.00% <0.00%> (ø)

...gramming/bot/commands/moderation/AuditCommand.java
0.00% <0.00%> (ø)

...rogramming/bot/commands/moderation/BanCommand.java
0.00% <0.00%> (ø)

...bot/commands/moderation/DeleteMessagesCommand.java
0.00% <0.00%> (ø)

...amming/bot/commands/moderation/TempBanCommand.java
0.00% <0.00%> (ø)

...g/programming/bot/commands/moderation/TimeOut.java
0.00% <0.00%> (ø)

...ogramming/bot/commands/moderation/WarnCommand.java
0.00% <0.00%> (ø)

...g/bot/commands/moderation/util/ModerationUtil.java
0.00% <0.00%> (ø)

...a/io/github/org/programming/database/Database.java
0.00% <0.00%> (ø)

... and 4 more

@all-contributors please add @RealYusufIsmail for infrastructure, tests and code

@all-contributors please add @Haleloch for code and review

@all-contributors please add @RealYusufIsmail for infrastructure, tests, review and code

https://github.com/all-contributors please add @Haleloch for code and review

@all-contributors please add @Haleloch for code and review

@all-contributors please add @Haleloch for code and review


Track moves in CollectionChangeset

It's very cumbersome given a CollectionChangeset to arrive at the original from and to indices of a moveItemAtIndex operation.
Is there any good reason there isn't a moves property in CollectionChangeset?

This feature is now implemented and moved to Bond framework that's now built on top of ReactiveKit :)


Fix switched descriptions

The text explaining the functionality for copySourcePRReviewers appears to be describing what should be controlled by copySourcePRLabels (at least that's my assumption based on the field names), and vice-versa.

duplicate of https://github.com/sorenlouv/backport/pull/492

Thanks for the contribution @resnickdeepfield !


missing saxon-dom dependency

I am getting the following exception
com.jamesmurty.utils.XMLBuilderRuntimeException: net.sf.saxon.trans.DynamicError: DOMSource cannot be processed: check that saxon8-dom.jar is on the classpath
at com.jamesmurty.utils.XMLBuilder2.wrapExceptionAsRuntimeException(XMLBuilder2.java:107)
at com.jamesmurty.utils.XMLBuilder2.toWriter(XMLBuilder2.java:523)
at com.jamesmurty.utils.BaseXMLBuilder.toWriter(BaseXMLBuilder.java:1251)
at com.jamesmurty.utils.XMLBuilder2.toWriter(XMLBuilder2.java:536)
at com.jamesmurty.utils.BaseXMLBuilder.asString(BaseXMLBuilder.java:1272)
at com.jamesmurty.utils.XMLBuilder2.asString(XMLBuilder2.java:478)
at com.jamesmurty.utils.BaseXMLBuilder.asString(BaseXMLBuilder.java:1310)
at com.jamesmurty.utils.XMLBuilder2.asString(XMLBuilder2.java:464)
Adding the saxon-dom dependency to my pom file fixed the problem.

Hi @jaceko your environment must be configured to use the Saxon XML library, this isn't normally the case